Problem in Post Editing Experiment

Hello,
By following the tutorial, I am trying to make basic experiments with NMT tool but even though I copied and pasted directly from the tutorial page, I couldn't run the described post editing experiment.
I am receiving parse value exception ("Failed to load INI File: Error on line 21:  Cannot parse value: "Vocabulary.from_dataset"). The line 21 describes the class=Vocabulary.from_dataset variable in configuration file.
Could you help?
ps : I am using the gpu version of tensorflow. And it seems like both TF and NeuralMonkey are installed correctly; I can import TensorFlow and NeuralMonkey from REPL.

Hi,
vocabulary should be with small v.
Cheers,
Jindra
Dne pátek 4. listopadu 2016 kadir-gunel<EMAIL_ADDRESS>napsal(a):

Hello,
By following the tutorial, I am trying to make basic experiments with NMT
tool but even though I copied and pasted directly from the tutorial page, I
couldn't run the described post editing experiment.
I am receiving parse value exception ("Failed to load INI File: Error on
line 21: Cannot parse value: "Vocabulary.from_dataset"). The line 21
describes the class=Vocabulary.from_dataset variable in configuration file.
I am using the gpu version of tensorflow. And it seems like both TF and
NeuralMonkey are installed correctly; I can import TensorFlow and
NeuralMonkey from REPL.
Could you help?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ufal/neuralmonkey/issues/133, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABwcs5FSCsSOQXxwsSf2lRR_GmHo5pOWks5q6xeMgaJpZM4KpcKU
.


enh: sparkle-ization of app design

Use Tab instead of subNavigation + simple title so that it can be closed
Rollout buttons and icons
Removes the old Button components

cc @Duncid I'm not super fan of the title + tab below but it's not that bad. Given that it's not our core focus I presume it's good enough but let me know if you have some ideas on that.

Super nit, but is the disabled state of the deploy button as expected ?

Yes => cc @Duncid

(@Duncid already noted that it's a bit meh meh)

Fixes #1220


Added cplusplus extern C scope

I've added an #ifdef for __cplusplus which sets the linkage to C-style, so this library can be used in C++.

Thanks


[High] Server crashes on big sites (otto.de)

Environment details  (please complete the following information):
JDN v 3.14.38 Back-end v 0.2.61
win10
Describe the bug
Server crashes on big sites (otto.de)
To Reproduce
Steps to reproduce the behavior:

Open https://www.otto.de/
Open JDN
Press Generate All button

Actual behavior
Server crashes. See log here: ServerCrash.txt
Expected behavior
No crashes

Validated on JDN v 3.14.49 Back-end v 0.2.63

Fixed.


JSS throws StackOverFlow exception when trying to resolve NameExpr

Hi,
I am trying to find out if a NameExpr resolves to a primitive type or not. In general it finds out type with the exception of unchecked types.
For example, trying to find out the the resolved type for the IfStmt:
 if (ind != numSamples) {                 samples.set(ind, key);               }
on line 194 of InputSampler.java from Hadoop branch 3.0.0 always crashes (it enters an infinite loop on JavaParserFactory line 114:
return getContext(parentNode, typeSolver);
The complete input file (inside Hadoop branch 3.0.0) is hadoop-branch-3.0.0/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/lib/InputSampler.java
Is there a way around this problem?
I am trying to resolve the type using the below approach:
						SymbolReference<? extends ResolvedValueDeclaration> rRef = JavaParserFacade 								.get(new ReflectionTypeSolver()).solve(n.getName());
where n is the left or right expression of an IfStmt...
As I said, it works in all my other cases, except the above...
Thanks in advance for any advice or perhaps a bug fix?

Hi! Please move this to https://github.com/javaparser/javaparser/issues because this project is now part of the main project :-)

Okay, well, don't know how to "move" it so I closed it here and opened it on the other tracker...


Add key binding note for macOS

Hi Jim, nice to see you here :-).
good note! I will edit it a bit as I think we should only suggest the meta key; the ctrl mapping makes sense too but is a bit too opinionated I guess :-)


About the  config file pseudo-distributed.ini

Dear Sir:
I try to modify the file desktop/conf/pseudo-distributed.ini, it seems that the modification do not come to effect。 And I try to   # Webserver listens on this address and port
http_host=<IP_ADDRESS>
http_port=9000
convert the port into 9000, but I still can only use localhost:8000 to browse the web.

Oh I see, I have to execute build/env/bin/supervisor in order to make the .ini file work.


Create Lyrics “the-difference”

Automatically generated by Netlify CMS

:construction_worker: Deploy Preview for thar-ai processing.
:hammer: Explore the source changes: c00f1b0b823aaf947bd17c1c2e5467e51fd6a476
:mag: Inspect the deploy log: https://app.netlify.com/sites/thar-ai/deploys/60ec2901b3864100089eb17e


Enable Pod Based Enricher and Node Aggregator

All test pass (default is broken for other reason), merging to uncomment some other e2e tests, @piosz please review in the morning your time.

LGTM


Allow that use if USB connected GPS

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Summary
Allow the use of USB connected GPSes. If the SX1302 HAL is used as-is, then it only enables UBlox binary messages on the UART port of the GPS. Even if the binary messages is enabled manually and saved to GPS flash, at startup the HAL disables binary messages on all ports except UART.
What is the use-case?
Adding a USB GPS to systems that doesn't ship with an integrated GPS, or that ships with a non Ublox GPS.
Implementation description
Fork the SX1302 HAL so that modifications can be made. The maintainers for the HAL cater for a very ideal world use case, and anything that diverge from that ideal case is ignored.
Once forked, during GPS init, enable ublox messages on all GPS ports, not just UART1
Can you implement this by yourself and make a pull request?
No, as the HAL needs to be forked.

Actually, I already made some changes which makes this possible using gpsd (https://gpsd.gitlab.io/gpsd/). This was needed to support the latest Multitech Conduit firmware.
By directly using the /dev/ttyXXXX, only one process can read from the GNSS module. Using gpsd, multiple processed can consume from the gpsd. In the case of the Multitech Conduit firmware, the gpsd initializes the GNSS module on boot (including the UBlox binary messages for GPS time), and the Concentratord can directly consume from the gpsd, including receiving the UBlox binary messages (in this case, Concentratord / the HAL doesn't do any GNSS setup).
This is the commit which added support for gpsd to the sx1302: https://github.com/chirpstack/chirpstack-concentratord/commit/54aa840b605715f6784f62f343905908e91ead42.

Adding a USB GPS to systems that doesn't ship with an integrated GPS

Please note that this is likely not going to work as expected. The PPS (pulse per second) pin of the GNSS module must be connected to the SX1301/2/3 for accurate scheduling of downlinks and for the fine-timestamp.

... or that ships with a non Ublox GPS.

Please note that the reason why the Semtech HAL requires a Ublox module is because it can expose the GPS time using the binary Ublox format. The NMEA data gives you the UTC time, which includes leap-seconds. The GPS time does not include leap seconds and for this reason Class-B scheduling is based on GPS time, not UTC time.
Supporting non-Ublox modules would therefore require more work as it requires setting up the GNSS module to expose GPS time (if the module supports this at all), as well as parsing the non-standard NMEA output of this GNSS module.

This is the commit which added support for gpsd to the sx1302: 54aa840.

I saw that, but couldn't figure out what the difference is, as it still references a /dev/ttyXXX path.

Please note that the reason why the Semtech HAL requires a Ublox module is because it can expose the GPS time using the binary Ublox format. The NMEA data gives you the UTC time, which includes leap-seconds. The GPS time does not include leap seconds and for this reason Class-B scheduling is based on GPS time, not UTC time.

Yes, I have been through this before. My board(waveshare hat) has a GPS. The PPS pin is connected to the SX1302, it outputs valid ublox data, but the RMC message isn't standard and is rejected by HAL.
So I wanted to add a USB Ublox GPS to get the GPS time, for ClassB messages, letting the onboard GPS provide the PPS, but the USB GPS providing coordinates and GPS time.
Looks like it will be simpler to just stick with the modified HAL and the default Semtech packet forwarder.
PSA: Stay away from the Waveshare Lora hat.

Btw, I have opened this issue: https://github.com/Lora-net/sx1302_hal/issues/92. I'm not sure if that will be accepted or not. If not, maybe it could be something to include in the Concentratord. In this case, maybe there isn't any need to use the Semtech HAL for GNSS reading / parsing, it could be done directly using Rust.

I saw that, but couldn't figure out what the difference is, as it still references a /dev/ttyXXX path.

It is an enum which could point to a path, to use gpsd you would set it to this:
https://github.com/chirpstack/chirpstack-concentratord/blob/master/chirpstack-concentratord-sx1302/src/config/vendor/multitech/mtac_003e00.rs#L176
In this case it will connect to the localhost gpsd port.

Btw, I have opened this issue: Lora-net/sx1302_hal#92. I'm not sure if that will be accepted or not. If not, maybe it could be something to include in the Concentratord. In this case, maybe there isn't any need to use the Semtech HAL for GNSS reading / parsing, it could be done directly using Rust.

That would be great. Either implemented in HAL, or even better, HAL's GPS code just ignored and implemented in concentratord.

It is an enum which could point to a path, to use gpsd you would set it to this:
https://github.com/chirpstack/chirpstack-concentratord/blob/master/chirpstack-concentratord-sx1302/src/config/vendor/multitech/mtac_003e00.rs#L176

Ahh, thanks. That now makes sense. Will try to implement that on my gateway.

Just to let you know, we came to the conclusion that using the time from the NMEA data is not accurate enough. A binary protocol is needed for accurate time synchronization. Please see the above issue.


Indexed view page should have more examples

The documentation of indexed view, partitioned views should have more examples, use cases and more significant information about how can I update materialized information (or it is automatically updated)
a good sample that what I would know: https://www.postgresql.org/docs/current/rules-materializedviews.html

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e7bfcbf2-4dbb-6a41-4390-40d6bc076606
Version Independent ID: 2992dd77-8134-c2ea-8957-3a696583a2c4
Content: Views - SQL Server
Content Source: docs/relational-databases/views/views.md
Product: sql
Technology: table-view-index
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

@ohrlando -- Orlando, thank you for your thoughtful feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@WilliamDAssafMSFT -- William, please look into this issue.

Hello, @WilliamAntonRohm. Thank you for the answer
I found some answers before in forums but there is no reference to official doc, because there is few information there. I would to know how indexed view works and I'm suggesting that it is a good information to see at documentation (like postgres doc does).
Is it possible to add this information? not just examples, but more information like "Do I need to update it frequently?". When the docs says "They are not well-suited for underlying data sets that are frequently updated" is that because I need to manually update? Thoses questions that I need to search in forums, but there there is no official references there.
thank you very much

Hi @Ohrlando-
Thanks for taking the time to provide this feedback. We have examples of creating indexed views in a different article here:
https://docs.microsoft.com/sql/relational-databases/views/create-indexed-views
And partitioned views:
https://docs.microsoft.com/sql/t-sql/statements/create-view-transact-sql#partitioned-views
Indexed views do not need to be manually updated, they physically exist and are transactionally maintained, and so create some overhead for writes to underlying tables. This is why indexed views on frequently-updated tables may hit write performance on the underlying tables.
Thanks again for this feedback, your suggestions help improve our documentation.


Only Shows when in group

I want to practice with this solo but I cannot get it to show unless I am in a group, any reason?  I have the option for always show checked.

Hey!  Unfortunately that’s a feature of Grid2.  I haven’t been able to figure out a work around.  You could reach out to the add on author though.

It's only something with your profile, when I create my own I don't have
that issue.
On Sun, Jul 11, 2021, 5:03 PM tolienbosheit @.***>
wrote:

Hey! Unfortunately that’s a feature of Grid2. I haven’t been able to
figure out a work around. You could reach out to the add on author though.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tolienbosheit/TolienGrid2-Profile/issues/1#issuecomment-877867565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUZ5KMZWB2QBZCOBB736ADLTXIIKFANCNFSM5AFVB5TQ
.

I think I may be misunderstanding your issue.   If you want the profile to show while solo you need to enable a display under General => Layouts => By Group Type => Solo.  By default it is set to none.  I have heard some issues with this displaying since prepatch and am working with the addon developer to try to identify why without a complete rebuild of the profile.

Thanks for the responses, that worked for me.  Is there any way to display
water shield? that's the last thing i am missing.
On Mon, Jul 12, 2021 at 1:25 PM tolienbosheit @.***>
wrote:

I think I may be misunderstanding your issue. If you want the profile to
show while solo you need to enable a display under General => Layouts => By
Group Type => Solo. By default it is set to none. I have heard some issues
with this displaying since prepatch and am working with the addon developer
to try to identify why without a complete rebuild of the profile.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tolienbosheit/TolienGrid2-Profile/issues/1#issuecomment-878496824,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUZ5KM7I7JOTSQ7ROX7GTVDTXMXS7ANCNFSM5AFVB5TQ
.

Hey! Sorry, I just saw your last comment about water shield.  I hope you got an answer, but if you didn't I would recommend using a WA.  I have the link to mine in the Shaman Discord pins (same post as this profile).


Add .yarnclean

This saves us 8.55 MB of valuable disk space. This not only saves disk space, but also has the advantage of faster cache on the CI.
λ yarn autoclean --force
yarn autoclean v1.22.0
[1/1] Cleaning modules...
info Removed 2423 files
info Saved 8.55 MB.
✨  Done in 2.23s.

https://classic.yarnpkg.com/en/docs/cli/autoclean

This is one of the best features of yarn 🙂


Add setuptools as optional dependency

This PR is alternative to #5753. I think this is superior because:

projects which uses not-setuptools build system will not have unneeded dependency
this is standard way how to manage dependencies which are optional - setuptools is needed only for building .c files.

Related to https://github.com/cython/cython/issues/5751

Can you elaborate on on which circumstances you need setuptools to run Cython? I have projects that use Cython on Python 3.12 without installing setuptools.

Can you elaborate on on which circumstances you need setuptools to run Cython?

setuptools is needed when one wants to build shared library. For compiling to .c file setuptools is not needed

I think the easiest solution is to have setuptoiols as an optional dependency for a cythonize variant. Something like:

This is already done in the PR with the same aproach you mentioned in an example.

I really don't want to have my projects depends (transitively) on setuptools after all the work that went into getting rid of it.

This PR is not adding hard dependency rather than optional one. In this PR when you install cython via pip install cython setuptools is not installed.

setuptools is needed when one wants to build shared library. For compiling to .c file setuptools is not needed

I don't think this is correct.

This is already done in the PR with the same approach you mentioned in the example.

Except that the variant is called setuptools and not cythonize. I think cythonize is a much better choice as the fact that cythonize requires setuptools is an implementation details that may change.

This PR is not adding hard dependency rather than optional one. In this PR when you install cython via pip install cython setuptools is not installed.

This PR raises ImportError when setuptools is not present to a code path that is traversed by code that does not require setuptools, making setuptools a requirement when before it was not.

Furthermore, setuptools is required only on Python >= 3.12, thus the optional dependency should probably be
[options.extras_require]
cythonize =
    setuptools;python_version>='3.12'

setuptools is needed when one wants to build shared library. For compiling to .c file setuptools is not needed

I don't think this is correct.

If not please correct me.

Except that the variant is called setuptools and not cythonize. I think cythonize is a much better choice

Yes I agree but I think this is more about how we name thinks not that implementation/approach is wrong. If there will be consensus among core devs on some name I have no problem changing it.

This PR raises ImportError when setuptools is not present to a code path that is traversed by code that does not require setuptools, making setuptools a requirement when before it was not.

which code path? Could you be please more specific? cython command is working without setuptools:
(cython312) matus@MacBook-Pro-2240:~/dev/cython/test$ python -V
Python 3.12.0
(cython312) matus@MacBook-Pro-2240:~/dev/cython/test$ pip freeze
-e git+https://github.com/matusvalo/cython.git@b0552058e178c0adab4d42fbdcf900284bbb95a7#egg=Cython
(cython312) matus@MacBook-Pro-2240:~/dev/cython/test$ cython -h
usage: cython [-h] [-V] [-l] [-I INCLUDE_PATH] [-o OUTPUT_FILE] [-t] [-f] [-v] [-p] [--cleanup GENERATE_CLEANUP_CODE] [-w WORKING_PATH] [--gdb] [--gdb-outdir GDB_OUTDIR] [-D] [-a] [--annotate-fullc] [--annotate-coverage ANNOTATE_COVERAGE_XML] [--line-directives]
              [-+] [--embed] [-2] [-3] [--3str] [--lenient] [--capi-reexport-cincludes] [--fast-fail] [-Werror] [-Wextra] [-X NAME=VALUE,...] [-E NAME=VALUE,...] [--module-name MODULE_NAME] [-M]
              [sources ...]

Cython (https://cython.org/) is a compiler for code written in the Cython language. Cython is based on Pyrex by Greg Ewing.

cythonize, pyximport and other tools raises an ImportError with message to install setuptools.
if you do know about any usage of Cython not using distutils/setuptools which fails in this PR please let me know.

which code path? Could you be please more specific?

You added an import setuptools to a source file that dind't contain it or import distutils before. The source file in question is imported, for example, when cython is run with the -M argument.

You added an import setuptools to a source file that dind't contain it or import distutils before. The source file in question is imported, for example, when cython is run with the -M argument.

Yes I missed that. I wanted to avoid check that directly in every file. Hence we can either:

check every distutils import
check at main functions called by commands

Thank you @dnicolodi for your review.

AFAIK, the only user visible interfaces that require distutils are cythonize and pyximport. If you really want to add this check, please add it to the main entry points for these facilities, ie the cythonize main function and to the pyximport module. Do not sprinkle it everywhere.

AFAIK, the only user visible interfaces that require distutils are cythonize and pyximport

I was thinking about that see comment about but there is also cython.inline and Jupiter notebook integration.

Repeating my comment from a now closed PR:
Just adding a dependency on setuptools is not a solution for making distutils importable
Setuptools uses this distutils-precedence.pth "trick" to hijack sys.path. That only runs on startup if setuptools is installed as a system site-package.
It doesn't run if setuptools is located through PYTHONPATH.
If you want to make it more robust, you have to use the alternative sys.path hijack, by importing setuptools before importing distutils.
But if you do that, you can just as well make a proper fix and stop using distutils altogether.

and stop using distutils altogether.

Not sure if we don't break someone who is not using setuptools and has python < 3.12. To be backward compatible, I am afraid we need to do this kind of dance:
https://github.com/cython/cython/blob/459f7c6236b7507b664703e6a20aafafe22f8a21/Cython/Distutils/build_ext.py#L16-L23

add it to the main entry points for these facilities, ie the cythonize main function and to the pyximport module.

Agreed for pyximport, which definitely depends on distutils. But cythonize has several modes of operation, only some of which actually build an extension module using distutils/setuptools. It would be great if we could also avoid keeping -M and the plain .c file generation from importing distutils/setuptools and avoid failing if it's not available.

the variant is called setuptools and not cythonize. I think cythonize is a much better choice

I disagree because cythonize is ambiguous. The cythonize() function shouldn't depend on setuptools, and several operations of the cythonize command could also work without it. We may well end up with two or more build implementations that our tools use, and setuptools would probably remain one of them for quite a while. That makes it appear quite reasonable to install Cython[setuptools] if you want your build to use setuptools.

@scoder I was thinking about the cythonize command line tool, not the function. However, there are more interfaces that depend on distutils that I initially thought about, therefore using cythonize as a variant does not seem appropriate. However, using setuptools as a variant name does not seem right either. In this case it is simpler to just install setuptools alongside cython:
python -m pip install cython[setuptools]

is not any simpler than
python -m pip install cython setuptools

it is simpler to just install setuptools alongside cython

The difference is that listing it in setup.cfg makes it available as package metadata, thus listing setuptools explicitly as an optional dependency. Otherwise, you'd have to find out yourself that Cython can make use of setuptools.

I don't think there is a way to discover optional dependencies that is not through documentation. AFAIK, variants are not listed anywhere on PyPI or via any installation tool. Am I missing something?

Technically, setuptools is required only with Python 3.12 and later. That is one reason why I was proposing to name the variant something else than setuptools and have it depend on setuptools conditionally to the Python version, so that installation instructions do not need to be diversified per Python version. https://github.com/cython/cython/pull/5754#issuecomment-1752050969

I don't think there is a way to discover optional dependencies that is not through documentation. AFAIK, variants are not listed anywhere on PyPI or via any installation tool. Am I missing something?

So, you are suggesting that we should not expose this metadata in any of our wheel distributions because PyPI is currently missing a very common feature of package indices and because you haven't used any tools yet that evaluate optional package dependencies?
I think we should expose it. And allow users to decide whether they want to use it or not. As you say, it makes almost no difference in usage, so we can just as well make it explicit.

I wonder, instead of changing all of our imports, can't we just try-import setuptools and then rely on distutils being there magically? Probably something for Cython 3.1 rather than 3.0.x…

Yeah, that's a valid way to deal with it, also works when setuptools is in PYTHONPATH.

We also struggled with the best way to handle this issue while fixing up Python 3.12 support for CFFI... Glad to see you've reached the same main conclusion that we did: a hard dep on setuptools is almost certainly the wrong approach. At least for CFFI, there are common situations (eg cryptography) that have a runtime cffi dep, but only need setuptools/distutils at build-time. Forcing everything that uses cffi to take an implicit setuptools dep would've been easy and convenient, but arguably creates a lot more problems than it solves.
CFFI's setuptools / distutils imports were limited enough that we ended up rolling our own internal shim module that basically exposes the various imports we need on a single module, but provides more prescriptive error messaging for users/devs that are missing a dep beyond ModuleNotFoundError: no module named 'setuptools'. I didn't love it, but it was at least simple...
We'd debated the extras_require as well. It would allow any given Cython version to (optionally) specify a supported/tested setuptools range that would be opaque to consumers just depending on eg Cython[setuptools]>=3.1- the same technique could work for both runtime and build-time deps, and at least pip seems to just warn if you ask for an extra spec that doesn't exist (eg if the extra was dropped for some reason in the future). We ended up not doing it for the moment, but knowing that at least modern versions of pip just warn on a missing extra instead of failing, I'm warming to the idea for CFFI as well.
It's a minor point, but there's also the build-time UX around PEP517 build requirements- requiring folks to do requires=['Cython[setuptools]'] in their pyproject.toml isn't so bad, but still suboptimal over just declaring Cython and letting it figure out if setuptools is needed from there. I can't remember off the top of my head if Cython provides its own native PEP517 build backend or does any shimming of the setuptools backend (some of which would be required to dynamically augment the build-time deps to completely hide setuptools), or if all the magic still occurs down at the legacy setuptools/distutils command level. I've been playing with a setup.py-free rewrite of PyYAML's libyaml extension build with custom in-tree backend shims over both setuptools and Cython >= 3.0, and while I can make it work, it feels "icky" and brittle in some ways. OTOH, I understand Cython not wanting to provide a full PEP517 backend as long as most of the setuptools integration is still happening down at the distutils/command layer (I feel the same way right now about trying to implement a native PEP517 backend for CFFI).


when will 0.2 releases?

Thanks for paying attention to Phantoscope. All features about 0.2.0 have been developed.
We have already prepared docs about api changes and feature introduction in Chinese version, but new preview front-page supporting and docs in English remains not ok.
Look forward to the release of the new version, which should be an easier-to-use and more powerful version.

thinks!


Wrong length zero bytes for UTF-8 encoding strings

Data+String.swift
in case when comment have encoding .utf8 and we looking for zero bytes end of string we must wait ONE zero byte not two
two zero bytes only for UTF16
// unicode strings are terminated by \0\0, while latin terminated by \0
your comment is wrong only utf16 strinds are termonated by \0\0
http://id3.org/id3v2.4.0-structure
Frames that allow different types of text encoding contains a text
encoding description byte. Possible encodings:
 $00   ISO-8859-1 [ISO-8859-1]. Terminated with $00.
 $01   UTF-16 [UTF-16] encoded Unicode [UNICODE] with BOM. All
       strings in the same frame SHALL have the same byteorder.
       Terminated with $00 00.
 $02   UTF-16BE [UTF-16] encoded Unicode [UNICODE] without BOM.
       Terminated with $00 00.
 $03   UTF-8 [UTF-8] encoded Unicode [UNICODE]. Terminated with $00.

Do you know if this is for all versions, or just for 2.4? In 2.3, it just mentions "unicode" but doesn't differentiate between UTF-8 and UTF-16.
All numeric strings and URLs [URL] are always encoded as ISO-8859-1. Terminated strings are terminated with $00 if encoded with ISO-8859-1 and $00 00 if encoded as unicode.

http://id3.org/d3v2.3.0

I've just pushed an update which makes this change. Please let me know if this works better for you.

Yes, you are right about versions before 2.4
Your fix correctly forking for me, thanks!


Fix link to Freer Monads

Thanks.

As I said https://github.com/michaelt/freemonad-benchmark/pull/1 freer has an easy way to add the customary representation of state actions
data S s r where
      Get_ :: S s s
      Put_ :: s -> S s ()

implemented in my dubious fork here https://github.com/michaelt/freemonad-benchmark/blob/master/Freer.hs#L39  This is what free-vl is using, for example, and oleg mentions it. Extensible effects  doesn't use it because it needs a functor which S s isn't. It can be emulated with Lan.


local model problem

Hi! Because of network restrictions, I can only use local models. Can I use a local model rather than LLAMA_INFERENCE_SERVER_URL?

LLAMA_INFERENCE_SERVER_URL can be used for local execution, e.g., it can be <IP_ADDRESS>:some_port
To be compatible with the APIs, we have to use the local server setup here: https://github.com/GATECH-EIC/mg-verilog/tree/main/inference_server_setup

Thank you so much! Do I have to use docker to build a server or I can use other URL instead?  (I cant reach huggingface in my own server).

We have an internal doc for setting up the inference server without docker, which is adapted from the official text generation inference repo
But we observe that this setup is highly unstable and significantly slower in generation speed as compared to the docker version.
However, regardless of using docker or not, you have to get the pre-trained weights of LLM from somewhere? Either from hugggingface or other frameworks ... Potentially, you can manually download the weights somehow and put it in the huggingface default cache directory? You will have to look into the resources of getting the pretrained model from huggingface and where it's stored by default

Thanks! That helps a lot.


Error in publishing Third Party APIs to external Dev Portal

Description:
An error occurred when trying to publish an Third Party API to an external dev portal/portals

Errors occur after tick the external devportal and click save.
Following error can be found in the terminal
[2022-03-22 16:26:04,573]  WARN - OAuth2AuthzEndpoint No session state found for the received Session ID : 4d23fb27-616e-48cb-aa59-3d3e611726c2
[2022-03-22 16:26:31,856] ERROR - WSO2APIPublisher Import API service call received unsuccessful response: Bad Request
This combination of host and port requires TLS.
status: 400
[2022-03-22 16:26:31,857] ERROR - APIProviderImpl org.wso2.carbon.apimgt.api.APIManagementException: Import API service call received unsuccessful response: Bad Request
This combination of host and port requires TLS.
status: 400
[2022-03-22 16:26:31,857] ERROR - GlobalThrowableMapper Failed to publish API to external Store.
[2022-03-22 16:26:45,322] ERROR - WSO2APIPublisher Import API service call received unsuccessful response: Bad Request
This combination of host and port requires TLS.
status: 400
[2022-03-22 16:26:45,323] ERROR - APIProviderImpl org.wso2.carbon.apimgt.api.APIManagementException: Import API service call received unsuccessful response: Bad Request
This combination of host and port requires TLS.
status: 400
[2022-03-22 16:26:45,407] ERROR - WSO2APIPublisher Import API service call received unsuccessful response: Bad Request
This combination of host and port requires TLS.
status: 400
[2022-03-22 16:26:45,408] ERROR - APIProviderImpl org.wso2.carbon.apimgt.api.APIManagementException: Import API service call received unsuccessful response: Bad Request
This combination of host and port requires TLS.
status: 400
[2022-03-22 16:26:45,411] ERROR - GlobalThrowableMapper Failed to publish API to external Store.

Hi @Tharanidk, I was not able to recreate this issue on latest APIM pack. (carbon apimgt version 9.20.59)
https://user-images.githubusercontent.com/28379317/159482868-528d38f4-24d6-4429-8ea6-c222c0e45fee.mov

This issue could be reproduced when trying to push a non-revisioned third party API to an external store.


Open external links in new tab and add utm parameters + Rename twitter to X

This is not an issue per say but an enhancement that I would like to contribute and before working on a PR wanted feedback from the maintainers.
External links when clicked upon redirect the user away from the site. It would be nice to open these links in new tab and add utm source parameters.
Example: Pastebin, Chat, Community and Social links in header, footer, user profile etc
Using X instead of twitter. Twitter word is being used in profile, socials etc.
@driesvints Please let me know what you think

Heya, in general I'm not a fan of adding target="_blank" to links as I think it should be a user choice to open these in a new tab.
What would the benefit for us be to add the utm source params?
Afaik we already use X everywhere?

Heya, in general I'm not a fan of adding target="_blank" to links as I think it should be a user choice to open these in a new tab.

Makes sense 👍

What would the benefit for us be to add the utm_source params?

For social and chat links no benefit as such but for community links and paste bin link it can be helpful for analytics

Afaik we already use X everywhere?

Not everywhere, here are some screenshots:

Maybe change the X Twitter naming to "X (Twitter)" and the direct links to x.com indeed 👍

Hi @driesvints
I have sent a PR #1170 for this.
Also I noticed that form labels display the name attribute in the slot rather than what's in the slot.

The label should be X (Twitter) handle rather than Twitter. Is it intentional or it's a bug?

I fixed the label bug in Blade UI Kit itself.


[Suggestion] Allow server to have unlimited client slots

It would be a cool feature if you could set the maxClientCount parameter in the Server.Start method to like 0 and this would mean that the server doesn't have a max client limitation.
This would be pretty useful in some applications. Like if you dont want the server to be a game server.

Well due to the fact that client IDs are a ushort, there's going to be a limit of 65535 clients anyways, but you'll almost certainly hit hardware and bandwidth limitations long before you reach that.
What are you trying to use this for that requires (and could theoretically handle) unlimited clients?

I'm trying to write a master server where users can register their own server to. And the master server lists all the servers currently registered and gathers their ping and some informations. So i think a master server shouldn't have any max clients limitations i guess, because the master server should obviously list all the servers available.
I'm not sure if this is the right way to create a master server tho. Do you think this is the right approach?

If you just want a server list, you should probably use a web server or something for that. No need for fast UDP transfer if that's all you're trying to do.
Also, you'd probably end up with multiple master servers for various regions, as listing every server in the world for users in completely different regions wouldn't make much sense. If you need more than 65k connections at once (assuming you have the resources for it, which is unlikely in more general game-server use cases), you could also create a second instance of the Server class and use that to listen on a separate port.

Yeah you're right. I'll see what i can do about that. Thanks for the idea with multiple master servers for various regions, i haven't thought about that to be honest.


debug dataframe

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi @lfjbb, this extension provides a command R: Preview Dataframe. Perhaps that does what you want? There is also an issue already open about an environment/variable viewer (#47). If neither of these are what you have in mind, could you provide some more information? Thank you.


Table.add will cost much time when add a big object (15M)

console.time('params');
          putParams({
            start: startIndex,
            column: {
              sheet: sheetIndex,
              key: columnIndex,
            },
            sheets,
            type: FIELD_TYPE_MAPPING[field],
            noCash: true,
          })
          .then((id) =>
              validateWorkerPromise.postMessage({ id }),
            )
          .then((error) => {
            columnSettingValidated({
              benchmarkId,
              field,
              error,
            });
          });
          console.timeEnd('params');

I have this code,  the putParams is actually call add method, when the params is very big, this code will block the process (almost 2000ms) even it is a async code, this dose not make sense to me.

Hard to tell when only watching app code an no Dexie-facing code.
However, I hope you are not indexing any big data property? This will lead to expremely slow performance. Should typically only index small strings and numbers and never images, movies or sound files. How is your table indexed?

Can you share the code where you define your tables and indexes? (db.version(1).stores ({...})

I don't think this is the Dexie.js issue, because even I use original indexeddb api which the browser provide it is slow also, anyway, thank you very much.


Typing error: is incompatible with protocol "Hashable"

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
vscode pylance/pyright errors with:
error: Argument of type "MyConfig" cannot be assigned to parameter "args" of type "Hashable" in function "__call__"
    "MyConfig" is incompatible with protocol "Hashable"
      "__hash__" is an incompatible type
        Type "None" is not assignable to type "() -> int" (reportArgumentType)

Example Code
from functools import lru_cache

from pydantic import BaseModel, ConfigDict

class MyConfig(BaseModel):
    model_config = ConfigDict(frozen=True)

    read_timeout: float = 60

@lru_cache
def client(mc: MyConfig):
    pass

client(MyConfig())

Python, Pydantic & OS Version
pydantic version: 2.10.2
        pydantic-core version: 2.27.1
          pydantic-core build: profile=release pgo=false
               python version: 3.10.15 (main, Sep  7 2024, 00:20:06) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]
                     platform: macOS-14.6.1-arm64-arm-64bit
             related packages: typing_extensions-4.12.2
                       commit: unknown

Static type checkers have no knowledge of the Pydantic model_config attribute, so they won't be able to understand that the class is hashable (because it is frozen). As shown in the class example of the typing specification, you can use class arguments:
from functools import lru_cache

from pydantic import BaseModel

class MyConfig(BaseModel, frozen=True):
    read_timeout: float = 60

@lru_cache
def client(mc: MyConfig):
    pass

m = MyConfig()

client(m)


Error in keyd-application-mapper script.

/usr/bin/keyd-application-mapper:465: SyntaxWarning: invalid escape sequence '\W'
  return re.sub('[\W_]+', '-', s).strip('-').lower()
ERROR: could not find app.conf, make sure it is in ~/.config/keyd/app.conf

The error occurs when running the keyd-application-mapper script. The configuration is also at the correct location at: ~/.config/keyd/app.conf.

Same here, maybe this extra data helps
Using Fedora 39 and GNOME 45.3 and Hyprland as Window manager
during installation I had this prompt:
make & sudo make install [1] 5282 mkdir bin          cp scripts/keyd-application-mapper bin/                                                 sed -e 's#@PREFIX@#/usr/local#' keyd.service.in > keyd.service                                                                                cc -DVERSION=\"v2.4.3\ \(ccd17ad\)\" -I/usr/local/include -L/usr/local/lib -Wall -Wextra -Wno-unused -std=c11 -DSOCKET_PATH=\"/var/run/keyd.socket\" -DCONFIG_DIR=\"/etc/keyd\" -DDATA_DIR=\"/usr/local/share/keyd\" -D_FORTIFY_SOURCE=2 -D_DEFAULT_SOURCE -Werror=format-security  -O3  src/*.c src/vkbd/uinput.c -lpthread -o bin/keyd                                                                                mkdir -p /etc/keyd mkdir -p /usr/local/bin/ mkdir -p /usr/local/share/keyd/ mkdir -p /usr/local/share/keyd/layouts/ mkdir -p /usr/local/share/man/man1/ mkdir -p /usr/local/share/doc/keyd/ mkdir -p /usr/local/share/doc/keyd/examples/ groupadd keyd install -m755 bin/* /usr/local/bin/ install -m644 docs/*.md /usr/local/share/doc/keyd/ install -m644 examples/* /usr/local/share/doc/keyd/examples/ install -m644 layouts/* /usr/local/share/keyd/layouts src/daemon.c: In function ‘add_listener’:                                          src/daemon.c:84:33: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]                                                 84 |                                 write(con, layer->type == LT_LAYOUT ? "/" : "+", 1);                        |                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                                src/daemon.c:85:33: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]                                                                                85 |                                 write(con, layer->name, strlen(layer->name));                                                                                |                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                          src/daemon.c:86:33: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]                                                 86 |                                 write(con, "\n", 1);                                                                                |                                 ^~~~~~~~~~~~~~~~~~~                                             install -m644 data/*.1.gz /usr/local/share/man/man1/

Gnome 45 broke older extensions. The latest commit should have support for it, though you will need to follow the instructions produced by keyd-application-mapper and install the new extension.

i have this problem too in archlinux using hyprland.

I'm getting the same issue, maybe an r needs to be added to it? r'[\W_]+'


Support custom client and transport for the Flutter SDK

Hi, Currently we are using the self-hosting Sentry back-end.
Due to security concerns, our devOps asked me to add some extra properties to request headers.
Im tried implements a BaseClient and pass it to the option, but it doesn't work.
  if (kReleaseMode) {
    await SentryFlutter.init(
      (options) {
        options.dsn =
            'http://##########dnsLink########/2';
        // Set tracesSampleRate to 1.0 to capture 100% of transactions for performance monitoring.
        // We recommend adjusting this value in production.
        options.httpClient = MySentryHttpClient;
        options.tracesSampleRate = 1.0;
      },
      appRunner: () => runApp(const MyApp()),
    );
  } else {
    runApp(
      const MyApp(),
    );
  }

my_sentry_http_client.dart
class MySentryHttpClient extends BaseClient {
  final Client _httpClient = Client();

  MySentryHttpClient();

  @override
  Future<StreamedResponse> send(BaseRequest request) {
    var defaultHeaders = <String, String>{
      'Content-Type': 'application/json; charset=UTF-8',
      "Extra-Property": "tokenABC",
    };
    request.headers.addAll(defaultHeaders);
    return _httpClient.send(request);
  }
}

Sorry for my broken english and thank you for everything....

@KeterVM we'll look into it, thanks.

The Java SDK makes it possible having a ITransportFactory and a RequestDetails parameter, so users could override the transportFactory property in the SentryOptions and mutate the requestDetails, we could do the same here.
For the Dart SDK that would be fine but for the Flutter SDK, we'd need to sync down this configuration to the Android and iOS SDK, the iOS SDK currently does not support this feature either, so we are blocked for now.
@philipphofmann @brustolin to confirm.
@KeterVM your option should work for the Dart SDK but not for the Flutter SDK.

Adding extra HTTP headers should be possible with SentryOptions.urlSessionDelegate. Can you confirm @brustolin?

@brustolin can you confirm that?

Adding extra HTTP headers should be possible with SentryOptions.urlSessionDelegate. Can you confirm @brustolin?

No, not possible. URLSession Delegate is to respond to state changes, not mutate the request. Currently is not possible to do this in the iOS SDK.


refactory with typescript ,and use tsc or  babel  to  compatible old node

See #492 and #497. We're not likely to switch the whole thing to typescript, but do want to move the code (gradually) forward into the future. Cheers!


Write documentation for this project

[ ] Overview
[ ] Preview Images
[ ] Installation #3
[ ] Create CODE_OF_CONDUCT.md
[ ] Create CONTRIBUTION.md

Hi!
I am a beginner here and
I would like to create CODE OF CONDUCT and CONTRIBUTION.md for this repo

sure @the-amazing-atharva you can work on it. I've created an issue #4 . if you are not sure what to add in CODE_OF_CONDUCT.md, just find a project that has one and use it. Let me know if you need something else.

Ok!


Revert "Removes GVKRequired struct"

Reverts operator-framework/operator-controller#400

Still need the -s/Signed option.
So, why are we reverting this?


P1 content bug: 1459745

Docs for <bdi> are misleading.
Acceptance criteria
https://bugzilla.mozilla.org/show_bug.cgi?id=1459745 is fixed.

Docs page is updated and reviewed, and the bug has been closed.


Remove DashboardController, addresses #125

DashboardController is now fully in Sufia. So remove it and its unit tests, but leave a feature test for future-proofing.

This did not require an upgrade to sufia in the Gemfile?  Thanks for following up with this.

Nope. The DashboardController hasn't changed much.


AppArmor disabled

towards https://github.com/giantswarm/giantswarm/issues/24677
pspUseAppArmor set to false in values.yaml to enable grafana-app to be deployed.

Tested on both WC and MC and working well


Add compatibility with lines that start with "0" or "::"

As an example, currently running ghosts on the following would generate no output:
:: www3.doubleclick.net
0 1app.blob.core.windows.net

I didn't find the ip parameter at the time apparently. However having ghosts detect common host prefixes by default would be nice, such as <IP_ADDRESS>, 0, and ::.


Support for unpacking sprites

A lot of the bundles I'm extracting have sprite atlases in them and it's pretty tedious to manually cut them out and rotate them to the correct position.  I've been looking for an automated way to do this with no luck.
https://docs.unity3d.com/Manual/SpritePacker.html
How difficult would this be?

If you can find the spatial and rotational metadata for each sprite then it could be added as a utility method, iff it's consistent.

I think it would be consistent since (according to the link above) it's handled by Unity automatically.  If I export the YAML for one such bundle, there are some dictionaries that contain spriteDefinition metadata:
!!python/object/apply:collections.OrderedDict
- - - m_GameObject
    - !PPtr [0, -1300960676681644065]
  - [m_Enabled, 1]
  - - m_Script
    - !PPtr [0,<PHONE_NUMBER>048467122]
  - [m_Name, '']
  - [version, 3]
  - [materialIdsValid, 1]
  - [needMaterialInstance, 0]
  - - spriteDefinitions
    - - !!python/object/apply:collections.OrderedDict
        - - [name, Fry]
          - - boundsData
            - - !!python/object/apply:collections.OrderedDict
                - - [x, 0.004999995231628418]
                  - [y, -0.004999995231628418]
                  - [z, 0.0]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 2.309999942779541]
                  - [y, 3.309999942779541]
                  - [z, 0.0]
          - - untrimmedBoundsData
            - - !!python/object/apply:collections.OrderedDict
                - - [x, 0.004999995231628418]
                  - [y, -0.004999995231628418]
                  - [z, 0.0]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 2.309999942779541]
                  - [y, 3.309999942779541]
                  - [z, 0.0]
          - - texelSize
            - !!python/object/apply:collections.OrderedDict
              - - [x, 0.009999999776482582]
                - [y, 0.009999999776482582]
          - - positions
            - - !!python/object/apply:collections.OrderedDict
                - - [x, -1.149999976158142]
                  - [y, -1.659999966621399]
                  - [z, 0.0]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 1.159999966621399]
                  - [y, -1.659999966621399]
                  - [z, 0.0]
              - !!python/object/apply:collections.OrderedDict
                - - [x, -1.149999976158142]
                  - [y, 1.649999976158142]
                  - [z, 0.0]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 1.159999966621399]
                  - [y, 1.649999976158142]
                  - [z, 0.0]
          - - normals
            - []
          - - tangents
            - []
          - - uvs
            - - !!python/object/apply:collections.OrderedDict
                - - [x, 0.0009770507458597422]
                  - [y, 0.0009770507458597422]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 0.1137690395116806]
                  - [y, 0.0009770507458597422]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 0.0009770507458597422]
                  - [y, 0.1625971645116806]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 0.1137690395116806]
                  - [y, 0.1625971645116806]
          - - normalizedUvs
            - - !!python/object/apply:collections.OrderedDict
                - - [x, 0.0]
                  - [y, 0.0]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 1.0]
                  - [y, 0.0]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 0.0]
                  - [y, 1.0]
              - !!python/object/apply:collections.OrderedDict
                - - [x, 1.0]
                  - [y, 1.0]
          - - indices
            - [0, 3, 1, 2, 3, 0]
          - - material
            - !PPtr [0, -139266242881986336]
          - [materialId, 0]
          - [sourceTextureGUID, f7e9e7708ec334bd596cfa71cd2c503d]
          - [extractRegion, 0]
          - [regionX, 0]
          - [regionY, 0]
          - [regionW, 0]
          - [regionH, 0]
          - [flipped, 0]
          - [complexGeometry, 0]
          - [physicsEngine, 0]
          - [colliderType, 0]
          - - customColliders
            - []
          - - colliderVertices
            - []
          - - colliderIndicesFwd
            - []
          - - colliderIndicesBack
            - []
          - [colliderConvex, 0]
          - [colliderSmoothSphereCollisions, 0]
          - - polygonCollider2D
            - []
          - - edgeCollider2D
            - []
          - - attachPoints
            - []

