Super slow speed on large data set of strings for UPDATE operation

Problem
Perfoming the following operations:
Given a dataset of 1000 elements, update matching string with a new string
Takes a ridiculously huge amount time
And the time complexity grows dramatically when increasing the dataset to 10 000 elements
Code snippet
    override suspend fun update(param: String, objectToInsert: NewsReportDto) {
        realm.query<NewsReportDto>("title == $0", param)
            .find()
            .forEach { oldNewsReportDto ->
                realm.write {
                    findLatest(oldNewsReportDto)?.apply {
                        title = objectToInsert.title
                        description = objectToInsert.description
                    }
                }
            }
    }

Link to this update method source code
Link to the data generator
Performance result

Let me know if i'm doing something wrong, the corresponding update query for Room
        "UPDATE newsReport " +
                "SET title = :newTitle," +
                "description = :newDescription " +
                "WHERE title = :title"

takes very little time for example, 20 miliseconds for 1k elements, and 20 miliseconds for 10k elements

Hi @konnovdev. Thanks for the report. I am currently investigating a memory leak in the JVM/Native transition. I will see if I can replicate your observations and investigate if it is related.

One thing to note is that you seem to be doing 10k writes rather than 1 write updating 10k elements. This is significantly different from the Room query.

But the tests for doubles uses the same approach, so seems like the issue is in fact related specifically to strings.

@nirinchev oh, is there a way to make 10k updates in one operation?

@rorbech with the test on real numbers - actually it's 99.9% unique numbers, so in that case there will be just a few rows updated, whereas in the test of strings it's 0.01% unique strings

@konnovdev Ahh, ok, then I guess the amount of transactions is probably the cause. You can just do the query inside the write instead. Then you are also doing the query on the latest version of the realm and you can skip the findLatest.
realm.write {
    query<NewsReportDto>("title == $0", param)
       .find()
      .forEach { oldNewsReportDto ->
         oldNewsReportDto.apply {
             ....
         }
       }
    }

@rorbech it says I cannot skip the findlatest step
    java.lang.IllegalStateException: Cannot set `NewsReportDto.$title` to `not strange`: changing Realm data can only be done on a live object from inside a write transaction. Frozen objects can be turned into live using the 'MutableRealm.findLatest(obj)' API.

But moving realm.write like this does the trick indeed! Now update for 1k items takes 19 milliseconds, and update for 10k items 121 milliseconds.
Here is the code snippet:
        realm.write {
            realm.query<NewsReportDto>("title == $0", param)
                .find()
                .forEach { oldNewsReportDto ->
                    findLatest(oldNewsReportDto)?.apply {
                        this.apply {
                            title = objectToInsert.title
                            description = objectToInsert.description
                        }
                    }
                }
        }

Btw I think bulk update could be a good addition to the documentation in Readme.md ))

@konnovdev Great.
Regarding findLatest: You have to do the query against the mutable realm (the receiver of the write) which is referencing the latest version of the realm and not against realm. So
realm.query<NewsReportDto>("title == $0", param)

should just be
query<NewsReportDto>("title == $0", param)

instead and then you should be able to skip the findLatest.

Strange, when I don't use findLatest, it's 10 times slower.
So doing this:
        realm.write {
            query<NewsReportDto>("title == $0", param)
                .find()
                .forEach { oldNewsReportDto ->
                    oldNewsReportDto.apply {
                        title = objectToInsert.title
                        description = objectToInsert.description
                    }
                }
        }

Takes 10 times more time than this
        realm.write {
            realm.query<NewsReportDto>("title == $0", param)
                .find()
                .forEach { oldNewsReportDto ->
                    findLatest(oldNewsReportDto)?.apply {
                        title = objectToInsert.title
                        description = objectToInsert.description
                    }
                }
        }

@konnovdev I have been digging a bit and what happens is that the underlying query is most likely re-run when you update the  objects, so for now the best thing is probably to iterate the stable result from the frozen realm.query. I will have to dig a bit further to understand the details and, if the hypothesis is right, the options of providing a stable results for these use cases.

Closing this issue as the various root causes of the lower speed has been explained, but created #914 to enhance the API to provide a stable result for fast and predictable iteration.


Suport for WiFI

Support for WiFi in the arduino code.

DRAFT CUZ IDK HOW TO TURN IT INTO A DRAFT (caps intended)

Closing this for now, but feel free to reopen if/when you're still working on wifi :)

Ok

#72


Check if revision exists before downloading

The sync command is fairly slow even if the cache contains every revision being
synced. This is because a download is attempted even if it is unneeded.
This change changes the flow of context.Sync to attempt to RevisionSync with
whatever is in the cache, if that fails then it calls Download followed by
another RevisionSync.
I am making this pull request because at my work we would like to use govendor for our vendoring system, but for doing quick dev builds in docker containers, govendor sync is very slow, even with the cache. We currently use glide which sync's in sub 1 second with a cache, while govendor takes much longer.
Here are before and after time stats for govendor sync in github.com/kardianos/govendor with an empty vendor directory with the cache up to date.

I love your tool, please let me know if this pull needs any changes. Please help us switch from glide.

I am not sure this was my problem, but it looks very promising.


Problems building the solution

How to exactly build the solution using Visual Studio? I get a lot of errors about missing header files etc.

@gpeipman, it looks like your change to the AdditionalIncludeDirectories in #3 would fix the problem with missing header files.
Did this resolve your issue?

