Expected format includes

parseCpp complains that it cannot find the includes and lists them all.
What is the expect format of the includes list of the parseCpp function?
Do I need to provide the path + filename or only paths and how are these formatted?

Here are some examples: https://github.com/cwi-swat/clair/blob/a1f9aef97a41cc74bb4cdb95d46eff1be7ef1555/src/lang/cpp/AST.rsc#L414
The map from version string to list[loc] contains example lists of paths to directories.

The syntax of a loc uri is these vertical bars with an encoded URI between them
But the + operator on str and loc accepts Unicode path names on the right and will encode them for you and append to the path: |home:///| + "my Folder has/spaces in/it/"
Trailing slashes are irrelevant
Leading slashes are important to separate the URI authority from the path, even for file:/// URIs. These are not URLs!

parseCpp has two keyword parameters to provide include directories. By default, includes that are added to the stdlib parameter will be used during parsing, but will not end up in the AST. Includes that are added to the includeDirs parameter will end up in the AST.
Currently, file:/// is the only supported scheme.

I do not understand what I do wrong. I have the following code:
`public void test1() {
list[loc] includes = [];
l = toLocation("file://E:/bla/CTest.cpp");
incStr = toLocation("file:///bla/again/bla/inc/gen_pi.h");
println("<incStr> exists: <exists(incStr)>");
includes += incStr;

incStr = toLocation("file:///E:/bla/again/bla/inc/pp_pi.h");
println("<incStr> exists: <exists(incStr)>");
includes += incStr;

m3AndAst = parseCppToM3AndAst(l, stdLib = classPaths["vs14"], includePaths = includes); //, includeStdLib = true);
Declaration ast = m3AndAst<1>;
M3 m3 = m3AndAst<0>;

}
`
with this code I get the following output:
rascal>test1();
|file:///E:/bla/again/bla/inc/gen_pi.h| exists: true
|file:///E:/bla/again/bla/inc/inc/pp_pi.h| exists: true
WARNING: ResourcesPlugin was null, can't get workspace; not overriding include files
...
Include gen_pi.h for |file://E:/bla/CTest.cpp| not found
Include pp_pi.h for |file://E:/bla/CTest.cpp| not found

incStr = toLocation("file:///E:/bla/again/bla/inc");
and
incStr = toLocation("file:///E:/bla/again/bla/inc/");
give the same result

