some references in patch_sm READMEs to restricted repo DaisyPatchSM

small fyi that there's a number of references in READMEs to https://github.com/electro-smith/DaisyPatchSM which appears to not be public (yet?)

That's correct.
The DaisyPatchSM repo is currently private. We will be making it public when the device is officially released.

ahha. no problem, look forward to reading about it


Netlify page

Fixes Issue
This PR fixes the following issues:
#98

Added a new page called netlify and added an route in App.js
Changes proposed
Here comes all the changes proposed through this PR

Check List (Check all the boxes which are applicable)

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots and video
Add all the screenshots and a video which support your changes

bro didn't you read that issue careflly

we wanted a dedicated page for deployment

hi @krishpatel023 basically what we want is the dedicated page should be for deployment. that page shall include all the tools and methods to deploy your sites. so netlify can be one of the tools.
also in the main page, the card is titled "netlify" change it to "deployment". Hope it helps


Allow unmasked data

We could have the option of not providing mask data. Note that we also need to implement random starts for this.

Dear Dan, Max,
I'm trying to setup the old spikesorting pipeline with Neurosuite and kk2_legacy script, but can't find the format description of the 'fmask' file. Could you please point me to it? I'd highly appreciate.
Thanks very much!
Andrey Sobolev
LMU Munich


DNS objects should be garbage collected when their workspace is "deleted"

https://github.com/kcp-dev/kcp/pull/2293 follow-up.
DNS objects are currently never deleted and they should be when there is no more objects in their corresponding workspace.
Part of https://github.com/kcp-dev/kcp/issues/1986
@davidfestal

Moving this to v0.11 (if we're able to merge a PR in time for v0.10, awesome, but don't want to hold up the release).

/transfer-issue contrib-tmc


[EUWE] point amazon provider gem to euwe branch

for now point to euwe branch of the amazon provider repo
once we start finalizing and release a version of the gem we can point it to that release
of course we can do this asap - but this is to make bin/setup work on euwe again
note that naming the branch 'euwe' on amazon is just for starters and to minimize developer confusion.
we can give the amazon release any number we want and further down the road we can name the branches what we want.
@miq-bot assign @chessbyte
cc @Fryguy @blomquisg

@durandom LGTM :+1:
@chessbyte We decided for now to create a euwe branch on that repo for consistency.  We're not quite there yet for a full blown numeric release with separate cadence and all that.

@durandom @Fryguy 👍 will merge when green

Travis failures seem unrelated.  Merging.


docs: Describe current state of IPI vs UPI testing

Per discussion with @vrutkovs:

Right, this needs careful wording in a README of okd repo.
At the moment we have 2 blocking e2e tests for okd-specific repos - AWS IPI install and vSphere UPI install in our CI. Other platforms - like gcp/azure/metal would be added as optional in order to test platform-specific things. We don't have a capacity to test all platforms at once sadly. Also note, that OKD nightlies are promoted from OCP nightlies which run all kind of platform tests, so I think we're sufficiently covered by those.
In terms of supported installs both IPI and UPI are equal (although reproducing IPI issues is easier)
Lets file a bug in okd repo to describe current state of this in the README

Current state as of Feb 1:
CI tested:

AWS IPI - blocking test to accept releases
Installer / MCO forks have vSphere UPI tests (these are flaky now)

User testing:

Baremetal seems to work
GCP - won't bootstrap, see https://github.com/openshift/okd/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+label%3Aplatform%2Fgoogle
Azure - issues with uploading FCOS image, see https://github.com/openshift/okd/issues?utf8=✓&q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+label%3Aplatform%2Fazure
oVirt - support should be in the installer, no user reports so far
Openstack - ditto

As of https://origin-release.svc.ci.openshift.org/releasestream/4.4.0-0.okd/release/4.4.0-0.okd-2020-03-13-191636 oVirt and Openstack should work, so current installer supports all installation methods.


How to display 'week' in 'calendar-nav'?

I see the view in the readme as:

My current view is:

ps: “九月” means September
How can I change the month to 'week'?

currently week number is displayed only if you enable weekly notes in the journal
I plan to make this block highly configurable in v2

currently week number is displayed only if you enable weekly notes in the journal I plan to make this block highly configurable in v2

Got it ! thanks a lot!


Show and remotely log out multiple connections

Adds a feature in private mode, to show current user connections. HTML side needs a bit fixing. Server side should have a better way to get hostnames on async. Added bonus, to remotely disconnect connections.  Example if you forgot lounge open in someone computer, now you can remotely disconnect them.

Changed callbacks to promise. Better?