The data for the Texture2D itself looks like this:
!unitypack:Texture2D
image data: !!binary |
  <image data>
m_ColorSpace: 1
m_CompleteImageSize: 4194304
m_Height: 2048
m_ImageCount: 1
m_IsReadable: false
m_LightmapFormat: 0
m_MipCount: 1
m_Name: Cards-1
m_ReadAllowed: true
m_StreamData: !unitypack:StreamingInfo {offset: 0, path: '', size: 0}
m_TextureDimension: 2
m_TextureFormat: 12
m_TextureSettings: !!python/object/apply:collections.OrderedDict
- - [m_FilterMode, 1]
  - [m_Aniso, 1]
  - [m_MipBias, 0.0]
  - [m_WrapMode, 1]
m_Width: 2048

However, I don't see how the Texture2D asset points to the spriteDefinitions or vice versa.

This would need to be explored more closely then. If you can figure it out, I recommend you write a script that uses unitypack to extract the stripes, then we can figure out if we can upstream it :)

OK, cool - I'll keep looking into it and see if I can rope some others into helping.  Thanks!

I realised that I missed some of the data off at the bottom of the sprite definitions object.  I've added it in now and it clearly states which Texture the items map to in that data at the bottom:
  - - textures
    - - !PPtr [0,<PHONE_NUMBER>669429443]

