RFC: Update Current Workflow for Maintaining Major Release Tags

Issue
Currently, our process of maintaining the major release tags is as follows:

We create a PR with the necessary changes.
We merge the PR.
We create a new release with a new tag to publish in the marketplace.
We move the major release tag to point to the latest release i.e

# 1 indicates the major release tag
git tag -f 1 <latest_release_tag>

Our current process

doesn't follow GitHub recommended practice. GitHub encourages creating a release branch i.e release/v1 before creating the release tag i.e v1.0.2
manually updating the major release tag might result in human errors like forgetting to update it.

Solution
In order to follow the GitHub recommended practice and reduce the effort of manually updating the release tag, here is what we can do:

Create a new Release Pull Request: We can create a new PR corresponding to the new release tag we want to publish. We are already following this convention in client projects.
Create a new GH Workflow for updating Major Release Tag: We can introduce a new GH Action workflow that will trigger right after we create a new release. The workflow will update the major release tag to point to the latest release. Popular Actions like actions/checkout follow this practice as well. Our workflow file could look like the following:

name: Move Major Release Tag

on:
  release:
    types: [created]

jobs:
  update_major_release:
    runs-on: ubuntu-latest

    steps:
    - uses: actions/checkout@v3

    - name: Get the major version num and update the tag
      run: |
        LATEST_TAG=${{github.ref_name}}
        MAJOR_VERSION=${LATEST_TAG%%.*}
        # why Hoang? explanation after this
        git config user.name 'Hoang Mirs'
        git config user.email<EMAIL_ADDRESS>        git tag -fa ${MAJOR_VERSION} -m "Update major version tag"
        git push origin ${MAJOR_VERSION} --f

Why @hoangmirs's username and email in the action? 😅
Because I can see his email on the security contact email:

Who Benefits?
developers
What's Next?

[ ] Agree on creating a release PR before publishing a new release tag.
[ ] Agree on using GH action to automate updating the major release tag to point to the latest release.
[ ] If we agree on the above, then we talk about the Git configuration inside the action

Resources

actions/checkout
cicirello/jacoco-badge-generator

The only concern is the event that triggers the workflow to update the latest tag. This workflow should only be run when the release PR is merged.

Thanks for the catch @olivierobert
I think instead of triggering the workflow when the release PR is merged, we can trigger the workflow when we publish the release. It's also recommended by GitHub as well

Our workflow file can be updated accordingly to get triggered after publish
on:
  release:
    types: [published]


chore: renovate for Python

🧭 What and Why
🎟 JIRA Ticket: -
Changes included:
just realized it did not worked because that's not the correct filename, also remove poetry from the automated managers because we only want to update the templates

gg, don't forget that you can test locally with https://github.com/algolia/data-ingestion/blob/main/docs/renovate.md 😉

yup I was too lazy to setup the whole stuff D:


faps

Mail<EMAIL_ADDRESS>

@captain239 I am closing this request as a duplicate to already merged #12504. If you have troubles while applying for the free educational license please send a screenshot of the page containing error message to<EMAIL_ADDRESS>Thank you.


'All' checkbox not getting selected on selecting all treeview items programatically.

Hi,
Is there anyway to select 'All' checkbox, as it is not being selected though i have selected all items in the treeview?

Ok. I will support this function


MM-49868: Remove OpenId-custom free feature from cloud starter

Summary

Hide OpenId-Custom configuration to show upgrade notice for Cloud Starter instances.
Ticket Link

Related Pull Requests

Has server changes (please link here)

Screenshots

before
after

Release Note

Fixed bug where OpenId Connect (Others) was configurable for cloud starter licenses.

E2E tests not automatically triggered, because the PR is not in a mergeable state. Please update the branch with the base branch and resolve outstanding conflicts.

/update-branch

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

@plant99
All scenarios are passed. However, I see the Save button gets enabled when I select OpenID Connect option from the dropdown

Can you fix it in this PR or should I create a bug ?

@plant99 All scenarios are passed. However, I see the Save button gets enabled when I select OpenID Connect option from the dropdown

Can you fix it in this PR or should I create a bug ?

very good catch @furqanmlk , I'll fix it here, we shouldn't let them save configs.

@furqanmlk could you please review again?

Cherry pick is scheduled.


Accept CLLocation parameter instead of coordinates as double?

Passing lat/long as double values for the coordinates doesn't seem as clean as simply passing a CLLocation object (easier to directly use with Core Location's user location). [As used in UberKit: github.com/sachinkesiraju/UberKit]

You're right that a CLLocation object makes more sense to pass around. For the beta release of the SDK, we didn't want to force use of CoreLocation just to use the RequestButton which just deep links to the Uber app. When the SDK is built out to expose all the endpoints, it'll make more sense to use the CLLocation object. In the meantime, the location.coordinate.latitude/longitude properties can be passed in if using CoreLocation. I'll leave this open. Thanks!

Fixed in v0.4.0


Support SBT 1.4.x.

With sbt-tc-logger version 6.0.0, TC builds fail with exit code 1 if the SBT project version is set to 1.4.x.

Cross-linking the related ticket TW-68077.

The issue was not in this plugin but in one of ours.


Residual translations

There are a few translations that got missed in the last round.  These were Google Translated and need to be checked by professional translators. Mark as fuzzy before sending to translators.
Spanish

Percentage (%%) indicators
Only completed target periods are included in the “indicators on track” calculations.
Last completed target period
If an indicator only has a Life of Program (LoP) target, we calculate performance after the program end date.
The actual value is 95% of the target value. An indicator is on track if the result is no less than 85%% of the target and no more than 115%% of the target. Remember to consider your direction of change when thinking about whether the indicator is on track.
For midline/endline and event-based target periods, we begin tracking performance as soon as the first result is submitted.

French

%(percentNonReporting)s% unavailable
No active country
Proceed anyway?
Please select a target.
Find
Percentage (%%) indicators
Only completed target periods are included in the “indicators on track” calculations.
Last completed target period
If an indicator only has a Life of Program (LoP) target, we calculate performance after the program end date.
The actual value is 95% of the target value. An indicator is on track if the result is no less than 85%% of the target and no more than 115%% of the target. Remember to consider your direction of change when thinking about whether the indicator is on track.
For midline/endline and event-based target periods, we begin tracking performance as soon as the first result is submitted.

QA Passed:
I checked in as many places I could look through for translation errors in french and spanish:
Admin lite create user
Admin lite create country
Admin lite Create program
Admin lite create Organization
and across country country > Program page > indicator form > IPTT > change log > Results form


[BUG][Android][2.0.0-beta.1] Proguard obfuscation crashes the app

Describe the bug
The app crashes when ACS SDK is instantiated on production environment ( code is obfuscated)
Exception or Stack Trace
Same as https://github.com/Azure/Communication/issues/69
To Reproduce
Steps to reproduce the behavior
Deploy ACS SDK on a project and obfuscate the code and run.
Expected behavior
The ACS SDK should not crash
Question
Are the rules below enough for this not to happen or we require additional rules ?
-keep class com.skype.rt.** {*;}
-keep class com.azure.** {*;}
-keep class com.skype.android.** {*;}
-keep class com.microsoft.media.** {*;}
-keep class com.microsoft.dl.** {*;}

These rules should be on the main page of the Android repository or in any other form of documentation.

Hi @bogdanmunteanu, thanks for reporting this issue. Would you mind taking a look @jsaurezlee-msft? As reported, it looks like a similar issue was reported a while ago and a using the proposed ProGuard rules solved it. I guess it could be possible there's more packages to include.
On another note, I agree that documentation should mention this configuration if requiring it is a common occurrence. Do you know how often developers would need to do this @jsaurezlee-msft? If the answer is "all of them" we should look into modifying our own ProGuard config so this doesn't trickle down to end-users, unless making that change ends up break peoples builds.


AbsenteeVotes for Assignments

I tried to implements Absentee votes also for assignments. The problem is, that the voting intention is much more complex as for motions (instead a simple Y, N or A, it has to be a candidate number or a Y, N, A for every candidate). Some basic things are set up, but not finished. This has a low priority and is huge work. Todo:

find a way to represent the voting intention. For assignments I think it could be a JSONField, so all kind of information can be saved.
make a nice UI to enter the voting intention.
Bind the absenteeVote not to an assignment, but to an assignment poll! For the intention, we need the candidates for the poll.
implement the logic in AssignmentBallot.create_absentee_votes.
Make the import work for all kinds of stuff.

@jwinzer Can you please check this issue? Is something still open and important for 3.0 release?


Rename lint-build-test workflow

Rename from lint-build-test.yml to pr-verifications.yml

Completed as part of #95 - will close as done


[ macOS ] Install pylint error Permission denied: '/Library/Python/2.7/site-packages/enum'

Environment data
VS Code version:  1.18.0 (1.18.0)
Python Extension version: 0.8.0
Python Version:  Python 2.7.10
OS and version:  macOS Sierra 10.12.6
Steps to reproduce:

Install ms-python.python
Open a Python source file
Error Linter pylint is not installed
Press "Install pylint"
Install fails tryin to mkdir

   Permission denied: '/Library/Python/2.7/site-packages/enum'

Tips or suggestions appreciated.  ( I am not a Python expert )
Logs
Output from Console window (Help->Developer Tools menu)
/usr/bin/python -m pip install pylint
edwards-mbp:source edward3$ /usr/bin/python -m pip install pylint
Collecting pylint
  Using cached pylint-1.7.4-py2.py3-none-any.whl
Collecting astroid>=1.5.1 (from pylint)
  Using cached astroid-1.5.3-py2.py3-none-any.whl
Collecting singledispatch; python_version < "3.4" (from pylint)
  Using cached singledispatch-<IP_ADDRESS>-py2.py3-none-any.whl
Collecting mccabe (from pylint)
  Using cached mccabe-0.6.1-py2.py3-none-any.whl
Collecting backports.functools-lru-cache; python_version == "2.7" (from pylint)
  Using cached backports.functools_lru_cache-1.4-py2.py3-none-any.whl
Requirement already satisfied: six in /Library/Python/2.7/site-packages/six-1.10.0-py2.7.egg (from pylint)
Collecting isort>=4.2.5 (from pylint)
  Using cached isort-4.2.15-py2.py3-none-any.whl
Collecting configparser; python_version == "2.7" (from pylint)
  Using cached configparser-3.5.0.tar.gz
