Use UIScrollView's gesture instead of adding a new UIPanGestureRecognizer

I am using this library to present several UIViewControllers. Many of them have UIScrollView subviews.
ModalTransitionDelegate.interactiveController.gestureRecognizer is private and this makes it impossible for me to tell my views to prefer their gesture recognizers over the interactive controllers gesture recognizer.
What is your recommended way around this?

@marcosgriselli hey man, sorry to bother, any news on this issue?

Hey @sevgjan no worries. I'm gonna be totally honest I haven't had the time to work on this. I took a look at it when the issue was created and realized it wasn't as easy as applying a hack so I can't give an estimate of when this will be fixed/supported. Sorry for the inconvenience.

No problem @marcosgriselli , just wanted to check if there's any news on this issue.


Add fantasy-premier-league-rankings extension

Description
This is an extension which allows a user to input their fantasy premier league team id and keep track of their rankings across all the leagues they are entered in.
Screencast
https://user-images.githubusercontent.com/47414652/189547553-1dc0f772-aa34-4dbf-896e-c06f57172108.mp4
Checklist

[X] I read the extension guidelines
[X] I read the documentation about publishing
[X] I ran npm run build and tested this distribution build in Raycast
[X] I checked that files in the assets folder are used by the extension itself
[X] I checked that assets used by the README are placed outside of the metadata folder

I'm going to fix the screenshots shortly and shoot them on a retina screen instead to make sure they are the right resolution

Hi @hwb15 👋
Thanks for your contribution, could you look into the linting issues?
When I test this extension I just get this result, do you happen to know what might cause this?