That PPtr references the Texture2D's path_id.


[🐞] QWIK ERROR Error: must be non null

Which component is affected?
Qwik Runtime
Describe the bug
After upgrading to 0.22.0 Qwik throws QWIK ERROR Error: must be non null
This is what gets printed
QWIK ERROR Error: must be non null     at createError (file:///[redacted]/node_modules/@builder.io/qwik/core.mjs:74:17)     at logError (file:///[redacted]/node_modules/@builder.io/qwik/core.mjs:68:54)     at must (file:///[redacted]/node_modules/@builder.io/qwik/core.mjs:7831:15)     at serializeSubscription (file:///[redacted]/node_modules/@builder.io/qwik/core.mjs:7718:21)     at file:///[redacted]/node_modules/@builder.io/qwik/core.mjs:3106:24     at Array.map (<anonymous>)     at _pauseFromContexts (file:///[redacted]/node_modules/@builder.io/qwik/core.mjs:3102:14)     at beforeClose (file:///[redacted]/node_modules/@builder.io/qwik/server.mjs:515:30)     at file:///[redacted]/node_modules/@builder.io/qwik/core.mjs:3608:28     at file:///[redacted]/node_modules/@builder.io/qwik/core.mjs:4087:25 null
When I try to log at serializeSubscription with:
...
else if (type <= 2) {
        console.log('----------------------------------------', sub[0]);
        console.log(2, getObjId(sub[2]), sub[2]);

        base += ` ${must(getObjId(sub[2]))} ${must(getObjId(sub[3]))} ${sub[4]}`;
    }
...

I get
2 null SignalDerived {
  '$func$': [Function (anonymous)],
  '$args$': [
    SignalImpl {
      untrackedValue: 'https://..... 100w,\n' +
        'https://i......jpg 200w',
      [Symbol(proxy manager)]: 0,
      [Symbol(proxy manager)]: [LocalSubscriptionManager]
    }
  ],
  '$funcStr$': 'p0.value'
}

I am testing the still unreleased Image component of gioboa - it might be something wrong with the component but I still believe this error is very strange and does not give any information. I still do not understand what is wrong the component. If I remove the assignment srcSet={srcSetSignal.value} the error is no more.
Reproduction
none
Steps to reproduce
I have no idea how to reproduce this.
System Info
System:
    OS: Linux 5.19 Ubuntu 22.10 22.10 (Kinetic Kudu)
    CPU: (4) x64 Intel(R) Core(TM) i5-6400T CPU @ 2.20GHz
    Memory: 17.99 GB / 31.25 GB
    Container: Yes
    Shell: 5.2.2 - /usr/bin/bash
  Binaries:
    Node: 18.12.1 - ~/.nvm/versions/node/v18.12.1/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 9.4.2 - ~/.nvm/versions/node/v18.12.1/bin/npm
  Browsers:
    Chrome: 111.0.5563.64
    Chromium: 111.0.5563.64
  npmPackages:
    @builder.io/qwik: ^0.23.0 => 0.23.0
    @builder.io/qwik-city: ^0.6.5 => 0.6.5
    @builder.io/qwik-react: ^0.5.0 => 0.5.0
    undici: 5.21.0 => 5.21.0
    vite: 4.2.0 => 4.2.0

Additional Information
No response

Can you provide a piece of code, or even better a stackblitz that reproduces the issue?
Without a repo case, i can't prioritize this issue to get it fixed!

@manucorporat I faced similar problem while learning qwik. Here's stackblitz https://stackblitz.com/edit/qwik-starter-xscsrx?file=src/routes/index.tsx

Hey Manu, sorry for my delayed response. I tried several times to reproduce this in stackblitz but i couldn't. Since the error is so obscure I am not sure that i understand exactly what is null and how to make it not null - or how to reproduce it.
Hopefully the stackblitz provided by @bhadreshp5 is the same issue as mine and that will help you debug it.

+1, It only happens when upgrading from 0.21.0 to 0.22
Looks like it has to do something with conditional chaining and a value is resolved to null or undefined.

I just got this error with my local Qwik/QwikCity build (latest stuff). I traced it to a signal in my JSX.
<img
            src={imagePathSignal.value}
            alt={`${product.name}`}
            class="h-full w-full object-cover object-center transition group-hover:opacity-75"
          />

Switched to this (trimmed the .value off imagePathSignal.value), and it works:
<img
            src={imagePathSignal}
            alt={`${product.name}`}
            class="h-full w-full object-cover object-center transition group-hover:opacity-75"
          />

hth

@manucorporat, I just wanted to

@manucorporat I faced similar problem while learning qwik. Here's stackblitz https://stackblitz.com/edit/qwik-starter-xscsrx?file=src/routes/index.tsx

@manucorporat, Please let us know if there is anything else needed.

Just got the same issue, can't seem to get rid of it... All i did was rename some variables and add another item to initial state. Seems to be some kind of caching issue now

removing any useSignals seems to fix it for me


JavaScript for-loop colouring broken

VSCode Version: Code - Insiders 1.5.0-insider (55168276e4c0b6ec2404ac20af04eeed5f5b1000, 2016-08-17T06:08:10.227Z)
VSCode Version: Code 1.4.0 (6276dcb0ae497766056b4c09ea75be1d76a8b679, 2016-08-04T16:45:31.680Z)
OS Version: Windows_NT ia32 10.0.10586

This occurs in both the latest nightly and release.
For-loops with a particular condition and arrangement of spaces cause the coloring to break from that point onward. Example code is below and attached screenshot - the last loop breaks.
The format has to be:

Less then symbol
No spaces
Variable (foo, foo.bar. foo['bar'] is okay)
One or more spaces
; character

If you change any one of those, as in the example below, the colouring is fine. It's almost as though it's seeing what looks like the start of an HTML tag and getting confused.
for (var i = 0; i <400; i++) { // Start loop
    doSomething();
}

for (var i = 0; i < foo ; i++) { // Start loop
    doSomething();
}

for (var i = 0; i < foo; i++) { // Start loop
    doSomething();
}

for (var i = 0; i <foo; i++) { // Start loop
    doSomething();
}

for (var i = 0; i <foo ; i++) { // Start loop
    doSomething();
}

@aeschli Assigning to you as this might require a fix on our side. IMHO the Javascript.tmLanguage.json grammar should not contain any jsx-* rules which are triggered by this sample code.

@alexandrudima It's on purpose that for JavaScript we use the TypeScriptReact grammar.
React Native uses the .js extension (not .jsx) and we want it to work out of-the-box.
The issue is a duplicate of #11.


Keyboard shortcut for command palette stops working

Issue Type: Bug
Keyboard shortcut for the command palette often fails on Mac. Have to restart Mac to bring it back. Issue is only in VSCode.
When I try to re-assign keybaord shortcut to "show all commands" as shift+cmd+p (because it has stopped working), the P will not register, just "shift+cmd+". P key works fine on Mac in all other apps.
Keyboard shortcut fails after some time -- a few hours. Not sure how to reproduce exactly. Just leave VSCode open for a day and you should notice it. Only started doing this in last 2 or 3 versions of VSCode.

Just tried the default keyboard shortcut for showing the command palette (cmd+shift+p) in VSCode on my home Mac (I was using my work Mac when I reported this), and it also fails. My workaround is to use another shortcut, but hopefully this bug will get fixed instead. Again, my "P" key works fine in general and in all other apps.

Having issues with shortcuts and command pallete here too.
Current version (no updates available). Mac OS.
Given this js:
function getDataForYear(year){
    const sheetUrl = sheets[`${year}`];  try {
        const sheetData = await fetch(sheetUrl)
        const jsonArray=await csv().fromString(sheetData)
    } catch (error) {
           console.log(error)
        console.log(`Failed to fetch sheet data for ${year}`)
    }
    return jsonArray;     }

This is the only code in the file. The file extension is js.
After setting keyboard shortcut for format document to Sh+Cm+L (no conflicts) and then pressing the combination, nothing happens. Then I press Sh+Cm+P and select Format Document and nothing happens.

@jcollum @peterneely sorry for reaching out late. Could you please help follow the wiki https://github.com/microsoft/vscode/wiki/Keybinding-Issues to help provide more information to help troubleshoot?

@rebornix I'm watching this issue because I have the exact same problem; macOS 10.14 on a MacBook Pro, with external Apple keyboard. I just ran through the "Troubleshoot Mac Keybindings" points with the following results. Nothing too helpful I'm afraid, but I am more than happy to run any other kind of test.

Are you using a custom installed keyboard layout with its own sub-layouts?

No

Are you using an external ISO keyboard attached to a laptop with an ANSI keyboard?

Yes, I guess? (Apple bluetooth keyboard). I tried my MacBook Pro's built-in keyboard and the same issue exists

Are you using some keyboard layout that remaps more than the printable characters?

No

An important action is not mapped to a good keybinding by default.

I suppose that's a way of describing the issue? (command-shift-p should open the Command Palette but doesn't.)