Hi,
I found one issue and I am not able to understand what’s going on. When I try to add RTSP source to graphedt.exe then it immediately crashes.
When I try to create instance of it through Activator then it also crashes with stack overflow exception.
Br,
Gunnar
From: fyehoo<EMAIL_ADDRESS>Sent: neljapäev, 10. august 2017 19:58
To: fyeh/RTSPSource<EMAIL_ADDRESS>Cc: Gunnar Peipman<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [fyeh/RTSPSource] Problems building the solution (#2)
@gpeipmanhttps://github.com/gpeipman, it looks like your change to the AdditionalIncludeDirectories in #3https://github.com/fyeh/RTSPSource/pull/3 would fix the problem with missing header files.
Did this resolve your issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/fyeh/RTSPSource/issues/2#issuecomment-321611666, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAJBAlmG7x5NZBWF-a3jKoqWtK-1Sg3Nks5sWzaugaJpZM4OzHFR.

Yes, issue is solved and on Windows 7 and Visual Studio 2012 it builds okay now.


Is version 3.0.30512.0 still supported?

The AJAX Control Toolkit 3.0.30512.0
Message from the NuGet community:  Is version 3.0.30512.0 still supported?

@ChrisSfanos I recommend all users of past ASP.NET AJAX Control Toolkit upgrade to the latest version because many security issues and bugs have been resolved since the early versions:
ASP.NET AJAX Control Toolkit - Release Blog posts


Metrics as a list not working

Hello, Thanks for this awesome package, it's saved me a ton of time. One issue I have ran into is when I attempt to pass metrics as a list, I get that the metrics are not found in the report suite.
report = suite.report
.element('mobiledevicetype',disable_validation=True)
.metric(['pageviews', 'visitors'])
.filter('s300001088_595cfe5403d383079e8701d8')
.breakdown('evar30')
.range('2018-05-01', '2018-05-01').run()
**

KeyError: "Cannot find ['pageviews', 'visitors'] among the available metrics"

**
It works fine if I daisy-chain them like .metric('pageviews').metric('visitors), just not as a list as described in your documentation.

You can pass a list of metrics if you wish to process the report using multiple metrics.
report = suite.report.element('evar1').metric(['visits','pageviews','instances'])

Same issue here!
I built a function and I'm passing the metrics' list as an argument but it won't work.
`def reporting(suite, metrics, start_date, end_date, granularity, segments):
report = suite.report\
        .metric(metrics)\
        .range(start_date, end_date)\
        .granularity(granularity)\`

I'm giving it the list as you would expect:
metrics = ['metric1', 'metric2']
I upgraded the package and everything.
Hope you can help us!

Can you provided the full error message? It's working fine on my end

This was many months ago. It’s been working fine for a while now.
Thanks,
Will Austin

On Sep 24, 2018, at 5:51 PM, Adib Behjat<EMAIL_ADDRESS>wrote:
This message is eligible for Automatic Cleanup<EMAIL_ADDRESS>Add cleanup rule | More info
Can you provided the full error message? It's working fine on my end
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Fix ValidationError import in forms.py

Closing this PR as there is already a PR with the fix in flight https://github.com/derek-schaefer/django-json-field/pull/41.


Right sidebar renders poorly with wide plugins

I'm not sure if this is the theme's fault or the calendar plugin I'm using, but it would be nice if the entire spacing on the right would be utilized.

It looks to be from the css variable --sidebar-size-right: 270px. Is there an easy way to configure this within the UI rather than forking and writing my own theme?

This is indeed part of the design, but you can adjust or disable that through the Style Settings plugin!

Style Settings > Layout Section > Sidebar Section > Fixed Width (Right) or Width size if fixed (Right)

Oooo very nice! TIL Style Settings plugin is a thing.
Thank you! I love the theme.


Valid meshes do not require len(points) == len(obj:vt)

Closes https://github.com/nschloe/meshio/issues/1261
This PR removes a check for point data consistency, since that restriction is not actually fulfilled by many valid mesh files.
https://github.com/nschloe/meshio/blob/0138cc8692b806b44b32d344f7961e8370121ff7/src/meshio/_mesh.py#L165-L169
This PR also adds another test for this case.

Also in that PR https://github.com/nschloe/meshio/pull/1067/commits/8e87d45bf7796695d5b5141659ace6bbbe7bbf1d, a pytest skip line was added to an existing test of the elephav.obj mesh.
@pytest.mark.skip("Fails point data consistency check.")

There's no description in the text of the PR, so I'm a little unclear about what problems this check for point data consistency was intended to prevent, or why the test was skipped.
So, while the test suite is passing for me, it may also be possible that I'll need to add another test (to check we really are preventing whatever bad thing we need to avoid). Please let me know if that's the case.

Hello, just posted this comment on issue #1487 which is related to this fix
How we could validate a different way
The problem with this assertion for .obj loading is that the vertex texture attribute "obj:vt" and other keys in valid .obj files commonly are not the same count as the number of vertices.
I have a suggestion how this could be changed to validate the "face" data which appears as
f 1/1 2/2 3/3 4/4
f 5/1 8/2 7/3 6/4
f 1/1 5/2 6/3 2/4
in an obj file.
 With corresponding vertex textures index 1 2 3 4 defined in the file as vt 0.000000 0.000000 etc.
Reading this documentation about the obj format https://learnwebgl.brown37.net/, what we should validate is that the indices in each face data doesn't exceed the length of the available attributes.
So if there are 4 "obj:vt" we should read each face, so
f 1/1 2/2 3/3 4/4 would pass the validation since for each vertex_index/texture_index, texture_index <= len(point_data["obj:vt"])
f 1/1 2/2 3/3 4/5 would fail since 4/5 has a texture_index = 5 > len(point_data["obj:vt"])
...still just a thought, however I would be happy to open a PR to try this approach


ipaexfont: use mkDerivation

Description of changes
https://github.com/NixOS/nixpkgs/issues/103997#issuecomment-1258186428
Changed $out/share/fonts/opentype to $out/share/fonts/truetype as the font is in .ttf format.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

We don't need or want to layer fonts into two fixed output derivations.


user-agent

Please add the ability to set a custom user-agent.
According  to the docs, streamable wants a user-agent for bots.

Usage of the API requires a Streamable account and valid email. All requests must be authenticated with Basic Auth. If you're a bot, please provide a descriptive user agent as well.

https://streamable.com/documentation
If I read it correctly this lib has no way to specify a custom user-agent, and also doesn't set any user agent at all.

this looks easy: http://docs.python-requests.org/en/master/user/quickstart/#custom-headers
I might even do this myself (... if I have time)

Hi, thanks for pointing this out, currently there is no option to specify user agent, and the user agent is not set. I'll add the user-agent, but no ability to override or disable it for now.

I've just uploaded version 1.0.1 to pypi, so you can upgrade using pip. I'll close this issue, please reopen or create a new one if something's not right :)


maybe improve grammar: t.get(find.text('hi')).tap()/.should(...) -> find.text('hi').tap()/.should(...)

by implement CommonFinders, then it even work with normal extension methods

t\.get\(find\.(.+?)\)\)
t.$1)

wait, this is not possible...the types are wrong

Method B: can we implement find.text('hi').should(...)? In other words, remove "t.get"


ValueError: No method was found starting with 'addObject' and with keywords set()

Operating System: macOS 12.3 Intel chip

Description
As per https://github.com/samschott/maestral/issues/516#issuecomment-1026782249
and request to log https://github.com/samschott/maestral/issues/516#issuecomment-1027245681
When starting the sync via terminal, following error is generated;
Notification failed
Traceback (most recent call last):
  File "Contents/Resources/app_packages/rubicon/objc/api.py", line 206, in __call__
KeyError: frozenset({''})

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "Contents/Resources/app_packages/desktop_notifier/base.py", line 204, in send
  File "Contents/Resources/app_packages/desktop_notifier/macos.py", line 252, in _send
  File "Contents/Resources/app_packages/desktop_notifier/macos.py", line 366, in _create_category_for_notification
  File "Contents/Resources/app_packages/rubicon/objc/api.py", line 236, in __call__
  File "Contents/Resources/app_packages/rubicon/objc/api.py", line 215, in __call__
  File "Contents/Resources/app_packages/rubicon/objc/api.py", line 153, in __call__
  File "Contents/Resources/app_packages/rubicon/objc/api.py", line 1519, in ns_from_py
  File "Contents/Resources/app_packages/rubicon/objc/api.py", line 236, in __call__
  File "Contents/Resources/app_packages/rubicon/objc/api.py", line 208, in __call__
ValueError: No method was found starting with 'addObject' and with keywords set()
Known keywords are:
frozenset({'toPropertyWithKey', ''})
frozenset({'', 'toBothSidesOfRelationshipWithKey'})

This should be fixed with recent releases of rubicon-objc which made significant improvements to loading of ObjC methods.


Multival

Code refactoring, transitioned the library to look more like the C connector, added functionalities.
Known bugs:

if there are more than 2 configurations, they are not parsed correctly
connection issues(perhaps resolved with the mqtt client reconnect mechanism, remains to be tested)

The simple example should send a random value


Fallback to Jedi if user opts not to proceed downloading analysis engine

@qubitron @brettcannon

Once the extension has been installed, we'll attempt to download .net core runtime (if not available)
The user is presented with the option of proceeding with this download & installation
If the user selects No do we:

Fall back to Jedi
Not have any intelliense

@DonJayamanne during the preview we would fall back to Jedi. After a things are going well with the stable release we are likely to remove Jedi, in which case the user would have no intellisense if we fail to download. At the stable release, I don't know if we need to ask before downloading, but we should display an error if downloading fails. It is one more step that could go wrong, and other VS code extensions automatically download their dependencies.

This is sad that jedi is going to be removed. Dave (the author of jedi) has been working really hard to make it better in recent months, jedi is literally getting better and bugs are fixed every week. Though I certainly understand that it's a burden for the vscode-python team to support two IntelliSense backends...

Dave's hard work on Jedi is definitely appreciated by us and we did not take this decision lightly, but since our larger team already has to support the IntelliSense engine in the Python workload for VS we just couldn't keep justifying the duplication of work when our own teammates have an IntelliSense engine we weren't using.

Currently there is no plan to offer The user is presented with the option of proceeding with this download & installation. If engine is activated, download is automatic since activation is an explicit option in settings. Do we need another confirmation? As @qubitron said, if new engine is default then Jedi is gone.

I'm closing this issue as we are in agreement


text/scanner: Allow users of ScanRawStrings to get a RawString

Currently, even with ScanRawStrings enabled, all "raw" (delimited by backticks) strings will trigger String tokens rather than RawString tokens. From a text/scanner-only perspective, this could be fixed by changing L624 in scanner.go as outlined in this patch:
https://github.com/go-forks/go/commit/d869c9a9ed55a79cb626d223736ff1c40d172cce
Regrettably I don't have the time for the endless litany of steps outlined in https://golang.org/doc/contribute.html just to submit such a brazenly trivial PR, so whoever contributor wants to pick this up, please be my guest =)

We now accept GitHub pull requests.  Want to send in your patch?


is there a way to uninstall tiscamera?

Is there a way to uninstall tiscamera?
I tried rm -r tiscamera in the home dir. but it only removes the directory.

Yes. If you installed with
sudo make install
then you use
sudo make uninstall.
That removes the files from the /usr/* directories. After doing this, you can remove the tiscamera directory from your home directory.
Stefan

Yes. If you installed with sudo make install then you use sudo make uninstall.
That removes the files from the /usr/* directories. After doing this, you can remove the tiscamera directory from your home directory.
Stefan

Do we run the sudo make uninstall from the build directory? or the tiscamera directory?

Hello

Do we run the sudo make uninstall from the build directory? or the tiscamera directory?
From the build directory.

Stefan

Closed due to inactivity.


python312Packages.pyrisco: 0.6.4 -> 0.6.5

Diff: https://github.com/OnFreund/pyrisco/compare/refs/tags/v0.6.4...v0.6.5
Changelog: https://github.com/OnFreund/pyrisco/releases/tag/v0.6.5

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 349399

x86_64-linux

:white_check_mark: 5 packages built:

home-assistant-component-tests.risco
python311Packages.pyrisco
python311Packages.pyrisco.dist
python312Packages.pyrisco
python312Packages.pyrisco.dist


[XML] Update OnOff Cluster XML to match updated spec

The data types names has been updated in OnOff Cluster and must be updated in the XML as well.
Added:

OnOffControlBitmap
EffectIdentifierEnum
DelayedAllOffEffectVariantEnum
DyingLightEffectVariantEnum

Changed feature names:

Level Control for Lighting -> Lighting
Dead Front Behaviour -> DeadFrontBehaviour

Adjusted value names in DelayedAllOffEffectVariantEnum and DyingLightEffectVariantEnum:

FadeToOffIn_0p8Seconds -> DelayedOffFastFade
50PercentDimDownIn_0p8SecondsThenFadeToOffIn_12Seconds -> DelayedOffSlowFade
20PercentDimUpIn_0p5SecondsThenFadeToOffIn_1Second -> DyingLightFadeOff

Spec PR: https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/6511 (already merged)

I am not sure whether it's worth working on this until whatever changes are needed for https://github.com/CHIP-Specifications/connectedhomeip-spec/issues/6336 happen too....  Right now the XML in the SDK cannot in fact be aligned with the spec.


3 instances for load testing of the tier

using instance 3 for the load testing

Building AWS Modernization Workshop: Cloud Automation with Quali following AWS workshops


Create SilverStripe.gitignore

Adding SilverStripe default .gitignore
https://silverstripe.org/download
https://github.com/silverstripe

I'm going to close this out because the value of maintaining a template like this isn't clear to me.
@zanderwar if you feel like this should still be a template listed here, please read the section about specialized templates and open a new pull request that follows the new pattern.


Unable to update from chiadog v0.3.0

trying to update chiadog with
docker pull artjacobson/chiadog-docker:latest
I get a message saying:
Status: Image is up to date for artjacobson/chiadog-docker:latest
but when I run the chiadog container it's still on 0.3.0 (not v0.4.1)

The latest tag for the Docker image has been updated to v0.4.1.
Make sure you update config.yaml to include the daily_stats section, it will crash on start without it.

Working perfect now, Thanks!


mxisd: 1.2.0 -> 1.4.3

Motivation for this change
Things done

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

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

[x] 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 nix-review --run "nix-review wip"
[x] 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] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

cc @fpletz @hschaeidt :spider_web:
I suggest to backport to 19.03

@mguentner Sorry for the delay on this. Do you mind doing a git rebase? Please keep in mind that PermissionsStartOnly has been deprecated.

@aanderse No worries. Thanks for taking a look.
mxisd 1.4.x seems stable enough. I will test it very soon.
I will might simply push to that branch and we can omit the 1.3.1.

OK sounds good. I'll wait to hear back on a version update.

@aanderse Ready :)

@GrahamcOfBorg build mxisd
@GrahamcOfBorg test mxisd

So it appears someone has removed mxisd from nixos/tests/all-tests.nix :disappointed:
@mguentner bonus points if you push another commit on top of this to add mxisd (back?) to all-tests.nix

@aanderse Pushed

@mguentner thanks!
@GrahamcOfBorg test mxisd

@mguentner can you briefly explain why the code in preStart is no longer required? It looks like you're moving from a mutable config file to immutable so are there any backward compatibility/upgrade issues to consider?

@aanderse
At some point mxisd required the config to be named application.yaml. Pointing it to the /nix/store path produced by writeText did not work because of the filename like hash-something-application.yaml.
That's why I opted for simply copying the config to the work directory. I used chmod 444 to prevent mutation.
So it was done because of the configuration requirements of mxisd < 1.3, not because it mutates the config file during runtime.
I don't expect / didn't run into any upgrade issues because of that.
If an existing config has something in extraConfig however, this might need to be adjusted as config format changed slightly. It's no longer some yaml-inspired format but standard-yaml.
More on that: https://github.com/kamax-matrix/mxisd/wiki/Upgrade-Notes#v130

@mguentner you have been super helpful in my review of this change. Thank you!


Improvements to the docs.

• Add section explaining that devise_token_auth uses the devise verbiage by default, but if you wish to customize it, you can create an overrides file in config/locales/devise.en.yml.
• Add link in e-mail auth section pointing to e-mail template overrides.
• Add link pointing to default devise verbiage file.
• Add link to e-mail template overrides demoing custom e-mail subjects.

Thanks @aarongray

:+1:


how to get count of any particular attributes value from map.graphics array of object

there is an array of  object of feature like this. i am taking graphics array from map and i need UCRType count from that array of object. UCRType is an feature attribute so i need how many Burglary and arson attribute i have in feature.graphics array
[
   {
      infoTemplate:{

      },
      symbol:{

      },
      geometry:{

      },
      attributes:{
         DayName:"Friday",
         ReportedCrimeDate:"1/20/2012 9:30:00 AM",
         UCR:"Burglary - Residence"         UCRCLASS:"21B"         time:"09:30"
      }
   }
],
[
   {
      infoTemplate:{

      },
      symbol:{

      },
      geometry:{

      },
      attributes:{
         DayName:"Friday",
         ReportedCrimeDate:"1/20/2012 9:30:00 AM",
         UCR:"Arson"         UCRCLASS:"21B"         time:"09:30"
      }
   }
],
[
   {
      infoTemplate:{

      },
      symbol:{

      },
      geometry:{

      },
      attributes:{
         DayName:"Friday",
         ReportedCrimeDate:"1/20/2012 9:30:00 AM",
         UCRType:"Burglary - Residence"         UCRCLASS:"21B"         time:"09:30"
      }
   }
]

how to get count of UCRType
array=[buglary:2, arson:1, , , , ]

Have you tried something like
SELECT _->infoTemplate->attributes->UCRTypes, count(*) FROM tablename GROUP BY  _->infoTemplate->attributes->UCRTypes

My feature array of object is like this and i need UCR Count and Its Name
"graphics":[
{
"symbol":{   },
"infotemplate":{   },
"attributes":{
"UCR":"Burglary"
}
},
{
"symbol":{  },
"infotemplate":{    },
"attributes":{
"UCR":"Arson"
}
}
]

If you do a
alasql("select _->0->attributes->UCR as name, count(*) as total from ? GROUP BY _->0->attributes->UCR", [feature]);

You get
[
	{
		"name": "Burglary - Residence",
		"total": 2
	},
	{
		"name": "Arson",
		"total": 1
	}
]

Live example: http://jsfiddle.net/zLgmqotL/
Its not exactly what you asked for, but I am sure you can find your way from here.
Please ask questions like this on stack overflow next time.


Unable to delete relays from network menu

Describe the bug
Steps to reproduce
Setup: Already have relays added

Navigate to menu > settings > network
Tap the “-“ button next to a relay
Tap “ok” on the “are you sure” modal
Relay is not removed from the list

Expected behavior

The relay should be removed from the list

Environment

iOS version 1.0.4

https://github.com/PrimalHQ/primal-ios-app/assets/123519473/d74b322b-365f-4c81-8955-a5a190d65194

Fixed


GPUParticles2D incorrect particle position when locol_coords disabled

Godot version
4.0.alpha3
System information
Ubuntu, Nvidia
Issue description
https://user-images.githubusercontent.com/23386940/156674293-ad56f7dc-90f6-4290-8a57-c7ff3071ba71.mp4
Steps to reproduce

Create a blank project
Create Node2D scene
Add a GPUParticles2D and add a ParticlesMaterial, perhaps crank its scale_min up for better visibility
Disable local_coords
Move the GPUParticles2D node away from 0, 0 and observe erroneous offset appearing between the node position and the particles

Minimal reproduction project
No response

Duplicate of #56892


Roll Engine from 03d645e782db to 5b92a197f0ee (3 revisions)

https://github.com/flutter/engine/compare/03d645e782db...5b92a197f0ee
2021-06-04<EMAIL_ADDRESS>Remove "unnecessary" imports. (flutter/engine#26580)
2021-06-04<EMAIL_ADDRESS>fixes crash when sets mouseTrackingMode before view is loaded (flutter/engine#26576)
2021-06-04<EMAIL_ADDRESS>Roll Skia from 568ef84d3f8a to 2692b0dd119f (1 revision) (flutter/engine#26582)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Closing for long-queueing ci.yaml validation.

Commit queue failed; closing this roll.


no graceful shutdown of tgtd

When running tgtd -f, stopping the docker container causes a rude kill of the process instead of a graceful shutdown.
This is because tgtd does not listen to SIGTERM or SIGINT - so docker is sending a SIGKILL to the process.
See https://github.com/fujita/tgt/blob/master/scripts/tgtd.service for details

When will this be built and pushed to docker hub?
At the moment, the yaml does not work, for the command is not available in the official image.

True, there was some problem in docker hub, I've recreated repo


error: invalid use of incomplete type 'SSL {aka struct ssl_st}

I am trying to compile my application using openssl1.1.1c version.
When I am trying to reference S3 variable inside SSL structure it gives and error
main.cpp:297:12: error: invalid use of incomplete type 'SSL {aka struct ssl_st}'
In file included from /home/../include/openssl/crypto.h:25:0,
from /home/v../include/openssl/comp.h:16,
from /home/../include/openssl/ssl.h:17,
I check the difference between the previous openssl1.0.2r code with openssl1.1.1C I could see that the SSL structure which is actually a typedef to ssl_st structure.
In Openssl1.0.2r, the ssl_st structure and number of other structures were present in the ssl.h include file with the Openssl1.1.1c this structure is moved to a ssl_locl.h which is present is ssl directory. This looks to be the reason it is not finding the "ssl_st" structure
Could anyone please help to get a workaround on this problem

OpenSSL 1.1.x is not fully source compatible with OpenSSL 1.0.2. Lots of structures were made opaque which means you have to use accessor functions to obtain data rather than reaching into the structure directly. What were you trying to do that needed access to s3?

I am trying to use the flags variable of s3 structure
s3->flags |= SSL_OP_NO_RENEGOTIATION;

The equivalent is:
SSL_set_options(s, SSL_OP_NO_RENEGOTIATION

Will try it. Thanks a lot for quick response.

This question seems to have been answered. Closing.

what about ssl->version ?


All flavors must now belong to a named flavor dimension.

Error:All flavors must now belong to a named flavor dimension. Learn more at https://d.android.com/r/tools/flavorDimensions-missing-error-message.html

@teolemon , Is this issue has been solved now?

thanks @AbsolutelySaurabh for spotting this. @VaiTon can you confirm ? If so, we may close it.


Adding Options to Enable ECS Service Connect

Adding Options to Enable ECS Service Connect

Looks good to me, but as it's pretty complex, better to try it out first.

this will overlap with consul


Access Zoom or other meeting URLs

Can we add another option to Open Event Locations? Currently we only use Google/Apple maps to open event locations.
Lots of meetings are now hosted online, and I typically save Zoom links to event location (or at event description). It would be really nice if I can open these links directly from calendar events using Command + Return. Thanks.

See #90.


Server: Log level settings not taking effect

What version of Music Assistant has the issue?
Server 2.0.0b72 (Docker)
The problem
I'm having issues with my logs filling up with info-level logs from unavailable metadata etc. I've tried setting the log level of the metadata controller to be less verbose, first to warning and then to error (indeed, I think all my logs are currently set to error) but nothing has changed in the log output.
Additional question - I see there's a global setting for logs, but I don't see where to change that, is there somewhere I'm missing?
How to reproduce

Set log level for a core module to error
Check the logs and see that the logs are still outputting lower levels e.g. info

Relevant log output
2023-08-16 20:07:25.389 INFO (MainThread) [music_assistant.metadata] Unable to get musicbrainz ID for artist Anjunadeep
 - using lookup-album(s): Anjunadeep The Yearbook 2020/Anjunadeep The Yearbook 2020
 - using lookup-track(s): none

Additional information
No response
What version of Home Assistant Core are your running
2023.8.x
What type of installation are you running?
Home Assistant Container
On what type of hardware are you running?
Raspberry Pi

I assume you mean the metadata controller you have changed the logging for? Try the Musicbrainz provider instead. Better yet tag those tracks with the MB IDs. I see Anjunadeep is in the MB database. As for where the global setting is I’m not sure.

Try the Musicbrainz provider instead

Those are all already set to error unfortunately. It's not just the metadata, I'm getting info-level logs for everything it seems, despite the log settings all being error.

Better yet tag those tracks with the MB IDs.

That is the medium term goal but I have a lot of tracks that would need tagging - the joy of trying to reconcile a vast collection coalesced over many years from many sources 😅
Here's my settings.json, you can see all logs are set to ERROR:
{
  "server_id": "[redacted]",
  "providers": {
    "chromecast": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "player",
      "domain": "chromecast",
      "instance_id": "chromecast",
      "enabled": false,
      "name": "Chromecast",
      "last_error": null
    },
    "airplay": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "player",
      "domain": "airplay",
      "instance_id": "airplay",
      "enabled": false,
      "name": "Airplay",
      "last_error": null
    },
    "dlna": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "player",
      "domain": "dlna",
      "instance_id": "dlna",
      "enabled": false,
      "name": "UPnP/DLNA Player provider",
      "last_error": null
    },
    "sonos": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "player",
      "domain": "sonos",
      "instance_id": "sonos",
      "enabled": false,
      "name": "SONOS",
      "last_error": null
    },
    "url": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "music",
      "domain": "url",
      "instance_id": "url",
      "enabled": true,
      "name": "URL",
      "last_error": null
    },
    "slimproto": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "player",
      "domain": "slimproto",
      "instance_id": "slimproto",
      "enabled": true,
      "name": "Slimproto",
      "last_error": null
    },
    "fanarttv": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "metadata",
      "domain": "fanarttv",
      "instance_id": "fanarttv",
      "enabled": true,
      "name": "fanart.tv Metadata provider",
      "last_error": null
    },
    "musicbrainz": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "metadata",
      "domain": "musicbrainz",
      "instance_id": "musicbrainz",
      "enabled": true,
      "name": "MusicBrainz Metadata provider",
      "last_error": null
    },
    "theaudiodb": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "metadata",
      "domain": "theaudiodb",
      "instance_id": "theaudiodb",
      "enabled": true,
      "name": "TheAudioDB Metadata provider",
      "last_error": null
    },
    "ugp": {
      "values": {
        "ugp_1": [
          [redacted]
        ],
        "log_level": "ERROR"
      },
      "type": "player",
      "domain": "ugp",
      "instance_id": "ugp",
      "enabled": true,
      "name": "Universal Group Player",
      "last_error": null
    },
    "filesystem_local": {
      "values": {
        "path": "/Music",
        "missing_album_artist_action": "track_artist",
        "log_level": "ERROR"
      },
      "type": "music",
      "domain": "filesystem_local",
      "instance_id": "filesystem_local",
      "enabled": true,
      "name": "Filesystem (local disk)",
      "last_error": null
    },
    "radiobrowser": {
      "values": {
        "log_level": "ERROR"
      },
      "type": "music",
      "domain": "radiobrowser",
      "instance_id": "radiobrowser",
      "enabled": true,
      "name": "RadioBrowser",
      "last_error": null
    }
  },
  "players": {
    [redacted]
  },
  "core": {
    "music": {
      "values": {
        "sync_interval": 60,
        "log_level": "ERROR"
      },
      "domain": "music",
      "last_error": null
    },
    "metadata": {
      "values": {
        "log_level": "ERROR"
      },
      "domain": "metadata",
      "last_error": null
    },
    "webserver": {
      "values": {
        "log_level": "ERROR"
      },
      "domain": "webserver",
      "last_error": null
    },
    "player_queues": {
      "values": {
        "log_level": "ERROR"
      },
      "domain": "player_queues",
      "last_error": null
    },
    "players": {
      "values": {
        "log_level": "ERROR"
      },
      "domain": "players",
      "last_error": null
    },
    "streams": {
      "values": {
        "log_level": "ERROR"
      },
      "domain": "streams",
      "last_error": null
    }
  }
}

Ok thanks. This will get looked at.

Still looking but the global setting is set on the addon configuration page

Please check the global logging level on the addon page. If that is set at a more verbose level then I believe it will override a less verbose level on the individual component

Just need some followup here or we will close this

Thanks for the report in a future update the following will be addressed

adjust log level higher than global should work
set global log level for docker users

This should be fixed in beta 75. Let us know if not.

Seems mostly fixed thanks but might need another tweak - I'm still getting warning-level messages from the webserver component with global set to warning and webserver set to error. Think that's the only one I've seen not behaving itself since updating though 👍🏻

Can you open a new issue for that? Thanks.


[DESIGN] design a landing page for LinksHub in Figma

What would you like to share or ask?
As our project is open source and we don't have a landing page yet, it would be great to have a place where users can learn about the project, its features, and how to get started. This issue is to track the design of a landing page for LinksHub in Figma.
The landing page should include:

A brief overview of the project and its purpose
A list of key features
Screenshots and/or demo videos to showcase the project
Links to github and documentation to help users get started
A section for contributors to learn how they can contribute to the project
A way for users to contact the development team for support or to report issues

Design and layout should be modern and user-friendly, and should be accessible to users of all abilities.

Please provide a Figma link to the design and share it with the development team. Also, let me know if you have any questions or need clarification on any of the above points.

It's also important to note that the design should be accessible, responsive and mobile-friendly as well.

Additional information
No response

I will try this :)

Working on it :)

Can you provide me with the key features of the LinksHub, It will be helpful so that I can work on the second point. :)

Can you provide me the key features of the LinksHub, It will be helpful so that I can work on the second point. :)

Add Links
Get Links
Save Time
Easy to use

Thanks :)

Hi @shubhsharma19, regarding pushing design, you can create a design dir under public dir and push files there. and if it's a link, then add this link in a json format.

@shubhsharma19 How's it going?

@shubhsharma19 How's it going?

I will be uploading the design idea today and will be making it better with updates. :)

@shubhsharma19 Any updates?

Sorry for the delay i will keep giving u updates now, almost forgot


does the params shape follow the particle.js params shape

doesn't appear to respond at all to these inputs.
                "move": {
                  "enable": true,
                  "speed": 12,
                  "direction": "top",
                  "random": false,
                  "straight": false,
                }
                "particles": {
                  "number": {
                    "value": 80
                  },
                  "size": {
                    "value": 50,
                    "random": true,
                  }
                },
                "interactivity": {
                  "events": {
                    "onhover": {
                      "enable": true,
                      "mode": "repulse"
                    }
                  }
                },

id recommend making the params API follow what particle.js uses,  this seems pretty useless :(

They do follow the particles.js interface.
Here you can find the detailed shape of the params object and here you can create it interactively.
The snippet you sent is missing a comma after the move object and it should be placed under particles object.


Readall after provisioning is not working

Hi -
I have been testing different units with the freeacs software and I cannot get the "read all" to work after I have provisioned a device. The provisioning works fine, but then I want the CPE to send the new configuration to the ACS to verify changes through the freeacs GUI and I get a timeout error. It's only after I reboot the CPE that the read all command that I sent works.
I have attached the Packet captures for each change that I made. Please let me know if you need more information.
20190424acs-readall.zip

Are your device listening on port 8890? Can you reach them in web browser?

Yes, I can reach them via HTTP (GUI). The CPE takes the configuration values that I change from the ACS but once the conversation is done. I cannot send another change whether it be a read all or a provisioning command. I get a read timeout error.

Please show me ConnectionRequestURL displayed in cpe page. Maybe you need set A flag for this parameter

In your readall file, cpe not respond on get request from ACS.
Sometimes I had same problems with some CPEs.

yes, you are correct. I have another CPE and the ACS server works as expected with it.


Add possibility to override ActivityLogger

Hi.
I have quite specific scenario in my app where I need to override some functions in ActivityLogger in order to perform some actions on the model before actually passing it futher (performedOn()).
It is nice that we can already use customized Activity model, but would be also nice to add possibility to do same with ActivityLogger.
I would just override activity() helper function, but as it is loaded before app helpers it is impossible to do so without ugly hacks with composer autoloader.
For now, I am just using custom function act() witch is using my own Logger.

It is already possible. The class is bind to the service container in the ServiceProvider. You only have to bind your own class to it.
https://github.com/spatie/laravel-activitylog/blob/3bee85adf8e19e3bc0cafab56599a442014a9169/src/ActivitylogServiceProvider.php#L39
$this->app->bind(\Spatie\Activitylog\ActivityLogger::class, \App\ActivityLogger::class);

Well, fun thing is that I even tried to override it in IoC before, but for some reason it didn't work (maybe misspelled?).
This time it worked.
Thank you @Gummibeer, issue solved :-)


Runtime error if number of training iterations is large  on Ascend

It is the bug introduced by #254 , fix later
Traceback (most recent call last):
File "/home/work/user-job-dir/V0003/tools/train.py", line 243, in
main(config)
File "/home/work/user-job-dir/V0003/tools/train.py", line 202, in main
dataset_sink_mode=cfg.train.dataset_sink_mode)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/mindspore/train/model.py", line 1051, in train
initial_epoch=initial_epoch)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/mindspore/train/model.py", line 98, in wrapper
func(self, *args, **kwargs)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/mindspore/train/model.py", line 618, in _train
self._train_process(epoch, train_dataset, list_callback, cb_params, initial_epoch, valid_infos)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/mindspore/train/model.py", line 935, in _train_process
list_callback.on_train_epoch_end(run_context)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/mindspore/train/callback/_callback.py", line 371, in on_train_epoch_end
cb.on_train_epoch_end(run_context)
File "/home/work/user-job-dir/V0003/mindocr/utils/callbacks.py", line 268, in on_train_epoch_end
train_loss = ms.ops.stack(self._losses).mean().asnumpy()
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/mindspore/ops/function/array_func.py", line 1209, in stack
return _stack(input_x)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/mindspore/ops/primitive.py", line 296, in call
return _run_op(self, self.name, args)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/mindspore/common/api.py", line 98, in wrapper
results = fn(*arg, **kwargs)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/mindspore/ops/primitive.py", line 733, in _run_op
output = real_run_op(obj, op_name, args)
RuntimeError: Single op compile failed, op: concat_d_3769627495608740356_2
except_msg: 2023-05-12 03:25:57.396075+00:00: Query except_msg:Traceback (most recent call last):
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/te_fusion/parallel_compilation.py", line 1643, in run
optional_input_mode=self._optional_input_mode)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/te_fusion/fusion_manager.py", line 1337, in build_single_op
compile_info = call_op()
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/te_fusion/fusion_manager.py", line 1315, in call_op
opfunc(*inputs, *outputs, *new_attrs, **kwargs)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/common/register/operation_func_mgr.py", line 98, in wrapper
return func(*args, **kwargs)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/common/utils/para_check.py", line 547, in _in_wrapper
return func(*args, **kwargs)
File "/usr/local/Ascend/ascend-toolkit/latest/opp/built-in/op_impl/ai_core/tbe/impl/dynamic/concat_d.py", line 183, in concat_d
concat_d_dsl(input_values, output_data, concat_dim, kernel_name)
File "/usr/local/Ascend/ascend-toolkit/latest/opp/built-in/op_impl/ai_core/tbe/impl/dynamic/concat_d.py", line 144, in concat_d_dsl
ins = classify([input_values], "concat", extra_params)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/dsl/api.py", line 1088, in classify
return shape_classifier.classify(ins, mode, extra_params)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/common/register/operation_func_mgr.py", line 177, in wrapper
return func(*args, **kwargs)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/common/register/operation_func_mgr.py", line 177, in wrapper
return func(*args, **kwargs)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/dsl/classifier/shape_classifier.py", line 83, in classify
return classifier_func(ins, extra_params)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/dsl/classifier/shape_classifier.py", line 98, in wrapper
return func(*args, **kwargs)
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/dsl/classifier/concat_classifier.py", line 50, in classify
return classifier.classify()
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/dsl/classifier/concat_classifier.py", line 79, in classify
self._check_inputs()
File "/usr/local/ma/python3.7/lib/python3.7/site-packages/tbe/dsl/classifier/concat_classifier.py", line 110, in _check_inputs
raise RuntimeError(dict_args, get_error_message(dict_args))
RuntimeError: ({'errCode': 'E90001', 'detailed_cause': 'input numbers error, input numbers must be greater 0 and less equal 63 , now, it is 112'}, 'Compile operator failed, cause: Parameters check failed, detailed information: input numbers error, input numbers must be greater 0 and less equal 63 , now, it is 112.')

How to reproduce this error?

Training with a dataset that needs roughly >7000 iterations per epoch on Ascend. Then the ms.ops.stack on EvalSaveCallBack will raise this error. I guess it is due to the loss list is too huge for concatenation.
Btw we can use a loss_meter  as implemented in https://github.com/mindspore-lab/mindpose/blob/master/mindpose/callbacks/eval_callback.py to prevent this, and also save some memory for storing the long loss list.

solved by #285


Rename KeyHandlers -> IoHandler

Technically it handles mouse events as well. This was confusing developers.

Should be no functionality change.


[🐛 BUG] Chat control "new message" pop-up is weirdly placed and often inaccessible

What went wrong? 🤔
The chat control has a pop-up button saying "A new message is available" when you type a new message and the chat has not auto-scrolled down. Clicking on the button scrolls down to the last message. This pop-up is placed weirdly. For example, here, it overlaps with the chat control:

In another app, it was hidden behind the sidebar:

In another one, it's hidden off-screen:

Expected Behavior
This pop-up should probably be in a fixed position on the page, and it should be laid on top of anything else, like a notification (the ones spawned by gui.notify), for example
Steps to Reproduce Issue
Run this code and type many messages until they fill the screen. The pop-up will appear.
from taipy.gui import Gui
import taipy.gui.builder as tgb

messages = []

def evaluate(state, var_name: str, payload: dict):
    state.messages.append([10, "Loading...", "Robot"])
    state.messages = state.messages

with tgb.Page() as page_home:
    with tgb.layout(columns="1 3", columns__mobile="1"):
        tgb.chat(
            messages="{messages}",
            on_action=evaluate,
            height="80vh",
        )

if __name__ == "__main__":
    gui = Gui(page=page_home)
    gui.run()

Browsers
Chrome
OS
Windows
Version of Taipy
4.0.0
Acceptance Criteria

[ ] A unit test reproducing the bug is added.
[ ] Any new code is covered by a unit tested.
[ ] Check code coverage is at least 90%.
[ ] The bug reporter validated the fix.
[ ] Related issue(s) in taipy-doc are created for documentation and Release Notes are updated.

Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

/attempt#2112 please assign me this issue

@sahilborse You are also assigned here


Install package in workflows before running test

This PR changes the test workflows. The workflows now:

Install Poetry
Use Poetry to install the package and dependencies, including pytest
Use Poetry to run pytest

This fixes the failing tests caused by #6 😅.
Closes #9
constraints.txt
An additional opinionated change I made is lock the Poetry version by specifying it in .github/workflows/constraints.txt. This would prevent surprise CI failures in the future if Poetry has further breaking changes. This will also prevent the CI from always installing the latest version of Poetry. The Poetry version in the CI must then be upgraded by bumping the version in .github/workflows/constraints.txt either manually or with GitHub's Dependabot. I am also happy to help on the Dependabot end if wanted.
Of course, if this is too strict and complex, we can remove constraints.txt and let the workflow always install the latest version of Poetry.

Looks good to me! Part of me would always let the latest version of Poetry install itself and fix it when it breaks, but this is fine. Cheers!


Combinatorial generation on every prompt word?

I'm curious if this could be done without needing to use wildcards or enclose anything inside of {x|y}? Basically, I want to test how removing 1 word and then adding it back to the prompt affects it, and to do this for all words in the prompt in sequence.

have you tried the built-in prompt matrix script - I think it does something like that.

Prompt matrix script is similar, but not the same. As you can see in olivio's demo, he ends up with 2^4 prompt variations, which is 16 different images. If he had a prompt of 30 words, that would be 2^30, which is<PHONE_NUMBER> images. I only wish to only ever have 1 word removed from the prompt at a time, which would result in 30 images + 1 image where no word is removed. The purpose is to see how that specific word impacts the overall prompt without going into the madness of testing every conceivable combo.
Also using prompt matrix for this means we can't use the other scripts like x/y plot.

Actually, I found https://github.com/Extraltodeus/test_my_prompt which does this. It still would be nice if we didn't have to take up a script slot just to do this, but going to close this issue.

It's possible that jinja templates might offer that functionality - worth
investigating. If you do and find a solution, I'll add it to the cookbook.
On Thu, 25 May 2023 at 08:50, PaperOrb @.***> wrote:

Actually, I found https://github.com/Extraltodeus/test_my_prompt which
does this. It still would be nice if we didn't have to take up a script
slot just to do this, but going to close this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/adieyal/sd-dynamic-prompts/issues/426#issuecomment-1562370758,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADGXMXNGRY3POQQVZSQFMLXH36KJANCNFSM6AAAAAAX52U7CY
.
You are receiving this because you commented.Message ID:
@.***>


Added source ru.readmanga

Checklist:

[x] Updated source's version for individual source changes
[x] Updated all sources' versions for template changes
[x] Set appropriate nsfw value
[x] Did not change id even if a source's name or language were changed
[x] Tested the modifications by running it on the simulator or a test device
[x] Run cargo +nightly clippy

make sure to run cargo +nightly clippy and correct the warnings that it gives you.

make sure to run cargo +nightly clippy and correct the warnings that it gives you.

Ran clippy, fixed warnings

@Skittyblock , do I have to do any changes or everything is ok?

https://readmanga.live gives me a 404. Is that because of my location?

https://readmanga.live gives me a 404. Is that because of my location?

I guess it is. But it opens from my internet provider and with VPN.


Function#bind/call/apply doesn't return or have same signatures

TypeScript Version:
master
Code
interface F {
  (): number;
  (p): string;
}
var f: F = (<F>null).bind(null);
var c: number = f.call(null);
var a: number = f.apply(null);

Expected behavior:
$ node built/local/tsc.js index.ts

Actual behavior:
$ node built/local/tsc.js index.ts
index.ts(5,5): error TS2322: Type '(this: void, ...argArray: any[]) => string' i
s not assignable to type 'F'.
  Type 'string' is not assignable to type 'number'.
index.ts(6,5): error TS2322: Type 'string' is not assignable to type 'number'.
index.ts(7,5): error TS2322: Type 'string' is not assignable to type 'number'.

@sandersn looks like the changes to the library had this unintended effect. we should revert these changes, specially that they do not really provide much added type safety.

I think this is a real bug with this function types, not with lib.d.ts. The bug goes away when I rewrite the example plus Function.bind/call/apply to have explicit "this-like" parameters.

Never mind, there was an error in my rewrite.

It turns out this behaviour is in the spec, section 3.10.6. When performing type inference of types with multiple call signatures, the compiler pairs them all up but ignore extra ones at the beginning. So F's second call signature will be the only one considered.
In any case, TypeScript doesn't have the capability to have Function.call/apply/bind choose between different overloads based on number of arguments.

@sandersn , yes. But before it used to return any, after your change it returns something more concrete, which breaks user code. since we concluded that the change does not provide much value in terms of type safety, it seems better to roll it back and unbreak such scenarios.

I came across this issue while using the nightlies and noticed I lost some type safety.
Is there a recommended practice for this common pattern (imho):
// Note that JSX.Element everywhere in this comment is automatically inferred—I'm sticking it in here for the reader.
class Foo extends React.Component<Props, State> {
    _someCallback(): JSX.Element {
        …
    }
    someCallback = this._someCallback.bind(this);

    render() {
        // pass along this.someCallback in props somewhere
        // we use the pre-bound method so that shallow comparison of props works correctly
    }
}

The specific error I received (error TS2453: The type argument for type parameter 'P' cannot be inferred from the usage.) was because, when using helper function for creating react elements, and the props containing this callback, the type inference is crucial to avoid specifying a nested template argument.
I've worked around it by re-writing as:
class Foo extends React.Component<Props, State> {
    someCallback = (): JSX.Element {
        …
    }

However, such a method is unavailable to subclasses.  It doesn't matter to me at the moment—but I'd avoid having to make bigger changes if I did need it to be available.
Is there a cleaner bind syntax available that preserves the types or a way of specifying it in the someCallback = this._someCallback.bind(this); line?

I'm not sure what you mean when you say that assigning an arrow function is unavailable to subclasses. Could you do the following, or are you talking about a different case?
class Bar extends Foo {
    someCallback = () => this._someCallback();
}

There's not much overhead since _someCallback has zero arguments, but of course you'd end up repeating all the arguments if it had more.

@sandersn
The 2nd code snippet in my comment skipped the _someCallback declaration and that's why the arrow function wouldn't be available in the callback.  The common thing in both snippets is that the signature is specified exactly once.  As you correctly point out, a solution is to repeat the full signature a 2nd time when defining the arrow function.  Typically, these bound callbacks, when passed as props to child components, are expected to be called with parameters (that I didn't show in the example.)  If these parameters don't have simple names to refer to them (inline interface definitions, repeated template params), it's ugly repeating them.
This is not a deal breaker or a complaint.  I'm trying to figure out a clean way to create such bindings.  It sounds like the arrow functions are the way to go (or I'd have received an alternate suggestion), so this satisfies my question.  Thanks.

I just answered my own question.  I can write a helper,
function bind<F extends Function, T>(func: F, self: T) {
    return func.bind(self) as F;
}

and then
someCallback = bind(this._someCallback, this);

and that preserves the types.

However, such a method is unavailable to subclasses.

Try not to create deep hierarchies with overrides in ReactJs components.

I'm trying to figure out a clean way to create such bindings

If you want to use bind (don't recommend that either as you already figured) you can use the pattern shown here to return to type safety : https://basarat.gitbooks.io/typescript/content/docs/project/declarationspaces.html :rose:
class Foo {
	someCallback() { }
}
declare let _foo: Foo;

class Bar extends Foo {
	_someCallback: typeof _foo.someCallback = this.someCallback.bind(this);
}

@basarat—gotcha.  I don't actually have subclasses in my own code (preferring composition) so point taken.
Thanks for your link to your typescript docs—there's some nice stuff there.


Disability question is required on yes.

Description
Required disability
If the user selects "yes" for the disability question in the signup page, then the text field is required.
Acceptance Criteria
Must enter some text in the yes text box.
Test Instructions

Navigate to branch name: required-disability
Go to the sign up page
Submit the form as is. (all empty)
Should see 4 errors.
Now, select yes for the question: "Do you identify as a person with a disability?"
Re-submit the form.
Should see 5 errors.

Help Requested

any constructed feedback is greatly appreciated!

Checklist

[x] Strings use placeholders for translation (No hard coded strings)
[x] Unit tests have been added/updated

Looks good, the only small problem is that the question after answering yes still says optional.

That could lead to confusion.

Thank you I added the required notification as needed. The question itself is optional, but if you choose to answer with a "yes", then it becomes required.


Update FAQ on firewall error fix

Update FAQ on what to do if firewall error when sync job is provisioning Azure DB

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

This appears to duplicate work done in the private repo. We'll close the pull request. Thanks.


decreased DNA content during G0

to correct annotations to FYPO:0000160 from PMID:19366728

DNA content decreased during G0 FYPO:0007327
edit file: 6ddc8dcd95ec79f7c7bc17a67ecb9d46409985ec
release: 84789e208e241b6502caf8a570be07496eea232a


Rewrite the program to use transaction_id

We need to update dependencies and rewrite the part related to the NFT transactions to use transaction_id.

Done in #12.


workrave: 1.10.21 -> 1.10.23

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/workrave/versions.
meta.description for workrave is: '"A program to help prevent Repetitive Strain Injury"'.
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23/bin/workrave had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23/bin/.workrave-wrapped had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 1.10.23 with grep in /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23
directory tree listing: https://gist.github.com/78773d24ebc9737b274bd20e46b3b8b1
du listing: https://gist.github.com/961647d9d82990c8eb8d6d844e365902

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:
nix-shell -p cachix --run 'cachix use r-ryantm'

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/f44a115b770f3788a55ce9cad943473674f5bd46.tar.gz -p workrave
ls -la /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23

cc @prikhi for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: workrave
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: workrave
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
Wrapping program /nix/store/pdl32l5laj761dy9ji7mh45k5ilmqaq5-workrave-1.10.23/bin/workrave
shrinking RPATHs of ELF executables and libraries in /nix/store/pdl32l5laj761dy9ji7mh45k5ilmqaq5-workrave-1.10.23
shrinking /nix/store/pdl32l5laj761dy9ji7mh45k5ilmqaq5-workrave-1.10.23/lib/libworkrave-private-1.0.so.0.0.0
shrinking /nix/store/pdl32l5laj761dy9ji7mh45k5ilmqaq5-workrave-1.10.23/bin/.workrave-wrapped
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/pdl32l5laj761dy9ji7mh45k5ilmqaq5-workrave-1.10.23/lib  /nix/store/pdl32l5laj761dy9ji7mh45k5ilmqaq5-workrave-1.10.23/bin
patching script interpreter paths in /nix/store/pdl32l5laj761dy9ji7mh45k5ilmqaq5-workrave-1.10.23
checking for references to /build in /nix/store/pdl32l5laj761dy9ji7mh45k5ilmqaq5-workrave-1.10.23...

Success on x86_64-linux (full log)
Attempted: workrave
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
Wrapping program /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23/bin/workrave
shrinking RPATHs of ELF executables and libraries in /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23
shrinking /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23/lib/libworkrave-private-1.0.so.0.0.0
shrinking /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23/bin/.workrave-wrapped
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23/lib  /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23/bin
patching script interpreter paths in /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23
checking for references to /build in /nix/store/fjd6jypyc3hcpwmjas9sdnmzmhgh0zkd-workrave-1.10.23...


Update performance summary

Purpose

Update performance summary

Codecov Report

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

@@               Coverage Diff               @@
##             next-release   #12669   +/-   ##
===============================================
  Coverage                ?   59.86%
  Complexity              ?      659
===============================================
  Files                   ?     2064
  Lines                   ?   101597
  Branches                ?    12931
===============================================
  Hits                    ?    60820
  Misses                  ?    35581
  Partials                ?     5196

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 e3d04fc...72e8d3f. Read the comment docs.


add check if server url containing http, if not add it

Hi this is my first PR,
I fixed this bug https://github.com/prometheus/prometheus/issues/6397
thanks,

Thank you for your pull request @ronshuli.
This does not seem to be the right fix however. I think we should do that earlier, at the point where we parse the parameter, so that it will benefit all the commands.
Ping me if you need more assitance to move forward, or when you have updated your pull request, and once again, welcome on the Prometheus project!

Hello @roidelapluie,
I think I fixed the PR according what you asked,
I would love that you will check this once again and thank for notice.
I'm happy to join Prometheus project :)

I will review in a few days ; however I think the approach should be to parse the URL and see if there is a protocol ; instead of checking for a string. I will come back to you :)
Also, did you test this? I was under the impression that we check the URL when we parse the arguments, using prometheus/common

hello,
Somehow this has been addressed in the meantime: #7956
Thanks for your work and sorry this went un-noticed!

hello,
Somehow this has been addressed in the meantime: #7956
Thanks for your work and sorry this went un-noticed!


Support HTTP oci registries

What would you like to be added:
Lakom to support resolving tags to digests and verify images with registries that are exposed only via HTTP, not HTTPS.
Why is this needed:
Mainly for local development setup where skaffold registries are not exposed via HTTPS.
/kind enhancement
/area ipcei
/assign @rrhubenov

Fixed by #93


fix(core): deepClone did not clone RegExp objects correctly

Information

Type
Breaking change

Fix
No

Fixes an issue where deepClone won't clone RegExp instances correctly.
Required for #2061 to work as the Store deepClones values on Store.merge()
Note that I did not look closely at deepMerge.
Todos

[x] Tests
[ ] Coverage
[ ] Example
[ ] Documentation

Hello @jojoxd
Perfect PR ;). I'll merge your PR when all tests are passed!
Thanks a lot for this fix!
See you
Romain

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

v6.128.10
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.0-rc.6 :tada:
The release is available on:

v7.0.0-rc.6
GitHub release

Your semantic-release bot :package::rocket:


Better config for Twin Missile Rail

I made some slight edits to the Twin Missile Rail that I believe make it better for missiles, in that the salvo size is 1 and that it doesn't fire the symmetry twin.
If the missiles are fired without these altercations, there is a (small) chance that all the missiles will be lost to countermeasures and reloading takes time, increasing the probability of being shot down. With these changes, however, the supply lasts more, meaning more engagements and more opportunities to launch a second missile at an evading craft should one missile be ineffective.
The current config for the rail is more ideal for bombs imho, but the colliders... collide, and the bombs will detonate prematurely, if not bounce off-course. The same issue is present with larger missiles like the Maverick. Could this be fixed in some way, for instance having the ability to change the ordinance offset?

I would also like to see a fix for this.
Speaking of the rail, could it get updated to have an adjustable height for larger missiles once they are legitimately compatible?


Meta.json's "TestRenderModel" is not pushed with templates

I'd like to include some default test data in the TestRenderModel of a template's meta.json, however it isn't kept when I push my templates. Instead, it's replaced with:
{
	"company": {
		"primaryColor": "primaryColor_Value",
		"logo": "logo_Value",
		"name": "name_Value"
	},
	"body": "body_Value",
	"product_name": "product_name_Value",
	"product_url": "product_url_Value",
	"bodyText": "bodyText_Value",
	"company_name": "company_name_Value",
	"company_address": "company_address_Value",
	"subject": "subject_Value"
}

The "company" field is our own custom properties we've added. We don't use the others, and have removed them from the template, yet they come back. If we then pull the templates our meta.json gets overwritten with the default data again.

Hey @mattjennings, can you send me both the HTML and Text version of the template that's giving you trouble? You can email it to<EMAIL_ADDRESS>And by pull, do you mean pulling via the postmark templates pull command or git?

Correct! I'll email them to you.

Closing this since we touched base via email. Thanks @mattjennings!


Use SVG or higher resolution badge F-Droid

:bulb: Idea
Use SVG or hIgher resolution badge F-Droid
:computer: Target components
Website
PS Consider adding the three badges also to the README.

If SVG is available, always use it!

yes, see e.g. https://github.com/f-droid/artwork/blob/master/badge/get-it-on.svg


Add aswin koliyot

@akoliyot, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

Hey Roshan,
So my interest in contributing to open source came when I had to work with
a couple of people from different countries.
We were using Git and I didn't know how to use it, so I figured learning
that would be a good investment.
And the best way to learn is to actually do it, so I took a couple of
tutorials and then participated in Hacktoberfest 2017.
It feels great to work with amazing people you barely know and them making
a positive impact by contributing to their project.
Also, I want to thank you for helping more people get into open source.
Best Regards,
Aswin Koliyot
On 26 October 2017 at 01:28, Roshan Jossey<EMAIL_ADDRESS>wrote:

@akoliyot https://github.com/akoliyot, I'm quite elated about your pull
request. I wanna evolve this project to addresses various problems faced by
first-time contributors. I'd love to learn about your journey in open
source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a
pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app.
Could you please go to https://roshanjossey.github.
io/first-contributions/#social-share and share your first contribution to
open source? Also, check out projects with easy issues while you’re there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Roshanjossey/first-contributions/pull/1321#issuecomment-339452543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIQkrvHjqkjDbp6diqUIR7tZFDxQQwPsks5sv5L0gaJpZM4QGR8E
.


Merge polygons differences between Clipper1 and Clipper2

Hi,
I have some differences between Clipper 1 (27 February 2017 v6.4.2) and Clipper 2 (3 August 2022 v1.0.0) when I merge several polygons.
With Clipper 1 I have the expected result: 1 polygon and with Clipper2 I have 3 polygons.
I made 2 samples to reproduce the issue: 1 with Clipper1 and 1 with Clipper2, I tried to keep the same structure: https://gist.github.com/scemino/89fd2b3cddc95b319d6ec07cf0aa8159
PS: thank you for your awesome library.

Hi scemino.
Unfortunately sometimes Clipper1 does better that Clipper2 WRT merging touching polygons.
http://www.angusj.com/clipper2/Docs/Overview.htm#touching


feat(ghActions): don't run workflows on draft PRs

Description
Changes proposed in this pull request:

feat(ghActions): don't run workflows on draft PRs

Related issue(s)

Definition of done

[x] The PR's title starts with one of the following prefixes:

feat: A new feature
fix: A bug fix
docs: Documentation only changes
refactor: A code change that neither fixes a bug nor adds a feature
test: Adding tests
chore: Maintainance changes to the build process or auxiliary tools, libraries, workflows, etc.

[x] Related issues are linked. To link internal trackers, use the issue IDs like backlog#4567
[x] Explain clearly why you created the PR and what changes it introduces
[x] All necessary steps are delivered, for example, tests, documentation, merging

/override tide


Could not pass event NPCRightClickEvent to Quests v4.0.1-b233 java.lang.IllegalStateException: QuesterPreCompleteQuestEvent may only be triggered asynchronously

What is your Quests version / build number (do not say "latest")?
Quests v4.0.1-b233
Describe the problem and how to replicate it. Any console errors to post using a pastebin?
cannot right click NPC, when clicked, this error shows
This problem does not occur when i was using b230 version
https://paste.gg/p/anonymous/53c5abec20a7481d94019215ced3d3c1

If applicable, can you provide an example quest from quests.yml using a pastebin?
https://paste.gg/p/anonymous/da22036804ef49178f3844da213b6487

Thanks for the report. Please try https://ci.codemc.io/job/PikaMug/job/Quests/234/


chore(dep) Upgrade to Concerto 1.0.0-alpha.10

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>Changes

Upgrade to Concerto 1.0.0-alpha.10

Coverage remained the same at 95.616% when pulling d555fce28baff1a1156fd31222f34cd140668ac7 on jeromesimeon:js-alpha.9 into 06f1d21689f763633f40e2e25247b364f998a7a0 on accordproject:master.


scripts: Mount /run/udev for gamepads

To use an Xbox controller via libmanette, /run/udev needs to be mounted. Otherwise I see errors such as openat(AT_FDCWD<..>, "/run/udev/data/c13:63", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory) and the gamepad isn't detected.

Looks good, thanks!


Handle throttling

What type of PR is this?
/kind feature

What this PR does / why we need it:
To handle throttling
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
It handle code error and code type.
No Mock.
First version that will be changes when throttling will have more restrict rules.
TODOs:

[ ] squashed commits
[ ] includes documentation
[ ] adds unit tests

https://github.com/outscale-dev/cluster-api-provider-outscale/actions/runs/3388862809/jobs/5638128850
Freeza failed

Frieza should have to destroy resources, the tests should

Closes https://github.com/outscale-dev/cluster-api-provider-outscale/issues/3


set privacy threshold from config

Add a config option for privacy filter and use it in summary functions. Error when set < 2?

🚢 'd
privacyThreshold is a new config attribute (added to template, but also has a default fallback) respected by the summary algorithm for trips and flows of all space & time dimensions.
https://github.com/sharedstreets/sharedstreets-mobility-metrics/commit/fefdceced1c5c25bad584b63a2a2957aff97ddef


After updating atom/ftp package the local and remote panels swapped sides

After updating both Atom and the package the remote and local panels have swapped places and I can see a way to get them back to the correct way around.

Hi, thanks for report issue.
Try to new Remote-FTP version.
This version Supported Dock / Pane method.


Self docking on charger

Hello,
Thanks for your work, I appreciate a lot to connect to my Cozmo directly without any android device ;) 🥇
I just discovered this script based on cozmosdk that allow Cozmo to self dock. It seems that this script is using internal image recognition :
https://forums.anki.com/t/program-self-docking-on-charger/13830
Do you think it is possible to have the same functionnality here with pycozmo ?? It would be nice to can charge it automatically
Thanks
Max

You are correct - the script uses at least the following features, implemented in the Cozmo app which are thus not available in PyCozmo today:

text-to-speech

robot.say_text()

image recognition

robot.world.wait_for_observed_light_cube()

path planning

robot.place_object_on_ground_here()
robot.place_on_object()
robot.pickup_object()
robot.go_to_object()

behaviors

robot.start_behavior()

From my perspective, such features do not belong to PyCozmo, which provides only "low-level communication" with the robot.
I am planning to start working on a new project in the near future though, code named PyCozmoBrain. I think of it as a replacement for the Cozmo app, written in Python and using PyCozmo for communicating with the robot. It will provide at least:

image recognition - cubes, pets, people, etc.
object tracking
path planning

Help is welcome!

Hello zayfod,
I agree completly your approach.
I have experience in SW dev and python so It will be a pleasure to help you if you want !
Will opencv be mainly used for pyCozmoBrain or tensorflow maybe ?
See You,
Max

Image recognition will definitely be based on Tensorflow. OpenCV may still come handy for image manipulation though.


Expand test coverage for experimental Pipeline API

Change Summary

I have added the following test cases to test_pipeline.py:

test_ge_le_gt_lt

Previously there were only ge and le, so I added test codes for gt and lt.

test_parse_multipleOf

Previously there were only cases for int, so I added test codes for float and decimal.

test_len_constraints

Newly added.

test_interval_constraints

Newly added.

I'm not confident if this test case is correctly constructed, so I would like you to review it.

test_parse_tz

Added because there was no pattern for ValueError.

Pre-PR Coverage of pipeline.py: 83.30%
Current Coverage of pipeline.py w/ this PR: 93.01%
Related issue number

fix https://github.com/pydantic/pydantic/issues/9524
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[x] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @sydney-runkle

please review

@sydney-runkle
Thank you for merging the PR!

What did you think about this API? We're looking for feedback on this new experimental feature 🚀 !

I genuinely think this is a great feature.
In the past, when defining complex constraints and transformations for model fields in Pydantic, we had to use separate Field Validators and Model Validators. However, with the introduction of this pipeline API, I believe it has enhanced type safety and improved readability with more concise syntax.
If there are any other benefits or effective ways to use this pipeline API, I would love to learn about them. :)
I understand this is currently an experimental feature, but I look forward to incorporating it once it is officially released. Thank you for implementing such a useful feature!

@K-dash,
Wonderful, thanks so much for your feedback! We're super excited to get your feedback. @adriangb did all of the initial work on this awesome feature, and even wrote a PEP to help support this! https://peps.python.org/pep-0746/
I think the main benefits are:

Type safety
Readability
Ease of reuse
Customizing constraint orders (like for string constraints, which isn't currently easily doable without this feature)

I'm sure there are even more things, but these were the main motivating factors!

@sydney-runkle
Thank you for your response.
I have looked at PEP 746 created by Adrian.
I see, in order to validate the consistency between the type annotations specified in Annotated and the associated metadata, the metadata needs to implement the  __supports_type__  protocol.
I hadn't understood that part.

Ease of reuse
Customizing constraint orders (like for string constraints, which isn't currently easily doable without this feature)

Indeed, I also think the above two benefits are significant. Thank you.
Through the implementation of these test codes, I have learned a lot not only about the pipeline API but also about type annotations. I am very grateful!


Show vesting period and amount

Please add your planning poker estimate with ZenHub @Rooty2020

Please add your planning poker estimate with ZenHub @calvinkei

@calvinkei
The UI is ready.
https://www.figma.com/proto/w7LpoB8DejNVumjJPDrp7j/Forbole-X-_web-app_1.0?page-id=192%3A2&node-id=3451%3A201873&viewport=241%2C48%2C0.24&scaling=scale-down&starting-point-node-id=3992%3A239605

BDJuno will starts to support querying vesting tokens and periods next week. This task is blocked until then


denokv serve: 500 An internal server error occurred when attempting to sum a non kvu64

To reproduce: run denokv serve locally, connect via Deno.openKv, then attempt a sum operation on an existing non kvu64 value.
Expected:
http 4xx response, this is a non-retryable client error
Actual:
An error is logged: [ERROR denokv] Failed to write to database: Failed to perform 'sum' mutation on a non-U64 value in the database and an http 500 is returned to the client with body An internal server error occurred.  The spec encourages retrying 5xx, so it's probably important to return a 4xx here instead.

Upgrading to Deno 1.38.1 from 1.37.2, I see the built-in Deno.openKv('http://localhost:4512/') client will actually retry forever in this case - another reason to use 4xx here.

Upgrading to Deno 1.38.1 from 1.37.2, I see the built-in Deno.openKv('http://localhost:4512/') client will actually retry forever in this case - another reason to use 4xx here.

Thanks!


Translate Function not Initializing Properly

Translate function is not responding to initialization command.

Run the developer build release with the file 'S.A.N.E.Assistant.py'
Speak the word 'translate' into the computer (surrounding words should not matter so 'translate a phrase' should work fine)
No response given

The program should ask you to input words to translate then ask for a language to translate from.
Windows 10, Visual Studio Code

Issue was with capitalization when speaking, if you speak a word on its own, the speech recognition library automatically capitalizes it. We can fix this issue in the future with an automatic lowercasing line that transforms the audio to all lowercase letters.

fixed this issue and surrounding problems, new code is going to be released with the update on July 1st


Rescheduler Doc changes

Rescheduler now works with only DaemonSet pods and it knows priorities.

/cc @bsalamat

Done PTAL.

Thanks @bsalamat for the review


Hotfix/hostnetwork

Fixes
The flannel CNI is no longer supported. See https://github.com/kubernetes/website/commit/a9d7ba33446914de7c38adfc6825a5828facaff5
To make it work with the Calico CNI, we had to adjust the Chart to start pods with
      hostNetwork: true
      dnsPolicy: ClusterFirstWithHostNet

I suspect that in the case of dripline, this is a good default setting. I don't suspect that it will break flannel setups that are already running.
Testing
Tested with helm 3.2 and kubernetes 1.18.2 on bare metal. Using Calico CNI.
Prior to merging for releases:

[ ] update the project's version in the top-level CMakeLists.txt file
[x] update the appVersion to be the new container image tag version in chart/Chart.yaml

Actually, the problem really was the application pool I picked for Calico.
When I initially installed Calico, I unwittingly picked an IP pool that included my subnet ip address. I reinstalled Calico on a different IP pool and it now works well.


Refactor Server to use blueprints

Refactoring the code for the server to use blueprints will make it easier to group code together and break the different subsystems of the application into usable chunks. Known subsystems so far include:

Admin
API
Authn
Host

Closed in 3b1f163ff75820b78089840259ecb65e855038eb. Introduced /admin, /api, /auth, and the main blueprint. Updated the agent to hit the new /api/getwork and /api/submit as well.


refactor: remove redundant check for camera node

@StraToN @dploeger @dploeger FYI
I couldn't find any reason you needed to set camera_node in ESCItem to get push commands to work. As far as I can tell it's not necessary so I've removed it. If I'm wrong and it ends up that it's needed, the camera node setting in ESCItem (what it's for and what you set it to) needs to be documented.

Can the camera_node variable be removed entirely from ESCItem then? I'm not sure what it was put there for.

Can the camera_node variable be removed entirely from ESCItem then? I'm not sure what its purpose is - it looks deprecated to me, the only function that uses it is the one I've edited, but I'm not sure why it was originally put there.

No, it can be used together with camera_set_target if you don't want to use the global position of the item, but of a secondary node.

(at least that's the plan :D )

Happy to be overruled, but I'd suggest removing it and re-adding it as part of the proposed feature to keep the code/interface cleaner. If the camera_set_target change is planned for 4.0 though, maybe leave it and add an issue to track the work?

No sorry, I wasn't specific enough. It is already possible with the current code. It is not planned.
I was saying "that's the plan" because I hope, that that it's implemented currently. You might have a look.

As far as I can see it's not implemented, a search of the specific file returns no results.
/mnt/e/Godot Projects/escoria-demo-game$ grep -i camera_node ./addons/escoria-core/game/core-scripts/esc/commands/camera_set_target.gd
/mnt/e/Godot Projects/escoria-demo-game$

And a search of all files shows the same. The esc_item reference is the one we're updating here, the others don't point to the camera_node entity that is currently manually set.
/mnt/e/Godot Projects/escoria-demo-game$ grep -ir camera_node

addons/escoria-core/game/core-scripts/esc/commands/camera_push.gd:#   has a child node called `camera_node`, its location will be used. If not,
addons/escoria-core/game/core-scripts/esc_item.gd:export(NodePath) var camera_node
addons/escoria-core/game/core-scripts/esc_item.gd:func get_camera_node():
addons/escoria-core/game/core-scripts/esc_item.gd:      if camera_node and get_node(camera_node):
addons/escoria-core/game/core-scripts/esc_item.gd:              return get_node(camera_node)
addons/escoria-core/game/scenes/camera_player/esc_camera.gd:    elif p_target.has_method("get_camera_node"):
addons/escoria-core/game/scenes/camera_player/esc_camera.gd:            if "global_position" in p_target.get_camera_nod
():
addons/escoria-core/game/scenes/camera_player/esc_camera.gd:                    _target = p_target.get_camera_node().global_position
addons/escoria-core/game/scenes/camera_player/esc_camera.gd:            if "zoom" in p_target.get_camera_node():
addons/escoria-core/game/scenes/camera_player/esc_camera.gd:                    _zoom_target = p_target.get_camera_node().zoom

No, it's actually used in esc_camera!
p_target is the target for the set_target command and it can be of various type: A location, a vector2d or an escitem. So it calls get_camera_node on the esc item and sets the target to the position of that node instead of the escitem itself.
Or am I misreading that?

How that reads to me is that it looks for a "get_camera_node" method - not the node_path that's specified in ESCitem. As ESCitem has the function, it resolves the function and gets back whatever the ESCitem's "camera_node" node is. It could be changed to return the nodepath of camera_node if one's set, but that's not how the function is currently written.
(Assuming I'm reading it correctly 😄 )

How that reads to me is that it looks for a "get_camera_node" method - not the node_path that's specified in ESCitem. As ESCitem has the function, it resolves the function and gets back whatever the ESCitem's "camera_node" node is. It could be changed to return the nodepath of camera_node if one's set, but that's not how the function is currently written. (Assuming I'm reading it correctly 😄 )

Exactly, so the camera_node variable still is required (to set the path to the node which position should be used).

Not as written in this pr though.
if camera_node and get_node(camera_node): if has_node(camera_node): return get_node(camera_node)
This will return the node called "camera_node", not the contents of a nodepath variable.

? But the function in esc_camera.gd requires the node and not the nodepath?

Let me play around with it and get back to you.

Thanks! :partying_face:


Move manifest to discrete package

This moves the manifest oriented type definitions out of the storage package. This is the first attempt at resolving #6. Specifically, this covers item 4.
cc @jlhawn @dmcgowan @dmp42 @jfrazelle @icecrime

LGTM

It's difficult to review, but as far it does what #6 states: LGTM for me.

LGTM


lib.pci: Support short/canonical ("01:00.0") PCI addresses

Proposed alternative to #657 after tomatoes were thrown in response to my more complicated proposal.
Adds these two functions:

pci.canonical(address) => returns address in short canonical format (01:00.0)
pci.qualified(address) => returns address in fully qualified format (0000:01:00.0)

with the intention that users can specify either format and Snabb Switch will prefer the canonical format internally but expand to the qualified format when needed (e.g. for resolving sysfs paths).
This is a user visible change. The PCI device scan and selftest now uses canonical addresses:
$ sudo ./snabb snsh -t lib.hardware.pci
selftest: pci
pciaddress    vendor  device  interface  status    driver      usable
01:00.0       0x8086  0x10fb  p1p1       down      apps.intel.intel_app yes
01:00.1       0x8086  0x10fb  p1p2       down      apps.intel.intel_app yes
02:00.0       0x8086  0x10fb  p2p1       down      apps.intel.intel_app yes
02:00.1       0x8086  0x10fb  p2p2       down      apps.intel.intel_app yes
03:00.0       0x8086  0x10fb  p3p1       down      apps.intel.intel_app yes
03:00.1       0x8086  0x10fb  -          -         apps.intel.intel_app yes
81:00.0       0x8086  0x10fb  p5p1       down      apps.intel.intel_app yes
81:00.1       0x8086  0x10fb  p5p2       down      apps.intel.intel_app yes
82:00.0       0x8086  0x10fb  p6p1       down      apps.intel.intel_app yes
82:00.1       0x8086  0x10fb  p6p2       down      apps.intel.intel_app yes

and software now accepts either format in parameters:
$ sudo SNABB_PCI0=01:00.0 SNABB_PCI1=0000:01:00.1 ./snabb snsh -t apps.intel.intel_app
selftest: intel_app
100 VF initializations:

Running iterated VMDq test...
test #  1: VMDq VLAN=101; 100ms burst. packet sent: 628,830
test #  2: VMDq VLAN=102; 100ms burst. packet sent: 1,745,985
test #  3: VMDq VLAN=103; 100ms burst. packet sent: 2,891,700

This change can potentially break software that depends on always having qualified paths: that would now be considered a bug. (This PR includes a fix for such a problem in packetblaster.)

Seems reasonable.


CHANGE: remove bintray from docs.

@qianzhangxa Like we talked, here my PR to remove bintray from the docs.

@andreaspeters Thanks for your contribution!


Support for Timers

Is this already supported? I was not able to find it.
http://docs.aws.amazon.com/amazonswf/latest/developerguide/swf-dg-timers.html

Hey Kumar,
Timers are supported, but weren't documented. I've updated the docs: https://botoflow.readthedocs.io/en/latest/reference/workflow_time.html
Let me know if it makes sense or not.

@darjus This is great. I will try it out and if any problem will raise an issue. Lets close this one 👍
BTW, great to run into you again, I think we worked around the same time at Amazon.


Resolves #10

Let's pair on this first thing on wednesday. Will like to work with the entire team on this.


protobuf was added to lochap2pd

Could we move the protobuf serialization to the src/ dir (locha-p2p crate)? So that way we share code between the app and the command line program, e.g.: we modify the send_message logic on the Runtime to accept the parameters we need in a message.