install.sh circumvents package manangement of major linux distributions

Hello,
I was going to test your genie package, but stumbled over the code in install.sh. What bugs me about it is how it manually does all the steps a linux package manager is supposed to take care of, like dependency management and installation of said dependencies:
[wget -qO- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.1/install.sh | bash](https://github.com/stanford-oval/genie-sdk/blob/9d64c8eac22170ebd81646a9f5b69852a4e6dc3b/install.sh#L9)
This code is really dangerous, as pointed out in many places all over the web, e.g. here.
And
[if grep -qE "ID(_LIKE)?=.*fedora.*" /etc/os-release ; then](https://github.com/stanford-oval/genie-sdk/blob/9d64c8eac22170ebd81646a9f5b69852a4e6dc3b/install.sh#L31) these lines only cover a very small variety of distributions (important ones, I won't deny that, but that's not the point). You'll never be able to cover all the different versions of distributing packages to linux distributions in a single bash script. The way to go here are creating packages for distributions (e.g. .deb for Debian/Ubuntu).
Please remove these lines from your install script and instead add an installation section to the README detailing how to install dependencies in general.

Thank you for your suggestion.
I checked the link about "curl bash piping", it seems the dangerousness is debatable. The main concern is the download is interrupted, but the script by nvm is safe from that. But thanks for the reminder, I didn't actually check if that script is safe or not, as I just copied their recommended installation method.
For your second point, we try to simplify the process for the majority. But I agree we should include a README to introduce what dependencies are needed for manual installation, to avoid the need to read install.sh. I'm not sure about why creating packages will be better though, it does not seem to solve the issue of not being able to cover all different Linux distributions.


Begin refactor

Too much referenced code in the master branch
I'd like to explore various options as to how we could structure the server before merging the PR

This branch is farther in progress than the master branch. Going to merge.


Async await usage

How does this work with async await?
I assume that, if the pool is depleted, the async event loop would be blocked. Is that correct?

checkout returns None in this case, in which case you should push a waker somewhere and return Poll::Pending from your poll_ready. Then when you put something into the pool, you should wake the waker(s).
It would be nice if this was provided as part of this library, but it's not too hard to code.


CMultiSelect - If user click on X to remove and CMultiSelect is inside , page is submitted

Hi! And thank you for looking at my issue.
Has title mention, I have this weird issue where if the CMultiSelect is inside a , the submit happens. I went in the html and it seems like the x button is missing type='button' since the default value of type is 'submit' :

If I add the type='button' manually on the button in the browser inspector, it works fine.
<template>
  <CContainer fluid class="step-container py-2">
    <!-- no issue here -->
    <CFormMultiSelect id="customId"
        label="custom label"
        :options="getOptions()"
        v-model="customModel" />
    <form>
      <!-- issue here -->
      <CFormMultiSelect id="customId2"
        label="custom label 2"
        :options="getOptions()"
        v-model="customModel" />

@matgenois which version of @coreui/vue-pro do you use?

Bug is in 5.0.1 and 5.1.0

@matgenois ok, I'm working on it.

@matgenois fixed in v5.2.0

Thank you!


Delete unused fonts

Issue No. #1792
Delete unused fonts

Thank you for your interest in this issue, @imdebamrita! This issue has already been solved in #1783, so I'm going to close this PR. Would you be interested in solving the other part of #1781 ?


Add husky & lint-staged for pre-commit hooks (#1)

feat: testing husky pre-commits

chore: remove now-unnecessary line

fix: formatting after committing

test husky

fix: add lint-staged so we don't commit all files

fix: attempt to fix eslint and tsc not checking

fix: incorrect glob pattern

fix: noemit

fix: remove test file changes

feat: update README

chore: remove unnecessary husky setup step

feat: run tsc from bash

@futurepaul I botched my other branch attempting to manually squash from the cli, fixed and squashed this one in this new PR which has all the other changes and is rebased & 1 commit.


Import-ObjectsToNAVContainer

Continuing my question posted here: Developing Business Central Extensions (part 2) – Repository/Environment
and your response:

If your container is started with -enableSymbolLoading, then the Import-Objects above should rebuild the symbols for these objects.
If this doesn’t happen, you might be able to just call compile-objectsinNavContainer with a filter identifying the objects you need to rebuild symbols for (you still need enablesymbolloading)

I am using the Local-Sandbox.ps1 script from your Helloworld repository.  It calls Create-Container.ps1.  This script is unmodified and does contain the -enableSymbolLoading switch.
New-NavContainer @parameters `
                 -containerName $containerName `
                 -imageName $imageversion.containerImage `
                 -auth NAVUserPassword `
                 -Credential $credential `
                 -alwaysPull:$imageversion.alwaysPull `
                 -updateHosts `
                 -includeCSide `
                 -doNotExportObjectsToText `
                 -enableSymbolLoading ` <<<<<----------------------
                 -useBestContainerOS `
                 -shortcuts $shortcuts `
                 -licenseFile "$licenseFile" `
                 -additionalParameters $additionalParameters `
                 -myScripts $myscripts

I have added the Compile-ObjectsInNavContainer command and that appears to resolve the issue without needing to regenerate the symbols.
I don't understand what the original problem was as it should have happened correctly from the start.

Compile-Objects will regenerate symbols for the objects you compile.
I think it should happen during import as well - not sure why it doesn't.

Also, please remember that the template project is a work in progress and ment to be a starting point for people to setup their CI/CD - it isn't something that is cast in cement and works for all.
Thanks


Support feature gates on NDM

Users should be able to provide an option to enable certain features via feature gates. This can be enabled via the NDM ConfigMap.
Providing feature gates will be useful mainly in case of features such as the new UUID algorithm, so that user can opt in and try out the new feature.

Initial implementation of feature gate configuration is in #377

Done in #377


ERROR Linking tmp mounts to host: Exchanging /tmp <=> /tmp.tmp: Invalid cross-device link (os error 18)

I have hit this error ERROR Linking tmp mounts to host: Exchanging /tmp <=> /tmp.tmp: Invalid cross-device link (os error 18) with example image in install.md file, i am currently using ubuntu 23.04 as OS from which i run podman command using bootc built from source in git repo
if i mount /tmp to /tmp.tmp with podman run flag -v error changes to ERROR Linking tmp mounts to host: Symlinking /proc/1/root//tmp to /tmp.tmp: File exists (os error 17) mounting /tmp to /tmp changes error to ERROR Linking tmp mounts to host: Exchanging /tmp <=> /tmp.tmp: Device or resource busy (os error 16)
There might be some commonality with issue #91.
podman version 4.3.1
command: sudo podman run --privileged --pid=host --net=none --security-opt label=type:unconfined_t ghcr.io/cgwalters/c9s-oscore bootc install --target-no-signature-verification /dev/vda

with different image error also appeared as ERROR Linking tmp mounts to host: Exchanging /var/tmp <=> /var/tmp.tmp: Invalid cross-device link (os error 18)

/proc/1/root//tmp that looks like it is missing a component

This must be the same thing as https://github.com/containers/bootc/issues/91 so closing as a duplicate.
However, can you just show the output of ls -ald /tmp && findmnt /tmp on the host?


Trigger demo-site build manually

Since automatic builds off master fails when triggered post a pull-request merge.

@jekyllbot: merge +dev


dragonflybsd adding ucontext type

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

Thanks! r? @JohnTitor @bors r+

:pushpin: Commit 783783d9af4e8bcf783f713f52374845ea70076b has been approved by JohnTitor

:hourglass: Testing commit 783783d9af4e8bcf783f713f52374845ea70076b with merge 858d54794163083d98f33f442d726e646969f756...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 858d54794163083d98f33f442d726e646969f756 to master...


[SPARK-28251][SQL] Fix error message of inserting into a non-existing table

What changes were proposed in this pull request?
Before this PR inserting into a non-existing table returned a weird error message:
sql("INSERT INTO test VALUES (1)").show
org.apache.spark.sql.AnalysisException: unresolved operator 'InsertIntoTable 'UnresolvedRelation [test], false, false;;
'InsertIntoTable 'UnresolvedRelation [test], false, false
+- LocalRelation [col1#4]

after this PR the error message becomes:
org.apache.spark.sql.AnalysisException: Table not found: test;;
'InsertIntoTable 'UnresolvedRelation [test], false, false
+- LocalRelation [col1#0]

How was this patch tested?
Added a new UT.

Test build #107240 has started for PR 25054 at commit 9ac59ba.

Can one of the admins verify this patch?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12396/
Test PASSed.

@dongjoon-hyun this is a PR for https://github.com/apache/spark/pull/24860#discussion_r300215009

ok to test

Thank you, @peter-toth !

Test build #107240 has finished for PR 25054 at commit 9ac59ba.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107240/
Test PASSed.


739 comprehensive documentation for station massa wallet security audit

@gregLibert  for some reason the second sequence diagram for Sign Message you added is not showing.


Update web_application_scanner.py

Description
Depth Parameter: Add a depth parameter to control the level of scanning, allowing users to specify how many levels of sub-links to scan.
Track Scanned Links: Implement a mechanism to track scanned links and prevent duplicate scanning, improving efficiency.
Scan Results: Store the vulnerabilities detected during the scan and associate them with the respective URLs for better analysis and reporting.
Recursive Scanning: Enhance the script to recursively scan sub-links within the specified depth, enabling a comprehensive scan of the web application.
Improved Link Handling: Skip internal links (starting with '#') and email links (starting with 'mailto:') during the scan to focus on external URLs.
Vulnerability Scanning: Include a modular vulnerability scanning logic that can be easily extended with additional checks for common vulnerabilities.
Report Generation: Implement a report generation function that can generate a comprehensive report based on the scan results, allowing users to save it to a file or generate HTML output.
Customizable Depth: Allow users to specify the depth of the scanning process, giving them flexibility in determining the extent of the scan.
Optional Vulnerability Scanning: Provide an option to enable or disable vulnerability scanning, allowing users to choose whether to perform vulnerability checks during the scan.
Enhanced Error Handling: Improve error handling by implementing appropriate exception handling and displaying meaningful error messages when encountered.

Fixes #(https://github.com/avinashkranjan/Pentesting-and-Hacking-Scripts/issues/316)
Replace issue_no with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project

[x] I have performed a self-review of my own code

[x] I have commented my code, particularly in hard-to-understand areas

[x] I have created a helpful and easy to understand README.md

[x] My documentation follows Template for README.md

[x] My changes generate no new warnings

[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@avinashkranjan
pls review this PR


Climber Subsystem

Hardware

1x Kraken X60 (non-pro)

Details

The climber will be lifts the robot off the ground.

Desired Behavior

The arm should use a PID controller and a motion profile on the climber's position (see 2k24-Robot-Code for specifics on how to do this)
The position of the climber should be able to be set in a method in the climber subsystem.
The climber should have a zeroing sequence that is mapped to a button on the controller. This sequence will slowly move the climber down until it hits its limit.

REMEMBER TO FOLLOW THE CODE STANDARD, REGARDLESS OF EXAMPLE CODE QUALITY

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


There is still random intermittent instances of the UI not being blue when it should be.

Describe the bug
I fear this will continue and make the new side panel design more confusing.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on 'Climate change"
Scroll down to 'Pop loss - is 94, is not blue'
See error

Expected behavior
Can Emma spend some time thinking about why this is happening and how to fix it?
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@kameron - just validated that this may no longer be happening? Does it still happen for you?

I don't think I've seen it since the update was made to get the flags via some Emma sorcery @vim-usds


Cannot install because webview2 is missing (and I cannot install it)

Microsoft PowerToys version
0.55.0
Running as admin

[X] Yes

Area(s) with issue?
Installer
Steps to reproduce
Any version newer than 0.53.0 does not install.
✔ Expected Behavior
Install correctly
❌ Actual Behavior
The update process stops when it tries to update Edge. On my PC, Edge is under the control of my organization and it cannot be modified. Hope this dependency can be removed, as it was the case until version 0.53.0 (the last one I can install properly). Thanks a lot anyway for your amazing work. You are making Windows much better!
Other Software
Edge, version 97.0.1072.62 (64 bit)

What if you try to install manually?
https://msedge.sf.dl.delivery.mp.microsoft.com/filestreamingservice/files/257e16c7-6b27-41bc-b212-64d0a9418970/MicrosoftEdgeWebView2RuntimeInstallerX64.exe
/needinfo

I receive the following message (my translation): "The organization has applied group criteria that prevent the installation".

@antodc You'll have to contact your system administrator for help. The webview2 is a required dependency for the preview pane.

It is really a pity that PowerToys might not be installed because of one specific feature (that maybe the user not even uses). What about making the installation of the different PowerToys features optional? I mean, the user could choice what to install and what to leave. Independently of my own problem, I think it would be appreciated also by other users and much more respectful towards their choices about what to install on their PCs. Maybe it is worth a thought from your side.

@antodc Thanks for your suggestion, that's tracked down in #3170

i have the same problem, because of domain policy, Thanks @antodc for the working version  0.53.0

@rpiombi-bna You are welcome! Actually, I am now able to install the latest versions of PowerToys. I do not know whether it depends on some changes made to the installer of PTs or to my PC from my organization, but it is a very appreciated development. :-)


Sticky header even without quick tips

I usually disable git-log--graph.hide-quick-branch-tips  because I dislike the scrolling area that sometimes appear and I find most of the text a bit redundant. But then I lose the GREAT sticky header (I even forgot about it):

So I what about keeping an empty space at the top the size of the History and All branches buttons:

You would align them horizontally, I feel like moving History to the left is the right call
IMO This also looks better than having the two buttons over the table occluding info.
The region can increase its size if required when many branch names are stickied (as it does now)

As always, I guess I would keep an option to toggle this/previous behavior to keep everybody happy.

Regarding the quick tip area info, I feel like a ghost commit in the graph indicating existing changes is simpler:

good ideas here, and the quick tip area info text thingy is just the output of cli git status that I put there because frankly I didn't know what else to put in this area, and having some information about for example an ongoing merge or rebase situation can be helpful. it looks horrible though.


A command returning non-zero exit executed via lagoon ssh -C breaks terminal output

Describe the bug
If I run lagoon ssh -C with a command that errors, my terminal goes crazy 😅

(Also, there is no output from the command)

Text version
ambp% lagoon ssh -p project -e dev -C 'some-error-command'
Process exited with status 127
                            %
ambp% echo "notice this\nline 2\n line 3"
notice this
         line 2
                line 3
                      %
ambp% echo "how \n it \n does \n it ? 😂"
how
   it
       does
             it ? 😂
                    %
ambp% echo "how to undo it?"
how to undo it?
             %
ambp%

STR

lagoon ssh -p MY-PROJECT -e MY-ENV -C 'some-error-command'
Try to run some other command that outputs something

It looks especially nice if you run something that rewrites the output dynamically. For example, npm i
Expected behavior
My terminal stays sane 😁
Additional context
Environment: latest macos, zsh without profile loaded (zsh --no-rcs)
ambp% lagoon --version
lagoon v0.17.0 (go-1.16)
                        built 2023-02-03T00:42:39Z
                                                  %

Thanks, I just put up a PR that handles errors differently that should help prevent this from happening.

Thanks for the super quick fix! Is there a chance for a release?

Just need to verify some other merges and then will aim for a release tomorrow/day after

Upgraded, tested, works! Thank you 🤍

Thanks for letting me know, glad it helps you out :)


Add explanation of all known non-mob materials.

For starters:

entity
entity_alphatest
slime_outer

I would love to know more about this, and would write something up...how would one even get this information? Looking through the vanilla files, I see things like fishing hook, camera, armor stand and arrow...but I don't have any sense for what those mean.

writing


httpcore.UnsupportedProtocol: Request URL is missing an 'http://' or 'https://' protocol.

Issue Description
When running K-shot LLM, I encountered a missing input_variables parameter in the code at this line I noticed the absence of the input_variables key in template_kwargs. I updated the code as follows:
 messages.append(FewShotChatMessagePromptTemplate(
           input_variables=["intention"],
           example_selector=template_kwargs["example_selector"],
           example_prompt=template_kwargs["example_prompt"]
)

However, I am still encountering an error. I am running this in a Docker environment. Can you give me any suggestions about this ...
Error Traceback
Traceback (most recent call last):
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_transports/default.py", line 69, in map_httpcore_exceptions
yield
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_transports/default.py", line 233, in handle_request
resp = self._pool.handle_request(req)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpcore/_sync/connection_pool.py", line 167, in handle_request
raise UnsupportedProtocol(
httpcore.UnsupportedProtocol: Request URL is missing an 'http://' or 'https://' protocol.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_base_client.py", line 978, in _request
response = self._client.send(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_client.py", line 914, in send
response = self._send_handling_auth(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_client.py", line 942, in _send_handling_auth
response = self._send_handling_redirects(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_client.py", line 979, in _send_handling_redirects
response = self._send_single_request(request)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_client.py", line 1015, in _send_single_request
response = transport.handle_request(request)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_transports/default.py", line 232, in handle_request
with map_httpcore_exceptions():
File "/usr/local/lib/python3.10/contextlib.py", line 153, in exit
self.gen.throw(typ, value, traceback)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_transports/default.py", line 86, in map_httpcore_exceptions
raise mapped_exc(message) from exc
httpx.UnsupportedProtocol: Request URL is missing an 'http://' or 'https://' protocol.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_base_client.py", line 978, in _request
response = self._client.send(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_client.py", line 914, in send
response = self._send_handling_auth(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_client.py", line 942, in _send_handling_auth
response = self._send_handling_redirects(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_client.py", line 979, in _send_handling_redirects
response = self._send_single_request(request)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_client.py", line 1015, in _send_single_request
response = transport.handle_request(request)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_transports/default.py", line 232, in handle_request
with map_httpcore_exceptions():
File "/usr/local/lib/python3.10/contextlib.py", line 153, in exit
self.gen.throw(typ, value, traceback)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/httpx/_transports/default.py", line 86, in map_httpcore_exceptions
raise mapped_exc(message) from exc
httpx.UnsupportedProtocol: Request URL is missing an 'http://' or 'https://' protocol.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/local/lib/python3.10/runpy.py", line 86, in run_code
exec(code, run_globals)
File "/app/src/opencole/inference_launcher/k_shot_llm.py", line 169, in
main()
File "/app/src/opencole/inference_launcher/k_shot_llm.py", line 141, in main
detail = tester({"intention": example.intention})
File "/app/src/opencole/inference/tester.py", line 592, in call
output = self.chain.invoke(input)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/langchain_core/runnables/base.py", line 2499, in invoke
input = step.invoke(input, config)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/langchain_core/language_models/chat_models.py", line 280, in invoke
self.generate_prompt(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/langchain_core/language_models/chat_models.py", line 713, in generate_prompt
return self.generate(prompt_messages, stop=stop, callbacks=callbacks, **kwargs)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/langchain_core/language_models/chat_models.py", line 570, in generate
raise e
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/langchain_core/language_models/chat_models.py", line 560, in generate
self._generate_with_cache(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/langchain_core/language_models/chat_models.py", line 785, in _generate_with_cache
result = self._generate(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/langchain_openai/chat_models/base.py", line 547, in _generate
response = self.client.create(**payload)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_utils/_utils.py", line 277, in wrapper
return func(*args, **kwargs)
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/resources/chat/completions.py", line 643, in create
return self._post(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_base_client.py", line 1266, in post
return cast(ResponseT, self.request(cast_to, opts, stream=stream, stream_cls=stream_cls))
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_base_client.py", line 942, in request
return self._request(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_base_client.py", line 1002, in _request
return self._retry_request(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_base_client.py", line 1079, in _retry_request
return self._request(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_base_client.py", line 1002, in _request
return self._retry_request(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_base_client.py", line 1079, in _retry_request
return self._request(
File "/root/.cache/pypoetry/virtualenvs/opencole-9TtSrW0h-py3.10/lib/python3.10/site-packages/openai/_base_client.py", line 1012, in _request
raise APIConnectionError(request=request) from err
openai.APIConnectionError: Connection error.

I think this is a problem with my network. :(


UoB-HPC / miniBUDE Public/data   problem

Hello  tom91136
I am trying do the miniBUDE)/sycl/ project and try to implement on opeapi. However i find the data files' content(.in file)  from data/bm1 and data/bm2 are almost garbled text. May you update correct data?

Are these .in files in binary format ? Thanks.

These .in files in binary are not correct because they are not representative of real data. Is that right ?
https://github.com/oneapi-src/oneAPI-samples/tree/master/DirectProgramming may be helpful for your work.

No, i mean this problem as below picture so i cannot make sure if they are
correct.
[image: image.png]
zjin-lcf @.***> 於 2022年7月7日 週四 下午2:58寫道：

These .in files in binary are not correct because they are not
representative of real data. Is that right ?
https://github.com/oneapi-src/oneAPI-samples/tree/master/DirectProgramming
may be helpful for your work.
—
Reply to this email directly, view it on GitHub
https://github.com/UoB-HPC/miniBUDE/issues/25#issuecomment-1177659273,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ53XHJPZYX7L6364CGCHTDVS3O77ANCNFSM52X5GPCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

The image shows garbled contents inside the .in file. Did you successfully
get the data when you run your code?
zjin-lcf @.***> 於 2022年7月7日 週四 下午3:12寫道：

The image is not visible...
—
Reply to this email directly, view it on GitHub
https://github.com/UoB-HPC/miniBUDE/issues/25#issuecomment-1177679722,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ53XHPJDZIBAMNXZLKZGCTVS3QTHANCNFSM52X5GPCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Do you consider a binary file as a garbled text file ?
A sample run:
./main --deck ../minibude-sycl/data/bm1  --wgsize 256 --iterations 100
Poses     : 65536
Iterations: 100
Ligands   : 26
Proteins  : 938
Deck      : ../minibude-sycl/data/bm1
WG        : 256

Kernel time:    2311.810 ms
Average time:   23.118 ms
Interactions/s: 69.136 billion
GFLOP/s:        2836.202
Largest difference was 0.003%.

Oh, it is not your fault. Is there a SYCL device_selector for FPGAs ?  Do you mind posting the error message at https://community.intel.com/t5/Toolkits-SDKs/ct-p/toolkits-sdks ?  Your question will be answered by developers.

Hi zjin
yes, I added these parts into the code as below.
in the title I add these:
#if FPGA || FPGA_EMULATOR#include
<sycl/ext/intel/fpga_extensions.hpp>#endif
in the runkernel function I add these:
#if FPGA_EMULATOR        // DPC++ extension: FPGA emulator selector on
systems without FPGA card.    ext::intel::fpga_emulator_selector
d_selector;#elif FPGA        // DPC++ extension: FPGA selector on systems
with FPGA card.    ext::intel::fpga_selector d_selector;#else        // The
default device selector will select the most performant device.
default_selector d_selector;  //  gpu_selector  d_selector;#endif  //  try
{        // queue q(d_selector, dpc_common::exception_handler);
sycl::queue q(d_selector, dpc_common::exception_handler);   // queue
q(d_selector);
ok, I will send a case via the Intel community link. If you have any idea
how to modify the code, please provide me with some suggestions.
thanks
zjin-lcf @.***> 於 2022年7月7日 週四 晚上10:58寫道：

Oh, it is not your fault. Is there a SYCL device_selector for FPGAs ? Do
you mind posting the error message at
https://community.intel.com/t5/Toolkits-SDKs/ct-p/toolkits-sdks ? Your
question will be answered by developers.
—
Reply to this email directly, view it on GitHub
https://github.com/UoB-HPC/miniBUDE/issues/25#issuecomment-1178269232,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ53XHNKW6RKMMJK437X5S3VS5HJVANCNFSM52X5GPCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi zjin
According to the error message, I think these issues are about local_work_size and  global_work_size. Maybe the local_work_size exceed global_work_size while running Intel FPGA hw. Do you have any idea how to modify smaller size or something else size definition about your fasten_main BUDE kernel function?
void fasten_main(
handler& h,
//    size_t posesPerWI,
size_t wgSize,
size_t ntypes,
size_t nposes,
size_t natlig,
size_t natpro,
accessor<Atom, 1, sycl_read, sycl_gmem> protein_molecule,
accessor<Atom, 1, sycl_read, sycl_gmem> ligand_molecule,
accessor<float, 1, sycl_read, sycl_gmem> transforms_0,
accessor<float, 1, sycl_read, sycl_gmem> transforms_1,
accessor<float, 1, sycl_read, sycl_gmem> transforms_2,
accessor<float, 1, sycl_read, sycl_gmem> transforms_3,
accessor<float, 1, sycl_read, sycl_gmem> transforms_4,
accessor<float, 1, sycl_read, sycl_gmem> transforms_5,
accessor<FFParams, 1, sycl_read, sycl_gmem> forcefield,
accessor<float, 1, sycl_discard_write, sycl_gmem> etotals)
{
constexpr const auto FloatMax = std::numeric_limits<float>::max();

size_t global = ceil((nposes) / static_cast<double> (NUM_TD_PER_THREAD));
global = wgSize * ceil(static_cast<double> (global) / wgSize);

accessor<FFParams, 1, sycl_read_write, sycl_lmem> local_forcefield(range<1>(ntypes), h);

h.parallel_for<class bude_kernel>(nd_range<1>(global, wgSize), [=](nd_item<1> item) {

    const size_t lid = item.get_local_id(0);
    const size_t gid = item.get_group(0);
    const size_t lrange = item.get_local_range(0);

    float etot[NUM_TD_PER_THREAD];
    cl::sycl::float3 lpos[NUM_TD_PER_THREAD];
    cl::sycl::float4 transform[NUM_TD_PER_THREAD][3];

    size_t ix = gid * lrange * NUM_TD_PER_THREAD + lid;
    ix = ix < nposes ? ix : nposes - NUM_TD_PER_THREAD;

    for (int i = lid; i < ntypes; i += lrange) local_forcefield[i] = forcefield[i];
    //if (ix < ntypes) local_forcefield[ix] = forcefield[ix];

    // Compute transformation matrix to private memory
    for (size_t i = 0; i < NUM_TD_PER_THREAD; i++) {
        size_t index = ix + i * lrange;

        const float sx = cl::sycl::sin(transforms_0[index]);
        const float cx = cl::sycl::cos(transforms_0[index]);
        const float sy = cl::sycl::sin(transforms_1[index]);
        const float cy = cl::sycl::cos(transforms_1[index]);
        const float sz = cl::sycl::sin(transforms_2[index]);
        const float cz = cl::sycl::cos(transforms_2[index]);

        transform[i][0].x() = cy * cz;
        transform[i][0].y() = sx * sy * cz - cx * sz;
        transform[i][0].z() = cx * sy * cz + sx * sz;
        transform[i][0].w() = transforms_3[index];
        transform[i][1].x() = cy * sz;
        transform[i][1].y() = sx * sy * sz + cx * cz;
        transform[i][1].z() = cx * sy * sz - sx * cz;
        transform[i][1].w() = transforms_4[index];
        transform[i][2].x() = -sy;
        transform[i][2].y() = sx * cy;
        transform[i][2].z() = cx * cy;
        transform[i][2].w() = transforms_5[index];

        etot[i] = ZERO;
    }

    item.barrier(access::fence_space::local_space);

    // Loop over ligand atoms
    size_t il = 0;
    do {
        // Load ligand atom data
        const Atom l_atom = ligand_molecule[il];
        const FFParams l_params = local_forcefield[l_atom.type];
        const bool lhphb_ltz = l_params.hphb < ZERO;
        const bool lhphb_gtz = l_params.hphb > ZERO;

        const cl::sycl::float4 linitpos(l_atom.x, l_atom.y, l_atom.z, ONE);
        for (size_t i = 0; i < NUM_TD_PER_THREAD; i++) {
            // Transform ligand atom
            lpos[i].x() = transform[i][0].w() +
                linitpos.x() * transform[i][0].x() +
                linitpos.y() * transform[i][0].y() +
                linitpos.z() * transform[i][0].z();
            lpos[i].y() = transform[i][1].w() +
                linitpos.x() * transform[i][1].x() +
                linitpos.y() * transform[i][1].y() +
                linitpos.z() * transform[i][1].z();
            lpos[i].z() = transform[i][2].w() +
                linitpos.x() * transform[i][2].x() +
                linitpos.y() * transform[i][2].y() +
                linitpos.z() * transform[i][2].z();
        }

        // Loop over protein atoms
        size_t ip = 0;
        do {
            // Load protein atom data
            const Atom p_atom = protein_molecule[ip];
            const FFParams p_params = local_forcefield[p_atom.type];

            const float radij = p_params.radius + l_params.radius;
            const float r_radij = 1.f / (radij);

            const float elcdst = (p_params.hbtype == HBTYPE_F && l_params.hbtype == HBTYPE_F) ? FOUR : TWO;
            const float elcdst1 = (p_params.hbtype == HBTYPE_F && l_params.hbtype == HBTYPE_F) ? QUARTER : HALF;
            const bool type_E = ((p_params.hbtype == HBTYPE_E || l_params.hbtype == HBTYPE_E));

            const bool phphb_ltz = p_params.hphb < ZERO;
            const bool phphb_gtz = p_params.hphb > ZERO;
            const bool phphb_nz = p_params.hphb != ZERO;
            const float p_hphb = p_params.hphb * (phphb_ltz && lhphb_gtz ? -ONE : ONE);
            const float l_hphb = l_params.hphb * (phphb_gtz && lhphb_ltz ? -ONE : ONE);
            const float distdslv = (phphb_ltz ? (lhphb_ltz ? NPNPDIST : NPPDIST) : (lhphb_ltz ? NPPDIST : -FloatMax));
            const float r_distdslv = 1.f / (distdslv);

            const float chrg_init = l_params.elsc * p_params.elsc;
            const float dslv_init = p_hphb + l_hphb;

            for (size_t i = 0; i < NUM_TD_PER_THREAD; i++) {
                // Calculate distance between atoms
                const float x = lpos[i].x() - p_atom.x;
                const float y = lpos[i].y() - p_atom.y;
                const float z = lpos[i].z() - p_atom.z;

                const float distij = cl::sycl::sqrt(x * x + y * y + z * z);

                // Calculate the sum of the sphere radii
                const float distbb = distij - radij;
                const bool zone1 = (distbb < ZERO);

                // Calculate steric energy
                etot[i] += (ONE - (distij * r_radij)) * (zone1 ? 2 * HARDNESS : ZERO);

                // Calculate formal and dipole charge interactions
                float chrg_e = chrg_init * ((zone1 ? 1 : (ONE - distbb * elcdst1)) * (distbb < elcdst ? 1 : ZERO));
                const float neg_chrg_e = -cl::sycl::fabs(chrg_e);
                chrg_e = type_E ? neg_chrg_e : chrg_e;
                etot[i] += chrg_e * CNSTNT;

                // Calculate the two cases for Nonpolar-Polar repulsive interactions
                const float coeff = (ONE - (distbb * r_distdslv));
                float dslv_e = dslv_init * ((distbb < distdslv&& phphb_nz) ? 1 : ZERO);
                dslv_e *= (zone1 ? 1 : coeff);
                etot[i] += dslv_e;
            }
        } while (++ip < natpro); // loop over protein atoms
    } while (++il < natlig); // loop over ligand atoms

    // Write results
    const size_t td_base = gid * lrange * NUM_TD_PER_THREAD + lid;

    if (td_base < nposes) {
        for (size_t i = 0; i < NUM_TD_PER_THREAD; i++) {
            etotals[td_base + i * lrange] = etot[i] * HALF;
        }
    }
    });

Hi zjin
Also, why you define WGSIZE 4 and what's the meaning of it ? How to change it? I guess this parameter is also related to the issues I mentioned above.
#ifndef DEFAULT_WGSIZE
#define DEFAULT_WGSIZE 4
#endif
thanks

size_t global = ceil((nposes) / static_cast<double> (NUM_TD_PER_THREAD));
global = wgSize * ceil(static_cast<double> (global) / wgSize);

accessor<FFParams, 1, sycl_read_write, sycl_lmem> local_forcefield(range<1>(ntypes), h);

h.parallel_for<class bude_kernel>(nd_range<1>(global, wgSize), [=](nd_item<1> item) {

For the global and local work size:
range<1> gws (global);
range<1> lws (wgSize);
h.parallel_for<class bude_kernel>(nd_range<1>(gws, lws), [=](nd_item<1> item) {

Work-group size can be set at the command line using "--wgsize ". Does the above changes help ? If not, please wait for answers from Intel.

Hi zjin
ok, I will try to modify wgSize. Can I directly change this definition on the code? (#define DEFAULT_WGSIZE 4), cause I see you define it as params.wgSize = DEFAULT_WGSIZE;
Also, do you know where can i find ceil funtion original definition? I might need to refer it to consider how to change wgsizw.
    size_t global = ceil((nposes) / static_cast<double> (NUM_TD_PER_THREAD));
    global = wgSize * ceil(static_cast<double> (global) / wgSize);

Another issue is about (CL_INVALID_BINARY), after checking some websites. It seems about binary file issues.
CL_INVALID_BINARY if an invalid program binary was encountered for any device. binary_status will return specific status for each device.
https://www.khronos.org/registry/OpenCL/sdk/1.2/docs/man/xhtml/clCreateProgramWithBinary.html

Hi zjin
ok, I will try to modify wgSize. Can I directly change this definition on the code? (#define DEFAULT_WGSIZE 4), cause I see you define it as params.wgSize = DEFAULT_WGSIZE;
Also, do you know where can i find ceil funtion original definition? I might need to refer it to consider how to change wgsizw.
    size_t global = ceil((nposes) / static_cast<double> (NUM_TD_PER_THREAD));
    global = wgSize * ceil(static_cast<double> (global) / wgSize);

https://cplusplus.com/reference/cmath/ceil/

Hi zjin
If I want to try different conditions about decreasing total local size,
how to change the parameters on the code directly? I am using visual studio
to compile the code. Can you provide me some suggestion about changing the
number of the parameters?
zjin-lcf @.***> 於 2022年7月8日 週五 下午3:07寫道：

size_t global = ceil((nposes) / static_cast (NUM_TD_PER_THREAD));
global = wgSize * ceil(static_cast (global) / wgSize);
accessor<FFParams, 1, sycl_read_write, sycl_lmem> local_forcefield(range<1>(ntypes), h);
h.parallel_for(nd_range<1>(global, wgSize), [=](nd_item<1> item) {
For the global and local work size:
range<1> gws (global);
range<1> lws (wgSize);
h.parallel_for(nd_range<1>(gws, lws), [=](nd_item<1> item) {
Work-group size can be set at the command line using "--wgsize ". Does the
above changes help ? If not, please wait for answers from Intel.
—
Reply to this email directly, view it on GitHub
https://github.com/UoB-HPC/miniBUDE/issues/25#issuecomment-1179030556,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ53XHN4EXGNAGBPPLJWY5DVTAY3FANCNFSM52X5GPCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Any updates from Intel Forum about the errors ?

It seems they are on holiday...
[image: image.png]
zjin-lcf @.***> 於 2022年7月11日 週一 晚上10:41寫道：

Any updates from Intel Forum about the errors ?
—
Reply to this email directly, view it on GitHub
https://github.com/UoB-HPC/miniBUDE/issues/25#issuecomment-1180897800,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ53XHPUCEMO6UOXKA4Z4GTVTSIH5ANCNFSM52X5GPCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Isn't the work-group size a power-of-two number when the runtime reports the following errors ?
'what(): Native API failed. Native API returns: -42 (CL_INVALID_BINARY) -42 (CL_INVALID_BINARY)
make: *** [run_fpga_hardware] Aborted
for stratix10 error:
terminate called after throwing an instance of 'cl::sycl::nd_range_error'
what(): Non-uniform work-groups are not supported by the target device -54 (CL_INVALID_WORK_GROUP'

It's power of two, I tried 256 and 128 by modified --wgsize command
zjin-lcf @.***> 於 2022年7月11日 週一 晚上11:26寫道：

Isn't the work-group size a power-of-two number when the runtime reports
the following errors ?
'what(): Native API failed. Native API returns: -42 (CL_INVALID_BINARY)
-42 (CL_INVALID_BINARY)
make: *** [run_fpga_hardware] Aborted
for stratix10 error:
terminate called after throwing an instance of 'cl::sycl::nd_range_error'
what(): Non-uniform work-groups are not supported by the target device -54
(CL_INVALID_WORK_GROUP'
—
Reply to this email directly, view it on GitHub
https://github.com/UoB-HPC/miniBUDE/issues/25#issuecomment-1180937284,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ53XHL7KXTQQ4WFXRWYWT3VTSNS5ANCNFSM52X5GPCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Maybe I can try to decrease the complexity of the BUDE kernel. For example,
making the arithmetical operation simpler and then try on devcloud ?
Wayne Huang @.***> 於 2022年7月11日 週一 晚上11:31寫道：

It's power of two, I tried 256 and 128 by modified --wgsize command
zjin-lcf @.***> 於 2022年7月11日 週一 晚上11:26寫道：

Isn't the work-group size a power-of-two number when the runtime reports
the following errors ?
'what(): Native API failed. Native API returns: -42 (CL_INVALID_BINARY)
-42 (CL_INVALID_BINARY)
make: *** [run_fpga_hardware] Aborted
for stratix10 error:
terminate called after throwing an instance of 'cl::sycl::nd_range_error'
what(): Non-uniform work-groups are not supported by the target device
-54 (CL_INVALID_WORK_GROUP'
—
Reply to this email directly, view it on GitHub
https://github.com/UoB-HPC/miniBUDE/issues/25#issuecomment-1180937284,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ53XHL7KXTQQ4WFXRWYWT3VTSNS5ANCNFSM52X5GPCA
.
You are receiving this because you authored the thread.Message ID:
@.***>

It is a good idea for debugging a simplified kernel.
My ssh access to DevCloud fails for a while. I will try to build the mini-bude example for an FPGA when the issue is solved.

I hope that the the course instructor may work with you to understand the cause of the errors.  You are welcome.


Add Dark mode and Light Mode

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

I can work on this issue. Can you assign this issue to me!


Form styling

Demo: https://calvintam.dev/p/ce382342-c4dd-4d0a-8bf5-bd63d55a6611/
Looks like there is no styling for form? I am trying to setup a contact form using formspree.io.

Thanks for the heads up, I already added styles to form controls, although if you want to distribute the controls in some specific way, you will have to add your own styles in the Code injection section.

Thanks @eddiesigner


Detect US_PASSPORTS

A United States passport is an identifier for a U.S. citizen or national. Advocates for passports say that they are a way to ensure secure and efficient travel, by verifying the identity and nationality of the holder. The passport number is used to track the passport's use and validity. Therefore, it's important to keep the passport protected, as it contains private information. Passports are common in the United States and are required for international travel.
Every U.S. passport is unique, each number is a combination of letters and numbers. The format of the passport number is not publicly disclosed to maintain security.
By meeting the requirements, a passport can be identified by its format. For example, a U.S. passport number could look like this: C12345678. Please note that this is a hypothetical example and real U.S. passport numbers will have a different format.
For a more in-depth understanding regarding U.S. passports, you can visit here. Please remember to always protect your personal identification documents and numbers.

This sounds great, What's the difference between a real and fake passport identifier?

@jbrockington please assign this issue to yourself and add the sprint-1 milestone

This sounds great, What's the difference between a real and fake passport identifier?

I edited my submission. I plan to determine this by checking the dataset for whether the passport ID is registered.

This was very informative. The emphasis on the uniqueness of every U.S. passport was great. You conveyed the message of why a passport is important to be protected quite nicely.


❓ What do you want to learn?

What is it you want to learn? What specific scenarios are you interested in? Comment below 👇

@abelberhane

I hope to learn how to import an Action that helps with AWS Deployments 🙏🏽
I want to be able to confidently migrate Jenkins pipelines ✈
I would love to learn more about setting up Self-Hosted runners for our more on demand Actions

I want to learn how to deploy to my environment ✈
I want to learn how to write my own
I want to learn best practices around CI workflows ✅

I want to learn more about the scope of what Actions is capable of
I'm interested in running ansible playbooks through actions

I want to get comfortable with the basic structure, usage and setup of actions so I can explore more advanced capabilities.

I want to learn best practices, see some good examples that I could possibly apply for our Org

I want to make sure I'm prepared to support whatever the organization asks me to support, whatever that ends up being.
I'd like to increase my overall knowledge base and increase my skillset

I want to learn best practices and increase my skillset
See how actions can be used to do some neat things for the org.

environment management
deployment best practices including visualization


Unable to compile/build ohif viewer 4.4.0

Im unable to compile/build ohif viewer 4.4.0. My yarn and node version are up to date. The errorI get is in the screenshot below.

I just pushed a fix for this but apparently the CI did not cut a release properly. I'll try to fix that. You can clone from master and it should work fine.

Please review the latest code in the master branch. This issue might have been resolved. If it persists, kindly reopen the issue with updated details.
Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing


[MNT] Remove Reshape layer from Deep Learning Clusterers

Describe the issue
The tf.keras.layers.Reshape was introduced in clustering/deep_learnin to make the input shape of the encoder equal to the output shape of the decoder which is already being tested in test_all_networks.py. So, basically it is unnecessary at the moment.
Suggest a potential alternative/fix
Remove it.

maybe add some context as to why?

@TonyBagnall @aadya940 Would you be able to provide more context on this?
@aadya940 Is it open for me to solve?

@av6-github Yes, go ahead :)
Basically, you have to remove the following layer from the clusterer's since this is never actually needed.
https://github.com/aeon-toolkit/aeon/blob/main/aeon/clustering/deep_learning/_ae_fcn.py#L232

@aadya940 thank you!

@TonyBagnall I have forked the repo and requested the mentioned change.

@aadya940 since there is no output layer variable, what do I put in the outputs parameter in the model?

@av6-github You have to pass the decoder_output layer.

@aadya940 i am having some issues with pre commit and certain checks, so i am currently unable to make a proper pull request. :(
Apologies!

Hello, your PR seemed good to me. If you're having issues with Pre-Commit here are the steps:
Install:
python -m pip install pre-commit

Run:
pre-commit run --files path_to_file/file.py

Do let us know if you need any help.

@av6-github do you still need a hand with this? It is fine to keep a pull request open even if it is failing the CI currently.

@av6-github do you still need a hand with this? It is fine to keep a pull request open even if it is failing the CI currently.
Oh, thank you so much for the consideration, it means a lot!
I am really new to this so am a little confused here!
It'd be great if you could help me out here!

@av6-github Hello, your PR seemed good to me. If you're having issues with Pre-Commit here are the steps:
Install:
python -m pip install pre-commit

Run:
pre-commit run --files path_to_file/file.py

Do let us know if you need any help.

I tried this, but still am having issues!
Should I fork the repo again and do the changes on that and open a new pr?

Not sure what the issue is without logs really. You can check the errors by creating a PR and then clicking on details on the failing test.

hi @av6-github we were just reviewing open issues, are you still interested in resolving this? If so, @aadya940 could you advise? thanks.

@tanishy7777 Yes, sure go ahead.


Speed and memory concern

Thanks for README. I understand it's for structural reorganization.
go-whosonfirst-pip-v2 consumes days for Indexing. Is it possible to speed up indexing? hopefully ten or hundred times.
Another concern is memory usage. I'm not sure that memory thrashing is the main cause of the slow indexing or not. Of course we could use a larger machine, but we have some restriction.

Keeping in mind that this is all very much still work-in-progress, and the documentation remains spotty, you might have better luck with go-whosonfirst-spatial SQLite packages.
This demonstrates how the SQLite databases are created and queried:

https://github.com/whosonfirst/go-whosonfirst-spatial-sqlite#databases

And this demonstrates how those SQLite databases are queries over an HTTP interface, like the original go-whosonfirst-pip-v2 server:

https://github.com/whosonfirst/go-whosonfirst-spatial-http-sqlite#server

The SQLite databases take less time to index than the original in-memory RTree implementation and can be created independently of the PIP server itself.
They also have better support for "extra" properties to be included with the default responses and support indexing alternate geometries. As of this writing there is not support for filtering alternate geometries either by type or name.
The original go-whosonfirst-pip-v2 RTree implementation was always constrained on indexing time due to the volume of files being indexed, I/O limits and the sequential nature of building the RTree index itself. There is an equivalent RTree implementation of the go-whosonfirst-spatial interfaces but that hasn't been updated to reflect the most recent developments yet:

https://github.com/whosonfirst/go-whosonfirst-spatial-rtree

There is nothing about this package that will address any of the problems around indexing time, though. For large volumes of data something like the SQLite package is probably a better alternative.

Update the go-whosonfirst-spatial-rtree package has been updated to reflect the current state of the go-whosonfirst-spatial interfaces:

https://github.com/whosonfirst/go-whosonfirst-spatial-rtree#query
https://github.com/whosonfirst/go-whosonfirst-spatial-rtree/blob/main/cmd/query/main.go

Update 2: The go-whosonfirst-spatial-http-local package has been updated as well. This package assumes an rtree spatial index and a local checkout/repository of WOF records for indexing and for appending properties:

https://github.com/whosonfirst/go-whosonfirst-spatial-http-local#server

That seems nice.
I use "-allow-geojson -plain-old-geojson" flags with the v2 server to use GeoJSON files shaped from OSM's pbf files. Is it difficult to have a feature that makes db files from the geojson files with go-whosonfirst-sqlite-features-index (or something)?

It should be possible to use these tools with "plain old" GeoJSON documents, although that hasn't been tested yet.
If they don't work that will be considered a bug.

OK, thanks.

I'm glad to see you are active again after the shutdown. Be careful with COVID-19.

"How do I make a sqlite database from a GeoJSON file?"
Ultimately, you should be able to use the same wof-sqlite-index-features tool used fof WOF-style GeoJSON documents. I can't guarantee that this works today but that is the goal.
The code has reached the stage where ensuring that everything works with both "alternate" geometry files (WOF) and "plain old" GeoJSON is the next thing to work on, shortly.

There is initial support for indexing, serving and querying "plain old" GeoJSON in both the in-memory (RTree) and SQLite implementations:

https://github.com/whosonfirst/go-whosonfirst-spatial-http-local#indexing-plain-old-geojson
https://github.com/whosonfirst/go-whosonfirst-spatial-http-sqlite#indexing-plain-old-geojson

The in-memory server only support returning results as "standard places results" which mirrors the functionality of the go-whosonfirst-pip-v2 package. The SQLite servers supports optional formatting of results as "properties" dictionaries (appending properties to the SPR response) or GeoJSON.
It is entirely possible there are still bugs, edge cases or gotchas in both.

You mean the "-mode geojsonl" option of wof-sqlite-index-features?

I'm not sure I understand but if you're saying there's a bug in the go-whosonfirst-index code for processing line-separated GeoJSON then yes, that too :-)
I think things have reached the stage with the go-whosonfirst-spatial- packages that it will time, shortly, to focus on the documentation which will try to address the different layers. On of them being the indexing (or really "crawling" or "walking") of an arbitrary data source, independent of how those data are finally processed.
If there are bugs with the indexing stuff please file an issue in this repo or here:
https://github.com/whosonfirst/go-whosonfirst-index

Sorry, I wanted to know usage of wof-sqlite-index-features for plain GeoJSON files.
I'll stop here. I'll give it a try someday not so far away.
Thanks much!


script installs incompatible version on WSL

Code of Conduct

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

What happened?
when installing withcurl -s https://aliae.dev/install.sh | bash -s on wsl, bash responds with
/home/<user>/bin/aliae: line 1: Not: command not found
tool works when building from source
What OS are you seeing the problem on?
Linux
Which shell are you using?
bash

@cethien this seems to be an issue with the download (the executable isn't there, it's only an HTML page), I'll make the script more robust.


Gifs don't play

Hi, I installed Giphy and used native bottom sheet to select Gif but Gifs don't play. Also I used GiphyMediaView with ID but it didn't play as well.
I configurations are like;
in App.tsx
GiphySDK.configure({
apiKey: Constants.giphy_key,
});
<GiphyMediaView
ref={mediaRef}
resizeMode={'contain'}
media={{id: 'Ac0fCix8D3oN7DwCEB'}}
style={{
width: '100%',
aspectRatio: 1,
}}
/>

Giphy React Native SDK version: 3.2.4
React Native version: 0.73.6
Platform(s): Android
Device info: Emulator API 33 - Real device API 32 Debug

gif.webm

Hey @Tarikmnk,
please refer to this answer

@Tarikmnk is your issue  was resolved ?
I was facing same issue GIF not playing
RN version :- 0.73.4
GIPHY SDK - 3.2.6

@Nav-kr Hey, yes it solved my problem.


As an analyst, if revisions are not submitted within 2 weeks by the operator, send an automated reminder email to the operator

Describe the task
A clear and concise description of what the task is.
Acceptance Criteria

[ ] first
[ ] second
[ ] third

Additional context

Add any other context about the task here.
Or here

Deleted this issue. Will be opened in CIIP repository


Anyone got a jar?

Many people have a v0.06 version of the RTF jar. If possible can anyone send me any? I tried finding anything in discord but no one seems to have uploaded the jar.

Please don't ask for jars on the GitHub and close this issue. 0.0.7 isn't that far off so you should just wait for that to release.


possible problem with width/height: 26px definition in .leaflet-bar button:hover style

We've recently noticed that with Internet Explorer 11 button sizes are not always sizing properly to match the icon/glyph placed on the button, while Chrome, Firefox, and Safari render the buttons just fine. After a little trial and error, I seem to have narrowed this down to the the use of an explicit size of 26px in the definition of .leaflet-bar button:hover in easy-button.cs. If I change the width/height from 26px to 100%, the problem goes away and the button sizes render fine.
Old, before local change:
.leaflet-bar button,
.leaflet-bar button:hover {
background-color: #fff;
border: none;
border-bottom: 1px solid #ccc;
width: 26px
height: 26px;
line-height: 26px;
display: block;
text-align: center;
text-decoration: none;
color: black;
}
After change:
.leaflet-bar button,
.leaflet-bar button:hover {
background-color: #fff;
border: none;
border-bottom: 1px solid #ccc;
width: 100%;
height: 100%;
line-height: 26px;
display: block;
text-align: center;
text-decoration: none;
color: black;
}
Could someone please review this and see if a change is warranted to easy-button.cs?
Thanks,
Dave

A better solution might be to use:
min-width: 26px
min-height: 26px;
The use of width/height:100%, proposed earlier, may cause other problems.
Note that the issue of buttons not resizing to their content properly may only appear if you use your own styles for the button glyphs which are larger than 26x26 pixels.

For anyone else facing this problem you can use the following as a work-a-round until the plugin source gets fixed.
// The following method corrects a given easy button's size to match its content.
// Easy buttons under IE do not resize properly due to the easy button Leaflet
// plugin using explicit width/height styles of 26px instead of min-width/min-height.
//
// Root of this problem can be found in the easy button plugin's definition of
// .leaflet-bar button:hover in easy-button.cs, which contains the following
// lines:
//
// width: 26px
// height: 26px;
//
// These lines should use min-width/min-height instead. The reason for these
// lines is to ensure that the buttons rendered by the easy button plugin
// are at least the same size as Leaflet's internal buttons, such as the
// zoom buttons, but causes problems should the glyphs placed on such buttons
// exceed a size of 26x26 pixels, in that under IE the buttons do not
// size properly to accommodate larger glyphs.
//
// @param { L.easyButton } button - The easy button object to modify.
function _fixEasyButtonSize(button) {
	var buttonElement = button.button;
	buttonElement.style.width = "auto";
	buttonElement.style.height = "auto";
	buttonElement.style.minWidth = "26px";
	buttonElement.style.minHeight = "26px";
}

Example:
	button = L.easyButton("someButtonClassThatIsLargerThan26x26Pixels",  clickListenerFunction, "Description").addTo(map);
	_fixEasyButtonSize(button);

thanks for the extra detail, I'll fix this at the beginning of next week!

Thanks so much!!
Dave

hanginwithdaddo, thank you for this function of resolution. It works like a charm!


Warning because of the DoctrineCacheBundle recipe during install

  -  WARNING  doctrine/doctrine-cache-bundle (>=1.3): From github.com/symfony/recipes-contrib:master
    The recipe for this package comes from the "contrib" repository, which is open to community contributions.
    Review the recipe at https://github.com/symfony/recipes-contrib/tree/master/doctrine/doctrine-cache-bundle/1.3

    Do you want to execute this recipe?
    [y] Yes
    [n] No
    [a] Yes for all packages, only for the current installation session
    [p] Yes permanently, never ask again for this project
    (defaults to n):

A quick fix would be to move this recipe to the official repository.

I'm quite reluctant to add anything from Doctrine in the official repo as we are not able to influence these in any way.

Or maybe could we hide this warning during composer create-project?

AFAIK doctrine-cache-bundle is going to be (is already?) not-required anymore by doctrine-bundle,so that this issue will disapear.


Provide built-in boolean aggregate functions

Provide built-in boolean aggregate functions for supporting DBMS:

or_agg
and_agg

As I am using these I'd like to contribute these methods. I am just not sure what the default method is to provide custom functions. Do we have any example of built-in functions that we already provide that I can use as an example?

Take a look at how GROUP_CONCAT is implemented.

The default SQL names for bool_or and bool_and aggregation are EVERY and SOME in SQL. I've noticed other criteria builders also use ALL and ANY as function names for these aggregations (https://github.com/querydsl/querydsl/blob/2bf234caf78549813a1e0f44d9c30ecc5ef734e3/querydsl-core/src/main/java/com/querydsl/core/types/Templates.java#L275).
The PostgreSQL documentation states the following:

Note: Boolean aggregates bool_and and bool_or correspond to standard SQL aggregates every and any or some. As for any and some, it seems that there is an ambiguity built into the standard syntax:
SELECT b1 = ANY((SELECT b2 FROM t2 ...)) FROM t1 ...;

Here ANY can be considered either as introducing a subquery, or as being an aggregate function, if the subquery returns one row with a Boolean value. Thus the standard name cannot be given to these aggregates.

WDYT? Which naming should we go for?

SQL standard all / some (undecided) or any
QueryDSL standard for SQL : all / any
QueryDSL standard for JPQL :  every / some
PostgreSQL convention : bool_and / bool_or
@Mobe91 's suggestion: or_agg / and_agg

IMO the question is whether we actually can implement aggregate functions for booleans for all DBMS. It's functionally a bit different to what the quantifiers EVERY and SOME/ANY provide, because the quantifiers only work for subquery predicates. An aggregate function can appear anywhere as an expression and one can also use it as a window function.
I like or_agg and and_agg as names for the aggregate functions.

I know about the lack thereof in ANSI and the ambiguity in other dialects. MySQL and SQL Server seem to use ANY / EVERY as allowed boolean aggregates nontheless (at least that is suggested in https://stackoverflow.com/a/6060414/2104280 ).


Fix name of variable in description

I'm not 100% sure about this one, but I think you meant arr in the description to refer to the arr variable defined in the next line.

Thank you


graphql-java v2.1.0

@jcrygier As seen here the travis build has some issues with the bintray user; Would it be possible for you to move that to a separate profile/task? https://travis-ci.org/timtebeek/graphql-jpa
And would you be willing to merge these changes and push a release out that I can use in my projects? I'd like to start using this if possible..

@timtebeek Thanks for the pull...I'm no BinTray / Travis Expert, but I'll definitely work on getting them integrated, so that builds are trivial to push out.
I'm traveling for work this week, so it may take me a few days to get this merged in and deploys to BinTray automatically.

@timtebeek I had some extra time....so I think I have everything squared away.  Please check out the v0.2, that's recently been deployed from Travis, and let me know of any issues.
Thanks!

Great thanks! Couldn't find where it's been deployed to, but the current master branch works fine when checked out locally. :)
Having a minor issue now with mapping Enums, but I'll post the details in a separate issue.


[Parse] Skip attributes in isStartOfDecl().

Previously, @ token was unconditionally considered as isStartOfDecl()
Since canParseTypeTupleBody() fails at isStartOfDecl(),
generic argument containing type attributes, e.g.
Array<(@convention(block) () -> Int) -> Void>
failed to be parsed.

Alternatively, can we simply  drop isStartOfDecl() check in canParseTypeTupleBody()?
That passes all test cases we currently have.

This change looks great.
I think the Parser::canParseTypeTupleBody code you referenced is trying to prevent tentative parsing from going off the rails in ill-formed code. It's probably better to leave it.

@swift-ci Please smoke test and merge

Sorry, I should have throughly tested locally 😓
The failures are because this change can't handle these cases.
@ [code_complete]
func foo() {}

  @something(incomplete
  func foo() { }
}

@swift-ci Please smoke test

@DougGregor Sorry to bother you. Could you take a look again?
Updated the skipping logic.

This revision LGTM, thanks!

@swift-ci Please smoke test and merge


Stress Detection

ML-Crate Repository (Proposing new issue)
:red_circle: Project Title : Stress Detection
:red_circle: Aim : Stress detection is a challenging task, as there are so many words that can be used by people on their posts that can show whether a person is having psychological stress or not. So we would find if the person is in stress or not .
:red_circle: Dataset : https://www.kaggle.com/ruchi798/stress-analysis-in-social-media
:red_circle: Approach :  Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.
Hello, ML-Crate contributors, this issue is only for the contribution purposes and allocated only to the participants of SWOC 2.0 Open Source Program.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.
A Demo Project has been created to guide you about the whole structure of presenting the Project in this repository. Here's the link of the Demo Project - https://github.com/abhisheks008/ML-Crate/tree/main/Project-Demo-Folder

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'SWOC' contributors of 'ML-Crate' project.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Participant ID :
Approach for this Project :
Are you a participant of SWOC 2.0?

[ ] YES
[ ]  No

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Full name : Bhaswati Roy
GitHub Profile Link : https://github.com/BhaswatiRoy
Approach for this Project : NLP techniques to preprocess the texts and predict outputs
Are you a participant of SWOC 2.0?
YES


How to pad input sequences for implementing CuDNN LSTM in TF2.0?

For implementing CuDNN LSTM, according to docs there are 6 requirements as follows-

activation == 'tanh'
recurrent_activation == 'sigmoid'
recurrent_dropout == 0
unroll is False
use_bias is True
Inputs are not masked or strictly right padded.

According to my understanding, the last one say to have input sequences not right padded. However, in text classification I want to pad sequences and I am using the function, now I do not know how to left pad using this function, as I could check it always right pad the input sequences, which means one can never use CuDNN LSTM for training on GPU(s) if they are padding the inputs. However, it can be done easily using tf.keras..pad_sequences function, defined here.

I think you are misreading the requirement. Inputs should be either [not masked] or [strictly right-padded]. For reference, see the following function used to determine CuDNN compatibility: https://github.com/tensorflow/tensorflow/blob/8e423e3d56390671f0d954c90f4fd163ab02a9c1/tensorflow/python/keras/layers/recurrent_v2.py#L1182
Note: the LSTM with CuDNN is experiencing a lot of bug reports atm (e.g. in loops, in distributed strategies, etc.), all of which are open issues as of right now. However, concerning your question, I think you can close this issue.

@jkamalu thank you for your answer and clarification. I still have a question about how to left pad in TF2.0, without using tf.keras..pad_sequences function. Is it possible?
-Rishabh Sahrawat

Use tf.pad. Choose the constant mode and parameterize the padding on the left-hand side with the max desired length and current length of the sequence and use zero padding on the right side. Play around with eager tensors to get this right if it doesn't seem obvious from the documentation.
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/pad
If you want your input data padded, use tf.pad with the Dataset map function during data feeding.
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/data/Dataset#map
However, in my experience, this adds unnecessary overhead and can bottleneck training... So, what you should do if you can is preprocess all sequences to be padded however you like, save them to disk, and then load them as such during Dataset creation/iteration.

@jkamalu thank you for your answer. I appreciate your help.


Update copypaste-purchasing-transactions-from-excel.md

add info

Docs Build status updates of commit 6c1ca0f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/whats-new/copypaste-purchasing-transactions-from-excel.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 38bf44b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/whats-new/copypaste-purchasing-transactions-from-excel.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


build map view

As a visitor I want to view hub listings with a map and filter by continents

@TimJones97  I might be building the filter components today and sharing it on bit.dev, If I do then just plugin the filters with the list view using the props given. Simply update the version number of the '@buildcities'  components package and run yarn to get latest component updates.


Suggested Updates to New-PnPTenantSite.md

The LCID references do not list the ID codes for creating a site with a specific LocaleID with a type of UINT32 so have added a command to get that from SharePoint online. Minor corrections to the other commands and seemingly corrected a bad URL at the bottom of the article.

:x: Validation status: errors

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-pnp/New-PnPTenantSite.md
:x:Error

Details

:x:Error

Details

whiteboard/whiteboard-ps/whiteboard/whiteboard.md
:warning:Warning

Details

sharepoint/sharepoint-ps/sharepoint-pnp/New-PnPTenantSite.md

[Error] Unable to load file: sharepoint-ps/sharepoint-pnp/New-PnPTenantSite.md via processor: AzureCliDocumentProcessor: :231:(853) '* '
Expect hyperlink, but got *

[Error] Markdown.MAML.Transformer.HelpSchemaException: :231:(853) '* '
Expect hyperlink, but got *
at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.GetInnerContexts(DocumentBuildParameters parameters, IEnumerable1 processors, TemplateProcessor templateProcessor, IHostServiceCreator creator) at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.BuildCore(DocumentBuildParameters parameters) at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.Build(DocumentBuildParameters parameters) at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.BuildCore(DocumentBuildParameters parameter, IMarkdownServiceProvider markdownServiceProvider, BuildInfo currentBuildInfo, BuildInfo lastBuildInfo) at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.Build(IList1 parameters, String outputDirectory)
at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument(BuildJsonConfig config, TemplateManager templateManager, String baseDirectory, String outputDirectory, String pluginDirectory, String templateDirectory)
at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument()

whiteboard/whiteboard-ps/whiteboard/whiteboard.md

Line 12: [Warning] Invalid file link:(~/whiteboard-ps/whiteboard/overview.md).

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.

Feel free to reject, just seen the notice on the homepage readme. If someone contributes via the edit button on docs site they would not see the warning, as I have just done.

@pkbullock - thanks for contribution, much appreciated! There is a check that looks at the PowerShell schema ( https://github.com/PowerShell/platyPS/blob/master/platyPS.schema.md ). It is super picky and never very clear what causes an error. I have even had just a single character complain on me and it has taken me forever to figure it out. :)
Also, thanks for pointing out the warning in whiteboard content. I will check that out and get that resolved. Feel free to re-open or ping me if you want to go ahead with a change to the PnP cmdlets.


Automate deployment

In #90, we have an (awesome!) protocol for deploying the app. It would be even better to automate most or all of it, using one of (in the increasing order of complexity):

A bash script
Makefile
Chef cookbook

@tskir I suggest I dockerize the app and its dependencies to make deployment much easier. You already did mention some services within EBI that use Docker images by default, however, since I got some Docker experience since we discussed that issue, I believe we could fully automate deployment on Embassy as well using webhooks once the app and its dependencies have been dockerized. That would of course require setting up pipelines, which I suggest we do using GitHub Actions.

@joj0s All of that sounds great! I agree with using both Docker & GitHub Actions.
Also migrating to Docker will pave the path to an even more exciting possibility: Embassy has something called “Embassy Hosted Kubernetes” (EHK). Once we have a Docker image and we can successfully deploy it inside our self-managed virtual machines, we could try going even further and trying out that EHK thing.
However, that's a story for a more distant future; but in the meanwhile you have my full approval to go ahead with the Docker & GitHub Actions.


[BUG] Consistent server freeze

Version of Advanced Rocketry

1.16.5-2.0.0-10

Have you verified this is an issue in the latest unstable build

[x] Y
[ ] N
[ ] N/A

Version of LibVulpes

1.16.5-1.0.2-18

Version of Minecraft

1.16.5

Does this occur without other mods installed

[ ] Y
[ ] N
[x] N/A

Crash report or log or visualVM (if applicable)

https://spark.lucko.me/NpdDkl4eB9

Description of the problem

Server consistently freezes for a short period of time. In my case (on default config) I have 94 files ~4kb each, which results in "Can't keep up! Is the server overloaded? Running 5456ms or 109 ticks behind" every 5 mins. See profiling info above

Looks like it's related to waiting for disk write.  This is a really tricky one I think because in order to determine if the files are stale I'd still need to read from the disk, which would also cause lag.  These json files are generated from the planetDef.xml, and I can't think of any way to reliably determine if that file has been editted.
I suppose what I could do is only write the JSONs the first time the world is saved, so instead of lagging every time the world is saved, it only lags the first time.
Ideally I'd like to do away with the datapack thing entirely but have not found a way to load dimensions in minecraft directly yet since they changed everything.

Is there a way to read/write dim files using another thread? I don't know whether it's a good idea in minecraft ecosystem or not, just curious.

yeah, it's possible and honestly not something I'd really thought about because all the other save things called in the same method can be edited in game, but the .json datapack config doesn't really change outside of some command being executed.
Worst case I suppose is that the time it takes to save is longer than the interval between saves and it ends up hanging waiting for the previous save to finish.


fix(buildozer): add cfg parameter to the buildifier attribute

Sorry for these obvious issues, I missed them when initially adding this attr.


[Validation] report duplicate function parameters

#16 adds duplicate identifier warnings, but doesn't cover duplicate function parameters.

@mattmasson I know this is existing SDK behavior, but how important is this if we're already validating the number of arguments and if their types match?

@mattmasson I know this is existing SDK behavior, but how important is this if we're already validating the number of arguments and if their types match?

Never mind. I was misreading this as a InvokeExpression issue, not as a FunctionExpression issue.


bug: Wiggly lines at rust method and macro calls

Did you check docs and existing issues?

[X] I have read all the flow.nvim docs
[X] I have searched the existing issues of flow.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.11.0-dev-387+g158ffd646 -> With Neovide
Operating system/version
Win 11
Describe the bug
Hey,
I just tried your colorscheme and really like it! There is just one thing which is kinda distracting IMHO.
Wiggly lines
If I use Neovide + Rust, many method and macro calls get some wiggly red lines below them.

LSP messages get the straight underline:

In Alacritty + NVIM + Rust they just get underlines, which most likely is the expected behavior:

Also the LSP messages get the straight underline:

Because my brain is wired to see wiggly lines as spelling or code mistakes, this is kinda distracting.
Dark blue contrast
Maybe this should be a different issue/suggestion, but the dark blue for function/method names could be a tad brighter for improved readability? The contrast to the background is a bit low, at least without a transparent background.
Best regards
Steps To Reproduce

Install Neovide
Use lazy.nvim
Add this repo as a plugin

Expected Behavior
No wiggly lines in Neovide for method and macro calls but for the LSP warnings these would be nice, if not necessary.
Repro
I use lazy.nvim with the colorscheme plugin with the following content of the colorscheme.lua:

return {
  {
    "0xstepit/flow.nvim",
    lazy = false,
    priority = 1000,
    opts = {},
    config = function()
      require("flow").setup({
        transparent = false,
        fluo_color = "pink",
        mode = "normal", -- "bright", "normal", "dim"
        aggressive_spell = false,
      })
    end,
  },

  {
    "LazyVim/LazyVim",
    opts = {
      colorscheme = "flow",
    },
  },
}

Hey @Drazhar, thanks for opening the issue and for the feedback.  I tried with some rust project I have but I cannot see all the underline you have.

For what concern the blue function name/ function, I usually prefer to have more focus on the inside of a function instead of its name. But this is my personal feeling, maybe I could provide a way to change some nvim group color

Hey @Drazhar, thanks for opening the issue and for the feedback.  I tried with some rust project I have but I cannot see all the underline you have.

These where just some small tests I did with Bevy. The issue also shows up in a brand new hello world Rust project initialized with cargo new:

But again, it's only in combination with Neovide. NVIM in the Alacritty terminal only shows the underline:

For example with the theme kanagawa, I don't get any underlines:

After some digging
I realized, that only some methods/macros from the bevy and the std library had the wiggly lines and also didn't show any hover information (lazy.nvim default +K). Methods from my own structs or macros didn't show any lines.
After some more testing, it seems, that the issue is within the Pixi environment I'm using. Inside the pixi shell, no definitions from bevy or the std library can be read by the rust-analyzer. But the project can be build and run without any issues.
If I use my global rust installation, I don't have any issues with the rust-analyzer and no lines are shown.
Conclusion
The issue isn't caused from your color-scheme.
But thanks to the flow theme I spotted an issue with my pixi environment. So an additional thanks for that.

For what concern the blue function name/ function, I usually prefer to have more focus on the inside of a function instead of its name. But this is my personal feeling, maybe I could provide a way to change some nvim group color

On my main monitor, the readability of the dark blue color for the functions is way better than on the go on my laptop screen. So this also isn't really an issue anymore. But an option to change the brightness would still be nice to have.

I will close this topic, because the main issue is solved ⭐


does it work well with openjdk-8-jdk

i am using ubuntu
if i use the openjdk instead of oracle jdk !
can the plugin work properly?

I'm not sure. Do you have any issues?

i have tried , the plugin can work properly!


Type cast error because of "remainingBackends"

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
After I applying this policy to APIM it cause errors when it try to loadbalance to other endpoints.
https://github.com/Azure-Samples/openai-apim-lb/blob/main/apim-policy.xml
Any log messages given by the failure
[{"elapsed":313,"source":"retry","reason":"ExpressionValueEvaluationFailure","message":"Expression evaluation failed. Unable to cast object of type 'System.Int32' to type 'System.String'.","scope":"operation","section":"backend"}]
Expected/desired behavior
There seems to be an issue with https://github.com/Azure-Samples/openai-apim-lb/pull/26.
It occured type cast error because of "remainingBackends".
OS and Version?
Ubuntu 22.04
Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

I am also experiencing this type cast issue

I tested the changes proposed in this PR and the errors went away for me. Hope that PR gets merged soon.

Hello,
This is the same issue as in #29
I proposed another solution in the comment.
But the fix from #18 is working as well 🙂


real time tracelog

Is it possible to do tracelog real time at the moment you execute it.  Currently it seems the messages are buffered and displayed at the end. Real time tracelog is useful for debugging issues that would otherwise be hard to do in a debugger. I'm not sure if there is a macro or some switch to make it real time?

Actually, TraceLog() messages are sent to console output in real time; if youy execute the program from command line you will see those messages as soon as they occur.
Maybe you refer to Notepad++ output console? It seems they are not displayed until program is closed...

you are correct.  All though i'm using VSCode output console. However; it displays fine on command line.  These plugins probably open a new process and try to read the buffer output.  Thus they get buffered until exit.


Minimized Card - Description not visible

Hi,
When minimizing cards on the board, the decription is not available.
I don't know if it's a bug as when we hover the title link, the title is put in the tootip or if it's a new feature to add the small icon to have the description in this mode.
thanks,

@elvince

Not a bug.
Since minimizing cards -  a.k.a. "Collapse tasks" - was introduced in v 1.0.12, description icon has not been present on the collapsed task.

Thanks,
I implemented locally, but I didn't take into account the Title length. So on big title, it goes over the icon ^^
in /app/Template/board/task_private.php
Line 23:
add:
<?php if (! empty($task['description'])): ?>
        <span title="<?= t('Description') ?>" class="tooltip" data-href="<?= $this->url->href('board', 'description', array('task_id' => $task['id'], 'project_id' => $task['project_id'])) ?>">

    <?php endif ?>


compat break

fixing runtitle for compat scenario

Approved.

Is the fix already in master? Not yet.
Does the fix change the AT/DT contract? No
Does the fix include any database schema changes? No
Code Review Sign-off: prmittal
Test Sign-off: allendm
What testing was done? Manual validation


how can I get the  initial  index of the current target

Result like this

I use
$('#jcarousel').jcarousel('target').index();
but not work right
I look the source code find Pagination Plugin can get the current page, but this Plugin must work with some element and create some elements. I don't want to do that. so could you tell me how can I get the  initial  index of the current target just with jCarousel core code??
good luck happy work

This should work. But hard to say without actually seeing some code. You probably need to call this method in an event handler, eq. on createend.

ok thx


Working with AWS Lambda

Is there any ways to make local telegram-bot-api work with AWS Lambda?

No.


update(k8s-metacollector): bump application version to 0.1.1

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

/area event-generator-chart

/area k8s-metacollector

What this PR does / why we need it:
update(k8s-metacollector): bump application version to 0.1.1
Which issue(s) this PR fixes:
https://github.com/falcosecurity/k8s-metacollector/issues/102

Fixes #
Special notes for your reviewer:
Checklist

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: alacuku
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [alacuku]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add PRIVMSG by command

Should relay response back to the appropriate sender (if possible? probably not without ircv3 tags).
Simplest option: output to console like it does right now
/thump message <network> <target> <rest of message>

Technically possible to do now with /thump sendraw <network> <IRC message>


Allow defining convertors for types collections in spring data r2dbc

Currently, it's impossible to define custom converters for typed collections fields (and documentation clearly states that)
It becomes a problem when a mapped field in the database is JSON/JSONB type (I'm working with Postgres).
So, if I have multiple JSON fields in the database, like this:
public class Example {
    ...
    @Column("a")
    private List<A> a;
    @Column("b")
    private List<B> b;
    ....
}

it's impossible to define proper converters for these types (as MappingR2dbcConverter ignores generics, and only 1 converter can be defined).
On the other hand, though the documentation says that in this case Converter should be defined for inner types (A and B in the example), it doesn't work in the case of JSONB fields.

Have you considered a wrapper object that holds your collections?
class MyListA {
private List<…> a;
}

Generic type parameters need to be sourced from somewhere. We generally only apply converters to elements within a collection. Generic type parameters are not available from object instances so we cannot consider these with Spring's Conversion Service.


Daily processes failing

Cronic detected failure or error output for the command:
bash -c sudo /home/ubuntu/compiler-explorer-image/update_compilers/install_nonfree_compilers.sh

RESULT CODE: 2

ERROR OUTPUT:
python: can't open file '/list_compilers.py': [Errno 2] No such file or directory

...will take a look.

My fault :/


Request: Customized Image Download Path

The MediaSync plugin provides a great image download feature, but the downloaded images are saved in the "_media-sync_resouces" folder, which is not ideal for file management. Could you please add different options so that users can set their preferred download path for images?
In other words, when pasting local images into Obsidian, according to Obsidian's settings, the images are saved in the corresponding folder. For the MediaSync plugin, it would be great if it could follow a similar approach and automatically download images to the attachment folder set in Obsidian's settings.

@iplaywind
Implemented in version 0.0.7. Please try it.
https://github.com/fnya/media-sync/releases/tag/0.0.7


Remove codeowners validator from pre-commit

Fixes

Fixes and issue reported by @dhruvkb and @stacimc they had running the pre-commit hook on their local machines.
Description

The docker image has issues running on arm64, so we will instead run it in CI using the GitHub action, with an utility just recipe to run the same checks locally as needed.
I also updated the CI/CD documentation to explain the new job and document the lint-codeowners recipe.'
Marked high because it is affecting the ability for folks to work on the codebase without this tedious interruption.
Testing Instructions

CI should pass with the new job and the just lint-codeowners recipe should work, including just lint-codeowners all to enable the experimental checks.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[N/A] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[N/A] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Yes, that's precisely the issue with arm64 that Staci and Dhruv ran into that caused us to need to switch away from running it locally in pre-commit. The recipe is there for folks who can run it to be able to easily do so, which the documentation notes:

It is still possible to run this hook locally, assuming the docker image works on your computer’s architecture, by invoking the lint-codeowners just recipe:

(Emphasis mine)
In other words, that's intended behaviour :)
Thanks for the reviews, y'all!


test: Update expectations for firefox support of provideResponse

Needs to wait until https://bugzilla.mozilla.org/show_bug.cgi?id=1853882 lands.
Summary:

add missing mock-binary-response.png
update multiple headers test to handle Firefox, which uses , as the default separator for header values
update various expectations

Could you please specify which test is it? and let's wait till it starts to pass on the PR?

The test is [requestinterception.spec] request interception Request.respond should allow mocking binary responses

Fine with me to wait until the test passes to accept the PR, I'll just apply the change locally for now. I guess I can keep the PR and update the test expectations at the same time.

Bug  https://bugzilla.mozilla.org/show_bug.cgi?id=1853882 should now be in Nightly

Note that CI will fail because we also need the changes from #12674 (which on its own fails CI as well because it misses the changes from this PR).
@OrKoN do you want use to do a combined PR with both?

Sounds good, thanks!


Does it support operating monitoring items by calling the api?

Question
是否支持通过调用api来操作监控项
例如添加和删除监控项

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

Question
Whether to support operating monitoring items by calling API
For example, adding and deleting monitoring items

支持的 有对应的rest api可以在swagger页面找到 http://localhost:1157/swagger-ui/index.html

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

The supported rest api can be found on the swagger page http://localhost:1157/swagger-ui/index.html