try: use "keyboard.dispatch": "keyCode" in your settings and restart VS Code

I made this change and restarted VS Code, without improvement.

I've also tested a second MacBook Pro and it's fine on that one. Cmd-shift-P works without issue.

@rebornix, here are my replies to those questions.
Troubleshoot Mac Keybindings

Are you using a custom installed keyboard layout with its own sub-layouts?

No

Are you using an external ISO keyboard attached to a laptop with an ANSI keyboard?

I'm using the Apple wireless keyboard (with and without number pad, both US models).

Are you using some keyboard layout that remaps more than the printable characters?

No

An important action is not mapped to a good keybinding by default.

Not applicable since I'm using a US QWERTY keyboard.

Experiencing any other issue?

I tried using "keyboard.dispatch": "keyCode" in settings and restarted VS Code. No joy.

@rebornix, here is a gif showing the problem.
When I go to Preferences > Keyboard Shortcuts > edit the shortcut for "Show All Commands" > press Shift Cmd P, VSCode won't capture the P key when it follows Shift Cmd. All other keys, and the P key, work in all other cases. This is the one case where the P key simply won't register that it was pressed.
In the gif, I'm pressing P several times after Shift Cmd, but ... nothing.

@rebornix, also, this behavior is the same on the built-in MacBook Pro keyboard; it's not just a wireless keyboard problem.

