Weird coverage report generated

The script, which extracts the coverage report from gcov's output seems to not work correctly, at least not under Cygwin. It contains much cruft, and does not contain a lot of important data (very few files in the src/ folder, for instance).
I know it works fine on Travis CI.
I'll upload an example report. I'll also check what happens on OS X.
UPDATE: I don't know what made me make the claim above, but we cannot assume the output is any better on Travis CI.

The gcov_report.txt is weird on OS X, also :-(

Actually, I have no idea whether it looks any better on Travis, as it is not recorded anywhere. This is an excerpt from gcov_report.txt:
  0.00%   ../include/CppUTest/TestFailure.h
[more]
  5.88%   ../include/CppUTestExt/MockCheckedActualCall.h
  Creating AllTests.cpp.gcov
  Creating TestFailure.h.gcov
[more - surely these are not here intentionally?]
  Creating TestHarness_c.cpp.gcov
  Removing allocator.h.gcov
 23.53%   ../tests/CppUTestExt/MockFailureTest.h
 [more of the same]
 30.00%   ../include/CppUTest/TestFailure.h
 60.95% of 105
 [huh?]
 62.26% of 106
100.00%   ../include/CppUTest/MemoryLeakDetector.h
[more reasonable entries]
100.00%   ../tests/TestResultTest.cpp
Creating AllocLetTestFree.c.gcov   Lines executed:100.00% of 9
[more of the same
Creating UtestTest.cpp.gcov   Lines executed:96.03% of 151
File ../include/CppUTest/MemoryLeakDetector.h
File ../include/CppUTest/TestFailure.h   Lines executed:100.00% of 362
File ../include/CppUTest/TestFilter.h
[more]
File ../include/CppUTestExt/MockExpectedCall.h
[more]
File ../tests/TestUTestMacro.cpp   Lines executed:70.83% of 24
Removing basic_string.h.gcov   No executable lines

I wonder what this was meant to look like?

I concur. gcov_output.txt is generated as follows, which somehow seems scrambled. Might it be that the multiple cores are used and thus having concurrency problems when generating this output?
gcov_output.txt: http://pastebin.com/mf7Kd0KU
gcov_report.txt: http://pastebin.com/QdPSLthR
gcov_report.txt.html: http://pastebin.com/2KcNpcLi
This is running the make check_coverage from a fresh mingw-64, msys (from the mingw64 sourceforge project) on windows 7 x64.
Also, a small note, for us mingw(64) users there is no autotools available in msys and we/I have to skip the "autoreconf .. -i" step on the front page. "../configure" works fine as well as everything else.

@daantimmer you can run lcov on the output and it will give you decent results. It is not part of MSYS, either, but should be easy to install.

The pastebin output is just the standard coverage output when compiling CppUTest with make check_coverage, it really looks a lot like scrambled output due to concurrency problems, but I can't see a flag/option somewhere to enable/disable concurrency.

I believe you; you don't have to convince me :)
It's just that I don't know that anyone (you?) has the time or knowledge to fix this anytime soon.... Therefore I tried to point you to an alternative that actually works, now. As a bonus, some unreachable lines are disregarded by Lcov, so the coverage is a little more accurate.

Personally, I think gcov_output.txt looks pretty much okay; filterGcov.sh could definitely use improvement though, it is very simplistic in my opinion

Personally, I disagree, the following is generated with only three source files:
File 'src/HomeAutomation/LightScheduler.c'
Lines executed:96.97% of 66
Creating 'LightScheduler.c.gcov'

File 'src/HomeAutomation/RandomMinute.c'
Lines executed:100.00% of 6
Creating 'RandomMinute.c.gcov'

Lines executed:97.22% of 72
File 'src/HAL/UART/baudrate.c'
Lines executed:100.00% of 7
Creating 'baudrate.c.gcov'

As I said, there are only three source files, however there are four messages of 'lines executed: x% of y' That feels wrong to me?

Right. I missed that. No idea what might be causing it though.

As far as I know, you have to explicitly set a flag for make to enable concurrancy.

Problem isn't concurrany.
Problem is that newer versions of "gcov" seem to output a "grand total" at the end, and cpputest is taking a LOT of assumptions on the output of gcov to generate the report.
This patch fixes the output for me, it strips off the grand total at the of the gcov report, making the output match expectations:
diff --git a/build/MakefileWorker.mk b/build/MakefileWorker.mk
index bd054ad4..c0afa5c1 100644
--- a/build/MakefileWorker.mk
+++ b/build/MakefileWorker.mk
@@ -534,10 +534,10 @@ ifeq ($(CPPUTEST_USE_VPATH), Y)
 else
        $(SILENCE)for d in $(SRC_DIRS) ; do \
                FILES=`ls $$d/*.c $$d/*.cc $$d/*.cpp 2> /dev/null` ; \
-               gcov $(GCOV_ARGS) --object-directory $(CPPUTEST_OBJS_DIR)/$$d $$FILES >> $(GCOV_OUTPUT) 2>>$(GCOV_ERROR) ; \
+               gcov $(GCOV_ARGS) --object-directory $(CPPUTEST_OBJS_DIR)/$$d $$FILES | head -n -1 >> $(GCOV_OUTPUT) 2>>$(GCOV_ERROR) ; \
        done
        $(SILENCE)for f in $(SRC_FILES) ; do \
-               gcov $(GCOV_ARGS) --object-directory $(CPPUTEST_OBJS_DIR)/$$f $$f >> $(GCOV_OUTPUT) 2>>$(GCOV_ERROR) ; \
+               gcov $(GCOV_ARGS) --object-directory $(CPPUTEST_OBJS_DIR)/$$f $$f | head -n -1 >> $(GCOV_OUTPUT) 2>>$(GCOV_ERROR) ; \
        done
 endif
        $(CPPUTEST_HOME)/scripts/filterGcov.sh $(GCOV_OUTPUT) $(GCOV_ERROR) $(GCOV_REPORT) $(TEST_OUTPUT)

Extra note, if you just ignore the "make gcov", but still enable "CPPUTEST_USE_GCOV" in the makefile. The tool "gcovr" can parse the generated files just file.
In my case, from a jenkins pipeline:
					sh 'make all_no_tests'
					sh(script: 'make all CPPUTEST_EXE_FLAGS=-ojunit', returnStatus: true)
					junit '*.xml'
					sh 'gcovr --xml > coverage.report'
					cobertura(coberturaReportFile: 'coverage.report', onlyStable: false)

The above patch didn't seem to work for me for Mac... leaving this open for now.

Hi, I have a similar issue that involves the use of gcov with the CppUTest Makefileworker. mk.
It generates everything perfectly but it also includes the header files, which essentially contain dead code.
Any advice as to how to remove the header files?


Change notification implementation for windows

Current implementation of notify on windows uses powershell. And it may take even few seconds to start to show it. I would suggest changing code to native implementation.
I tried to do that and I got it working on win xp, win 7 and win 10:

The only drawback I currently face - it doesn't show app icon, just an empty spot.
Another problem is that I rely on object constructor/destructor to clean up notification. So I need to change
pfd::notify(...);
to
pfd::notify notification(...);
And now it doesn't look very logical in terms of naming. And we need to have an object that will be destroyed so we are 100% sure that notification will be deleted if app is closed before timeout.
And we can't just rename notify to something else because notifications don't require object name on other platforms.
@samhocevar what would you suggest? Native implementation is much better than powershell

Currently the only option I see is to create some helper class like WinNotification to handle construction and destruction and in notify function have static instance:
static WinNotification notification;
notification.create(title, message, icon);

If owner approves this approach, I can make another PR.

@samhocevar I've added notification part to separate branch for a review:
compare
What would you like to change? Currently it works (no need to change how pfd::notify is called), the only thing to change is to do a check of buffer length where I copy title and message. But this is a minor tweak, I'll add it before creating another pull request.
It's better than current implementation since its faster than calling powershell and it also uses custom application icon if there is one (falls back to default windows app icon). And it also works on windows XP (where you don't have powershell).
In summary, I've added new helper class WinNotification to namespace internal. This helper class is created as static object in notify function. At each notify call this object destroy previous notification and shows a new one. If executable is terminated before notification disappears, object will destroy it in its destructor.
I've tested this code and don't see any problems with it.

Thanks, I have merged your branch in master and changed the code using a different pattern so that WinNotification could be reduced to a class with only a destructor. Let me know what you think!

It looks much better! There is a one catch though. RT_GROUP_ICON is different with unicode enabled and disabled. That's why I had ifdefs to generate different callback functions to enumerate icons. You can either rollback those changes or I can make a PR that will compile with both unicode enabled and disabled. For example, mingw by default disables it.
Or if you know how to bypass this in a different way - that'll be good too.

I don’t think I understand. The callback now uses LPCTSTR and LPTSTR instead of LPCSTR/LPSTR or LPCWSTR/LPWSTR so that the #ifdef should no longer be necessary. Did I miss something? It seems to work with both Unicode and MultiByte for me (the icon appears in the SysTray in both cases).

You're right I didn't actually test code at that time, only looked around and didn't notice your change to LPCTSTR/LPTSTR. Now I ran it, everything is working, I'm closing this issue.


Typescript error when providing a ref to an argument of Ref<>

Error I see in VScode since the last update was released today (1.8.20 (2023/10/23)).
Property '[RefSymbol]' is missing in type 'import("/Users/USER/SOME/PATH/node_modules/vue-demi/node_modules/vue/types/v3-generated").Ref<boolean>' but required in type 'import("/Users/USER/SOME/PATH/node_modules/vue/types/v3-generated").Ref<boolean>'.

This is in a Vue 2.7.15 project and was working just fine earlier today and yesterday. In this case it is an object that contains a key with :Ref<boolean>. Additionally, my editor is setup in "takeover" mode with Volar.

Property '[RefSymbol]' is missing in type 'import("/Users/USER/SOME/PATH/node_modules/vue-demi/node_modules/vue/types/v3-generated").Ref' but required in type 'import("/Users/USER/SOME/PATH/node_modules/vue/types/v3-generated").Ref'.

Could you please provide a minimal reproduction?

Yeah happy to. What is the best way to do that? Is there a base codesandbox people typically use?
To add more context, I am also seeing errors in my template like Type 'Ref<boolean>' is not assignable to type 'boolean | undefined'.ts(2322).
Perhaps it is just some weird state of the work I am in. I will keep trying to debug today and report back with more info.

Ah this is my mistake. I think I upgraded vue to 2.7.15 but not vue-template-compiler and that likely caused some issues. Going to close this as I think this was entirely my fault.


Add ability to remove toctrees from pages

This is an attempt at making it possible to remove the toctree objects from pages before outputs are built. For example, you could configure with:
html_theme_options = {
    remove_toctrees_from = ["demo/api"]
}

and any toctree items in those pages will not make it into the sidebar.
For example, take a look at our docs. Here is the api docs page, note that we have :toctree: there, so the stub files will be generated:
https://github.com/pydata/pydata-sphinx-theme/blob/4034d1bf7a97a347c6ee6dabf395ed1c34776620/docs/demo/api.rst#L11-L21
but, in our rendered docs, the demo/api page does not have any children in the navbar, and you can still click through to the stub pages from the table:
https://pydata-sphinx-theme--492.org.readthedocs.build/en/492/demo/api.html

Thanks for this analysis @rossbar - I agree with your conclusion! The main goal of this PR is precisely to give documentation authors more precision over where they remove elements of their sidebar. So you don't have to say "remove everything over level 2 across the board", you can say "remove only the sub-tress from this specific page"

I also wonder if this would be more useful as a standalone sphinx extension so that it is theme-agnostic. For example, something like sphinx-remove-toctrees or something like that. Then using it would be as simple as:
$ pip install sphinx-remove-toctrees

and in conf.py:
extensions = ["sphinx_remove_toctrees"]
remove_toctrees_for = ["list", "of/pages"]

And it could be used across the pydata theme, the book theme, and other themes like ReadTheDocs

@rgommers your diff makes me think that it'd also be useful to support glob-like patterns here :-)

I gave remove_toctrees_from a try for Dask's documentation over in https://github.com/dask/dask/pull/8238. This cut the documentation build time down (as measured by our RTD PR preview build) from ~14 minutes to ~7 minutes. Thanks @choldgraf!

@jorisvandenbossche that's a clever idea!
looking at the toctree object that is generated automatically, here it is:
<bullet_list>
    <toctree caption="True" entries="(None,\ 'demo/generated/pandas.DataFrame.drop') (None,\ 'demo/generated/pandas.DataFrame.groupby')" glob="True" includefiles="demo/generated/pandas.DataFrame.drop demo/generated/pandas.DataFrame.groupby" maxdepth="-1" rawentries="">
    <toctree caption="True" entries="(None,\ 'demo/generated/pandas.Series.array')" glob="True" includefiles="demo/generated/pandas.Series.array" maxdepth="-1" rawentries="">

Everything in entries should have the folder prefix that is defined in the :toctree: kwarg. So if the api docs are in demo and they use:
.. autosummary::
   :toctree: generated/

then all of the entries in the toctree will have a prefix of demo/generated
If that's the case, then another way to configure this might be to ask users to provide a list of folders with auto-generated stubs in them, and anything in those folders will be removed from the toctrees.
So the config would be like
remove_toctrees_from = ["demo/generated/*"]

The main difference is that you give the location of the stub pages to be removed from the sidebar, rather than the location of the parent page where the autosummary directive is located.
That seems pretty reasonable to me - curious which you think is more intuitive

Ah, I see you already implemented it in https://github.com/executablebooks/sphinx-remove-toctrees, and now you can specify it in both ways?


What happened to typing numbers?

Good day to all:
I have noticed that typing numbers in many of TW's forms is compromised.
Maybe you are already aware of this and are working on the problem, maybe not.
I have even bought myself a new keyboard thinking it was a problem on my side.

@adrik29 can you give us an example or screenshot please?

Here:
Number 2 never enters -- I tried to type page 20 and instead I got page zero.

Also in "New source" module, numbers 12345 never enter.

And I have tried different browsers.

I cannot replicat (production and development)

Does it mean your numbers are entering normally?

Tried in Chrome only

I have now tried with:
Chrome = fail
Microsoft Edge = fail
Mozilla Firefox = OK

computer restart?

No, Dmitry -- this is going on since many many days, through many restarts.
Big mystery to me -- I suspected it could be some change you introduced in the system.

I will inquire in my team to see if someone has the same problem.

could you try different computer?

Well, I recently had a problem with my mouse when it did double click instead of sing (only occasionally), I thought I am getting crazy.

Hm, I tried in another computer and the numbers take a while but they end up appearing.
This is all very strange to me.
For now, I will migrate my work in TW to Mozilla, but there must be some weird stuff going on.
I hope it is not a virus.

https://www.techbout.com/keyboard-not-typing-numbers-windows-10-48426/

@proceps @adrik29 I have had this happen to me before (in general not just in TW). Thanks @proceps. Let us know if any of these steps help @adrik29 !

https://www.techbout.com/keyboard-not-typing-numbers-windows-10-48426/

@debpaul -- thanks for the hints, I will make sure I'll try those.
 If you need help to sort this out, please contact me by email or gitter and we can meet online to solve it together. Windows does have some wonderful "features."


chat bot bug fixed and nabvar refactored

it closes issue #456

double chat box show - FIXED
ui for navbar is refactored - REFACTORED

https://github.com/user-attachments/assets/3db8e74c-721b-4e04-8cbf-f01eb1ae8ff6

@Shariar-Hasan  conflicts

got it.
i will push it tonight inshaAllah.

Conflicts


[#noissue] fix useless use of cat

Hello.
I modified the UUOC(useless use of cat)
Pointing out UUOC is a long standing shell programming tradition, and removing them from a short-lived pipeline in a loop can speed it up by 2x. However, it's not necessarily a good use of time in practice, and rarely affects correctness.
what do you think?
Thanks.
Ref: https://en.wikipedia.org/wiki/Cat_(Unix)#Useless_use_of_cat

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

Codecov Report

Merging #6055 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6055      +/-   ##
==========================================
- Coverage   40.72%   40.72%   -0.01%
==========================================
  Files        3019     3019
  Lines       90457    90457
  Branches    11814    11814
==========================================
- Hits        36843    36842       -1
  Misses      50452    50452
- Partials     3162     3163       +1

Impacted Files
Coverage Δ

...pinpoint/profiler/instrument/AccessorAnalyzer.java
57.14% <0%> (-8.58%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
82.92% <0%> (-2.44%)
:arrow_down:

...ollector/cluster/zookeeper/ZookeeperJobWorker.java
80.93% <0%> (-1.4%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.92% <0%> (+0.7%)
:arrow_up:

...p/pinpoint/rpc/client/WriteFailFutureListener.java
75% <0%> (+18.75%)
:arrow_up:

...collector/receiver/grpc/ShutdownEventListener.java
25% <0%> (+25%)
:arrow_up:

Continue to review full report at Codecov.

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

@dyyim741
I perfectly agree with you're opinion.
Thanks for you're contribution. :)


Improve binding syntax

Pact's binding syntax leaves something to be desired, by overloading object syntax and using a special sigil to indicate assignment:
(defun age-times-height-cm (id)
  (with-read people id
    { "age" := age
    , "height" := height-inches }
    (* age (* height-inches 2.54))))

The overloaded object syntax also raises the possibility of putting the wrong type in the key column.
Proposal is to recognize a second syntax reminiscent of let forms, allowing atom key names, and adding the ability to simply bind the column as named with a sole atom:
(defun age-times-height-cm (id)
  (with-read people id
    (age
     (height-inches height))
    (* age (* height-inches 2.54))))

Backwards compat is maintained by lexer accepting both forms.

FWIW, here's how Clojure does map destructuring, where client is a map:
(let [{nm :name
       loc :location
       desc :description} client]
  (println name location "-" description))

I always thought that maybe it should have been the other way around, to reflect that values live "on the right hand side" in the map:
(let [{:name nm
       :location loc
       :description desc} client]
  (println nm loc "-" desc))

/cc @joelburget @emilypi

To be clear, this destructures a client map like {:name "brian" :location "ny" :description "foo"}

Alongside a more vanilla binding, and vector destructuring:
(let [some-number 15
      {nm :name
       loc :location
       desc :description} client
      [four, five, six] [4,5,6]]
  (println name location "-" description))


Update ScratchJr FAQ

Will ScratchJr be available in other languages? needs to be updated for version 1.2.5 which has 12 languages.

I can take this, when was version 1.2.5 released?


MBP Retina 100% Scale - 4 x 4 Template Saves + Operates - Invalid Data Error on Win + ` + Template Erased

ℹ Computer information

PowerToys version: 0.21.1
PowerToy Utility: Fancy Zones
Running PowerToys as Admin: Yes
Windows build number: [run "winver"]: V2004 B20221.1000

📝 Provide detailed reproduction steps (if any)

Edit Rows Template
Add 4 columns to top + bottom
Save with name MBP Retina 100% - 4x4
Close Fancy Zones
Arrange windows successfully
Press Win + ` to open Editor
Invalid Data Error

✔ Expected result
No Error.
Or at least don't forcibly remove a template that actually works, but just can't be edited.
❌ Actual result
Error + Template Removed
📷 Screenshots

@JoeBrockhaus
can you please update to 0.23? https://github.com/microsoft/PowerToys/releases/download/v0.23.0/PowerToysSetup-0.23.0-x64.exe
After updating try again and let us know if this is still a problem.
Also consider testing with a 8 zones grid.


Design and implement dependency matrix for kraken-lib dependencies

There should be a tool that

keeps track of which combinations of dependencies have passed CI
verifies this against  customer specified versions in incoming config file
makes recommmendations based on known successful integrations of these dependencies.

Comment from @philoserf on #929 ---
an example of a kubernetes specific support matrix
https://github.com/kubernetes/client-go#compatibility-matrix


test: sum, prod, mean can be applied to zero-dim ndarray

Description
at integration test, prod, sum, mean test for zero-dim ndarray is commented out. we can uncomment them, as the function call is correct as expected.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
[X] Code is well-documented:

For user-facing API changes, Java doc has been updated.
For new examples, README.md is added to explain the what the example does.

test on Linux is OK, but failed in other platforms


Solution to day 1 problem 1

Work from the mob programming meetup on 01.12.2021.

Thanks to all participants for the awesome mob-programming spirit! Looking forward to more sessions 🎉


chore(testing-helpers): alphabetize exports

Force a release of the testing-helpers package so that it picks up the types in CI now...
fixes #1438

oh yeah 💪


Create CAP History Tables

Update the database to include CAP history table that can be used to track the changes to CAP events.  Also create a cap area table that can be used to track changes between the relationship from caps to areas/basins for cap events.
Acceptance Criteria

[ ] tables defined in functioning alembic migration
[ ] migration completes locally
[ ] implemented in test instance

CAP event status should only allow these values:

create
update
cancel

Need to either create a domain or a lut to restrict this.  Domain is easier as doesn't effect work already completed as much as implementing a lut would

tests adjusted and passing when run locally

see https://github.com/bcgov/nr-rfc-alertauthoring/pull/106


Add a Windows section for Linux oci on LCOW

When creating a default OCI spec on Windows that is targeting the LCOW
platform it needs to contain a Windows section as well. This adds the
Windows section by default. It also protects against this case for all
OCI creation that doesnt use the OCI package in the runhcs-shim.
Signed-off-by: Justin Terry (VM<EMAIL_ADDRESS>

LGTM


Bulk capabilities

Hi,
We are currently facing an issue where we are using cqrs with event-sourcing inside our application & we now need a way to bulk a bunch of commands in order to accept a high volume of data modifications. I was wondering what would be the best way to optimize performance as at the moment a single command would potentially make x number of SQL transactions. Making it an exponential behavior is more commands are sent from the same request.
We would need a way to aggregate all those commands SQL transactions into a single roundtrip.
I was thinking maybe about adding some contracts to the IStoreProvider interface as well as the ICommandSender in order to be able to handle a list of commands. This would allow the aggregation of the events processing through multiple commands with a single commit savechanges at the end.

Well, let's start with what you have done in https://github.com/clinia/cqrs:

Copy/paste of this project
All namespaces changed
No notice of changes of this original work
Not a single reference to Kledex.
License removed

Naughty naughty

Hi @lucabriguglia ,
I apologize for any inconvenience this could have caused, it is indeed not the right way of supporting open source community. I want to make this right has it is not what I'm up for. To try and go into this direction and I will for starter

Delete the repository
Create a fork from this one
Leave all references to Kledex

I hope this could make things right, as the initial purpose was mainly to add some capabilities without modifying your core library. I do apologize again.

@etiennecl we all makes mistakes and it's important to makes things right.
Thanks for the understanding.


[Feature Request] Update Average CpS units

Currently there are options to display Average CpS for past 1m, 5m, 10m, 15m, and 30m.  Please add an option for 30s and 10s, or other smaller units of your choice.
Reason for request:
Smaller units allow the user to see the difference that their big cookie clicking is making between the base CpS and what they are making overall.  I find this especially useful when attempting Speed Baking.

Finally started to look at this, and with the way I set it up before, this isn't as easy as I once thought =(

Added in Version 2.0106.2


Smartthings discovers, but shows HA-B "offline"

Hello, I'm running HA-B 5.2.1.  Echo finds and works with the Bridge fine.  Before Google Home's Hue change, that worked fine too.  For some reason, however, both Smartthings and OpenHab can discover HA-B, then immediately reports it as "offline".
Focusing on Smartthings, I have tried removing/rediscovering several times and got the same results.  Because HA-B shows offline to ST, the configured devices cannot be discovered or used.  I tried pairing using button and without, no difference.
Also, curiously, under "Manage Links", it always shows the initial time/date stamp of pairing.  That is, I remove the ST link, re-link, but the time/date stamp shows the time/date from yesterday.
Edit: Nevermind on the paragraph immediately above--I just didn't "Save" after deleting.
Any idea what is going on and how to fix it?
This post indicates that ST-HAB can be done: https://community.smartthings.com/t/controlling-x-10-switches-with-smartthings-using-hue-bridge-emulation/112365
Thanks
Edit:  I enabled debug logging and saw the following:
hue lights list requested: 71ca903d8ec4444bxxxxxxxxxx from [ip]
validateWhitelistUser: found a user <71ca903d8ec4444xxxxxxxxxxx>
The [ip] is the Smarthings IP.  Does this mean that ST is in fact talking to HA-B?  If yes, why does ST "think" that HA-B is offline?

I believe a hue Hub is required for the method described.
SmartThings states you must press a button on the hub once ST sees the hue bridge this indicates it needs a connection response from HA-Bridge. I'm not sure if that can be emulated yet from HA-Bridge.

You can emulate the button push.  Under the Bridge Control tab, Update Security Settings button, tick the Use Link Button, you will then see a Link button under the Bridge Devices tab.  That simulates a button push.  Based on what I see in the log, HA-B is responding either way.

I don't have a hub of any sort so I can't test this. Looks like a ST Hub is required.
I use the developer console to add my devices which show up in the ST app.

Not sure what you mean--I do have an ST (Smartthings) hub and I am trying to get the ST hub to work with HA-Bridge.  The ST hub is able to recognize the HA-Bridge and make the initial connection, but shows the HA-Bridge as offline.  When I add it using the IDE, it defaults the HA-B to cloud execution and I have no way to change it to local.


Fix extensions to work for all protocols

Fixes the iptables extensions and allows them to work for all protocols.
In addition, when an extension is programmed, the original rule does not need to be programmed.

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1149",
    "commit-sha": "f0b200c4fa424b1ab088811d4feeb56d0ede9d2b"
  },
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "784",
    "commit-sha": "d9a75841eb81b6c0ceba92c5534ac9500b8403fe"
  }
]


Version the Stripe Description

Right now the description on Stripe transactions is "elementary OS download' but we should make it more specific. For example, including the version name and number, like "elementary OS Freya 0.3.2" instead.

This issue is merged with  #998 pull request !


Spawners lossing data after pickup/placement

When picking up a spawner (rogue or dungeon) it losses it's original data regarding what mob it spawned and the stats it had, leaving an empty spawner in it's place

What mod version are you using? And could you send your mod list?

Mod version is Zenith 0.1.7 - 1.20.1
The mod list of the server running the world:

Bookshelf-Fabric-1.20.1-20.1.6.jar
cardinal-components-api-5.2.2.jar
cloth-config-11.1.106-fabric.jar
EnchantmentDescriptions-Fabric-1.20.1-17.0.8.jar
fabric-api-0.90.4+1.20.1.jar
fabric-carpet-1.20-1.4.112+v230608.jar
fakerlib-0.0.7.jar
InventorySorter-1.9.0-1.20.jar
Jade-1.20-fabric-11.6.2.jar
lithium-fabric-mc1.20.1-0.11.2.jar
no-telemetry-1.7.0.jar
Patchouli-1.20.1-81-FABRIC.jar
sodium-fabric-mc1.20.1-0.5.3.jar
travelers-backpack-1.20.1-9.1.6.jar
zenith-0.1.7-1.20.1.jar
zenith_attributes-0.0.4.jar

Updated the mod and its dependencies (fakerlib, zenith attributes) to the latest available (fakerlib-0.0.9, zenith-0.2.0-1.20.1, zenith_attributes-0.1.0) and the bug is present, when you pick up a spawner it displays correct on the inventory (correct entity and stats) but losses all data when placed on the world

Try with 0.2.2, it should be fixed now

Yes is fixed now, thanks for the help.
Quick question on a different topic, are rogue spawners implemented?


No results when running basic_plotting_example.py

Hello!
I am new in python but found the pysplit package that could help me a lot in my research.
I am a Linux user and was trying to run some examples. I successfully got the trajectory files after running the bulk_trajgen_example.py.
However when I want to plot a map using the basic_plotting_example.py:
$ python basic_plotting_example.py
I get the following error:
"mypath/pysplit/hyfile_handler.py:373: FutureWarning: The pandas.datetime class is deprecated and will be removed from pandas in a future version. Import from datetime module instead.
times.append(pd.datetime(*[int(x) for x in i]))"
Any idea on what can be the issue?

Hi,
I am not familiar with this error message. Packages like pandas that pysplit is dependent on is being updated, so some old codes might be out of date. However, I think it is just a warning message and I don't think it is because you used datetime in pandas. If nothing shows up, I would think of if adding a line of "%matplotlib inline" at the beginning would solve.
Zhengyu

Fixed by #66


How do you pass arguments that include variable names?

const
TestSource = '@CDAB@CD@CD@EF@GH@;';
TestP2 = True;
[TestCase('Test1',!!TestSource,TestP2!!)]
How to do?

Does this work?
const
  cTest = '@cdab@CD@CD@EF@GH@,true';

[TestCase('Test1',cTest)]

I have never tried passing a const as a parameter to an attribute, I have only ever needed inline strings.

These test constants,
I now put them in the test initialization code
It's just a drop in flexibility
It's settled
Text parsing doesn't seem like a good idea.
Machine Translation... I hope you understand.

I don't fully comprehend what you are saying, but it appears as though you have solved or otherwise worked around the issue you were having.  :1st_place_medal:


manage signed timestamped values

# Comments for current PR

add your comments here
...

!!! WARNING : DO NOT EDIT THE CHECKLIST BELOW !!!
Checklist : Merge from DEVELOP branch
Code Quality

[ ] Each function has a header (description, inputs, outputs)
[ ] Code is commented (particularly in hard to understand areas)
[ ] There are no new warnings that can be corrected
[ ] Commits policy is respected (constitancy commits, clear commits comments)

Tests Review

[ ]  Manual Tests for new features have been reviewed
[ ]  Automatic Tests for new features have been reviewed

Tests are passed the best as I can

[ ]  Non regression existing tests are passed OK
[ ]  New tests for new features & bug fixes are passed OK

Documentation

[ ] Documentation is up to date
[ ] Changelog for next release is up-to-date : fill here
[ ] If needed, tests are briefly described in test repository

We don't need to use int64_t so I changed my PR.
After reading your comments, I think we should put this feature on hold for a moment. This feature is tighly coupled with our tools as the Saas tool. I think we should wait for this feature to have a clear view of what we need to do.


module remains in state "loading" if maxUnreachableDepartures > 0

Hi Ray,
example config:
{
  module: "MMM-PublicTransportHafas",
  position: "bottom_center",

  config: {
    stationID: "008012202",  // Replace with your stationID!
    stationName: "bla bla",  // Replace with your station name!

    direction: "",                    // Show only departures heading to this station. (A station ID.)
    ignoredLines: [],                 // Which lines should be ignored? (comma-separated list of line names)
    excludedTransportationTypes: [],  // Which transportation types should not be shown on the mirror? (comma-separated list of types) possible values: StN for tram, BuN for bus, s for suburban
    timeToStation: 1,                 // How long do you need to walk to the next Station?
    timeInFuture: 15,                 // Show departures for the next *timeInFuture* minutes.

    showColoredLineSymbols: true,     // Want colored line symbols?
    useColorForRealtimeInfo: true,    // Want colored real time information (timeToStation, early)?
    showTableHeadersAsSymbols: true,  // Table Headers as symbols or text?
    maxUnreachableDepartures: 2,      // How many unreachable departures should be shown?
    maxReachableDepartures: 7,        // How many reachable departures should be shown?
    customLineStyles: "leipzig",      // Prefix for the name of the custom css file. ex: Leipzig-lines.css (case sensitive)
    showOnlyLineNumbers: false,        // Display only the line number instead of the complete name, i. e. "11" instead of "STR 11"

    updatesEvery: 60,
    marqueeLongDirections: false,
  }
},

Sometimes the HAFAS-API is slow. In this case the module remains in loading state.
Please try again and report it this problem persists.

I've found the problem! Your setting of timeInFuture is to low. Please set it to a higher value like 60 or so. That way the module will have enough departures to ”play with“.
See also issue #39.

Solved with PR #41.


Pin to<EMAIL_ADDRESS>for IE8-10 compatibility

We've recently closed a Dependabot upgrade to iframe-resizer@4

https://github.com/alphagov/govuk-design-system/pull/2568

But based on recent investigations govuk-frontend found other issues above v3.5.15
Until we're ready to drop IE8-10 let's pin this package

Looks good. Should we tell Dependabot about this pin?

@domoscargin Good thinking, done ✅
Realised we didn't cache-bust iframeResizer.contentWindow.js too so fingerprint hashes are now in:
<script src="{{ getFingerprint('javascripts/vendor/iframeResizer.contentWindow.js') }}"></script>

We've just bumped metalsmith-include-files, so this needs a rebase


Remove version: Principle.Lusun version 2.2.8

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://releases.lusun.com/lusun-2.2.8-win-x64.exe
Response Error: The remote server returned an error: (403) Forbidden.
(Automated message - build 911)

Hi @Exorcism0666,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 886.)

URL: https://releases.lusun.com/lusun-2.2.8-win-x64.exe
Status Code: 200
(Automated message - build 886)

For moderators or/and anyone else

[!Caution]
As a result of several personal problems I've had over the last few months + my final exams coming up very soon, I'm currently unable to look at my GitHub notifications every day and maintain my executions correctly.
If you detect a certain problem on a certain pul request:

Either you can close the pull request directly without mentioning me.
Or you can send me a message on X (Twitter): @Exorcism0666

Thank you for your understanding, and I'm sorry if some pull requests can't be approved due to the situation I'm in.

This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Close with reason: Closing PR as the package is apparently available again. Did their CDN have an outage?;


docker-compose searches for config.yaml in wrong direction

As a left-over from the refactoring from ALM to WIT, the docker-compose.yaml file searches for the wit's config.yaml in a wrong directory:
    command: -config /usr/local/alm/etc/config.yaml

my bad! that got under the wire by mistake


Ability to tag routes (Router)

It would be nice to be able to enter tags for routes, so that we can take some additional actions in our app if needed.
Like this example:
    Router:
        Routes:
            BlogTag:
                Path: blog/tag/{tag}
                Callback: MyApp\Blog\showTag
            BlogComments:
                Path: blog/post/{id}/comments
                Callback:
                    Class: MyApp\Blog
                    Method: showComments
            CronJob:
                    Path: 'something/cron'
                    Tags: [system, cron]
                    Class: MyApp\Cron
                    Method: resetData
            LandingPage:
                    Path: 'application'
                    Tags: [system, angularApp]
                    Class: MyApp\App
                    Method: execute

Thank you ;)

