Can't build recent gitoxide versions

https://github.com/Byron/gitoxide doesn't successfully build with cargo-auditable anymore. It fails in https://github.com/rust-secure-code/cargo-auditable/blob/9fa7fb3988b4d2414397b9550a3f3fd1aa1bc8f2/cargo-auditable/src/collect_audit_data.rs#L77-L80. I'm not sure if gitoxide or cargo-auditable is to blame here, and the error message isn't all that helpful to me either:
thread 'main' panicked at 'cargo metadata failure: error: Package `gix-features v0.31.0 (/home/jcgruenhage/dev/github.com/Byron/gitoxide/gix-features)` does not have feature `prodash`. It has an optional dependency with that name, but that dependency uses the "dep:" syntax in the features table, so it does not have an implicit feature with that name.
', cargo-auditable/src/collect_audit_data.rs:77:9

That reads to me as if something tried to enable a prodash feature on gix-features, but I couldn't find such a thing.
For context, afaict, v0.23.0 is the last version that builds. Anything newer than that runs into the above issue. I ran into this while trying to update the version of gitoxide we ship on Void Linux.

The error originates in cargo metadata command that is part of the official Cargo distribution, and that cargo auditable runs to get information about the crates.
I can reproduce the issue on the latest version of the repo from git and Rust 1.70. I will investigate further. Thank you for the report!

Thanks for looking into this, reporting it is the least I can do when seeing something like this ;)

It seems to be a bug in cargo. It passes --cfg 'feature="prodash"' to rustc even though there is no such feature in the crate, only a dep: declaration. Then, when cargo auditable extracts the list of features passed to rustc and feeds it back to cargo, cargo itself reports an error because the feature it passed to rustc doesn't actually exist.
Unfortunately this is something that will have to be fixed in Cargo, so I cannot provide an ETA for the fix. I will make a minimal reproducing example and submit the issue upstream.

I've filed an issue upstream: https://github.com/rust-lang/cargo/issues/12336
But the turnaround on Cargo issues is not very quick, so I might have to work around this somehow - perhaps with another call to cargo metadata to get the list of actually existing features, and filter the --cfg 'feature=foo' directives against that`? That will slow down the build but at least it will work.

I have the same issue building Wasmtime.
The conflict seems to be fixed by https://github.com/rust-lang/cargo/issues/10788, but cargo-auditable still does not work even if I use the latest Rust version.
The build succeeds without cargo-auditable.

I found a workaround that seems to work.
Patched Wasmtime.
-wasi = ['wasi-cap-std-sync', 'wasmtime-wasi', 'cap-std', 'wasi-common']
+wasi = ['wasi-cap-std-sync', 'wasmtime-wasi', 'dep:cap-std', 'wasi-common']

Any update on this? It affects other packages as well, like lightningcss https://github.com/parcel-bundler/lightningcss/issues/702.

Unfortunately this is a bug in Cargo itself, and AFAIK there is nothing I can really do to fix that in cargo auditable.
The proper way to do this would be in Cargo, or to integrate cargo auditable functionality into Cargo itself making the external tool obsolete. Sadly the progress on upstreaming is slow - the prerequisite https://github.com/rust-lang/rfcs/pull/3553 is not yet merged (although a prototype implementation is underway) and the actual RFC for cargo auditable-like functionality has been recently postponed.

I don't really understand, it works in cargo, but with auditable it errors out. Usual workaround is to remove dep: from the features table.

https://github.com/rust-lang/rfcs/pull/3553 would be the proper, correct solution to this problem.
Until then, there may be hope for working around Cargo passing nonexistent features to rustc, which causes these failures.
When compiling a crate, Cargo sets the current working directory of the child rustc process to the crate’s directory. Therefore, we could discover the Cargo.toml file for the crate, parse it, extract a list of all actually existing features, and filter the features passed by cargo to rustc against that list.
However, that ties cargo auditable to the evolving Cargo.toml format with a lot of complexity and edge cases. We might be able to bypass parsing it ourselves by invoking cargo metadata --offline --no-deps and parsing its output.

Fortunately that is not a problem if I run cargo metadata --no-deps, which is what I am going to do. This creates its own problems - it is difficult to determine what package the Cargo.toml actually belongs to, which is silly. Starting with Cargo 1.71 I can use the default_workspace_members field, but for earlier versions I will probably need some cursed heuristics if I want to keep --no-deps.
Besides that, I will only need to run cargo metadata on workspace members, and all workspace members already have their Cargo.lock generated or the build would not proceed. So I could actually afford to run without --no-deps, if I'm willing to take the compilation time hit.

I have prototyped a fix in the https://github.com/rust-secure-code/cargo-auditable/tree/fix-dep-features branch. Tests pass, but I don't have a way to check if that fixes the issue because the latest versions of both gitoxide and lightningcss build successfully even without these changes.

Okay, I managed to reproduce this on gitoxide if I check out v0.24.0 tag. My "fix" doesn't actually fix it. Debugging time!

Okay, it seems that cargo metadata is buggy and reports that gix-features has the feature prodash when it actually doesn't, it just has a dep:prodash syntax without the actual feature.
I don't think I can work around the issue in cargo auditable when there are this many different bugs in Cargo around it :face_with_diagonal_mouth:


OpenPolicyContainers.Policy.v0.2.16

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I was not associated with this pull request (I received relevant information.)


Roll Engine from 9d2377f0d8aa to 6136cbd8b054 (10 revisions)

https://github.com/flutter/engine/compare/9d2377f0d8aa...6136cbd8b054
2021-06-30<EMAIL_ADDRESS>Give FlutterView a view ID (flutter/engine#27052)
2021-06-30<EMAIL_ADDRESS>Roll Skia from c6804edbaefc to 55b401ed9e6c (1 revision) (flutter/engine#27055)
2021-06-30<EMAIL_ADDRESS>Roll Dart SDK from 5103185fdff6 to 9d7c40ba84c4 (1 revision) (flutter/engine#27054)
2021-06-29<EMAIL_ADDRESS>Fix use-after-free. (flutter/engine#27053)
2021-06-29<EMAIL_ADDRESS>Roll Skia from 62ce2488f744 to c6804edbaefc (4 revisions) (flutter/engine#27050)
2021-06-29<EMAIL_ADDRESS>Removes the licence sheck from cirrus (flutter/engine#27051)
2021-06-29<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from qq5J5tHIA... to eMHAbJpmO... (flutter/engine#27049)
2021-06-29<EMAIL_ADDRESS>--sound-null-safety instead of enable-experiment where possible (flutter/engine#26999)
2021-06-29<EMAIL_ADDRESS>[web] Render RTL text correctly (flutter/engine#26811)
2021-06-29<EMAIL_ADDRESS>Temporarily opt out of reduced shaders variants till roll issues are resolved. (flutter/engine#27048)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from qq5J5tHIADGD to eMHAbJpmOD9J
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Firebase - Unauthenticated request is not raising an exception

When injecting the FirebaseClaims object, no exception is raised for an unauthenticated user. It looks like it should raise a 401 exception because auto_error is set to True.
Additionally, it is unclear how to test this via the Swagger UI. There does not appear to be any authentication UI, as there is with cogito.
Code:
async def get_current_user(settings: Settings = Depends(get_settings)):
    return FirebaseCurrentUser(project_id=settings.firebase_project_id)

@app.get("/protected")
async def protected(current_user: FirebaseClaims = Depends(get_current_user)):
    return f"Hello, {current_user.user_id}"

@br-allstreet ,
try to do something like this:
from fastapi.security import HTTPAuthorizationCredentials, HTTPBearer

async def get_current_user(
    settings: AppSettings = Depends(get_app_settings),
    http_auth: Optional[HTTPAuthorizationCredentials] = Depends(HTTPBearer(auto_error=False)),
) -> CognitoCurrentUser:
    current_user_auth = FirebaseCurrentUser(project_id=settings.firebase_project_id)
    return await current_user_auth(http_auth)

@app.get("/protected")
async def protected(current_user: FirebaseClaims = Depends(get_current_user)):
    return f"Hello, {current_user.user_id}"

I didn't check it, jsut a quick sketch...


Resizing of messages pane isn't quite right

The messages pane under the editor should be resizable by dragging on the border between the editor and the pane.
Right now, you need to drag the lower-right corner of the pane to resize it. This probably means we can't rely on CSS resizing for this feature.

#59


Support models with same names across different packages

Today, model namespacing looks different than macro namespacing. When calling macros, there's a notion of precedence. So, {{ mymacro() }} will search:

First, in the local package
Then, in the "global" package (includes in dbt repo)
Then, package-specific lookups if package.mymacro syntax is used

We can / should use a similar notion of namespace precedence with models. This would let us support multiple models of the same name in packages, allowing us to generate docs for projects built into different namespaces with overlapping model names.

closing in favor of the discussion in #1269


Finish Sprint Backlog

Discuss whether the document is complete and ready to turn in

Sprint Backlog 1 ( /10)
Use this template. See example(s) here.
Contains only the stories (user stories and technical and non-technical stories) you commit to working on and delivering for this Deliverable.  Should reflect the product backlog. That is, all the user stories that are assigned to this Sprint in the product backlog, should be shown in this Sprint Backlog.
For any missing user story, deduct -2 points.
If the Sprint Backlog 1 does not reflect the product backlog, deduct -5 points.
You should also have other tasks in the backlog, like documentation, requirements, UI mockups, etc. As covered in class sessions, these are called non-technical stories.
You don’t need to have all the tasks for all future Deliverables for the project, just the ones for the sprint that you planned to do (and perhaps completed)
See the sprint backlog in the examples
All columns based on the projects’ details document should be in the product backlog and must be filled out. For any missing column or missing information deduct -2 points.


refactor(runtime-core): enum attribute refers to other attributes of itself

enum attribute refers to other attributes of itself, you can remove enum name.
An example modeled after ShapeFlags.
enum Example {
  A   =  1
  B   =  1 << 1
  C   = Example.A | Example.B
}

But we can do
enum Example {
  A   =  1
  B   =  1 << 1
  C   = A | B
}

Due to code conflict, resubmit PR


Display correction candidate if an incorrect cop name is given

Maybe we cannot memorize correctly cop names.
if an incorrect cop name is given with --only or --except options on CLI option,
it just only raises exception. I think it's not friendly.
So I've added its error message to correction candidate.
Example:
$ bundle exec rubocop --only Lint/Li
Inspecting 1081 files

0 files inspected, no offenses detected
Unrecognized cop or department: Lint/Li.
Did you mean? Lint/LiteralAsCondition, Lint/LiteralInInterpolation
...

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

...Mmm, It seems travis's bundler is a bad mood.

You should be good with a rebase.

BTW, the message should also include candidates that are found with some similarity algorithm (e.g. Levenshtein distance).

Yeah, that could be a nice future improvement.


cmd/cue: tool failure is hard to debug

What version of CUE are you using (cue version)?

$ cue version
cue version v0.0.0-20231004170206-6c71b2c90d63

go version devel go1.21-af8f94e3c5 Tue Jul 11 21:30:51 2023 +0000
      -buildmode exe
       -compiler gc
  DefaultGODEBUG panicnil=1
     CGO_ENABLED 1
          GOARCH amd64
            GOOS linux
         GOAMD64 v1
             vcs git
    vcs.revision 6c71b2c90d63846f2983deb07102253298a568c0
        vcs.time 2023-10-04T17:02:06Z
    vcs.modified false

Does this issue reproduce with the latest stable release?
Yes
What did you do?
! exec cue cmd foo
stdout 'foo'
stdout 'bar'

-- cue.mod/module.cue --
module: "example.com"

-- foo_tool.cue --
package foo
import "tool/file"

command: foo: bar: file.Create & {
	filename: "/something/non-existent"
	contents: "x"
}

What did you expect to see?
A passing test: the failure should at least mention the path to the task that failed.
What did you see instead?
> ! exec cue cmd foo
[stderr]
task failed: open /something/non-existent: no such file or directory
[exit status 1]
> stdout 'foo'
FAIL: /tmp/testscript573993553/x.txtar/script.txtar:2: no match for `foo` found in stdout
error running /tmp/x.txtar in /tmp/testscript573993553/x.txtar

The error message does not mention which task has failed, making it quite hard to determine where the error has arisen, particularly when there are many hundreds of tasks.

Another possibility would be that passing the -v flag to cue cmd might print information about commands being run, but it doesn't change the output at all.


nl-BE (Flemish, Dutch Belgian) misrepresented as Dutch-Belarusian

I'm submitting a ...

[x] bug report

General infomration
We have the problem that Netflix has streams for children movies in our native Dutch language, but it is never selected correctly. It is wrongly translated into Dutch-Belarusian, but also not considered Dutch.
Prerequisites

[x] Are you running the latest version?

Description
Apart from not selecting native Dutch(nl-BE), it is defaulting to German (ger).
Expected behavior: [What you expected to happen]
Actually what we would like to have (for children movies):

nl-BE
nl
en (with nl subtitles)
native language (with nl subtitles)

Actual behavior: [What actually happened]
Usually it defaults to German (ger) for audio and subtitles
Installation

[x] I installed the plugin via zip from the Releases page
[ ] I´m using the Netflix Repo
[ ] I´m using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[x] Linux (ARM)
[ ] Android

Additional informatin on the environment
We are using LibreELEC v8.95.003 using Kodi v18.0-RC5 and plugin.video.netflix v0.13.21.

@dagwieers can you provide me with the files you modified to correct the language?
I need to figure out how to best manage

The filenames are in the output at the start of each line.


test: Test IPsec+VXLAN on 4.19

The IPsec+VXLAN test was disabled on 4.19 long ago and never reenabled. Since we now run with kube-proxy on 4.19, we can run it there.

/test

/test

test-gke


Update to upstream

Material UI Commit Head e99f2afbd350f659412dcbba1a158f0a0246040a

Up to date with Material UI from 4.2.1 -> 4.4.5

New File/Directory Icons
Title bar things, etc.

[x] Works on MacOS

[x] Works on Windows


Update packages list

cc @zerebubuth

build-essential depends on g++, so that package shouldn't need to be explicitly installed. Otherwise looks good.
We should add something about running git submodule update --init as well.


In preprocessing, which part of the video captions code is in your folder?

Thank you for your work, it's a great job!
I cannot find the code for generating video captions in your code. May I know which part of the code is in which folder? I really need this part of the code to verify my idea. Thank you for your help!

Please refer to https://github.com/YoadTew/zero-shot-video-to-text for generating video captions.


Trying to make a line over contourf returns TypeError

I need to plot a contourf and overlay a line on it. If I try this with proplot I get the following error:
File [~/micromamba/envs/py_env/lib/python3.10/site-packages/proplot/internals/process.py:284](https://file+.vscode-resource.vscode-cdn.net/Users/raulvalenzuela/Library/CloudStorage/Dropbox/PythonProjects/Paper_ZAR_ROS/notebooks/~/micromamba/envs/py_env/lib/python3.10/site-packages/proplot/internals/process.py:284), in _preprocess_args.<locals>.decorator.<locals>._redirect_or_standardize(self, *args, **kwargs)
    281             ureg.setup_matplotlib(True)
    283 # Call main function
--> 284 return func(self, *args, **kwargs)

File [~/micromamba/envs/py_env/lib/python3.10/site-packages/proplot/axes/plot.py:2879](https://file+.vscode-resource.vscode-cdn.net/Users/raulvalenzuela/Library/CloudStorage/Dropbox/PythonProjects/Paper_ZAR_ROS/notebooks/~/micromamba/envs/py_env/lib/python3.10/site-packages/proplot/axes/plot.py:2879), in PlotAxes.plot(self, *args, **kwargs)
   2875 """
   2876 %(plot.plot)s
   2877 """
   2878 kwargs = _parse_vert(default_vert=True, **kwargs)
-> 2879 return self._apply_plot(*args, **kwargs)

File [~/micromamba/envs/py_env/lib/python3.10/site-packages/proplot/axes/plot.py:2853](https://file+.vscode-resource.vscode-cdn.net/Users/raulvalenzuela/Library/CloudStorage/Dropbox/PythonProjects/Paper_ZAR_ROS/notebooks/~/micromamba/envs/py_env/lib/python3.10/site-packages/proplot/axes/plot.py:2853), in PlotAxes._apply_plot(self, vert, *pairs, **kwargs)
   2850         objs.append((*eb, *es, obj) if eb or es else obj)
   2852 # Add sticky edges
-> 2853 self._add_sticky_edges(objs, 'x' if vert else 'y', xsides, only=mlines.Line2D)
   2854 self._update_guide(objs, **guide_kw)
   2855 return cbook.silent_list('Line2D', objs)
...
   2361 # masked_invalid previously never returned nomask as a mask and doing so
   2362 # threw off matplotlib (gh-22842).  So use shrink=False:
   2363 if res._mask is nomask:

TypeError: ufunc 'isfinite' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

With matplotlib runs ok.
Steps to reproduce
import proplot as pplt
import numpy as np
from datetime import datetime

fig, ax = pplt.subplots()

x=[datetime(2023,1,1,1), datetime(2023,1,1,2), datetime(2023,1,1,3)]
y=range(125)
Z=np.random.rand(125,3)
ax.contourf(x,y,Z)
ax.plot([datetime(2023,1,1,2), datetime(2023,1,1,2)],[0,100])

Expected behavior:
Draw contour filled and line.
Actual behavior:
Throw TypeError described above.
Equivalent steps in matplotlib
import matplotlib.pyplot as plt
import numpy as np
from datetime import datetime

fig, ax = plt.subplots()

x=[datetime(2023,1,1,1), datetime(2023,1,1,2), datetime(2023,1,1,3)]
y=range(125)
Z=np.random.rand(125,3)
ax.contourf(x,y,Z)
ax.plot([datetime(2023,1,1,2), datetime(2023,1,1,2)],[0,100])

Proplot version
'0.9.7'

Cannot reproduce this issue on my end:

Hi @cvanelteren ! Could you confirm you are using the following versions?:
proplot: '0.9.7'
numpy: '1.24.3'
matplotlib: '3.4.3'

This was some time ago, but currently I am proplot 0.9.5. The env you listed produces the same error.

Ah for me it was related to cartopy, removing it runs the code above fine on the listed versions.

Thanks! I will have to figure it out then because cartopy is within the libs I use frequently.


Example with exception from sttp testing docs causes "dead code following this construct" warning

There is an example for simulating exceptions in stub backend where you need to throw an exception which causes dead code warning or error (with tpolecat plugin).
URL: https://sttp.softwaremill.com/en/latest/testing.html#simulating-exceptions
val testingBackend = SttpBackendStub.synchronous
  .whenRequestMatches(_ => true)
  .thenRespond(throw new SttpClientException.ConnectException(
    basicRequest.get(uri"http://example.com"), new RuntimeException))

Warning:
dead code following this construct
          throw new SttpClientException.ConnectException(

Perhaps it should have some specific methods for simulating errors like:
val testingBackend = SttpBackendStub.synchronous
  .whenRequestMatches(_ => true)
  .respondWithError(...)

But isn't it a problem in the compiler, with a false warning, rather than a problem in the API?
There is already thenRespondServerError, but this completest the request successfully, but with a 503 response code. There's also thenRespondF, where you can return a failed effect (instead of throwing an exception)

But isn't it a problem in the compiler, with a false warning, rather than a problem in the API?
There is already thenRespondServerError, but this completest the request successfully, but with a 503 response code. There's also thenRespondF, where you can return a failed effect (instead of throwing an exception)

I thought it was some problem with thenRespond since i have no such warning in other places with throw. It's work well with thenRespondF(IO.raiseError(...)) or with MonadError[F].error.

I thought it was some problem with thenRespond since i have no such warning in other places with throw. It's work well with thenRespondF(IO.raiseError(...)) or with MonadError[F].error.


amp-video can't load tokyo.mp4

It works for me - but the video is way to big for the sample anyway which I fixed in the referenced PR.


Long-running actions cause Alexa intent to fail

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.49.0
Python release (python3 --version):
Python 3.5.2
Component/platform:
homeassistant.components.alexa
Description of problem:
I've followed this guide to create an Alexa intent that activates scenes in my home. When I attempt "simple" scenes (those that have short activation lifespans), the scene activates and my Alexa responds as expected. However, with long-running scenes, the scene "works" (in that the actions occur), but Amazon responds with, "The remote endpoint could not be called, or the response it returned was invalid."
Expected:
Even if it takes a bit for the scene to complete, I expect Alexa to see a successful output.
Problem-relevant configuration.yaml entries and steps to reproduce:
alexa:
  intents:
    ActivateSceneIntent:
  action:
    service: scene.turn_on
    data_template:
      entity_id: scene.{{ Scene | replace(" ", "_") }}
  speech:
    type: plaintext
    text: OK

scene:
  name: Good Night
entities:
  alarm_control_panel.simplisafe:
    state: armed_home
  cover.garage_door:
    state: closed
  lock.front_door_locked:
    state: locked
  lock.garage_door_locked:
    state: locked
  group.living_room_lights:
    state: off
  group.living_room_tv:
    state: off

Follow the guide on creating an Alexa intent to activate scenes.
Utilize the above scene and corresponding intent.
Launch the intent via the Alexa device or console.

Traceback (if applicable):
N/A
Additional info:
I imagine there's some sort of timeout going on with long-running scenes (which, in this example, I expect is SimpliSafe). In this case, my debug log shows Amazon firing the intent:
2017-07-23 02:01:02 DEBUG (MainThread) [homeassistant.components.alexa] Received Alexa request: {'session': {'application': {'applicationId': 'amzn1.askskill.45e173ad-5c02-44ba-ad43-fba96ad246c1'}, 'sessionId': 'SessionId.15821ecd-81a5-4796-8936-89497b63167d', 'user': {'userId': 'amzn1.ask.account.AEGLRE4CL3BAQI66HZABV2M5PVAH43R5KJ7MDGO72UY6KRYV3S7WCC5LEK5PSCMMM2QW3NNWQYGCLUOYKDKU2MF3WBJD2XSI6RVH7UR22QUQQAAV2IWZWF73Q7YG3Z4JJKVPKRDJN7QZDRYU5BPBELXLQFYRNJOXIO3TO6QUCVAJVH66JIO7F4I4KW5VXSSJMVTW5TTZ73UA3WI', 'accessToken': None}, 'new': False,'attributes': {}}, 'request': {'timestamp':<PHONE_NUMBER>603, 'intent': {'slots':{'Scene': {'value': 'good night', 'name': 'Scene'}}, 'name': 'ActivateSceneIntent'}, 'type': 'IntentRequest', 'locale': 'en-US', 'requestId': 'EdwRequestId.ebb87b64-cc29-4020-80c8-f59409f38d2b'}, 'context': {'System': {'application':{'applicationId': 'applicationId'}, 'user': {'userId': 'userId', 'permissions': {'consentToken': 'consentToken'}, 'accessToken': 'accessToken'}, 'device':{'supportedInterfaces': {}, 'deviceId': 'deviceId'}, 'apiEndpoint': 'apiEndpoint'}}}

...but no corresponding response ever comes back.

It seems the default timeout is around 10 seconds (that cannot be configured by users). If the skill does not respond within that window, it will time out.
Having so many activities in a scene may be time consuming. Limit the number of activities in a skill to increase the speed. You can also create an ActivateScriptIntent that can turn on a scene.

@arsaboo Appreciate the confirmation. Unfortunately, what you suggest (limiting activities) isn't feasible here; even if, say, arming the security system is a long-running action, removing it defeats the entire purpose of the scene.
Additionally, I'm not how your suggestion:

You can also create an ActivateScriptIntent that can turn on a scene.

...differs from what I'm already doing, minus naming the intent something different?
As I consider this, a workaround comes to mind: I can attach long-running actions (like turning on the above scene) to Emulated Hue switches and trigger them via Alexa that way; that should execute very quickly. I'll save intents for simpler items.


Delete Column During Update

Since the .update() method calls .mergeObj(), there does not seem to be any way to delete a column from a record in taffy once it is created. I have also tried to use function(){ delete this[column]; return this; } but this seems to have no effect ... in fact it doesn't even throw an error.
A. Is this intentional? I noticed that .merge() in lodash also lacks a method to remove a key from an object. Is best practice in nosql inspired databases not to remove columns?
B. Without the ability to delete a column, what is the best practice to ensure that this record does not appear in subsequent .filter() calls?
PS. I have cross-posted this issue on Stackoverflow

Can you help me understand your use case for removing columns? I could see it potentially being nice to keep your data structures clean, but not a breaking lack of feature. The recommended solution would be to either to remove the offending record entirely or set the offending column to a negative value and filter it out.
I did a little poking around and I'm about 90% sure you can use .extend() to add a custom method to remove the record in its current form and insert a new copy with the column(s) removed.

Yes. Remove + Insert = Update. That sounds like the best way to delete columns... and it will likely work for my use case as well.
I have been putting together an npm module which adds a sync method to Taffy for synchronizing data with a REST API. After working with PouchDB, I wasn't entirely pleased with the overhead it requires, and I didn't see anything like this for Taffy, so thought I'd wrap a function around Taffy which extends Taffy to include client server data synchronization and turn it into a module for others to use as well.
However, technically .update() in Taffy == merge and Remove + Insert == overwrite. So, since my wrapper uses onUpdate, onInsert and onRemove settings, there is the potential edge case that two transactions (DELETE and POST) will not equal one update transaction (PUT) if the triggered processes are non-blocking.
I haven't tested this yet to see... but if it is an issue, your suggestion to use extend to build a special method for this overwrite edge case may be the best option rather than adding overwrite as an option to .update(). Thanks.

So given your use case what I'd suggest not using the native onUpdate / onRemove methods and extend (replace) both .remove() and .update() with your own custom methods that do there own handling for changes (trigger calls to the REST api). That way you could have update always do a remove, insert (or make it smart enough to do it contextually based on what is changing). Then adding taffy and your custom script would give you the intended effect.

That's a good idea.
I've been thinking about creating a new method called .overwrite() which can be called instead of .update() if the goal is to delete columns or otherwise completely replace a record. This would allow me to map .update() -> onUpdate to 'PATCH' and .overwrite() -> onOverwrite to 'PUT' without interfering with the normal operations of update(), insert() or remove(). Since update and PATCH can both function as a merge (non-delete upsert) request, it becomes 'safer' for use cases where records which could be edited from multiple sources might create an inconsistent state at the cost of not being able to delete columns, while at the same time allowing a true PUT replacement to occur when circumstances deem it necessary at the cost of potential inconsistency.
It seems to me that this approach creates the smallest footprint on Taffy's existing architecture while achieving the goal of my use case. Neither insert nor remove need be tinkered with... overwrite would simply call both with the normal event triggers disabled. Once the record has been recreated, it would call its own event trigger which prompts a sync (PUT) request if auto-synchronization is enabled.
Since I am only creating overwrite() for the purposes of mapping PUT, I'm not sure that there is a need to mess with all the settings to have a real onOverwrite trigger... so what I have in mind is merely an internal trigger at the moment.


Document whether or not NetworkPolicy works in Docker Desktop for Mac

File: docker-for-mac/networking.md
If NetworkPolicy works in Docker Desktop for Mac, please document it.
If not, please add it to the "Known limitations, use cases, and workarounds" section.
I'm running Docker for Mac
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.1", GitCommit:"4485c6f18cee9a5d3c3b4e523bd27972b1b53892", GitTreeState:"clean", BuildDate:"2019-07-18T14:25:20Z", GoVersion:"go1.12.7", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.6", GitCommit:"96fac5cd13a5dc064f7d9f4f23030a6aeface6cc", GitTreeState:"clean", BuildDate:"2019-08-19T11:05:16Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes v1.14 has NetworkPolicy
However, when I kubectl apply -f the following file
kind: NetworkPolicy
apiVersion: networking.k8s.io/v1
metadata:
  name: default-deny-all-egress
  namespace: default
spec:
  policyTypes:
  - Egress
  podSelector: {}
  egress: []

I'm still able to send egress network traffic
$ kubectl run --generator=run-pod/v1 -i --tty netshoot --image=nicolaka/netshoot -- bash
If you don't see a command prompt, try pressing enter.

bash-5.0# curl https://example.com
<!doctype html>
<html>
<head>
    <title>Example Domain</title>
...

FYI this is the same for Windows & WSL2.
I have weave-net installed as my CNI, and yet my policies are ignored.
easy to reproduce following :
kubectl apply -f "https://cloud.weave.works/k8s/net?k8s-version=$(kubectl version | base64 | tr -d '\n')"
https://kubernetes.io/docs/tasks/administer-cluster/declare-network-policy/

Does someone manage to make NetworkPolicy work or have an answer on the question ?
I've created an issue

@etoews @ChrisScottThomas Did you manage to make it work ?

It sounds like they don't work on Docker for Desktop? That makes sense, as RBAC does not work either. We use K3D when we need that.

@alexec the problem with RBAC is different. They created an annoying ClusterRoleBinding that thwarts RBAC. Just delete it.
$ kubectl -n kube-system delete clusterrolebinding docker-for-desktop-binding

NetworkPolicies don't work on Docker-Desktop for Mac and getting them to work is difficult. You can't do much with the Docker-Desktop VM, because most of it's read-only; so Calico is out of the question.
For anyone else that just needs something to test on, I've just spent the last half hour on this and found that minikube works great.


[Question] Why not a PWA?

Firstly I like the idea of having a mobile app for coding-coach! And Xamarin is a great tool to work with!
But why not implementing as a PWA? With this it could be possible to reuse nearly all content from the website?
Or will it be like a Mobile App for the app store and still having a PWA ?
Just curious about it...

This app is for the app stores, my idea is also to use it as a playground for any mentee that wants to work with Xamarin. Probably the PWA is currently under development, you should ask in the front-end channel in slack.

@SergejDK also you can join the Xamarin channel in slack if you want to participate in the development.


Split page to make it more clear

Since the instructions are fundamentally different for Kubernetes 1.18 and higher, it would be better to split this into two pages:  one for versions below 1.18 and another for later versions of Kubernetes.

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

ID: 2ba1dad2-9792-2a7b-cdb6-68da0524cff3
Version Independent ID: 42736112-f6db-d241-e89b-48fd2c35ceb7
Content: Manage an Azure Kubernetes Service cluster with the web dashboard - Azure Kubernetes Service
Content Source: articles/aks/kubernetes-dashboard.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@CharlesCara
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@mlearned
Can you please check and add your comments on this doc update request as applicable.

Thank you @CharlesCara for the feedback.
We added a task to our backlog to consider this and will prioritize accordingly.
#please-close


Changes to UI?

It would be nice if we could have the UI come up with only the left part filled in, e.g. treat the Details groupbox as part of the per-pattern elements you get when selecting a pattern in the left-side list.
Maybe just put the three requirements boxes inside Details box and show that box only when a pattern is selected.
As it is the empty Name and Pattern boxes you get on first opening the dialog are misleading.

We would need to code a WPF behaviour that collapsed the right part when no branch pattern is selected. Since it is admin only ui it is not high on the priority list...


Do not see exception messages in console

Is any instruction how to set .env variable.
Probably thus I do not see any exception messages in console.
https://github.com/yiisoft/demo/blob/991551df1d761eb5370eb2e28a0717de3edd89ba/yii#L12

Is any instruction how to set .env variable.

env=test ./yii

Thank you


Storage disk not working

Issue description: Appears that some disks are not working in my disk drive
What happens: I insert them into my disk drive and although I can see them in the GUI it does not affect the total storage or update the block texture
What you expected to happen: Storage disk to hold items and display on block
Steps to reproduce:

Have Storage disk with no 0/XXXX stored.
Put into disk drive

Minecraft: 1.12.2
Forge: <IP_ADDRESS>38
Refined Storage: 1.6.15
Does this issue occur on a server? LAN server and Singleplayer

What kind of disk are you trying to insert into the drive?

Two 16K Storage Disks and two 256K Fluid Storage Disks. I am playing the current version of FTB Academy if that helps.

I can't reproduce this.


Adding updated notebook for "Exploratory Data Analysis with R and BigQuery" tutorial

@gogasca Hey Gonzalo - We have made some updates to this Notebook (mainly simplification and clean up). Could you please review and merge?

LGTM


Error after upgrading the Appium to 1.6.2.

After I have upgraded the Appium to 1.6.2 I keep getting this error when I'm opening the iOS inspector. "

Appium source at http://localhost:4723/wd/hub/bf439b9f-a7ce-4415-9fd5-0b5d071ea160/source cannot be received after 60 seconds timeout. Click this message to retry.

Thank you,

Please check https://github.com/facebook/WebDriverAgent/issues/372


.NET 8 (and possible all non-NETSTANDARD and higher versions) attempt to load EF 6 types

After upgrading one of our projects from .NET Core 3.1 to .NET 8 we were seeing a performance issue when using a Radzen data grid that is loaded or sorted for the first time when an entire list is directly bound to it. After some debugging I was able to narrow it to the usage of the OrderBy method in this library that is being used.
The static constructor of the PredefinedTypesHelper class attempts to load a set of types from Syste.Data.Objects and System.Data.Entity. There is a list of preprocessor directives around it which, I believe, should only be triggered in full .NET framework cases.
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/5cf83570b4f6f2d302a6cf669d48c538a78f1967/src/System.Linq.Dynamic.Core/Parser/PredefinedTypesHelper.cs#L54-L75
Our old implementation had a .NET Standard class library loading the Radzen library, which in turn loaded System.Linq.Dynamic.Core. This resulted in the NETSTANDARD directive being set, so the entire block was discarded. Our new implementation upgraded everything to .NET 8, were the NETSTANDARD flag is not set this the entire set of types is loaded.
Due to the nature of our deployment we have a lot of assemblies in the working directory which might or might not need to be loaded depending on the configuration. So they are not necessarily already present in the AppDomain but exist on disk in the current working directory.
The Type.GetType(typeName); call will force load all the assemblies in the current folder to be loaded in the AppDomain. If there are hundreds of assemblies there, this takes quite some time and results in the performance impact we are seeing.

This is probably related to / the same as #783.
Can you try the suggestion described there?

I initially thought so too, but my understanding is that the TypeFinder is not yet being used. When I'm following my stack trace, I seem to be getting into a path that will always do this. My experience is not the scanning of the assemblies that is taking a long time or causing the issue, it's actually forcing the load dozens of assemblies from disk which only then are attempted to be scanned for the full framework EF types.
The Radzen library calls the .OrderBy method directly:
https://github.com/radzenhq/radzen-blazor/blob/421bb3b7012dd59edf2a86995f9a61b3768d4321/Radzen.Blazor/RadzenDataGrid.razor.cs#L1728
Which is here:
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/f8eddd61cf5026bdd57b0de66694e124437b1bf0/src/System.Linq.Dynamic.Core/DynamicQueryableExtensions.cs#L1495
Which then calls the InternalOrderBy method: https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/f8eddd61cf5026bdd57b0de66694e124437b1bf0/src/System.Linq.Dynamic.Core/DynamicQueryableExtensions.cs#L1551
Which is instantiating the ExpressionParser:
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/f8eddd61cf5026bdd57b0de66694e124437b1bf0/src/System.Linq.Dynamic.Core/DynamicQueryableExtensions.cs#L1575
Which is instantiating the KeywordsHelper:
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/f8eddd61cf5026bdd57b0de66694e124437b1bf0/src/System.Linq.Dynamic.Core/Parser/ExpressionParser.cs#L78
Of which the static constructor is loading the PredefinedTypesHelper:
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/f8eddd61cf5026bdd57b0de66694e124437b1bf0/src/System.Linq.Dynamic.Core/Parser/KeywordsHelper.cs#L56
It's the static constructor of the PredefinedTypesHelper that loads all those assemblies from disk.
From my understanding, the IDynamicLinqCustomTypeProvider is only used after these classes have been instantiated.
The TypeFinder uses the KeywordsHelper, thus that static constructor will always be invoked:
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/f8eddd61cf5026bdd57b0de66694e124437b1bf0/src/System.Linq.Dynamic.Core/Parser/TypeFinder.cs#L24

I had some time to further test this.
So, there are 2 things in play:

The loading of the EF libraries from the PredefinedTypesHelper, which I was able to bypass by hooking into the AppDomain.CurrentDomain.AssemblyResolve event and skipping that if the requesting assembly is System.Linq.Dynamic.Core.

The force loading of all assemblies by the DefaultDynamicLinqCustomTypeProvider, which I could indeed bypass by setting my custom provider on the ParserConfig.Default.CustomTypeProvider.

For that, I believe the same as with the PredefinedTypesHelper, the conditions here should be extended with something like NETCOREAPP3_1_OR_GREATER:
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/f8eddd61cf5026bdd57b0de66694e124437b1bf0/src/System.Linq.Dynamic.Core/ParsingConfig.cs#L52
So, for now I've managed worked around it.

We see the exact same issue in our application (which happens to also use the Radzen DataGrid that ends up calling the .Order).
Our application is quite large with a lot of dependencies, which leads to a slowdown of 16 seconds on every first query after application startup, because of the loading of the assemblies.
I'm not sure how to implement a custom type provider as was suggested, but I'd very much appreciate any fix or workaround to improve the performance as the wait is considerable.

@BoudewijnPopkema
I've used the following implementation, which is a simplified type finder. It doesn't use anything related to the KeywordsHelper that exists in the default implementation:
public class CustomDynamicLinqTypeProvider : AbstractDynamicLinqCustomTypeProvider, IDynamicLinkCustomTypeProvider
{
    public HashSet<Type> GetCustomTypes()
        => GetCustomTypesInternal();

    public Dictionary<Type, List<MethodInfo>> GetExtensionMethods()
        => GetExtensionMethodsInternal();

    public Type? ResolveType(string typeName)
    {
        var assemblies = AppDomain.CurrentDomain.GetAssemblies();
        var type = ResolveType(assemblies, typeName);

        return type;
    }

    public Type? ResolveTypeBySimpleName(string simpleTypeName)
    {
        var assemblies = AppDomain.CurrentDomain.GetAssemblies();
        var type = ResolveTypeBySimpleName(assemblies, simpleTypeName);

        return type;
    }

    private HashSet<Type> GetCustomTypesInternal()
    {
        var assemblies = AppDomain.CurrentDomain.GetAssemblies().ToList();
        var hashSet = new HashSet<Type>(FindTypesMarkedWithDynamicLinqTypeAttribute(assemblies));

        return hashSet;
    }

    private Dictionary<Type, List<MethodInfo>> GetExtensionMethodsInternal()
    {
        var types = GetCustomTypes();

        var list = new List<Tuple<Type, MethodInfo>>();

        foreach (var type in types)
        {
            var extensionMethods = type
                .GetMethods(BindingFlags.Static | BindingFlags.Public | BindingFlags.NonPublic)
                .Where(x => x.IsDefined(typeof(ExtensionAttribute), false))
                .ToList();

            extensionMethods.ForEach(x => list.Add(new Tuple<Type, MethodInfo>(x.GetParameters()[0].ParameterType, x)));
        }

        return list.GroupBy(x => x.Item1, tuple => tuple.Item2).ToDictionary(key => key.Key, methods => methods.ToList());
    }
}

Then, in my Program.cs I've set this up using the following line:
ParsingConfig.Default.CustomTypeProvider = new CustomDynamicLinqTypeProvider();

@jnsn / @BoudewijnPopkema
Which project are you using?

System.Linq.Dynamic.Core ?
or
Microsoft.EntityFrameworkCore.DynamicLinq.EFCore8 ?

We are using 2. Microsoft.EntityFrameworkCore.DynamicLinq.EFCore8

I've added some workaround logic to only load the old EF when a special EFType is found.
https://github.com/zzzprojects/System.Linq.Dynamic.Core/pull/807
Would this solve your issue in .NET 8 ?

I'm using the Radzen library, so my use of System.Linq.Dynamic.Core is implicit.
At first glance, this would indeed solve the issue.
But as I mentioned earlier, some of these types are excluded using a NETSTANDARD preprocessor directive. Is there a reason these aren't excluded in anything more recent than NETSTANDARD, like NETCOREAPP3_1 and higher?

@jnsn
I did use the same workaround for NETSTANDARD in that file.
Can you please provide a simple console app which shows your issue? This makes it easier for me to check if this fix solves that problem.

@jnsn I did use the same workaround for NETSTANDARD in that file.

Maybe I'm not understanding it entirely. To me, it looks like these types should only exist in a Full Framework environment, as they are from EF 6. In .NET Core or .NET 5 and higher projects, these will never exist. Or at least never loaded from the Full Framework EF 6 libraries, but always from potential EFCore libraries, if at all.
The NETSTANDARD preprocessor directive is not set in .NET 5 or higher, so my initial idea was that the list should be extended with the NET5_0_OR_GREATER directive. This would mean that it's also excluded from any newer projects (like .NET 8).

Can you please provide a simple console app which shows your issue? This makes it easier for me to check if this fix solves that problem.

I've attached a ZIP file which contains 2 projects: linq-core-demo.zip

Demo.Plugin: A simple class library, which will be published to the output folder but not referenced by any project.
Demo.Host: A simple console app, which references System.Linq.Dynamic.Core, 1.3.13.

I've also included a build.ps1 file which will build the solution and publishes the Demo.Plugin and Demo.Host projects into a output folder. It's important to note in my scenario that the Demo.Host project never references the Demo.Plugin project.
The Demo.Host project can then be run from the output folder, which produces the following output:
PS E:\jnsn\linq-core-demo\output> dotnet Demo.Host.dll
11:50:57.127 [Warning] Attempted to resolve assembly "System.Data.Entity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089" by "System.Linq.Dynamic.Core"
11:50:57.145 [Warning] Attempted to resolve assembly "System.Data.Entity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089" by "System.Linq.Dynamic.Core"
11:50:57.146 [Warning] Attempted to resolve assembly "System.Data.Entity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089" by "System.Linq.Dynamic.Core"
11:50:57.147 [Warning] Attempted to resolve assembly "EntityFramework, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089" by "System.Linq.Dynamic.Core"
11:50:57.148 [Warning] Attempted to resolve assembly "EntityFramework, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089" by "System.Linq.Dynamic.Core"
11:50:57.148 [Warning] Attempted to resolve assembly "EntityFramework, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089" by "System.Linq.Dynamic.Core"
11:50:57.149 [Warning] Attempted to resolve assembly "EntityFramework.SqlServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089" by "System.Linq.Dynamic.Core"
11:50:57.150 [Warning] Attempted to resolve assembly "EntityFramework.SqlServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089" by "System.Linq.Dynamic.Core"
11:50:57.176 [Warning] Attempted to resolve assembly "Demo.Plugin, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null" by "System.Private.CoreLib"
11:50:57.252 [Information] Found 1 customers: [Customer {Id=0de3c65d-d6ee-4289-9aec-1eba2c8fc46d, Orders=[Order {Id=6acedbef-9570-4f99-a3de-27a51a577033}, Order {Id=9f464cc5-8c9a-455f-8a3d-ec33d9bab75d}, Order {Id=632afba1-387c-406a-aa7a-891d05e65af3}, Order {Id=f5cc1f15-0058-470a-bb8c-3161e49843de}]}]

As you can see from the output, all the EF types are attempted to be resolved from by System.Linq.Dynamic.Core. This isn't a major problem. They are not found, I log them and the program continues. So these are to me not really the issue.
The main issue is the attempted load of the Demo.Plugin assembly by System.Private.CoreLib, which is triggered by the DefaultDynamicLinqCustomTypeProvider, as I mentioned in an earlier reply. That's the case that should be avoided.

:one:
I did change the code for PredefinedTypesHelper so that it checks for
static PredefinedTypesHelper()
    {
        if (Type.GetType("EntityFramework.DynamicLinq.EFType, EntityFramework.DynamicLinq") != null)
        {
            TryAdd("System.Data.Objects.EntityFunctions, System.Data.Entity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089", 1);
            TryAdd("System.Data.Objects.SqlClient.SqlFunctions, System.Data.Entity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089", 1);
            TryAdd("System.Data.Objects.SqlClient.SqlSpatialFunctions, System.Data.Entity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089", 1);
            TryAdd("System.Data.Entity.Core.Objects.EntityFunctions, EntityFramework, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089", 2);
            TryAdd("System.Data.Entity.DbFunctions, EntityFramework, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089", 2);
            TryAdd("System.Data.Entity.Spatial.DbGeography, EntityFramework, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089", 2);
            TryAdd("System.Data.Entity.SqlServer.SqlFunctions, EntityFramework.SqlServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089", 2);
            TryAdd("System.Data.Entity.SqlServer.SqlSpatialFunctions, EntityFramework.SqlServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089", 2);
        }

        if (Type.GetType($"Microsoft.EntityFrameworkCore.DynamicLinq.EFType, Microsoft.EntityFrameworkCore.DynamicLinq, Version={Version}, Culture=neutral, PublicKeyToken={PublicKeyToken}") != null)
        {
            TryAdd($"Microsoft.EntityFrameworkCore.DynamicLinq.DynamicFunctions, Microsoft.EntityFrameworkCore.DynamicLinq, Version={Version}, Culture=neutral, PublicKeyToken={PublicKeyToken}", 3);
        }
    }

Which results now in:
12:54:46.910 [Warning] Attempted to resolve assembly "EntityFramework.DynamicLinq, Culture=neutral, PublicKeyToken=null" by "System.Linq.Dynamic.Core"
12:54:46.944 [Warning] Attempted to resolve assembly "Microsoft.EntityFrameworkCore.DynamicLinq, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=974e7e1b462f3693" by "System.Linq.Dynamic.Core"
12:54:47.006 [Warning] Attempted to resolve assembly "Demo.Plugin, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null" by "System.Private.CoreLib"

(Maybe this is not yet perfect, but it's a workaround)
:two:
Note that the last call is done via the DefaultAssemblyHelper which loads indeed all .dll files from the same folder where the main program is running. So in this case it also finds the Demo.Plugin.dll file
Loading from these extra files could be made configurable via the Config. So by default it will not load extra files which are not referenced by the main program.

I don't think that will solve the issue for me, as nothing changes in the behavior that I'm trying to avoid. It's still loading in the hundreds of Demo.PluginX DLL's that are in my folder.
From my testing, I believe the problem is that the Type.GetType() call force loads all the assemblies that were not yet loaded from disk into the AppDomain, and that should not happen.
I still believe the issue is located here:
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/f8eddd61cf5026bdd57b0de66694e124437b1bf0/src/System.Linq.Dynamic.Core/ParsingConfig.cs#L48-L58
As the comment on line 53 suggest, that should only be done in case of the full framework. I believe the fix is to add the NET5_0_OR_GREATER  directive to line 52.

A]
I've added if NET452_OR_GREATER || NETSTANDARD2_1 to PredefinedTypesHelper.cs to only load the older EntityFramework types for old NET-framework and standard2.1 (because EntityFramework can also be used for netstandard 2.1).
See:
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/da82150b984cd43616c54263b5bf50906d570db0/src/System.Linq.Dynamic.Core/Parser/PredefinedTypesHelper.cs#L64
B]
Adding NET5_0_OR_GREATER to line 52 will have a side-effect that no default CustomTypeProvider is loaded, which means that types with the [DynamicLinqTypeAttribute] will not be found anymore, which breaks current behavior.
C]

I don't think that will solve the issue for me, as nothing changes in the behavior that I'm trying to avoid. It's still loading in the hundreds of Demo.PluginX DLL's that are in my folder.

I've implemented option :two: by adding a extra config setting:
        /// <summary>
        /// Load additional assemblies from the current domain base directory.
        ///
        /// Note: only used when full .NET Framework and .NET Core App 2.x and higher.
        ///
        /// Default value is <c>false</c>.
        /// </summary>
        public bool LoadAdditionalAssembliesFromCurrentDomainBaseDirectory { get; set; }

Default it's false so the Demo.PluginX DLL's are not loaded.
When set to true, the Demo.PluginX DLL's in that folder are loaded.
See logging from that demo-app:
11:20:29.245 [Information] --- LoadAdditionalAssembliesFromCurrentDomainBaseDirectory = False ---
11:20:29.284 [Warning] Attempted to resolve assembly "Microsoft.EntityFrameworkCore.DynamicLinq, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=974e7e1b462f3693" by "System.Linq.Dynamic.Core"
11:20:29.390 [Information] Found 1 customers: ["Customer { Id = 2ff7ca04-0753-4b50-bd40-a99342cfb3af, Orders = System.Collections.Generic.List`1[Order] }"]
11:20:29.394 [Information] ********************************************************************************
11:20:29.395 [Information] --- LoadAdditionalAssembliesFromCurrentDomainBaseDirectory = True ---
11:20:29.422 [Warning] Attempted to resolve assembly "Demo.Plugin, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null" by "System.Private.CoreLib"
11:20:29.453 [Information] Found 1 customers: ["Customer { Id = 2ff7ca04-0753-4b50-bd40-a99342cfb3af, Orders = System.Collections.Generic.List`1[Order] }"]