@rebornix, here is a gif showing the problem.
When I go to Preferences > Keyboard Shortcuts > edit the shortcut for "Show All Commands" > press Shift Cmd P, VSCode won't capture the P key when it follows Shift Cmd. All other keys, and the P key, work in all other cases. This is the one case where the P key simply won't register that it was pressed.
In the gif, I'm pressing P several times after Shift Cmd, but ... nothing.

The same thing happened here.
Troubleshoot Mac Keybindings

Are you using a custom installed keyboard layout with its own sub-layouts?

No

Are you using an external ISO keyboard attached to a laptop with an ANSI keyboard?

I'm using a Mac Mini, so I do have a wired keyboard (with number pad).

Are you using some keyboard layout that remaps more than the printable characters?

No
VSCode Info
1.36.0 (1.36.0)

2019-07-03T12:59:35.697Z

@rebornix, does this issue still need the label needs more info?

same issue.

I've noticed something else about this: sometimes when the format doesn't happen it's because there is a syntax error in the code.
It's very easy to miss it when this happens, the UX could be improved.

@jcollum Please open a separate issue. It appears you have trouble with formatting JS files, which is unrelated to the cmd+shift+p issue the OP has.

@peterneely @jeff-h @rosuH
The fact that the keybinding is not registered in the input box means that we are not receiving the keybinding in VS Code. From my dealing with such things in the past, I think it means that the OS or some 3rd party software is eating it up. When cmd+shift+p is not working in VS Code, does it work in other software on your machine? What is the log when pressing cmd+shift+p at https://www.w3.org/2002/09/tests/keys.html in Chrome on the same machine?
Here is what I get on my machine, where cmd+shift+p always works fine:

