Unmarking an attribute by prepending "-" doesn't work

Describe the bug
The linguist docs shows that prepending the linguist attribute with "-" will unmark it but that doesn't work. In this page I would expect the hidden file to be visible.
Expected behaviour
I would expect that prepending "-" would do the same thing as appending "=false" like in this page.

It should and does definitely work. I notice your fork is really old and doesn't include the override.  This definitely used to work, we've made no changes in Linguist around this, and Linguist definitely acknowledges these the same:
➜ ls -lR */package-lock.json
-rw-r--r--  1 lildude  staff  24 15 Sep 09:21 a/package-lock.json
-rw-r--r--  1 lildude  staff  24 15 Sep 09:21 b/package-lock.json
-rw-r--r--  1 lildude  staff  24 15 Sep 09:38 c/package-lock.json
➜ cat .gitattributes
a/package-lock.json -linguist-generated
b/package-lock.json linguist-generated=false
➜ github-linguist a/package-lock.json --json | jq
{
  "a/package-lock.json": {
    "lines": 3,
    "sloc": 3,
    "type": "Text",
    "mime_type": "application/json",
    "language": "JSON",
    "large": false,
    "generated": false, <----- cos of -linguist-generated
    "vendored": false
  }
}
➜ github-linguist b/package-lock.json --json | jq
{
  "b/package-lock.json": {
    "lines": 3,
    "sloc": 3,
    "type": "Text",
    "mime_type": "application/json",
    "language": "JSON",
    "large": false,
    "generated": false,  <----- cos of linguist-generated=false
    "vendored": false
  }
}
➜ github-linguist c/package-lock.json --json | jq
{
  "c/package-lock.json": {
    "lines": 3,
    "sloc": 3,
    "type": "Text",
    "mime_type": "application/json",
    "language": "JSON",
    "large": false,
    "generated": true,  <----- cos of no override
    "vendored": false
  }
}
➜

I suspect something has changed on the GitHub side. I'll do some more research and see what I can find.

Oh wow!!! I've just found the minus syntax has never worked in diffs on GitHub since Linguist introduced support for the minus syntax way back in 2020 in https://github.com/github-linguist/linguist/pull/4780. I've opened a PR to correct this. I'll close this issue once it's been deployed.
I feel really silly for continuously telling peeps this does work when it clearly doesn't 😊

Things should now be working and all looks good from your original link @SupaJoon


ion-content scrollTo not working

Ionic version:  (check one with "x")
2.2.1
I'm submitting a ...  (check one with "x")
[ ] bug report
Current behavior:
When I use ion-content in complex view, there is 1 nested ion-content inside main ion-content that allows scrolling in the app. Please see the screenshot:

When the list inside nest ion-content added new item, I want to scroll to that item position. But not works.
@ViewChild('ordersContent') ordersContent: Content;

this.ordersContent.scrollTo(0, position); // not works
this.ordersContent.scrollToBottom(); // not works

this.ordersContent.isScrolling => false => why ?

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Your system information:

 ordova CLI: 6.5.0
Ionic Framework Version: 2.0.1
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.1
Xcode version: Not installed

Hi there! How can we replicate this? The issue template included a link to a V2 plunkr.
Please try to recreate this in a plunkr or some sort of demo.
We need one in order to debug.

Hello all! As it seems it has been a while since there was any activity on this issue i will be closing it for now. Feel free to comment if you are still running into this issue. Thanks for using Ionic!

I have the same issue here after I upgrade my ionic framework to 2.2.1, actually I believe it is related to ionic-angular 2.1.0. I was using ionic-angular 2.0.0-rc2 and it was working fine, but after I upgrade it to ionic-angular 2.1.0, then the scrolling features are not working. After I downgrade it to 2.0.0-rc3, then it works again.
I was following the instructions on https://ionicframework.com/docs/v2/api/components/content/Content/
I don't know how to replicate this on plunkr, maybe it is just me having this case.

I am having the same issue like @CarsonChen1129 . After i updated ionic-angular from 2.0.0-rc3, content.scrollTo stopped working.

I came here to follow up my questions before, my problem was resolved by adding a resize() function after every scrolling. Hope this helps.


Drag & Drop functionality + Upload status notification on Tempus

Child of #953

Do not need this anymore.


Make list of available mutants more compact

The current version v10 of muter lists all mutants only printing their file name to the console. In my project, I have almost 2k available mutants and I can't see it all because of a limited number of lines in iTerm. If it doesn't display detailed info about mutant, it should display by default a compact list like e.g.:
1) CustomViewModel.swift (5 mutants)
2) CustomViewController.swift (2 mutants)
3) SomeModel.swift (12 mutants)

Your project has a total of 19 mutants in 3 files that will be tested.

Thanks for this feature request @sprzenus! You'll find it implemented it in v11, which will be released shortly. :)


if it's possible to loose docstring-parser to ^0.16 for conflict with instructor

Hi @brentyi, thx for making this amazing project!
Currently,
https://github.com/instructor-ai/instructor/blob/main/pyproject.toml restricts docstring-parser = "^0.16"
this usually be parsed by pip to be >=0.15, <0.16.
However, tyro uses docstring-parser>=0.16, so I always have pip conflict when trying to install them both under the same environment.
So I wonder if this dependency in tyro can be loosed? If this is for some compatibility consideration, I will also make an issue to instructor repo.
Thx!

I downgraded tyro to 0.8.4 and this solves the problem.

Hi @zhuoqun-chen, recent versions of tyro should work fine with 0.15 also. I'll relax this requirement in #213.


Flux metadata lists the previously loaded checkpoint

Expected Behavior
When looking at the metadata for previous image generations, the actual checkpoint used for the generation should be listed.
Actual Behavior
The metadata for images generated with Flux still list the previously loaded checkpoint.
Steps to Reproduce
After loading the provided example workflow for Schenll and sending it to the Generate tab:
User comment	{
  "sui_image_params": {
    "model": "sdxlpony/DucHaiten-Pony-Real - Pony-Real_v10",
    "aspectratio": "Custom",
    "width": 1024,
    "height": 1024,
    "comfyrawworkflowinputtextcliptextencodenodetextg": "a bottle with a rainbow galaxy inside it on top of a wooden table on a table in the middle of a modern kitchen",
    "automaticvae": true,
    "seed":<PHONE_NUMBER>,
    "prompt": "",
    "negativeprompt": "",
    "swarm_version": "<IP_ADDRESS>",
    "date": "2024-08-02",
    "generation_time": "0.01 (prep) and 17.08 (gen) seconds"
  }
}

When checking all the options on the left, their metadata are listed but it still lists the previous checkpoint.
Examples:

Debug Logs
Not relevant.

Other
No response

This isn't a bug, this is a jankily configured custom comfy workflow. You do in fact have a stray pony model loading there, as the comfy workflow isn't taking on the Swarm model parameter (normally this automates itself with CheckpointLoaderSimple node, but Flux uses the UNETLoader node so it doesn't automate atm)

I see. So whatever model I last loaded will always be (apparently) there while I'm using Flux?
That is not what I said at all?
This is a problem specific to the comfy workflow setup you have, and is fixed by either just not doing the custom workflow, or by linking the swarm model param (Just make a primitive SwarmUI: Model and link it to the unetloader input)

That looks correct.
I can't diagnose that error without more info.
I just did myself to test:

and it works as intended:

This is on a very clean workflow though, so there might be something more to what's going wrong.
It might be in the swarm interface you have the wrong model selected? It should autoswap to the one in your workflow but sometimes that doesn't happen right

make sure your Swarm install is up to date, Flux support was naturally just added yesterday

Oh, nope, that was wonked in the code from the Flux update - my bad, fixed.


Added support for GROUP all operations MAX, MIN, SUM and AVG

CLA is valid!

Coverage decreased (-0.07%) to 98.721% when pulling 1448484d46cef13119970320b405c6ee9aff959e on group-all-operations into 1a2c6ab86e385dbd9ce6b01ef18ca9f31cfa14af on master.

Coverage decreased (-0.07%) to 98.721% when pulling 1639c38ecf51ad5fe0ecd2f58fd7da6ce48edcf3 on group-all-operations into 1a2c6ab86e385dbd9ce6b01ef18ca9f31cfa14af on master.

Coverage decreased (-0.07%) to 98.721% when pulling b416b26dacf7767c29746274ca0bc516cc80ba5e on group-all-operations into 1a2c6ab86e385dbd9ce6b01ef18ca9f31cfa14af on master.


pre-configured "null" webapps

The webapps in the list all show up as "null"

closing, see https://github.com/tobykurien/WebApps/issues/15


Add --save_references_path into args

Arguments like --save_generations --save_generations_path --save_references exists, but --save_references_path misses. We can add it and it does not affect existing code.

Hello :) I am new to this and I do not know why this https://github.com/bigcode-project/bigcode-evaluation-harness/actions/runs/6604468842/job/17970522424 failed. I have made few modifications about args. Can you kindly help me review this PR? Thank you~


Crash after adding character/Icon not showing up

So, im making a mod called 'Get Fake' and there seems to be a problem with adding the characters when i reload the character

(^seconds before it crashes....)
guest's .json is located in Get Fake/images/characters
guest's icon is located in Get Fake/images/icons
if you know what im doing wrong feel free to point it out as soon as possible.
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

themod/images/characters are where the spritesheet goes
the jsons go in the themod/characters/ folder

No luck


cli: fix panic in cilium bpf sha get command

Fixes #8423
Fixes panic issue in cilium bpf sha get command when no sha is provided

Signed-off-by: Deepesh Pathak<EMAIL_ADDRESS>

This change is

Coverage decreased (-0.02%) to 44.187% when pulling cdf672baf69e1fa47c3cd50591e8290ecfc6ee31 on fristonio:fristonio/cilium-cli-issues into 83f68ae55288cb9174efa44e332282c0b9b5d395 on cilium:master.

test-me-please

@fristonio This looks great. Do you still want to work on it or can we merge it?

@tgraf since he moved it to being 'Open' from 'Draft' PR status, that signifies it is ready for review - I think we can merge.

Yeah, Thanks @ianvernon
I will be fixing the other issue in a separate Pull Request.


ID field is displayed as a truncated value

The ID field is displayed truncated, but the EdgeX API is not able to find via truncated IDs, it needs exact IDs.

There is a shortened ID field on:
Gateway
Device
Device Service
Device Profile
Export
Rule Engine

@brandonforster ok, assign it to me , i know the reason

@badboy-huaqiao I don't have permissions on this project to assign.

@brandonforster ok, i assign it to myself

i have fixed , i close it now


LUCENE-10493: move n-best logic to analysis-common

This is a follow-up of https://github.com/apache/lucene/pull/805 - NBest logic can be separated from Japanese-specific stuff, and reside in analysis-common.
This doesn't reduce code duplication but tries to clarify the boundary between graph (or lattice) construction and language-specific tweaks. Also, this may allow KoreanTokenizer to support n-best as well as JapaneseTokenizer if the feature is worth adding to it.

@mikemccand thank you for revewing!


Can't install sharp on windows(x64).

i have install<EMAIL_ADDRESS>same issue with install bcrypt.
`C:\Users\dell>npm install sharp -g

><EMAIL_ADDRESS>install F:\Program Files\nodejs\node_global\node_modules\sharp
> node-gyp rebuild

F:\Program Files\nodejs\node_global\node_modules\sharp>if not defined npm_config_node_gyp (node "F:\Program Files\nodejs\node_global\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )

`

Hello, are the Windows pre-requisites for node-gyp installed? This usually include the Visual C++ Build Tools.

I don't  install  the Visual C++ Build Tools  , because the prompt error: already installed C++ Build Tools Visual. Python has been installed: 2.7.11 Python. Studio Visual 2015 I have been updated and selected Common Tools for Visual  C++ during setup.

Thanks for confirming, are you able to share the full log, perhaps as a gist, when running npm install --verbose sharp (preferably without the -g global flag)?

This appears to be a cross-post of https://github.com/nodejs/node-gyp/issues/1034 - please feel free to re-open if not.


Request: Add Tanto view to normal views list (Linear/Hex Editor etc)

Request: Add Tanto view to normal views list (Linear/Hex Editor etc)
i.e. add it to this view:

This would make it much more discoverable in addition to allowing us to switch to it easily without needing a separate pane etc.

It also gets you location sync. 😜

(planned, along with loads of other goodies, for the to-be-announced 2.0 release)


Update components.yaml

Part of the master -> main rename. Changes some develop: branches to the new name.

Closing in favor of new omnibus PR


HSEARCH-2126 Checkstyle fails when explicitly called via goal checkstyle:check

@gsmet: can you run this on your machine as well?

I still have an error
[ERROR] /data/home/gsmet/git/gsmet/hibernate-search/orm/src/test/resources/org/hibernate/search/test/spatial/hibernate.cfg.xml:1: Line does not match expected header line of '<?xml version="1.0" encoding="UTF-8"?>'. [Header]

utf-8 should be UTF-8 in the header. Once this one is fixed, it works OK.

I added some additional fixes that showed up (for some reason).

We never meant to run checkstyle on XML files. The formatting improvements in these changed XML look fine, but do we want to enforce it on XML?

I don't have a strong opinion about it but it doesn't look like too much of a hassle.
Just wanted to have checkstyle:check working.

@Sanne:
Well, I haven't added any rules regarding xml. Also, this
<module name="Header">         <property name="header" value="<?xml version="1.0" encoding="UTF-8"?>\n<!--\n ~ Hibernate Search, full-text search for your domain model\n ~\n ~ License: GNU Lesser General Public License (LGPL), version 2.1 or later\n ~ See the lgpl.txt file in the root directory or <http://www.gnu.org/licenses/lgpl-2.1.html>.\n  -->"/>         <property name="fileExtensions" value="xml"/>     </module>
suggests that at some point this was intended.

@Sanne: bump

yes we can merge this, but beware I think people will eventually make XML formatting mistakes and the regular build wouldn't catch it (the same way it didn't catch these before).
Could we adjust this to make sure the checkstyle:check coverage is the same as when doing a normal (full) build?

Errors in .xml files are also reported during a normal build, but as it seems the default behaviour has changed. We could just disable all checkstyle checks on .xml files apart from the header check, this would do it as well, right?

@Sanne : i changed it so only the Header check is run on xml files.

@s4ke try this: remove the header from one of the files you fixed, then run the build again. It won't complain...
That's what I meant by saying that it's not regression-proof.
Sure, it's an improvement in style but I'm pretty sure it won't last if it's not automatically prevented ;)
If you want I can merge this.. not a problem. Just saying that if we could enable the automatic check as well, then you'll have solved the problem for longer term.

This is what I'm testing:

I fetch your branch
I revert your commit 3541607
I run mvn clean install

I would expect it to fail, but it's not. Am I running the wrong Maven goal?
BTW please squash 2232f66c4f74c7c58f6e547564139274f1975086 into its parent ;)

@Sanne:
During the build checkstyle:checkstyle is used which generates reports. For some reason this apparently behaves differently to checkstyle:check when it comes to error checking.

Awesome, seems you untangled the mystery :+1:
Merged


Weird AppMenu behaviour on page re-render

The AppMenu component sort of reloads when any action triggers a page re-render. This only happens when the AppMenu bar is on a specific media query range, as presented on the video below.
https://user-images.githubusercontent.com/76881129/185696954-840d178c-d460-4ebf-88f6-cb78d53757ae.mp4

Magically solved.


added friendly_id

Fixes #2055
Describe the changes you have made in this PR -
Added friendly_id for users model.
Screenshots of the changes (If any) -

Coverage decreased (-0.9%) to 83.735% when pulling 7b187f5a05db6e3921cdfc9a6800aac5cfaf2548 on SingingApple:friendly-users into 129182c0f422c9482abe149f730e03292ec1a7ef on CircuitVerse:master.

Coverage decreased (-0.9%) to 83.735% when pulling 7b187f5a05db6e3921cdfc9a6800aac5cfaf2548 on SingingApple:friendly-users into 129182c0f422c9482abe149f730e03292ec1a7ef on CircuitVerse:master.

@gr455, Can I work on this issue?


Use specific index commands instead of one general one

Some data structures are likely to need specialized ones, such as JavaScript's Map and Set.

Split out into getter issues.


How do I install Issacsim 2023.1.1

In the Nvidia Omniverse launcher I can no longer find the 2023.1.1 version of Issacsim, what should I do to install the correct version?

You may use the container installation of Isaac Sim 2023.1.1 if no workstation version is available in Omniverse Launcher. You can use streaming client with native=True to achieve a similar experience as workstation.
https://github.com/OpenRobotLab/GRUtopia/blob/aadfca80080f26e06360bb216d47f491553a271c/demo/h1_locomotion_native.py#L15
Currently we are working on migrating GRUtopia to Isaac Sim 4.x version, and will release it when stable.


cephfs: disallow creating small volumes from snapshot/volume

as per the CSI standard, the size is an optional parameter, as we are allowing the clone to a bigger size today we need to block the clone to a smaller size as its a have side effects like data corruption, etc.
Note:- Even though this check is present in the kubernetes sidecar as CSI is CO independent adding the check here.
fixes: #2718
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@mergifyio rebase

Jan 17 11:36:11.447: FAIL: failed to create storageclass: etcdserver: request timed out

@ceph/ceph-csi-maintainers @ceph/contributors need a review if anyone has free time.

@mergifyio rebase

Actually this will never happen from Kube CO and also on Openshift. In our documentation we have mentioned, only those as tested or Known to Work https://github.com/ceph/ceph-csi#known-to-work-co-platforms .. The  confusion this could bring here is, if we started to think about other COs , then many areas have to be looked upon in various  code paths..

Actually this will never happen from Kube CO and also on Openshift.
In our documentation we have mentioned, only those as tested or Known to Work https://github.com/ceph/ceph-csi#known-to-work-co-platforms .. The confusion this could bring here is, if we started to think about other COs , then many areas have to be looked upon in various code paths..

we never mentioned that this will won't work or we should only fix it as per the kubernetes/OCP requirement. most of the time we will try to align with the standard CSI specification.

@nixpanic do you have any comments on this one?

@mergifyio rebase

@mergifyio rebase

@mergifyio refresh


enhance the nav

Add a link to create issues from the nav

Deploy preview processing.
Built with commit ccb2acbd24d49508dc3e121dc6a3b304cdc3905a
https://app.netlify.com/sites/open-sauced/deploys/59e1c35da6188f4cdde1ce54


fix(): use console warning for zoid version mismatch

Fixes #225
For various reasons it's not always possible to deploy the parent / child at the same time.
Currently zoid crashes on a version mismatch even if there are no breaking changes between two versions.
Rationale for using console.warn() instead of new Error():

In case of a breaking change the software still crashes with a warning on console
non-breaking changes allow independent deployments (think of a security patch)

Ultimately it's the developers responsibility to make sure that the software is working as intended and that there are no console warnings.

@bluepnume any reason why this wasn't merged? Does PayPal want to intentionally throw error on version mismatch?

@songz -- it's a choice between:
a) Fail at this point with a clear error message
b) Fail at some future indeterminate point due to some unforeseen incompatibility between the messaging protocol in two different zoid versions (e.g. some new key expected in payloads, etc.)
We were hitting b) a lot. So a) at least gives a super clear reason.
Typically with zoid, the expectation is you create a CDN script with your component + the zoid library, and make sure you're loading the exact same CDN script in the parent window and in any child frames. That avoids any such compatibility issues, and it also has performance benefits (you usually don't want to force your user to download multiple versions of zoid).
^ We should probably document the above a little better.

Typically with zoid, the expectation is you create a CDN script with your component + the zoid library, and make sure you're loading the exact same CDN script in the parent window and in any child frames. T

Could you point out where paypal/paypal-checkout-components does this.


fix: cross-site post form submissions are forbidden when using load balancer in front

In my use case

end-client -(https)-> GCP load balancer -(http)-> docker containerized app using svelte-adapter-bun

key points

gcp load balancer request to my svelte app by using http://my-domain.com url but https is set to X-Forwarded-Proto. <-- so I want to modify request's base-url on every request. (even ORIGIN === get_origin(request.headers) case).
docker containerized app such as k8s, google-cloud cloudrun uses health check request which dose not contain x-forwarded-for header. <-- I want to ignore "ADDRESS_HEADER" check of those health-check requests

(GCP load balancer http-header forwarding rules are described here)
I think this PR will help issue #54

Do we have any updates on this? Will this be merged? Seems like the issue still persists

Please merge this - it's the only thing holding us back from deploying it in production, due to issue #54 which If I understand correctly, is related to this :)


Argmax unexpected key and Cant convert Cuda tensor to Numpy error

Hi I am facing the issue below. I have installed fast-bert using pip and just copied the code from the readme.
Any suggestions on how to fix?
model/tensorboard
Traceback (most recent call last):
  File "/usr1/home/rjoshi2/envs/myenv/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 61, in _wrapfunc
    return bound(*args, **kwds)
TypeError: argmax() got an unexpected keyword argument 'axis'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "train_bert.py", line 67, in <module>
    main()
  File "train_bert.py", line 62, in main
    optimizer_type="lamb")
  File "/usr1/home/rjoshi2/envs/myenv/lib/python3.7/site-packages/fast_bert/learner_cls.py", line 406, in fit
    results = self.validate()
  File "/usr1/home/rjoshi2/envs/myenv/lib/python3.7/site-packages/fast_bert/learner_cls.py", line 524, in validate
    all_logits, all_labels
  File "/usr1/home/rjoshi2/envs/myenv/lib/python3.7/site-packages/fast_bert/metrics.py", line 15, in accuracy
    outputs = np.argmax(y_pred, axis=1)
  File "<__array_function__ internals>", line 6, in argmax
  File "/usr1/home/rjoshi2/envs/myenv/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 1153, in argmax
    return _wrapfunc(a, 'argmax', axis=axis, out=out)
  File "/usr1/home/rjoshi2/envs/myenv/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 70, in _wrapfunc
    return _wrapit(obj, method, *args, **kwds)
  File "/usr1/home/rjoshi2/envs/myenv/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 47, in _wrapit
    result = getattr(asarray(obj), method)(*args, **kwds)
  File "/usr1/home/rjoshi2/envs/myenv/lib/python3.7/site-packages/numpy/core/_asarray.py", line 85, in asarray
    return array(a, dtype, copy=False, order=order)
  File "/usr1/home/rjoshi2/envs/myenv/lib/python3.7/site-packages/torch/tensor.py", line 433, in __array__
    return self.numpy()
TypeError: can't convert CUDA tensor to numpy. Use Tensor.cpu() to copy the tensor to host memory first.

I even tried to install fast-bert using github. Same error.

Also, this error comes after 1 epoch is done and for train and validation.

This occurs with the new version released today. Try pip installling version 1.5.0.

should be fixed in v1.5.2.  please check.

Tried v1.5.2 also. Same problem.

@josh-cooper 's answer (v1.5.0) fixed it!

will this be fixed in the new version? I tried 1.6.0, same problem.

facing same issue in lastest version, i belived its 1.60 , i am using google colab

still getting the same issue for me as well
can't convert CUDA tensor to numpy.


Consider Major Version Update For Breaking Changes

Hi,
I saw a similar thread here, but wanted to bring this issue up to stand on its own. My team and I spend quite a while debugging an obscure issue in our lower environments, to eventually track it down to the parameter order swap in propEq during the 0.28 to the 0.29 release.
Like the linked thread mentioned, our CI process (and I would wager most others out there) auto increment packages for patch and minor version changes. Please consider updating your versioning to match semvar practices to prevent issues like this in the future.
I've written a LOT of code at previous companies using propEq and I can only imagine the fires happening over there.

Like the linked thread mentioned, our CI process (and I would wager most others out there) auto increment packages for patch and minor version changes. Please consider updating your versioning to match semvar practices to prevent issues like this in the future.

The observed behavior is compatible with item 4 in the linked document. Any CI that treats x increments in 0.x.y as anything other than a major bump needs to be fixed. Also not all packages follow semver anyways. Single package upgrades gated by end to end tests should make it doable to identify upgrade issues.

Any CI that treats x increments in 0.x.y as anything other than a major bump needs to be fixed

This is a very naive and egotistical way of thinking, declaring that the entire industry must conform to your specific way of versioning your software. When the industry as a whole has created standards that state that x is in fact a minor update and not a major one.
Regardless of whether your project follows semvar practices or not, the platform that you publish you code on (npm) and the tooling the industry has build around JS package management does make general assumptions when it comes to package updating, specifically when it comes to how to version your software so you are not responsible for breaking other peoples code.
As an aside, the particular change that was made here that swaps the parameter order or the propEq function goes against ramda's design philosophy of "data last". This function is now data first, and cannot be partially applied without first flipping the parameter order. To propEq function should be renamed to eqProp be more semantically inline with the change.

Are you suggesting we notify npm and yarn and have them adjust the way they treat dependency versioning?

I'm not. That's how they work since ever. They take a conservative approach for versions styled 0.x.y because that's how some projects used to operate in the early days of nodejs, including this one it seems. Think about it, a software versioned 0.x.y is quite expected to have breaking changes all the time, otherwise they could have gone with 1.x.y.
Some CI tooling would also propose semver-major updates, so I don't see what difference does it make were 0.28.x to 1.0.0.

As an aside, the particular change that was made here that swaps the parameter order or the propEq function goes against ramda's design philosophy of "data last". This function is now data first, and cannot be partially applied without first flipping the parameter order. The propEq function should be renamed to eqProp be more semantically inline with the change.

I see a lot was discussed around this topic #2938 #3369. Introducing an export under a different name while deprecating the older one could be a better approach to avoid breaking changes. I think avoiding a breaking change is more valuable than going for it while declaring it a breaking change with a major version.

I wasn't aware that npm or yarn treated 0.x.y packages differently. I know it's been discussed many times before in previous issues, but is there an update on the reason ramda doesn't want to move to a 1.x.x version? It's been around for quite a while now and has quite high adoption. I personally think the 0.x.y days might be behind you. Not just for jumping on the semantic versioning wagon, but to declare that this library is beyond experimentation and ready for some stable releases.

is there an update on the reason ramda doesn't want to move to a 1.x.x version?

There is no strong reason. We have had many discussions over the years about what should get done before 1.0.0. I'm of the mind we should just go for it. I will continue to discuss it with @Harris-Miller and I believe it will get done soon, say in the next month or three.

Awesome, love that. I'll keep my eyes out for the possible change.


balena-raspberrypi: replace references to resin-vars

Replace all references to the 'resin-vars' script with 'balena-config-vars' as it has been renamed.
This PR needs to be merged when https://github.com/balena-os/meta-balena/pull/2142 has been merged.
Change-type: patch
Changelog-entry: balena-raspberrypi: replace references to resin-vars
Signed-off-by: Mark Corbin<EMAIL_ADDRESS>

@balena-ci I self-certify!

@resin-jenkins retest this please


Promises

@hairyhenderson  - Have you considered retooling the lib to use Promises? I'm considering a fork to make this happen using Q.

@ingageco - I hadn't considered it, but I'm not opposed to it. Is Q the best approach? There's native Promise support in JavaScript now, which is supported in Node.js 4 & 5. I'd much rather use the runtime support rather than add a dependency...
But, it would mean dropping support for Node.js 0.10/0.12, which I'm not opposed to, but not quite ready to do :)
Which version of Node are you using?

So I've been working on this a bit, and I've hit a snag...
Promises can only resolve a single value, so functions like F1.get_token, which yields two (creds, url), won't work as-is.
After looking through the project I think get_token is the only offender, so it might be legit to just deprecate that function (at least for Promise-based use).

Hey Davy - looks great. Unfortunately, the office staff at the church is so bent on leaving F1 that I've not worked much with it. They're looking to close the deal on a new system very soon, so I've decided to put my efforts into other areas. To be honest, I'm disappointed - you've done the best work here that Ive seen for integrating with the F1 api and I was hoping to get to work with you on it.

@ingageco - lol. Actually, we're looking at alternatives too - out of curiosity, what system is your church switching to?
The reason I started writing this library initially was to try and ease some frustrations that my church was having with F1's interface... It's a common theme with F1 users, I think :wink:

They're looking at a newer company called Seraphim - can you connect via email? jay [at] weingage.com

e-mail sent :smile:

Won't be implementing this - we're not using F1 anymore...


reply: support tts

Add tts (text-to-speech) field in poise::reply::CreateReply. Backwards-compatible, defaults off.
I use this functionality in my bot and am migrating from serenity's standard framework, which supports it.

Try again with the rebasing, next, not serenity-next.

Hey, since this hasn't been touched in a couple weeks I'll close this. If you want to continue working on it, just let me know and I'll re-open.


Fixed IE11 format bug

Fixed formatting issues that made the modal and carousel unusable in Internet Explorer 11
Fixes issue #304
Tested on

Windows 10 1903

Chrome 75
Internet Explorer 11
Firefox 67

iOS 12.3.1

Safari

Android 9

Firefox 68
Chrome 75

Checks:

[x] Please confirm yarn run lint ran successfully
[x] Please confirm that only /src and /examples/src are committed

Moved to pull request #329


Add Comments to front-end

For LoggedInUserContext, you can comment Allows us to globally access the logged in user instead of having to manually pass it through props. TODO: Change our code to use this context.


[feat]:issue #49 (Footer Elements must be in center) has been solved

After solving the issue
👇

Kindly mention the issue this pr solves so that it will be easier to award points.

@SSHSRN kindly check it now

#49

Sorry for the delay.

Thanks for the contribution!

Thanks for the contribution!

Thanks for accepting my contribution


Extract symbols from .so files

Describe the solution you'd like
A function to extract symbols on the AppCenter side when the obj directory generated when building Android is compressed and uploaded.
Describe alternatives you've considered
I tried to build and generate dump_syms command from Google Breakpad repository.
In macOS used for development and CI, dump_syms command that supports only Mach-o was generated, and it was difficult to generate commands that supported Android ELF.
Additional context
I was migrating from HockeyApp and HockeyApp had this feature.

Thanks for the request! Right now, we only support extracting symbol files if you use App Center build and distribute. We don't have any immediate plans to change this but I'll monitor this thread for support and post here with any updates. Thanks!

Hi,
My name is Yukio Murakami.
Thank you for your reply.
Future trends will be monitored!

2019/08/30 8:51、Winnie Li<EMAIL_ADDRESS>Thanks for the request! Right now, we only support extracting symbol files if you use App Center build and distribute. We don't have any immediate plans to change this but I'll monitor this thread for support and post here with any updates. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/microsoft/appcenter/issues/884?email_source=notifications&email_token=ACFNHPPSAGE6ARVAP2JMGIDQHBOJLA5CNFSM4ILQOZY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5QEA6I#issuecomment-526401657, or mute the thread https://github.com/notifications/unsubscribe-auth/ACFNHPNDMLWYB454OS56TITQHBOJLANCNFSM4ILQOZYQ.

Hi @m-yukio - we just enabled support for this. You can find the docs here: https://docs.microsoft.com/en-us/appcenter/diagnostics/android-ndk#generate-a-zip-file-to-upload
Give it a try and let me know if you run into any issues.


docs: concurrency env variables

The concurrency for cds-* and hdi-* commands can now be controlled through the env variables

MTX_CDS_CONCURRENCY
MTX_HDI_CONCURRENCY

This behavior should be documented.

Fixed by https://github.com/cap-js-community/mtx-tool/commit/759ce8c1eb5c9938299319b4b55cf3fe47c96c89


Add a tag to corresponding version change

Can you please along with changing version in VERSION file add corresponding tag to Releases. It'll update composer database and will be reflected here: https://packagist.org/packages/mikecao/flight
It will also allow to install those versions via Composer. Right now latest version available via composer is v1.2.13 but in fact latest version v1.2.18
Thanks.

Thanks, I always forget to do that. Tag pushed.


Ruby on Rails Group Project.

In this project, I , @assadounto  and @RitobrotoMukherjee  have implemented the following;

Followed the layout of the wireframes provided. You should personalize the rest of the design including colors, typographies, spacings, etc.

Login page and registration page:
built with Devise gem.

Food list:
displays a list of food added by the logged-in user as in the wireframe (for teams with only 2 people - display also quantity of a given food).
leads to a form that allows users to add new food.

Recipes list:
displays a list of recipes created by the logged-in user as in the wireframe.
leads to recipe details.
If the user is the owner of the recipe, it allows the user to delete it.

Public recipe list:
displays a list of all public recipes ordered by newest as in the wireframe.
leads to recipe details.

Recipe details:
If the recipe is public or the user is the owner of the recipe, displays the recipe details as in the wireframe.
If the user is the owner of the recipe, should lead to the form that allows the user to add new food.

Inventories list:
displays a list of inventories created by the logged-in user as in the wireframe.
leads to inventory details.
If the user is the owner of the inventory, should allow to delete it.

Inventory details:
displays the inventory details as in the wireframe.
leads to a form that allows users to add new food.

Recipe details:
has a modal to choose an inventory to create a shopping list with, as in the wireframe.

Inventory shopping list:
shows the list of food that is missing by comparing the food in the recipe with the food in the inventory.
counts the total food items and total price of the missing food.

@codepantha Thank you so much for approving our project


Bug with positioning on overflow window

When i click on calendar icon, flatpickr makes nasty screen scroll.
http://joxi.ru/8An0REOHvbdDmO

Link?

@jaredatch what?)

Can you provide the link to where this can be viewed?

@jaredatch this is the same issue as #22

Ah ok, for some reason from my previous testing I thought it accommodated that already.

Closing as a duplicate of #22


scripts: Fix CMake spelling

As per its creators, CMake is written with a capital "M".
If approved I'll also fix the spelling in areas other than the scripts folder.
Note: CI (compliance checks) is failing due to what seems unrelated findings? I'd rather not address that in this PR. suppressed

Any chance to get this merged before RC1 hits? :pray:

Any chance to get this merged before RC1 hits?

that is a fix that can be merged after RC1 as well.

Please rebase on main and resolve any merge conflicts.

Please rebase on main and resolve any merge conflicts.

@nashif , can you please approve again.


Support typing.TypeVar annotations

Hamilton fails on TypeVar annotations.
Current behavior
If you define functions like the following:
import typing

StrSeries = typing.TypeVar("StrSeries", bound="pd.core.series.Series[str]")

def normalize_situs(situs_addr: StrSeries) -> StrSeries:
    normalized = situs_addr.map(str.upper)
    return normalized

def foo_bar(normalize_situs: StrSeries) -> StrSeries:
    return normalize_situs

and then try to create a DAG it breaks.
Stack Traces
Traceback (most recent call last):
  File "sf/hamilton/examples/temp/run.py", line 21, in <module>
    dr = driver.Driver({'situs_addr': pd.Series(['a', 'b', 'c'])}, repo)
  File "sf/hamilton/hamilton/driver.py", line 47, in __init__
    self.graph = graph.FunctionGraph(*modules, config=config, adapter=adapter)
  File "sf/hamilton/hamilton/graph.py", line 183, in __init__
    self.nodes = create_function_graph(*modules, config=self._config, adapter=adapter)
  File "sf/hamilton/hamilton/graph.py", line 117, in create_function_graph
    add_dependency(n, node_name, nodes, param_name, param_type, adapter)
  File "sf/hamilton/hamilton/graph.py", line 78, in add_dependency
    if custom_subclass_check(required_node.type, param_type):
  File "sf/hamilton/hamilton/graph.py", line 41, in custom_subclass_check
    if issubclass(requested_type, param_type):
TypeError: issubclass() arg 1 must be a class

Expected behavior
Hamilton should be happy to handle TypeVar defined types.
Library & System Information
E.g. python version, hamilton library version, linux, etc.
Steps to replicate behavior

define functions above into module.
Run

import repo
dr = driver.Driver({'situs_addr': pd.Series(['a', 'b', 'c'])}, repo)

Additional context
Flagged by discord user https://discord.com/channels/842466558483496997/842467113611821087/955688991749980200

It's easy to fix enabling the DAG to be created:
Add
inspect.isclass(requested_type) and inspect.isclass(param_type) and


Merge all data in main branch

Upload all data to the main branch. To generate new python library.

Well done team!!!!


checkov: 3.2.266 -> 3.2.267

Diff: https://github.com/bridgecrewio/checkov/compare/refs/tags/3.2.266...3.2.267
Changelog: https://github.com/bridgecrewio/checkov/releases/tag/3.2.267

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

x86_64-linux

:white_check_mark: 2 packages built:

checkov
checkov.dist


Numbered buildings in German addresses

For query:
infanteriestr. Gebäude C
the result is:
"house": "infanteriestr. gebäude c"

but the expected result is
"road": "infanteriestr.",
"house": "gebäude c"

For similar query:
bahnhofstr. Gebäude C
the result is proper:
"road": "bahnhofstr.",
"house": "gebäude c"

Checked on v1.1 and master using src/address_parser.

Hi @Aknilam, the v1.1 branch is work-in-progress and should not be used. Our releases take a little more time than in other software libraries because we have to first generate the planet-wide training data so the parser has sufficient examples of the patterns we want, then train the model on a billion examples, then upload to S3 and release it so people can use it. It's not just a simple code change to change behavior (or even if it is, switching branches doesn't help unless there's a new model). So v1.1 is not functional yet, and in fact performance will likely be worse when compiling on that branch because it will be using different code with a model which was trained on and expects 1.0. As such, only master should be used. When I have new models available on S3, I'll merge the new branch and release 1.1.
The 1.1 release will include this case (however, since things like "Gebäude C" are relatively rare in OSM, we will have to generate them like we do unit/floor numbers, and instead of "house" it will be labeled either as "house_number" or possibly a new tag for "building").


Support loop replaceStream

I have a object for replacement which key is search and value is replace, I use Object.keys to loop the object and to replace the stream, but failed, code like
const loop = {path1:'Addr1', path2:'Addr2'}
const str = fs.createReadStream(filePath, 'utf8')
    .pipe(loopStream.bind(this, chunk, encoding, done));

function loopStream(chunk, encoding, done) {
  let regex
  Object.keys(loop).map(key => {
    regex = new RegExp(key, 'gi')
    replaceStream(regex, loop[key])(chunk, encoding, done)
  })
}

I got a error
(node:23686) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 5): ReferenceError: chunk is not defined

node -v is 8.5.0
replaceStream version is 4.0.3

Support loop replaceStream

Not needed. Use /Addr(1|2)/ in this case.

ReferenceError: chunk is not defined

It's not a bug of replacestream. Just an error of your code.

No, I wanna say one  regex of search with a alone replace, not one  regex of search include every thing.
and, above code is a demo to help you understand what I want,  I do not want say it's a bug, I wanna add a feature.

So, use switch statement in the replaceFn.

above code is a demo to help you understand what I want

Broken code doesn't help understanding, or rather has the opposite effect.

So, use switch statement in the replaceFn.
I wanna say one regex of search with a alone replace, not one regex of search include every thing. They are different, if use /Addr(1|2)/, Addr2 will never be replaced when match Addr1, because it's OR.

Broken code doesn't help understanding, or rather has the opposite effect.
Sorry for that

You probably wanna ask this on Stack Overflow.
Next time please write more detailed issue description, for example expected input - output pairs.

thank you very much, I miss g flag in regex, thanks again for your patient


add support for VS like smart indentation in vscode

this issue is related to this issue on LSP (https://github.com/microsoft/language-server-protocol/issues/1326)
VS has this API (https://docs.microsoft.com/en-us/dotnet/api/microsoft.visualstudio.text.editor.ismartindent?view=visualstudiosdk-2019) that VS editor uses to put caret at right location while user is moving around in the code using arrow up/down or hitting "enter" which put caret at next blank line.
currently vscode uses text (regex) based indentation which sometimes not enough for more complex expression to put caret at desired location, it would be nice if vscode has API similar to VS smart indenter which in turn can be used by LSP implementation to take advantage to ask LSP server desired indentation using syntactic information.

This is about having the VS Code API. Assigning to Joh.


Kaisong1990/update changelog pipeline

Proposed changes
Reused the yml file from CPP team. When changelog is not updated by developer, a conversation will show and ask to confirm if this is expected. No failures will be triggered anymore.
Type of change

[ ] Feature work
[ ] Bug fix
[ ] Documentation
[x] Engineering change
[ ] Test
[ ] Logging/Telemetry

Risk

[ ] High – Errors could cause MAJOR regression of many scenarios. (Example: new large features or high level infrastructure changes)
[ ] Medium – Errors could cause regression of 1 or more scenarios. (Example: somewhat complex bug fixes, small new features)
[x] Small – No issues are expected. (Example: Very small bug fixes, string changes, or configuration settings changes)

Additional information

If you didn't change changelog file in the future, the checks will not fail anymore. Instead, it will show a new conversation as above image and asking if not changing is expected.


Christmas wish (Request): Support for iTunes/Apple Music

Hey,
we talked about it so I thought to open an extra issue/request.  :)
Would be very nice :) ❤ As I occasionally switch between my iTunes library and Spotify :) 🎵
Thank you for considering it. 👍

The player I used before I discovered SpotMenu was "MiniPlay" from Denys Star (twitter: @zyafa) which can handle both iTunes/Apple Music and Spotify. However the development of app seem to be stopped. Maybe Denys could help to implement AppleMusic/iTunes support.

I also found this 😊
https://developer.apple.com/documentation/mediaplayer

This would be awesome! Currently using also "MiniPlay" which is available in the MacApp Store.
Which bugs were fixed... But this app to be seems very nice. iTunes Support would be the cherry on the cake though.

How should this work, should it automatically detect which player's info it should show/control (iTunes or Spotify) or should there be a switch somewhere in the app?
I think it would be best if:
If Spotify is open then Spotify is used
If iTunes is open then iTunes is used
If they're both open and none of them or both of them are playing music-> Spotify is preferred
If both are open and only one is playing music -> the one that is playing music (or atleast the last one that was playing music) is used

Yeah that sound very good. Maybe there could also be an indicator in the widget or menu bar which player is currently used (e.g. Spotify icon or iTunes/Apple Music icon and also the button in the left upper corner of the cover). Or maybe the color of the slider (time)
 could be blue for iTunes and green for Spotify or something like that.
Maybe the preferred player if both are open and none playing could be selected in the settings.

Apple seems to work on its Music Widget as well. But no idea if they support 3rd party apps like Spotify.
https://9to5mac.com/2017/07/24/macos-high-sierra-beta-4/

Another interesting project: https://github.com/xzzz9097/Muse

@fabi94music Nice! if I can ever afford a touch bar Macbook, I'll keep that project in mind! (Looks almost as awesome as Spotmenu :D)

Any updates on that? :)

Hey i am currently working on ditching SpotifyAppleScript and using ScriptingBridge instead. I have some working code, but i need to port it to swift 4.0

initial support for iTunes has been added
SpotMenu.zip
👆🏻here's a sneak peek

SpotMenu.zip
ive improved itunes support
known issue:

if neither player was playing before spotmenu was opened -> play, next, prev buttons dont do anything until music is started manually (by keyboard keys or from the music player)

It does not open on my Mac. :(

SpotMenu.zip
yep, sorry, I just noticed I forgot to embed the music player framework in the app, thats why it didnt open.

with that version the Notification Center widget freezes and seems not to update anymore.

Sorry, mea culpa. Easy fix though.
SpotMenu.zip

Thanks, now it doesn't work with iTunes :) The menu bar widget works fine with iTunes :P

SpotMenu.zip

if neither player was playing before spotmenu was opened -> play, next, prev buttons dont do anything until music is started manually (by keyboard keys or from the music player)

fixed this

sorry to bug you again, but I think this didn't solve the issue I was having. I attached a screenshot. The menu bar widget does well but the notification centre widget seems to struggle 🙈

did you completely delete SpotMenu.app from your Applications folder and replace it with the new version?

yep, and I even removed it with AppCleaner, that removes additional files.

does it work with Spotify?

Yes it works with Spotify 😃 !

okay, then i know what the problem is. i just need to fix it now 🤦♂

SpotMenu.app.zip
how about now? this should fix it hopefully 🙏🏻

Perfect :) Only a quick link to iTunes (in the menu bar widget its the upper left icon on the album art) in the notification center widget would be great :) I have added German translations on the localisation issue :)

SpotMenu.app.zip
that was a bug 🐛💥🔨


Update clients.rst - base_url should be base_uri

The docs give 'base_url' as the param name for specifying the default URL for the client, whereas the code expects "base_uri". This commit updates the docs to say "base_uri".

This commit shouldn't have been merged into 5.3, it's wrong.

@mtdowling

Fixing this now. This was a bad PR/merge that I didn't catch.


SimpleCylinderActor.cpp at line 213

should be
if (QuadIndex != 0 && QuadIndex != InCrossSectionCount-1 && bInCapEnds)

Fixed! Thanks :)


Fixes #203 : Fixed deprecated api warning for flutter_facebook_auth.

Fixes #203
Describe the changes you have made in this PR -
Fixed deprecated API warning for flutter_facebook_auth.
Screenshots of the changes (If any) - No warning shown for flutter_facebook_auth

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 70.7%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
3909

Relevant Lines:
5529

💛  - Coveralls


fix vulnerable memory modification to gpu shared memory

This PR fixes a vulnerable memory modification to gpu shared memory in CUDA kernel function single_query_cached_kv_attention_kernel.

@WoosukKwon Sorry for bothering you. It seems that I cannot invite a reviewer for this PR. I have read CONTRIBUTING.md and are there any other instructions? thanks


Remove outdated element tests

_datasetIndex and _index were removed, so these tests no longer make sense. Plus tons of other tests end up testing element construction indirectly

These do test that the provided parameters end up as properties (in addition to the element actually getting constructed). The properties tested are outdated but the test works. I think we should keep this functionality and the tests. To me it would be ok to change the tested properties. Maybe something obviously bogus. foo / bar should be widely known.

The non-test code never passes any properties though, so it seemed weird to have a test for that. There are other tests that use the functionality such as should determine if in range, so it's still covered, but I'm not quite sure it's valuable to test explicitly. I'm fine closing this one though if we think the tests are still valuable


stuck script on ubuntu 20.04 server

kaushik@ubuntu:~/project$ ./bk-linux-amd64-1.1.0
>>> Starting local agent 🤖
>>> Starting build 👟
>>> Executing initial command: buildkite-agent pipeline upload

Any idea what's the issue? Also there is no steps included in documentation of installation of bk script.

👋 Given the age of this issue and the number of versions release since, as well as the most recent rework in 3.x.x I'm going to close this issue for now. Cheers!


Fix the snowflake extractor import in dev docker

Getting below error in container
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/airflow/plugins_manager.py", line 227, in load_entrypoint_plugins
    plugin_class = entry_point.load()
  File "/usr/local/lib/python3.9/site-packages/importlib_metadata/__init__.py", line 203, in load
    module = import_module(match.group('module'))
  File "/usr/local/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/usr/local/lib/python3.9/site-packages/openlineage/airflow/plugin.py", line 25, in <module>
    from openlineage.airflow import listener
  File "/usr/local/lib/python3.9/site-packages/openlineage/airflow/listener.py", line 74, in <module>
    extractor_manager = ExtractorManager()
  File "/usr/local/lib/python3.9/site-packages/openlineage/airflow/extractors/manager.py", line 15, in __init__
    self.task_to_extractor = Extractors()
  File "/usr/local/lib/python3.9/site-packages/openlineage/airflow/extractors/extractors.py", line 80, in __init__
    extractor = import_from_string(extractor)
  File "/usr/local/lib/python3.9/site-packages/openlineage/airflow/utils.py", line 303, in import_from_string
    raise ImportError(f"Failed to import {path}") from e
ImportError: Failed to import  astronomer.providers.snowflake.extractors.snowflake.SnowflakeAsyncExtractor


 This is bad because it emphasizes the footnotes as much as the main text. Decreasing the font-size for footnotes should be fine.

@melissasamworth

Fixed by https://github.com/StampyAI/stampy-ui/pull/867


feat: Update to Apache Arrow v15

Update the apache arrow version to the latest major one

The change was done via
gomajor get github.com/apache/arrow/go/v15
go mod tidy
go mod vendor

however, I think automating it with renovate would be nice

@kravets-levko any chance to get this merged & released, too?
We use arrow v15 & there isn't a straightforward way to map v12 to v15 now (only custom converter), but on the wire it's the same.

Closing in favor of https://github.com/databricks/databricks-sql-go/pull/216


Add catch for calculation not being done

Looks like calculated dose is not always calculating with the javascript.  I can either have the python calculate it, or throw an exception -- which may be a better alternative to force the calculation.

Various error catching has been added

Various error catching has been added


¿how restart client after install plugins?...

Hi,
I have the following doubt, after installing plugins from the marketplace, I must restart the client, but it does not work for me, what steps do you recommend to do this, modify the docker image?
Thanks in advance, greetings ...

You just have to restart the Tomcat instead of reloading the tab/window on browser.

thank you for the answer, finally have problem with the kubernetes.


Invite_Limit_Reached

Hey guys,
I'm having an issue where upon sign up, the API returns an error stating invite_limit_reached. Is this something I need to take up with Slack? Or do you think there's a bug in the code?
I've taken a look around and couldn't see anything obvious...

Was an issue with Slack.

Good


Improve source file remapping

When running tests on Linux I was getting the following crash:
10:57:45   Error Message:
10:57:45    System.ArgumentNullException : Value cannot be null. (Parameter 'directory')
10:57:45   Stack Trace:
10:57:45      at Guard.AgainstEmpty(String value, String argumentName) in /_/src/Verify/Guard.cs:line 108
10:57:45    at Guard.BadDirectoryName(String name, String argumentName) in /_/src/Verify/Guard.cs:line 59
10:57:45    at VerifyTests.PathInfo..ctor(String directory, String typeName, String methodName) in /_/src/Verify/DerivePaths/PathInfo.cs:line 15
10:57:45    at VerifyTests.PathInfo.DeriveDefault(String sourceFile, String projectDirectory, Type type, MethodInfo method) in /_/src/Verify/DerivePaths/PathInfo.cs:line 30
10:57:45    at VerifyNUnit.Verifier.GetPathInfo(String sourceFile, Type type, MethodInfo method) in /_/src/Verify.NUnit/DerivePaths/VerifierSettings.cs:line 10
10:57:45    at VerifyNUnit.Verifier.BuildVerifier(String sourceFile, VerifySettings settings, Boolean useUniqueDirectory) in /_/src/Verify.NUnit/Verifier.cs:line 51

After investigation, it appears that:

My project is built on Windows
The resulting assemblies are passed to another Linux machine to run tests
In that scenario, source file path retrieved through [callerFilePath] attribute is a Windows path and Path.GetDirectoryName will return null with such a path on Linux.

I then stumbled on GetMappedBuildPath (introduced in https://github.com/VerifyTests/Verify/pull/729) but it's not detecting my case as I'm not running in a WSL context.
After some discussions with @JanKrivanek, we ended up:

Stop using Path.GetDirectoryName as it returns null with a Windows path on Linux
The method is now called IoHelpers.ResolveDirectoryFromSourceFile to be more honest about what it does
Improve the remapping algorithm to cover more use cases (new approach is described here)
Make the code testable (by introducing a new interface IEnvironment) and add some tests to cover the old and new use cases

@JanKrivanek thoughts

I'll need to have a closer look and thought.. but the very first idea is that it'll crash anyway down the road :-/
Basically Verify relies on build-time paths info - that is completely lost once the build outputs are copied to another machine. Normalizning the paths will unblock the Path.GetDirectoryName, but the results will be a nonexistent path. I'm cannot now recall what the caller info is exactly used for - is it just naming? Then it might be proably overriden via custom naming info (UseTypeName, UseMethodName).
tl;dr; "What is your scenarion?" :-) We might have similar in https://github.com/dotnet/templating/blob/main/tools/Microsoft.TemplateEngine.Authoring.TemplateVerifier/VerificationEngine.cs

I feel like bypassing Path.GetDirectoryName totally is probably the best as it's meant to be used on a local path. In this case, the path can have different separators so having its own logic would be better and will have a consistent behavior across platforms.
I would rename it to IoHelpers.GetDirectoryNameFromSourceFile as the input would always be a file path.

Overall looks good to me.
Albeit beyond origninal scope - it would be nice to include handling of the 'build outputs lives next to the solution dir' case as well

@jairbubbles Can you please add me write permissions to your fork?
I've created a change that handles the case of execution folder being sibling to source folder.
Thanks

Sure! You should have received an invite.

Looking good, I updated the PR title/description.

@SimonCropp If you're willing to merge, I believe you have an option to coauthor when squashing, @JanKrivanek made most of the work so he should get credit.

@jairbubbles @JanKrivanek thanks for all the effort on this one.

this is deployed. can u smoke test

this is deployed. can u smoke test

looks good for basic scenarios

On my side my CI on Linux / macOS is now fixed.


Support for controlling "heat once" mode and "set down" temperatures

Should address #77 and #80.
Tested both of them on my own setup.
Only issue I still have with this is that the translations are not applied even though I added the texts to strings.json and used the same key as in const.py.
@dominikamann, Any idea what's missing to make this work?

Hi @simonangerbauer thanks so much for your input 💪😎 I really appreciate it 👍. I’ll look into why the translations aren’t working and see what I can find out. Best, Dominik


Inconsistent results for wasi-nn different backends

Test Case
nn_image_classification, nn_image_classification_named and nn_image_classification_onnx have different inference results. The first two cases are based on openvino backend, while the third one is based on onnxruntime backend.
Steps to Reproduce
Run the cases above locally or check the output of GitHub Actions.
An example output:
https://github.com/bytecodealliance/wasmtime/actions/runs/8716252474/job/23909494323#step:17:3914
Expected Results
Since both of them use mobilenet v2 and the same input tensor data, they should have similar results. We don't expect them to be exactly the same because of different model formats.
Actual Results
The result of wasi-nn openvino backend is [InferenceResult(963, 0.7113049), InferenceResult(762, 0.07070768), InferenceResult(909, 0.036356032), InferenceResult(926, 0.015456118), InferenceResult(567, 0.015344023)].
The result for onnx backend is [InferenceResult(470, 479.08182), InferenceResult(862, 378.7252), InferenceResult(626, 364.8759), InferenceResult(644, 334.28488), InferenceResult(556, 288.65884)].
CI for WinML backend is not enabled yet, but it has the same result as onnx backend.
Versions and Environment
Wasmtime version or commit: 19.0.1
Operating system: Windows
Architecture: x86_64
Extra Info
Although all these tests use mobilenet v2, openvino model requires input data to be BGR format, with mean values: [127.5, 127.5, 127.5](openvino model description). ONNX model (used by both onnxruntime backend and winml backend) requires input data to be RGB format, in the range of [0, 1] and then normalized using mean = [0.485, 0.456, 0.406] and std = [0.229, 0.224, 0.225] (onnx model description).
I'm not sure if we missed input data preprocessing, or it was processed somewhere else.
The test case for WinML backend uses a different input at this time. I'm trying to unifying the inputs for all backends so we can double check the correctness.

cc: @devigned

Wasi-nn example classification-component-onnx has images pre-processed here. Applying the same process for test inputs should fix this issue for onnxruntime and winml backends.

Nice catch, @jianjunz! Thank you for opening this issue.
I'm happy to open a PR for this, but if you are interested, I'd gladly give it a review. Are you interested in contributing?

Thanks, David. #8442 is opened for fixing this issue. It also makes ONNX Runtime backend and WinML backend share the same test code because both of them use ONNX models.


GroupBy with Join is evaluated locally

I'm in the process of porting an app which uses EF6 to a .NET Core one with EFCore. In a GroupBy which is grouping for the ForeignKey and counting the elements of each group, as soon as I add a Join to get data from the related entity, the GroupBy cannot be translated and it's evaluated locally.
Steps to reproduce
I've created a simple project reproducing the issue: https://github.com/cjaliaga/GroupByWithJoinTest/
The queries that I'm trying are:
var mostFollowedQuerySyntax = (from f in context.FollowShows
                        group f by f.ShowId
                        into g
                        join show in context.Shows on g.Key equals show.Id
                        select new
                        {
                            Id = g.Key,
                            Followers = g.Count(),
                            Name = show.Name,
                        })
                    .OrderByDescending(s => s.Followers)
                    .FirstOrDefault();

var mostFollowedMethodSyntax = context.FollowShows
                    .GroupBy(f => f.ShowId, (key, grp) => new {Id = key, Followers = grp.Count()})
                    .Join(context.Shows, f => f.Id, s => s.Id,
                        (f, s) => new
                        {
                            Id = f.Id,
                            Followers = f.Followers,
                            Name = s.Name
                        }).OrderByDescending(s => s.Followers)
                    .FirstOrDefault();

The models
public class Show
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public List<FollowShow> Followers { get; set; }
    }

   public class FollowShow
    {
        public int Id { get; set; }
        public int ShowId { get; set; }
        public Show Show { get; set; }
        public DateTime Followed { get; set; }
    }

The logging output when running the first query:
dbug: Microsoft.EntityFrameworkCore.Query[10101]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      Compiling query model:
      '(from IGrouping<int, FollowShow> g in
          (from FollowShow f in DbSet<FollowShow>
          select [f]).GroupBy([f].ShowId, [f])
      join Show show in DbSet<Show>
      on [g].Key equals [show].Id
      order by
          (from FollowShow <generated>_1 in [g]
          select [<generated>_1]).Count() desc
      select new <>f__AnonymousType3<int, int, string>(
          [g].Key,

              (from FollowShow <generated>_1 in [g]
              select [<generated>_1]).Count(),
          [show].Name
      )).FirstOrDefault()'
dbug: Microsoft.EntityFrameworkCore.Query[10104]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      Optimized query model:
      '(from IGrouping<int, FollowShow> g in
          (from FollowShow f in DbSet<FollowShow>
          select [f]).GroupBy([f].ShowId, [f])
      join Show show in DbSet<Show>
      on [g].Key equals [show].Id
      order by
          (from FollowShow <generated>_1 in [g]
          select [<generated>_1]).Count() desc
      select new <>f__AnonymousType3<int, int, string>(
          [g].Key,

              (from FollowShow <generated>_1 in [g]
              select [<generated>_1]).Count(),
          [show].Name
      )).FirstOrDefault()'
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'GroupBy([f].ShowId, [f])' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'join Show show in value(Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryable`1[GroupByTest.Models.Show]) on [g].Key equals [show].Id' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'orderby {[g] => Count()} desc' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Count()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Count()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Count()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'FirstOrDefault()' could not be translated and will be evaluated locally.
dbug: Microsoft.EntityFrameworkCore.Query[10107]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      (QueryContext queryContext) => IEnumerable<<>f__AnonymousType3<int, int, string>> _InterceptExceptions(
          source: IEnumerable<<>f__AnonymousType3<int, int, string>> _ToSequence(() => <>f__AnonymousType3<int, int, string> FirstOrDefault(IEnumerable<<>f__AnonymousType3<int, int, string>> _Select(
                      source: IOrderedEnumerable<TransparentIdentifier<IGrouping<int, FollowShow>, ValueBuffer>> _OrderBy(
                          source: IEnumerable<TransparentIdentifier<IGrouping<int, FollowShow>, ValueBuffer>> _Join(
                              outer: IEnumerable<IGrouping<int, FollowShow>> _GroupBy(
                                  source: IEnumerable<FollowShow> _ShapedQuery(
                                      queryContext: queryContext,
                                      shaperCommandContext: SelectExpression:
                                          SELECT [f].[Id], [f].[Followed], [f].[ShowId]
                                          FROM [FollowShows] AS [f]
                                          ORDER BY [f].[ShowId],
                                      shaper: BufferedEntityShaper<FollowShow>),
                                  keySelector: (FollowShow f) => f.ShowId,
                                  elementSelector: (FollowShow f) => f),
                              inner: IEnumerable<ValueBuffer> _ShapedQuery(
                                  queryContext: queryContext,
                                  shaperCommandContext: SelectExpression:
                                      SELECT [show].[Id], [show].[Name]
                                      FROM [Shows] AS [show],
                                  shaper: ValueBufferShaper),
                              outerKeySelector: (IGrouping<int, FollowShow> g) => g.Key,
                              innerKeySelector: (ValueBuffer show) => int TryReadValue(show, 0, Show.Id),
                              resultSelector: (IGrouping<int, FollowShow> g | ValueBuffer show) => TransparentIdentifier<IGrouping<int, FollowShow>, ValueBuffer> CreateTransparentIdentifier(
                                  outer: g,
                                  inner: show)),
                          expression: (TransparentIdentifier<IGrouping<int, FollowShow>, ValueBuffer> t0) => int Count(t0.Outer),
                          orderingDirection: Desc),
                      selector: (TransparentIdentifier<IGrouping<int, FollowShow>, ValueBuffer> t0) => new <>f__AnonymousType3<int, int, string>(
                          t0.Outer.Key,
                          int Count(t0.Outer),
                          string TryReadValue(t0.Inner, 1, Show.Name)
                      )))),
          contextType: GroupByTest.ShowsDbContext,
          logger: DiagnosticsLogger<Query>,
          queryContext: queryContext)
dbug: Microsoft.EntityFrameworkCore.Database.Connection[20000]
      Opening connection to database 'TestShows' on server '(localdb)\mssqllocaldb'.
dbug: Microsoft.EntityFrameworkCore.Database.Connection[20001]
      Opened connection to database 'TestShows' on server '(localdb)\mssqllocaldb'.
dbug: Microsoft.EntityFrameworkCore.Database.Command[20100]
      Executing DbCommand [Parameters=[], CommandType='Text', CommandTimeout='30']
      SELECT [f].[Id], [f].[Followed], [f].[ShowId]
      FROM [FollowShows] AS [f]
      ORDER BY [f].[ShowId]
info: Microsoft.EntityFrameworkCore.Database.Command[20101]
      Executed DbCommand (81ms) [Parameters=[], CommandType='Text', CommandTimeout='30']
      SELECT [f].[Id], [f].[Followed], [f].[ShowId]
      FROM [FollowShows] AS [f]
      ORDER BY [f].[ShowId]
dbug: Microsoft.EntityFrameworkCore.Database.Command[20300]
      A data reader was disposed.
dbug: Microsoft.EntityFrameworkCore.Database.Command[20100]
      Executing DbCommand [Parameters=[], CommandType='Text', CommandTimeout='30']
      SELECT [show].[Id], [show].[Name]
      FROM [Shows] AS [show]
info: Microsoft.EntityFrameworkCore.Database.Command[20101]
      Executed DbCommand (13ms) [Parameters=[], CommandType='Text', CommandTimeout='30']
      SELECT [show].[Id], [show].[Name]
      FROM [Shows] AS [show]