@jnsn Did you have time to take a look at my comment?

Hi,
Sorry for the late reply.
This indeed looks like it would fix the issue for me. Thanks!


Android Basics: Viewmodel and State in Compose. Course text issue

This is an issue on the course text, not on the code. I hope this is the correct way of sumitting as the contributing guidelines do not say anything on the course content.
In step 5 Add a View Model
Section Display random scrambled word
item 3 (declare currentWord) should come before item 2, where it is used
alternatively the text at the end could say
current text: Android studio flags an error for the undefined function
corrected   : Android studio flags an error for the undefined variable and function

Hello @jderijke,
Thank you for reaching out to us and sharing your feedback. The text will be updated and published.


Allow Capture of Response Headers with Invoke-RestMethod

Problem
Invoke-RestMethod is extremely useful for working with APIs that return parsable objects. My single gripe has been that it is not possible to capture the response headers. Many API's provide (sometimes critical) details via the response headers, especially rate-limiting quotas and session timeouts. In situations where information in the the response headers are required, the user must currently fall back to Invoke-WebRequest and then perform their own Content-Type detection and response-to-object conversion.
Proposal
Add a -ResponseHeadersVariable parameter to the Web Cmdlets that accepts string. this string will be used as the name of a variable in the calling scope. This variable will be set with the same contents of BasicHtmlWebResponseObject.Headers. The user can then access the variable to retrieve the headers from the previews call to Invoke-RestMethod
Example:
$GetData = Invoke-RestMethod -Uri 'http://localhost:8083/Get/' -ResponseHeadersVariable 'ResonseHeaders'
$ResponseDate = [DateTime]$ResonseHeaders.Date[0]

This example converts the Date response header to a DateTime.
The functionality would be similar to -SessionVariable. This would be available for both Web Cmdlets, though it would be redundant on Invoke-WebRequest as it would be a mirror of the returned Headers property.

This is a great idea!


Getting optional values combining schemas

I am composing base schemas and then create other from those base ones. I am running into an issue that the base schemas keys are getting the optional: true property, they lose the require property and I am not able to change that.
const searchByFieldSchema = new SimpleSchema({
  searchBy: {
    type: [Object],
  },
  'searchBy.$': {
    type: String,
  },
  'searchBy.$.field': {
    type: String,
  }
});

const searchByStringSchema = new SimpleSchema([
  searchByFieldSchema,
  {
  'searchBy.$.value': {
    type: String,
  },
}]);

searchBy and searchBy.$ are getting optional:true.  Is that an expected behavior?

The extending is intentionally done in that way and is often useful. In SimpleSchema v2, you can pass requiredByDefault: false as an option to the constructor, and then it flips so that you do required: true in the definitions and otherwise they are optional. Alternatively, I think simply doing optional: false in the override in v1 SimpleSchema will work.


Add hidden files to gitignore

Can we either add .* to ignore all hidden files or add just .DS_store to ignore DS_Store files?

@djalova @bdwyer2
Any suggestion on this?