I would share more.
in my case, it looks like (not sure) it disfunction, after I open a large file(which will not tokenize by default).
my extensions includes vim and Prettify JSON.

@viix Does VSCode receive cmd+shift+p for you?

no.
On Wed, Aug 7, 2019 at 10:43 PM Alexandru Dima<EMAIL_ADDRESS>wrote:

@viix https://github.com/viix Does VSCode receive cmd+shift+p for you?
[image: keyboard-input]
https://cloud.githubusercontent.com/assets/5047891/25478333/fe31c830-2b40-11e7-865c-b2c3bb10dd75.gif
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/75525?email_source=notifications&email_token=ACC2JY33L5QQBY2CN5CJLPLQDLNSFA5CNFSM4HYLQIUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3YUGJY#issuecomment-519127847,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACC2JY5AJC75QWOGK4ENRETQDLNSFANCNFSM4HYLQIUA
.

--
*Best Regards *此致
Nate Guo 郭云鹏

@alexandrudima no.

@alexandrudima Wow, https://www.w3.org/2002/09/tests/keys.html is illuminating. You're right, cmd+shift+p does NOT register there. I feel bad blaming VS Code! :)
Despite cmd+shift+p having no apparent effect on my Mac, the above link prompted me to go digging. First I checked System Preferences > Keyboard > Shortcuts but the combination wasn't defined there. Next I started checking all the other apps I run which have global keyboard shortcuts. And what do you know, I use an app called TickTick which has registered this combination.
Their release of 3.0.10 on Jun 12, 2019 includes the note "Added several new shortcut keys " and correlates suspiciously with the creation date of this issue :)
So I removed the shortcut in Ticktick and of course, it now works in VS Code again! 👍

You are right! The TickTick has registered this combination. And I launch it when the OS login...

Thanks for following through. I will close this issue since a 3rd party is eating up the keybinding, as I suspected :)

@jeff-h, you're the bomb! That was my problem -- and fix -- too.
I wish Mac had a centralized directory of all registered keyboard shortcuts so that these issues are easier to track down. Not the first time this has happened.

Does any OS have that? It would be nice...


this week’s GLAD update

Thanks for the update! Unfortunately I'm getting an error from cartodb: Pgraster Plugin: unable to lookup available table overviews due to unknown schema  encountered during parsing of layer 'layer0' in Layer.


feat: Upgrade platformadmin's yurtappset dependencies to v1beta1

What type of PR is this?
/kind feature
What this PR does / why we need it:
Upgrade IoT component platformadmin's yurtappset dependencies to v1beta1
Which issue(s) this PR fixes:

Fixes #1941
Special notes for your reviewer:

@luc99hen @LavenderQAQ

iot's e2e has been fixed with pr #2110.

I don't want to codecov error in unit tests to prevent this pull request, so i will merge this pull request at first, and we will solve the problem of codecov lately.

/LGTM


Upgrade lxml to 4.5

Whether we want to have it or not, we do as of tonight, b/c it came along with the latest judge-pics. Ugh. Check and see what kind of trouble this may cause for Juriscraper as a top priority.

This seems to have gone down without a hitch. Going to deploy shortly.


Root project not yet available if subproject listed in settings.gradle is missing.

Expected Behavior
I apply an initialization script that uses settingsEvaluated and it.rootProject against a project whose settings.gradle includes a project that doesn't exist. Gradle should either ignore the missing sub-project (since the build itself still works in this case), or it should provide a more informative error message than the current "The root project is not yet available for build.", i.e. "The root project is not yet available for build due to a missing sub-project .".
Current Behavior
None of --info, --debug and --stacktrace are useful in identifying the root cause of this problem and the current error message is not very informative, so it's hard to debug this problem without understanding the gradle lifecycle.
"* What went wrong:
The root project is not yet available for build."
Context
I'm using some generic initialization scripts to modify third-party builds so I can run static checkers, re-sign output etc. without needing to modify the original build files which I don't own.
Steps to Reproduce (for bugs)
In this project: https://github.com/microg/android_packages_apps_GmsCore, apply the signing init script from this project: https://github.com/vertu20140207/gradle-init-scripts. i.e. ./gradlew -I /signing-init.groovy clean
Remove the first line from settings.gradle and the problem goes away.
It's probably not hard to generate a reduced toy example.
Your Environment
Not really relevant, I've explained the root cause.

A couple things would help us consider this:

Gradle version or a build scan
A simpler example

As it stands now, I think it'll take a lot of time for us to break this down into a simple example. As someone more familiar with the code, would you please consider this?

OK, it's not what I originally thought, so the title is misleading/wrong. The reduced example is that this init script doesn't work in a multi-project build:
settingsEvaluated { settings ->
    settings.rootProject.children.each {
        rootProject.logger.log(LogLevel.INFO, it.toString())
    }
}

The rootProject I want to get a logger from isn't yet available. In this case, I see I should just be using logger.info(), so it's no big deal. Is it documented somewhere when rootProject will be available?

@vertu20140207 Thanks for the update.
I'm not sure this is documented, but tracing the code the rootProject is set after init scripts are executed in DefaultGradleLauncher. findAndLoadSettings() eventually calls setRootProject.
TL;DR: the root project isn't available in init scripts.


Proposal: Prioritize declaration file resolutions when inside node_modules

Reverses the priority order of .ts/.d.ts in node_modules so we prefer loading declaration files from libraries when .ts and .d.ts files are colocated.
We have encouraged libraries to consider shipping their source .ts files and declaration maps to npm to improve the user experience of things like go-to-definition. However, this has problems if the library didn’t use an outDir or declarationDir to separate their declaration files from source files. It’s more expensive for us to load .ts files, and it can cause checking errors like the ones in #58353, where globals only needed for checking implementations are missing, or those implementations have checker errors due to the user’s compiler options.
I think the biggest risk here is that it somehow adversely affects monorepos with node_modules symlinks. However, when used with project references, a different mechanism handles remapping from declaration files to implementation source files or vice versa, depending on settings and whether the program is being built on the command line or as part of editor services. Some monorepo users don’t use project references, but I think in that case, they’re also not generating declaration files. Additionally, most people use an outDir. (JSR’s reason for not wanting to compile into an outDir is it changes the runtime structure of import.meta.url from what the author expected, from the perspective of authoring in, and only ever thinking about, .ts files.) I think this is ok, but @sheetalkamat may know of extra things that need to be handled for monorepos / project references, or may think of reasons why this is a bad idea generally.
Closes #58353

