Bug located in export mesh as a .key file (DYNA format)

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
The user is unable to export the mesh in the form of a dyna file
the error observed is
Traceback (most recent call last):
File "D:\CustomerProjects\Microsoft\Solder_reflow_consulting\PyPrime\mesh_script.py", line 137, in
file_write.export_lsdyna_keyword_file(
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\ansys\meshing\prime\core\fileio.py", line 512, in export_lsdyna_keyword_file
result = super().export_lsdyna_keyword_file(temp_file_name, params)
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\ansys\meshing\prime\autogen\fileio.py", line 581, in export_lsdyna_keyword_file
result = self._comm.serve(self._model, command_name, self._object_id, args=args)
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\ansys\meshing\prime\internals\error_handling.py", line 552, in wrapper_error_code
result = func(*args, **kwargs)
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\ansys\meshing\prime\internals\error_handling.py", line 507, in wrapper_handle_errors
func_result = func(*args, **kwargs)
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\ansys\meshing\prime\internals\grpc_communicator.py", line 187, in serve
message = get_response(response, '')
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\ansys\meshing\prime\internals\grpc_communicator.py", line 67, in get_response
return separator.join(response.data for response in get_response_messages(response_iterator))
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\ansys\meshing\prime\internals\grpc_communicator.py", line 67, in
return separator.join(response.data for response in get_response_messages(response_iterator))
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\ansys\meshing\prime\internals\grpc_communicator.py", line 58, in get_response_messages
for response in response_generator:
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\grpc_channel.py", line 542, in next
return self._next()
File "C:\Users\jhe.ansys_python_venvs\venv\lib\site-packages\grpc_channel.py", line 968, in _next
raise self
grpc._channel._MultiThreadedRendezvous: <_MultiThreadedRendezvous of RPC that terminated with:
status = StatusCode.UNAVAILABLE
details = "Connection reset"
debug_error_string = "UNKNOWN:Error received from peer  {created_time:"2024-08-28T13:30:13.6434084+00:00", grpc_status:14, grpc_message:"Connection reset"}"
has context menu
📝 Steps to reproduce
Create a volume mesh and export key file. (Ensure LsDyna is not installed on the machine)
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
2024R2
🐍 Which Python version are you using?
3.10
📦 Installed packages
ansys-meshing-prime=0.6.2

@agvarghe confirmed this suspected to be an installation issue and has not been reproducible.  Closing now.  Please reopen if the issue repeats. Thanks


Adds enrichment usage to readme

Adds enrichment usage to the README as requested in slack.
I'm open to moving it somewhere else if there's a better place to put it?

Adds enrichment usage to the README as requested in slack.
I'm open to moving it somewhere else if there's a better place to put it?

Yeah I've a better place to put it, basically in /examples, and I've a WIP enrichment readme as well. I'll be able to have that pr-ed tomorrow - will be adding you to that.

Closing in favor of #62


Brand Sign Up Page made as responsive

Issue #24 Fixed
Screen Shot

@BHS-Harish  eveything is fine just one issue there are 2 background colors

@RohitGupta1237
Now It's ok.. Check it

@BHS-Harish  its all fine now


add / remove / add "model already exists error" (doing global check, which prevents local action)

I think its checking from globally installed instead of aimodels.json ?

confirmed


DM-41124: Exclude panda-client 1.5.65.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


Fix: typos at mask propagation test in MultiHeadAttention

This PR will fix the typos in MultiHeadAttention by replacing progagation -> propagation.

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

Comparison is base (1c0d997) 78.30% compared to head (fd06665) 57.94%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #18616       +/-   ##
===========================================
- Coverage   78.30%   57.94%   -20.37%
===========================================
  Files         334      334
  Lines       32484    32484
  Branches     6342     6342
===========================================
- Hits        25438    18823     -6615
- Misses       5482    12225     +6743
+ Partials     1564     1436      -128

Flag
Coverage Δ

keras
57.94% <ø> (-20.25%)
:arrow_down:

keras-jax
?

keras-numpy
57.94% <ø> (ø)

keras-tensorflow
?

keras-torch
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 180 files with indirect coverage changes

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

cc: @fchollet


Changed message to show bot messaging endpoint based on env variable

@jotrick,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Fix cache permission calculator integration with threads

In the cache permission calculator integration, the channel method is causing a failing build.
Threads inherit their permissions from their parent channel. The method needs to get the parent
channel overwrites, and in the case of private threads, merge the two overwrite lists.

Fixed with the latest force push to feature/threads


pub get failed (66; , errno = 1314))

Running "flutter pub get" in dino_run...
Cannot create link, path = 'G:\AndroidStudioProjects\flutter.pub-cache_temp\dir76fa0be4\ios\Classes\AudioplayersPlugin.h' (OS Error: 客户端没有所需的特权。
, errno = 1314)
pub get failed (66; , errno = 1314))
Process finished with exit code 66

I'll need some more information to investigate this.

Does this happen if you just clone and run this project? Or have you made your own modifications?
What platform are you building for?
What are the version of packages you are using? Specially audioplayers.

yes,I am clone and run this project
Flutter 2.5.3
Dart 2.14.4
Android Studio 11.0.10
but I just found a solution.
1.Run CMD as Administrator.
2.flutter pub get
Thank you very much


Parameters being ignored when installing artifacts with vm create and inline JSON.

Describe the bug
Command Name
az lab vm create
Errors:
The VM is created but the choclatey install reports the following error:

Unable to generate valid template for the artifact - Install Chocolatey Packages with error(s) - Deployment template validation failed: 'The value for the template parameter 'packages' at line '13' and column '21' is not provided. Please see https://aka.ms/resource-manager-parameter-files for usage details.'.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.
$artifacts = '[{"""artifactId""": """/subscriptions/*******/resourceGroups/safetestrg/providers/Microsoft.DevTestLab/labs/testlab/artifactSources/public%20repo/artifacts/windows-chocolatey""", """parameters""": [{"""packages""": """firefox"""}]}]'

az lab vm create --resource-group TestRG --lab-name TestLab --name 'ScriptVM5' --image "Windows 10 Pro, Version 1809" --image-type gallery --size 'Standard_B2s' --admin-username '****' --admin-password '****' --artifacts $artifacts

Expected Behavior
The VM is created and Firefox is installed via chocolatey
Environment Summary
Windows-7-6.1.7601-SP1
Python 3.6.6
Shell: powershell.exe

azure-cli 2.0.74

Additional Context
i know there is already an artifact for Firefox. This is just a proof of concept around deploying other chocolatey packages and seems to highlight a bigger issue.

@1dasdave7 Apologies for the delayed response. This GitHub issue has been open for quite some time. Could you please let us know if you still facing this issue and need assistance? Awaiting your reply.


Sequential coupling in route/middleware initialization

It seems like the routes are bound to the middleware stack in the state that it was at the moment the first route was created, which creates hard to track sequential coupling for complex initialization processes.
Given the lightweight/kiss nature of express, I understand that this might be done by design. However, I'd advise documenting the exact behavior clearly. (ie: That the middleware stack bound to a route corresponds to the state of the middleware stack at the moment where the first route was created.)
To reproduce:
var express = require('express');
var should = require('should');

// Create express app;
var app = express.createServer();

// Route here, which fails
app.get('/', function(req,res){
  // We *should* receive the value here given the custom middleware
  // was supposed to have put it.
  should.exist(req._magic);
  return res.send('Hello World!');
});

// Middleware stack
//Small mw setting a custom var in the req for the tests.
app.use(function(req,res,next){req._magic = '42';next();});
app.use(app.router); //Router right there

// Another route, defined after the mw stack. it will also fail.
app.get('/test', function(req,res){
  should.exist(req._magic);
  return res.send('Hello World!');
});

// Start server
app.listen(3000);

Move the middleware stack above all the routes definition to get it all working.
I'm on express 2.5.2

Hm, true :| forgot about that.
Maybe a quick note could be added in the documentation about it then, just so that the behavior is clearly stated somewhere. (I can do it if it's the way to go.)

yeah ill be sure to make it clear on the new site. it really is essentially the same as doing:
var router = new Router;

router.get(...);
router.get(...);
app.use(router);

just sugar really

Perfect, thanks!

Noted :) thanks for the follow up!


[BUG] encode error in gpt ask

when I use apt ask prompt_file, I upload excel or pdf file, it will raise the error. Could you please help deal with this problem?

You cannot use excel or PDF files with prompt_file, only text file


adding delays between routing tests

Running presubmits at https://istio-testing.appspot.com/job/manager/job/presubmit/287/ ...

manager/presubmit passed. Details at https://istio-testing.appspot.com/job/manager/job/presubmit/287/.


Downgrade xz: 5.6.1 -> 5.4.6

Description of changes
This downgrades xz to 5.4.6, the last version before a backdoor was added to it. This is unlikely to be properly exploitable on NixOS, in a practial manner at least, since sshd doesn't link against xz/liblzma.
Things done

Built on platform(s)

[ ] x86_64-linux
[X] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[X] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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.

https://github.com/NixOS/nixpkgs/pull/300028


List of related videos

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Hello! Is there at the moment a possibility to get a list of related videos for a given video without downloading anything?
Thanks!
Provide verbose output that clearly demonstrates the problem

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

Complete Verbose Output
[debug] Command-line config: ['-vU']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (pip)
[debug] Python 3.9.2 (CPython x86_64 64bit) - Linux-5.10.0-21-amd64-x86_64-with-glibc2.31 (OpenSSL 1.1.1n  15 Mar 2022, glibc 2.31)
[debug] exe versions: ffmpeg 4.3.5-0, ffprobe 4.3.5-0, phantomjs broken, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.9.7, brotli-1.0.9, certifi-2020.06.20, mutagen-1.45.1, pyxattr-0.7.2, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Extractor Plugins: KodicIE, porndeadIE
[debug] Loaded 1788 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>

No

yt-dlp can get information about video, playlist, channel, notifications, subscriptions, history from youtube, but can't get list of related videos?


Incremental building

How can Nix do incremental builds?
Could include:

Derivation-based incrementality
The new pkgs.checkpointBuildTools
Ccache
Experimental Dynamic derivations
edolstra's nix-ccache