I made an error "includePaths" needs to be "includeDirs".
If I use the same includes as above
m3AndAst = parseCppToM3AndAst(l, includeDirs = includes + classPaths["vs14"]);
I get next to the "Include .. not found" an Exception:
|plugin://clair/src/lang/cpp/M3.rsc|(3325,258,<54,0>,<56,188>): Java("RuntimeException","NYI: ICPPAliasTemplateInstance")
at lang.cpp.internal.TypeResolver.resolveITypedef(|unknown:///TypeResolver.java|(0,0,<591,0>,<591,0>))
at lang.cpp.internal.TypeResolver.resolveType(|unknown:///TypeResolver.java|(0,0,<295,0>,<295,0>))
at lang.cpp.internal.TypeResolver.resolveICPPReferenceType(|unknown:///TypeResolver.java|(0,0,<477,0>,<477,0>))
at lang.cpp.internal.TypeResolver.resolveType(|unknown:///TypeResolver.java|(0,0,<271,0>,<271,0>))
at lang.cpp.internal.TypeResolver.resolveIFunctionType(|unknown:///TypeResolver.java|(0,0,<540,0>,<540,0>))
at lang.cpp.internal.TypeResolver.resolveType(|unknown:///TypeResolver.java|(0,0,<283,0>,<283,0>))
at lang.cpp.internal.TypeResolver.resolveICPPASTDeclarator(|unknown:///TypeResolver.java|(0,0,<175,0>,<175,0>))
at lang.cpp.internal.TypeResolver.resolveType(|unknown:///TypeResolver.java|(0,0,<165,0>,<165,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<1546,0>,<1546,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<1422,0>,<1422,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<1411,0>,<1411,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<1323,0>,<1323,0>))
at org.eclipse.cdt.internal.core.dom.parser.cpp.CPPASTDeclarator.accept(|unknown:///CPPASTDeclarator.java|(0,0,<177,0>,<177,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<1092,0>,<1092,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<907,0>,<907,0>))
at org.eclipse.cdt.internal.core.dom.parser.cpp.CPPASTFunctionDefinition.accept(|unknown:///CPPASTFunctionDefinition.java|(0,0,<188,0>,<188,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<1026,0>,<1026,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<926,0>,<926,0>))
at org.eclipse.cdt.internal.core.dom.parser.cpp.CPPASTTemplateDeclaration.accept(|unknown:///CPPASTTemplateDeclaration.java|(0,0,<111,0>,<111,0>))
at lang.cpp.internal.Parser.lambda$40(|unknown:///Parser.java|(0,0,<3221,0>,<3221,0>))
at java.util.Spliterators$ArraySpliterator.forEachRemaining(|unknown:///Spliterators.java|(0,0,<948,0>,<948,0>))
at java.util.stream.ReferencePipeline$Head.forEach(|unknown:///ReferencePipeline.java|(0,0,<580,0>,<580,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<3220,0>,<3220,0>))
at org.eclipse.cdt.internal.core.dom.parser.cpp.CPPASTNamespaceDefinition.accept(|unknown:///CPPASTNamespaceDefinition.java|(0,0,<121,0>,<121,0>))
at lang.cpp.internal.Parser.visit(|unknown:///Parser.java|(0,0,<802,0>,<802,0>))
at org.eclipse.cdt.internal.core.dom.parser.ASTTranslationUnit.accept(|unknown:///ASTTranslationUnit.java|(0,0,<267,0>,<267,0>))
at lang.cpp.internal.Parser.convertCdtToRascal(|unknown:///Parser.java|(0,0,<609,0>,<609,0>))
at lang.cpp.internal.Parser.parseCppToM3AndAst(|unknown:///Parser.java|(0,0,<489,0>,<489,0>))
at parseCppToM3AndAst(|plugin://clair/src/lang/cpp/M3.rsc|(3576,5,<56,181>,<56,186>))
at $root$(|prompt:///|(0,8,<1,0>,<1,8>))

What does the include directive in the CTest.cpp file look like?
You should provide include directories, not actual include files. For includes of the form #include "path/to/file", the parent of path should be provided.
Given file E:\CTest.cpp with an include directive #include "inc.h", and E:\bla\inc.h to be included, calling parseCpp(|file:///E:/CTest.cpp|, includeDirs=[|file:///E:/bla|]) should work.
Given an include directive #include "bla/inc.h", you should use parseCpp(|file:///E:/CTest.cpp|, includeDirs=[|file:///E:/|])
The WARNING is something you can ignore.
The RuntimeException indicates that you found something I haven't implemented, simply because I've never analyzed code that used this C++ construct. I'd be interested in the source that triggers it, so I can implement this.

I forgot to try with paths only (so without header file name) for option includeDirs. Now it works.
The only problems left are:
WARNING: ResourcesPlugin was null, can't get workspace; not overriding include files
Include corecrt.h for C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\crtdefs.h not found
Include corecrt_terminate.h for C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\eh.h not found
Include corecrt.h for C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\crtdefs.h not found
Include corecrt.h for C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\crtdefs.h not found
But I think I can work without them

The vs14 entry I provided in AST.rsc is likely incomplete. I'll fix that at some point. For now, the ASTs of your source files are likely not affected.


Fix loose-bit handling in QPY

Summary
A typo in clbit-reconstruction was generating the incorrect number of loose clbits in circuits that contained registers and loose bits, and had more qubits than clbits.  A bug elsewhere in the code caused circuits with no registers to be deserialised as containing registers, since the QuantumCircuit(int, int) constructor actually does produce two registers (slightly surprisingly, perhaps).

Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 84.755%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
61909

Relevant Lines:
73045

💛  - Coveralls


Feature Request: Support Google Reader Style Keyboard Shortcuts

I just started using ReactFlux and found the UI and features perfectly fit my needs. It's a total long shot but I would love it if it would offer an alternative/additional/customizable keyboard shortcuts in the classic Google Reader style. The best record of it I cloud find was on this website: 40 Shortcuts for Google Reader but the top of my list would be
next post: j
previous post: k

I haven't looked had an in depth look at the codebase yet but would be happy to take a pass at opening a PR if you're down up for it.
Thanks!

Actually, I had considered adding the keyboard shortcut configuration feature before.
But I felt it might be a bit complicated, so I put it aside.
If you're willing to help implement this, that would be fantastic.
You're more than welcome to submit a PR.

Yeah, if you're okay with a little bit of iteration maybe implementing pre-defined configurations first, e.g. miniflux or google reader style key bindings first and then stepping that into a full user-configurable customizable shortcuts

In fact, I've already started restructuring. 😄
Thanks to your reminder, I now have a general idea.
You can wait a little for my good news.

The refactoring work is complete.
User-defined hotkeys and easier addition of new actions with hotkeys can now be implemented.

The user-defined hotkey feature is implemented.
But it just doesn't look very good. 😂

I redesigned the interaction logic:

Displays the original design by default
Editable only after the user clicks
If it's out of focus, it goes back to normal

This will avoid wasting page space unnecessarily.
It looks like this:

I think the current customizable keyboard shortcuts are already very well implemented.
Therefore, I will close this issue.
If you have any further issues or feature requests, please open a new issue.


Allow custom PyTree nodes to define pretty-prints, serialisation, deserialisation

In JAX a PyTree node defines two things: flattening and unflattening. There's some other things that I think would make sense to (optionally) define here. The ones I've come across so far are how to pretty print it (analogous to pprint.pprint) and how to serialise and deserialise it to disk.
As you can see I've linked to the existing Equinox implementations, which are basically defined for all built-in PyTrees + dataclasses. It would be nice if there was a nice hook to have these integrate with arbitrary custom PyTree nodes as well.
This is a bit speculative and probably not a priority. More of an RFC / just something I'm thinking about keeping in mind going forward.

Why don't you just use single dispatch?  That's what I do to pretty print pytrees in tjax?

Of course one can design separate systems for this. I'm just commenting that it would be nice to have some broader extensible way of doing this in JAX; i.e. make it a JAX-notion or a PyTrees(-beyond-JAX)-notion. Rather than just a library-specific thing we each reinvent.

I really like the idea of being able to pretty print pytrees; my fingers get RSI from typing jax.tree_map(lambda x: x.shape, some_large_pytree) everywhere, and yet the string is typically not nice to read in a log
cc @mattjj and @hawkinsp for the extensibility aspect of this proposal

What does serialization have to do with Jax?  Do you serialize the static fields differently than the dynamic ones?
As for pretty-printing, the only advantage that I can see to having registration within Jax is that Jax errors can make use of it.  That would be useful since whenever I have an error, I added a call before it to my pretty-printer, and re-run the program to hit the same error.

Our usual starting position is that anything supporting more than some basic tree manipulation is out of scope for jax's pytree library. Namely, jax.tree_util is scoped narrowly to correspond to the jit (etc.) API contract and not much else.
I do think it would be nice for another tree library to do this. (I'd likely try and reach for such a thing in my work!) What are the main disadvantages to that? I can imagine wanting to check/ensure some correspondence with jax's pytree node registry.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

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

Codecov Report

Merging #176 (3ffb039) into main (2974dfe) will increase coverage by 9.55%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #176      +/-   ##
==========================================
+ Coverage   83.33%   92.88%   +9.55%
==========================================
  Files           2        2
  Lines         126      478     +352
  Branches       31       49      +18
==========================================
+ Hits          105      444     +339
- Misses         21       33      +12
- Partials        0        1       +1

Impacted Files
Coverage Δ

example/src/main.ts
95.65% <100.00%> (+7.19%)
:arrow_up:

src/index.ts
91.76% <100.00%> (+9.76%)
: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 3675b7f...3ffb039. Read the comment docs.


Shorten host property in network statsbeat

can you add some tests for the regular expression?
It's very similar to mine.


Vulnerability roundup 98: postgresql-9.6.20: 2 advisories [8.8]

search, files

[ ] CVE-2018-1058 CVSSv3=8.8 (nixos-20.09)
[ ] CVE-2019-9193 CVSSv3=7.2 (nixos-20.09)

Scanned versions: nixos-20.09: aa5b9cd16b9.
Cc @danbst
Cc @globin
Cc @ocharles
Cc @thoughtpolice

See also: #106234

See also: #106234

The second CVE is disputed.

The second CVE is disputed.

CVE-2018-1058 was fixed in 9.6.8: https://www.postgresql.org/about/news/postgresql-103-968-9512-9417-and-9322-released-1834/

CVE-2018-1058 was fixed in 9.6.8: https://www.postgresql.org/about/news/postgresql-103-968-9512-9417-and-9322-released-1834/

I'm closing this since the remaining CVE is disputed: https://www.postgresql.org/about/news/cve-2019-9193-not-a-security-vulnerability-1935/.

I'm closing this since the remaining CVE is disputed: https://www.postgresql.org/about/news/cve-2019-9193-not-a-security-vulnerability-1935/.


@pwittrock - please take a look. PR for #891

This PR would cover two commits. The first one has new PNG files, plus attempted changes in
logging.md
to point two URL's at those PNG files. The second commit fixed the URL's to make them correct.

This change is

Do we have a single directory for images?  I notice "images" and "diagrams".  Is there a standard for where we store images associated with a doc?

In Issue 891, Mr. Rutsky mentioned that the release-1.3.0 branch had the missing PNGs. In that release branch, I found the files in directory
kubernetes/examples/blog-logging/diagrams/
and as a judgment call, I figured that the last two directories might have some importance for later bug fixes / work that might depend on release-1.3.0. I can certainly place the files in
kubernetes / images
and repoint to them if you want. LMK . . .

@fbsolo Thanks for taking care of this.  LGTM pending approval by @jaredbhatti


Should restore focus on text field when switching apps

If cmd-tab is executed while the cursor is on a textfield and ready for text input, returning to the app will cause the currently selected text input field to lose focus, and occasionally causes the backspace key to stop working.

"If cmd-tab is executed while the cursor is on a textfield and ready for text input, returning to the app will cause the currently selected text input field to lose focus, and occasionally causes the backspace key to stop working."

No longer as of Nativefier 7.1.0 / electron 1.6.6 :) , closing.


Update ModbusServerTCPasync.cpp

With this code change, the response time for any ethernet or WiFi Modbus requests will go from 900mS to 30mS. Therefore much more requests per seconds can be handled.
The previous version was using the aSyncPollTrigger ->onPoll option. This onPoll gets triggered every 500 to 900mS or so and that explains the slow response in the first place.
I also do not see any benefit of using the onPoll option as onData would suit much better; it is the same thread and it already knows when there is a message to be send out and thus it could just be handlet inside onData.
Tested this code. Compared to previous, this will go from around 900mS to 30mS. I tested for an hour running at 35 messages per seconds and works fine.

It is even faster when logging is disabled and only 1 client is connected / fetching data... an awesome 4mS response time!

@bertmelis Bert, can you approve this PR, since it is your code affected?

I'm looking at it. And I'm trying to recall why I didn't do this in the first place. Because there is no real reason why the the outbox shouldn't be handled when adding something to it.
@EmbeddedDevver Can you achieve the same solution when adding handleOutbox to addResponseToOutbox while also leaving it in onPoll? The lock_guard has to be moved to avoid deadlock of course.
I vagely remember that while stress testing no all outgoing messages are sent because the TCP buffer ran out of space. If you keep it in onPoll they will be sent as soon as there is space again.

A question from the (async) layman: will this work alike on the ESP8266?

@bertmelis
Thank you for your kind and fast messaging :-).
I can imagen that during stress testing AND other functions also use the tcpasync, that there is a possibility that the tcp buffer can be full and thus not send out the message.
So I think to use both options is the best. Than usually it will be send out directly, and if not, than the poll will handle the message 900mS later.
Would you like me to test your adjusted code in practice?

Yes please. Since I've moved abroad to a rental house I'm short on testing devices.

@bertmelis
Tested the adjust code. Works like charm. Reached almost 55 messages per second getting 10 holding registers in each request. Impressive. Less then 2mS response time:

I would say, merge it 👍


Bootloader doesn't meet size limits when compiled

$ make -C bootloader build
make: Entering directory '/build/rv003usb/bootloader'
riscv64-linux-gnu-gcc -o bootloader.elf  bootloader.c ../rv003usb/rv003usb.S  -g -Os -flto -ffunction-sections -fdata-sections -fmessage-length=0 -msmall-data-limit=8 -march=rv32ec -mabi=ilp32e -DCH32V003=1 -static-libgcc -I/usr/include/newlib -I../ch32v003fun/ch32v003fun/../extralibs -I../ch32v003fun/ch32v003fun -nostdlib -I. -Wall -I. -I../lib -DUSE_TINY_BOOT -I../rv003usb -Wl,--print-memory-usage -L../ch32v003fun/ch32v003fun/../misc -lgcc -T ch32v003fun-usb-bootloader.ld -Wl,--gc-sections
/tmp/ccwYoG6F.s: Assembler messages:
/tmp/ccwYoG6F.s:561: Warning: setting incorrect section attributes for .rodata
/usr/lib/gcc-cross/riscv64-linux-gnu/11/../../../../riscv64-linux-gnu/bin/ld: bootloader.elf section `.text' will not fit in region `FLASH'
/usr/lib/gcc-cross/riscv64-linux-gnu/11/../../../../riscv64-linux-gnu/bin/ld: region `FLASH' overflowed by 720 bytes
/usr/lib/gcc-cross/riscv64-linux-gnu/11/../../../../riscv64-linux-gnu/bin/ld: bootloader.elf: error: PHDR segment not covered by LOAD segment
Memory region         Used Size  Region Size  %age Used
           FLASH:        2640 B       1920 B    137.50%
             RAM:         280 B         2 KB     13.67%
collect2: error: ld returned 1 exit status
make: *** [../ch32v003fun/ch32v003fun/ch32v003fun.mk:235: bootloader.elf] Error 1
make: Leaving directory '/build/rv003usb/bootloader'

I just cloned the project locally (with submodule), and it outputs this message that the bootloader is oversized.
Why is this happening and how can we fix it?

Please run riscv64-linux-gnu-gcc -v -- for the bootloader you should be using gcc14

Please run riscv64-linux-gnu-gcc -v -- for the bootloader you should be using gcc14

gcc version 11.4.0
Feels like I'm stuck at GCC v12 for now. Thanks for your quick reply (you can probably add minimum requirements in the README, btw).

Or even with GCC v11 for RISCV64.

Ok, honestly, gcc v11 should be OK.  What configuration do you have?  Some configurations for the bootloader are not able to fit in the bootloader space.  Or is your install a virgin install?


Adds def site feature to super-unstable

As per my extra comment to #132 this adds the def_site feature to proc-macro2 so it'll compile with the latest nightly.

Thanks!

No problem! Out of curiosity when will this be released?

Already done!


[master] run timeout task at new thread to avoid the worker thread blocking

What is the purpose of the change
fixes #8949 on master

Codecov Report

Merging #8956 (4547a5b) into master (11e04fb) will decrease coverage by 0.65%.
The diff coverage is 70.00%.

@@             Coverage Diff              @@
##             master    #8956      +/-   ##
============================================
- Coverage     60.79%   60.14%   -0.66%
+ Complexity      446      445       -1
============================================
  Files          1100     1100
  Lines         44472    44474       +2
  Branches       6475     6474       -1
============================================
- Hits          27036    26747     -289
- Misses        14452    14748     +296
+ Partials       2984     2979       -5

Impacted Files
Coverage Δ

...rg/apache/dubbo/common/timer/HashedWheelTimer.java
79.52% <70.00%> (-0.21%)
:arrow_down:

...ookeeper/ZookeeperDynamicConfigurationFactory.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...gcenter/wrapper/CompositeDynamicConfiguration.java
0.00% <0.00%> (-63.34%)
:arrow_down:

...pport/zookeeper/ZookeeperDynamicConfiguration.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...fig/configcenter/TreePathDynamicConfiguration.java
31.42% <0.00%> (-54.29%)
:arrow_down:

.../dubbo/metadata/report/MetadataReportInstance.java
12.50% <0.00%> (-40.63%)
:arrow_down:

.../configcenter/support/zookeeper/CacheListener.java
0.00% <0.00%> (-32.44%)
:arrow_down:

...gistry/client/migration/MigrationRuleListener.java
55.88% <0.00%> (-23.53%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-20.69%)
:arrow_down:

...mmon/config/configcenter/DynamicConfiguration.java
68.18% <0.00%> (-18.19%)
:arrow_down:

... and 39 more

Continue to review full report at Codecov.

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


support 16bit tif image

Can it support 16 bits tif image? image  size almost  70Mb

@gavincoder2020 Does it? I want to load 16 bit images too.

@gavincoder2020 Does it? I want to load 16 bit images too.

You can change the loadFile function to support loading of other types of images, I think.

You can change the loadFile function to support loading of other types of images, I think.

I am using pyqt5. the file size is 578kb. tif image is not opening in my case as well. @YashRunwal  can you tell me what to edit?

@SrutiBh I hope this helps a bit:

Install opencv: pip install opencv-python
import opencv in labelImg.py: import cv2
go to function read() in labelImg.py
replace the contents of the try block with the following (untested, let me know if it works for you):

image = cv2.imread(filename, cv2.IMREAD_ANYDEPTH)
return QImage(image, image.shape[1], image.shape[0], QImage.Format_RGB888)

The parameter cv2.IMREAD_ANYDEPTH will give you the image in actual 16 bit pixels. I have not tested what Qt will do to it when you load a 16 bit image into a QImage. It's possible that you have to do some adaptations after loading the image to optimize the contrast, but in my book that's an advantage of using 16 bit images in the first place, since you have the full intensity range and can dynamically adapt it as you need.

@Cerno-b  it is still coming the same.  if you are free can you come in email now?  that will be helpful

Solving this via e-mail will not be helpful to anyone else, so I'd prefer if we could try to solve this here.
Could you post a screenshot of what it looks like? Also could you upload your image somewhere? You mentioned 60MB, so maybe dropbox or something similar might be useful.

@Cerno-b  sure. I am unable to upload the tif images. It's saying not supported here. I have changed the code as you suggested. Whenever I am trying to open the image the kernel is dying and the output window is getting off.

Okay, turns out it's a little bit more complicated. This works for me.
It now supports RGB, RGBA and grayscale in 8 and 16 bit.
I'll put this on a branch. Would you be so nice and test whether it works with your 16 bit images and whether they look okay?
def read(filename, default=None):
    try:
        image = cv2.imread(filename, cv2.IMREAD_UNCHANGED)
        if image.max() > 255:
            image = (image - image.min()) / (image.max() - image.min()) * 255
            image = image.astype('uint8')

        if len(image.shape) == 2:
            image = np.tile(image, (3, 1, 1)).transpose((1, 2, 0))

        height, width, channels = image.shape
        if channels == 3:
            # convert BGR to RGB
            image[:, :, [0, 1, 2]] = image[:, :, [2, 1, 0]]
            image = image.copy()
            return QImage(image, width, height, image.strides[0], QImage.Format_RGB888)
        elif channels == 4:
            # convert BGRA to RGBA
            image[:, :, [0, 1, 2, 3]] = image[:, :, [2, 1, 0, 3]]
            image = image.copy()
            return QImage(image, width, height, image.strides[0], QImage.Format_RGBA8888)
        else:
            assert False
    except:
        return default

@Cerno-b  Thank you so much. This works well for me now

@SrutiBh If you run into any incompatibility problems with other image formats, please let me know.

@Cerno-b  sure, I will

Okay, turns out it's a little bit more complicated. This works for me. It now supports RGB, RGBA and grayscale in 8 and 16 bit.
I'll put this on a branch. Would you be so nice and test whether it works with your 16 bit images and whether they look okay?
def read(filename, default=None):
    try:
        image = cv2.imread(filename, cv2.IMREAD_UNCHANGED)
        if image.max() > 255:
            image = (image - image.min()) / (image.max() - image.min()) * 255
            image = image.astype('uint8')

        if len(image.shape) == 2:
            image = np.tile(image, (3, 1, 1)).transpose((1, 2, 0))

        height, width, channels = image.shape
        if channels == 3:
            # convert BGR to RGB
            image[:, :, [0, 1, 2]] = image[:, :, [2, 1, 0]]
            image = image.copy()
            return QImage(image, width, height, image.strides[0], QImage.Format_RGB888)
        elif channels == 4:
            # convert BGRA to RGBA
            image[:, :, [0, 1, 2, 3]] = image[:, :, [2, 1, 0, 3]]
            image = image.copy()
            return QImage(image, width, height, image.strides[0], QImage.Format_RGBA8888)
        else:
            assert False
    except:
        return default

Thank you @Cerno-b . It's very helpful. It's saved me considerable time😊


npm stay in version 0.8.5?

npm stay in version 0.8.5?

As you can see here https://www.npmjs.com/package/file-loader, 0.9.0 is properly released. You probably have something wrong in your package.json, but we can't do anything about it.


[fix][broker] fix incorrect unack stats when dup ack a message

Fixes #19268

Modifications
For shared-type consumers, if can't find the message position in pendingAck set, skip the following process.
Verifying this change

[x] Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[x] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/labuladong/pulsar/pull/20

Since we will start the RC version of 3.0.0 on 2023-04-11, I will change the label/milestone of PR who have not been merged.

The PR of type feature is deferred to 3.1.0
The PR of type fix is deferred to 3.0.1

So drag this PR to 3.0.1

https://github.com/apache/pulsar/issues/19268 has been fixed by https://github.com/apache/pulsar/pull/20990, so close this PR.


improving list view scroll

I have introduced mixin AutomaticKeepAliveClientMixin to every listview element.
This keeps the element alive even if the item goes out of the viewport.
Reasoning: Our ListView item can be very heavy in terms of number of widget it is built width. If we have to built the heavy item we scroll this make scroll janky. So by keep them alive, list view can reuse them.
If we keep item alive even if we go out of view port, this will increase the memory usage as we scroll to very large number of items.

Ensemble widget tree
https://github.com/EnsembleUI/ensemble/assets/36638657/6d4eece1-9b59-4ba0-9a62-629e039f900f
Flutter widget tree
https://github.com/EnsembleUI/ensemble/assets/36638657/5bd80b48-c0e3-4c05-82c9-10ce91a692a3
Our ensemble widget tree for similar widget is quite heavy compare to flutter implementation tree. This make quite a difference in scroll performance between two for similar UI.
So, short term solution is in this PR reusing the already built heavy widget by keeping them alive even after the go out of viewport but long term will be to optimise the ensemble widget tree as well.
fyi: both video and scroll are taken in debug mode.


Cannot type >= in SQL editor

Description
Hello,
I'm not sure if this is an issue on my side, but the following occurs only in DBeaver (community edition, portable):
When I type >= as soon as I press the spacebar, >= changes to ≥ which of course fails when I execute the query.
Any pointers?
DBeaver Version
Community Edition 23.3.0 Portable
Operating System
Windows 11 Enterprise 22H2

Hmm.. It's not happening for me. Have you installed some additional plugins?

@E1izabeth first off, thanks for the follow-up.
I think I have found what is causing this.
When I set 'JetBrainsMono Nerd Font' (https://github.com/ryanoasis/nerd-fonts/releases/tag/v3.1.1) as a monospace font, then >= changes automatically to ≥.
Switching back to another monospace font, such as Courier, all is good. However, this occurs only in DBeaver. In other editors >= is maintained regardless the font used.
Is this something that could be fixed in DBeaver or I should report it to Nerd Font GitHub?

I think we should try it on Eclipse to be sure it's not a DBeaver issue.

@valsamis-d Sorry to interrupt, but could you please check for the error?
The font simply tells the user how the actual value will be rendered.
The actual value has not changed, and neither has the script.
Fonts can even contain images rather than Unicode values, so it would be really strange for these values to be passed through a shell or script.
So it seems to me that the error you encountered is not caused by this, but by some other cause.
working screenshot

@PzaThief thank you so much for the update. I tested it in the latest DBeaver release and indeed it works just fine:

I'm happy I will keep using my favorite monospaced font :)


Many Dual-Division Team Lists are wrong for RelicR, leading to messed up team pages.

Good examples of where the divisions are messed up are: https://theorangealliance.org/teams/8813  (NSR and CMP-IA are listed in both divisions for both events) vs. https://theorangealliance.org/teams/11316 (shown as competing in neither event).  Both teams competed at both NSR and CMP-IA but in different divisions. The incorrect team list screws up their team pages, leading to either two divisions showing for the same team at the same event, or the team not showing up at all at an event that it competed in.

This has been corrected please let me know if you come across any more with this issue and they will be corrected.


can't load image from outer source

i am trying to load image from http source and image was not loaded?
      <Image                     x="0" y="0"                     width="50"                     height="40"                     href={{uri: 'https://fluent-panda.appspot.com.storage.googleapis.com/dumbbell.svg'}}                 />

were you able to solve this?

@msand
<SvgUri
    width="200"
    height="200"
    uri="http://thenewcode.com/assets/images/thumbnails/homer-simpson.svg"
  />


protoc error

Descriptors cannot be created directly. If this call came from a pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0. If you cannot immediately regenerate your protos, some other possible workarounds are: 1. Downgrade the protobuf package to 3.20.x or lower. 2. Set PROTOCOLBUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower). More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

sentencepiece == 0.2.0 and protobuf == 4.25.5 works in our server


Add instructions for launching project in browser using local host

In the very helpful instructions you have already written, it would probably be good to add how to launch index.html using a local host, for example nmp http-server, so that you can see all the images when editing the viewpoints with your tool. It's very easy to do if you have nmp or npx installed. This would be at step 7. Change:

Open your marzipano project in your browser

To:

Open your marzipano project in your browser using a local host server, for example  nmp http-server.
If using http-server, you can launch it from the terminal in the app-files directory directly with:
npx http-server
Then you can point your browser to http://<IP_ADDRESS>:8080/ to view the project.

and then your original instructions can continue from there.
Thanks!

@isaacullah Thanks for your contribution.
I am not sure if this should be in the scope of this project.
Why?
To use npx you need npm installed and i don't think "normal" user have this installed. The goal is to be user friendly for non-technical people.
To view the panorama created with marzipano tool you need a (local) server and i think that the user has figured this out as this question has a lot of answers on google.
I am going to add a hint that the user needs a server to view it in the browser.

@isaacullah i am sorry i just tested your idea with npx http-server and it works easier than expected. I am going to add it to the instructions.

Glad you could add this! the npx http-server is by far the easiest and most cross-platform way that I've discovered to preview a static site from disk. Again, I really appreciate your work with this helpful tool!!


ARROW-16597: [Python][FlightRPC] Force server shutdown at interpreter exit

Hmm, also this needs rebasing.

Thank you @lidavidm !


Move dump tests to their own file

Codecov Report

Merging #57 (75e124f) into main (936e26d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #57   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          597       597
=========================================
  Hits           597       597

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 936e26d...75e124f. Read the comment docs.


Discussion/question: What to do with the generated images ? (texturing 3d models)

Hi , the tool works great. Thanx a lot!
Do you know of any good workflow for texturing my 3d model though ? i already projected the initial texture from the front, but now i need to texture the rest of my monsters body, from the side and back at least.
I thought maybe i can use stencil projection mode in texture painting -> which camera settings would i need to set in blender or substance painter to exactly match the generated images from the different angles?
if you know of any python code to do this automatically would be nice too. zero12345 wouldnt work as they generate their own mesh , i already have my mesh from which i made the first original texture (front projection).
maybe this could be partly scripted in blender to become a useful and outstanding texture pipeline tool.
EDIT: Now that i thought about it. Would it be possible to send an arbritrary (or fixed) angle to zero123+ ??
Like, we move around the object in blender, when we find a good angle (this might be different for different objects) , then send the camera position and angle (and maybe even an image for controlnet) to zero123+, generate the next image, receive the image ... then move around it again , and again send to zero123+ and again receive the image.
the intermediate steps in the diffusion could be kept  in the mean time so we always get the consistent style and shading from the first/initial picture (similar to how comfyui only evaluates changed parts of the diffusion pipeline) just like it does now..
Maybe comfyUI would be the tool to make this all alot easier as it can already communicate with other progs such as blender and has a really good codebase and makes integrating new modules "easy", without having to write everything from scratch.

Hi @Ulf3000 As far as I understood this paper and codebase, it will not allow you to pass any other view to project the texture on. They have their set of angles, only they can be used (all at once). If you want to do one at a time, then  you will need to look somewhere else that generate consistent images with respect to your geometry.
By the way, if you find such models, please share with me too.


hub pull-request -i flag overwrites commits

I realize that hub pull-request is deprecated, but I accidentally wrote over someone's commit in a project repository.
not sure what there is to do about that now.

Hello World

I'm sorry you had a problem. The pull-request -i command is indeed deprecated, but it doesn't do anything other than converting an issue to a PR based on a branch that you specify. It doesn't change anything in the git repo, and it doesn't have the power to eliminate git commits from your repository either.
I'm not sure what happened, but the only way of “overwriting” git commits in your repo is if you had done git push --force.

Hi @mislav thanks for the explanation. I apologize for misrepresenting the issue - what happened was that I wrote over an issue in a separate repo, not a commit.

@cbcoutinho Thanks for following up. If I understand, you wrote over an issue in a repo that is different from the one you were operating on when you wanted to create a PR?

No not exactly, I originally created an issue on a repository that I don't
have write access to. I wanted to make the change so I forked it, committed
my change to my fork and then created a pull request on the original
repository with the '-i' flag. The number I supplied was the issue number
that I created. This converted my 'issue' into the pull-request I was
trying to create.
I think it makes more sense to get an error like 'an issue or pull-request
already exists at that number', but instead it just had just overwritten
the issue. Luckily it was my own issue, but maybe it's possible to
overwrite other people's issues - I haven't tried that yet.
On Thu, Nov 30, 2017, 11:45 Mislav Marohnić<EMAIL_ADDRESS>wrote:

@cbcoutinho https://github.com/cbcoutinho Thanks for following up. If I
understand, you wrote over an issue in a repo that is different from the
one you were operating on when you wanted to create a PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/github/hub/issues/1628#issuecomment-348151820, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMTd7EIENAeRTomZCfhJDxnr3hMVSLsQks5s7odZgaJpZM4QqnJb
.

I wanted to make the change so I forked it, committed my change to my fork and then created a pull request on the original repository with the '-i' flag. The number I supplied was the issue number that I created. This converted my 'issue' into the pull-request I was trying to create.

This is literally what the -i flag does. It converts an issue (usually your own) to a pull request. The original issue is then gone.


Combining mask and profile in widgets/silx view

Allowing to use current mask to compute profile.
Similar to #3124 but for profile.
Regarding how to connect mask tool to profile, any better idea than what I proposed in #3124, i.e., using:
def setSelectionMaskSource(self, source):
        """Set the source of the selection mask.

        :param source:
            QObject with sigMaskChanged signal and getSelectionMask method.
        """

An alternative I see is to attach the mask to the image at the item level, that might be more generic since it does not rely on the mask tool.

The mask tool would be OK to me.
Needed for ID17

If the NeXus convention is respected, you should get the mask information together with the detector data.

I didn't even though of NeXus for pre-loading the mask...

The item-level way is probably best for supporting more than the mask tool... but it requires more internal rework.

I didn't even though of NeXus for pre-loading the mask...

At a certain moment one has to profit from the information contained in the file.
One cannot ask the user to specify uncertainties, masks, calibrations, pixels sizes, motors, ... They should be read from the file if available.

They should be read from the file if available.
I completely agree but apparently so far people are asking for authoring masks or loading masks saved independently.

But that is already handled by the mask tool, isn't it?

I don't think so.
The mask tool in itself has nothing to do with NeXus, and I don't know if silx view is loading the mask (I would say no).
Anyway, if we're supporting the Nexus mask, we might not want users to edit it.

?????
The mask tool has nothing to do with NeXus, therefore should be able to load and to save masks.
Where is the problem?

What I am saying is:

That you can take a lot of information directly from the file.

That for the cases where the mask is not in the file, it can be read or saved by the mask tool itself.

Yes, I agree, but from a quick search there is no Nexus mask support in silx, so you first point is not implemented.
The second one is there since the beginning.

Well, files generated by Lima are not really Nexus compliant. The mask is still given by a dummy value (4e9) which turns out to be more annoying than having the dummy at "-1".
One could argue the issue may be addressed by Nexus, the NexusWriter, ... but ID17 won't switch to Bliss in a foreseeable future, hence I am locked with LImA files.


fix: use asset host if set during development

Description 📖
This pull request fixes the loading of the Vite client and react refresh preamble when an asset_host is defined and the dev server is running.
Background 📜
This was happening because when an asset host is defined and a dev server is running the vite client is loaded twice. Once with the asset_host domain and once with the relative path (so the current domain).
Consider the case where all assets are provided by my-cdn.com. The Vite client loaded with the vite_client_tag tag will be loaded without any subdomain (/vite-dev/@vite/client) while the client as fetched from within the code will be loaded from the same domain as the asset itself (in this case my-cdn.com/vite-dev/@vite/client).
This results in two vite clients being loaded and a crashing page.
The Fix 🔨
By changing the prefixing method of the vite assets with the assets subdomain.
Screenshots 📷

Thanks for your contribution Stef, looks good!
Out of curiosity, would you explain your current use case for setting asset_host during development?

We have an application that uses subdomains to switch tenants so to mimic this behaviour locally we use the *.lvh.me domains to resolve localhost.
Next to this we use a separate subdomain for our assets. This way we can easily put a CDN in front of it. So to ensure this works properly we set our asset_host to http://assets.lvh.me:3000 during development. This way we can for example check CORS settings etc.

Thanks for the quick replies and the release 👍

Unfortunately the proposed solution causes the lookup to fail in production, because it affects the name of the entries.
I yanked 1.2.19, and will temporarily revert main. I can revisit the implementation tomorrow, another PR is welcome.

Ow, for me it seemed to work 😱 I will have a look tomorrow as well! Thanks

I had a short look into this and I wanted to check in the ideas with you before trying to fix it and break it again.
For me the lookup actually seems to work and since I'm kinda new to vite, I'm not really sure why this is failing. So if you have any better ideas, please.
Path 1: Fix it in vite_rails
The loading of the vite_client can quite easily be corrected in the ViteRails::TagHelpers#vite_client_tag by converting
tag.script(src: src, type: 'module')

into
javascript_include_tag(src, type: 'module', extname: false)

So that's all good but then the react_refresh_preamble is more problematic. Since it path is generated within ViteRuby::Manifest. So it might be an option to explicitly prefix it only for the preamble.
Path 2: Separate method in vite_ruby
Both the @vite/client and @react-refresh are no lookups in the manifest and rather the serving of a package directly. So it might be an idea to create a second method next to prefix_vite_asset called prefix_node_asset to return the full url.

@StefSchenkelaars Path 2 has the benefit of fixing the issue for all frameworks that support an asset_host configuration.
I've released<EMAIL_ADDRESS>please upgrade and let me know how it goes!

@ElMassimo I've tested it and it seems to be all good 👍  Thanks!


Handle conflicts on changelog update; print debugging info

Concurrent runs of the CD pipeline against main cause a conflict when we try to parse the changelog update. I'm not sure why. This fails more gracefully (we should still do the deployment, but the changelog won't get updated) and prints some debugging information that might be useful.

@devennavani if you merge two PRs in quick succession can you give me a heads up so I can check the logs? Trying to get to the bottom of these deployment conflicts.

@prbot approve


Move yolox detector to examples

Why this PR?
This PR treats the yolox detector as a user plug-in detector (in examples), to

isolate yolox code dependencies from VQPy
ease the future detection models built in (minimize dependencies).

Changes in VQPy

To install VQPy only, users don't need to install yolox and the pre-trained in model directory that we specified, or install the dependencies yolox introduced.
Add how to customize and register a yolox detector to VQPy in examples

Future Plan

Refine detector interface and add built-in detectors

Thanks for the review and suggestions!
I will merge the PR to unblock the next-step development. And I will create a new PR to address any further comments.


website demo is broken (.js files 403 forbidden)

When trying to view the demo on the website, I only see a rotating square and javascript console is showing multiple 403 errors for javascript files.

Hi, probably was a temporary issue. Can u confirm you can navigate to it now?


Add an option to present the posts in a feed

The posts should be presented such that users can continually scroll down to view the posts in the study, without needing to press any buttons to continue.
Once a post is out-of-sight (scrolled past), it should be treated as if "next post" has been pressed. The dynamic feedback should be applied, and the user should not be able to scroll back up to the post any longer.
The "continue to next post" button should be replaced by a button that asks the user to continue after they have completed the study. This should become activated once the user scrolls to the final post in the study. A message below the last post in the study could also inform the user that they can press the continue button.
We may also need some way to limit the scrolling speed such that users can't accidentally scroll past posts by scrolling too fast.

This is now available


[Feature][Selectdb-cloud] Support sink ddl

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Updates

connector
add e2e
update docs

Reference

https://github.com/apache/seatunnel/blob/dev/seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/sink/JdbcSink.java#L67C17-L67C43
https://github.com/apache/seatunnel/blob/dev/seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/sink/AbstractJdbcSinkWriter.java#L61
https://github.com/apache/seatunnel/blob/dev/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-cdc-mysql-e2e/src/test/java/org/apache/seatunnel/connectors/seatunnel/cdc/mysql/MysqlCDCWithSchemaChangeIT.java
https://github.com/apache/seatunnel/pull/8134
https://github.com/apache/seatunnel/pull/8082

Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Would you like to submit a PR to solve this issue? It's not difficult, and I encourage you to give it a try! Here are some helpful resources to get started:

Contribution Setup Guide
Code Submission Guide

If anyone else would like to help implement this function, please leave a message here, and you can submit a PR. Thank you!
By the way, since this might be your first contribution, I think we can have a more in-depth communication to help you get familiar with SeaTunnel. Feel free to contact me on LinkedIn (David Zollo) or WeChat(davidzollo). When adding me, please let me know your GitHub ID. I'd be happy to assist you!

hi, I'm interested in this issue and will fix this issue

please assign to me


CVE-2020-7691 Security Vulnerability Issue

With the latest version of jspdf:2.5.1 integrated into the project getting security vulnerability issue
CVE-2020-7691 EPSS: 0.17%CVSS: 6.1
In all versions of the package jspdf, it is possible to use <

Could you give us some steer on this @eKoopmans

Thanks for the heads up!
The good news is that the fromHTML method reported in CVE-2020-7691 no longer exists in jsPDF:

It was deprecated in 2018 and removed sometime after that
It's not defined on a jsPDF object
The replacement html method is actually just a clone of htmlpdf.js (fun with recursion 🙃)

Even if that method did still exist in jsPDF, we only use the methods addPage, addImage, output, and save.
I think this should be safe to close.


Remove redundant next checks

Pull Request Test Coverage Report for Build 56

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 84.896%

Totals

Change from base Build 50:
-0.2%

Covered Lines:
2417

Relevant Lines:
2847

💛  - Coveralls

Pull Request Test Coverage Report for Build 56

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 84.896%

Totals

Change from base Build 50:
-0.2%

Covered Lines:
2417

Relevant Lines:
2847

💛  - Coveralls

Pull Request Test Coverage Report for Build 51

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 85.072%

Totals

Change from base Build 50:
-0.04%

Covered Lines:
2422

Relevant Lines:
2847

💛  - Coveralls

Pull Request Test Coverage Report for Build 51

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 85.072%

Totals

Change from base Build 50:
-0.04%

Covered Lines:
2422

Relevant Lines:
2847

💛  - Coveralls


[IN-104][OSF] Add work as a choice on the preprint provider model

Purpose
Add "work" as a word choice for preprint providers.
Changes
Add "work" to PREPRINT_WORD_CHOICES on the preprint provider model.
QA Notes
Doesn't need QA. Will be tested once we use it on the frontend.
Side Effects
None
Ticket
https://openscience.atlassian.net/browse/IN-104

Updated in abe77139af1c89bd395c59e03977bd610e93213b, consider this merged


Add the ability to modify tags

In the UI, this would ideally be represented more abstractly as "tags". When exporting the library, you could choose to export the tags to a specific field (e.g. grouping) for the purposes of use in Rekordbox.

I've started adding this functionality using a SwiftUI form editor. The entry point looks like this:

which pushes an interface like this onto the side pane's navigation controller:

All of this is currently ephemeral and needs to be wired up to the database in some form. Still considering whether to make changes instantly or to implement some form of an editing mode with a Cancel/Done button.


add release-process.md

Signed-off-by: hang.jiang<EMAIL_ADDRESS>
What type of PR is this?

/kind documentation
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm

/lgtm


Running on HoloLens 2 Emulator crashes system

This shouldn't happen regardless of a problem with the engine, but I'm experiencing a system crash (blue screen) when trying to run DirectX12CoreWindowApp on the HoloLens 2 emulator.
Have you tried this, or are you running on actual hardware?

Never mind, it's clearly a Microsoft bug!  Again!
If you disable hardware acceleration, it's all working as expected.  Very nice.
You may want to add a note about this problem.  Please close this non-issue.

Interesting, I don't have a real HoloLens, I just tried it with a Windows Mixed Reality headset.


Default postfix configuration error

The default latest V4 installation has problems with Postfix docker container. It incorrectly resolves its own domain.
Steps to reproduce:

Install fresh V4 on example.com with WordPress
Try to send an email from WordPress to<EMAIL_ADDRESS>
Emails work to all domains except the same domain as web setup.
It starts with this error:
Jan 21 20:53:59 ip-xxxxxxxxx mydomain.com/smtpd[10882]: error: open database /etc/aliases.db: No such file or directory
When you go to email docker and create /etc/aliases file (even empty) it moves to this error:
Jan 24 12:19:59 ip-xxxxxxx website.com/smtpd[223]: NOQUEUE: reject: RCPT from websitecom_php_1.website.com[<IP_ADDRESS>]: 550 5.1.1<EMAIL_ADDRESS>Recipient address rejected: User unknown in local recipient table<EMAIL_ADDRESS><EMAIL_ADDRESS>proto=SMTP helo=
Which makes me think that postfix has problems resolving the email and tries to send it to another container rather than to MX record.

root@ip-172-26-15-187:~# ee shell example.com
^_^[www-data@example.com:~/htdocs]$ php -r<EMAIL_ADDRESS>"Hello Human", "Hello from EasyEngine!");'
sendmail: RCPT<EMAIL_ADDRESS>(451 4.3.0<EMAIL_ADDRESS>Temporary lookup failure)

Having the same issue on EE v4. I have two domains running on 1 server.
When you test send emails from the server it works!
I use Office 365 smtp and can see the emails at the sent inbox.
But when I try send email from Wordpress/Woocommerce it doesn't show on sent inbox but the email get through eventually to spam inbox.
Hope you guys find solutions to this.
On my old server I have EE v3 running with the same setup and it all works fine!


Change date to be more clear

On a new day the time will look like it went backwards so add back the full date on the message field

Fixed


Auto merge should complete without review when using frontmatter reviewer assignment and content owner only modifies their content

Feature Request
Is your feature request related to a problem? Please describe.
Current solution introduces a blocker in the workflow when waiting on others to approve content changes to content you own
Describe the solution you'd like
When using the frontmatter reviewer assignment feature, and a content owner modifies their own content the workflow should not require that someone else reviews it. It should just pass through without any reviewer requirement.
When adding new files it should allow for specifying a default reviewer (in case the repo doesn't have CODEOWNERS configured)
When renaming a file, this workflow should be thought through. It should initially work the same as modifying files. Maybe in the future it needs to be a configurable setting.
Describe alternatives you've considered
Only alternative right now is to disable the minimum required reviews which affects everyone not only the content owners.
Teachability, Documentation, Adoption, Migration Strategy
TBA

Initial release in v1.2.4


iOS Firebase dependencies cause App Store rejection

Older versions of Xamarin.Firebase.iOS.Auth contain references to the deprecated UIWebView API. Apple no longer accepts App Store publications with the following message:
"ITMS-90809: Deprecated API Usage - New apps that use UIWebView are no longer accepted. Instead, use WKWebView for improved security and reliability. Learn more (https://developer.apple.com/documentation/uikit/uiwebview). "
Newer versions of the package supposedly fixed the issue, according to this issue. Can you please update the dependencies in your package?

The package is updated and build successfully according to AppCenter.
But I have not test it yet, please do me a favour on testing it.
Please use <IP_ADDRESS> with XF 4.6+ and read about readme.md as there is additional settings in your AppDelegate.cs

It builds fine with XF 4.6.726 (not with the newly released 4.6.772 sadly). But I get NullReferenceException when I try to set RemoteConfig.SharedInstance.ConfigSettings in AppDelegate.cs.

It builds fine with XF 4.6.726 (not with the newly released 4.6.772 sadly). But I get NullReferenceException when I try to set RemoteConfig.SharedInstance.ConfigSettings in AppDelegate.cs.

I made mistake on the calling sequence, please do it in below sequence
Firebase.Core.App.Configure();
RemoteConfig.SharedInstance.ConfigSettings = new RemoteConfigSettings();

Just finding some error after update.
Working on it

@nyakaspeter
So I finished update and testing on iOS.
Please use <IP_ADDRESS> and it should be working now.
Please let me know if there is any issue.
(I do not have Android to test)

Hi I have updated to <IP_ADDRESS> with XF 4.6+ but I am still facing this issue. Is there something I should check?

@remco-rebelit
You can rebuild all to try. If it is not working, you may need to check if there is any code is using UIWebView.

I removed all obj and bin folders and I am still getting the App Store rejection because of ITMS-90809: Deprecated API Usage. There is no code using UIWebView.

@remco-rebelit
How about if you remove my package and submit?
I think there is a library using uiwebview but you may need to try it one by one.

Thanks, I solved it by adding: --optimize=experimental-xforms-product-type as additional touch arguments.

Thanks, I solved it by adding: --optimize=experimental-xforms-product-type as additional touch arguments.

good to know that.
Similar issue for reference
https://forums.xamarin.com/discussion/179071/deprecated-uiwebview

I've tested <IP_ADDRESS> on both Android & iOS. It works great now. Thanks for the fast response.


content-sources: submit content-sources repositories to image builder.

Description

Validate image ThirdParties repos when Creating/Updating Image before imagebuilder Compose, some repos exists and need updates, some needs to be created locally .
Update Image Builder request repositories with new repository fields, for now image builder support only GPGKey, CheckGPG

FIXES: THEEDGE-3083
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[X] Tests update
[ ] Refactor

Checklist:

[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] I run make pre-commit to check fmt/vet/lint/test-no-fdo

Codecov Report
Patch coverage: 87.50% and project coverage change: +0.34 :tada:

Comparison is base (3ef96ec) 58.86% compared to head (80ddc96) 59.21%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1958      +/-   ##
==========================================
+ Coverage   58.86%   59.21%   +0.34%
==========================================
  Files          71       71
  Lines       10267    10369     +102
==========================================
+ Hits         6044     6140      +96
- Misses       3704     3710       +6
  Partials      519      519

Impacted Files
Coverage Δ

pkg/services/images.go
59.67% <85.57%> (+1.88%)
:arrow_up:

pkg/clients/imagebuilder/client.go
72.63% <100.00%> (+0.98%)
:arrow_up:

... and 1 file 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.

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

@mgold1234 @loadtheaccumulator thank you for your view, updated the comments, and responded to the questions.


Agent trips up over UTF-8 characters from some commands

Not sure how to quantify this much more, as I don't have a super firm grasp on what's happening. The easiest way I know to see the issue is to try to "rm" (with no flags) a file that doesn't exist on a Centos 7 system. The issue does not occur on either Centos 6 or Centos 8. But on Centos 7 there is apparently some UTF-8 returned in the stderr message that causes Choria to return the error "Cannot decode output from Shim for Ruby action shell#run".
When this happens the command seems to run successfully, you just fail to get the proper response information back.

thank you


[MISC] Adding type T specification for BaseIterable and enhance funct…

…ionalities with empty method

Ok I have taken a look at the comments and came up with a new implementation, which incorporates @aiola suggestion, although in a slightly different fashion.
The idea is that type is actually abstract in the _BaseIterable. In fact, Documents and Datasets (which extends from _BaseIterable) do not need this specification, as we know that objects should be Document and Sample, respectively. On the other hand, the more generic implementation BaseIterable (to be used in generic cases where the Iterable is not typed), the @aiola inferring strategy is used.
I'm starting to like this new implementation but I'm eager to hear on what you think


Widget border line drawing error

Hi,
I have just tried installing this but when I run the example I get an error where the lines around each widget are letters rather than lines.

I am running OSX El Capitan and using the font Monaco.
Any ideas anyone?

Hi
Do you see the same issue with a simple blessed example?
var blessed = require('blessed');
var screen = blessed.screen();
var box = blessed.box({
  top: 'center',
  left: 'center',
  width: '50%',
  height: '50%',
  content: 'Hello {bold}world{/bold}!',
  tags: true,
  border: {
    type: 'line'
  },
  style: {
    fg: 'white',
    bg: 'magenta',
    border: {
     fg: '#f0f0f0'
    },
    hover: {
      bg: 'green'
    }
  }
});

screen.append(box);
screen.key(['escape', 'q', 'C-c'], function(ch, key) {
  return process.exit(0);
});
screen.render();

If so the question is more relevant to the blessed project.
You can also try with a different font (e.g. consolas).
It may also depend in the terminal software you use (iTerm2 works well).

Hi
I am using iTerm2 and tried the consolas font but that doesn't seem to be fixing anything.
I have also tried your example and that seems to also be having the same problem.
As you mentioned I will take this question across to the blessed project and see if they have any ideas :smile:


Replace logging outputs

Fixes #19

Add console class with support for output to stdout and stderr
Replace print statements with calls to Console.<stdout/stderr>

Wouldn't it be better to log using the logging library? - to utilize from its advantages such as seeing when/where a log call is being made

Well that highly depends, generally even though you may use a logger to output status to stderr the format on stderr most of the times should not include timestamps, log-level, ... if it is a command line tool (long running server processes are a different story).
So here are some of the questions I have been asking myself:

Do we need to report status of different importance ("log levels")?
How hard is it to decide on the log level for each individual log message for me right now?
Do the log levels bring additional benefits to the user of the cli?
Do we need to be able to report status to other outputs and formats than stderr (e.g. logfile)?
Do the benefits make up for the extra complexity and effort?

Initialize logger vs calling a function
Decide on the level for each individual message vs just reporting status
The maintainer needs to know about the logging library vs the maintainer needs to be able to call a function

Those and a couple more questions lead me to the conclusion that using the logging framework isn't the best solution right now.

Advantages of logging: 1- See where/when the logs occur 2- Log leveling 3- Logging to different outputs. While I found it necessary because of the 1st item, you found it unnecessary for the 2nd and 3rd items :)
Since it's a small project, we can leave it that way for now. Maybe later we can set a standard for all our python projects.

@umitbuyuksahin @Nicoretti I think, this will stay a case by case decision. We probably can create some guidelines, but not a standard. It depends on the type of project, cli, server, UDF, but also on size, runtime and complexity. For example, the container tool is a CLI, but has a high runtime and complexity, such that logging is necessary for diagnostic, however we don't want the user to see all of it, such that we log the details in a file and to console only warning and errors.


Recent change to left nav behaving strangely

Per this pull request, https://github.com/carbon-design-system/carbon-website/pull/1456, I updated the left nav heading Community to Community assets but the change is not sticking. Not sure what's happening here.
In Chrome, the new heading initially shows and then disappears. See the attached Quicktime movie.
community asssets.mov.zip
It works fine in Safari.

Seems like it's just a caching issue, I'm seeing the correct naming. Can you try holding shift and click the refresh icon?

Closing because this issue has been resolved 👍


Allow video input

The app now only allows an AudioDocument as input and a MMIF file with just a VideoDocument will be returned as is.
This means that users are responsible for creating an audio file before running and adding it to the MMIF file. It would be nice if that was not needed.

fixed in 9aa8248422998869173431f2602ae3ccf1546529


Crash on iOS 14.6

🐛 Bug Report
I have a user whose app randomly crashes when location sharing is enabled, but when location is denied the application starts up fine.
Expected behavior
Application should startup up even when location permissions are set to allowed.
Reproduction steps
Not sure, I only have the crash logs.
Configuration
Version:
geolocator: 9.0.0
** Trace **
Fatal Exception: NSInvalidArgumentException
-[CLLocation sourceInformation]: unrecognized selector sent to instance 0x282ec5060

Fatal Exception: NSInvalidArgumentException
0  CoreFoundation                 0x129754 __exceptionPreprocess
1  libobjc.A.dylib                0x287a8 objc_exception_throw
2  CoreFoundation                 0x2cc3c -[NSOrderedSet initWithSet:copyItems:]
3  CoreFoundation                 0x12c2ac ___forwarding___
4  CoreFoundation                 0x12e5b0 _CF_forwarding_prep_0
5  geolocator_apple               0x8218 (Missing UUID 720753cd252738f1be0ac8f3f52507b7)
6  geolocator_apple               0x75a4 (Missing UUID 720753cd252738f1be0ac8f3f52507b7)
7  geolocator_apple               0x7024 (Missing UUID 720753cd252738f1be0ac8f3f52507b7)
8  Flutter                        0x55a33c (Missing UUID 1aed3005d544309887cd044022e3ec31)
9  Flutter                        0x424e8 (Missing UUID 1aed3005d544309887cd044022e3ec31)
10 libdispatch.dylib              0x2a84 _dispatch_call_block_and_release
11 libdispatch.dylib              0x481c _dispatch_client_callout
12 libdispatch.dylib              0x12c70 _dispatch_main_queue_callback_4CF
13 CoreFoundation                 0xa3398 __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__
14 CoreFoundation                 0x9d270 __CFRunLoopRun
15 CoreFoundation                 0x9c360 CFRunLoopRunSpecific
16 GraphicsServices               0x3734 GSEventRunModal
17 UIKitCore                      0xbca584 -[UIApplication _run]
18 UIKitCore                      0xbcfdf4 UIApplicationMain
19 Runner                         0x6b0c main + 5 (AppDelegate.swift:5)
20 libdyld.dylib                  0x1cf8 start

I am also getting this error.

🐛 Bug Report
I have a user whose app randomly crashes when location sharing is enabled, but when location is denied the application starts up fine.
Expected behavior
Application should startup up even when location permissions are set to allowed.
Reproduction steps
Not sure, I only have the crash logs.
Configuration
Version: geolocator: 9.0.0
** Trace **
Fatal Exception: NSInvalidArgumentException
-[CLLocation sourceInformation]: unrecognized selector sent to instance 0x282ec5060

Fatal Exception: NSInvalidArgumentException
0  CoreFoundation                 0x129754 __exceptionPreprocess
1  libobjc.A.dylib                0x287a8 objc_exception_throw
2  CoreFoundation                 0x2cc3c -[NSOrderedSet initWithSet:copyItems:]
3  CoreFoundation                 0x12c2ac ___forwarding___
4  CoreFoundation                 0x12e5b0 _CF_forwarding_prep_0
5  geolocator_apple               0x8218 (Missing UUID 720753cd252738f1be0ac8f3f52507b7)
6  geolocator_apple               0x75a4 (Missing UUID 720753cd252738f1be0ac8f3f52507b7)
7  geolocator_apple               0x7024 (Missing UUID 720753cd252738f1be0ac8f3f52507b7)
8  Flutter                        0x55a33c (Missing UUID 1aed3005d544309887cd044022e3ec31)
9  Flutter                        0x424e8 (Missing UUID 1aed3005d544309887cd044022e3ec31)
10 libdispatch.dylib              0x2a84 _dispatch_call_block_and_release
11 libdispatch.dylib              0x481c _dispatch_client_callout
12 libdispatch.dylib              0x12c70 _dispatch_main_queue_callback_4CF
13 CoreFoundation                 0xa3398 __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__
14 CoreFoundation                 0x9d270 __CFRunLoopRun
15 CoreFoundation                 0x9c360 CFRunLoopRunSpecific
16 GraphicsServices               0x3734 GSEventRunModal
17 UIKitCore                      0xbca584 -[UIApplication _run]
18 UIKitCore                      0xbcfdf4 UIApplicationMain
19 Runner                         0x6b0c main + 5 (AppDelegate.swift:5)
20 libdyld.dylib                  0x1cf8 start

The minimum supported iOS version of this API（[CLLocation sourceInformation]） is iOS15

Was a cocoa pod configuration issue on my side

Hell all to resole this problem you can add  Background modes => Location update in Signing & Capabilities


⚠ Bofrost.be has degraded performance

In 5f3b7e6, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1286 ms

Resolved: Bofrost.be performance has improved in 8e90a46 after 6 minutes.


Further improved how the PMT is created WCSimDetectorConfig.cc

I now return the pointer to PMT with the CreatePMTObject command to make things more concise. I also added an exit command if an unknown type of PMT is specified. Finally, I cleaned up the indentation a bit in the WCSimDetectorConfigs.cc file.

OK this looks good. I will merge it.


Applying plugin with the plugins DSL

Is the extension released in a way that it can be used using Plugin DSL (https://docs.gradle.org/current/userguide/plugins.html#sec:plugins_block)
Example:
	plugins {
		id("com.google.cloud.tools.jib-ownership-extension-gradle")
                ...
	}

tried and the artifact cannot be found while jib is:
	plugins {
                ...
		id("com.google.cloud.tools.jib")
	}

thanks

No, the extensions are not Gradle plugins and not meant to be "applied." You just want the classes in the JAR file available (i.e., be on the classpath) inside the build script.

It's a good point. Plan was to put
id("com.google.cloud.tools.jib-ownership-extension-gradle") version("${jibOwnershipVersion}") apply false

So it wasn't applied but loaded into the classpath.
I'll keep my OCD aside and add the buildscript part as you propose in documentation:
buildscript {
  dependencies {
    classpath('com.google.cloud.tools:jib-ownership-extension-gradle:0.1.0')
  }
}


Add Plugin interface

Idea: The automatic-tracer would use this interface to automatically patch list of configured/provided modules i.e. Every instrumentation plugin would implement this interface.
Something like this.
// @example
export class RedisPlugin extends Plugin {
    constructor() {}
    enable<T>(moduleExports: T, tracer: Tracer): T {}
    disable(): void {}
}

Let me know if this makes sense or suggest an alternative.

Codecov Report

Merging #110 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #110   +/-   ##
=======================================
  Coverage   98.61%   98.61%
=======================================
  Files          26       26
  Lines        1732     1732
  Branches      193      193
=======================================
  Hits         1708     1708
  Misses         24       24


Add ; before printing TSIG

TSIG doesn't have a presentation format, when converting to a string
prefix the record with a semi-colon so it will be parsed as a comment.

Codecov Report

Merging #1051 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #1051   +/-   ##
=======================================
  Coverage   54.97%   54.97%
=======================================
  Files          41       41
  Lines        9872     9872
=======================================
  Hits         5427     5427
  Misses       3423     3423
  Partials     1022     1022

Impacted Files
Coverage Δ

tsig.go
40.84% <0%> (ø)
: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 aae7df6...e69d3ae. Read the comment docs.

Can we get a space after the semi-colon so it's slightly more readable?

good point. done

/merge


[IMPROVEMENT] Definition for lcm is unnecessarily complicated

[IMPROVEMENT] Simplify the definition for lcm
Description
Currently, lcm has the following snippet:
from functools import reduce
import math

def spread(arg):
  ret = []
  for i in arg:
    if isinstance(i, list):
      ret.extend(i)
    else:
      ret.append(i)
  return ret

def lcm(*args):
  numbers = []
  numbers.extend(spread(list(args)))

  def _lcm(x, y):
    return int(x * y / math.gcd(x, y))

  return reduce((lambda x, y: _lcm(x, y)), numbers)

This is hardly a 30-second snippet, especially if you are looking at it in isolation. Most of the code is tangential to calculating the LCM. It is also an unlikely usage pattern (I don't think anyone expects to use lcm as lcm([[5, 10], [4, 6, [2]], 8])). If this were to follow the pattern all other snippets follow, it would probably look like this:
from math import gcd
from functools import reduce

def lcm(*nums):
  def _lcm(x, y):
    return int(x * y / gcd(x, y))

  return reduce(_lcm, nums)

Makes some sense in general, PR it maybe? :+1:

Closing as resolved - although current implementation is a bit more verbose, it allows for some additional use-cases.


1.16.x: 1.1 - Register biomes at app start

Other mods assume that biomes are registered at app start, since OTG registers/unregisters biomes when loading/unloading worlds, this causes mod compatibility problems, with modded mobs/objects not spawning in OTG biomes. To fix this, we've discussed adding a new mode to OTG that allows users to define a single preset (world + dimensions), for which the biomes are registered at app start. While this mode is active, the selected preset should be the only useable preset and only its pre-defined dimensions can be created/deleted.
See also: #154.

See also: #275

Included for 0.0.1, all biomes for all presets are loaded and registered on app start. may need re-working when Forge biome registry / DeferedRegister works properly .


训练intermLM2_5_7b报错

pt-p1wvaor8-worker-0 logs: 07/05 15:59:30 - mmengine - WARNING - Dataset Dataset has no metainfo. ``dataset_meta`` in visualizer will be None.
pt-p1wvaor8-worker-0 logs: [rank2]: Traceback (most recent call last):
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/home/linjunpeng/reranker_llm/xtuner/xtuner/tools/train.py", line 360, in <module>
pt-p1wvaor8-worker-0 logs: [rank2]:     main()
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/home/linjunpeng/reranker_llm/xtuner/xtuner/tools/train.py", line 356, in main
pt-p1wvaor8-worker-0 logs: [rank2]:     runner.train()
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llmreranker/lib/python3.10/site-packages/mmengine/runner/_flexible_runner.py", line 1182, in train
pt-p1wvaor8-worker-0 logs: [rank2]:     self.strategy.prepare(
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llmreranker/lib/python3.10/site-packages/mmengine/_strategy/deepspeed.py", line 381, in prepare
pt-p1wvaor8-worker-0 logs: [rank2]:     model = self.build_model(model)
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llmreranker/lib/python3.10/site-packages/mmengine/_strategy/base.py", line 306, in build_model
pt-p1wvaor8-worker-0 logs: [rank2]:     model = MODELS.build(model)
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llmreranker/lib/python3.10/site-packages/mmengine/registry/registry.py", line 570, in build
pt-p1wvaor8-worker-0 logs: [rank2]:     return self.build_func(cfg, *args, **kwargs, registry=self)
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llmreranker/lib/python3.10/site-packages/mmengine/registry/build_functions.py", line 232, in build_model_from_cfg
pt-p1wvaor8-worker-0 logs: [rank2]:     return build_from_cfg(cfg, registry, default_args)
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llmreranker/lib/python3.10/site-packages/mmengine/registry/build_functions.py", line 121, in build_from_cfg
pt-p1wvaor8-worker-0 logs: [rank2]:     obj = obj_cls(**args)  # type: ignore
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/home/linjunpeng/reranker_llm/xtuner/xtuner/model/sft.py", line 84, in __init__
pt-p1wvaor8-worker-0 logs: [rank2]:     llm = self._dispatch_lm_model_cfg(llm, max_position_embeddings)
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/home/linjunpeng/reranker_llm/xtuner/xtuner/model/sft.py", line 225, in _dispatch_lm_model_cfg
pt-p1wvaor8-worker-0 logs: [rank2]:     llm_cfg = AutoConfig.from_pretrained(
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llmreranker/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 974, in from_pretrained
pt-p1wvaor8-worker-0 logs: [rank2]:     config_class = get_class_from_dynamic_module(
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llmreranker/lib/python3.10/site-packages/transformers/dynamic_module_utils.py", line 514, in get_class_from_dynamic_module
pt-p1wvaor8-worker-0 logs: [rank2]:     return get_class_in_module(class_name, final_module)
pt-p1wvaor8-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llmreranker/lib/python3.10/site-packages/transformers/dynamic_module_utils.py", line 213, in get_class_in_module
pt-p1wvaor8-worker-0 logs: [rank2]:     return getattr(module, class_name)
pt-p1wvaor8-worker-0 logs: [rank2]: AttributeError: module 'transformers_modules.internlm2_5-7b.configuration_internlm2' has no attribute 'InternLM2Config'

确认intermLM2.5为当前hf最新版本

transformers版本4.42.1，更换4.41.0后正常。

新的错误：
pt-5vf2t9a1-worker-0 logs: [rank2]: Traceback (most recent call last):
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/home/xxxxxxxx/llm/xtuner/xtuner/tools/train.py", line 360, in <module>
pt-5vf2t9a1-worker-0 logs: [rank2]:     main()
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/home/xxxxxxxx/llm/xtuner/xtuner/tools/train.py", line 356, in main
pt-5vf2t9a1-worker-0 logs: [rank2]:     runner.train()
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/runner/_flexible_runner.py", line 1182, in train
pt-5vf2t9a1-worker-0 logs: [rank2]:     self.strategy.prepare(
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/_strategy/deepspeed.py", line 394, in prepare
pt-5vf2t9a1-worker-0 logs: [rank2]:     self.param_schedulers = self.build_param_scheduler(
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/_strategy/base.py", line 658, in build_param_scheduler
pt-5vf2t9a1-worker-0 logs: [rank2]:     param_schedulers = self._build_param_scheduler(
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/_strategy/base.py", line 563, in _build_param_scheduler
pt-5vf2t9a1-worker-0 logs: [rank2]:     PARAM_SCHEDULERS.build(
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/registry/registry.py", line 570, in build
pt-5vf2t9a1-worker-0 logs: [rank2]:     return self.build_func(cfg, *args, **kwargs, registry=self)
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/registry/build_functions.py", line 294, in build_scheduler_from_cfg
pt-5vf2t9a1-worker-0 logs: [rank2]:     return scheduler_cls.build_iter_from_epoch(  # type: ignore
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/optim/scheduler/param_scheduler.py", line 787, in build_iter_from_epoch
pt-5vf2t9a1-worker-0 logs: [rank2]:     return cls(*args, begin=begin, end=end, by_epoch=by_epoch, **kwargs)
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/optim/scheduler/lr_scheduler.py", line 20, in __init__
pt-5vf2t9a1-worker-0 logs: [rank2]:     super().__init__(optimizer, 'lr', *args, **kwargs)
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/optim/scheduler/param_scheduler.py", line 759, in __init__
pt-5vf2t9a1-worker-0 logs: [rank2]:     super().__init__(
pt-5vf2t9a1-worker-0 logs: [rank2]:   File "/mnt/data/conda/envs/llm/lib/python3.10/site-packages/mmengine/optim/scheduler/param_scheduler.py", line 68, in __init__
pt-5vf2t9a1-worker-0 logs: [rank2]:     raise ValueError('end should be larger than begin, but got'
pt-5vf2t9a1-worker-0 logs: [rank2]: ValueError: end should be larger than begin, but got begin=0, end=0

log中scheduler：
pt-5vf2t9a1-worker-0 logs: param_scheduler = [
pt-5vf2t9a1-worker-0 logs:     dict(
pt-5vf2t9a1-worker-0 logs:         begin=0,
pt-5vf2t9a1-worker-0 logs:         by_epoch=True,
pt-5vf2t9a1-worker-0 logs:         convert_to_iter_based=True,
pt-5vf2t9a1-worker-0 logs:         end=0.12,
pt-5vf2t9a1-worker-0 logs:         start_factor=0.025,
pt-5vf2t9a1-worker-0 logs:         type='mmengine.optim.LinearLR'),
pt-5vf2t9a1-worker-0 logs:     dict(
pt-5vf2t9a1-worker-0 logs:         begin=0.12,
pt-5vf2t9a1-worker-0 logs:         by_epoch=True,
pt-5vf2t9a1-worker-0 logs:         convert_to_iter_based=True,
pt-5vf2t9a1-worker-0 logs:         end=4,
pt-5vf2t9a1-worker-0 logs:         eta_min=1.0000000000000002e-06,
pt-5vf2t9a1-worker-0 logs:         type='mmengine.optim.CosineAnnealingLR'),


Godot 3.0 texture import is extremely slow

Operating system or device - Godot version:
Windows 10, Godot v. 3.0 alpha build 1
Issue description:

it takes a very long time to import textures into Godot. This is especially true for large textures, 2k and up.
When we imported 5 textures in 4k resolution into our project, it took around 4-5 minutes to import them. Compared to Unity, Unreal engine and other 3d programs this is a very long time.
Steps to reproduce:
Import a few textures that's at least 2048x2048 in size.
Link to minimal example project:

Tested on Ubuntu 16.04, Godot_v3.0-alpha1
Importing 6 6k textures, it takes a while, but quicker than the 4-5 minutes.
Also noticed both on Windows and Ubuntu, after importing a texture, it gets reimorted  when applied to a model.

@Keetz the reimport happens because by default it imports Textures in 2D mode. But when Godot notices that it's used in 3D, it reimports with suitable 3D settings.

it's slow due to etc2comp, it needs to be disabled if you are not doing
mobile.
It gets reimported when applied to a model because it will recompress it to
video mem (by default it just uses regular png compression when importing)
On Thu, Aug 3, 2017 at 6:21 AM, Andreas Haas<EMAIL_ADDRESS>wrote:

@Keetz https://github.com/keetz the reimport happens because by default
it imports Textures in 2D mode. But when Godot notices that it's used in
3D, it reimports with suitable 3D settings.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/10059#issuecomment-319916124,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z26PJetGnaFezCqiSBjwnBMXw6YrNks5sUZEWgaJpZM4OsCSN
.

mhh, could we defer the etc2comp stuff until it's needed? (e.g exporting to mobile / debug on device)

@Hinsbart No, that would be pretty complicated, and you would hate it to have to wait 2 hours to test your project on a mobile device.
Eventually I think the best solution would be to move the reimporting of textures to a background thread.

@reduz how is etc2comp disabled?
Why do the etc2comp happen as default? Just a design choice or?
I see the practicality of this if you develop for mobile, but if not, it just becomes a real slowdown and annoyance, but this isn't a problem if it is easily disabled of course.
It seems more intuitive to import quickly, and then let the user decide how to compress if needed.

Is it possible to make importing of textures to run in a background? My biggest problem with new Godot importer is that whole engine becomes unresponsive when importing. Could it be possible to replace (re)imported textures temporally with some replacement texture? Maybe checkerboard one or some with importing written all over it?

@Keetz It probably should be an option for when you create the project, though if we make it happen on background it might not be too bad.
@MrMaidx
the initial import is usually fast because it uses the existing file as base compression (or should be), it's the reimport into vram that is slow. And yes, the idea is that it should run on a thread.. but one thing at a time :D. Need to make a stable release first, then will work on that

@reduz Thank you for a quick answer. I'm glad that it's planned. :)

@reduz Making it happen in background sounds good yea.
If by setting an option at the project creation means it can't be changed in the project later on i would say it's a no go. That would make it a big hurdle to export your game to multiple platforms.

For me as an artist it makes sense to have textures import as fast as possible, so I can try them on my shaders right away and then later make changes to import settings.
In Unity I just tried importing 7 images in 6k resolution and it took around 15 seconds. Same textures in Godot took around 10 minutes to import (at least on my Windows 10 machine).

Rasmus changing it later on means welcome to reimporting everything and
wait a few hours for it to finish :P
On Fri, Aug 4, 2017 at 10:51 AM, jesperkondrup<EMAIL_ADDRESS>wrote:

For me as an artist it makes sense to have textures import as fast as
possible, so I can try them on my shaders right away and then later make
changes to import settings.
In Unity I just tried importing 7 images in 6k resolution and it took
around 15 seconds. Same textures in Godot took around 10 minutes to import
(at least on my Windows 10 machine).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/10059#issuecomment-320254360,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z26MEFUn6N8JXtcAhzCC4aUFUyX1Mks5sUyHwgaJpZM4OsCSN
.

I added a note to the first post, on how this issue probably should be addressed.

This call:
Ref<Texture> tex = ResourceLoader::get_singleton()->load(image, "Texture");

For a 1200x1200, 990kb PNG takes ~160ms on a really fast machine with a fast SSD (Samsung EVO Pro).
It seems to me like that's too high, not sure if it's related to this issue. I tried looking into etc2comp but I'm not sure how you disable it, I can find the thirdparty library files but no references inside Godot's source.
I feel like this is an important issue that should be looked at for 3.2?

@eligt To disable etc2comp, uncheck Rendering → Vram Compression → Import Etc2 in the Project Settings.

@Calinou I did that but it does not seem to have any effect on the ResourceLoader::get_singleton()->load call at run-time.

@eligt I believe large textures will take a while to be uploaded to the GPU if they're not in a VRAM-compressed format (uncompressed, WebP or PNG). PNG is also relatively slow to decode compared to an uncompressed format like TGA.

@Calinou I feel like 100+ milliseconds just to load the texture is way too much though, even when factoring in PNG decompression and loading in VRAM, considering my hardware:
Processor (CPU)	  Intel® Core™ i7 Quad Core Processor 7700HQ (2.8GHz, 3.8GHz Turbo)
Memory (RAM)	  32GB Corsair 2400MHz SODIMM DDR4 (2 x 16GB)
Graphics Card	          NVIDIA® GeForce® GTX 1060 - 6.0GB GDDR5 Video RAM - DirectX® 12.1
1st M.2 SSD Drive	  1TB SAMSUNG PM961 M.2, PCIe NVMe (up to 3000MB/R, 1700MB/W)

I think that this was one of the reasons why I was on/off with Godot. I would leave godot for up to an hour to just load my textures into a material. My focus is primarily on Art.
When I disabled Rendering -> Vram Compression -> Import Etc 2, all of my textures would import really fast. That was the turning point for me. This reason IS the primary reason why I am now able to STAY in Godot. Now I can drink clean water (Godot) instead of going back to Mountain Dew (ahemunreal) to quench my thirst.
I think it would be really cool to have a dialog box to specify the target platform of your project. This way the artists who have no clue as to what compression is, etc, will be able to adapt to Godot easy and be retained. Just my two cents :)

@phyffer Could you define "really fast"? Like, how many textures of what size and how long do they take to load? I've disabled all texture compression and load times are still what I'd consider slow, look at my above comment for reference.

@eligt Here are my results:
Machine : cpu @ i8700k
4 textures are:
1 AO+Rough+Metallic,
1 Normal,
1 Albedo
1 Grayscale Mask
Time: 2048 x 2048 * 4 PNG Textures
< 1 Second to import
< 1 second to use in a spatial material
Time: 4096 x 4096 * 4 PNG Textures
~40 Seconds
Time: 4096 x 4096 * 4 PNG Textures to replace 2048 PNGs already used in a spatial material
< 9 Seconds to import
Compared to etc2 on, it definitely made Godot workable. But it is definitely similar to results you are getting

godot 3.2 Still not resolved.
I still hope to make some useful updates, such as addressing the slow import of resources, and the performance optimization of GD scripts. At present, it takes about an hour to import 1G resources.  However, the resource editor file system browser of the 700M-1G import is now broken, and nothing will be displayed. File list, this may be the shortcomings of the editor itself, so I think that optimizing and solving the existing problems is the better way for Godot. If I can only import 100-500M resources, then it is destined that Godot cannot develop large games , But my path is now blocked and I cannot move forward. Hope that the next version will solve such problems, I wish Godot better and better!

@qq715152910 Have you disabled Import Etc2 in the Project Settings?

@qq715152910 Have you disabled Import Etc2 in the Project Settings?
Either way, asset importing will be significantly faster in Godot 4.0.

I only enable the s3 tc and etc options, but not etc2, because I am working on a 2d project, and the project resources are large and large, so I try to subcontract the resources and package them into PCK / ZIP one to one, Unfortunately, the following error occurs when using the load method call after loading:
NOTE:
Loading textures is normal and can be used normally, the program will not crash, but there will be multiple consecutive errors below, similar to #35179
CODE:
ProjectSettings.load_resource_pack("res://scene/1464.zip")
var index = 0
for h in 14:
    for l in 18:
	var texture = load("res://scene/1464/map/%04d.png" % (index+1))
	index = index + 1

ERROR:
**> E 0: 00: 00.518 FileAccessWindows :: _ get_modified_time: Failed to get modified time for: D: /godot/Samples/game/scene/1464/map/0001.png.

<C ++ Error> Method failed. Returning: 0
   <C ++ source program> drivers \ windows \ file_access_windows.cpp: 344 @ FileAccessWindows :: _ get_modified_time ()
    app.scene.map.gd:33 @_ready ()**

I've run into some textures that can take near-infinite time to import.  There's a loop of a  loop of a loop with some very large values here:
            for (int iRed2 = iMinRed2; iRed2 <= iMaxRed2; iRed2++)
            {
                for (int iGreen2 = iMinGreen2; iGreen2 <= iMaxGreen2; iGreen2++)
                {
                    for (int iBlue2 = iMinBlue2; iBlue2 <= iMaxBlue2; iBlue2++)
                    {

iMaxRed2 = 252645152, iMaxGreen2 = 252645152, iMaxBlue2 = 252645152
This code looks suspect:
		int iMaxRed2 = iColor2Red + (int)a_uiRadius;
		if (iMaxRed2 > 15)
		{
			iMinRed2 = 15;
		}

Should it be setting iMinRed2 there or iMaxRed2 there?
Here's a texture that exhibits the problem:

If you import it as a normal map with just the red and green channels, it imports.  If you import as a RGB texture with compression, Godot freezes for some undetermined amount of time.
I also entered an issue on the etc2comp github, but it looks like that hasn't been touched in years: https://github.com/google/etc2comp/issues/46

@jitspoe Note that you don't need to use ETC2 compression if you're only targeting desktop platforms. In this case, disable Import Etc2 in the Project Settings to speed up import times.

Here's a texture that exhibits the problem:

If you import it as a normal map with just the red and green channels, it imports. If you import as a RGB texture with compression, Godot freezes for some undetermined amount of time.
What exact import settings are you using? I can't reproduce a freeze here with this texture.

Is this before or after your change? :D
I think this is what I was using:

Is this before or after your change? :D

I'm testing with 3.2.1-stable to be sure :)
I set the same settings that you have but I still get an instant import for that texture. :/
In any case, the problematic code that you found definitely seems bogus, and the huge max values you had are likely due to uninitialized variables, so it could explain why it doesn't happen reliably.

That's interesting.  I was having this issue back when I was using 3.1, and sometimes I'd have a texture that wouldn't import, so I'd load up 3.2 to import it, then go back to 3.1.  I just assumed it was a bug that had been fixed, so I never entered an issue.  Now that I've fully migrated over to 3.2, I was surprised to see it again.  Perhaps it's something specific to my project settings?


no-unsafe-mutable-readonly-assignment should whitelist Array.from

Ditto Array.of

Be careful of this: https://github.com/danielnixon/eslint-plugin-total-functions/issues/730

Should also whitelist immediate assignment from an array spread, like this:
      // eslint-disable-next-line total-functions/no-unsafe-mutable-readonly-assignment
      const foo: readonly string[] = [...new Set(["a"] as const)];


fix fix test_query_sys_tables schema_privileges regression case.

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run p1


Fix test for fetching css from remote source

Use create-test-server or nock or something similar to respond with either CSS or an error.

Will be closed with #97


trackDependency is not in typings

Hi,
The trackDependency method is not present in the typings file of DefinitelyTyped.
The trackAjax call is defined but I assume it is deprectated? The "method" property isn't filled in when you use trackAjax so it is not exactly the same.
Code:
        AppInsights.prototype.trackAjax = function (id, absoluteUrl, pathName, totalTime, success, resultCode, method) {
            this.trackDependency(id, null, absoluteUrl, pathName, totalTime, success, resultCode);
        };

Kind Regards,

I sent a pull request to DefinitelyTyped repo with a fix.


fix: merge beta to main

fix: merge beta to main

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: