text:
I want to add a apply and cancel button in it. how it is possible?

Subject of the issue
I want to add a apply and cancel button in date range picker. how it is possible?

Hi @Tariqali13 , you can embed <DateRangePicker in your own component that keeps the date changes in its state and handles the apply/cancel actions.

Cancel button would just hide the range picker.
Apply button would trigger a function to handle the current selection

Also, for the cancel button to work nicely, you would need to save the initial selection so, if a user hits Cancel, you would restore your calendar selection to prior values.
I hope this helps. I'm planning to work on documentation that would explain scenarios like this one.

@nerychucuy please provide the documentation for achieving this easily.

Hi. @nerychucuy  thanks for the response, but i have already achieve this approach. it's will be good if it is provided by default.

authors: Tariqali13 iamsaurabhgupt nerychucuy

repo: Adphorus/react-date-range


text:
Added WebCompiler in the tooling section

Please consider adding https://github.com/excubo-ag/WebCompiler to the tooling section.

Thanks!

source: gharchive/pull-request

authors: AdrienTorris stefanloerwald

repo: AdrienTorris/awesome-blazor

license_type: permissive

license_source: github-api


text:
Allow profiles to dynamically select templates based on current job.

A feature that I love from Glamourer is the ability to apply glamours automatically when changing jobs. I wish that I could apply CustomizePlus templates in the same way, for example, to reshape one job's gear without it affecting other jobs. Profiles could be enhanced to select templates based on a dropdown list of jobs and apply these when the plugin detects that the relevant job is equipped. A default option of "Any" would function as CustomizePlus already does. Crude example drawing provided:

Built-in automation is not planned in Customize+. I believe DynamicBridge plugin should be able to do what you want. Another option is using a macro combining Customize+'s profile changing commands and ffxiv's /gearset command.

source: gharchive/issue

authors: DannyFDev RisaDev

repo: Aether-Tools/CustomizePlus


text:
Encountered an AffdexException：Unable to initialize affdex

I'm running the affdex-win-samples.sln provided by the sample file through VS2017 on a 64-bit Windows system, I've already set the include directory and library directory, etc. I'm using opencv 3.4 and boost 1.59 (because it can't find libboost_filesystem-vc120-mt-1_59.lib when I use the latest version boost)
My arguments are set to: -d D:\code\cpp-sdk-samples-emotion-sdk\data\joy.mp4 -r 640 480
where the path is where I put my test video
The opencv-webcam-demo and video-demo both give me this on the terminal:
Initializing Affdex FrameDetector
Setting the webcam frame rate to: 30
Max num of faces set to: 1
Face detector mode set to: LARGE_FACES
Encountered an AffdexException Failed to initialize: Unable to initialize affdex
how to fix the last line problem? Thanks for your help!

I would suggest looking at the README for instructions on how to run the sample apps.
OpenCV webcam demo: https://github.com/Affectiva/cpp-sdk-samples#opencv-webcam-demo-c
Video demo: https://github.com/Affectiva/cpp-sdk-samples#video-demo-c
The video-demo utility expects an input file (-i) and the path to the directory consisting of the Affectiva SDK models (-d).
Please go over the dependencies for the sample apps as well (https://github.com/Affectiva/cpp-sdk-samples#additional-dependencies)
opencv-webcam-demo.exe -d C:\Path\To\AffdexSDK\data and
video-demo.exe -d C:\Path\To\AffdexSDK\data -i C:\Path\To\Input\video.mp4
By default the screen resolution is 640x480.

Hi, would you mind sharing the video that resulted in that error?  We'll investigate.

I'm sorry that's my fault! The problem goes from the empty frame in my previous video, I recorded a new video which my head stay relatively clear and stable in the center and the problem was fixed. Thank you so much!

Well, it shouldn't produce an error like that anyway, but thanks for the tip, we'll try to reproduce with an empty frame.

Rats, I can't seem to reproduce it; I'm moving my head all over the place and in/out of the frame :-).  Do you still have the first video that causes the probem?

test.zip
I think it's the problem of the video

Hi thanks for the test video.  I still haven't been able to reproduce the problem, unfortunately.  When I look at the frame in that video with timestamp 1.7s, it doesn't look like an empty frame.  I just want to confirm that this is the video that gave you that error.

Hi, that's weird because every time I test using that video I receive the error, but the program works fine when I use other videos.

Sorry I encountered a problem again :) the video-demo works but the webcam-demo (in the same solution as video-demo) doesn't work
I followed all the instructions you suggested above and compiled the opencv-webcam-demo in VS2017, and I got

when I run the exe file, nothing shown in the window

any help will be greatly appreciated!

Hi, usually that error indicates that the data directory path you specified on the command line is not pointing to a valid data directory.

authors: adennie hetingjane umangmehta12

repo: Affectiva/cpp-sdk-samples

license: mit

license_source: bigquery


text:
new test file

merge requesting test-branch-3.

Confirming merge request.

Success!

deleting test-branch-3

delete success!

license: MIT


text:
[Bug]  Allow standard functions to be used in Custom Evaluation Code