12:54:43.422 [AxiosError: Request failed with status code 403] {
  code: 'ERR_BAD_REQUEST',
  config: {
    transitional: {
      silentJSONParsing: true,
      forcedJSONParsing: true,
      clarifyTimeoutError: false
    },
    adapter: [Function: httpAdapter],
    transformRequest: [ [Function: transformRequest] ],
    transformResponse: [ [Function: transformResponse] ],
    timeout: 0,
    xsrfCookieName: 'XSRF-TOKEN',
    xsrfHeaderName: 'X-XSRF-TOKEN',
    maxContentLength: -1,
    maxBodyLength: -1,
    env: { FormData: [Function] },
    validateStatus: [Function: validateStatus],
    headers: {
      Accept: 'application/json, text/plain, */*',
      authority: 'users.premierleague.com',
      'cache-control': 'max-age=0',
      'upgrade-insecure-requests': '1',
      origin: 'https://fantasy.premierleague.com/',
      'user-agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36',
      'sec-fetch-site': 'same-site',
      'sec-fetch-mode'<…>

Hi @hwb15 👋
Thanks for your contribution, could you look into the linting issues?
When I test this extension I just get this result, do you happen to know what might cause this?

12:54:43.422 [AxiosError: Request failed with status code 403] {
  code: 'ERR_BAD_REQUEST',
  config: {
    transitional: {
      silentJSONParsing: true,
      forcedJSONParsing: true,
      clarifyTimeoutError: false
    },
    adapter: [Function: httpAdapter],
    transformRequest: [ [Function: transformRequest] ],
    transformResponse: [ [Function: transformResponse] ],
    timeout: 0,
    xsrfCookieName: 'XSRF-TOKEN',
    xsrfHeaderName: 'X-XSRF-TOKEN',
    maxContentLength: -1,
    maxBodyLength: -1,
    env: { FormData: [Function] },
    validateStatus: [Function: validateStatus],
    headers: {
      Accept: 'application/json, text/plain, */*',
      authority: 'users.premierleague.com',
      'cache-control': 'max-age=0',
      'upgrade-insecure-requests': '1',
      origin: 'https://fantasy.premierleague.com/',
      'user-agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36',
      'sec-fetch-site': 'same-site',
      'sec-fetch-mode'<…>

👋 @pernielsentikaer
Let me take a look at this, seems to be an error with the API the extension uses to fetch the fantasy premier league data.
All the linting errors are sorted, the test error that currently shows seems to be when the extension was going to be built and added to the store but another extension visual-studio-code-recent-projects that was being updated at the same time had a build failure for No loader is configured for ".wasm" files which is why it looks like the build failed on this extension.
I'll get back to you shortly!

Hi @pernielsentikaer !
I'm happy to reopen this now, and feel the extension is ready again for store - I have worked on sorting the issues with 403 responses to requests and have fixed it up both through the extension and also a few changes on the backend.
RE: linting errors, explanation above from the build failure message is in regards to another extension so this shouldn't cause any issues w/ my own extension.
Thank you 🙏


[Snyk] Security upgrade web-ext from 4.3.0 to 6.6.0

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

packages/react-devtools-extensions/package.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Regular Expression Denial of Service (ReDoS) SNYK-JS-ASYNC-7414156
  50

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

⛔ Snyk has found 3 issues.

Icon
Severity
Issues

Critical
0

High
0

Medium
3

Low
0

⛔ security/snyk check is completed. Snyk found 1 issues. (View Details)
⛔ license/snyk check is completed. Snyk found 2 issues. (View Details)


Online examples

Good news! readthedocs.org now has numpy, scipy, and matplotlib installed!
Not only does this mean that we don't need to bother with the "MOCK_MODULES" nonsense, but it also means that we can include examples and have the plots show up! Check them out:
http://pyabel-dh-fork.readthedocs.org/en/latest/example_direct_gaussian.html
http://pyabel-dh-fork.readthedocs.org/en/latest/example_hansenlaw.html
http://pyabel-dh-fork.readthedocs.org/en/latest/example_O2_PES_PAD.html
http://pyabel-dh-fork.readthedocs.org/en/latest/example_hansenlaw_Xe.html
The one catch is that the examples need to actually run each time we build the docs. So, if we want to feature an example in the documentation, it needs to run pretty fast. Currently, it's only really the basex examples that take a long time. So, we could just run these with smaller images if we want to include them with the docs.

That's really good news! Thanks for checking that up.

Some of the examples are now part of the online docs. I'm going to declare victory on this issue and close it.


BackDrops:set_files loading .DS_Store as image file (blank background)

Once I've deleted two or more backgrounds from the backgrounds folder, I get a bland background added to the cycle. So, as I cycle to the next image, I get the normal backgrounds in the folder but then an additional blank one. It's as if there is a blank background image in the folder.
This happens on a fresh install of the configuration too. If I put the deleted backgrounds back that doesn't get rid of the problem. I can add images to the background folder just fine, but once I've deleted two or more then the blank shows up.
To get the backgrounds I wanted I had to create a new empty backdrop folder and then add the images in.

Hey @username-adk, I've been trying to reproduce the issue for a while now with no luck.
Could you tell me which OS and version of wezterm you're running?
wezterm --version

wezterm 20240203-110809-5046fc22
macOS 15.0.1 (24A348)
I only noticed this problem after updating to Sequoia 15.0.1 but I don't know if I just missed it before.
I tried to recreate the problem again and it's still there.

Backed up .config/wezterm then deleted it.
Deleted the folder in .local/share.
Brew uninstall wezterm.
Brew install wezterm.
Git clone your repo.
Opened up wezterm and the backgrounds work fine.
Closed it, deleted one pic from .config/wezterm/backdrops, opened it again and the backgrounds work
fine (with deleted one not showing up).
Did the same with a second pic and now a blank background shows up.

I think what really throws me is that if I put the deleted pics back the problem persists.
Anyway my normal config doesn't have this problem. I love yours so it's just a copy with a couple changes.
I fixed it by adding an empty backdrops folder and then only adding the images I wanted.

From what you've described, what I think is happening is somehow the BackDrops:set_files() is finding files that aren't supposed to exist in the backdrops directory?
Cuz I was able to replicate the blank background image while cycling, but only after I hard-coded a file path to an image that doesn't exist:
function BackDrops:set_files()
   self.files = wezterm.read_dir(wezterm.config_dir .. PATH_SEP .. 'backdrops')
   table.insert(self.files, 'C:\\Users\\kevin\\.config\\wezterm\\backdrops\\i-dont-exist.jpg')
   wezterm.GLOBAL.background = self.files[1]
   return self
end

So what I'm wondering now is how you deleted the files.
Did you use the command line to rm image.jpg or did you use Finder?
This is just a guess on my part, but if you did use Finder and you're getting this issue, I think what may have happened is in the OS update you just got, the way Finder handles deleting files may have changed (maybe for restoring deleted items?).
If so, try deleting the image you no longer want in the command line.
On the other hand, if you did use the command line and are getting the issue, I am stumped.
Not a clue how this happened.
That being said, it might still be worth checking out the nightly build, since this might be an issue that is already patched.
brew install --cask wezterm@nightly

You could also check if BackDrops:set_files really is finding non-existent files by:

Press F12 to enter debug mode
Paste the following line in the debugger to dump the current BackDrops staterequire('utils.backdrops').files

You should get something like this:

If neither the nightly version nor deleting with the command line don't work, it might be worth filing an issue to wezterm.

You've got it! Deleting using the command line doesn't raise this issue.
Following your debug mode steps I got:

require('utils.backdrops').files

[
"/Users/anthony/.config/wezterm/backdrops/.DS_Store",
"/Users/anthony/.config/wezterm/backdrops/final-showdown.jpg",
"/Users/anthony/.config/wezterm/backdrops/pastel-samurai.jpg",
"/Users/anthony/.config/wezterm/backdrops/frieren.jpeg",
]
So it looks like .DS_Store is being picked up as an additional background.

Nice!
I forgot that .DS_Store has a tendency to slip into some unwanted places 😂.
I'll switch over to wezterm.glob instead of read_dir to load only the supported image files instead so you won't have to keep deleting the file.
Thanks for finding the bug 😎

And thanks for your config!
wezterm.glob worked nicely. In BackDrops:set_files
I just changed the line:

self.files = wezterm.read_dir(wezterm.config_dir .. PATH_SEP .. 'backdrops')

to

self.files = wezterm.glob(wezterm.config_dir .. PATH_SEP .. 'backdrops/*')

and now I can remove backgrounds without any issues.

And thanks for your config!
wezterm.glob worked nicely. In BackDrops:set_files I just changed the line:

self.files = wezterm.read_dir(wezterm.config_dir .. PATH_SEP .. 'backdrops')

to

self.files = wezterm.glob(wezterm.config_dir .. PATH_SEP .. 'backdrops/*')

and now I can remove backgrounds without any issues.

Nice!
You might want to add the glob pattern to only select the supported image files though:
self.files = wezterm.glob(wezterm.config_dir .. PATH_SEP .. 'backdrops/*.{jpg,jpeg,png,gif,bmp,ico,tiff,pnm,dds,tga}')


variant type is obsolete

back-link: ch04/index.html#quick-starter
The official documentation states that the variant type is obsolete and that var should be used instead.

Correct. Needs to be fixed


Feature pybullet

Continuation of incomplete PR from https://github.com/facebookresearch/mbrl-lib/pull/87
This is my first time contributing to an open-source project so any advice is welcome, technical or otherwise
Types of changes

[x] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue

This adds support for PyBullet, an open-source alternative to MuJoCo. MuJoCo-compatible and RobotSchool environments are supported via pybullet-gym.
How Has This Been Tested (if it applies)

python -m pytest tests/pybullet
Checklist

[ ] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CONTRIBUTING).
[ ] All tests passed, and additional code has been covered with new tests.

I refactored the pybullet-relevant code into a separate mujoco/util/pybullet.py as suggested in #87 .
Have verified that tests/pybullet/test_util.py works.
Currently working on making tests/pybullet/test_diagnostics.py tests work.

I need some help with understanding the config dictionary setup in tests/{mujoco,pybullet}/test_diagnostics.py.
Currently I get the following error in my tests:
FAILED tests/pybullet/test_diagnostics.py::test_eval_on_dataset - omegaconf.errors.ConfigAttributeError: Missing key seed
FAILED tests/pybullet/test_diagnostics.py::test_finetuner - omegaconf.errors.ConfigAttributeError: Missing key seed
FAILED tests/pybullet/test_diagnostics.py::test_visualizer - omegaconf.errors.ConfigAttributeError: Missing key seed

I made that the config dictionary setup in  tests/pybullet/test_diagnostics.py mirrors that used in  tests/mujoco/test_diagnostics.py. I am unable to verify whether this issue also exists in  tests/mujoco/test_diagnostics.py because I do not have a Mujoco license.
Appreciate help with getting this to work correctly.

I resolved the missing seed error (albeit rather hackily.)
The next problem is that mbrl/diagnostics is hardcoded to use mbrl.util.mujoco functions which doesn't work well with the new pybullet code. I intend to solve this by creating an abstract class for gym env types and subclassing to get polymorphic behaviour.

Commit 6512c94  adds a proposal for a general interface to gym backends. Instead of having if-else statements in every function, we can have a single if-else statement that constructs a handler which implements polymorphic behaviour.

Ah, this is normal. This just means that black changed some of your
formatting automatically and you have to stage some of your files again.
On Sat, Oct 2, 2021, 10:28 AM Daniel Tan @.***> wrote:

@.**** commented on this pull request.
In mbrl/util/pybullet.py
https://github.com/facebookresearch/mbrl-lib/pull/135#discussion_r720686143
:

       self._exit_method = self._exit_pybullet_gym

   else:

       raise RuntimeError("Tried to freeze an unsupported environment.")

def _enter_pybullet_gym(self):

   # For now, the accepted envs are limited to ease implementation and testing

   from pybulletgym.envs.roboschool.robots.locomotors.walker_base import  WalkerBase as RSWalkerBase

   from pybulletgym.envs.mujoco.robots.locomotors.walker_base import WalkerBase as MJWalkerBase

   env = self._env.env

   robot = env.robot

   assert isinstance(robot, (RSWalkerBase, MJWalkerBase))

   self.state_id = env._p.saveState()

   self.ground_ids = env.ground_ids

   self.potential = env.potential

   self.reward = float(env.reward)

   robot_keys = [("body_rpy", tuple), ("body_xyz", tuple), ("feet_contact", np.copy), ("initial_z", float), ("joint_speeds", np.copy), ("joints_at_limit", int), ("walk_target_dist", float), ("walk_target_theta", float), ("walk_target_x", float), ("walk_target_y", float)]

I fixed the above by switching my development environment to python 3.7.
Now I'm struggling to understand why this happens with black:
(mbrl-lib-py37) @.***:~/code/mbrl-lib$ git commit
[WARNING] Unstaged files detected.
[INFO] Stashing unstaged files to /home/dtch1997/.cache/pre-commit/patch1633184818-865.
black....................................................................Failed

hook id: black
files were modified by this hook

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/mbrl-lib/pull/135#discussion_r720686143,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEKAIRMV3YA7UR2N3O52EDUE4JJZANCNFSM5FGH3DOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I've implemented PybulletEnvHandler and MujocoEnvHandler based on the agreed interface.
PybulletEnvHandler has been tested with tests/pybullet/test_util.py.
MujocoEnvHandler has not been tested at all.
TODO:

[ ] Implement DmControlEnvHandler
[ ] Implement GymEnvHandler
[ ] Refactor mbrl.diagnostics to use handlers

[ ] DatasetEvaluator
[ ] FineTuner
[ ] Visualizer

After thinking about it, I think make_env can be detached from EnvHandler since it is not engine-dependent. Then make_env can be modified to also return an EnvHandler of the appropriate type.

@luisenp would be helpful to get your input on the progress thus far and the suggested next steps

Hi @dtch1997, sorry for the delay, I'll take a look at this today.

@luisenp good to hear back from you!
This is what I got when I ran pre-commit:
(mbrl-lib) dtch1997@DESKTOP-AR4R24K:~/code/mbrl-lib$ pre-commit run --all-files
black....................................................................Passed
flake8...................................................................Passed
mypy.....................................................................Failed
- hook id: mypy
- exit code: 2

tests/pybullet/test_util.py: error: Duplicate module named 'test_util' (also at 'tests/mujoco/test_util.py')
tests/pybullet/test_util.py: note: Are you missing an __init__.py? Alternatively, consider using --exclude to avoid checking one of them.
Found 1 error in 1 file (errors prevented further checking)

isort....................................................................Passed
isort....................................................................Passed

I fixed the flake8 error by deleting the problematic file. The relevant code has been fully refactored into pybullet_handler.py anyhow.
The mypy error seems more difficult to resolve since it seems to be complaining that mujoco and pybullet both have the same internal directory structure. I could fix it by 1) excluding the directory from mypy or 2) changing the file name but neither seem ideal. I found a relevant Git issue here: https://github.com/pre-commit/mirrors-mypy/issues/5

Also, my tests/pybullet failed with
FAILED tests/pybullet/test_diagnostics.py::test_visualizer - RuntimeError: Tried to freeze an unsupported environment.

Yes this doesn't work yet. I need to refactor the mbrl.diagnostics module to use the new handlers, which requires instantiating the handler from a cfg object.

Now that I think about it, for PyBullet env_cfg.target will probably look something like pybulletgym.envs.something.something, no?

Yup you're exactly right, that's how I've implemented it currently

@luisenp good to hear back from you!
This is what I got when I ran pre-commit:
(mbrl-lib) dtch1997@DESKTOP-AR4R24K:~/code/mbrl-lib$ pre-commit run --all-files
black....................................................................Passed
flake8...................................................................Passed
mypy.....................................................................Failed
- hook id: mypy
- exit code: 2

tests/pybullet/test_util.py: error: Duplicate module named 'test_util' (also at 'tests/mujoco/test_util.py')
tests/pybullet/test_util.py: note: Are you missing an __init__.py? Alternatively, consider using --exclude to avoid checking one of them.
Found 1 error in 1 file (errors prevented further checking)

isort....................................................................Passed
isort....................................................................Passed

I fixed the flake8 error by deleting the problematic file. The relevant code has been fully refactored into pybullet_handler.py anyhow.
The mypy error seems more difficult to resolve since it seems to be complaining that mujoco and pybullet both have the same internal directory structure. I could fix it by 1) excluding the directory from mypy or 2) changing the file name but neither seem ideal. I found a relevant Git issue here: pre-commit/mirrors-mypy#5

I'm a bit more lenient when it comes to test code conventions, so changing the file name wouldn't be too bad. On the other hand, it seems that adding __init__.py to both test folders resolves the mypy issue.

On the other hand, it seems that adding init.py to both test folders resolves the mypy issue.
Oh that's neat! Implemented

All the tests in tests/pybullet should work now. All the linter checks should also pass

Hi @dtch1997, thanks for the update! I just took a quick look, and given the line where the error is thrown, it's more likely that the missing reset would be an issue for the environments used for planning, a copy of which each process would have to use for planning, and not in eval_env, which is the environment that lives in the main thread were the actions are being executed.
If I add the line env__.reset()at the end of function init() (line 31), the error changes to
WalkerBase::__init__
argv[0]=
argv[0]=
pybullet build time: Jul  9 2021 21:27:33
/private/home/lep/.conda/envs/mbrl/lib/python3.7/site-packages/gym/logger.py:30: UserWarning: WARN: Box bound precision lowered by casting to float32
  warnings.warn(colorize('%s: %s'%('WARN', msg % args), 'yellow'))
WalkerBase::__init__
argv[0]=
argv[0]=
b3Warning[examples/SharedMemory/PhysicsDirect.cpp,1204]:
restoreState failedb3Warning[examples/SharedMemory/PhysicsDirect.cpp,1204]:
restoreState failedmultiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/private/home/lep/.conda/envs/mbrl/lib/python3.7/multiprocessing/pool.py", line 121, in worker
    result = (True, func(*args, **kwds))
  File "/private/home/lep/code/mbrl/mbrl/diagnostics/control_env.py", line 59, in evaluate_sequence_fn
    handler__.set_env_state(current_state, env)
  File "/private/home/lep/code/mbrl/mbrl/util/pybullet.py", line 180, in set_env_state
    return PybulletEnvHandler._set_env_state_locomotion(state, env)
  File "/private/home/lep/code/mbrl/mbrl/util/pybullet.py", line 211, in _set_env_state_locomotion
    env._p.restoreState(state_id)
pybullet.error: Couldn't restore state.
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "mbrl/diagnostics/control_env.py", line 185, in <module>
    trajectory_eval_fn, callback=compute_population_stats
  File "/private/home/lep/code/mbrl/mbrl/planning/trajectory_opt.py", line 561, in optimize
    callback=callback,
  File "/private/home/lep/code/mbrl/mbrl/planning/trajectory_opt.py", line 172, in optimize
    values = obj_fun(population)
  File "mbrl/diagnostics/control_env.py", line 174, in trajectory_eval_fn
    current_state__,
  File "mbrl/diagnostics/control_env.py", line 49, in evaluate_all_action_sequences
    res = [res_obj.get() for res_obj in res_objs]
  File "mbrl/diagnostics/control_env.py", line 49, in <listcomp>
    res = [res_obj.get() for res_obj in res_objs]
  File "/private/home/lep/.conda/envs/mbrl/lib/python3.7/multiprocessing/pool.py", line 657, in get
    raise self._value
pybullet.error: Couldn't restore state.

@luisenp thanks for asking! I've been quite pressed for time recently, but I am continuing to work on this. I will keep you posted when I make progress or need help

@luisenp can you point me to a version of dm_control that works with Mujoco 2.0.x? The latest version on the Github repository is for Mujoco 2.1.0, which is incompatible with mujoco_py at the moment.
On a side note: DeepMind has recently acquired Mujoco and made it (freely available)[https://deepmind.com/blog/announcements/mujoco]. Going forward it is still unclear whether mujoco_py will be supported: https://github.com/openai/mujoco-py/issues/642. I am of the personal opinion that dm_control will become the preferred way to access Mujoco envs. Let me know if you have opinions on what you'd like to support moving forward

mmm, actually, I don't think there is, now that I think about it. Are you able to do the dm_control part using 2.1.0 (if you haven't already)? Then, once you have that working, for mujoco-py you can try to put things more or less in place w/o testing, then I can run on my side and fix any remaining errors. Is that feasible?

Awesome, thanks a lot for your hard work!

Hi @dtch1997. I'm really sorry that is taking me so long to review this, but my other commitments are taking all of my time. It might be hard for me to take a look at this until mid-November, but rest assured that this is a really nice contribution, and I'm looking forward to merge it in.

@luisenp let me know if there's anything else I need to do for this PR.

Hi @dtch1997, thanks for reaching out and apologies again for the lack of input on my part. My other commitments should clear out by next week, so you can expect some final comments to merge this in from me by Wednesday 30 or so.

Hi @dtch1997, PR merged! Let me know if you are still interested in the followups we had discussed and we can discuss a bit a plan for it. Thanks again for the contribution :)