We decided to focus on this topic for this weeks Nix Hour, this Tuesday at 18:00 CET! If anybody has any extra questions about this topic, either:

Ask them in this issue in advance with a comment
Join the livestream and use the chat to ask questions live

Questions:

Even with 100% of the inputs cached it seems like the evaluation of a derivation can take quite some time - why?
Understanding the build output in more detail would be nice.
[1/2/37 built, 14/42/66 copied (86.9/251.3 MiB), 24.1/59.6 MiB DL] fetching zstd-1.5.5-man from https://cache.nixos.org
Especially the 1/2/37  and 14/42/66  in the above case.
What's the best way to minimize builds from source on deployments (is cachix the only solution?)

@tcurdt We touched upon your questions at 48:26 :)
Since we didn't get through everything, we'll focus on this topic again for this weeks Nix Hour, this Tuesday at 18:00 CET! If anybody has any extra questions about this topic, either:

Ask them in this issue in advance with a comment
Join the livestream and use the chat to ask questions live

@tcurdt We touched upon your questions at 48:26 :)

Sorry, I didn't make the live steam.
But that was super helpful. Thanks!
Still so much more to learn 🙄


Issues when splitting windows

When using startify with lightline.vim installed, an error occurs when splitting the window.
How to reproduce:

Start vim with startify and lightline enabled
Split the window (any of sp and vsp)

Error:
Error detected while processing function lightline#update[9]..lightline#statusline[2]..lightline#highlight:
line   18:
E254: Cannot allocate color guibg=

Not sure where this bug belongs exactly, so duplicated for both plugins:
https://github.com/mhinz/vim-startify/issues/254

I cannot reproduce this problem with following configuration.
set nocompatible
filetype off
set runtimepath+=/path/to/lightline
set runtimepath+=/path/to/startify
set background=dark
set laststatus=2
filetype plugin indent on
syntax enable

vim -u /path/to/the/above/vimrc
:vsp

Looks like this happens even without startify, but I really cannot track it down. My dotfiles are here: https://github.com/n9code/dotfiles

https://github.com/n9code/dotfiles/blob/313cdf278654b33a08e2aed409615f44ed6eee8b/.vim/autoload/lightline/colorscheme/solarized_dark_n9code.vim#L60-L62
Here you have to specify gui colors. lightline does not accept abbreviation like this so that colorschemes should be available for both cui and gui environments.

Oh man, you saved my day, thanks a lot @itchyny!
Off-topic: After fixing this, I started to get the following (the ^s instead of spaces):

Could this be because of :help fillchars?

Any one that is omitted will fall back to the default.  For "stl" and
"stlnc" the space will be used when there is highlighting, '^' or '='
otherwise.

If so, what would be a workaround?

Hmm, looks weird. You seem to be specify spaces here. I'm not sure but the highlight configuration here can be related to it.

yep, that's what I'm concerned about. However, setting ctermfg and ctermbg colors to different values, does not help 😕

Can you test what it looks like without lightline?

With bg and fg set to the same color:

lightline specifies different colors for bg and fg so can you try with the different colors?

I explained it wrong. The screenshot I posted is with lighline turned off and with:
hi StatusLine ctermbg=10 ctermfg=10 cterm=NONE guibg=NONE guifg=NONE gui=NONE
hi StatusLineNC ctermbg=10 ctermfg=10 cterm=NONE guibg=NONE guifg=NONE gui=NONE

Actually, the original reason I touch StatusLine and StatusLineNC is this effect I want to avoid (see left to NORMAL):

The last character of inactive window status line (or first character of active status line) is highlithed with the default StatusLine group (or default StatusLineNC group).

Okay. How about using cterm=bold instead of cterm=NONE for StatusLine (only, not for StatusLineNC)? It will make no differences in the appearance.

@itchyny it worked with this config 👍 :
hi StatusLine ctermbg=10 ctermfg=10 cterm=bold guibg=NONE guifg=NONE gui=NONE
hi StatusLineNC ctermbg=10 ctermfg=10 cterm=NONE guibg=NONE guifg=NONE gui=NONE

Thanks.


[QUESTION] How can I user a "private" variable (_var) as a field on models

Description
I'm using mongoDB an I want to match the internal id of mongo documents _id to a model field, so my model and route looks similar to:
from typing import List

from fastapi import FastAPI
from pydantic import BaseModel

app = FastAPI()

class Item(BaseModel):
    name: str
    description: str = None
    price: float
    tax: float = None
    tags: List[str] = []
    _id: str

@app.get("/items/{_id}", response_model=Item)
async def get_item(_id: str):
    # Getting the item from DB
    item = db.item.find_one({"_id": _id})
    return item

So I have notice that the variable _id is omitted on parsing from the model so not included in this response.
So I was wondering if it is possible to include it somehow?
Checking Pydantic field parsing I found:
https://github.com/samuelcolvin/pydantic/blob/8c0f805d15d6fddca20ec691c6299a92e1ab15c1/pydantic/main.py#L749-L750
So probably is on Pydantic side?

Yep, what @Slyfoxy said :point_up:
Thanks for the help @Slyfoxy ! :clap: :cake:


Add Step for SSH Server to README

If the curl command runs on the pi without the pi managing the SSH session, it will time out. Add the raspi-config step to enable the SSH Server.

No need anymore, there is a page about it on the Engine webserver wiki

No need anymore, there is a page about it on the Engine webserver wiki


Odometer

Adding funtion:  transfer the odometer data  from smart car to Android APP, and display it.

guscuizi requested a review from platisd

I've already left my review. Please address the comments.

