undefined method 'keepalive'

Hello,
When following the tutorial on http://godrb.com/ I get
$ rvmsudo god -c simple.god -D
I [2012-01-13 21:54:09]  INFO: Loading simple.god
I [2012-01-13 21:54:09]  INFO: Syslog enabled.
I [2012-01-13 21:54:09]  INFO: Using pid file directory: /var/run/god
There was an error in simple.god
	undefined method `keepalive' for #<God::Watch:0x00000001377ef8>
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/task.rb:219:in `method_missing'
	/home/ec2-user/god/simple.god:4:in `block in <top (required)>'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god.rb:269:in `task'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god.rb:253:in `watch'
	/home/ec2-user/god/simple.god:1:in `<top (required)>'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/cli/run.rb:156:in `load'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/cli/run.rb:156:in `load_god_file'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/cli/run.rb:148:in `block in load_config'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/cli/run.rb:147:in `each'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/cli/run.rb:147:in `load_config'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/cli/run.rb:72:in `default_run'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/cli/run.rb:84:in `run_in_front'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/cli/run.rb:23:in `dispatch'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/lib/god/cli/run.rb:8:in `initialize'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/bin/god:122:in `new'
	/usr/local/rvm/gems/ruby-1.9.2-p290/gems/god-0.11.0/bin/god:122:in `<top (required)>'
	/usr/local/rvm/gems/ruby-1.9.2-p290/bin/god:19:in `load'
	/usr/local/rvm/gems/ruby-1.9.2-p290/bin/god:19:in `<main>'
E [2012-01-13 21:54:09] ERROR: File 'simple.god' could not be loaded

Is the tutorial outdated?

This is not clear on the tutorial.
gem install god installed 0.11.0
Maybe an update to the tutorial?

Actually, I believe you just updated the gem on rubygems :)

Yes, I just did the release yesterday. I'll update the docs to make
sure it's more clear.
On Fri, Jan 13, 2012 at 7:13 PM, pcasaretto
<EMAIL_ADDRESS>wrote:

Actually, I believe you just updated the gem on rubygems :)
—
Reply to this email directly or view it on GitHub.


Problem displaying the logs when the lines contains '<'

When I see the logs from the extension I see this:
	from /app/config/application.rb:24:in `'
	from /app/config/application.rb:22:in `
'

but the original lines are:
     from /app/config/application.rb:24:in `<module:ModuleName>'
     from /app/config/application.rb:22:in `<main>'

It happens on every line when some class name is enclosed in '<' '>'

@arbesulo the new version has been released 2 weeks ago. Have you had the chance to verify it? Do you still face this issue? Let us know, thanks!!

Now it works perfect. Thanks for solving the issue!

Cool, you're welcome!! Closing it then. If you face any other issue or have some ideas how to improve the extension, let us know please. Thanks!!


Duplicate classes in _ide_helper.php (cartalyst themes)

Versions:

ide-helper Version: ^2.12
Laravel Version: 9.11.0
PHP Version: 8.1.5

Question:
Tell me, for what reason can there be two classes in _ide_helper.php?

Similar problem.

same problem here

same here

I have the same issue with \Barryvdh\DomPDF now. What is the correct way to handle this? Why does ide_helper generate the class twice?


fix some spelling

Very nice. What tool did you use for spell checking?

@JamesNK r#

what are tspecials?
// tspecials are required to be in a quoted string.  Only non-ascii needs to be encoded.

@Tratcher should this be rebased on release/2.2 branch?

Having this in release/2.2 would be nice but is not blocking. @SimonCropp want to rebase?

cherry picked+squashed+resubmitted https://github.com/aspnet/HttpAbstractions/pull/1035


Can not reinstall chainer, please help me

I install chainer and want to reinstall. When I reinstall ,it happen errors. My computer is Ubuntu 16 system, and in my computer I have two version python ,python2.7 and python3.5.
error:
Exception:
Traceback (most recent call last):
File "/home/suiishii/.local/lib/python3.5/site-packages/pip/basecommand.py", line 215, in main
status = self.run(options, args)
File "/home/suiishii/.local/lib/python3.5/site-packages/pip/commands/install.py", line 342, in run
prefix=options.prefix_path,
File "/home/suiishii/.local/lib/python3.5/site-packages/pip/req/req_set.py", line 784, in install
**kwargs
File "/home/suiishii/.local/lib/python3.5/site-packages/pip/req/req_install.py", line 851, in install
self.move_wheel_files(self.source_dir, root=root, prefix=prefix)
File "/home/suiishii/.local/lib/python3.5/site-packages/pip/req/req_install.py", line 1064, in move_wheel_files
isolated=self.isolated,
File "/home/suiishii/.local/lib/python3.5/site-packages/pip/wheel.py", line 345, in move_wheel_files
clobber(source, lib_dir, True)
File "/home/suiishii/.local/lib/python3.5/site-packages/pip/wheel.py", line 316, in clobber
ensure_dir(destdir)
File "/home/suiishii/.local/lib/python3.5/site-packages/pip/utils/init.py", line 83, in ensure_dir
os.makedirs(path)
File "/usr/lib/python3.5/os.py", line 241, in makedirs
mkdir(name, mode)
PermissionError: [Errno 13] Permission denied: '/usr/local/lib/python3.5/dist-packages/cupy'

It says, you don't have permission. Please use sudo or install chainer to your local environment with --user option: pip install -U chainer --user

It works. Thank you very much.


Is there a way of decoding only I-frames for h.264

You can leverage SetSkipMode API function:
Description

This function sets the decoder skip mode. The application may use it to increase decoding performance by sacrificing output quality. The rising of skip level firstly results in skipping of some decoding  operations like deblocking and then leads to frame skipping; firstly, B then P. Particular details are platform dependent.

Right, but noticeable moment - it doesn't allow you to control skipped steps, so for example if you want to skip all frames except I, but want to kip deblocking - it won't allow you to perform right way, we are working on new functionality to allow skip level selection


drivers: lora: sx126x: Support software-managed RF switches

The SX126x supports using DIO2 to control an TX switch, but only if the switch can be operated using a single control signal. Add support for RF switches that are wired to a GPIO instead of the radio chip itself. This makes it possible to use RF switches that require two control signals (one for the RX port and one for the TX port) by wiring them to two GPIOs on the MCU.
Testing: Tested with custom E22 module and an NRF52840 DK board and TTN.
NOTE: Depends on: https://github.com/zephyrproject-rtos/loramac-node/pull/5
NOTE: Check patch errors look like false positives.

Added DNM, because this PR contains pending west.yml update.

Updated to work with new SX126x HAL API from upstream.

It turns out that using the mode tracker to control the RF switch potentially has some reliability issues. The MAC currently updates the mode after sending a command to the radio. That is, there is a short period between enabling TX and enabling the TX path in the RF switch. This is usually not a problem, but a preemption could delay that indefinitely and cause undesirable behaviour (some RF switches reflect RF energy back to the power amplifier, which could cause hardware damage). Re-ordering mode updates and commands is non-trivial since that makes it impossible for the HAL to know if it needs to wake the radio from sleep. Upstream is working on a long-term solution, but the unexpected complexity means that this will take some time.
My suggestion is that we track the state by monitoring the commands sent to the radio and the interrupts. This allows us to perform more accurate state tracking and can be done with no/minimal patching of the MAC. See https://github.com/andysan/zephyr/tree/sx126x-rf-switch-v2 for an implementation that does that.

It turns out that using the mode tracker to control the RF switch potentially has some reliability issues. The MAC currently updates the mode after sending a command to the radio. That is, there is a short period between enabling TX and enabling the TX path in the RF switch. This is usually not a problem, but a preemption could delay that indefinitely and cause undesirable behaviour (some RF switches reflect RF energy back to the power amplifier, which could cause hardware damage). Re-ordering mode updates and commands is non-trivial since that makes it impossible for the HAL to know if it needs to wake the radio from sleep. Upstream is working on a long-term solution, but the unexpected complexity means that this will take some time.
My suggestion is that we track the state by monitoring the commands sent to the radio and the interrupts. This allows us to perform more accurate state tracking and can be done with no/minimal patching of the MAC. See https://github.com/andysan/zephyr/tree/sx126x-rf-switch-v2 for an implementation that does that.
Let me know if we should switch to the V2 approach. This could be merged today if we cherry-pick a single change from upstream into the LoRaMAC repo.

I have reviewed the HAL version on upstream (the relevant changes are included in https://github.com/zephyrproject-rtos/loramac-node/pull/5) and the current HAL version seems OK in most cases. The only potentially problematic case now is a direct transition from TX->RX in which case there is a (theoretical) possibility that the RF switch could be in the wrong state for some time. The only cases I can think of where this could happen (and cause a problem) are:

Test modes (continuous wave or continuous preamble transmission) that are stopped by transitioning in to RX mode.
Transmissions that are aborted prematurely by a transition into RX mode.

None of these should normally happen, so this should be safe for normal users.

Regarding current and V2 approaches I am more keen to stay with current code. I don't know the details of all the risk involved with wrong rx/tx switches, when they arise, etc. However it sounds to me like there is a problem in loramac-node repository that we would try to solve it in Zephyr. At least I don't see clear reason for now to do it that way.

Regarding current and V2 approaches I am more keen to stay with current code. I don't know the details of all the risk involved with wrong rx/tx switches, when they arise, etc. However it sounds to me like there is a problem in loramac-node repository that we would try to solve it in Zephyr. At least I don't see clear reason for now to do it that way.

They should never occur unless you are testing the device (CW or continuous preamble) or there is something wrong in the MAC (premature termination of TX by a direct transition to RX). Even then, you need a certain amount of bad luck that causes a preemption within a relatively small window. I don’t think this is a blocker, so I agree that the current code is the best solution. I’ll try to find time to prepare a patch for upstream, but we don’t need to wait for that before merging this.

UPDATE: Rebased and updated to use pin handling helpers by @mniestroj.

@Mani-Sadhasivam could you review please?

Please update the SHA now that https://github.com/zephyrproject-rtos/loramac-node/pull/5 is merged

Please update the SHA now that zephyrproject-rtos/loramac-node#5 is merged

Thanks for merging it. I have now updated the SHA.

Rebased to trigger new test run as the previous run contained unrelated failures.

Ping @carlescufi.  This should be ready to be merged. The style warning is triggered incorrectly by a macro that uses designated initialisers.

@andysan can you rebase and fix merge conflict.

@andysan can you rebase and fix merge conflict.

Done.

checkpatch error is N/A, since it's a macro.


Animate NewReactionButton

Adds a bounce-in effect to the hover state of NewReactionButton

[x] performance

@keybase/react-hackers this is ready!

@keybase/react-hackers bump

OK @chrisnojima, addressed comments PTAL


can you add bloons td 4 or 5 with html5 support?

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

@fnfgo


Tagging release 0.6

I am going to tag the current master branch as release 0.6. The recent commit 2c62cc9 seems important, and we want to start using it in our research pipeline. I will include a Dockerfile. Let me know if you have any concerns.

I updated version numbers in source/docs, updated the Dockerfile, updated the linux binary, and tagged a 0.6 release. I am unable to update the OSX binary without immediate access to an OSX machine. But I'll leave this ticket open till I am able to do that, and then add a new commit that tag 0.2 will point to.

Thanks Cyriac for releasing 0.6 and made a Dockerfile. The commit 2c62cc9 fixed or added 1.4% MS sites across the whole human genome in scan step.

I was unable to make a macos binary - so I decided to remove the binaries altogether - and attach them to the releases instead - https://github.com/ding-lab/msisensor/releases - this will allow us to add binaries later, without having to add commits or moving tags. Release 0.6 should be good to go.


Negation control

v3.0.0 and later allows to use negation rule unrestricted (although at least not again) for a specific task. This leads to a lot more strain on the control mechanism. v1.6.5's handling was better, let's go back to it. Let negation rule apply to negations only.
The inference rules allow for negation anyway, example:
<a --> A>. %0.00;0.90%
<(--,<a --> A>) ==> <c --> C>>. %1.00;0.90%
|-
(--,<a --> A>). %1.00;0.90%
<c --> C>. %1.00;0.81%
Note that it also produced the negation, but only because the implication existed.

Maybe there was an example where the restricted treatment was insufficient.  Let's carefully investigate before changing it back.

Resolved with https://github.com/opennars/opennars/commit/41cf1e2635bac9e600df33c206a5738dc6ae88b3 in v3.0.2 dev branch.


fix(build): deply to pypi org from 3.6

:tada: This PR is included in version 1.10.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.10.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Failure by proxy.

Under normal circumstances I Goagent as proxy access youtube.com.
In commen-line:
youtube-dl --proxy "https://<IP_ADDRESS>:8087" -v "https://www.youtube.com/watch?v=Moy1nWM3Zmg"

and the output is:
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--proxy', 'https://<IP_ADDRESS>:8087', '-v', 'https://
www.youtube.com/watch?v=Moy1nWM3Zmg']
[debug] youtube-dl version 2013.05.14
[debug] Python version 2.7.3 - Windows-7-6.1.7601-SP1
[debug] Proxy map: {'http': 'https://<IP_ADDRESS>:8087', 'https': 'https://127.0.0.
1:8087'}
[youtube] Setting language
[youtube] Moy1nWM3Zmg: Downloading video webpage
[youtube] Moy1nWM3Zmg: Downloading video info webpage
[youtube] Moy1nWM3Zmg: Extracting video information
[download] Got server HTTP error. Retrying (attempt 1 of 10)...
[download] Got server HTTP error. Retrying (attempt 2 of 10)...
[download] Got server HTTP error. Retrying (attempt 3 of 10)...
[download] Got server HTTP error. Retrying (attempt 4 of 10)...
[download] Got server HTTP error. Retrying (attempt 5 of 10)...
[download] Got server HTTP error. Retrying (attempt 6 of 10)...
[download] Got server HTTP error. Retrying (attempt 7 of 10)...
[download] Got server HTTP error. Retrying (attempt 8 of 10)...
[download] Got server HTTP error. Retrying (attempt 9 of 10)...
[download] Got server HTTP error. Retrying (attempt 10 of 10)...
ERROR: giving up after 10 retries
Traceback (most recent call last):
  File "youtube_dl\FileDownloader.pyo", line 850, in _do_download
  File "urllib2.pyo", line 126, in urlopen
  File "urllib2.pyo", line 406, in open
  File "urllib2.pyo", line 519, in http_response
  File "urllib2.pyo", line 444, in error
  File "urllib2.pyo", line 378, in _call_chain
  File "urllib2.pyo", line 527, in http_error_default
HTTPError: HTTP Error 500: Internal Server Error

I don't know how to do .. help~
I'm try it in window7-64 now and will use it in KUbuntu.
Give me some valuable suggestions.
thanks~!

Obvious problem with your proxy.


build errors

CMake Error at CMakeLists.txt:79 (add_subdirectory):
add_subdirectory given source "third_party/googletest/googletest" which is
not an existing directory.
CMake Error at CMakeLists.txt:98 (add_subdirectory):
The source directory
/home/local/SoftwareDownloads/QLever/third_party/abseil-cpp

does not contain a CMakeLists.txt file.
CMake Error at CMakeLists.txt:121 (add_subdirectory):
The source directory
/home/local/SoftwareDownloads/QLever/third_party/stxxl

does not contain a CMakeLists.txt file.
CMake Error at CMakeLists.txt:136 (add_subdirectory):
The source directory
/home/local/SoftwareDownloads/QLever/third_party/re2

does not contain a CMakeLists.txt file.

Hi,
These modules are integrated as git submodules, so you need to pass the --recursive flag when performing git clone:
git clone --recursive https://github.com/ad-freiburg/QLever.git qlever

Alternatively, you ca go to your existing non-recursive clone and add the submodules via
git submodule init
 git submodule update

Please let us know, if this works for you.

That worked.   Silly me.


Added repeatCount as an editable property

Thought this might be useful

Okay, let me give this a test. Thanks, that was fast!

Only bug I found was I couldn't add HUGE or INFINITY into the InterfaceBuilder attribute area so I just ended up using 999

That's fine. I think resetAll needs it to make sure that it can chain and that it doesn't get properties from stored animations.

Made the change, good call :+1:


0.12.0版本本地执行示例教程报错

If you are trying graphscope inside docker, please specify enough shared memory argument like
docker run --rm -it --shm-size=8Gi ......

OK

Close as resolved.


Youtube live_chat download not working anymore

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Provide a description that is worded well enough to be understood
Youtube live_chat download doesn't work anymore. It seems like it stopped working sometime around Aug 24. It just downloads ~410KiB (first fragment?). Same thing happens if I try to download it live or afterwards.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '--ignore-config', '--skip-download', '--sub-langs', 'live_chat', '--write-sub', 'ZvjP1RcAHJQ']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.8.10 (CPython x86_64 64bit) - Linux-5.4.0-159-generic-x86_64-with-glibc2.29 (OpenSSL 1.1.1f  31 Mar 2020, glibc 2.31)
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2019.11.28, mutagen-1.46.0, secretstorage-2.3.1, sqlite3-2.6.0, websockets-9.1, xattr-0.9.9
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[youtube] Extracting URL: ZvjP1RcAHJQ
[youtube] ZvjP1RcAHJQ: Downloading webpage
WARNING: [youtube] unable to extract initial player response; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[youtube] ZvjP1RcAHJQ: Downloading ios player API JSON
[youtube] ZvjP1RcAHJQ: Downloading android player API JSON
[youtube] ZvjP1RcAHJQ: Downloading iframe API JS
[youtube] ZvjP1RcAHJQ: Downloading player 0a835141
[youtube] ZvjP1RcAHJQ: Downloading web player API JSON
[debug] Loading youtube-nsig.0a835141 from cache
[debug] [youtube] Decrypted nsig Zw6gTqd7psj1WLtFMM => MNVAiudIXbsqOA
[youtube] ZvjP1RcAHJQ: Downloading m3u8 information
WARNING: [youtube] unable to extract yt initial data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: [youtube] Incomplete data received in embedded initial data; re-fetching using API.
[youtube] ZvjP1RcAHJQ: Downloading initial data API JSON
[info] ZvjP1RcAHJQ: Downloading subtitles: live_chat
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] ZvjP1RcAHJQ: Downloading 1 format(s): 303+251
[info] Writing video subtitles to: life is half 2 ending [ZvjP1RcAHJQ].live_chat.json
[debug] Invoking youtube_live_chat downloader on "https://www.youtube.com/watch?v=ZvjP1RcAHJQ&bpctr=9999999999&has_verified=1"
[youtube_live_chat] Downloading live chat
[youtube_live_chat] Total fragments: unknown (live)
[download] Destination: life is half 2 ending [ZvjP1RcAHJQ].live_chat.json
[download]  410.11KiB at    1.10MiB/s (00:00:00) (frag 1)

Duplicate of #7594
yt-dlp --update-to nightly
or
https://github.com/yt-dlp/yt-dlp/wiki/Installation#pip-master
or
https://github.com/yt-dlp/yt-dlp-nightly-builds/releases/latest


Upgrades methods in Account V1

Summary

Related Issue: # (issue number)
Change Type

[ ] Bug Fix
[ ] Refactor
[ ] New Feature
[ ] Breaking Change
[ ] Documentation Update
[ ] Performance Improvement
[ ] Other

Checklist

[ ] My code follows this project's style guidelines
[ ] I've reviewed my own code
[ ] I've added comments for any hard-to-understand areas
[ ] I've updated the documentation if necessary
[ ] My changes generate no new warnings
[ ] I've added tests that prove my fix is effective or my feature works
[ ] All unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published

Additional Information

Branch Naming

branch feat/SMA-73-SA-upgrades also have same code, can delete that branch


Adding target framework for dotnet8

Description
Adding target framework for dotnet8
Testing
Added targets for dotnet 8 to test projects and executed
Types of changes

[X] Platform Upgrade (non-breaking)
[ ] Bug fix (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 change)

Checklist

[X] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[X] I have read the README document
[X] I have added tests to cover my changes
[X] All new and existing tests passed

License

[X] I confirm that this pull request can be released under the Apache 2 license

Hi @jon-armen,
Thank you for submitting the PR. Our team shall review it and get back to you soon.
Regards,
Chaitanya

Per was released as part of version 6.1.0. Thanks for the PR.


Edited the inner text as per the given instruction

Issue Number #7

@P2dwivedi I'm merging your request. Thanks for contributing.


Support to add properties in @Schema

For Maps it should be possible to set known properties to the generated documentation.
e.g.:
@Schema(description = "A map with known and custom properties.")
private Map<String, Object> map = new TreeMap<>();

returns an output like:
map:
  type: object
  additionalProperties:
    type: object
    description: A map with known and custom properties.
  description": A map with known and custom properties.

But required is an output with known properties like this:
map:
  type: object
  properties:
    knownValue1:
      type: string
    knownValue2:
      type: string
  additionalProperties:
    type: object
    description: A map with known and custom properties.
  description": A map with known and custom properties.

Therefore '@Schema' should get an additional variable that allows to override or add properties.

I have the almost exact opposite problem, of wanting to add custom properties to a schema with fixed fields, i.e. to add additionalProperties to an object with properties already correctly generated.
I think a solution to this would be to allow setting additionalProperties inside @Schema, as well as allowing adding properties there.

#4129 introduces support to define Schema properties via annotations. Please see this and this examples for usage scenarios.


JSPWIKI-1181 - Fix: Correct URL Generation for Pages with Special Characters

Summary
This pull request aims to resolve the issue described in JSPWIKI-1181. The problem was that the URLs generated for search results in quick navigation were incorrect when the page name contained a '/'.
Changes

Updated the cleanPageName function to correctly handle the '/' character.

Txs. Looks ok to me.

You're absolutely correct, JP. The version has been updated to
2.12.2-git-03.
Arturo
On Sun, Oct 1, 2023 at 8:09 PM Brushed @.***> wrote:

Txs. Looks ok to me.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/jspwiki/pull/310#issuecomment-1742156411, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABD3THQGUFDKRZIVYH5T6BTX5GWUHANCNFSM6AAAAAA5OJJ2AU
.
You are receiving this because you authored the thread.Message ID:
@.***>


onDeactivate event doesn't fire when another node is activated

If you activate a node and then activate a different node without multiselect, there is no onDeactivate event fired for the first node.  This basically forces you to maintain your own list of selected nodes instead of relying on the tree to handle it for you.
I'd submit a PR, but I want to wait until you decide what to do with #43.

The more I think about this, more I think this is at least partially a documentation issue.  I'm not sure what onDeactivate, onActivate and onActiveChanged actually mean.  Is this behaving the way you intended it to?
How I thought it would work
My initial assumption, just from looking at the names, is this:

onActivate: fires any time a node goes from not active to active
onDeactivate: fires any time a node goes from active to not active
onActiveChanged: fires any time a node changes active state in either direction

So a test case would go like this:
1. Node1 clicked

onActivate(Node1)
onActiveChanged(Node1, true)

2. Node2 clicked

onDeactivate(Node1)
onActiveChanged(Node1, false)
onActivate(Node2)
onActiveChanged(Node2, true)

3. Node1 shift-clicked (to multi select)

onActivate(Node1)
onActiveChanged(Node1, true)

4. Node3 clicked (back to single select)

onDeactivate(Node1)
onActiveChanged(Node1, false)
onDeactivate(Node2)
onActiveChanged(Node2, false)
onActivate(Node3)
onActiveChanged(Node3, true)

5. Node3 clicked (to deselect)

onActiveChanged(Node3, false)
onDeactivate(Node3)

How it currently works

onActivate: fires any time a node goes from not active to active
onDeactivate: fires when a node is explicitly clicked to deactivate it (not when clicking another node)
onActiveChanged: fires only when a node has active state toggled

The same test case:
1. Node1 clicked

onActiveChanged(Node1, true)
onActivate(Node1)

2. Node2 clicked

onActiveChanged(Node2, true)
onActivate(Node2)

3. Node1 shift-clicked (to multi select)

onActiveChanged(Node1, true)
onActivate(Node1)

4. Node3 clicked (back to single select)

onActiveChanged(Node3, true)
onActivate(Node3)

5. Node3 clicked (to deselect)

onActiveChanged(Node3, false)
onDeactivate(Node3)

Fixing....

Fixed in 1.2.2


removed ko.utils functions that are not part of minified knockout.js and added addOrRemoveItem() issue#4192

knockout/knockout.d.ts
to authors(@borisyankov @Igorbek @moonpyk). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

ko.utils.domData contains: clear, get and set functions.

I couldn't find get or set on ko.utils.domData, or removeNode on ko.utils.domNodeDisposal. Can you make sure you are referring to the right minified file. http://knockoutjs.com/downloads/knockout-3.3.0.js
.On a similar issue https://github.com/knockout/knockout/issues/670 SteveSanderson from knockout team has confirmed that they have exposed limited APIs and they don't want to expose such functions that are subject to change or can have backward compatibility issues in later versions. So, i guess it make sense to only include in typed file functions that are available in minified files. Any thoughts?

@SergioMorchon, any update on this? Can you approve the pull request if it looks good.?

It think that it could be provisionally ok, but we have to keep in mind this Knockout issue: https://github.com/knockout/knockout/issues/1858
The next release is almost here, and maybe with changes here.

Maybe we could ask the to the Knockout team which is the API or how we can get it.

We don't have an "official" API, but anything that's exported using ko.export... or is specifically made public using a quoted property name should be fine. It's also pretty safe to just examine the objects in the ko namespace to find which methods aren't minified.

This is something I also noticed as I made the switch from the debug knockout build to production build and noticed that certain methods defined in the TypeScript definition file for ko.utils no longer appeared to be exposed (specifically ko.utils.stringStartsWith is what I ran into).
The major addition to the TypeScript definition file for v3.4.0 that immediately stands out to me is the addition of the ko.onError callback.

Hello and thank you for your contribution!
Due to an excessively long queue of pull requests that have become stale, we are "declaring bankruptcy" and closing all PRs opened before May 1, 2016. If you'd still like to merge this code in, please open a new PR that has been merged and rebased with the master branch.
Going forward, we are committing to review or merge all PRs on a regular basis so this bankruptcy will not occur again. We apologize for the incovenience and hope you will continue to contribute to DefinitelyTyped in the future.
Thanks
The TypeScript and DefinitelyTyped teams


 Though memo was added to StartChildWorkflow here, it is not actually getting passed from the options in execute_workflow.
This PR just corrects that omission.

Thanks!


MailTo Encoding

Using special characters in mailto: strings causes Outlook (and possibly other email clients) to stop parsing the string.

I would gladly merge a PR for this, if any of you can make the necessary changes.

I forked a copy of the repo to my local and made changes to the specific snippet file (Mailto.md). This was my veryfirst pull request on GitHub like ever so hopefully I did it right and respected the terms of the CONTRIBUTING.md file. Please let me know if it worked fine and what I should do going forward regarding PRs.

@Dusch4593 I don't see the PR...

I think I got it. Check to see if it's there now.


Wait for pods to reach desired count after workloads get to active state.

@bmdepesa Can you review this PR?
Problem:
This fix is made to address random test case failures when the following validation method fails , since we assume the correct number of pods to be available as soon as the workload gets to active state.  Although this is true most of the time , we have seen random test failures around this check.
Fix:
We are now waiting for the desired number of pods to be made available for a maximum of 2 minutes, before we proceed to asserting the pod check.
def validate_workload(p_client, workload, type, ns_name, pod_count=1,
                          wait_for_cron_pods=60):
        workload = wait_for_wl_to_active(p_client, workload)
        assert workload.state == "active"
        # For cronjob, wait for the first pod to get created after
        # scheduled wait time
        if type == "cronJob":
            time.sleep(wait_for_cron_pods)
        pods = p_client.list_pod(workloadId=workload.id).data
>       assert len(pods) == pod_count
E       AssertionError

LGTM


SDK move from a .framework to a .xcframework is breaking the Mac Catalyst build

Recently Google updated their Google Mobile Ads SDK (as well as the Google User Messaging SDK in the previous release), to move from a .framework to a .xcframework.
Unfortunately while including the libraries in the pod.file as defined in the README for the script, the build (exclusion) for Mac Catalyst fails.
--
pod.file:
pod 'Google-Mobile-Ads-SDK', '~> 7.68.0' (or without version, note requires Firebase 7 if that's also in use)

def catalyst_unsupported_pods
  [
    "Google-Mobile-Ads-SDK", ...
  ]
end

--
Build errors
ld: warning: directory not found for option '-F/Users/.../Library/Developer/Xcode/DerivedData/something.com-blqqpxgyaaldrudoafizlmebhztt/Build/Products/Debug (test server)-maccatalyst/XCFrameworkIntermediates/GoogleMobileAds'
ld: warning: directory not found for option '-F/Users/.../Library/Developer/Xcode/DerivedData/something.com-blqqpxgyaaldrudoafizlmebhztt/Build/Products/Debug (test server)-maccatalyst/XCFrameworkIntermediates/UserMessagingPlatform'
ld: framework not found GoogleMobileAds
Script errors
warning: [CP] Unable to find matching .xcframework slice in '/Users/.../Development/ios-app/Pods/GoogleUserMessagingPlatform/Frameworks/Release/UserMessagingPlatform.xcframework UserMessagingPlatform framework ios-armv7_arm64_arm64e ios-i386_x86_64-simulator' for the current build architectures (x86_64).
warning: [CP] Unable to find matching .xcframework slice in '/Users/rmulder/Development/ios-app/Pods/Google-Mobile-Ads-SDK/Frameworks/GoogleMobileAdsFramework-Current/GoogleMobileAds.xcframework GoogleMobileAds framework ios-armv7_arm64 ios-i386_x86_64-simulator' for the current build architectures (x86_64).

Hey hi @rowanmulder , thanks for raising this, I'll give it a look shortly

@rowanmulder are you using the last version of the script? I have this Podfile:
target 'CollectionViewLayoutExample' do
  use_frameworks!

  pod "Google-Mobile-Ads-SDK"
end

def catalyst_unsupported_pods
  [
    "Google-Mobile-Ads-SDK"
  ]
end

with the Manifest.lock:
PODS:
  - Google-Mobile-Ads-SDK (7.68.0):
...

And it seems to work, Build and Archive.

Once I changed it to 7.64.0, it will fail on Google User Messaging Platform (a dependency from Google Mobile Ads), and if I change it to 7.68.0 both Google Mobile Ads and User Messaging Platform give the same errors.
I'm using CocoaPods 1.10 by the way, could that be related, I believe it also had some changes related to .xcframeworks integration?

@rowanmulder I just checked with cocoapods 1.10.0 and that might seem to be the issue. With 1.9.0 it seems to work.
I'll give it a look, thanks

@rowanmulder  I think I got it, will you give it a look please? Use the last version, I just committed

Updated the script, and I am able to build the latest releases of CocoaPods (1.10), Google Mobile Ads (7.68.0) and Firebase (7.0). Excellent, thanks!

Cheers! It was a simple change anyways.
Thanks a million for noticing and opening this issue, it was very helpful


OPTIONS method

Hi everyone,
Just wondering when this copy of scotty-0.9.1 will be available on hackage? The current version on hackage doesn't have support for "options"; but the github 0.9.1 version does.
I'm using cabal sandbox add-source in the meantime.
Thanks!

Apologies, I'll hopefully get back around to Scotty next week sometime. Trying to finish up with school. :-)

Ah np. Sorry, just noticed the "Away Please Read!" issue. No rush. I'm good with sandbox's "add-source".
Thanks for the response!


remove leftover assets

removes app/assets.py, which is leftover from the Flask Assets era and no longer needed.
removes /front/js/lib/typeahead.js which we aren't using.

:+1:


No demo

Here's a phrase for you:

No pics, no clicks

I've heard it enough times, I think you should hear it too. It's a real thing and you should really add at the very least a picture.

@LoganDark Agreed. If you still want to see it I configured it here:
https://stackblitz.com/edit/angular-k8zphy

A little late, but thanks! Could you add it to the readme?

I'm actually not the author.  If I get time, I'll submit a pull request for it.

A pull request is what I meant 😛
Anyway, thanks for the demo!

@LoganDark
I'll add something when I have time. But feel free to create a PR, this is the point of open source. You are not supposed to use it and complain if it doesn't work but try to make it better ;)

Alternatively you can clone the repo, then npm install & npm start you'll see the example running.


Setting the normal value in the property skin, breaks the skin

So this works:
['property'] = {
	['label normal'] = '#241414',
	['label hover'] = '#241414',
	['label active'] = '#241414',
},

and this does not:
['property'] = {
	['normal'] = '#241414',
	['label normal'] = '#241414',
	['label hover'] = '#241414',
	['label active'] = '#241414',
},

The result is a half broken skin, with some f the default values set and others taken from the skin

Does this do what you were expecting?
['property'] = {
	['normal'] = '#241414',
	['label normal'] = '#241414',
	['label hover'] = '#241414',
	['label active'] = '#241414',
	['edit'] = {
		['normal'] = '#241414'
	},
	['inc button'] = {
		['normal'] = '#241414'
	},
	['dec button'] = {
		['normal'] = '#241414'
	}
}

Well this doesn't work either. it seems okay, if I don't set the normal value

I'm not sure what you're seeing or expecting. Could you upload some screenshots of the problem - one with normal and one without?

Without:

With:

My Löve fileas a zip if you want to try it yourself.
http://uploaded.net/file/r36jiihx

Thanks for the catch, I believe the issue should be solved with the latest commit. Here are some tentative binaries for you to test. Let me know whether they fix the problem.
love-nuklear-1.1.3-linux32.zip
love-nuklear-1.1.3-linux64.zip
love-nuklear-1.1.3-win32.zip
love-nuklear-1.1.3-win64.zip

That fixes it 👍


Update plugins because of traffic light OFF

All IDE plugins must be able to handle the OFF value introduced with #809 .
Remark: With #809 the java client library is automatically up-to-date but the plugin must use the new library

Link here the links for the corresponding plugin issues
Ensure all plugins are changed and new versions available on marketplace

[ ] Eclipse
[ ] IntelliJ
[ ] VSCodium/Theia (code changes necessary)

sub issue for #808


Question about Windows build, where is your "wx-config.exe"?

Hi, I just see this:
		<Compiler>
			<Add option="-std=c++11" />
			<Add option="`wx-config --cflags`" />
		</Compiler>
		<Linker>
			<Add option="`wx-config --libs`" />
		</Linker>

Which looks like you need a "wx-config.exe" under Windows?
If that is correct, how does the "wx-config.exe" know about the wxWidgets library's location?
For me, under Windows + Code::Blocks, I use this kind of setting:
asmwarrior/cb_projects_for_wxWidgets: Code::Blocks projects for building wxWidgets sample code
With such kinds of setting, I can either build a wx project against the wx library from msys2 or my self built wx library.

Thank you for posting that @asmwarrior!
Currently, I put wx-config.exe from Codelite in %PATH%, works well also with custom-built wx installations.
Apparently there is a spinoff at https://github.com/eranif/wx-config-msys2, probably it can be installed from MSYS2 directly

Currently, I put wx-config.exe from Codelite in %PATH%, works well also with custom-built wx installations.

Good to hear. But I'm not sure why the above steps work. How does a wx-config.exe find a valid wx installation?

I have the default path and build set in environment variables %WXWIN% and %WXCFG%, but also the --prefix option should work


Default user and password in v0.8.0 on Raspberry 3

RancherOS Version: (ros os version)
v0.8.0
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
raspberry pi 3
As stated in http://docs.rancher.com/os/running-rancheros/server/raspberry-pi/ there is no possibility to pass in a cloud-config and therefor i can't set a SSH pubkey.
 What user and password should I use, how can I set it, or is there any chance to autologin?

the rpi version stil uses rancher and rancher :/

Are you sure, even v0.8? This was one of my first combinations I tried with no success. I couldn't find anything about the credentials in the docs and also tried the default from http://blog.hypriot.com/faq/

I'm running into this issue using ami: rancheros-v0.8.0-hvm-1-3e977279-1fad-4ebf-bb68-5d6e0d2d0b37-ami-4600ce50.4 (ami-9fc3e6fa)
rancher as user and blank password doesn't work
rancher as user and rancher as password also doesn't work
I'm specifying the public key equivalent to the one that was used when the EC2 instance was launched.

Is there anything new on this topic?
Should I update to v0.8.1 as soon as it gets released for rpi?


Output Current Time

Love the component!
It'd be nice to have access to the current time, perhaps in milliseconds?
I can help contribute.... but I need some assistance getting started. I've set up and got Livesplit able to run via VS2019, but when I try to add your component as a sub-component via the solution manager, it doesn't appear after running, in the Edit Layout > Add component menu.

https://github.com/LiveSplit/LiveSplit.Server
This project might be a better fit when trying to get the current time data


Question on React-Native usage

Hi,
Thank you so much for such an amazing work! Need clarification on using React-Native for mobile apps!
Is this library compatible with React-Native?
Thanks
Jay

It's pure JS so it can work with React-Native, Angular, Vue...etc.
For instance, I've used it with AngularJS and Electron together.

Thank you @charles-merlinguides for responding so quickly 👍
I was wondering what would be storage location. I think produced whole indexed content as a JSON which I can save it on Mobile or reload for offline.

Thanks

So, storage isn't available out of the box as everything is in memory. To get around this you can basically copy the Object and save it. What I do is I store it as a .JSON object on the users machine, however you could store it any way you want really.


Question: Can speechbrain accept mfcc streams?

Prob a silly question as likely what are the parameters of MFCC for speechbrain so I can send a MFCC stream.
I am working on a multiroom network mic solution that uses broadcast KW hit to select best stream and my KWS has 2 models one for KW & another for VAD/Diarisation where most of the load is the creation of the MFCC pipeline of a 20ms stream of S16_LE.
https://ai.googleblog.com/2021/02/lyra-new-very-low-bitrate-codec-for.html
The Lyra codec was a bit mweh to me as conversion from and back to audio is quite heavy in comparison to feeding a network ASR with an already computed MFCC stream.
Is that possible or could it be?

Yes, you can save store your mfccs in pkl files and read MFCCs directly. You can figure out how to do it, if you take a look into one of the existing recipes. The difference is that instead of reading audio files you have to read features. We understand your special need in your project but let me take this opportunity to remark anyway why in speechbrain we prefer on-the-fly feature computation:

Feature computation is super fast (especially on GPUs)
You can easily combine on-the-fly feature computation with on-the-fly data augmentation
The pipeline is cleaner and do not require annoying preliminary steps such as dumping feature on disk.
We can backpropagate the gradient through the features and make them learnable
Conceptually, we just see feature extraction as an early layer of a deep architectuere


Duplicate and outdated location data is posted

Your Environment

Plugin version:

"@transistorsoft/capacitor-background-fetch": "^1.0.2",
"@transistorsoft/capacitor-background-geolocation": "^4.9.4",

Platform: iOS
Device OS version: iOS 16.2
Device manufacturer / model: iPhone XR
XCode version:
Capacitor info (npx cap doctor)

@capacitor/cli: 4.6.1
@capacitor/core: 4.6.1
@capacitor/ios: 4.6.1
@capacitor/android: 4.6.1

Plugin config provided to #ready:

	  autoSync: true,
      // Geolocation Config
      locationAuthorizationRequest: "Always",
      useSignificantChangesOnly: true,
      desiredAccuracy: BackgroundGeolocation.DESIRED_ACCURACY_HIGH,
      logLevel: BackgroundGeolocation.LOG_LEVEL_ERROR,
      stopOnTerminate: false, // Allow the background-service to continue tracking when user closes the app.
      startOnBoot: true, // Auto start tracking when device is powered-up.
      disableStopDetection: true,
      pausesLocationUpdatesAutomatically: false,

Expected Behavior
We expect to receive latest location data when our server receives location update from the plugin.
Actual Behavior
The location data posted to the server is duplicate and outdated. We got several location updates during the trip while device is moving, which is nice and expected. But all the location data sent is with the same UUID during the trip and the timestamp is also several hours or longer time ago.
Steps to Reproduce
N/A
Context
We want to get rough location update when the device is moving.
Debug logs

Logs
N/A

I checked the log and I found the new location is recorded as expected. It turns out that the duplicated and outdated data is caused by none-200 response from our server because how the plugin handles http failures:

If your server does not return a 20x response (eg: 200, 201, 204), the SDK will UNLOCK that record. Another attempt to upload will be made in the future (until maxDaysToPersist) when:
When another location is recorded.
Application pause / resume events.
Application boot.
onHeartbeat events.
onConnectivityChange events.
[iOS] Background fetch events.

Thank you very much for your great work and support.


some schematic snapshots SVG's aren't rendering the symbols correctly.

start with the battery test.

/bounty $10

fixed in #314


Bug 1921139: remove override for fast cert rotation for release

This switches back to normal 30 days rotation period (from 6h during development).

/lgtm
/approve

/retest

/retest

/override ci/prow/e2e-upgrade
Adding override as we don't want to ship with short cert rotation by accident. The failures in the upgrade are unrelated to this change.


.warn can be used by anyone with ban perms to ban anybody from the server, even admins.

The way the bot is written allows anybody with ban permission to use .warn to ban anyone from a server, regardless of their position in the role order. This means literallly a low level moderator can ban a high ranking admin, as long as the admin isnt the owner.
For example, a moderator could wait until the server owner is away and then just warn ban anybody else with power to do anything and then they could do serious damage to a server

Fixed in 2.0


Popup Category

Right now, we were putting popups in #Alerts, but I don't think it counts as an alert.
Popup projects:

KLCPopup - A simple and flexible class for presenting custom views as a popup in iOS.
MMPopupView - Pop-up based view(e.g. alert sheet), can easily customize.
STPopup - STPopup provides a UINavigationController in popup style, for both iPhone and iPad.
NMPopUpView - Simple iOS class for showing nice popup windows. Swift and Objective-C versions available. :large_orange_diamond:
CNPPopupController - Simple and versatile class for presenting a custom popup in a variety of fashions. It includes a many options for controlling how your popup appears and behaves.

@dkhamsing what do you think?


Make it possible to listen to UNIX socket file

Offering services to the world by listening some TCP high port is not particularly secure, since a hostile unprivileged user can replace the server with a hostile server that steals user's passwords shall the actual server crash.
While this can be prevented with SELinux, it is quite a heavy-weight solution. My proposal is to allow listening of UNIX sockets. Then the reverse proxy or xinetd could be configured to redirect connections to that socket.

Hey @ilmaisin, that would be great! Do you think you could help us with that by opening a PR with your proposal?
Thanks!

My development skills are unfortunately quite rudimentary, but maybe I can try and take a look at it.

Sounds good! And if you have any question, feel free to post them here and/or to open a PR early, even with non-functional or pseudo-code, and we can work with you in order to make that happen 😄.

I made a tentative request here: https://github.com/thelounge/lounge/pull/691
Things that need yet to be done:

Modify the stdio output to tell when we listen to socket
Add a code to clean up old socket if the server has exited non-gracefully due to power outage or service crash
Possibly add a command line option for socket listening


ESLint couldn't find the config "plugin:react/jsx-runtime" to extend from.

I created a NextJS (TS) application and decided to set up a linter. It connected correctly and displayed errors, but when fixing errors, I noticed the react-in-jsx-scope rule, which is not relevant in NextJS realities. Using the ESLint documentation, I added "plugin: react / jsx-runtime" to extends and got this error

eslintrc.json:
{
  "extends": [
    "airbnb",
    "airbnb-typescript",
    "plugin:react/jsx-runtime"
  ],
  "parserOptions": {
    "project": "./tsconfig.json"
  }
}

package.json dependencies:

The only thing that worked was the alternative with disabling this rule:
{
  "extends": [
    "airbnb",
    "airbnb-typescript"
  ],
  "rules": {
    "react/react-in-jsx-scope": "off"
  },
  "parserOptions": {
    "project": "./tsconfig.json"
  }
}

Updating to the latest version solved the problem, thanks


removeFile() Usage

This PR:

Fixes removeFile failure on Windows in slic_macros.cpp unit test by closing the files associated with ifstream and the ofstream  member variables of LogStream instances.
Checks the result of removeFile in unit tests to verify file has been deleted

Closes issue #1415

@bmhan12 , thank you for your attention to detail on this.

@bmhan12 -- could you please update the documentation for removeFile about it not working on Windows when there are open file handles?


Great work

Hi. I've been trying to really dig into this protocol. I've read your thesis and I just stumbled upon this: https://tools.ietf.org/id/draft-pouwelse-trustchain-00.html . I noticed you are making a few changes over on cothority too... I assume to test a CHECO implementation?
Are you doing any work to convert this protocol into golang, then? Is there a good place to discuss trustchain and checo?
I have a couple of (probably naive) ideas on how one could simplify the protocol given the addition of a BFT consensus that I'd love to discuss.
I'm also wondering what the licensing is? Tribler seems GNU, this is MIT and the link above is BSD. any ideas for licensing moving forward?

Hi, thank you for your interest. At the moment, I'm not actively involved with Checo or Tribler. But @synctext and his team are. Also, I can't comment on the license situation.
Regardless, we are always happy to hear about new ideas, especially simplifications.

Thanks for the reply! Where’s the right place to discuss?

On Feb 3, 2018, at 22:03, kc1212<EMAIL_ADDRESS>wrote:
Hi, thank you for your interest. At the moment, I'm not actively involved with Checo or Tribler. But @synctext and his team are. Also, I can't comment on the license situation.
Regardless, we are always happy to hear about new ideas, especially simplifications.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

You're welcome to discuss it here. The Tribler-related projects are all discussed in the open.


[Install] Use custom prompt for sudo

This will show the information message only when sudo requires to insert the password. Is that sufficient here and what was intended?

Coverage remained the same at 61.35% when pulling 03281a1a5d50b7404344448427d5c481f9a1c14f on mrackwitz:patch-1 into 8942ad3f48d6c01d68c41ff5b1fa8c3823b47d96 on neonichu:master.

This is great :thumbsup:


fix: ensure upstream_protocol is passed to the Worker

What this PR solves:
In wrangler dev it is possible to set the upstream_protocol, which is the protocol under which the User Worker believes it has been requested, as recorded in the request.url that can be used for forwarding on requests to the origin.
This setting defaults to https. Previously, it was not being passed to wrangler dev in local mode. Instead it was always set to http.
Note that setting upstream_protocol to http is not supported in wrangler dev remote mode, which is the case since Wrangler v2.0.
Fixes #4539
How to test:
Using the build of Wrangler from this PR run wrangler dev on a simple worker as follows:
export default {
  fetch(request: Request) {
    return new Response(request.url);
  }
}

Note that the response will be a URL that has https as its protocol (e.g. https://localhost:8787).
Then run the same worker with wrangler dev --upstream-protocol=http. Now the response will have a URL that has http as the protocol.
Author has addressed the following:

Tests

[x] Included
[ ] Not necessary because:

Changeset (Changeset guidelines)

[x] Included
[ ] Not necessary because:

Associated docs

[ ] Issue(s)/PR(s):
[x] Not necessary because: bug fix - no docs changes needed.

It's surprising to me that it defaults to https. In my opinion, request.url should be the actual url to reach the user worker by default. If there are config overrides to opt-in to change request.url, fine. But by default, it shouldn't be different from the eyeball.

It's surprising to me that it defaults to https. In my opinion, request.url should be the actual url to reach the user worker by default. If there are config overrides to opt-in to change request.url, fine. But by default, it shouldn't be different from the eyeball.

Yes, ideally the URL the user hits and the upstream URL should be equivalent by default, but this is the current defaults, which I think are a legacy of the fact that it is easier for the remote preview worker to be HTTPS and it is easier for the local proxy server to be HTTP.
I think it is a breaking change to swap that default which has been like that since Wrangler 1.

It's surprising to me that it defaults to https. In my opinion, request.url should be the actual url to reach the user worker by default. If there are config overrides to opt-in to change request.url, fine. But by default, it shouldn't be different from the eyeball.

Yes, ideally the URL the user hits and the upstream URL should be equivalent by default, but this is the current defaults, which I think are a legacy of the fact that it is easier for the remote preview worker to be HTTPS and it is easier for the local proxy server to be HTTP.
I think it is a breaking change to swap that default which has been like that since Wrangler 1.

Actually, thinking about this more... Given that the current behaviour was not to send this through at all in local mode, then it is probably not a breaking change to change the default for local mode. For remote mode, https is the only possible value anyway, so that would have to stay the same.

Actually, thinking about this more... Given that the current behaviour was not to send this through at all in local mode, then it is probably not a breaking change to change the default for local mode. For remote mode, https is the only possible value anyway, so that would have to stay the same.

Yep that was my thinking too


Save data parsing error 20518

Hello again! There is an error occured when I cleared the first chapter and tried to edit items (normal tickets).
Here what it says:
Error: An error has occurred while parsing your save data (address = 20518):
Traceback (most recent call last):
File "C:\Users\Alexander\AppData\Roaming\Python\Python310\site-packages\BCSFE_Python\parse_save.py", line 1457, in start_parse
save_stats = parse_save(save_data, country_code)
File "C:\Users\Alexander\AppData\Roaming\Python\Python310\site-packages\BCSFE_Python\parse_save.py", line 1608, in parse_save
current_sel = get_event_stages_current()
File "C:\Users\Alexander\AppData\Roaming\Python\Python310\site-packages\BCSFE_Python\parse_save.py", line 223, in get_event_stages_current
clear_progress = list(helper.chunks(clear_progress, stars_per_sub_chapter))
File "C:\Users\Alexander\AppData\Roaming\Python\Python310\site-packages\BCSFE_Python\helper.py", line 161, in chunks
for i in range(0, len(lst), chunk_len):
ValueError: range() arg 3 must not be zero
Save_data.zip

I don't seem to get the error with the provided save file, are you using the latest editor version?

I don't seem to get the error with the provided save file, are you using the latest editor version?

Yes

So if you use the saves given in the zip file, they don't work?

Still, I don't have the issue, can you try to run this command py -m pip install -U battle-cats-save-editor and retry

Sorry for taking me too long to respond, but I think I solved the problem. Though it's very weird and funny, as if the program is sorta "religious" and feels offended: the error happens ONLY when the input for normal tickets is 666. I spontaneously and lazily entered this number in order to have many tickets without aтн second thoughts.
It is still happening with the last update too.

Oh, I just found out why, it's because the current number of cats in the game is 666 atm, so it happened to break one piece of code, I'll fix for the next release

This should be fixed in the latest release


ReadOnlySpan results in "Cannot create boxed ByRef-like values." (System.InvalidProgramException)

TracingOn with a method like the one below throws an exception of type System.InvalidProgramException saying "Cannot create boxed ByRef-like values.".
Example method to duplicate
public void Test3(ReadOnlySpan path)
{
    }

Happens in

ReadOnlySpan is a ref struct which is a new feature in C#7.2. I have to change the rewriter to support it. As you cannot box a ref struct I think the only viable option is to skip this parameter from logging.

For a char span it can be converted to a string and from a string. Other
spans are different.
On Tue, 30 Jul. 2019, 1:10 am Csaba Nemes<EMAIL_ADDRESS>wrote:

ReadOnlySpan is a ref struct which is a new feature in C#7.2. I have to
change the rewriter to support it. As you cannot box a ref struct I think
the only viable option is to skip this parameter from logging.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/csnemes/tracer/issues/83?email_source=notifications&email_token=AADB44MVM4G5NZMJ37AGA2DQB4B5NA5CNFSM4H4KE4NKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3BARWA#issuecomment-516032728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADB44KM4GURCALJNVMYELLQB4B5NANCNFSM4H4KE4NA
.

The problem is that the log signature has to be very general, thus the method parameters are passed as an array of object. ref structs like ReadOnlySpan cannot be boxed and added to an object array.


Command to package electron application into a single exe (without configuration file)

Hi,
Currently using @electron-forge/cli - 6.0.0-beta.22
I will like to execute a command to package up my electron application into one single exe (with whatever dependencies and runtime inside this exe).
May I know if there is any reference out there for me to use to achieve my goal? Without involving any configuration file.
I think the command is electron-forge make, but I keep getting some errors, such as "Failed to locate makeable Electron application".
Thanks you

My understanding is that a single-binary of an Electron app is a limitation of Chromium, not Electron Forge or Electron. (@MarshallOfSound please correct me if I'm mistaken). You can create a single .exe via the Squirrel installer to set up and install the app, though. I'm sure there are wrappers/bundlers around that would generate a single .exe for you to run, but it would have to temporarily extract them someplace in order to execute the app.

My understanding is that a single-binary of an Electron app is a limitation of Chromium, not Electron Forge or Electron. (@MarshallOfSound please correct me if I'm mistaken). You can create a single .exe via the Squirrel installer to set up and install the app, though. I'm sure there are wrappers/bundlers around that would generate a single .exe for you to run, but it would have to temporarily extract them someplace in order to execute the app.

Hi. Thanks for the resply. But i still dont quite get it. So does electron-forge provide any feature to generate a single .exe from the electron app?

What Electron Forge can do: create a single Setup EXE file for the purposes of installing onto a user's system.
What Electron Forge cannot do (due to Chromium limitations): Once installed, only have one EXE to run the application.

We've built a Maker plugin that builds a portable Windows .exe file using electron-builder's portable target (NSIS) under the hood:
https://github.com/rabbit-hole-syndrome/electron-forge-maker-portable


Add exclude option

Allow excluding files/paths in the following ways:

Add a new config property ignore which accepts file paths/glob regexes.
Detect and ignore via a comment in the file: // import-conductor-ignore

Hi, Thanks for this awesome library, I already integrated it with File watcher of my IDE Webstorm which runs it on save.
although I faced a problem which this enhancement will fix it.
in a Typescript project I use TypeDi for dependency injection and I need to reflect-metadata be the first line but after running the plugin it will be pushed to third line.
import 'reflect-metadata';

I'll be eagerly waiting for this ☺

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


substack style formatting options

@AbhayVAshokan Please take a look at https://github.com/bigbinary/neeto-quiz-web/issues/1440#issuecomment-1270388945
I asked my kids to type on neeto KB and both of didn't know how to bolden the header. So the current solution needs a revisit. We need to migrate towards substack style where the formatting options are always there. However as I posted at https://github.com/bigbinary/neeto-quiz-web/issues/1440#issuecomment-1267733168 sometimes we don't want rich text and sometimes we want the screen to look clean.
Even if options are not always there there needs to be a hint that if you select something then something can happen. Most of the folks have used google doc or word and they know that if you select something then items can be made Bold, Italic etc. However in the absence of any visual clue they felt that it's a text area.
Lemme know your thoughts on this.

@neerajdotname please have a look: https://vimeo.com/758068794/dd2a93073f
Figma: https://www.figma.com/file/Osl7GBbBk0iwVMc57366nZ/neetoKB
cc: @karthiknmenon please share your thoughts as well.

@AbhayVAshokan https://www.loom.com/share/43637a93d2c34f3e9fd47951cbf2ae35

@AbhayVAshokan This is looking good. We have exactly the same issue in neetoCourse too.

@AbhayVAshokan Let's move ahed with this.
The issue is that soliciting more feedback from others who all belong to tech world is not going to give a better perspective. At this time I'm fully convinced that a non tech person wouldn't know what to do with blank canvas as was the case with my kids.
This issue https://github.com/bigbinary/neeto-engineering-ui-ux/issues/19 is another example of a case where tech folks will approve of something which outside the tech world is a big no-no.
With that comment I'll say let's move ahead with it.

Agreed. I'll update the POC to a more elegant UI and assign it to @goutham-subramanyam for review.
@karthiknmenon, @vysaghsekharbb please share your thoughts if it needs further improvements.

@AbhayVAshokan
https://www.loom.com/share/f723f9a4922748b291b3f541c250cf6e
cc @neerajdotname, @karthiknmenon, @praveen-murali-ind

This design looks great. Looking from the technical point of view, the Menu must be extracted to a separate component and placed at the top of the page. The title is maintained in a separate state in an Input component. I shall scope out these issues and start the work.

The sticky header is looking great. 😍

Extracted the issue to #405


fix mislined entry box and label text

In debug_frame the label and text entry boxes appeared to be misaligned

Oh, my bad! Thanks for fixing this!


Modules not extracted to lazy loaded chunk if coming via an export of an index file

Is this a bug report?
Yes
Steps to Reproduce
Set up a project similar to https://github.com/penx/cra-code-splitting-exports
Specifically, say you have an index file for your components:
import Component1 from './component-one'
import Component2 from './component-two'

export {Component1, Component2};

Then in your application, you import Component1 in the main App and import Component2 via React.lazy/import()
Expected Behavior
I would expect the code from /src/components/component-two.js to be included in the lazy loaded chunk.
Actual Behavior
The code from /src/components/component-two.js is included on initial page load because it comes from /src/components/index.js so is included in the same bundle as src/components/component-one.js.
In the case of local modules, you can import from the module directly instead of going via an index file. Unfortunately you don't always have this option if you want to lazy load part of a dependency:
i.e.
if you have the following on app load:
import { someUtil } from 'my-library'
And the following is in a component that is lazy loaded:
import { aMuchLargerUtil } from 'my-library'
Then I would not want aMuchLargerUtil to be included in the initial bundle.
Reproducible demo
git clone git@github.com:penx/cra-code-splitting-exports.git
cd cra-code-splitting-exports
yarn && yarn build
serve -s build

When you import ./components in App.js, webpack loads the whole module and evaluates it.
The only "issue" I can see here is why webpack can't tree-shake ComponentTwo, since it isn't used in main chunk.

I found a "documented hack".
Create a package.json file in ./components directory with the following contents:
{
  "sideEffects": false
}

it worked! Thanks <3

Demo of it working, including for CSS modules, here https://penx.github.io/govuk-frontend-react-example/


Deploy Catalyst Voices UI with "coming soon" landing page  [M2]

As a user, I want to be able to visit a simple landing page for Catalyst Voices, that has been deployed to a testnet URL, so that I can see progress on delivery for Catalyst Voices & signoff on Milestone 2.
Design Tasks
Simplify the proposal editor prototype originally planned for M2, so that it only includes the landing page. Remove any references to proposals. Add "Coming Soon!" text.

[x] Extract color gradient from splash screen OR create new simplified one
[x] Remove "proposal list" and "create proposal"
[x] Add "Coming Soon" text

Dev Tasks
Implement the new splash screen designs in the Catalyst Voices flutter application.

[ ] Implement landing screen in flutter app

Deploy Tasks
Deploy the updated Catalyst Voices flutter application to a domain like "voices.testnet.projectcatalyst.io"

[ ] Deploy flutter app

Thanks @BojanKocijan ! The animation is a nice touch.
@minikin any concerns with translating this to flutter?


Update RoBERTa vocabulary files

import torch
roberta = torch.hub.load('pytorch/fairseq', 'roberta.base')
roberta.eval()

tokens = roberta.encode('Hello world!')
print(tokens)  # [    0, 31414,   232,   328,     2]

import texar.torch as tx
tokenizer = tx.data.RoBERTaTokenizer(pretrained_model_name='roberta-base')

input_ids, _ = tokenizer.encode_text('Hello world!', max_seq_length=5)
print(input_ids)  # [0, 31414, 232, 328, 2]

resolve #246

input_ids, _ = tokenizer.encode_text('Hello world!', max_seq_length=5)
is max_seq_length necessary here? what's the result in this case without setting max_seq_length?

It will be

input_ids, _ = tokenizer.encode_text('Hello world!', max_seq_length=5)
is max_seq_length necessary here? what's the result in this case without setting max_seq_length?

max_seq_length is used here just to show the same result between ours and theirs. If max_seq_length is not specified, it will be [0, 31414, 232, 328, 2, 0, 0, 0, ..., 0](zero-pad up to the maximum sequence length).

input_ids, _ = tokenizer.encode_text('Hello world!', max_seq_length=5)
is max_seq_length necessary here? what's the result in this case without setting max_seq_length?

max_seq_length is used here just to show the same result between ours and theirs. If max_seq_length is not specified, it will be [0, 31414, 232, 328, 2, 0, 0, 0, ..., 0](zero-pad up to the maximum sequence length).

If the user doesn't want padding, it's difficult for them to know the correct seq length just as in this case ('Hello world!' needs seq_length 5).. Can we add an argument (or allow a special value of max_seq_length) so that the user can get the encoded text without padding (and without need to specify max-seq-length explicitly)?

input_ids, _ = tokenizer.encode_text('Hello world!', max_seq_length=5)
is max_seq_length necessary here? what's the result in this case without setting max_seq_length?

max_seq_length is used here just to show the same result between ours and theirs. If max_seq_length is not specified, it will be [0, 31414, 232, 328, 2, 0, 0, 0, ..., 0](zero-pad up to the maximum sequence length).

If the user doesn't want padding, it's difficult for them to know the correct seq length just as in this case ('Hello world!' needs seq_length 5).. Can we add an argument (or allow a special value of max_seq_length) so that the user can get the encoded text without padding (and without need to specify max-seq-length explicitly)?

There are two returns, input_ids and input_mask, for the function encode_text. If the user can compute the correct sequence length by aceessing input_mask. In this case, input_mask is [1, 1, 1, 1, 1, 0, 0, ..., 0].

pls try to pass the test and merge asap

I tried to disable codecov/patch because some of our tests (pre-trained tests) are done locally. codecov/patch is also reported to be not very reliable. Some other projects have met such issues (https://github.com/argoproj/argo-cd/pull/1926).  I think codecov/project is good enough to check the code quality for now. I tried many ways to disable codecov/patch but failed. It seems that someone also met a similar problem (https://community.codecov.io/t/cannot-disable-codecov-patch-check/682/4). Do you think it is reasonable to merge PRs regardless of the status of codecov/patch? @ZhitingHu

As long as the build is "passing" we're good for now

https://github.com/pytorch/fairseq/issues/1432


content-type issues upon transfer to S3

When I manually upload the files (html, JS and CSS) using the S3 "upload" option (in Actions), the correct content-types are applied i.e. "text/html" for HTML files.
When the same files are deployed to S3 from github - as a result of a PR merge - their Content-Type are being set to "application/octet-stream", which means the files to not render in the browser. Have you seen this behaviour? I'm using the built JAR file in the releases tab.

Hi Adrian,
yes, you are right, the Content-Type was not set by the uploader. The latest release fixes this.
The Content-Type will be set, after a upload operation, which means, after the file has changed. All existing files must be changed manually or you remove the affected items from the bucket and start the sync again (you can do this without new commit when you go to the integration panel of your GitHub project, select the SNS integration and trigger a test event with the upper right button).

Yes, content-type is correct with uploads now when using v1.1 - thanks for resolving.


build: Pin hatchling version

Pin hatchling version. This version should be kept up-to-date: my working assumption is that Dependabot will handle it.
I think we want to do something like this: IMO our build artifacts should not change if hatchling releases a new version (which is what happened right after we released 1.1.0: now verify_release will say the artifacts do not match because WHEEL file includes the hatchling version number).
Whether this exact change is the correct one, I'm not sure. Does dependabot update build-system.requires? I'm hoping so but I don't know :shrug:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.326%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1187

Relevant Lines:
1203

💛  - Coveralls

Not yet https://github.com/dependabot/dependabot-core/issues/3290


[Development] Mirror system

Templates won't be stored on git repository, to make sure we don't infringe copyright, what I'm proposing instead can be described in a couple of steps:

Upload templates as archives to 3rd party file storage

Add direct download links to text file for specific template, we can call it "template TXT"

Upload those template TXTs  to 3rd party file storage

Add links to template TXTs to "mirrorlist" file with md5 and sha256 hashes included per archive of specific version, e. g. "Github Template 0.3"

Implement code to automatically:

Fetch mirrorlists
Parse them for links
Fetch template TXTs
Parse them for links
Download latest version of template
Verify it with hash
If valid - move from temporary folder to main templates storage
If invalid - delete and cleanup, throw an error and make sure to write it to debug logs

As per templates versioning, we can use creation date, e.g. "GitHub 23/08/2021"

As per templates versioning, we can be using creating date, e.g. "Github 23/08/2021"

@sTiKyt Hum

Can we chat privately on telegram

how
On Sep 18, 2021, at 03:10, im2gm @.***> wrote:

Can we chat privately on telegram
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

I'm closing all comments due to offtopic

Reiterated upon this concept, closing the issue accordingly.


Checkup config: read and parse checkup's config from ConfigMap object

This add the ability to read and parse checkup's configuration from a ConfigMap object.

Change: Fixed a problem discovered by the linter.

Change: Changed a commit message.

Changes: Optimized the unit test code.

Change: Fixed linter problems.

Changes: Fixed code review comments + linter errors.

Change: Moved types form loader_test.go to outside of Test*() functions.
Also changed their field names according to an offline code-review comment by @ormergi.

Change: Changed Config.Params to Config.EnvVars of type []corev1.EnvVar.

Change: Added config.ErrRoleNameIsIllegal.

Change: Fixed comments from code review in production code.

Change: Fixed comments from code review in unit test code.

Change: Added a test case - fail when ConfigMap object doesn't exists.

Change: Moved low level K8s client usage to a dedicated function.
Try to get ClusterRole and Role objects, only after successful parsing of their names.

Change: Extracted the parsing logic from config.Loader to a new entity.

Change: Fixed code review comments.

Change: Moved ConfigMap namespace and name back to loader.go.

Change: Rebase and integration with PR #9.

Change: Added a configmap package to unify actions against a ConfigMap object.

Change: Put the logic of splitRoleName() inline.

Change: Inlined testLoaderFailureWhenConfigMapIsMissing()

Change: Fixed commit messages.

Change: Changed commit messages.

@orelmisan
Thanks for the changes it look much better!
I have some more comments, most of them are suggestions to simplify the code even more,
Please take a look and see if it make sense to do.

Change: Answered code review comments.

Change: Fixed a small bug in newFakeClientWithObjects()

Change: Fixed a review comment.


Bump htmx version to 1.9.12

There are no breaking changes. I tested and everything works as before.

BTW, You don't need to bump single dependencies unless there is reason to, I do them all at once.

BTW, You don't need to bump single dependencies unless there is reason to, I do them all at once.

I monitor a few like htmx and relative-time-element with special attention

Sure, it's good to test as I don't always test all specifically. htmx is probably not mature enough to upgrade blindly :)


Merge upstream

@at-wat PTAL

@at-wat PTAL

@DaikiMaekawa using squash merge is not good for this PR.

@DaikiMaekawa using squash merge is not good for this PR.

@at-wat Should I revert it back?

@at-wat Should I revert it back?

I'll force push to fix it.

I'll force push to fix it.

fixed by https://github.com/alpine-ros/rosdistro/pull/60

fixed by https://github.com/alpine-ros/rosdistro/pull/60


Hackathon: A First Timer Experience

Hackathon: A First-Timer Experience
Description
Detailed look into arguably one of the wildest roller-coaster rides of my entire student life: My first-time experience in joining hackathon!

test!

Your vivid recollection really sets the tone for this year's rendition of Hack2Hire. Extremely nervous but excited to experience it firsthand!

Your vivid recollection really sets the tone for this year's rendition of Hack2Hire. Extremely nervous but excited to experience it firsthand!

Thanks a lot for the compliment! I'm glad that you found it worthwhile reading. All the best once again for your team in the Hack2Hire competition this week!


ACAG updates - subscriptions

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/cloud-adoption-guide/intermediate-stage/subscription-controls.md
:warning:Warning
View
Details

docs/cloud-adoption-guide/intermediate-stage/automation.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/enterprise-onboarding.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/enterprise-subscription.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/subscription-intermediate.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/subscription-multiple-original.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/subscription-multiple.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/intermediate-stage/subscription-ownership.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/intermediate-stage/subscription-patterns.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/subscription-controls.md

Line 11: [Warning] Invalid file link:(~/cloud-adoption-guide/intermediate-stage/also known as quotas).

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/cloud-adoption-guide/intermediate-stage/automation.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/enterprise-onboarding.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/enterprise-subscription.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/subscription-controls.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/intermediate-stage/subscription-intermediate.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/subscription-multiple-original.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/subscription-multiple.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/intermediate-stage/subscription-ownership.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/intermediate-stage/subscription-patterns.md
:white_check_mark:Succeeded

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

Hey Alex - could you please also add where you think these docs will go in the outline (overview.md)? This will help us figure out where these land in the "multiple teams/multiple projects" narrative.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/cloud-adoption-guide/intermediate-stage/automation.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/enterprise-onboarding.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/enterprise-subscription.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/overview.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/intermediate-stage/subscription-controls.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/intermediate-stage/subscription-intermediate.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/subscription-multiple-original.md
:white_check_mark:Succeeded

docs/cloud-adoption-guide/intermediate-stage/subscription-multiple.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/intermediate-stage/subscription-ownership.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/intermediate-stage/subscription-patterns.md
:white_check_mark:Succeeded

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

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/mspnp/architecture-center, working directory: W:\lvng-s, target branch: acag-intermediate-working, target branch commit: 0e7f1d98c9ab0ce78234d978ec8ca3afb14d725a, source branch: alex-acag-intermediate-working, source branch commit: .

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

Closing PR due to validation errors. Will submit a new PR instead.


[ISV-1856] The Operator pipeline integration tests should always increment the bundle semver.

Note: I have used the Pull request number rather than github run_id to represent the version_prefix because the long run_id value will be a cause in character limit excess for GitHub branch name we use in purpose to prepare the E2E testing as a parameter in the future. Also, I verified that if we re-run the E2E testing with the same version it doesn't cause a failure, so I avoid using the unique version semvers for the same commit.

@haripate The e2e usually runs after the PR is already merged. Are you sure the PR number will work as a version?

@haripate The e2e usually runs after the PR is already merged. Are you sure the PR number will work as a version?

I think it should work even after merging the PR, but to avoid confusion and make it still efficient I used github.run_number as it is a unique number as same as github.run_id.

@haripate The e2e usually runs after the PR is already merged. Are you sure the PR number will work as a version?

* I think it should work even after merging the PR, but to avoid confusion and make it still efficient I used `github.run_number` as it is a unique number as same as `github.run_id`.

With github.run_number I think we can face the same issues as I described in the first comment. See the description of the variable: "A unique number for each run of a particular workflow in a repository. This number begins at 1 for the workflow's first run, and increments with each new run. This number does not change if you re-run the workflow run ". The fact that the value remains the same if we re-run the workflow might cause the same issue.

Yes, I added the run_attempt to the version_suffix to prevent any future failure.


[Authz] Fix dependencies conflict by replacing jersey-bundle with jersey-client

Why are the changes needed?

jerysey-bundle 1.x , which is depended by ranger-plugins-common, contains javax.ws.rs.* (like javax.ws.rs.core.*) packages that conflicted to jarkarta.ws.rs-api-2.1.6.jar in Spark 3.0+ jars
exclude jerysey-bundle 1.x and include jersey-client 1.19.4 to align with ranger-plugins-common

Diff in Authz dependency jar list:
gethostname4j-1.0.0.jar
jackson-jaxrs-1.9.13.jar
- jersey-bundle-1.19.3.jar
+ jersey-client-1.19.4.jar
+ jersey-core-1.19.4.jar
jetty-client-9.4.50.v20221201.jar
jetty-http-9.4.50.v20221201.jar
jetty-io-9.4.50.v20221201.jar
jetty-util-9.4.50.v20221201.jar
jna-5.7.0.jar
jna-platform-5.7.0.jar
ranger-plugins-audit-2.3.0.jar
ranger-plugins-common-2.3.0.jar
ranger-plugins-cred-2.3.0.jar

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

cc @yaooqinn @pan3793 @zhouyifan279

Does jersey-client has the transitive deps? Can you list the final jars that users should be copied?

Does jersey-client have transitive deps? Can you list the final jars that users should copy?

OK, diff in jar list added to the description. See above.

The conflict classes javax.ws.rs.* you mentioned in description still existing in jsr311-api-1.1.1.jar

Maybe we should try to exclude jsr311-api-1.1.1.jar from jersey-client?

Maybe we should try to exclude jsr311-api-1.1.1.jar from jersey-client?

Agree. Let's do this.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5fab9b7). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #4437   +/-   ##
=========================================
  Coverage          ?   53.25%
  Complexity        ?       13
=========================================
  Files             ?      569
  Lines             ?    31146
  Branches          ?     4208
=========================================
  Hits              ?    16586
  Misses            ?    12989
  Partials          ?     1571

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks, merged to master/1.7

The spark web ui  can be displayed normally

The spark web ui  can be displayed normally

Thank you for the validation. @ASiegeLion


Document multiple drivers in projects

@adammhaile - how's this?  https://github.com/ManiacalLabs/BiblioPixel/wiki/Projects#projects-with-multiple-drivers

:+1:


Add logic to handle communicator bindings in kubectl

Description
Changes proposed in this pull request:
Enhancements

Currently, we allowed to have only one executor binding on the list.
Now executor manager is able to take the bindings list and calculate proper restrictions for a given channel.

E2E and unit test coverage to ensure no regression.

Change commands list output:

 Before   After

allowed verbs:
  - api-resources
  - api-versions
  - auth
  - cluster-info
  - describe
  - diff
  - explain
  - get
  - logs
  - top
allowed resources:
  - configmaps
  - daemonsets
  - deployments
  - namespaces
  - nodes
  - pods
  - statefulsets
  - storageclasses
allowed namespaces:
  include:
    - botkube
    - default

enabled:
  kubectl:
    kubectl-read-only:
      namespaces:
        include:
          - botkube
          - default
      enabled: true
      commands:
        verbs:
          - api-resources
          - api-versions
          - cluster-info
          - describe
          - diff
          - explain
          - get
          - logs
          - top
          - auth
        resources:
          - deployments
          - pods
          - namespaces
          - daemonsets
          - statefulsets
          - storageclasses
          - nodes
          - configmaps
      defaultNamespace: default
      restrictAccess: false
    kubectl-wait-cmd:
      namespaces:
        include:
          - botkube
          - default
      enabled: true
      commands:
        verbs:
          - wait
        resources: []
      restrictAccess: false`

Bugs:

On v0.12.4 @BotKube  get po/botkube-56c8c5874b-f2bbm -n botkube results in:Command not supported. Please run /botkubehelp to see supported commands.

This was addressed on this PR. And now it's validated properly.
On main the -A was returning all data even if not authorized namespace.include: ["default"]
This was addressed on this PR. Now, the -A works only if you have binding with namespace.include: ["all"]
On v0.12.4 BotKube get -n botkube po returns,Command not supported. Please run /botkubehelp to see supported commands.

On main returns:Sorry, the kubectl command is not authorized to work with '-n' resources on cluster 'dev'. Use 'commands list' to see all allowed resources.

In this PR it was fixed to:Please specify the resource name after the verb, and all flags after the resource name. Format <verb> <resource> [flags]

The better approach will be to remove all flags and find the resource name, however it's not simple. The proper solution should be to delegate this to kubectl which can do a proper parsing.

Testing
The merger was fully tested with unit-tests

https://github.com/kubeshop/botkube/pull/672/files#diff-efacb45b078225d6d6058c5ca9152fb14cb291663d78dd64f3984bcae17d49dd
https://github.com/kubeshop/botkube/pull/672/files#diff-189b9a00b5d538a5926c3c1543d0039ce151dd3e732dbefa346c103f4db44568

and e2e tests:

It's tested as a part of the e2e tests. See new values: https://github.com/kubeshop/botkube/blob/c5b791b7ba3d0d0e07f12e2df9618af08658ee93/helm/botkube/e2e-test-values.yaml#L31-L48

It doesn't make sens to test the rest of the communication platform as the logic was not changed for them.
Related issue(s)
Fixes https://github.com/kubeshop/botkube/issues/668

Hi @ezodude
From the output that you posted, I can see that you are not using the image from this PR. Please use:
--set image.repository=kubeshop/pr/botkube \
--set image.tag=672-PR \

In the e2e tests, you can see that we test that scenario. I specify multiple executors:
https://github.com/kubeshop/botkube/blob/d12b76ac1a1ee85c3ed2125d6a6a7dd6e394e34b/helm/botkube/e2e-test-values.yaml#L34-L70
and assert that only those enabled would be displayed:
https://github.com/kubeshop/botkube/blob/d12b76ac1a1ee85c3ed2125d6a6a7dd6e394e34b/test/e2e/slack_test.go#L124-L175
The new format was also mentioned in the PR description.
It has to be changed as a given list of resources and verbs need to be associated with a given set of namespaces. I could try to come up with some merging strategy. However, for now, IMO it's good to print the config as it is so user don't need to be familiar with different syntaxes. In the long term, it would be nice to revisit all BotKube's commands and come up with some consistent approach (better UX). In this way, configuration would be nicely presented (with some field/modals?) to the user, but he will still have an option to get plain config format e.g. using raw flag.

Merging this PR as it's blocking https://github.com/kubeshop/botkube/pull/670


Add bash/zsh completion support

Would be cool to add support for bash and zsh completion like we have in kompose

/me will start contributing to kedge by taking this one :)

this is closed in https://github.com/kedgeproject/kedge/pull/169


Manually filling in configs and entering the TitleId in it's hex format causes unclear error

The TitleId is given in hex format in the title's settings in xdp.xboxlive.com. Entering the hex format in the XboxServices.config will cause an error when deserializing, which is caught silently and hard to debug.
I had to debug into xbox-live-unity-plugin\CSharpSource\Source\Microsoft.Xbox.Services.UnityEditor.CSharp\Common\XboxLiveAppConfiguration.cs's Load method to understand the issue. A better error handling would be appreciated.

Thanks for raising this. We will address this in the next release.

vc Add addisplayer.cs

xs gamemanager.cs


Fix no-poly1305, no-siphash and no-blake2

Make sure we don't include files that we don't need if we've disabled
them.

Pushed. Thanks.


Building projection with 18362 SDK results in empty binary

using cswinrt 1.3.0
Building a projection for Microsoft.ReactNative (see Microsoft.ReactNative.ProjectReunion).
If I set
    <TargetFramework>net5.0-windows10.0.19041.0</TargetFramework>

Then I get a working DLL, size 329216 bytes.
If I set
    <TargetFramework>net5.0-windows10.0.18362.0</TargetFramework>

Then the resulting DLL is 4 kB and has no information in it.
Attaching bad DLL, deps.json and binlog:
Microsoft.ReactNative.ProjectReunion.zip

@Scottj1s @j0shuams does this sound familiar by chance?

The binlog shows only Windows SDK and Project Reunion winmds as input, but no ReactNative winmds, which explains the output.  Is the solution missing a package/project reference in the interop project, to pick these winmds up?
https://github.com/microsoft/CsWinRT/blob/f3d14e5cac610f2251a8cabdddfc4570343ac997/docs/usage.mda

This worked when using 19041 for whatever reason (the only thing that I changed was the TFM). See PR linked in the original description if you're curious :)

@scottj1s I found something interesting: a warning that I had missed:
C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\Bin\Microsoft.Common.CurrentVersion.targets(2526,6):
warning MSB3851: This project targets "Windows, Version=10.0.18362.0", but it is attempting to reference "C:\rnw\vnext\target\x64\Release\Microsoft.ReactNative\Microsoft.ReactNative.winmd" targeting "UAP,Version=10.0.19041.0", which is invalid. [C:\rnw\vnext\Microsoft.ReactNative.ProjectReunion\Microsoft.ReactNative.ProjectReunion.csproj]