Update README.md

First edition of the general readme for GuideLLM - WIP.

Added drafts of README and GuideLLM CLI guide, still more to do, but got a rough outline in place. we can also discuss again what should live where in regards to the Using GuideLLM sections and such.


⚠ Spryr has degraded performance

In ceeab39, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3003 ms

Resolved: Spryr performance has improved in 53cf884 after 8 minutes.


Fix switching between two tabs when clicking on a tab in the tabbar. …

…r=linclark
Signed-off-by: Jordan Santell<EMAIL_ADDRESS>

Coverage remained the same at 41.225% when pulling 2162d295a82283cd0f1a33350f3e84e25e8f3061 on jsantell:fix-tab-switching into 88800aa2ec5793f3f1ba514cff7857361a0091f5 on mozilla:master.

works for me

Coverage decreased (-0.08%) to 41.15% when pulling b22ce09be6e9f640fe6f0a47a733b5b1c47828f3 on jsantell:fix-tab-switching into 88800aa2ec5793f3f1ba514cff7857361a0091f5 on mozilla:master.


MSBot Azure services deployed in default region rather then specified location

Tool
Name: Msbot
Version: 4.3.4
OS: Windows 10
Describe the bug
Some of the services are deployed into a default region rather than the location specified in the --location parameter even though they are available in that region.
To Reproduce
Steps to reproduce the behavior:

Deploy bot to Australia eg --australiaeast
Some resources are deployed to australiaeast but other eg search and appinsights are deployed to souteast asia

Expected behavior
All resources should be deployed into the region specified in the --location parameter if the service is available there
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
it looks like the utils.ts class has out of date both in terms of region service availability and service to region mapping for example
export const regionToSearchRegionMap: { [region: string]: string } = {
australiaeast: RegionCodes.SOUTHEASTASIA,
australiacentral: RegionCodes.SOUTHEASTASIA,
australiacentral2: RegionCodes.SOUTHEASTASIA,
australiasoutheast: RegionCodes.SOUTHEASTASIA,
[bug]

Hi @stevengum , PR submitted by @toddwhitehead is a partial solution to this issue.
There are some changes needed on az bot create to fix this issue. Otherwise this error is thrown in msbot: ERROR: az bot create: 'Australia East' is not a valid value for '--insights-location'. See 'az bot create --help'.
Can you please take a look?

Since we are deprecating msbot as a provisioning and deployment option, replaced with ARM template support, I'm going to resolve this issue as "won't fix"


Settings - Add Palettey Color Scheme

The settings page should contain an option to pick a color which would be used to generate a 10 color "palette" used in the site.

[x] Add palettey functionality
[x] Save color settings to localStorage
[x] Add default color
[x] Toggle white/black text depending on the color contrast

Repo that generates tailwind like color palettes: https://github.com/bartbergmans/Palettey
Repo that generates color pairs based on contrast: https://github.com/KyleAMathews/color-pairs-picker

Workflow

User Loads Page
Check localStorage if colorSettings are present
a) Generate settings from default if not present
b) Load existing settings if present
Inject color data into dom? ( I am not sure how this step works yet )

Feature is complete.


HomeWindow: small cleanup

@adeebshihadeh OffroadHome will be refreshed on showEvent. It seems that this connection is unnecessary,am I right? In addition, I don’t understand why we need to refresh alerts when open settings

Yes, the refresh on openSettings is unnecessary.


Coolest WD feature ever :)

🛠 Fixes Issue (Number)

👨💻 Changes proposed and Brief Description

✅ Check List (Check all the applicable boxes)

[ ] My code doesn't break any part of the project
[ ] This PR does not contain plagiarized content.
[ ] My Addition/Changes works properly and matches the overall repo pattern.
[ ] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

test this please


Fixed-#36 - Updated requirements_dev.txt: Added hypothesis

Issue link https://github.com/audreyr/binaryornot/issues/36


User Scenario Documentation Typo

'You know your app well, but other users Microsoft does not'
should be
'You know your app well, but other Microsoft users do not'
(?)

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

ID: 5c32e452-ab94-4557-4062-feba4951c112
Version Independent ID: 421e93e8-857d-4f1d-c433-afc9910f8c2f
Content: User Scenario Documentation - Business Central
Content Source: dev-itpro/compliance/apptest-userscenario.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: freddyk

Thanks for catching this. It will be fixed with the next refresh of the docs.


RuntimeException during installation of composer

We use deployphp to deploy projects to our servers but sinds a little while composer changed there output from writing lines to writing error's in some cases but the installation is still succesvol. So there is an open issue at deployphp https://github.com/deployphp/deployer/issues/201 library that try's to fix this and they solved it for PhpSecLib server implementation but they also use this library to connect to servers.
In conclusion is it possible to check the output result on exit code like deployphp does for PhpSecLib instead of checking the error output?

https://github.com/Herzult/php-ssh/blob/master/src/Ssh/Exec.php#L26

More details

https://github.com/deployphp/deployer/issues/201
https://github.com/deployphp/deployer/pull/199
https://github.com/composer/composer/issues/1905#issuecomment-41631659

To avoid confusion here:
You are using php-ssh to connect to a server via SSH and run composer over there.
So there there is no problem installing php-ssh via composer?
And the cause for your ticket is, that composer does some "... older 30 Days build ..." output on STDERR, which is interpreted by this library as a Error and then throws a RuntimeException?

You are using php-ssh to connect to a server via SSH and run composer over there.
So there there is no problem installing php-ssh via composer?

Yes

And the cause for your ticket is, that composer does some "... older 30 Days build ..." output on STDERR, which is interpreted by this library as a Error and then throws a RuntimeException?

Yes :)

Is it possible to check return code via php ssh ext?

I checked the SSH2 PHP Documentation and could not find anything describing a way to receive exit codes of executed commands.
I checked:

http://php.net/manual/de/function.ssh2-exec.php
http://php.net/manual/de/function.ssh2-shell.php

The only idea i have is something like this:
Appending another command that will output the exit code of the previous command ssh2_exec($connection, $cmd.' ; echo "PHP_SSH_EXIT_CODE="$?');.
Some example implementation:
<?php
$username = 'julius';
$password = '';
$server = 'localhost';
$serverPort = 22;

$connection = ssh2_connect($server, $serverPort);
ssh2_auth_password($connection, $username, $password);

executeCommand($connection, 'cd /etc');
executeCommand($connection, 'cd /foo');

function executeCommand($connection, $cmd) {
  echo "\n--- Executing: $cmd \n";

  $stream = ssh2_exec($connection, $cmd.' ; echo "PHP_SSH_EXIT_CODE="$?');
  $errorStream = ssh2_fetch_stream($stream, SSH2_STREAM_STDERR);

  stream_set_blocking($errorStream, true);
  stream_set_blocking($stream, true);

  $out = stream_get_contents($stream);
  $err = stream_get_contents($errorStream);

  $exitCode = 0;
  if(preg_match('@PHP_SSH_EXIT_CODE=([0-9]+)@', $out, $matches)) {
    $exitCode = (int)$matches[1];
    $out = str_replace($matches[0], '', $out);
  }

  echo "STDOUT: " . trim($out)."\n";
  echo "STDERR: " . trim($err)."\n";
  echo "ExitCode: ".$exitCode."\n";

  fclose($errorStream);
  fclose($stream);
}

Looks like only solution.

@Herzult Might that be a solution we could try?
It would need to modify the Exec::run signature or introducing a second method.

Solved with: https://github.com/Herzult/php-ssh/issues/43
Contained in release: v1.1.1


Local repo syntaxes

Describe the enhancement
The thing is that, for people how want to create a programming language, and upload their code on github can't see their code with a highlighted syntax.
So I propose a local repository syntax :
In the directory .github/syntaxes/, you put your textmate grammar file. And when a file with the extension (specified in the grammar) is open, their is highlighting thanks to the file in .github/syntaxes/.
e.g of a tree with the local syntax :
$ tree
.
|____*Some code files ...*
|____...
|____.github
| |____syntaxes
| | |____MyProgrammingLanguage.tmLanguage.json
|____LICENSE

Or with a .gitattributes, like that :
*.myopl linguist-syntax https://github.com/[username / organization]/myopl-grammar

