Adding Offensive360 to list

Hello @JameelNabbo,
thanks for your contribution.
The tool looks great, however the README.md file is auto-generated from the files in data/tools. Can you please go ahead and create a new YAML file in that directory with the information for the tool? You can take a look at the existing files to get an idea of the file format.
The README.md will be rendered automatically then, so the manual changes can be reverted.

Many thanks @mre . Done

Merged to master now as part of #780. Thanks for your contribution @JameelNabbo.


Non-uniform support for ISO time format in CK browser

I have performed a HOG experiment and recorded its data into a local repository. I can view the created entry via a local webserver (http://localhost:3344/?wcid=experiment:) and on disk (ck find experiment:* --tags=hog). It is timestamped at 2015-09-04 13:29:20.
However, pasting this timestamp into the "After date (ISO)" filter in the CK browser (to filter out all but this entry) doesn't work - I need to replace the space in the timestamp with the 'T' character: 2015-09-04T13:29:20
Now, Wikipedia says: "It is permitted to omit the 'T' character by mutual agreement." But the CK browser should either print the date format with 'T' or accept it without 'T'.

I fixed that. You can now search with or without T. May I ask you to check it and close this ticket, please? Thanks!

Yep, works fine now - thanks!


can't go get

go get github.com/go-playground/locales<EMAIL_ADDRESS>unexpected EOF

@deankarn @teaoea why was this closed? im getting this in a github action using courtney
open  downloading github.com/go-playground/locales v0.14.0: no such file or directory
Error reading files from  downloading github.com/go-playground/locales v0.14.0
github.com/dave/courtney/tester.(*Tester).processDir
	/home/runner/go/pkg/mod/github.com/dave/courtney@v0.4.0/tester/tester.go:195


Create lr-wolfenstein3d.sh

@HerbFargus
I finally took a look at this. Worked well once I understood how it worked. One problem, it creates a .chocolate_wolfenstein_3d directory for savegames in $home. When I attempted to use moveConfigDir for it, it complained about directories, etc... I don't really have the time to look into this further at the moment.
The current version of the script in the chocolate-wolf3d branch has the addition of moveConfigDir. If you can take a look at it and fix whatever is causing the issue with it, I'll probably then move it into the master branch.

It may just be worth waiting until libretro actually ports it proper. I don't know if that will be anytime soon and we already have wolf4sdl on the main RetroPie branch so I'm happy to just let it sit til upstream sorts it.

Fair enough.

But I'll give the save settings a look if I have some free time

I also noticed that they are working on an sdl 1.2x engine for libretro for some reason. Not sure how that would work but maybe it could be helpful when it gets completed.


ARROW-2467: [Rust] Add generated flatbuffer source code

Adding generated flatbuffers code for Schema, File, Message, and TensorIt would be nice to generate on the fly as part of the build process but this doesn't seem feasible today because:

We would have to download and compile Flatbuffers first
The Rust implementation of Flatbuffers has a couple minor issues that required manually fixing the generated code

Did you delete and recreate your fork?

Yes, I did. Looks like you found the new PR. In case anyone else is looking for it, it is https://github.com/apache/arrow/pull/2986


Set up a "Hello world" project with two apps

Milestone 💪

[x] Create a new React app called 'hello-react-front-end'
[x] Create the App component with react-router.
[x] Create the Greeting component that will display a greeting. Set it up as a route in your App component.
[x] Create a store, an action and a reducer that will connect to you API endpont to get the random greeting.
[x] Display the random greeting in your Greeting component.
PR link for Back end

Hi @Tekhlay
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
Kindly add link to the backend repo in the frontend readme.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Can't retrieve attributes of the top-level component

We need to pass arguments from the server, to the Angular2 component.
I cannot manage to pass via attributes like I managed to do with AngularJS.
the host page contains:
<scrollable-table
             [table-header-url]="${g.createLink(action: 'itemTableHeader')}"
             [table-data-url]="${g.createLink(action: 'itemTableData', params: [offset: 0, rowCount: 100])}"
             [number-of-row-displayed]="5">Loading...</scrollable-table>

the component:
@Component(
    selector: 'scrollable-table',
    inputs: const ['number-of-row-displayed', 'action-load-table-url'],
    styleUrls: const ['cybertable_component.css'],
    templateUrl: 'cybertable_component.html')
class ScrollableTable implements OnInit {
  @Input('action-load-table-url')
  String actionLoadTableUrl;
  @Attribute('number-of-row-displayed')
  String numberOfRowDisplayed;
  String dataUrl;

  ngOnInit() {
    print("${actionLoadTableUrl} ${numberOfRowDisplayed}");
}

the Chrome console do print:

null null

whatever I tried (putting @Attribute, @Input, changing cases on attribute names ...) the properties are not initialized and no errors are printed.
Is it possible to get attributes from the server?
Is there a documentation telling how to process?
I am using

angular2: 2.0.0-beta.21

Thanks for your support!
Adrien

ok, I managed to get attributes reading:
[(https://angular.io/docs/dart/latest/guide/attribute-directives.html)]
the document

See also http://stackoverflow.com/questions/37155082/initialize-angular-2-component-with-attribute/37155204#37155204


Use the same buildspec across all services.

Coverage remained the same at 86.771% when pulling 30073a74e35607f232b04e586ff30d1e1a651e1f on johngian:fix-buildspec into 79d9f43246b4bb1f2c850e6baebc8c60ea8c4d95 on mozilla:master.


URLs with an extension are not recognized

Urls such as

https://msdn.microsoft.com/fr-fr/library/windows/apps/br229576.aspx
https://www.cdiscount.com/n-380205/cdiscount-immobilier/immobilier-neuf.html

are not recognized. I've done a small fix on a fork
https://github.com/nautilebleu/my-name-is-url

Brilliant, thanks!
Could you add a test for this functionality to prevent regressions too?

Closed via https://github.com/lukechilds/my-name-is-url/pull/19


[Go Live][Dec 30]

Push to live

Learn Build status updates of commit be4b0ef:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/build-tabs-for-meeting.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/meeting-apps-apis.md
:white_check_mark:Succeeded
View

msteams-platform/assets/images/deep-links.png
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/deep-link-application.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/deep-links.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/deep-link-teams.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/deep-link-workflow.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/test-app-overview.md
:white_check_mark:Succeeded
View

msteams-platform/tabs/how-to/tab-navigation.md
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/task-modules/invoking-task-modules.md
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/task-modules/task-modules-bots.md
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/what-are-task-modules.md
:white_check_mark:Succeeded
View

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

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


Add support for caBLE. Bump to 3.1.7.

Fixes #91

Codecov Report

Merging #92 (63f0468) into master (1ea8f5e) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 63f0468 differs from pull request most recent head 551bbc9. Consider uploading reports for the commit 551bbc9 to get more accurate results

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   92.93%   92.93%
=======================================
  Files          15       15
  Lines        5818     5818
=======================================
  Hits         5407     5407
  Misses        411      411

Impacted Files
Coverage Δ

lib/validator.js
92.06% <100.00%> (ø)

Continue to review full report at Codecov.

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

Thanks, you're doing amazing work!!


Socket observable swallows exceptions

When an exception happen to be thrown from inside .subscribe on any watched query, the unsubscription is being made by rxjs, but the error remains silent.
I found this particular line in my callstack after trapping on caught exceptons
https://github.com/rethinkdb/horizon/blob/next/client/src/socket.js#L236
Btw, what do you think about using the socket subject implementation from rxjs?
https://github.com/ReactiveX/rxjs/blob/master/src/observable/dom/WebSocketSubject.ts
I haven't found any official documentation for that though.

Thanks for opening an issue @Frizi .
This sounds like a duplicate of https://github.com/rethinkdb/horizon/issues/500
I'm not quite sure if the line in socket.js that you found is actually relevant, as it seems like subscribe always discards exceptions. Pinging @deontologician .

I saw this recently, I believe the error suppression was to avoid situations where we would error an observable that was already closed. That was probably not the right way to handle it.
Rxjs 5's WebSocketSubject is based off the same original implementation I based our socket off of (https://github.com/blesh/RxSocketSubject) which is unmaintained, so we couldn't add it as a dependency. In addition, I integrated some of our serialization/request handling logic into the socket layer, since I was writing it from scratch anyway. Using the rxjs 5 socketsubject would probably reduce some of the file-size, but we'd still need that socket class to handle things like the handshake and request_id handling. It's probably a good idea though

Should be fixed in #619


custom_comp_test2.py color orange not working

examples/tutorial/custom_components/custom_comp_test2.py

"bg-{self.bg_color}-500 hover:bg-{self.bg_color}-700 " for color orange doesn't seem to be working

The color orange is not available in tailwind 2.2.7

see https://v2.tailwindcss.com/docs/background-color for available colors in tailwind 2.2.7

To repair the example we change orange to yellow which is the closest to orange in the supported tailwind version.


"You exceeded your current quota, please check your plan and billing details."

This error appeared after I tried to update my API key, also I had already deleted my old key from the OpenAI system, which caused the issue. Initially, the error showed that I had an incorrect API key, but it displayed my old key. Now, despite paying for the pro plan, the error still appears.

Hmm, now it works. Before, I only had ChatGPT-Plus, but now I purchased a subscription here: https://platform.openai.com/account/billing/overview and now it works. I had no clue that was the problem. :)

Huh... thanks for the report, I guess! Closing!


WSO2 startup error because of relative path start with repository/ within the code

Description:
When we use above way of starting the server we get some error regarding relative path which start with repository/ within the code.
We also have found out following classes having above path problem.
APITemplateBuilderImpl.java
ThrottlePolicyTemplateBuilder.java
APIUtil.java
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Comment out the cd to carbon home command in the server startup script and start the server.
Related Issues:

Fixed with https://github.com/wso2/carbon-apimgt/pull/5748


Removing duplicate keys from translation file

Subject of the issue
I want to detect if my translation has duplicate keys or same text that has duplicate texts..
Your environment

version of angular-translate: 2.12.1
version of angular: 1.5.8
which browser and its version: 1.8.0

Steps to reproduce
Lets assume that I have an translation js file called messages_en.js which looks like this:
var translationsEN = {
	TASKS:'Tasks',
        TASKS:'Tasks'
}

In this case both the keys and values are same.
Expected behaviour
As the angular-translate-handler-log displays error message on the browser when there are missing translation, I expect my duplicate values also to be shown on the console of the browser.
Actual behaviour
There is no console right now even if I have duplicate keys and values, only duplicate keys, or only duplicate values.

Hello,
as we had already mentioned in the issue's template, we need a working and as minimal as possible demo covering your specific use case or issue.
Follow these steps:

Please open this plnkr base.
Fork it (at the top left).
Ensure the version of AngularJS and angular-translate is correct.
Ensure all angular-translate plugins are available; reduce and remove anything you can.
Write a minimal as possible demo for your specific issue.
Save and freeze. Provide us the final link to your demo.

Thanks.

Hello,
I'm closing this issue due inactivity. We have asked for additional details, but you could not helped us. Therefore we close this issue as not reproducible.
If you still have an issue, please create a new issue because we cannot be sure to track closed issues.


[SPARK-9489] Remove unnecessary compatibility and requirements checks from Exchange

While reviewing @yhuai's patch for SPARK-2205 (#7773), I noticed that Exchange's compatible check may be incorrectly returning false in many cases.  As far as I know, this is not actually a problem because the compatible, meetsRequirements, and needsAnySort checks are serving only as short-circuit performance optimizations that are not necessary for correctness.
In order to reduce code complexity, I think that we should remove these checks and unconditionally rewrite the operator's children.  This should be safe because we rewrite the tree in a single bottom-up pass.

/cc @marmbrus for review.

Merged build triggered.

Merged build started.

Test build #39110 has started for   PR 7807 at commit 9d76ce9.

Test build #39110 has finished for   PR 7807 at commit 9d76ce9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

LGTM

Thanks.  I'm going to merge this into master then will rebase the other commit.


Something is wrong wit the calculator_tool.ipynb cookbook

Cookbook located at tool_use/calculator_tool.ipynb is unable to be opened from GitHub website, nor after downloading the file.
Hyperlink to the file: https://github.com/anthropics/anthropic-cookbook/blob/main/tool_use/calculator_tool.ipynb
The exact error displayed on GH website is:
Invalid Notebook
The Notebook Does Not Appear to Be Valid JSON
Using nbformat v5.10.4 and nbconvert v7.16.1
The exact error displayed in VS Code after attempt to open the file:
The editor could not be opened due to an unexpected error: Bad escaped character in JSON at position 1785 (line 61 column 39)

Should be fixed now!


Update cosmos buildinfo.json

High-level description
Migrate config updates to open DC/OS
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS-19855 Migrate Describe
DCOS-19858 Refactor Update
DCOS-19856 Migrate Update
DCOS-19857 Release DC/OS

Related tickets (optional)
Checklist for all PRs

[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: the test can be found in the Cosmos repo
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): diff
[x] Test Results: link to CI job test results for component
[x] Code Coverage (if available): link to code coverage report

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot bump-ee

@jsancio @takirala

@mesosphere-mergebot label Ready For Review


InteractionManager freeze Unity from Editor to PlayMode

If press play, unity freeze in playmode bevor it starts the "game".
Is InteractionManager disable, playmode is running.. .
System:
CPU: 1505M v6
graphic card: Q5000
Windows Creators Update 1703
Unity 2017.1.0f3

Enable dev mode in settings for win 10

Thanks for your help!
I will check this!

Oh man, this was a great Info!
This is the a new laptop and i had forget to set this option!!!
Many many Thanks!!!


Add action to register user

Add action to register user.
At first, I thought that would need to create an admin user via command line and then use it to create via API but then I found out this documentation Register API that make it possible to create user using a shared secret (defined in the configuration file) and a nonce.
This makes things much easier now.

I'll continue working on the reviews of the tests on Monday.


really looking forward to your TU3 fixer, thanks in advance for all your effort ❤

Always happy to help! It was only prefabs this time around, if you are needing to update them, grab it here: https://github.com/keegars/MHR-PrefabFixer/releases/tag/v0.5

that really helps tks, your the best 🙂


E5Encoder, documentation dependencies

Fixes:
#15
Two simple E5 encoders were implemented: E5Encoder and E5InstructionalEncoder.
The only difference is what prefix they give to documents.
Documentation is updated.
#16
Doc dependencies listed in pyproject.toml
#17
Temporary fix: commented L45 out in mkdocs.yml (the custom_templates line)
Site looks ok.

good comments, thanks!

@x-tabdeveloping reworked the encoder, check it out

there's a new problem with documentation:
newline characters break the code blocks
what is should be:
    Examples
    --------
    Instructional models can also be used.
    In this case, the documents should be prefixed with a one-sentence instruction that describes the task.
    See Notes for available models and instruction suggestions.
    ```
    from turftopic.encoders import E5Encoder

    def add_instruct_prefix(document: str) -> str:
        task_description = "YOUR_INSTRUCTION"
        return f'Instruct: {task_description}\nQuery: {document}'

    encoder = E5Encoder(model_name="intfloat/multilingual-e5-large-instruct", preprocessor=add_instruct_prefix)
    model = GMM(10, encoder=encoder)
    ```

    Or the same can be done using a `prefix` argument:
    ```python
    from turftopic.encoders import E5Encoder
    from turftopic import GMM

    prefix = "Instruct: YOUR_INSTRUCTION\nQuery: "
    encoder = E5Encoder(model_name="intfloat/multilingual-e5-large-instruct", prefix=prefix)
    model = GMM(10, encoder=encoder)
    ```

what it is:

Site won't be rebuilt, code commited in another pull request


Update Composer dependencies (2021-07-23-00-13)

Visual regression test passed!
View the visual regression test report


Calendar Questions

Hello, thanks for creating this plugin. It's awesome! How long did it take to create?
I have a few questions about it.

User mappings are overridden by default? I have two functions that allow me to
resize my windows with repeat.vim. Calendar.vim uses these keys as well as many
other keys that I need. So do I have to do this autocommand for all my mappings that
are used by calendar.vim ?

function! MoveRight(mycount) abort
	let defaultRepeat=15
	let size=  a:mycount == 0 ? 1 : a:mycount
	execute "vert resize +"  . size
		call repeat#set("\<Plug>(MoveRight)",size == 1 ? defaultRepeat:size)
endfunction
function! MoveLeft(mycount) abort
	let defaultRepeat=15
	let size=  a:mycount == 0 ? 1 : a:mycount
	execute "vert resize -"  . size
		call repeat#set("\<Plug>(MoveLeft)",size == 1 ? defaultRepeat:size)
endfunction

nnoremap <silent> <Plug>(MoveRight) :<C-u>call MoveRight(v:count)<CR>
nmap <c-l> <Plug>(MoveRight)
nnoremap <silent> <Plug>(MoveLeft) :<C-u>call MoveLeft(v:count)<CR>
nmap <c-h> <Plug>(MoveLeft)

augroup calendar-mappings
		autocmd!
		autocmd FileType calendar nunmap <buffer> <c-l>
		autocmd FileType calendar nunmap <buffer> <c-h>
		"This is needed to enter the command line q:
		autocmd FileType calendar nunmap <buffer> q
		autocmd FileType calendar nunmap <buffer> <c-g>
		autocmd FileType calendar noremap <buffer> q q
		autocmd FileType calendar nmap <buffer> <c-k> <Plug>(calendar_view_left)
		autocmd FileType calendar nmap <buffer> <c-j> <Plug>(calendar_view_right)
		autocmd FileType calendar nmap <buffer> J <Plug>(calendar_down_large)
		autocmd FileType calendar nmap <buffer> K <Plug>(calendar_up_large)
		nnoremap <c-G> :tabn<cr>
augroup END

Maybe a setting 'g:calendar_override_user_mappings=0'?

Can I edit the ~/.cache/calendar.vim/ files or will that break something
in the plugin? I presume my edits would have to be made before calling
Calendar. My idea is to import ics events into these json files. It would
look something like this:

function! AddEvents() abort
if  'saved_md5sum' !~ 'events_file_md5sum'
"Events file was changed
	call system('Script to read/write events to json files in ~/.cache/calendar.vim.')
endif
execute "normal!  :Calendar -position=topleft  -width=100 \r"
endfunction
command! Cal call AddEvents()

This would replace the call back function from script 52 'calendar_sign'
Ideally an ics import feature as mentioned before would be nice in
calendar.vim. This is pretty much irrelevant if I decide to stay with google
calendar but would like to know for future reference.

What is the difference between agenda and event? It looks like they both
add an event when 'i' is entered. Here is my setup.

let g:calendar_views =['year', 'month', 'week', 'day_3', 'day','event', 'agenda']

How do I change the syntax for the current calendar buffer? I read the
docs for ownsyntax, I could not figure it out. When attempting to change
the background I end up changing all of my window buffers background.

How do you create a task with a note? According to the docs I should
add the 'note:' item to it but don't know if it's saving properly as a
note because the json file looks different. It saves it as a title
and without an entry for 'note' or 'notes'. I think that the
notes section is only relevant for google calendar and not the local/.

:Task:Some Task note:This is something that needs to get done.

{"id":"202101301628360005","items":[{"id":"202101311716540004","title":"Some Task note:This is something that needs to get done."}],"title":"Task list"}

According to the docs :
Task with note:
TASK: [task-title] note: [task-note]
Is there a way to view the note in a task or just enter 'i' and view it in
the command line?

Events - How come for the events on google I can't see the details but for the tasks I can?
Tasks seem to work fine but I can't add an event with a note or read the notes in an event.

Thanks for everyone's contributions to this project.

Sorry for keeping this unanswered for years, several questions at the same time demotivated me from answering. Let me close with a few comments.

Yes, currently there's no way to disable the plugin mappings. If you could send me a patch, I can review that.
You can modify the cache files at your own risk, but there's no guarantee that the file structure does not change in the future.
The agenda view shows the upcoming events (including the events starting tomorrow), as requested by #88.
You can overwrite the syntax with something like autocmd FileType calendar highlight CalendarToday ....
Yes, the note: is implemented only for Google Task.
Maybe you mean the description in calendar events? Unfortunately it's not supported in this plugin.


Initial Commit

Data preprocessing by removing special characters and numbers in column "message"
Performed SVM, Naive Bayes, Logistics Regression (Still finding way to lower overfitting)

not to be added

.

.


sls-dev-tools v.1.10.0 does not start at all

Describe the bug
I am trying to launch sls-dev-tools with one of our stacks following the required cli command structure: "sls-dev-tools --stack-name ... --profile ... --region ..." but the command returns immediately without any message. The same command works correctly with version 1.0.6.
Is there any debug flag to get more details about the reason of this behaviour?

@TLaue Looking into this now, we have been able to replicate your error and should be able to fix this soon. Having a debug mode for the tool is something we hope to have in the future

@TLaue apologies this appears to have come from a regression in the v1.1.0 release where we broke the mfa prompt we had put in place from your previous issue. This should now be working in the latest release v.1.1.1.
Would you be able to confirm the tool now works for you?

@DotGav The MFA prompt appears now, I can enter the MFA code, submit the form which is closed afterwards but nothing happens. All windows are empty (no lambdas...) and I am getting the original error (#115) back after some time. I cannot confirm that the latest fix has solved this problem.

@TLaue We've looked into this issue again, and have released a new version to fix this. Can you confirm the tool is working as expected on v1.1.3?

@mansurpasha I can now confirm that it is working again! Thanks!


Include command to install codejail from pip in readme

Thanks for the pull request, @miquelvir! I've created OSPR-6228 to keep track of it in JIRA, where we prioritize reviews. Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.
:warning: We can't start reviewing your pull request until you've submitted a     signed contributor agreement     or indicated your institutional affiliation. Please see the CONTRIBUTING     file for more information.

@miquelvir Thank you for the contribution! Once you have signed our Contributor Agreement and it has been processed, you will receive a confirmation via email from our Legal team. We then will be able to review this and all your future code contributions.

@miquelvir We have not heard from you in a while, so I will close this for now. Please feel free to re-open should things change.

@miquelvir Even though your pull request wasn’t merged, please take a moment  to answer a two question survey so we can improve your experience in the future.


ingest: implementation discussion

@mgrauer suggested that we might want some additional ec2 instance for the data injest -- to validate data before we let it/them into the archive.
I thought about it and wondered if should bother with any such staging area atll.  And my main argument for myself was: whatever we (pynwb + DANDI) consider valid today might become "invalid" tomorrow as more validation checks get established.  E.g., we did see quite a number of new checks added (and some other fixed) in bids-validator.
So I wonder if we should just limit validation to client(s) (to start with the cmdline dandi-cli, later possibly some JS fronted for web UI upload), and then (eventually, not for SfN ;-) ) develop into Web UI an indicator (e.g., an exclamation in some red or yellow triangle), either any particular file (or dataset as a whole) passes validation at that particular moment in time.  We could also provide email notifications (to uploader/maintainer(s), need to add to DB schema) whenever a file/dataset goes from "valid" to "invalid" state or vise versa.
NB metadata extraction could also be done "dynamically" on already present in the archive content
If we decide to go this route for re-validation of data upon changes in NWB/DANDI validation procedure, there will probably be no need for separate ingestion staging area.
Or may be there would be other purposes for staging area beyond validation/metadata extraction?

@mgrauer suggested that we might want some additional ec2 instance for the data injest -- to validate data before we let it/them into the archive.

I think you maybe are conflating two different things that I've said, probably I did not clarify enough the difference between them.
I have specifically mentioned the idea of a ingest process to the dandiarchive (note that this is not necessarily the same thing as an ingest process to Girder), but I haven't described what the ingest process would be because I don't have a clear idea yet, as we don't yet have clear requirements. This is something that I see as a longer term goal.
One example of an ingest process (I'm not advocating for this one, this is just an example) could be: upload compressed files to an S3 bucket, the S3 bucket object creation events trigger lambdas which download the data, uncompress it, validate it, extract metadata and compute summary statistics, then move the uncompressed file to a different S3 path and update some database with the metadata, summary statistics, provenance, and S3 storage path. Even if Girder+S3 is the ultimate place to store the metadata and blob data, having this separate ingest process from Girder allows us to scale and evolve the ingest process independently of the archive, and to keep the errors from one system from impacting the other.
I've also mentioned the use of an EC2 instance in the same region as the dandiarchive as a development helper in the near term. In these early days we are simply trying to find valid data and figure out where to put it and what metadata to extract. As there are some downloadable NWB files available to us (e.g. Buzsaki data), I thought it would be easier to pull the data into the EC2 instance and play around with it there manually, as the EC2 instance might have better bandwidth for pulling data in than other non-AWS servers, and further, once the data is on the EC2 instance, we can very quickly push it into S3 or some other dandiarchive piece, as the EC2 instance is inside AWS in the same region as the rest of dandiarchive (as opposed to downloading heavyweight data to some server and then pushing that same data to AWS). This is kind of a ingest process, but much more of a manual ingest process that is exploratory rather than what I would see as a scalable and production ready ingest process.

I think this gets subsumed into the discussion of the unified API. Some of the ideas are still relevant, but should be revisited anew in light of current circumstances.


docs: add extensive documentation

Add documentation for the compliance suite
Add images for reference
Update README.md with home page documentation
Add license

I have made the changes. Kindly review them please.


Integrations tests are warning of an unknown mark integration_timescale

Bug Description
The integration tests are showing a warning about an unknown mark pytest.mark.integration_timescale.
For example.
tests/ops/integration_tests/test_sql_task.py:1172
  /fidesops/tests/ops/integration_tests/test_sql_task.py:1172: PytestUnknownMarkWarning: Unknown pytest.mark.integration_timescale - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html
    @pytest.mark.integration_timescale

Steps to Reproduce

Run nox -s pytest_integration
View the logs

Expected behavior
No warning.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Version:
OS:
Python Version:
Docker Version:

Additional context
Add any other context about the problem here.

argh, I forgot to add that integration_timescale to https://github.com/ethyca/fidesops/blob/main/pytest.ini


Initial build files for the docker image

Renamed the shell script to Docker-ESP-IDF.sh
Added a PowerShell script (untested) to build and run the Docker container
Added a section to the README.md file to explain how to build and run the Docker

The changes look good.


Doesn't change focus when activating command palette in Atom Beta, events dispatch to the last active pane.

When activating the command panel from an editing pane, keystroke events are dispatched to the editing pane rather than the command palette.
When activating the command palette from settings view or another non-editable pane, the following error is thrown when editing:
Uncaught TypeError: Cannot read property '_atomicEmacs' of undefined
System Info
Atom: 1.19.0-beta0 x64
Electron: 1.6.9
OS: Mac OS X 10.12.6
Thrown From: atomic-emacs package 0.11.0
`

Any chance you have a stack trace?

Yeah, sorry about that.
To reproduce:

Go to settings pane
activate the command palette
kill-line

TypeError: Cannot read property '_atomicEmacs' of undefined
  at Function.module.exports.EmacsEditor.for (/path/to/.atom/packages/atomic-emacs/lib/emacs-editor.coffee:19:20)
  at getEditor (/path/to/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:24:3)
  at HTMLElement.atomic-emacs:kill-line (/path/to/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:85:44)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (<embedded>:21771:35)
  at <embedded>:21509:65
  at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (<embedded>:66623:22)
  at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (<embedded>:66414:28)
  at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeyEvent (<embedded>:18112:42)
  at HTMLDocument.<anonymous> (<embedded>:18015:65)


feat: support v2 backing image download file from sync service

ref: https://github.com/longhorn/longhorn/issues/6341

Modify download backing image url handler for creating v2 backing image.

For qcow2, we will need to transform to raw first
We don't need to compress the data

TODO:

[x] Handle qcow2 image, when creating qcow2 image, the checksum and the size is based on the qcow2. It will fail if we download the raw content to the spdk lvol. The checksum will not be the same.

pass dataEngine to the datasource pod
in the finish processing, if this is a v2 backing image, we convert it to raw and update the size and checksum

Summary by CodeRabbit
Summary by CodeRabbit

New Features

The file download route now supports both "GET" and "HEAD" methods, allowing users to retrieve headers without downloading the file.
Enhanced handling for file downloads, including support for different file formats and improved processing for QCOW2 files.
Introduced a new query parameter to control file processing behavior.
New constants for data engines have been added to improve configurability.

Bug Fixes

Improved error handling and logging during file operations to ensure better user feedback in case of issues.

@coderabbitai pause


Update JCB minutes 17 Oct 2024

@Zsailer did you mean to approve the pull request? https://github.com/jupyter/governance/pull/238


FancyZones: Not working with Application Guard

Environment
Windows build number: 1903 18362.10024
PowerToys version: <IP_ADDRESS>
MS Edge version: 44.18362.10024.0
MS Edge(Chromium) version: 79.0.309.5 (Official build) dev (64-bit)
PowerToy module for which you are reporting the bug (if applicable):
FanzyZones
Steps to reproduce
Start Edge in Application guard(regular edge or edge chromium).
Try to Shift-drag
Expected behavior
Zones should show and the window should snap to zone.
Actual behavior
No zones is showing when trying to Shift-dragged.

@jopose
thank you!

So, it turns out those Application Guard windows are weird. First, they do not send EVENT_SYSTEM_MOVESTART nor EVENT_SYSTEM_MOVEEND messages. We depend on them to detect if a window is dragged by mouse. It sends other messages though.
And secondly - if moved with WinKey + arrow key, they bug out, and instead of being moved they end up cropped (note the right side of the window, the minimize/maximize button is missing):

While we might have some luck with moving the window in some other way to prevent cropping, there is little we can do if the window does not sends mouse move events.

@bzoz: WinKey + Arrow works for me.(Normal windows behaviour, that is. No FanzyZones)
Windows build number: 1903 18362.10024, Swedish.

There is a setting for overriding the Windows snap button, make sure it is enabled.

@bzoz: Ah, ok then the window gets cropped. But the window moves between the diffrent zones cropped.

@crutkas
I don't think this is fixable.
Each Application Guard window is hosted in the hvsirdpclient.exe process.
We don't receive any system notification from individual Application Guard Edge windows.
Also using Win+arrow, to snap those windows, result not only if cropped windows, but also in unresponsive windows.
Btw, Windows 10 Snap shows a very different rendering when snapping regular Edge windows vs Application Guard Edge windows, suggesting something custom is going on for Application Guard windows.

Lets revisit later then based on this investigation.
Idea is to treat app guard windows like we do when we aren't elevated and there is an elevated window.  "hey, this is an application that we can't interact with due to ..."

Tracking this issue in #6193

Nothing that we can do about it. When an app runs in Application Guards, FancyZones doesn't have access to keyboard/mouse events.
Added to https://github.com/microsoft/PowerToys/wiki/Application-Compatibility


Explicitly open text-mode files with UTF-8 to fix Windows issues and ensure correct cross-platform behavior

Previously, an encoding was not specified when opening the setup.py, resulting in a platform- and user-environment-dependent encoding being selected; in particular, Windows defaults to the user's local code page, not UTF-8. This results in the setup.py not being readable on Windows (or in other cases where the user has selected a non-UTF-8 encoding) if it contains any non-ASCII characters, resulting in a crash. This PR prevents this by specifying encoding="utf-8", as should be standard practice for the vast majority of cases for non-trivial cases of opening text-mode files.
In addition, it does the same several other places in the tests to ensure they work cross-platform as well, and also fixes several instances where a with context manager was not being used to open files/resources, resulting in ResourceWarnings and other potential downstream problems.
Fixes #28

@regebro Are the tests not run in the CI :open_mouth: ? I see a .travisci; was that disabled due to Travis sunsetting open-source project support? Shouldn't be too much effort to port that to Github Actions, if that's the issue...I'd be happy to help.

I'm worried about hardcoding to UTF-8, that's limiting. I'll think about some other options. The other changes are good.
There are modules to guess encodings, that's an option as well.

Getting help moving to github actions would definitely be nice, btw.

Yeah, given the extreme rarity of a setup.py in properly-packaged code that using an obsolete (non-UTF-8 compatible) encoding in (I've yet to see one in the wild, at least that actually had an intentional encoding marker that could be detected per PEP 263, I didn't address that corner case. However, see PR #56 for my further comments on that and a proposed improvement.

Getting help moving to github actions would definitely be nice, btw.

Sure, let me take a look. From my experience, for security reasons newly-added actions won't actually run on the PR that adds them unless the PR feature branch is on this repo, not mine. Options to deal with this:

Temporarily add me as a collaborater on the repo so I can create a new branch here and create a PR for it against master, so the port can be tested and debugged immediately before merging, without any extra PR churn or other steps from you.
Create a new branch add-github-actions on this repo, I target the PR against that; the actions will only run after you merge my PR to that branch and any fixes would require more PRs, but at least we can treat that branch as a feature branch and you can do fixups/squashing/cleanup as necessary once everything's working before merging to master
Just target master directly, and use followup PRs if needed to get everything working


Request for Dark Mode

I really like this theme and have been applied for my personal site, are you considering to implement dark mode?

Of course, I'll be working with @dreamerblue  on this.

great!  I'll look forward to the update, by the way, thank you!

@natkingchloe close for https://github.com/MeiK2333/github-style/pull/28


Add NLP demo to site

Description
https://jira.mesosphere.com/browse/DCOS_OSS-3807
Urgency

[ ] Blocker
[ ] High
[ x] Medium

Requirements

See the contribution guidelines.
Build content locally and test for formatting/links.

retest me please

retest this please


Removal of outdated Controller parts

Currently, controller contains several outdated (sometimes even broken) parts:

bfd application (controller/src/bfd), even doesn't contain needed include files;
bgp daemon (controller/src/bgp/daemon), the purpose of the program is unclear;
OVS ToR agent (controller/src/vnsw/agent/ovs_for_tor_agent), most likely even not used today by anyone in the world;
VxLAN agent (controller/src/vnsw/agent/vxlan_agent/linux) looks like contains wrong linker settings, i.e., most likely not used today.

All these programs are not included in contrail.spec file


Need whitelist or blacklist of status which to be considered for reminder notifications.

Please share the whitelist or blacklist of status for which the remainder scan to be ran for legislative due date and CFR date seperately.
Below given are consolidated list of status from system:
1 Open
2 Call For Records
3 Closed
4 Redirect
5 Unopened => Ignored; coz the reminder checks are ran only for opened request
6 Intake in Progress => Ignored; coz the reminder checks are ran only for opened request
9 Consult
10 Ministry Sign Off
11 On Hold
12 Deduplication
13 Harms Assessment
14 Response
8 Fee Estimate
7 Records Review
15 Archived => Ignored; coz the reminder checks are ran only for opened request

@sumathi-thirumani-aot Due date reminders to be ignored for the following states:

Unopened
Intake in Progress
Redirect
On Hold
Closed
Archived

All other states have due dates applicable and reminders should be generated. Any other thoughts @m-prodan?

Per discussion at Sprint Planning on Jan 19, once a request has moved past the Call for Records state (options would be to Deduplication, Harms Assessment, Fee Estimate, Records Review, Ministry Sign Off) then the Call for Records state is considered complete and no reminder notifications are required for the Call for Records due date.
Legislated Due Date reminder notifications will continue until the request has been Closed, except for the exceptions noted in previous comment.
Let me know if you have any questions @sumathi-thirumani-aot

This covers 2 scenarios:

Reminder for CFRs: CFR reminders will be sent only when the application is in CFR state and nearing due.

Technical: This applies to status id = 2

Reminders for LDDs: Reminder notifications should be sent only when the application is NOT in status listed Unopened = 5
Intake in Progress = 6
Redirect = 4
On Hold = 11
Closed = 3
Archived = 15

To test this we need to be able to search for Closed requests. Testing will required the completion of search.

@nkan-aot Can you help test this one? let's connect.

@sumathi-thirumani-aot, let's connect on Monday how to expedite this one through QA.
cc @m-prodan


Bomb out if dangling nodes, and warn if unary nodes present

As discussed with @awohns

This looks perfect, I say go ahead and merge!

I'll add another test just to get the coverage up , then merge.


Add ability to get output as plain-text

This is mostly useful when running Bombardier client on local machine where the output should be human readable.

@sebastienros @Maoni0


roslisp_common: 0.2.10-0 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository roslisp_common to 0.2.10-0:

upstream repository: https://github.com/ros/roslisp_common.git
release repository: https://github.com/ros-gbp/roslisp_common-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.7
previous version for package: null

Holding for sync


Needs support for OpenXR, Unity 2020 LTS, MRTK v2.7, and Package Manager Plugins for ARCore/ARKit

The current release for Spectator View doesn't align with MRTK team's plan to release mixed reality tools with OpenXR and Unity Package Manager support in Unity 2020+. A few namespaces used for Spectator View are also deprecated. As a result, working around these issues in a modern Unity project gets messy very quickly.

@chrisfromwork @davidkline-ms @keveleigh  Quick q-- is the Spectator View project still active at Microsoft? Attempting to get a forecast for implementing Spectator View into more XR projects, but there are many outstanding issues in this repo going without a response. Our team isn't sure where things stand with this product and having some clarity would be a huge help. No worries if there isn't an answer available right now.

I second this! Any word? Especially when you need to upgrade to 2020.3 or above in order to get around UI canvas elements not recording in holocapture mode: https://stackoverflow.com/questions/68692824/hololens-video-recording-not-showing-ui-elements

How would I go about "removing" the spatial anchors, as iam using QR in my existing project?


[Question] Skype - long message truncated?

System Information

SDK Language: .NET
SDK Version: <IP_ADDRESS>
Development Environment: App Service

Bot Info

Bot name: ICMTestBot
Bot handle: ICMTestBot
App ID: f9cf18d8-9e89-459b-86c2-5ce1840f449a
Connected channels: Skype,Teams, SMS

Issue Description
Skype Message text is getting truncated.
Steps to Reproduce the Issue

Enter long text in Skype message

Actual Results
Message is truncated and ellipsis (...) is shown.

@ashice Use <br/> with the closing slash.

@nwhitmont Got it.  Longer messages are still truncated though.

duplicate of https://stackoverflow.com/questions/45684118/skype-bot-truncating-display


Add Puhti web interface R4 changes

Adds all changes in Puhti Release 4 in one PR to simplify merging, #1110, #1111, #1114, https://github.com/CSCfi/csc-user-guide/tree/wwwpuhti-R4-release-notes

Wait until the new changes are in production (deployment starting at 15:00)


Bug in conditional statement in for loop

while giving condition in for loop, when someone gives space between "<" and  "=" it gives error.
forexample if i write "< =" with space between them it gives error. ideally it should not give an error.

Challenge Name

https://www.freecodecamp.com/challenges/iterate-odd-numbers-with-a-for-loop#
Issue Description

i wrote the statements like this
for (var i = 1; i < =9; i += 2) {
ourArray.push(i);
}
space in the conditonal statement between < and = sign is clearly visible and it gives error.
Browser Information

Mozilla firefox
windows 10:
Desktop:

Your Code

Screenshot
picture with error

picture without error

@engrfaisal90 thanks for the issue but this is expected behavior. The operator <= is very specific and you can't separate it with a space. If you get stuck or have questions with a challenge in the future, please first ask questions to the Help Room. Happy coding!

Thank you for your response. As i haven't seen such kind of issue in other compilers that's why i raised this issue. In  most compilers white spaces doesn't matter. Thank you by the way.


Lifecycle method called correctly once

This change is to fix both of #1161 and #1222.
To fix #1222, RootAgent (and Agent) should not retrieve outputs of output plugins dynamically in #lifecycle. Removing it solves the problem to call shutdown methods twice.
But this fix re-opens #1161 problem.
If all plugins are flat/static plugins or MultiOutput plugins which initialize all stores by static configuration, all plugin instances are registered in agent.outputs.
But once MultiOutput plugin is initialized dynamically by any plugins (e.g., fluent-plugin-forest), its sub-store/sub-plugin instances are not registered in agent.outputs. Lifecycle methods of these plugins are never called.
This change will add a feature to call lifecycle methods in MultiOutput. It's a kind of dirty workaround, but works well.

@repeatedly Could you review this change?

It seems good. Removing recursive_output_traverse.call(store) in lifecycle also resolves same inssue with copy_ex like v0.12 MultiOutput based plugin.

@repeatedly updated this change.

@repeatedly ping.

LGTM


implement parallelized version of hill climbing

we can evaluate all successors in parallel, which will help when the heuristic evaluation is the bottleneck (which it is for predicate learning)

i can look into this but i think we might find that the overhead is too great

This looks good at a glance. Why didn't you want to make it into a PR? Couple of questions though:
1 - will it eat up the entire processor on our Macs?
2 - did you verify the scores are the same as in sequential mode?

I just didn't make a PR because I didn't have time yet to test it (so no I didn't do 2 yet). Re: 1, my computer didn't seem to slow down, but we should see.

i can take over on it later


⚠ Bofrost.be has degraded performance

In f73d564, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1034 ms

Resolved: Bofrost.be performance has improved in 94c4096.


the code out of the box doesn't work

Hello,
I tried the attached sample and I got an error: Cant find controller for update, code is below:
const Telegram = require('telegram-node-bot')
const TelegramBaseController = Telegram.TelegramBaseController
const tg = new Telegram.Telegram('I placed my token here')
class PingController extends TelegramBaseController {
/**
* @param {Scope} $
*/
pingHandler($) {
$.sendMessage('pong')
}
get routes() {
    return {
        'ping': 'pingHandler'
    }
}

}
tg.router
.when(['ping'], new PingController())

It's just a warning, you don't have a handler for this command


Look into recognizing and rendering latex within docstrings for generated documentation

Sometimes we use latex in our docstrings. It would be ideal if it was actually rendered as latex on the documentation website.

Fixed by https://github.com/quantumlib/Cirq/pull/1184


Document: Creating Service Account

[ ] Verify the steps shared
[ ] Add the documentation to the Wiki

Initial Draft: https://github.com/splunk/ta-webex-meetings-add-on-for-splunk/wiki/Troubleshooting

Added steps to the README.md

ae26c20
830f207
6f964c3


Integrity backend

@YurkoWasHere thoughts on this?
We can use these commands to simply preprocessor a bit:
type: docker
    name: docker-compose ps (Name)
    image: docker inspect --format='{{.Image}}' $INSTANCE_ID

type: git
    repository: git remote get-url origin
    branch: git branch --show-current
    commit: git rev-parse HEAD
    clean: is_empty(git status --porcelain) [see: https://unix.stackexchange.com/a/155077]

    clean: is_empty(git status --porcelain) [see: https://unix.stackexchange.com/a/155077]
This will be hard, there are modifications we make to things we don't own
For example docker-compose.yml in Browsertrix :
We change the map the data to a mounted volume
Change the access ports from 80 to 81 on authsign so it doesn't conflict with nginx
We expose a port that normally is not exposed for authsign
Wouldn't want to ignore a docker compose, and even if we had access to the repo (its 3rd pary)

git remote get-url origin
We have to be careful here since it will also include Personal Access Codes when dealing with private repos.

This will be hard, there are modifications we make to things we don't own

Then it'll simply return false and that's fine for now. But what this tells us to do is to actually commit those changes in a fork, this way the actual code we run is in a commit. Otherwise we could just change the entire program, and the whole recorder metadata would be pretty pointless.

We have to be careful here since it will also include Personal Access Codes when dealing with private repos.

Yes I saw that. Was wondering why we don't deploy read-only without PAT. I think we also can set PAT as env var.

Hiding the PAT
You need to hide the credentials somewhere. Make this file:
starling@integrity:~$ cat ~/git.store
<EMAIL_ADDRESS>
Then set up git to use it:
starling@integrity:~$ git config --global credential.helper 'store --file ~/git.store'

I suggest using a filename like ~/.git-credentials instead.
I did this on stg, and was able to clone a private repo without entering username and password:
starling@integrity:~$ git clone https://github.com/starlinglab/organizing-private.git
Cloning into 'organizing-private'...
remote: Enumerating objects: 8, done.
remote: Counting objects: 100% (8/8), done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 8 (delta 0), reused 0 (delta 0), pack-reused 0
Receiving objects: 100% (8/8), done.

You'll see that the in-repo URL now does not contain creds:
starling@integrity:~/organizing-private$ git remote get-url origin
https://github.com/starlinglab/organizing-private.git

This is the way to go :)

some changes are environment specific and wouldn't ever get committed to the the repo we pull from (unless we fork and then we have to maintain a fork).

Something is either:

in .gitignore, or
included in a commit

When someone audits the repo at the hash, it is clear that (1) something is ignored but it is a restricted set of things that can change (e.g. creds, some output/) which do not affect the integrity, (2) the code is included in a commit.
In the absence of this, someone can easily git clone -> rm everything -> cp another code in -> run that which makes the whole thing pointless.


Test gntp.notifier fails

Hi, I installed gntp with pip on my machine running ubuntu 15.10. The installation looks ok, but when I type:
python -m gntp.notifier  #to check the installation

I get the following error:
INFO:__main__:Sending registration to localhost:23053
ERROR:__main__:Growl error
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/gntp/notifier.py", line 242, in mini
result = growl.register()
File "/usr/local/lib/python2.7/dist-packages/gntp/notifier.py", line 97, in register
return self._send('register', register)
File "/usr/local/lib/python2.7/dist-packages/gntp/notifier.py", line 204, in _send
raise errors.NetworkError(exc)
NetworkError: [Errno 111] Connection refused

Do you have an idea what is going on and how to fix it? Thank you

I'm not certain about Growl on Linux but do you have a copy of Growl running?
I know there is a Growl for Linux project ( http://mattn.github.io/growl-for-linux/ ) but I have never used it so I do not know how well it is supported. I think most people who are using this library, are running it to send messages to the OSX Growl ( http://growl.info/ ) or Growl for Windows ( http://www.growlforwindows.com/gfw/ )
This library by itself does not create the messages on your desktop, but it talks to a Growl application to display those messages.

Actually I do not have a copy of Growl running. My first attempt to get growl notification was following the tutorial at http://masnun.com/2013/08/18/python-sending-growl-notifications.html. Do you think that having Grow installed is a prerequisite to make gntp work?
I would like to use growl notification triggered from my ubuntu localhost targeting a windows virtual machine.

To send a Growl message to a different computer, you need to specify the hostname argument, which the default "run as module" quite reasonably does not do -- it tries to send to localhost, so if you don't have a local Growl client running, it will fail.
I've tested sending Growl messages going the other way (sending a message from Linux virtual machine to Windows host), and it works:
import gntp.notifier
import logging

# enable display of debug messages
logging.basicConfig(level=logging.DEBUG, format='%(asctime)s - %(levelname)-10s - %(message)s')

# send a test (use IP address or hostname)
gntp.notifier.mini('foo', hostname="<IP_ADDRESS>")  # target host IP address

The Windows growl settings under the Security tab need to have "Allow network notifications" on, and if the "require password" options are enabled, a password needs to be assigned in the growl "password manager", and password must be passed as a parameter in addition to hostname.

I think onionradish covered most of it. The default "run as a module" is meant as a simple check to see if it's installed correctly, but perhaps I should make the error messages more clear and add a bit of a better note in the documentation.
This library ships with a "gntp" command that you can use to send a message and check to see if it is hitting your VM correctly.
gntp -m test -H <IP_ADDRESS> -P password

@onionradish and @kfdm Thank you for your support, that helps. However I am still fighting to make it work. I guess the problem is I am not able to install growl on my windows VM, i.e. when I run the GrowlInstaller.exe it asks me for .NET version 2.0.50727 which is weird because .NET is already installed on my VM. Thus, if I try to run the example suggested by @onionradish at first seems to work but then I get a growl error:
2016-03-03 09:15:15,839 - INFO       - Sending registration to <IP_ADDRESS>:23053
2016-03-03 09:15:15,844 - DEBUG      - To : <IP_ADDRESS>:23053 <<class   'gntp.core.GNTPRegister'>>
GNTP/1.0 REGISTER NONE
Application-Name: PythonMini
Origin-Platform-Version: Linux-4.2.0-30-generic-x86_64-with-Ubuntu-15.10-wily
Origin-Software-Version: 1.0.2
Origin-Machine-Name: HP-EliteBook-Folio-9470m
Origin-Software-Name: gntp.py
Origin-Platform-Name: Linux
Notifications-Count: 1
Notification-Enabled: True
Notification-Name: Message

2016-03-03 09:15:18,847 - ERROR      - Growl error
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/gntp/notifier.py", line 242, in mini
result = growl.register()
File "/usr/local/lib/python2.7/dist-packages/gntp/notifier.py", line 97, in register
return self._send('register', register)
File "/usr/local/lib/python2.7/dist-packages/gntp/notifier.py", line 204, in _send
raise errors.NetworkError(exc)
NetworkError: timed out

So I managed to install Growl on windows10 VM. The .NET required was problematic because I had updates disabled. After re-enable them I could install .NET and Growl.
  However no Growl Screen appears...Is this normal? Is there another way I can access the Growl screen in order to "Allow network notifications" and "password" options? I will keep trying

You should address these questions to the GfW Google group - make sure that Growl for windows is running as it should first! https://groups.google.com/forum/#!forum/growl-for-windows
(try double-click on the Growl icon in the try to get settings, and then test it using the installed growlnotify tool on the windows machine... !)

I navigate to the folder GrowlforWindows in my cmd and then run the growlnotify.exe. After that I can write messages that appear on my desktop. So the growlnotify works. I addressed my original question the GrowlGooglegroup.

If you have Growl working on your Windows VM, then I suppose the next step is to make sure you can send a message from Linux. Make sure the Windows firewall is allowing messages on the Growl port (default is 23053) and that Growl is listening.

I managed to send desktop notification from my linux host through Ansible executing a Powershell script. But when using python I have again an error:
gntp.error.NetworkError: [Errno 104] Connection reset by peer

I changed the advanced setting for the Windows Firewall allowing port 23053 to be opened but that does not improve the situation. The main problem is that I cannot access to the Growl Screen where I can change the settings. For example I would like also to set Growl to start automatically when I log into Windows VM but I am unable to do that. Is there an alternative way to change settings (so that I can follow the tutorial at http://www.growlforwindows.com/gfw/help/#notifications) when you do not have access to Growl Screen?

You may be able to manipulate the growl settings directly
/Users//AppData/Local/Growl/<IP_ADDRESS>/user.config I think
Something like this
<?xml version="1.0" encoding="utf-8"?>
<configuration>
    <configSections>
        <section name="userSettings" type="System.Configuration.ClientSettingsSection, System, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089" />
    </configSections>
    <userSettings>
        <setting name="AllowNetworkNotifications" serializeAs="String">
            <value>True</value>
        </setting>
    </userSettings>
</configuration>

On my windows 10 VM the file you are referring to is called Growl.exe.config and it is under C:\Program Files (x86)\Growl for Windows/`. I edited this file and changed to true the values:
Autostart = True
AllowNetworkNotification = True
EnableLoggin = True

I restarted the VM but I still need to click on Growl icon to start growl, and still gntp from linux host does not work...something else to change that you are aware of?

I'm not sure. At this point you may need to wait for a response on the GfW forum. I don't use Windows as my primary environment (and I'm still running Win7) so I'm not aware of all the nuances of Windows 10.

Thanks for your help..

If it's alright with you, I'm going to close this ticket for now while you wait to hear back from the GfW forum. If they get back to you and think it's a problem with this library, please feel free to re-open this thread :)


[chore](cancel message) Simplify cancel message of BufferControlBlock

What problem does this PR solve?
Issue Number: close #xxx
Related PR: Introduced by https://github.com/apache/doris/pull/41798
Too much cancel msg in be.INFO now. Stop printing them.
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

duplicated by https://github.com/apache/doris/pull/43267


web_benchmarks_html text_canvas_cached_layout.html.layout.average and text_canvas_color_grid.html.text_layout.average regression...

Looks like roll https://github.com/flutter/flutter/pull/59222.

This is the same roll as implicated in https://github.com/flutter/flutter/issues/59335, but they may not be related.

Reducing priority because canvas-based text layout is under experiment flag.

The increase in text_canvas_color_grid.html.text_layout.average is expected, though it's higher than I saw on my laptop. The reason I was okay with it is because the actual effect on the app was not significant.
That being said, I have a few ideas that will help here.

the actual effect on the app was not significant.

If the benchmark is not representative, please change the benchmark.

Reducing to P4 because this is not a customer critical issue. This is a performance regression of an experimental feature that is under active development.


TextureProgress not working when the value is bigger than 62

The same issue : https://github.com/godotengine/godot/issues/17011
and for some reason no one set it as a bug
Godot version:
Godot 3.0.1 Stable and Godot 3.0.2 RC1
Issue description:
when seting TextureProgress Fill mode to : Clockwise or to counter Clockwise and set the value more than 62 .. it does not render and there is an error say : Bad polygon
the value from 0 to 62 work fine , from 63 to 99 not working , and 100 work fine
in Godot 3.0 stable works fine
Steps to reproduce:
add TextureProgress node , add your progress texture to it , set the value more than 62 .. it's not rendering and if you run the project you will get an error say Bad polygon !
set it to 100 or less than 63 it will work fine
Minimal reproduction project:
TestProgressTexture.zip

ERROR: canvas_item_add_polygon: Bad Polygon!
   At: servers/visual/visual_server_canvas.cpp:675.

confirmed this error occurred.

it's regression from #16419
91215e191992f3cbbbf4fe047b000ac5a403085c master branch
a0e59a7259990a8c55ddfaec5eb5481373172a5a 3.0 branch
@nical do you have an idea what is wrong here?

I'll have a look in the next few days.

@nical Thank you :smile:

I've added this bug to the 'known bugs in 3.0.1' section of the release notes.

The bug is caused by the first check in Triangulate::snip
	if (CMP_EPSILON > (((Bx - Ax) * (Cy - Ay)) - ((By - Ay) * (Cx - Ax)))) return false;

Which tries to rule out triangles that are negative or zero-area.
If the expression on the right hand side is negative it means the winding is wrong and the triangle is outside the shape. If it is zero, it means we have a zero-area triangle (the 3 points are aligned).
The problem is that it can happen that the last three points at the end of the triangulation are indeed aligned, and in this case the test prevents the triangulation from completing. In practice, zero area triangles (while not very useful), aren't problematic when it comes to rendering correctness, so I believe we should relax this check and accept them.
The easiest way to go about it is to just relax the check for all triangles. It is a simple change but will potentially generate several zero-area triangles if there are many aligned vertices in the polygon. We can try something a little more complicated to mitigate that, for example relaxing the check only if we have exhausted all combinations. I'll submit a pull request that does the latter, but we can also chose to go for the simple fix if degenerate triangles aren't an issue.


Update to Next.js 12

Also going to update the rest of the dependencies, including Bootstrap, which I have been avoiding.


[Papercut][SW-13998] - Fix image position attribute

Description
Please describe your pull request:

Why is it necessary?
The image position attribute can now be used by plugins / extensions. Before all images got "position = 1" and not the value from the database
What does it improve?
it now sets the right position from the database
Does it have side effects?
no

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
https://issues.shopware.com/#/issues/SW-13998

How to test?
outprint $sArticel.images in template /bare/frontend/detail/images.tpl and see the right position attribute

Please do not add the position to the media struct since the media struct itself does not care about a position. The position is article related. You may have to refactor the image selection on the detail page to fix this issue.

@janbuecker the position on the detail page is correct. but the position value in the struct is always "1"
if we don't need this value, we should delete it or fill it correctly, so other can use this

yeah, I told that @taltholtmann face2face :)

removed the attribute
https://github.com/shopware/shopware/pull/761


Not working

Was working properly a day before, but now not working

What do you mean, is there an error or something?

What do you mean, is there an error or something?

No error at all, the library is displaying the same message in result despite the package being updated to a lower verison.
{
"other": {
"isFlexibleUpdateAllowed": false,
"isImmediateUpdateAllowed": false,
"packageName": "com.mediconnect.oladoc",
"status": 0,
"totalBytes": 0,
"updateAvailability": 1,
"updatePriority": 0,
"versionCode": 95
},
"reason": "status: 1 means there's no new version available",
"shouldUpdate": false
}

So are you saying it is working (it does update your app) but to a lower version or do you mean the library says there's no update even though there is according to you?

So are you saying it is working (it does update your app) but to a lower version or do you mean the library says there's no update even though there is according to you?

Ok so my apps version was 95 on play store. So i tested this library by downgrading the version to 93 yesterday. And the library was working correctly with the right response in result and everything. Today however even if i downgrade my version to 93 the library produces the above result. Which is incorrect

Got it, right. So I see that Google responded with update availability 1 which means they think (on their backend side) that there's no update for your app.
https://developer.android.com/reference/com/google/android/play/core/install/model/UpdateAvailability.html#UPDATE_AVAILABLE
This is definitely an in app updates native issue and it's not related to this library which only exposes that native functionality to react native apps.
I'm afraid I can't help you, but rest assured this has nothing to do with our lib.
Make sure you uninstall the app and clear everything before you retry testing in case there were files there from your previous tests that make Google think you don't need an update.
Also that's a good one for stack overflow.

Ok. Let me check


[lit-element] Use of aria in lit Element

Should this be an RFC?

[X] This is not a substantial change

Which package is this a feature request for?
Lit Core (lit / lit-html / lit-element / reactive-element)
Description
It would be useful if we could reflect the aria attributes of an HTML element to the shadowdom elements, and upon hydration, remove the aria attributes (but somehow keeping its reactivity) from the HTML element and keep them on the shadowdom elements.
For example, when I was creating a stepper component, I ran into an accessibility problem. If a user uses the stepper component two times (or two components that contain the  element) on a page like this:
<stepper></stepper>
<stepper></stepper>

It would render like this:
<stepper>

</stepper>
<stepper>

</stepper>

But this is wrong according to https://dequeuniversity.com/rules/axe/4.6/landmark-unique?application=axeAPI because there are no unique landmarks applied to .
The solution would be to add an aria-label to describe what the  is for, so screen readers can tell what the  is used for.
When rendering the component using the aria-label method:
<stepper aria-label="stepper 1"></stepper>
<stepper aria-label="stepper 2"></stepper>

It would be rendered like this:
<stepper aria-label="stepper 1">

</stepper>
<stepper aria-label="stepper 2">

</stepper>

The problem with this is that when someone is using a screen reader, it would read out:
"stepper 1, group"
"stepper 1, navigation"
"stepper 2, group"
"stepper 2, navigation"
This is wrong. What we want to be rendered and read out is:
<stepper>

</stepper>
<stepper>

</stepper>

So that when a user is navigating with a screen reader, things would be read out like:
"nav 1, navigation"
"nav 2, navigation"
The user should be able to control the aria-label of  by setting an aria-label attribute to <stepper>. Adding an extra prop like navAriaLabel would not be developer-friendly because the developer would have to remember an extra non-native HTML property.
Alternatives and Workarounds
it's possible to achieve this only for the first render by doing:
export class StepperElement extends LitElement {
  render() {
    return html`<nav aria-label=${this.ariaLabel}>...</nav>`;
  }

  updated(p) {
    if (this.ariaLabel) {
      this.ariaLabel = '';
    }
  }
}

const renderResult = render(html`
  <stepper-element aria-label="main navigation"></stepper-element>
`);

This works on first render, but if you update your element again, nav aria-label will be empty. Besides, this doesn't work with SSR.
I also tried (With a developer from the discord channel), to find a loophole and make it work for ssr:
@customElement('stepper-element')
export class StepperElement extends LitElement {

  @state() navAriaLabel = undefined;

  willUpdate(p) {
    if (this.navAriaLabel === undefined && this.ariaLabel) {
      this.navAriaLabel = this.ariaLabel;
      this.removeAttribute('aria-label');
    }
  }

  render() {
    return html`<nav aria-label=${this.navAriaLabel}>...</nav>`;
  }
}

const renderResult = render(html`
  <stepper-element aria-label="main navigation"></stepper-element>
`);

But if you try to render this as is, it doesn't work because aria-label is removed from stepper and loses its reactivity. Besides that, this dosn't work with SSR.

@asyncLiz and @dfreedm have done a lot of work in this area. There are standards for ARIA delegation in progress, but maybe they have a workaround to recommend for now.

@justinfagnani @asyncLiz @dfreedm I tried to make a custom property decorator with using this.dataSet. But this created some issues with SSR. I had some node hydration mismatch in Nuxt when using lit to render SSR. Apparently the approach that I did, did not work since the custom property hydrated before Vue could hydrate the component, this created some bugs. Haven't figured out a workaround. I think it's important for LIT to support a11y props, since accessibility is a big subject these days.
Any ideas?


[Bug] [dolphinscheduler-server] process instance is always running: task is failure when process instance FailureStrategy.END

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
[Bug] [dolphinscheduler-server] process instance is always running: task is failure when process instance FailureStrategy.END
What you expected to happen
[Bug] [dolphinscheduler-server] process instance can finish as normal
How to reproduce
pressure test
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!


134 bug recipesmith thread panics at runtime due to multithreading error

Description
This PR addresses a runtime panic in the RecipeSmith thread on the Horizon Server caused by an error in how the RecipeSmith subsystem is written.
Issue(s) Solved
Fixes #134
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
The following steps can be used to reproduce and verify the fix:

Clone the latest affected commit (2130768a901da09bc6619579572eb1c3ea02d1e5 as of this PR being filed).
Open the project in a VSCode DevContainer or on a Unix machine.
Execute cargo run in the root directory of the project.
Verify that the runtime panic described in #134 no longer occurs and the RecipeSmith thread runs without errors.

Test Configuration:

OS: Linux (WSL)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I have added tests that prove my changes are stable
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Ready to merge when approved


Added support and documentation on how to start the app using Lando

Added the ability to use Lando as a loca; environment.

Hello, could you please provide some documentation on Lando as I've never
herd of it?
Thanks!
On Sun, Oct 7, 2018, 5:18 PM Marcos Ibanez<EMAIL_ADDRESS>wrote:

Added the ability to use Lando as a loca; environment.
You can view, comment on, or merge this pull request online at:
https://github.com/frankcash/HackerQueue/pull/77
Commit Summary

Added support and documentation on how to start the app using Lando

File Changes

A .lando.yml
https://github.com/frankcash/HackerQueue/pull/77/files#diff-0 (38)
M README.md
https://github.com/frankcash/HackerQueue/pull/77/files#diff-1 (6)

Patch Links:

https://github.com/frankcash/HackerQueue/pull/77.patch
https://github.com/frankcash/HackerQueue/pull/77.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/frankcash/HackerQueue/pull/77, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFu9RzoWfksuCu8R-8EiOj2hAHv2z7DMks5uim-kgaJpZM4XMCv3
.

Lando its a set of tools wrapped around docker-compose, IMO its easier to use and has a bunch of pretty good recipes to setup local envs for MEAN, LAMP, LEMP, etc stacks.
More info can be found here: https://docs.devwithlando.io/

This branch is stale, please update if you wish to merge.


Update README.md

This seemed to be confusing for some people, calling with a method providing data is done by passing data={ ... }.

Looks great. Can you fill out our CLA at https://developer.vimeo.com/cla?
Once that's done, let me know and I'll merge this right away.

I've just filled out CLA and submitted the patch.

Awesome, thanks!


basemaps: mapbox tab

saving category (mapbox) for basemaps without category
changing category when adding a currently existing basemap

tests added

CR @xavijam

Things I'd appreciate the most:

Screenshot
Description with the bullets
❤

🚀


Microsoft SQL Server 2014

Why is Microsoft SQL Server 2014 not supported anymore? In the next next release(wave 2) it is supported.

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

ID: a2b0aa47-5276-2fd9-173f-f05adcd420c2
Version Independent ID: fb89206b-f5c1-6f2c-1f83-899c02aa8e89
Content: System Requirements for Business Central - Business Central
Content Source: dev-itpro/deployment/system-requirement-business-central.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: solsen

Hell, sorry for the mistake. SQL Server 2014 is not supported on Business Central 2019 Release Wave 2. The article have been fixed and an update will be published soon. Thanks for the feedback!


Sponsor Model

name string
logo string
registration_password string
active boolean
inactive_reason string
fiscal_year_start_month integer
notes text
has_many :pledges
has_many :sponsor_tasks
has_many :tasks, :through => :sponsor_tasks
has_many :sponsor_users
has_many :contacts => slightly gross SQL query
has_one :primary_assignee => gross SQL query
has_one :secondary_assignee => gross SQL query

has_many :contacts, :class_name => "User", :through => :sponsor_users, -> { where(:role => :sponsor) }
has_one :primary_assignee, :class_name => "User", :through => :sponsor_users, -> { where(:role => :primary_assignee) }
has_one :secondary_assignee, :class_name => "User", :through => :sponsor_users, -> { where(:role => :secondary_assignee) }

Let's leave this open until we get the has_manys finished


TKinter issue on latest version

On version 4.0a7 there's an from tkinter import N in validation module.
That import forces python to be configured for Tk.
8/22/2022, 1:32:06 PM	from fondat.validation import validate_arguments	*redacted*
8/22/2022, 1:32:06 PM	File "/opt/venv/lib/python3.10/site-packages/fondat/validation.py", line 5, in <module>	*redacted*
8/22/2022, 1:32:06 PM	import fondat.types	*redacted*
8/22/2022, 1:32:06 PM	File "/opt/venv/lib/python3.10/site-packages/fondat/types.py", line 11, in <module>	*redacted*
8/22/2022, 1:32:06 PM	from tkinter import N	*redacted*
8/22/2022, 1:32:06 PM	File "/usr/local/lib/python3.10/tkinter/__init__.py", line 37, in <module>	*redacted*
8/22/2022, 1:32:06 PM	import _tkinter # If this fails your Python may not be configured for Tk     *redacted*

Spurious import added by my IDE. Fixed in 4.0a9.


App Service - Multi Container - Kubernetes Pod Scaling Capabilities

We'd like to use a kubernetes based manifest for deploying a pod manifest to an app service. Does the multi-container feature support kubernetes scaling capabilities like minReplicas, maxReplicas, AverageUtilization, etc?

@erikschlegel can you provide me with the link to the documentation you are referring to? If not related to a specific doc, we do have some scaling docs
https://social.msdn.microsoft.com/Forums/en-US/655cd5c0-a261-4634-af48-5151b00e448c/hns-failed-with-error-object-already-exists?forum=AzureServiceFabric
https://docs.microsoft.com/en-us/azure/aks/cluster-autoscaler

@erikschlegel Thanks for the Comment. We are actively investigating and will get back to you soon.

@erikschlegel I will close this for now. If you still have questions feel free to ping me offline and I can give you some additional resources you can use for questions like these.


Environment variables not updated when creating a new tab.

Windows Terminal version
1.11.2921.0
Windows build number
10.0.19042.1348
Other Software
No response
Steps to reproduce

Open a Command Prompt tab (it also occurs with a Powershell tab).
Type "SET" to display environment variables.
Create or edit any Environment Variable
Open a new Command Prompt tab.
Type "SET" to display environment variables.

Expected Behavior
The environment variable value displayed in the tab opened AFTER creating/editing the variable is the current one.
Actual Behavior
The environment variable value displayed in the tab opened AFTER creating/editing the variable is NOT the current one.

SET in cmd.exe only applies to the current session, always. What you're doing is basically the same as:

Open conhost running cmd
use set to set some variable
open a new conhost running cmd via the start menu or run dialog

In this example, you'd expect the variables set in 1 wouldn't apply to the cmd spawned in 3. That's basically what's happening here in the Terminal.
SETX is the command you can use in cmd to update the environment not just for the current process, but the block that's used for future launches as well. However, that's sorta gonna run into a similar issue, because the Terminal doesn't (yet) reload environment variables while it's running. There's more discussion about this in #1125, an issue we're still sorting out the right way of handling. There's also #2785 tracking adding custom environment blocks to the profile. (that's related, but maybe not relevant). Does that make more sense now?

However, that's sorta gonna run into a similar issue, because the Terminal doesn't (yet) reload environment variables while it's running.

Terminal already calls CreateEnvironmentBlock() for new tabs. But problems have surfaced due to the order in which this API function loads the environment, which is inconsistent with the private RegenerateUserEnvironment() function that the shell uses. I think the two should be made consistent by adding CreateEnvironementBlock2() to the public API.

Any mention of setx.exe has to be accompanied by a disclaimer that it should never be used on its own to modify the system or per-user "PATH". It can be combined with reg.exe query to get the original system or user value of "PATH", but it's difficult to do if you're not experienced with batch scripting.
Using PowerShell is easier, but if "PATH" is a REG_EXPAND_SZ value that uses percent variables, use the registry provider to read the original value without expansion instead of using [System.Environment]::GetEnvironmentVariable(name, target). The latter should be avoided because it expands the percent variables based on the current environment.

Sorry, @zadjii-msft, I probably didn't explain the issue well enough: I'm using the SET command just to dump the environment variables in the terminal, not to modify actual variables.
The environment variables are modified in Windows's "Environment Variables" dialog. And once these variables are modified, I open a new command prompt tab.
I expect the new command prompt tab to inherit its environment variables from the current state, but changes that occurred since the terminal was started are ignored.

Anyway, you mentioned issue #1125, it looks exactly like my scenario. Good to know you have this under your radar

Calling CreateEnvironmentBlock() for new tabs has been implemented for a while now, but it's only available in Windows Terminal Preview.

Oh okay yea, that's my bad.  That's definitely #1125 (and the myriad of pain that's followed in the wake of #7243). Or at least, that's the thread we've generally been using to track that. Thanks!
/dup #1125


chore: bdd-gh-1583880107 to 0.0.1

chore: Promote bdd-gh-1583880107 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Closed tree on 'C:\Users\flutter\.cocoon\flutter' (OS Error: Access is denied."

Full log here. The specific failure is
{
  "textPayload": "ERROR: FileSystemException: Deletion failed, path = 'C:\\Users\\flutter\\.cocoon\\flutter' (OS Error: Access is denied.\r",
  "insertId": "nbluntg1p7sjhp",
  "resource": {
    "type": "global",
    "labels": {
      "project_id": "flutter-dashboard"
    }
  },
  "timestamp": "2021-03-08T17:39:25.972045728Z",
  "logName": "projects/flutter-dashboard/logs/ahFmbHV0dGVyLWRhc2hib2FyZHJfCxIJQ2hlY2tsaXN0Ij9mbHV0dGVyL2ZsdXR0ZXIvbWFzdGVyL2FjZGEyNzJjZjI2MjM3OWNiYTIxNWFmNzUwNDM0ZDAyMTIzMWMwZTkMCxIEVGFzaxiAgNi1yb75Cww_2",
  "receiveTimestamp": "2021-03-08T17:39:25.972045728Z"
}

I'm guessing this is an infra issue?
/cc @keyonghan

Manually recovered win2, and rerun failed tasks.


Enable heat in qa_devstack

Is this PR sill active or needed to be kept open? It needs a rebase as well.
Could it live as a branch in your fork if a merge is not needed now?


[OOM Error] Out of Memory with 32k tokens

Thank you for your valuable contribution! I have been experimenting with your evaluation codes on the LongChat-Lines dataset. However, I encountered an Out of Memory Error when the token length reached 32k.
I am fortunate to have multiple 80G A100 GPUs at my disposal. However, I noticed that your evaluation code does not incorporate parallel processing, and only one GPU is utilized during evaluation.
I would greatly appreciate it if you could provide more information about the resources used in the experimental section of your paper. Additionally, I am curious if you implemented any form of parallelization to enhance the evaluation process.
Thank you once again for your assistance!

hi, for this line:
inputs = tokenizer.encode(prompt, return_tensors="pt").to(f"cuda:{local_rank}")

hi, for this line: inputs = tokenizer.encode(prompt, return_tensors="pt").to(f"cuda:{local_rank}") does it mean for every gpu(local_rank), it will encode the same inputs? also which model-name-or-path do you use? Thanks.

There is a blog about DeepSpeed Inference, which may help you to understand clearly how DeepSpeed accelerates inference.
For the first problem, DeepSpeed uses tensor parallelism to shard the model and generate results through communication. This means that each GPU will not encode the same inputs. For more details, you can refer to the issue: https://github.com/microsoft/DeepSpeed/issues/4154.
For the second problem, model-name-or-path refers to the local path where llama2-7b  model is stored. You can download it from HuggingFace.

hi, @JL-Cheng , how did you get 32K of data? As far as I know, in https://huggingface.co/datasets/abacusai/LongChat-Lines/viewer/default/100, the maximum data length is 26K.


Redisson Client hangs on creation

Hey guys,
currently trying redis + redisson (v3.5.5), so this might be an easy question for some of you. Anyways as soon as I create my Redisson Client (config seems right), the client hangs in the create statement indefinitely without any exception or timeout, etc (waited for over an hour to hopefully see some error, sadly none).
To the setup: Win10 + Ubuntu subsystem running redis on standard port (6379), single server setup. In its config the redis server is binding to <IP_ADDRESS> (also tried disabling protected mode). RedisDesktop Manager and Jedis connect just fine without any issues.
Code (Kotlin):
val config = Config()
    config.useSingleServer()
            .setAddress("redis://localhost:6379") // redis://<IP_ADDRESS> does not work either
            .setDatabase(0)
            .setSslEnableEndpointIdentification(false)
            .setConnectTimeout(10)

    val redisson = Redisson.create(config)

Hope someone can help, as I couldn't find anything related to this anywhere online.
Stacktrace :
"main" #1 prio=5 os_prio=0 tid=0x0000000004f46000 nid=0x37450 waiting on condition [0x0000000004f3e000]
   java.lang.Thread.State: WAITING (parking)
        at sun.misc.Unsafe.park(Native Method)
        - parking to wait for  <0x000000076b9ae160> (a java.util.concurrent.CompletableFuture$Signaller)
        at java.util.concurrent.locks.LockSupport.park(LockSupport.java:175)
        at java.util.concurrent.CompletableFuture$Signaller.block(CompletableFuture.java:1693)
        at java.util.concurrent.ForkJoinPool.managedBlock(ForkJoinPool.java:3323)
        at java.util.concurrent.CompletableFuture.waitingGet(CompletableFuture.java:1729)
        at java.util.concurrent.CompletableFuture.join(CompletableFuture.java:1934)
        at org.redisson.misc.RedissonPromise.syncUninterruptibly(RedissonPromise.java:168)
        at org.redisson.misc.RedissonPromise.syncUninterruptibly(RedissonPromise.java:39)
        at org.redisson.connection.MasterSlaveConnectionManager.initSingleEntry(MasterSlaveConnectionManager.java:277)
        at org.redisson.connection.MasterSlaveConnectionManager.<init>(MasterSlaveConnectionManager.java:165)
        at org.redisson.connection.SingleConnectionManager.<init>(SingleConnectionManager.java:32)
        at org.redisson.config.ConfigSupport.createConnectionManager(ConfigSupport.java:231)
        at org.redisson.Redisson.<init>(Redisson.java:117)
        at org.redisson.Redisson.create(Redisson.java:156)

Can you show us the full thread dump please?

There you go:
redissonHang.dump.txt

It looks very much like so. The workaround -Dorg.jboss.netty.epollBugWorkaround=true is only a flag to indicate a different logic is required to work around this issue, and that has to be implementation specific, in this case it is only useful for jboss. Maybe we too need to do something similar. @mrniko can you advice?

Hmm I have never met such problem on Windows 7

Yeah I was not aware that jboss is an own project and the netty package actually "io.netty" is.
Which makes it obvious that that workaround wouldnt fix anything. Anyway they closed that netty ticket as it was for version 3.9 & at that point they had 4.0 on master branch.
As for Redisson 3.5.5 seems to be using 4.0.23 of netty, which makes it seem like its not fixed.

@Flakesz I'm sorry but Redisson 3.5.5 uses 4.1.16 version of netty

Strange I see all netty imports in RedisClient referencing to netty 4.0.23. Meanwhile in the pom for redisson it says for all netty components 4.1.16 as ex:
<dependency>
            <groupId>io.netty</groupId>
            <artifactId>netty-transport-native-epoll</artifactId>
            <version>4.1.16.Final</version>
            <scope>provided</scope>
</dependency>

Anyways it should be fixed for version 4+. So I guess I wont be able to use Redisson under Win 10 anytime soon.

I think you may have a version conflict with netty

Yeah I believed the same and created a clean project with only Redisson as a dependency and it worked. So for the bigger project, I disabled all dependencies and reenabled one by one to find the cause.
Turns out its was:
 compile group: 'org.apache.hbase', name: 'hbase-client', version: '1.3.1'
Was expecting sth like jetty to be the cause but in the end it was the hbase-client.

So I guess I wont be able to use Redisson under Win 10 anytime soon.
Overriding netty-all to 4.1.16 seems to work for both

Do not give up hope :)


Update org.everit.json.schema to 1.14.0

Updates com.github.everit-org.json-schema:org.everit.json.schema from 1.13.0 to 1.14.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.everit-org.json-schema", artifactId = "org.everit.json.schema" } ]

labels: library-update, semver-minor

Codecov Report

Merging #139 (9cd88fb) into master (bbbb5b1) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #139   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           35        35
  Branches         2         4    +2
=========================================
  Hits            35        35

Continue to review full report at Codecov.

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


How to enable client side pagination

Is there any way to enable client side pagination? In one my grid, I would normally have small data and for that I would like to have pagination on the local data. How it would be possible?

you would need to implement your own datasource to do this. see the example on pagination, this is in fact client side pagination, just with a timer to make it appear like server side.