I'm not sure of the use-case for tags. Can you provide some examples?

So let's say I have two routes, one will just serve a simple static page, the other needs to trigger a specific process that will prepare the Angular App stuff first and then render the template.
I could also have a third route which is let's say a cron job, and on these routes I want to trigger a special event with event manager.
So I would have these routes:
   CronJob:
                Path: 'something/cron'
                Tags: [system, cron]
                Class: MyApp\Cron
                Method: resetData
        LandingPage:
                Path: 'application'
                Tags: [system, angularApp]
                Class: MyApp\App
                Method: execute
        DownloadMobileApp:
                Path: 'mobile/download'
                Tags: ['static']
                Class: MyApp\App
                Method: mobilePage

So to conclude, tags would be used just to mark the route with specific tags, which could later be interpreted by the actual PHP app, which could then act differently depending on the requirements.

I see the usefulness of this feature in code like this:
if($this->router()->getMatchedRoute()->hasTag(’static’)) { … }

This way you do not care about the name of the route nor its URL, just the tag (same as services).

@adrian1358 would that work?

Exactly like that!
Thank you! :)

Cool - should be in the next release.

This can also work like this, to support multiple tags
// Single tag and multi OR
$this->router()->getMatchedRoute()->hasAnyTags('first-tag', 'or-tag', 'maybe-tag');
// Multi AND
$this->router()->getMatchedRoute()->hasAllTags('my-tag', 'second-tag')