We don't want to add .* because that would ignore .dockerignore but we should add .DS_store and other files used by IDEs.

Is .dockerignore the only file that we need to force add if we include .*?

.gitignore, .travis.yml

I thought of suggesting explicit include using ! prefix. If there are many files, I am not sure if it makes sense.

we have added .DS_store to gitignore on other max repos previously so we should just add it here. wildcard ignores are generally not good practice unless in a subdirectory (eg tests)

Ref: #29


Subdirectories in Casks directory?

I already asked this question for Formulas (https://github.com/Homebrew/homebrew/issues/45046)
but I just wanted to be sure it's the same for Casks.
So is it possible to put casks into subdirectories in the Casks directory of a tappable repository?

Nope. Same as them.

…and let me guess: You wouldn't accept a pull request? ^^

Our conversation from the other issue has nothing to do with this. Like I said there, the work you put into it is still appreciated, and we’re not in the habit of declining stuff based on who they came from.
I never even considered the feature previously, so I don’t even have an opinion yet. In addition, other maintainers should also vouce their opinion.
That said, I don’t think you should spend the time on implementing it here, no, because we’re in talks of merging with homebrew (there’s a recent open issue, I’d link you, but am currently on a phone), so that work would likely end up being thrown out if/when that happens. If you submit it there and they accept it, though, perhaps both projects can benefit.


Entity resolution

I have the original skill working, but am having trouble getting the newer entity resolution feature added using synonyms. It would be great if you could update the documentation to explain how to use this feature.

Thanks for the suggestion.  We'll evaluate incorporating entity resolution / synonyms into the sample.  In the meantime, here are a couple of examples using entity resolution:
https://github.com/alexa/skill-sample-nodejs-petmatch
https://github.com/alexa/skill-sample-nodejs-the-foodie
https://github.com/alexa/alexa-cookbook/tree/master/feature-demos/skill-demo-entity-resolution


Disable tests in language-nix

They currently fail on 32-bit linux
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Hm. Actually looks like this is unnecessary...


Fix kotlin unit tests

Kotlin tests are run by a JUnit 5 test runner. The styx-proxy component has Kotlin unit tests as well as TestNG unit tests, but the Maven Surefire plugin does not, by default, run both JUnit and TestNG tests in the same project.
The surefire plugin needs to be configured with two executions - one each for JUnit5 and TestNG - and the default execution disabled.
Surefire determines which test framework to use by examining the test classpath. If it finds the appropriate dependency ("junit:junit", "org.junit.platform:junit-platform-engine" or "org.testng:testng" for JUnit4, JUnit5 and TestNG respectively) it executes that framework.
The two custom executions defined here disable the unwanted frameworks for each case, leaving only one that will run. It does this by changing the name of the dependency artifacts that the surefire plugin is looking for. (i.e. by setting the TestNG artifact name to "none:none", surefire will not find that on the test classpath and so will not try to run TestNG tests in that execution.)
The config parameters used are testNgArtifactName, junitArtifactName and junitPlatformArtifactName - as listed here: http://maven.apache.org/surefire/maven-surefire-plugin/test-mojo.html
The original inspiration for this approach came from here: https://issues.apache.org/jira/browse/SUREFIRE-377

Please add a quick comment in the POM file explaining how the TestNG and JUnit5 activation works.

I added details in the PR


Migrate Kafka console tool examples

What changed, and why it matters
Migrated https://help.aiven.io/en/articles/5344081-kafka-console-tool-examples-for-testing-aiven-for-apache-kafka

Thank you Tibs, I added the suggested changes

👍 Thanks @lornajane I was thinking the same myself.

Just trying to sort out the make linkcheck problems before merging.


Fix annotations for decorated classes

This patch fixes a problem with using dataclasses in Cython. Using Cython master, the following code:
# cython: language_level=3

import dataclasses

@dataclasses.dataclass
class Foo:
    bar: int

print(Foo(1))

Fails with:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "x.py", line 11, in init x
    print(Foo(1))
TypeError: __init__() takes 1 positional argument but 2 were given

This is because decorated classes do not have an __annotations__ attribute. The root cause of this problem is that annotations are looked for after wrapping the class with the decorator(s), and since SimpleCallNode does not forward the class' annotations, none are found. This patch fixes the problem by checking for annotations before wrapping the class.

@scoder I think you need to approve github-actions to run for this PR.

Beautiful little change with good tests. Thanks!


Replacing references of "kill" in documentation

I like these changes! We can also use "stop" as a synonym for "terminate" when talking about apps or nodes or operating system processes.


Fix scaling alert for SNO

Issue:  https://github.com/open-cluster-management/backlog/issues/13181

/lgtm


Validation Hangs

Changing the Identifier/"ProcessId" to numbers throws a 422 (Unprocessable Entity) error, and the user cannot make any further changes to the Process because it will no longer save. Even changing the Identifier back to "ProcessId" does not allow it to save.

Create New Process
Change Process Identifier from "ProcessId" to any string of numbers
Press save twice

Duplicate of #323


Add option for custom server filtering/mapping logic

Motivation
We use tags in Consul to attach metadata to service instances, e.g. version information, environments or proxy addresses to reach the service from outside the company network. A simple tag filter is not sufficient to express our needs when retrieving service information from the discovery.
Changes in this pull request
We introduced another option called mapServers which -- if set -- will be used to transform the list of services retrieved from the discovery service to a list of service addresses based on custom logic. If the option is not set, the default implementation will be used, making this change downwards compatible.

The PR is more than welcome! Well done!

Version 0.1.3 is available now via npm and bower.
BTW, I would appreciate a brief feedback from your side about Resilient:

Are you using it in node.js or webapps?
How many time are you using it?
Are you using it in production?
Which are your general impressions about using Resilient? It's easy to use? It's useful?

Wow, you're really fast. Thank You! We just started writing our own implementation of a resilient HTTP client when I read about resilient.js on reddit. Were not using it yet, but we are planning to abandon our implementation and use resilient.js, in node.js and in the browser. I'll let you know about our impressions.


CI: add workflow for building a windows binary

PyGPSClient Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Testing
Only test I did so far is make sure the software opens on windows.
Checklist:

[x] I agree to abide by the code of conduct (see CODE_OF_CONDUCT.md).
[x] My code follows the style guidelines of this project (see CONTRIBUTING.MD).
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] (if appropriate) I have added test cases to the tests/test_*.py unittest suite to maintain test coverage.
[ ] I have tested my code against the full tests/test_*.py unittest suite.
[ ] My changes generate no new warnings.
[x] Any dependent changes have been merged and published in downstream modules.
[x] I have signed my commits.
[x] I understand and acknowledge that the code will be published under a BSD 3-Clause license.

Hi @Williangalvani
Thanks for this contribution, but I'm not seeing a great deal of value in incorporating a PyInstaller-based deployment into the automated GitHub workflow.
The existing platform-agnostic pip install already creates an executable (in the Python scripts/bin folder) which runs fine under Windows 10 & 11, and I've had no reports of any difficulty deploying or running on Windows using this standard method.
Happy to be convinced otherwise (and happy to add your deploy.yml to the \examples folder for anyone who does want to use it).

Hi!
I am using PyGPSClient to debug some U-Blox GPSs over UDP (u-center just refuses to connect).
My goal with this PR is to avoid having inexperienced (windows) users having to install python and using the terminal for launching PyGPSClient.
I would like to have users just download a file from a release, double-click it, and that is it.

The existing platform-agnostic pip install already creates an executable (in the Python scripts/bin folder) which runs fine under Windows 10 & 11, and I've had no reports of any difficulty deploying or running on Windows using this standard method.

For someone who knows how to deal with python/pip that is fine. but I'm always reluctant in telling users (specially in windows where things never work the way I expect) to open the terminal.
Is there another way we could accomplish this?

Hi! I am using PyGPSClient to debug some U-Blox GPSs over UDP (u-center just refuses to connect).

Glad to hear PyGPSClient is proving useful :-).

My goal with this PR is to avoid having inexperienced (windows) users having to install python and using the terminal for launching PyGPSClient. I would like to have users just download a file from a release, double-click it, and that is it.
For someone who knows how to deal with python/pip that is fine. but I'm always reluctant in telling users (specially in windows where things never work the way I expect) to open the terminal.

Appreciate the intent but remain unconvinced this adds significant utility. Assuming Python is installed (I recommend the latest Python.org distribution rather than the Microsoft Store version), all we're asking the Windows user to do is open Terminal (which is a standard Windows app, available via Search or the CMD+R wt shortcut) and copy and paste a single command from the README. They would normally only need to do this once - subsequent updates can be accomplished using PyGPSClient's 'Check for updates' feature (provided the user has appropriate file system privileges). I myself have placed an iconised shortcut to the ../Scripts/pygpsclient.exe executable in my Windows 11 Start menu which works in exactly the same way as any other app icon:

If opening Terminal and/or creating a suitable iconised shortcut to ../Scripts/pygpsclient.exe is genuinely the issue here, I can happily provide (a link to) additional instructions in the README.

Is there another way we could accomplish this?

Aways open to suggestions and appreciate all contributions, but I have a number of reservations about the use of pyinstaller:

The principal intention behind PyGPSClient was to provide a platform-agnostic GNSS utility which works on practically all end-user operating systems and CPU architectures using - as far as is practicable - a single set of deployment artefacts and installation procedures. To my mind, Python, PyPi and pip meet these requirements quite happily.
As far as I am aware (I'll happily stand corrected on this point), pyinstaller is not a cross-compiler and can only create 'single-file' runtime executables for the platform it's run on - in the case of a GitHub Actions windows-latest workflow, that means Windows Server 2022 on x64 architecture. There is, as far as I am aware, no GitHub Actions support for Windows on ARM (e.g. for someone using the latest MS Surface Pros), and if you wanted to provide single-file runtime executables for other operating systems and/or architectures, you'd need a discrete pyinstaller workflow for each, which significantly increases the Actions runtime as well as introducing additional potential points of failure and an additional (unpaid) support overhead for me.
It would still be incumbent upon the end user to download the executable from the ../dist folder, save it to a location of their choosing (subject to applicable user file system privileges) and create an appropriate Start menu shortcut to this location. I would imagine any novice Windows user who struggles to open Terminal may well struggle to perform these steps.
For platforms that already have a Python environment installed, a pyinstaller 'single-file' executable bloats the file system by (in effect) replicating elements of this Python environment. On my Windows machine the resultant ../dist/pygpsclient.exe file was over 20MB in size - the ../Scripts/pygpsclient.exe created by pip is only 106kB.
The load time for the pyinstaller executable was over 5 seconds for every invocation on my Windows machine. The pip executable loads almost instantly.
Unlike the pip installation, the pyinstaller workflow would not create runtime executables for the various pygnssutils CLI utilities that come bundled with PyGPSClient.
I seem to recall older versions of pyinstaller (<=5.0) having multiple platform compatibility issues, though those seem to be largely resolved in the latest versions.

Happy to be challenged on any or all of these points.

These are fair points.
I agree with most of your points,  so I won't insist too much, but I do have some interesting notes:

There is, as far as I am aware, no GitHub Actions support for Windows on ARM (e.g. for someone using the latest MS Surface Pros)

Windows 11 on arm can run x86 apps on ARM, it seemed fine on a VM on my M1 Mac. It is not going to run at native performance, but pygpsclient is not a heavy software anyway, right?

It would still be incumbent upon the end user to download the executable from the ../dist folder

while this is not what this pr currently does, this is what I had in mind long-term: https://github.com/Williangalvani/PyGPSClient/releases/tag/test
just a one-click download and then run the binary
Upon re-reading the install instructions, I think some of my issues were because I didn't check the "Add to PATH" checkbox.

If opening Terminal and/or creating a suitable iconised shortcut to ../Scripts/pygpsclient.exe is genuinely the issue here, I can happily provide (a link to) additional instructions in the README.

Yes I think that would be great!

Given you do have strong points, I think I'll close this now.
Thank you for the feedback and keep up the good work =]

Given you do have strong points, I think I'll close this now. Thank you for the feedback and keep up the good work =]

Thank you for the contribution. FYI I've updated the README to include enhanced instructions on creating desktop application launchers for Windows, MacOS and Linux. I believe there are Python packages that will do this more-or-less automatically (e.g. https://pypi.org/project/pyshortcuts/) but I haven't evaluated them myself. I dare say it's also been raised as a feature request for the PyPi pip installer.


Maybe it is wrong

Yep, this is definitely wrong. :laughing:
I'll fix soon. thanks to report this.


支持GLM4微调

[ ] I checked to make sure that this is not a duplicate issue

Describe the solution you'd like
希望支撑GLM-4的微调

暂不支持，有qwen2-7b。


chore(pitr): pitr agent add test workflow

Type of change:

[ ] Bugfix
[ ] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

add pitr agent workflow.
Pre-submission checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?

Codecov Report

Merging #320 (1268c52) into main (46be6a7) will decrease coverage by 20.54%.
The diff coverage is 41.83%.

@@             Coverage Diff             @@
##             main     #320       +/-   ##
===========================================
- Coverage   62.36%   41.83%   -20.54%
===========================================
  Files           2       14       +12
  Lines         279     1310     +1031
===========================================
+ Hits          174      548      +374
- Misses        103      686      +583
- Partials        2       76       +74

Impacted Files
Coverage Δ

pitr/agent/internal/pkg/pkg.go
0.00% <0.00%> (ø)

pitr/cli/internal/pkg/shardingsphere-proxy.go
0.00% <0.00%> (ø)

pitr/agent/internal/pkg/opengauss.go
4.08% <4.08%> (ø)

pitr/agent/pkg/gsutil/conn.go
35.71% <35.71%> (ø)

pitr/cli/internal/pkg/local-storage.go
42.73% <42.73%> (ø)

pitr/cli/internal/cmd/restore.go
44.58% <44.58%> (ø)

pitr/agent/pkg/cmds/cmd.go
49.46% <49.46%> (ø)

pitr/cli/internal/cmd/show.go
56.09% <56.09%> (ø)

pitr/agent/pkg/xerror/error.go
61.53% <61.53%> (ø)

pitr/cli/internal/cmd/backup.go
63.15% <63.15%> (ø)

... and 4 more

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Formatting After function argument in call expression

TypeScript Version:  2.4.0 / nightly (2.5.0-dev.201xxxxx)
Code
Try formatting the following:
let a : any;
a(function () { })
     (a);

Expected behavior:
let a : any;
a(function () { })
(a);

Actual behavior:
Indentation is unchanged.

The actual behavior looks correct for me. Due to the actual JS behavior, when the (a); expression on 3rd line calls a function that was returned from the 2nd line. So, it behaves and looks like chained calls.
Another example. If we have the following function:
function harmfulResult() {
  return function() {
    // there may be a logic which will do a scary thing
    console.log('command was successful: rm -rf /');
  };
}

We may unintentionally write the following code:
harmfulResult()
(console.log('Everything is OK'))

And with the "expected" behavior it'll look harmless. However, actually, it'll remove your files.

thanks @igelbox you are right, this is a correct production per the grammar.


Assemble PhotonOS based VM

Feature Request
Detailed Description

VM must include all planned components
PhotonOS based
Include solo binary

no other dependencies other than docker

The assembly is expected to be manual, as opposed to tools driven (i.e. packer)
Context

Possible Implementation

Complexity

[ ] Low - Simple enhancement or bug fix, no architectural changes or refactoring
[ ] Medium - Change requires some thought, but is relatively isolated
[ ] High - Significant architectural change or large refactor

Yes, this is done


open()  + save() probably don't need to be in the C-core.

Since we can handle them via Lua too.

Actually I'm going to close this and leave as-is.  But I will implement "insert_file" as a lua-function.


Is there any plan to add page sizes dropdown?

Is there any plan to add page sizes dropdown?
By defining new array variable in grid options like 'pageSizes':[10,20,30] and it will create drop down control for page size options to user can dynamically change page size.
Thanks for considering request.

Hi @tabalinas  even i need pagesize dropdown,  is this enhancement done or can you suggest an alternative way to achieve this.

Hi,
jsGrid or alternative way.
<select id="pageSize" >
	<option value="10">10</option>
	<option value="30" selected="selected">30</option>
	<option value="50">50</option>
	<option value="100">100</option>
</select>

$("#pageSize").on('change', function(event) {
     $("#jsGrid").jsGrid("option", "pageSize", this.value);
});


fix: if null in array

Hello, in the case of an array's item is null, fuse crash. I just added a check, else return.

Is it possible to merge my PR ? It would help my project and others.
Thanks


Extensions Web View does not follow zoom

- VSCode Version: 1.3.0
- OS Version: Windows 10
Steps to Reproduce:

code --enable-use-zoom-for-dsf (Due to bug #1896)
Open an extension tab

this is #5745


protein modification regions [sf#108]

Reported by eilbeck on 2008-07-25 17:15 UTC
I am adding this to the tracker as I think Robin brought up a good point about capturing the region where a modification occurs - which is slightly different from the modification itself.  This discussion came from a request for terms to describe the GPI anchor. The suggestion is to make cross products with psi-mod.
--K
On 7/21/08 4:33 AM, "Robin Houston"<EMAIL_ADDRESS>wrote:
Interesting. Is it planned to move some of the SO terms into the PSI-
MOD ontology, then? It would be useful to see an explanation of the
(intended?) formal relationship between the two ontologies, if there
is such a thing.
To my naive eyes, it's not altogether clear how this fits what we're
looking for. MOD:00818 (glycosylphosphatidylinositolated residue) is_a
MOD:00000 (protein modification), whereas we would like a term to
describe a feature of the polypeptide (the GPI anchor). Is it planned
to introduce some cross-product terms into SO to make this possible
somehow?
Thanks,
Robin
On 17 Jul 2008, at 10:55, Sandra Orchard wrote:
> Hi Christian
>
> Sorry about the delay in getting back to you, but the status at the
> moment is
>
> GPI_anchor is already present (MOD:00818)
> I have spoken to John Garavelli about including the cleavage site in
> the MOD ontology as it really is a PTM - I'll let you know his
> decision.
>
> UniProtKB annotate the cleaved region as a propeptide, removed in
> the mature form and I agree with Gabby that this is covered by
> cleaved_membrane_spanning_domain. You can use the label feature to
> add more explanation if you wish.
>
> I hope this goes someway towards dealing with your query and I'll
> get back to you as soon as I have heard from John.
> Best wishes
> Sandra
>
>
> Gabrielle Reeves wrote:
>> Dear Christiane,
>>
>> Apologies for the multiple emails...  I am leaving the EBI and
>> therefore I am currently in the process of handing over the
>> responsibility of the protein feature terms to my colleague Sandra
>> Orchard, who is CCd on this email.
>>
>> Sandra will be very pleased to discuss these terms with you.
>>
>> Regards
>>
>> Gabby
>>
>> Christiane Hertz-Fowler wrote:
>>> Hi,
>>> sounds good. as far as the position in the ontology is concerned,
>>> is this where signal peptides sit as well?
>>> Christiane
>>> On 1 Jul 2008, at 12:11, Gabrielle Reeves wrote:
>>>
>>>> Dear Christiane,
>>>>
>>>> I can sort these terms out but help for definitions and
>>>> classification would be greatly appreciated.
>>>>
>>>> Presumably we need to add these terms:
>>>>
>>>> GPI_anchor
>>>> GPI_anchor_cleavage_site
>>>> cleaved_membrane_spanning_domain (we already have
>>>> cleaved_peptide_region - maybe this works?)
>>>>
>>>> I am slightly unclear of the exact biology.
>>>>
>>>> Can they all sit as a child of "immature_peptide_region ->
>>>> cleaved_peptide_region"
>>>>
>>>>
>>>> Gabby
>>>>
>>>>
>>>>
>>>> Karen Eilbeck wrote:
>>>>> Hi Christiane,
>>>>> This term does not exist currently, but we can make it for.
>>>>> Would you call it GPI_anchor_cleavage_site?
>>>>> I am hoping that Gabby is still around to help with this...
>>>>> Would you need a term also for the part of the protein that is
>>>>> cleaved and left in the membrane? What would the name be?
>>>>> Cleaved_membrane_spanning_doman?
>>>>> Could you both think about the position in the ontology and
>>>>> definitions for these terms? I can help too, but Gabby is in a
>>>>> better position for this.
>>>>> Thanks,
>>>>> Karen
>>>>> On 6/30/08 11:33 AM, "Christiane Hertz-Fowler"
>>>>><EMAIL_ADDRESS>wrote:
>>>>>    Hi,
>>>>>    I am trying to find the appropriate SO term to describe the
>>>>> cleavage
>>>>>    site for GPI anchor addition ... doe such a term currently
>>>>> exist in
>>>>>    the ontology?
>>>>>    Regards,
>>>>>    Christiane
>>>>>    --
>>>>>     The Wellcome Trust Sanger Institute is operated by Genome
>>>>> Research
>>>>>     Limited, a charity registered in England with number 1021457
>>>>> and a
>>>>>     company registered in England with number 2742969, whose
>>>>> registered
>>>>>     office is 215 Euston Road, London, NW1 2BE.
>>>>>

Commented by eilbeck on 2008-08-07 21:25 UTC
Logged In: YES
user_id=742851
Originator: YES
Hi Robin,
I have added 21 modified amino acid terms as children of amino acid to SO.
These terms can be used to annotate which feature (residue) is modified although the exact type of modification would still need to be specified further by another resource devoted to modifications such as psi-mod.
I have added  cross references for these terms to psi-mod, and included their names and synonyms as SO synonyms. I also created a new synonym category amino acid modification to distinguish these synonyms.
There are no plans to make cross products at this time.
Thanks for pointing this out.
--Karen

Updated by eilbeck on 2008-08-07 21:25 UTC

status: open --> closed-accepted


fix(fix bug , add nexttime feature): ServeFileDownload filename doubl…

…e quotes cause underscores before and after the final file

Codecov Report

Merging #1637 (5c23c0c) into master (68bdf7d) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1637      +/-   ##
==========================================
- Coverage   71.07%   71.04%   -0.03%
==========================================
  Files         444      444
  Lines       42294    42322      +28
==========================================
+ Hits        30059    30068       +9
- Misses      10340    10358      +18
- Partials     1895     1896       +1

Flag
Coverage Δ

go-1.15
71.04% <0.00%> (-0.01%)
:arrow_down:

go-1.16
?

go-1.17
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

net/ghttp/ghttp_response.go
27.05% <0.00%> (ø)

os/gcfg/gcfg_adapter_file.go
60.49% <0.00%> (-8.03%)
:arrow_down:

net/gtcp/gtcp_pool.go
34.09% <0.00%> (-4.55%)
:arrow_down:

protocol/goai/goai_response.go
80.95% <0.00%> (-3.54%)
:arrow_down:

os/gcfg/gcfg.go
56.43% <0.00%> (-2.06%)
:arrow_down:

protocol/goai/goai_shema.go
70.27% <0.00%> (-0.79%)
:arrow_down:

container/gtree/gtree_avltree.go
97.21% <0.00%> (-0.60%)
:arrow_down:

net/gtcp/gtcp_conn.go
31.15% <0.00%> (-0.51%)
:arrow_down:

os/gfile/gfile.go
75.86% <0.00%> (ø)

net/ghttp/ghttp_request_param_param.go
85.71% <0.00%> (+0.71%)
:arrow_up:

... and 3 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 68bdf7d...5c23c0c. Read the comment docs.


azure acs kubernetes browse: No such file or directory: 'C:\Users\username/.ssh/config'

When invoking az acs kubernetes browse -g <resource group> -n <name> I get the following error:
[Errno 2] No such file or directory: 'C:\\Users\\mrichman/.ssh/config'
Traceback (most recent call last):
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\main.py", line 36, in main
    cmd_result = APPLICATION.execute(args)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\core\application.py", line 212, in execute
    result = expanded_arg.func(params)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\core\commands\__init__.py", line 377, in __call__
    return self.handler(*args, **kwargs)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\core\commands\__init__.py", line 620, in _execute_command
    reraise(*sys.exc_info())
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\six.py", line 693, in reraise
    raise value
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\core\commands\__init__.py", line 602, in _execute_command
    result = op(client, **kwargs) if client else op(**kwargs)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\command_modules\acs\custom.py", line 154, in k8s_browse
    _k8s_browse_internal(name, acs_info, disable_browser, ssh_key_file)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\command_modules\acs\custom.py", line 164, in _k8s_browse_internal
    _k8s_get_credentials_internal(name, acs_info, browse_path, ssh_key_file)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\command_modules\acs\custom.py", line 835, in _k8s_get_credentials_internal
    '.kube/config', path_candidate, key_filename=ssh_key_file)
  File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\azure\cli\command_modules\acs\acs_client.py", line 63, in secure_copy
    conf.parse(open(ssh_config_file))
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\mrichman/.ssh/config'

Environment summary
CLI version 2.0.19 installed via https://aka.ms/InstallAzureCliWindows
c:\Users\mrichman\.ssh>az --version
azure-cli (2.0.19)

acr (2.0.13)
acs (2.0.17)
appservice (0.1.18)
backup (1.0.1)
batch (3.1.5)
batchai (0.1.1)
billing (0.1.5)
cdn (0.0.9)
cloud (2.0.8)
cognitiveservices (0.1.8)
command-modules-nspkg (2.0.1)
component (2.0.7)
configure (2.0.11)
consumption (0.1.5)
container (0.1.11)
core (2.0.18)
cosmosdb (0.1.13)
dla (0.0.12)
dls (0.0.15)
eventgrid (0.1.4)
extension (0.0.4)
feedback (2.0.6)
find (0.2.7)
interactive (0.3.10)
iot (0.1.12)
keyvault (2.0.12)
lab (0.0.11)
monitor (0.0.10)
network (2.0.16)
nspkg (3.0.1)
profile (2.0.14)
rdbms (0.0.7)
redis (0.2.9)
resource (2.0.16)
role (2.0.13)
servicefabric (0.0.4)
sql (2.0.13)
storage (2.0.17)
vm (2.0.16)

Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\mrichman\.azure\cliextensions'

Python (Windows) 3.6.1 (v3.6.1:69c0db5, Mar 21 2017, 17:54:52) [MSC v.1900 32 bit (Intel)]

Legal docs and information: aka.ms/AzureCliLegal

The same thing happens when I create a cluster use azure cli on Ubuntu.
az acs kubernetes get-credentials --resource-group myResourceGroup --name clusterName
The cluster is created successfully. But when I try to get credentials using the below command, I get the exception:
[Errno 2] No such file or directory: '/home/myname/.ssh/config' Traceback (most recent call last):   File "/opt/az/lib/python3.6/site-packages/azure/cli/main.py", line 36, in main     cmd_result = APPLICATION.execute(args)   File "/opt/az/lib/python3.6/site-packages/azure/cli/core/application.py", line 212, in execute     result = expanded_arg.func(params)   File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 377, in __call__     return self.handler(*args, **kwargs)   File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 620, in _execute_command     reraise(*sys.exc_info())   File "/opt/az/lib/python3.6/site-packages/six.py", line 693, in reraise     raise value   File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 602, in _execute_command     result = op(client, **kwargs) if client else op(**kwargs)   File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/acs/custom.py", line 814, in k8s_get_credentials     _k8s_get_credentials_internal(name, acs_info, path, ssh_key_file)   File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/acs/custom.py", line 835, in _k8s_get_credentials_internal     '.kube/config', path_candidate, key_filename=ssh_key_file)   File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/acs/acs_client.py", line 63, in secure_copy     conf.parse(open(ssh_config_file)) FileNotFoundError: [Errno 2] No such file or directory: '/home/myname/.ssh/config'

This issue is expected to be fixed in the latest release 2.0.20

cc @rjtsdl


Add compile option --strictNullChecks

Add compile option --strictNullChecks
Remove null or undefined checks from validate condition
Add DateValue as ValueObject

Coverage remained the same at 100.0% when pulling 03fbe983991056994f2ffc54947957dc7974350f on strict-null-checks into a90675d25db59d6190a60e4d06e698cad5d1421b on develop.

@yoshimasa36g
I saw some 'sapn'.

Coverage remained the same at 100.0% when pulling 31175bdc0e083ffe9ea32e2c41cb26d761595a95 on strict-null-checks into a90675d25db59d6190a60e4d06e698cad5d1421b on develop.

@MasayukiFukada Oops, I Fixed it.

OK.
LGTM

Merge #15 and fix null checks

Coverage remained the same at 100.0% when pulling 1a78a04e1d9f8c1dd05f177ca43194da19c67096 on strict-null-checks into b0c91aeb9d10de26b4654b906d5903b7478c8ac7 on develop.


⚠ Bofrost.lu has degraded performance

In 0f27b2b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1209 ms

Resolved: Bofrost.lu performance has improved in aed0771 after 7 minutes.


A new web component is needed to translate markdown into html sanitized

We use the marked npm package in the brands site to display class documentation. We need something similar in RP, to allow custom formatted markdown to be used to output people data. We could use another package like DOMPurify to sanitize the text.
<ucdlib-md ignore="H*"> - Means don't render it exclude="img" - Means excise it from the output subset?="core|full|parsimonious" - Specifies some common sets of ignores/excludes, that we use
The docs show the ability to override a list of renderer functions which would override the default renderer. We could use these to ignore specific tags if specified in the element attributes. From the docs-
const renderer = {     heading(text, level) {       return something;     } };
Then marked.use({ renderer }); with our list of overridden render functions. In our case, we would return an empty string or perhaps the default text, depending on the element attributes.
We can override these functions:
Block-level renderer methods
code(string code, string infostring, boolean escaped)
blockquote(string quote)
html(string html)
heading(string text, number level, string raw, Slugger slugger)
hr()
list(string body, boolean ordered, number start)
listitem(string text, boolean task, boolean checked)
checkbox(boolean checked)
paragraph(string text)
table(string header, string body)
tablerow(string content)
tablecell(string content, object flags)

Inline-level renderer methods
strong(string text)
em(string text)
codespan(string code)
br()
del(string text)
link(string href, string title, string text)
image(string href, string title, string text)
text(string text)

These elements shadowDOM styling needs to match the patternLabs styling.

@UcDust Looks pretty good to me.  My thinking is that we can have a few default renderers that have our overridden functions for various types.  Can you be thinking of how we can be showing examples of different MD renders?

IMHO, if you want flexible client side markdown rendering, just use the NPM packages.  ie:
document.querySelector('#md-rendered').innerHTML = sanitize(mdRenderer(mdText));

I would suggest the advantages of a custom element, that we put time into supporting, would be that there is an opinion layer.  So we are saying; this is our renderer, this is our sanitize utility, this is our standard setup.  Of course you can allow for a couple optional params.  The question I would have, is that even helpful?  Perhaps a little.  Our standard setup with single npm install command.
If we are going to implement a markdown element, the rendered markdown should be in the local dom.  So if you have <ucdlib-md id="md"></ucdlib-md> and did something like document.querySelector("md").data = mdTxt then the markdown HTML would render into the local dom. Or if you wanted to declaratively set the markdown you could do something like:
<ucdlib-md id="md">
  raw markdown goes here
</ucdlib-md>

And then the element hides the raw and inserts a new rendered tag as child element.
You could also look to do something like:
<ucdlib-md id="md">
  raw markdown goes here
</ucdlib-md>

Completely replacing the raw markdown with the rendered, however it doesn't provide a nice update cycle.  ie it's rendered and done.

Doesn't the document.quewrySelector solution suffer from the same update problems?   I guess one thing I think is nice is that the custom elements are more declarative, you add it in and it works.
Does it make sense to think about the old polymer "everything is an element" and have an engine-like thing:
<ucdlib-md-renderer render-class="md-rendered" subset="parsimonious"> which then sort of does what you suggested?
I guess markdown can get really messy anyways but is it better to use a tag and avoid things like


Clarify admin UI dev instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

lgtm! Definitely clears up the .env bits.


Add named pipes support

Similar to the .Net SqlClient behavior:

Automatically infer named pipe connection from the np: prefix on the server name or from a name formatted as a UNC path like \\servername\pipe\sql\query
Allow packages to register a Dialer instance associated with a protocol. EG the core package will register a tcp: dialer and a new namedpipe package will register a np: dialer during init()
Enable the app to specify a global setting that is the order of which dialers to try when there is no prefix on the server name. The default will be tcp only.

the connection timeout is cumulative for all dialers

Fixed by #61

I want to use this, can you add a tag?


Support for in-class templates?

It seems that currently CodeMaker supports only the creation of new classes, any thoughts on supporting in-class templates just like toString or equals?
So that we could customize templates for adding any codes to an existing class? More specifically:

Support adding arbitrary contents. (e.g. many fields/functions)
Support the selection of members. (just like the default implementation for toString)
Support overwriting existing methods/fields.

Ideally, the built-in getter/setter, toString, hashCode/equals could be replaced by CodeMaker.

Close this. I've made a plugin for myself https://github.com/lotabout/CodeGenerator. Thanks very much for your idea and code base.


Rightmost profile link in a blurb starts highlighted

Also, the highlight for each link is too low.
(I wish I had any idea how to CSS so I could fix it instead of just pointing it out...)

Yup, noticed this too. I might just turn the highlight off.
I'm looking at alternatives to this dialog modal so the whole thing might get restyled and this won't be a problem anymore

Fixed


README_TH.md - TH translation for 'Branch Rules' section

改善詳細 / Details of Improvement

Translate English to README_TH in 'Branch Rules' section

スクリーンショット / Screenshot

期待する見せ方・挙動 / Expected behavior

New translation in README_TH.md, it affects GitHub page

動作環境・ブラウザ / Environment

macOS / Windows / Linux / iOS / Android
Chrome / Safari / Firefox / Edge / Internet Explorer

I close this issue as the related PR has been merged.