Unable to login with swedish letters in nick

i was trying to connect to hub via nmdc using nick Lördag containing swedish letters using encoding CP1251. hub disconnects me with message:
handshake failed: read tcp <IP_ADDRESS>:1411->A.B.C.D:59640: i/o timeout

I'm getting a different error:
invalid name: encoding: rune not supported by encoding. ("Lördag")

This one is from the name validation - it tries to convert the name to a fallback encoding (CP1251 for that hub) and fails. This check is to make sure that NMDC clients will still see the whole user list.
Your error is different though.

but what is solution for this? some clients maybe have this nick and want to use your hub, they can not? letter ö in CP1252 is same byte as ц in CP1251. why hub can not use CP1251 (as defult) instead? like every other nmdc hubs without encoding support do > Lцrdag.

Because it won't be possible to enter with Lцrdag name. Hub will have to track that name collision as well.
I agree that this is a problem, but the only "good" and correct solution would be to not show users with UTF-8 names to users with CP1251. Or switch clients to UTF-8.

ok. so maybe hub should atleast send message to client saying that he should add this hub to favorites and select UTF-8 encoding in favorute hub settings? so he can use hub properly.

Makes sense. Although I want to try a different approach first.
As described in nmdc/1 draft, we can make a new protocol version called "nmdc/1" and make it work with only UTF-8. This will only work with ALPN, of course.
Currently, only Ncdc does full ALPN negotiation, but this is not a problem because switching NMDC<->ADC is harder than switching to a new protocol version.
So I plan to first make a few more changes to clients so they support ALPN with a single protocol properly, then make redirects to nmdcs://. Next, then most of the users are connected via TLS, we can discuss the features included in "nmdc/1" and I will update the hub and the clients to support it and force UTF-8 on it.


Highlighting for unhandled exceptions from docstrings

Thanks for your work and the awesome tool ❤
Problem
Some docstring formats support raise blocks for describing potential exceptions a function may throw. However, neither LSP (pyright) nor IDEs (PyCharm) can suggest handling these exceptions when a function is called. This limitation often leads to two bad practices:

Handling all types of errors using a broad except Exception block, which can mask specific issues and make debugging more difficult.
Completely overlooking potential exceptions, resulting in unhandled errors that can cause unexpected runtime bugs.

Is it possible to create a rule for highlighting unhandled exceptions? Most related feature in PHP Storm - https://blog.jetbrains.com/phpstorm/2017/11/bring-exceptions-under-control/#unhandled-exception
def positive_sum(a: int, b: int) -> int:
	"""
	Summarize two positive integers

	Raises:
		ValueError: when value is not positive
	"""
	if a < 0 or b < 0:
		raise ValueError('value must be positive integer')

	return a + b

positive_sum(15, 15)   <-- RUF999: Unhandled ValueError exception

try:
	positive_sum(15, 15)  <-- OK
except ValueError:
	...

try:
	positive_sum(15, 15)  <-- OK, but may be should consider case when it can be blocked as well
except Exception:
	...

