Falsely detects "online mode" when monitoring the connection between bungeecord and pandaspigot 1.8

Hi, I'm trying to determine why pandaspigot 1.8 is disconnecting bungeecord, or why bungeecord is disconnecting from pandaspigot 1.8, when I try to use a plugin I made for bungeecord that replaces most of the client-to-proxy network stack.
I'm trying to use this tool to see if I can catch what part of the proxy-to-server network stack's handshake is causing it to disconnect because I don't want to drive myself insane looking at wireshark hex dumps, but I get the "this server is online mode, please copy this code into your browser" message when I try to intercept the connection between bungeecord and pandaspigot.
Both bungeecord and pandaspigot have online-mode set to false.
The bug also happens when I try to connect to bungeecord using a vanilla client instead of my custom stack.
To reproduce, simply set up bungeecord and pandaspigot 1.8, monitor the connection between bungeecord and pandaspigot, and then connect to bungeecord with vanilla minecraft 1.8.

oops, spigot got disconnected because I forgot newer versions have a bungeecord config.yml boolean, didn't fix the crash though

I had issues with pakkit detecting a 1.18.2 server as online mode when the server was offline mode. So this issue might not be specific to bungeecord and stuff

Same for 1.17 purpur, cant join offline server

Sort of fixed in https://github.com/Heath123/pakkit/commit/9698706b17252215d74ea3446104ee4839d2d888, but not really because I just turned off online mode properly. I need to fix the auto detection or add a toggle

well hello lax, didnt expect to see you here.


envoy: b1219ef0d

brings in: https://github.com/envoyproxy/envoy/pull/19010
Signed-off-by: Jose Nino<EMAIL_ADDRESS>

Likely needs #1968


How does the redirect for this backend work.

 I'm having trouble figuring out how to 'force' a user login with this app. How would one, for example, create a 'login' button that redirects to Keycloak?
I've tried to do:
if not request.user.is_authenticated:
    return redirect('%s?next=%s' % (settings.LOGIN_URL, request.path))

In a view, but only redirects to:
http://localhost:8000/path/keycloak_login?next=/path/
I tried to put in the whole url of my keycloak server as value for LOGIN_URL, in settings.py, but then I'm getting an error message saying 'Missing parameters: client_id'
Any advice would be highly appreciated.
Cheers!

Hi @makkus,
I might have forgotten something in the documentation. You also have to configure the Keycloak URL's by adding something like this to your urls.py:
urlpatterns = [
    ... some other url configs ...
    url(r'^account/', include('django_keycloak.urls')),
]

Please don't forget to configure your Keycloak server and REALM using the Django Admin before trying to login.

I've updated the docs.

Thanks, much appreciated!

I just ran into the same issue. Unfortunately, the documentation seems to be outdated again. Instead of  using keycloak_login as LOGIN_URL one has to use keycloak/login.
I hope this helps anyone else trying to get the library up and running. Apart from that minor "bug" very nice job 😄


Github_user_ finder

Related Issue
#583
Closes: #583
Describe the changes you've made
An app to find  github user and we can keep track of name,company,bio,pubic repos,etc
There will be a button directing to the github profile of user
This app fetches some of the recent repos and on clicking on those repos ,the user will be directed to that repo on github
Describe if there is any unusual behaviour of your code
NA
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] New and existing unit tests pass locally with my changes.

Screenshots

@urvashi-code1255 Please review it!

Please check it now!


Update minor typo in Docs

Cals to InvokeEndpoint -> Calls to InvokeEndpoint

Codecov Report

Merging #1875 into develop will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1875      +/-   ##
===========================================
+ Coverage    92.56%   92.59%   +0.03%
===========================================
  Files           53       53
  Lines        10001    10001
===========================================
+ Hits          9257     9260       +3
+ Misses         744      741       -3

Impacted Files
Coverage Δ

botocore/credentials.py
98.86% <0%> (+0.34%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for the contribution, @Jarryd-rk! This request was passed on to the service team who owns these docs and they've been able to address it in more recent versions of the SDK. Going to close this now that the issue is resolved.


Redesign named variables in FuzzIL

This change replaces the three operations LoadNamedVariable, StoreNamedVariable, and DefineNamedVariable with a single new operation: CreateNamedVariable. This operation now simply creates a new FuzzIL variable that will be assigned a specific identifier during lifting. Optionally, the named variable can be declared using any of the available variable declaration modes: global, var, let, or const.
Below is a small example of how CreateNamedVariable can be used:
// Make an existing named variable (e.g. a builtin) available
v0 <- CreateNamedVariable 'foo', declarationMode: .none
v1 <- LoadString 'bar'
Reassign v0, v1

// Declare a new named variable
v2 <- LoadInt 1337
v3 <- CreateNamedVariable 'baz', declarationMode: .var, v2
Update v3, .PostInc
v4 <- CreateNamedVariable 'print', declarationMode: .none
v5 <- CallFunction v4, v3

This will now lift to the following JavaScript code:
foo = "bar";
var baz = 1337;
baz++;
print(baz);

With this, we now have a single, flexible way of creating variables that have a specific name. We now use this:

For builtins, which are effectively just existing named variables in the global scope. This also now makes it (easily) possible to overwrite builtins. As such, The LoadBuiltin operation was removed.
During code generation, to sometimes create variables with specific names in generated code (we use random property names).
In the compiler. This is the main user of named variables and this is where this change has the most impact: we now compiler every variable declaration to a CreateNamedVariable operation. This now makes it possible to correctly compiler any code that relies on variable names, for example due to using eval, with statements, or similar constructs. See some of the added/modified tests for examples. The consequence of this is that compiled code will now often have a lot of CreateNamedVariable operations. However, as these are now just regular FuzzIL variables, this change should not significantly affect mutability of the programs. In the future, we could consider implementing a specific minimizer (that we could also run during corpus import) to remove unneeded CreateNamedVariable operations. However, it will likely be somewhat difficult to determine when such an operation is not needed.

Had some time on the train today so I gave the idea from https://github.com/googleprojectzero/fuzzilli/pull/474#discussion_r1896696197 a try. @carl-smith  and @TobiasWienand, could you both take a look at this and see what you think? This should make named variables much more flexible and powerful and I think it should now make it easy to handle issues such as the one we're encountering in PR #474 now. But it's also a somewhat deep change and for example also removes LoadBuiltin.

Overall it looks really awesome :)
I noticed that
before
for (const a of ["a"]) {}
for (let b of ["b"]) {}
for (var c of ["c"]) {}

after
for (const v2 of ["a"]) {}
for (const v5 of ["b"]) {}
for (const v8 of ["c"]) {}

Apart from that, my small testbench regarding scoping of variables passed. I am currently working on a more comprehensive test bench for semantic correctness based on differential testing. I don't know if that already exists or has been done before but I think we will gain more confidence in any case. Will keep you updated


Add Histogram combiner

Design document: here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

assign set of reviewers

What is the status of this PR. I see @tilgalas reviewed it. If the change looks good to you, reviewer could approve it and I can help merge

What is the status of this PR. I see @tilgalas reviewed it. If the change looks good to you, reviewer could approve it and I can help merge

* What went wrong:
Execution failed for task ':sdks:java:extensions:combiners:analyzeClassesDependencies'.
> Dependency analysis found issues.
  usedUndeclaredArtifacts
   - org.apache.commons:commons-lang3:3.14.0@jar

need to add   implementation library.java.commons_lang3 in combiners/build.gradle
after that you can also run ./gradlew :sdks:java:extensions:combiners:build locally to check if build now pass

different task failed in different trials of Java PreCommit, not related to the PR, merging for now
PS: in the future when address comment please avoid rebase-and/or-force-push which erases the linear history of the change. Just push another commit and reviewer can just check the latest changes and not need to get back to the whole code


{Synapse} Fix test fail in yum package

Description

test_notebook fails in Test Yum Package
It might caused because the operation system ADO used can't deal with ipynb file. This PR changed ipynb file in test to json file.
Testing Guide

azdev test test_notebook

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

is *.ipynb required to test?

It doesn't quite make sense to me about this file extension renaming fix.

this is just file renaming, while file content doesn't change. It's strange your pipeline works with .json but not with .ipynb
we would like to name this file as .ipynb, which is a well known file format in Notebook world.

I think we should dig further and figure out the exact reason why the test case fails

@fengzhou-msft Could you help take a look why it only fails in yum package?

Closed since It fails infrequently.

Resolved in https://github.com/Azure/azure-cli/pull/17704


[extractors] get_title() filters Cyrillic and other but CJK Unified Ideographs

 Issue by KMiNT21
Mon Jun  4 13:57:32 2018
Originally opened as https://github.com/codelucas/newspaper/issues/574

class ContentExtractor(object):
...
def get_title(self, doc):
...
filter_regex = re.compile(r'[^\u4e00-\u9fa5a-zA-Z0-9\ ]')

^^^
This filter work only for Chinese, Japanese and Korean and drops lots of other languages.
If anyone needs to fix this - https://apps.timwhitlock.info/js/regex

added in v0.9.3


Change app name to HuatAh!

Pull Request Test Coverage Report for Build 404

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.887%

Totals

Change from base Build 399:
0.0%

Covered Lines:
2223

Relevant Lines:
2650

💛  - Coveralls


User manual: minor fixes

Erased wrong semicola.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #909   +/-   ##
=========================================
  Coverage     92.86%   92.86%
  Complexity      393      393
=========================================
  Files             2        2
  Lines          5834     5834
  Branches       1527     1527
=========================================
  Hits           5418     5418
  Misses          203      203
  Partials        213      213

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 fa232b3...765bd1d. Read the comment docs.

Merged into master.
Thank you as always!!


How to remove console message for mobX.

Hi,
I am getting this on my console.

Type mobxAngularDebug(true) or mobxAngularDebug(false) to trigger MobX debugging

Please tell me how to remove/disable the same.
Thanks.

Closing this issue due to inactivity. Please open a new issue if the problem still exists.


feat: add codeql workflow

Is your feature request related to a problem? Please describe.
The repository contains code in javascript, it does not have workflow for code scanning.
Describe the solution you'd like
I want to add the codeql workflow to automate security checks. CodeQL is the code analysis engine developed by GitHub to identify vulnerabilities in code. It will analyze your code and display the results as code scanning alerts. It will be enabled on every push, commit and pull request using GitHub actions.
Record

[x] I agree to follow this project's Code of Conduct
[x] I'm a SSOC 2.0 contributor
[x] I want to work on this issue

Please assign me this issue under SSOC 2.0
Thanks

I agree to follow this project's Code of Conduct
I'm a SSOC 2.0 contributor
I want to work on this issue
Please assign me this issue under SSOC 2.0
Thanks