@typescript-bot test it

I think main concern was mixing input and output files in. Even with this PR you can run into this.
Eg structure:
// /repos/node_modules/foo symlinked to /repos/foo
// /repos/node_modules/foo/index.d.ts
import { y } from "./y";

// /repos/node_modules/foo/index.ts
import { y } from "./y";

// /repos/node_modules/foo/y.ts
import { something } from "./index";
export const y = 10;

// /repos/node_modules/foo/y.d.ts
export const y = 10;

// /repos/bar/src/index.ts
import {something} from "foo";

With your PR we will include /repos/foo/index.d.ts from bar (though node resolution but if there is no preserveSymlinks  = value of this is false by default so always will resolve to actual file path) and any imports in this will now resolve to ts files preference since they dont have node_modules in them and we end up getting foo/index.d.ts and foo/inded.ts as well in the program.

/repos/node_modules/foo symlinked to /repos/foo

I don’t think it’s realistic for a realpath in node_modules to be symlinked to a location outside node_modules; every real scenario I’ve ever seen has either been the opposite (e.g. npm link ../foo or monorepo workspaces) or both the realpath and symlink location are inside node_modules (e.g. pnpm).

/repos/node_modules/foo symlinked to /repos/foo

I don’t think it’s realistic for a realpath in node_modules to be symlinked to a location outside node_modules; every real scenario I’ve ever seen has either been the opposite (e.g. npm link ../foo or monorepo workspaces) or both the realpath and symlink location are inside node_modules (e.g. pnpm).

This is pretty common:
cd foo
npm link
cd ../bar
npm link foo

this creates bar/node_modules/foo whose real path is "../foo" .. The first resolution inside foo will take place with "node_modules" in it but resolutions inside that file will be done for "foo" as the path is resolved for the file to real path

@cloudflare/workers-types ships an index.ts and index.d.ts side-by-side with different content, apparently used for different purposes 😐

@sheetalkamat I see your point now. That would be solved with something like this:
- const prioritizeDeclarationFiles = pathContainsNodeModules(candidate);
+ const prioritizeDeclarationFiles = pathContainsNodeModules(preserveSymlinks ? candidate : realPath(candidate));

if it doesn’t have an unacceptable perf impact. Would that address your concern, or do you have another suggestion, or do you think we shouldn’t try to do this?

If .d.ts is preferred for resolution over .ts, wouldn’t that prevent GTD (the entire reason to ship .ts files in the first place) from working anyway?

No, that mapping is requested explicitly in the server.


Table sort column highlighting and default

Highlight the current column used for sorting
Sort by the first sortable column by default

Addresses #832

Highlight looks like this:

I changed the color... I think I prefer light yellow.


Replace entity references with function calls

fix #137

This is purely a refactoring change; I verified it does not change the output of the projects that use docca.

@vinniefalco Okay if I rebase and merge this PR?

Yep!


containers: workaround for ganesha package error on reef

Currently the tests on the released reef image fails due to an RPM transaction error when updating nfs-ganesha. Since go-ceph doesn't need ganesha packages to be up-to-date to ensure that lib{cephfs,rados,rbd}-devel are installed we temporarily disable ganesha repo when running yum update.
See also, discussion in #910
Testing infra only change.

@anoopcs9 according to github your "lgtm" comment is not an approving review. If you approve can you please let github know so this can auto-merge?  Thanks!


Remove outline around stars after selection/during keyboard navigation?

Chrome:

Firefox:

Hi @dandv, there is a SASS variable to turn this off, the outline is mainly for accessibility reasons there. More explaination is here already: https://github.com/LunarLogic/starability/pull/19
I think I will turn this off by default, as it seems to be confusing for many people.


Deployment: CAS Gradle DSL

Deployments are today done via either Maven or Gradle. As an extra option, provide a gradle plugin for CAS to handle deployments via a dedicated DSL. Could possibly be used as an option for the backbone for #599. Working with @scalding on this.

Initial pass available here https://github.com/UniconLabs/gradle-cas-plugin
One documentation improves more and the plugin gets more adoption and feedback, we should be able to list this on the official docs.


Deprecate Python 2.6 support

Since one of our dependencies (Twisted) has dropped Python 2.6 support, and another (Cryptography.io) has deprecated it (https://github.com/pyca/cryptography/blob/master/src/cryptography/init.py#L21), and it is getting super super long in the tooth, maybe we should consider phasing out support next release.
Opinions, @oberstet, @meejah ?

makes sense. cool, I'm +1

I think we should raise a DeprecationWarning ASAP, and then drop it in next release + 1. Then it gives some people warning, although I'm not sure how many people actually use 2.6 and would keep their AB up to date to see such a warning...

Awesome! That's even better. Btw: what do you think rgd the fully PEP8 naming stuff. That'll break API. Or should we provide backward compat. classes? At least for some time?

The backwards compat adapters that raise DeprecationWarnings would be ideal -- since it means people have a transitionary period. Then I think 1.0 would remove all deprecated code, leaving only the new API left.


db: Consider removing the truncate and destroy database functions

They are likely very dangerous to accidentally call on a production server.
destroy is dubious, not sure what it does. But truncate is very useful for sequential tests. It should be made available for them I think.

In default.db.queries.ts

Ok, yeah, agreed, we don't really need those function as generics. If we ever need to do those operations, we could reimplement them with adequate protection around them.


K01: Implement Full Validate Profile Method & Required Methods

Describe your changes
This PR finishes fleshing out the profile validation work in SmartChargingHandler. It finalizes the implementation for a few functional requirements:

K01.FR.06
K01.FR.38
K01.FR.03

With this PR, we can now call validate_profile() and, based on the type of the given profile, run the specific validation methods that are applicable. We also simplify the function signatures of some of the previously added methods to take the EVSE ID.
Issue ticket number and link
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I have made corresponding changes to the documentation
[x] If OCPP 2.0.1: I have updated the OCPP 2.0.1 status document
[x] I read the contribution documentation and made sure that my changes meet its requirements

CC: @shankari

@the-bay-kay the high-level description is not quite correct, this PR addresses the following functional requirements:

K01.FR.06
K01.FR.20
K01.FR.26
K01.FR.38
K04.FR.03

Please read through my review and review the changes yourself so you get familiar with the process.

@christopher-davis-afs Just give me a sign that it is ready to merge and I will merge it ;)

It is ready :)

@couryrr-afs @drmrd @wjmp I believe that there were several comments that here that the reviewers wanted to change, but didn't want to hold up the merge for. Now that the merge is done, it would be great to see a cleanup PR address the comments. Also, some of my comments were marked as resolved, but I don't agree with the resolution 😄

@couryrr-afs @drmrd @wjmp I believe that there were several comments that here that the reviewers wanted to change, but didn't want to hold up the merge for. Now that the merge is done, it would be great to see a cleanup PR address the comments. Also, some of my comments were marked as resolved, but I don't agree with the resolution 😄

@shankari  Yep — we haven’t forgotten about the code items and can get the comments added to the code base you requested. We have been prioritizing getting the Validate Profile PR merged, the Add Profiles PR ready in support of Alfen’s timelines and some other blocker around that. I thought it best to resolve comments prior to presenting the PR to the community as to avoid any confusion or further delay based on your comment about the review.
We will have a PR out soon.


pin gdal 2.1.x

Unfortunately fiona is not compatible with gdal 2.2.* yet, so we need to pin everything to 2.1.* is we want to install them along side with fiona. Note that installing the previous build number with latest gdal is still possible when not installing fiona.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@ocefpaf I think we can do the same (ie remove gdal pinning) for tuiview and python-fmask...

I sent PRs https://github.com/conda-forge/tuiview-feedstock/pull/17 and https://github.com/conda-forge/python-fmask-feedstock/pull/4 to try this. I believe python-fmask is safe to unpin, but I am not sure about tuiview.


How to scroll top cell and left fixed cell?

I want to scroll the cell on the top along with the cell in left if it is scrolled left. Any Solutions?

This can be solved by change some source codes in ExcelPanel.java, you may start from initWidget() method


Time to update "Getting and Building the Code" :-)

https://github.com/aspnet/EntityFrameworkCore/wiki/Getting-and-Building-the-Code
(No PRs for wiki pages, sadly)

@bricelam @smitpatel @AndriySvyryd @maumar @anpete Made a stab at updating this--all feedback welcome.

Looks good and useful!


[Bug] Update weights from disk ends in runtime corruption for different size model

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.
[X] 4. If the issue you raised is not a bug but a question, please raise a discussion at https://github.com/sgl-project/sglang/discussions/new/choose Otherwise, it will be closed.
[X] 5. Please use English, otherwise it will be closed.

Describe the bug
In our unit test test_update_weights_from_disk.py, we only test the update of the Engine with a wrong model name. But if you update llama 3.2 1B with llama 3.2 3B, the engine will be corrupted. This is found in our document native_api.ipynb. I now change our native_api.ipynb to be the same as our unit test. But I fix the error of updating weights from disk with a model of a different size.
Reproduction
Easy to reproduce.
Environment
Easy to say.

This should not be closed since this bug is not fixed right now.


kodi-addon-pvr-plutotv: Bump to v21.0.1

Hi, this is the first of a serie of PR.
I have a RPi4 (arm64) at home running Kodi and I wanted to update to nanbield, but the initial layer is not ready for it. So I’ve used yours which is far more active. May I suggest you to ask to become the official maintainer of meta-kodi ?
I am using the nanbield branch, because I want to have a rather stable system. First of all, my system could not compile until I had taken patch:
https://github.com/yoctoproject/poky/commit/daee9d9eae7c391edecf972e008ec5f124260dc3
from Poky master branch (upgrade of ffmpeg) and the following ones from your master branch: from:
https://github.com/MarkusVolk/meta-kodi/commit/a83bfc378bb7e11b4fe2c85a70aad7746281182f
to:
https://github.com/MarkusVolk/meta-kodi/commit/d7eb727d398e7f87e413253aeddeb93f81ea115c
and:
https://github.com/MarkusVolk/meta-kodi/commit/947095b4bef6544f171d1e81b567973592c32747
Maybe could you update the nanbield branch with at least these patchs and also with the ones from my PRs.
So, this first PR is simply to update the Kodi addon pvr-plutotv.
Don’t hesitate and tell me if you need me to re-work this patch and the following ones, and also if you want me to also create a backport for the nanbield branch.