But there can be problem (may be, I didn't read a whole ruff code to understand how parser and semantic work):
def validate_number(num: int) -> int:
	"""
	Validate number and return error if it's not positive

	Raises:
		ValueError: when value is not positive
	"""
	if num < 0:
		raise ValueError('value must be positive integer')

	return num

# There is no docstring
def positive_sum(a: int, b: int) -> int:
	validate_number(a)  <-- RUF999: Unhandled ValueError exception
	validate_number(b)  <-- RUF999: Unhandled ValueError exception

	return a + b

positive_sum(15, 15)  <-- RUF999: Unhandled ValueError exception

Also there could be cases when we use assert statements without docstring, like
def positive_sum(a: int, b: int) -> int:
	assert a > 0
	assert b > 0

	return a + b

# Strongly not sure if should cover such case, because assertion is supposed to find bugs by `fuzzing`
positive_sum(15, 15)  <-- RUF999: Unhandled AssertionError exception

Is it possible to implement such a rule at all? Because my concern is that is job for LSP, not for linter

Thanks for the kind words.
If I'm not mistaken than this rule shipped as part of ruff 0.5.5 but under preview mode only. See https://github.com/astral-sh/ruff/pull/11471

Thanks for the kind words.
If I'm not mistaken than this rule shipped as part of ruff 0.5.5 but under preview mode only. See #11471

I guess this rule just prevent cases when user forgets to update its docstring, but do nothing when user actually executes function with Raises block, see my example again (or look at PHPStorm blog which I've attached to at the top)

Oh yeah sorry. That's something else. I don't think we have the necessary infrastructure in place today because ruff can't do any cross-file analysis.


Can't install with go

I can't install with go

error:
go<EMAIL_ADDRESS>reading github.com/pborman/getopt/go.mod at revision v2.0.0: unknown revision v2.0.0

Sorry for getting back so late, now it should work using the latest version! Let me know any issue still occours


rez-env Fails Due to Build Metadata in Version String

I have a package that is using the following as its version string: 3.3.0+b75c1cb
This does follow the semantic versioning conventions, as you can check here
However, it causes the following exception:
Traceback (most recent call last):
  File "c:\opt\rez\lib\site-packages\rez\vendor\version\version.py", line 867, in __init__
    parser = _VersionRangeParser(range_str, make_token,
  File "c:\opt\rez\lib\site-packages\rez\vendor\version\version.py", line 669, in __init__
    raise ParseException("Syntax error in version range '%s'" % part)
rez.vendor.version.util.ParseException: Syntax error in version range '3.3.0+b75c1cb'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Program Files\Python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Program Files\Python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\opt\rez\Scripts\rez\rez-env.exe\__main__.py", line 7, in <module>
  File "c:\opt\rez\lib\site-packages\rez\cli\_entry_points.py", line 148, in run_rez_env
    return run("env")
  File "c:\opt\rez\lib\site-packages\rez\cli\_main.py", line 191, in run
    returncode = run_cmd()
  File "c:\opt\rez\lib\site-packages\rez\cli\_main.py", line 183, in run_cmd
    return func(opts, opts.parser, extra_arg_groups)
  File "c:\opt\rez\lib\site-packages\rez\cli\env.py", line 203, in command
    context = ResolvedContext(
  File "c:\opt\rez\lib\site-packages\rez\resolved_context.py", line 220, in __init__
    req = PackageRequest(req)
  File "c:\opt\rez\lib\site-packages\rez\utils\formatting.py", line 74, in __init__
    super(PackageRequest, self).__init__(s)
  File "c:\opt\rez\lib\site-packages\rez\vendor\version\requirement.py", line 156, in __init__
    self.range_ = VersionRange(
  File "c:\opt\rez\lib\site-packages\rez\vendor\version\version.py", line 871, in __init__
    raise VersionError("Syntax error in version range '%s': %s"
rez.vendor.version.util.VersionError: Syntax error in version range '3.3.0+b75c1cb': Syntax error in version range '3.3.0+b75c1cb'

Environment

OS: windows
Rez version (2.112.0
Rez python version: 3.9.13

To Reproduce

Request a package including semantic versioning build metadata. The package does not have to exist for the exception to be raised. e.g rez-env blah-0.0.1+badf00d
Observe the exception

Expected behavior
Rez should locate the package
Actual behavior
The above exception
I'm guessing that the presence of a + is being treated as a range, but should that only be the case when the character to the right is a space (rez-env blah-1+ bob-0.2.0) or a <, >' symbol (rez-env blah-1+<2 bob-0.2.0`).
So the fundamental question is, should rez cover this use case?

I think this is a use case that should be supported since it's a feature of a popular and widely used versioning scheme.
There are a couple issues that may complicate supporting this:

Versions and Package Orderers aren't currently designed to be plugin-able.
Version parsing isn't configurable, it is global across all packages and package-families.
Orderers are configurable, but not in a way that's scalable IMO, since it's configured in the global configuration, and not by package-family (indeed, there are no package-family configuration files at the moment).


#395: Set up Code Climate

Resolves #395.

👍

👍

[Current coverage][cc-pull] is 90.29%

Merging [#396][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #396   diff @@
==========================================
  Files            60         60
  Lines          2256       2256
  Methods           0          0
  Messages          0          0
  Branches        328        328
==========================================
  Hits           2037       2037
  Misses          135        135
  Partials         84         84

[][cc-pull]

Powered by Codecov. Last updated by 9585adc
[cc-base-branch]: https://codecov.io/gh/sleekbyte/tailor/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/sleekbyte/tailor/pull/396?src=pr


add sensor Beaufort based on m/s

HI,
since Beaufort is the scale news headlines and meteorology institutes use in our region for wind and warnings, would younplease consider adding a sensor for that too. You probably know about the scale, here's a quick link for references in this feature request:
https://en.wikipedia.org/wiki/Beaufort_scale
I use 2 template sensors myself, but it would be great if your integration could create it:
      windkracht:
        friendly_name: Windkracht
        value_template: >
          {% set wind = 3.6 * states('sensor.windspeed')|float %}
          {% set wind_round = wind|round(1) %}
            {% if wind <= 1 %}Bft: 0, Wind stil, {{wind_round}} km/u
            {% elif wind <= 5 %}Bft: 1, Zwak: flauwe wind, {{wind_round}} km/u
            {% elif wind <= 11 %}Bft: 2, Zwak, {{wind_round}} km/u
            {% elif wind <= 20 %}Bft: 3, Matig: lichte bries, {{wind_round}} km/u
            {% elif wind <= 28 %}Bft: 4, Matig: flauwe bries, {{wind_round}} km/u
            {% elif wind <= 39 %}Bft: 5, Vrij krachtig, {{wind_round}} km/u
            {% elif wind <= 50 %}Bft: 6, Krachtig, {{wind_round}} km/u
            {% elif wind <= 62 %}Bft: 7, Hard, {{wind_round}} km/u
            {% elif wind <= 75 %}Bft: 8, Stormachting, {{wind_round}} km/u
            {% elif wind <= 89 %}Bft: 9, Storm, {{wind_round}} km/u
            {% elif wind <= 103 %}Bft: 10, Zware storm, {{wind_round}} km/u
            {% elif wind <= 117 %}Bft: 11, Zeer zware storm, {{wind_round}} km/u
            {% else %} > 117 %}Bft: 12, Orkaan, {{wind_round}} km/u
          {%- endif %}

      wind_bft:
        friendly_name_template: >
          Bft: {{states('sensor.wind_bft')}}
        value_template: >
          {% set wind = 3.6 * states('sensor.windspeed')|float %}
            {% if wind <= 1 %}0
            {% elif wind <= 5 %}1
            {% elif wind <= 11 %}2
            {% elif wind <= 20 %}3
            {% elif wind <= 28 %}4
            {% elif wind <= 39 %}5
            {% elif wind <= 50 %}6
            {% elif wind <= 62 %}7
            {% elif wind <= 75 %}8
            {% elif wind <= 89 %}9
            {% elif wind <= 103 %}10
            {% elif wind <= 117 %}11
            {% else %} > 117 %}12
          {%- endif %}
        entity_picture_template: >
          {% set state = states('sensor.wind_bft') %}
          {% set path = '/local/weather/beaufort/' %}
          {% set ext = '.jpg'%}
            {{[path,state,ext]|join('')|lower}}

thanks for considering!

Sure, I will add this. Could also be an interesting test case to do some translations. It looks like you are German speaking, so I think I will do two new sensors - one with the Beaufort value and one with the text describing the value as you show above. I can steel the German text from above and then default to English of no translation. Now I just need to figure out how to that.

haha, almost...
no I am Dutch ;-)

Yes I did - but then I read your text more carefully. I am actually the VP for an IT company managing companies across Europe, and Holland is one of them - so I should have spotted it right away. Anyway, you saw I changed it.

Fixed in release 0.22


FlakyTestRunIncrementalBuildPod

❌ TestRunIncrementalBuildPod (15m9.17s)
  Copy catalog camel-catalog-3.8.1 from namespace default
  OLM is not available in the cluster. Fallback to regular installation.
  Camel K installed in namespace test-306a76c0-ad81-4122-95c6-47f9e39b137c
  2024-05-07T00:49:56Z	DEBUG	camel-k	Finding available platforms in namespace test-306a76c0-ad81-4122-95c6-47f9e39b137c
  2024-05-07T00:49:56Z	DEBUG	camel-k	Found active integration platform camel-k-incremental-build-pod
  Integration "javafyfbn" created
      incremental_build_test.go:99:
          Timed out after 900.001s.
          Expected
              <v1.PodPhase>:
          to equal
              <v1.PodPhase>: Running

Seen again in https://github.com/apache/camel-k/actions/runs/8977585130/job/24656597456?pr=5458

I think we have some pending PR that deals with fixing this kind of 900s timeouts in the E2E tests: https://github.com/apache/camel-k/pull/5415 @valdar is this right?


add link to outreach presentation

add link to outreach presentation https://drive.google.com/drive/folders/1qsqteFng74C7Kvm2BdIoDASWOaWaqEjN from Nov. 17

link added


Demzne 1131 copy rank over to custodians in newperiod

This PR

Renames the field total_vote_power to rank and re-uses it to store the rank of the candidate at time of newperiod
Updates daccustodian and stakevote tests to account for the above
Adds a unit test for the rank calculation to the stakevote tests

This allows the UI to display the rank data for both candidates and custodians.

This PR was reworked to include a full table migration for the custodians table in order to add the desired additional fields.
Deployment instructions:

Compile the daccustodian contract without any flags using yarn build and deploy
Run daccustodian::migrate1 for all dac ids. This moves all custodians from the custodians table to the temporary custodians2 table.
Compile the daccustodian contract with the MIGRATION_STAGE_2 flag like this: yarn build -DMIGRATION_STAGE_2 and deploy.
Run daccustodian::migrate2 for all dac ids. This moves all custodians from the temporary custodians2 table to the  custodians table.

At the next newperiod the additional fields will be filled with meaningful values.


Address #720 review comments

This should fix version detection for cilium/cilium CI images.
Ref. https://github.com/cilium/cilium-cli/pull/720#discussion_r813626778
See individual commits for details.

Thanks a lot for this PR. The original PR was merged before I could comment about using the versioncheck package.


Inconsistent Footer Design Across Pages on Responsive Layouts

Describe the bug
The footer design and content are inconsistent across different pages when viewed on responsive layouts. This inconsistency results in a non-uniform user experience, particularly on mobile and tablet views. Please standardize the footer appearance across all pages to ensure a cohesive design.
To Reproduce
Steps to reproduce the behavior:

Open the website on a mobile or tablet device.
Navigate to various pages.
Observe the footer on each page and note the inconsistencies in layout or content.

Expected behavior

The footer should appear consistent in terms of structure, content, and styling across all pages.
The footer should adapt seamlessly to different screen sizes.

Screenshots

Additional context
Implement a shared footer component or ensure the same footer structure is applied across all pages in responsive views.

hey @ANSHIKA-26  I was thinking of fixing
This post-tags div sizing:

and This carousel slider alignment:

let me know what do you think


[Hub Generated] Review request for Microsoft.ContainerService/aks to add version stable/2023-10-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM (Control Plane) API Specification Update Pull Request
PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

[1] ARM review queue (for merge queues, see [4])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] https://aka.ms/azsdk/support/specreview-channel
[3] List of SDK breaking changes approvers in pinned Teams announcement
[4] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[ ] New API version. (Such PR should have been generated with OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[ ] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to Step 2, "ARM Review", for this PR.

Breaking changes review (Step 1)

If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

ARM API changes review (Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
Getting help

For general PR approval workflow, see the diagram at the top of this comment.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

    "ipTagType": {

enum?

Refers to: specification/containerservice/resource-manager/Microsoft.ContainerService/aks/stable/2023-10-01/managedClusters.json:3358 in 1e9488b. [](commit_id = 1e9488b3013bd7bad9c1c95328a264aa3e79e811, deletion_comment = False)

/pr RequestMerge

There is a trivial merge conflict on file custom-words.txt. Could you please help resolve it? Thanks

@FumingZhang you can resolve the conflict and come back with the merge request, thanks

/pr RequestMerge


Crash with a UTF-8 error

Here's a small reproducer which works reliably for me.
In a GHCI session,
λ> :set -XOverloadedStrings
λ> import qualified Data.Text as T
λ> import qualified Text.Regex.Pcre2 as Pcre2
λ> length $ Pcre2.captures ".*" (T.replicate 10000 "a\n")
*** Exception: pcre2: UTF-8 error: byte 2 top bits not 0x80

I'm using the latest version, pcre2-2.2.1.

Confirmed.  Thanks for the report.
In fact, running the test a bunch of times I randomly get the following 3 errors:
pcre2: UTF-8 error: byte 2 top bits not 0x80
pcre2: UTF-8 error: isolated byte with 0x80 bit set
malloc(): unsorted double linked list corrupted
Almost certainly related to https://github.com/sjshuck/hs-pcre2/issues/34.

I would also expect the answer to be 1 if PCRE2 is anything like Ruby/Oniguruma:
/.*/.match("a\n" * 10000)
# #<MatchData "a">

but in those uncommon instances where the test completes, it produces 20001.  That might be a separate bug though.

According to PCRE docs, . is any char except newline, so I would expect it to be 10000. But yes, this is a separate issue altogether.
λ> Pcre2.captures "a+" "abab" :: [NonEmpty Text]
["a" :| [],"a" :| []]
λ> Pcre2.captures "a*" "abab" :: [NonEmpty Text]
["a" :| [],"" :| [],"a" :| [],"" :| [],"" :| []]


UI Enhanced

When we scroll the screen
Icon to toggle dark mode always stays on the top, it doesn't disappear as the screen moves
Please fix that

here are the screenshots for your reference

@UjjwalSharma01 it was already like that but no problem i will change that. Give me a sec plz

@UjjwalSharma01 can you check if this issue has a hacktoberfest label cause its not showing in the hacktober site

@UjjwalSharma01 can you check if this issue has a hacktoberfest label cause its not showing in the hacktober site

Issue has but lemme add tag on this PR too don't worry it sometimes takes time, happened with me also

@UjjwalSharma01 it got accepted just now.

@UjjwalSharma01 can you merge this PR. I will make another issue for fixing the logo. is it ok with you? It will help me do more PRs

@UjjwalSharma01 can you merge this PR. I will make another issue for fixing the logo. is it ok with you? It will help me do more PRs

Sure just a min plz

@Milind-Palaria you can create more issues now


Tabbar color in Linux

I'm trying out KDE Plasma and obviously want to use your style there as well.
I checked that all the requires keys are set, and I've enabled the Australis theme but it looks all bland and solid-color.

The only time I notice anything is when hovering above a tab:

What am I missing?

I don't see any issues on my side. Although I checked on Firefox 107 Beta. Firefox 106 update has not come to me yet.

I'll try and see once you get 106 :-)

I checked several screenshots with color capture. I don't think this is a bug. It's just that the color of the tab bar is almost the same as the color of the toolbar. Try turning on the shadow or change the theme to something else. If you have KDE 5.25+, then you can activate the accent color as on Win10/11.

No interface issues found. (Firefox 106.0.1 + WaveFox v<IP_ADDRESS>)


Add early stopping

How should I try to implement early stopping and avoid overfitting? Looks like eval.py takes the latest model and not the best one. I'm studying this but I'm not sure about where it should go.
thanks and keep up the excellent work!

This is more of a general Tensorflow question so I recommend you post it on Stackoverflow to get a detailed answer with code.
You need to wrap the model code into tf.contrib.learn.Estimator and then you can use the ValidationMonitor from the tutorial you linked to.

@carloandrea Did you end up posting this question to Stackoverflow or solving it? I am interested in the same thing.


python312Packages.google-cloud-language: 2.14.0 -> 2.15.1

Changelog: https://github.com/googleapis/google-cloud-python/blob/google-cloud-language-v2.15.1/packages/google-cloud-language/CHANGELOG.md

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 356498

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.google-cloud-language
python311Packages.google-cloud-language.dist
python312Packages.google-cloud-language
python312Packages.google-cloud-language.dist


Apple home

There’s surely something I’m missing here but in the homebridge hub under the accessories tab I  have my little icon to turn off the tv and on but i can’t get it on the Apple home app only via hitting the add accessory option then selecting can’t scan my tv then appears but it asks for the 8 digit code Is this an issue or how it is
Supposed to be? or perhaps just a bad initial setup? Hope someone can please help me.

Interestingly experiencing the same after updating node and some plugins today. I wanted to reconfigure my TV for different reasons. Now I face this issue.

Multiple restarts resolved it. Maybe need to give HomeKit some time before accepting the code again.

That seemed to do it! it’s in my home now Thank you.


Fixes error when before is called without a target

Quick fix for when before is called without a valid target.
Fixes #241.

@lizell For us to be able to accept your merge, please can you follow these guideline in the wiki and confirm you have read the guide please
Contributor License Agreement
In order for us to accept pull-requests from a new contributor, the contributor must indicate in their first Pull Request that they accept and agree to be bound by the terms of the IBM Contributor License Agreement that can be found here:
https://github.com/RuntimeTools/appmetrics/raw/master/IBM-RuntimesTools-V1.0.1-CLA.odt

@lizell - what's the use case for this?  Seems to me that if the target is null then the probe has been written incorrectly so it's good to throw an error.  I don't like the idea of just exiting silently.

closing as no response from raiser


Sample code not play video

I am trying to play video but it display following screen.

Hello!
Sorry for such a long delay. Seems that this issue was forgotten.
Closing it for now, as a long time has passed. But feel free to reopen if you still have questions.
Please note that this iOS FPS app is only intended to be used for integration purposes with Axinom DRM. Before using it, make sure you have set up an Axinom DRM demo account. Registration can be performed here: https://drm.axinom.com/evaluation-account/.


Data only container backend

Some times it could be simple to use data only container inside an Rancher service / stack, but sidekicks not supports shared data only containers as needed (each app container get it's own data container and so the volume isn't shared).
Is it possible to build a data only container backend additional to solutions like xfs or dm?

Convoy saves volumes inside the docker volumes directory with names instead of hashes. Docker volumes gets deleted if unused, but convoy volumes are persistent.
So I close that old issue.


Incorrect link for UpdateAssessment structure

https://docs.microsoft.com/en-us/windows/desktop/sysinfo/update-assessor-structures
On this page, both of the links to the structure descriptions take you to the OSUpdateAssessment structure, but the second one should actually be taking you to this page: https://docs.microsoft.com/en-us/windows/desktop/api/waasapitypes/ns-waasapitypes-tagupdateassessment

@GrantMeStrength Can you please take a look at this issue?

Now fixed.


Redirect on changed user and org name

After a user's or organization's name is changed, redirect the old name to the new name until a new user/org is created with the old name.
Similar to #807
Fix #9531

Do not change translation files except English, Drone CI will take care of it when merged

Should we perhaps redirect only repositories like GitHub does? Currently I see that it also redirect on user profile.

In addition I would definitely add logic like GitHub has where notice is printed about new link on Git action.

Should we perhaps redirect only repositories like GitHub does? Currently I see that it also redirect on user profile.

I see now how Github does it.  I guess the question would be then, do we want to match Github, or do we want to have more features?  Do you see any major issues for this that wouldn't arise for repository redirects?

No specific security concerns that I can think of.
The only reason i could think of not to do it would be making others aware quicker that name has changed.

@AndrewBezold whats the state?

Codecov Report

Merging #11649 (cd4c6eb) into master (91e59a6) will decrease coverage by 0.02%.
The diff coverage is 24.57%.

@@            Coverage Diff             @@
##           master   #11649      +/-   ##
==========================================
- Coverage   41.76%   41.74%   -0.03%
==========================================
  Files         748      749       +1
  Lines       80103    80197      +94
==========================================
+ Hits        33458    33478      +20
- Misses      41123    41188      +65
- Partials     5522     5531       +9

Impacted Files
Coverage Δ

models/org.go
72.23% <0.00%> (-0.29%)
:arrow_down:

modules/context/context.go
51.48% <0.00%> (-3.84%)
:arrow_down:

modules/context/org.go
37.73% <0.00%> (-2.27%)
:arrow_down:

modules/context/repo.go
53.90% <0.00%> (-0.61%)
:arrow_down:

routers/api/v1/user/user.go
44.44% <0.00%> (-5.56%)
:arrow_down:

routers/repo/http.go
41.81% <0.00%> (-0.54%)
:arrow_down:

routers/user/profile.go
47.29% <0.00%> (-0.71%)
:arrow_down:

models/user.go
53.03% <11.11%> (-0.27%)
:arrow_down:

routers/api/v1/api.go
78.39% <11.76%> (-1.11%)
:arrow_down:

routers/api/v1/user/key.go
51.88% <28.57%> (-1.52%)
:arrow_down:

... and 13 more

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 9872b8d...cd4c6eb. Read the comment docs.

Codecov Report

Merging #11649 (cd4c6eb) into master (91e59a6) will decrease coverage by 0.02%.
The diff coverage is 24.57%.

@@            Coverage Diff             @@
##           master   #11649      +/-   ##
==========================================
- Coverage   41.76%   41.74%   -0.03%
==========================================
  Files         748      749       +1
  Lines       80103    80197      +94
==========================================
+ Hits        33458    33478      +20
- Misses      41123    41188      +65
- Partials     5522     5531       +9

Impacted Files
Coverage Δ

models/org.go
72.23% <0.00%> (-0.29%)
:arrow_down:

modules/context/context.go
51.48% <0.00%> (-3.84%)
:arrow_down:

modules/context/org.go
37.73% <0.00%> (-2.27%)
:arrow_down:

modules/context/repo.go
53.90% <0.00%> (-0.61%)
:arrow_down:

routers/api/v1/user/user.go
44.44% <0.00%> (-5.56%)
:arrow_down:

routers/repo/http.go
41.81% <0.00%> (-0.54%)
:arrow_down:

routers/user/profile.go
47.29% <0.00%> (-0.71%)
:arrow_down:

models/user.go
53.03% <11.11%> (-0.27%)
:arrow_down:

routers/api/v1/api.go
78.39% <11.76%> (-1.11%)
:arrow_down:

routers/api/v1/user/key.go
51.88% <28.57%> (-1.52%)
:arrow_down:

... and 13 more

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 9872b8d...cd4c6eb. Read the comment docs.

@zeripath ping ;)

@zeripath ping ;)

@zeripath :point_up:

@zeripath :point_up:

🚀

🚀


use new Context API from React 16.3

bumps peer and dev deps to React 16.3
uses new Context API in <Provider> and <Helmet>
wraps test renders in <StrictMode>
2 client tests have race conditions that might be too hard to mock, for now

bump

bump


ExecutePath fails on Ada: 'JointTrajectory' object has no attribute 'GetDescription'

I get an error when executing a path with Ada. The error goes away when I comment out https://github.com/personalrobotics/prpy/blob/master/src/prpy/base/robot.py#L392.
In the following trace, the path being sent was generated by robot.PlanToConfiguration(dofs) for some hand selected values.
@mkoval @jeking04
In [4]: robot.ExecutePath(path)
[INFO] [prpy.base.robot:robot.py:377]:do_execute: Post-processing path with 2 waypoints.
[WARNING] [prpy.base.robot:robot.py:314]:do_postprocess: Post-processing smooth paths is not supported. Using the default post-processing logic; this may significantly change the geometric path.
[INFO] [prpy.planning.base:base.py:236]:plan: Sequence - Calling planner "ParabolicSmoother".
[INFO] [prpy.planning.base:base.py:245]:plan: Sequence - Planning succeeded after 0.201 seconds with "ParabolicSmoother".
[INFO] [prpy.base.robot:robot.py:387]:do_execute: Post-processing took 0.397 seconds and produced a path with 4 waypoints and a duration of 1.961 seconds.
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
/home/lcv/pr_catkin/src/ada/adapy/scripts/console.py in <module>()
----> 1 robot.ExecutePath(path)

/home/lcv/pr_catkin/src/prpy/src/prpy/base/robot.pyc in ExecutePath(self, path, defer, executor, **kwargs)
    402             return wrap_future(executor.submit(do_execute))
    403         elif defer is False:
--> 404             return do_execute()
    405         else:
    406             raise ValueError('Received unexpected value "{:s}" for defer.'.format(str(defer)))

/home/lcv/pr_catkin/src/prpy/src/prpy/base/robot.pyc in do_execute()
    390             with Timer() as timer:
    391                 exec_traj = self.ExecuteTrajectory(traj, defer=False, **kwargs)
--> 392             SetTrajectoryTags(exec_traj, {Tags.EXECUTION_TIME: timer.get_duration()}, append=True)
    393             return exec_traj
    394

/home/lcv/pr_catkin/src/prpy/src/prpy/util.pyc in SetTrajectoryTags(traj, tags, append)
    312
    313     if append:
--> 314         all_tags = GetTrajectoryTags(traj)
    315         all_tags.update(tags)
    316     else:

/home/lcv/pr_catkin/src/prpy/src/prpy/util.pyc in GetTrajectoryTags(traj)
    286     import json
    287
--> 288     description = traj.GetDescription()
    289
    290     if description == '':

AttributeError: 'JointTrajectory' object has no attribute 'GetDescription'

Is the exec_traj returned by the controller a ROS joint trajectory? It's
supposed to be converted to an openrave trajectory somewhere.
On Jul 20, 2015 3:07 PM, "Laura Herlant"<EMAIL_ADDRESS>wrote:

I get an error when executing a path with Ada. The error goes away when I
comment out
https://github.com/personalrobotics/prpy/blob/master/src/prpy/base/robot.py#L392
.
In the following trace, the path being sent was generated by
robot.PlanToConfiguration(dofs) for some hand selected values.
@mkoval https://github.com/mkoval @jeking04
https://github.com/jeking04
In [4]: robot.ExecutePath(path)
[INFO] [prpy.base.robot:robot.py:377]:do_execute: Post-processing path with 2 waypoints.
[WARNING] [prpy.base.robot:robot.py:314]:do_postprocess: Post-processing smooth paths is not supported. Using the default post-processing logic; this may significantly change the geometric path.
[INFO] [prpy.planning.base:base.py:236]:plan: Sequence - Calling planner "ParabolicSmoother".
[INFO] [prpy.planning.base:base.py:245]:plan: Sequence - Planning succeeded after 0.201 seconds with "ParabolicSmoother".
[INFO] [prpy.base.robot:robot.py:387]:do_execute: Post-processing took 0.397 seconds and produced a path with 4 waypoints and a duration of 1.961 seconds.
AttributeError                            Traceback (most recent call last)
/home/lcv/pr_catkin/src/ada/adapy/scripts/console.py in ()
----> 1 robot.ExecutePath(path)
/home/lcv/pr_catkin/src/prpy/src/prpy/base/robot.pyc in ExecutePath(self, path, defer, executor, **kwargs)
402             return wrap_future(executor.submit(do_execute))
403         elif defer is False:
--> 404             return do_execute()
405         else:
406             raise ValueError('Received unexpected value "{:s}" for defer.'.format(str(defer)))
/home/lcv/pr_catkin/src/prpy/src/prpy/base/robot.pyc in do_execute()
390             with Timer() as timer:
391                 exec_traj = self.ExecuteTrajectory(traj, defer=False, **kwargs)
--> 392             SetTrajectoryTags(exec_traj, {Tags.EXECUTION_TIME: timer.get_duration()}, append=True)
393             return exec_traj
394
/home/lcv/pr_catkin/src/prpy/src/prpy/util.pyc in SetTrajectoryTags(traj, tags, append)
312
313     if append:
--> 314         all_tags = GetTrajectoryTags(traj)
315         all_tags.update(tags)
316     else:
/home/lcv/pr_catkin/src/prpy/src/prpy/util.pyc in GetTrajectoryTags(traj)
286     import json
287
--> 288     description = traj.GetDescription()
289
290     if description == '':
AttributeError: 'JointTrajectory' object has no attribute 'GetDescription'
—
Reply to this email directly or view it on GitHub
https://github.com/personalrobotics/prpy/issues/160.

@herlant Has this been resolved on ADA?


fix component as a tool

added class variable is_tool that enable the component to be tool. added basic bool input to turn on/off as tool.
TODO to solve But the toggle doesnt update is_tool as true or changes the output.

This PR is Set to draft:
Multiple PR created solving individual issues.


Can activity or fragment have multiple viewmodel

i have this dashboard activity which shows advertismentList and other Property List
I have made AdvertisementModel which consult with AdvertismentRepository for data, in similar way
I have made PropertyViewModel which consult with PropertyRepository for data
Now,In DashBoardActivity
*First Approach
i set both view model(property and advertisement) with ViewModelProviders --> is this right ??
*Second Approach
should i have to make separate DashBoardActivityViewModel and then it consist of these two dependent component viewmodel in its fields,property and advertisement.

which one is the better approach ? currently i am using first approach

Sure.
You can have as many ViewModels as you want aslong as you bind them to a livecycle (activity, fragment).

@mynote
Thank you so much :)

Should you?


breaks webmock?

I like using WebMock in my tests.
When trying to write my own test against a shrine uploader using shrine-url, and using WebMock to mock out the HTTP request that shrine-url would be doing...
I get a mysterious error. I haven't yet been able to isolate this error into something that only uses down and not shrine-url, is why I'm reporting it here. It's really quite mysterious and I'm not sure what's going on, when I try to isolate it into a simple reproduction case, i end up not being able to reproduce.
Here's the error/stack trace as reported by my rspec, in case it gives you any ideas
     Failure/Error: response.connection.close unless @client.persistent?

     NoMethodError:
       undefined method `close' for #<HTTP::Response::Streamer:0x00007ff14aa89380>
       Did you mean?  clone
     # /Users/jrochkind/.gem/ruby/2.5.3/gems/down-4.7.0/lib/down/http.rb:109:in `stream_body'
     # /Users/jrochkind/.gem/ruby/2.5.3/gems/down-4.7.0/lib/down/chunked_io.rb:282:in `each'
     # /Users/jrochkind/.gem/ruby/2.5.3/gems/down-4.7.0/lib/down/chunked_io.rb:282:in `block in chunks_fiber'

(Does this also suggest it's using http_rb instead of net_http?  Ah, I see, even though down says net-http is the default, shrine-url makes http_rb it's default. Any particular rationale?)

Trying to get a better stack trace, but having trouble, maybe down's use of fibers/threads is confusing things.

Yeah, the WebMock monkey patches for HTTP.rb still appears to be broken with later versions (https://github.com/httprb/http/issues/212).
The main reason I switched shrine-url to HTTP.rb backend for the user is because HTTP.rb has lower memory usage, which is beneficial when uploading large files. Also, when you configure HTTP.rb with a timeout, that's the actual timeout you get (Net::HTTP will retry the request once automatically).
The reason for me was that I didn't want to deal with Net::HTTP anymore.  And I think that HTTP.rb is a great library, so I think it's a better choice. But if you want to use the Net::HTTP backend because of its WebMock support, and you run into issues, I'll still help you out 😃
Net::HTTP will probably stay the default backend for Down, just because I don't want to force users to install more gems (because in that case I'd need to add http as a runtime dependency). Not really a great reason, but I'm trying to strike a balance.

Hmm. That commit is in webmock, odd it's not fixed.  I haven't actually been able to reproduce the problem even with just down -- I can only reproduce with shrine-url, I can't figure out what shrine-url is doing to trigger the problem. But that commit and test give me a little bit of orientation, thanks.  Any more orientation welcome!
I think shrine-url ends up using http_rb in an uncommon way, so other people aren't running into the bug in webmock, if that's what it is.
I'd really like to use http_rb, but testing my app reasonably without webmock is challenging. Hmm.

That fix and test are still in latest version of webmock... so it must be something else?  I will try to investigate a bit more.
I'm confused about why I can't get any more of a stack trace than those three lines, to see exactly what shrine-url is doing, to be able to write an isolated reproduction of the problem.  I guess it's got something to do with fibers, which shrine-url uses (?), that limits my backtrace?
If you had any hints for where in the code actually calls things, prior to the stack trace I'm able to get, that would also be helpful.

That's weird, shrine-url is using Down in the simplest way. Do you think you could come up with self-contained example that reproduces the error? Something like
require "shrine/storage/url"
require "webmock"

storage = Shrine::Storage::Url.new

storage.download(some_url)
# or
storage.open(some_url).read

Aha!  I was accidentally using WebMock 2.0, before your included fix.  Everything is good with Webmock 3.0.
Not sure why I was having trouble reproducing, but no longer necessary to find out.
Sorry for the diversion, thanks for posting the commit that fixed, that led me to figure out why the fix wasn't in my app. Sorry thanks!


[question] LocalAddress in custom predicate

Hi,
is there way how to obtain local address in custom route predicate? I need to check on which interface was the request accepted and then perform action. I mean something like ServletRequest#getLocalAddr
Thanks

You have access to the server web exchange. See the webflux javadocs for details.
https://github.com/spring-cloud/spring-cloud-gateway/blob/master/spring-cloud-gateway-core/src/main/java/org/springframework/cloud/gateway/handler/predicate/RemoteAddrRoutePredicateFactory.java

@spencergibb  Thanks, I've seen RemoteAddrRoutePredicateFactory and I'm already working with exchange.getRequest().getURI().getHost(). But there's always url of "balancer".
If I have two virtual addresses <IP_ADDRESS> and <IP_ADDRESS> and there is LB sitting before them at address some.address.com can I have one of 192* somehow?

I don't know. Have you looked at the javadocs? @rstoyanchev any ideas?

I think that ugly hack with netty's HttpServerRequest could be
if ((((AbstractServerHttpRequest) exchange.getRequest()).getNativeRequest()) instanceof HttpServerRequest) {
    final InetSocketAddress localAddress = ((HttpServerRequest) (((AbstractServerHttpRequest) exchange.getRequest()).getNativeRequest())).hostAddress();
    logger.debug("!!! Hostname {}, HostString {}, Address {}",
            localAddress.getHostName(),
            localAddress.getHostString(),
            localAddress.getAddress()
    );
}

This is beyond the scope of spring cloud gateway. Please open an issue in spring framework


Allow multi-line f-string with format spec

Summary
Reported at https://github.com/python/cpython/issues/110259
Test Plan
Add test cases for the fix and update the snapshots

Current dependencies on/for this PR:

main

PR #7787   👈

This comment was auto-generated by Graphite.

The CPython PR has been merged: https://github.com/python/cpython/pull/110271

Friendly reminder to review the discussion in https://github.com/python/cpython/issues/110259 so you end implementing the same as CPython 😉

Friendly reminder to review the discussion in python/cpython#110259 so you end implementing the same as CPython 😉

I did, thanks for the heads up! :)

Actually, there's more to it for single-quoted f-string that I missed (sorry!). The newline should basically end the lexing of format spec which we currently don't.

That's why I suggested to double check :)

Hehe, I must've missed that part 😅

I'm going to try a couple solutions and look at the benchmarks, please bear with me :)

Update after the first review is that the following case will now raise a syntax error as it should:
f'__{
	x:a
		b
}__'

The reason it didn't earlier is because we didn't end the format spec lexing after we reached the newline (after a). Now, we do which means the lexer will emit a Name token for b for which the parser will raise an unexpected token error.


[1.0.2b2] Unable to upload Self Service Icons

Hi,
Currently unable to upload Self Service Icons, here is the error I'm receiving.
All other functionality is working as intended.
'self_service_icon': u'MSTeams.png',
JSSImporter: Found file: /Users/administrator/Library/AutoPkg/RecipeRepos/#####/MSTeams/MSTeams.png
Response Code: 404	Response:  The origin server did not find a current representation for the target resource or is not willing to disclose that one exists. https://#####:8443//JSSResource/fileuploads/policies/id/587.
Failed.
Error in #####.MSTeams: Processor: JSSImporter: Error: Response Code: 404	Response:  The origin server did not find a current representation for the target resource or is not willing to disclose that one exists. https://#####:8443//JSSResource/fileuploads/policies/id/587.
This is running as verbose with -vv
Thanks!

@qu1gl3s Are you using Jamf Cloud? I wonder if this is a delay issue where it is expecting to find policy 587 but it hasn't yet been replicated to the cloud node that it's attempting to upload the icon to. We're seeing a few issues with this.

I believe this is a duplicate of #81

Hi Graham,
I'm not a Jamf Cloud customer so I can't see any reason so far that could explain the issue at hand.
Thanks!

OK, is this a generic issue with all icons or just the MSTeams icon? Have you tried running it on more than one Mac?

Yes, I've tried running it on more than one Mac and all icons are affected, including existing ones.

On prem or a cloud DP e.g. S3?

On-prem distribution point.

Hi,
I've run into this issue recently also. For me, icons failed to upload for an On-prem distribution point yet uploaded without any trouble to a Cloud based DP.
I still have both instances in a testing environment, the Cloud based DP is a Jamf 10.13 Beta cloud instance.
I was also using 1.0.2b2, however, I can test with 1.0.2b3 shortly.

Ok, have tried this and 1.0.2b3 has the same issue with On-prem DP for me.
All good if manually uploading the icon and using STOP_IF_NO_JSS_UPLOAD to continue even if the package uploaded ok earlier.

Hi @perryk you say this affected you recently, could that be related to a Jamf Pro update? I started having some icon upload issues from last week using regular curl (not JSSImporter issues) where the icon is corrupted, around the same time I upgraded to 10.12. It’s probably unrelated but worth an ask!

Sorry, this is the first time I have stood up Autopkg/JSS Importer so actually I have only tested on 10.13 beta for On-Prem and 10.12 for Cloud.

Just confirming the issue is still occurring in 1.0.3.
Not sure whether there is an issue with the API accepting the image.

Having the same issue in 1.0.5, cloud distribution point.

@aluminummonster00 Jamf Cloud or some other cloud? Distribution point is kind of irrelevant for icons.

Self hosted in AWS, Jamf Pro 10.15.1.

Can you provide a full output from autopkg -vvvv SomeRecipe.jss (removing any sensitive passwords etc)?

apparently this issue gets fixed in 1.0.5 but still exists in 1.0.3. Autopkgr doesn't apparently see 1.0.5 as being available yet.
The problems seems to be that it is looking for an icon and if it doesn't exist, it errors.
two different macs, two different JSS's
Failure - Policy exists but has no icon:
SSImporter: Looking for Policy template file NoMADAutoInstallPolicy.xml...
JSSImporter: Found file: /Users/autopkg-admin/Library/AutoPkg/RecipeRepos/repo/NoMAD/NoMADAutoInstallPolicy.xml
GET https://my.jss.com:8443/JSSResource/policies/name/NoMAD: Success.
JSSImporter: Setting policy to 'Install' package.
PUT https://my.jss.com:8443/JSSResource/policies/id/5658: Success.
Retrieving data from JSS...
GET https://my.jss.com:8443/JSSResource/policies/id/5658: Success.
GET https://my.jss.com:8443/JSSResource/policies/name/NoMAD: Success.
JSSImporter: Policy ID '5658' verified on server
GET https://my.jss.com:8443/JSSResource/policies/name/NoMAD: Success.
JSSImporter: Policy 'NoMAD' updated.
JSSImporter: Looking for Icon file NoMAD.png...
JSSImporter: Found file: /Users/autopkg-admin/Library/AutoPkg/RecipeRepos/repo/NoMAD/NoMAD.png
JSSImporter: Icon name in existing policy: None
Response Code: 404      Response:  The origin server did not find a current representation for the target resource or is not willing to disclose that one exists. https://my.jss.com:8443//JSSResource/fileuploads/policies/id/5658.
Failed.
Success - Policy exists and has icon:
JSSImporter: Looking for Policy template file NoMADAutoInstallPolicy.xml...
JSSImporter: Found file: /Users/engh/Github/autopkg/NoMAD/NoMADAutoInstallPolicy.xml
GET https://my.jss.com:8443/JSSResource/policies/name/NoMAD: Success.
JSSImporter: Setting policy to 'Install' package.
PUT https://my.jss.com:8443/JSSResource/policies/id/1256: Success.
Retrieving data from JSS...
GET https://my.jss.com:8443/JSSResource/policies/id/1256: Success.
GET https://my.jss.com:8443/JSSResource/policies/name/NoMAD: Success.
GET https://my.jss.com:8443/JSSResource/policies/name/NoMAD: Success.
JSSImporter: Policy 'NoMAD' updated (ID: 1256).
JSSImporter: Looking for Icon file NoMAD.png...
JSSImporter: Found file: /Users/engh/Github/autopkg/NoMAD/NoMAD.png
JSSImporter: Icon matches existing icon, moving on...

That's not an error. If the recipe refers to an icon file and it is not accessible in the RECIPE_SEARCH_PATH, it will correctly error out.

Just want to mention that the issue is still occurring in 1.2.2
JSSImporter: Looking for Icon file Mendeley Reference Manager.png... JSSImporter: Found file: /AUTOPKG/CACHE/LOCATION/MendeleyReferenceManager/Mendeley Reference Manager.png JSSImporter: Icon name in existing policy: None Traceback (most recent call last):   File "/Library/AutoPkg/autopkglib/__init__.py", line 658, in process     self.env = processor.process()   File "/Library/AutoPkg/autopkglib/__init__.py", line 465, in process     self.main()   File "/Library/AutoPkg/autopkglib/JSSImporter.py", line 1388, in main     self.handle_icon()   File "/Library/AutoPkg/autopkglib/JSSImporter.py", line 743, in handle_icon     icon.save()   File "/Library/AutoPkg/JSSImporter/jss/misc_endpoints.py", line 221, in save     error_handler(PostError, response)   File "/Library/AutoPkg/JSSImporter/jss/tools.py", line 96, in error_handler     raise exception jss.exceptions.PostError: Response Code: 404	Response:  The origin server did not find a current representation for the target resource or is not willing to disclose that one exists. https://JAMFPRO//JSSResource/fileuploads/policies/id/824.   File "/Library/AutoPkg/autopkglib/__init__.py", line 658, in process     self.env = processor.process() Response Code: 404	Response:  The origin server did not find a current representation for the target resource or is not willing to disclose that one exists. https://JAMFPRO/JSSResource/fileuploads/policies/id/824. Failed.


Feature request: Upgrade to Angular 1.4

I might provide a PR but I can't promise anything.

Thanks @twogood it would be great


updated globals.py

measure unit rendering looks better in the results page on the summary table

Pull Request Test Coverage Report for Build 548124877

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.525%

Totals

Change from base Build 540092146:
0.0%

Covered Lines:
739

Relevant Lines:
861

💛  - Coveralls

This will not affect how units are rendered in graphics, right?

This will not affect how units are rendered in graphics, right?

No, I have created new dictionaries in globals just for the html rendering :)

ok, we should try to keep the globals.py as small as possible, otherwise adding e.g. a new metric becomes very tedious.


Fix/34 editor context after section swap

Coverage decreased (-0.4%) to 49.664% when pulling 0538e3745d54d1742074be6af02b2107b2d3fc24 on fix/34-editor-context-after-section-swap into 5fd29a005e0280a45e158c0344a0c89ec6268c0d on master.

Coverage decreased (-0.4%) to 49.664% when pulling 0538e3745d54d1742074be6af02b2107b2d3fc24 on fix/34-editor-context-after-section-swap into 5fd29a005e0280a45e158c0344a0c89ec6268c0d on master.

Coverage decreased (-0.8%) to 49.333% when pulling e791ecfb91c91ef25e5d69a39e10f3ec5c434b6b on fix/34-editor-context-after-section-swap into 5fd29a005e0280a45e158c0344a0c89ec6268c0d on master.


Add LDAP connector groups implementation

The first commit cleans up the LDAP connector a bit. A follow up one will add the groups implementation.
Cleanup includes:

Remove some unlikely to be used fields to help configurability.

Combined "serverHost" and "serverPort" into "host"
Remove "timeout" (just default to 30 seconds).
Remove "maxIdleConn" will add it back if users feel the need
to control the number of cached connections.
Remove "trustedEmailProvider" (just always trust).
Remove "skipCertVerification" you can't make this connector
ingore TLS errors.

Fix configs that don't search before bind (fixes #481).
Add more examples to Documentation
Refactor LDAPPool Acquire() and Put() into a Do() function which
always does the flow correctly.
Added more comments and renamed some functions.

Groups implementation added. Need to figure out how to have this work with refresh tokens (hopefully without too much refactoring).

This PR has been changed to just include the cleanups to the LDAP connector.
New docs here: https://github.com/ericchiang/dex/blob/c629cfaf1d6298fe0f532f80e48a4c852c86cdac/Documentation/connectors-configuration.md#ldap-connector
Ready for review.

Woops, left some docs unfinished.

Ready for review

found a few minor things, otherwise LGTM

:+1: updated


Buffering two streams from the same client

If I configure threadfin to use buffering (either VLC or ffmpeg) and play a single stream from a client, it works great. If i start another stream for a different channel both channels will play for a bit, but then the 1st channel i played will stops, the second stream continues to play.
If i turn off buffering and repeat the above step, both streams play without issue on the same client.
I want to use buffering because it does seem to help with occasional stream hiccups, but I always want to be able to play more than one stream at the same time from a client.

Please join the discord for support. This is an issue with your setup


Catch Exception when GCM is not available

should fix #487
replaces #490

I'm declining this PR. I'm going to handle this directly in the isSupported() method.


Emit parsing errors as diagnostic records

PR Summary
Fixes #1041
Emit parsing errors as diagnostic records.
I'm attempting to bring analyzer into a more lint like behavior which treats parsing/syntax errors as just another type of diagnostic.
Currently, errors during parsing are written as error records, but they should probably just be diagnostic records.  This PR creates a new diagnostic record type and emits errors found during parsing into the result stream. This is far more useful to downstream readers (such as VSCode/EditorServices) because there's a single stream for information about the script. Rule violations and syntax errors.
I have also removed the current limitation of any time there are more than 10 parse errors, the analyzer simply returns, which seems to me to be pretty undesirable. We will now emit diagnostic records for all parse errors.
PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[ ] Change is not breaking
[ ] Make sure all .cs, .ps1 and .psm1 files have the correct copyright header
[X] Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

@bergmeister - suggestions for documentation additions?

Looks very nice. I also checked how it looks with the integration into VS-Code and it looks so much better

@JamesWTruher How would people suppress it in vscode (where no command line exists and most people do not use settings files)

How would people suppress it in vscode (where no command line exists and most people do not use settings files)

The red squiggly in VSCode comes from PSES, not PSScriptAnalyzer, because those are legitimate parse errors. I'm not sure if we currently have a way of suppressing them. Installing the PowerShell extension and then suppressing parse errors would be like buying a fighter jet and ripping out the flight computer.

@rjmholt Thanks for the info, that's good to know, I agree, lets keep it how it is, PSES can keep its flight computer ;-)

LGTM

I've looked at implementing support for [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute] and while it's possible, it doesn't work like I want. (the bare attribute can't exist without a param statement).  I could also do something like #pragma, etc, but I haven't looked at that. For the moment, -Severity can be set to Warning,Error,Information which will suppress the output.

Sounds good. We have a separate issue to introduce pragma like region or line based suppression anyway

How would people suppress it in vscode (where no command line exists and most people do not use settings files)

The red squiggly in VSCode comes from PSES, not PSScriptAnalyzer, because those are legitimate parse errors. I'm not sure if we currently have a way of suppressing them. Installing the PowerShell extension and then suppressing parse errors would be like buying a fighter jet and ripping out the flight computer.

What is PSES? I am experiencing this error in relation to PowerShell class inside a PowerShell module and although my  code is correct (as a result of concatenating all source files into a combined .psm1 file), VSC is still flagging in the original source. This is rather unfortunate because I'm not sure how to disable the flagging of this non-error if it isn't being generated by PSSA built into VSC.

I am experiencing this error in relation to PowerShell class inside a PowerShell module and although my code is correct (as a result of concatenating all source files into a combined .psm1 file), VSC is still flagging in the original source

If you run the PowerShell parser itself on your file, it will raise this error. The PowerShell VSCode extension just reports that. There's no way to suppress a parser error currently, because parser errors are the most serious kind of error. This is to say that what you're seeing isn't an error in any of the tooling -- it's a mismatch between how you're handling the file and what PowerShell expects. The two options I can imagine are:

Open a feature request on the PowerShell extension for VSCode to suppress parser errors
Change the way you handle these scripts in some way (for example, marking them as template files)

The script as you have it won't run as a standalone. That's what the PowerShell extension and all the other tools assess currently.

What is PSES?

it is PowerShell Editor Services - https://github.com/PowerShell/PowerShellEditorServices

* Open a feature request on the PowerShell extension for VSCode to suppress parser errors

* Change the way you handle these scripts in some way (for example, marking them as template files)

The script as you have it won't run as a standalone. That's what the PowerShell extension and all the other tools assess currently.

I've decided that the best way forward is to constrain all class references to a single file. So I currently have the need for 2 classes. Instead of implementing them in separate files, I am going to implement them all in the same file including defining a factory function to create the class instances:
controller.ps1:
class Controller {
  [System.Collections.Hashtable]$_passThru;

  Controller(
    [System.Collections.Hashtable]$passThru
  ) {
    $this._passThru = $passThru;
  }
}

class ForeachController : Controller {
  ForeachController(
    [System.Collections.Hashtable]$passThru
  ) {

  }
}

function New-Controller {
  # return appropriate new instance
}

No other module code will make explicit reference to classes. This is the best way I can see going forweard for the time being. It's less than desirable, but at least I don't have to put up with none-errors being reported by VSC and potentially hiding real issues.
So @rjmholt, when you say raise a feature request against the powershell extension for VScode, is this https://github.com/PowerShell/vscode-powershell or if the error is being emitted from PSES, shouldn't I submit to that instead?

So @rjmholt, when you say raise a feature request against the powershell extension for VScode, is this https://github.com/PowerShell/vscode-powershell or if the error is being emitted from PSES, shouldn't I submit to that instead?

Almost all the issues in the vscode-powershell repo are actually things implemented in PSES, but people were confused by that distinction and redirecting them just increased friction. So you're free to open the issue wherever, but we have more set up to track it properly in vscode-powershell.

Actually on second thoughts, having chatted some folks on POwerShell group on Gitter, I've decided to avoid using POwerShell classes altogther because there are potentially lots of gotchyas. I'm going to resort to attaching functions to PSCustomObjects and just be done with it. As much as I love PowerShell, the number of gotchyas in it are infuriating, so I'd rather just avoid classes which I have not used in PowerShell up to this point. I could do without the grief!


Command to Force System Sync with Backend

Would be good to have a command to force that system syncs with backend, I haven't seen anything on the API that we could use for this functionality, probably we'll have to talk with @3scale/system

out of scope


feat: add SceneWorker

This PR introduces SceneWorker, an ergonomic way to spawn a web worker and invoke functions on the worker. Note that SceneWorker is very much WIP, and is subject to API changes as we spruce it up. Also note that SceneWorker does not currently support pooling.

Codecov Report
Attention: 45 lines in your changes are missing coverage. Please review.

Comparison is base (9913855) 27.41% compared to head (f1a5114) 25.78%.

Files
Patch %
Lines

src/lib/SceneWorker.ts
0.00%
40 Missing and 5 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   27.41%   25.78%   -1.63%
==========================================
  Files           9       10       +1
  Lines         715      760      +45
  Branches      176      182       +6
==========================================
  Hits          196      196
- Misses        435      475      +40
- Partials       84       89       +5

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


[Error in Example code] SyntaxError: 'yield' outside function

Your example in the README.md does not work.
~ $ ./example.py
  File "./example.py", line 10
    yield s.connect('your.email.host',587)
    ^
SyntaxError: 'yield' outside function

I might have skipped the function declaration there.


Broken texture for some stuff when in hand (and in inventory)

Describe the bug
Here are some of the guns that are broken when in hand, plus images:
- Chiappa Rhino:

- FN F2000

100rnd 5.56x45mm Stoner Magazine

L96A1 (casing sticking out)

M200 Intervention (casing sticking out)

S&W .500 Magnum

These are some weapon parts and other stuff that looks broken in the inventory (and some in hand too):
NOTE: I will always put them in my hotbar from left to right
- 25/50rnd .45 ACP Vector (Drum) Magazine

- G11 K1/K2 Handguard

- SIG-556 Grip

- Vector Taped Grip

- M249 (Upper) Handguard/Stock/Mil-Specs Stock/Para Stock/HK416 Stock

- item.mwc_FiveSevenSlide.name (no proper name) (no texture)

- HK416 Receiver (no texture)

- item.mwc_C8SFWReceiver.name (no proper name)

- Leupold VX-6 CDS TMOA Rifle Scope

- Scorpion Rear/Front Iron Sight

- item.mwc_oldiepinkcamo.name (no proper name)

NOTE: The camo was displayed correctly when applied back in MWC 0.1-Dev-12, but is now broken. Inventory texture remained broken throughtout the builds
Here are some guns that render the muzzle flash incorrectly:
- Bren MKII

- M1941 Johnson (LMG)

- M1941 Johnson (Rifle)

- DP-28

- MG-34

- MG-42

- M60E4

- L96A1 (inside the barrel)

- M200 Intervention

Versions:

Modern Warfare Cubed: MWC 0.1-Dev-14
Forge: 1.12.2-<IP_ADDRESS>60
Java: JDK 1.8.0_351-b10
Java JVM: 25.351-b10 (64-bit)

Specification:

OS: Windows 10 Enterprise
CPU: Intel Core i7-2600U 3.40GHz
GPU: NVIDIA Quadro K620
RAM Allocated: approx. 10GB

There will be a big internal write up of all the weapon issue, a lot of theses have been fixed too


[MetricsAdvisor] Investigate failing live tests authentication issue

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=685460&view=results

Fixed with the help of the EngSys team.

Fixed with the help of the EngSys team.


[web]: Clipboard support on Flutter Web

Clipboard support should be implemented in Flutter Web. It needs to be wired with Flutter's Clipboard Service.
For Flutter Web implementation: there are two ways to interact with the clipboard in the web. Document.execCommand() and more modern asynchronous ClipboardAPI. More info on article.
The a11y will be supported automatically after these changes.

I was able successfully to implement clipboard just by implementing my own https://api.flutter.dev/flutter/services/TextSelection-class.html that use Clipboard API provided by a browser, so this is possible. The browser asks for clipboard permission first. All major browsers support that feature.

@uKL can you give me an example? thanks

nturgut@ Is the support added?

Yes, web clipboard support is available now. With the recent changes we made in the text editable, now we are using native input elements for providing clipboard (and other context menu) functionalities.
Flutter Web repository is deprecated so please use the main Flutter repository. More details on the announcement: https://developers.googleblog.com/2019/09/flutter-news-from-gdd-china-flutter1.9.html

Cool, thanks!

I'm seeing the call to Clipboard.setData(data) hang on the dev channel (can't test on master due to https://github.com/flutter/flutter/issues/40876)

We are not using the Clipboard.setData and getData method calls.
In web, our solution is using the native input elements. This way we can utilize browsers' built in clipboard functionalities.
(1) when a user clicks (or taps on) an editable field, the framework sends its location and styling to the web engine. Web engine positions an invisible input element on the same location.
(2) Users only see the editable widget rendered by Flutter. When they do clipboard related activities, example right click, paste, the native element responds.
(3) We make sure the text is in sync.
(4) Ctrl+C, Ctrl+V, Ctrl+A, Ctrl+X also works the same way.

Ok makes sense, thanks nturgut!

The Clipboard.setData doesn't work in Flutter for Web

I'm using the clippy package https://pub.dev/packages/clippy. Works fine!

I'm using the clippy package https://pub.dev/packages/clippy. Works fine!

Thanks bud.

Is there a workaround for Clipboard.getData(), e.g. to provide our own "paste" button in the app?  I notice that this works in Chrome in development mode but not when compiled for release (and maybe not in other browsers at all).  An obvious alternative would be to provide the user with a text field and ask the user to paste in the text, but we have some places in our app where a paste button is preferable.  Thanks.


Add more thorough testing

As a reasonable approximation "more testing" = "more good" does hold. And rsasl already does some testing but it could definitely profit from more.
Especially as it's a rather abstract framework integration tests are relevant to make sure updates don't break stuff.
Especially integration tests that just test the client portion of a mechanism against its server portion should be a good place to start as they won't require knowledge of the inner workings of the mechanisms or rsasl itself.

Would you be interested in a PR that adds fuzz-testing to rsasl? Fuzz-testing everything that is potentially exposed to untrusted data seems like a good start.

Oh absolutely :)
I was looking into fuzz testing rsasl a while back, the main reason why I didn't go for it was that it currently doesn't expose any of the mechanisms directly so I felt like it'd mostly fuzz test the parsers. But that too would be a benefit so if you want to help with it, please do! ^^


ENH: Use free positions for XTPot

Closes #13.

Not a super clean PR w.r.t to the CI but it makes more logical sense to merge as in and push a lint commit on main...


Changes to README badges

Changes:

Separated by newline
Uses SVG for all badges

PNG results in a pixelation on macOS when using a retina display

Coverage remained the same at 0.0% when pulling df2369714c90f346a05eba30c68eccc0b3a8c01d on nvzqz:badges into 0d690191e67351a47848ded05f4e87ff74414313 on gimli-rs:master.


feat(sdk-trace-web): ignore unavailable span network events

Which problem is this PR solving?
When doing cross-origin XHR/Fetch requests browsers expect Timing-Allow-Origin HTTP header to explicitly allow getting network performance timings. See https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Timing-Allow-Origin for more.
Right now, on such cross-origin requests, we record all events with negative timings (relative to the span creation time).
It can be often misleading and can contribute to invalid metrics when calculating same event from multiple spans.
Short description of the changes
Network events equal 0 are not recorded.

I don't remember who, but it was already like this and it has been changed to explicitly log such information too

CC @johnbley . I've found the negative/zero performance timings were allowed in https://github.com/open-telemetry/opentelemetry-js/pull/1769 . I'm good with negative timings, but zero often means that the timing is not available.

CC @johnbley . I've found the negative/zero performance timings were allowed in #1769 . I'm good with negative timings, but zero often means that the timing is not available.

And zero also can mean "instantaneous", "was in cache", "not applicable", or "legitimately 0"  (e.g., there were no redirects so redirectStart == 0.0, as Chromium does it at least).  I wish the spec explicitly said "just leave the property undefined if it wasn't / isn't / can't be measured".  As another special case, the spec states that startTime for PerformanceNavigationTiming entries is supposed to be exactly 0, which we definitely want to treat as "correct"/"true".
I think that for this piece of tech, our telemetry should report whatever the browser says with minimal processing, and let downstream/backend/easier-to-modify components deal with normalization/truncation/processin if desired (e.g., throwing away absurd times like "that page took 6 years or -5 hours to load", or declaring that "no redirects happened so the redirect timing is not 0 but non-existent").  Does that not work in your world for some reason?

Unfortunately I don't think we can always be sure that 0 equals missing value. One example is Firefox which for security reasons reduces precision. It is possible that you have sub 2ms or 100ms(or more depending on settings) connect time which will show up as 0. Here is this reproduced in a non localhost page:

There are cases where we know that timings will be missing eg:

HTTP span always has missing secureConnectionStart
Third party requests without Timing-Allow-Origin header have most timings missing

In those cases we can predict that the timings will be missing and sanitize them server side. Although not sending them at all could save some bandwidth. If it is not possible for someone to have that processing maybe make this configurable? That seems a bit of an overkill tho.

@johnbley @m-tu Thanks for clarification. I was not aware how complex this matter is.
Your idea about handling it on backend side sounds good.
I think, the biggest issue for me was that the 0 is relative to the page time origin, so on some traces from long-running websites I've seen that span event occurred 30 minutes before trace. That was confusing, because I was checking one particular transaction with no knowledge about user session and how the page was opened. But now I don't think it can be easily fixed just by removing 0. Thanks for your help.


Correct potential issues with article on Ansible and SSH bastion host

Per a reader, there are some potential issues with the article on using Ansible through an SSH bastion host (http://blog.scottlowe.org/2015/12/24/running-ansible-through-ssh-bastion-host/):

The use of both ProxyCommand and ForwardAgent in an SSH configuration file is not required/recommended. The reader suggests that the commands "compete" with each other.

Some of the SSH configurations are duplicated between the SSH configuration file and the Ansible configuration file.

I need to review these potential errors and correct them.

Closing this issue---created it in the wrong repository.


Add 50 additional airports

Sourced from https://www.partow.net/miscellaneous/airportdatabase/ under MIT license.
Work towards https://github.com/adaptive/iata/issues/136.

@Celeo  Your commit has left out the attribution requirements of the license.


Azure Functions lab for Japanese

I translated this page into Japanese.
A lot of Japanese engineers are interested in Azure Functions so that this PR might help them.

 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.:x: toshi0607 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@rajenki @conceptdev
Would you review this PR?

Thank you very much for taking the time to do this. Apologies for the delayed response, I have merged this now and moved it under the Azure-Functions/Getting-started/jp/ folder. Thanks again!


XCode 9 push notifications support

Notifications are not enabled if the files are not added to the project

Description

Add the default entitlement files to the xcode project with a  tag
Related Issue

https://github.com/phonegap/phonegap-plugin-push/issues/2264
Motivation and Context

Many people are struggling with this one and nothing really relevant on SO or GitHUB.
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ X ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ X ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ X ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Closed, as explained on #2264, this is something cordova-ios should do, not the plugin, in #2264 there is a plugin that can be used as workaround


Trigger for clientside callbacks

From TriggerTransform:

NOTE: This transform cannot be implemented for clientside callbacks since the JS can't be modified from here.

I respectfully beg to differ.
We can wrap the callback JS function in another function which takes out the trigger args.
Roughly like this:
// original callback:
function(important, _irrelevant, interesting) {
    return important * interesting;
}

// wrapped callback:
function(important, _irrelevant, interesting) {
    const func = function(important, interesting) {  // <-- '_irrelevent' removed
        return important * interesting;
    };
    return func(important, interesting);
}

Implementation is in this PR.

Ah, yes, you are correct! Very elegant solution - I didn't think of that at the time ;)


hi,this is a swift version

https://github.com/liyoro/LIPAImageView

Hi,
I already created a Swift version ;)
https://github.com/abiaad/PASImageView

On 28 juil. 2015, at 12:19, liyoro<EMAIL_ADDRESS>wrote:
https://github.com/liyoro/LIPAImageView
—
Reply to this email directly or view it on GitHub.

en，thanks，i like it


Unknown I/O error when using whitespace in file path

If the file path contains a white space, I get this error: UNKNOWN_IO_ERROR.
Example path with error: /storage/emulated/0/My Folder/test.txt
Searching the web, I found the following two ways:

Replace whitespace with \\ , for example: /storage/emulated/0/My\\ Folder/test.txt
Replace whitespace with %20, for example: /storage/emulated/0/My%20Folder/test.txt

But I get this error in both ways: FILE_NOT_FOUND

I checked again, storage permission was wrong.


Extract image parts for Handwriting Recognition

It would be super interesting to isolate single lines of handwritten text to further process them in a handwriting recognition apporach.
Unfortunately I am quite new to OpenCV and would be super happy to get a hint how to start that?
Thank you for any help/ideas.

Hi, I will close the issue for now. Feel free to open it again if you have any question.


remove functions which take uncurried functions

We have several functions which take uncurried functions as arguments:
$ grep '//# .*_ :: .*,' index.js
  //# reduce_ :: Foldable f => ((b, a) -> b) -> b -> f a -> b
  //# flip_ :: ((a, b) -> c) -> b -> a -> c
  //# on_ :: ((b, b) -> c) -> (a -> b) -> a -> a -> c
  //# encase2_ :: ((a, b) -> c) -> a -> b -> Maybe c
  //# encase3_ :: ((a, b, c) -> d) -> a -> b -> c -> Maybe d
  //# encaseEither2_ :: (Error -> l) -> ((a, b) -> r) -> a -> b -> Either l r
  //# encaseEither3_ :: (Error -> l) -> ((a, b, c) -> r) -> a -> b -> c -> Either l r
  //# groupBy_ :: ((a, a) -> Boolean) -> Array a -> Array (Array a)

Thanks to #289 we now have curry2, curry3, curry4, and curry5. These provide a way to make uncurried functions compatible with functions such as encase2 which require a curried function:
S.encase2(S.curry2(f)) = S.encase2_(f)

We could remove all the _-suffixed functions listed above. Not only would this provide a small simplification now, it would save us from adding similar _-suffixed functions in future. Thoughts?

@JAForbes, @rjmk, @svozza: Any objections?

No objections.
Does this cut off a possible route of optimisations for a user of Sanctuary? i.e. using js functions with no closures or indirection in cases where that's useful

Does this cut off a possible route of optimisations for a user of Sanctuary?

Yes, it does. Currently S.reduce_(S.add, 0) is more efficient than S.reduce(S.add, 0) because it avoids unnecessarily using uncurry2 to wrap S.add before passing it to Z.reduce.
Empathy for the Machine is a trait shared by many programmers. I believe that in many cases it's a liability rather than an asset. Performance is sometimes very important, but we tend to consider it—and make concessions to it—even when it's unimportant.
I don't like the idea of some Sanctuary users having a mental note to use S.reduce_ if possible, as S.reduce is slower. I'd prefer to provide just one option (in accordance with the Zen of Python).

Is the performance penalty significant?
-----Original Message-----
From: "David Chambers"<EMAIL_ADDRESS>Sent: 6/21/2017 10:41 PM
To: "sanctuary-js/sanctuary"<EMAIL_ADDRESS>Cc: "James Forbes"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>Subject: Re: [sanctuary-js/sanctuary] remove functions which take uncurriedfunctions (#416)
@JAForbes, @rjmk, @svozza: Any objections?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Is the performance penalty significant?

I'll let you decide:
> var S = require('sanctuary')
> function time(thunk) { console.time(''); thunk(); console.timeEnd(''); }
> time(function() { S.reduce(S.add, 0, S.range(0, 5000)); })
: 1113.919ms
> time(function() { S.reduce_(S.add, 0, S.range(0, 5000)); })
: 1664.283ms
> time(function() { S.reduce(function(x) { return function(y) { return x + y; }; }, 0, S.range(0, 5000)); })
: 1045.408ms
> time(function() { S.reduce_(function(x, y) { return x + y; }, 0, S.range(0, 5000)); })
: 1613.222ms
> var S_ = S.create({checkTypes: false, env: []})
> time(function() { S_.reduce(S_.add, 0, S_.range(0, 5000)); })
: 16.418ms
> time(function() { S_.reduce_(S_.add, 0, S_.range(0, 5000)); })
: 3.827ms
> time(function() { S_.reduce(function(x) { return function(y) { return x + y; }; }, 0, S_.range(0, 5000)); })
: 1.372ms
> time(function() { S_.reduce_(function(x, y) { return x + y; }, 0, S_.range(0, 5000)); })
: 0.859ms

I'm sure we can all agree that type checking is the real source of slowness. ;)

I think if you need to optimize a code path for performance, you will use Z directly, or even functions specialized for your data type. I see no reason to keep the "non-curried-function-taking functions" around for performance reasons, as I would avoid even those when optimizing.

Interesting.  Yeah type checking is definitely taking the most time.

time(function() { S_.reduce(S_.add, 0, S_.range(0, 5000)); })
: 16.418ms
time(function() { S_.reduce_(S_.add, 0, S_.range(0, 5000)); })
: 3.827ms

This jumps out to me as a meaningful difference for UI programming, typically a frame is 16ms, but we can really only spend ~12ms of that frame for JS to run.  The curried example would not fit in a frame, the uncurried one would.
I am pretty stunned currying makes that much of difference even for just summing a list of 5000 items.  Micro benchmarks should always be treated with scepticism, but I'd expect that to take a lot less time.
I don't necessarily have objections because if you have a hot code path, you probably should use a for loop to sum a list of numbers instead of sanctuary.  And... I've never used the uncurried versions.  But I am surprised currying alone, without type checking takes that much of a toll.
For reference, in ramda, currying doesn't have the same impact:
function time(thunk) { console.time(''); thunk(); console.timeEnd(''); }

time(function(){
  reduce(add, 0, range(0, 5000))
})
// 2.31ms

I don't know why currying in sanctuary without type checking takes 16ms, but only ~2ms for ramda.  Is Sanctuary's curry method doing more work than ramda's even with type checking off?

Is Sanctuary's curry method doing more work than ramda's even with type checking off?

Yes. It defines inspect/toString.  It would be interesting to profile our internal curry function in order to better understand why it is slower than the Ramda equivalent.

Yes. It defines inspect/toString.

I don't imagine inspect/toString would have a cost on execution.  It would be interesting to explore this further.  That's a significant performance difference.  Maybe it's worth while to export separate curry functions depending on typeCheck's value?
It might even be helpful to disable custom inspect/toString for production, as they are really are just a development convenience (but only if they have any impact on performance).


Using too many tiles and a Camera2D will make TileMapLayer vanish on camera zoom + offset

Tested versions
Version: 4.3.stable.mono
System information
Godot v4.3.stable.mono - Windows 10.0.22631 - GLES3 (Compatibility) - NVIDIA GeForce RTX 2080 SUPER (NVIDIA; <IP_ADDRESS>03) - AMD Ryzen 7 3700X 8-Core Processor (16 Threads)
Issue description
At a specific number of tiles (in my test its 128 * 192 = 24 576 tiles) I have a problem when using a Camera2D.
When I zoom out of the tilemaplayer and then move the camera around via the Offset property, on specific positions in a direction (for example moving down) - the tilemaplayer vanishes. If I keep moving a bit more, it appears again. Then vanishes again. Like on specific breakpoints or something.
Steps to reproduce
Download the example project and then zoom out to max zoom and keep holding, the down arrow. Then you will see the effect.
Minimal reproduction project (MRP)
TestTilemap.zip

When I scroll down until the tilemaplayer vanishes im about at camera offset.y 720. When I stop right there and in the Remote Tab go to my tilemaplayer and set the quadrant size to for example (from default 16) to 15 (or any other number) it appears again. Thats one thing I found out.
Don't fully get the quadrant size, just want to throw that in.


Opening notes in a new tab is broken

Summary
With the latest update, opening notes from a wiki-link with custom text in a new tab is broken.
Expected Behavior
When holding ctrl while clicking on a wiki-link with custom text, the respective file opens in a new tab.
Current Behavior
When holding ctrl while clicking on a wiki-link with custom text, the respective file opens in the same tab.
Steps to reproduce

Open a Obsidian Vault.
Install and enable Mathlinks.
Create two files 'File A' and 'File B'.
Inside 'File A', write: 'Hello [[File B|Test B]]'.
Make sure to move the cursor outside of the wik-link, such that only the text 'Test B' is displayed.
Hold ctrl while clicking on the wiki-link.

Thanks for leaving this issue! It will be fixed in 0.3.3.


HDDS-7090. EC: delete empty closed EC container

What changes were proposed in this pull request?
we need to check more states for EC container and delete empty closed EC container.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7090
How was this patch tested?
Unit test

We are refactoring how the container checks are handled in the new RM in #3743, so once that goes in this one will need changed to work in the way described there.

see @siddhantsangwan has already completed this work, so this patch just add the handler to RM.
@adoroszlai @sodonnel please take a look!

We have left this new handler out of the RM chain deliberately for now, as its not really complete without the deleting handler. The plan is to implement the deleting handler and then add both it and this handler to the chain in the next PR.
Feel free to go ahead with implementing this change and the deleting handler in the next PR.

We have left this new handler out of the RM chain deliberately for now, as its not really complete without the deleting handler. The plan is to implement the deleting handler and then add both it and this handler to the chain in the next PR.
Feel free to go ahead with implementing this change and the deleting handler in the next PR.

i have implemented deleting container handler, #3881 , please take a look! @sodonnel

since #3881 is merged , we can merge this patch now, please take a look! @sodonnel @siddhantsangwan

@sodonnel @siddhantsangwan
since this patch is simple,  if no more comments from you, i will merge it tomorrow!

LGTM!

thanks @sodonnel @DaveTeng0 @adoroszlai for the review!

Hi @JacksonYao287, do we need to cherry-pick this PR and HDDS-7384 to ozone-1.3 branch?

Hi @JacksonYao287, do we need to cherry-pick this PR and HDDS-7384 to ozone-1.3 branch?

yea , please cherry-pick both HDDS-7090 and HDDS-7384, thanks! @kaijchen


RecyclerView

Is is possible to create such thing for recyclerview?

It's working nice for me in a RecycleView.


[TypeError: table.point is not a function]

Environment
knex: "^0.95.11",
MariaDb:
OS: manjaro linux
Bug
1.In documentation it shows up the point data type supported in knex, but when I use it throws a error as points not exists

[TypeError: table.point is not a function]

Missing / erroneus documentation
https://knexjs.org/#Schema-point

Try upgrading knex. Geometry schema helpers were released in 0.95.12 or 0.95.13


Performance bottleneck numbers

Immutable data structure in pgoutil
locking
libraries

We no longer have pgoutil in the runtime. Performance numbers will be measured in the future but this issue is outdated.


Fix support for runtime theme in the Visitor Code view

The previous solution didn't behave as expected. Tested it out and fixed issues.
Jira issue:
https://glia.atlassian.net/browse/MOB-1952

!merge

!merge


updated readme

Coverage remained the same at 92.162% when pulling 90efcaef6c6dd45652714b25a00a71348c3542bb on ALim95:readme into 97588e124db8a6047f51a9919b4bda59c923d966 on CS2103AUG2017-T15-B4:master.


no matching package named rustpython-vm found

macOS Big Sur, clean install of Rust.
vlad@Vladimirs-MacBook-Air ~ % cargo install rustpython
Updating crates.io index
Installing rustpython v0.1.2
error: failed to compile `rustpython v0.1.2`, intermediate artifacts can be found at `/var/folders/yc/2061m39j2bxg37kz2wxb5sl00000gn/T/cargo-installVJRhrB`

Caused by:
no matching package named `rustpython-vm` found
location searched: registry `https://github.com/rust-lang/crates.io-index`
required by package `rustpython v0.1.2`

vlad@Vladimirs-MacBook-Air ~ % cd /var/folders/yc/2061m39j2bxg37kz2wxb5sl00000gn/T/cargo-installVJRhrB
vlad@Vladimirs-MacBook-Air cargo-installVJRhrB % ls -la
total 8
drwxr-xr-x    3 vlad  staff    96 Aug 23 12:57 .
drwx------@ 152 vlad  staff  4864 Aug 23 12:57 ..
-rw-r--r--    1 vlad  staff  1044 Aug 23 12:57 .rustc_info.json
vlad@Vladimirs-MacBook-Air cargo-installVJRhrB % more .rustc_info.json
{"rustc_fingerprint":7515179083630404878,"outputs":{"931469667778813386":{"success":true,"status":"","code":0,"stdout":"___\nlib___.rlib\nlib___.dylib\nlib___.dylib\nlib___.a\nlib___.dylib\n/Users/vlad/.rustup/toolchains/stable-x86_64-apple-darwin\ndebug_assertions\nproc_macro\ntarget_arch=\"x86_64\"\ntarget_endian=\"little\"\ntarget_env=\"\"\ntarget_family=\"unix\"\ntarget_feature=\"fxsr\"\ntarget_feature=\"sse\"\ntarget_feature=\"sse2\"\ntarget_feature=\"sse3\"\ntarget_feature=\"ssse3\"\ntarget_os=\"macos\"\ntarget_pointer_width=\"64\"\ntarget_vendor=\"apple\"\nunix\n","stderr":""},"17598535894874457435":{"success":true,"status":"","code":0,"stdout":"rustc 1.54.0 (a178d0322 2021-07-26)\nbinary: rustc\ncommit-hash: a178d0322ce20e33eac124758e837cbd80a6f633\ncommit-date: 2021-07-26\nhost: x86_64-apple-darwin\nrelease: 1.54.0\nLLVM version: 12.0.1\n","stderr":""},"2797684049618456168":{"success":true,"status":"","code":0,"stdout":"___\nlib___.rlib\nlib___.dylib\nlib___.dylib\nlib___.a\nlib___.dylib\n","stderr":""}},"successes":{}}

Try installing directly from the main branch.

Clone this project and compile it with cargo.

Yeah, this is documented in the README use:
cargo install --git https://github.com/RustPython/RustPython

Same as #2996


Consolidate materials related to JupyterHub

Currently materials related to JupyterHub are presented in 2 places: Pre-event preparation and Tutorials/Getting Started which makes it confusing.
I think we can consolidate all JupyterHub-related materials to under Pre-event preparation and just add a link at the Tutorials section.
Pinging @emiliom and @ocefpaf -- not urgent.

I don't have the bandwidth this week. Let's wait till next week, b/c I'm also adapting some of that same material for Waterhackweek, which started today.
I forget if I made that distinction or if I inherited it -- and liked it -- from ICESat-2 hackweek. It's there for a logical reason, though we could well argue that it's better to consolidate the materials. We can discuss next week.


Working with swagger-editor in VM

I am trying to configure swagger in VM and then access it from a local desktop browser(windows'7).
I set port forwarding:
dev.vm.network "forwarded_port", guest: 3000, host: 3000, host_ip:
"<IP_ADDRESS>", auto_correct: true

Followed steps given in swagger official documentation:
npm install -g swagger
Create a new swagger project
swagger project create hello-world

Since I am working with VM and local desktop, I had to run command to open swagger-editor on host <IP_ADDRESS>:
swagger project edit --host <IP_ADDRESS> -p 3000 -s
Till here I don't have any issues.
Now, when I try to open the editor in the local desktop browser it's just giving me some text in YAML format instead of something like this.
Refer attached screen shot to see how it's looking in a web browser.

try  npm install -g swagger  , don't use yarn


Problem with minimal theme and excalidraw plugin

Is the bug present when using the default Obsidian theme?
No
Is the bug present when snippets and plugins are disabled?
Yes
Minimal theme version
6.0.5
Describe the bug
Then I turn on dark mode in the obsidian and Excalidraw is in light mode, I get bad colours

Excalidraw author redirect me to you about this issue.
more details in a ticket:
https://github.com/zsviczian/obsidian-excalidraw-plugin/issues/817
Debug info
no one

Also experiencing this issue.

Also seeing this. Does anyone know of a workaround for this issue?

Here is a CSS snippet workaround to make it usable. Depending on your theme it may not look very pretty.
body .excalidraw, body .excalidraw.theme--dark {     --island-bg-color: #F5F5F5; }

I am also having this issue - just want to clarify the issue.

It happens when I am using dark theme and I switch to excalidraw light mode (the dark mode = black paper background, works fine)
Issue is on mac OS
Obsidian v1.0

Here is a CSS snippet workaround to make it usable. Depending on your theme it may not look very pretty.
body .excalidraw, body .excalidraw.theme--dark { --island-bg-color: #F5F5F5; }

I had to change it to
body .excalidraw  { --island-bg-color: #F5F5F5; }

to make the switching work properly

Fixed in 6.1.10


Password reset

Any chance we can get a password reset page? Mariadb console is cumbersome for any people not familiar with it.

Hi, this already exists in the dev branch. Clone it using this command:
git clone -b dev https://github.com/fethica/PHP-Login.git
Then navigate to whatever your sites's base directory is +
/install/index.php to run the automated installation process. If this
fails for whatever reason, try again using the manual installation
method. If that fails, you may have to do some extra legwork to get it
running. It's still a dev version, so ymmv, but it is fairly reliable
on most systems, from what I've seen. Just make sure that your server
satisfies all of the requirements, including having a working PHP CLI
On Wed, Sep 27, 2017 at 11:54 AM, factulas<EMAIL_ADDRESS>wrote:

Any chance we can get a password reset page? Mariadb console is cumbersome
for any people not familiar with it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fethica/PHP-Login/issues/121, or mute the thread
https://github.com/notifications/unsubscribe-auth/AH24AkV1C4wWRBLP3aCITBwlyC_UOQdIks5smm-0gaJpZM4Pl_5a
.

Well, hell, where do I see what branches there are.. They used to be at the top by the issues/ etc.


level=error component=activeQueryTracker msg="Failed to create directory for logging active queries“

Describe the bug
Description of the bug here.
I encountered this problem when running node, is it a bug?
aiko_client_prover_relayer-1  | [INFO  wait] --------------------------------------------------------
taiko_client_prover_relayer-1  | [INFO  wait] docker-compose-wait - Everything's fine, the application can now start!
taiko_client_prover_relayer-1  | [INFO  wait] --------------------------------------------------------
taiko_client_prover_relayer-1  | Required flags "l1.proverPrivKey, prover.capacity" not set
prometheus-1                   | ts=2024-03-03T04:24:30.865Z caller=query_logger.go:115 level=error component=activeQueryTracker msg="Failed to create directory for logging active queries"
prometheus-1                   | ts=2024-03-03T04:24:30.865Z caller=query_logger.go:93 level=error component=activeQueryTracker msg="Error opening query log file"
Steps to reproduce
Steps to reproduce here.
Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

Please reach out in the correct discord channel.

ok, ths