The native microsoft.reactnative project has:
    <WindowsTargetPlatformVersion Condition=" '$(WindowsTargetPlatformVersion)' == '' ">10.0.19041.0</WindowsTargetPlatformVersion>
    <WindowsTargetPlatformMinVersion>10.0.17134.0</WindowsTargetPlatformMinVersion>

We recently moved WindowsTargetPlatformVersion from 18362 to 19041, because 19041 became the default that's installed with VS, so new developers installing VS today would get 19041, not 18362.
It seems odd that we would get an error for targeting 19041 if min version is 17134. It sounds like that check should be done against min version, not target version.
Thoughts?

TPV/TPMV have always tripped up innocent souls like yourself.  TPV is the SDK version that you actually build against, and TPMV is the version that you promise you've tested down to.  That's a very brittle system IMO, and I've always advised to set them equal unless you have light-up code (ApiInformation checks) requiring otherwise.
Given that, I think your fix should be to change your projection to target 19041 as well (not 18362)

After addressing Alex's concerns, opened a dedicated bug for the last bullet, closing.

Making the warning an error is tracked by #906


More changes for riscv. It now builds and fails to link.

Do you want to merge this or to fix linking before?

Merge it. Linking is going to be a huge task.

Push update branch.
LGTM


mac-7, mac-24 and mac-2 lost external connection phone device.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
mac-7, mac-24 and mac-2 lost external connection phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Rebooting all above phone devices to restore all connections.


