Save Draft page action is not overridable

Using the construct_page_action_menu hook, we can override the actions that appear in the drop-up box at the bottom of the page editor. However, this doesn't seem to include the "Save Draft" action that appears in the default position.
I think it would be nice if "Save draft" was also configurable using this hook and it was possible to choose a different action as the default one.
Implementation
We could add a default flag to the ActionMenuItem class so you could specify which is the default?
For example, to make publish the default action:
@hooks.register('construct_page_action_menu')
def make_publish_default_action(menu_items, request, context):
    menu_items[:] = [
            UnpublishMenuItem(order=10),
            DeleteMenuItem(order=20),
            SubmitForModerationMenuItem(order=30),
            SaveDraftMenuItem(order=40),
            PublishMenuItem(default=True),
    ]

If more than one item has default=True an error would be raised. If no items have default=True then we should automatically add a SaveDraftMenuItem as this is what existing implementations would expect.
Alternatively, we could pick which ever item is first (or last) by the order field and treat that as the default action.

I will give it a try.

Thanks @rjpruitt16!

I finished the feature. I am adding test and updating documentation. Should submit pull request by friday.

https://github.com/wagtail/wagtail/pull/5500


Update PlaylistSoundConfig to Foundry VTT 0.7.9

Which class should be updated?
PlaylistSoundConfig
For which version should the class be updated?
Foundry VTT 0.7.9

Taking this.


Vertical Pod Autoscaler Addon

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

if AMP is enabled then set values are used to apply the service account name and service annotations.


fix(Designer): Keyboard Focus Moves Out of TokenPicker Navigating Using Keyboard

Please check if the PR fulfills these requirements

[X] The commit message follows our guidelines

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
fixes #4042

What is the current behavior? (You can also link to an open issue here)
in tokenpicker, moving by keyboard takes the focus away to somewhere irrelevant

What is the new behavior (if this is a feature change)?
in tokenpicker, moving by keyboard stays within tokenpicker unless esc is pressed

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Please Include Screenshots or Videos of the intended change:

https://github.com/Azure/LogicAppsUX/assets/144840522/12390b19-7ff5-4794-924c-1e88a9f9ab4a

this causes a weird little bug where if you have a token in the editor, and press tab you can get multiple tokenpicker buttons:
https://github.com/Azure/LogicAppsUX/assets/95886809/f0177c36-14e3-419e-b44b-fd63bba35d22

this causes a weird little bug where if you have a token in the editor (html editor doesn't need it due to the toolbar), and press tab you can get multiple tokenpicker buttons:
Screen.Recording.2024-02-09.at.11.50.51.AM.mov

I checked out the main branch and am still seeing the weird little bug. Would you be able to double check if the bug persists on main to confirm my change is causing the issue?
https://github.com/Azure/LogicAppsUX/assets/144840522/a6eae1aa-62bd-442a-8c24-616502a56081

this causes a weird little bug where if you have a token in the editor (html editor doesn't need it due to the toolbar), and press tab you can get multiple tokenpicker buttons:
Screen.Recording.2024-02-09.at.11.50.51.AM.mov

I checked out the main branch and am still seeing the weird little bug. Would you be able to double check if the bug persists on main to confirm my change is causing the issue?
tokenPickerFocus.bug-1.mp4
is this from main? because it doesn't seem to be happening here - in my other branch i currently have checked out from main this issue isn't happening

this causes a weird little bug where if you have a token in the editor (html editor doesn't need it due to the toolbar), and press tab you can get multiple tokenpicker buttons:
Screen.Recording.2024-02-09.at.11.50.51.AM.mov

I checked out the main branch and am still seeing the weird little bug. Would you be able to double check if the bug persists on main to confirm my change is causing the issue?
tokenPickerFocus.bug-1.mp4
is this from main? because it doesn't seem to be happening here - in my other branch i currently have checked out from main this issue isn't happening

Thanks for confirming, must be my local ends problem. I'll clear cache and try again!

this causes a weird little bug where if you have a token in the editor (html editor doesn't need it due to the toolbar), and press tab you can get multiple tokenpicker buttons:
Screen.Recording.2024-02-09.at.11.50.51.AM.mov

I checked out the main branch and am still seeing the weird little bug. Would you be able to double check if the bug persists on main to confirm my change is causing the issue?
tokenPickerFocus.bug-1.mp4
is this from main? because it doesn't seem to be happening here - in my other branch i currently have checked out from main this issue isn't happening

I have addressed the issue and fixed it.
https://github.com/Azure/LogicAppsUX/assets/144840522/e56c6755-f830-45b1-a76b-024656a6d8b9


Several  statements can be used

Quoting SLAPD.ACCESS(5):
The dn, filter, and attrs statements are additive [..]
Example: to dn.subtree="dc=ldap1,dc=example,dc=org" attrs=userPassword,shadowLastChange by dn.exact="cn=replicator,dc=ldap1,dc=example,dc=org"

@sathieu we won't merge this.


Correlate source of trap

I am trying to log (Source_IP, OID, Value) tuples for each varBind in a trap message received, but the documentation does not have very much information on traps, or I am not finding the relevant section.  The only example I could find was in the Advanced Topics section.
I am registering a trap receiver with ntfrcv.NotificationReceiver(snmpEngine, cbFun) but in the callback function, def cbFun(snmpEngine, stateReference, contextEngineId, contextName, varBinds, cbCtx): I can not see how to identify the source address of the sender, only a contextEngineId.  Is there any way I can correlate the EngineID to an IP address of the source?
The other method I tried was to use the registerObserver() method, which has the transportAddress variable but that does not let me easily process the PDU, or I don't understand how to, because it is a complex data structure.
I can't see anywhere in the documentation which explain these methods, or how to use them, despite them being used in several examples.
Can anyone please point me in the direction of some information on the API functions and how to unpack the data from the data model.
Many Thanks

In the standard, SNMP processing model does not reveal peer addresses to the application on top od SNMP stack. That's probably purposefully hidden. Therefore pysnmp has a workaround for pulling such details from the internals of the SNMP engine. Yes, your take on observer is correct.
If you take this example callback, the variables parameter is a dict having transportAddress key holding both local and remote address information. You also have to register your callback to pysnmp.
Now, the callback is guaranteed to be called right before you receive the TRAP. You can have a shared data structure between your callback and your TRAP receiving function so that the former will store the address for the latter to retrieve it.
Does it make sense?

Thanks,  I think that makes sense.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1803

Questions ?
Please refer to the Backport tool documentation


Fixes #26: remove todo dir when empty

Addresses #26 writeTodos() should ensure .lint-todo dir does not exist if no errors are present #26

Closing since .lint-todo dir should actually be expected once authors have opted in to use todos.


Fix typo

I noticed there was a typo in an error message when updating the gem.
This PR fixed it. Please review when it's convenient for you.

😳


VS2017 hangs on new R project

Please briefly describe what you were doing that led to the issue, if applicable:
I recently added the data science workload to try out R tools in VS 2017. When I try to create a new R project, the application just hangs. I tried doing a repair install and making sure I had all the latest updates. These things did not help. This is happening on two of my machines, so it's not a local VS issue.
...
If you have any screenshots demonstrating the issue, please include them as well to help us diagnose it better.

Additional information:
OS Information
Version:       Microsoft Windows NT 10.0.15063.0
RTVS Information:
Assembly: Microsoft.VisualStudio.R.Package, Version=1.1.30413.947, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a

Hi. Sorry about that. Can you do a send frown from R Rools menu and attach
the logs? Thx
On Sun, May 14, 2017 at 8:16 PM Dustin Ewers<EMAIL_ADDRESS>wrote:

Please briefly describe what you were doing that led to the issue, if
applicable:
I recently added the data science workload to try out R tools in VS 2017.
When I try to create a new R project, the application just hangs. I tried
doing a repair install and making sure I had all the latest updates. These
things did not help. This is happening on two of my machines, so it's not a
local VS issue.
...
If you have any screenshots demonstrating the issue, please include them
as well to help us diagnose it better.
[image: capture]
https://cloud.githubusercontent.com/assets/466287/26041498/fbee83e6-38f2-11e7-9417-d055631e69d7.PNG
Additional information:
OS Information
Version: Microsoft Windows NT 10.0.15063.0
RTVS Information:
Assembly: Microsoft.VisualStudio.R.Package, Version=1.1.30413.947,
Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/RTVS/issues/3525, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtUVLEIBs8C4pRgWkd6EYAVc7y32huqks5r58OWgaJpZM4Nan6t
.

Which version for R are you using? Does File | New | R Script work? If it is R 3.4.0, it is not yet supported in VS 2017 15.2. (support comes in 15.3).
BTW, R Tools do not require projects you can just work with files if you want. R Project is not a typical VS project, it just a display of what is in the project folder.

File -> New R Script gives me a script, but it doesn't look like there's any way to open it. Additionally it will not bring up the R interactive Window.
I have three versions of R on my computer.
From the logs:
Installed R Engines (from registry):
Microsoft R Client (<IP_ADDRESS>) <IP_ADDRESS> C:\Program Files\Microsoft\R Client\R_SERVER
Microsoft R Open (<IP_ADDRESS>) <IP_ADDRESS> C:\Program Files\Microsoft\R Open
R 3.4.0 3.4.0 C:\Program Files\R\R-3.4.0

That being said, I figured it out. Hat tip to @MikhailArkhipov for the version reminder. Once I uninstalled R 3.4.0, it worked again.
Ideally, there would be some better error handling around that one 'cause a) that's a nasty error, and b) lots of people who are using RStudio and 3.4.0 are going to be trying this same thing.
Thanks a bundle for the quick response. Have a good night.

Yep we should have a better error. 3.4 came out just after VS lock down.
The new update has a fix. Out in ~ 2 weeks.
On Sun, May 14, 2017 at 8:59 PM Dustin Ewers<EMAIL_ADDRESS>wrote:

File -> New R Script gives me a script, but it doesn't look like there's
any way to open it. Additionally it will not bring up the R interactive
Window.
I have three versions of R on my computer.
From the logs:
Installed R Engines (from registry):
Microsoft R Client (<IP_ADDRESS>) <IP_ADDRESS> C:\Program Files\Microsoft\R
Client\R_SERVER
Microsoft R Open (<IP_ADDRESS>) <IP_ADDRESS> C:\Program Files\Microsoft\R Open
R 3.4.0 3.4.0 C:\Program Files\R\R-3.4.0
That being said, I figured it out. Hat tip to @MikhailArkhipov
https://github.com/mikhailarkhipov for the version reminder. Once I
uninstalled R 3.4.0, it worked again.
Ideally, there would be some better error handling around that one 'cause
a) that's a nasty error, and b) lots of people who are using RStudio and
3.4.0 are going to be trying this same thing.
Thanks a bundle for the quick response. Have a good night.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/RTVS/issues/3525#issuecomment-301371061,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtUVO6Aj43AwyDi9lYOcz_EgQ6XlPc_ks5r582TgaJpZM4Nan6t
.

You don't have to uninstall 3.4. Start VS, open   R Tools | Windows | Workspaces. Select R 3.3.3 or below. After you made the selection, RTVS will remember and use it. It only uses latest when no selection was ever made.
Unforrtunately R 3.4 changed C++ API so there is no good way to know if it was compatible. 15.2 was already out when R 3.4 got published.

Alternatively, there is 15.3 available in preview channel https://www.visualstudio.com/vs/preview/. It installs side by side and does not affect main VS install. I, in fact, have 3 different ones :-)


fix: update runtime's polynomial fn to adjust formula of WeightToFeeCoef

Description of proposed changes
update runtime's polynomial fn to adjust formula of  WeightToFeeCoef to make tx fees on cc3 the same as it is on cc2.
with a new updated formula the fees on cc3 and cc2 are:
transfers tx: 9.8mili vs 11.35mili
staking payout_stakers: 10.27mili vs 11.38mili

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

The check version job is failing because if we modify the runtime we also need to bump the version, this release should be v3.36.0. See the job here https://github.com/gluwa/creditcoin3/actions/runs/11730824518/job/32679537016?pr=521
I'm not sure why the danger-will-brick-the-blockchain job is failing https://github.com/gluwa/creditcoin3/actions/runs/11730824518/job/32679537942?pr=521 because I don't see any changes in epoch duration in this pr, maybe @atodorov can shed some light on this?
Other 3 CI jobs are known to fail, so I think its okay. I'm testing the changes locally and will let you know how it all works out

I'm not sure why the danger-will-brick-the-blockchain job is failing

b/c it detects MILLISECS_PER_BLOCK as part of the git diff context.  This is a false positive in this particular case and can be ignored.

Merging, see https://gluwa.slack.com/archives/C03MQ532BGA/p1731078810606819


Ruby Capstone Project

Catalog of my things 📚 🎶 🎮 🎥
members of this project:

Javier Aybar GitHub
Bruno Kambere GitHub
Karim Barragan GitHub
Diariatou Diakite GitHub

In this project we did:
✅  Create a console app that will help you to keep a record of different types of things you own:( books, music albums, movies, and games)
✅ Everything will be based on the UML class diagram
✅ The data stored in JSON files
You can see the project requiremets in the following links:
https://github.com/microverseinc/curriculum-ruby/blob/main/group-capstone/catalog_of_my_things.md https://github.com/microverseinc/curriculum-ruby/blob/main/group-capstone/catalog_of_my_things_divided.md

Hello @Owusu-Desmond!
Thanks for reviewing our project! but we believe that our implementation has no problems, if you check the same link that you give us, you can see the following:

We are following that approach, and as it says there, it is a completely basic good practice.
If we follow the approach you give us, we will have a linters error related to cyclomatic complexity because we have too many cases.

✅ The main.rb present the user with a list of options to perform
✅ Let users choose an option, If needed, ask for parameters for the option, and have a way to quit the app.
✅ The main.rb IS the entry point of our application.


[onprem] HDFS Interface

Continued from previous PR

@HaileyJang how you will obtain parallelism when reading data from HDFS?

@HaileyJang how you will obtain parallelism when reading data from HDFS?

Hey @gilv for now, the parallelism is restricted to reading multiple files concurrently. Obviously, this isn't very smart as we are bottlenecked by the name node. Once we have an end 2 end system and a good understanding of how this is impacting the performance we plan to revisit it. If you have thoughts, happy to hear!

@HaileyJang HDFS can perfectly deal with many reads and name node is never an issue, as all cached in memory. I would suggest you to extend hadoop-distcp. This way you will get perfect parallelism over HDFS cluster and then use your techniques to efficiently copy data


is it ok to use gRPC interceptors in go-micro

as the title

Not sure about v4, never tried, you could try I guess, but it might be difficult since the server is tucked away a bit. v5 will support it out of the box

Btw feel free to join the discord for any questions you have

I will try discord

Failed to try discord, I was not able to register it with a resource limited prompt, with twice tries.
Really prefer to use go-micro framework in my new project, but the last problem in my way is interceptors, which I want to use to secure every gRPC call with my user authentication service. Any other solution? or any advices? Thanks very much.
Btw, when will v5 come out? I'm really looking forward to it.

@leoujz You did you join the discord server twice, strange that you can't use it.
You can secure your services by requiring auth at the proxy level. I like to use Traefik's forward auth functionality for that, but I'm sure other reverse proxies also have auth build in.
If interceptors don't work with v4, you could also manually add the auth logic in your proto handlers temporarily.
v5 should be ready for testing in a month, or two. I'll be sure to let you know

that's a solution. I'd rather wonder that service mesh already has service discovery and registration, load balancing, what's the main difference or motive of implemention in go-micro framework?


fix(cloudcommon): supplement i18n translations

这个 PR 实现什么功能/修复什么问题:
修正：补充共享的错误消息i18n翻译

是否需要 backport 到之前的 release 分支:

release/3.6

/cc @yousong
/area util

Codecov Report

Merging #9295 (fbdcbeb) into master (516f5ab) will increase coverage by 10.99%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #9295       +/-   ##
===========================================
+ Coverage    5.85%   16.85%   +10.99%
===========================================
  Files         835       85      -750
  Lines      129737     8900   -120837
===========================================
- Hits         7597     1500     -6097
+ Misses     121453     7268   -114185
+ Partials      687      132      -555

Flag
Coverage Δ

aFlag
16.85% <ø> (+10.99%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/compute/models/cloudproviders.go

pkg/util/stringutils2/ignorecase.go

pkg/compute/models/natstable.go

pkg/compute/models/loadbalancerbackendgroups.go

pkg/compute/models/dbinstancejoints.go

pkg/cloudcommon/db/virtualjointbase.go

pkg/compute/regiondrivers/aws.go

pkg/multicloud/esxi/devtools.go

pkg/keystone/tokens/authmethods.go

pkg/compute/models/inter_vpc_network_routeset.go

... and 739 more

Continue to review full report at Codecov.

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


Make pg_dump compatible with 13.3 db server

Change Overview
This PR updates the postgres-kanister-tools base image to postgres:13.3-alpine
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[x] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[ ] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test

Issues

#XXX

Test Plan

[x] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

./build/integration-test.sh RDSPostgreSQLDump
https://gist.github.com/A-kanksh-a/d7d75a6718a15c63a265713006a483c0

@A-kanksh-a could you please also verify if it doesn't break stable/postgress app?

Verified :
./build/integration-test.sh PostgreSQL

Running integration tests:
~/KASTEN/kanister/pkg/testing ~/KASTEN/kanister
=== RUN   Test
OK: 1 passed
--- PASS: Test (61.77s)
PASS
ok  	github.com/kanisterio/kanister/pkg/testing	61.806s
~/KASTEN/kanister


heterogeneous trace context extraction

These changes remedy the following scenario:
A tracer is configured to inject in the W3C (tracecontext, i.e. OpenTelemetry) style, perhaps among other styles. The same tracer is configured to extract first in one or more non-W3C styles (DATADOG and/or B3), and then in the W3C style.
When an incoming request contains the desired non-W3C headers, such as X-Datadog-Trace-Id, then trace context is extracted from them, and any W3C headers in the request are ignored.
This is problematic when the tracestate W3C header contains non-Datadog information. We have ways of putting Datadog information into tracestate, but there is not a way to put arbitrary tracestate into Datadog headers.
So, this pull request adds the following logic: If we extract due to non-W3C headers, but we are configured also to extract W3C headers, then look for the tracestate header and keep some of its data if the trace ID from the corresponding traceparent header matches the trace ID extracted from the non-W3C header.
See the included unit test for clarification.
Note to the reviewer: There are some unnecessary changes included in this pull request, per discussions we've had previously about where the extraction code should live. I didn't go all-in on the reorganization, but extraction_util is a start. See the individual commits if you want to subtract that noise.

I'll run these against Zach's parametric system tests tomorrow.

When I run the parametric system tests locally, I get 97 errors of the following type:
----------------------------------------------------------------------------------- Captured stdout setup ------------------------------------------------------------------------------------
[]
____________________________________________ ERROR at setup of Test_Span_Sampling.test_multi_rule_independent_rate_limiters_sss013[library_env0] _____________________________________________
[gw11] linux -- Python 3.10.12 /home/david/src/system-tests/venv/bin/python
Could not create docker network 'apm_shared_tests_network_6e867f', see the log file '/tmp/tmpzactx35z'

The referenced log file (/tmp/tmpzactx35z, in this case) never exists.
I'm going to proceed with a release of this code.


Consider partitioning tests for behavior, bug fixes, exceptions, etc

The larger test modules are a mess of different tests. Partitioning may make them easier to sort through.

I'm not sure if you've seen TestNG before, but I really like how it handles tests cases.
For example, this is what one of my test cases looks like for a section of my excel reader that parses the values in the header for column names:
public class ColumnReadHelperTest {
    /**
     * @return data for testing
     */
    @DataProvider
    private static String[][] sanitizeColumnNameData() {
        return new String[][] {
            {
                null, ""
            }, {
                "", ""
            }, {
                "BOB", "bob"
            }, {
                "BoB", "bob"
            }, {
                "   BOB   ", "bob"
            }, {
                "   B_O B   ", "b o b"
            }, {
                "   BOB__   ", "bob"
            }, {
                "   BO  B   ", "bo b"
            }, {
                "   BO   B   ", "bo b"
            }, {
                "   BO ___ ---   B   ", "bo b"
            }, {
                "   B--___OB_   ", "b ob"
            }, {
                "      ", ""
            }, {
                "_______________", ""
            }, {
                "---------------", ""
            }, {
                "_ -_ -_ -_ -_ -_ -_ -_ ", ""
            }, {
                "123", "123"
            }, {
                "AbCd123", "abcd123"
            }, {
                " - _ AbCd123 _ - ", "abcd123"
            }
        };
    }

    /**
     * @param s
     *            string to convert
     * @param expected
     *            expected result
     */
    @Test(dataProvider = "sanitizeColumnNameData",
            timeOut = 1)
    public void sanitizeColumnName(final String s, final String expected) {
        final String actual = ColumnReadHelper.sanitizeColumnName(s);
        assert actual.equals(expected) : "Expected : [" + expected + "] but got: [" + actual + "]";
    }

    /**
     * Initializes the class we are using to ensure all static fields are constructed before calling
     * the tests. This will help ensure timing of the tests is not effected by "startup costs" for
     * the class.
     */
    @BeforeClass
    private void init() {
        ColumnReadHelper.sanitizeColumnName("a");
    }
}

TestNG converts the dataProvider into the parameters that the actual test uses. Each set of parameters is run as a single test so it will run every set of data even if the first one fails. I find that it makes finding and fixing bugs a lot easier. i.e in my test above if the first parameter set { null, "" } errored (NPE, or the expected value didn't match) then TestNG would just flag that single data item as a fail and move onto the next one: { "", "" }
It supports all the JUnit Assert functions and it has it's own versions of most of them too (I just dislike using them, for some reason I like using actual asserts directly... mostly because the assert keyword pops up in my syntax highlighter)
It also allows you to assign tests to groups ("Regression", "Bug", etc). My test annotation could have looked like this for example:
@Test(dataProvider = "sanitizeColumnNameData",
            timeOut = 1, groups = {"group1", "Another Group"})

TestNG does have some performance test options (like that timeout one I have in my test) but it's not really made for formal performance testing.
Anyway just my 2cents.

Oh, here's an example of what that test run looks like when run in Eclipse and has no failures.

Thanks, have never used testng, will look into it.

Not prioritized


RFC - display progress during link recovery

Scope & Purpose
Displays progress report during arangosearch link recovery, e.g.
...
2022-09-05T18:02:00Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 4792073 (stage1, operation 1/1 (0%)
2022-09-05T18:03:00Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 4792073 (stage3, operation 67/209 (31%)
2022-09-05T18:04:01Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 4792073 (stage3, operation 136/209 (64%)
2022-09-05T18:05:01Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 4792073 (stage3, operation 208/209 (99%)
2022-09-05T18:05:03Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 5507397 (stage1, operation 1/1 (0%)
2022-09-05T18:06:04Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 5507397 (stage3, operation 68/209 (32%)
2022-09-05T18:07:05Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 5507397 (stage3, operation 137/209 (65%)
2022-09-05T18:08:05Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 5507397 (stage3, operation 209/209 (99%)
2022-09-05T18:08:07Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 5507407 (stage1, operation 1/1 (0%)
2022-09-05T18:09:07Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 5507407 (stage3, operation 63/210 (29%)
2022-09-05T18:10:08Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 5507407 (stage3, operation 123/210 (58%)
2022-09-05T18:11:08Z [65850] INFO [d1f18] {arangosearch} recovering arangosearch link 5507407 (stage3, operation 192/210 (90%)

The output format may need some tweaking to become easier to understand. Especially the labels "state1" to "stage4" are not really comprehensible without knowledge of the internals.

[ ] :hankey: Bugfix
[x] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.10: (Please link PR)
[ ] Backport for 3.9: (Please link PR)
[ ] Backport for 3.8: (Please link PR)

Related Information

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket:
[ ] Design document:

operation 1/1 (0%)

Shouldn't that be 100%? Or is op 1 ongoing?
0/1 (0%) would be more intuitive in that case.

operation 1/1 (0%)

Shouldn't that be 100%? Or is op 1 ongoing? 0/1 (0%) would be more intuitive in that case.

It is operation 1 of 1 ongoing.

👍
It's a bit misleading that the percentage shows the degree of completion but the op fraction does not


Integrate llama as a LLM backend for API-less inference of a search results

Basically the whole idea in the title.
Currently the project heavily relies on OpenAI completion API.
It would be great to integrate with something like:
llama.cpp
llama-rs
pyllama
This way the inference could be done locally without a reliance on 3rd party services

Indeed the first two need to have python wrapper around them, but the third one could be integrated pretty easily.
I believe dependency injection technique would be quite suitable here.

After an investigation and creating a small wrapper around llama.cpp it turns out that I can't get it to response with a NEXT COMMAND normally. I guess training of a model is needed, or more examples within a prompt.
Link to reproduce test env:
https://gist.github.com/nenkoru/ebf99934fbd7935735901d5ab6cf5da1

Thank you for the tremendous work. My guess is that indeed out of the box LLaMA will perform poorly (especially the smaller models). More promising direction is to use the alpaca (instruction-tuned llama):

https://github.com/tatsu-lab/stanford_alpaca

I will investigate this


to do list

[x] Turn on GitHub Pages
[x] Outline my portfolio
[ ] Introduce myself to the world

completed


TypicalPersons: Make all methods and fields static #569

Fixes #569
TypicalPersons is a utility class that creates preset Persons for tests
to use. Some methods and fields are non-static.

This makes it troublesome for test classes to access these preset
Persons as they have to create a TypicalPersons object prior to
accessing the Person objects.

Let's make all methods and fields in TypicalPersons to be static.

v1
@Zhiyuan-Amos submitted v1 for review.

[v1 1/2] TypicalPersons: Make all methods and fields static
[v1 2/2] TypicalPersons: Remove getTypicalPersons()

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/574/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v2
@Zhiyuan-Amos submitted v2 for review.

[v2 1/3] TypicalPersons: Make all methods and fields static
[v2 2/3] TypicalPersons: Remove getTypicalPersons()
[v2 3/3] TypicalPersons: Add private constructor

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v1 and v2)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/574/2/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

Good day dear sir @yamgent,
Thanks very much for your kind and prompt review. I have made the amazing changes that you have requested, and I sincerely hope that you find my PR satisfactory and pleasing to your eyes.
Thank you once again dear sir.


[End Game Testing] Run Cells Above / Below should be disabled when run by line debugging

Testing #129467
When doing run by line debugging if I click the buttons to run a cell above they get stuck on pending and stopping the pending cells will error out the debug session and cause it to stop.

Seems to have caused my debug session to be broken as well

they get stuck on pending and stopping the pending cells will error out the debug session and cause it to stop.

That's expected, You can only run one cell at a time. And there is no per-cell cancellation, you are just interrupting the kernel which stops everything.

Info 2021-08-24 13:15:38: Attempting search for colors ...
Info 2021-08-24 13:15:38: Loading colors from /snap/code-insiders/796/usr/share/code-insiders/resources/app/extensions/theme-defaults/themes/dark_plus.json ...

Info 2021-08-24 13:15:38: Loading base colors from /snap/code-insiders/796/usr/share/code-insiders/resources/app/extensions/theme-defaults/themes/dark_plus.json ...

Info 2021-08-24 13:57:17: Attempting search for colors ...
Info 2021-08-24 13:57:17: Loading colors from /snap/code-insiders/796/usr/share/code-insiders/resources/app/extensions/theme-defaults/themes/dark_plus.json ...
Info 2021-08-24 13:57:17: Kernel switching to busy
Info 2021-08-24 13:57:17: Notebook Session status file:///home/parallels/jupyter-dbg/ipykernel-env/test.ipynb # Busy
Silly 2021-08-24 13:57:17: [Debug] event: {"header":{"msg_id":"53937680-c73089e921546f08818bc6fe_668","msg_type":"status","username":"parallels","session":"53937680-c73089e921546f08818bc6fe","date":"2021-08-24T17:57:17.289147Z","version":"5.3"},"msg_id":"53937680-c73089e921546f08818bc6fe_668","msg_type":"status","parent_header":{"date":"2021-08-24T17:57:17.286000Z","msg_id":"96481e1a-3f1e-4e54-8de0-4bce1d544511","msg_type":"execute_request","session":"9eb98a94-4b88-43c6-99f6-655c1014cfda","username":"","version":"5.2"},"metadata":{},"content":{"execution_state":"busy"},"buffers":[],"channel":"iopub"}
Info 2021-08-24 13:57:17: Attempting search for colors ...
Info 2021-08-24 13:57:17: Loading base colors from /snap/code-insiders/796/usr/share/code-insiders/resources/app/extensions/theme-defaults/themes/dark_plus.json ...

 Looks like we correctly set the breakpoint, then execute the cell, but the kernel doesn't stop. I can't repro it. Is it easy for you to hit this @lramos15?

Yes it is, let me know if you want me to demo it over teams


Can You Bring Discussions back,APRIL FOOLS ENDED!

What feature do you want to get added on the base engine?
doing easier lua stages
and about the tittle
guys,it ended,give it back
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

heres the thing: its probably not coming back.
give some sort of proof other than the fact that's its april fools

can ShadowMario add a trusted and good moderator

The answer to why it's not coming back it's because ShadowMario already said that discussions won't come back because of how hard it is to moderate.
Stop asking about this and answering with false answers, this is what they need to learn.

https://github.com/mayo78/PSYCHDISCUSSIONS/discussions/1


Feature/clear outbreak history home screen

Summary | Résumé
The copy for the ClearExposureOutbreakView view was not in the JSON file, and was outdated.
Updated copy for EN and FR copied to JSON.
Test instructions | Instructions pour tester la modification
Scan a QR code to trigger an Outbreak Exposure. Go to Home Screen, and click the "Negative Test Result" button to display this view.  Verify that all copy is accurate to the Figma file. Repeat for both EN and FR.

Looks good - just need to fix the conflict.


LuxCore v2.2: Brightness difference between Path OpenCL and Path CPU

Reported by Dez!.
How to reproduce:

Open official Blender 2.80 with BlendLuxCore v2.2 installed and open this file:
https://drive.google.com/file/d/1Tv7HFhW6papNMIVBGtqJtxLJ6UzlWwgN/view?usp=sharing
Press F12, the OpenCL version will render
After the render is done, switch to slot 2 in the image editor on the right
change device to "CPU"
Render again

You should obtain these results:
OpenCL:

CPU:

I have not tried to debug this yet, it might be an addon issue, in which case we can move this issue to the BlendLuxCore repo.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

I should have fixed this problem.

Do you think it would make sense to create a v2.2.1 bugfix release for fixes like this one, or will we release v2.3 so soon that we shouldn't bother?

I'm not sure if we have a procedure for Azure to release v2.2.1, I think not. The bug is in since 16 July and it is triggered by mixing triangle lights with other types of light sources (otherwise it works fine).
I would prefer to  just aim to release the v2.3 before the end of the year.

Ok. I have confirmed that it's fixed, so I'm closing this. Thanks!

How do I get these fixes for version 2.2?

I'm not sure if we have a procedure for Azure to release v2.2.1, I think not.

I can check what we need to set up a v2.2.1 release.
Having the artifacts should be easy, it might be that we have then to set up the release in GitHub manually.


Support for sequelize v4

The library doesn't appear to be working properly with the latest version of sequelize. Installing v4.1.1 and running the tests results in the following failures: https://gist.github.com/cbargren/1191a5f64a9aa7682800c1f185c18c8b.
Sequelize v4 breaking changes: http://docs.sequelizejs.com/manual/tutorial/upgrade-to-v4.html

Hi @cbargren
Indeed, this tools was not design for the new sequelize major version
I will try for the next weeks to make some evolutions
Regards

Awesome, thanks!


Parse modules.txt of go modules to create dependency snapshot

Glide and godeps are deprecated. Using go modules is the current way to go for managing dependencies.
Right now, modules.txt looks like a file that contains the right information to get all the dependencies and their corresponding revisions/commits/SHAs/....
Idea:

read https://github.com/gofed/symbols-extractor/blob/master/vendor/modules.txt
translate versions/tags into SHAs
translate v0.0.0-20160126235308-23def4e6c14b git describe bits into SHAs

Open to better way how to get SHAs of all dependencies.

In python for the moment: https://github.com/gofed/symbols-extractor/commit/d69f1150a65439f79e5eaf0b5dbdc644d978ef81


enable deepcopy for DictWrapper

i run into this because i want to dump the response object into a yaml file.
products_api = mws.Products()
products = products_api.get_x()
mydict = vars(products)  # same as products.__dict__
type(mydict['_response_dict'])

this returns
luigipipe.mws.mws.utils.ObjectDict
i think it should return: dict
A very simple approach for problems of that kind i found here.
If you think that is a good idea i can try to understand and fix this.

I haven't ignored this one, just been tossing it around in my head for a while.
I see no real issue offering that functionality, since it's unlikely to get in the way of other methods.
Note that in your pseudocode, products is an instance of DictWrapper, and that parsed attribute is a shorthand to response_dict.
Thus, products.parsed will result in an instance of ObjectDict, which is similar to the dict-like object in the example you've linked. That's the part where we'd add a __deepcopy__ to enable this function.

I have a bit of bandwidth this weekend, so I'll play around with this stuff, see what we can work out. :)

I try to understand how manipulated dictionaries or dict like classes from  scratch work.
It seem those problems are the price you pay for your class like dicts.
What i find striking is that everyone implements this different.
I am trying a lot of them out.
Opinions are: don't inherit from dict,that would break things like get(item , default) will return a dict and not a DotDict, it somehow isn't using your getattr . Those people are using the collections module.
What's interesting with our ObjectDict it does not the heavy lifting of recursion. It takes as input only none nested dictionaries.
That have to change if we want to pass a nested dictionary once (like that we get from xmltodict).

Like argued in the comment section of that link, the copy should work without a dunder deecopy method.
In my new version it does. Tough it's a bit inconvenient since vars returns the dict nested:
{'_DotDict__data': {'EndDate': '2018-05-29T04:24:10+00:00',
'ReportProcessingStatus': 'SUBMITTED',
'ReportRequestId': '548585017',
'ReportType': 'GET_FLAT_FILE_OPEN_LISTINGS_DATA',
'Scheduled': 'false',
'StartDate': '2018-05-29T04:24:10+00:00',
'SubmittedDate': '2018-05-29T04:24:10+00:00'}}
But all nested Dicts are Dicts and not DotDict objects.


gitea: 1.9.5 -> 1.9.6

Motivation for this change
Gitea Bugfix release: https://blog.gitea.io/2019/11/gitea-1.9.6-is-released/
Since 1.10 was also released, maybe we should drop this pr in favour of 1.10? See #73416 for the 1.10.0 version.
Things done

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

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

[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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg test gitea
I'm thinking that maybe we want 1.9.5 -> 1.9.6 and 1.9.6 -> 1.10.0? We need to decide if we want to backport 1.10.0 to stable or if we want to stick it to the 1.9.X series. If we have the latter we should probably get 1.9.6 in master to be able to cherry-pick that commit.

IMHO we should put 1.10. in stable, since 1.9.X won't get much support now because 1.10. is out. We simply don't have the resources to provide an LTS version, we're only a community open source project.


wrong link 404

The link under LocalDateTime For details on the pattern syntax read here should be https://docs.oracle.com/javase/8/docs/api/java/time/format/DateTimeFormatter.html

Fixed. Thanks! :+1:


Add missing os_disk property for vm_linux

Linux VM resource is missing the diff_disk_settings property of the os_disk setting.
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/linux_virtual_machine#diff_disk_settings

OK for Azurerm 2.88.1

thanks for your contribution to Cloud Adoption Framework @a-ferri


Streams menu breaks at certain viewport widths

At certain viewport widths (as wide as 869px), the last stream cannot be scrolled to. This is related to the shifting stream buttons, which should be presented with a modern layout technique that prevents buttons from shifting onto a second line:

CZO thread

@zulipbot claim

Hi, I was curious about the status of this issue. Is there anything I can do to help?

@zulipbot claim

@asub10 It's a solved problem from my side just waiting for @karlstolley to review.

@zulipbot abandoned


Improving section + content browsing and navigation

What is it and who's it for?
All site users
Due to the long-running iterative development of the site, we've ended up with a lot of variety in our page templates, making it much harder to see a consistent site hierarchy and build out pages that contain a lot of content.
Given that we're now expanding the amount of content on the site (through shutting down wellcomelibrary.org and recreating all of its content based on user needs), we need to take a holistic view of the site to make improvements across the whole design system, as it relates to page templates and content hierarchy.
This project's aim is to improve the page-by-page, section-by-section browsing experience on the site.
Project Brief: https://www.notion.so/wellcometrust/Project-Brief-Improving-section-content-browsing-and-navigation-8d11582ef0e14a95a1a617a394f919ea
Initiatives

[ ]  Section-level page headers with custom functionality (#5432)
[x] Harmonizing section, list and content page (with/without featured image) headers (#5411)
[ ] Headings AA compliance #4607
[x]  Headings hierarchy #4860
[x] Anchor links on content pages https://github.com/wellcomecollection/wellcomecollection.org/issues/5561

Measurement

Increase on-site page views

Other considerations
<---Delete as appropriate--->

Impact on URL design
Cardigan documentation

Relates to https://github.com/wellcomecollection/platform/issues/4646

21 July 2020:

[x] List/content pages
[x] Content pages with hero image
[ ] Section page with custom functionality, with bigger header and a different treatment in addition to photography

Dealing with this as now-separate issues.

Dealing with this as now-separate issues.


Close folders

Auto-close folders is not working for me. I install it today. The first few hours it worked but then it stopped.
I have google-chrome version: 40.0.2209.0 dev (64-bit)

It appears that this issue has been fixed. Unselect option "Remember open folders" and select option "Auto-close folders" Thank you developer!!

I'll try to improve the wording in the options so it's more clear what they do.


Add support for updating materialized views

Updating materialized views used to be an error because providing a new
definition for a materialized view requires dropping and recreating it.
This will cause any indexes on the old materialized view to be dropped.
For this reason, we suggested that users manually drop_view,
create_view and add_index. This is a frustrating user experience.
This changes view updates to record any indexes that were in place
before the drop_view and then re-create those indexes once the new
materialized view is in place. Any indexes that are now invalid
(most-likely because the column no longer exists) will be dropped. This
behavior is similar to what happens when you alter the schema of a
table, so I do not believe it to be surprising.
We log the results of each index creation for the users information in
the migration output. For example:
==<PHONE_NUMBER>1117 UpdateSearchesToVersion2: migrating =========================
-- update_view(:searches, {:version=>2, :revert_to_version=>1, :materialized=>true})
   -> index 'searches_test_1' on 'searches' has been recreated
   -> index 'searches_test_2' on 'searches' is no longer valid and has been dropped.
   -> 0.0148s
==<PHONE_NUMBER>1117 UpdateSearchesToVersion2: migrated (0.0149s) ================

Closes #98

@calebthompson I moved the PG specific code into the adapter. I think there's some further refactoring (as pointed out by hound's ABC warning here) to slim down and break up the adapter, but I'd like to attack that in a separate refactoring.

lgtm


fix nightly release bash script

There are two errors in the past nightly CI:

https://github.com/lycheeverse/lychee/actions/runs/5876633509/job/15935180970

because the gh release delete --cleanup-tag  won't delete the local tag, refer to https://github.com/cli/cli/blob/4a57a812f5db9db72812030f19b214c93b18b297/pkg/cmd/release/delete/delete.go#L136

https://github.com/lycheeverse/lychee/actions/runs/5888738531/job/15970534873

because the previous nightly CI deleted the remote tag & release but failed to generate a new one

This PR should fix them. It will detect if the nightly tag exists:

exist: check if there are new commits
not exist: release a new nightly

Fingers crossed. 😎 🤞

Huh https://github.com/lycheeverse/lychee/actions/runs/5891881988/job/15979942353


Add website?

Creating a static website for this repository should be pretty straight forward, anyone up to the task? See https://pages.github.com/

Yes, I would love to work on this! @ScottBrenner
Can I work on this? @the-ethan-hunt

Great idea! Perhaps the site could look much like this one?

Hello @the-ethan-hunt I am participant of GSSOC. I would be happy to contribute to it if you want

I would also like to work with you guys @ScottBrenner @pbiswas101 can i work with them? @the-ethan-hunt

@Dhiraj240 , you already have been assigned an issue.
Issue assigned to @ScottBrenner only for now

Hey Scott, gonna build it using static site generator? Like Hugo or Jekyll?

Yes - also, someone else is welcome to work on this instead of me.

Then I would like to work on it!

Assigned to @pbiswas101

Issue unassigned and up for grabs!

@the-ethan-hunt I want to work on this issue!

Assined to @Dhiraj240

Assigned to @shreyateeza

@shreyateeza @the-ethan-hunt  can we work together? If you don't mind @shreyateeza

@Dhiraj240 Just saw your comment. A PR regarding this has been created(#101).
Feel free to take another issue!

okay, cool.

@the-ethan-hunt  I would like to work on this.Can I work on this ?

I would like to work on this as well!
If you don't mind @Pallavi024 . Can we both work on this!

@Pallavi024 @megha070 I am working on this and will soon update my PR(#101).

@Pallavi024 and @megha070 why don't you two work on this together while @shreyateeza works on her site.

Hi @the-ethan-hunt ,
Can I take this up?

I'm interested how far is the website

@Qamar30 there has been no progress. You are free to take up the issue.

Hello @the-ethan-hunt!
Must be bored of getting people to comment on this...
Anyway I want to take it. I participated in GCI this year and have kind of good knowledge in making websites using github pages and jekyll/HTML & CSS...
I need some more details tho... What should the website look like? I mean does the first page needs to only correspond to the README.md?
Also, I'm thinking of working on this issue.

I created an index.rst as the first page of the website after the README.md file. Let me know if it's good.


Variable Height Width Input Support

First of all, Love your work on GAN Inversion and Editing.
Was experimenting with this codebase. I had a question.
We were trying to train this codebase with say a variable input StyleGAN like (1024,512) but this was throwing some errors (Shape Mismatch Errors). Like the codebase expects fixed inputs like (1024,1024)
Is there a way to alleviate this by say small changes to the code? I am posting the error below.
Thanks
"""    self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for Generator:
size mismatch for input.input: copying a param with shape torch.Size([1, 512, 4, 2]) from checkpoint, the shape in current model is torch.Size([1, 512, 4, 4]).
size mismatch for noises.noise_0: copying a param with shape torch.Size([1, 1, 4, 2]) from checkpoint, the shape in current model is torch.Size([1, 1, 4, 4]).
size mismatch for noises.noise_1: copying a param with shape torch.Size([1, 1, 8, 4]) from checkpoint, the shape in current model is torch.Size([1, 1, 8, 8]).
size mismatch for noises.noise_2: copying a param with shape torch.Size([1, 1, 8, 4]) from checkpoint, the shape in current model is torch.Size([1, 1, 8, 8]).
size mismatch for noises.noise_3: copying a param with shape torch.Size([1, 1, 16, 8]) from checkpoint, the shape in current model is torch.Size([1, 1, 16, 16]).
size mismatch for noises.noise_4: copying a param with shape torch.Size([1, 1, 16, 8]) from checkpoint, the shape in current model is torch.Size([1, 1, 16, 16]).
size mismatch for noises.noise_5: copying a param with shape torch.Size([1, 1, 32, 16]) from checkpoint, the shape in current model is torch.Size([1, 1, 32, 32]).
size mismatch for noises.noise_6: copying a param with shape torch.Size([1, 1, 32, 16]) from checkpoint, the shape in current model is torch.Size([1, 1, 32, 32]).
size mismatch for noises.noise_7: copying a param with shape torch.Size([1, 1, 64, 32]) from checkpoint, the shape in current model is torch.Size([1, 1, 64, 64]).
size mismatch for noises.noise_8: copying a param with shape torch.Size([1, 1, 64, 32]) from checkpoint, the shape in current model is torch.Size([1, 1, 64, 64]).
size mismatch for noises.noise_9: copying a param with shape torch.Size([1, 1, 128, 64]) from checkpoint, the shape in current model is torch.Size([1, 1, 128, 128]).
size mismatch for noises.noise_10: copying a param with shape torch.Size([1, 1, 128, 64]) from checkpoint, the shape in current model is torch.Size([1, 1, 128, 128]).
size mismatch for noises.noise_11: copying a param with shape torch.Size([1, 1, 256, 128]) from checkpoint, the shape in current model is torch.Size([1, 1, 256, 256]).
size mismatch for noises.noise_12: copying a param with shape torch.Size([1, 1, 256, 128]) from checkpoint, the shape in current model is torch.Size([1, 1, 256, 256]).
size mismatch for noises.noise_13: copying a param with shape torch.Size([1, 1, 512, 256]) from checkpoint, the shape in current model is torch.Size([1, 1, 512, 512]).
size mismatch for noises.noise_14: copying a param with shape torch.Size([1, 1, 512, 256]) from checkpoint, the shape in current model is torch.Size([1, 1, 512, 512]).
size mismatch for noises.noise_15: copying a param with shape torch.Size([1, 1, 1024, 512]) from checkpoint, the shape in current model is torch.Size([1, 1, 1024, 1024]).
size mismatch for noises.noise_16: copying a param with shape torch.Size([1, 1, 1024, 512]) from checkpoint, the shape in current model is torch.Size([1, 1, 1024, 1024]).
"""

Managed to bring variable (height,width) support in my case by changing the following lines in model.py for (1024,512). Model Wanted (1024,1024). I believe one could do a future PR and incorporate this permanently in the code.. Interesting feature to add.
For Input:
self.input = nn.Parameter(torch.randn(1, channel, size, size ))
to
self.input = nn.Parameter(torch.randn(1, channel, size, (size//2) ))
For noise Layers:
for layer_idx in range(self.num_layers):
res = (layer_idx + 5) // 2
#edited
shape = [1, 1, 2 ** res, 2 ** res]
to
shape = [1, 1, 2 ** res, (2 ** res)//2]
def make_noise(self):
    device = self.input.input.device

    #edited
    noises = [torch.randn(1, 1, 2 ** 2, 2 ** 2, device=device)]
    to
    noises = [torch.randn(1, 1, 2 ** 2, (2 ** 2)//2, device=device)]

    for i in range(3, self.log_size + 1):
        for _ in range(2):
            #edited
            noises.append(torch.randn(1, 1, 2 ** i, 2 ** i, device=device))
            to
            noises.append(torch.randn(1, 1, 2 ** i,  (2 ** i)//2, device=device))

But be forewarned if anyone tries this. This adds issues with the Wencoder which need to be retrained to match the dimensions.


Search doesn't work when invoked from the "Products to be completed" screen

Search doesn't work when invoked from the "Products to be completed" screen

It should be fixed, but this needs to be verified.

This is fixed.


Error when training PV-RCNN on multiple GPUs

I am successfully able to train PV-RCNN on single GPU. I am able to train other networks say pointrcnn on multiple gpus. However when I try to train PV-RCNN on multiple GPUs, I get the following error:
2020-11-16 18:07:54,162   INFO  Start training kitti_models/pv_rcnn_ours_10(default)
epochs:   0%|          | 0/80 [00:00<?, ?it/s]
epochs:   0%|          | 0/80 [00:00<?, ?it/s]
epochs:   0%|          | 0/80 [00:00<?, ?it/s]
epochs:   0%|          | 0/80 [00:00<?, ?it/s]
train:   0%|          | 0/988 [00:00<?, ?it/s]
epochs:   0%|          | 0/80 [00:11<?, ?it/s]
Traceback (most recent call last):
File "train.py", line 205, in
main()
File "train.py", line 177, in main
merge_all_iters_to_one_epoch=args.merge_all_iters_to_one_epoch
File "/team1/codes/3dObjDet/OpenPCDet_ravi/tools/train_utils/train_utils.py", line 100, in train_model
dataloader_iter=dataloader_iter
File "/team1/codes/3dObjDet/OpenPCDet_ravi/tools/train_utils/train_utils.py", line 38, in train_one_epoch
loss, tb_dict, disp_dict = model_func(model, batch)
File "/team1/codes/3dObjDet/OpenPCDet_ravi/pcdet/models/init.py", line 30, in model_func
ret_dict, tb_dict, disp_dict = model(batch_dict)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/parallel/distributed.py", line 376, in forward
output = self.module(*inputs[0], **kwargs[0])
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/team1/codes/3dObjDet/OpenPCDet_ravi/pcdet/models/detectors/pv_rcnn.py", line 11, in forward
batch_dict = cur_module(batch_dict)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/team1/codes/3dObjDet/OpenPCDet_ravi/pcdet/models/backbones_3d/spconv_backbone.py", line 139, in forward
x = self.conv_input(input_sp_tensor)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/spconv/modules.py", line 127, in forward
input.features = module(input.features)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/batchnorm.py", line 83, in forward
exponential_average_factor, self.eps)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/functional.py", line 1697, in batch_norm
training, momentum, eps, torch.backends.cudnn.enabled
RuntimeError: cuDNN error: CUDNN_STATUS_EXECUTION_FAILED
Traceback (most recent call last):
File "train.py", line 205, in
main()
File "train.py", line 177, in main
merge_all_iters_to_one_epoch=args.merge_all_iters_to_one_epoch
File "/team1/codes/3dObjDet/OpenPCDet_ravi/tools/train_utils/train_utils.py", line 100, in train_model
dataloader_iter=dataloader_iter
File "/team1/codes/3dObjDet/OpenPCDet_ravi/tools/train_utils/train_utils.py", line 38, in train_one_epoch
loss, tb_dict, disp_dict = model_func(model, batch)
File "/team1/codes/3dObjDet/OpenPCDet_ravi/pcdet/models/init.py", line 30, in model_func
ret_dict, tb_dict, disp_dict = model(batch_dict)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/parallel/distributed.py", line 376, in forward
output = self.module(*inputs[0], **kwargs[0])
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/team1/codes/3dObjDet/OpenPCDet_ravi/pcdet/models/detectors/pv_rcnn.py", line 11, in forward
batch_dict = cur_module(batch_dict)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/team1/codes/3dObjDet/OpenPCDet_ravi/pcdet/models/backbones_3d/spconv_backbone.py", line 139, in forward
x = self.conv_input(input_sp_tensor)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/spconv/modules.py", line 127, in forward
input.features = module(input.features)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/batchnorm.py", line 83, in forward
exponential_average_factor, self.eps)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/functional.py", line 1697, in batch_norm
training, momentum, eps, torch.backends.cudnn.enabled
RuntimeError: cuDNN error: CUDNN_STATUS_EXECUTION_FAILED
                                          Traceback (most recent call last):

File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/usr/local/lib/python3.6/dist-packages/torch/distributed/launch.py", line 235, in
main()
File "/usr/local/lib/python3.6/dist-packages/torch/distributed/launch.py", line 231, in main
cmd=process.args)
subprocess.CalledProcessError: Command '['/usr/local/bin/python', '-u', 'train.py', '--local_rank=0', '--launcher', 'pytorch', '--cfg_file', 'cfgs/kitti_models/pv_rcnn_ours_10.yaml']' returned non-zero exit status 1.

Check if torch.cuda.is_available() is True. Sometimes it switches to False due to system changes or device occupancy. Try to restart your shell or system?

@raviteja-kvns Did you use the latest version? I also met such problem that I can train with one GPU but fail with multi GPU. However, I did not get the error information as I run the code in the server. I do not meet the problem in the previous version.

Thank you @NLCharles and @LZDSJTU for your suggestions. Turns out the issue is with batch size 2 i.e. GPU memory is not sufficient. When I change batch size to 1, multi-gpu training works fine.

Hello, I have encountered a similar error when training with multiple 2080ti GPUs. What is your GPU model you use, how much GPU training solves this problem, thank you


[bug] Multiple Injected Providers for same wallet

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
2.0.1
wagmi Version
2.5.7
Current Behavior
I'm working on doing some performance refactors in my NextJS app, moving from pages router to app router and rendering a lot more on the server.
When I click between routes in my app without my wallet connected, each navigation adds another duplicate row for injected wallet.

Entirely possible this is on my end, I'm not very familiar with Next SSR. But it also happens with Rainbow extension (different chrome profile)

Expected Behavior
The discovered wallets should be deduped. Although maybe it's an indication I'm doing something odd?
Steps To Reproduce
My getDefaultConfig uses ssr: true
I'm happy to try put together an reproducible example but if there's a simpler explanation here that would be ideal! lmk, i can fork the rainbowkit example and switch it over to app router and see what's what
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

@sbauch Thanks for raising this issue. This is a bug that comes from the wagmi package when rendering in server-side. I've raised a PR in their repo here: https://github.com/wevm/wagmi/pull/3683.
For now can you try to not render a lot on the server and try to use RainbowKit and Wagmi only in client side ?
Check out some of our examples using next-js (SSR) and RainbowKit:

https://github.com/rainbow-me/rainbowkit/tree/main/examples/with-next-app-i18n
https://github.com/rainbow-me/rainbowkit/tree/main/examples/with-next-app
https://github.com/rainbow-me/rainbowkit/tree/main/examples/with-next

I know this is a bit crucial, but please bear with us 🙏

thanks for the quick reply!! i love Set() lol often forget about it. but awesome ok, glad I didn't do the whole repro.
I do think I'm only rendering wagmi and RainbowKit on the client? I've got an EthereumProviders component that's got the "use client"; directive, but admittedly am still wrapping my head around it all. Feels very similar to the second example above. I don't have a wallets key in my config seems like the only difference.
"use client";

export const config = getDefaultConfig({
appName: "Speedtracer",
  projectId: "my id",
  chains: [base],
  ssr: true,
  transports: {
   // ...

  },
});

// ...

export const EthereumProviders = ({ children }: Props) => (
  <WagmiProvider config={config}>
    <QueryClientProvider client={queryClient}>
      <RainbowKitProvider>{children}</RainbowKitProvider>
    </QueryClientProvider>
  </WagmiProvider>
);

but no worries, i still have a few more things to switch over before i can deploy this anyhow. ill keep an eye on the wagmi PR if you wanna close this

@sbauch Gotcha. Where do you wrap your  ?

Not sure what you mean by wrap, but I import and render the component in what is essentially the root of my client components. so it should be at the top of the suspense boundary?

@sbauch Yeah i meant more like this, but can you share a small portion of your code here so i can see what other approach you can take ?

I push it a little further down. im under the impression that this would cause the page.tsx component to render client side, as it's parent Providers component include the "use client" directive?
so im still moving some things around with this all to try to address some hydration errors, but something like this:
// layout.tsx
export default function RootLayout({
  children,
}: {
  children: React.ReactNode;
}) {
  return (
    <html lang="en">
      <body className={cn("font-Bebas antialiased", bebasNeue.variable)}>
        {children}
      </body>
    </html>
  );
}

// page.tsx
const PlayPage = async ({ params: { s } }: { params: { s: string } }) => {

  // a bunch of server-side contract calls with viem and redis caching

  return (
    <div
      className="w-[100vw] h=[100vh]"
      style={{ backgroundColor: theme.background }}
    >
      <PageComponent
        seed={BigInt(s)}
        updatedAtBlock={BigInt(updatedAtBlock)}
        currentId={BigInt(today)}
        d={d}
        theme={theme}
        ads={ads}
        leaders={leaders}
      />
    </div>
  );
};

// PageComponent.tsx
"use client";

// ...

const PlayPage = (props: Props) => {
  return (
    <Providers>
      <div className="max-h-[100dvh] flex flex-col items-center overflow-hidden">
      // ...
      </div>
    </Providers>
    );
}

@sbauch Gotcha. The best thing for now is to move your <Providers> in layout.tsx since that's where your provider will exist on top of your app instead of providing it in page.tsx + other pages. Are you able to do that ?

I dont think thats right tbh. imo placing the providers component in the loop would make the page render client side. and i only have one page in this route tree, which i want to render server side, so my providers component is only added on this page

I dont think thats right tbh. imo placing the providers component in the layout would make the page render client side. and i only have one page in this route tree, which i want to render server side, so my providers component is only added on this page

Sounds good. Yeah if you don't want to wrap it to your whole app that's fine. We'll keep this issue open and ping you once it's resolved on wagmi's side 🙏

@sbauch Sorry for the delay here. Can you upgrade your wagmi version to the latest one and see if the issue still persists ?

@sbauch Any updates ?

Closing this issue since it's wagmi has fixed this issue 👍


Why python_requires>=3.10?

Hi, is there a particular reason you only support >=3.10?
What is the limiting factor for lower versions?
Thanks, mark

We cannot test 3.9 or below internally, so we don't have a way to ensure compatibility anymore. You are free to try it on your side, but you'd have to build from source.

Fair, thanks for answering quickly


[Select[material-ui] Add regression test

Related to https://github.com/mui/material-ui/issues/40128

[x] I have followed (at least) the PR section of the contributing guide.

The test fails before the revert: https://app.circleci.com/pipelines/github/mui/material-ui/115575/workflows/7d2730a8-3438-45db-b234-b5a15d48811e/jobs/620597/parallel-runs/0/steps/0-113


FNIRSI GC-01 (CA32F0502L3): Support

Hi there!
I ordered a FNIRSI GC-01 off Aliexpress to experiment with your fantastic firmware.
Upon teardown I ran into a completely different hardware to that described here on Githud.
No battery, no visible piezo and an unmarked IC and screen ribbon.
Am I completely out of luck?
Any idea as to which electronics it is and compatibility?
Thank you!!
Federico

Please read #48.

Thanks for the quick reply.
I did have a suspension that my was the funky unit.
Difficult to spot the good units, FNIRSI doesn't seem to have an official store on Amazon.
Should you be interested I have found a data sheet for the chip version.
Thanks again!!
CA32F0156系列MCU_产品介绍.pdf


Add the numpad divide key for toggle comment

Add the ctrl + numpad divide key to the list of keyboard shortcuts for the toggle comment feature

I don't think that is really necessary. We have a pretty good default. Folks that want to use the numpad for toggle comment can customize this in their keybindings.json. We ship with a couple of keybindings using numpad keys, but those are for zooming in and zooming out. The argument in favor of those is that other software (e.g. web browsers) can zoom in/out with those key combinations.

I mainly use Atom and I always use the slash key from the keypad for comment toggle because as a french speaking user the other slash key is often used for the "é" character!
I tried this shortcut and it did not work, so I checked the settings and found that it should work. It took me a little bit of time to understand that only the slash near the enter work.
Not a critical issue but I still think that all the slash key should work.


iOS 14 - Error loading page, Domain: NSURLErrorDomain, Error Code: -1202, Description: The certificate for this server is invalid.

I just ran into a cert validation issue when loading a local development server into my webview. Before I updated to iOS 14 this was not a problem and iOS 13.7 had no issues validating the self signed certs that I was using. Suddenly, after applying the iOS 14 update, I am no longer able to load my local web server in the webview using the same certs.
Does anybody know if the TLS cert trust requirements have changed from iOS 13 to iOS 14?
The error I am getting contains the following info:
Error loading page
Domain: NSURLErrorDomain
Error Code: -1202
Description: The certificate for this server is invalid. You might be connecting to a server that is pretending to be "XX.X.X.XX" which could put your confidential information at risk.
Metro WARN:
Encountered an error loading page {"canGoBack": false, "canGoForward": false, "code": -1202, "description": "The certificate for this server is invalid. You might be connecting to a server that is pretending to be “<IP_ADDRESS>” which could put your confidential information at risk.", "didFailProvisionalNavigation": true, "domain": "NSURLErrorDomain", "loading": false, "target": 7, "title": "", "url": ""}

Device:: iPhone 11 Pro, iPhone 11 Pro Max
iOS: 14.0
react-native: 0.63.2
react-native-webview: 10.9.0

I have the same issue. but it seems to happen only when VPN is on.

Happening here as well over an https server hosted with caddy + docker.
I navigated to my caddy instance' certificates folder and installed them onto my emulator... no dice - still getting "An SSL error has occurred and a secure connection to the server cannot be made."
react: 16.13.1 => 16.13.1
react-native: 0.63.2 => 0.63.2

Having the same issue:

on iOS: Domain: NSURLErrorDomain, Error Code: -1202, Description: The certificate for this server is invalid...
on Android: Domain: undefined, Error Code: 3, Description: SSL error: The certificate authority is not trusted
This happens when trying to load a webpage that has self-signed SSL.

Same thing with me, when I have the VPN on it won't work at all.  Self signed server.  When I turn off the VPN it works well.

This is still happening. My particular case is when trying to load any site lacking a valid SSL cert via props.source.uri. Did anyone make any progress with this?

This is still happening. My particular case is when trying to load any site lacking a valid SSL cert via props.source.uri. Did anyone make any progress with this?

Still this issue, some times work sometimes not

Still this issue, some times work sometimes not

I'm also still encountering this error. I've got a weird local setup with Charles Proxy and SSL Proxying. On iOS (where I'm testing), it seems to stem from the fact that webviews don't handle self-signed certificates in the same way as Safari.

I'm experiencing this as well since updating to React Native 0.64.0, it works fine on Android, but getting a -1003 error code on iOS 14, don't have any other version of iOS to test with.

I'm experiencing this as well since updating to React Native 0.64.0, it works fine on Android, but getting a -1003 error code on iOS 14, don't have any other version of iOS to test with.

hello there, do you found the solution for this issue? getting it as well

Can we re open this, alot of people are facing this issue.

Another "me too" here, iOS 14.4 simulator, happened twice in the span of a couple hours.

Hey everyone, check if the SSL is expired on the URL you are using, if so there is nothing you can do much except to include a SSL or completely remove it.

is there any solution?

I am looking for solution as well

try check your SSL certs on you domain

Any solution?

+1

Any solution?

We did such trick https://github.com/react-native-webview/react-native-webview/issues/1834#issuecomment-1482454783


Added more unit tests for zookeeper.

Added more testing for zookeeper.
Coverage before this pull request:
Statements   : 79.17% ( 1543/1949 )
Branches     : 62.25% ( 442/710 )
Functions    : 76.82% ( 348/453 )
Lines        : 80.31% ( 1489/1854 )

Coverage after this pull request:
Statements   : 82.2% ( 1602/1949 )
Branches     : 64.65% ( 459/710 )
Functions    : 82.34% ( 373/453 )
Lines        : 83.39% ( 1546/1854 )

Thanks for the PR @lightswitch05!


Issue patch release with fix for issue #50

PR #51 was merged, which contains a fix for issue #50. Can the maintainers please issue a patch release containing this fix?

@hchokshi  the same edit from PR #51 needs to be applied to PxPostGateway.php in order to fully resolve issue #50.
I'll make a PR for this later tonight, unless you're able to do so earlier.

I've opened PR #53 which removes the Guzzle dependency from the PxPost gateway.

Ahh thanks for that @mikeybeck. Hopefully gets merged and released soon. Didn't notice earlier as all the tests passed without changing this one. Might be a test coverage issue there?

@barryvdh can we please get a semver tagged release with all the HTTP client fixes now merged? Thanks :)

Yes tagged v3.0.1


[tuify][giti] ideas for making our Rust CLI / tuify git buddy called giti

Bin target in r3bl_tuify/Cargo.toml
We have a bin target called giti here:
https://github.com/r3bl-org/r3bl-open-core/blob/main/tuify/src/bin/giti.rs#L65

Why
git is notoriously difficult to use. giti aims to simplify some of the git workflows by using r3bl_tuify. One example of this is the git branch -D workflow, when you have many local branches to choose from. You have to run the git branch -D on each one of the branches, which is tedious.
Better UX
Instead we can use r3bl_tuify to allow the user to select multiple branches and then perform an operation on them (like delete, or whatever else). The UX of this command still needs to be flushed out.
"Shell out" to git executable and run git branch commands
Here's more information on the git branch command and how we can pass arguments to it, so that we can output data from the command in a way that we can use in giti. We will "shell out" from the binary target to run git executable, since this is more of a demo of what tuify can do.

Here's the Rust API for that: https://doc.rust-lang.org/std/process/struct.Command.html
For now, we are ok shelling out, despite its limitations. For a more serious implementation, we can look at https://github.com/Byron/gitoxide so we don't have to shell out to git binary on the user's computer.

Here are some commands to get the output from git branch command in a way that is easy for our Rust program to use:

git branch --format '%(refname:short)' --show-current
git branch --format '%(refname:short)'

git branch --format is not a valid command. The git branch command is used to list, create, or delete branches in a Git repository. However, you can use git for-each-ref command to format the output of git branch command.
Here's an example of how to use git for-each-ref to output the branch name, author, and date for all branches:
git for-each-ref --format='%(refname:short) %(authorname) %(authordate)' refs/heads/
In this example, %(refname:short) is the placeholder for the branch name, %(authorname) is the placeholder for the author name, and %(authordate) is the placeholder for the author date.
References and guides
You can find a list of all available placeholders in the git for-each-ref documentation.

https://git-scm.com/docs/git-for-each-ref

Tutorial on how to use clap & tuify:

https://developerlife.com/2023/09/17/tuify-clap/

Video w/ ideas on "type driven" API design:

https://www.youtube.com/watch?v=bnnacleqg6k

Other projects that were started a long time ago in this vein

The following repo contains some Kotlin code that makes it easy to search git log output to find related commits in long histories: https://github.com/r3bl-org/git-tools

@NadiaIdris Just added this idea here as an issue; didn't know if it is better as an issue or github discussion.

@Harshil-Jani FYI - more info on giti

select_from_list() method returns Vec<String> for Both SelectionMode's, Single and Multiple. But In case of Single it looks much cleaner to have String datatype returned from the API

@Harshil-Jani @NadiaIdris That's a great idea. Simple but useful developer experience / usability / ergonomic improvement for the API.

@NadiaIdris @Harshil-Jani I don't think using lib-git2 is a path we want to go down. It might just be better to use process exec and "shell out" to execute git commands directly and use stdin and stdout to communicate w/ these git subcommands.
Alternatively we can use something called gitoxide. It is git but rewritten in Rust 100%. Here's an example of using gitoxide as a library (not binaries)
https://github.com/Byron/gitoxide/discussions/1020
Here's an example of how the git clone subcommand is implemented in Rust code, w/out using process exec to shell out:
https://github.com/Byron/gitoxide/blob/072ee32f693a31161cd6a843da6582d13efbb20b/gitoxide-core/src/repository/clone.rs#L75-L84

https://asciinema.org/a/14V8v3OKKYvDkUDkRFiMDsCNg
Command to run this:
cd tuify ; ./run.nu install-giti ; giti branch -c delete


Draft: [9] Create unit tests

Fixes #9
Hi, I started implementing unit tests, I create a sample one.
Of course I had to change the initialization strategy for the KeycloakWrapper class otherwise I could not use mocks.
So if you are okay with the changes I did, I will continue with the tests.
Let me know if you think anything can be changed and improved.
Best 😄

I'll close this PR since the package has gone thru many changes and unit tests are not required anymore since massive refactoring is needed for creating mocks. If you want to help, you can refer to the open issues.
BTW, thank you for your contribution. Really appreciated.


Update libphonenumber dependency

Hello there..
I'am having an issue while sending to some new phone numbers and I figured out that new format number was published earlier in libphonenumber 8.13.3 and the current used version is 8.12.32..
Upgrading libphonenumber to latest version 8.13.27 will fix that issue.

Hey there,
Thanks for letting me know about this issue.
If you're using the local server mode, you can give version 1.3.1 a try from our GitHub releases here. If you're using the cloud server, I'll update the server in the next few days to include the latest libphonenumber version.


Spatial indexing of GeoAgents

What's the problem this feature will solve?
Mesa-Geo uses RTree for spatial indexing of GeoAgents. When such indexing is not necessary (e.g., no neighborhood queries are needed), it could be a performance bottleneck, especially if there's a huge number of GeoAgents. See https://github.com/wang-boyu/agents-and-networks-in-python/pull/2.
On the other hand, when indexing is actually needed, it happens only when adding agents: https://github.com/projectmesa/mesa-geo/blob/a12f73570c0cf56e94c36675ec0686083882c792/mesa_geo/geospace.py#L292-L308
Whenever agents move and need re-index, users need to manually call GeoSpace._recreate_rtree(). For example: https://github.com/projectmesa/mesa-geo/blob/a12f73570c0cf56e94c36675ec0686083882c792/examples/geo_sir/model.py#L109
Describe the solution you'd like

Do not perform spatial indexing when it is not needed.
When spatial indexing is needed, automatically keep track of agents' movements and re-index accordingly.

Thanks for opening he issue here!
Maybe we could add an additional layer in GeoSpace specifically for fast moving objects?
As of know there is the AgentLayer which index everything with rtree and then the StaticLayers with rasters, images and geodataframes
https://github.com/projectmesa/mesa-geo/blob/a3f01ad47a454246c2da05d0d6c7ccc27bcd963b/mesa_geo/geospace.py#L84
Maybe I could add the fast agents as a vector layer and keep track of the movements there
I could try to create a small PR with this change if you think it is interesting

Sounds good! A PR would be great - appreciate your time and effort on this!
Just a note on visualization that might be related - MapVisualization.py renders layers and agents separately in different ways: https://github.com/projectmesa/mesa-geo/blob/a3f01ad47a454246c2da05d0d6c7ccc27bcd963b/mesa_geo/visualization/modules/MapVisualization.py#L76-L80
If you have any questions regarding creating a PR, please feel free to ask. We also have a contributors guide for your reference.

Sure! and thanks for the heads-up on the layer rendering, this is indeed important to consider.
I'll look at the contrib guide and post something here as soon as possible

Implemented in https://github.com/projectmesa/mesa-geo/pull/179. Closing.


Page title not updating immediately after render on Chrome Desktop

Do you want to request a feature or report a bug?
(POSSIBLE) BUG / WEIRD BEHAVIOR
What is the current behavior?
I'm rendering the following structure:

The App component renders a <Helmet/> with a default <title/>
It calls some API and when the data arrives. It renders a <Component/>
The component renders its own <Helmet/> with its own <title/> changing the page title.

<App>

  <Helmet>
    <title>DEFAULT TITLE</title>
  </Helmet>

  // APP MAKES SOME API CALLS AND ONLY RENDERS <Component>
  // WHEN DATA ARRIVES

  {  dataFromAPI &&
    <Component>
        // THE <Helmet/> IS REPRESENTED HERE, BUT IT IS RENDERED BY <Component/>
        <Helmet>
          <title>COMPONENT TITLE</title>
        </Helmet>
    </Component>
  }

</App>

The fact is that from Component, on Chrome Desktop, we get this:
  function Component() {

    useEffect(()=>{
      console.log(document.title);                          // Logs  "DEFAULT TITLE"
      setTimeout(()=>console.log(document.title),0);        // Logs  "COMPONENT TITLE"
    },[]);

    return(
       <React.Fragment>
          <Helmet>
             <title>COMPONENT TITLE</title>
          </Helmet>
           // SOME OTHER STUFF
       </React.Fragment>
    );
  }

WEIRD BEHAVIOR ON DESKTOP CHROME
This weird behavior is happening on Chrome Version 81.0.4044.129 (Official Build) (64-bit)
And Firefox 75.0 (64-bit) is working fine. This is all I have tested so far.
How is it possible that from a useEffect from a <Component/> that just rendered a <Helmet/> changing the page title, I'm logging the DEFAULT TITLE which was rendered by a parent?
When I log it inside a 0ms setTimeout(), it logs the proper COMPONENT TITLE
What is the expected behavior?
I would be able to log the updated title from the useEffect immediately, without using a setTimeout.
Which versions of React and react-helmet, and which browser / OS are affected by this issue?
Did this work in previous versions of React and/or react-helmet?
REPRODUCIBLE EXAMPLE
I tried my best to create a CodeSandbox with this issue, but I wasn't able to reproduce it. Anyway, I'm getting this behavior on my local environment. Chrome Desktop does not work properly. Firefox works fine.
CHROME:

FIREFOX:

QUESTION
Is this a known issue? Is there a way to "fix" it for the Chrome Desktop browser?
MY USE CASE
I need to access the updated title inside the useEffect in order to send an Analytics Page View hit from my web app, which is an SPA. So I need to use the current title to send it over the analytics gtag call.
"react": "^16.13.1",
"react-dom": "^16.13.1",
"react-helmet": "^6.0.0",

@cbdeveloper you can pass defer={false} to Helmet component. It solved the same problem for me.

Same issue.Do you have any solution about this bug?

I don't have a definitive solution to this yet. This bug was a problem to me because I send analytics events from those useEffect and I was getting lots of page_title: not set in my reports. I feel that the setTimeout(() => ... , 0) definitely helps, but sometimes I'm still getting page_title: not set, which means the code wasn't able to read the document.tile. I'm even thinking about trying with 100 or 200ms timeout to see if it improves. Anyway, that is how I'm dealing with this so far.

I didn't test this yet. A workaround is you put analytics scripts at the bottom of the index.html exactly before </body>.

The most prominent idea here is that the problem is due to react-helmet.
It is for the topic due to the react render process and how to understand it. For other comments the usage/when and how of the tracking api.

React useEffect is not a one time render; it will console log once and after the timeout again (next tick; is most probably after the render of helmet; maybe)
Why are you relying on document.title
Generally you do not send the title to the tracking api just the page view, i.e. the route.
The page view should be sent on route change; i.e. utilize routing. (in a SPA on route load if the route has the data then send the page view; not before)

Let me know if I missed something! :sheep:

@AubreyHewes thanks for the ideas.
Since my app is an SPA, gtag won't fire pageViews on client route changes. That's why my code needs to implement this behavior. For a multiple page APP rendered on the server, it would be possible to rely solely on the gtag default <script/> from Google Analytics. That would fire every time a page was rendered.
Everything is working fine. But I'll definitely use the title that comes on my data rather than relying on the document.title to update (which is working fine, but not in the order/sync I was expecting it to).
Thanks again.

@cbdeveloper I fully understood that your app was a SPA and gtag wont fire changes. With pageView and page view I was trying to be agnostic. i.e. when you want to register a page view you use the api for it. So on route change use your analytics api to register a page view.
There are libraries for this.. like react-ga and such. Though you still need to feed them "extra" data if required. In your case the title.

Did not know about the react-ga package. Will give that a try. What I can take from this issue is that I can't rely on the assumption that the browser will update the document properties synchronously, so I have to work around that. I'll close this issue.
My conclusion is:

<Component/> rendered a new Page <Helmet/>
useEffect from <Component/> must be aware that document properties may not have been update at this point.

What I can take from this issue is that I can't rely on the assumption that the browser will update the document properties synchronously

Remember your React lessons.. how does React work?.. assumptions are not part of the process.

What I can take from this issue is that I can't rely on the assumption that the browser will update the document properties synchronously

Remember your React lessons.. how does React work?.. assumptions are not part of the process.

@cbdeveloper Did you figure out a work around for this? It is causing a ton of SEO issues for us since bots are going with the default title as that gets caught before the actual value gets passed.


Use New CodeBuild Local Cache

AWS CI/CD is awesome, with the one drawback of the slow builds when docker image is being built. I saw that AWS recently released this blogpost about the new Local Cache feature.
I think it can be simple and awesome to see it also part of this Terraform module

Yes, that would be pretty sweet. That said, this is not something we can prioritize at this time. If you want to open a PR for it, we'll promptly review.

May you tag this as feature request? :)
Currently it is still blocked by https://github.com/terraform-providers/terraform-provider-aws/issues/7643


DeprecationWarning

Hey,
I followed the instructions in the README and tried training the rnn-struct model. While running python -m experiments.exp_train_test cdcp --method rnn-struct --model strict I get the following warning multiple times:
/data/defacto/env/lib/python3.5/site-packages/sklearn/preprocessing/label.py:151: DeprecationWarning: The truth value of an empty array is ambiguous. Returning False, but in future this will result in an error. Use array.size > 0 to check that an array is not empty.
Complete Output:
[dynet] random seed:<PHONE_NUMBER>
[dynet] allocating memory: 512MB
[dynet] memory allocation done.
2018-04-03 19:55:42,096 - root - INFO - rnn-struct strict on cdcp ({'max_iter': 10, 'mlp_dropout': 0.2})
2018-04-03 19:55:56,745 - root - INFO - Setting node class weights fact: 1.0, policy: 1.0, reference: 1.0, testimony: 1.0, value: 1.0
2018-04-03 19:55:56,745 - root - INFO - Setting link class weights False: 1.0, True: 30.359851988899166
2018-04-03 19:55:56,745 - root - INFO - Overriding n_embeds to glove size 300
2018-04-03 19:55:56,752 - root - INFO - Initializing embeddings...
2018-04-03 19:55:57,006 - root - INFO - ...done
[epoch=1 eta=0.001 clips=568 updates=579] Iter 0 loss 0.5513
74.4% integer, 24.4% fractional, 1.2% not solved
[epoch=2 eta=0.001 clips=544 updates=567] Iter 1 loss 0.5325
84.9% integer, 13.9% fractional, 1.2% not solved
[epoch=3 eta=0.001 clips=502 updates=560] Iter 2 loss 0.5270
83.0% integer, 16.2% fractional, 0.9% not solved
[epoch=4 eta=0.001 clips=465 updates=541] Iter 3 loss 0.5169
84.5% integer, 13.8% fractional, 1.7% not solved
[epoch=5 eta=0.001 clips=435 updates=540] Iter 4 loss 0.5120
82.3% integer, 15.8% fractional, 1.9% not solved
[epoch=6 eta=0.001 clips=418 updates=535] Iter 5 loss 0.5089
84.7% integer, 12.9% fractional, 2.4% not solved
[epoch=7 eta=0.001 clips=437 updates=526] Iter 6 loss 0.5083
85.0% integer, 12.6% fractional, 2.4% not solved
[epoch=8 eta=0.001 clips=425 updates=513] Iter 7 loss 0.4793
86.6% integer, 11.9% fractional, 1.5% not solved
[epoch=9 eta=0.001 clips=395 updates=489] Iter 8 loss 0.4506
82.3% integer, 16.0% fractional, 1.7% not solved
[epoch=10 eta=0.001 clips=373 updates=459] Iter 9 loss 0.4102
85.0% integer, 13.1% fractional, 1.9% not solved
2018-04-03 21:22:50,187 - root - INFO - Training time: 579.24s/iteration (1.00s/doc-iter)
/data/defacto/env/lib/python3.5/site-packages/sklearn/preprocessing/label.py:151: DeprecationWarning: The truth value of an empty array is ambi
guous. Returning False, but in future this will result in an error. Use array.size > 0 to check that an array is not empty.
  if diff:
/data/defacto/env/lib/python3.5/site-packages/sklearn/preprocessing/label.py:151: DeprecationWarning: The truth value of an empty array is ambi
guous. Returning False, but in future this will result in an error. Use array.size > 0 to check that an array is not empty.
.
.
.
.
2018-04-03 21:23:48,273 - root - INFO - Prediction time: 0.37s/doc
2018-04-03 21:23:48,274 - root - INFO - Test inference status: 100.0% integer

Also, I am not able to run python -m experiments.plot_test_results.py cdcp which gives the following error:
/data/defacto/env/bin/python: Error while finding spec for 'experiments.plot_test_results.py' (AttributeError: module 'experiments.plot_test_results' has no attribute '__path__')

deprecation warnings are just warnings, they do not cause any further issues, you can safely ignore them. You can probably fix it by updating your scikit-learn version.
As for the second issue, you have a typo: the command should be:
python -m experiments.plot_test_results cdcp

note that there is no .py at the end, because we are talking about modules, not files.


Yet another refactoring

This PR adds:

Better logs http status in websocket
Move config objects to configuration.go
Remove providerTemplates dir, moved in templates
Adds traefik.frontend.rule and traefik.frontend.value labels in Docker and Marathon.
Fixes BoltDB backend.

Fixes #64
Fixes #73
Fixes #68

@emilevauge couple of comments :wink:

Thank you @vdemeester for you code review :)

Overall looks good.
LGTM 🐨


[COGNITO] Unhandled exceptions after synchronize anonymous data set

This happens only if pool has disabled anonymous login.
Steps to reproduce:

In AWS panel disable anonymous account for test pool.
Open Cognito sample scene and provide pool id and region name.
Play scene.
Click synchronize.

Result (from Unity console):
anonymous_exceptions.txt

i'm not sure what is it you are trying to say. Are you saying that the sdk is not handling the exceptions correctly?
I can see that the cognito client throws a NotAuthorizedException and the cognito sync fails the synchronization. Is there any other behavior that you are expecting?

I miss Debug.LogException in log.
I forget about: http://docs.aws.amazon.com/mobile/sdkforunity/developerguide/setup-unity.html


Support sg rule generation for VPE

It looks like the generated code only includes security group rules. It needs to also produce the security groups themselves, along with a comment that tells the user which endpoints need to be assigned this SG.
The name of the SG should not be exactly identical to the name of the resource it protects (I doubt Terraform will allow that).

Both addressed in #66


Samair

Added samari.ru to be parsed

Thanks for the PR! I like the css parsing step but there seem to be two different styles currently in the proxy page - the first one is causing parser failure:

PS: Some proper exception handling could help in such cases. As a step further you could also take a look at issue #3  if you are interested.
Cheers,
Panagiotis

I've improved the extraction of the css link so, if order or page is changed, it'll still work:

    for href in soup.findAll('link'):

        if '/styles/' in href.get('href'):

           style = "http://www.samair.ru" + href.get('href')

           break

    css = requests.get(style).content.split('\n')

Looks good, thanks!


fix: remove apt dependency requirement when managing repositories manually

fixes https://github.com/elastic/puppet-logstash/issues/359

Looks great, thanks! Sorry about the test failures, they are my fault.
Please consider signing the CLA and I'll get this merged and released.
Cheers.

Hey @jarpy. Sorry for the delay, I have now signed the contributors license agreement. Thanks

Excellent! Thanks for the contribution.
Resource ordering for Debian packaging has always been tricky. I really appreciate the help.