Collecting enum34>=1.1.3; python_version < "3.4" (from astroid>=1.5.1->pylint)
  Using cached enum34-1.1.6-py2-none-any.whl
Collecting lazy-object-proxy (from astroid>=1.5.1->pylint)
  Using cached lazy-object-proxy-1.3.1.tar.gz
Collecting wrapt (from astroid>=1.5.1->pylint)
  Using cached wrapt-1.10.11.tar.gz
Installing collected packages: enum34, singledispatch, lazy-object-proxy, backports.functools-lru-cache, wrapt, astroid, mccabe, isort, configparser, pylint
Exception:
Traceback (most recent call last):
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/basecommand.py", line 215, in main
    status = self.run(options, args)
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/commands/install.py", line 342, in run
    prefix=options.prefix_path,
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/req/req_set.py", line 784, in install
    **kwargs
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/req/req_install.py", line 851, in install
    self.move_wheel_files(self.source_dir, root=root, prefix=prefix)
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/req/req_install.py", line 1064, in move_wheel_files
    isolated=self.isolated,
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/wheel.py", line 345, in move_wheel_files
    clobber(source, lib_dir, True)
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/wheel.py", line 316, in clobber
    ensure_dir(destdir)
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/utils/__init__.py", line 83, in ensure_dir
    os.makedirs(path)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/os.py", line 157, in makedirs
    mkdir(name, mode)
OSError: [Errno 13] Permission denied: '/Library/Python/2.7/site-packages/enum'
edwards-mbp:source edward3$

This is happening because you're using the system Python which means you need to install Pylint with sudo or into a virtual environment. I will need to discuss this with @DonJayamanne , but my suspicion is our recommendation will be to create a virtual environment and install into that (versus installing into a private cache directory).

@brettcannon Thanks.  A virtual environment sounds less risky.
Whether macOs or Red Hat Enterprise Linux I am afraid to mess with the system Python because of potentially bad and unknown consequences.
I would prefer to install everything with homebrew.
Is not there a utility that can switch environments, for example, between system Python, Python 2.7, and Python 3.0?

@esutton Yep, you're after pyenv.

Closing in favor of #386


Bump dependency on rsyslog cookbook to 1.15.0.

The rsyslog 1.15.0 cookbook is needed because 1.13.0 tries to set up systemd logging on Amazon Linux, a platform which doesn't actually use systemd.

Thanks, @ziggythehamster!


c.Format TypeError: replace() argument 2 must be str, not int

Initial Check

[] Python version is 3.6; check
[] Updated Twint with pip3 install --user --upgrade -e git+https://github.com/twintproject/twint.git@origin/master#egg=twint; check
[] I have searched the issues and there are no duplicates of this issue/question/request. check

Command Ran
python twitter.py
code:
import twint
c = twint.Config()
c.Search = "data science"
c.Format = '{username} | {tweet}'
c.Limit = 1
c.Pandas = True
twint.run.Search(c)
output:
(venv) C:\Users\31622\Documents\test>python twitter.py
Traceback (most recent call last):
File "twitter.py", line 9, in
twint.run.Search(c)
File "c:\users\31622\documents\test\venv\src\twint\twint\run.py", line 410, in Search
run(config, callback)
File "c:\users\31622\documents\test\venv\src\twint\twint\run.py", line 329, in run
get_event_loop().run_until_complete(Twint(config).main(callback))
File "c:\users\31622\appdata\local\programs\python\python36\lib\asyncio\base_events.py", line 466, in run_until_complete
return future.result()
File "c:\users\31622\documents\test\venv\src\twint\twint\run.py", line 235, in main
await task
File "c:\users\31622\documents\test\venv\src\twint\twint\run.py", line 286, in run
await self.tweets()
File "c:\users\31622\documents\test\venv\src\twint\twint\run.py", line 226, in tweets
await output.Tweets(tweet, self.config, self.conn)
File "c:\users\31622\documents\test\venv\src\twint\twint\output.py", line 166, in Tweets
await checkData(tweets, config, conn)
File "c:\users\31622\documents\test\venv\src\twint\twint\output.py", line 137, in checkData
output = format.Tweet(config, tweet)
File "c:\users\31622\documents\test\venv\src\twint\twint\format.py", line 23, in Tweet
output = output.replace("{replies}", t.replies_count)
TypeError: replace() argument 2 must be str, not int
Description of Issue
c.Format gives the error, without c.Format it does work normally.
Environment Details
Windows, visual studio code, in virtual environment, python 3.6.0

@successor1 try adding labels like: Username {username} | Tweet {tweet}
The docs for this are here: https://github.com/twintproject/twint/wiki/Tweet-attributes

I'm also running into the same issue when trying to format output. It seems that the problem happens here:
https://github.com/twintproject/twint/blob/e7c8a0c764f6879188e5c21e25fb6f1f856a7221/twint/format.py#L23
Python's replace function requires its arguments to be strings, and doesn't automatically coerce other types, opting to throw a TypeError instead. Seems like an easy solution would be to go through and wrap all replacement values in str().

Thought that I would leave this here. This is what fixed the issue for me
output = output.replace("{replies}", str(t.replies_count))
output = output.replace("{retweets}", str(t.retweets_count))
output = output.replace("{likes}", str(t.likes_count))
output = output.replace("{mentions}", ",".join(str(t.mentions)))

Since it looks like this isn't being fixed any time soon, I made a fork to fix it until they do. (if they ever decide to)
https://github.com/cyxv/twint


Removed IF(APPLE) hack to use /usr/local/lib as library directory

retest this please

retest this please

retest this please


Authorization or acl support?

Is there authorization or acl or rbac implementation in revel or golang? Will this be supported by revel in future? I searched such library, but didn't found good implementation by golang.

Hi, I'm the author of casbin. It is an authorization library that supports models like MAC, RBAC, ABAC.
Related to RBAC, casbin has several advantages:

roles can be cascaded, aka roles can have roles.
support resource roles, so users have their roles and resource have their roles too. role = group here.
the permission assignments (or policy in casbin's language) can be persisted in files or database.

Of course you can also don't use RBAC at all, and only use the basic ACL model. casbin can be highly customized to adapt to multiple scenarios.
Please consider it and let me know if there's any question:)

Duplicated


fix fix Illegal aggregate node: group by and output is empty

fix Illegal aggregate node: group by and output is empty, introduced by #33091

run buildall

run buildall


Not following RCLONE_RETENTION

Unifi Protect Backup version: latest
Unifi Protect version: 2.1.2
Python version:
Operating System:
Are you using a docker container or native?: docker

Description
When I set retention via environment variable, it doesn't appear to follow that. In my docker compose file, I have the retention set to 5d. However, when the nightly delete happens, it follows the default 7d retention. Aka, folders and all the files that are up to 7 days old stay around. Folders and files older than 7 days are deleted nightly. It's just not following the overwritten value of 5d.
Docker compose file:
version: '3'
services:
  unifi-protect-backup-box:
    restart: always
    image: ghcr.io/ep1cman/unifi-protect-backup:latest
    environment:
      - UFP_USERNAME=removed
      - UFP_PASSWORD=removed
      - UFP_ADDRESS=removed
      - UFP_SSL_VERIFY=false
      - RCLONE_RETENTION=5d
    volumes:
      - /unifi-protect-backup/data:/data

What I Did
cont-init: info: /etc/cont-init.d/99-custom-files exited 0
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service init-mods: starting
s6-rc: info: service init-mods successfully started
s6-rc: info: service legacy-services: starting
services-up: info: copying legacy longrun unifi-protect-backup (no readiness notification)
s6-rc: info: service legacy-services successfully started
s6-rc: info: service 99-ci-service-check: starting
[ls.io-init] done.
s6-rc: info: service 99-ci-service-check successfully started
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	Config:
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  address='removed'
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  port=443
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  username='removed'
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  password=REDACTED
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  verify_ssl=False
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  rclone_destination='local:/data'
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  retention='5d'
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  rclone_args=''
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  ignore_cameras=()
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  verbose=1
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  detection_types=['motion', 'person', 'vehicle', 'ring']
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	  file_structure_format='{camera_name}/{event.start:%Y-%m-%d}/{event.end:%Y-%m-%dT%H-%M-%S} {detection_type}.mp4'
2022-08-22 18:48:18 [INFO]:unifi_protect_backup.unifi_protect_backup:	Starting...
2022-08-22 18:48:18 [INFO]:unifi_protect_backup.unifi_protect_backup:	Checking rclone configuration...
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	rclone found: /usr/bin/rclone
2022-08-22 18:48:18 [DEBUG]:unifi_protect_backup.unifi_protect_backup:	ffprobe found: /usr/bin/ffprobe
2022-08-22 18:48:18 [INFO]:unifi_protect_backup.unifi_protect_backup:	Connecting to Unifi Protect...
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	Found cameras:
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	 - removed camera name
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	 - removed camera name
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	 - removed camera name
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	 - removed camera name
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	 - removed camera name
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	 - removed camera name
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	 - removed camera name
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	 - removed camera name
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	 - removed camera name
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	Setting up purge task...
2022-08-22 18:48:19 [INFO]:unifi_protect_backup.unifi_protect_backup:	Listening for events...

 (7 days)

I just tested the functionality and the files are being deleted as expected. What might be happening however is that empty directories are being left behind. Can you check if this is the case?

Hi! thanks for getting back to me and testing the functionality. I think I found my issue. I was basing the timeframe off my timezone and not UTC. Looking at the timestamps on the files, it's starting to make sense. Based on my timezone, the delete runs at 8pm ET. So I would expect 5 days before that to delete. Which it does, for example, this file just barely under 5 days old. I was looking at my ET time on the filename and saying that's bigger than 5 days. So when the service ran at 8pm last night, it wasn't old enough... idk if that makes sense. Sorry for wasting your time, appreciate your quick response and love what you've created!
-rw-r--r-- 1<PHONE_NUMBER>026 Aug 17 00:01 '2022-08-16T20-01-14 motion.mp4'

You can set the container time zone with the "TZ" env variable. Refer to https://en.wikipedia.org/wiki/List_of_tz_database_time_zones for a list of possible timezones
With that set, the timestamps of the files should match the timestamps of the file names


Change XxxS anonymous field name to non-anonymous fields named Value

This applies across the board in go-crypto, change all the type XxxS struct, to have a Value Xxx field instead of just the anonymous interface?
https://github.com/tendermint/go-crypto/blob/3f47cfac5fcd9e0f1727c7db980b3559913b3e3a/signature.go#L31
Example change, From:
type SignatureS struct {
	Signature
}

to
type SignatureS struct {
	Value Signature
}

This solution does make the constructor safer, but it seems harder to use for me.  I made an example of another approach in 8ff4ce2 (feature/improve-godata), which rejects manually constructing the XxxS objects (which is where the error came from), rather using a WrapXxx() constructor that makes sure we don't nest multiple levels.
This was inspired by the usage of embedding anonymous interfaces in github.com/pkg/errors Especially look at:

The struct
Wrapping
Unwrapping

I combined the unwrapping and the constructor in one, as here there is no need to have multiple nested levels (while nested errors are useful in pkg/errors if handled properly).
Please take a look and let me know what you think about that. (It's not much code, but faster to write it than explain what I intend to do).

Solution agreed to at the meeting (which solves this) is being worked on in #6
If you can "fix" the issue about not being able to use go-wire with private fields, that would be AWESOME.  (But either hacking go runtime or modifying go-wire).

Seconded.  Lets see how #6 goes.


CSS is not loading for components if routes have name that begins similar

Describe the bug
CSS is not loading for component which referred in route if routes have similar names:

routes/first.svelte
routes/first_2.svelte

Logs
To Reproduce
https://github.com/asv1/svelte_bug_css_not_load
Also video below.
Expected behavior
CSS is loaded properly.
Stacktraces
Information about your SvelteKit Installation:
 System:
    OS: Windows 10 10.0.17763
    CPU: (12) x64 AMD Ryzen 5 2600 Six-Core Processor
    Memory: 11.29 GB / 15.93 GB
  Binaries:
    Node: 14.16.0 - d:\portable\node\node.EXE
    npm: 6.14.11 - d:\portable\node\npm.CMD
  Browsers:
    Internet Explorer: 11.0.17763.1852
  npmPackages:
    @sveltejs/kit: next => 1.0.0-next.88
    svelte: ^3.29.0 => 3.37.0
    vite: ^2.1.0 => 2.2.1

Severity
Bug is annoying. Hard to find it.
Additional context
https://user-images.githubusercontent.com/13184913/115953070-1669b780-a4f2-11eb-9011-4a21c5ba5b25.mp4

Downgrade vite to version 2.1.5 should fix this issue

Looks like fixed now.


Update docs to add latest explain and give examples of latest changes

Pull request

What this PR does / why we need it
Docs updates
Which issue(s) this PR fixes

N/A
Describe testing done for PR

N/A
Additional information or special notes for your reviewer

Codecov Report

Merging #525 (61b4108) into main (ac2a5be) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #525   +/-   ##
=======================================
  Coverage   85.95%   85.95%
=======================================
  Files          65       65
  Lines        4123     4123
=======================================
  Hits         3544     3544
  Misses        453      453
  Partials      126      126

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Create class to find developers' organizations

This is research for #779. @joemasilotti I think this should be enough to get you started, but let me know if you need anything more.

Thanks for this! Can you please add company to the query and some output? Ideally, in the following format:
Developer profiles

Total: XXX
With GitHub organization(s): XXX
With company: XXX
With both: XXX

Thanks! Once that's in I'll run the numbers and see if it makes sense to implement.

@joemasilotti Done

OK, I finally got my GitHub access token working! Here are the results:
Developer profiles
Total: 828
With GitHub organization(s): 188
With company: 362
With both: 122

So we are looking at over 40% with either one. Which feels high enough to validate my assumption. Let's move forward with the original issue!
I'll close this PR and we can continue conversation back in #779.


azm v0.1.5

Plus minor refactoring in model migrations to reduce code duplication.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

6 of 39 (15.38%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 31.157%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/bdb/migrate/mig006/migrate.go
0
8
0.0%

pkg/bdb/migrate/mig/mig.go
6
31
19.35%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
1136

Relevant Lines:
3646

💛  - Coveralls


Add special characters

form_feed is used in https://github.com/abapGit/abapGit/pull/5404

thanks 👍


My outdoor string lights and new ceiling light 15" made the plug in freak out

homebridgelog.txt

Describe The Bug:

To Reproduce:

Expected behavior:

Logs:
I'm new to this but i have attached this config as an attachment

Plugin Config:
I'm new to this but i have attached this config as an attachment

Screenshots:

Environment:

Plugin Version:
Homebridge Version:
Node.js Version:
NPM Version:
Operating System:

I have the same error on a new install with the new BR30 matter LIFX bulbs


Show all prices including VAT/Tax

When doing an online shop in Germany, everything needs to be displayed including tax. What's the matter with this?
At them moment everything is shown and saved "without" tax.

This isn't currently supported I'm afraid


Add merge completion case

Thanks! Would you mind retargeting this to the series/0.6.x branch?

Re-triggering CI ...

I believe that the tests run on pg 11, while MERGE has been introduced in pg 15. Should we run the test conditionally?

Can we make the test check for merge support and then skip if not enabled on the server?

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (7e8ae17) 84.33% compared to head (b90205f) 84.29%.

Files
Patch %
Lines

...d/src/main/scala/net/message/CommandComplete.scala
50.00%
1 Missing :warning:

Additional details and impacted files
@@               Coverage Diff                @@
##           series/0.6.x    #1026      +/-   ##
================================================
- Coverage         84.33%   84.29%   -0.04%
================================================
  Files               126      126
  Lines              1736     1738       +2
  Branches            197      180      -17
================================================
+ Hits               1464     1465       +1
- Misses              272      273       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

It doesn't change the discussion above, but PG 11 seems EOL since November 2023 (https://www.postgresql.org/support/versioning/). Should we upgrade the docker to PG 12 (in a separate PR)?

Yeah that's a good idea.


RFC: Support Parameter Directions and Optional Parameters for MSSQL Stored Procedures.

Why make this change?

Adds support for OUTPUT Stored Procedure parameters from schema generation to data retrieval and return.
Backwards compatible - if no OUTPUT parameters are defined in the Stored procedure, it will use the legacy schema.graphql schema format.
Adds support for optional Stored Procedure parameters. They can now be excluded from the config if they are optional in the Stored Procedure.
Fixes several other bugs I encountered.
Refactored style consistency.
Closes #1429

What is this change?
Support for Parameter Direction:
Given the following Stored Procedure:
CREATE PROCEDURE [dbo].[GetBooks]
    @page INT,
    @pageSize INT,
    @totalBooks INT OUTPUT,
    @totalPages INT OUTPUT
AS
BEGIN
    SET NOCOUNT ON;

    SELECT *
    FROM dbo.Books
    ORDER BY Id
    OFFSET @pageSize * (@page - 1) ROWS
    FETCH NEXT @pageSize ROWS ONLY;

    SELECT @totalBooks = COUNT(*)
    FROM dbo.Books;

    SET @totalPages = CEILING(CAST(@totalBooks AS FLOAT) / @pageSize);
END;

Which you can query as follows:
DECLARE @totalBooks INT;
DECLARE @totalPages INT;

EXEC dbo.[GetBooks]
    @page = 1,
    @pageSize = 5,
    @totalBooks = @totalBooks OUTPUT,
    @totalPages = @totalPages OUTPUT;

SELECT @totalBooks AS "Total Books";
SELECT @totalPages AS "Total Pages";

Giving you results:

Running DAB will generate the following schema.graphql:
schema {
  query: Query
}

enum OrderBy {
  ASC
  DESC
}

input DefaultValue {
  Byte: Byte
  Short: Short
  Int: Int
  Long: Long
  String: String
  Boolean: Boolean
  Single: Single
  Float: Float
  Decimal: Decimal
  DateTime: DateTime
  ByteArray: ByteArray
}

type GetBooks {
  id: Int!
  title: String!
  year: Int
  pages: Int
}

"Represents the results of the executeGetBooks stored procedure execution."
type ExecuteGetBooksResult {
  "The GetBooks result set from the stored procedure."
  resultSet: [GetBooks!]!
  "The totalBooks InputOutput parameter from the stored procedure."
  totalBooks: Int
  "The totalPages InputOutput parameter from the stored procedure."
  totalPages: Int
}

type Query {
  "Execute Stored-Procedure GetBooks and get results from the database"
  executeGetBooks(page: Int = 1 pageSize: Int = 5 totalBooks: Int = 0 totalPages: Int = 0): ExecuteGetBooksResult!
}

enum ApplyPolicy {
  BEFORE_RESOLVER
  AFTER_RESOLVER
}

"The `Byte` scalar type represents non-fractional whole numeric values. Byte can represent values between 0 and 255."
scalar Byte

"The `Short` scalar type represents non-fractional signed whole 16-bit numeric values. Short can represent values between -(2^15) and 2^15 - 1."
scalar Short

"The `Long` scalar type represents non-fractional signed whole 64-bit numeric values. Long can represent values between -(2^63) and 2^63 - 1."
scalar Long

"IEEE 754 32 bit float"
scalar Single

"The built-in `Decimal` scalar type."
scalar Decimal

"The `DateTime` scalar represents an ISO-8601 compliant date time type."
scalar DateTime

scalar ByteArray

"The `@oneOf` directive is used within the type system definition language\n to indicate:\n\n - an Input Object is a Oneof Input Object, or\n - an Object Type's Field is a Oneof Field."
directive @oneOf on INPUT_OBJECT

directive @authorize("The name of the authorization policy that determines access to the annotated resource." policy: String "Roles that are allowed to access the annotated resource." roles: [String!] "Defines when when the resolver shall be executed.By default the resolver is executed after the policy has determined that the current user is allowed to access the field." apply: ApplyPolicy! = BEFORE_RESOLVER) repeatable on SCHEMA | OBJECT | FIELD_DEFINITION

"A directive to indicate the type maps to a storable entity not a nested entity."
directive @model("Underlying name of the database entity." name: String) on OBJECT

"A directive to indicate the relationship between two tables"
directive @relationship("The name of the GraphQL type the relationship targets" target: String "The relationship cardinality" cardinality: String) on FIELD_DEFINITION | INPUT_FIELD_DEFINITION

"A directive to indicate the primary key field of an item."
directive @primaryKey("The underlying database type." databaseType: String) on FIELD_DEFINITION

"The default value to be used when creating an item."
directive @defaultValue(value: DefaultValue) on FIELD_DEFINITION

"Indicates that a field is auto generated by the database."
directive @autoGenerated on FIELD_DEFINITION

Note: I chose this schema design because it makes the most sense. It wouldn't make sense for the scalar outputs to be a part of the results as they are not a property of the results (nor are they a column in a table). It makes sense for the scalar outputs to be properties within the same result object because they represent scalar outputs, just like the resultSet represents a tabular, or property-based output of a Stored Procedure, commonly referred to as a database "Result Set".
This more closely follows the pagination-style results where rather than having a “Connected” Type with an “items” property, we instead have an “Execute<STORED_PROC_TYPE>Result” Type with a “resultSet” property, as well as one property per OUTPUT parameter. In the case above we have “totalBooks” and “totalPages” as those were the output parameters to our Stored Procedure.
This can now be queried as follows:

Backwards Compatibility:
Additionally, it is backwards-compatible when no OUTPUT parameters are defined.
If we alter the procedure as below:
ALTER PROCEDURE [dbo].[GetBooks]
    @page INT,
    @pageSize INT
    -- @totalBooks INT OUTPUT,
    -- @totalPages INT OUTPUT
AS
BEGIN
    SET NOCOUNT ON;
    DECLARE @totalBooks FLOAT;
    DECLARE @totalPages FLOAT;

    SELECT *
    FROM dbo.Books
    ORDER BY Id
    OFFSET @pageSize * (@page - 1) ROWS
    FETCH NEXT @pageSize ROWS ONLY;

    SELECT @totalBooks = COUNT(*)
    FROM dbo.Books;

    SET @totalPages = CEILING(CAST(@totalBooks AS FLOAT) / @pageSize);
END;

We can see queries still retrieve the books:
-- DECLARE @totalBooks INT;
-- DECLARE @totalPages INT;

EXEC [dbo].[GetBooks]
    @page = 1,
    @pageSize = 5
    -- @totalBooks = @totalBooks OUTPUT,
    -- @totalPages = @totalPages OUTPUT;

-- SELECT @totalBooks AS "Total Books";
-- SELECT @totalPages AS "Total Pages";

If there are no output parameters defined in the procedure, I have DAB generate the following schema:
schema {
  query: Query
}

enum OrderBy {
  ASC
  DESC
}

input DefaultValue {
  Byte: Byte
  Short: Short
  Int: Int
  Long: Long
  String: String
  Boolean: Boolean
  Single: Single
  Float: Float
  Decimal: Decimal
  DateTime: DateTime
  ByteArray: ByteArray
}

type GetBooks {
  id: Int!
  title: String!
  year: Int
  pages: Int
}

type Query {
  "Execute Stored-Procedure GetBooks and get results from the database"
  executeGetBooks(page: Int = 1 pageSize: Int = 5): [GetBooks!]!
}

enum ApplyPolicy {
  BEFORE_RESOLVER
  AFTER_RESOLVER
}

"The `Byte` scalar type represents non-fractional whole numeric values. Byte can represent values between 0 and 255."
scalar Byte

"The `Short` scalar type represents non-fractional signed whole 16-bit numeric values. Short can represent values between -(2^15) and 2^15 - 1."
scalar Short

"The `Long` scalar type represents non-fractional signed whole 64-bit numeric values. Long can represent values between -(2^63) and 2^63 - 1."
scalar Long

"IEEE 754 32 bit float"
scalar Single

"The built-in `Decimal` scalar type."
scalar Decimal

"The `DateTime` scalar represents an ISO-8601 compliant date time type."
scalar DateTime

scalar ByteArray

"The `@oneOf` directive is used within the type system definition language\n to indicate:\n\n - an Input Object is a Oneof Input Object, or\n - an Object Type's Field is a Oneof Field."
directive @oneOf on INPUT_OBJECT

directive @authorize("The name of the authorization policy that determines access to the annotated resource." policy: String "Roles that are allowed to access the annotated resource." roles: [String!] "Defines when when the resolver shall be executed.By default the resolver is executed after the policy has determined that the current user is allowed to access the field." apply: ApplyPolicy! = BEFORE_RESOLVER) repeatable on SCHEMA | OBJECT | FIELD_DEFINITION

"A directive to indicate the type maps to a storable entity not a nested entity."
directive @model("Underlying name of the database entity." name: String) on OBJECT

"A directive to indicate the relationship between two tables"
directive @relationship("The name of the GraphQL type the relationship targets" target: String "The relationship cardinality" cardinality: String) on FIELD_DEFINITION | INPUT_FIELD_DEFINITION

"A directive to indicate the primary key field of an item."
directive @primaryKey("The underlying database type." databaseType: String) on FIELD_DEFINITION

"The default value to be used when creating an item."
directive @defaultValue(value: DefaultValue) on FIELD_DEFINITION

"Indicates that a field is auto generated by the database."
directive @autoGenerated on FIELD_DEFINITION

Which we can query as we did before introducing OUTPUT parameter directions:

Optional Parameters:
We currently require every parameter to a Stored Procedure in the config. We also do not have a concept of nullity for these config parameters; there is no way that I know of to pass NULL (it would come in as a string) or DBNull, etc.
With this change, I added config support for optional Stored Procedure parameters for MSSQL.
Given a stored procedure as below:
CREATE PROCEDURE [dbo].[GetBooks]
    @page INT,
    @pageSize INT,
    @id INT = NULL,
    @title NVARCHAR(1000) = NULL,
    @year INT = NULL,
    @pages INT = NULL,
    @totalBooks INT OUTPUT,
    @totalPages INT OUTPUT
AS
...

Where id, title, year, pages are optional parameters (since they have default values assigned). You can now omit these from the configuration, and DAB will not send these parameters to the MSSQL DB.
Before:
{
  "entities": {
    "Books": {
      "source": {
        "type": "stored-procedure",
        "object": "[dbo].[GetBooks]",
        "parameters": {
          "page": 1,
          "pageSize": 5,
          "id": "?",
          "title": "?",
          "year": "?",
          "pages": "?",
          "totalBooks": 0,
          "totalPages": 0
        }
      }
    }
  }
}

After:
{
  "entities": {
    "Books": {
      "source": {
        "type": "stored-procedure",
        "object": "[dbo].[GetBooks]",
        "parameters": {
          "page": 1,
          "pageSize": 5,
          "totalBooks": 0,
          "totalPages": 0
        }
      }
    }
  }
}

Note: I do think we should define a config value that maps to DBNull for providing null values for other DB.

There have been many changes since this was opened. Closing for now and you can re-open with latest changes and conflicts resolve.


Wrong syntax highlighting after put

Steps to reproduce

Run vim --clean
Source the following file:

syn region hsBlockComment start="{-"  end="-}" contains=hsBlockComment
hi def link hsBlockComment Comment
call setline(1, ['{-', '-', '-}', '- Test'])
call feedkeys('GddPPP2jp', 'tni')  " Note: no x flag

The last two lines are not highlighted as Comment

Expected behaviour
All lines highlighted as Comment, as after a :redraw!
Version of Vim
9.0.354
Environment
Operating system: Arch Linux
Terminal: kitty
Value of $TERM: xterm-kitty
Shell: fish
Logs and stack traces
No response

This bug doesn't happen if the feedkeys() is called on startup (e.g. using -S to source the file). The keys need to be fed later in this case.

Steps to reproduce

Run vim --clean
Source the following file:

syn region hsBlockComment start="{-"  end="-}" contains=hsBlockComment
hi def link hsBlockComment Comment
call setline(1, ['{-', '-', '-}', '- Test'])
call feedkeys('GddPPP2jp', 'tni')  " Note: no x flag

The last two lines are not highlight as Comment

Expected behaviour
All lines highlighted as Comment, as after a :redraw!

You have not defined any syncing.  When using a region that crosses line
boundaries you need to define syncing, otherwise Vim does not know it
has to look in previous lines for a region to start.
For this example you might want to use something like:
syn sync match hsCommentStart grouphere hsBlockComment "{-"
--
'Psychologist' -- Someone who looks at everyone else when
an attractive woman enters the room.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

This line is a simplified version of a line in runtime/syntax/haskell.vim. I guess there is a bug in haskell syntax file then.


Made an i18n plugin based on gettext and moment.js

It uses PO/MO files which I find cool to use along with Poedit and Moment.js for date and time localization

Thank you!


npm dev script for local live reload

watches views/ content, such as presentation.ejs, for file changes to live-reload browser preview;
courtesy of browser-sync
closes #2

Current coverage is 100% (diff: 100%)

Merging #4 into master will not change coverage

@@           master    #4   diff @@
===================================
  Files           1     1
  Lines          47    47
  Methods         5     5
  Messages        0     0
  Branches        6     6
===================================
  Hits           47    47
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update fbba174...1958ff9


Incomplete instructions?

Following the instructions and I'm getting a 502 Bad Gateway.
My exact steps
$ git clone https://github.com/scrapinghub/portia
$ cd portia
$ vagrant up
$ vagrant ssh
$  slyd

Open web browser to http://localhost:9001/
Getting a 502 error.
What am I missing?

Changing the path according to #442 fixed it for me.


SV2 roadmap

Hi, I've been making some contributions to SV2 and this project got me curious.
What is the vision for SV2 integration here?

We're R&D phase about Stratum V2 into library for miners. We're open for ideas and suggestions.

What is the vision for SV2 integration here?
Our idea is help decentralization for mining.


Should CustomTopLevel constructor require the identity parameter

Currently identity has a default value of None.  This means it is possible to create a CustomTopLevel object with no URI which could turn out to be a problem.  Is there a reason why that is allowed (maybe it is initialized to None in case the constructor is called as a builder method)?

I think identity has a default value of None to turn it into a keyword argument. I agree that we don't really want to let that travel too far before flagging it as an error.
I think it is the case that all TopLevel instances must receive an identity argument (either full URI or display ID). So maybe we need to add a check in the TopLevel constructor to ensure that there is a value.

I think identity has a default value of None to turn it into a keyword argument. I agree that we don't really want to let that travel too far before flagging it as an error.
I think it is the case that all TopLevel instances must receive an identity argument (either full URI or display ID). So maybe we need to add a check in the TopLevel constructor to ensure that there is a value.

So maybe we need to add a check in the TopLevel constructor to ensure that there is a value.

That sounds ideal to me as well.

So maybe we need to add a check in the TopLevel constructor to ensure that there is a value.

That sounds ideal to me as well.


Weird error when building the project on different machines

Hi there,
we are having issues with our local build on different machines with same version of TypeScript installed (mine and my colleague one). This is the error that we have:
ERROR in<EMAIL_ADDRESS>(219,15): In ambient enum declarations member initializer must be constant expression.

but this is not happening on my local environment (Angular).
We have an Angular front end project and I have created a separate module for managing Sentry only on particular environments (production and staging in this case).
We have also many external plugins/libraries in the project that are causing Non-Error exception captured exceptions that we do not want to track and send to Sentry as we are not interested in them. We basically want to track just our platform errors and exceptions.
For this reason, in my Angular module, I have implemented a check in the beforeSend like this:
Sentry.init({
    dsn<EMAIL_ADDRESS>    beforeSend(event, hint) {
        if (
            event.exception.values[0].value.startsWith('Non-Error exception captured')
            || hint.originalException['message'].startsWith('Non-Error exception captured')
        ) {
            // We want to ignore those kind of errors
            return null;
        }
        return event;
    }
})

I am still not sure about this check because it is checking strings so I was wondering if there is something more smart already implemented and/or if there are some sort of filtering that we may use instead.
P.S.: I may had to post this issue in another repository, sorry about that... 😄

We have just released v3.0.0 of the SDK which includes a lot of changes and improvements since this issue was opened.
If you can still reproduce this with v3, please open a new issue and I'll look into it.


crd: Add categories for cilium CRDs.

This commit is to add categories for cilium CRDs, so that users can
easily query cilium CRs with kubectl commands (e.g. kubectl get
cilium or kubectl get ciliumpolicy). Two categories (e.g. cilium and
ciliumpolicy) are added to start with.
Signed-off-by: Tam Mach<EMAIL_ADDRESS>Related slack conversation
crd: Add categories for cilium CRDs

It would be nice if we could add some CI to ensure that future crds do not forget to add the categories field.

Sure, CI has evolved a lot since last time I take a look at it. Will do it in separate PR.

I don't think the full CI is required, might need input here.

@sayboras I think a linter that checks for the existence of "categories" in that line should be enough?

I think a linter that checks for the existence of "categories" in that line should be enough?

Sure, I just add a simple check in second commit, let me know if this is what you have in mind. Manual testing was done by removing one category in ciliumclusterwidenetworkpolicies CRD.
$ make manifests
cd "./vendor/sigs.k8s.io/controller-tools/cmd/controller-gen" && \
go run ./... "crd:crdVersions=v1" paths="/home/tammach/go/src/github.com/cilium/cilium/pkg/k8s/apis/cilium.io/v2;/home/tammach/go/src/github.com/cilium/cilium/pkg/k8s/apis/cilium.io/v2alpha1" output:crd:artifacts:config="/tmp/tmp.WavdeFMNZO";
go run ./tools/categorycheck "/tmp/tmp.WavdeFMNZO"
2021/08/29 18:54:30 CRD categories missing for ciliumclusterwidenetworkpolicies.cilium.io
exit status 1
make: *** [Makefile:303: manifests] Error 1

test-me-please

/test-1.16-netnext

/test-net-next


Invoke-DBOQuery - unhandled datatype aclitem[]

Summary
Using Invoke-DBOQuery, queries returning non-null data of type aclitem[] (for example pg_database.datacl) throw an error. This data should be gracefully returned
Repro
Given Set-DBODefaultSetting -Name rdbms.Type -Value PostgreSQL,
PS > Invoke-DBOQuery 'select cast(null as aclitem[]);'

aclitem
-------

PS > Invoke-DBOQuery "select cast(null as aclitem[]), cast('{=c/postgres}' as aclitem[]);" -Verbose
VERBOSE: [17:50:25][Initialize-ExternalLibrary] Npgsql was found among the loaded libraries, assuming that the library is fully loaded
VERBOSE: [17:50:25][Invoke-DBOQuery] Establishing connection with PostgreSQL localhost
VERBOSE: Performing the operation "Executing query select cast(null as aclitem[]), cast('{=c/postgres}' as aclitem[]);" on target "localhost".
VERBOSE: [17:50:25][Invoke-DBOQuery] Executing sub-query select cast(null as aclitem[]), cast('{=c/postgres}' as aclitem[]);
[17:50:25][Invoke-DBOQuery] 42883: no binary output function available for type aclitem
Exception calling "GetResult" with "0" argument(s): "42883: no binary output function available for type aclitem"
At /Users/pvandivier/.local/share/powershell/Modules/PSFramework/1.1.59/PSFramework.psm1:3304 char:23
+         if (-not $Cmdlet) { throw $records[0] }
+                             ~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], Exception
    + FullyQualifiedErrorId : dbops_Invoke-DBOQuery
PS >

Full Error
PS > Invoke-DBOQuery "select cast(null as aclitem[]), cast('{=c/postgres}' as aclitem[]);"
[17:53:06][Invoke-DBOQuery] 42883: no binary output function available for type aclitem
Exception calling "GetResult" with "0" argument(s): "42883: no binary output function available for type aclitem"
At /Users/pvandivier/.local/share/powershell/Modules/PSFramework/1.1.59/PSFramework.psm1:3304 char:23
+         if (-not $Cmdlet) { throw $records[0] }
+                             ~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], Exception
    + FullyQualifiedErrorId : dbops_Invoke-DBOQuery
PS > $error[0] | select *

PSMessageDetails      :
Exception             : System.Exception: Exception calling "GetResult" with "0" argument(s): "42883: no binary output function available for type aclitem" --->
                        System.Management.Automation.MethodInvocationException: Exception calling "GetResult" with "0" argument(s): "42883: no binary output function available for type
                        aclitem" ---> Npgsql.PostgresException: 42883: no binary output function available for type aclitem
                           at Npgsql.NpgsqlConnector.<>c__DisplayClass161_0.<<ReadMessage>g__ReadMessageLong|0>d.MoveNext() in C:\projects\npgsql\src\Npgsql\NpgsqlConnector.cs:line 1034
                        --- End of stack trace from previous location where exception was thrown ---
                           at Npgsql.NpgsqlConnector.<>c__DisplayClass161_0.<<ReadMessage>g__ReadMessageLong|0>d.MoveNext() in C:\projects\npgsql\src\Npgsql\NpgsqlConnector.cs:line 1034
                        --- End of stack trace from previous location where exception was thrown ---
                           at Npgsql.NpgsqlDataReader.NextResult(Boolean async, Boolean isConsuming) in C:\projects\npgsql\src\Npgsql\NpgsqlDataReader.cs:line 526
                           at Npgsql.NpgsqlCommand.ExecuteDbDataReader(CommandBehavior behavior, Boolean async, CancellationToken cancellationToken) in
                        C:\projects\npgsql\src\Npgsql\NpgsqlCommand.cs:line 1231
                           at CallSite.Target(Closure , CallSite , Object )
                           --- End of inner exception stack trace ---
                           at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
                           at lambda_method(Closure , Object[] , StrongBox`1[] , InterpretedFrame )
                           --- End of inner exception stack trace ---
TargetObject          :
CategoryInfo          : NotSpecified: (:) [], Exception
FullyQualifiedErrorId : dbops_Invoke-DBOQuery
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at Stop-PSFFunction, /Users/pvandivier/.local/share/powershell/Modules/PSFramework/1.1.59/PSFramework.psm1: line 3304
                        at Invoke-DBOQuery<Process>, /Users/pvandivier/repos/git/dbops/functions/Invoke-DBOQuery.ps1: line 386
                        at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo : {}

PS >

Possible background context here

Don't use [aclitem], it's slated to disappear soon.

... dated 2000-10-31 ... 🤔
Might explain lack of support in the lib though

This type is not the only one that is not supported by npgsql, check out this FAQ section.
The only thing that works is this suggestion:
using (var cmd = new NpgsqlCommand(...)) {
  cmd.AllResultTypesAreUnknown = true;
  var reader = cmd.ExecuteReader();
  // Read everything as strings
}

It requests all the types to be returned as text, and, although it's not optimal, this is the only solution for now. I'm testing a branch that will add a -ReturnAsText switch to handle such situations.

Resolved in #101


ModuleNotFoundError: No module named '_tkinter'

I am running linux mint and have downloaded the requirements for the tool, but when I run HiyaCFW_Helper.py i get this error
Traceback (most recent call last):
File "./HiyaCFW_Helper.py", line 8, in
from tkinter import (Tk, Frame, LabelFrame, PhotoImage, Button, Entry, Checkbutton, Radiobutton,
File "/usr/local/lib/python3.8/tkinter/init.py", line 36, in
import _tkinter # If this fails your Python may not be configured for Tk
ModuleNotFoundError: No module named '_tkinter'

https://stackoverflow.com/questions/26357567/cannot-import-tkinter-after-installing-python-3-with-pyenv


perf/solver: don't zero fill weight gradients

Please verify this PR! I'm not completely sure that it's correct.
Weight gradients aren't used before they are overwritten at backpropagation
step, so initialization is redundant.
If #89 is applied, this patch improves performance 30% on
leaf-examples mnist.

I highly doubt that it works well when removing this.
The idea of clearing the gradients at that point is that the training of each minibatch starts out with a clean slate. If we don't do that, the gradients accumulate over multiple minibatches leading to unexpected behaviour.
As an alternative the clearing of the weights could be moved to the end of the function, but that would make some diagnostics like looking at the gradients after each minibatch hard/impossible. Personally I prefer the current way between the two alternatives.

Hmm, I was under impression that weight gradients for i-th layer are recomputed in the backpropagation pass from the following data:

output of i-th layer (computed during forward pass),
output gradients of i-th layer (computed in the previous backpropagation step for i+1-th layer or provided by solver for the last layer),
weights of i-th layer if layer has weigths (computed in the previous epoch),

So weight gradients for i-th layer aren't used in computation, so their contents doesn't matter.
I've also tried to verify it in practice, here are graphs for mlp and conv with and without this patch. It looks like there is no visible change.

Hey @alexandermorozov, thanks for the PR and the charts :+1:
Max and I agree, that it makes sense to merge the PR as it provides immediate value. If we should decide later to handle the gradients differently, it would be quite easy to put the one line back in.
I am happy to merge!

@homu r+

:pushpin: Commit 01be854 has been approved by MichaelHirn

:hourglass: Testing commit 01be854 with merge 434a0db...

:boom: Test timed out

@MichaelHirn, thanks! It's great that this PR is useful )
I've rebased this branch, it should merge now.

@homu r+

:pushpin: Commit 6c4482c has been approved by hobofan

:zap: Test exempted - status


issue #3: create spider russell with test settings

see issue #3

@k-allagbe need to finish up and merge this pull request to


fixed links to az cli query article in learning bash article

@dbradish-microsoft
Hi Delora, I fixed the links to the query az cli article in Carl's learning bash article.

Docs Build status updates of commit dbf9382:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-learn-bash.md
:white_check_mark:Succeeded
View

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


Devspace project name in devspace.yaml has to be unique

What happened?
We just tried to upgrade to devspace 6.x from 5.x
We have a monolith repo with multiple k8s deployment from one helm chart. With 5.x, devspace.yaml name is not required. We use to run devspace dev targetting different deployments using label selector. With 6.x, the name in devspace.yaml is required field. But with value set for name, we can't run multiple devspace dev in one namespace targetting different deployments.
When we try to run, we are getting there is already session using devspace in this namespace
Any suggestions on how to proceed with this use case ?

Hi @EdwinPhilip!
Is this an error, or a message being logged? I see where we log Skipping dependency X as it is currently in use by another DevSpace instance in the same namespace, but that is expected since it's already been deployed.
Is this multiple users running DevSpace against the same namespace, or the same user?
It would be helpful if you could share the log output from running DevSpace with the --debug flag on.
Thanks!

https://github.com/devspace-sh/devspace/blob/825e4d2c4c7dfd6e10acaf2767011e4f0dbf76c3/pkg/devspace/pipeline/pipeline.go#L108
This is the error i get.

If all your deployments are in the same devspace.yaml then I think your best option is to remove any dev.*.terminal configuration from your devspace.yaml and use devspace enter -l app=app1 for a terminal to each deployment.
If your deployments are in dependencies, then the default pipeline for each dependency defaults to deploy. You should be able to devspace dev to each dependency as needed without issue. Another approach is to change the pipeline to dev for each dependency, and have one process handle the file syncing and port-forwarding for all deployments. In this case you would also need to use devspace enter for a terminal as above.
The major change with DevSpace 6 is it tries to run things concurrently to improve performance. As such, we needed a way to deduplicate the concurrency in the case of a dependency tree.
Let me know if neither of these scenarios applies to your situation, and provide an example devspace.yaml so I can reproduce the issue.

My usecase is little different. We don't deploy with devspace instead we use devspace to replace existing pod and develop in it. So no dependencies or deploy pipelines, just only dev
I went with profiles to solve this issue
profiles: |-
  $(
  namedProfile="
  - name: namedProfile
    patches:
    - op: replace
      path: name
      value: ${label}
  "
  echo "$namedProfile"
  )

and ${label} replaces with pod/deployment labels. With this, i was able to run multiple devspace dev in my monolith repo.
We can close this ticket


docs: improve readme

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Files
Coverage Δ

src/transformer.ts
100.00% <100.00%> (ø)

src/transformers/oembed/oembed.ts
94.58% <100.00%> (+0.33%)
:arrow_up:


Fix missing status bar

JupyterLab 3.4 introduced new css variable --jp-statusbar-height. Themes created from older theme-cookiecutter missed it and statusbar is not shown in the UI.

Hi @konodyuk , would you also update npmjs package?

Sure, just published 0.3.1


feat(obstacle_stop_planner): add optional check for predicted trajectory

Signed-off-by: Berkay Karaman<EMAIL_ADDRESS>Description
Related issue: https://github.com/autowarefoundation/autoware.universe/issues/2252
In current planning control design, autoware does not check the predicted trajectory for collision. Because trajectory is not draw from base_link of ego vehicle for some situations, we need to check the predicted trajectory in addition to trajectory output of planning module.
This PR checks the predicted trajectory and motion planning trajectory for collision, then compare them. It makes decision for the nearest collision or slowdown points.
It is optional, in default it will be disabled.

Related links

Tests performed

I tested changes in planning simulator. It works good but there is a problem for some scenarios:
https://user-images.githubusercontent.com/45468306/203044604-08a24063-2cf5-4ad1-9136-b5891f482539.mp4
Controllers depend trajectory velocities while calculating the predicted trajectory. If there is a object on predicted trajectory, obstacle_stop_planner adds stop point to trajectory. Then, this affect the predicted trajectory so on.
Except this, it works as we wanted.
Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@brkay54 Thanks for your PR, but I would rather not feedback control outputs into planning module for the following reason:

Planning module and Control module would be tightly coupled, and decrease modularity.
Not all vehicles use MPC and this method cannot be used when using pure pursuite.

On the other hand, other solution is proposed in discussion, so could you check that?

@brkay54 Thank you for this PR.
We must be careful about adding this functionality: return predicted path information computed in control to the planning module. First, this PR will significantly change the interface between modules. It also changes the role of each module. Additionally, having FB loop between modules also brings up complex issues, exactly as in the video you posted.
There are reasons for separating control and planning, mainly to maintain high modularity as an OSS. However some performance is sacrificed by separating the modules, for example, path planning has to be done without consideration of control performance.
However, it is clear that we want to do the stop planning from the actual ego position. A solution using object_collision_checker is proposed in the issue, but is this not enough? Using this module will not degrade modularity and will allow stop planning for the predictive path output by the control while maintaining the existing interfaces between the modules.
In addition, we can suggest that instead of getting the predicted path from control, the predicted path be calculated in the planning stage (It was suggested before). There are several calculation methods. For instance,

To integrate the vehicle's speed and angular velocity over a certain period of time (the simplest one).
To create a model that converges to the target path from ego (e.g., lateral deviation decreases at a constant rate).
The duplicated MPC (or the same controller used in the control module) runs in the planning module (if the computational resources are available).

What do you think?
@xmfcx @mitsudome-r If you have any comments.

Thank you for your comment @TakaHoribe san.

However, it is clear that we want to do the stop planning from the actual ego position. A solution using object_collision_checker is proposed in the issue, but is this not enough? Using this module will not degrade modularity and will allow stop planning for the predictive path output by the control while maintaining the existing interfaces between the modules.

Actually, it can not solve our problem exactly. obstacle_collision_checker just send emergency if the object inside of braking distance. If emergency_hold not true, It will cause unwanted behavior (like move - stop - move - stop and go on).

In addition, we can suggest that instead of getting the predicted path from control, the predicted path be calculated in the planning stage (It was suggested before). There are several calculation methods. For instance,
* To integrate the vehicle's speed and angular velocity over a certain period of time (the simplest one).

* To create a model that converges to the target path from ego (e.g., lateral deviation decreases at a constant rate).

* The duplicated MPC (or the same controller used in the control module) runs in the planning module (if the computational resources are available).

Of course we can calculate an estimated path which start from ego base_link, however all of these methods will produce worse estimated path than controller produce. And also these methods cause a lot of false positive collisions (because we will use just current state of vehicle for prediction). Therefore, in my opinion, the most reliable predicted path will be output of controller.

We must be careful about adding this functionality: return predicted path information computed in control to the planning module. First, this PR will significantly change the interface between modules. It also changes the role of each module. Additionally, having FB loop between modules also brings up complex issues, exactly as in the video you posted.
There are reasons for separating control and planning, mainly to maintain high modularity as an OSS. However some performance is sacrificed by separating the modules, for example, path planning has to be done without consideration of control performance.

I agree with you, separating the control and planning module is better to avoid this type conflicts. But if one module of planning or control will do collision checking in vehicle's actual path, it shouldn't be planning because control has better and more reliable estimated path and we should use it.
Alternative to this PR, I want to propose that we can add new functionalities to obstacle_collision_checker. It is currently checking obstacles in path for emergency stop and also it can be used for smooth stop too. It can communicate with the vehicle_cmd_gate to stop the vehicle if there is an obstacle on footprints of predicted path. For example it can use pause_interface to stop vehicle or it can calculate the stop accelerations and send them to vehicle_cmd_gate etc. We can design the structure of how we stop.
Of course it will cause conflict between obstacle_collision_checker and obstacle_stop_planner. To avoid this, obstacle_collision_checker check only some part of predicted_path that is separated from motion_planning trajectory output.
With these, we can separate the modules and we have more reliable predicted path for collision. What do you think about it?

Can we discuss it here since it will be an architecture issue?
IMO : https://github.com/orgs/autowarefoundation/discussions/2859#discussioncomment-4235577


Update UI to match mockup

Codecov Report

Merging #66 (9f29b2f) into master (9939c07) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #66      +/-   ##
============================================
- Coverage     74.40%   74.31%   -0.10%
  Complexity      499      499
============================================
  Files            77       77
  Lines          1594     1596       +2
  Branches        184      184
============================================
  Hits           1186     1186
- Misses          348      350       +2
  Partials         60       60

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/HelpWindow.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/PersonListPanel.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/UiPart.java
84.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9939c07...9f29b2f. Read the comment docs.


Added http includes

Added http includes as requested by venum's house discord server.

W

W

K


Jabu

Please refer to our Commit Guidelines for documenting changes.
Changes proposed in this pull request:

Acceptance Criteria
General

[x] Tested on desktop (see [supported browsers](https://www.
Application-bundle-Java.zip
lightningdesignsystem.com/faq/#what-browsers-are-supported))
[x] Tested on mobile (for responsive or mobile-specific features)
[x] Confirm Accessibility
[x] Confirm RTL

Feature

[x] Reference User Story work item number in description
[ ] Add usage examples
[ ] Add documentation for new usage guidelines
[x] Add tests to test new components and implementation usage
[x] Add component specific Release notes mentioning the changes
[x] If feature requires the implementor to move to a new version, please provide a migration description in the component specific Release notes

Fix

[x] Reference Bug work item number in description
[x] Add tests to ensure bug does not happen again
[x] Add component specific Release notes mentioning the changes

Design Changes

[x] Add component specific Release notes mentioning the changes
[x] If change requires the implementor to move to a new version, please provide a migration description in the component specific Release notes

Deprecated

[x] If css selector is being deprecated, apply the @deprecate annotation to selector comment block

 /**
  * @selector .slds-bar
  * @deprecated
  */
 .slds-bar { ... }

[x] If css selector is being deprecated, move deprecated code to _deprecate.scss file and wrap in deprecate mixin.

[x] Provide deprecate mixin with version code will become deprecated and a brief description for what the code is being deprecated for.

 @include deprecate('4.0.0', 'Please use slds-foo instead of slds-bar') {
    .slds-bar { ... }
 }

Thanks for your contribution


how to remove searchBox ?

i want just remove searchBox
However, children must be redefined. Is there no other way?

I had a quick look for that but it's seems like there is no other way, I can't find any workaround, but I might be wrong.

You’re right. Currently the only way to achieve this is to define your own children. I probably will creat a new prop “hasSearch” to determine if the elements in the “items” has the function prop “search”, and only display the search box if “search” is provided.
It should be a simple request but I’m not available for this until the end of this month. You could either send a PR or wait for 2 more weeks :-)


Regex and token prefix causes error match

I'm trying to lex the Python language, which has the keyword not and the separate keyword not in (which may have arbitrary spaces between the not and in). My attempt at that is:
use logos::Logos;

#[derive(Logos, Debug, PartialEq, Eq)]
enum PythonToken {
    #[token("not")]
    Not,
    #[regex("not[ ]+in")]
    NotIn,
    #[regex(" +", logos::skip)]
    #[error]
    Error,
}

#[test]
fn python_token_test() {
    let mut lexer = PythonToken::lexer("not not");
    let mut result = Vec::new();
    while let Some(x) = lexer.next() {
        result.push(x);
    }
    assert_eq!(result.as_slice(), &[PythonToken::Not, PythonToken::Not])
}

However, this gives the result [Error, Not], when the first not could be validly parsed as a single Not. Is there any way to work around this issue? Or is the example I've provided buggy in some way?

Try not([ ]+in)?

Any way to do that and have them end up with different tokens? Or is Not{has_in: bool} the best approach?

Ah yes, since they can have conflicting matches you can force the short one to be correct with #[token("not", priority = 100)]

The priority doesn't appear to make any difference. Moving to not([ ]+in)? and using a single NotOrIsNot(bool) also doesn't work - if I replace the two Not and NotIn cases with:
    #[regex("not([ ]+in)?", |lex| lex.slice().len() > 3)]
    NotOrNotIn(bool),

I get [Error, NotOrNotIn(false)] for the input not not.

Ok, that's a bug, looks similar to #181 too.

Thanks for the info, that does look related. To fix the problem I've switched to treating them as two separate keywords, and then merging them in the parser I used afterwards.

I'm hitting a similar issue. I'm using a wrapper around Logos to handle modal lexing (to support string interpolation for example). I could reduce the error to the following example:
use logos::Logos;

#[derive(Logos, Debug, PartialEq)]
enum Token {
    #[token("[^ab]+")]
    Literal,

    #[regex("a")]
    SingleA,

    #[regex("a(b+)c")]
    Special,

    #[regex("b+")]
    Bs,

    #[error]
    Error,
}

fn main() {
    let mut lex = Token::lexer("ab");

    assert_eq!(lex.next(), Some(Token::SingleA));
    assert_eq!(lex.next(), Some(Token::Bs));
}

On confirmation that it has to do with backtracing is that if we replace SingleA with a token that matches more characters, it is still ambiguous, but it doesn't fail. I imagine it's because when it fails in the second case, it just to select the AThenBs token, while in the first case, it needs to go 1 character back in the stream.
use logos::Logos;

#[derive(Logos, Debug, PartialEq)]
enum Token {
    #[token("[^ab]+")]
    Literal,

    #[regex("ab+")]
    AThenBs,

    #[regex("a(b+)c")]
    Special,

    #[regex("b+")]
    Bs,

    #[error]
    #[regex(r"[ \t\n\f]+", logos::skip)]
    Error,
}

fn main() {
    let mut lex = Token::lexer("ab");

    assert_eq!(lex.next(), Some(Token::AThenBs));
}


Visually identify rules that use fs.*

Enabling any of these rules causes dramatic slow down.
For context, running eslint against our project with 3,000+ files takes ~3 minutes with any of these rules enabled, and just another 20 seconds otherwise.
A simple indication in documentation that these rules cause the slowdown would be sufficient.
{
  "import/no-relative-parent-imports": 0,
  "import/no-relative-packages": 0,
  "import/no-unresolved": 0,
  "import/no-extraneous-dependencies": 0,
  "import/no-unused-modules": 0,
  "import/no-restricted-paths": 0,
  "import/no-duplicates": 0,
  "import/no-useless-path-segments": 0,
  "import/no-cycle": 0,
  "import/no-internal-modules": 0,
  "import/extensions": 0,
  "import/no-self-import": 0
}

I identified these rules by checking which rules include eslint-module-utils/resolve.

Yeah, for the sake of eslint in vscode, i set eslint.nodeEnv: "editor" in my config, and use that environment variable to disable the slow rules in my .eslintrc.js


Increment the numeric array index for cloned field names

When field name come with array format, sometime it required to increment the array index. So i added the code to do that in the redos method. Please merge it if you feel it's useful.
Example :
Default field : <input type="text" name="emails[0]"/>
Cloned Field : <input type="text" name="emails[1]"/>

The merged code is wrapped in:
if ($this.config.serializeIndex) { ... }
There is no 'serializeIndex' config option. Shouldn't it be 'serializeID'?

@DamiansDesign : It's not related to this pull request, because those changes are applied recently, still it's good find


Train my own datasets

Hello, thank you for sharing the code.
But I got some trouble training my own data. I was wondering if anyone can share their own training code including preparing data for network and the training part.

Same issue.


Drag and drop slow

The drag and drop feature, especially with adding/removing spaces is now slower than before since moving the app to ace.jlab.org and upgrading to latest jQuery UI.
The jQuery UI lib is used for drag and drop and the version went from 1.10.3 to 1.13.2 and jQuery went from version 1.10.2 to 3.6.1.  Changes to the lib API resulted in the following changes to calendar:  https://github.com/JeffersonLab/calendar/commit/9fc7b4cbf2f8d5e68f77f46d1fe5740339360320
Note: It appears switching from outlook or month view to week view minimizes the number of occurrences and speeds up drag and drop.

When SAD Calendar was first created back in 2011 (released early 2012) jQuery UI was a good choice for drag and drop.  Today more than a decade later there are probably better choices, especially ones that leverage the browser built-in drag and drop API such as:

https://github.com/lukasoppermann/html5sortable
https://codepad.co/snippet/sortable-list-with-drag-and-drop

Alternatively, we could roll back to older versions of jQuery and jQuery UI since those were working fine.


Powerthesaurus is broken because Powerthesaurus API change

It has also been reported in upstream.
https://github.com/SavchenkoValeriy/emacs-powerthesaurus/issues/11
pressing SPC s T on emacs, doesn't show any user feedback, no error or message at all on the messages buffer or mini-buffer.

As of 27c64a3 this issue should be resolved. I'll keep this open until it is resolved upstream and we can revert back to the source package. Thanks for bringing it to my attention!

Hi. The original upstream repo has been updated so there is no longer a need for pulling from the fork.

Great! @maxchaos Thanks for letting me know!


[Feature] min_p sampling parameter

Motivation
The min_p sampling parameter is becoming quite popular. It's conceptually simple and "makes sense", and (at least anecdotally, according to opinions of many model fine-tuners and users in the LocalLlama community) it tends to perform better than the usual top_p+top_k approach. You can see the readmes of HF repositories of many new model finetunes/merges recommend to use min_p instead of top_p and top_k.
Related resources

vLLM: https://github.com/vllm-project/vllm/blob/8ea5e44a435e8731fd6f5ba4c329dd112752532a/vllm/sampling_params.py#L64C9-L66C57

min_p: Float that represents the minimum probability for a token to be considered, relative to the probability of the most likely token. Must be in [0, 1]. Set to 0 to disable this.

So e.g. a min_p of 0.07 means that if a token's probability is less than 7% of the size of the highest-probability token, it will be disqualified. A min_p of 0.5 would mean that if a token's probability is not at least half the size of the highest-probability token, then it is disqualified. Said another way, min_p allows you to set a minimum fraction of the most likely token's probability, else the token cannot be sampled.

https://github.com/vllm-project/vllm/pull/1642
https://github.com/oobabooga/text-generation-webui/pull/4449
https://github.com/ggerganov/llama.cpp/pull/3841

Please see the above links for more info.

@irexyc Please put this feature in the work list

Wondering if there's any chance this could get implemented soon? Currently supported in vLLM, SGLang, llama.cpp, text-generation-webui with increasing usage across the community - seems to basically a "free" intelligence boost without hurting creativity.

Hi, @josephrocca
@irexyc used to work it out in PR #1966
This PR involves other features and improvements, making it hard to review.
So, @irexyc is splitting the PR into smaller ones.
I think min_p will be supported soon.
Stay tuned.

Thanks! I've tested this, and there are no issues as far as I can tell, so I think this can now be closed. If I come across any issues I'll re-open. To test it via the official docker image, I had to add min_p as an additional field here:
https://github.com/InternLM/lmdeploy/blob/e2aa4bd7a434944f2f95f7f55a8272ebc786bf2f/lmdeploy/serve/openai/api_server.py#L610
And here:
https://github.com/InternLM/lmdeploy/blob/e2aa4bd7a434944f2f95f7f55a8272ebc786bf2f/lmdeploy/serve/openai/protocol.py#L254


[WFCORE-4346] Allow tests to configure LegacyKernelServicesInitialize…

…r to not load certain resources from the parent CL
https://issues.jboss.org/browse/WFCORE-4346
This is a longer term solution to the test problem hit by #3685. https://github.com/wildfly/wildfly/pull/12098 was a temporary workaround.
@fl4via and @stuartwdouglas FYI.

Thanks, @bstansberry ! I created https://issues.jboss.org/browse/WFLY-11759 to use the new exclusion feature in the tests and then revert the workaround in https://github.com/wildfly/wildfly/pull/12098/commits/8c4127aa1cf1e8a50e4cbd05b6259f93540b04c4

I added tests for this, so it's ready to go now (assuming they pass on CI as they do locally.)

This PR violates build from source principles by containing child.jar and parent.jar.


[WIP] Adapt e2e scripts so they can be used more generally (and locally)

So far:

make them all respect a base directory, to avoid stomping on
/usr/local
add some notes in each about what they depend on

Superseded by https://github.com/weaveworks/flux/pull/2058


Modified files are not found anymore

In Atom 1.8.0 (and I think in the previous 1.7.3 as well) the tree view panel has no content anymore.
I have attached a screenshot and I hope it's not related to other packages I use.
Would be great to get it fixed, I really like this package!

Thank you for reporting this, I will updated it when I have some time. Cheers.

Seems to be ok again with version 0.6.3 on atom 1.8.0. Thanks

It's just not picking up workspace changes :(

Ok I updated the package and it should all be working fine on 0.6.4. Try it out! Cheers


Improve documentation of missing features and workarounds

KaTeX is optimized for rendering a subset of LaTeX fast—which is great! What's not so great, though, is when you don't know quite what that subset is and run into issues. Right now, KaTeX's main documentation doesn't cover well what it's missing. There's a listing of what is present, but not what isn't. CONTRIBUTING.md tells you how to add a new symbol, but doesn't tell you where you should start. Aside from paging through the issue tracker, there's no clear way to know what's missing or where you should help out first.
I believe a central source of missing features would greatly help with both directing contributors at what's missing and with making sure people considering KaTeX for LaTeX rendering on the web know what they're setting up. If a user needs more intense LaTeX rendering, KaTeX's documentation should direct them to MathJax, perhaps with a guide to set it up as a fallback. For the features that are partially supported, this would be a great place to put or link workarounds so that they're easy to find. If a user wants to update their site's math after a few months of KaTeX improvements, they could just scroll through for what they used and see if there are better workarounds (or if they've been eliminated entirely) without having to subscribe to or bookmark a (potentially large) set of issues.

It sounds like you're looking for an index of the feature requests (say, "regular LaTeX features") that are in the issues? (Unless you have in mind a way to also find LaTeX features we're missing that are not yet issues...)  I worry about keeping such a list up-to-date...  Maybe if there was a tag on issues for "missing LaTeX feature", we could link to the list of them (as opposed to the entire list of issues, which has many other KaTeX-specific things on it).
Could you walk through a use-case where you want a list of missing features?  My impression is that most users who are considering KaTeX just try it out and see if it's missing anything they need, and if so, they search for that feature in the issues (it's usually clear that some \command is not supported), follow it, and/or report it as a new issue. Then we try to add it when possible...  Personally, KaTeX has basically all the features I want, and the many users seem to agree...
Personally I'd be more interested in some way to automatically detect (and list) features supported by MathJax (say) but not KaTeX.  Not sure how difficult this would be.

The list of what isn't in KaTeX could be very large.  LaTeX supports lots of packages.  I guess we could list some popular packages that we don't support at all, and then for packages where we do have partial support list those things from those packages which we don't support.  This would take a big effort though so I'd want to have some way to automate this.

There is an Alphabetical List of TeX Commands available in MathJax. Maybe we can use it to build a list of missing features and compare with MathJax.

This page makes a pretty good attempt at documenting which MathJax commands are not yet in KaTeX.  The KaTeX Contributing page already alludes to it.

Several of those MathJax features are not present in any LaTeX package. And some of them are duplicates of LaTeX functionality. Like \cases in addition to \begin{cases}. Such features may never make it into KaTeX.

As to a fallback, I used to have a site in which I would make the following call after calling renderMathInElement;
(function () {
  var script = document.createElement("script");
  script.type = "text/javascript";
  script.src  = "http://cdn.mathjax.org/mathjax/latest/MathJax.js?config=TeX-AMS_HTML";
  document.getElementsByTagName("head")[0].appendChild(script);
})();

That code is now out of date. MathJax has changed their CDN and they now have version numbers, not "latest" in the address.
But I stopped calling a fallback because there was no longer any point. KaTeX renders all that I need.

Linking this page prominently in the main KaTeX documentation instead of in the contributing page would be enormously helpful if someone installs KaTeX only to later find out that something they want to use for a new document isn't supported yet.

@kevinbarabash Query: Should I put this page into the list of links on the function support page?

I can't think of a title short enough to fit onto the list. Maybe I'll just add the link to the introductory paragraph at the top of the page.

@blackhole12 thank you for reminding me about that page.
Interestingly % fails b/c we require a new line at the end of comments.  The test for \left and \right fail b/c we require those to be balanced as does LaTeX (MathJax appears to be more liberal with what it accepts).  We should probably file issues with https://github.com/utensil-site/available-in-katex/issues for the \left and \right at least.
The list appears to be out of date a bit, e.g. \S and varGamma etc were added recently.  It's use the version from khan.github.io/KaTeX which explains why.  I need to update that.
Some of the tests should fail, e.g. \end on its own.  Some are easy to add, e.g. \cal is the implicit group style version of \mathcal which we already support.  I think \unicode would be pretty easy to add too.
The list is also highlighting some bug, e.g. the \llap test.  We aren't allowing \, in text mode but we should.  Same thing with \rlap except it's \;.

I've also opened https://github.com/utensil-site/available-in-katex/issues/2 to find a way to run the tests in node so that we can use the data to update our README.md at publish time.

I think \brace and \brack wouldn't be too hard to support as we already support \over, \choose, and \atop.  They're all very similar.

Yeah,  \llap and\rlap are difficult. In LaTeX, they work in math mode, but only sometimes, depending on what's in the argument. And I have not found a robust rule that tells me exactly when they work and when they don't.

The list of what isn't in KaTeX could be very large.

Yes, but we can focus on a subset taken from two sources:

KaTeX's GitHub issues page.
http://utensil-site.github.io/available-in-katex/

Those are items that somebody either cared about enough to program them or at least to sit down and write a request.
That subset is much smaller, with fewer items to manage than the KaTeX function support page. So I've written it up. It's a KaTeX wiki page.
Please feel free to edit it and improve it.

I think \brace and \brack wouldn't be too hard to support

Technical notes on the amsmath package deprecates the whole family of functions that use the infix syntax similar to \atop. It suggests using \genfrac instead.  I can write a PR that will add \genfrac to KaTeX.
Once we have \genfrac, do we still want to include \above, \brace, \brack, and \atopwithdelims ? \genfrac can be used as an alternate for any of those functions.
The AMS Tech Notes says that they deprecated those functions because the infix syntax causes problems for macros. Maybe @edemaine can weigh in here?

I'd say we should ideally support both.  We already support \over and \atop (which I'm very happy about, as I use those myself day-to-day -- I find they are more intuitive than say \frac).  It also shouldn't be hard to support one given the other (and I'd be happy to work on that aspect of it, as I've added these kinds of implicit-argument commands before).  But \genfrac sounds like a good starting point!

@edemaine Say that we add the other infix functions. Should the function support page then include a disclaimer saying that they might coexist poorly with macros?
Or are those problems just the sort of thing that a macro writer is expected to know?

I think a warning to not use infix functions inside macros is a good note as lots of people will probably be writing macros even though they don't know some of these issues.

Personally, I find the AMS's deprecation of a standard TeX/LaTeX feature kind of obnoxious, even if it is well-intentioned.  Many users don't \usepackage{amsmath}, so they don't even have a choice here.  I believe KaTeX should be agnostic on these points (we shouldn't deprecate them), in particular because it's unlikely that KaTeX-level macros will get into the issues mentioned.  But it certainly wouldn't hurt to link to the AMS document for reference.


docs: Automated Changelog Update for v5.0.0-beta5.3

copilot:all
Automated changes by create-pull-request GitHub action

🤖 Generated by lychee action
Summary

Status
Count

🔍 Total
1258

✅ Successful
378

⏳ Timeouts
0

🔀 Redirected
0

👻 Excluded
879

❓ Unknown
0

🚫 Errors
0

Full action output
Full Github Actions output


403 when opening project settings with the aim to delete a project

I previously deleted a draft of a project, which worked fine. For anther project some weeks later I also wanted to delete a draft, but got an 403 trying to view the project settings.

--> Project settings

Any hints on that?
Windows 10, Chrome v.107.0.5304.91.

Its not a bug, as this is connected to permissions and an owner is not allowed to access the project settings. I will add more information, as soon as we have got more details on that via the account of the team creator.

Created a NFDI4Chem Lead-by-Example team was just about to add people. Following screenshot tells...

...that an owner is allowed to read, update and delete a project, study and dataset, while deleting is not possible (anymore). If an owner can't delete a project, study and dataset, that he has the same rights as the collaborator, right?
I am the "owener" with Team "IPB Halle", but can't delete a project.

Duplicate #https://github.com/NFDI4Chem/nmrxiv/issues/590


Compute weight only once.

Seems that currently dynamic weight might be computed multiple times. The PR materializes weight expression to a single variable and then just passes a reference to avoid that.

bot merge


HTTP 431 issue snapshot (DO NOT MERGE)

I hit https://github.com/istio/pilot/issues/978 while writing PoC code for supporting headless TCP services.
Environment:
isaiah@xerxes:~$ minikube version
minikube version: v0.21.0
isaiah@xerxes:~$ kubectl version
Client Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.1", GitCommit:"b0b7a323cc5a4a2019b2e9520c21c7830b7f708e", GitTreeState:"clean", BuildDate:"2017-04-03T20:44:38Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.0", GitCommit:"d3ada0119e776222f11ec7945e6d860061339aad", GitTreeState:"clean", BuildDate:"2017-07-26T00:12:31Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

Steps to reproduce:

Run pilot/bin/e2e.sh, wait until pods have started, then ctrl+c tests.
Use kubectl to send a request from a to b:

isaiah@xerxes:~$ kubectl exec -it -n istio-test-x5hx2 a-1570096860-rd4c1 -c app -- client -url http://b
2017/08/16 20:06:11 [0] Url=http://b
2017/08/16 20:06:12 [0] StatusCode=431
2017/08/16 20:06:12 All requests succeeded

Seems to go away after 10-15 minutes.

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
03962ce8d7de23dc1f2f91900799b98f90d3288d
link
/test pilot-presubmit

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

Closing, because it does not seem to reproduce reliably.


Accept config.driver as a module for browserify/webpack compatibility

The node-mssql tries to resolve driver module path at runtime (from 'driver' configuration option), making the project incompatible with popular bundler systems (browserify, webpack, etc) which can only resolve static module paths. This patch makes possible to pass the driver module directly in 'driver' option (instead of string), eliminating the need for runtime module path resolving. An example:
mssql = require('mssql')
mssql.connect({ driver: require('mssql/lib/tedious') })
Fixes https://github.com/patriksimek/node-mssql/issues/318

Thank you Jaakko and sorry for not replying for so long. I was finally able to get some time to redesign the whole library. This PR is no longer relevant since I have completly dropped the current behavior of dynamic driver loading. Anyway, thank you for you effort, very appreciated.