Modify

Currently we have the following entry in organs for "Breast" from the UBKG:
    {
        "code": "C030027",
        "organ_cui": "C0006141",
        "organ_uberon": "UBERON:0000310",
        "rui_code": "BS",
        "sab": "SENNET",
        "term": "Breast"
    }

After discussing with the team from Indiana we learned that the RUI tool supports  “Mammary Gland (Left)” and “Mammary Gland (Right)“.
Is it possible to modify this existing entry into a left and right breast? Are there UBERON and CUI codes to support that? Would it be better to change this to be "Mammary Gland" instead of "Breast"?
@bhonick @AlanSimmons @shirey

Notes regarding the RUI tool

The 2 digit code is not required, it is used on the HuBMAP side as a quick lookup for the UBERON or FMA term (https://github.com/hubmapconsortium/ingest-ui/blob/5bce9fba3162a64faf0433a541fd8a465a883f87/src/src/constants.jsx#L139)
The UBERON code for "Mammary Gland" is http://purl.obolibrary.org/obo/UBERON_0001911

An FMA code is required to specify left or right and would need to be stored instead
Mammary Gland L -http://purl.org/sig/ont/fma/fma57991
Mammary Gland R - http://purl.org/sig/ont/fma/fma57987

After discussing with the Indiana team I have applied a band-aid fix to allow for RUI registration of Breast Blocks. I have added a check for this organ and set the RUI organ to be "Mammary Gland, L"

So, do we add codes for left and right mammary glands and delete the code for breast in SenNet?

@AlanSimmons I added this as an agenda item for the CODCC meeting tomorrow

In source
SENNET:C030082 - Mammary gland (left) - cross-referenced to FMA:57991
SENNET:C030083 - Mammary gland (right) - cross-referenced to FMA:57987
These will be available in the next release of the HuBMAP/SenNet UBKG.