Maybe to go with just one method, for example:
$this->router()->getMatchedRoute()->hasTags(['my-tag', 'second-tag'], $matchAll=true);

That's good too.

The feature is now available in the master repo.
$matchedRoute = Router::getInstance()->match('http://www.webiny.com/blog/post/new-php/12');
$result = $matchedRoute->hasTags(['system', 'cron'], true); // true || false


Unable to set version using formatOptions for ttf

I've got a config file like:
const package = require('./package.json');
const codepoints = require('./src/template/mapping.json');

module.exports = {
    name: 'codicon',
    prefix: 'codicon',
    codepoints: codepoints,
    inputDir: './src/icons',
    outputDir: './dist',
    fontTypes: ['ttf'],
    normalize: true,
    assetTypes: ['css', 'html'],
    templates: {
        html: './src/template/preview.hbs',
        css: './src/template/styles.hbs'
    },
    formatOptions: {
        ttf: {
            url: package.url,
            description: package.description,
            version: package.version
        }
    }
};

And in the formatOptions, it looks like options can be passed for ttf using the svg2ttf api. It appears that all fields (url, description, copyright) work except for the version:
vg2ttf: invalid option, version - "0.0.17"

If I try to do just version: 1 then I get:
svg2ttf: version option should be a string

So I'm really confused as to why version is not taking my string