And in the repo where the grammar is, why not have a special file like github-linguist.json where you can find the path to the text mate grammar and some useful data, like the display name, the color etc...

This isn't really the right place for this as the syntax highlighting engine is completely independent of Linguist.
GitHub's syntax highlighting engine is a lightweight, self-contained micro-service which uses cached and consistently transformed grammar files (see the linguist-grammars.tgz attachment on each release) and only considers one file in a repo at a time. In order to implement this, it would require the service to read into every single repo to look for the grammar definition, transform it into the form it requires, handle errors etc, and then adjust the rendering. This will be costly in terms of resources, duplicates work linguist's "compiler" does, and would over complicate what is a simple service that works really well.
So whilst a nice idea, it's almost certainly never going to happen, so I'm closing this issue.


Removed underscore from GraphEdit begin/end_node_move signals

There is no other signal across the engine with an underscore prefix - the only exception is GraphEdit. I think this should be fixed.

Thanks!


[SPARK-34180][SQL] Revert SPARK-33888

What changes were proposed in this pull request?

This PR reverts SPARK-33888 (#30902) .
That PR has the two serious problems.

A regression which affects PostgresDialect causing PostgreSQLIntegrationSuite to fail.
That PR changes the JDBC-Catalyst type mapping. sql.Types.TIME is mapped to sql.types.IntegerType so we'll get millisecond value. But the start time point is different between MsSqlServer and other RDBMSs. For MsSqlServer, it starts from 1900-01-01 00:00:00 while 1970-01-01 00:00:00 for other RDBMS.

More about problem-2, the following assertion passes before SPARK-33888.
// This is from jdbc.MsSqlServerIntegrationSuite
assert(row.getAs[Timestamp](5).equals(Timestamp.valueOf("1900-01-01 13:31:24.0")))

But after SPARK-33888, the following modified assertion fails because the millisecond value is considered starting from 1970-01-01 00:00:00.
assert(row.getAs[Integer](5).equals(Timestamp.valueOf("1900-01-01 13:31:24.0").getTime.toInt))

Why are the changes needed?

To fix the serious issues.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

This is just reverting a previous change.

What do you mean by "start time point"? The TIME type has no date part and should return seconds(with certain precision) of the hour:minute:second fields.

What do you mean by "start time point"? The TIME type has no date part and should return seconds(with certain precision) of the hour:minute:second fields.

@cloud-fan

What do you mean by "start time point"? The TIME type has no date part and should return seconds(with certain precision) of the hour:minute:second fields.

Yes it has no date part. So in MsSqlServer, the date part is assumed 1900-01-01 00:00:00 while other RDBMS assumes it as 1970-01-01 00:00:00.
https://docs.microsoft.com/ja-jp/sql/t-sql/data-types/time-transact-sql?view=sql-server-ver15

@cloud-fan

What do you mean by "start time point"? The TIME type has no date part and should return seconds(with certain precision) of the hour:minute:second fields.

Yes it has no date part. So in MsSqlServer, the date part is assumed 1900-01-01 00:00:00 while other RDBMS assumes it as 1970-01-01 00:00:00.
https://docs.microsoft.com/ja-jp/sql/t-sql/data-types/time-transact-sql?view=sql-server-ver15

@cloud-fan
Or, do you think the assert(row.getAs[Timestamp](5).equals(Timestamp.valueOf("1900-01-01 13:31:24.0"))) is wrong because it contains date part?

@cloud-fan
Or, do you think the assert(row.getAs[Timestamp](5).equals(Timestamp.valueOf("1900-01-01 13:31:24.0"))) is wrong because it contains date part?

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38890/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38890/

@sarutak can you take a look at https://github.com/apache/spark/pull/30902/files#r561601546 ? I think the test is wrong.

@sarutak can you take a look at https://github.com/apache/spark/pull/30902/files#r561601546 ? I think the test is wrong.

@cloud-fan O.K, I'll close this PR and open another PR to fix the wrong assertion. Thanks.

@cloud-fan O.K, I'll close this PR and open another PR to fix the wrong assertion. Thanks.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38890/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38890/

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

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


Global Configuration

It would be really nice if we could specify global settings. For instance, if you want all of your switches to have labels, or globally change the labels to something else.
Thank you very much for the plugin.

Hey thanks for a question.
You can create a wrapper with default params.
It seems to me that it's easily solvable problem and should not be a part of the package.
Links to read:
https://vuejs.org/v2/api/#vm-attrs
https://vuejs.org/v2/api/#vm-listeners

Fair enough, thank you


fix(#904): fix environment save when empty secret exists

Description
Closes #904
Allows empty value for an environment variable(useful as user might want to create variables and assign values later)

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request.

Thanks for taking care of this @adarshlilha !


Add test cases and adjust title for User/Developer Guide

What

Adds 100% line coverage for the DeleteCommand and InterviewCommand`.
Refactors Storage classes for HMHero
Edit Site Header for User Guide and Developer Guide

LGTM


Verify if a file exists as a policy

Hello!
I'd like to know if it is possible to create a policy that will verify the existence of a specific file inside a repository?
Thanks in advance,
Matias

Hi @matias-jt,
Yes, that's doable if you duplicate the security.go check and replace SECURITY.md variable in polName.

Possibly a duplicate of #450, although that one is more specific to the SECURITY.md check.

Though #500 was created later, there is more design work there, so closing this as duplicate of that.


Duplicate key

In data/controls.csv, this:
{"source":"cis_csc_v7.1","id":"cis_csc_v7.1:12.10","id_raw":"12.1","tier_raw":"Sub-Control","tier":"1","seq":"84","title":"Decrypt Network Traffic at Proxy","description":"Decrypt all encrypted network traffic at the boundary proxy prior to analyzing the content. However, the organization may use whitelists of allowed sites that can be accessed through the proxy without decrypting the traffic."},
Should be:
{"source":"cis_csc_v7.1","id":"cis_csc_v7.1:12.10","id_raw":"12.10","tier_raw":"Sub-Control","tier":"1","seq":"84","title":"Decrypt Network Traffic at Proxy","description":"Decrypt all encrypted network traffic at the boundary proxy prior to analyzing the content. However, the organization may use whitelists of allowed sites that can be accessed through the proxy without decrypting the traffic."},
-Jarrod

Quite right, Jarrod, thank you for filing the issue!  Found a few others that were missing trailing zeroes as well.  Fixed in 2c16691.

Quite right, Jarrod, thank you for filing the issue! Found a few others that were missing trailing zeroes as well. Fixed in 2c16691.

Also:
{"source":"cis_csc_v7.1","id":"cis_csc_v7.1:7.10","id_raw":"7.1","tier_raw":"Sub-Control","tier":"1","seq":"49","title":"Sandbox All Email Attachments","description":"Use sandboxing to analyze and block inbound email attachments with malicious behavior."},
Should be:
{"source":"cis_csc_v7.1","id":"cis_csc_v7.1:7.10","id_raw":"7.1","tier_raw":"Sub-Control","tier":"1","seq":"49","title":"Sandbox All Email Attachments","description":"Use sandboxing to analyze and block inbound email attachments with malicious behavior."},

Yessir - there were 6 others I found and fixed, the one you just mentioned shows corrected here in the CSV and here in the JSON.  Thanks again!


Regression for object browser keys

@keithamus
#127 has introduced regression for object browser keys. From what I see they are kinda ignored.
Example:
const importer = '/Users/mateuszburzynski/Desktop/node-uuid/src/v1.js'
const importee = './rng'
const options = { browser: true }

// what gets passed in as arg for packageFilter
const pkg = {"name":"uuid","version":"3.1.0","description":"RFC4122 (v1, v4, and v5) UUIDs","keywords":["uuid","guid","rfc4122"],"license":"MIT","bin":{"uuid":"./bin/uuid"},"devDependencies":{"babel-cli":"^6.26.0","babel-plugin-add-module-exports":"^0.2.1","babel-plugin-transform-es2015-modules-commonjs":"^6.26.0","eslint":"4.5.0","mocha":"3.1.2","runmd":"^1.0.1"},"scripts":{"build":"babel src --out-dir lib","test":"mocha test/test.js","md":"runmd --watch --output=README.md README_js.md","prepare":"npm run build && runmd --output=README.md README_js.md"},"browser":{"./lib/rng.js":"./lib/rng-browser.js","./lib/sha1.js":"./lib/sha1-browser.js","./lib/md5.js":"./lib/md5-browser.js","./src/rng.js":"./src/rng-browser.js","./src/sha1.js":"./src/sha1-browser.js","./src/md5.js":"./src/md5-browser.js"},"repository":{"type":"git","url":"https://github.com/kelektiv/node-uuid.git"},"dependencies":{}}
packageBrowserField {"./lib/rng.js":"./lib/rng-browser.js","./lib/sha1.js":"./lib/sha1-browser.js","./lib/md5.js":"./lib/md5-browser.js","./src/rng.js":"./src/rng-browser.js","./src/sha1.js":"./src/sha1-browser.js","./src/md5.js":"./src/md5-browser.js"}

// result of resolveId call
const resolved = '/Users/mateuszburzynski/Desktop/node-uuid/src/rng.js'
// and this actually should be '/Users/mateuszburzynski/Desktop/node-uuid/src/rng-browser.js'

resolveId has no knowledge of object browser key - it accept only simple strings in main, so it resolves the whole thing using a simple node algo and the result is not adjusted - especially that importee path was relative to the importer directory and browser field specifies overrides relative to the package root.

I believe this to be fixed. So I'm going to close this. If you think its an issue still, please comment and I'll re-open.


Let benchmarks perform multiple runs internally

I'd like to try out Benchpress, but for my purposes it's awkward (=slow) to start up each benchmark several times.  I'd much rather Benchpress just ask it to run N times, and get N measurements back.  Would this be a feasible extension?

Fixed in e684a4d582eb2442c277bdcc5b674735a9fb2930, which make all visualizers use all matches of --parse-regex.
The commands within your suite file can then run each benchmark multiple times.


Added exponential backoff waiting for AWS changeset to stabilize

We appear to be the only stacker users experiencing long wait periods on CFN changeset generation, but we've seen some stacks esp. our VPN take up to a minute to stabilize. This backoff is meant to function very similarly to the retry_on_throttling

This looks good, thanks - one thing I think it needs is more logging, especially if it's going to hang for up to a minute.  Maybe post a logger.debug message saying we're waiting on the change set still every 10 retries?

Yep that seems reasonable. I switched it to logging every time it's waiting the maximum amount; that'll give frequent feedback once it's looping longer than should be expected. What do you think?

Just hit this for the first time today. By the time I went into the console to look at the error it had stabilized, so I suspect this would be helpful for us too.

Man, sorry - I've been swamped and meant to merge this.  It's in now.  Thanks for the ping @troyready, and thanks @smverleye for the PR!


Compiling Flutter Web apps in a better way

Use case
Make so that when I compile my Flutter website that each page/route is compiled in its own and different .js file instead of the entire code all in 1 single .js file and having the user download the entire website. I am aware that I can use deferred loading to load imports only when they are actually needed, but that's different than loading pages. I am also aware the this is helpful to provide the PWA functionality but it shouldn't take a long time for the user to see the actual website content while Flutter starts up and starts loading all its resources which takes quite some time every time I visit a Flutter website.
Proposal
Compile each page that is provided in the routes property for the MaterialApp in main.dart to its separate .js file and when the user goes to a specific route, the resources for that route are loaded. It would also be useful if package/imports are loaded only when they are actually being used instead of loading the entire library regardless if it is being actually used, example:
I have a function that I call when a user clicks on a button:
import 'some_library.dart' as some_library;

Future<void> trigger() async {
  // Makes use of the `some_library` import. This import is only being used in this function.
}

And then I have this button:
SomeButton(
  text: 'Button',
  onPressed: trigger,
),

Now that trigger function is the only place that is using the some_library import. If the user doesn't click on the button, the library should never be loaded. Once there is a place where it calls to use the some_library then it should actually load the library.
It would be great for Flutter web to expand and go beyond serving single-page web apps.

I think there is already something similar for your proposal in flutter.
Take a look at this article Only load code when needed with deferred loading

Yes, I am aware of that.
However, my proposal was that Flutter would do that out of the box without
me having to migrate my code.
On Tue, Aug 10, 2021 at 11:14 AM Schaban Bochi @.***>
wrote:

I think there is already something similar for your proposal in flutter.
Take a look at this article
https://medium.com/flutter/optimizing-performance-in-flutter-web-apps-with-tree-shaking-and-deferred-loading-535fbe3cd674 Only
load code when needed with deferred loading
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/87902#issuecomment-895826383,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANRAMR5BKZMKE2ZD77M6QBDT4DNXLANCNFSM5BYONS3Q
.

--
Yours sincerely,
Ziyad Farhan

Doing something like this out-of-the-box would mean the tooling would need to be aware of a particular screen/route structure in the app. However, in practice our users can roll all kinds of screen structures, and not even use Flutter's built-in router. So we will continue relying on deferred  imports for this use-case.
However, if you can somehow serialize the structure of routes from your app (e.g. using annotations), you could build your own code generator that generates deferred imports automatically. Something like this would be out of scope for Flutter to support though.


Surf display kiosk session

Motivation for this change
Adds surf-display as a package which is kiosk browser session using the surf browser.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

cc @talyz @adisbladis @aanderse

I'm not sure how I broke eval with this. I've tried it all on my system... So meh, restarting.
@GrahamcOfBorg eval
@GrahamcOfBorg build surf-display

@GrahamcOfBorg build surf-display
Building again now when openjpeg should be fixed on master

Currently everything works perfectly fine except for the configuration of the session. I will have a look at this soon. I know how to solve it.

And now the config works properly, I just imported the template config and overwrote the variables that I implemented support for in the module.

@worldofpeace Still works perfectly fine locally :+1:

@michaelCTS Hello, first of all I want to note that 4 year old pull requests aren't a support forum. For support please turn to support channels such as Matrix or Discourse mentioned here: https://nixos.org/community/
Then second of all, if you read what you have written in your config it should be fairly obvious why you get fluxbox and not surf-display since... you have fluxbox enabled and surf-display disabled.
I personally haven't had the need for surf-display for a couple of years so I'm a bit unsure of the current state. But I would expect that it works.


fix the demo code

@dsky1990
Thank you for the recommendation to change this ->
be advised that the } was left off on purpose , to indicate that the async function was intended to be left open to leave later code. This is not actually an error but a stylistic choice!
Appreciate your close eye - if you have any suggestions on how to better clarify, let me know!


cmd/compile: invalid memory address or nil pointer dereference

What version of Go are you using (go version)?

$ go version
go version go1.16 darwin/amd64

Does this issue reproduce with the latest release?
Yep.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/Users/$USER/gopath/bin"
GOCACHE="/Users/$USER/Library/Caches/go-build"
GOENV="/Users/$USER/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/$USER/gopath/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/$USer/gopath"
GOPRIVATE=""
GOPROXY="$PROXY"
GOROOT="/usr/local/go"
GOSUMDB="$SUMDB"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/2b/74fz3jhd4wz4vnbf4z7ywzww0000gp/T/go-build2406540359=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

https://play.golang.org/p/vhFwohxt9c3
What did you expect to see?
Anything other than an ICE.
What did you see instead?
An ICE.

I believe this is fixed at tip.
% ../bin/go build ~/gowork/issue45044.go
# command-line-arguments
../../../gowork/issue45044.go:8:6: invalid recursive type chain
	/Users/khr/gowork/issue45044.go:8:6: chain refers to
	/Users/khr/gowork/issue45044.go:8:6: chain

Yep. Thanks @randall77.
$ gotip run x.go
# command-line-arguments
./x.go:8:6: invalid recursive type chain
	/tmp/x.go:8:6: chain refers to
	/tmp/x.go:8:6: chain


Update to version 1.7.12

PPA with proposed package:
https://launchpad.net/~lucaskanashiro/+archive/ubuntu/container-backports/+packages
autopkgtest summary:
autopkgtest [19:10:34]: @@@@@@@@@@@@@@@@@@@@ summary
basic-smoke          PASS

We are using go 1.20 is closer to what upstream supports so far (look at go.mod) and it is available in all supported Ubuntu series. Moreover, the container stack in general is still using go 1.20. I think version 2.0 which is in beta now will support a newer go version.


lighttpd HTTP ERROR 500 on admin page

I have installed bluddit but when i got to the admin page i get
HTTP ERROR 500
I suspect it's to do with rewrite rule i have tried all the ones in this thread https://forum.bludit.org/viewtopic.php?t=121 but to no avail
relevant lighttpd.conf
[code]server.document-root        = "/bludit"
url.rewrite-if-not-file = ( "^/(.*)" => "/index.php?q=$1" )[/code]
lighttpd log
[code]2023-02-12 10:22:11: (response.c.247) URI-path (raw)  :  /admin/
2023-02-12 10:22:11: (response.c.248) URI-path (clean):  /admin/
2023-02-12 10:22:11: (response.c.249) URI-query       :
2023-02-12 10:22:11: (response.c.383) -- before doc_root
2023-02-12 10:22:11: (response.c.384) Doc-Root     : /bludit
2023-02-12 10:22:11: (response.c.385) Rel-Path     : /admin/
2023-02-12 10:22:11: (response.c.386) Path         :
2023-02-12 10:22:11: (response.c.438) -- after doc_root
2023-02-12 10:22:11: (response.c.439) Doc-Root     : /bludit
2023-02-12 10:22:11: (response.c.440) Rel-Path     : /admin/
2023-02-12 10:22:11: (response.c.441) Path         : /bludit/admin/
2023-02-12 10:22:11: (response.c.243) -- splitting Request-URI
2023-02-12 10:22:11: (response.c.244) Request-URI     :  /index.php?q=admin/
2023-02-12 10:22:11: (response.c.245) URI-scheme      :  http
2023-02-12 10:22:11: (response.c.246) URI-authority   :  <IP_ADDRESS>
2023-02-12 10:22:11: (response.c.247) URI-path (raw)  :  /index.php
2023-02-12 10:22:11: (response.c.248) URI-path (clean):  /index.php
2023-02-12 10:22:11: (response.c.249) URI-query       :  q=admin/[/code]

please ask in the forum


soc: arm64: Rename "TI Sitara" to "TI K3"

Rename "TI Sitara" to "TI K3":

The Keystone 3 (K3) family encompasses a wider variety of SoC's
This aligns the soc/arm64 naming with the soc/arm directory
This naming is more consistent with other projects like Linux

Maintainers: Update TI K3 to cover all related files/paths:

Some newer TI K3 files/paths were not part of the paths/globs

Just two questions want to confirm.
Do you need to rename it in boards/arm64/phycore_am62x_a53/doc/index.rst together？
Is there another place that needs to rename?

Just two questions want to confirm.
Do you need to rename it in boards/arm64/phycore_am62x_a53/doc/index.rst together？ Is there another place that needs to rename?

@SgrrZhf The AM62x is part of the Sitara family, which is part of the wider Keystone 3 family/architecture.
So using "Sitara" in "phycore_am62x_a53/doc/index.rst" still makes sense
I have checked the remaining instances of "Sitara" in the project and they are all ok


[Feature Request] Ability to apply certs when connecting to database

YugabyteDB Managed require to specify cert location in order to connect to the database. But currently there is no way to specify certs location when connecting to the database.
@YourTechBud

This is an easy fix. Adding it for the next release, for now you can download the certs on your filesystem (or mount it in your container) and specify a path to it in the postgres connection string.
Ref: https://pkg.go.dev/github.com/lib/pq#hdr-Connection_String_Parameters

ok thanks


Path find source currency limits (RIPD-1062)

Reviewers: @ximinez @vinniefalco

Current coverage is 49.67%

Merging #1442 into develop will decrease coverage by -0.03% as of 60c3a7f

@@            develop   #1442   diff @@
=======================================
  Files           963     963
  Stmts         67038   67062    +24
  Branches          0       0
  Methods           0       0
=======================================
- Hit           33318   33312     -6
  Partial           0       0
- Missed        33720   33750    +30

Review entire Coverage Diff as of 60c3a7f

Powered by Codecov. Updated on successful CI builds.

"PF" == "Pathfinding"? I haven't seen that used as an abbreviation before, and it's not obvious without looking at the code changes. I suggest spelling it out in the commit message.

Changes look good. Nice tests. I'll note that as of 12/28 10am PT, unit tests fail unless you rebase on https://github.com/ximinez/rippled/commits/mao (specifically for the commit at https://github.com/ximinez/rippled/commit/ce0290351498a92d2941ec3aa937e31e6c07c12b).

Needs rebase: mao changes are in develop.

Fantastic, this increases coverage!

Needs to be rebased against develop because of conflicts.

Seems legit
:+1:

Merged 796ee8e3de6003c6b39e0306d5811d8d0fd78066


Mapping a material using the material editor leads to bad UVS

Either select a model in the mission or load one in the shape editor.  Then map a new material and the UVs on that model break (are skewed and distorted).  If the changes are saved and torque is restarted the issue goes away.

Hi, This is still a bug? :S I loaded different models with UVs of course and I don't remember this happen to me. How can we test this? or it can be closed? @Areloch ???

Thinking this was fixed at some point, and still can't replicate it, so I'm just going to close this.


NIFI-5623 Upgraded OkHttp to 4.9.1 and updated unit tests

Description of PR
NIFI-5623 Upgrades OkHttp3 to version 4.9.1 and updates several unit tests to work with the new version.

Created okhttp.version property in root Maven configuration to reflect version used in NiFi framework components
Refactored unit test for HttpNotificationService to remove unnecessary test methods and streamline testing approach
HttpNotificationServiceTest includes URL property creation to work on all platforms following OkHttp version updates for OkHttp MockWebServer, and also removes the need for binary JKS files in nifi-bootstrap test resources
Preserved OkHttp version override in nifi-toolkit-api to maintain support for generated classes

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on JDK 8?
[X] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Merged to main, thanks @exceptionfactory


Add test coverage for icon services

As requested by @nathansobo while reviewing #281.

I really appreciate the speedy followup! ✨


Add a keyboard shortcut handler to the TabRowControl

This makes the scenario mentioned in #8480 work. It's maybe not as holistic a solution as we'd like, but it definitely works.
Tested both with Narrator, and using the TabView scroll handles to get tab focus into the tab row manually

[x] Closing the active tab with Ctrl+Shift+w works (not the TabViewItem that has focus, but that's how Edgium works so that seems fine)
[x] Opening a tab with Ctrl+Shift+t works
[x] Opening the cmdpal with Ctrl+Shift+p works
[x] Will take care of #8480 once we get the a11y team to validate
[x] I work here

Notes:
None of
PreviewKeyDown="_KeyDownHandler"
KeyDown="_KeyDownHandler"
KeyUp="_KeyDownHandler"

On the TerminalPage directly seem to fire when the focus is in the TabViewItem or the New Tab flyout. But they fire just fine when focus is in the TermControl. Interesting, because you'd think that the TermControl would have already handled the key...

@msftbot merge this in 1 minute


Using TaskScheduler across DLLs

I have just moved my codebase to enkiTS, having previously been using Micha Mettke's ANSI C port of it. I ran into some issues with occasionally reading garbage from the pipe in Windows, and thought I would try the original enki, as it is more actively developed and uses full on std::atomics. Seems to have fixed the problem, great! It also runs faster, many thanks!
Now, my usecase is inside a scientific python library, which is compiled as a DLL/so. I also allow extra DLLs (not known at compile time) from which replacement functions are manually loaded. I want to exploit the braided parallelism inside these extension methods loaded from the DLLs, but the DLLs end up containing a secondary copy of the implementation of enkiTS (it's non-trivial to link them to each other). This is mostly fine as I can ensure that they are compiled with the same compiler/ABI. However, the problem is gtl_threadNum, which won't be initialised if accessed inside a DLL function. This tends (not with 100% reliability) to deadlock the scheduler.
My current "fix", which seems fine in every test I have run, is to make TaskScheduler::GetThreadNum virtual, and pass around TaskScheduler*, so that GetThreadNum is called through a vtable, and update all accesses to gtl_threadNum to call this->GetThreadNum() instead. Whilst this solution works fine, I fear that some optimiser is going to become sufficiently "smart" to devirtualise GetThreadNum, and end up loading the wrong value of gtl_threadNum.
I was wondering if you had any recommendations of a better way to address this. I have considered putting gtl_threadNum as thread_local in a static struct, and carrying a pointer to this struct in TaskScheduler, but that also feels quite untidy.
I know this is a weird usecase, and not something I expect to be handled by the core enkiTS, but I felt it also good to document my approach (plus any advice you may have to offer) in case someone else finds themselves in a similar situation.

My mistake, many thanks for your time. I thought (very incorrectly) that each shared library linked against the enkiTS DLL would end up with its own copy of the thread local variables. Having tested this, I now understand how it works.


No need for BastionHost

In Step 7, this guide has users enable/create a BastionHost. This is not at all necessary to creating a virtual network. Bastions are also very expensive, unnecessary, and hard to delete.
[Enter feedback here]

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

ID: 0f0990fc-9834-bde3-5e06-071c75fa89a5
Version Independent ID: 96f5df37-aa95-443d-2500-37665179ce89
Content: Quickstart: Create a virtual network - Azure portal - Azure Virtual Network
Content Source: articles/virtual-network/quick-create-portal.md
Service: virtual-network
GitHub Login: @asudbring
Microsoft Alias: allensu

@jwilliams45-aug ,
Thank you for sharing your feedback. As this QuickStart is intended to show how virtual-network works in Azure by depicting the connectivity between two VM's deployed in the same Virtual Network and Bastion is used to securely connect to the VM.
Azure Bastion is the recommended way to connect to a VM in Azure as Bastion provides secure RDP and SSH connectivity to all of the VMs in the virtual network in which it is provisioned. Using Azure Bastion protects your virtual machines from exposing RDP/SSH ports to the outside world, while still providing secure access using RDP/SSH.

@jwilliams45-aug ,
Since we did not hear back from you, we will now proceed to close this issue out, please tag me in your response if you have any questions. Thank you!


Issue with change button

After click correct answer or incorrect bgcolor of button should be green or red.
Find any way into change color of button

Solution
In first function we set switch function to check wich button was click.  When we knows which button was click set class name. In second function we check for button value is correct or incorrect, when button value is equals to correct answer set class btn btn-success but when incorrect btn btn-danger
btn btn-success / green color
btn btn-danger / red color
function change_color($post, $class) {
	global $classNameT, $classNameN, $classNameA, $classNameB, $classNameC;
	switch ($post) {
		case "T":
			$classNameT = $class;
			break;

		case "N":
			$classNameN = $class;
			break;

		case "A":
			$classNameA = $class;
			break;

		case "B":
			$classNameB = $class;
			break;

		case "C":
			$classNameC = $class;
			break;
	}
}

function check_answer() {
	global $correct_answer;

	//correct answer
	$correct_answer = get_answer();

	//button background color
	if($correct_answer == $_POST['answer']) {
		change_color($_POST['answer'], "btn btn-success");
	}
	else {
		change_color($_POST['answer'], "btn btn-danger");
	}
}


Moving overall from table to map card

Closes the eng parts of #993
Additionally, the map card was still supporting querying for multiple breakdowns at once (age, sex, race) based on an older version that we moved away from a long time ago. I don't think we'll bring that back so I went ahead and removed that logic.

This all looks good to me, thanks @kkatzen !

@kkatzen do you think this would make sense to push to a branch on the main repo so I can change it later if I need to ?

@kkatzen do you think this would make sense to push to a branch on the main repo so I can change it later if I need to ?

I think since this PR actually changes quite a bit of logic in the Map Card I'd be worried about annoying merge conflicts later if it sits in another branch for awhile- if you need to get rid of this in a hurry in the main repo I would just comment it out

@kkatzen do you think this would make sense to push to a branch on the main repo so I can change it later if I need to ?

I think since this PR actually changes quite a bit of logic in the Map Card I'd be worried about annoying merge conflicts later if it sits in another branch for awhile- if you need to get rid of this in a hurry in the main repo I would just comment it out

I went ahead and just disabled the text and added a todo for #993 to reenable when the copy is reviewed

@kkatzen Awesome, I guess the only problem is that we lose the all measure all together right now. How hard would it be to leave that in the table for now?

@kkatzen Awesome, I guess the only problem is that we lose the all measure all together right now. How hard would it be to leave that in the table for now?

oh good catch- reverted that change and added a TODO with the new code in the comment also linked with 993 :)

Thank you!


chore(router) use table.new to process header_values

Summary
table.new may be better than raw {}.
Full changelog

use table.new to create new table in processing headers

Good


refactor timer to redux-saga

https://github.com/markliu2013/reactjs-redux-snake-game/commit/2ba1100c0c5a0604952106c656a569ec4d92c2dd


run weekly 2.12.x community build with optimizer enabled

we don't have enough disk space on the Jenkins nodes for this at present. (and they aren't easy to expand, they would need to be rebuilt)

we have plenty of disk space now, since the recent big Jenkins overhaul


Man pages for CLI commands

The dotnet/cli repo contains bunch of documentation that includes man pages for the cli tooling (such as dotnet publish). Unfortunately, those are not very well maintained - the current docs still refer to project.json! See https://github.com/dotnet/cli/issues/7291 for more background.
I noticed that this repo contains documentation that appears on https://docs.microsoft.com/, such as https://docs.microsoft.com/en-us/dotnet/core/tools/dotnet-publish. These tool docs look very, very close to man pages. And unlike the cli docs, these are well maintained.
Would it be possible to generate man pages from these docs? I can help out with implementing this if this idea seems sensible.

@nguerrera @livarcocc @richlander Z used to generate the man pages from our docs in the past, but I thought the process had changed since then?

It looks like no one picked up that task when Z left. As I see it, there is no process currently :(

@omajid is right. We need to figure out the best way to get the man pages to sync directly from the docs.microsoft.com content. Any manual steps will just risk us getting back to this bad state in the future.

It looks like pandoc can generate man pages from existing docs already.
pandoc -s -f markdown -t man -o dotnet.1 docs/core/tools/dotnet.md
man ./dotnet.1

The generated docs do need some cleanup, though. Here is a quick list of issues from running the commands above:

No section name in man page docs
Section names are not consistent with other programs (Name vs NAME)
Option names/descriptions are not indented correctly. Also a problem for examples and environment variables
Tables are completely messed up
Links to other documents in docs are completely messed up

Ping. Anyone have any thoughts on how to proceed from here?

Ping. I would appreciate some guidance on what a new man page workflow should be.

Sorry for making noise, but I would appreciate some comments and/or suggestions on how to proceed.

Thanks for pushing on this, @omajid. I'm sorry for not responding sooner.
Adding @KathleenDollard, our new PM, for her thoughts too.
I'm now thinking that it is going to be tough to implement this in a way that is fully automated and does not interfere with the daily flow of documentation writers or CLI developers.
Also, we only got feedback (that I saw) about the bad man pages on the verge of 2.0 RTM and yet they are all about project.json! This makes me wonder if folks are even finding the man pages. I didn't even know they existed until this was raised as they were never updated during any of my time on the team.
Initially, I think we should just delete the bad man pages or just have man dotnet with a basic description and a pointer to dotnet help that can launch the up-to-date docs in a browser.

I'm now thinking that it is going to be tough to implement this in a way that is fully automated and does not interfere with the daily flow of documentation writers or CLI developers.

Maybe we can handle it async a couple of times every release cycle? In other words, at some point a developer (say, me) can get a heads up from doc folks that the general docs are complete and the dev can sync them?

Also, we only got feedback (that I saw) about the bad man pages on the verge of 2.0 RTM and yet they are all about project.json! This makes me wonder if folks are even finding the man pages. I didn't even know they existed until this was raised as they were never updated during any of my time on the team.

This might be a chicken and egg problem. I don't think Microsoft binaries include them. I looked through https://www.microsoft.com/net/core#linuxfedora for example and only dotnet.1 is included there. I know it is included in RHEL RPMs and Fedora RPMs (only because I found them and added them).

Initially, I think we should just delete the bad man pages or just have man dotnet with a basic description and a pointer to dotnet help that can launch the up-to-date docs in a browser.

Some distributions, like Debian, consider it a bug if man pages are missing. I would strongly consider keeping them unless there is a very good reason not to.

@nguerrera
I think man pages become more important once .Net Core becomes part of package repositories of distributions.
Right now, to install .Net Core, I have to find a guide like https://www.microsoft.com/net/core#linuxubuntu, which can direct me to the documentation. It also means I have used the browser recently, so I might use it again when looking for documentation.
But once I can just do something like sudo apt install dotnet, then I think man dotnet is an obvious next step, especially since I used only shell and I might not even have a browser open.

My thoughts?
We should do this.
What? Still working on. Have a call to my favorite Linux folks later today.
At least ifno on the level of –help, which is also on my radar to see improved, and a docs link.
We’re cross platform as an org. I’ll work with docs to see what they think that should mean.
Kathleen
From: Nick Guerrera<EMAIL_ADDRESS>Sent: Wednesday, August 30, 2017 7:53 AM
To: dotnet/docs<EMAIL_ADDRESS>Cc: KathleenDollard<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/docs] Man pages for CLI commands (#2791)
Thanks for pushing on this, @omajid https://github.com/omajid . I'm sorry for not responding sooner.
Adding @KathleenDollard https://github.com/kathleendollard , our new PM, for her thoughts too.
I'm now thinking that it is going to be tough to implement this in a way that is fully automated and does not interfere with the daily flow of documentation writers or CLI developers.
Also, we only got feedback (that I saw) about the bad man pages on the verge of 2.0 RTM and yet they are all about project.json! This makes me wonder if folks are even finding the man pages. I didn't even know they existed until this was raised as they were never updated during any of my time on the team.
Initially, I think we should just delete the bad man pages or just have man dotnet with a basic description and a pointer to dotnet help that can launch the up-to-date docs in a browser.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dotnet/docs/issues/2791#issuecomment-326015911 , or mute the thread https://github.com/notifications/unsubscribe-auth/AFktXgyspaIBham8CteFUb0_udkgNUaqks5sdXdQgaJpZM4Oox6h .  https://github.com/notifications/beacon/AFktXkor59D2Fs7N4wkFI_TbWgYTZMcvks5sdXdQgaJpZM4Oox6h.gif

Hi @KathleenDollard , any updates on this? Is there anything I can help with?

Ping. Just trying to make sure we come to a decision and implement it!

Ping. Has anyone given this any thought?

We haven't given any extra thought on this yet. My main point is that this needs to be automatic somehow, otherwise it will just go out of sync completely again.

Please let me know how I can help in addressing this.

It looks like pandoc can generate man pages from existing docs already.
pandoc -s -f markdown -t man -o dotnet.1 docs/core/tools/dotnet.md
man ./dotnet.1

Seems like the CI script that runs pandoc command is not open sourced.. can one of the admin update the script with the proposed parameters and see the impact?

My main point is that this needs to be automatic

Seems like it's already automatic. All these docs are generapted by pandoc util https://github.com/search?q=org%3Adotnet+pandoc&type=Code but the caller script is not in open.. maybe it belongs to dotnet/core-eng private repo?

I'm closing this one given that we have issue https://github.com/dotnet/cli/issues/7291 tracking this work. I'm not aware of any work items for the docs team re. man pages at the moment.
/cc @livarcocc @KathleenDollard

Would it be possible to keep this open? I agree that there is no well defined work at the moment and docs team may not be responsible for this, but having severely out of date man pages is an issue and it needs to be (somehow) addressed.

@mairaw, @mmitche, can dotnet-bot send followup PRs to dotnet/docs repos after something is merged by contributors?
The packaging team can then pick up the manpage changes. With all the cool features @dotnet-bot is bringing via https://github.com/dotnet/dotnet-ci, this seems like a very simple / low-cost thing with huge advantage for Unix consumers.
Using PowerShell:
$pandoc = 'C:\pandoc-2.1.2\pandoc.exe'
if (!(Test-Path $pandoc)) {
  [Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12

  Invoke-WebRequest -output pandoc-2.1.2.zip `
    https://github.com/jgm/pandoc/releases/download/2.1.2/pandoc-2.1.2-windows.zip

  Expand-Archive .\pandoc-2.1.2.zip -DestinationPath c:\
  Remove-Item pandoc-2.1.2.zip
}

$mdFiles = Get-ChildItem -Recurse "C:\repos\dotnet-docs\docs\*.md"

for ($i=0; $i -lt $mdFiles.Count; $i++) {
    $mdFile = $mdFiles[$i].FullName
    $manFile = [io.path]::ChangeExtension($mdFile, '1')

    &$pandoc -s -f markdown -t man -o $manFile $mdFile
}


Cannot completely clear an input.datetime component

Home Assistant release (hass --version):
0.57.3
Python release (python3 --version):
hass.io?
Component/platform:
input.datetime
Description of problem:
Attempting to clear the field like so:
2017-12-14 21:22:42 INFO (MainThread) [homeassistant.core] Bus:Handling <Event call_service[L]: domain=input_datetime, service=set_datetime, service_data=entity_id=input_datetime.door_keypad_3_date_start, date=, time=, service_call_id=1966370448-58>
2017-12-14 21:22:42 ERROR (MainThread) [homeassistant.core] Invalid service data for input_datetime.set_datetime: Could not parse date for dictionary value @ data['date']. Got ''
Invalid time specified:  for dictionary value @ data['time']. Got ''

does not work

Looking at the code  it can be solved to set the sensor to unknown if time and date are empty ''


unyaffs: init at 0.9

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Wait, this is 13 years old...

Yes, quite old, but not that scary; the halloween is over :)
This is a runtime dependency for the new binwalk which I'm trying to init.
https://github.com/ReFirmLabs/binwalk/blob/d1ecf723d45df99f70f04733c050c5f7c783b91f/dependencies/ubuntu.sh#L21