dbug: Microsoft.EntityFrameworkCore.Database.Command[20300]

Further technical details
EF Core version: 2.1.0-rtm-30799
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10
IDE: Visual Studio 2017 15.7.3

Duplicate of #10012


Issue missing identifier column

according to the last merged PR
for  ScormManager.php you define a new column called identifier while save a new created scorm however isn't defined at create_scorm_tables.php.stub migration and  ScormModel.php

Hi @AmrAbdalrahman ,
Thanks for the highlight. The column has been added in version 4 and main repo. I will add it in version 3 soon.
Regards,
Dev


Annotation colors not saved.

Annotation colors are not saved within an Operator. All default back to (1.0,1.0,1.0)...

Colors seem to persist now in the latest dev branch, but I've noticed that all annotation boxes will move relatively to another location on the graph and all the text gets defaulted.

Yeah, I've noticed this too. When you pop into an Operator and then come back out the Operator has moved.

Should be fixed in upcoming version.


ModuleNotFoundError: No module named 'jsonschema.compat'

I using the pybit project, which is using also the jsonschema.
Since updating to version 4.0.1 I get the following error:
Traceback (most recent call last):
File "E:\programming\Project\TradeBot\TradingBot.py", line 12, in
from bybit import HTTP
File "E:\programming\Project\TradeBot\env\lib\site-packages\bybit_init_.py", line 3, in
from bravado.client import SwaggerClient
File "E:\programming\Project\TradeBot\env\lib\site-packages\bravado\client.py", line 51, in
from bravado_core.param import marshal_param
File "E:\programming\Project\TradeBot\env\lib\site-packages\bravado_core\param.py", line 13, in
from bravado_core.marshal import marshal_schema_object
from bravado_core.model import Model
File "E:\programming\Project\TradeBot\env\lib\site-packages\bravado_core\model.py", line 13, in
from swagger_spec_validator.ref_validators import attach_scope
File "E:\programming\Project\TradeBot\env\lib\site-packages\swagger_spec_validator_init_.py", line 8, in
from swagger_spec_validator.util import validate_spec_url
File "E:\programming\Project\TradeBot\env\lib\site-packages\swagger_spec_validator\util.py", line 9, in
from swagger_spec_validator import validator12
File "E:\programming\Project\TradeBot\env\lib\site-packages\swagger_spec_validator\validator12.py", line 29, in
from swagger_spec_validator.ref_validators import default_handlers
File "E:\programming\Project\TradeBot\env\lib\site-packages\swagger_spec_validator\ref_validators.py", line 14, in
from jsonschema.compat import iteritems
ModuleNotFoundError: No module named 'jsonschema.compat'
Any help or ideas?
Regards,

Hi there. This is a downstream bug you should file on swagger-spec-validator, unless you see it filed already.
More context in #843.

Thanks for the reply, but I using bybit and if I go back to the version 3.1.1 of jsonschema, all is running correct.
I give back your replay to the bybit developer.
Regards,


Option to fail if there are skipped tests

🚀 Feature Proposal
Add a CLI option to fail if there are any skipped tests. This option would be a boolean,
possibly called disallowSkipping, or something like this.
Motivation
Skipping tests is really useful when maintaining tests, however, sometimes, a skipped test may be committed and pushed to the repository.
It'd be really useful if there was a CLI argument to fail the tests if there are any skipped tests. This argument would then be configured in the hooks and possibly in the CI to ensure all tests are running.
Example
$ jest --disallowSkipping
Pitch
Recently we had a test flagged with .only finding its way all the way to the develop branch because the developer committed the test file with it. We then had two other developers, including myself, reviewing that PR and also we didn't realise that only that test case was being run for the whole file. Of course that was an oversight on our end, but we're all subject to pressure to deliver.
Much to my surprise, there isn't any options to disallowing skipped test cases or to ignore skipping altogether. Although there might be arguments for both cases, I believe to be cleaner to fail the tests if there are any being skipped than to ignore skipping because it'd force the code owner to actually fix the commit and remove the skip rule.

I'm not exactly sure if this is a good idea. Reason being, you have other tools out there that match your use case. You can use eslint along with this plugin that was developed for this exact use case. https://github.com/jest-community/eslint-plugin-jest

This type of feature is best implemented as a lint rule; and already has been.
no focused tests:
https://github.com/jest-community/eslint-plugin-jest/blob/master/docs/rules/no-focused-tests.md
a variety of other rules:
https://github.com/jest-community/eslint-plugin-jest/blob/master/docs/rules/