Hi,
I added this code by mistake, just delete it, it won’t affect the software function.
I will also delete it, in the project.
Sincerely,
Zizhuang Cui
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Dimitris Platis @.>
发送时间: Saturday, July 31, 2021 12:20:09 AM
收件人: DIT112-V21/group-17 @.>
抄送: guscuizi @.>; Author @.>
主题: Re: [DIT112-V21/group-17] Odometer (#174)
@platisd commented on this pull request.

In android/app/src/main/AndroidManifest.xmlhttps://github.com/DIT112-V21/group-17/pull/174#discussion_r680053118:

@@ -1,7 +1,8 @@

<manifest xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:tools="http://schemas.android.com/tools"

package="com.delirover.app">

package="com.delirover.app"
tools:ignore="MissingLeanbackLauncher">

What is MissingLeanbackLauncher and why should it be ignored?
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/DIT112-V21/group-17/pull/174#pullrequestreview-719267697, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATMHVNGEFTPKUEXDUWBN7E3T2LGLTANCNFSM5BIX2FPQ.

I added this code by mistake, just delete it, it won’t affect the software function.
I will also delete it, in the project.

First of all, I don't understand what code you are referring to. Please go to the pull request on GitHub and respond to the comments there.
Secondly, this is a code review for you to fix the problems, not me. This is a process you should have already been familiar with...

Create another pull request towards the master branch so it can be properly reviewed please...

Okay, I will do this, thank you for your patience.
Sincerely，
ZiZhuang Cui
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Dimitris Platis @.>
发送时间: Monday, August 2, 2021 4:13:26 PM
收件人: DIT112-V21/group-17 @.>
抄送: guscuizi @.>; State change @.>
主题: Re: [DIT112-V21/group-17] Odometer (#174)
Create another pull request towards the master branch so it can be properly reviewed please...
―
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/DIT112-V21/group-17/pull/174#issuecomment-890819523, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATMHVNATB3ZCFIB33XCCPWDT2ZHSNANCNFSM5BIX2FPQ.


rescue any error that occurs deleting the file

The error occurs whenever you restart a job and can mask other exceptions being thrown in other parts of the code.

@awead added the error message to the rescue logging, since we are rescuing one line I do not think we need a backtrace.


Firing

when firing can do you have to press the pause button or can you still use left click?

Thank you. I went through the code and it seems to be one of the best recoil scripts out for PUBG :)

Right, if it doesn't work, try to open the logitech software with administrator privileges. Or if it doesn't work well, just adjust the targeting sensitivity to an appropriate value.


The aurelia-breeze.d.ts file is missing after installing

I used the navigation skeleton for asp dot net to build an app, and it runs well, then I want integrate breeze. I followed the guide but get an error : cannot find the module 'aurelia-breeze', according to other issues, I am sure it is caused by the missed ts file, but I cannot find it, is there a way to make work? thanks.

find it, thanks:
https://github.com/jdanyow/aurelia-breeze/blob/master/dist/aurelia-breeze.d.ts


AttributeError: 'CutSet' object has no attribute 'find'

Traceback (most recent call last):
File "/workspaces/vall-e/egs/libritts/bin/tokenizer.py", line 204, in
main()
File "/workspaces/vall-e/egs/libritts/bin/tokenizer.py", line 146, in main
cut_set = CutSet.from_manifests(
File "/root/miniconda3/envs/valle/lib/python3.10/site-packages/lhotse/cut/set.py", line 317, in from_manifests
return create_cut_set_eager(
File "/root/miniconda3/envs/valle/lib/python3.10/site-packages/lhotse/cut/set.py", line 2937, in create_cut_set_eager
supervisions.find(
AttributeError: 'CutSet' object has no attribute 'find'
getting this error when I try to run
bash prepare.sh --stage -1 --stop-stage 3

Is it the dataset in egs/?

I solved this issue here
https://github.com/lhotse-speech/lhotse/issues/1024


(Suggestion): create a "cycle through themes" command.

Thanks for this plugin. I have been playing with the appearances over the past few weeks and jumping into settings to change themes is rather slow and cumbersome. So I am grateful for the developer creating this theme.
I don't know if other users would find this useful, but as I only have 3 themes installed ( default, primary, minimal) and I only use dfeault and primary, it would be smoother to use a command to cycle through those themes. And I would map the command to a hotkey.
otherwise, two thumbs up from me, thanks! 👍 👍

This, and I would also like to be able to assign hotkeys to specific themes.


Issue when a query string is used on url without trailing slash

Using Angular 1.4 or 1.5 in an asp.net mvc site
I have a base tag:
<base href="/fr/">
and html5mode is set.
The site works perfectly. I use ui-router... no issues.
But today I hit one and this is when I try to browse to this kind of url:
http://example.com/fr?arg=1
I get an error from angular:

url is undefined
trimEmptyHash@http://............/angular.js?tag=2016020919:12165:3

If I navigate to /fr/?arg=1 (with the trailing slash) the error goes away.
In this context, is this a bug? How to get the query string to work without a trailing slash? "Normal" people typing a url normally will never put a slash before the question mark...
Note: I don't know if this is a ui-router or an angularjs issue. I checked this and tried the 2 mentioned solutions:

strictMode to false didn't change anything
even if I use the latest ui-router, I tried using a rule. A breakpoint inside the rule is not even reached and so the problem is still there.

In this context, is this a bug?

It's hard to tell.. I'll need more information.  How can I reproduce this?  Can you post a reproduction in a plunker?

Seems I can't. Tell me if I'm wrong, but how can I create a plunkr to simulate what happens when a url is typed on the browser address bar?
That said, I now think the issue is more on the angularjs side.

Closing due to inactivity


camellia-redis-proxy support http api for redis commands

Module: camellia-redis-proxy
Content: support http api for redis commands, ref: webdis

http api see redis.io


[Bug]: The duplicated application has a master branch details instead of default branch

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The duplicated application has a master branch details instead of default branch
Steps To Reproduce

Connect the application to the git
now create a new branch
From git hub change the default branch to the newly created branch b1
Now come back to the application sync the branch
now master branch will be removed as a default branch
Now go to the home page
duplicate the application
The duplicated application has the details of master branch instead of default branch

Public Sample App
No response
Version
release

Same issue with the json import too


Quest giver npc in worldguard regions

Quests by a npc cant be taken in worldguard regions.
"cant use that here"

This is intentional. Your 'use' flag (or possibly another flag) is set to DENY for your test region, or perhaps globally. You will have to change this within your WorldGuard region(s) to take quests.


fix: use start and end times from langchain

this is necessary because the timing is no longer real when the handler is called in a separate thread

@RogerHYang add a bit more color that you need to respect the timing from langchain when running in a separate thread.
Also is this testable? Not blocking but would be good to have some coverage at some point.


[FEATURE] Please, fix the permanent visibility of "Keywords:" subsections on the resume.

Edit: Wrong type of issue.

Closing.


Select input

Still to do:

default story of selectInput uses a dropdown - I will move it as soon as we have a select component to a dedicated component

Also, there's an error I also see in the stories all the time. Not sure why it's not picking up the defaultProps?

Awesome 🎉
_Travis says we probably need to update snapshots after merging, I'll take care of that in #28


Question: does not-expanding macros can cause program misbehavior?

I'm currently dealing with a related issue to #8925. After updating rustc and rust-analyzer, my current project which is based on pest-parser, is complaining that the proc macro server crashed. Although my program compiles, it doesn't returns the expected output. I wonder if this rust-analyzer bug might be causing this misbehavior?

rust-analyzer does not perform any disk writes, we call Cargo to build procedural macros, so what you're seeing is probably unrelated (unless there's some incremental compilation issue in rustc)

Since the question is answered, I think we can close this.


Connections

Coverage decreased (-0.2%) to 68.697% when pulling 117cf6b4af7d880ab55b763c000ebb5bc5f52a92 on connections into 9b5eb11817902677713679f169010a6ab9482e5c on dev.


Remove the Glowing strips in Documentation

these diagonal strips make the documentation harder to read.
Even though its a nice touch for the site, it distracts users from reading the docs. Especially when scrolling.

Thanks for the feedback. The background color has been removed. Thank you for using farm.

I think it crashed the website

I just tried to fix this problem. You can try it again

I've requested access to the deployment preview to see if it works :)


Question about visibleTimeStart

Hello and thank you very much for this cool component !
I have question about how to use visibleTimeStart and defaultTimeStart.
In my project i want to limit Timeline just for one day. I set visibleTimeStart(or defaultTimeStart), but after component render first time i see a day before visibleTimeStart(or defaultTimeStart). Once i force to re-render component (for example just resize browser window) i see the right date. I tried to use fixed date or just "today" date but got the same problem.
How can i fix it or force component to update again after start ?
First render

After re-render

@KuroiNeko6610 you need to set both the visibleTimeStart and visibleTimeEnd. Try it and let me know if this fixed your issue.

@Ilaiwi Unfortunately the issue still persists even after setting those props.

@florinciubotariu please provide a codesandbox example so I can help you

@florinciubotariu @Wax6610 apparently this was a bug that i didn't know about. I fixed it in 0.26.6
example here
https://codesandbox.io/s/timeline-demo-controlled-visible-time-no-scroll-659jb

question still exists in example https://codesandbox.io/s/timeline-demo-controlled-visible-time-no-scroll-659jb


Expand sorting for parameter tuning

@bendichter Part (b) on the current to-do list.
Since the changelog is unreadable and the notebook may not render on GitHub, here's a screenshot of the changes.

Simply showcases how to use the spikesorters.run_sorters() multiple times over the same method, but with different parameters over a pre-defined mesh. The user would then afterward be expected to define and setup their own custom cost functions for evaluating the output of each iteration, but this I will leave to them.

# define parameters
vary_sorter_params = dict(
    # Sorter name is the first key
    ironclust=dict(
        # Parameter name is the second key
        # It's value is a list, ranging either numerical values or string options
        adjacency_radius=list(range(25, 80, 5)),
        feature_type=["gpca", "vpp"]
    )
)

# create gridsearch
all_runs_params = list()
for sorter_name, sorter_values in vary_sorter_params.items():
    this_run = dict(sorter_name=sorter_name)
    for args in product(*[x for x in sorter_values.values()]):
        for arg, key in zip(args, sorter_values.keys()):
            this_run.update(key=arg)
    all_runs_params.append(this_run)

# run sorters
sorting_outputs = list()
for param_iter in all_runs_params:
    sorter_name = all_runs_params["sorter_name"]
    sorter_params = ss.get_default_params(sorter_name)
    sorter_params.update(param_iter)
    sorting_outputs[sorter_name].append(
        ss.run_sorters(
            sorter_list=sorter_name,
            recording_dict_or_list=dict(rec0=recording_processed),
            working_folder=spikeinterface_folder / "working1",
            mode="overwrite",
            sorter_params=sorter_params,
            run_sorter_kwargs=dict(raise_error=False)
        )
    )

Implemented the changes as described; search grid now forms at earlier cell, user may inspect and modify before it passes into the sorting loop.


drivers: display: stm32_ltdc: fix set orientation API

Fixes orientation set return value for normal display orientation.
Refactors invalid switch into if statement.

@tmilkovic51


[ruby] Instrument service with Sentry

Feature Request
Once the Ruby SDK work is in a first working state, lets add it to the relevant service(s) here for dogfooding and demo purposes
Describe the solution you'd like:

[x] Project in Sentry SDKs org (slug: sentry-sdk )named with opentelemetry-demo-SERVICE_NAME
[x] add Sentry SDK to service
[x] merge and deploy

Describe alternatives you've considered.
some services may be relatively simplistic, let's consider ways for relevant platforms to add beefier demo examples

this is a nice to have to be clear

Additional Context
this demo assists with us assessing how well the data works with the Sentry product.

performance
tracing
vitals
links to errors
releases and so on

all will not work perfectly but gives us a vehicle by which to asses. Further this demo will become a powerful to share with users

https://github.com/getsentry/sentry-ruby/issues/1957
done and deployed but distributed tracing in the deployed demo site cannot be confirmed, as well this issue where the transaction do not appear to be filled correctly. Could be an issue with gRPC vs HTTP but we should confirm


Need a help in text i want  to align in vertical form is it possible???? if yes please do let me know

Thank you for submitting an issue to jsPDF. Please read carefully.
yes I am using the latest version
Have you tried using jspdf.debug.js?
No
Steps to reproduce
Ideally a link too. Try fork this http://jsbin.com/rilace/edit?html,js,output
What I saw
doc.text(15,52,order details);

What I expected
I want this text in the vertical form in that column!!!!!!!!!!!!!!!

Hey @pranaydp  and thanks for posting this! This issue looks like a question that would be best asked on StackOverflow. StackOverflow is amazing for Q&A: it has a reputation system, voting, the ability to mark a question as answered. Because of the reputation system it is likely the community will see and answer your question there. This also helps us use the GitHub bug tracker for bugs only. Will close this as this is really a question that should be asked on SO.

got a solution thanx


Update camera.cpp

According to page 84 - page 86 of slambook, The relationship betweent pixel coordinate and World coordinate is u=fx*(X/Z)+cx; v= fy*(Y/Z)+cy.  But in this function, It represent the relationship of Camera coordinate and pixel Coordinate?

Nope. Actually we define three coordinates: the pixel, the camera and the world. From pixel to camera we check the intrinsics like fx, fy, cx and cy. From camera to world we need extrinsics like the rotation and translation. In some chapters of this book where we only have two frames, we may call the first one as the world and compute the transform of the second one.
So here we should just keep the original version. Thank you.


As a User I want all gnfinder options be available

I decided to make gndoc a text-conversion library, so no need for CLI or name-finding anymore


Request body logging isn't working for server requests

Hello everyone,
Trying to use request body logging in some cases, but it doesn't seem to be working
request := sentry.Request{}
request = request.FromHTTPRequest(req)

request.Data is always empty :(
I guess a problem can be here
if request.GetBody != nil {
		if bodyCopy, err := request.GetBody(); err == nil && bodyCopy != nil {
			body, err := ioutil.ReadAll(bodyCopy)
			if err == nil {
				r.Data = string(body)
			}
		}
	}

request.GetBody is always == nil and Doc says

For server requests, it is unused.

request.Body looks like a right field.

up

I'm having the same issue

@flash286 @aschade92 I'll take a look at it in the morning

@kamilogorek let me know if it's a bug. I'm ready to make PR.

Hi @flash286! I can confirm this is a bug, thank you for reporting it 😄
Please go ahead and send in a PR, I'll be happy to help you get that merged. Thanks!


DCOS-22454: improve performance of composite store nodes list getter

Checklist

[X] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

How to review

Check that we did not break the tasks list by accident (or anything else using the composite state store (e.g. Nodes View))
Add console.count to the getter and see the amount of calls before and after the change

This repo has @mesosphere-mergebot integration. You can interact with the following commands.
@mesosphere-mergebot integrate
@mesosphere-mergebot ship-it
@mesosphere-mergebot test

@orlandohohmeier @juliangieseke Could you take another look. I think it is a bit messier, but I understand that we won't forget about adding some of the update calls in the future.

@orlandohohmeier Yeah, deferring the computation seems to make sense, took your code now 👍


[READY] ci: gpuocelot cache, remove fast-apt

The gpuocelot install consistently triggered a re-compilation (see for example this job).
On this MR the build is cached.
restore-keys was superfluous, but I am not certain if that is the root issue. I suspect it was the extra slash behind the cached folder.
Was 2m 44s, now it's 1s.
For faster C++ builds (everywhere), we could consider ccache and mold, but they are not necessary here.

So this is trading complexity for potential speed. How much speed? It might not be worth it.

So this is trading complexity for potential speed. How much speed? It might not be worth it.

This seems to me to be a bugfix of the original implementation. Saves around 2m30s on the cuda tests.

Now that's a nice PR!


Image Enhancer using Python

Name of your Project: Image Enhancer using Python
Brief Summary:
Image enhancer using Python imaging library in python. A simple image enhancer code where you can add any image by just changing the input file name and can adjust the enhancement scale also.
Issue no. : #44
Self Check:

[x] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My code is in proper folder with proper syntax (E.g. Password_generator)
[x] I have clean code and Documentation(README.md, requirements.txt) and my code is well linted with flake8.

NOTE: If issue was not assigned to you it'll become invalid.

@Divya2163 kindly rename requirements.md to requirements.txt and try to make it in a format with flake8lint

@palanioffcl Made the required changes, kindly check


pulumi: init at 0.16.2

Motivation for this change
Things done

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

[ ] NixOS
[x] macOS
[ ] other Linux distributions

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

i tried to build from source but it does so much crap with pip + yarn and maybe more that i gave up and fallback to just fetch the binary

@GrahamcOfBorg build pulumi-bin

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: pulumi-bin
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: pulumi-bin
Partial log (click to expand)
shrinking /nix/store/05zjpida7yzvsfmy2lsllh4rzj2szvbk-pulumi-0.16.2/bin/pulumi-language-nodejs
shrinking /nix/store/05zjpida7yzvsfmy2lsllh4rzj2szvbk-pulumi-0.16.2/bin/pulumi-language-go
shrinking /nix/store/05zjpida7yzvsfmy2lsllh4rzj2szvbk-pulumi-0.16.2/bin/pulumi-language-python
shrinking /nix/store/05zjpida7yzvsfmy2lsllh4rzj2szvbk-pulumi-0.16.2/bin/pulumi
strip is /nix/store/n4hb93w6j076xcjw5pm09rdmc09s075b-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/05zjpida7yzvsfmy2lsllh4rzj2szvbk-pulumi-0.16.2/bin
patching script interpreter paths in /nix/store/05zjpida7yzvsfmy2lsllh4rzj2szvbk-pulumi-0.16.2
/nix/store/05zjpida7yzvsfmy2lsllh4rzj2szvbk-pulumi-0.16.2/bin/pulumi-resource-pulumi-nodejs: interpreter directive changed from "/bin/sh" to "/nix/store/yal8k8rgbaj8p9vi08q1kh21ls0ndq95-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/05zjpida7yzvsfmy2lsllh4rzj2szvbk-pulumi-0.16.2...
/nix/store/05zjpida7yzvsfmy2lsllh4rzj2szvbk-pulumi-0.16.2

Success on x86_64-darwin (full log)
Attempted: pulumi-bin
Partial log (click to expand)
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
strip is /nix/store/mvpvjar6m4jpjcz48715w2pax53djv6g-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/4f6ggf2ii4daxwbb5r4l4cr2g7ff4wjy-pulumi-0.16.2/bin
patching script interpreter paths in /nix/store/4f6ggf2ii4daxwbb5r4l4cr2g7ff4wjy-pulumi-0.16.2
/nix/store/4f6ggf2ii4daxwbb5r4l4cr2g7ff4wjy-pulumi-0.16.2/bin/pulumi-resource-pulumi-nodejs: interpreter directive changed from "/bin/sh" to "/nix/store/sv35yjk452mggg76bz94nppibj96a66h-bash-4.4-p23/bin/sh"
/nix/store/4f6ggf2ii4daxwbb5r4l4cr2g7ff4wjy-pulumi-0.16.2

@peterromfeldhk Thank you! :)


How do I can manage load balancer tagets via api

I want to dynamically getList add and remove targets on load balancer. Is any api exposed for it?

To get: https://rancher.clicman.ru/v1/serviceconsumemaps?limit=-1
To set: {{RANCHER_URL}}/v1/loadbalancerservices/1s4/?action=setservicelinks
Set request is simple and clear, but Get is to complicated.
It will be good to see simple load balancer configuration... maybe in rancher-cli?

An example of implementation here

Are you still looking for help on this? I wasn't clear on what the current request is.

Oh sorry I should close this issue. Iveimplemented my own cli :)


Building successfully but not generating gem5.opt or gem5.fast  or gem5.debug

Command used for building:
scons  build/RISCV/gem5.opt -j12
Linux version:
NAME="SLES"
VERSION="15-SP5"
VERSION_ID="15.5"
PRETTY_NAME="SUSE Linux Enterprise Server 15 SP5"
ID="sles"
ID_LIKE="suse"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:suse:sles:15:sp5"
DOCUMENTATION_URL="https://documentation.suse.com/"
Already tried methods:

using git clone and build
downloading and unzip the .zip file and build
Deleting the build directory and rebuild gem5
building gem5.opt / gem5.fast / gem5.debug

Could you please help me with this?

Hi @soniab, when including terminal output in a message, please copy and paste the text rather than attaching a screenshot. Thank you!
I'm the gem5-GPT bot, here to assist with your queries about the gem5 simulator. Please note that while I aim to provide accurate information, my responses could be wrong or may have been slightly edited for clarity. You can access the gem5-GPT on chat.openai.com/g/g-o3y1y4t8J-gem5-gpt if you subscribe to chatGPT.
It seems like you've been trying multiple methods to build gem5, but you're not getting the expected gem5.opt, gem5.fast, or gem5.debug binaries after running the build command. Let's troubleshoot and identify the possible reasons for this issue, followed by solutions.
Steps to Investigate and Resolve:

Verify the SCons Version:
Make sure you have an appropriate version of SCons installed. gem5 requires SCons 3.0.0 or later. Run the following command to check the version:
scons --version

If the version is outdated, you can install a newer version using the package manager or pip:
sudo zypper install scons

Or with pip:
pip install --user scons

Check for Errors in the Build Process:
The build command you are running (scons build/RISCV/gem5.opt -j12) is correct for generating the optimized binary (gem5.opt), but if there were errors during the build process, it may not generate the binaries.
Re-run the build command, but this time, use the --debug=explain option to see more detailed information about what is happening during the build process:
scons build/RISCV/gem5.opt --debug=explain -j12

Check if any errors or warnings are being reported.

Check the Build Directory:
The binaries (gem5.opt, gem5.debug, gem5.fast) should be located inside the build/RISCV directory if the build is successful. Run the following command to see if any files are being generated:
ls build/RISCV/

If you don't see any of the expected binaries (gem5.opt, gem5.debug, or gem5.fast), this suggests an issue during the build.

Ensure the Appropriate Toolchain is Installed:
Since you're building for the RISCV architecture, make sure you have the appropriate cross-compilation toolchain installed. For SUSE Linux Enterprise Server, you can install the necessary toolchain by running:
sudo zypper install gcc gcc-c++ make python3
sudo zypper install gcc-riscv64-linux-gnu

Verify that the riscv64-linux-gnu-g++ compiler is available and working:
riscv64-linux-gnu-g++ --version

Verify SCons Target Architecture:
Ensure that you are targeting the correct architecture. You mentioned using RISCV, but double-check that the architecture folder is correct. You can list available architectures using:
scons -Q

Ensure that you're specifying the correct target in your build command. If it needs to be corrected, change the target as follows:
scons build/ARCH/gem5.opt -j12

Replace ARCH with the appropriate architecture, such as RISCV, X86, or others as per your configuration.

Clean Build and Logs:
Try doing a clean build and capture the log output:
scons -c
scons build/RISCV/gem5.opt -j12 > build_log.txt 2>&1

Afterward, review build_log.txt for any errors or warnings that might have been overlooked.

Dependencies and Libraries:
Make sure that you have installed all necessary dependencies for gem5 on your system. For SUSE Linux, the following packages are generally required:
sudo zypper install gcc gcc-c++ python3-devel libzstd-devel libboost-devel

The libboost and libzstd libraries are commonly used in gem5, and missing them can cause builds to fail.

Check for Available Memory:
Since you are using 12 build threads (-j12), it’s important to ensure that your system has enough memory to handle the parallel compilation. If your system runs out of memory, the build might silently fail. Try reducing the number of jobs:
scons build/RISCV/gem5.opt -j4

Summary of Steps:

Check SCons version and upgrade if necessary.
Re-run build with --debug=explain to gather more information about the build process.
Ensure the correct toolchain for RISCV is installed.
Check that the riscv64-linux-gnu-g++ compiler is functioning.
Clean the build directory and recompile while capturing logs.
Ensure you have enough memory for the build process and dependencies are installed.

Once you have more detailed build logs or outputs from any of the steps, it will be easier to narrow down the root cause if the issue persists.


Adding FunctionK values for Task, Coeval

I'm using Coeval lately, for describing resources like Resource[Coeval, ?] and then I'm converting those to Task when needed. This is because you may want to initialize a resource in a dirty context, like Play Framework's dirty dependency injection and lifecycle mechanism and working with Coeval there is easier and more truthful.
But then doing a mapK to convert from Resource[Coeval, ?] to Resource[Task, ?] is freaking painful. Therefore we now have toK to help:
val res: Resource[Coeval, Something] = ???

res.mapK(Coeval.toK[Task]).use { r =>
  Task(???)
}

For this I also added the lawless CoevalLift typeclass, to mirror the lawless TaskLift. So now we have a generic to[F[_]] method on Coeval too:
coeval.to[IO]

coeval.to[Task]

I no longer see the point of .toIO and .toEval, because they are not special, so I deprecated them.

TBH, I'm not very big fan of lawless typeclasses, and wins overres.mapK(FunctionK.lift(Task.coeval)) seem marginal, though I'd take to[F[_]: Sync] over toIO.

@oleg-py I'm not a fan of lawless type classes either, but due to all the restrictions in Cats-Effect's Bracket and Sync, there are now many interesting types that are left out.
Eval and Observable are two such examples.

@oleg-py reliance on FunctorK.lift doesn't cut it, as you can easily end up with an error like this ...
missing parameter type for expanded function ((x$1: <error>) => x$1.task)
[error]       .mapK(FunctionK.lift(_.task))

The type inference does work with FunctionK.lift(Task.coeval), but I don't think many people will know that. And there is no such function already described for cats.effect.IO.
Will take your suggestions under consideration. I especially like the idea of making the *Lift type classes inherit from FunctorK.

@alexandru is it ready to merge? I could do the realese tomorrow evening :)

@Avasil yes, it's ready


mp3, ogg, webm, wav, aac... for audio

mp3, ogg, webm, wav, aac... for audio

This issue was imported from Google Moderator
Moderator votes: +7
Issue added by mh_nichts on 2014-09-02
To vote this issue up or down, simply include +1 or -1 in your comment.

+1

Duplicate of #988 so moving half the votes there.


Active characters in JSON file break things

Active characters in the JSON source break things, but escaping them with just a single backslash is not proper JSON.
Maybe replace these characters by their TeX escape sequence immediately after reading the file contents?

Probably better to rethink parsing:

Change cctab, so that everything is catcode 12 and only upper-half of 8-bit range is active. (Backslash active?)
Rewrite parsing of objects (similar to how arrays are parsed).
Rewrite parsing of \", \b etc.

As of version 0.8, the package has been largely rewritten in order to cope with these things. When reading JSON data, it is now parsed as string with all characters catcode 12, except for space (catcode 10) and backslash (catcode 0). Additionally, in order not to break UTF-8 support of PDFLaTeX, the upper half of the 8-bit range is set to active (onyl for PDFTeX).
The parsing of objects now works in an analoguous manner as parsing of arrays, which allows to omit the use of catcode 1 and 2.
Before typesetting, JSON escape sequences are replaced by relevant TeX code. Then, characters are rescanned with the default TeX catcodes and macros.
Escaping is now active per default. The starred variants output the value without rescanning.
A new escape sequence \x has been introduced that allows to nest JSON strings.


fixed main_megamenu.less for compiling

Errors because of auto-formatting of comments that broke the less compiling

Thanks for this contribution.
I will check why the boosted syntax checker let it passed thru


PHPLIB-462: ensure compatibility with PHP 7.4

While we're not going to be running any PHP 7.4 features for a while, we at least want to make sure that we test the library on PHP 7.4. This requires upgrading to newer PHPUnit releases to avoid errors due to the deprecation of the __toString methods in reflection classes.
Upgrading to PHPUnit 7 or newer is non-trivial, as support for PHP 5 was dropped and PHPUnit introduces type hints. To cope with that, we use symfony/phpunit-bridge to provide compatibility layers for the TestCase class. Once https://github.com/symfony/symfony/pull/33078 is merged, we can also drop the constraint traits from this packages. However, since the traits are marked as @internal we don't need to wait for a merge of that pull request and can drop them once they're available in symfony/phpunit-bridge.

symfony/symfony#33078 appears to have been merged already.

Yes, I originally included the code I PR'ed to Symfony in this library, but now that it has been merged we no longer need to include it ourselves.

Was that related to using ^4.4@dev in composer.json, or do we still need that until a tagged release is made upstream?
Symfony 4.4 will be released in November, so we have to use an @dev release until then. I don't like relying on pre-release versions, but in this case it's the only option we have if we want to test on PHP 7.4 without PHPUnit causing deprecation errors and tripping over them itself. PHPUnit 7.5.15 will fix this (once released), but that version wouldn't be installable without changes either so I decided to update to the newest version where possible. Once Symfony 4.4 is released, I'll update the version constraint to require a stable release.

Once Symfony 4.4 is released, I'll update the version constraint to require a stable release.

Please make a PHPLIB ticket as a reminder to do so (if you haven't already).

Created PHPLIB-465 to track changing the dev requirement after the release of Symfony 4.4.


Follow-up to "offers"

From #1799

[ ] should you be allowed to edit accepted/archived offers?
[ ] add "loading massive image into canvas" indicator?
[ ] on offer detail page when there is no offer it just shows loader
[x] pagination (infinite scroll, trigger load more, etc)
[ ] multicroppa should have "cursor: pointer" on mouse hover
[x] you get email notified about your own offer https://github.com/yunity/karrot-backend/pull/883
[x] cannot access the conversation you were part of after the offer gets accepted by the owner https://github.com/yunity/karrot-backend/pull/886
[x] remove concept of "accepting" offers - marking as archived is sufficient (see forum thread)
[ ] show generic "this offer is not available" when it cannot be found (see forum thread) - this is either the case of "id does not exist" or "offer is not visible to you"

In the app, choosing a picture directly opens the gallery. It would be nice if it would allow to take a picture from the camera. (That's the same for all Croppa elements.)

Just tested on production and one issue I noticed is that I get an e-mail notification for an offer that I created.

Yes, I noticed this too actually when fixing https://github.com/yunity/karrot-backend/pull/878 - but decided to leave it for another fix. Will fix it :)

I encountered the issue me and @tiltec spoke of at some point - if you're in an offer conversation for somebody elses offer, then they accept/archive it, you can no longer access that conversation (but you still get the conversation notifications). Maybe we can do the permission check to also include offers that you are part of the conversation for, a bit fiddly, but maybe ok....

Maybe it would be nice to have the possibility of creating an offer without the need to upload a picture? Then a default image would need to be in place, maybe the gift icon.
The use case is to make it easier for people to put requests there. Also I suggested a friend to use Offers to keep an update list of things they need at the Bike Kitchen.

This seems fine for now, only minor points left in here -> closing.


Load time delayed for preview CameraKit v0.12.0

Thanks for the new Release.
Is this a bug report?
No. Its Optimization
Have you read the Contributing Guidelines?
No
Environment
Android Studio 3.0
Moto G3
Wonderkiln  - CameraKit v0.12.0
Steps to Reproduce

Capture a photo
Delay in returning a preview of the captured image

Expected Behavior
As soon as the image is captured the method should return the image file for the preview immediately.
Actual Behavior
As of now, there is a delay to see the preview of the image.
 I believe I'm facing the same issue

I had the same issue and finally found the solution for it, kindly have a lock at my post here https://github.com/wonderkiln/CameraKit-Android/issues/235

Thanks for the update @MuhammedRefaat -- we will include a note in the documentation!

The quality of the photo is not good in case of 'still method'. is there any way to make fast the capture in standard mode?

We will be addressing this in 0.13.0 @karandeep26 and we will look at setting sane defaults =)

This issue has been resolved by v1.0.0. If you are still having issues with capture speed in 1.0.0 I will create a new ticket to track it!


Update Characteristic

Hi,
I need help please.
I want to update the characteristic of an accessory. When I print to console I get this:
console.log(accessories[0]);

Accessory {
displayName: 'Kitchen Light',
UUID: 'b9a6b2ff-c36f-4484-b8ac-5d6b26094bfd',
aid: null,
_isBridge: false,
bridged: true,
bridgedAccessories: [],
services:
[ Service {
displayName: undefined,
UUID: '0000003E-0000-1000-8000-0026BB765291',
subtype: undefined,
iid: null,
characteristics: [Object],
optionalCharacteristics: [Object],
_events: [Object],
_eventsCount: 1 },
Service {
displayName: 'Kitchen Light',
UUID: '00000043-0000-1000-8000-0026BB765291',
subtype: undefined,
iid: null,
characteristics: [Object],
optionalCharacteristics: [Object],
_events: [Object],
_eventsCount: 1 } ],
deviceid: 3,
_events:
{ identify: [Function],
'service-characteristic-change': [Function: bound ] },
_eventsCount: 2 }

I just want to update the value stored and push that back to the phone....

on the Characteristic object there is an updateValue function to do this.


Config template

https://github.com/gammasim/gammasim-tools/blob/76c43f1965bddcc9ecd77f2f438861ff34615733/config_template.yml#L6
A few comments regarding the config template (I understand that this is not the final solution):

do we really need the paths on Orel's machine or can we generalize it?
unclear in the docu (https://gammasim.github.io/gammasim-tools/framework.html) if all of these files are needed. Do we e.g., need the files located in modelFilesLocations ? If yes, then we should add this to the installation instructions.
what about using environmental variables for e.g., the simtelPath? This would simplify the settings also for e.g. the Docker file.
do we want to add an example for ./dbDetails.yml ? I cannot see what should be in this file.

Will let Raul comment on the other stuff.

do we want to add an example for ./dbDetails.yml ? I cannot see what should be in this file.

Not sure, it's a private file with a username and password for the DB. I don't see how an example would be useful. Open to discussion though.

Problem is that at this point we don't even know what is expected in the file. So I would prefer an file with empty username/pw field compared to know knowing what is missing.

https://github.com/gammasim/gammasim-tools/blob/76c43f1965bddcc9ecd77f2f438861ff34615733/config_template.yml#L6
A few comments regarding the config template (I understand that this is not the final solution):

do we really need the paths on Orel's machine or can we generalize it?

We can generalize it, no problem.

unclear in the docu (https://gammasim.github.io/gammasim-tools/framework.html) if all of these files are needed. Do we e.g., need the files located in modelFilesLocations ? If yes, then we should add this to the installation instructions.

Right. I'll expand the description and make clear which files area needed for what.

what about using environmental variables for e.g., the simtelPath? This would simplify the settings also for e.g. the Docker file.

I don't like environmental variables and I'm trying to live without them. I like even less the idea of having some paths as environmental variables and some given at the config file.

do we want to add an example for ./dbDetails.yml ? I cannot see what should be in this file.

I think just a skeleton, without any private info, is fine.
I will apply all the changes to the develop branch and merge it in the end, after approaching all the issues/comments.

what about using environmental variables for e.g., the simtelPath? This would simplify the settings also for e.g. the Docker file.

I don't like environmental variables and I'm trying to live without them. I like even less the idea of having some paths as environmental variables and some given at the config file.

There may be a way to catch environmental variables in yml files. I'll have a look at this. If that is possible, I will document it.

But isn’t it easier to 1-3 environmental variables then to work with a config.yml file plus a shell script which sets one single environmental variables?

On 7. Oct 2020, at 11:43, RaulRPrado<EMAIL_ADDRESS>wrote:

what about using environmental variables for e.g., the simtelPath? This would simplify the settings also for e.g. the Docker file.
I don't like environmental variables and I'm trying to live without them. I like even less the idea of having some paths as environmental variables and some given at the config file.
There may be a way to catch environmental variables in yml files. I'll have a look at this. If that is possible, I will document it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

But isn’t it easier to 1-3 environmental variables then to work with a config.yml file plus a shell script which sets one single environmental variables?

It may be easier to implement but it just make more complicated to use it. The thing is that users sometimes don't know if there are only 1, 3 or 15 environmental variables to be set. It's harder to make everything clear for the user with environmental variables.
With the current design, the user will always know for sure that everything he/she needs to setup is there, in one single file, consistently.
I found a way to allow for enviromental variables in the config file. You will be able to set
simtelPath=$SIMTELPATH
if that is easier for you.

@GernotMaier , I did all these changes on the develop branch. If you want to have a look, you can checkout develop , go to docs, make html and open it with firefox build/html/index.html. The gh-pages will only be updated when I merge the branch with the master.
I added

more details to the config part of the docs, specially about the modelFilesLocations (including links to the gitlab).
a comment on evironmental variables (and implemented it).
comments to the config_template.yml
a dbDetails_template.yml file. I just removed the user and password. I don't know if everything else is actually public (@orelgueta ?)

Checked everything, this is good (actually easier to compare the develop with the master branch than to look at the html output)
On 7. Oct 2020, at 16:43, RaulRPrado<EMAIL_ADDRESS>wrote:

@GernotMaier , I did all these changes on the develop branch. If you want to have a look, you can checkout develop , go to docs, make html and open it with firefox build/html/index.html. The gh-pages will only be updated when I merge the branch with the master.
I added
more details to the config part of the docs, specially about the modelFilesLocations (including links to the gitlab).
a comment on evironmental variables (and implemented it).
comments to the config_template.yml
a dbDetails_template.yml file. I just removed the user and password. I don't know if everything else is actually public (@orelgueta ?)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

This should be added to config.yaml?
I tried
simtelPath: '$SIMTELPATH'

But get

python applications/derive_mirror_rnda.py --tel_name North-MST-FlashCam-D --mean_d80 1.4 --sig_d80 0.16 --mirror_list mirror_MST_focal_lengths.dat --d80_list mirror_MST_D80.dat --rnda 0.0075
Traceback (most recent call last):
  File "applications/derive_mirror_rnda.py", line 209, in <module>
    tel = TelescopeModel(
  File "/workdir/external/gammasim-tools/simtools/model/telescope_model.py", line 100, in __init__
    self._loadParametersFromDB()
  File "/workdir/external/gammasim-tools/simtools/model/telescope_model.py", line 221, in _loadParametersFromDB
    db = db_handler.DatabaseHandler(self._logger.name)
  File "/workdir/external/gammasim-tools/simtools/db_handler.py", line 78, in __init__
    if cfg.get('useMongoDB'):
  File "/workdir/external/gammasim-tools/simtools/config.py", line 86, in get
    if config[par][0] == '$':
TypeError: 'bool' object is not subscriptable

On 7. Oct 2020, at 14:23, RaulRPrado<EMAIL_ADDRESS>wrote:

But isn’t it easier to 1-3 environmental variables then to work with a config.yml file plus a shell script which sets one single environmental variables?
It may be easier to implement but it just make more complicated to use it. The thing is that users sometimes don't know if there are only 1, 3 or 15 environmental variables to be set. It's harder to make everything clear for the user with environmental variables.
With the current design, the user will always know for sure that everything he/she needs to setup is there, in one single file, consistently.
I found a way to allow for enviromental variables in the config file. You will be able to set
simtelPath=$SIMTELPATH
if that is easier for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Oh sorry, this was a bug created by the previous change that I fixed on the next commit.  I just pushed itIt shoudl work now

Yes, thanks!
This is nice, as I use the docker image and can set SIMTELPATH automatically (as in the docker image, simtel_array is installed always in the same directory)

On 8. Oct 2020, at 11:31, RaulRPrado<EMAIL_ADDRESS>wrote:

Oh sorry, this was a bug created by the previous change that I fixed on the next commit. I just pushed itIt shoudl work now
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Let's close it?


fix: Correct length check in load_data

修改 FixPacket 检查越界边界条件

duplicated https://github.com/robomaster-oss/rmoss_core/pull/34#pullrequestreview-1345773672.


Update WeDeploy JS API to 3.0.2

LGTM
Is there some other way we should be "notifying" users of version releases besides just updating the docs? Like if there was a cli tool would list or something. I don't know what is common for other organizations to do.

Hey @jonnilundy,
In general, these channels are:

Announcements - Tweeter, Facebook, etc.
Blog posts - especially in case of major feature releases
For some packages and especially CLI tools there might be hints - for example when you run the package, it says: "There is a new version available, type 'npm update -g package-x' to update". Maybe @henvic may consider that for WeDeploy CLI.

Thanks,

Is there some other way we should be "notifying" users of version releases besides just updating the docs?
As a user of a library package I usually don't want to be bothered just because the vendor decided to draft a new release. Unless it is a security advisory.

I could add a command of  we list flag on the CLI to check versions of used Docker images maintained by WeDeploy. Also, I could notify about security advisories on normal usage.
As for notifying about changes on the CLI tool itself, it already does so every other day.

@henvic I like this idea. So to clarify, you could check the image versions but not the api client version?


microsoft docs "Themes" feature unavailable

I noticed the "Theme"  feature missing from .net products documentation I mostly refer.  Please bring back this feature.

The Themes chooser is still available, it just moved to the very bottom of the page next to the language chooser.
/cc @dend

thanks!


v0.138.1

v0.138.1, 17 March 2021

Bundler: [Prerelease] Add a stubbed out native helper for Bundler 2
Bundler: [Prerelease] Allow the v2 native helper to be invoked via an options argument

Looks like https://github.com/dependabot/dependabot-core/pull/3283 is on main, also roll into this release?

Release notes updated 👍🏻


Add AccessOwl Friends of SSO

---
name: AccessOwl
free_sso_providers:
  - Google
  - Slack
scim_pricing:
scim_starting_on_plan:
sources:
  - https://www.accessowl.io/pricing
sso_on_free_plans: true
sso_on_all_paid_plans: true
updated_at: 2024-04-11
vendor_url: https://www.accessowl.io/

Thank you, @PhilipLR !


261 bug closing window event not properly handled

Fix features

Fix deadlock in InputManagement on close window event

Codecov Report

Merging #278 (b8d0a34) into master (c899e58) will increase coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #278      +/-   ##
==========================================
+ Coverage   24.49%   24.51%   +0.02%
==========================================
  Files         113      113
  Lines        2441     2439       -2
  Branches     1356     1353       -3
==========================================
  Hits          598      598
+ Misses       1356     1354       -2
  Partials      487      487

Impacted Files
Coverage Δ

libs/GraphicECS/SFML/Systems/InputManagement.cpp
23.02% <0.00%> (+0.32%)
:arrow_up:

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


Changed "FeatureCollectionLayerFromPortal" to use a different portal item by default

Also added punctuation to comments and updated the year.

@ZackAllen I think you can go ahead and merge this now.


Sample minor improvement: "Click a disk" is clickable in "Implicit animation"

In the sample Implicit animation is written on screen "Click a disk", but this text is clickable as well. Shouldn't it be excluded from widget tree under the GestureDetector?
Here's a fix of this:
import 'dart:math';
import 'package:flutter/material.dart';

class DiscData {
  static final _rng = Random();

  final double size;
  final Color color;
  final Alignment alignment;

  DiscData()
      : size = _rng.nextDouble() * 40 + 10,
        color = Color.fromARGB(
          _rng.nextInt(200),
          _rng.nextInt(255),
          _rng.nextInt(255),
          _rng.nextInt(255),
        ),
        alignment = Alignment(
          _rng.nextDouble() * 2 - 1,
          _rng.nextDouble() * 2 - 1,
        );
}

void main() async {
  runApp(
    MaterialApp(
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        body: Container(
          color: const Color(0xFF15202D),
          child: const SizedBox.expand(
            child: VariousDiscs(50),
          ),
        ),
      ),
    ),
  );
}

class VariousDiscs extends StatefulWidget {
  final int numberOfDiscs;

  const VariousDiscs(this.numberOfDiscs);

  @override
  _VariousDiscsState createState() => _VariousDiscsState();
}

class _VariousDiscsState extends State<VariousDiscs> {
  final _discs = <DiscData>[];

  @override
  void initState() {
    super.initState();
    _makeDiscs();
  }

  void _makeDiscs() {
    _discs.clear();
    for (int i = 0; i < widget.numberOfDiscs; i++) {
      _discs.add(DiscData());
    }
  }

  @override
  Widget build(BuildContext context) {
    return Stack(
      children: [
        const Center(
          child: Text(
            'Click a disc!',
            style: TextStyle(color: Colors.white, fontSize: 50),
          ),
        ),
        GestureDetector(
          onTap: () => setState(() {
            _makeDiscs();
          }),
          child: Stack(children: [
            for (final disc in _discs)
              Positioned.fill(
                child: AnimatedAlign(
                  duration: const Duration(milliseconds: 500),
                  curve: Curves.easeInOut,
                  alignment: disc.alignment,
                  child: AnimatedContainer(
                    duration: const Duration(milliseconds: 500),
                    decoration: BoxDecoration(
                      color: disc.color,
                      shape: BoxShape.circle,
                    ),
                    height: disc.size,
                    width: disc.size,
                  ),
                ),
              ),
          ]),
        ),
      ],
    );
  }
}

You're right, that's a minor-but-legitimate improvement. Thanks for the suggestion.
Because the code for the samples is stored in gists in a dedicated GitHub account, it's not possible for me to ask you to submit a PR for this change, but I can make the change to the gist on your behalf as long as you've signed the Google Contributor License agreement.
Could you sign it at https://cla.developers.google.com/clas and verify for me here that you've done so?

Done: "your CLA has been submitted"

Thanks! I've updated the gist to match. DartPad aggressively caches gists, so if you've already got it loaded into the editor, you may need to hard-reload the page to verify the change. It's in there, though. 😄

Amazing! Thank you ;)


get salt from environment variable pls

u could (and probably should) use environent variables for the other options like admin password too.

it should be a really easy pr to make, i think all you need is std::getenv

k
On Sun, Aug 15, 2021, 1:02 AM Hri7566 @.***> wrote:

it should be a really easy pr to make, i think all you need is std::getenv
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nagalun/multiplayerpiano-server/issues/35#issuecomment-899012411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF4LCHODB2DMTPEA6FWBIK3T45X7VANCNFSM5CCM4PYQ
.
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&utm_campaign=notification-email
.

i made a config.json that should be used instead tbh

but that would make it commit-able to the repo, salts and db passwords and stuff should be in an environment-level config

well dont commit it then ru (gitignore)
but env variable ideal for this


Support generic "fsspec"-like input for data files/stores

It would be nice if we generally support fsspec-style {protocol}{path} (e.g. az://bogus/data.zarr) for input and output data configs. This would make it easier moving data between multiple cloud providers. Users can assume they'd pass in credentials through default environment variables (see #55).
This is more consistent behavior with tools our users are already using. I think didn't set it up this way initially because I was having trouble with adlfs when we first started prototyping the pipeline tools.
Want to get this behavior in before our first release.
Drawback to this is that we become more sensitive to breaking changes in the fsspec-verse -- which is still pretty young.

very cool idea! It would be great to be as cloud provider-agnostic as possible.

Yeah, I need to think through how this influences how we test things. Testing things that I/O from cloud storage can be hard and I'm trying to make it look simple.
It took a bit of getting used to but I've warmed up to how simple it is test services with the 'memory files system', right now. I don't know if you feel the same fondness, @dgergel
If services just take fsspec URLs, they don't need to pass ZarrRepositories through the storage argument... in fact we'd prob just drop that storage arg on services all together. So, then for us to test services I/O we'd have to write files to a temporary directory because I don't know that we can use memory as a fake file system like we're doing now - I need to try it out.
We might move some tests from services and into the core, because it's easier to pass around Datasets (which is how the core works right now).


[Feature Request] Dual page option for landscape mode

I love the app, but when using it on my new tablet, there seems to be a lack of the option to view two pages at once in landscape mode like most other readers allow for. Is there any possibility of adding this feature, or is it already there?

No plans at this time.  Most of the reader code is still Tachiyomi based
https://github.com/inorichi/tachiyomi/issues/149


Introducing Auto-Expiry and Time-to-Birth Features for APKAM Keys

Is your feature request related to a problem? Please describe.
Currently, the APKAM keys do not have an auto-expiry feature. Enhance the APKAM keys by allowing users to set an auto-expiry, after which the keys will no longer be valid for authentication.
Additionally, introduce a time-to-birth mechanism, enabling users to specify when the atKeys will become active, alongside the existing time-to-live functionality.
Describe the solution you'd like

In the enrollment request, allow user to specify "TTL" and "TTB" which represents the time duration in minutes. When TTL represents "time to live", beyond which the APKAM keys will be inactive for usage.
When TTL is met, the APKAM keys cannot be used for authentication and subsequently the enrollment data will be deleted from the keystore.

Pending work on ticket is to publish at_onboarding_cli package.

@sitaram-kalluri at_onboarding_cli version 1.7.0 was published yesterday so it should include your changes. ( Version 1.8.0 will be published once the latest atServer canary release has been promoted to prod and rolled out )


pack error

Warning:com.googlecode.mp4parser.authoring.tracks.mjpeg.OneJpegPerIframe: can't find referenced class javax.imageio.ImageIO
Warning:com.googlecode.mp4parser.authoring.tracks.mjpeg.OneJpegPerIframe: can't find referenced class java.awt.image.BufferedImage
Warning:there were 6 unresolved references to classes or interfaces.
Warning:Exception while processing task java.io.IOException: Please correct the above warnings first.
Error:Execution failed for task ':app:transformClassesAndResourcesWithProguardForRelease'.

Job failed, see logs for details

Please @shiyanjie explain what you tried to do to have this error occurred


Upgrade jQuery to 1.12.4

Performance of the selector engine has improved thanks to a shortcut path that immediately uses precompiled Sizzle selectors when the selector cannot be processed by the native querySelectorAll or matchesSelector methods. This results in a significant speedup in some real-world cases.

Release notes:
https://blog.jquery.com/2016/01/08/jquery-2-2-and-1-12-released/
https://blog.jquery.com/2016/02/22/jquery-1-12-1-and-2-2-1-released/
https://blog.jquery.com/2016/03/17/jquery-1-12-2-and-2-2-2-released/
https://blog.jquery.com/2016/04/05/jquery-1-12-3-and-2-2-3-released/
https://blog.jquery.com/2016/05/20/jquery-1-12-4-and-2-2-4-released/
Similar to previous upgrade jQuery PR:
https://github.com/alphagov/static/pull/628
(Holding off merging until upgrade PRs on other repositories are merged.)

Nice work 👍
I'd like to test it like the previous PR and if possible point https://github.com/alphagov/govuk-component-guide at the integration deployed version of static.

Branch deployed to integration for testing.
I've tested in Chrome:

Collections (browse)
Smart answers
Search facets
Finder facets
Licence finder
Video player
Analytics


nixUnstable: 2.4pre20210908_3c56f62 -> 2.4pre20210917_37cc50f

Motivation for this change
Changes: https://github.com/nixos/nix/compare/3c56f62...37cc50f
I figured that now with an actual 2.4 release around the corner[1] we
could bump it a bit more often considering that it seems to contain
mostly bugfixes, so that upstream receives a bit more feedback.
[1] https://discourse.nixos.org/t/tweag-nix-dev-update-17/15037
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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

[x] Fits CONTRIBUTING.md.

darwin build failed

Failed update system if using custom buildMachines.
My configuration:
  nix = {
    distributedBuilds = true;
    buildMachines = [
      {
        hostName = "aarch64.nixos.community";
        maxJobs = 64;
        sshKey = "/root/.ssh/id_ed2551_aarch64";
        sshUser = "izorkin";
        system = "aarch64-linux";
        supportedFeatures = [ "big-parallel" ];
      }
    ];
  };

Checking with this nix versions: 2.4pre20210917_37cc50f and 2.4pre20210908_3c56f62
Build log:
...
Failed to find a machine for remote build!
derivation: xmycamdgf2rx7gdhffg022z7pqfv8zkz-curl-7.79.0.drv
required (system, features): (x86_64-linux, )
1 available machines:
(systems, maxjobs, supportedFeatures, mandatoryFeatures)
(aarch64-linux, 64, big-parallel, )
...
Failed to find a machine for remote build!
derivation: dlfvvg65bg8zchn4l2015jylh2fvlbgp-git-2.33.0.drv
required (system, features): (x86_64-linux, )
1 available machines:
(systems, maxjobs, supportedFeatures, mandatoryFeatures)
(aarch64-linux, 64, big-parallel, )
...

darwin build failed
git diff btw<EMAIL_ADDRESS>
With luck this build failure is fixed by https://github.com/NixOS/nix/commit/bcd73ebf60bb9ba6cb09f8df4366d5474c16e4a4 and this PR can be updated to include it


TxType RDO is broken

Only DCT is chosen always with speed 0,1,2,3. TxSize is fine.

Seems false alarm.
What is broken is AOMAnalyzer, since it shows DC_DCT 100% only for tx_type histogram.
I uses AOMAnalyzer like this:
/Users/yushin/Downloads/AOMAnalyzer-darwin-x64/AOMAnalyzer.app/Contents/MacOS/AOMAnalyzer  https://people.xiph.org/~mbebenita/analyzer/inspect.js test.ivf


Update some limitations

4 TB is now generally available in selected regions
MAXSIZE default for Standard is 250 GB, not 500 MB
Remove P3 (no longer available)
Add AS COPY OF to ##Syntax
Remove statement that says connection to master database is required. I have successfully created a new database when connected to a user database.
Fix some typos, including errant ( and )

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thank you @SvenAelterman for help with the content.

Thanks @craigg-msft. This is mostly about Azure limits. @CarlRabeler is probably the best person to validate these figures.

@SvenAelterman thanks - reviewing today

@CarlRabeler Thanks. Perhaps we can immediately include the new service objectives in the standard tier that just became available (in preview...): S4 | S6 | S7 | S9 | S12

@Svankam going live tomorrow

@SvenAelterman these updates all going live tomorrow in substantial update to multiple topics - including your fixes here - one follow-up question - what do you mean by : "Add AS COPY OF to ##Syntax"

@SvenAelterman For your edification, we delayed publishing this update for all regions until this improvement had been rolled out in all regions

@CarlRabeler AS COPY OF was not present in the syntax (Syntax heading or ##Syntax 🙂) but it was referenced later in the doc.

@CarlRabeler can you please review the conflicts. Or does it make sense to just close this PR. the PR conflicts with files in the repo, which is not a surprise given the set of edits you mentioned that just pushed out.

Please close this PR - all changes made in other mega-PR
@SvenAelterman @craigg-msft

Thanks, closing


Evaluate ORA 2 PG

Open in MURAL
CAMD 3-PI Roadmap - EPA CAMD Business Suite - ECMPS and AMPD

Marking invalid, covered by #32


Referencing causal BO in section 11.9

Description of issue
I think the reference to causal Bayesian optimization would fit beautifully in section 11.9 where you talk about scenarios where optimization is influenced by exogenous factors. Causal inference is explicitly defined as exogenous factors (see page 207 of Pearl, 2009) in that modelling paradigm. Recent work (see Aglietti et al. 2020, Aglietti et al. 2020, Aglietti et al. 2021) have introduced this notion into the BO framework*.
*This is a shameless plug but I am joint first-author of the second paper.
Proposed resolution
Because causal inference so clearly models exogenous factors, reference to related causal BO work may be useful to some readers. There is also additional BO robot work by e.g. Roberto Calandra that would fit very nicely in this section.
Additional context
None
Additional context
None

@neildhir can you provide links to the latter two papers?

Yeah sure, the three papers in question are:

Causal Bayesian Optimization
Multi-task Causal Learning with Gaussian Processes
Dynamic Causal Bayesian Optimization

There we go.

I'm afraid I wasn't able to squeeze this in in the end (maybe the next edition!). But I will leave this issue open so that these important references remain accessible!


made service status configurable

this helps in cases where you want to configure but not start the service imediately.

Setting prometheus_enabled: false in this example will cause prometheus not to be started after node reboot. This is not the desired outcome at any setup apart from demo purposes.

What is the scenario where you run this role and don't want to start prometheus at the end?

Configure it now but start it later on demand is something we find helpfull for some machines with quite low memory available or for dev/test VMs on our local cloud thing.
So basically saving resources but configure it so a tester just needs to start it without knowing about ansible or other things.
This also helps disabling it in case you are low on memory at some later point and monitoring is not the most important process.

The aim of the role is to provide production-ready prometheus installation and as such, I don't really want to promote such features. However, we could move it into vars/main.yml and prefix with underscore like in https://github.com/cloudalchemy/ansible-prometheus/blob/master/vars/main.yml#L10.
Also maybe _prometheus_started would better reflect what this is actually doing, as the primary goal is to control if prometheus is started or not.

As long as one can override the var from outside the role I agree with _prometheus_started like in a filter
prometheus_started | default(_prometheus_started)

As long as one can override the var from outside the role

Variables from vars/main.yml can still be overwritten.

As long as one can override the var from outside the role

Variables from vars/main.yml can still be overwritten.

Yes but they have a very high precedence. vars/main.yml overrides inventory and play vars which is not what you would want for this one. For variables like this one the lowest possible rank should be picked.

Ansible best practices in role design state that everything in defaults/main.yml is meant as a role supported option. This, in turn, causes that those variables are de facto something that role needs to support "officially". With that in mind, I feel this poses a future maintainability problem, where users have this set to false and after some time report that prometheus is not working (I've already seen such cases in a different role).
Putting it in vars is a workaround for this and provides a path for more experienced users to leverage unsupported options. I am aware it has much higher precedence, but this pattern is what we have for now. Maybe we can break it and put such options into a separate file, like defaults/internal.yml and still prefix with _ to mark this as "unsupported but possible", WDYT?
Lastly, please understand that @cloudalchemy is mostly an after-hours side project, and to have high role quality I prefer to put a focus on future maintainability over new features (that's why there is a fairly extensive molecule test suite put in place :slightly_smiling_face:  )

Ansible best practices in role design state that everything in defaults/main.yml is meant as a role supported option. This, in turn, causes that those variables are de facto something that role needs to support "officially". With that in mind, I feel this poses a future maintainability problem, where users have this set to false and after some time report that prometheus is not working (I've already seen such cases in a different role).

I would not fear such cases.
With the pattern suggested in the previous comment prometheus_started | default(_prometheus_started) you would not need to have it in defaults but can override it from everywhere. You can also skip the _ var then but it would make finding it more easy.

With the pattern suggested in the previous comment prometheus_started | default(_prometheus_started) you would not need to have it in defaults but can override it from everywhere.

I do prefer to have variables initialized (in defaults/ or /vars/). This greatly simplifies the role and eases discoverability of features.


doc: Grammar fixes

What problem does this PR solve?
Minor grammar fixes, add some additional Markdown formatting to the PR template
What is changed and how it works?
Just some changes to the Markdown files.
Check List
N/A

Please merge the master branch into your branch.

This branch is out-of-date with the base branch

Please merge the master branch into your branch.

This branch is out-of-date with the base branch

Merged, thanks.


Fix centos image filename

have a service with an associated centos vm,
open the service in My Services,
click on the VM detail,
an icon is missing in the leftmost card.
The file was named os-centos.svg, but vm.os_image_name says linux_centos, so we need os-linux_centos.svg
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1687059

Reproducing in rails console:
find a centos vm: vm = Vm.all.find { |v| v.os_image_name == 'linux_centos' }
find a service: s = Service.first
associate: s.add_resource(vm); s.save!
Open /services/#{s.id}/resource=#{vm.id}


An attempt at some code cleanup, renaming things

Description
Uncomment if you want to provide a custom description.

Mostly renaming and simplifying methods, for consistency
The component itself is only aware of two 'built-in' views,
named presentation (used for start and resume events)
and speaker (used for the join-as-speaker event)
Only listen for keydown on body, and add/remove listener on connect/disconnect
A lot of 'keydown' event-handler cleanup to try and achieve proper priority in different situations

cmd/ctrl-k always works, open/closes control panel
if the control panel is open, esc will blur and close the panel
if a blank slide is open, any non-cmd/ctrl key will close it (this allows you to open the control panel while a blank slide is shown)
cmd/ctrl-enter will always resume presentation (or start from the top, with shift)
alt-enter will always join presentation as speaker
other events are only triggered when key-control is enabled

I was hoping to remove all known views, and allow those to be totally defined by the page author and the stylesheets. But we have useful shortcuts to start and join-as-speaker, so it seemed like those two views need to be provided. We could allow them to be changed? Or just make sure we've chosen good names. I went back and forth a bit between presentation and slideshow for the public-facing one.
Steps to test/reproduce
Please explain how to best reproduce the issue and/or test the changes locally (including the pages/URLs/views/states to review).

try lots of key combinations in different arrangements
try all the buttons?

I made the changes based on our conversation this morning.

The shortcut for opening the control panel opens the search bar in Chrome and Firefox.

Oh, I accidentally lost the event.preventDefault();. Fixed in bfdc38c

Should there be a visible confirmation that keyboard navigation is turned on?

I'm not sure what sort of indication we would provide, besides the pressed state of the button (which is subtle but there). Do you have ideas?
I'm not actually sure if we need this? The other option is to always have keyboard control turned on. I expect that's the way most people will use this. I was just worried about conflicts like the ones you mentioned above: key-controls hijacks a number of things, like arrow keys, period, comma, etc…

When switching views, it seems to be a two step process to use keys to navigate slides:

Press a button to change the view
Press escape
Navigate with keys.

Is that expected?

This isn't new in this branch, but comes out of exactly the same concern. If we're hijacking common keys while another interactive element has focus that seems especially risky. In the context of an embedded demo, you press an arrow key, and the slide changes?
Maybe the current behavior is too cautious, though. I could imagine a solution that only allows form input focus to override key control, and stops propagation of keyboard events inside iframes (embeded demos)? That way you could still use keyboard controls despite a top-level link or button having focus?

I also think it would be helpful to provide a view-switching keyboard shortcut. Maybe we should open a new issue about that.

@dvdherron Oh I see, I also broke the key-control pressed state with one of the name changes. That should also be fixed now.

If I am in "grid" view (by clicked the button on the first slide) and slide 4 is my active slide, when I open the command palette and click resume, what should happen?

I feel like there's no feedback that a button was pressed in the control panel (except for the View: slideshow or speaker buttons).
I also can't seem to get keyboard navigation working (as in, I click the keyboard navigation button in the control panel, then close the panel, then hit arrow down and the slide doesn't move but the whole document scrolls like 30 pixels). Is that just me?

If I am in "grid" view (by clicked the button on the first slide) and slide 4 is my active slide, when I open the command palette and click resume, what should happen?

Oh hey, I broke two of those command-panel buttons. 🤦🏻♀ Thanks for catching it. They should be fixed. Resume should take you into presentation view, turning on key-control and follow-active.

I feel like there's no feedback that a button was pressed in the control panel (except for the View: slideshow or speaker buttons).

Yeah, several of these buttons aren't 'toggles' - they don't control a single thing on-or-off. So it's hard to know what they should show. Another issue here is that key-control and follow-active (the things that change in addition to the view) are kinda invisible and abstract concepts? Do we need to think a bit more about the UX around those?
But it doesn't help that two of these buttons were just totally broken and doing nothing. Does it feel better now that they actually do something?

I also can't seem to get keyboard navigation working (as in, I click the keyboard navigation button in the control panel, then close the panel, then hit arrow down and the slide doesn't move but the whole document scrolls like 30 pixels). Is that just me?

Key-control is on by default. When you click the keyboard navigation button, you might be turning it off?

The shortcut for opening the control panel opens the search bar in Chrome and Firefox.

Oh, I accidentally lost the event.preventDefault();. Fixed in bfdc38c

👍🏽

I'm not actually sure if we need a key-control toggle? The other option is to always have keyboard control turned on. I expect that's the way most people will use this. I was just worried about conflicts like the ones you mentioned above: key-control hijacks a number of keyboard events. Not just cmd-k, but arrow keys, period, comma, etc…

Hmm, with those conflicts in mind it could be nice to keep the key-control option available.

When switching views, it seems to be a two step process to use keys to navigate slides:

Press a button to change the view
Press escape
Navigate with keys.

Is that expected?

This isn't new in this branch, but comes out of exactly the same concern. If we're hijacking common keys while another interactive element has focus that seems especially risky. In the context of an embedded demo, you press an arrow key, and the slide changes?
Maybe the current behavior is too cautious, though. I could imagine a solution that only allows form input focus to override key control, and stops propagation of keyboard events inside iframes (embeded demos)? That way you could still use keyboard controls despite a top-level link or button having focus?

I think this sounds like a good approach.

ok, I pushed a less cautious approach to keyboard controls. Happy for anyone to review - but especially @jamesnw or @jgerigmeyer.

@jamesnw Yes, that describes the idea correctly. And yeah, I had the same thought. Maybe I'll open a new issue for it, though. We can call this a good start?


Update Dutch translation

I just did this manually, maybe you could put the project up for translation on Transifex (/Weblate/...)?

Thanks, I'll consider that :)


ReferenceError: TextEncoder is not defined

Hey, i want to use this library in embedded environment where no TextEncoder, can you switch to pure js way for this?

https://gist.github.com/Yaffle/5458286

we have no hard dependency on textencoder though

fixed in e1046f5, will release new release soon