Yes, it build correctly now.

About the other pull requests where ofborg tried to evaluate before I added myself to maintainers list: how to make it retry evaluation?

 From there, git switch branch-1 and git rebase master && git push --force-with-lease for all of your PRs' branches.


Passcode/Touch ID to the app?

When will you add Passcode/Touch ID when opening the app to the Tofu Authenticator app?

Hi @briannyeko! I currently have no plans on adding this feature.
I'd love to know more about what kind of situations you think it would be valuable in, though!

Hi Calle!
I see 2FA authentication apps(Could be used to access information(emails,financial or technological information) that could be valuable to individuals) similar to a bank vault(you have multiple locks and passwords in order to access the money/gold) 😁 .
So I thought it would be a good idea to suggest to you when you would add a passcode/touch ID function to your app(in case you lose your phone or if someone steals the phone from you, if would be hard to access the verification code if touch ID/Passcode is a feature of the Tofu Authenticator app.

Thanks Brian, I appreciate the suggestion!

in case you lose your phone or if someone steals the phone from you

Aren't these cases already covered by the passcode lock on the phone?

No, it doesn't. You mean the passcode (TouchID, FaceID) that you have to enter to get into your iPhone? In the scenario I proposed, an attacker (robber, border control agent etc.) would have already gotten into your phone, for example, by forcing you to unlock it. Then there's nothing to stop them from getting into Tofu, correct?

Yes, that's the lock I mean.
If I'm forced to unlock the phone's passcode lock, I'm not sure how another passcode/Touch ID lock would prevent access to Tofu? Wouldn't I be forced to unlock that one too?
I'm not trying to be difficult or negative here. I just want to be sure this feature would add value  before someone spends time implementing it and before I commit to supporting it going forward.

Okay, you're right, maybe my example was flawed. :P
How about this though: If a user for some reason does not use a code, TouchID or FaceID on their iPhone, then a 3rd party could easily get into the phone and then into Tofu. A code or TouchID for Tofu would then be a good defense.

Ok, so the point of 2FA is to have 2 things to authenticate (thing you know and thing you have for example), but now most of us use some sort of password manager (hopefully) and usually this manager and 2FA token generator are on the same device. If somehow somebody acquires your device in unlocked state and you recently used your PM so the lock timeout hasn't yet triggered they will have both the password and token. It's not that unlikely scenario, you may want to hand your device to someone to show them some photos and go to the toilet. Usually this will be trusted person, but if not, then saying "give my my phone back, I will show you the rest of the photos when I'm back" is just something socially awkward to say because it may look like you're obsessively paranoid.
When you think of security layers that you can add you think of the balance between ease of use and security it actually provides. When poor security upgrade makes something annoying to use it may actually lower the security. Windows UAC prompts in Vista was a good example as it taught its users like Pavlovian dogs to click just any prompt in order to use the computer. But in the age of easy system level biometric bypass to PIN codes, using this in Tofu would be actually free as it would be almost invisible in terms of typical use cases. Yet it will give you a peace of mind in various situations like the one above so you won't have to stress about them too much.
Please consider adding this mechanism as an option. It's just a good security practice, adds another layer for the attacker, would not annoy normal users and should be easy enough to add and maintain in code.
Fun fact - similar attack vector (access to unlocked device with unsecured 2FA generator) was exploded in second season of Mr. Robot.

It seems that a second TouchID/FaceID prompt in the app would protect you against one case: When your phone is snatched from you while unlocked, and kept unlocked until it can be used for 2FA. This seems like a very unlikely threat scenario, though, and if someone is that determined to get access to your services, you probably have larger problems that are not easily solved by this app.
Having a separate passcode in the app would protect you against the additional case where someone gets your locked phone, and has some method of getting around the the phone lock. As far as I know this would be tricky but possible for TouchID, and I do not know of any successful attacks against FaceID.
Both seem a bit like security theatre, but the passcode is the more effective of the two.

Thanks for the arguments! I'm convinced this feature could provide value and since it is mostly orthogonal to existing features I hope it won't prevent us from adding other features down the line.
If someone would like to work on this, I'd be happy to review a PR. If not, I'll get to it when I have time.

I'll add another example.  I frequently unlock my phone and hand it to someone so they can navigate while I drive or some other common scenario.  They're using my phone with my permission without direct monitored access.  This feature would prevent someone from sneakily gaining access to my TFA pins while they have access to my device.  Sounds like you already accepted this as valid, but wanted to add a little fuel to the fire :)