Describe the bug
Currently it is not possible to use functions such as set in the Custom Evaluation Code.
Sample code:
from typing import Dict
def evaluate(
    app_params: Dict[str, str],
    inputs: Dict[str, str],
    output: str,
    correct_answer: str
) -> float:
    output_list = [int(x) for x in output.split(',')]
    correct_answer_list = [int(x) for x in correct_answer.split(',')]
    common_elements = len(set(output_list) & set(correct_answer_list))
    score = common_elements / len(correct_answer_list)
    return score

Returns a 500 with:
Failed to execute custom code evaluation: name 'set' is not defined

To Reproduce
Set the python code, and run a custom evaluation
Expected behavior
The evaluation would work
Proposed fixed here : https://github.com/Agenta-AI/agenta/pull/1160

Closed #1160

authors: mmabrouk romainrbr

repo: Agenta-AI/agenta


text:
Bug: All evaluations do not work without OpenAI API key

Describe the bug
Starting any evaluation now requires an OpenAI API key
To Reproduce
https://github.com/Agenta-AI/agenta/assets/4510758/189122fd-4c5b-4904-b089-5b1bbfc38464
Expected behavior
Only when starting AI Critic evaluation, would the OpenAI API key be required

fixed in https://github.com/Agenta-AI/agenta/pull/325

authors: aakrem mmabrouk


text:
Add a LICENSE file

I know that jspicl has an MIT license, but you should also add a LICENSE file, to the repository, to have it "light up" in the GitHub UI
https://help.github.com/articles/adding-a-license-to-a-repository/

❤

repo: AgronKabashi/jspicl


text:
nh下载漫画,发送到群里的全都是第一张

但是下载到本地的却是正确的

看错了,里面都是最后一张

已在 52a0de0fb24e0a0763f21a82a95a5b0739ac736d 修复

实测可用,感谢

authors: Ailitonia lancer-wang

repo: Ailitonia/omega-miya


text:
Update Proxy.cs

Added RegEx Matching for "IP:PORT" during Proxy(string) function before passing Address to proxy.Trim(); some proxy lists have errors or incorrectly formatted proxies that will throw an exception during running. This prevents anything invalid from being passed to Address variable that isn't an IP:PORT.
Includes: using System.Text.RegularExpressions;

in the comment to the previous your pull request, I did not mean this, I will submit the commit as soon as I can use my pc

opened to not forget

reworked

authors: Airkek LeftBased

repo: Airkek/Youtube-Viewers


text:
add more options

added options for reconnectAwaitSeconds, postDisconnectAwaitSeconds, signingCheckDelaySeconds

You should add at least the encryption button visibility option as well. Other encryption options probably don't need to be duplicated.

I do not see that option in the wiki

I do not see that option in the wiki

https://github.com/Aizistral-Studios/No-Chat-Reports/wiki/To-Encrypt-or-Not-to-Encrypt#other-config-options

repo: Aizistral-Studios/No-Chat-Reports

license: WTFPL


text:
error after starting the Addon

Hi,
After starting the Addon, I'll get the following error:
s6-overlay-suexec: fatal: can only run as pid 1

this seems to be related to this topic at HomeAssistant:
https://community.home-assistant.io/t/hello-world-example-addon-from-developer-docs-stopped-working-s6-overlay-issue/421486/2

Thank you for reporting and even providing a PR! :)

authors: Akulatraxas ChristophCaina

repo: Akulatraxas/ha-modbusproxy


text:
Logging

Swapped all PSIOPT printing to spdlog. Output is now duplicated to terminal and log file. Current implementation has file level set to trace, and console level tied to the old PSIOPT::PrintLevel.
I don't love the filename uniqueness code, but I think we have to do something like that for when there are multiple PSIOPT instances, like in Jet.
Probably closes #25

I'm having some issues building it on Windows, but I'll get that part figured out. In the meantime, from glancing at the code it looks like printlevel and log level are coupled together, so by default we would be dumping a bunch of uniquely named textfiles into the users directory everytime the console comes up. Instead, I think we should have it be toggleable and be off by default.

The unique files are for multiple PSIOPT instances and aren't directly related to the level. I wanted to avoid the case where multiple instances are all simultaneously dumping to the same file, which would make it basically unreadable. I do agree there should be a toggle though.

authors: jbpezent wgledbetter


text:
Irmaku/wc generator issue

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #294
** #330
** #337
Summary:
...
Type of change:

[x] Revision of an existing capability

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Irma Kurnia seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant irma-kurnia-alaska

repo: AlaskaAirlines/WC-Generator


text:
SWC-82: Endpoint_Creation

Created temp endpoint link for each project

looks good to me

repo: Aldokimi/API-Builder


text:
enhance edit data flows ports list

Netbox Version

3.X & 4.X

Actual Behavior
When I create a data flow I have selected source ports from 1 to 65000 :

It looks good on the summary page.
When I want to edit the ports I get the following :

Desired Behavior
As you can see the source ports is listing all the ports. From a UX perspective I believe it would be better to have :

Thanks