Hi,
thanks for your contributions. They all look reasonable to me.
I'll have to mention that my intention was never to take over this layer. @dev-0x7C6 has done a great job keeping it up to date, and due to lack of privileges requested changes from @koenkooi over a year ago with no response to this day.
https://github.com/koenkooi/meta-kodi/issues/65
So I wouldn' even know who to ask to get an answer.


greengrass-lite_git.bb: upgrade git version

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

Backport failed for kirkstone-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin kirkstone-next
git worktree add -d .worktree/backport-10748-to-kirkstone-next origin/kirkstone-next
cd .worktree/backport-10748-to-kirkstone-next
git switch --create backport-10748-to-kirkstone-next
git cherry-pick -x 9e3fd7184486fcad35fc43353fdd7938061efde0

Successfully created backport PR for scarthgap-next:

#10752

Successfully created backport PR for styhead-next:

#10753


Unable to create new Angular project in VS 2017 15.8.3

README.md says:
Something went wrong with the creation of an Angular CLI application
Project structure created:

I've narrowed it down to my naming convention.
The error occurs if I have a dot (.) in the project name


Custom role without gb permissions will receive ugly stack trace on home page instead of access denied error

To replicate:

Add a custom role to existing site called "Course Builder"
Assign some permissions to build content but dont assign any GB permissions
Assign a user to the new "Course Builder" role
Become User
Go to the site and go to Gradebook tool

Caused by: java.lang.SecurityException: You do not have permission to perform this operation  at org.sakaiproject.component.gradebook.GradebookServiceHibernateImpl.getCategoryDefinitions(GradebookServiceHibernateImpl.java:2574)  . . . at org.sakaiproject.gradebookng.tool.pages.GradebookPage.getUiSettings(GradebookPage.java:729)  at org.sakaiproject.gradebookng.tool.pages.GradebookPage.<init>(GradebookPage.java:193)

Good catch, will correct this exception handling to take the user to the access denied page.

PR sent

Ready to test

Confirmed in trunk:


feat(gsoc'24): Implemented LayoutElements Panel vue component

Fixes #108
Describe the changes you have made in this PR -
Created a Vue component out of LayoutElement Panel which was previously in the extra.vue file

moved all DOM manipulation to vue's reactives
moved all styles to scoped view

Screenshots of the changes (If any) -
https://github.com/CircuitVerse/cv-frontend-vue/assets/91966855/baa644af-feb7-42ba-b4a6-f62f13568369

@niladrix719 can you check the failed tests?

@niladrix719 are you sure you take care of :

fix the dropdown and icons not showing up
you can take reference from the circuit element panel


Installed according to directions and still got 404

Installed according to directions and still got a 404
Ubuntu20, Apahce2, Django 3.1.4

Can you tell me a bit more about your configuration? Can you include a stack trace of some kind? I don't have much to work with here from your issue. Please include your:

Environment details
Any related stack traces
A pip freeze output
And any other debug details

Please also take a look at the example project in this GitHub and compare to your Django.
After you have additional details please re open this issue.


Updated HTTP2 Reader to fix missing header state

Motivation:
When a malformed request comes in where a stream error is thrown, data is still left on the reader and not properly cleaned. When a valid request comes in, it is not properly parsed because of the leftover data being included.
Modifications:
Updated DefaultHttp2Reader to clean up leftover data if these types of stream exceptions come in the wire.
Result:
After this change, data should be properly parsed after a malformed request
Fixes #13788
If there is no issue then describe the changes introduced by this PR.

Can you include test case?

@idelpivnitskiy @bryce-anderson PTAL

Added a simple test case that fails before the changes in the PR

@isaacrivriv, I imagine you're busy but I wanted to see what your plan was for this. It's a great find and I'd love to see it fixed. 🙂

Sorry for the late reply, I was trying to get to this last week but couldn't get it to a state I was happy with. I added a push with the changes we discussed in mind. I separated this following these steps

Parsing the framing layer component of the frame (everything all the frame types have in common)
Verifying the data read from the framing layer based off the different types
Slicing the buffer after verifying and processing the payload for this specific frame
Resetting the flags specifically the readingHeaders flag to the default value once a stream reset occurs while processing the headers/payload

The test I added passes and ran all the tests locally for http2 which pass as well. I did some manual testing to replicate garbage requests submitting more data than actually announced in the payloadLength and that still fails as expected. I'm not sure of a way to be able to identify these cases and respond to the client that something may be wrong with their implementation if anyone had anything in mind.

Thanks for the detailed review @bryce-anderson! I'm getting a better understanding on how everything works behind the scene while doing this. I pushed out changes based on the comments. I also had to update the test I added to add dummy data to the buffer for the reader to consume it and continue since the logic to identify if the request is valid or not (extra data or not enough data for the request) is not added. Let me know what you think!

/cc @ejona86

Let's plan to get this in after we cut the next release just to give it a bit more time to bake and for people like @ejona86 to check it out.

@isaacrivriv thanks a lot! I also cherry-picked into main


[Bug]: vue style loss

Version
System:
OS: macOS sonoma 14.6.1
Binaries:
Node: 18.20.4 - /usr/local/bin/node
npm: 10.7.0 - /usr/local/bin/npm
Browsers:
Chrome: 128.0.6613.120
npmPackages:
@rspack/core: 1.0.2

Details
When I used<EMAIL_ADDRESS>the page style of vue3 was lost (each page was loaded asynchronously). When I rolled back the version to 1.0.1-beta.13, everything was normal
Reproduce link
No response
Reproduce Steps
rsbuild dev

https://github.com/ritilyn1991/testdemo.git

same problem


Investigate why ActiveSupport is being pulled in and see if it can be removed

Currently a dependent Gem is pulling in ActiveSupport. Figure out what Gem is pulling that in and see if it's really needed for the project. If it is then this can be closed with the details.

The Gem that is pull in active support is pivotal-tracker. We should look for an alternative to this gem so the time it takes to download and install the toolshed bundle is reduced. We don't need active support for this gem.


+default/search-project-for-symbol-at-point looks for symbol starting at buffer directory instead of project directory

What did you expect to happen?
I expected for the symbol to be looked at project root, instead of only at subdirectories starting at the file's directory.
What actually happened?
Symbol was being looked at starting from the directory of the file I use SPC * at a symbol, not from project root directory as it was before.
Describe your attempts to resolve the issue
The behaviour has changed since 411ed0c8f8e12d106a64a56ae437ba45c1bcd1f2
Since then I have to add a .dir-locals.el that sets default-directory to project root, to make +default/search-project-for-symbol-at-point actually look at project for symbol, not at project subdirectories starting from the file dir.
I have a workaround for this, just wasn't sure if this new behaviour was intended or not by the change, as it looks somewhat erroneous for my case.
This workaround, however has side effects for M-x compile and other things that read default-directory and now point to project base, which is not wanted..
Steps to reproduce
(i) open multi dir project:

root
root/file0.txt (contains symbol sym)
root/subdir
root/subdir/file1.txt (also contains symbol sym)

(ii) reproduce:

open file1.txt
hit SPC * on symbol sym that is present in file0.txt too
notice how file0.txt is not shown on the match list as SPC * didn't look for symbol in all project files

(even though the name +default/search-project-for-symbol-at-point suggests it should)

System Information
emacs 28

:warning: This issue was closed and marked for deletion for the checked reasons below:

[X] Issue cannot be acted on or investigated further because:

[X] Could not be reproduced in vanilla Doom on any of our test environments.
[ ] Not enough information to act on.
[X] Vital information was omitted or inaccessible.
[ ] Explanations are too vague or issue is too open ended.
[ ] It's unclear what you are trying to say.

[X] The following possibilities are more likely explanations and haven't been ruled out:

[ ] A pilot error.
[X] Private configuration.
[ ] A third-party package is behaving correctly, but in a manner you're not expecting.
[ ] A corrupt or broken Emacs installation.
[X] Your system environment (hardware issues, the OS, shell environment, missing system dependencies, etc).

This ruling is not absolute and can be reversed if you follow up with enough information to further investigate this issue. Our How to Debug Issues guide can help you find out more about it (note: our discourse currently requires an account).
If you still feel lost after reading our debugging guide, raise your issue on our Discord or Discourse -- there may be others who have experienced it. Otherwise, this issue will be automatically deleted after 14 days.

I apologize if this is inconvenient or you feel lost, but it takes a lot of time to review issues by myself. The complexity and extensibility of Emacs tends to produce a lot of false positive or open ended bug reports, so I've restricted our issue tracker to issues that are actionable, immediately reproducible in vanilla Doom, or can be investigated in a reasonable amount of time.

I'm also facing this issue. Here is the output of doom/info: https://pastebin.com/ptfVizY4

I'm also facing this issue. Here is the output of doom/info: https://pastebin.com/ptfVizY4

@jvalkeejarvi I see there is a duplicate issue to follow:

https://github.com/hlissner/doom-emacs/issues/5831

#5831 helped me realize what the cause of this issue was. Sorry about that! As of f0ba6c99b26b0bf23495323a3097cab7d057ea3d this should be fixed.

❤


trim levels read from properties file to remove trailing spaces

trim levels after reading them from properties and before initializing any loggers.
If the level in a properties file contains a trailing space log4j2 won't initialize the logger correctly. In Effect the logger for whicht the level is defined will keep the inherited loglevel.

I guess there should be a jira issue for this, so I created https://issues.apache.org/jira/browse/LOG4J2-1890

Du bist auch überall ;-) Danke für den Issue
Am 26. April 2017 19:49:52 MESZ schrieb Pascal Schumacher<EMAIL_ADDRESS>
I guess there should be a jira issue for this, so I created
https://issues.apache.org/jira/browse/LOG4J2-1890
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/logging-log4j2/pull/75#issuecomment-297489676

--
Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet.

Hi, so what can we do about the problem with the travis-ci build? Is there any chance this PR can be part of the next release?

No worries about the travis bit, something is broken there at the moment. The PR looks good, ready for merge, though I myself will not get a chance to do this for at least a few hours if anyone else wants to handle this right now. Thanks for the PR!

Any news on the PR?

Yikes, I totally forgot about this, sorry. This is ready to be merged.

Seems the PR is still not integrated. Can i help with something?

Thank you for you patch. It is now in Git master! :-) Keep them coming :-)


Develop to master - appstore

I get a 404 when trying to view what failed in the ci/dockercloud-stage output.


Native UI Components for Android to JS response speech slowly

My native is surfaceView,When I was in MotionEvent.ACTION_MOVE, starting from the native to JS, also good, JS behind the response time becomes very long

We're cutting down on the number of outstanding issues, in order to allow us to focus. I'm closing this issue because it has been open for over 60 days with no activity. If you think it should still be opened let us know why. PRs are always welcome.