Any news on when this feature will be implemented? Just downloaded and app and loving it, although I feel this is the only feature that is missing.

@NC740, I'm glad you like the app. I have no timeline for when this feature will be ready. I don't have much time to work on the app at the moment. If you want help, please consider supporting my work: https://github.com/sponsors/calleerlandsson

Are you planning on adding it?


chore(IsAlgClosed): move some defs about lift into IsAlgClosed namespace

All of the moved theorems are basically auxiliary things which need not be used by users anyway.

Once the import change is reverted this is good to go, thanks!
bors d+


front autofocus props

Need for a bugfix.

ERROR in /home/travis/build/coveo/react-vapor/src/components/input/Input.tsx
./src/components/input/Input.tsx
[tsl] ERROR in /home/travis/build/coveo/react-vapor/src/components/input/Input.tsx(168,21)
      TS2339: Property 'autofocus' does not exist on type 'DetailedHTMLProps<InputHTMLAttributes<HTMLInputElement>, HTMLInputElement>'.

     TS2339: Property 'autofocus' does not exist on type 'DetailedHTMLProps<InputHTMLAttributes<HTMLInputElement>, HTMLInputElement>'. in your build 🤕

@MichelML @GermainBergeron it should be fixed now (waiting for the build). I added a test too.

Codecov Report

Merging #585 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #585   +/-   ##
=======================================
  Coverage   99.72%   99.72%
=======================================
  Files         259      259
  Lines        6500     6500
  Branches     1516     1516
=======================================
  Hits         6482     6482
  Misses         18       18

Impacted Files
Coverage Δ

src/components/input/Input.tsx
100% <ø> (ø)
:arrow_up:

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 dc2616b...9554423. Read the comment docs.

C'est fixed pour vrai la!


可以处理聚合结果吗

比如把聚合结果转存到其他表
select date, count(1) as from table group by 1

还有可以把多张不同字段名称的表通过字段映射到同一张表吗？

Hi! Thank for taking the time

可以处理聚合结果吗
如把聚合结果转存到其他表
select date, count(1) as from table group by 1

Unfortunately, not today, however, I have spoken to a few folks about the ability to control in what form they would like the data to be synced, and aggregation is a common one.
I am thinking, users can write the sql and sync + transform + route the data to another table. And manage the sql through a yaml file. Would something like this, help w/ your use case?

还有可以把多张不同字段名称的表通过字段映射到同一张表吗？

I would not recommend, writing to the same table again, because of logical replication and the replication slot is created under the same table. Its better if you route to a different table, using the routing feature.

Thanks for your response.

Looking forward to the the data sync with aggregation.
I have some ETF requirements, for example, union different table, yes, it would sync to another table, but maybe the column name or column type is not unified, as follows.