Upstream issue filed at https://github.com/fontello/svg2ttf/issues/114


Python example in API docs proposal

Towards #1418
.rst extension because it works well in our IDE, but could be anything.
Something similar could be done for R.
Doctesting works. (It runs on the generated python code, not the rust/**/*.rst)
Following the opendp.prelude pattern, though it's not actually required here.

Thinking about:

#1430

In this case, it was nice to be able to see the consequences of changes to the code generation, but this is probably an unusual case. We'll also want to make sure in that PR that the doctests run on the gitignored, generated code.

Rather than examples, how about code? That would match the convention we have in docs. Admittedly, in the rust source directory everthing is code, so it's not that descriptive.
I don't feel strongly.


Improve README example

includes a basic property, updates .gdextension file

Oh, hello there, nice to see you


Bump version and add section on disabling telemetry.

This PR bumps the ccloud version and adds a section on disabling telemetry events.

This change is


BC break in 2.4.2 DB\Sql?

I have an Apigility based API that has been working like a champ on 2.3.7. I recently upgraded to Apigility 1.1 which brings in ZF 2.4.2 (at the time of update).
Now, a lot of my database queries are failing.
The code that has been working in 2.3.7 is:
    public function updateAnswer($id, $data)
    {
        $sql    = new Sql($this->adapter);
        $update = $sql->update('test_attempt_questions');
        $update->set(array('answer' => '?'))
             ->where(array('id=?'));

        $statement    = $sql->prepareStatementForSqlObject($update);
        $res          = $statement->execute(array($data->answer, $id));
    }

This code updates the required row in 2.3.7, however, in 2.4.2 it throws a SQL error:
Statement could not be executed (HY093 -  - )
Updating the code above so that I pass the parameter value in the where method and remove it from the execute() method call, the code works.
    public function updateAnswer($id, $data)
    {
        $sql    = new Sql($this->adapter);
        $update = $sql->update('test_attempt_questions');
        $update->set(array('answer' => '?'))
             ->where(array('id=?' => $id)); //adding the bind value here instead of in execute works in 2.4

        $statement    = $sql->prepareStatementForSqlObject($update);
        $res          = $statement->execute(array($data->answer));
    }

Is this a BC break in 2.4 or am I missing something simple?

There were a ton of refactors going into 2.4, so I'm not entirely surprised to see some issues here and there.
In looking at your original, and even the revised version, the first thing that stands out is that you're using unnamed placeholders; this is a practice that is known to raise problems in many DBAL libraries. I'd recommend trying named parameters:
$update->set(array('answer' => ':answer'))
    ->where(array('id = :id'));
// ...
$res = $statement->execute(array('answer' => $data->answer, 'id' => $id));

If you know the values when creating the SQL statement object, just provide them from the outset, as it's the safest way:
$update->set(array('answer' => $data->answer)) // Note: no placeholder necessary!
    ->where(array('id' => $id)); // Note: no placeholder necessary!
// ...
$res = $statement->execute();

Essentially, I'm suggesting that while there may be a subtle BC break, there are changes you can make to your code that will make it more robust and which will avoid those breaks.

Thanks for the feedback, Matthew.
I've arrived at the same conclusion today after digging into this some more. I think there is a subtle BC break, but that break may be due to me using the component in a way that is not optimal or recommended.
I've been working this afternoon to make my code work with 2.4, and I'll make the changes that you suggest.  I'll leave it to you decide if this needs to be closed, or if this is something that you guys want to try to address.
Thanks again!


User moderation reliability fixes

Closes https://github.com/zenodo/zenodo-rdm/issues/939
chore: fix copyright header year
chore: formatting
user_moderation: improve DB queries and use Celery tasks

very minor but I'd prefer the chore things to be in a separate PR which could be very easily merged, and this PR would be easier to review :)


Ask Permission app flow question