This is a really cool feature, @metsjeesus, thanks!
I am seeing a bug when connecting from my phone to an instance started on my laptop (how I test PRs locally 😄). Server crashes with:
.../lounge/src/server.js:429
			if (err && host.length) {
			               ^

TypeError: Cannot read property 'length' of undefined
    at .../lounge/src/server.js:429:19
    at QueryReqWrap.asyncCallback [as callback] (dns.js:62:16)
    at QueryReqWrap.onresolve [as oncomplete] (dns.js:216:10)

This does not appear from laptop to itself.
Outside of that, a couple comments:

Regarding the config option, I agree with @xPaw: this should be there in private and that's all. You can get rid of displayConnection 🎉
Regarding the UI, don't worry about that for now. We'll come up with adequate UI/UX once the server side works flawlessly (though I would definitely see this under the CSS block until we have better settings).

Hey @metsjeesus, I let myself in and made some UI changes since you mentioned you were having issues with HTML/CSS. I think it's nice like that, let me know what you think :)
Regardless of that remote log out / event @xPaw mentioned, a few things:

When I connect from my mobile, if it loses connection, its line does not get removed from the list.
At the moment, opening the app on a same device, same browser but different tabs creates multiple entries. That means that if I disconnect another tab, I get kicked out (but not instantly, upon refresh, which is also weird. These should not create a new line, but different browser / private connection should. Not sure how feasible this is.
It would be nice to have more details: last time something happened or connection duration or something time-related, browser, OS, maybe geolocation (but at the moment my link to a geo-ip service will do), etc. That as well I'm not sure how feasible.

Seems you dont like underscore on variable names. Now i know.
I try to understand what happens on mobile. If you lose connection, disconnection event should be fired on server and it will repopulate all connected clients with new info. On other hand, disconnected client will not see changes, because client is disconnected and wont recieve data.
location.reload(); should force refresh on disconnecting every connection. And it removes token too from localStorage. Because localStorage is shared between all the tabs, other tabs wont recieve reload event, but lose the token. If you do refresh, you generate new token and need to log in again.
It is possible to have more info about client. Anything what socket can give you can use, http headers are bit tricky, but usable too. Currently, client wont save any info about timings, so it needs to be implemented to have them. Moment, lounge lists every connection and every open tab counts as one. Maybe group them as (2 tabs) or so?

Seems you dont like underscore on variable names. Now i know.

Ahah, nothing like that, it's just for consistency sake :)
Also, in JS it's mostly camelCase convention, but it this were Ruby or Python, it would not be.

Welcome to networking where nothing ever works as expected. Reason for disconnect event not being called is silent network failures or incorrectly closed sockets. This is why you always have to just terminate the connection in server without waiting for client to respond (you can wait to ACK that it got a final logout event, but it has to be on a timeout)

If i get it right, socket.io plays ping pong. By default, pingTimeout is 60000 and pingInterval is 25000 milliseconds. What gives 60-85 seconds until disconnect event happens. Question is, do both client and server fire disconnection event? There should be a reliable way to test it?

@metsjeesus are you still stuck with this? Also, mind rebasing on master and fixing conflict?

@astorije I know what should be done, use cookies per connections, not one for all. I looked on it and it will interfere with https://github.com/thelounge/lounge/pull/1011 what is made to be a start of rewriting the authentication system. I feel that the change of the cookie managment is too big for that pull.  On server side, it changes alot how auth and client works. I would rather keep those things on separate PR.

It looks like you made some progress, @metsjeesus. Are you still stuck with this / do you need some more help? What's the next step? 😃

@astorije next step would be rewriting auth system with better cookie managment and that needs async calls from #1011 to be functional. From html/css style, feel free to change as you need.

next step would be rewriting auth system with better cookie managment and that needs async calls from #1011 to be functional.

Is this actually needed for this PR or just a nice-to-have? It seems to me like this PR could do without, but I haven't deeply looked at the server code so I might be wrong. Also interested in @xPaw's opinion.

From html/css style, feel free to change as you need.

Did you delete my commit from 27 days ago? 😱 I had made a lot of changes, I really hope not :( I don't have your copy of the repo on my laptop anymore so if you force-pushed over my commit, then my changes are in the trash...

Alright, it turns out, I had this partially git-stashed somewhere, so it was not completely lost. Let's not do that again 😅
@metsjeesus, I added a commit to your branch (Alert! Alert! git pull your branch!) and it now looks like this:

It's not perfect of course, but I think that's way slicker :D
Good luck with the rest!

@metsjeesus, now that #1199 is merged, we'll definitely want this PR! :D
Actually, #1199 came with a feature regression (it's not possible to log out other sessions anymore) and your PR re-introduces it, so we know we cannot release the next version without your work :)
Do you know what else is needed on this PR? Do you need help with the rebase and fixing conflicts?
Make sure you pull locally before committing/force-pushing or you'll lose my commit again :)
Thanks for your help!!

@metsjeesus, do you think you might be able to take this up soon?

Rebased on master to fix conflicts (that was quite the pain!). It's not working right now and it's missing 30 lines compared to before rebase (kinda expected it with such a rough rebase), but I'll figure it out.

I also did some separate work: https://github.com/thelounge/lounge/compare/xpaw/session-list
But since sessions that don't use 'remember me' will not generate a token, we have to combine both approaches.

@xPaw, now that we don't have the "Remember me" option, does your branch need any work?

@metsjeesus, @xPaw, pretty ping? 😁

Superceded by #1536


Update to use only CA cert

This PR updates README to use only SDP CA certificate for authentication.

apoorva918:update-for-ca-certs has commits from https://github.com/pravega/pravega-sensor-collector/tree/master
CLosing this PR in favour of https://github.com/pravega/pravega-sensor-collector/pull/56


UI-7645 - Bump to allow usage of _removeWidgetFromPage

See the new util added in https://github.com/activeviam/activeui/pull/3139, that we need to leverage here.
With the recent introduction of the first ESM package in the monorepo (@activeviam/utils), we need to change the Jest file transform and an experimental flag to make Jest work.
See https://jestjs.io/fr/docs/ecmascript-modules

You're going to actually import _removeWidgetFromPage in a following PR then?

Yes


Solution to problem 67. create your own promise

add solution to problem 67. create your own promise

Sure, no problem


TypeError: evts.filter is not a function error when rendering calendar

I'm trying to render the calendar with some fake data, and I'm getting the TypeError: evts.filter is not a function error @ react-big-calendar/lib/Month.js:80. Is this a known bug?
Here's my code:
import React, {Component} from 'react';
import BigCalendar from 'react-big-calendar';
import moment from 'moment';
import events from 'events';

BigCalendar.momentLocalizer(moment);
class Calendar extends Component {
  constructor(props, context) {
    super(props, context);
  }

  render() {
    return (
      <div>
        <BigCalendar
          culture='en-GB'
          events={events}
          />
      </div>
    );
  }
}

export default Calendar;

Sounds like your events prop isn't an array

I'm loading the events from a file, like that:
export default [
    {
      'title': 'All Day Event very long title',
      'allDay': true,
      'start': new Date(2015, 3, 0),
      'end': new Date(2015, 3, 1)
    },
    {
      'title': 'Long Event',
      'start': new Date(2015, 3, 7),
      'end': new Date(2015, 3, 10)
    },
...list goes on
]```
I made sure the data was in array format.

You might have something wonky with your import or something. I'd set a break point or console.log the value out when rendering the calendar


[ValueCard] values not showing precision

What package is this for?

[x] React
[ ] Angular

Describe the bug
Precision can be set to 4, but the whole numbers are still being truncated. If precision is set, it should be assumed that the user wants to see all numbers
To Reproduce
Steps to reproduce the behavior:

Go to ValueCard, medium size and add 2-3 attributes with precison: 4 and values > 1000
See that the value will be truncated to 1K

Expected behavior
If I set precision to 4 I should be able to see 4 digits of information in whole numbers and decimal numbers.
Environment/versions:

OS: [e.g. MacOS, Windows]
Browser: [e.g. chrome, safari]
carbon-addons-iot-react version: [e.g. v2.60.0]

Additional context
internal issue: https://github.ibm.com/wiotp/monitoring-dashboard/issues/1694

Internal design spec: https://github.ibm.com/wiotp/monitoring-dashboard/issues/1694#issuecomment-26851354

Internal design spec: https://github.ibm.com/wiotp/monitoring-dashboard/issues/1694#issuecomment-26851354

Complete design sec: https://ibm.box.com/s/oxez903ifgk784i83i9frj1s0upvrr7w

Complete design sec: https://ibm.box.com/s/oxez903ifgk784i83i9frj1s0upvrr7w


libxlsxwriter: 1.0.5 -> 1.0.6

Motivation for this change
https://github.com/jmcnamara/libxlsxwriter/releases/tag/RELEASE_1.0.6
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 124771 at 5d873fde run on x86_64-linux 1

2 packages built successfully:

libxlsxwriter
sc-im


Unskip bound attribute test

Since we were talking in the working group meeting, thought I'd give it a go and see if it passes

✅ Integration tests are green


Output abi, networks and bytecode to different json files during packaging

Disclaimer: do not check this in until the new Registry is out. This PR will be updated to output both the old Registry JSON files and the new Registry JSON files in the same npm package once the new Registry is checked in. The reason for having both the old and the new Registry JSONs in the same npm package is that the dapp will need to support both the old and the new Registry and npm doesn't have a concept of aliases like yarn does, so you cannot pull two different versions of the same package with a different name.
Changed package-npm.js script to take either a string with a file path or an object as the value for the source files map.
If an object is provided, each key will represent a key in the JSON source file that you want to save in a separate output file, and its value will be the destination file.

@erickearns ok I'll extract the bytecode too. I don't know why we are exporting .sol for AgentFactory and Registry, I hadn't joined yet when this script was made.
P.S. I'm making a similar PR for the token contract too

Are the agent and job contract being included in the abi of the factories?
On Tue, Jul 10, 2018, 00:07 Valerio Versace<EMAIL_ADDRESS>wrote:

@erickearns https://github.com/erickearns ok I'll extract the bytecode
too. I don't know why we are exporting .sol for AgentFactory and Registry,
I hadn't joined yet when this script was made.
P.S. I'm making a similar PR for the token contract too
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/singnet/alpha-blockchain/pull/23#issuecomment-403635797,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADbhOjF51oBHOPQISxAkgAG7ENwA9hEGks5uE9QWgaJpZM4VGIzM
.

They are in separate files


External event recorder adhere to EventRecorder

:warning: Removes controller.Events.
This change updates the external event recorder to be compatible with the k8s EventRecorder interface and embed the k8s event recorder. Since the events.Recorder implements the k8s EventRecorder, it can be directly used in reconcilers that embed the k8s EventRecorder interface. This makes the controller.Events redundant. The k8s event recorder is embedded in the events.Recorder to be able to post events to k8s and any external recorders.
This also introduces a new event type EventTypeTrace to map with the severity type trace.
Since the kubernetes API only accepts Normal and Warning event types. Sending a Trace event results in:
E1018 10:37:56.133861 1768805 event.go:334] Unsupported event type: 'Trace'

In order to keep the trace events only in k8s, the trace events are passed to k8s as Normal events. Trace events are not passed to the external event recorders, refer https://github.com/fluxcd/pkg/issues/139.
Since the EventRecorder interface methods don't return any error, the external event recorder has a logger to log any event recording related errors.
Replacing corev1.ObjectReference with runtime.Object due to the function signature change required updating the existing tests objects to be runtime.Object compatible.
Closes #134

Rebased and added object information in the recorder logger.
The error log from the retry test contains the object info, name/namespace:
E1019 00:36:16.462960 2396418 recorder.go:195]  "msg"="unable to record event" "error"="POST http://<IP_ADDRESS>:33683 giving up after 3 attempt(s): unexpected HTTP status 500 Internal Server Error" "name"="webapp" "namespace"="gitops-system"


stage 3 cpkt for zero-shot NextQA

Hello, could you please release the stage 3 checkpoint for zero-shot NextQA, which in the paper is obtained by performing stage 3 instruction tuning without the NextQA dataset?

Hi! Thanks for your interest! These days, we have conducted some explorations about VideoChat2 for reasoning tasks. Considering the experiments are based on the checkpoint, the weight may be not publicly available now. We will release the checkpoints with improved reasoning abilities.


No formatting for Citation in Markdown blog page

No formatting for Citation in Markdown blog page

Also keen to revise the style of lists.

Closed on 08Jun


refactor: Move flags.go to pkg/option

Logically flags.go belongs to pkg/option, so let's just move it there. I also had to move ParseMetricsLabelFilter to pkg/option to break a cyclic dependency.

Cool, I wanted to do that one day and I didn't!


Delete the Test Plan

Description
I should be able to delete a test plan and hide the test from the list.
Possible Solution
Create a logical delete for the test plan.

@dyegowermelinger Do you mean to remove Run from History?
We have also RunPlan as a part of TestConfiguration that basically represents JMeter execution plan.


same address for listarguments and listoutputs??

when I call the following two functions, I found that out_str_array1 and out_str_array2 have same address. It may not be a critical problem, but I think it should be better if you can make the couple be different.
int MXSymbolListArguments(SymbolHandle symbol,
                          mx_uint *out_size,
                          const char ***out_str_array1)

int MXSymbolListOutputs(SymbolHandle symbol,
                        mx_uint *out_size,
                        const char ***out_str_array2)

I don't get your point. Does it hurt to reuse temp space?!

In which case will this matter?

It will not pose a problem, the requirement is that the data is copied out as soon as function finishes, and these temp variables are thread local

not a critical problem anyway.
yes, copy the out as soon as function finishes is a solution.


Replaced Qt 5.15 deprecated method calls

Qt Documentation for the obsolete methods
https://doc.qt.io/qt-5/qstring-obsolete.html#split-1
https://doc.qt.io/qt-5/qprocess-obsolete.html#start-1

Even though it says "Some checks were not successful" on the Github thing under your post, it compiled nicely for me with QT 5.15.0

Compiled nicely for me to.

Compiled without any issues.


Feature Request - Introduce --provider-state-header-name in pact-verifier

The pact-stub-server has a --provider-state-header-name that allows for selecting a better matching interaction based on a header containing the requested provider state. The pact-verifier-cli (rust implementation) is missing this option. We should probably also add the analog header in the verifier.

The reason that flag exists, is because the stub server is loaded with the entire pact contents in advance, so state handlers and descriptions can't be used to delineate the incoming request - specifying a header that can override it makes sense in this context.
In the verification context, only one interaction should be coming in at a time, which can be disambiguated through the state name.
What problem are you trying to solve here?

The short answer is twofold:

speed gained by parallel execution.
The docs state that requests should contain the provider state, and it would be symmetrical.

NOTE: For verifying messages fetched via HTTP, the provider state is also passed in the request to fetch the message, so the state change URL is not required.

The long story, without going into the full details, is:
We are a pretty big organisation and attempted to adopt contract testing several times across different parts of the organisation. This failed in the past, mostly because the gains were largely for the consumer where the producer carried most of the burden and gained the least benefit. To address this (and other points) we reduce the amount of work required to maintain the setup for verifying the consumer contracts by the provider. Part of this is to re-use the contracts the provider has with its providers again as the backing, together with running the binary exactly as it goes to production.
This means that when verifying the contract the request from the consumer contract flows through the app and reaches the pact stub server where a header can be used to disambiguate between the interactions. We previously integrated with the pact-jvm library where we add the additional headers. And we are successful in enabling providers to easily support contract testing.
Because we were unable to upload pact verification results to the broker using the pact-jvm library we want to pivot to using the platform/language agnostic pact-verifier-cli. It works, except for that we lose the ability to send the provider state header. Therefore, we would like to see it added and are happy to contribute it even.

The docs state that requests should contain the provider state, and it would be symmetrical.

NOTE: For verifying messages fetched via HTTP, the provider state is also passed in the request to fetch the message, so the state change URL is not required.

You misunderstood that section. It is explicitly talking about message queue verification where the normal producer - consumer interaction doesn't involve HTTP requests (a Pact V3.
For Pact verification involving a backend service, I would avoid using Pact contract testing as a stub. Pact contract verification should really only verify your producer layers, not downstream service implementations, including their contracts. But, if you really must do this, then you already have instrumented your downstream API client and it should be triivial to add your own header in your state change handler.
Running pact verifications concurrently is fraught with issues because you have to confine the producer state for each test to a single request-response pair, but most mocking libraries operate on global state. If you have a lot of different consumers, I'd just run a test matrix on the CI with each job confined to a specific consumer and so run the verification in parallel against isolated producer servers.

But, if you really must do this, then you already have instrumented your downstream API client and it should be triivial to add your own header in your state change handler.

Indeed, we have done so, but it limits to executing/verifying a single interaction at a time.

Running pact verifications concurrently is fraught with issues because you have to confine the producer state for each test to a single request-response pair, but most mocking libraries operate on global state. If you have a lot of different consumers, I'd just run a test matrix on the CI with each job confined to a specific consumer and so run the verification in parallel against isolated producer servers.

We have this under control for downstream web services and it works with our custom command based on the pact-jvm. We prefer moving to the official pact-verifier-cli in order to gain the other features of that client (mainly communication with the broker).
Note that we have services that have over 100 consumers and setting these kind of CI jobs up manually is a pain which is why we take a lot of orchestration away. By verifying in parallel we also ensure that verification is feasible to run instead of it becoming a hurdle for teams.

Note that we have services that have over 100 consumers

That's where your problem lies. Is there no SDK for your consumers? SDKs would let you reduce that number drastically, because you consolidate all consumer interactions into that SDK. You reduce the cardinality to the number of programming languages / ecosystems your SDK supports.

So the pact stub server is used by consumers to stub the backend - i.e. it stubs the provider. The reason why we have that header, is to disambiguate requests - we need this because all of the interactions are loaded at once. It looks something like this:
sequenceDiagram
    User->>Stub Server: ./pact-stub-server {options}
    Stub Server->>User: PID 1234
    User->>Test: run tests
    Test->>Consumer: execute test
    Consumer->>+Stub Server: GET /a/request [x-state-change-header=state happy]
    Stub Server->>-Consumer: HTTP 200 {...}
    Consumer->>Test: results
    Test->>Consumer: execute test
    Consumer->>+Stub Server: GET /a/request [x-state-change-header=state unhapp]
    Stub Server->>-Consumer: HTTP 404 {...}
    Test->>Consumer: ...

The verifier is the reverse of this - it acts as the consumer. Each request from the verifier comes in to the API one at a time anyway, so we don't need to be able to disambiguate interactions at the request level:
sequenceDiagram
    User->>Verifier: ./pact-verifier {options}
    Verifier->>+API: POST /state/change/URL {"state": "state happy", "params": {...}}
    API->>-Verifier: state change response [OK|ERROR]
    Verifier->>+API: GET /a/request
    API->>V-erifier: HTTP 200
    Note right of Verifier: Next interaction to test...
    Verifier->>+API: POST /state/change/URL {"state": "state unhappy", "params": {...}}
    API->>-Verifier: state change response [OK|ERROR]
    Verifier->>+API: GET /a/request
    API->>-Verifier: HTTP 404
    Note right of Verifier: ...
    Verifier->>User: results

I think what you're proposing is to change this to:
sequenceDiagram
    User->>Verifier: ./pact-verifier {options}
    Verifier->>+API: GET /a/request [x-state-change-header=state happy]
    API->>-Verifier: HTTP 200
    Note right of Verifier: Next interaction to test...
    Verifier->>+API: GET /a/request [x-state-change-header=state unhappy]
    API->>-Verifier: HTTP 404
    Note right of Verifier: ...
    Verifier->>User: results

If so, I'd be supportive of this change. If not, I think I'm missing something - could you please perhaps draw out how you'd like to see it work?

One thing on the final diagram above, if we were to support that, we need to consider the impact on #441 - how would be support provider supplied parameters (if at all?).


Fix env var check for ClientSecretCredential.

Description
ClientSecretCredential asserts all 3 of them to have value, not just exist:

https://github.com/Azure/azure-sdk-for-python/blob/8fe60e912fa9f2074c39b70465f834675ca07e09/sdk/identity/azure-identity/azure/identity/_credentials/client_secret.py#L44-L49

Currently when any of these env var exist but empty, EnvironmentCredential routes the request to ClientSecretCredential and will fail the auth, with undocumented exception ValueError while only CredentialUnavailableError is allowed:

https://learn.microsoft.com/en-us/python/api/azure-identity/azure.identity.environmentcredential?view=azure-python

This PR fixes the check.
Ideally we should cast ValueError or any other reasonable exceptions to CredentialUnavailableError as well, but that's out of the scope of this PR.

Thanks for the contribution.
It is by design that if any of these environment variables are explicitly set, we don't want to silently swallow them even they are empty string.

As I mentioned this is an inconsistency of ClientSecret and EnvCred assert different things. Regardless of the design choice they should be the same. If we want to take empty secret, it should continue as that into the normal workflow and return as auth failure, not ValueError.

Thanks for the feedback.
We treat this case (client_id is set in env with empty string) as configuration mistakes. Users should not handle the error in runtime. Instead, they should fix the configuration.

No the use case is on partial environment where those env var are not intended to be used and we want a fallback to other auth method after catching exception. Do you think we should then be explicit about the existence of ValueError in doc? Lib is not the owner of process lifespan so it should not unconditionally blow up the process on error even when it believes it is user error.

To make sure we are on same page, if any of these env vars are missing, e.g. client_id is not set, identity library does not attempt to use ClientSecretCredential.
Because the env var is not set automatically, the case is you explicitly set client_id, but the value is empty string. (it is different from client_id is not set).
In this case, our recommendation is double check the value of the env var you set or remove the empty client_id env var.

I agree with your that empty value can be considered as error.
Whether to accept empty value is just a design choice, but that's not my point.
The problem here is it raises ValueError while doc only mentions CredentialUnavailableError, so either a bug in code or doc.

Exceptions
CredentialUnavailableError
environment variable configuration is incomplete

https://learn.microsoft.com/en-us/python/api/azure-identity/azure.identity.environmentcredential?view=azure-python

Thanks for the feedback.
Let's fix the docs. :)

I took another look into the doc https://learn.microsoft.com/en-us/python/api/azure-identity/azure.identity.environmentcredential?view=azure-python
Exceptions
CredentialUnavailableError
environment variable configuration is incomplete
is for method get_token().
We don't document exceptions for constructors.
I will discuss with the crew how we want to address it.
It may take some time.


Color-blind-mode: Displays not always the right background

Create a build
Fail the build once
Run the build now:
The field is now blinking, and it is red, but there is not the right color at the backgroud which a not blinking red field has.

Hey @Luke081515, thanks for pointing this out! The latest build should resolve the problem. Looking forward to hearing your thoughts.
Best,
Jan

@jan-molak I checked it out, it works now as expected. Thank you for your fix.
Best,
Luke

Perfect, thanks for your help! 😄


⚠ ClamAV has degraded performance

In 369f6ee, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 224 ms

Resolved: ClamAV performance has improved in ad0f9a9 after 13 minutes.


CUDNN_STATUS_NOT_INITIALIZED with mgpu

Hi,
I really like PointRCNN and wanted to try some things with it out.
When I try to train the RPN with multiple GPUs just the way you describe in the README I get following error.
For some more information:
Cuda Version: 10
Cudnn Version: 10
GPUs: 2x 980 TI (non SLI)
Pop! OS 19.04
I tried running PointRCNN on both GPUs separately and they worked, so the GPUs themself should work.
It also works if I use a batch-size of just 1. Probably because it then only uses 1 GPU. I also tried various other batch sizes.
This is the error I get when running with --mgpu
Traceback (most recent call last):
  File "train_rcnn.py", line 250, in <module>
    lr_scheduler_each_iter=(cfg.TRAIN.OPTIMIZER == 'adam_onecycle')
  File "/-redacted-/PointRCNN/tools/../tools/train_utils/train_utils.py", line 199, in train
    loss, tb_dict, disp_dict = self._train_it(batch)
  File "/-redacted-/PointRCNN/tools/../tools/train_utils/train_utils.py", line 132, in _train_it
    loss, tb_dict, disp_dict = self.model_fn(self.model, batch)
  File "/-redacted-/PointRCNN/tools/../lib/net/train_functions.py", line 35, in model_fn
    ret_dict = model(input_data)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 143, in forward
    outputs = self.parallel_apply(replicas, inputs, kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 153, in parallel_apply
    return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/parallel/parallel_apply.py", line 83, in parallel_apply
    raise output
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/parallel/parallel_apply.py", line 59, in _worker
    output = module(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/PointRCNN/tools/../lib/net/point_rcnn.py", line 33, in forward
    rpn_output = self.rpn(input_data)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/PointRCNN/tools/../lib/net/rpn.py", line 74, in forward
    backbone_xyz, backbone_features = self.backbone_net(pts_input)  # (B, N, 3), (B, C, N)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/PointRCNN/tools/../lib/net/pointnet2_msg.py", line 61, in forward
    li_xyz, li_features = self.SA_modules[i](l_xyz[i], l_features[i])
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/PointRCNN/tools/../pointnet2_lib/pointnet2/pointnet2_modules.py", line 40, in forward
    new_features = self.mlps[i](new_features)  # (B, mlp[-1], npoint, nsample)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/container.py", line 92, in forward
    input = module(input)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/container.py", line 92, in forward
    input = module(input)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 320, in forward
    self.padding, self.dilation, self.groups)
RuntimeError: cuDNN error: CUDNN_STATUS_NOT_INITIALIZED

Sorry, I have no idea about the error since I could not reproduce it. Maybe you could write a simple multiple GPUs code to test your environment first.

Hi,
I really like PointRCNN and wanted to try some things with it out.
When I try to train the RPN with multiple GPUs just the way you describe in the README I get following error.
For some more information:
Cuda Version: 10
Cudnn Version: 10
GPUs: 2x 980 TI (non SLI)
Pop! OS 19.04
I tried running PointRCNN on both GPUs separately and they worked, so the GPUs themself should work.
It also works if I use a batch-size of just 1. Probably because it then only uses 1 GPU. I also tried various other batch sizes.
This is the error I get when running with --mgpu
Traceback (most recent call last):
  File "train_rcnn.py", line 250, in <module>
    lr_scheduler_each_iter=(cfg.TRAIN.OPTIMIZER == 'adam_onecycle')
  File "/-redacted-/PointRCNN/tools/../tools/train_utils/train_utils.py", line 199, in train
    loss, tb_dict, disp_dict = self._train_it(batch)
  File "/-redacted-/PointRCNN/tools/../tools/train_utils/train_utils.py", line 132, in _train_it
    loss, tb_dict, disp_dict = self.model_fn(self.model, batch)
  File "/-redacted-/PointRCNN/tools/../lib/net/train_functions.py", line 35, in model_fn
    ret_dict = model(input_data)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 143, in forward
    outputs = self.parallel_apply(replicas, inputs, kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 153, in parallel_apply
    return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/parallel/parallel_apply.py", line 83, in parallel_apply
    raise output
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/parallel/parallel_apply.py", line 59, in _worker
    output = module(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/PointRCNN/tools/../lib/net/point_rcnn.py", line 33, in forward
    rpn_output = self.rpn(input_data)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/PointRCNN/tools/../lib/net/rpn.py", line 74, in forward
    backbone_xyz, backbone_features = self.backbone_net(pts_input)  # (B, N, 3), (B, C, N)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/PointRCNN/tools/../lib/net/pointnet2_msg.py", line 61, in forward
    li_xyz, li_features = self.SA_modules[i](l_xyz[i], l_features[i])
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/PointRCNN/tools/../pointnet2_lib/pointnet2/pointnet2_modules.py", line 40, in forward
    new_features = self.mlps[i](new_features)  # (B, mlp[-1], npoint, nsample)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/container.py", line 92, in forward
    input = module(input)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/container.py", line 92, in forward
    input = module(input)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 489, in __call__
    result = self.forward(*input, **kwargs)
  File "/-redacted-/.local/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 320, in forward
    self.padding, self.dilation, self.groups)
RuntimeError: cuDNN error: CUDNN_STATUS_NOT_INITIALIZED

so have you finally solved this problem?


Fix lights issue #8098

Description:
Related issue (if applicable): fixes #8098
Checklist:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Cherry-picked for 0.47.1


Cannot open Tianchi files

Hi. :)
I just downloaded the MuCGEC data files from Tianchi but cannot open them because they are password protected.
Can you you help?

Hi! Dear Dr. Christopher, thank you for your interest!
Since we are conducting a CGEC evaluation task based on MuCGEC, only those who participated in our evaluation can get the unzip password. Once our shared task is finished, we will remove the password protection as soon as possible.
I have sent an e-mail to you with the password. Hope this will help you :)

Understood, thanks!


upd(discord): 0.0.16 -> 0.0.17

Today I opened up discord by accident and discovered it needed an update
might as well just script a github actions check lol

@aranym0 Do you wish to continue maintaining this script?

Well, 1 day has passed, merging for now.


[bug] Not Displaying the air quality on 400S

Describe The Bug
Air quality does not show in the latest build.

Expected behavior
Been able to see the air quality
Logs
No error messages
Plugin Config
{
"name": "Levoit Air Purifiers",
"email": "my_email@my_email",
"password": "my_password",
"platform": "LevoitAirPurifiers"
}
Plugin Version
1.2.1-alpha
Homebridge Version
1.6.0
Node.js Version
18.12.1
NPM Version
8.19.2
Operating System
Raspbian 1.0.39

Hey, the alpha version is to release a support for a new device and could have bugs, also the air quality is now splited into a separate accessory, so each device has 2, the purifier and air quality sensor

I switched to 1.2.1 and still not able to see the air quality


Message {"status":"success"} on GET call on /login endpoint?

Issue opened for the creation of a wiki page that summarizes the doubts and problems for newbies (https://github.com/volatiletech/authboss/issues/210).

I'm using authboss in API mode.
I wonder why I get the message {"status":"success"} on GET calls on /login endpoint?

It's a layover from using authboss in API mode. Because it simply translates all the handlers to JSON, the handler for GET from the http side is still there. It has no effects, it renders empty JSON so it shouldn't matter. I realize it's annoying though.


Update README.md

make it clear that this sentence is about eventemitter3, not reflux
shorten sentence

I'm pretty much refactoring all docs, so I wouldn't spend too much time worrying about the docs as they are (I'd rather have you put that time into perfecting the new docs after the fact!). That said, you're right, that sentence is confusing.


Better SMPP multipart_info handling

At the moment, this uses the week-long third_party_id_expiry and is never cleared. Instead it should use the day-log submit_sm_expiry and be cleared after all parts of a message have been acknowledged.
(There may be some overlap with #965, but running out of memory in Redis is a worse problem than potential extra acks.)

Left a couple of comments, otherwise looks good.

:+1: once Travis is happy.


Unusual appearance on Windows Web Server 2008 R2

Before you post
Make sure to search through the issue list to check if others have reported the same bug already. If you find an existing issue, make your voice heard there. Your case may provide valuable extra clues towards a solution and will increase its priority.
Describe the bug
Unusual menu appearance when opening it.
To Reproduce

Install Windows Web Server 2008 R2
Download win-acme.v<IP_ADDRESS>2.x64.pluggable
Run wacs.exe

Expected behavior
Normal options.
Log
[INF] Connection OK!
[DBG] Running with administrator credentials
[DBG] IIS version 7.5
[WRN] Scheduled task points to different location for .exe and/or working directory
[WRN] Scheduled task exists but does not look healthy
[INF] Please report issues at https://github.com/win-acme/win-acme
[VRB] Unicode display test: Chinese/語言 Russian/язык Arab/لغة
[VRB] Exiting with status code 0
[DBG] Found 1 secrets in secrets.json
[DBG] Renewal period: 55 days
[VRB] Sending e-mails false
[INF] No command line arguments provided
[VRB] ExePath: C:\Users\Administrator\Downloads\win-acme.v<IP_ADDRESS>2.x64.pluggable\wacs.exe
[VRB] ResourcePath: C:\Users\Administrator\Downloads\win-acme.v<IP_ADDRESS>2.x64.pluggable
[VRB] PluginPath: C:\Users\Administrator\Downloads\win-acme.v<IP_ADDRESS>2.x64.pluggable
[INF] Software version <IP_ADDRESS>2 (release, pluggable, standalone, 64-bit) started
[INF] Connecting to "https://acme-v02.api.letsencrypt.org/"...
[DBG] [HTTP] Send GET to "https://acme-v02.api.letsencrypt.org/directory"
[DBG] Connection failed: The request was canceled due to the configured HttpClient.Timeout of 10 seconds elapsing.
[DBG] [HTTP] Send GET to "https://acme-v02.api.letsencrypt.org/"
[VRB] [HTTP] Request completed with status "OK"
[VRB] [HTTP] Response of type text/html (1540 bytes)
[DBG] Connection failed: Unable to parse response content
[DBG] Initial connection failed, retrying with TLS 1.2 forced
[DBG] [HTTP] Send GET to "https://acme-v02.api.letsencrypt.org/directory"
[VRB] [HTTP] Request completed with status "OK"
[VRB] [HTTP] Response content: {
"9YG7hfP7yUs": "https://community.letsencrypt.org/t/adding-random-entries-to-the-directory/33417",
"keyChange": "https://acme-v02.api.letsencrypt.org/acme/key-change",
"meta": {
"caaIdentities": [
"letsencrypt.org"
],
"termsOfService": "https://letsencrypt.org/documents/LE-SA-v1.3-September-21-2022.pdf",
"website": "https://letsencrypt.org"
},
"newAccount": "https://acme-v02.api.letsencrypt.org/acme/new-acct",
"newNonce": "https://acme-v02.api.letsencrypt.org/acme/new-nonce",
"newOrder": "https://acme-v02.api.letsencrypt.org/acme/new-order",
"renewalInfo": "https://acme-v02.api.letsencrypt.org/draft-ietf-acme-ari-02/renewalInfo/",
"revokeCert": "https://acme-v02.api.letsencrypt.org/acme/revoke-cert"
}
[INF] Connection OK!
[DBG] Running with administrator credentials
[DBG] IIS version 7.5
[WRN] Scheduled task points to different location for .exe and/or working directory
[WRN] Scheduled task exists but does not look healthy
[INF] Please report issues at https://github.com/win-acme/win-acme
[VRB] Unicode display test: Chinese/語言 Russian/язык Arab/لغة`
Platform:

OS: Windows Web Server 2008 R2
Version: <IP_ADDRESS>2, 64-bit, pluggable

Additional context

Thank you, Shalom.

I don't think win-acme (or any other software) supports Server 2008 anymore. You should consider not connecting this OS to the internet especially if it hosting anything remotely business critical. I would suggest moving your application/workload to a new server as soon as possible.

Duplicate of https://github.com/win-acme/win-acme/issues/2515


Navigate to a page without a page transition animation

Is it possible to immediately present a page without the page transition animation?

https://github.com/eclipsesource/tabris-js/issues/1118#issuecomment-277994625

Thanks @cookieguru !


[8.x] Add ConfirmableTrait to ConfigCacheCommand

We've had a few instances where config:cache has been ran, so then that is being used over the values in phpunit. So it's taken down a few databases unintentionally.
I can see that this was added in 5.2 to increase throughput, and it's in the docs to not use this command locally.
However we've still seen it happen (to myself personally also).
All this does is follow the same convention when running migrations on production.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

I feel like this one should get a rethink.
If config:cache shouldn't be run outside of production there should absolutely be a warning
It also doesn't seem like something which could be shared as a package as suggested, since you'd have to overwrite the config:cache command via the package, which i'm not sure is possible.

@taylorotwell
I appreciate your point about trying to reduce the amount of code included in the framework, but we've been burned by this quite badly in the past. And while we can tell people not to run config:cache locally, it still seems like a pretty big accident waiting to happen.
I'm not sure how this could be a package either. So if you have any thoughts on how best to achieve that I'm all ears.

If config:cache shouldn't be run outside of production there should absolutely be a warning

There isn't a golden rule about any environment here. It's about knowing what the command does and what the consequences are. All of which is explained in the docs here: https://laravel.com/docs/7.x/deployment#optimizing-configuration-loading
People still might want to use the command locally to test things out before deploying to production. All of the past examples of people getting tripped over this is not reading the docs properly.

How about this solution @CraigHarley ?
In Foundation/Bootstrap/LoadConfiguration.php it checks for the existence of the cached files:
https://github.com/laravel/framework/blob/0c93bf343dd54496b026e6fa900061c60421e8d5/src/Illuminate/Foundation/Bootstrap/LoadConfiguration.php#L27
Which uses the getCachedConfigPath
https://github.com/laravel/framework/blob/0c93bf343dd54496b026e6fa900061c60421e8d5/src/Illuminate/Foundation/Application.php#L966
This uses the APP_CONFIG_CACHE variable, so presumably you can set that in your phpunit.xml like so:
<server name="APP_CONFIG_CACHE" value="cache/test_config.php"/>

It's not ideal, but it would solve 99% of cases and allow you to run tests that involve the cached config without affecting the the non test environment.
I imagine at the moment if you ran a test were php config:cache occurred during the test, it would overwrite the config from the app with whatever your test config is, which isn't ideal? (I doubt anyone would run it in a test, but you never know...)
If this was in phpunit.xml by default or mentioned in the docs that might be better?

@robjbrain
That makes sense and seems like a great solution.
For others reading this I think thephpunit should look like this:
<server name="APP_CONFIG_CACHE" value="some-non-existent-file" force="true"/>

+1 to adding to phpunit.xml by default.


Fix deprecations on Buffer and Crypto

Fixes #14

package.json and yarn.lock should be updated too, right_

Updated package.json however I don't see any changes in yarn.lock

When you force push you must request a review by me. If not I will not get a notification.

Ok.
I think we can switch to 10.15.3 on CI as it is LTS at this moment so it is wise to be on LTS. What you think? I am testing using LTS locally as always.

Also, I would encourage you to use alpine images on CI as much as possible as it speeds up the pipelines.

Okay. Change it to alpine.

I also created https://github.com/Levino/mock-jwks/issues/16 fyi.

misclick

Replaced by #17


Unescaping already escaped xml URLs

I made a change fixing unescaped URLs in report page. For details see #441.
Description
Current implementation expects unescaped URLs in suite XML (e.g (...)?a=b&c=d ) which are then escaped before serializing XML to TestSuite. If you put correctly escaped XML in URL (e.g (...)?a=b&c=d) then the query params are escaped to (...)?a=b&amp;c=d, parsed back (while serializing) to (...)a=b&c=d and this version is then displayed in report page.
This redundancy in escaping is what causes the issue. If the URLs are already escaped they shouldn't be tampered with. My fix uses the simpliest solution, i.e it tries to unescape before escaping, thus handling correctly both unescaped (unescape has no effect here) and escaped URLs.
I also added a unit test for escaped URLs.
Motivation and Context
Closes #441
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed (and updated if needed) the documentation regarding this change

I hereby agree to the terms of the AET Contributor License Agreement.

Hello @wiiitek, you've made a few good points and I feel obliged to clear up some misunderstandings.
First of all:

But something is not right here. Isn't it that:
StringEscapeUtils.escapeXml11(StringEscapeUtils.unescapeXml( something )) === something
??

That's true only for escaped strings. Take for example this string: &. The above statement evaluates to escape(unescape(&)) == escape(&) == & =/= &.
As you can see unescape() has no effect on not-escaped strings. This little workaroud of unescaping and escaping ensures that at the end we get an escaped string regardless of initial input being escaped or not.
Secondly:

But what should we do with this URL:
<url href="https://www.google.pl/search?param=12&q="" />
If something like that is placed inside suite - what page should AET test? Currently it will do search for >", but after the proposed change it will search for "?

You are right that this url will in the end be serialized to (...)?param=12&q=", however I have some problems with this url. To me, it would never be necessary to use XML escaping on query strings. If you have a valid URL, all XML special characters (&<>"') are already escaped. The exception is & delimiting query parameters
In conclusion:
To my mind the only enemy among XML special characters (&<>"') is the ampersand because I don't see any possibility of any of those <>"' occurring unescaped in a valid URL, although I might be wrong. With a valid URL user has to decide whether to escape all & chars with & or keep them unescaped (thus making suite.xml invalid XML file). Both of those options will be correctly serialized but I agree that it might be misleading and sticking to one option would be nice.

Thank you, yes I agree :

To me, it would never be necessary to use XML escaping on query strings. If you have a valid URL, all XML special characters (&<>"') are already escaped. The exception is & delimiting query parameters

But it might be confusing what one should do in case of testing escaped special characters on search pages. Maybe we should update documentation? ;p
What should i put into XML if i want to test something like that:

?

Dealing with <>"
Hmm, I delved deeper into the subject and according to RFC1738 characters such as  <>" are regarded unsafe and should ALWAYS be encoded.
In fact, in your screenshot they are escaped but it appears that chrome tries to be user friendly and displays prettified version in the url bar. Fortunately, if you copy the address in the url bar and paste it in a text editor you get escaped version.
Example:
the link you provided:
https://www.google.pl/search?source=hp&q="%26amp%3B+vs+%2526"
copied from the url bar into notepad looks like this:
https://www.google.pl/search?source=hp&q="%26amp%3B+vs+%2526"
I carried out a small experiment and pasted
http://nonexistingpage.mock/?checkmark=✓&q="heh"
in url bars of different browsers. Results:

If you copy it back from the url bar to the notepad you get the original (escaped) URL.
Concluding, i think that unless you type the url by hand you shouldn't have to deal with escaping query parameters.
Dealing with ;
The only problem left is ; character. Similarly to & it is a reserved character and thus can occur unescaped in URLs. We wouldn't want strings such as &, < existing in valid URLs because they would get wrongly interpreted.
W3 reccommends (here) the possiblity of using ; as a query string separator instead of &.
So i guess a set of query strings like this: "a","amp","b" can IN THEORY be represented as ?a&amp&b or ?a;amp;c or even ?a&c (this is a a strange mix-up but i guess it's valid?). This remains a problem but it seems like an edge case almost impossible to occur.
I have to admit this is more complicated than it seemed :p I wonder what are your thoughts

Thanks @wblachowski for your answer! I see you put an effort to look into this.
Please make a decision on this topic after some consultations with @Skejven and @mchrominski.
Personally I like the approach with with "valid XML" and making no escaping/unescaping but i understand that this could be not acceptable because of end-users. My other thought is that the XML we are using is not friendly format - we don't have XML schema (I remember there were attempts to prepare one). So any other format would be better ;p
Thank you again for your work. I believe you will make good decision. Cheers!

Whole this topic is resolved here: https://github.com/Cognifide/aet/pull/524
I think this PR can be closed.


Not rendering configuration

Hey there,
i'm currently testing laravel-trix as a new editor for my webapp. Unfortunately the custom configuration is not rendering as expected.
Blade:

Comment?
@trix(\App\Events::class, 'description', [ 'containerElement'=>'div', 'id'=>'comment_editor', 'hideToolbar'=>'true', ])

Generated HTML:

Output:

Any ideas?

@realtarget should be fixed in version 2.0.1


add sync-api action

Codecov Report

Merging #5 (f65fd0e) into master (3be63bb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   55.40%   55.40%
=======================================
  Files          85       85
  Lines        8258     8258
=======================================
  Hits         4575     4575
  Misses       3151     3151
  Partials      532      532

Flag
Coverage Δ

unittests
55.40% <ø> (ø)

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

Continue to review full report at Codecov.

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

Codecov Report

Merging #5 (dfc5cff) into master (3be63bb) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   55.40%   55.41%   +0.01%
==========================================
  Files          85       85
  Lines        8258     8343      +85
==========================================
+ Hits         4575     4623      +48
- Misses       3151     3182      +31
- Partials      532      538       +6

Flag
Coverage Δ

unittests
55.41% <ø> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/controller/utils/utils.go
18.88% <0.00%> (-3.62%)
:arrow_down:

cmd/core/main.go
13.52% <0.00%> (-0.17%)
:arrow_down:

...troller/core.oam.dev/v1alpha2/application/apply.go
62.93% <0.00%> (-0.04%)
:arrow_down:

...dev/v1alpha2/application/application_controller.go
53.09% <0.00%> (+0.46%)
:arrow_up:

...ller/core.oam.dev/v1alpha2/application/revision.go
70.05% <0.00%> (+1.63%)
:arrow_up:

pkg/oam/util/test_utils.go
60.27% <0.00%> (+7.81%)
:arrow_up:

Continue to review full report at Codecov.

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


Error: improper code

Browser: Supermium 118 (Chromium based)
Text to paste:

Lorem ipsum dolor sit amet, consectetur adipiscing elit. Etiam eu sem placerat, dapibus massa eu, consectetur orci. Fusce ultrices blandit ante a ullamcorper. Phasellus pellentesque est sed ligula bibendum, ac lobortis enim sollicitudin. Nunc tincidunt leo in sapien tincidunt volutpat. Vestibulum metus lectus, rutrum non arcu a, molestie rhoncus metus. Etiam ornare dolor mauris, et scelerisque urna pharetra ac. Aenean feugiat porta nulla eu tristique. Aliquam sed elit auctor, dignissim neque hendrerit, finibus ipsum. Interdum et malesuada fames ac ante ipsum primis in faucibus.

Save and get URL
ERROR: Improper code!

Why?

Hi @sergeevabc, The online service running at https://mathb.in/ expects the input code to contain some LaTeX mathematics delimiter. When no such delimiter is found, an error is returned.

@susam, it is NOT obvious, consider adding a tip or a more meaning error message at least. For example, the frontpage says Mathbin is for math snippets. Okay, let's expand 'lorem ipsum' (looks proper Markdown to me) with '2+2=4' (not LaTeX, still proper math), yet 'Improper code' again. Otherwise, one has to use hacks like @enzotib did.

You lucky, I waited more than two months to get an answer I already found myself

Hi @sergeevabc, Thank you for the feedback. There are currently mainly two reasons why the error message for this specific error is unclear.

The error message (see https://github.com/susam/mathb/blob/b15f3c8/mathb.lisp#L439-L440) is not configurable.  However, the condition for generating this error message is configurable via the :expect configuration option documented at https://github.com/susam/mathb#runtime-options.  For example, the live instance running at https://mathb.in/ is configured to generate this error when LaTeX delimiters are missing.  However, someone else running a separate instance of MathB may configure it to generate an error when a different set of tokens is missing from the post.  It is also possible to configure MathB to not generate this error at all.

Keeping the error message unclear also reduces drive-by spamming to some extent.  Pastebins like this face significant spamming problem.  The measures to discourage spam in MathB.in have evolved over many years.  The "improper code" error message was one of the things that had a positive correlation with reduced spam.

I have to admit though that these are really mundane reasons for keeping an unclear message.  A clearer error message will certainly be helpful to genuine authors of a post.  The unclear error message can indeed be frustrating.  The error message can definitely be made configurable, so that as one configures the :expect option, they can configure the corresponding error message too.  However, this is a volunteer driven open source project and this particular issue is not a high priority issue for me.  If someone else is strongly motivated to resolve this issue, a pull request or patch is very welcome.

Hi @enzotib, Sorry, it took so long for you to get an answer when you reported this issue earlier at #29.  I maintain this project in my spare time and there are some months when I am too busy to devote any time to this project.  I do respond to messages when I can, based on how much spare time I have and how I want to spend that spare time.  I know this is not ideal for someone stuck due to an unclear error message.  I'd love for their to be more volunteers who could help out with the issues and questions being submitted.  However, the fact is that this is a small project with not many volunteers, so the time to respond to new issues and questions can be rather long.


Promote SPI/RemoteSecret from staging to prod

Included PRs:

https://github.com/redhat-appstudio/remote-secret/pull/275
https://github.com/redhat-appstudio/remote-secret/pull/263
https://github.com/redhat-appstudio/remote-secret/pull/274
https://github.com/redhat-appstudio/remote-secret/pull/266
https://github.com/redhat-appstudio/remote-secret/pull/269
https://github.com/redhat-appstudio/remote-secret/pull/271
https://github.com/redhat-appstudio/remote-secret/pull/272
https://github.com/redhat-appstudio/remote-secret/pull/270
https://github.com/redhat-appstudio/remote-secret/pull/264
https://github.com/redhat-appstudio/remote-secret/pull/248
https://github.com/redhat-appstudio/remote-secret/pull/267
https://github.com/redhat-appstudio/service-provider-integration-operator/pull/808
https://github.com/redhat-appstudio/service-provider-integration-operator/pull/798
https://github.com/redhat-appstudio/service-provider-integration-operator/pull/807

/lgtm


class='MJX-TeXAtom-ORD' in MathML crashes mathml2chtml

Issue Summary
If class='MJX-TeXAtom-ORD' is present in MathML being converted by mathml2chtml, then the function crashes. This happens in both Firefox and Chrome
Steps to Reproduce:

Open https://mathjax.github.io/MathJax-demos-web/input-mml2chtml.html
Paste the following MathML into the input area:

 <math display='block' id='Mna33kyx-0'>
    <mn class='MJX-TeXAtom-ORD' id='Mna33kyx-9'>2</mn>
 </math>

Click Render and open the debugger to see the message:

Uncaught Error: Unexpected text node "2"
    at t.error (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:205225)
    at t.addText (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:204033)
    at t.addChildren (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:203371)
    at t.makeNode (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:202222)
    at t.addChildren (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:203503)
    at t.makeNode (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:202222)
    at t.addChildren (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:203503)
    at t.makeNode (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:202222)
    at t.addChildren (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:203503)
    at t.makeNode (cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js:1:202222)

I know it might seem strange to have this internal class set, but the situation is that the math is being grabbed from the web, processed a little, and being sent back to MathJax for rendering. Since the original source (Wikipedia) used MathJax, that shows up.
Technical details:

MathJax V3.2.0

The example came from https://en.wikipedia.org/wiki/Quadratic_formula, so I guess they are still stuck on v2.
The movement of the attr to the <mn> comes from there being a single mrow around the <mn>.  The MathML spec says that mrow with a single child can be ignored unless they have attrs on them. I think the only MathML attr allowed on mrow is dir and since I'm not concerned about display in my app, I just coalesce any attrs on mrow into the child figuring that things like classes will still do what they were intended to do on the child. Apparently, that's not a good assumption. Normally this (bad) assumption won't be a problem for AT, but some AT may decide to display the math during navigation for sighted folks who are watching so display is potentially used (as it is in my demo for how navigation might work in AT).
Based on your reply, removing MJX-TeXAtom-* and data-mjx- should solve the problem in v2 and v3. Anything else that should be removed?

I guess they are still stuck on v2.

Yes, wikipedia is using v2.

I think the only MathML attr allowed on mrow is dir

I think that overlooks the attributes shared by all MathML elements, which includes class (and style, id, href and so on).  There is also attributes shared by all presentation elements, which include mathcolor and mathbackground, and also apply to mrow.  So it seems that you should not remove mrow when it has class set (or any of these other attributes).  In fact, I'd argue that, in a setting like HTML pages where dataattributes are allowed, you shouldn't removeif it has adata-* attribute either (these are mentioned in the section on [maction](https://www.w3.org/TR/MathML3/chapter3.html#presm.maction), so the spec does recognize these in the HTML setting, and they are allowed on any element, including mrow`).

Anything else that should be removed?

If you are going to remove these attributes, then I'd recommend removing any class starting with MJX-, not just MJX-TeXAtom-*, as there are others used in v2 to capture MathJax data that isn't available in MathML directly (in v3 they should all be in data attributes).

Apologies for not being precise -- I meant the only mrow-specific attr is dir. My code merges any mrow attrs into the single child element which is why the mn ended up with class='MJX-TeXAtom-ORD' . I was trying to avoid removing attrs, although since I'm mainly targeting speech and braille, and any display will likely be in a different environment than the originally authored code, leaving the attrs may not be ideal. I'll learn more when the code gets used more.
I'll change my code to remove MJX-*. Again, thanks for letting me know how to fix the problem.


Add support for Eto?

"Boo hoo, no cross-platform UI framework in .NET Core"Um... Folks... Y'all need to pay more attention to the ecosystem. pic.twitter.com/fp01OoqkhZ— Jo Shields (@directhex) May 7, 2018
We need someone to research if / when we should support https://github.com/picoe/Eto and what's involved.

If thinking of cross-platform support, there’s also Avalonia that looks super great! Other than that, WPF is coming to .NET Core although !

We won't be supporting Eto in the short term.


HttpHostConnectException  without any response and no timeout

When I request an unkonwnhost that didnot exit  like this   http://<IP_ADDRESS>:8888 （you can reqeust it by your browser ，it just loading all the times）
it without any response  even  I set  connectTimeout（10，TimeUnit.SECONDS）
it will be failure by several minutes but not 10 seconds and throw HttpHostConnectException like this
Connection to http://<IP_ADDRESS>:8888 refused
12-28 19:40:25.412 4813-5174/? E/beacon: execute error:org.apache.http.conn.HttpHostConnectException: Connection to http://<IP_ADDRESS>:8888 refused

I realize  HttpHostConnectException that  not belong to  connecttimeout or readtimeout
But I want to konw ， if there no response by 10 seconds ，how can it return a callback anyway。

Can you provide a reproducing test case.  The connect timeout is firing correctly for me on both desktop and an Android emulator.
If you call call.execute it should throw an exception you can catch, and if you call enqueue it should call Callback.onFailure

It's also strange you have an apache http exception (org.apache.http.conn.HttpHostConnectException)


CFE: Implement missing override checks

CFE doesn't fully implement all checks related to overriding/inheriting members yet.
This includes reporting the following errors:

When method is "overridden" with getter/field and vice versa.
When static member conflicts with inherited instance member with the same name.

Currently, some of these errors are reported by VM. However, we would like to add tree shaking kernel transformation which removes conflicting members and VM no longer has a chance to report these errors. This issue will be used to silence failing tests for override checks in status files.
/cc @kmillikin @stefantsov @mraleph

As of 8f6c6c2a8edd514cdb6533541189a6572318facd it is now an error to declare an instance method and inherit an instance getter or setter with the same name, or to declare an instance getter or setter and inherit an instance method with the same name, or to inherit both an instance method and an instance getter or setter with the same name.
We don't catch the cases of static members conflicting with inherited instance members.  While we do catch duplicated declarations of methods and getters in the same class, we do not catch duplicated declarations of a method and a setter with the same name (presumably we should, which is related to issue #33077).
Note that there's an unfortunate interaction where we will signal an error for any conflicting inherited members, and also signal an error that they are not both implemented if the class is not abstract.  Since you can't actually fix the second problem by implementing both of them, we should find a way to suppress that error.

Please note that the specification of member declaration conflict rules are being updated, cf. https://github.com/dart-lang/sdk/issues/33235. I believe that the behavior requested here matches the updated specification, but it may be useful to consult the table here in order to clarify corner cases.

I'm not 100% clear where the implementation is on this.  It sounds like most (but not all) of this is implemented in the CFE?  Probably relatively rare that this breaks something, but it will be a breaking change to fix.  I'm ok with leaving it as an open bug against Dart 2, but we will need to fix it at some point.

From the tables here: https://github.com/dart-lang/sdk/issues/33235#issue-326617285
It seems that
has instance setter `n=`      has instance method `n`       2450 or 2451

is not handled, i.e. it should give an error, but doesn't.
and that
declares constructor `C.n`    has instance method `n`
declares constructor `C.n`    has instance getter `n`

should be allowed but isn't.

Oh, and there's also some tests tagged with 32613 that fails...

Any updates on this, @jensjoha ?

https://dart-review.googlesource.com/c/sdk/+/64500 should make it comply with the tables at https://github.com/dart-lang/sdk/issues/33235#issue-326617285
https://dart-review.googlesource.com/c/sdk/+/64542 should fix tests tagged with 32613

and https://dart-review.googlesource.com/c/sdk/+/64840

@stefantsov Can this be closed?

I believe so.


KHR_binary_glTF should just be binary_glTF

Fixes schema errors in these Cesium sample binary glTF models.

Nice, thanks for the fixes, @lasalvavida!


The attachments are displaced after pasting

I am using grape_juice's isometric grid. Sometimes the selected token will copy and paste as intended, but others it will paste in the correct position, but the actor, walls, and drawings will be separated.

As you can see on the right, the walls have been displaced by two grid spaces. The token has also been moved from the bottom-most floor square.
Original paste location:

Note that the textures are not tiles; they are drawings.

Can you reproduce the problem without the isometric module as I've no access to it?

Should be fixed in 3.1.1. If not then open a new issue.


Add --test262-esnext option to run-tests.py

Changes:

Download a quite new test262 revision and run it with the bugfixed test262 harness
Run tests on CI to be able detect regressions and progressions too

JerryScript-DCO-1.0-Signed-off-by: Csaba Osztrogonác<EMAIL_ADDRESS>

We agreed that we import the 3rd party test262 harness instead of cloning and patching it. I'm going to update the PR soon.


add userstamps to timestamps()

Can you add the created_by and updated_by fields to timestamps() so that all new tables created will include the userstamps fields

I've considered adding something along those lines - will try to take a look at it some point - happy to accept a PR though!

:-)    What is a PR?

Handy section on Pull Requests (PRs) here https://guides.github.com/activities/contributing-to-open-source

I'm aware the issue is old & closed but I've documented the way to combine timestamps() & userstamps() below. This is tested on 5.7 but any version of Laravel that has Blueprint able to use Macro should be ok.
AppServiceProvider:
Blueprint::macro('userTimeStamps', function() {
            $this->timestamps();
            $this->unsignedBigInteger('created_by')->nullable();
            $this->unsignedBigInteger('updated_by')->nullable();
            $this->unsignedBigInteger('deleted_by')->nullable();
        });

The created_by, updated_by & deleted_by also use unsignedBigInteger as this is consistent with the id columns. You could also change them to foreign keys if you wished.
You then simply need to do
$table->userTimeStamps();

inside your migrations.

Good spot on the unsignedBigInteger @harry-synergi - have updated the readme.

Good spot on the unsignedBigInteger @harry-synergi - have updated the readme.

No problem. I'm also confused why you're doing default(null) when nullable() should take care of that? Is this due to the strict DB mode in laravel 5.3?

Good spot on the unsignedBigInteger @harry-synergi - have updated the readme.

No problem. I'm also confused why you're doing default(null) when nullable() should take care of that? Is this due to the strict DB mode in laravel 5.3?

Nope - just a mistake! ☺


Fix interactive figures in scipy fitting notebook

@wardsimon @celinedurniak @rozyczko sorry for assigning for review you without asking. It would be great if one of you can do it as I am bit pressed for time with the other stuff.


To redux

@mcdonnelldean I did this just so you can check it out. I haven't had time to finish it. so, don't merge. I'll finish it in the morning :)

@ckiss Leave this open, both you and @mirceaalexandru can jump on this monday.


cmd/go: origin does not appear to be a git repo error on all go mod commands

What version of Go are you using (go version)?

$ go version
go version go1.14.1 darwin/amd64

Does this issue reproduce with the latest release?
no.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/balintp/Library/Caches/go-build"
GOENV="/Users/balintp/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/balintp/go"
GOPRIVATE=""
GOPROXY="direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/balintp/skaffold/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/lk/h2pmhk456999cwnrm162297400fv8w/T/go-build176808651=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
go list -m -json or any go mod commands
What did you expect to see?
{
        "Path": "github.com/GoogleContainerTools/skaffold",
        "Main": true,
        "Dir": "/Users/balintp/skaffold",
        "GoMod": "/Users/balintp/skaffold/go.mod",
        "GoVersion": "1.14"
}

What did you see instead?
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>invalid version: git fetch -f origin refs/heads/*:refs/heads/* refs/tags/*:refs/tags/* in /Users/balintp/go/pkg/mod/cache/vcs/3930fbb77689992ff1e965e902478ed592c6673062a4ad1276ed7dda23a6a131: exit status 128:
        fatal: 'origin' does not appear to be a git repository
        fatal: Could not read from remote repository.

        Please make sure you have the correct access rights

Deleting the cache folder ended up helping:
rm -rf /Users/balintp/go/pkg/mod/cache/vcs/3930fbb77689992ff1e965e902478ed592c6673062a4ad1276ed7dda23a6a131
Which means that somehow the cache gets into an invalid state (not sure how that happened). And I wish I'd moved  the folder instead of deletion - now I can't give you more detail around what the state of that folder was.

Thanks for the report. Since this appears to be filesystem corruption and we don't know the cause, I'm going to go ahead and close this issue, but we'll watch for a pattern — if we get more reports of similar issues we'll certainly want to take a closer look.
CC @jayconrod @matloob

I'd just like to note that this happened to me as well.
$ go get -u github.com/matthewmueller/diff
go get github.com/matthewmueller/diff: module github.com/matthewmueller/diff: git ls-remote -q origin in /Users/m/Go/pkg/mod/cache/vcs/5f9f95779422363e8e24e203ef2f34dbab3a4cb052f98aa116323dc518593f13: exit status 128:
        fatal: 'origin' does not appear to be a git repository
        fatal: Could not read from remote repository.

        Please make sure you have the correct access rights
        and the repository exists.
$ git remote -v
origin  git@github.com:matthewmueller/duojs.git (fetch)
origin  git@github.com:matthewmueller/duojs.git (push)
$ rm -rf /Users/m/Go/pkg/mod/cache/vcs/5f9f95779422363e8e24e203ef2f34dbab3a4cb052f98aa116323dc518593f13
$ go get -u github.com/matthewmueller/diffgo: github.com/matthewmueller/diff upgrade => v0.0.0-20191220174011-88a65b538395
go: github.com/sergi/go-diff upgrade => v1.1.0
go: github.com/kr/pretty upgrade => v0.2.0
go: github.com/kr/text upgrade => v0.2.0
go: downloading github.com/kr/text v0.2.0

@matthewmueller Which repository did you make public? github.com/matthewmueller/diff? The public status of the main repository (git@github.com:matthewmueller/duojs.git shouldn't matter here.
Are you able to reproduce this? If so, can you post the config file for the repository with the error? It would have been /Users/m/Go/pkg/mod/cache/vcs/5f9f95779422363e8e24e203ef2f34dbab3a4cb052f98aa116323dc518593f13/config.
Could you post any relevant parts of ~/.gitconfig? I'd be particularly interested in any url and insteadOf lines.

Which repository did you make public? github.com/matthewmueller/diff?

Yep!

Are you able to reproduce this? If so, can you post the config file for the repository with the error? It would have been /Users/m/Go/pkg/mod/cache/vcs/5f9f95779422363e8e24e203ef2f34dbab3a4cb052f98aa116323dc518593f13/config.

Sorry, I deleted this repo.

Could you post any relevant parts of ~/.gitconfig? I'd be particularly interested in any url and insteadOf lines.

[url "git@github.com:"]
	insteadOf = https://github.com/

This has been here for awhile though.
One thing I did recently is add GOPRIVATE to my rc file so I can pull from private repositories
export GOPRIVATE=github.com/matthewmueller

@matthewmueller Your ~/.gitconfig file explains the remote SSH URLs. And it seems like those URLs should still be valid. I can't think of a reason why anything here would cause problems.
If you (or anyone else) run into this problem in the future, it would be helpful if you could post:

Git version.
Any changes to repository privacy, remote URLs, authentication that might cause this.
Repository config file.
Ideally, a .zip of the repository inside the module cache (maybe a lot to ask I know; it would help us rule out file corruption).

@balopat @matthewmueller you are a life-saver! I have been looking for the solution for almost a day, and this is exactly what fix mine. Thank you for sharing!

I wish I had read to the bottom of this issue about saving the cache folder for troubleshooting. I turfed my cache after running into this error:
	github.com/dlclark/regexp2: module github.com/dlclark/regexp2: git ls-remote -q origin in /Users/jesseredl/go/pkg/mod/cache/vcs/41f57ef0dad41000850e307b84f39892ac7069a4e73d6b9c629532f5cbf182af: exit status 128:
	fatal: 'origin' does not appear to be a git repository
	fatal: Could not read from remote repository.

	Please make sure you have the correct access rights
	and the repository exists.

We faced this issue recently and for us it was introduced due to the recent git vulnerability update i.e, the pkg mod cache was owned by some other user and go process was being run by another user. Since git prevents you from running git commands inside dirs not owned by you after the recent update (fatal: unsafe repository (‘/dir/path’ is owned by someone else), go was unable to run git ls-remote -q origin and hence the failure.
We also initially thought that the cache got corrupted somehow but that wasn't the case for us. So, for others who face this issue in the future, please try this first before deleting the cache to find the RC:
# cd to the cache dir reported in the error msg, eg.
cd $GOPATH/pkg/mod/cache/vcs/41f57ef0dad41000850e307b84f39892ac7069a4e73d6b9c629532f5cbf182af
git remote -v # should show origin as a remote
# for us all git commands were failing but if the above command works for you then try the actual command being run by go. i.e,
git ls-remote -q origin


Ajax Base Component - Query Parameter pass-through support

JS-centric version of #479
Setting  [cq:Component]@ajaxPassQueryParameters = true on the Ajax'ified component will persist the QPs from the page request into a data-* attribute, which the AJAX GET will pick up and apply to the XHR call to AJAX in the component.
@B-Johnson @ice1138 ... WDYT?  Can you double check the encoding in the JSP and the decoding the JS .. think it's right; encodings sometimes confuse me though.
Thanks!

You should use this to define parameters, rather than encoding them yourself.
Also, use the cache option on the get() call to do cache busting.


[#1] Generalize Reaction Tracking

Generalizes reaction tracking to work with multiple commands by including a type field in the tracking commands, which can be given the command's "type".
This makes the process of tracking slightly more complicated, as the type must be registered first, along with a "validator".
A "validator" is a function which accepts some tracking information and a message and determines if the message is tracked by that information. It needn't track only a single message, it can track entire channels if so desired. All validators return a Boolean.
The full usage information is given at the top of utils/reaction_tracking.js.

This is tied to issue #1.

works good


Adjust retry parameters for non-Pygithub requests

This changes the number of retries and the lengths of inter-retry sleeps to match the configuration for Pygithub requests.
Closes #147.

Thank you! Let's see how/if this would help.  To speed up updating etc, will release it as well


fix(XMLHttpRequest): resolve the "Illegal invocation" exception (conditional setter)

Fixes #339
Fixes an issue where setting XHR callbacks like ontimeout did nothing because we've treated them as method calls and not property setters.

Root cause for Illegal invocation on setters
Turns out, having a handler.set in the Proxy handlers added conditionally is quite an issue. If the setter function is always added to the proxy, there are no illegal invocations anymore.
Root cause for xhr.responseType not being respected
JSDOM sets a bunch of property descriptors on both XMLHttpRequest implementation and its prototype. We were not forwarding any of the prototype descriptors in our proxy, and that's why setting xhr.responseType didn't propagate to JSDOM and didn't affect the returned response.

This is fixed on the XHR controller level as this is something specific to XHR in JSDOM.

Released: v0.22.4 🎉
This has been released in v0.22.4!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


setting max-cache-ttl to 0 causes slowdown problems

Describe the issue you are having
with the change from last that set max-cache-ttl to 0, the lancache DNS service is now quite slow.  each lookup can take one or two full seconds, and subsequent checks are not cached at all so they also take a second or two.  Further more, even a downstream pihole will not cache the hits because the TTL was set to zero.
The change was made with commit #131
of the three changes to the options file:
'forward-only' seems fine, and it accomplishes the goal of that commit.
'max-ncache-ttl' doesn't seem to break anything, although the default bind value is 10800
'max-cache-ttl' of 0 is the problem.   Per the docs, the default value is 604800, and zero should not be used because it may cause SERVFAIL errors.
https://bind9.readthedocs.io/en/latest/reference.html#namedconf-statement-max-ncache-ttl
I've personally modified it to 10,000 and it works well.
my recommendation is to leave in the forward only line and strip out the other two.  (barring an enhancement to let the user set the cache values)
How are you running the container(s)
docker compose, pulled latest.

### DNS Configuration

version: '2.1'
services:
dns:
image: lancachenet/lancache-dns:latest
env_file: .env
restart: unless-stopped
ports:
  - ${DNS_BIND_IP}:53:53/udp
  - ${DNS_BIND_IP}:53:53/tcp

and in .env
IP address on the host that the DNS server should bind to
DNS_BIND_IP=<IP_ADDRESS>
DNS Resolution for forwarded DNS lookups
UPSTREAM_DNS=<IP_ADDRESS>;<IP_ADDRESS>

### Output of container(s)

I have nothing.

yeah would be really nice to have it configurable via env variable. +1


Support for shaderSampledImageArrayDynamicIndexing physical device feature

The shaderSampledImageArrayDynamicIndexing physical device feature is not supported in MoltkenVK. This feature is supported on 98% of all Vulkan platforms (according to https://vulkan.gpuinfo.org/listfeatures.php) and makes binding many textures very convenient. Would it be possible for MoltenVK to support this feature as well?

This feature would really be appreciated, any plans for it?

We'd need to have SPIRV-Cross translate arrays of textures to metal::array objects. This is supported as of Metal 1.2. Unfortunately, this is somewhat complicated by the fact that arrays of samplers aren't supported until Metal 2.0. This is a problem primarily for combined image/sampler objects--with MSL 1.2, we'd be able to have an array of the image parts, but not the sampler parts.

Or maybe it will work for written images. The docs lied to me!

That sounds great, enabling shaderSampledImageArrayDynamicIndexing is enough for us at the moment.


ENH: Allow FE regression to drop absorbed

Add option that allows absorbed to be dropped
closes #205

Hello @bashtage! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file exp.py:

Line 10:14: W292 no newline at end of file


Decoration is Missing

There is no Option for using Input decoration  with out decoration it can not be customized as we want . It should accept Input Decoration like border border color focus color ....

d_input is package for simple style with default border input and concise
code. If you want full custom style, you can use TextField or TextFormField
Pada Min, 14 Jan 2024 01.46, Abdu A @.***> menulis:

There is no Option for using Input decoration with out decoration it can
not be customized as we want . It should accept Input Decoration like
border border color focus color ....
—
Reply to this email directly, view it on GitHub
https://github.com/indratrisnar/d_input/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTLYDYFM7OVWKWMNDKJIBDYOLJADAVCNFSM6AAAAABBZQAAJSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA4DANBTGE4DEMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>