@TusharKesarwani please review this PR.
Thanks


PTFE-659: migrate registry bert-e

This PR migrates the download of bert-e from registry.scality.com to ghcr.io

Hello abubakarr99,
My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.
Status report is not available.

Waiting for approval
The following approvals are needed before I can proceed with the merge:

the author

one peer

The following reviewers are expecting changes from the author, or must review again:

@tcarmet

Integration data created
I have created the integration data for the additional destination branches.

this pull request will merge improvement/PTFE-659-migrate-registry-bert-e into
development/3.8
w/3.9/improvement/PTFE-659-migrate-registry-bert-e will be merged into development/3.9

The following branches will NOT be impacted:

development/1.0
development/2.0
development/3.0
development/3.1
development/3.2
development/3.3
development/3.4
development/3.5
development/3.6
development/3.7

You can set option create_pull_requests if you need me to create
integration pull requests in addition to integration branches, with:
@bert-e create_pull_requests

Waiting for approval
The following approvals are needed before I can proceed with the merge:

the author

one peer

The following reviewers are expecting changes from the author, or must review again:

@tcarmet

History mismatch
Merge commit #f6c8e8b2695d9df7a27533d8f7753e70a8205647 on the integration branch
w/3.9/improvement/PTFE-659-migrate-registry-bert-e is merging a branch which is neither the current
branch improvement/PTFE-659-migrate-registry-bert-e nor the development branch
development/3.9.
It is likely due to a rebase of the branch improvement/PTFE-659-migrate-registry-bert-e and the
merge is not possible until all related w/* branches are deleted or updated.
Please use the reset command to have me reinitialize these branches.

reset

@tcarmet  check my comment on the ticket

History mismatch
Merge commit #f6c8e8b2695d9df7a27533d8f7753e70a8205647 on the integration branch
w/3.9/improvement/PTFE-659-migrate-registry-bert-e is merging a branch which is neither the current
branch improvement/PTFE-659-migrate-registry-bert-e nor the development branch
development/3.9.
It is likely due to a rebase of the branch improvement/PTFE-659-migrate-registry-bert-e and the
merge is not possible until all related w/* branches are deleted or updated.
Please use the reset command to have me reinitialize these branches.
The following options are set: approve

@bert-e help

Help page
The following options and commands are available at this time.
Options

name
description
privileged
authored

:arrow_right: after_pull_request
Wait for the given pull request id to be merged before continuing with the current one.

:arrow_right: bypass_author_approval
Bypass the pull request author's approval
:star:

:arrow_right: bypass_build_status
Bypass the build and test status
:star:

:arrow_right: bypass_commit_size
Bypass the check on the size of the changeset TBA
:star:

:arrow_right: bypass_incompatible_branch
Bypass the check on the source branch prefix
:star:

:arrow_right: bypass_jira_check
Bypass the Jira issue check
:star:

:arrow_right: bypass_peer_approval
Bypass the pull request peers' approval
:star:

:arrow_right: bypass_leader_approval
Bypass the pull request leaders' approval
:star:

:arrow_right: approve
Instruct Bert-E that the author has approved the pull request.

:writing_hand:

:arrow_right: create_pull_requests
Allow the creation of integration pull requests.

:arrow_right: no_octopus
Prevent Wall-E from doing any octopus merge and use multiple consecutive merge instead

:arrow_right: unanimity
Change review acceptance criteria from one reviewer at least to all reviewers

:arrow_right: wait
Instruct Bert-E not to run until further notice.

Commands

name
description
privileged

:arrow_right: help
Print Bert-E's manual in the pull request.

:arrow_right: status
Print Bert-E's current status in the pull request TBA

:arrow_right: clear
Remove all comments from Bert-E from the history TBA

:arrow_right: retry
Re-start a fresh build TBA

:arrow_right: build
Re-start a fresh build TBA

:arrow_right: force_reset
Delete integration branches & pull requests, and restart merge process from the beginning.

:arrow_right: reset
Try to remove integration branches unless there are commits on them which do not appear on the source branch.

The following options are set: approve

History mismatch
Merge commit #f6c8e8b2695d9df7a27533d8f7753e70a8205647 on the integration branch
w/3.9/improvement/PTFE-659-migrate-registry-bert-e is merging a branch which is neither the current
branch improvement/PTFE-659-migrate-registry-bert-e nor the development branch
development/3.9.
It is likely due to a rebase of the branch improvement/PTFE-659-migrate-registry-bert-e and the
merge is not possible until all related w/* branches are deleted or updated.
Please use the reset command to have me reinitialize these branches.
The following options are set: approve

@bert-e reset

Reset complete
I have successfully deleted this pull request's integration branches.
The following options are set: approve

Integration data created
I have created the integration data for the additional destination branches.

this pull request will merge improvement/PTFE-659-migrate-registry-bert-e into
development/3.8
w/3.9/improvement/PTFE-659-migrate-registry-bert-e will be merged into development/3.9

The following branches will NOT be impacted:

development/1.0
development/2.0
development/3.0
development/3.1
development/3.2
development/3.3
development/3.4
development/3.5
development/3.6
development/3.7

You can set option create_pull_requests if you need me to create
integration pull requests in addition to integration branches, with:
@bert-e create_pull_requests

The following options are set: approve

/approve

In the queue
The changeset has received all authorizations and has been added to the
relevant queue(s). The queue(s) will be merged in the target development
branch(es) as soon as builds have passed.
The changeset will be merged in:

:heavy_check_mark: development/3.8

:heavy_check_mark: development/3.9

The following branches will NOT be impacted:

development/1.0
development/2.0
development/3.0
development/3.1
development/3.2
development/3.3
development/3.4
development/3.5
development/3.6
development/3.7

There is no action required on your side. You will be notified here once
the changeset has been merged. In the unlikely event that the changeset
fails permanently on the queue, a member of the admin team will
contact you to help resolve the matter.
IMPORTANT
Please do not attempt to modify this pull request.

Any commit you add on the source branch will trigger a new cycle after the
current queue is merged.
Any commit you add on one of the integration branches will be lost.

If you need this pull request to be removed from the queue, please contact a
member of the admin team now.
The following options are set: approve

I have successfully merged the changeset of this pull request
into targetted development branches:

:heavy_check_mark: development/3.8

:heavy_check_mark: development/3.9

The following branches have NOT changed:

development/1.0
development/2.0
development/3.0
development/3.1
development/3.2
development/3.3
development/3.4
development/3.5
development/3.6
development/3.7

Please check the status of the associated issue PTFE-659.
Goodbye abubakarr99.


⚠ Processors has degraded performance

In 7ef2643, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 63 ms

Resolved: Processors performance has improved in 251eb32.


Update operation-timed-out-error-disk-management-console.md

Correcting sfc /scannnow  command to sfc /scannow , By removing an extra n in scannnow

@mhdomerah : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Update navicat-for-mysql to version 12.0.7

Updated navicat-for-mysql to version 12.0.7 and replaced sha256 checksum. Modified appcast url and replaced checkpoint.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 62a263273f8e587889baf1344fe5c081c2d5d641. Your contribution is still credited to you.


feat(sbb-screenreader-only): initial implementation

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

:exclamation: No coverage uploaded for pull request base (main@f94d612). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2377   +/-   ##
=======================================
  Coverage        ?   93.63%
=======================================
  Files           ?      222
  Lines           ?    23289
  Branches        ?     2052
=======================================
  Hits            ?    21806
  Misses          ?     1445
  Partials        ?       38

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


Implement 1-2

Implement the portion of the script dealing with finding, rescuing, healing, and recruiting Elaine. Consists of several scenes and one battle. Details in the script.
This will require her to be a new sprite with her own art, animations, attributes, skills, and other data. I'll also need a sprite for the plant used to heal her, and for her bow.

Renamed to reflect abelon-2 story


Modified stl.py to correctly handle multi-part stl files. The ascii…

…reader read beyond the 'endsolid' so the header of the next part was wrong! Fix proposed in lines 133-135

Smart fix, thanks :)
I'm running the tests now but everything looks good, should have a new release pushed deployed within a week.


good afternoon !!! i keep gettin this error

This keeps popping up and cant fix it. Any suggestions ? Thank you in advance!! Cant wait to try this super cool tool out.
8380 INFO: Processing pre-find module path hook   distutils
Traceback (most recent call last):
File "/usr/bin/pyinstaller", line 11, in
load_entry_point('PyInstaller==3.5+498e6ee058', 'console_scripts', 'pyinstaller')()
File "/usr/lib/python3/dist-packages/PyInstaller/main.py", line 111, in run
run_build(pyi_config, spec_file, **vars(args))
File "/usr/lib/python3/dist-packages/PyInstaller/main.py", line 63, in run_build
PyInstaller.building.build_main.main(pyi_config, spec_file, **kwargs)
File "/usr/lib/python3/dist-packages/PyInstaller/building/build_main.py", line 844, in main
build(specfile, kw.get('distpath'), kw.get('workpath'), kw.get('clean_build'))
File "/usr/lib/python3/dist-packages/PyInstaller/building/build_main.py", line 791, in build
exec(code, spec_namespace)
File "/home/kali/OSRipper/update_or.spec", line 6, in
a = Analysis(['update_or.py'],
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/PyInstaller/building/build_main.py", line 243, in init
self.postinit()
File "/usr/lib/python3/dist-packages/PyInstaller/building/datastruct.py", line 158, in postinit
self.assemble()
File "/usr/lib/python3/dist-packages/PyInstaller/building/build_main.py", line 348, in assemble
self.graph = initialize_modgraph(
^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/PyInstaller/depend/analysis.py", line 613, in initialize_modgraph
graph.import_hook(m)
File "/usr/lib/python3/dist-packages/PyInstaller/lib/modulegraph/modulegraph.py", line 1473, in import_hook
target_package, target_module_partname = self._find_head_package(
^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/PyInstaller/lib/modulegraph/modulegraph.py", line 1657, in _find_head_package
raise ImportError("No module named " + target_package_name)
ImportError: No module named _bootlocale
Generated in dist

Good afternoon,
I have had a look and it seems to be a pyinstaller issue as mentioned here (not an issue with osripper)
https://stackoverflow.com/questions/68459087/pyinstaller-with-python-3-10-0b4-importerror-no-module-named-bootlocale
There is a fix for it so you can change the affected line in the code and add the argument described in the post.
If you aren't able to do that lmk and I'll make a release for you and others affected by this error


Allow third parties to bulk-add assets to the library

It would be great to mass import assets like Materials and finish in a BlenderKit style search inside godot editor. I mean: be able to let the free asset libraries to add their own free content to the godot library in bulk as a way to promote their services. Some kind of API or something like that...

The issue is that PBR materials need to be imported with Godot .tres files to be really usable out of the box. Otherwise, people have to set up the materials by hand, so you don't gain much time compared to downloading the textures manually and placing them in your project.

I was thinking about doing an automated import of materials due to this
problem (based on file names) . I think there is no standard related to
materials and someone should create it...
El jue, 10 de jun. de 2021 a la(s) 19:05, Hugo Locurcio (
@.***) escribió:

The issue is that PBR materials need to be imported with Godot .tres
files to be really usable out of the box. Otherwise, people have to set up
the materials by hand, so you don't gain much time compared to downloading
the textures manually and placing them in your project.
If these third parties don't do this work, there's no point in supporting
mass importing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Calinou/godot-asset-library-laravel/issues/303#issuecomment-858793471,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADY35JSR5VSUW4CAGHRP5TTSDWFZANCNFSM46O5COGQ
.

I was thinking about doing an automated import of materials due to this
problem (based on file names) . I think there is no standard related to
materials and someone should create it...

Creating a standard is easier said than done :slightly_smiling_face:
Loading textures into a StandardMaterial3D/ORMMaterial3D based on naming conventions is possible, but it's difficult to get it right for all textures available online. See also https://github.com/godotengine/godot-proposals/issues/2316.

Do you know if there are some kind of LOD but for materials?


Bluetooth: Controller: Add common ULL ticker stop with mark function

The same code block for marking an object with
ull_disable_mark, stopping the ticker and then unmarking
the object appeared multiple places; often with a regression
bug where instread of unmarking it in case of error, it would
be "remarked".
This commit fixes the bug, and moves the functionality to
a common function.
Signed-off-by: Emil Gydesen<EMAIL_ADDRESS>

CI does not catch misspelled "instread"  in commit body.... https://github.com/zephyrproject-rtos/zephyr/pull/29973/commits/d8e088db48c7a594f4478ba20bff9d4c8accf6be


Production ready?

I remember you mentioning this some time ago but I was unsure if this is ready for use now.

Hey rafde most of this work has been moved over to the marionettejs v3 branch I believe

Yo @rafde! :wave:
I'm using this in one app in production, and it's been going well. I strongly recommend that one read through the source / understand exactly what the code is doing before deciding to use it. It's only a few lines long luckily!
Another thing to keep in mind is that it's not compatible with Marionette v2's API. This lib returns the view from showChildView, while Marionette returns the Region. That's one little detail that might hang someone up if they're chaining, like:
this.showChildView( ... ).someRegionMethod();

Thanks for the response. I read through the file and it's pretty straight forward.
If you recall my work's code base, chaining is a rarity. And it's even rarer to chain regions.
Since there's a MarionetteV3 branch, I might check that out for reference.
Thanks again.

Here's the branch if you go looking for it
https://github.com/marionettejs/backbone.marionette/tree/next
On Wed, Oct 28, 2015 at 11:28 AM, Rafael De Leon<EMAIL_ADDRESS>wrote:

Closed #11 https://github.com/jmeas/marionette.base-view/issues/11.
—
Reply to this email directly or view it on GitHub
https://github.com/jmeas/marionette.base-view/issues/11#event-448133438.

Thanks @megawac , I looked through the change diff https://github.com/marionettejs/backbone.marionette/compare/2.4.2...next#diff-b8da5292cb07ec757ac64e78ff5af5c1 and didn't see the next code reflect what this code base is doing. It looks like you still have to use the regions hash or #addRegion function.


How to select and export database in windows 10？

OrientDB Version: <3.0>
Java Version: <1.8.0_211-b12>
OS: <Windows 10>
Expected behavior
Export the database in localhost by console
Actual behavior
Error occurred during export
Steps to reproduce
<Here are the command lines in cmd:

cd c:\orientdb-3.0.20\bin
console.bat--- OK
connect remote:localhost:mydb root password--- OK
export database mydb.zip
<Error: com.orientechnologies.common.exception.OSsystemexception: Database not selected. Use 'connect  ' to connect to a database.>

Any help would be appreciated!

Solved:
Orientdb would not report an error in Step 3 even for a nonexistent database. To correctly export database,  Step 3 should be corrected into:

change "localhost" into the actual domain name (e.g. <IP_ADDRESS>);
add "/"  before the database name;
Thus we get: connect remote: <IP_ADDRESS> :/ mydb root password

Hi @TianChangWu
connect remote:localhost:/mydb root password  you should have a slash before database name. I have successfully connect server once I have slash which have a short echo on terminal like orientdb {db=import_test}>  rather than orientdb {server=remote:localhost:import_test}>. It seem that orientdb console haven't a friendly echo. I think change hostname is not necessary.
by a way, I don't think exported format is zipfile , export database mydb.zip imply zipfile

@x007007007  Exactly!

Hi,
This seems solved, closing it.


PDFJS.webViewerLoad is not a function

Am getting PDFJS.webViewerLoad is not a function.
Am using Angularjs 1.6 and psf.js pdfjs-1.6.210.
Thx,

@dnamna i facing the same issue, how you solve this ?

@jasny you have reopened this, but not provided the solutions. Can you please let us know what we are doing wrong. Thanks

I've reopened it, simply because you are experiencing this issue. Please reproduce this issue in a jsfiddle, so we can look at what is causing it.

Hi,
Sorry guys for the late reply.
The issue was simply the version of the pdfjs.
Just add the latest package of pdfjs in your project, the one provided in the example.

Got the same issue:
window.PDFJS.webViewerLoad is not a function. Full log:
{compatibilityChecked: true, version: "1.9.426", build: "2558a58d", pdfBug: false, …}       angular.min.js:123
TypeError: window.PDFJS.webViewerLoad is not a function
at Object.link (angular-pdfjs-viewer.js:397)
at angular.min.js:17
at angular.min.js:84
at ra (angular.min.js:85)
at n (angular.min.js:70)
at g (angular.min.js:61)
at g (angular.min.js:62)
at g (angular.min.js:62)
at n (angular.min.js:70)
at g (angular.min.js:61) ""

I'm having the same issue. I've updated PDFJS to the latest version, did anybody find a way to make it work? window.PDFJS.webViewerLoad does not exist in the example provided as a globar variable (I checked it in the console), neither it does in my local project.

@fbrcc use "bower install angular-pdfjs-viewer --save" after instalation the pdf.js will be avaliable in folder "bower_components/pdf.js"

Thank you, this solved it

Hey Guys I'm experiencing the same issue and making that @diegosmotta says it's not resolving the issue, I've to do that before use this package but I'm still receiving the error... any words on this? (will be appreciated)

+1, please some help !

Doesnt work. Same error.

Same issue for me !

Same thing here.
pdf.js-viewer: 1.6.211

Unfortunately I've the same issue. Has someone finally found a way to fix that?
Thanks by advance.

same thing happening here.

Hi Dear Friends,
I am facing same issue.. Angular js:1.6
Can any one of the tell how to view the byte[] array to pdf viewer in angularjs. its Very argent. Am i facing the issues 4to 5 days.. now also didn't got it. please help me

Am getting PDFJS.webViewerLoad is not a function.. I already update the version of pdf.js..
thanks

any update on the above issue window.PDFJS.webViewerLoad("")..
because i am facing the same problem. I have installed the latest version of pdfjs

bower install angular-pdfjs-viewer --save" works for me..

The version of PDF.JS that is embedded is like this
function webViewerLoad() {
 var config = getViewerConfiguration();
 window.PDFViewerApplication = pdfjsWebApp.PDFViewerApplication;
 pdfjsWebApp.PDFViewerApplication.run(config);
}
if (document.readyState === 'interactive' || document.readyState === 'complete') {
 webViewerLoad();
} else {
 PDFJS.webViewerLoad = function (src) {
  DEFAULT_URL = src;
  webViewerLoad();
 };
}

When it's loaded in your code it directly calls the function webViewerLoad way before the directive is replaced. The workaround is to delete the first if statement so that the function webViewerLoad will be defined.
A quick and dirty solution is to override the "main" attribute of the pdf.js-viewer module in your main bower.json:
  "overrides": {
    "pdf.js-viewer": {
      "main": [
        "viewer.css"
      ]
    }
  }

Then copy and paste pdf.js and pdf.worker.js in a "lib" folder in your project drop the if statement
function webViewerLoad() {
 var config = getViewerConfiguration();
 window.PDFViewerApplication = pdfjsWebApp.PDFViewerApplication;
 pdfjsWebApp.PDFViewerApplication.run(config);
}
 PDFJS.webViewerLoad = function (src) {
  DEFAULT_URL = src;
  webViewerLoad();
 };

and include your modified pdf.js manually.
 <script defer src="lib/pdf.js"></script>
yes it's ugly, one could fork and do a push request, but I won't be that guy.


support of vscode-restclient style GraphQL

I tried the GraphQL with the extension exactly the same way as by vscode-restclient:
POST {{server}}
Content-Type: application/json
X-REQUEST-TYPE: GraphQL

query {
	test(id: 95, url: "/test95") { Url Name }
}

but got an error message, that the json is incorrect. Could you please show an working sample of GraphQL querying with the extension?
P.S. I used the extension version 1.6.0 on vscode 1.53.0 on windows

It will not work this way since httpyac is not a fork of vscode-restclient and the hack with the request header is intentionally not implemented. I want to support graphql even better, but for now that JSON still needs to be created manually.
POST {{server}}
Content-Type: application/json
{
"query": "{test(id: 95, url: "/test95") { Url Name }}"
}

My current idea would be to implement the following format.
POST {{server}}
Content-Type: application/json
gql`
query HeroNameAndFriends {
  hero {
    name
    friends {
      name
    }
  }
}
`

However, I want to read through the specification more before implementing this so I don't overlook special cases like graphql fragments.
I need to dig deeper into graphql and be able to run a use case. I am grateful for help.

I have discarded the idea with gql`. With the new version the query should work without the X-REQUEST-TYPE header. Fragments are also supported.
fragment IOParts on Repository {
  description
  diskUsage
}

POST https://api.github.com/graphql
Content-Type: application/json
Authorization: Bearer {{git_api_key}}

query repositoryQuery($name: String!, $owner: String!) {
  repository(name: $name, owner: $owner) {
    name
    fullName: nameWithOwner
    ...IOParts
    forkCount
    watchers {
        totalCount
    }
  }
}

{
    "name": "vscode-httpyac",
    "owner": "AnWeber"
}


Error handling

We streaming Twitter data from a service called GNIP suddenly I get a
ERROR [lt-dispatcher-2] c.s.r.k.KafkaActorSubscriber Stopping Kafka subscriber due to fatal error. WARNING arguments left: 1

As I read in order to understand what is happening I need to implement error handling with actors.
I would like some help on the example. Can someone show me how I can create a sink from the actor that supervises the KafkaActorSubscriber child?

There's an example in our Activator Template: https://github.com/softwaremill/activator-reactive-kafka-scala/blob/master/src/main/scala/sample/reactivekafka/KafkaWriterCoordinator.scala
Let me know if you need any further explanation on how to plug in your error handling.

Made some progress on the matter and ended up with this. Unfortunately I'm still getting the above mentioned error. I would like to know what is this  "arguments left: 1"
import akka.actor.SupervisorStrategy.Resume
import akka.actor.{ Props, Actor, ActorLogging, OneForOneStrategy, SupervisorStrategy }
import com.softwaremill.react.kafka.{ ProducerProperties, ReactiveKafka }
import com.typesafe.config.Config
import kafka.serializer.StringEncoder

import scala.collection.JavaConversions._

class ActorSupervisor(config: Config)(implicit val materializer: akka.stream.Materializer) extends Actor with ActorLogging {
  val subscriber: String = "Subscriber"

  override val supervisorStrategy: SupervisorStrategy = OneForOneStrategy() {
    case e =>
      log.error("Exception: {}", e)
      Resume
  }

  private def createSupervisedSubscriberActor() = {
    val kafka = new ReactiveKafka()

    val subscriberProperties = ProducerProperties(
      brokerList = config.getStringList("kafka.brokers").toList.mkString(","),
      topic = config.getString("kafka.topic"),
      clientId = config.getString("kafka.clientId"),
      encoder = new StringEncoder())

    val subscriberActorProps: Props = kafka.producerActorProps(subscriberProperties)
    context.actorOf(subscriberActorProps, subscriber)
  }

  override def aroundPreStart() = {
    createSupervisedSubscriberActor()
  }

  override def receive = {
    case msg => context.child(subscriber) match {
      case Some(ca) => ca ! msg
      case None =>
    }
  }
}

And the sink is created like this
val kafkaSink = Sink.actorSubscriber(Props(new ActorSupervisor(config)))

According to this you are missing a {} in KafkaActorSubscriber.
Should I make a PR?

Sure, a PR will be most welcome :)

Ok, I see that someone already changed the logging. I guess this will be part of 0.8.2.? @kciesielski any estimates on when you will put that up?

@alexanderzafirov I need to verify this one as well: https://github.com/softwaremill/reactive-kafka/issues/43, looks like a serious issue. As soon as it's resolved I'll release 0.8.2.

@kciesielski thanks!

@kciesielski Just to clarify - by labeling the issue as a bug are you referring to the fact that we cannot catch certain exceptions with the ActorSupervisor?

@alexanderzafirov Yes, because we cannot see the cause.

Fixed by 35f7a49


C:\Windows\System32\tar.exe --posix -z -cf cache.tgz -P -C D:/a/kit/kit --files-from manifest.txt
Cache Size: ~0 MB (30 B)
Cache saved successfully
Cache saved with key: pw-Windows-84213962f52cb541e4092ca888c1aa986057dc2ba0a2bed78a1292e3ee88eaa1

All CI runs are currently using this broken 30 byte cache for its Windows runs and promptly failing.
This PR works around this issue by changing the playwright key, because GitHub provides no way to manually evict caches and we'd have to wait 7 days with zero CI usage for it to automatically clear the cached browser.

Okay apparently this run also did the same exact thing:
Post job cleanup.
C:\Windows\System32\tar.exe --posix -z -cf cache.tgz -P -C D:/a/kit/kit --files-from manifest.txt
Cache Size: ~0 MB (30 B)
Cache saved successfully
Cache saved with key: playwright-Windows-84213962f52cb541e4092ca888c1aa986057dc2ba0a2bed78a1292e3ee88eaa1

So I guess the caching we've set up is just straight up broken on Windows. I'm suspecting a Unix vs Windows slash issue.

This is where the location to cache is defined for the browser on windows: https://github.com/sveltejs/kit/blob/master/.github/actions/env/main.js
const WINDOWS_BROWSER_PATH = '%USERPROFILE%\\AppData\\Local\\ms-playwright';

This should be correct according to the docs, or at least was at the time, not sure why it would have broken after working successfully unless there was a change with playwright.
I can take a look later if no one beats me to it.

Fixed by swapping %USERPROFILE% to os.homedir(). Maybe it was incorrectly resolving?


Possible to use HostApplicationBuilder instead of HostBuilder?

HostApplicationBuilder seems to be preferred by many, as its WebApplicationBuilder cousin is commonly used in ASP.NET apps. The use of ConfigurationManager is especially convenient when dealing with multiple interdependent configuration sources.
The existing documentation still uses HostBuilder. If it's possible to use HostApplicationBuilder, could an example be added? If not, could a note be added to indicate this limitation?

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

ID: 6ad5f362-cbfb-0621-4389-6378f50359e4
Version Independent ID: ebf1a8f3-ab9d-d295-e3a4-a982ed9df1e9
Content: Guide for running C# Azure Functions in an isolated worker process
Content Source: articles/azure-functions/dotnet-isolated-process-guide.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@hikarikuen
Thanks for your feedback! We will investigate and update as appropriate.

any updates?

Any update on this?

I would also like to know what the team's stance on this is.
We just ran into a massive problem that took us over a day to debug because of the behavior of the callback-based ConfigureAppConfiguration which contrary to what its documentation states, does not accumulate providers for subsequent calls:

https://github.com/dotnet/runtime/issues/43410

We needed to load the URL for Azure AppConfiguration from configuration, but the default Azure Function setup method only adds the EnvironmentVariables provider at the application level (when ConfigureFunctionsWorkerDefaults is called) meaning we cannot see environment variables in our own ConfigureAppConfiguration call to fetch the AzureAppConfig URL.
Now we have to either forcibly add the AZURE_FUNCTIONS_ on our url setting, or we are forced to reintroduce another EnvironmentVariables provider at the host level which duplicates the one at the app level using:
.ConfigureHostConfiguration(builder => builder.AddEnvironmentVariables())

I'm super disappointed that Microsoft has seemingly not yet updated Azure durable functions isolated to support the modern property-based host builder which does work much better with incremental configuration provider additions.

It appears this is now available as a preview feature; for example, https://learn.microsoft.com/en-us/azure/azure-functions/dotnet-isolated-process-guide?tabs=ihostapplicationbuilder%2Cwindows#start-up-and-configuration has IHostApplicationBuilder (preview) versions of the sample code.

It appears this is now available as a preview feature; for example, https://learn.microsoft.com/en-us/azure/azure-functions/dotnet-isolated-process-guide?tabs=ihostapplicationbuilder%2Cwindows#start-up-and-configuration has IHostApplicationBuilder (preview) versions of the sample code.

That's fantastic to see. I'm glad the team finally caught up to this. Looking forward to upgrading to v2 packages now as soon as they are out of preview.
Appreciate the information @hikarikuen .

It looks like 2.0.0 version released 8 hours ago :)


Revert raising of pyside6 version error

Description
This reverts most of the changes in #5244 because reordering the imports can cause circular imports. For example, try running pytest napari/_vispy/_tests. See the Zulip release discussion for more details.
This PR keeps the change to the tox.ini change, so that #5217 remains fixed.
Type of change

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

References
Fixes #5217

But why do you target it to the main, not the release branch?

I plan to add a note to the v0.4.17x release notes in the section regarding experimental Qt6 support (i.e. if you want to use pyside6, then use Python 3.10).

But why do you target it to the main, not the release branch?

Because the circular import is also present on main. I still plan to cherry pick the squashed commit from this PR to v0.4.17x.

Because the circular import is also present on main. I still plan to cherry pick the squashed commit from this PR to v0.4.17x.

I will solve it tomorrow. It is not a problem for the main branch.

It is not a problem for the main branch.

Yes, it is. Try running pytest napari/_vispy/_tests on the main branch. I get 5 errors.

I will solve it tomorrow.

This solves it today. And I won't be available tomorrow. But as you insist, I created a separate PR that targets the v0.4.17x branch and closed this.


[Proposal] Internal retry and DLQ mechanism

We're using OpenWhisk on IBM Cloud for streaming ETL and have implemented a custom retry mechanism as an ad-hoc addition to our pipeline. However, there are still two scenarios where lack of internal retry means we are out of options: internal failures and network failure at the machine running OpenWhisk.
If for whatever reason OpenWhisk cannot connect to any outside system, it becomes a black box that can only be investigated later by referring to log outputs. However, for security and practical reasons, using logs is not a feasible option for dealing with a high throughput system. Furthermore, log outputs might not be available for internal failures at all e.g. OOM crashes
The only alternative is to validate the information from upstream services that pass data to OpenWhisk (in our case Kafka) to downstream services; e.g. having an external arbiter that checks to see if there is a successful event in the downstream services for every message in our upstream service. However, that is not practically feasible because downstream services can be e.g. SaaS products that are not controlled by us and do not expose the required information.
My proposal is for OpenWhisk to handle retry internally after a payload has been handed off to it and it has acknowledged receiving it.
Thanks

@HunderlineK
This is a good idea.
While OpenWhisk only supports at-most-once semantic for some reason, it is still great to give options to users.
Why don't you create a POEM with this?
https://github.com/apache/openwhisk/tree/master/proposals


[ROCm] Fix for invalid llvm ir on AMDGPU in reduction

Address Space Casting initial value addr to generic space for a fix for the llvm ir generated on AMD GPU. This is a fix for the invalid llvm IR generated following fc9df104fdbbda6ee9096b40186a50badb147ac7.

cc @cheshire @chsigg gentle ping

@cheshire CastSharedToGlobal was reinstated. Can you rereview this PR please?


Can we run the dashboard container in Fargate?

Looking through the code, I don't think there's anything that reads from the filesystem in the dashboard there – but until we have the appraisal stuff working (see https://github.com/wellcometrust/platform/issues/4021), I don't want to fiddle with this.
This would let us get away with a smaller EC2 instance and/or break up the MCP worker.

No: the processing config is read from the disk.


BBT sentence case has stopped working for a few days now

Support log ID: WCKKVMWL-apse
Expected behavior: Change the title of article to sentence case.
Actual behavior: Nothing happens. This has stopped working since the last week, I have been trying with multiple articles from multiple sources, and I have received 2 - 3 updates in zotero-better-bibtex during this time. I have been using the regular zotero sentence case till now.

I can replicate this, on it.
As an aside, another plugin is generating (unrelated) errors, you might want to report this to the author:
[JavaScript Error: "TypeError: dois[0] is undefined" {file: "chrome://zotero-scite/content/Scite.js" line: 74}]
getDOI@chrome://zotero-scite/content/Scite.js:74:5
getCellX@chrome://zotero-scite/content/Scite.js:140:17
Zotero_ItemTreeView_prototype_getCellText@chrome://zotero-scite/content/Scite.js:161:12
Zotero_ItemTreeView_prototype_getCellText@chrome://zotero-better-bibtex/content/better-bibtex.js:59950:14

I can confirm that test build <IP_ADDRESS>4 fixes the issue. Thank you @retorquere

Thanks for the report; I've scheduled a new release, you'll be upgraded automatically, but it's simply a rebadged 774.


apm-server

Failed to connect to backoff(elasticsearch(http://elasticsearch:9200)): Get http://elasticsearch:9200: dial tcp <IP_ADDRESS>:9200: connect: connection refused

What is this issue about? Where are the logs requested in the issue template?

@antoineco default config
apm-server:
  host: localhost:8200

output:
  elasticsearch:
    hosts: http://elasticsearch:9200

Please edit your issue and include all the requested information, a partial screenshot does not help.


Provide option to switch between .cirlceci-contained configs and distributed configs

Describe Request:
Should allow switching between the default directory structure
.circleci/
    config.yml
    ... other configs

to stating the location of additional config.yml in other locations around the repo, like
.circleci/
    config.yml
test/
    cmd/
        config.yml
... other directories containing configs, etc.

Examples:
Supporting Documentation Links:

Unlikely this should be used in practice, closing.

Also covered by #18


Article does not fit to new "gsuite" connector

This article is written for the old G Suite gallery app that referenced the "Goog2OutDelta" connector. The current G Suite gallery app references the "gsuite" connector that has a totally different target schema. Also the the target directory name is no longer "GoogleApps" but "G Suite". Please update the article.

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

ID: c64daae8-0d9c-5986-043f-e383d06827d9
Version Independent ID: 65c4180b-3540-6de6-2b6e-833d1793fb65
Content: Tutorial: Configure G Suite for automatic user provisioning with Azure Active Directory
Content Source: articles/active-directory/saas-apps/google-apps-provisioning-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@MrAzureAD Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

@v-sonan Can you please fix this?

@jeevansd Work items have been created for this and the work is in progress.

@Chetansriv for tracking.

#please-close


Concisely describe the proposed feature
A clear and concise description of what you want. For example,

The log format collected to Kafka is not friendly to subsequent processing

Describe the solution you'd like (if any)
A clear and concise description of what you want to achieve and implement. For example,

I would like to add an log flush method for flusher_kafka.

Additional comments
Add any other context or screenshots about the feature request here.
For example, the ideal input and output logs.

The logs are supposed to have the following contents, e.g.,
{"_target_":"<IP_ADDRESS>","_field_":"DISMAN-EXPRESSION-MIB::sysUpTimeInstance","_oid_":".<IP_ADDRESS>.<IP_ADDRESS>.0","_type_":"TimeTicks","_content_":"10423593"}

请问，是否方便参与共建支持这个功能？

期待参与共建支持这个功能. 我如何参与?
在 2023-06-04 17:48:25，"徐可甲" @.***> 写道：
请问，是否方便参与共建支持这个功能？
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

可以先给个设计文档，设计文档可以放到https://github.com/alibaba/ilogtail/issues/893#issuecomment-1581926096


[transaction builder generator] make python test ready for CI

Motivation
I had to make a few fixes in preparation of https://github.com/diem/diem/pull/7635 (adding CI to Move Sdk).
This PR lands them early on while we are figuring out what to do with CI.
The pyre fix is similar to https://github.com/novifinancial/serde-reflection/pull/83
Test Plan
On my laptop:
cargo test -p transaction-builder-generator -- --ignored

Tested in CI as part of https://github.com/diem/diem/pull/7635

/land


Fix simple action client/server

These are a few fixes to make SimpleActionClient and SimpleActionServer work.
Remarks:

simple_action_client.dart and simple_action_server.dart are now exported. I didn't change the export status of actionlib_{client,server}.dart (exported) and `action_{client,server}.dart' (unexported).
I'm not sure of the best place for test cases, since we need quite a few messages. For now I'll make a PR with test code in https://github.com/TimWhiting/test_dart_ros .
I didn't find a way to properly shut down the nodes: they still appear in the output of rosnode list (unlike Python nodes which disappear after shutdown).
I've tested this with ROS Noetic.

Closes #12

Thanks, everything should be pushed and published!


adding exploratory analysis and visualizations, still WIP

I have been working on plotting the DEM data as well as samples of the NAIP data. This is a work in progress notebook. Next steps will be to attempt to load the DEM data into a torchgeo dataset to see if that makes sampling it easier. I do not think this is ready to merge yet though

Hi! We can merge these changes as the notebook does not really affect any other file in main. Please let me know if I should merge.

I have added the code in the notebook with a sample class that is close to the previous class but with some slight changes to help query bounding boxes. I am able to print the correct piece of the dem data when opening through rasterio but when plotting using the class object it does not work.

This code is ready to merge, but I hadn't read the message when I made the PR. I created a new branch for further work on the DEM class but should I merge this?


Problems with opacity in expo web

I'm having some issues on expo web with opacity prop. What I'm doing:
<AnimatePresence>
    {selected && (
        <MotiView
              from={{ translateX: 50, opacity: 0 }}
              exit={{ translateX: 50, opacity: 0 }}
              animate={{ translateX: 0, opacity: 1 }}
              transition={{ mass: 0.15, damping: 4 }}
          >
            <SelectedIcon />
        </MotiView>
    )}
</AnimatePresence>

The animation and timing are perfect, BUT when the animation ends the SelectedIcon component goes back to opacity: 0.
When I take off the opactity prop, it works fine.
I tried to debug with onDidAnimate but the value returns as undefined.

Hey, thanks for the issue!

What version of RNW? Please upgrade, since expo web uses an old version. I've had success with 0.15.3
This is likely a reanimated 2 issue with web. But we can try to debug it here!

Hey, thanks for the issue!

What version of RNW? Please upgrade, since expo web uses an old version. I've had success with 0.15.3
This is likely a reanimated 2 issue with web. But we can try to debug it here!

Hi, I just tested it with RNW 0.16.3, no success. Problem persists!
The weirdest thing is everything else works fine! The problem is only the opacity property. The project is almost production ready, so for now I switched to scale animation, althoug it's not the animation we've planned, it's doing the job so far.

What if you do opacity without scale? Still no luck?

Can you reproduce this in a snack?

What if you do opacity without scale? Still no luck?

Already tried, no luck!

Can you reproduce this in a snack?

I'm a little busy today, but I'll do it ASAP.

I use opacity all over my app so this must be an issue with your setup.
https://snack.expo.io/@kellycup8/tactless-cheese

I've tested in other components also, follow the same problem
Anyway, this is component:
function ProfessionalItem({ professional, selected, onPressItem }: Props) {
    const { palette } = useTheme();

    function handlePressItem() {
        if (onPressItem) onPressItem(professional);
    }

    function Selected() {
        return (
            <MotiView
                from={{ scale: 0, opacity: 0 }}
                exit={{ scale: 0, opacity: 0 }}
                animate={{ scale: 1, opacity: 1 }}
                transition={{ mass: 0.15, damping: 4 }}
            >
                <SelectedIcon />
            </MotiView>
        );
    }

    return (
        <ProfessionalItemContainer selected={selected} onPress={handlePressItem}>
            <ProfessionalItemContent>
                <Avatar source={{ uri: professional.image }} size={30} />

                <ProfessionalTitleContainer>
                    <ProfessionalTitle>{professional.name}</ProfessionalTitle>

                    <ProfessionalRatingContainer>
                        <Icon name="star" size={18} color={palette.primary.default} />

                        <ProfessionalRating>
                            {professional.rating.toFixed(1).replace('.', ',')}
                        </ProfessionalRating>
                    </ProfessionalRatingContainer>
                </ProfessionalTitleContainer>

                <AnimatePresence>{selected && <Selected />}</AnimatePresence>
            </ProfessionalItemContent>
        </ProfessionalItemContainer>
    );
}

export default React.memo(ProfessionalItem, (prev, next) => prev.selected === next.selected);

You shouldn't be putting selected into its own function like that. JSX components shouldn't be created in render code. If you want to do that, call it as a function, like {selected && Selected()}.
But stylistically, I don't see the benefit to the syntax in your example.
In any case, I can't reproduce this in my snack, so I think something is wrong with your setup. Please make sure you're on the latest versions of reanimated and moti. If you recreate this in a snack, I can reopen.


Wiki spam

you may want to clean that out: e.g. https://github.com/Netflix/Hystrix/wiki/Service-air-panas-081291315510--

Deleted.  Thanks @hgschmie

Uhm, that was just an example. There are ~200 additional pages that look the same....

Think I got them this time.


Additional gateways in IOP create duplicate PodDisruptionBudgets

apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
metadata:
  name: custom-ingressgateway-iop
spec:
  profile: empty
  components:
    ingressGateways:
      - name: custom-ingressgateway-1
        enabled: true
      - name: custom-ingressgateway-2
        enabled: true

generates these two PDB:
apiVersion: policy/v1beta1
kind: PodDisruptionBudget
metadata:
  name: custom-ingressgateway-1
  namespace: istio-system
  labels:
    app: istio-ingressgateway
    istio: ingressgateway
    release: istio
    istio.io/rev: default
    install.operator.istio.io/owning-resource: unknown
    operator.istio.io/component: "IngressGateways"
spec:
  minAvailable: 1
  selector:
    matchLabels:
      app: istio-ingressgateway
      istio: ingressgateway
---
apiVersion: policy/v1beta1
kind: PodDisruptionBudget
metadata:
  name: custom-ingressgateway-2
  namespace: istio-system
  labels:
    app: istio-ingressgateway
    istio: ingressgateway
    release: istio
    istio.io/rev: default
    install.operator.istio.io/owning-resource: unknown
    operator.istio.io/component: "IngressGateways"
spec:
  minAvailable: 1
  selector:
    matchLabels:
      app: istio-ingressgateway
      istio: ingressgateway

Notice that the matchLabels for both PDB are the same.
This causes problems during draining node:
This pod has more than one PodDisruptionBudget, which the eviction subresource does not support.

Should the match labels use a different selector that is unique to each gateway? such as:
service.istio.io/canonical-name=custom-ingressgateway-2
[ ] Docs
[ X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
Should the match labels use a different selector? such as:
service.istio.io/canonical-name=custom-ingressgateway-2
Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
1.7.6
How was Istio installed?
istioctl manifest generate
Environment where the bug was observed (cloud vendor, OS, etc)
IBM Cloud Kubernetes

cc @linsun

Thanks @rvennam for reporting this, makes sense to me.
Hi @morvencao - would like to get your thoughts?  I think there are 2 possible ways to fix this:

if we could detect a PDB for a given label selector for ingress gateway for a ns has been created, we don't create any further PDBs.
If the first option is not possible, allow ingress gateway to tell us if they want to have PDB created as part of the ingress gw.

cc @sdake @ostromart @costinm

@rvennam @linsun
In this case we should add label that is unique to each gateway:
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
metadata:
  name: custom-ingressgateway-iop
spec:
  profile: empty
  components:
    ingressGateways:
      - name: custom-ingressgateway-1
        enabled: true
        label:
          app: custom-ingressgateway-1
      - name: custom-ingressgateway-2
        enabled: true
        label:
          app: custom-ingressgateway-2

Then the operator will create different PDB for different gateway, with unique matchLabels:
---
apiVersion: policy/v1beta1
kind: PodDisruptionBudget
metadata:
  name: custom-ingressgateway-1
  namespace: istio-system
  labels:
    app: custom-ingressgateway-1
    istio: ingressgateway
    release: istio
spec:
  minAvailable: 1
  selector:
    matchLabels:
      app: custom-ingressgateway-1
      istio: ingressgateway
---
apiVersion: policy/v1beta1
kind: PodDisruptionBudget
metadata:
  name: custom-ingressgateway-2
  namespace: istio-system
  labels:
    app: custom-ingressgateway-2
    istio: ingressgateway
    release: istio
spec:
  minAvailable: 1
  selector:
    matchLabels:
      app: custom-ingressgateway-2
      istio: ingressgateway
---

@morvencao Thanks!
Is it safe for us to use a new label instead of app? For example:
    ingressGateways:
      - name: custom-ingressgateway-1
        label:
            gateway: zone1
        enabled: true
      - name: custom-ingressgateway-2
        enabled: true
        label:
            gateway: zone2

Yes, you can @rvennam


Multi selection SelectView

Hello,
I would like to select multiple items in a SelectView. Can this be achieved?

This is not currently possible, but looks like a use case we'd like to support.

In the short-term, it might be better served with a list of CheckBoxes.

I'm not sure what's your @gyscos concept for multi selection. Perhaps current SelectView should be extended.
But if not, I've done this.
It's strongly opinionated version for my use case.. .
What do you think?

+1 for this feature. It will be very useful if we could extend SelectView for multiple selections.


Integration of DeepL translation service

Integration of DeepL translation service as requested in #191.
DeepL API requires a key (see https://www.deepl.com/docs-api/)

:+1: :smile:
Fixes #191


embeddings: refactor EmbedderClient interface to reduce code duplication

Starting with the vertex embedder, but this is applicable to others too.
The common code fixes the combined vector issue, similarly to #357, for #356
This replaces a completely duplicated method in two places with a common function; the same function can be used for the embedders of other LLMs as well, if this approach is deemed good.
PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[x] Describes the source of new concepts.
[x] References existing implementations as appropriate.
[x] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.

Thanks @tmc - I'd be happy to apply this to other embeddings backends, but it's not clear to me why there are two at all - why is there one instance of LLM and one of Chat with the same embeddings methods and implementations? E.g. https://github.com/tmc/langchaingo/blob/main/embeddings/ollama/ollamachat/ollama_chat.go#L31 and https://github.com/tmc/langchaingo/blob/main/embeddings/ollama/ollama.go#L32 -- could you clarify?


[fix] Move CODEOWNERS to the root of the repo

It wasn't working in .github. Hopefully it'll work here.
Signed-off-by: Avi Miller<EMAIL_ADDRESS>

I stuffed up the rebase. I'll open a new PR later.

I stuffed up the rebase. I'll open a new PR later.


'Main_platform' can't be use into the model pipeline

Summary
In model_pipeline.py, we are trying to build a ML pipeline using all features in our dataset. The dataset contains categorical feature such as 'main_platform'. We use pd.get_dummies to transform this feature into a dummy one. However some values of 'main_platform' presents in training set are not in the test set, resulting into an error.
Steps to reproduce
Download the repo and run model_pipeline.py.
Expected behavior
We should get a MAE score printed in the terminal at the very end. Instead, we get an error.
Actual behavior
We get the following error :
  File "C:\Users\charl\miniconda3\lib\site-packages\pandas\core\indexes\base.py", line 3080, in get_loc
    return self._engine.get_loc(casted_key)
  File "pandas\_libs\index.pyx", line 70, in pandas._libs.index.IndexEngine.get_loc
  File "pandas\_libs\index.pyx", line 101, in pandas._libs.index.IndexEngine.get_loc
  File "pandas\_libs\hashtable_class_helper.pxi", line 4554, in pandas._libs.hashtable.PyObjectHashTable.get_item
  File "pandas\_libs\hashtable_class_helper.pxi", line 4562, in pandas._libs.hashtable.PyObjectHashTable.get_item
KeyError: 'main_platform'

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

Traceback (most recent call last):
  File "C:\Users\charl\miniconda3\lib\site-packages\sklearn\utils\__init__.py", line 396, in _get_column_indices
    col_idx = all_columns.get_loc(col)
  File "C:\Users\charl\miniconda3\lib\site-packages\pandas\core\indexes\base.py", line 3082, in get_loc
    raise KeyError(key) from err
KeyError: 'main_platform'

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

Traceback (most recent call last):
  File "c:\Users\charl\OneDrive\Documents\Projects\OWREstimator\src\model_pipeline.py", line 80, in <module>
    model_pipeline.fit(X_train,y_train)
  File "C:\Users\charl\miniconda3\lib\site-packages\sklearn\pipeline.py", line 341, in fit
    Xt = self._fit(X, y, **fit_params_steps)
  File "C:\Users\charl\miniconda3\lib\site-packages\sklearn\pipeline.py", line 303, in _fit
    X, fitted_transformer = fit_transform_one_cached(
  File "C:\Users\charl\miniconda3\lib\site-packages\joblib\memory.py", line 352, in __call__
    return self.func(*args, **kwargs)
  File "C:\Users\charl\miniconda3\lib\site-packages\sklearn\pipeline.py", line 754, in _fit_transform_one
    res = transformer.fit_transform(X, y, **fit_params)
  File "C:\Users\charl\miniconda3\lib\site-packages\sklearn\compose\_column_transformer.py", line 505, in fit_transform
    self._validate_remainder(X)
  File "C:\Users\charl\miniconda3\lib\site-packages\sklearn\compose\_column_transformer.py", line 332, in _validate_remainder
    cols.extend(_get_column_indices(X, columns))
  File "C:\Users\charl\miniconda3\lib\site-packages\sklearn\utils\__init__.py", line 403, in _get_column_indices
    raise ValueError(
ValueError: A given column is not a column of the dataframe```

Relevant documentation : https://stackoverflow.com/questions/62240050/how-to-run-model-on-new-data-that-requires-pd-get-dummies?noredirect=1&lq=1


Fixup serialization of ZZ gate

Logic was assuming that for gates with 1 input that input was 'phase' which is not true for ZZ, which just has 'theta'

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

ty :pray:


Stone Pine Crash

Crash Report:
https://mclo.gs/p3MXgMe
Latest Log:

This is not actually caused by Sniffer+. Some other mod is crashing your game. Please supply the log, not crash report, and I can point you to what mod is the problem

I See, in that case, my apologies for creating this issue under your mod!
This is from latest log : https://mclo.gs/yo3h9i4

You're all good! It's a common issue here. In the next version I'm gonna try and see if I can make this not happen. According to that log, the error seems to come from Patchouli. I'd double check you have the right version. That said, the event Patchouli is looking for should exist in 1.20.1 I'd think, so not sure why it's erroring.


text changed from upload to convert to image

using upload was making a confusion , so used "convert to image "

changed


Update defaults.py

Primary reason for update is higher quality images and more status codes images available.
The code is not breaking if #3 is incorporated.

Primary reason for update is higher quality images and more status codes images available.
The code is not breaking if #3 is incorporated.

is there an issue in overriding the defaults?

Primary reason for update is higher quality images and more status codes images available.

Thinking more about this, it won't necessarily be a breaking change since people won't be much concerned about the website behind the images, as long as the images are displayed.
Can you please combine all the changes from #3, and #2 here in this patch? it would make more sense to have a feature, its related test and documentation updated in the same patch.
Also, while updating the documentation, can you please make sure we no longer use the images from the earlier website?
Let me know, if you need any help or more clarification regarding this.


Investigate adding fast-math functions

Relevant: https://github.com/rust-lang/rust/issues/21690
My current thinking around this is that we probably shouldn't even do this.
However, if we do this, I'm kind of partial to having something like unsafe fn sqrt_approx() implemented on floats, that one needs to call explicitly within an unsafe block. As discussed in that issue, there are quite a few problems with the existing LLVM backend, it's optimizations etc. Many of which we don't need to care about because we're rolling our own. Having said that, SPIR-V and especially Vulkan have their own set of restrictions around floats: https://www.khronos.org/registry/vulkan/specs/1.2-extensions/html/vkspec.html#spirvenv-precision-operation

This probably isn't on our roadmap any time soon, so closing this until someone has a concrete need for it with opinions and desires we can design the feature around.


Fixes MicrosoftDocs/azure-docs#35489

Fixes MicrosoftDocs/azure-docs#35489
Updated description for the WorkspaceId parameter.

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

File
Status
Preview URL
Details

azps-0.10.0/Az.Monitor/Set-AzDiagnosticSetting.md
:white_check_mark:Succeeded
View (azps-0.10.0)

azps-1.8.0/Az.Monitor/Set-AzDiagnosticSetting.md
:white_check_mark:Succeeded
View (azps-1.8.0)

azps-2.8.0/Az.Monitor/Set-AzDiagnosticSetting.md
:white_check_mark:Succeeded
View (azps-2.8.0)

azps-3.8.0/Az.Monitor/Set-AzDiagnosticSetting.md
:white_check_mark:Succeeded
View (azps-3.8.0)

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Feature: SBOM documentation

As a user of the OpenSauced platform, I want documentation related to the Software Bill of Materials (SBOM) feature set so that I can have a good understanding of SBOM and the related tools.

Once this documenation has been added, it'd be good to update the learn more... link here, https://github.com/open-sauced/app/pull/3938/files#diff-5791fb55bff1d92f290ccb1d66c6b1a0c833c675c245466810fb72504e5a00d4R265-R266
Relates to https://github.com/open-sauced/app/pull/3938

Some additional references care of @jpmcb:

NIST also has an important resource: https://www.nist.gov/itl/executive-order-14028-improving-nations-cybersecurity/software-security-supply-chains-software-1
The OpenSSF also has a whole working group on this with some good resources: https://github.com/ossf/sbom-everywhere/tree/main

Quick note: It may be an anti-pattern to document exactly what an SBOM is since it's abit of a moving target: there are lots of government agencies, third party organizations (like the OpenSSF and Linux foundation), and companies that all have opinions on this. Even GitHub sort of has their own spin: https://docs.github.com/en/code-security/supply-chain-security/understanding-your-software-supply-chain/exporting-a-software-bill-of-materials-for-your-repository
Personally, I'd focus on the OpenSauced feature while referring people to the CISA or the NIST. Or even direct quoting from those gov agencies would be good 👍🏼


Product check fixes

perform cheaper bool checks before synchronisation methods
don't issue original request if product check fails
check original request error before response header check (avoiding panic)
replace sync.Once approach so transient network errors are not cached

LGTM!


ModelRocket's performance

Hi,
I was wondering about ModelRocket's performance compared to a few other popular JSON mapper libraries:
https://github.com/bwhiteley/JSONShootout
However, due to the way of creating the class variables (e.g. let title = Property(key: "title"), I'm not sure how I can add ModelRocket to do a comparison.
Looks like I will need to write an extension on these classes and member values:

ModelObjects/Program.swift
ModelObjects/Recording.swift

Any suggestions?

Hey @falcon4ever,
Thanks for reaching out! JSONShootout uses value types (i.e. structs) for its models, but ModelRocket uses reference types (i.e. classes) which subclass the Model class. Because of that, you'll need to create new models for the ModelRocket tests. They could look something like this:
final class RecordingMR: Model, JSONTransformable {
    enum Status: String, JSONTransformable {
        case none      = "0"
        case recorded  = "-3"
        case recording = "-2"
        case unknown
    }

    enum RecGroup: String, JSONTransformable {
        case deleted = "Deleted"
        case default = "Default"
        case liveTV  = "LiveTV"
        case unknown
    }

    let startTsStr = Property<String>(key: "StartTs")
    let status     = Property<Status>(key: "Status")
    let recordId   = Property<String>(key: "RecordId")
    let recGroup   = Property<RecGroup>(key: "RecGroup")
}

final class ProgramMR: Model {
    let title     = Property<String>(key: "Title")
    let chanId    = Property<String>(key: "channel.ChanId")
    let desc      = Property<String>(key: "Description")
    let subtitle  = Property<String>(key: "SubTitle")
    let recording = Property<RecordingMR>(key: "Recording")
    let season    = Property<Int>(key: "Season")
    let episode   = Property<Int>(key: "Episode")
}

If you're simply curious how ModelRocket's performance compares, I ran it in JSONShootout's test suite, and here are the results:

Test
Duration

testDeserialization
0.08s

testPerformance
0.55s

Let me know if you have any other questions. Thanks!

Thanks @jlandon ! I'm mostly interested in ModelRocket's performance, while it has been performing fine for my needs, I just wasn't sure how it would compare to others.
I migrated from SwifyJSON as that one was underperforming and was trying to decide between Argo and the (back then new) ModelRocket. I'm surprised that the performance of Argo isn't as good as expected (since it's developed by a bigger team).


hash Join Optimization Collection

Simdize Vectorhasher int64 range path.

Simdize retrieving hash join result rows.

Special inline path for normalized key hash join probe.

faster mixing of bits for normalized key to hash number conversion.

Drops single thread time for velox_tpch_benchmark from 40s to 36s with
10G uncompressed Parquet dataset.

@oerling There is a minor merge conflict

Looks like #2347 is the newest version


Add support for latest HTTP Signatures spec draft

Since Mastodon 3.2.1 servers require a Digest header in their POST requests.
https://github.com/tootsuite/mastodon/pull/14556
PR Forthcoming

@mediaformat fixed it with #105
Thanks a lot!

Cheers


Question about repetitive MemoryBackedStore construction pattern

What is the reason for this kind of pattern?
let files: Vec<_> = (0..40).map(|_| MemoryBackedStore::new()).collect();
let clf = ChildLayerFiles {
    node_dictionary_files: DictionaryFiles {
        blocks_file: files[0].clone(),
        offsets_file: files[1].clone(),
    },
//...

The above comes from from here, but there are a number of them:
src/layer/base.rs:1040:    let files: Vec<_> = (0..21).map(|_| MemoryBackedStore::new()).collect();
src/layer/builder.rs:299:  let files: Vec<_> = (0..21).map(|_| MemoryBackedStore::new()).collect();
src/layer/builder.rs:346:  let files: Vec<_> = (0..40).map(|_| MemoryBackedStore::new()).collect();
src/layer/child.rs:1597:   let files: Vec<_> = (0..40).map(|_| MemoryBackedStore::new()).collect();
src/storage/memory.rs:251: let files: Vec<_> = (0..21).map(|_| MemoryBackedStore::new()).collect();
src/storage/memory.rs:321: let files: Vec<_> = (0..40).map(|_| MemoryBackedStore::new()).collect();

I don't currently see why each of files[0].clone() to files[39].clone() shouldn't be replaced with MemoryBackedStore::new(), but I could be missing something.

.. Yeah that would actually be much better. The reason for this pattern is probably sleep deprivation.


story/continue-log4net

created pull request. New branch

ready to merge. I want to see if this will work for your ec2 virtual machine now.


Fix Payer Cost View

Fix Payer Cost View in Summary Row.

Codecov Report

Merging #1192 into development will increase coverage by <.01%.
The diff coverage is 85.71%.

@@               Coverage Diff               @@
##           development    #1192      +/-   ##
===============================================
+ Coverage        56.92%   56.92%   +<.01%
===============================================
  Files              277      277
  Lines            20734    20737       +3
===============================================
+ Hits             11802    11805       +3
  Misses            8932     8932

Impacted Files
Coverage Δ

MercadoPagoSDK/MercadoPagoSDK/PayerCostView.swift
69.64% <85.71%> (+1.71%)
:arrow_up:

Pongamos bien lo que ese corrige en el PR


Apply folding view state from untitled files to file when saved

VSCode Version:
OS Version:

Steps to Reproduce:

When we  save a file and open again the fold state must maintain..

@japsancha It works for me when I saved a file, closed it and opened it, folding state is in previous state.
Can you please provide more steps to re-create the issue you are seeing?
Thanks.

Closing. Haven't received any response. @japsancha please ping if the issue is still reproducible with steps that demo the problematic behavior.

Hi
In the images file I send an example

The screenshot shows an unsaved, untitled file.
Where did you save it? Is it in the workspace folder?

Assigning to @bpasero. It's the workbench who reopens the file with the save state (which contains the folding regions)

PR welcome 👍

When saving a untitled file (or using "Save As" on an existing file), the entire editor view state is now being applied again on the target.
Note that there is a corner case: If a save participant is running and changing the content as a matter of the save (removing / adding lines), the view state is ignored by the editor because the line count do not match anymore.


Remove crds on uninstall -k

Closes https://github.com/dapr/cli/issues/656.

@yaron2 PLease fix the upgrade path e2e tests ... since it also checks that the CRDs remain on uninstall ... that behavior is changed in the PR

I'm closing this PR and it's associated issue since deleting the CRD resource automatically deletes any component, configuration or subscription that's in the cluster and this is not a desired behavior.


Attempt fit to sco-cen

Both an aged component and a non-aged, no cov run are running on sco-cen subset (80,000 stars)

Fitting is progressing but is rather slow... performing 5 component fits, none with ages > 1 have emerged yet.


fix deprecation warnings in Node 8

Node 8 provides deprecation warnings when running with --pending-deprecation flag and we need small fixes in tailor core.

[ ] Move from Buffer() to Buffer.alloc (https://nodejs.org/en/blog/release/v8.0.0/#buffer-changes)
[ ] Include Node 8 as part of travis build

@vigneshshanmugam IMO we should use Buffer.allocUnsafe() instead of Buffer.alloc() to gain some performance in memory allocation. Ideally, we can create some benchmark tests to compare these 2, but I think Tailor doesn't have scenarios where memory leak can happen because of Buffer.allocUnsafe()`. Thoughts?

Memory leak can happen in tailor due to number of subscribers to tailor's event emitters. This might be a hint.


Support ORDER BY clause

We need to allow the user to specify one or more sort fields and directions in the ORDER BY clause for a Selectable.
Selectable.OrderBy() should accept Column, ColumnDef or string literals. The following code should be runnable:
users := meta.Table("users")

emailCol := users.Column("email")

sel1 := sqlb.Select(users).OrderBy(emailCol)
sel2 := sqlb.Select(users).OrderBy(users.ColumnDef("email"))
sel3 := sqlb.Select(users).OrderBy("email")

// sel1.String() == sel2.String() == sel3.String()

Not going to support the string literal variant...


Swarm automatic cluster creation

Adding the shell code below appears to allow swarm users the ability to automatically create the cluster without having to manually scale the seed and the cluster.  The node service can specify the total number of replicas in the service definition.  The cluster will be created and the seed will automatically be created after all the nodes have joined and the cluster is synced.  Note, I'm not a Galera expert so there may be a more efficient way to accomplish the same.  Suggestions are welcome.
I have added the following environment variables to the seed:
- IS_SEED=true
- SKIP_UPGRADES=1
- MYSQL_ROOT_SOCKET_AUTH=0
- NODE_REPLICAS=<integer of total number of nodes + seed>
- CLUSTER_NAME=<unique name.
Also, the following needs to be added to the seed service definition:
    deploy:
      restart_policy:
        condition: none

The following code is added after the 'gosu mysql mysqld.sh...' line in start.sh but prior to 'wait $! || true':
if [[ $IS_SEED = "true" ]] ; then
	cluster_size=1
	while [ "$cluster_size" -lt "$NODE_REPLICAS" ] ; do
	    cluster_size=`mysql -uroot -p$MYSQL_ROOT_PASSWORD information_schema -N -e "select VARIABLE_VALUE from GLOBAL_STATUS where VARIABLE_NAME='wsrep_cluster_size';"`
	    echo "cluster size: $cluster_size"
		sleep 10
	done

	local_state=""
	while [ "$local_state" != "Synced" ] ; do
	    local_state=`mysql -uroot -p$MYSQL_ROOT_PASSWORD information_schema -N -e "select VARIABLE_VALUE from GLOBAL_STATUS where VARIABLE_NAME='wsrep_local_state_comment';"`
	    echo "local state: $local_state"
		sleep 10
	done
        # The following sleep command may possibly be reduced but adding it appears to ensure that the state transfers are completed prior to the seed shutdown
        sleep 120
	shutdown
fi

So does this method assume that you will actually run N+1 cluster members? One issue with that is that if you only have N physical nodes it requires an additional node temporarily or double the disk space on one node which for a large database could be an issue. Manually starting the seed and rebooting it as a node allows the seed node to not have to be duplicated.

The method assumes only N cluster members.  For a majority of databases, disk space should not be an issue and it is assumed that the user has appropriately sized the swarm nodes.  Additionally, since volumes can be separately mounted, it does not require that the swarm node itself contain the disk space needed for the database.  For the clusters that we deploy, the seed is constrained to the master and one of the node containers is also started on the container.
I have now tested this with a number of different stacks and it appears to be running without issue.  The above dramatically simplifies deployment because the stack can be deployed without any manual intervention.

Ok, perhaps I don't fully understand it still. Would you mind putting it into a pull request for direct review? Thanks for your contributions!

I would rather spend time ensuring that you understand the code because a pull request, especially given its confined nature, will not answer any questions/concern.  What do you not fully understand and maybe I can answer your questions.
Happy to contribute back to your fantastic work.

Mainly what the correct parameters are for the docker-compose.yml file and the steps for creating a cluster after this update. How do you prevent the Nth node from starting until the seed is shut down?

Why is that needed?  On a 3 node + seed stack definition, all the nodes join and are synced creating a 4 node cluster after which the seed is shut down leaving the desired 3 node cluster.

What if your nodes are not so easily created/destroyed? E.g. high-dollar dedicated instances.

In the above code, the seed does not shut down until the number of NODE_REPLICAS (which is the total number of nodes plus the seed) have joined and are synced.  To clarify, in the 3+1 example, the value would be set to 4.  If the cluster size is less than the NODE_REPLICAS, then there is a different issue which would need to be manually addressed anyway since the stack did not initiate as defined.
For clarity, NODE_REPLICAS may be set to the actually desired cluster size and the above code modified to account for the same, i.e., change [ "$cluster_size" -lt "$NODE_REPLICAS" ] to [ "$cluster_size" -lt "$NODE_REPLICAS" +1 ].


Environment value are not correctly updated

Terraform Version
Terraform v1.3.9

Affected Resource(s)
Please list the resources as a list, for example:

postman_environment

Terraform Configuration Files
resource "postman_environment" "cohort" {
  name      = var.project
  workspace = local.workspace_id

  values = [
    {
      key   = "firebaseApiKeyWallet"
      value = data.google_firebase_web_app_config.app.api_key
      type  = "secret"
    },
    {
      key   = "firebaseApiKeyAdmin"
      value = data.google_firebase_web_app_config.admin.api_key
      type  = "secret"
    },
    {
      key   = "adminUrl"
      value = "https://admin-api.${var.root_domain}"
      type  = "default"
    },
 ]
}

Debug Output
Please provider a link to a GitHub Gist containing the complete debug output: https://www.terraform.io/docs/internals/debugging.html. Please do NOT paste the debug output in the issue; just paste a link to the Gist.
Panic Output
If Terraform produced a panic, please provide a link to a GitHub Gist containing the output of the crash.log.
Expected Behavior
When a value change this value is applied on the current value
Actual Behavior
When an environment variable is updated the current value is not correctly updated, only the initial value is updated.
In order to bypass that I need to remove the content of the env
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:

write code for creating an environment
terraform apply
edit some values of the environment
terraform apply
check postman

Hi, thanks for submitting this issue! Would you be interested in submitting a pull request to fix this?

Hi, thanks for submitting this issue! Would you be interested in submitting a pull request to fix this?

Yes, I can try the only things I didn't find is the api documentation for postman.
There is a documentation available or you are looking for network calls ?

Awesome! 🚀 (apologies for the late reply, hectic day yesterday!)
Good question, the documentation for the postman API is here: https://www.postman.com/postman/workspace/postman-public-workspace/api/72a32ca3-f06a-4e83-a933-2821a0e6616f/definition/024624cf-e622-4836-9d19-efd067f6cbdb/file/024624cf-e622-4836-9d19-efd067f6cbdb?version=860f56fe-6024-4403-a5ae-1b95532ba7b9
And you can find the sanitised version that used in the repo here: https://github.com/jonnydgreen/terraform-provider-postman/blob/main/openapi.yaml
More details around contributing can be found here: https://github.com/jonnydgreen/terraform-provider-postman/blob/main/CONTRIBUTING.md
If you have any further questions, don't hesitate to ask! :)


Error when comparing equality of indexes

In 0.16.0 comparing MultiIndices using == raises an exception.
from StringIO import StringIO
import pandas as pd

# create a series using a multiindex
df = pd.read_csv(StringIO('a,b,c\n1,2,3'), index_col=[0, 1])
series = df['c']

series.index == series.index
#   File "/Users/dan/venvs/pandas-0.16.0/lib/python2.7/site-packages/pandas/core/index.py", line 54, in wrapper
#     func = getattr(self._data.view(np.ndarray), opname)
# AttributeError: 'NoneType' object has no attribute 'view'

print pd.__version__
# 0.16.0

Another one I've come across is that there is an AttributeError when comparing indexes for which the underlying numpy arrays cannot be broadcast together. The error message itself is very obscure. I gather that .equals() can be used for what I'm trying to do in these examples, but still the errors when using == are surprising.
>>> pd.Index(['a', 'b']) == pd.Index(['a', 'b', 'c'])
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-2-44325051df92> in <module>()
----> 1 pd.Index(['a', 'b']) == pd.Index(['a', 'b', 'c'])

/Users/dan/venvs/pandas-0.16.0/lib/python2.7/site-packages/pandas/core/index.pyc in wrapper(self, other)
     57         # technically we could support bool dtyped Index
     58         # for now just return the indexing array directly
---> 59         if is_bool_dtype(result):
     60             return result
     61         try:

/Users/dan/venvs/pandas-0.16.0/lib/python2.7/site-packages/pandas/core/common.py in is_bool_dtype(arr_or_dtype)
   2510
   2511 def is_bool_dtype(arr_or_dtype):
-> 2512     tipo = _get_dtype_type(arr_or_dtype)
   2513     return issubclass(tipo, np.bool_)
   2514

/Users/dan/venvs/pandas-0.16.0/lib/python2.7/site-packages/pandas/core/common.py in _get_dtype_type(arr_or_dtype)
   2439     elif isinstance(arr_or_dtype, CategoricalDtype):
   2440         return CategoricalDtypeType
-> 2441     return arr_or_dtype.dtype.type
   2442
   2443

AttributeError: 'NotImplementedType' object has no attribute 'dtype'

.equals() is the comparator here as is compares a whether indexes of different types can be equal as well as the elements themselves (e.g. this is dtype and value). While == is an element-wise comparison and is pretty raw. It is pretty long established to use .equals().
All that said, If you can catch errors and produce a better error message, all for it.

closed by #9872


Shaka Player CAF V3 - DASH + WIDEVINE - 6008

Good evening, in advance sorry for the inconvenience, I come for your help since I do not know what else to try and I am desperate.
Currently I have an application developed with version 2 of the Chromecast SDK and a WEB application that uses Shaka Player, I want to develop a new application using the CAF V3 version.
We are using DASH + Widevine as encode, I have no problems wanting to get the manifest file, even starting to get the video chunks.
However, when I get to the service in which I obtain the license, it responds with status 200 but the Shaka Player that comes embedded in CAF v3 responds with an error 6008 - LICENSE_RESPONSE_REJECTED.
However, the license details that I send to the player are exactly the same as those that I send in the other applications that work correctly.
Here are some images of the tests I have done and the answers I get from the API.
I reiterate my thanks to you for your support to resolve this issue.

That error means that the license response was rejected by the platform.  So it may have been malformed in some way.
licenseHandler and licenseRequestHandler are part of CAF, not Shaka Player, so we can't help you debug that.  You may want to check out https://support.google.com/chromecast/contactflow or https://stackoverflow.com/questions/tagged/Google-Cast for support on CAF.  Those both appear to be official support channels.
Since it's the response that is rejected, my guess would be that the way you're handling it may be wrong.  But we're not familiar with CAF, so I can't guess what's wrong.

We use this for our DRM license with DRM Today
/**
 * Handling license for widevine.
 * @param  {string} licenseData Current license.
 * @return {Uint8Array}             Decoded licenses.
 */