I can see that this is over a year old, but I would like to argue that the best way to do this is NOT with a lint rule.
/eslint-disable-next-line jest/no-disabled-tests
        describe.skip("no success", () => {

There goes my lint rule! I would really like it if I could check for skipped tests like I check for code coverage. If build says "3 skipped" I want it to fail.

You can use a custom reporter for this, that's how coverageThreshold is checked and fails the run: https://github.com/facebook/jest/blob/7e5e27a6aa5f07037c6b8b23a3e49afc30008937/packages/jest-reporters/src/coverage_reporter.ts#L412-L415

Seems like a lot of work that I will probably do in a shell script instead. Capture the result output and just /(\D+) skipped/


not working YQL service

I did fix for proxy service here:
https://github.com/brainwallet/brainwallet.github.io/pull/70

Still doesn't seem to work. Could you please check it out? Also the POST mode (for sending tx).

fixed in d9b3f3c2f949eae3a578f3c96653f45aa051ee97


Can't indent code blocks more than once(using tab) on spyder 4

I'm on spyder4 dev master and can't tab indent a block of code that has any leading spaces and therefore it won't let me indent more than one tab. Shift tab works multiple times to remove indents.
I bisected this to #3958 ...I guess I should have tested better.
What steps will reproduce the problem?

highlight a block of code
press tab once (works)
press tab again (nothing)

Versions and main components

Spyder Version: 4 master
Python Version: 3.6
Qt Version: 5.6.2
PyQt Version: 5.6
Operating system: Linux

Dependencies
IPython >=4.0    :  6.1.0 (OK)
cython >=0.21    :  0.25.2 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
nbconvert >=4.0  :  4.2.0 (OK)
numpy >=1.7      :  1.12.1 (OK)
pandas >=0.13.1  :  0.19.2 (OK)
psutil >=0.3     :  5.0.1 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pyflakes >=0.6.0 :  1.5.0 (OK)
pygments >=2.0   :  2.1.3 (OK)
pylint >=0.25    :  1.6.4 (OK)
qtconsole >=4.2.0:  4.3.0 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
sphinx >=0.6.6   :  1.5.1 (OK)
sympy >=0.7.3    :  1.0 (OK)

I guess I should have tested better.

Don't worry about it. Thanks for letting us know this is a side effect of that PR.
@dalthviz, please work on this one.

Thanks @bcolsen for noticing this and again for all the help with testing this feature, we really appreciated all your effort :+1:


Editing the wrong dns zone

I'm trying to create a new certificate for libs.datalynk.ca and forms.datalynk.ca but the lego client is trying to add the challenge to the wrong zone in Route53.
I can successfully create a number of other names under the *.datalynk.ca but not the above 2 for some reason.
Information is found below:
root@balancer2:/etc/nginx/certs/letsencrypt# ./lego_linux_amd64
NAME:
   lego - Let's Encrypt client written in Go

USAGE:
   lego_linux_amd64 [global options] command [command options] [arguments...]

VERSION:
   v0.4.0-0-g192334c448f7

2017/08/21 14:55:35 [INFO][libs.datalynk.ca] acme: Obtaining bundled SAN certificate
2017/08/21 14:55:35 [INFO][libs.datalynk.ca] AuthURL: https://acme-v01.api.letsencrypt.org/acme/authz/CEFezAAFLlNnDLpmeu0kNVGZ3gFDxHgnOhUmgNiHmQA
2017/08/21 14:55:35 [INFO][libs.datalynk.ca] acme: Could not find solver for: http-01
2017/08/21 14:55:35 [INFO][libs.datalynk.ca] acme: Trying to solve DNS-01
2017/08/21 14:55:36 [libs.datalynk.ca] Could not obtain certificates
	Error presenting token: Failed to change Route 53 record set: InvalidChangeBatch: RRSet with DNS name _acme-challenge.libs.datalynk.ca. is not permitted in zone auxiliumgroup.com.
	status code: 400, request id: 50f1632c-86a2-11e7-b2f1-9b0de897ce05

2017/08/21 14:53:42 [INFO][foo.datalynk.ca] acme: Obtaining bundled SAN certificate
2017/08/21 14:53:42 [INFO][foo.datalynk.ca] AuthURL: https://acme-v01.api.letsencrypt.org/acme/authz/Y_Nr_ey9Zrtc257K-WrMSfODzYw15F2vgoG5e_6L5vI
2017/08/21 14:53:42 [INFO][foo.datalynk.ca] acme: Trying to solve DNS-01
2017/08/21 14:54:19 [INFO][foo.datalynk.ca] Checking DNS record propagation using [<IP_ADDRESS>:53 <IP_ADDRESS>:53]
2017/08/21 14:54:21 [INFO][foo.datalynk.ca] The server validated our request
2017/08/21 14:54:58 [INFO][foo.datalynk.ca] acme: Validations succeeded; requesting certificates
2017/08/21 14:54:59 [INFO] acme: Requesting issuer cert from https://acme-v01.api.letsencrypt.org/acme/issuer-cert
2017/08/21 14:54:59 [INFO][foo.datalynk.ca] Server responded with a certificate.

I think I know the problem you are facing. libs.datalynk.ca has CNAME. So, lego detects the zone as the underlying domain. After that it fails. I believe this is the same issue https://github.com/xenolf/lego/issues/330
I have a commit that tries to fix this issue. https://github.com/appscode/lego/commit/1af7a77b966607bfc78e90ef9f2635ceff3ed65e
If the auto detected zone is not in the same domain as the original domain, I use the publicSuffix library to guess the zone.

#449 resolves this issue.


Added Format Code functionality to PropertyFieldCodeEditor property control

Q
A

Bug fix?
[x]

Related issues?
fixes https://github.com/pnp/sp-dev-fx-property-controls/issues/165

What's in this Pull Request?
Added Format Code functionality to PropertyFieldCodeEditor property control for languages including JSON, JavaScript, Sass, TypeScript", HTML, Handlebars, XML, css.
When the language is Plain Text, the "Format Code" button is hidden.

Hi @AJIXuMuK
Please review this PR. Thanks

Hi @nanddeepn!
Thank you for the contribution!
Could you please check the markup of the element? In my tests "Format code" button location is ugly:

Another thing: you don't need to change all loc files. Please, make needed changes in en-us.js and we'll automatically translate them to other languages.

Thanks @AJIXuMuK
I will look into this.
This looks fine in SharePoint local workbench, but not in hosted workbench.
Local Workbench

Hosted workbench

Hi @AJIXuMuK
Fixed the layout for buttons: https://github.com/pnp/sp-dev-fx-property-controls/pull/261/commits/472494f4b9cbcb1ac2bf80bc0625a6c61ecabe24

merged!
thanks @nanddeepn


raft: peer shouldn't hibernate if it has pending reads

What have you changed?
Make peer not hibernate if it has pending reads.
Blocked on #6396 and #6348 .
Close #6439 .
What is the type of the changes?
Bug fix.
How is the PR tested?
Integration test.
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No.
Does this PR affect tidb-ansible?
No.
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Any examples? (optional)

/run-all-tests


Deliver Missing Module

Hey, AICoE-CI!
Please deliver the package module for the following tag, as it missing on PyPI:
Tag: 0.6.3

Successfully Delivered Package module to Pulp PyPI.
Package can be found at: https://pypi.org/project/jupyterhub-singleuser-profiles


Var in modeling object likely causing races across parallel tests

The var added to spatial.util.modeling in d48b35161316f94ace1baa25d6f57e6f4177ca3e for tracking accumulation registers with FMA is generally unsafe - should probably use global metadata for this instead.

Will be resolved with accum fix in #63

Closed with merging accum branch into master


chore: move to v4.11.0-rc2 (new variable command)

Looking at the declarations diff above, it actually looks okay: biprod_lift_id_sub_id has vanished, intentionally (it was awkward to arrange the variables, and Jöel (I think) just inlined it).

Looking at the declarations diff above, it actually looks okay: biprod_lift_id_sub_id has vanished, intentionally (it was awkward to arrange the variables, and Jöel (I think) just inlined it).

What about this Obj stuff?

What about this Obj stuff?

i believe that might be an unneeded part of the fun_prop_dev test? the file was adapted by @lecopivo who worked on the tactic (as far as i can tell), so i'd expect that to be an acceptable change. if you want specifics, i'm sure they can provide them?

What about this Obj stuff?

i believe that might be an unneeded part of the fun_prop_dev test? the file was adapted by @lecopivo who worked on the tactic (as far as i can tell), so i'd expect that to be an acceptable change. if you want specifics, i'm sure they can provide them?

I don't need specifics, just making sure that people who know about this are ok with its removal.
Then I guess the declarations diff above is completely fine.

bors d+

bors merge

bors p=10

bors merge

bors r-
needs merging master, @edegeltje is working on it

@jcommelin build succeeded, it should be ready to be merged

bors merge


Fragmented traces in LangChain callback handler

Tracking parent IDs through lang chain to try and prevent fragmented traces.

We took another approach.


Broken for Prosody 0.9.7 (stream is undefined)

When downgrading to Prosody 0.9.7 (current stable) in the backend, I get this error:
return f.emit("stream:end"),f.disconnect();n.lang||(n.lang=f.stream.lang
                                                             ^^^^^^^^^^^
Cannot read property 'lang' of undefined

This should be taken care of now in 7.0.6


Error while patching : java.io.IOException: This archive contains unclosed entries.

?? Patching /opt/sas/sashome/SASEnvironmentManager/2.5/installs/hypericsvr_lax_2.5_M3_HFE8M004/SASHome/xx/SASEnvironmentManager/2.5/server-5.8.0.tar.gz::server-5.8.0-EE/hqapi1-client-6.0.2/lib/log4j-core-2.11.1.jar
? null
? null
? null
? null
? null
? null
? null
? null
? null
? null
ERROR: Error while patching : java.io.IOException: This archive contains unclosed entries.
at org.apache.commons.compress.archivers.tar.TarArchiveOutputStream.finish(TarArchiveOutputStream.java:291)
at com.sas.vulnerabilities.utils.ArchiveCompressUtils.compressArchive(ArchiveCompressUtils.java:108)
at com.sas.vulnerabilities.patcher.SequentialPatcher.packageNextArchive(SequentialPatcher.java:38)
at com.sas.vulnerabilities.patcher.SequentialPatcher.packageNextArchive(SequentialPatcher.java:41)
at com.sas.vulnerabilities.patcher.SequentialPatcher.patchSingleCVE(SequentialPatcher.java:58)
at com.sas.vulnerabilities.patcher.SequentialPatcherInventoryTask.runSingleCveInventoryPath(SequentialPatcherInventoryTask.java:68)
at com.sas.vulnerabilities.patcher.SequentialPatcherInventoryTask.run(SequentialPatcherInventoryTask.java:93)
at lukfor.progress.tasks.Task.call(Task.java:39)
at lukfor.progress.tasks.Task.call(Task.java:1)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
at com.oracle.svm.core.thread.JavaThreads.threadStartRoutine(JavaThreads.java:519)
at com.oracle.svm.core.posix.thread.PosixJavaThreads.pthreadStartRoutine(PosixJavaThreads.java:192)

@boemska-nik is there any update?

What OS / version is this with, and what was the full command line you ran & with what user?

Hi @sridharamirneni,
Coming back to this. We weren't able to replicate this issue, but it could be resolved by the 3.2.0 release because of the changes that were made to the repacker. Are you able to test this release and let us know if you still face this issue?


Quick access: workspace symbols

Refs: https://github.com/microsoft/vscode/issues/69955

[ ] anyOS: @jrieken
[ ] anyOS @octref

Complexity: 2

Quick open is now using the quick input widget that leverages our new tree widget. All quick open providers have been rewritten from scratch.
All of the existing commands for driving quick open have been preserved. Everything should work as before with some additional features as noted in the test plan.
Note: configure "workbench.quickOpen.closeOnFocusLost": false to make testing easier by keeping quick open opened even if you navigate somewhere else.

verify you can see, filter and navigate to symbols of the active editor when typing # in quick open followed by a search string (note: at least for typescript this requires a typescript file to be opened in the background of the project you want to search in)
verify you see a symbol icon the reflects its kind
verify that any active word under the cursor is used as default query when opening
verify that you can narrow down symbols by container if you type the container name after a space (e.g. type foo bar to find all symbols that have foo in their label and bar in their description)
configure a files.exclude or search.exclude in the form of **/pattern/** and verify that symbols that have this pattern as their container do not appear (e.g. **/node_modules/** to exclude all results from that folder)

configure a files.exclude or search.exclude in the form of /pattern/ and verify that symbols that have this pattern as their container do not appear

@bpasero Is that a new feature? It might raise some eyebrows because there have been asks to support the same for go-to-symbol and reference search but keep saying that it is the responsibility of the underlying language and its project system

@jrieken yes new, and yes agree this needs to be done on the LSP side of things. I think C++ for example is already working on supporting this.
Refs: https://github.com/microsoft/vscode/issues/46718#issuecomment-599397590

Here is the one from C++: https://github.com/microsoft/vscode-cpptools/issues/5099

I'd say this is questionable because it also applies to other navigation feature. And also, why don't we pass this to the extension when making the request. This sounds waste, e.g search in node_modules so that we later drop all results.

@jrieken if you can point me to where I can pass this on to extensions via API then I can add it. I do not see such support in provideWorkspaceSymbols
But I am easy to remove this support until we figured out the better solution via extension API.

do not see such support in provideWorkspaceSymbols

Yeah, it would require new API and all of that. Tho, I still have trouble understanding the use-case, shouldn't I exclude folder from my project config if I don't want to see them? You are still getting compile errors and navigation results for those resources

I think the support for filtering via files.exclude and search.exclude is really a last resort solution in case the extension does not play by the rules. There may always be an extension that simply does not do this filtering properly and then this setting can be used.

But I am easy to remove this support until we figured out the better solution via extension API.

Given this reply https://github.com/microsoft/vscode/issues/46718#issuecomment-606794209 I think that it would be better to hold off for now


iOS 17 physical device not found

When running flutter devices or flutter run, iOS 17 physical device does not show in list.
I'm unsure if it's just me and my device.
When I run xcrun xcdevice list, it has 2 entries for the same device. One with an error and one without.
{
    "ignored" : false,
    "modelCode" : "iPad14,3",
    "simulator" : false,
    "modelName" : "iPad Pro (11-inch) (4th generation)\"",
    "error" : {
      "code" : -13,
      "failureReason" : "",
      "description" : "Victoria’s iPad is not connected",
      "recoverySuggestion" : "Xcode will continue when Victoria’s iPad is connected and unlocked.",
      "domain" : "com.apple.platform.iphoneos"
    },
    "operatingSystemVersion" : "16.5 (20F66)",
    "identifier" : "xxx",
    "platform" : "com.apple.platform.iphoneos",
    "architecture" : "arm64e",
    "interface" : "usb",
    "available" : false,
    "name" : "Victoria’s iPad",
    "modelUTI" : "com.apple.ipad-pro-11-4th-2"
  },
  {
    "ignored" : false,
    "modelCode" : "iPad14,3",
    "simulator" : false,
    "modelName" : "iPad Pro (11-inch) (4th generation)\"",
    "operatingSystemVersion" : "17.0 (21A5248v)",
    "identifier" : "xxx",
    "platform" : "com.apple.platform.iphoneos",
    "architecture" : "arm64e",
    "interface" : "usb",
    "available" : true,
    "name" : "Victoria’s iPad",
    "modelUTI" : "com.apple.ipad-pro-11-4th-1"
  },

change Xcode-beta.app to Xcode.app

Hi @safakarapkirli,
could you explain what you mean by changing from the beta version to the stable one?

@FabsHof he mean to rename the app, go to Applications and rename Xcode-beta.app to Xcode.app, that should work ;)

Thanks @frapeti, that helped a lot :)


New utilities for margin, padding, display, position and borders

the github/design-systems team has been hard at work on creating some more useful utilities to use in our sites.
I'm publishing the utilities here. There's a little bit of an overhaul to the utility file. I broke up the utilities into their own separate partials. and placed them in /scss/utilities with a root file that will allow you to import all of them. Or import specific ones via their own files eg /scss/utilities/_padding.scss

@primer/owners @mdo @sophshep @broccolini @pmarsceill

Btw, probably throws some shit off, but given the movement of classes across files, I might suggest we do a major version bump here or include this in Primer 3.

@primer/design-systems when you get a chance I could use a review.
My goal with this PR was to bring parity between the utilities here and the ad-hoc ones added to github.com. This will be released in the v3.0.0 release. So there's going to be work done in github/github to support some of these changes.