table1

id1
name1
age1

+
table2

id2
name2
age2

=>
table3

id3
name3
age3


Codegen doesn't present help when invoked without arguments

Bit of a shame. It just fails instead of presenting help:
➜  test-sdk yarn xata-codegen
yarn run v1.22.17
warning ../package.json: No license field
$ /Users/tejas/Sites/LAB/test-sdk/node_modules/.bin/xata-codegen
Using schema file: /Users/tejas/Sites/LAB/test-sdk/xata/schema.json
Using output file: /Users/tejas/Sites/LAB/test-sdk/xata.ts

Could not read schema file at ./xata/schema.json
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

It doesn't present the --help output, but you can invoke it:
xata-codegen --help
Options:
      --help     Show help                                            [boolean]
      --version  Show version number                                  [boolean]
  -s, --schema   The schema file       [string] [default: "./xata/schema.json"]
  -o, --output   The output file                  [string] [default: "xata.ts"]

It doesn't tell you you are missing paramteres, because all parameters are optional. In the --help output you can see the defaults.
I think the messages sent are useful enough:
Using schema file: /Users/tejas/Sites/LAB/test-sdk/xata/schema.json
Using output file: /Users/tejas/Sites/LAB/test-sdk/xata.ts

Could not read schema file at ./xata/schema.json


Add missing Prometheus metric "paths_bytes_sent"

Describe the feature
Currently, MediaMTX provides a nice Prometheus compatible metrics endpoint which can be used in Grafana for showing real-time information. Thanks for this feature. Unfortunately, although there is a "paths_bytes_received" metric, there is no corresponding "paths_bytes_sent" metric. One can add the other *_bytes_sent" metrics as a workaround, but as at least SRT is missing there, IMHO currently there is no way to graph the total traffic sent by MediaMTX. Could you please add a metric "paths_bytes_sent"? Many thanks.

In the above referenced pull request I added the missing exporters.


Grammar error on date range error message.  Placement could be better

yeah yeah yeah, those damn testers and their grammar bugs.  Sure but details count, especially to be a world class application. See the screen shot.
Also - I suggest moving the error message location closer to the actual date range fields.  After entering dates & location, the user's focus is on the "volunteer" button,  not where the error messsage shows up. It would be more fluid to put the system's error response in the vicinity of the date ranges.
bug found during starwest virtual conference.

Fixed the grammar mistake in PR#419 ("a end date" -> "an end date"), but did not move the error message.

Fixed in PR #419


Adding Pipfile

And excluding Pipfile.lock from repo

We now have requirements specified in setup.py, setup.cfg, requirements.txt and now we are adding another file? Can pipfile replace the others and could we clean them up?

Well, they are not interchangeable.  So, I'm afraid, no. But Pipfile is optional, pipenv can generate it from requirements.txt, so, we can skip that.


Verify CLion experience (Linux)

This is the "out of the box" experience.
When you open this project in Clion does it Just Work?
related #17

For me it worked perfectly first time: Ubuntu 20.04, 2023.1.4.

@lefticus  Works in CLion 2024.1 EAP (Nova) - Ubuntu 23.10 - Clang-17/++/Tidy/Format
fwiw : Similar to the poster above there were some compilation warnings coming from ftxui's sources. It's a bit of an exotic dependency to pull in as a hello world IMO it adds too much noise, it pollutes the clion run menu with a gazillion irrelevant targets for all their examples (see screenshot a couple posts above). It took me a good second to realize I needed to look for the "intro" target. It would be nice if that was called out in the readme or if the target would reflect the name of the project.

Environment: CLion 2024.1.4 dev container integration on OpenSuse TumbleWeed 202408
The container works in the default configuration, using a non root user inside the container requires modifying the Dockerfile to add the user (just like for VsCode, follow its documentation here).


MainEditor not excluded at build time

When installing uREPL via the Git URL the MainEditor is no longer under the special Editor folder under assets, as so it's not actually excluded at build time.
This leads to the build failing because of the dependency on UnityEditor
To solve this we would need to modify the package to add #if (UNITY_EDITOR)  around the entire file.
Could you update the MainEditor to use macros please?

uREPL.Editor.asmdef was added and the problem was fixed from v1.0.4.
https://github.com/hecomi/uREPL/releases/tag/v1.0.4

Thank you, it works now


Adjust CSS to render /summary component cleanly on mobile

Currently, the details pane on /summary is pushed off the screen, preventing proper reflow. Use media queries to adjust as needed for different screen sizes and orientations.

Less relevant because of redesign of summary component.


⚠ Bofrost.es has degraded performance

In 3721f30, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1567 ms

Resolved: Bofrost.es performance has improved in 99d698b after 6 minutes.


Add the option to create Property Node methods with a Get/Set prefix (instead of Read/Write)

When creating a new Property Node method, I can only select :

Read Property
Write Property

Depending on what I chose here, my method will get the prefix "Read" or "Write".
I would like to see the option to have this prefix be "Get" or "Set".
Either the option to change it here :

or possibly a Boolean selector somewhere to change between Read/Write and Get/Set.

I know this is personal preference and LabVIEW native also calls them Read/Write.
But I prefer to use the term Read\Write to the VIs that interact with instruments.
And prefer to use the term Get\Set to VIs that interact with a class (attribute accessors).
For example:
The accessor name ‘Read Data’ will make me think that this VI will read Data from an Instrument.
So I prefer ‘Get Data’ instead, because it's stored in the attributes.

Even better would be if this options could be toggled from the GOOP4 Options in the GOOP4 configurations.

I agree I also use Read and Write when talking to instruments and Get and Set when I only access the Instrument object's attribute And I most often just use Property Methods not Property Node Method.
And Yes I used the NI naming convention for this.
Okay it would be easy to add that extra option in the GOOP4 Config dialog

Hi, we are working on some major clean up tasks in the code right now.
This is to disconnect the UML modeller from the OpenGDS base function, just to make it smaller so we can publish it on tools network so you can find it in VIPM.
BTW I see that you are not using the latest version, the config dialog looks like this now:


Updated git plugin version

what

Updated git plugin in Jenkins to 4.5.1

why

The gitlab-oauth plugin requires 4.5.1
The gitlab-oauth plugin began failing to install, even though gitlab-oauth plugin version was unchanged

/test

/test