const licenseHandler = licenseData => {
  // This is the wrapped license, which is a JSON string.
  const wrappedString = fromUTF8(licenseData);
  // Parse the JSON string into an object.
  const wrapped = JSON.parse(wrappedString);
  // This is a base64-encoded version of the raw license.
  const rawLicenseBase64 = wrapped.license;
  // Decode that base64 string into a Uint8Array
  // The raw license will be fed to the Widevine CDM.
  return fromBase64(rawLicenseBase64);
};

function fromCharCode(args) {
  const max = 16000;
  let ret = '';
  for (let i = 0; i < args.length; i += max) {
    const subArray = args.subarray(i, i + max);
    ret += String.fromCharCode.apply(null, subArray);
  }

  return ret;
}

export default function fromUTF8(data) {
  if (!data) return '';

  let uint8 = new Uint8Array(data);
  // If present, strip off the UTF-8 BOM.
  if (uint8[0] === 0xef && uint8[1] === 0xbb && uint8[2] === 0xbf) {
    uint8 = uint8.subarray(3);
  }

  // http://stackoverflow.com/a/13691499
  const utf8 = fromCharCode(uint8);
  // This converts each character in the string to an escape sequence.  If the
  // character is in the ASCII range, it is not converted; otherwise it is
  // converted to a URI escape sequence.
  // Example: '\x67\x35\xe3\x82\xac' -> 'g#%E3%82%AC'
  const escaped = escape(utf8);
  // Decode the escaped sequence.  This will interpret UTF-8 sequences into the
  // correct character.
  // Example: 'g#%E3%82%AC' -> 'g#€'
  return decodeURIComponent(escaped);
}

/**
 * @param {string} str Input.
 * @returns String.
 */
export default function fromBase64(str) {
  // atob creates a "raw string" where each character is interpreted as a byte.
  const bytes = window.atob(str.replace(/-/g, '+').replace(/_/g, '/'));
  const result = new Uint8Array(bytes.length);
  for (let i = 0; i < bytes.length; i += 1) {
    result[i] = bytes.charCodeAt(i);
  }
  return result;
}

Hope this can help you
With regards
Morten


Scrolling behaviour is not always correct

What I do:

Click right mouse on a scroller
Open dev tools

Actual behavior:

Scrolling is scrolling down for some reason

Expected behavior:

Scrolling is not working

Closed, project is not maintained anymore.


Publish to NuGet

Currently, this is just some code that you can build. Not very discoverable.

Initial beta is now on nuget (as a placeholder) https://www.nuget.org/packages/KafkaClient/0.1.155-beta