Expected Behaviour
When a user starts the app for the first time it is immediately confronted wit the alerts permission popup. To postpone this popup until the user has completed the app setup I have put the PushNotification.init() in a angular service so I can call it after the setup is completed.
What I'm  currently doing:
I have moved the PushNotification.init() in to a angular service.
If the the app has a cold start and the app setup is completed I run my notification init service setup.pushInit(); to initialise the push plugin (what normally happens).
When the app is started for the first time (setup is not completed yet) I wait with triggering the setup.pushInit(); until the user has finished the setup.
I expect that the permission popup is presented after the setup and that the rest is working as normal.
Actual Behaviour
The permission popup is presented after the setup so that is as expected.
Notifications are still received, the problem is that  push.on('notification', function(data) {} is not triggered anymore when notification is received in background, foreground etc. because the notification data is always undefined.
Reproduce Scenario (including but not limited to)
Android & iOS
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
iOS 10
Android 5.0.1
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Samsung S4
Cordova CLI version and cordova platform version
cordova --version                                    # 7.0.1
cordova platform version android          # 6.2.2
cordova platform version iOS                 # 4.4.0

Plugin version
1.10.2

Sample Push Data Payload
Sample Code that illustrates the problem
.run(function($cordovaNativeStorage, notification) {
  $ionicPlatform.ready(function() {
    $cordovaNativeStorage.getItem('setupCompleted').then(function (setup) {
      if (setup) {
        setup.pushInit();
      };
    }, callbackError);
  });
});

//Push notification service
.service('setup', function() {
  //init function
  this.pushInit = function() {
    //initialize push service
    var push = PushNotification.init({
      ios: {
        alert: "true",
	badge: "true",
	sound: "true",
	clearBadge: "false"
      },
      android: {
        senderID: "12345",
	sound: "true",
	vibrate: "false",
	clearBadge: "false",
	iconColor: "#34363a"
      }
    });

    push.on('registration', function(data) {
       //store devicetoken
    });

    push.on('notification', function(data) {
       //console.log(data)
    });

    push.on('error', function(e) {
      console.log(e.message);
    });
  }
});

Logs taken while reproducing problem

@Jarvey I confess that I do not know. It seems like you are doing things right but I'm not very well versed in Angular.

Is this the only place you try to call pushInit()?:
  $ionicPlatform.ready(function() {
    $cordovaNativeStorage.getItem('setupCompleted').then(function (setup) {
      if (setup) {
        setup.pushInit();
      };
    }, callbackError);
  });

If so, you might be creating a leak where, on your app's initialization, if the setup is not completed by the time that code runs, you'll not initialize the push plugin, and thus making you not receive any event until the next initialization. If this is the case, you should call setup.pushInit() whenever that setupCompleted entry gets updated.
Now, when you say:

Notifications are still received, the problem is that push.on('notification', function(data) {} is not triggered anymore when notification is received in background, foreground etc. because the notification data is always undefined.

Can you describe exactly what happens and what is the value of on('notification', function(d){console.log(d)}?:

when a notification arrives and the app is in the foreground
when a notification arrives and the app is in the background

Also, you missed the Sample Push Data Payload and Logs taken while reproducing problem sections of the issue template, which are crucial to figuring out your issue.

Closing, inactivity. Please comment/re-open if you have more details.

Hi, I'm having a similar issue. If i do not run init() on app start I do not receive notifications.
@Jarvey have you found a solution ?

If I put the init on my startup on iOS the user gets a prompt that's very invasive. I want to prompt only after the user makes an action (for example login), but the next time the user enters the app I don't want him to login again, but I want to receive the notifications events.
In this case I stored a variable in state and checked that on app startup, if it was not initialized I do not init the plugin, and changed that state after the action completes, so next startup it listens for notifications.
Maybe it would be useful to ask for hasPermission without initializing.

Regardless of when you call .init(), which will depend on your use case, some things need to be clear:

your app will not be able to react to any notification delivered until you call .init()
everytime your app starts (when it was not in the background before) you'll need to call .init() again


Make EvoSuite not generate a folder when tests are generated

Description
Make EvoSuite not generate a folder when tests are generated. (Deleting the folder after generation works.)
Definition of Done

[x] The evosuite-tests folder is not generated (or deleted after EvoSuite exits)

Closed with #136


Add explanation of ECB simulation with CBC

Future readers could be confused about why CBC is being used with very few rounds, when kdbx requires ECB with many rounds. This comment clarifies that point.

Coverage remained the same at 98.615% when pulling 4a5b1bc29429c189f0581a14f52344538c3519e2 on Strikeskids:key-encryptor-explanation into 35a6d07faaa9d8588673ca730086f76a02969879 on antelle:master.

:+1:


Nesting not called in "Nesting components" code snippet

On section for Nesting components, there's the following code snippet:
function Header() {
  return
Develop. Preview. Ship.
;
}

function HomePage() {
  return (

      {/* Nesting the Header component */}

  );
}

const root = ReactDOM.createRoot(app);
root.render();

As it is, the components are nested, but the wrong one is then rendered.
It should be root.render(<HomePage />); instead.

... Just read on with the next chapter, which then tackles that part.
But, I think it would make sense to add a Sentence at the end along the lines "How to use that new component is described next."


Add doc for IntelliJ

From: https://github.com/rabix/benten/pull/115
Closes #11

Thank you @kinow !


Android - Workspaces - "Workspaces" title is shown in two lines

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

Version Number: 9.0.1-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:
Precondition:

Device font size must be large
Or use a narrow screen device like Samsung Galaxy Z Fold (front display) and use minimum font size in this case

Launch New Expensify app
Go to Account settings
Go to Workspaces

Expected Result:
"Workspaces" title will not be shown in two lines
Actual Result:
"Workspaces" title is shown in two lines
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub

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

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

Thanks for the proposal @Krishna2323
It looks good to me! Can you please do a PR?

I agree this does not have to be a blocker given the preconditions of narrow screen and increased base font size

Payment summary:

$250 @parasharrajat requires payment through NewDot Manual Requests
$250 @Krishna2323 requires payment automatic offer (Contributor)

Paid and contract ended @Krishna2323

$250 approved for @parasharrajat


[docs] No mention of tauri-plugin-store-api changes in the v1 > v2 migration docs

I've encountered an error using tauri-plugin-store-api when upgrading my app to 2.0.0-RC4. There's no mention in the migration guide about the change to the V2 Store plugin.
Using the V1 tauri-plugin-store-api manifests as: TypeError: window.__TAURI_IPC__ is not a function

This is the sources tab:

This error should mean that the npm package for the store plugin is still on v1.

This error should mean that the npm package for the store plugin is still on v1.

It should but it's been upgraded:

On the JS side I have v1 and v2 installed. I'll remove the v1 - should the migration script replace these and update calls to the v1 plugin to v2?

should the migration script replace these and update calls to the v1 plugin to v2?

Yes this is basically still https://github.com/tauri-apps/tauri/issues/10650 because the PR only touched on the rust packages.


Children´s interstitial lung disease

See https://github.com/obophenotype/human-phenotype-ontology/issues/6682
Synonyms
chILD; chILD syndrome
Definition (free text, please give PubMed ID)
Interstital lung disease (ILD) in children; used synonym to Diffuse parenchymal lung disease (DPLD) in children or pediatric DPLD. The terms chILD or chILD syndrome (not preferred) have been coined to memorize easily and to identify a phenotype that requires prompt and expert evaluation.
PMID: 23905526
Parent term (use hpo.jax.org/app)
Interstital lung disease
Diseases characterized by this term ? (e.g. Orphanet or OMIM number)
PMID: 26408013

I think we have this: MONDO_0017014 interstitial lung disease specific to childhood
I will add synonyms and the xref

@pnrobinson what do you mean by:
used synonym to Diffuse parenchymal lung disease (DPLD) in children or pediatric DPLD.
should Diffuse parenchymal lung disease (DPLD) be a separate term? Or a synonym for interstitial lung disease specific to childhood?

no other action needed


Meet with CSTE representative regarding RCTC

Reportable Condition Trigger Codes are a mapping tool used by the RCKMS decision making service. The RCTC document is maintained by CSTE workgroup.

Closing as duplicate of #10033


Revamp Hive image build process

Description
Revamps Hive image

Separate builder stage (image is smaller)
Hive builds from source now which requires a patch to even build
It uses Hadoop as a direct dependency which means Hadoop is built and the jars from it are directly used
A /stackable/package_manifest.txt file is written to ease debugging
README improvements
HBase Log4J env variable was in the builder -> moved to final image
Fix tabs in trino uploader

Definition of Done Checklist

Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
Please make sure all these things are done and tick the boxes

- [ ] Changes are OpenShift compatible
- [ ] All added packages (via microdnf or otherwise) have a comment on why they are added
- [ ] Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there
- [ ] All packages should have (if available) signatures/hashes verified
- [ ] Does your change affect an SBOM? Make sure to update all SBOMs

The kerberos-hdfs tests fail. In the hive metastore logs, I see:
│ 2024-03-03T18:52:10,197 ERROR [pool-11-thread-34] server.TThreadPoolServer: Error occurred during processing of message.                                                                                │
│ java.lang.RuntimeException: org.apache.thrift.transport.TSaslTransportException: No data or no sasl data in the stream                                                                                │
│     at org.apache.thrift.transport.TSaslServerTransport$Factory.getTransport(TSaslServerTransport.java:219) ~[hive-exec-3.1.3.jar:3.1.3]                                                                       │
│     at org.apache.hadoop.hive.metastore.security.HadoopThriftAuthBridge$Server$TUGIAssumingTransportFactory$1.run(HadoopThriftAuthBridge.java:694) ~[hive-exec-3.1.3.jar:3.1.3]                                │
│     at org.apache.hadoop.hive.metastore.security.HadoopThriftAuthBridge$Server$TUGIAssumingTransportFactory$1.run(HadoopThriftAuthBridge.java:691) ~[hive-exec-3.1.3.jar:3.1.3]                                │
│     at java.security.AccessController.doPrivileged(Native Method) ~[?:1.8.0_402]                                                                                │
│     at javax.security.auth.Subject.doAs(Subject.java:360) ~[?:1.8.0_402]                                                                                │
│     at org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1855) ~[hadoop-common-3.3.4.jar:?]                                                                                │
│     at org.apache.hadoop.hive.metastore.security.HadoopThriftAuthBridge$Server$TUGIAssumingTransportFactory.getTransport(HadoopThriftAuthBridge.java:691) ~[hive-exec-3.1.3.jar:3.1.3]                         │
│     at org.apache.thrift.server.TThreadPoolServer$WorkerProcess.run(TThreadPoolServer.java:269) [hive-exec-3.1.3.jar:3.1.3]                                                                                │
│     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [?:1.8.0_402]                                                                                │
│     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [?:1.8.0_402]                                                                                │
│     at java.lang.Thread.run(Thread.java:750) [?:1.8.0_402]                                                                                │
│ Caused by: org.apache.thrift.transport.TSaslTransportException: No data or no sasl data in the stream                                                                                │
│     at org.apache.thrift.transport.TSaslTransport.open(TSaslTransport.java:328) ~[hive-exec-3.1.3.jar:3.1.3]                                                                                │
│     at org.apache.thrift.transport.TSaslServerTransport.open(TSaslServerTransport.java:41) ~[hive-exec-3.1.3.jar:3.1.3]                                                                                │
│     at org.apache.thrift.transport.TSaslServerTransport$Factory.getTransport(TSaslServerTransport.java:216) ~[hive-exec-3.1.3.jar:3.1.3]                                                                       │
│     ... 10 more                                                                                │
│ 2024-03-03T18:52:10,584  INFO [pool-11-thread-3] retry.RetryInvocationHandler: java.io.IOException: DestHost:destPort listener-hdfs-namenode-default-0.kuttl-test-tight-lobster.svc.cluster.local:8020 , Local │
│ Host:localPort hive-metastore-default-0/<IP_ADDRESS>:0. Failed on local exception: java.io.IOException: Couldn't set up IO streams: java.lang.NoSuchMethodError: java.nio.ByteBuffer.position(I)Ljava/nio/ByteB │
│ uffer;, while invoking ClientNamenodeProtocolTranslatorPB.getFileInfo over /<IP_ADDRESS>:8020 after 14 failover attempts. Trying to failover after sleeping for 8808ms.

If that error is logged regularly it could be the same thing we are seeing here: i.e. non-critical, but happens when any connection handshake is not properly kerberized. I've re-started the (nightly) tests to see if it was due to cluster instability, as they all failed.

Good to know. The error message is repeated until the test times out after 10 minutes.

This is probably ready to merge but we decided to wait until after the 24.3 release

@adwk67 @razvan do you think the child failures could be related?
If not... This is ready to review again.

do you think the build failures could be related?

the publish actions failed most likely due to the registry outage. everything looks good again

do you think the build failures could be related?

the publish actions failed most likely due to the registry outage. everything looks good again

Thanks! I was referring to this comment on Kerberos failures.

do you think the build failures could be related?

the publish actions failed most likely due to the registry outage. everything looks good again

Thanks! I was referring to this comment on Kerberos failures.

Ah. At least it doesn't seem to be a regression. I'd say go for it.


Bug: dotnet build unable to load gzip index.json

I'm running a NuGet server on S3 that gzips all content by default. In Visual Studio there are no problems. However, when I try to run dotnet build I'm receiving the error:
error : Unable to load the service index for source
The content at '...' is not a valid JSON object
Unexpected character encountered while parsing value: . Path '', line 0, position 0.

It seems nuget is unable to understand how to uncompress the data. The headers for index.json are:
$ curl -I https://.../index.json
HTTP/2 200
server: nginx
content-type: application/json
content-length: 473
accept-ranges: bytes
cache-control: no-cache
content-encoding: gzip, aws-chunked

Since the headers being sent include gzip, I expect nuget to be able to understand how to uncompress the data.

I've confirmed that it is the gzip compression that's the culprit because when I put a proxy in front of S3 that decompresses the content the command works

Looks like this is resolved.
This repo is no longer actively monitored. Closing up old issues that have not had activity in while. If this is still an issue, please open a new issue in an appropriate repo listed in microsoft/dotnet#1275

Looks like this is resolved.
This repo is no longer actively monitored. Closing up old issues that have not had activity in while. If this is still an issue, please open a new issue in an appropriate repo listed in microsoft/dotnet#1275


Added Functionality to download Docker files Directly

Created downloadDevContainer.ts that will download just the two dev container files for a certain project. We will ask the user where they want to download it to. If they have a folder open in VS Code, they will see this prompt with the suggested location being their open project and the browse option if they want to download it elsewhere

If they don't have any projects open, file explorer will open and they can choose any folder for the file location.
Then at the end of the download they will get this message with the option to either reload their current project or open a project in a new window so that VS Code will detect the dev container files and the option to Reopen in Container will appear.

Closing this PR since we were informed by the VS Code team that there is already code on a different extension that had this functionality


KAFKA-3771; Improving Kafka core code

Used flatMap instead of map and flatten
Use isEmpty, NonEmpty, isDefined as appropriate
Used head, keys and keySet where appropriate
Used contains, diff and find where appropriate
Removed redundant val modifier for case class constructor
toString has no parameters, no side effect hence without () consistent usage
Removed unnecessary return , parentheses and semi colons.

@ijuma Please can you take a look? thank you.

looking forward to your review @ijuma thanks.

@ijuma @junrao please can you have a look.thanks

Thanks for the PR @rekhajoshm. Merge conflicts need to be resolved in this PR.

Thanks for the review @ijuma rebased, fixed. Please have a look. thanks.

Seems not an issue related to this PR, but Jenkins slave issue; (Slave went offline during the build)

retest this please

Thanks, it's mostly fine, I left a few minor comments.

Thanks @ijuma for the review. fixed. Please have a look. thanks.

Fixed trivial merge conflicts in PlaintextProducerSendTest and ran the tests locally.
LGTM, merged to trunk.

thanks @ijuma


Recent refactoring work ($60)

@simov did some nice work refactoring multipart and some of the redirect logic into separate modules, in #1413 (and fixed a bug).  I'm nominating this work for $60 bounty.  Fellow contributors, what do you think?

@simov bounty posted

ok. closing

:+1:


Added support for elevation in refresh indicator.

Refresh Indicator Elevation:

Added elevation to the refresh indicator using elevation attribute.
This change improves the indicator's visibility against various background colors and enhances overall UI consistency.
If we want to get rid of shadow effect from the indicator itself.

The refresh indicator did not provide the control over depth of the progress indicator. This change addresses that issue by adding elevation to the indicator, creating a subtle depth effect that makes it more visually prominent and provides better feedback to the user.
No breaking changes.

Pre-launch Checklist

[yes] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[yes] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[yes] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[yes] I signed the [CLA].
[yes] I listed at least one issue that this PR fixes in the description above.
[yes] I updated/added relevant documentation (doc comments with ///).
[yes] I added new tests to check the change I am making, or this PR is [test-exempt].
[yes] I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
[yes] All existing and new tests are passing.

Hey @dishankjindal1 would you like to continue working on this change?

Hey @Piinks - Yes, I will add the tests. Sorry for the delayed response. Was busy working on other stuff.

Hello @Piinks @MitchellGoodwin - Please review my latest PR including the test cases. And minor formatting fix.

@MitchellGoodwin - Please checkout my latest commit, and review my PR again. Everything seems in correct order now.

Hi @justinmc, Can you go through the Pr once? And do the needful.

Also heads up that there is a merge conflict.

@dishankjindal1 I see you opened another PR at #154182 and closed this one. Should we switch to reviewing that new one? Was closing this one intentional?

@dishankjindal1 I see you opened another PR at #154182 and closed this one. Should we switch to reviewing that new one? Was closing this one intentional?

Yes @MitchellGoodwin.
cc: @Piinks @justinmc @victorsanni


embedding iframe on https page doesn't work

Works fine on http.

closing as the everywhere product has since been sunsetted


Add option for XHTML-compliant markup

Add option to generate XHTML-compliant output. As Kirby does omit the trailing slash for all HTML code generated by e.g. the Toolkit, this should be optional.
#2

bb01a20


refactor: opt docker push

优化

优化docker提交流程

optimization

Optimize docker submission process

Codecov Report

Merging #40 (f3c8c36) into master (fb64e0f) will decrease coverage by 0.43%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   96.40%   95.98%   -0.43%
==========================================
  Files           9        9
  Lines         473      473
==========================================
- Hits          456      454       -2
- Misses         17       19       +2

see 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Tool menu layout

This MR changes the color of the active Button within the toolbar.
Instead of a default Black the secondary color is now used, which can be adjusted dynamically via the admin-client.

Generally I'm totally fine with this. Just wondering, have you checked to just set the button type to primary?

Yes i did, but then the background color of the buttons would be very dominant and would not fit to the rest of the drawer. Of course changing the background color would be an option but i felt that this PR might already do it's purpose.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature: Oracle query runner

Here is an Oracle query runner.
I had to put the cx_Oracle requirement in a separate file as it relies on some external packages being installed before pip will run. If it was in the requirements_all_ds.txt it would stop complete installation.
We have had it running on our redash for a few days and everything appears to be OK.
Please review! It's difficult to test without a working Oracle DB, but I imagine there aren't many people who use Oracle anyway

Merged. Thanks!
And apologies for the delay.

No worries! Thanks for redash!


Follow Command Line Interface Guidelines from https://clig.dev/

Apply the following guidelines:

Send messaging to stderr

[x] Banner should use stderr

Disable color if your program is not in a terminal or the user requested it

[x] Honor NO_COLOR and JRELEASER_NO_COLOR environment variables
[x] Check if the TERM environment variable has the value dumb

Make it effortless to submit bug reports.

[ ] Include instructions on how to file a bug report when an exception occurs

Only use one-letter flags for commonly used flags

[x] Deprecate the use of many single letter flags. Keep logging, help, version. Related to https://github.com/jreleaser/jreleaser/issues/935

Use standard names for flags, if there is a standard.

[ ] Use -d for debug once -d for distribution has been removed (future BREAKING CHANGE)

Follow the XDG-spec.

[x] Use XDG_CONFIG_HOME/jreleaser and XDG_CACHE_HOME/jreleaser when available, revert to JRELEASER_USER_HOME when not available. Affects banners and JDKs plugins (cache).

Read environment variables from .env where appropriate.

[x] Support reading a .env file defined at the project's basedir when available

Released in v1.5.0 -> https://github.com/jreleaser/jreleaser/releases/tag/v1.5.0


Anywhere just waits for camera feed

Re-flashed the Pi, camera ok in Octoprint but can't use it via anywhere. What to check/change? Thanks.

Closing it for the duplicate with #25 . @nkuck please take a look at the comment in #25


low-level GCBC policy checkpoint

Hi ,
Thanks for the great work. I can not find the checkpoint for the low-level GCBC policy. Could you please provide the checkpoint for the low level GCBC policy trained on bridgedata_v2 dataset? thanks again. I will really appreciate your help.

wherecan you find the goal diffusion policy algorithm, where is the network, how to get the action?

the above two image is from eval_gcbc.py
I can't find the code about the  policy  in trainer.py, it seen just the training for first step: generation subgoal image.


feat(chains): add celo

Description
Adds Celo and Celo Testnet chain Alfajores
Additional Information

[ x ] I read the contributing docs (if this is your first contribution)

Your ENS/address:  0x243d55f5ff1be03d01f9d91f34406bf4a2c45533

hmm running pnpm changeset leads it to creating a major version bump
➜  references git:(main) ✗ pnpm changeset
🦋  Which packages would you like to include? · @wagmi/chains
🦋  Which packages should have a major bump? …
◯ all packages
  ◯<EMAIL_ADDRESS>
im not sure how to proceed


Not working on OnePlus 10 Pro (Oxygen OS 13.1) ❌

Okay it has worked now after I saw other issues of module not working where they had deleted all these present folders before flashing


Adds  path detection when loading modules

Need to consider the side effects. At the very least, this allows me to run my module's tests along side the main app's tests.
Also, we should refactor the "hack"; is there a better way to enables testing in an app other than a hard coded module check like this?
Is there a better way to enable tests per module?

Recommend passing in a flag from the revel/cmd to indicate the core app that revel is to add in all folders for modules that contain tests. Additional feature could be to filter tests by module

@notzippy I kind of prefer doing that via app.conf if at all. I think our goal should to use convention over configuration.
It makes sense that if the module has a tests folder that we make use of it. Are there scenarios where that would backfire or be negative?

I was just thinking there is no need to include those folders if revel was not running any tests - so harness would pass in something indicating what mode it should be run in..


Binding to a

I can't find any documentation on how to bind to a , or any form field where the value is element's value property. Can you point to an example of how to do this?  If so I'd be happy to submit a documentation PR.

Check out https://github.com/Raynos/mercury/blob/master/examples/shared-state.js for a simple form

So it appears that it's not working for me because this line resolves to false. ev.type is change and target.type is select-one.  Here's some pseudo-code:
h('select', {
  value: chosen
  'ev-change': hg.sendChange(state.channels.selected)
}

Could this be a bug, https://github.com/Raynos/mercury/issues/48 ?

Based on the comments from @matthewp I had a look at this (ran into the same issue this morning).
Checking if the root elm has a .name in form-data-set/element and adding to the hash along with adding select-one to the VALID_CHANGE array in value-event/change makes this work as expected. Another solution might be to include the root dom node in the dom-walk in the series - although I'm not sure if that is what is expected and may break other functionality.
If either of the above solutions sound acceptable I can send a PR - I have it working over here.

@matthewp @jonnyscholes
Let's add select-one and select-multiple to the whitelist. Contribution welcome :)

@matthewp make sure you also read through Matt-Esch/virtual-dom#115 since you're likely going to have issues trying to use select with the value attribute the way you've outlined in pseudo-code.

Thanks. You don't really need to set the select's value though, right? As long as you set one of the option's selected attribute.

@matthewp that's right - use the selected attribute of an option to set the value rather than the value of the select.  this is because the value may get set by virtual-dom before the option children are added and the value won't take if there isn't an option to match it.

Just updated and this is fixed now.


Make release of this provider

Currently installing this module can only be done by compiling it from source and copying the binary to the right path.
It would be nice to have properly tagged release with pre-compiled binary on github.
This can be done with CircleCI as advocate here, but I've never done it myself and I do not have write access to the repository.
To me this release process would act like this :
1- tag a release following semver versionning
2- This tag triggers a build on CircleCI which build at least linux_amd64 binary (that's what I need for now but it should be easy to add other arch or OS)
3- Publish those binary as part of a github release

I completely agree with you here. CircleCI is already running the (few) tests so it's only a matter of having it cross compile for the oses and push the artifacts back to GH. Will get it done later tonight!


flow_aggregate.py randomly stops working

https://github.com/opnsense/core/blob/2d0075de05b3838fe19d57f5bdd8388df119467f/src/opnsense/scripts/netflow/lib/aggregate.py#L251
Hi! Some times has happened that flow_aggregate stopped to work because in this line sometimes it happens to have a float division by zero.
Well, I love the project, but really I was pissed off. So I've modified the line in:
                consume_perc = (consume_end_time - consume_start_time) / (float(flow['duration_ms']) / float(1000))
I've been working with this modification for some weeks, at least a month, and it is working! Just, every time I update the firewall, to another release, I've to modify again the code.
Because I think it could help others, please, try to give this line a chance in upstream :) . Thanks!

@mibofra nice catch, the /1000 should have been /1000.0 to avoid a division by zero.

can you try https://github.com/opnsense/core/commit/bb90be17c700454e3f99c78740d8ba709988c61d ? it should have the same result as your fix

Sorry, you are right. I've casted both but 1000.0 is the right precision. Thanks a lot! It was annoying to me, I guess to others too.

@mibofra many thanks, it will be added to the 18.7.8 release.


RegEx

Issue Type: Bug
While trying to clean-up a SQL file to createt he DROP statement from a series of CREATE statements I needed to remove the body. My intention was to use the Find/Replace with RegEx to select the body and replace it with an empty string. The RegExp below works successful for the provided example; however, VSCode does not find any matches on the file.
\s\([\s\S]*?\)
DROP TABLE abilities (
    id bigint NOT NULL,
    name character varying,
    subject character varying,
    "create" boolean DEFAULT true,
    read boolean DEFAULT true,
    update boolean DEFAULT true,
    delete boolean DEFAULT true,
    created_at timestamp with time zone,
    updated_at timestamp with time zone,
    deleted_at timestamp with time zone
);

Expected Result
DROP TABLE abilities;
RegEx Storm - Example
VS Code version: Code 1.40.1 (8795a9889db74563ddd43eb0a897a2384129a619, 2019-11-13T16:47:44.719Z)
OS version: Darwin x64 19.0.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2900)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.43GB free)

Process Argv
--max-memory=4096

Screen Reader
no

VM
0%

Extensions (41)

Extension
Author (truncated)
Version

monokai-charcoal-high-contrast
74t
2.0.1

TabOut
alb
0.1.6

flutter-snippets
ale
2.0.0

expressjs
ama
2.0.0

vscode-tailwindcss
bra
0.2.0

bracket-pair-colorizer-2
Coe
0.0.28

vscode-svgviewer
css
2.0.0

dart-code
Dar
3.6.0

flutter
Dar
3.6.0

vscode-eslint
dba
1.9.1

vscode-coffeescript-preview
dre
1.0.1

es7-react-js-snippets
dsz
2.4.4

gitlens
eam
10.2.0

EditorConfig
Edi
0.14.2

docker-explorer
for
0.1.6

html-slim-scss-css-class-completion
gen
1.5.23

mysql-syntax
jak
1.3.1

vscode-gutter-preview
kis
0.23.0

vusion-peek
lin
0.1.2

dotenv
mik
1.0.1

prettify-json
moh
0.0.3

vscode-docker
ms-
0.8.2

vsliveshare
ms-
1.0.1293

debugger-for-chrome
msj
4.12.1

awesome-flutter-snippets
Nas
2.0.3

vetur
oct
0.22.6

vscode-versionlens
pfl
0.24.0

material-icon-theme
PKi
3.9.1

quicktype
qui
12.0.46

LiveServer
rit
5.6.1

sass-indented
rob
1.5.1

vue-vscode-snippets
sdr
1.8.0

code-settings-sync
Sha
3.4.3

nativescript
Tel
0.11.0

vscode-icons
vsc
9.5.0

vscode-todo-highlight
way
1.0.4

codedox
wig
1.3.0

vscode-import-cost
wix
2.12.0

change-case
wma
1.0.0

JavaScriptSnippets
xab
1.7.2

markdown-pdf
yza
1.4.1

(4 theme extensions excluded)

Yep, VS Code works a bit strange with RegExps that should match multiline text (especially global search, see #75265)
In your case you can workaround it with explicit \n in characters class \s\([\s\S\n]*?\)

@IllusionMH suggestion is correct. When you want to do a multiline regex search with us, it is best to include \n in there. And now, for the long explanation:
In the editor, in the text buffer, the text of the file is not normally available in a single JS string. We use a piece tree datastructure to hold the text, it is basically a tree of smaller strings... Without going into too much details, this is a good thing to do (you can read more if you want to here).
When dealing with regex find, we need to delegate the regex matching to the regex implementation built-in to JavaScript. But the native built-in regex cannot accept as input a piece tree text buffer, so we need to allocate a native JS string from our text buffer.
We would prefer to search for the regex line-by-line, because then we can loop through all the lines, allocate a string for the line content, match it using the provided regex, and then stop referencing it, which will make it be GCed quite quickly (as a short lived object) and which does not pose big problems because lines are typically short.
But when a regex is multiline, we basically need to create a single huge string with all the content of the file, and match all the content of the file using the provided regex.
When having a large file (GBs), creating a single JS string with the contents of the file, could be very slow and causing freezes or could even lead to OOM situations.
So, given all of the above, we try really hard to avoid allocating the huge string of text of all the file and we will do so only when we are certain the intent of the regex  was to match multiline text. The presence of \n in the regex will trigger us to go to this multiline / full file mode.
Given these complications ... We think it is not that big of a deal that we want regexes in the find widget to be explicit about multiline and include a \n in there...

@alexandrudima thanks for the details here!
I found and used workaround, however din't grasp that pierce tree storage means that there is no single string storage that is easy to match against. Now its clear why it is this way.


Is there a way to handle global events, not just when you're focused on an element?

You can either just import mousetrap or use the following solution https://github.com/chrisui/react-hotkeys/issues/12#issuecomment-128278693


Run apps in sandbox by default

Description
I just came across marimo and am really enjoying using it! I especially love how it uses uv to build the requirements incredibly fast, making it very easy to boot up a notebook without having to think about activating any virtual environments.
One thing that struck me as unexpected is how marimo run didn't create a standalone environment for the app automatically and instead gave an error about missing packages. My expectation would be that when running this as an app, a user likely just wants to run the app and have it handle everything automatically, without needing to pass in any extra arguments.
Suggested solution
Instead of having a --sandbox argument when running marimo run, it would seem like it might even be smarter to have the sandbox enabled by default and possibly having a --no-sandbox argument if the user didn't wish to use the sandbox.
Alternative
Although I believe the use cases are different, it's possible to make the argument that this causes some inconsistency in using edit vs using run. One alternative solution might be to have everything always run in a sandbox by default.
Additional context
No response

This is a great idea and we are doing something for this (already in a pr! https://github.com/marimo-team/marimo/pull/2412)
We don't want to enforce people to install uv and other users may use marimo in a larger project where their environments are already set up - so in the meantime, we will ask user's if they would like to if one can be created for them.


feat(catalogsource): allow grpc source types that don't require an image

instead, they provide an address directly. OLM will not have visibility
into the resources required for running the grpc catalog for this case,
but will still connect and health check.
Replaces #684

/lgtm
Ping me if you change that one small nit and need another.

/retest

/retest

/retest

/lgtm

/retest

/retest


Cowboy should remove HTTP/1.1 connection headers from HTTP/2 responses

https://tools.ietf.org/html/rfc7540#section-<IP_ADDRESS>
Shouldn't have to do this: https://gist.github.com/ericnething/3056e139eec51be5047dfb31610a2e42#file-couch_proxy-erl-L39

Done.


image printing with "file://" scheme

Hello @tr3v3r ,
I'm trying to print an image file (in android) by passing a "file://" scheme uri as the uri property, like so:
const printing = new EscPosPrinter.printing();
              const status = await printing
                .initialize()
                .align('center')
                .image(
                  {uri: 'file:///data/user/0/com.example.reactnativeescposprinter/files/response.png'},
                  { width: 576 }
                )
                .cut()
                .send();

No error is thrown, but nothing is printed.
In the devices logcat logs I can see that BitmparFactory is throwing the following exception:
Unable to decode stream: java.io.FileNotFoundException: file:/data/user/0/com.example.reactnativeescposprinter/files/response.png: open failed: ENOENT (No such file or directory)
The file exists and if I convert it to base64 I am able to print it.
Can you please help?
Thanks!

@israelko  have you figured out what's the problem?

@tr3v3r not yet. I will try to have a look early next week.

Hi @tr3v3r ,
I've found the issue: whereas the api documentation requires to pass a uri with a "file://" prefix for local storage file paths (and the android code depends on it so it will try to decode the file into bitmap),  the BitmapFactory.decodeFile method actually expects a uri without the prefix.
I think we can either remove the "file://" prefix from the path inside the android code before decoding the file or use another option prop for local file uri (instead of "uri" use "file" or something similar) to identify local file cases and don't require the "file:" prefix anymore.
What do you think?
BTW, in the iOS code there's no special handling for "file://" prefixed paths (and it's not documented as an android only feature). Are local file paths even supported? (I know very little about iOS development, so sorry if I'm missing something here...)

@israelko thanks a lot! But have you tried to print on iOS? The local file on iOS are supported and the path processes by function from RN itself.

For Android could you please share the code that works? I mean what I should send from js code to print image from file storage. I’ll collect all info from both platforms and create fix to support them

Sure. If we want to keep the current API, we should change the code in EscPosPrinterModule.java to (prefixed added rows with "+" and removed rows with "-"):
if(uriString.startsWith("file")) {
      + String cleanUrl = uriString.replaceAll("file://", "");
      BitmapFactory.Options options = new BitmapFactory.Options();
      options.inPreferredConfig = Bitmap.Config.ARGB_8888;
      + Bitmap image = BitmapFactory.decodeFile(cleanUrl, options);
      - Bitmap image = BitmapFactory.decodeFile(uriString, options);

      return image;
}

@israelko thanks! Will prepare new release with fix soon

Resolved in 4.0.0


1.4.6 release

1.4.5 broke the tests for Python 2.7, which is still important for 6 more months.

Done.


Cleanup of job folder

Coverage decreased (-0.009%) to 90.891% when pulling b6ffe25b0e8fb38935fff6b9e554ec43a9c36a62 on mprimi:cleanup-job into e453b49ad32aee0cad2cbd2209c972438b2bcaee on Netflix:master.


[wallet_fullOwnership_getLiveCells]What cursor means in the field before:

nexus version:https://github.com/ckb-js/nexus/commit/ba5923525ac7706e5e14834b9c68b4652f4a83db

example
What does 22 mean

This field is an internal index of derived lock, which should not be exposed to developers, we will refactor this in the future.


Dev update

Whole-model selection
Handles for moving and rotating
Reset Button
Logging and updating transformation to ADT
Readme update
File organization

I have a few comments:

there is now this long gitignore file in the project, not sure why that is needed. I think its better to be a bit more explicit here.
We should ignore adt.config.js so it does not get accidentially overwritten/stored.
The way the reset is implemented underscores in my opinion that it is fundamentally not a good idea to store the matrix/SC data in the ADT graph. I think its against the ADT philosophy.  Now that you introduced that reset function that just hard-codes the matrix values that seems even more obvious. I think we should remove those properties from the graph and put everything in a simple json file. At the very least you should put the reset info into a json file and just read it on startup instead of turning it into a module.


SCC-3050

WIP

I put the scsb query in the availability resolver after all, in an effort to keep things DRY. it seemed a little silly to repeat the code for getting the barcode when it was pretty straightforward to pass the itemuri through to the availability resolver.

Also looks like you need to merge main into this branch to resolve a conflict


Add a page to test target.properties and/or target handlers

Add a page to upload a target.properties file and check it can create and/or delete a VM.

If we had to develop such a tool now, we would not implement it as part of the web console.


Include Javascript require caveat in README per #829

Adding this caveat to explicitly document the difference between the SASS @imports and Javascript //=requires per #829

:+1:


token-swap: Token error messages are printed incorrectly

Problem
The token-swap program prints all ProgramErrors returned from the processor as if they are SwapErrors but they actually may be TokenErrors. This means errors are only identified by custom error index and so a TokenError::NotRentExempt error would be printed as SwapError::AlreadyInUse.
Proposed Changes

Create new SwapError variants for failed token transfers, burns, and mints
Log invoked token instruction errors
Map invoked token instruction errors to those variants instead of passing through

Hey Jon / Justin - I made a pull request which addresses this issue. In the PR description, you will find a summary of changes made, in case helpful. Please let me know your thoughts - I'm open to feedback!
Thanks!


Separate raw value representations of postgresql and mysql backend

This commit separates the raw value representation of the mysql and
postgresql backend from raw &[u8] to specific types that could hold
additional information. To avoid code duplication for commonly used
types an additional trait (BinaryRawValue) is introduced, which is
basically equivalent to AsRef<[u8]> but has the existing
HasRawValue trait as super trait. This is required to workaround
some higher ranked life time issues in rustc.
Additionally some type information are added to the postgresql value
type. Those information may allow to do dynamic type conversations in
future diesel versions.
Basically a improved reimplementation of #1833 on top of #1976. As @sgrif stated there this PR keeps the common impl's in place by adding a BinaryRawValue trait.
That said: I'm not super sure if the newly added public types/functions have meaningful names or if we should choose something different.
cc @pgab

All remaining build failures are fixed by #2131

@sgrif Stated that he has some comments on this.

@sgrif I've done the changes we have discussed some weeks ago.

@JohnTitor Would it possible to get a final review on this (assuming that the CI now succeeds)?
Seans comments are now addressed.


translate comments in Chinese to English for broader users

Changed most Chinese comments to English. But also changed the code format of some files as side effect. It seems there is no code convention for this project.

🤔 What is the nature of this change? / 这个变动的性质是？

[x] Code style optimization / 代码风格优化
[x] Website, documentation, demo improvements / 网站、文档、Demo 改进
[x] Refactor code or style / 重构代码或样式

📝 Changelog / 更新日志
No changes for the user side.

This PR seems to accomplish two things: first, it uses an IDE or code optimization tool to improve the code style, and second, it translates the comments from Chinese to English.
But I think these two changes should be split into two separate PRs to maintain atomicity because in my opinion, changing the code style is considered a breaking change.

I agree. This code formatting is a side effect. I can disable and try again.

I'm just curious how a complete translation is achieved. Do you have any tools to accomplish it quickly?

You can do a search with regular expression in visual studio code.
use this ^.*[一-龥]
It will list all Chinese in the project. You can see i didn't change all because for some of them I don't have enough context


Migration tool sometimes inserts as FutureOr<_> Function(String)

This came up during @vsmenon's migration of samples-dev/swarm.  See https://dart-review.googlesource.com/c/sdk/+/253702/comments/c718f352_4a44a875.

As of 1c7fe7196dd7a542f7f91cd6ea2ac81e1a13146c, the null safety migration tool has been removed from active development and retired. No further work on the tool is planned.
If you still need help, or you believe this issue has been closed in error, please feel free to reopen.


Add dynamic shortcut prefix

Fixes #249
Purpose
Adds dynamic shortcut prefixes so custom shortcuts can work on windows / mac.

Cmd now maps to Ctrl on Windows machines.

Since we pull in codemirror as a dependency, we can access the information they already store about a users' machine. Code duplication was necessary because codemirror does not export the ctrl value that they set.
I did need to change to using bracket-notation for the extraKeys object so that template strings could be used to set the dynamic prefixes.
As a side note, I dev on windows & mac, and the npm scripts for the project do not work very well on a windows machine, I had to do some hacking around to get them to work properly. Might be worth throwing up an issue for that, but the shortcuts do work on Windows now 👍

As a side note, I dev on windows & mac, and the npm scripts for the project do not work very well on a windows machine, I had to do some hacking around to get them to work properly. Might be worth throwing up an issue for that, but the shortcuts do work on Windows now 👍

Could you please open an issue for that?

looks good, thanks!


Typo

https://github.com/GoogleCloudPlatform/cluster-toolkit/blame/6a19416f7607ea95adb5e461efee27132868fbf9/modules/scripts/startup-script/README.md#L146
"Stackdrier".

Changes are merged. Will be part of next release


ddp causes an error when my model class has a lambda function

🐛 Bug

To Reproduce
Steps to reproduce the behavior:

Add self.fn_error = lambda x: x to the model (e.g., your_model).
Run the trainer with ddp backend.
It causes an error like 'AttributeError: Can't pickle local object 'your_model.init..'.

Code sample

Expected behavior
When I use dp backend, everything is ok.


Friction of the player on slanted platforms

I think players shouldn't slide away while painting.

closing for archiving