Hello LaV,
This should be already fixed in v1.0.6 (#43 ) that was released on Sunday.

authors: Alef-Burzmali lvenier

repo: Alef-Burzmali/netbox-data-flows


text:
excep handling and F usage update pull

Updates/improvements 113021 2247:

function F development, inclusion and usage within functions cbrt, to_depressed and cbdelta, mainly consisting on setting values to fractions.Fraction object and more fractions module operations.
Inclusion of calling of functions check_types and check_vals for checking on the supportability of the input variables to be passed (variable type or variable matching values). These two functions are soon to be included on exceptions.py.

Also, the commit included the import of fractions module and removing numpy importing as it is not used yet.

repo: AlejandroPenaloza/ompy


text:
Patch 1

added missing icons for highlight and mathml ('https://github.com/Alex-D/Trumbowyg/tree/develop/src/ui/icons')

They are not missing.
I moved them into their respective plugin folders:

https://github.com/Alex-D/Trumbowyg/tree/develop/plugins/highlight/ui/icons
https://github.com/Alex-D/Trumbowyg/tree/develop/plugins/mathml/ui/icons

hi @Alex-D ,
one of the react version of this editor, is using from this path

Seams it use the built icons.svg file:
import svgIcons from 'trumbowyg/dist/ui/icons.svg'
https://github.com/RD17/react-trumbowyg/blob/master/src/react-trumbowyg.js#L3
Also, "missing" icons are in the latest dist icons:
https://raw.githubusercontent.com/Alex-D/Trumbowyg/v2.15.1/dist/ui/icons.svg
You can find: <symbol id="trumbowyg-mathml"
So I did not understand why you do not have icons :/

repo: Alex-D/Trumbowyg


text:
out folder not created back after delete

build extension
build it again
we have error build.js(19, 1) Microsoft JScript runtime error: Permission denied
in cmd output

Waiting after deleting
https://github.com/AlexanderGubarev/RecordVideo-TestComplete-extension/commit/12fbfcf1d5d4937ffb7ec9548e284931172601ba#diff-efd59cd1bdc5c9ac6d0eaa368f1e149f

authors: AlexanderGubarev longmaxx


text:
🛑 Nyam is down

In 269e6a5, Nyam ($SECRET_URL_NYAM) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Nyam is back up in f962735.

repo: AlexanderKlimashevskiy/upptime


text:
Add CellAssign process

Towards #415
The next step is running cell assign, done here! I think given all the nice organizational changes in #476 & #477 this is pretty straightforward for review. The one additional thing to note is that I realized we probably need to be including singler_reference_name in the same way that we include cellassign_reference_name. This wasn't previously needed since we had been doing singler celltyping and adding to SCE object in the same process, where the reference name was accessible from the trained singler model itself. Now that we will be adding singler & cellassign to the final step all at once in the last process, we'll need to grab both reference names for input.
I also have one discussion-y comment to bring up at this point - my memory from our figjam session 🎸  was that we had decided not to keep using indicator variables in meta, e.g. meta.has_singler=true (edit lowercase true, this isn't R). I am not seeing any notes that explicitly say this though, so let's make sure I am remembering correctly! We can/should certainly still add relevant information to SCE metadata as we had been doing, but with our current celltyping workflow structure I don't really see the need for anything in the json metadata unless I am missing something?
Noting the next PR will write the process & R script for adding celltypes into the processed SCE.

I also have one discussion-y comment to bring up at this point - my memory from our figjam session 🎸 was that we had decided not to keep using indicator variables in meta, e.g. meta.has_singler=true (edit lowercase true, this isn't R). I am not seeing any notes that explicitly say this though, so let's make sure I am remembering correctly! We can/should certainly still add relevant information to SCE metadata as we had been doing, but with our current celltyping workflow structure I don't really see the need for anything in the json metadata unless I am missing something?

The presence of a reference file(s) for singleR and/or cellassign will be enough to say whether that processing has happened. There should not be any need for a separate boolean value.

Removing the singler_reference_name variable. We didn't have it before because it is already in the .rds files.

I realized that I am not 100% sure whether the reference name is in the output .rds object. It was there when we included the full SCE object, but maybe not now? I'd still say it is better if we can include it in the object itself, rather than creating an extra meta variable and associated column that we would have to add to the input table.

A couple thoughts on reference names -

The singler trained model name and the cellassign reference names are both present in their respective file names, .e.g. BlueprintEncodeData_model.rds & PanglaoDB-blood.tsv. So, since these filenames are in meta, I guess we don't need to pass either the singler or cellassign reference names in separately. We could extract this info from filenames with some string manipulation. (Not loving that we have _ vs -, though..).
In terms of adding the singler model name to the singler object saved inrds, I'm not sure how that would play out - my idea was to keep that as the exact structure that singler exports so that people can actually use it if they want to with singler's native functions, e.g. heatmap. It's a DataFrame, so we'd have to add a column with model name which should be fine unless there is crazy strict checking in singler's functions. But, if we do my point in 1^ , then this isn't needed.

I realized that I am not 100% sure whether the reference name is in the output .rds object. It was there when we included the full SCE object, but maybe not now?

It is not in the output rds from the singler projcess at this point:
https://github.com/AlexsLemonade/scpca-nf/blob/fb5f3bf1d9dad7533046cf05230a5cf0b3ddabaf/modules/classify-celltypes.nf#L25

But we can add it quite trivially, as DataFrame objects have metadata slots just like the SCE. So we could add in something like metadata(singler_result)$singler_reference <- singler_reference_name to store the reference name along with the results itself.

Right, I would just want to make sure this doesn't cause singler's native functions to break, which it really shouldn't. Let me double check though...

It is possible for the meta object to become disconnected from the file name/reference that was actually used if we are not careful, but I expect that the repeat_celltyping logic should handle that when we add it.

One way to handle this would be to (optionally) pass in the spca-meta.json files that are in the cellassign and singler directories

As long as we are adding in metadata, I'm fine with passing in the reference names for both cellassign and singler directly as an argument to the assign script. The only thing we have to be cautious about (and we have to do this anyway) is that we need to make sure we are passing in the correct value.

Feels like this could serve as an extra check as well - we'd want to make sure the singler reference name that gets passed in is the same as the name recorded in the singler DataFrame's metadata.

Changes since last time:

https://github.com/AlexsLemonade/scpca-nf/pull/479/commits/9de11fc74436ccf2ae5229bf38f55fd158bdf39b: I got rid of all tracking of the reference names in favor of "regexing them," if you will..., out of the reference/model file names directly.
https://github.com/AlexsLemonade/scpca-nf/pull/479/commits/09565d13607fefecf064070d7db20b97e0ecaf3a: I updated the SinglerR script to add the reference name as DataFrame metadata, with a couple checks to go with it

Just for posterity...

stub failed for https://github.com/AlexsLemonade/scpca-nf/pull/479/commits/3c2715cbb808124be2b41b4b6b5a38ec393b6aed, but none of these changes really should have caused that?
Locally, stub ran fine for me at that commit!
I retriggered the CI stub check and now it's passing 🤷♀

authors: jashapiro sjspielman

repo: AlexsLemonade/scpca-nf


text:
Cover service.toggle problem

Hello
I have I have aqara curtains b1. And I faced the following problem
After some time, apparently, the position of the curtains is reset, the cover.toggle service also stops working
To fix this, you have to go into lovelace, and move the curtain position slider. After that, service.toggle starts working successfully. But after a while everything breaks down again

Probably related to #139
Or you can show logs with error

Probably related to #139
Or you can show logs with error

and where to look at the logs, there seems to be nothing in the usual log

and where to look at the logs, there seems to be nothing in the usual log

Wait new version with fix for 139. Maybe it helps

Wait new version with fix for 139. Maybe it helps

@AlexxIT Unfortunately the update didn't work for me. But I noticed another peculiarity. Toggle only stops working when the curtains are open. If the curtains are closed, even all night, the toggle will work in the morning. But in the evening the toggle does not close the curtains. Have to send position for curtains

@AlexxIT Unfortunately the update didn't work for me. But I noticed another peculiarity. Toggle only stops working when the curtains are open. If the curtains are closed, even all night, the toggle will work in the morning. But in the evening the toggle does not close the curtains. Have to send position for curtains

Check device state in developer tools when get the problem. There should be something like this:
current_position: 50
friendly_name: Aqara Curtain B1 Motor
supported_features: 15

Check device state in developer tools when get the problem. There should be something like this:
current_position: 50
friendly_name: Aqara Curtain B1 Motor
supported_features: 15

I apologize for missing. Curtains for some unknown reason do not store their history, maybe the problem is

I reconnected the curtains and they started to keep history. Until I called toggle again

Another observation. I reconnect the curtains open, then call toggle. The curtains close, then I call toggle the curtains open, but the curtains are closed. After that, toggle stops working

As I said before, toggle stops working only after the curtains are opened. If the curtains are closed, then the toggle works.
If you call toggle when the curtains are closed, then the curtains open, and all this time their status is opened, but as soon as they open completely, the status changes to closed

I tried to do a life hack and open the curtains to 99 and 1 positions. To build on these numbers, but as soon as the curtain opens up to these positions, it resets the position to 0 and writes that it is closed (

The temporary fix I am using. Maybe someone will come in handy
cover:
    input_boolean:

        cover_fix:
            name: Cover Open

    switch:
        - platform: template
          switches:
            cover_bedroom:
                friendly_name: 'Cover'
                value_template: "{{ is_state('input_boolean.cover_fix', 'on') }}"
                turn_on:
                    - service: cover.set_cover_position
                      entity_id: cover.bedroom
                      data:
                        position: 100
                    - service: input_boolean.turn_off
                      data:
                        entity_id: input_boolean.cover_fix
                turn_off:
                    - service: cover.set_cover_position
                      entity_id: cover.bedroom
                      data:
                        position: 0
                    - service: input_boolean.turn_on
                      data:
                        entity_id: input_boolean.cover_fix

    automation:

        - alias: 'Open and Close Cover Opple'
          trigger:
            - platform: state
              entity_id: sensor.0x04cf8cdf3c7974cc_action
              to: 'button_5_hold'
          action:
            - service: switch.toggle
              entity_id: switch.cover_bedroom

I am seeing the same issue as well. Bare in mind the current XiaoMi Gateway 3 (China ver.) does not support B1 motor so I can only get this pairred via Home Assistant Pairing function.
Once I opened the curtain, I cannot close the curtain and the position is wrong as well.
I am also seeing the same problem when using ZHA mode, however, people has reported that to ZHA and that seems to be a ZHA problem.
The only workaround for me is to use an old lumi gateway which officially supports B1 motor. Once I paired with old gateway, that works in HA correctly.

@uPagge write to me in Telegram. It will be faster to fix problem.

@AlexxIT good

@uPagge this is the link for ZHA mode that has issues with B1 motor.
https://github.com/home-assistant/core/issues/35738
this seems to be a generic issue with B1 motor that is different from the normal zigbee covers.

I am seeing the same issue as well. Bare in mind the current XiaoMi Gateway 3 (China ver.) does not support B1 motor so I can only get this pairred via Home Assistant Pairing function.
Once I opened the curtain, I cannot close the curtain and the position is wrong as well.
I am also seeing the same problem when using ZHA mode, however, people has reported that to ZHA and that seems to be a ZHA problem.
The only workaround for me is to use an old lumi gateway which officially supports B1 motor. Once I paired with old gateway, that works in HA correctly.

So if if got this XiaoMi Gateway 3, it's impossible to pair it? I just got the Aqara B1 curtain motor and found that the Mi Home Apps not support this motor. What's the proper way to pair it to the Gateway 3? Or should I get the Aqara M1 hub instead?

I am seeing the same issue as well. Bare in mind the current XiaoMi Gateway 3 (China ver.) does not support B1 motor so I can only get this pairred via Home Assistant Pairing function.
Once I opened the curtain, I cannot close the curtain and the position is wrong as well.
I am also seeing the same problem when using ZHA mode, however, people has reported that to ZHA and that seems to be a ZHA problem.
The only workaround for me is to use an old lumi gateway which officially supports B1 motor. Once I paired with old gateway, that works in HA correctly.

So if if got this XiaoMi Gateway 3, it's impossible to pair it? I just got the Aqara B1 curtain motor and found that the Mi Home Apps not support this motor. What's the proper way to pair it to the Gateway 3? Or should I get the Aqara M1 hub instead?

Yes, as far as I know, currently Xiaomi Gateway 3 does not support Aqara B1 motor officially. But they have said it is under development from Xiaomi forum.  I'd suggest to pair with the old version of xiaomi gatway or aqara gateway instead. These product supports Aqara B1 motor. I guess the reason behind it is that Xiaomi Gateway3 moves away from supporting Zigbee and lean to promoting BLE devices so there is less compatibility work done on supporting newer Zigbee products like Aqara B1 motor in the Xiaomi Gateway3.

The temporary fix I am using. Maybe someone will come in handy. Create new packages
cover:
    input_boolean:

        cover_fix:
            name: Cover Close

    switch:
        - platform: template
          switches:
            cover_bedroom:
                friendly_name: 'Cover'
                value_template: "{{ is_state('input_boolean.cover_fix', 'on') }}"
                turn_on:
                    - service: input_boolean.turn_on
                      data:
                        entity_id: input_boolean.cover_fix
                    - service: cover.set_cover_position
                      target:
                        entity_id: cover.bedroom
                      data:
                        position: 1
                turn_off:
                    - service: input_boolean.turn_off
                      data:
                        entity_id: input_boolean.cover_fix
                    - service: cover.set_cover_position
                      target:
                        entity_id: cover.bedroom
                      data:
                        position: 100

    automation:

        - alias: 'Open and Close Cover Opple'
          trigger:
            - platform: state
              entity_id: sensor.0x04cf8cdf3c7974cc_action
              to: 'button_5_hold'
          action:
            - service: switch.toggle
              entity_id: switch.cover_bedroom

Unfortunately, with one of the new updates, my crutch stopped working, now the integration simply does not see the engine ((This is very sad, since I have to climb the sofa to open and close the curtains

@uPagge you can downgrade component to find work version

@AlexxIT Unfortunately, my motor broke :( As soon as I fix it, I will try

Please reopen the issue if it is still relevant

authors: AlexxIT relliky skho1121 uPagge


text:
Switched editor from Ace to Monaco

Replaced Ace Editor with Monaco Editor in editor.html for an enhanced editing experience.

4 spaces for tabs in YAML should be fixed.

Implemented configuration validation with guidance prompts.

I don't like monaco-yaml-prebuilt idea.

I don't like monaco-yaml-prebuilt idea.

Me too. But this is the only option without local build and storage of the editor bundle. (however, it can be stored on the go2rtc GitHub pages)

Why do we need bundle? Is it only for tabs? Or also for yaml support?

Why do we need bundle? Is it only for tabs? Or also for yaml support?

It's needed for completion, hints, and validation support for YAML.
Without it the original package can be used
Unfortunately, extended YAML support is only available in external package, which is designed for use in the node-js environment. I couldn’t make it work without building it into a single package 😔
In any case, the build used has open source which allows it to be easily rebuilt with new versions, and the current versions used will always be available on npmjs and services like unpkg.

I have tested this https://monaco-yaml.js.org/ with # yaml-language-server: $schema=https:// start line.
It works awesome. We can try to use default monaco bundle and host this JS worker and schema.json on github pages.

I have tested this https://monaco-yaml.js.org/ with # yaml-language-server: $schema=https:// start line. It works awesome. We can try to use default monaco bundle and host this JS worker and schema.json on github pages.

Awesome. I couldn't get it to work in such configuration. I would like to see the code that worked

authors: AlexxIT skrashevich


text:
Make clean doesnt find the artifacts in artifacts_cache

The new-ish scripts/clean-artifacts.sh script doesnt find/delete the artifacts under artifacts_cache.  I believe that the file path on line 29 is incorrect.  Changed it locally to ./artifacts_cache/* and it cleans everything up.

That script was not meant to also delete the artifacts cache, but just the artifacts scattered across all the components folders.
That ! -path '*/artifacts_cache/*' is actually for skipping the artifacts_cache folder explicitly.
If you want to cleanup also the artifacts cache, beside simply running a rm artifacts_cache/*, you can run make clean_caches (there is also make clean which just run the above clean-artifacts script)

yeah I think previously the make clean target removed those in the cache, but noted.  as designed.  thanks

Thank to you and feel free to reach out with any new questions or concerns.

authors: gionn hulljaja

repo: Alfresco/alfresco-dockerfiles-bakery


text:
536 copying vector by assignment multiple times fails asserts

Bugfix for the following sequence that previously triggered an assertion failure for the reference_omp backend:
Vector<double> a(n), b(n);
set(a, 1.2);
set(b, 1.5);
a=b;
a=b; //Fails assertion

Issue: multiple-registration of the same container.
Cause: the previous implementation of the copy-assignment seems to predate the introduction of IDs. In its implementation, the container a is in fact replaced by a totally new container, with a different ID, through the use of std::swap and the creation of a new container within the copy-assignment. Additionally, this induces new memory allocations while in principle a dispatch to grb::set would have sufficed.
Solution: dispatch to grb::set instead. Apart from the performance gain, this also retains the container's original ID and does not trigger the earlier double-registration issue and thus does not trigger the reported assertion failure. Additionally, this MR adds a unit test for double copy-assignment.
This MR also includes

bugfix: BSP1D grb::set was not correctly updated as part of the previous MR (internal issue 533);
implement previously missing copy-assignment for BSP1D backend;
improved _DEBUG tracing;
minor code style fixes.

Reported by Anders Hansson -- many thanks!

Review OK. Waiting CIs OK. Also running all unit tests with LPF before merge.

BSP1D unit test failed. Traced back to previous MR. Tested hotfix in this branch now -- all unit tests with LPF OK.
After hotfix, all internal CIs also OK.

Code review of hotfix OK. Will merge.

repo: Algebraic-Programming/ALP


text:
Docs/readme edits

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

OS: [e.g. macos, linux, windows]
OS Version / Distribution: [e.g. 10.15.7, Ubuntu 20.04, Windows 10]
Version [e.g. 0.1.0]
Other: [e.g. python version, docker version, etc.]

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

:tada: This PR is included in version 1.1.1-next.1 :tada:
The release is available on:

GitHub release
crate.io packages (1 packages published)

Your semantic-release bot :package::rocket:

repo: AliSajid/gh-bofh-rs


text:
Not working.

Not receiving any updates since last week. Any issue with the app?

Yep is literally not working anymore I thought I had some issues. After restarting , delete and redownload its still the same. My last updated was on 21st of august 2024. That's it no more updates.

It's because this script get data from somewhere else which has bug:
https://github.com/Osb0rn3/bugbounty-targets/issues/6

repo: Alikhalkhali/programs-watcher


text:
Is it possible to use STS access key instead of RAM access key?

The use case is I do the AssumeRole call elsewhere, get STS access key id/secret and SecurityToken beforehand and let ack-ram-authenticator use that credentials to communicate with the server?
To put it simply, how can I use SAML SSO with this?

@andyzhang495  You can use STS Token now, example can be found in README.md like this:
{
  "AcsAccessKeyId": "xxxxxx",
  "AcsAccessKeySecret": "xxxxxx",
  "AcsAccessSecurityToken": "xxxxxx"
}

About the SAML SSO, this doc maybe helpful https://yq.aliyun.com/articles/712298 , more about SAML SSO on AlibabaCloud please access https://help.aliyun.com/document_detail/28627.html for reference.  https://www.alibabacloud.com/help/product/28625.htm  fo En version

Sorry for the late reply. @haoshuwei Thanks for your quick response and awesome job!
About SAML SSO, we're planning to use Role-based SSO which will generate an STS token if SSO is successful and use this STS token with ack-ram-authenticator. Like the use case with awscli, I can use the following kubeconf with aws-iam-authenticator:
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: {{ .CaData }}
    server: {{ .ServerAddress }}
  name: {{ .ClusterName }}
contexts:
- context:
    cluster: {{ .ClusterName }}
    user: {{ .ClusterName }}
  name: {{ .ClusterName }}
current-context: {{ .ClusterName }}
kind: Config
preferences: {}
users:
- name: {{ .ClusterName }}
  user:
    exec:
      apiVersion: client.authentication.k8s.io/v1alpha1
      args:
      - token
      - -i
      - {{ .TokenIdentity }}
      command: aws-iam-authenticator
      env:
      - name: AWS_PROFILE
        value: saml

The important part here is AWS_PROFILE being set to saml, which is a native login profile supported by awscli, which is implemented with AssumeRoleWithSAML. I saw Alibaba cloud also support this STS API.  Is this feature on your roadmap?

@andyzhang495 before you use aws-iam-authenticator in this way,  you must get a AssumeRoleWithSAML STS Token first in other ways, is it right?

@haoshuwei Ah, I get the idea now. I need to implement a cli tool fetching STS token using AssumeRoleWithSAML and save it as a profile named saml. I thought awscli provides this functionality out of the box but I was wrong.
Closing this issue since the function is already added. Thanks for replying!


text:
Consolidate device classes; update ephys schemas

Lots of changes here, happy to chat more about any of them! Since there are also some changes to ophys_rig.py and instrument.py, the ophys and imaging teams should be consulted before merging.
Consolidating classes across projects
There were a number of classes (e.g. Laser, Camera) that were duplicated across modules. I moved these to devices.py and made sure they contained all of the original fields were present in the merged version. This should help ensure that schema are more consistent across projects. I would recommend putting each device type in a separate file (e.g. lasers.py, cameras.py), but I didn't want to make that change yet. There are also some classes that will be needed across projects that still live in the ephys rig schema (e.g. HarpDevice, MousePlatform). We should probably move these out sooner rather than later.
Updated DAQ conventions
Rather than linking each device to one DAQ channel, I've changed it so each DAQ channel is associated with one device. I think this is better because there are some devices (e.g. lenses) that don't have DAQ channels, and some devices that have multiple DAQ channels. Ideally there could be some validation step to ensure that each DAQ channel refers to a device that exists elsewhere in the rig description, but this may be tricky.
Device-specific manufacturer enums
Each device type in devices.py now has a separate manufacturer enum. This is not strictly necessary, but it will make it much easier to manually enter new devices. After many years of adding devices to rigs, the list of all manufacturers will become quite long.
Naming devices
Devices such as probes and cameras are now named using open-ended strings rather than enums, because I don't think consecutive letter-based naming conventions are going to give us sufficient flexibility. This requires that whoever is creating the session JSON checks that data stream names match devices in the rig JSON. Eventually, it would be great if the form could ingest the rig JSON and display all of the valid stream names.
Other changes
I tried to avoid using type as a field name, because this is a Python built-in function. Same with filter, but I didn't change that yet.
Example files
Here are two example JSON files I generated with the latest changes:
ephys_rig.json
ephys_session.json

@saskiad and @dyf, the most recent commit addresses many of the issues you raised, please take a look at the latest code

Fixed!

repo: AllenNeuralDynamics/aind-data-schema


text:
Unable to remove allsky for a complete re-install

Environment

Camera:  RPi - HQCam
OS: Bullseye
Allsky version: n/a
Pi model and amount of memory: RPI 4 Model B Rev 1.2 / 4 GB

Bug Description
I would want to make a complete re-install. I could halt allsky service and remove most of the file in the allsky-directory.
However it seems that the tmp directory still is blocking complete removal.
(foobar) pi@meteotux:~ $ sudo rm -R allsky
rm: cannot remove 'allsky/tmp': Device or resource busy

I already dit a reboot but still get the same message.
Any help to this?

Run
cd
sudo umount allsky/tmp
rm -fr allsky

It worked - perfect!
Thank you

authors: EricClaeys WirthmU

repo: AllskyTeam/allsky


text:
Use a different configurable sync interval while Active full sync is ongoing

Is your feature request related to a problem? Please describe.
Users who enable active sync that that want to avoid having a sync interval all together may still encounter times where the active sync mechanism falls behind and triggers a sync on the entire sync point. Users will likely still want metadata synced in a timely manner. During the full sync, this may not occur.
Describe the solution you'd like
In order to preserve behavior in the best possible manner, during the full sync if the user's sync interval is configured to -1 and active sync has a full sync occurring, then we should add a configuration flag which will automatically use a specified sync interval during active sync.
Example

by default sync interval is -1, active sync is enabled
Active sync falls too far behind
full sync is triggered
All RPCs to paths underneath the sync point use a different (typically much shorter, 1-5s) sync interval

Describe alternatives you've considered
None at the moment
Urgency
Low

@da is this still valid? otherwise help me close this please

repo: Alluxio/alluxio


text:
High CPU usage when read data from Azure and seems has memory leak issue

When reading data from AzureBlob, we get high CPU usage in worker. Attached the profile results from async-profiler. Seems it caused by decode https response.
https-azure.zip
And we found worker doesn't free the memory of AzureStorageResponse. So when memory usage reach Xmx limit, the CPU usage will become much higher (about 900% in our env).
Currently, we use AzureStorage SDK v8, and it integrated with hdfs client. But v8 seems has many performance issues, maybe we can change to v12. Since v12 is not compatible with HDSF, need type of UFS need to be added.
Refer: https://docs.microsoft.com/en-us/azure/storage/blobs/storage-quickstart-blobs-java?tabs=powershell

@apc999 Please help take a look, maybe we should do some performance test for AzureStorage SDK

The memory dump link: https://alluxiotestms.blob.core.windows.net/dump/heap.zip?sp=r&st=2021-04-06T02:54:13Z&se=2021-04-30T10:54:13Z&spr=https&sv=2020-02-10&sr=b&sig=%2B1SZXtEAjrvEUaRTHUdYQR5Bfhp%2B%2FOs4tbE1EK8UN1I%3D

seems like resolved.Will close it for now, feel free to reopen it and contact us if this is still valid.


text:
Remove rocks page store log

What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

alluxio-bot, merge this please

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to main please

@maobaolong  can you create a same commit to main branch too? thanks

@apc999 Sure


text:
Normalize the HTTP RESTful API

Normalize the HTTP RESTful API:

the list files API: curl -X GET "http://<worker_ip>:<port>/v1/files?path=<path>"

Now the server will also return the lengths of the files.

the get page API: curl GET "http://<worker_ip>:<port>/v1/file/<file_id>/page/<page_index>"

alluxio-bot, merge this please.

The HTTP GET request requires that the string after the question mark in the URL represents the parameters. If there is a question mark elsewhere, it should be converted into an escape character.

I found the limitations stated in the Alluxio docs (https://docs.alluxio.io/ee/user/stable/en/administration/Troubleshooting.html).

authors: JiamingMai beinan


text:
[ALLUXIO-628] Fixed Style checker failed to find an indent problem

https://alluxio.atlassian.net/browse/ALLUXIO-628
This one prepares the change in the configuration in Checkstyle.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9504/Failed Tests: 0
Test FAILed.

authors: AmplabJenkins HorizonNet


text:
[Dewhale] implements #7

[Dewhale] This PR implements #7, created by Dewhale.

[Dewhale]: This JSX code will create a user profile info card with the details provided. The card includes an avatar, user information, and uses separators to distinguish between the different pieces of information. The card is centered and has a maximum width to ensure it looks good on all screen sizes.


text:
Fetch service owners from BFF and use them for filtering in frontend

Description
As of now, the service owners displayed through the filter component are limited to hard coded for test-values.
However, now that the BFF has an endpoint for fetching them (from backend), frontend can now make use of this to fetch the true owners.
This task should:

[x] Set up an rtk query for fetching service owners from the BFF
[x] Use the resulting data-value pairs in the filter component
[x] Use the same data-value pairs in the chip-components for displaying of chosen filters as well as in the search query.

Additional Information
No response

Det ser ut til å fungere fint. Lukker saken.

authors: allinox hannekristin


text:
Improve the display of headers when referenced in summary

Headers look ugly when referenced in a "type": "Summary" component and the Edit button shows up by default.

This PR makes a custom view for a header component.

Verification

Manual testing

[x] I have tested these changes manually
[ ] No testing done/necessary

Automated tests

[ ] Unit test(s) have been added
[ ] Cypress E2E test(s) have been added
[ ] No automatic tests are needed here
[x] I want someone to help me make some tests (if required)

User documentation @ altinn-studio-docs

[x] No changes/updates needed

Sprint board

[ ] The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board

Can we also get tag org/krt . This pullrequest is crucial for us, now that we are utilizing the new dynamic feature.

authors: FTLems ivarne

repo: Altinn/app-frontend-react


text:
De-Identification chart and integration

This PR adds a new Chart project to deploy an instance of the De-ID
service to a K8S cluster.
The chart deploys the container image created by @ibmatc to run the
de-id service JAR as described in the Alvearie de-id github page.

I don't think we should have de-identification (and the second FHIR server) enabled by default for the ingestion pattern.  Maybe for the enrichment pattern we could consider it, but for ingestion I'd lean the other way-  don't deploy it by  default and let them know the option to include it

I don't think we should have de-identification (and the second FHIR server) enabled by default for the ingestion pattern. Maybe for the enrichment pattern we could consider it, but for ingestion I'd lean the other way- don't deploy it by default and let them know the option to include it

The big question for me is whether most of this should go under health-patterns or under de-identification (and then have us use it from there).

Agree about the need to have this go under the de-id repo. Like we talked about earlier, I think let's deliver this here for now, while we make it more solid with tests, and work with the de-id team to see where they would want it, or some kind of hand off.

Also - the first line of this clinical-ingestion/helm-charts/deid/README.md should be De-identification (instead of FHIR)

Done. Thanks
Also, I will disable the de-id and 2nd fhir servers by default for the ingestion pattern.

repo: Alvearie/health-patterns


text:
fix: remove pattern for ref property in localization.metadata.schema.json

There is no point validating the ref if we don't validate the key.

I approve this change !

authors: amadeus-pmeneguzzi guillaumerochelle

repo: AmadeusITGroup/otter