Unable to Add Accessory

Hi,
I'm using a Generic ESP8266 to make accessories. I could pair accessories a couple of times after so many tries but almost always while pairing the accessories it take too long and in the end says Unable to Add Accessory.
I'm using ESP8266 library version 3.0.2 (Also tries 2.7.0), IDE 1.8.15 iOS 14.7.1 on iPhone 7 Plus and latest ESPHap library.
What should I do? Please help me with that
Does ESP32 have such this problem too?
Thanks

Esp32 is much better, but for the esp8266, since latest versions there are not any problem on our huge community, check if you are using latest version and attach logs
Sent from my iPhone

On 5 Sep 2021, at 18:33, alisadeghy @.***> wrote:

Hi,
I'm using a Generic ESP8266 to make accessories. I could pair accessories a couple of times after so many tries but almost always while pairing the accessories it take too long and in the end says Unable to Add Accessory.
I'm using ESP8266 library version 3.0.2 (Also tries 2.7.0), IDE 1.8.15 iOS 14.7.1 on iPhone 7 Plus and latest ESPHap library.
What should I do? Please help me with that
Does ESP32 have such this problem too?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

The problem is about the board, maybe it's shorted or something. see more #83


Querying a type with multiple variables ( variables based on Fields )

Good morning,
it is my first time to try GraphQL and i came to this repo, which i can find it's great !
I have a project that i am working on, and i need to query an Entity based on multiple parameters which are their properties.
for example, i have an Entity called  ( Booking ) , this Entity has multiple properties like
Booking
{
Id : int
ContactEmail : string
RoomsBookingDetails : (type RoomsBookingDetails )
{
number : int
.....
}
List Persons : (type Person)
{
id : int
name : string
}
}
i want to be able to query this object ( Booking ) / maybe also some sub objects, with multiple parameters
for example :
my query will be something like this
{
query {
booking(id:5, email: "" )
}
}
Sometimes, i want to query with only one parameters
This means that i want to pass dynamic parameters ( based on the Entity Fields )  to my query type,
and i want to resolve this in a single generic Resolver
this could be possible to do ?
i mean with a single resolver, i want to pass multiple params (those params are the Fields of the Entity )
i'm using Code-First approach with .NET Core
i have tried graphql.net,  but it seems like i should handle all the passed arguments with multiple resolvers

Note: i am using Elastic Search as a Data Source

So, you want to query an elastic search server?
What you can do if elastic search does not support IQueryable is to implement a query rewriter. The Filter API can be extended by implementing a query visitor.
Here is an example how we did it for queryable.
https://github.com/ChilliCream/hotchocolate/blob/master/src/Core/Types.Filters/QueryableFilterVisitor.cs

If you need help to implement it for nest head over to our slack channel.

That's fantastic, i really appreciate your great help.
Thank you very very much
Please let me know the Slack channel as the Invitation is no longer active
https://hotchocolategraphql.slack.com/join/shared_invite/enQtNTA4NjA0ODYwOTQ0LTBkZjNjZWIzMmNlZjQ5MDQyNDNjMmY3NzYzZjgyYTVmZDU2YjVmNDlhNjNlNTk2ZWRiYzIxMTkwYzA4ODA5Yzg

This invite link is no longer active.

OK, fixed the invite link.
https://join.slack.com/t/hotchocolategraphql/shared_invite/enQtNTA4NjA0ODYwOTQ0LTViMzA2MTM4OWYwYjIxYzViYmM0YmZhYjdiNzBjOTg2ZmU1YmMwNDZiYjUyZWZlMzNiMTk1OWUxNWZhMzQwY2Q

@AALGate  Can I close this issue?

Yes, thanks a lot @michaelstaib

Hello everyone, I also looking for a way to convert Queries to ElasticSearch Nest client.


Hello World in Ruby

This Pull Request resolves Issue #464
Description:
This PR prints 'Hello World!' text through a Ruby program.

@share-with-me Are there any updates or should we close this and mark it as spam?


lukasmonk.lucaschess version R2.03

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

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

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

Microsoft Reviewers: Open in CodeFlow

Looks like a download timeout

@wingetbot run


Performance Comparison

Hello. I thought about how productive this library is, especially in comparison with others. Please tell me what ways there are to measure the performance of react-native-svg, and for example compare with react-native fast-image?

I'm really curious about this as well, especially compared to React Native Skia. I've had 9-slice components where I had to switch from using react-native-svg to react-native-fast-image due to bad performance on Android.

I discovered that a small simple svg image used on an item in a listing of about 100 items was severely slowing down the scroll (mid-range android). After changing to fast-image and png format I noticeably improved the scroll. On iOS I did not notice the problem.


Add a single swift test to help the macOS CI to complete without error

The macOS CI is currently failing with what appears to be a bug in SwiftPM where it fails when there XCTests, but not Swift Tests. Add a trivial Swift Test to help the macOS CI to report as green when all of the XCTests pass.

@swift-ci test macOS

The experiment didn't solve the problem. Closing this.


[#447] Change the importer's language to be more explicit about the implication of its recommended dependencies

The text displayed throughout has been improved to provide more information to the common user of the module. This way, they are not urged to install 10+ modules that they may have no interest in.
Also changed the wording in the MuncherSettings.js to better describe that DAE is only required for highly automated games; it possesses no features (without other modules) that a core ActiveEffect cannot provide.

Whilst the DAE statement is true, the module won't generate a bunch of effects without it installed right now for legacy reasons that have yet to be addressed.

I have pushed a number of changes after removing some DAE requirements around effect generation.


Unauthorized when I am the reviewier for a work

I see the work here:

but when I click on it, I get the unauthorized message:

@vantuyls @mjgiarlo Is this how it's always worked? I can't find any evidence in the code that anyone has changed the search builder to allow people with workflow-roles to view the work.

Ah-ha! The code in question is https://github.com/projecthydra-labs/hyrax/blob/master/app/search_builders/hyrax/filter_suppressed_with_roles.rb#L27 and that query is returning an empty result for me.

I hadn't set myself up with the correct workflow role.


docs: SEC Sentiment Analysis Example

Summary
Adds a notebook with an example of how to use the SEC pipeline API and the unstructured core library to stage data for a labeling task and train a sentiment analysis model.
Testing

Install the requirements with pip install -r requirements.txt
Run the notebook

Also I think this is a great example, really showing an end-to-end use case. Most stuff you find when searching for information on these topics is not this in-depth or practical.

Could use an nbdev_clean --fname examples/sec-sentiment-analysis/sec-sentiment-analysis.ipynb


Support for Jakiper server rack batteries

Would be great to have support for Jakiper server rack batteries.
They have both the RS232 and RS485 ports.
https://jakiperbattery.com/2021/09/22/jk48v100-rs232-pc-monitoring-software/
The BMS is possibly using the PACE protocol?  Just judging by this thread, I have no inside info :)
https://diysolarforum.com/threads/jakiper-48v-sok-battery-bms-is-pace-brand-and-not-seplos-type.38015/

Also look at this info
https://diysolarforum.com/threads/monitoring-jakiper-maybe-sok-too-server-rack-batteries-via-modbus-to-esp8266-mqtt.45540/


SnackBar closeOnEscape is not working

SnackBar is supporting esc keydown action for closing.
But it looks not working.
I think cause of this problem is event listen target.
Now keydown event is attached to snakbar's div tag.
In order to work properly, we need to put an event on the window.

Maybe this isn't problem of react...

reopen in mdc-web


Proposal Creation Wizard: Show compensation in cycles & periods

Summary
As a DAO user I would like to see the time frame my compensation is for.
Description
Currently we only show the compensation in the proposal creation wizard (and proposal detail page) but don't say if this compensation is for 1 week (period), 1 month (cycle) or the whole duration of the assignment (e.g. 3 months) or one year
AC
Proposal Creation Wizard: Assignments "Step: Compensation"

[x] show default value for one cycle (ca. 1 month)
[x] Show toggle "Compensation for one period"
[x] Change copy to: "Salary compensation for one cycle ($70.000 USD / year)

Proposal Creation Wizard: Assignments "Step: Review"

[x] show default value for one cycle (ca. 1 month)
[x] Show toggle "Compensation for one period"

Proposal Detail Page: https://dao.hypha.earth/hypha/proposals/30633

[x] show default value for one cycle (ca. 1 month)
[x] Show toggle "Compensation for one period"

My own assignment:

[x] show default value for one cycle (ca. 1 month)
[x] Show toggle "Compensation for one period"

@jmgayosso @DaniPoot this looks great, just one small styling change please:
The headline "Compensation for one cycle" should be the same styling as "Duration" - "Commitment Level" etc... and all content should also be left aligned:

@jmgayosso can you add a little bit more spacing?


Create signature.json

We are waiting for ownCloud to provide a signed certificate owncloud/appstore-issues#94
Duplicate #57


[$500] Live Markdown conflicting with auto text replacement in mac

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.39-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @tgolen
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1707431230675759
Action Performed:

Open any report
Try to type the word "current" on mac Web

Expected Result:
The word "current" should be typed out
Actual Result:
The word "See yourrent" is the result. This is due to the default autocorrect which transforms cu into "see you". Normally, typing more characters wouldn't result in the autocorrect being used, but live markdown appears to have broken this so that the autocorrect is always applied.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/e37feeb8-dab8-4d5f-b4d0-a882d292f08c

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014026cc737dde97e6
Upwork Job ID:<PHONE_NUMBER>614876160
Last Price Increase: 2024-02-08

These are regressions from https://github.com/Expensify/App/pull/35837, keeping track of them in https://github.com/Expensify/App/issues/36071

@thienlnam , is this the same or should I file a new bug report?
https://github.com/Expensify/App/assets/22508304/7e4a7b7c-2ee7-4048-9cce-770d13a31481

That looks pretty related, I'm guessing it's the same root cause so we can keep track of it in this issue as well

Not a current bug, but being worked on as part of https://github.com/Expensify/App/issues/36071

Latest update: https://expensify.slack.com/archives/C06BDSWLDPB/p1709233583044319?thread_ts=1708724096.745109&cid=C06BDSWLDPB

Resolved in the repo


Add Zarr backend to write recordings and electrical series

Added option to write electrical series to Zarr.
This is just a proposal, but I'm happy to discuss.
This is briefly what I've added/changed:

unify compression parameters to compression_kwargs
add a backend parameter (hdf5 -default- or zarr) to use either the H5DataIO or the ZarrDataIO class.

TODO

[ ] Add and modify tests
[ ] Propagate to data interfaces

Accidentallyt included other changes, making a separate clean PR


windows-x64 and node-webkit >0.14.6

Hello,
first of all thanks for the work.
Everything works fine, but when I compile with node-webkit version greater than 0.14.6 then the application terminates immediatly after clicking the .exe.
The .exe for win 32 bit is working but not 64.
The process or programm is terminated without showing the window (so I could not debug easy)
Other systems I did not test.
Thanks for help.

You're very welcome :) I will take a look at this issue as soon as I get the chance. Thanks for reporting it!

Which version did you try exporting with specifically? I just tried 0.15.1 and 0.15.0 on Windows 7 x64 and Windows 10, and the compiled exe works for me. Is your code doing anything architecture specific with native modules or anything?

I tried on Windows 8.1.

Is your code doing anything architecture specific with native modules or anything?
I am not sure, I will try to exclude some modules and maybe find the problem in the code...

@jyapayne:
Can I give you (e-Mail?) you a deeplink to my source code?

Sure :) I can try to take a look at what's going on. My email is jyapayne at gmail.com

Hello  jyapayne,
thank you, I wrote you an E-Mail.

Thank you for help!
As I believe so far it is an nw.js issue as I can reproduce this with a blank project and download from nwjs?

Yeah, it's most likely an nw.js issue. Please file an issue with their repo :)


Packagist dependency version support

:clipboard: Description
A badge for Packagist to display the version constraint of a dependency. This should be similar/same to the already existing packagist/php-v badge but instead of showing hardcoded PHP version constraint it should return the version constrained for a dynamic dependency.
I'm not sure what the best URL structure would be as we would have to pass two package names to the shields.io service. First idea would be something like:
packagist/dependency-v/:user/:repo/:dependencyVendor/:dependencyRepo

:link: Data
The data is already available in the https://github.com/badges/shields/blob/master/services/packagist/packagist-php-version.service.js - but instead of packageVersion.require.php it should use something like packageVersion.require[vendor+'/'+name].
:microphone: Motivation
Some repositories want to showcase version constraints for important dependencies - for example the supported Symfony or Laravel versions.

I would like to work on this one

feel free :+1:


✨(frontend) create react app for standalone site

Purpose
Initialise web app for the standalone marsha site
Proposal

[ ] add a new yarn workspace
[ ] initialise a web app using CRA
[ ] rollback new CRA to react17
[ ] prevent hoisting on confliting modules

@kernicPanel do you know where can I find those assets ?

@kernicPanel do you know where can I find those assets ?

https://github.com/openfun/logos


Accept and handle AsyncIterators to facilitate streaming

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
When validating json using pydantic like so:
stream:  AsyncIterator[bytes]
model_iter: AsyncIterator[MyModel] = TypeAdapter(AsyncIterator[MyModel]).validate_json(stream)

I'd like Pydantic to return a AsyncGenerator.
Iterating over the returned generator should yield an instance of MyModel as soon as the stream provided at least enough data for a single item of the enclosing array.
This would potentially speed up processing responses received over a network, by allowing to process the data, while the event loop waits for the next chunk, for example with chunked encoding in HTTP
This has nothing to do with infinite generators, and would only apply to the data that's iterable at the top-level (since you couldn't return a model instance until you've read the entire document).
Analogously, when using dump_json()
data: AsyncIterator[MyModel]
stream: AsyncIterator[bytes] = TypeAdapter(AsyncIterator[MyModel]).dump_json(stream)

Conversely, handling AsyncIterator fields would be problematic, since Pydantic would need to consume the iterator field and leave the input object unusable.
EDIT: Actually models are often constructed for the sole purpose of sending them as JSON, so it might be useful.
PS. Probably should use new methods adump_json and avalidate_json instead.
Affected Components

[ ] Compatibility between releases
[X] Data validation/parsing
[X] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

While I do think lazier validation and serialization are in the works, I don't think we'll necessarily want to use this exact API.
I'll chat with the team about this proposal and get back to you after that.

Thanks!
I'm not attached to this particular API

This is a valuable and legitimate feature request, but it would require rewriting a lot of Pydantic, in particular pydantic-core.
The current story for interop between async Python and async Rust is quite poor. Things are improving but I expect it will take a couple years for things to settle properly, and even then we don't know if the interop will be good enough to make sense for this sort of implementation.
Because we won't be implementing this any time soon I'm going to close this as a wontfix 😞.


move oligo_U_tail and polyA_sequence [sf#156]

Reported by batchelorc on 2009-07-30 13:59 UTC
Just to check whether this is OK:
(1) polyA_sequence is_a mRNA_region
(2) oligo_U_tail is_a guide_RNA_region
best wishes,
Colin.

Commented by eilbeck on 2009-08-04 22:38 UTC
Hi Colin
Hi Colin
Right, these sequences are not really genomic seq unless they have somehow sneaked back as part of a pseudogene or something.
This is why I want to be able to have other topological and time relationships so we can describe the sequence of something that gets bits added to it as it matures. So I'm not sure where these two things belong at the moment. What were you thinking?
--K

Commented by batchelorc on 2009-08-06 10:06 UTC
OK, good, so is it the case that when there is an SO annotation of these, it really means a "fossilized" polyadenylylylyl sequence or oligouridine tail?
Equally, what is the story with "cap"?
Best wishes,
Colin.

Commented by eilbeck on 2010-03-12 18:27 UTC
Do you think we should have a parent to these terms called post translational region?

Commented by batchelorc on 2010-04-12 12:33 UTC
Resulting only from a post-translational modification?  Yes.
Colin.


@builder.io/sdk-vue runtime is not honored on Vercel

Environment
Nuxt 3.9.3, Vercel default (nodejs 18), nitropack 2.8.1
Reproduction
When building a minimal Nuxt3 project integrating the builder.io toolkit and deploying it on vercel, the output does not use the vercel-light edge export defined in the package's package.json.
See https://github.com/20x-dz/builder-vue-test for a simple project and https://builder-vue-test.vercel.app/ for the current output.
Describe the bug
After deploying to vercel, the package export for the edge-light environment is not used, but the default node environment is used instead, resulting in the following error: Cannot find module 'isolated-vm' Require stack: -<EMAIL_ADDRESS>As mentioned in https://github.com/BuilderIO/builder/issues/2933#issuecomment-1904494105 adjusting the import to use the edge export directly fixes this issue (import { Content, fetchOneEntry, isPreviewing } from "@builder.io/sdk-vue/edge";), but then breaks when developing locally.
Additional context
No response
Logs
No response

Hi. Please note that edge-light condition should be used for vercel_edge preset (not lambda which is default). For this, you might need to set NITRO_PRESET=vercel_edge environment variable or locally use build --preset vercel_edge

Thx @pi0 . I am using NITRO_PRESET=vercel because NITRO_PRESET=vercel_edge fails building with sentry (https://github.com/getsentry/profiling-node/issues/170). I just quoted edge-light because it was mentioned in the linked builder io PR.

These are two completely different runtimes:

vercel: Uses AWS Lambda and Node.js (resolving libraries with node condition)
vercel_edge: Uses a Worker to run your code in edge (resolving libraries with edge-light

One deployment/project can be against one of these. From your requirements, it seems you both need Node.js for sentry profiling and builder.io with edge-light conditions.
Also I notice that your reproduction the SDK is imported pages/[...app].vue (it is bundled by Vite and Nuxt not Nitro directly).
Have you tried to update your import to @builder.io/sdk-vue/edge?

Have you tried to update your import to @builder.io/sdk-vue/edge?

Yes, that works fine. On Vercel. Locally (running npm run dev, so node runtime I would guess?) this does not work.
From what I understand, using the import @builder.io/sdk-vue should resolve to the correct runtime automagically depending on the context given the exports defined in package.json, shouldn't it?

Yes it should work and i ever a library author, I would use main subpath export for universal builds (not a Node.js dependent variant).
Thanka for confirming that @builder.io/sdk-vue/edge works fine.
I think there is nothing much for us to do (vercel is a node preset and they never request any different condition that I am aware of. if there is one, both Nitro and libraries can use them. Best luck ;)

Thank you very much for your swift and detailed reponses. Very much appreciated! 🙏


Archlinux build script fix

script changes into x86_64. Afterwards it tries to copy the *.arch templates that are now under ../ not ../build-scripts.

Thanks!


Create a k6 load testing suite

Create a k6 load testing suite and use it against Chaz's Kubernetes cluster to see how the cluster behaves under load.
Have Grafana's node-exporter dashboard ready to show performance metrics during the test.
Try to assert a number of concurent users needed to make the cluster become unresponsive.
For reference, Chaz's k8s cluster runs in Oracle Cloud free tier, a 4 node cluster with 2x12Gb RAM on ARM and 2x1Gb RAM on x86.


Publish a codemod to migrate away from the deprecated "styled-components/native" imports

It'd be great to write a babel-codemod that quickly rewrites all imports and requires from styled-components/native to just styled-components;
Examples include:
// in
import /* ... */ from 'styled-components/native';
require('styled-components/native');

// out
import /* ... */ from 'styled-components';
require('styled-components');

Starting point would be the babel-codemod repo: https://github.com/square/babel-codemod
Here's a good example of how to write a babel-codemod plugin: https://github.com/kentcdodds/babel-codemod-example
And finally this is how we target imports/requires in our babel plugin:

https://github.com/styled-components/babel-plugin-styled-components/blob/master/src/index.js#L12-L14
https://github.com/styled-components/babel-plugin-styled-components/blob/master/src/visitors/assignStyledRequired.js#L4-L11

Hope this helps someone to get started with contributing to styled-components :heart:
The codemod should probably live in its own little repo that we can migrate to the styled-components org 💟

Is this still an issue? I have created a babel-codemod for this with the sole purpose of learning. Let me know if you'd like me to do anything with this.
https://github.com/bbuckland/styled-components-native-codemod

@bbuckland That's great! Want to add it to the RN section of the website?


CMS: Content overview sorting and filtering / search

Do this after pagination
Split into smaller sub-tasks

Laurens to add screenshots for inspiration from Polaris

Replaced with
#286
#287
#288
#289


Change import priority of React/RCTBridgeModule.h

Fixes redefinition of RCTMethodInfo in RN 0.48+

I need this, too.

More info here: https://github.com/facebook/react-native/issues/15775

Solid, thank you! Published with 0.7


fix(client): display non-JSON HTTP errors

This changes the new CLI to display the raw response body and status code for HTTP 50x errors if JSON decoding fails.

is there a reason for this ordering?
I'm following the convention of the JSON-formatted messages, which have the fields, then the status code. I thought it was preferable to be internally consistent, although I would have put the status code first in both cases...

Code LGTM.

Code LGTM

:+1: I found this annoying as well!


Not an issue but of exemple of implementation in real project!

Link to the module declarations
Module folder;
Store declaration
Store init
In component usage exemple
Usage in other files exemple
Hope it helps those who want to test this lib I really love it!

I was looking through your code and one thing confused me. In your Global Store you type the context of your actions as  async function fetchPaths(context: GlobalContext, svgPath: string) { but in other modules you don't type the context in actions. Why do you, in that call, call context.commit rather than as you do in the other calls, to Mutations.mutations.someCall?

@Gregoyle Hi! Thanks for looking into it. I struggled myself to put this in place so I understand that you have problems emulating it !
For the GlobalStore that was an old context commit I forgot to change, If you look into LoginStore for exemple you will see I always use Mutations.mutations.someCall. But i think you just need to put the parameter in place so the typings works.
That was a choice of myself to use namespaces like this and I don't know if it's good practice so if you find a better and proper solution don't forget to notify me! :)
Also if you have other questions on my code I'm here

@victorgarciaesgi
Hi Victor, I'm wondernig if you ever ran into this issue?
https://github.com/mrcrowl/vuex-typex/issues/11


Updating with two new activities for URL Links

@JohanFreelancer9 Copy editing is needed for this PR. Thanks

@JohanFreelancer9  done! Thanks

@markjjo PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@cebulnes PR merged. I re-ordered as activities in the table are listed in alphabetical order

@cebulnes FYI that this PR is live: https://docs.microsoft.com/en-us/microsoftteams/audit-log-events#teams-activities


Extend BuildSystem and CI to Ubuntu

Pattern rules and more Make improvements have been implemented in cb6be6bc9e2cebb04d859b169cf085971394abd7.
However, the full transparence/equivalence Win*/*nix is not yet available in the build system.

Fixed by 1289a4c647b307aa4cc886bfa7485f12bd70f2cf
Cc: @imahjoub and @Chalandi


[Additional Parameter Request] Codeunit 6620 Copy Document Mgt. - OnCopyPurchDocPurchLineOnAfterSetFilters

Hi Team,
In codeunit 6620 we need possibility to have additional checks in the procedure CopyPurchDocPurchLine. Standard publisher OnCopyPurchDocPurchLineOnAfterSetFilters would suit us if it had additional parameter var RecalculateLines: Boolean.

[IntegrationEvent(false, false)]
local procedure OnCopyPurchDocPurchLineOnAfterSetFilters(FromPurchHeader: Record "Purchase Header"; var FromPurchLine: Record "Purchase Line"; var ToPurchHeader: Record "Purchase Header"; var RecalculateLines: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


OIDC: state is too short minimum 8 characters expected

Description
Internal OIDC provider complains that state is too short.
Impact
OIDC not letting to authorize
Environment and steps to reproduce
usual setup
Expected behavior
OIDC provider should allow to login
Additional information
in headlamp.go file I added UUID after state as a temporary fix.
             oauthRequestMap[cluster] = &OauthConfig{Config: oauthConfig, Verifier: verifier, Ctx: ctx}
		state := uuid.New().String()
		cookie := http.Cookie{
			Name:    "state",
			Value:   state,
			Expires: time.Now().Add(time.Minute),
		}
		http.SetCookie(w, &cookie)
		uuidWithoutHyphens := strings.Replace(state, "-", "", -1)
		http.Redirect(w, r, oauthConfig.AuthCodeURL(cluster+":"+uuidWithoutHyphens), http.StatusFound)
	}).Queries("cluster", "{cluster}")

	r.HandleFunc("/oidc-callback", func(w http.ResponseWriter, r *http.Request) {
		mstate := r.URL.Query().Get("state")
		if mstate == "" {
			http.Error(w, "invalid state request ", http.StatusBadRequest)
			return
		}
		state := strings.Split(mstate, ":")[0]
		// nolint: nestif

The PR #447 was merged, so this should be fixed and in main and in the next Headlamp release.


No Unicode mapping for H11001 (3) in font COIHHG+MathematicalPi-One

several simple math operation symbols such as +, -, = with font COIHHG+MathematicalPi-One can't be mapped.

I worked out a solution:

make a file:  mathematicalpione.py that defines a dict for the code map and a function to do the mapping. put the file in the package dir
insert 2 lines of code in the encodingdb.py to import and call the function
at top, insert the line:
from .mathematicalpione import *
in the class function  get_encoding(klass, name, diff=None), between the line "elif isinstance(x, PSLiteral): " and the line "try", insert:
x = check_mathematicalpi_one(x)

the file:  mathematicalpione.py
---------------------------------------------:
from .psparser import PSLiteral
MathematicalPi_One_code = {'H11001':'plus','H11002':'minus', 'H9004':'Delta'}
def check_mathematicalpi_one(x):
""" function to transfer MathematicalPi_One_code to character name. """

if x.name in MathematicalPi_One_code:
    x = MathematicalPi_One_code[x.name]
    x = PSLiteral(x)
else:
    pass
return x


When ig job partially fails, KubeCF installation is left broken

Sometimes, when ig job fails, some of the instance groups get created but not all. It leaves KubeCF in a broken state that is hard to debug in production or demo systems, given that the failure is not clear.
It appears that the ig containers return non-zero exit code, but the cf-operator still wipes them out of the world, not leaving obvious traces of the failure. This often leads to messages like diego-cell is missing, it's a broken KubeCF.
Suggestion:

Don't delete the ig job in case of failure.
Make the creation of instance groups all or nothing. All of them get created, or none of them do.

@f0rmiga the ig job should stay around - do you have a repro scenario?
Your second suggestion might not be ideal - it would mean that we can't start things as configuration becomes available. Everything would start all at once.

@viovanov I could not reproduce this anymore. A change in the cf-operator might've fixed this. If it happens again I'll reopen with the repro steps.
The second suggestion indeed is not ideal.
Thanks for the links.


Increment Version

@JuliaRegistrator register() SequentialSamplingModels v0.1.1

Registration pull request created: JuliaRegistries/General/26317
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.1 -m "<description of version>" dd9440ca697e536730bbcf89cf96e01ba8f03ff0
git push origin v0.1.1


[Feature Idea] Text Substitutions

When the user types text that matches pre-defined expressions it will automatically get replaced. Some examples would be:

Typing * + space at the beginning of a paragraph will change it into a bullet list
1. + space converts into a numeric list
Two dashes will convert into an emdash
# + space converts into an h1
other markdown shortcuts such as > + space at the start of a paragraph for a block quote
auto-linking URLs

Hey,
sounds like #534 and #413...
We have commander branch with that feature, but not at the best shape.

Yes. I did search for existing stories that might cover this but couldn't find any. In my defense they are not named very well. 😄  I would say #413 is about a different issue with text substitutions only being mentioned. But this is a duplicate of #534 for sure.

Agree that our issues are not named well ;)
We named this feature set keyboard macros, being part of beta4.


Public vs. shared channels

Statement: "Only public channels are supported." under Notes.
Can you please clarify this statement? According to several of my test's, shared channels are not supported. Shouldn't it say "standard channels"?

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

ID: b755d9ae-13ac-019f-f5f4-a95b1eee2989
Version Independent ID: f7afd8ad-e6e0-6c79-aed1-4ee87d48ba12
Content: Create a call queue in Microsoft Teams - Microsoft Teams
Content Source: Teams/create-a-phone-system-call-queue.md
Service: msteams
GitHub Login: @DaniEASmith
Microsoft Alias: danismith

Thanks for your feedback!


Move the "zoom to this" button to the map controls

Re: #6601
Opening a PR for this in order to gather feedback. It's nice to have all the zoom controls together and the design is neater than the text button. But does it make sense to have a selection-aware control separate from the inspector?

Looks good 👍

@quincylvania I just noticed the "zoom to" button/feature on a relation member and I wonder, if that is a duplicate of this feature – just for relation members. If so, maybe it could be removed in favour of this newly positioned button?
Map: https://preview.ideditor.com/master/#background=Berlin-2018&disable_features=boundaries&id=r933526&locale=en&map=20.00/52.47423/13.44983

@tordans Nice thought, but I think the relation member button is still useful since you can use it without changing the selection. I often want to focus on a single member of a big relation, say an island in a lake, where the extent of the full relation is significantly different than the extent of the one member.


plugin flak8-html missing from the docs

The docs should mention that flake8-html is required for genbadge to run.

Thanks @sandeep-gh  !
For which step exactly is it required ? Do you get an error message when you run
flake8 <src_folder> --exit-zero --statistics --tee --output-file flake8stats.txt
?
As mentioned in https://smarie.github.io/python-genbadge/#prerequisite-a-flake8-report, the html part is optional.
Maybe the text of the documentation could be improved ? If so, which improvement would you suggest ? Thanks in advance !

@sandeep-gh : should I consider this issue closed ?

closing for now, feel free to reopen !


crash when hovering to display tooltips

Crash report:
http://paste.ee/p/vRDqR#
version: 3.0.51
steps to reproduce: hover cursor over crafting tweaks buttons in a crafting table

Weird! I used an incorrect format in the access transformer, but it worked fine in my dev environment, so I didn't notice. A fix is up on CurseForge now.


Import Ballots: Analyze Data from csv

Go here: http://localhost:8001/import_export_batches/batch_list/?kind_of_batch=IMPORT_BALLOT_ITEM
Upload a Ballot file

On the left side of the "Import Ballot at One Location" screen, there is an "Analyze" link that takes you to "batch_action_list_analyze_process_view" in WeVoteServer/import_export_batches/views_admin.py
That view calls "create_batch_row_actions" in WeVoteServer/import_export_batches/controllers.py which needs code to be added for kind_of_batch=IMPORT_BALLOT_ITEM
I started a function called "create_batch_row_action_ballot_item" that you can finish.
You will also need to add IMPORT_BALLOT_ITEM-specific code to WeVoteServer/templates/import_export_batches/batch_action_list.html

This is done.


table2py implementation

Probably the current implementation of table2py is totally out of date taking into account the several new features added on PySD.
I was thinking about adding some new tables to complete the test and have a better coverage. However, Vensim doesn't allow me to write or read table models. Maybe this type of model was already deprecated by Vensim. Could we then remove the table2py feature from PySD?
@JamesPHoughton

Yes, let's pull it out - vensim never made models in a table format, it was an experiment for interacting with another platform that I've forgotten all about. I don't think anyone is using it, but it might make sense to make a "final release to support table2py" and make a note in the documentation saying that if anyone needs it, that's the version to use. Same as we did when we moved to py3.7. Ok if I let you make those changes?


[BUG] Back to page animation does not work in XF <IP_ADDRESS>6.

Hi everyone, today I upgraded XF to version <IP_ADDRESS>6 and unfortunately the site animation does not work as before. Next page animation works well but back animation doesn't. I used to use XF version <IP_ADDRESS>2 and everything worked fine. I enclose a few screenshots. Thanks a lot.
Bug Information
Version Number of Plugin: 2.4.0
Device Tested On: Realme GT Android 11, OnePlus 9 Pro Android 11. Tested on other devices with the same effect
Simulator Tested On: Pixel3_XL_API_30
Version of VS: 16.11.5
Version of Xamarin: <IP_ADDRESS>6
Versions of other things you are using: Xamarin.Essentials 1.7.0
Steps to reproduce the Behavior
Just create a new Xamarin Forms project and upgrade XF to the latest build <IP_ADDRESS>6. That is all. You don't have to add anything else.
He tried it in XF.Shell and in a blank window. Same mistake.
Expected Behavior
The animation back to the page should work as in the previous version of XF <IP_ADDRESS>2. see video XF_<IP_ADDRESS>2.
Actual Behavior
The back to page animation does not work. see video XF_<IP_ADDRESS>6.
Code snippet
in XAML
NavigationPage.HasNavigationBar="False"  NavigationPage.HasBackButton="False" sharedTransitions:SharedTransitionNavigationPage.TransitionDuration="300" sharedTransitions:SharedTransitionNavigationPage.BackgroundAnimation="SlideFromRight"
in App.cs
MainPage = new SharedTransitionNavigationPage(new Page1());
Navigate to the page
 App.Current.MainPage.Navigation.PushAsync(new Page2());
Navigate back to the page
App.Current.MainPage.Navigation.PopAsync();
Screenshots
https://user-images.githubusercontent.com/5471789/141354257-932f97a3-a8e5-468d-baea-2e34f6e59cb7.mp4
https://user-images.githubusercontent.com/5471789/141354325-9c92df2c-f23a-4672-a214-4a2fdf25a619.mp4

Good luck the author of this project hasn’t responded in like over a year. I have similar problem and I’ve received little to no help. I wish I could help you but unfortunately with out the help of the creator it becomes difficult as to I am still trying to fix my problem.

Yes, it seems that this project will be unusable in the future. Therefore, I applied for the inclusion of this project in the XamarinCommunityToolkit. It would be great to have this option with a box with additional features in one XCT project.

I have not have time to check this fully out, but i noticed running the sample AndroidX app and debug the following method:
public static void RemoveTransition(View view, Page currentPage)...
currentPage.Parent = null <-- Which again will not trigger removing the transitionmap.
Not sure if thats the case why we get a whitescreen on navigating back.
currentPage.Parent should probably be the SharedTransitionNavigationPage reference?

Updates to AndroidX.Fragment have almost completely broken this plugin on Android.

Unfortunately yes, this was such a good lib... there are also no alternatives :(

Unfortunately the latest versions of xf modified a lot of things in navigations. for example the NavigationaPage (and i suspect Shell too)) in Android is completely broken for this plugin. The problem is that the fragments generation in the destination page now happens too late to attach a shared transition inside the available overrides provided by XF, there is no fix for that oter than rewrite all the platform navigation logic (not going to happen).
Also something is changed in the iOS navigation that broke the back button.
I spent a few hours trying to fix those issue but the Android one hit me hard, is unfixable :(
So i cannot be able to support this for XForms 5 anymore (the first version of XF5 should still work) and i'm going full in with MAUI with a full rewrite.
MAUI handlers are way more flexibile and expandable than XF renderers.
I think that this plugin was too "hacky" for XF and too suscetible at every minor changes made by the XF team in the navigation logic, incapsulated in renderers with very few entry point available to extend.
I have some nice idea for MAUI, i want to be able to make shared transition on grouped view more easily and semplify by A LOT all the source code, expecially the iOS one (where there are not OS support for shared transitions and i had to do some crazy things to emulate them).

erere


Need to escape the \n when there are multiple public keys so that the groovy script sees the literal \n, not the newline.

I did a 'bundle exec rake' on the master before making my change but it failed trying to converge smoke-package-centos-64.
Without the change, my wrapper cookbook throws a groovy syntax error in jenkins_user if there are two or more public keys being added because of the newline between the multiple keys. When joined with \n the groovy script sees only one line & correctly inserts the multiple keys into the user object.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@jcejohnson

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

Thx! This was merged as 135fd6a693b865ab66cff4df8eec2104e74fbba0 and integration test coverage was added in 3d4aa6f57eb49a675d24875c9c90ae2fbe8d76ca


[Orbit] prev and next not visible on white by default

the buttons vor next and prev are not visible by default until you hover over the button
maybe hover on orbit should show the buttons

Shouldn't that be on the designer to change the color of the buttons themselves? @gakimball @zurbrandon

What @zurbchris said, it's hard for us to account for varying use-cases, so in this case we defer to the designer to account for what content is in the carousel.
You can do the hover reveal with just CSS, by targeting the :hover state of the container, and then applying styles to the controls inside.
.orbit:hover .orbit-next,
.orbit:hover .orbit-previous {
  // Put a more explicit hover state in here
  background-color: rgba(0, 0, 0, 0.75);
}


Demo: Calendar >>>  ImportError: can't perform relative import

17:  from .date import DateCal

%Run -c $EDITOR_CONTENT

MPY: soft reboot
Traceback (most recent call last):
File "", line 17, in
ImportError: can't perform relative import

I can't replicate this.
You need to run the demos from the touch GUI root, so you run the demo with
>>> import gui.demos.calendar

You're right, everything worked out, thank you


Easy way to skip fields of an object

Say I have an object like this:
  Player = ref object
    username: string
    ready: bool
    conn: WebSocket
    curLevel: JsonNode
    invadingUsername: string
    heistSuccess: bool
    id: uint64

that I want to dump to json, but don't want the conn to be in the dumped json. What if jsony had some kind of skipHook you could define to describe what fields to skip serializing?

Beef on the discord advocates for a skip pragma:
Personally i'd go for a template jsonySkip*{.pragma.} so you can just do conn {.jsonySkip.}: WebSocket
I'd also say a template jsonyName*(name: string){.pragma.} would be nice
So you can give custom names
Without a user defined hook


[stable/datadog] Fix several aspect of KSM integration

What this PR does / why we need it:
this PR include several improvements around the stable/kube-state-metrics chart integration as stable/datadog chat's dependency.
Also fix another typo in Readme.md

cherry-pick: #15527 that fix  #15526
update kube-state-metrics chart version to 1.2.0 corresponding to the v1.5.0
fix kube-state-metrics values overwrite in the datadog values.yamlandReadme.md`

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @CharlyF

/ok-to-test

/lgtm


Cannot build/run/emulate on iOS

If I try to run the build for an Android platform, there are no issues and all functionality is present. However, if I try to run on any iOS device I receive the following error:
john-thornewills-imac:nextra-test johnt$ tns emulate ios
Successfully prepared plugin nativescript-insomnia for ios.
Successfully prepared plugin nativescript-sqlite for ios.
Processing node_modules failed. Error: Command failed: /bin/sh -c ruby -e "require 'xcodeproj'; Xcodeproj::Config.new('/Users/johnt/Dropbox/NativeScript/nextra-test/platforms/ios/plugins-debug.xcconfig').merge(Xcodeproj::Config.new('/Users/johnt/Dropbox/NativeScript/nextra-test/node_modules/nativescript-sqlite/platforms/ios/build.xcconfig')).save_as(Pathname.new('/Users/johnt/Dropbox/NativeScript/nextra-test/platforms/ios/plugins-debug.xcconfig'))"
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in require': cannot load such file -- xcodeproj (LoadError) 	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in require'
from -e:1:in `'
Any ideas folks?

I found the issue. I didn't have the correct gem installed.
You can install is using sudo gem install xcodeproj

Hmm, that is a new one.  I've installed this many times on my box and never seen that happen.  I'm glad you have found the solution!


Overriding URLs for binary downloads

We're trying to install jsbin, which depends on<EMAIL_ADDRESS>Unfortunately, we're firewalled and must use internal package hosting alternatives. We also can't build from source, because of a lack of tooling on the relevant boxes. What we can do is host the binaries internally, but we must have a way then to configure the download URLs. I can't seem to find any documentation related to this -- is it possible?

Hi @mstade, totally understand your situation however to the best of my knowledge there is no configuration option because these details are provided via the project's package.json.
I believe the correct place to request this feature would be on the node-pre-gyp repository, which is the module that handles the binaries.
The best workaround I can think of would be to fork this repository, and set the package.json details in your fork to your own aws mirror.
Alternatively, given your situation, perhaps including your dependencies in your deployment would work?

Thanks for the quick response @ErisDS! I just had a look at node-pre-gyp docs, and it looks like maybe this is an option: https://www.npmjs.com/package/node-pre-gyp#download-binary-files-from-a-mirror
The "because China" comment is tragically comical. I'm going to try this and report back any progress, thanks for the pointer!


Add generic domains for integration tests

What this PR does / why we need it:
This PR adds a util function to assign a generic default domain to shoots in the integration test framework.
Currently this domain is hardcoded to integrationTestName + ".shoot.dev.k8s-hana.ondemand.com".
Special notes for your reviewer:
@zanetworker I implemented the functionality as a util function in the framework.
Release note:

NONE

Why do we need this at all? Can't we just remove .spec.dns.domain and just specify .spec.dns.provider=aws-route53? When the shoot will be created Gardener will compute a proper domain for it.

@zanetworker If this also fits for your tests then this solution would be more reliable.

Updated dnsDomain and tested the shoot operations test suite


Code fixes often don't work (codeAction/resolve failed)

Type: Bug
Issue Description
I am running VSCode on Linux within a devcontainer.
The suggested C# code fixes often cannot be applied. It often states "Request codeAction/resolve failed.".
I currently have no idea when and why. If I restart VSCode after the issue appeared, the exact same code-fix works as expected.
(This issue template recommends enabling "dotnet.server.trace", so I will do so and update this issue once there is more information.)
Steps to Reproduce
In my current case:

Open a .cs class file within a subdirectory of a project.
Ensure the folder structure does not match the namespace of that class (which, in my case, is by default when creating the class using this C# extension within the Solution Explorer).
Try to apply the code fix "Change namespace to '{0}'".

Expected Behavior
The namespace is changed according to the folder structure and probably the project settings.
Actual Behavior
An error pops up "Request codeAction/resolve failed."
The C# log outputs "The solution does not contain the specified document." - which is not true.
Logs

C# log

[Error - 11:20:47 AM] [LanguageServerHost] System.InvalidOperationException: The solution does not contain the specified document.
   at Microsoft.CodeAnalysis.Shared.Extensions.ISolutionExtensions.GetRequiredDocument(Solution solution, DocumentId documentId) in /_/src/Workspaces/SharedUtilitiesAndExtensions/Workspace/Core/Extensions/ISolutionExtensions.cs:line 48
   at Microsoft.CodeAnalysis.ChangeNamespace.AbstractChangeNamespaceService`3.<>c__DisplayClass26_0.b__4(IGrouping`2 refInOneDocument) in /_/src/Features/Core/Portable/CodeRefactorings/SyncNamespace/AbstractChangeNamespaceService.cs:line 477
   at System.Linq.Enumerable.SelectEnumerableIterator`2.MoveNext()
   at System.Threading.Tasks.Task.WhenAll[TResult](IEnumerable`1 tasks)
   at Microsoft.CodeAnalysis.ChangeNamespace.AbstractChangeNamespaceService`3.ChangeNamespaceInSingleDocumentAsync(Solution solution, DocumentId id, String oldNamespace, String newNamespace, CodeCleanupOptionsProvider fallbackOptions, CancellationToken cancellationToken) in /_/src/Features/Core/Portable/CodeRefactorings/SyncNamespace/AbstractChangeNamespaceService.cs:line 475
   at Microsoft.CodeAnalysis.ChangeNamespace.AbstractChangeNamespaceService`3.ChangeNamespaceAsync(Document document, SyntaxNode container, String targetNamespace, CodeCleanupOptionsProvider fallbackOptions, CancellationToken cancellationToken) in /_/src/Features/Core/Portable/CodeRefactorings/SyncNamespace/AbstractChangeNamespaceService.cs:line 225
   at Microsoft.CodeAnalysis.CodeActions.CodeAction.ComputeOperationsAsync(IProgress`1 progress, CancellationToken cancellationToken) in /_/src/Workspaces/Core/Portable/CodeActions/CodeAction.cs:line 285
   at Microsoft.CodeAnalysis.CodeActions.CodeAction.GetOperationsCoreAsync(Solution originalSolution, IProgress`1 progress, CancellationToken cancellationToken) in /_/src/Workspaces/Core/Portable/CodeActions/CodeAction.cs:line 222
   at Microsoft.CodeAnalysis.LanguageServer.Handler.CodeActionResolveHandler.HandleRequestAsync(CodeAction codeAction, RequestContext context, CancellationToken cancellationToken) in /_/src/Features/LanguageServer/Protocol/Handler/CodeActions/CodeActionResolveHandler.cs:line 94
   at Microsoft.CommonLanguageServerProtocol.Framework.QueueItem`3.StartRequestAsync(TRequestContext context, CancellationToken cancellationToken)
[Error - 11:20:47 AM] Request codeAction/resolve failed.
  Message: The solution does not contain the specified document.
  Code: -32000
[object Object]

C# LSP Trace Logs
Post the output from Output-->C# LSP Trace Logs here.  Requires `dotnet.server.trace` to be set to `Trace`
Environment information
VSCode version: 1.84.2
C# Extension: 2.10.28
Using OmniSharp: false
Dotnet Information
.NET SDK:
 Version:           8.0.100
 Commit:            57efcf1350
 Workload version:  8.0.100-manifests.6c33ef20
Runtime Environment:
OS Name:     debian
OS Version:  12
OS Platform: Linux
RID:         linux-x64
Base Path:   /usr/share/dotnet/sdk/8.0.100/
.NET workloads installed:
Workload version: 8.0.100-manifests.6c33ef20
There are no installed workloads to display.
Host:
Version:      8.0.0
Architecture: x64
Commit:       5535e31a71
.NET SDKs installed:
8.0.100 [/usr/share/dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 8.0.0 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 8.0.0 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
Other architectures found:
None
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Visual Studio Code Extensions

Extension
Author
Version
Folder Name

angular-console
nrwl
18.10.1
nrwl.angular-console-18.10.1

code-spell-checker
streetsidesoftware
3.0.1
streetsidesoftware.code-spell-checker-3.0.1

copilot
GitHub
1.139.0
github.copilot-1.139.0

copilot-chat
GitHub
0.10.2
github.copilot-chat-0.10.2

csdevkit
ms-dotnettools
1.0.14
ms-dotnettools.csdevkit-1.0.14-linux-x64

csharp
ms-dotnettools
2.10.28
ms-dotnettools.csharp-2.10.28-linux-x64

ilspy-vscode
icsharpcode
0.17.0
icsharpcode.ilspy-vscode-0.17.0

markdown-all-in-one
yzhang
3.5.1
yzhang.markdown-all-in-one-3.5.1

nugetpackagemanagergui
aliasadidev
2.0.6
aliasadidev.nugetpackagemanagergui-2.0.6

prettier-vscode
esbenp
10.1.0
esbenp.prettier-vscode-10.1.0

vscode-docker
ms-azuretools
1.28.0
ms-azuretools.vscode-docker-1.28.0

vscode-dotnet-runtime
ms-dotnettools
2.0.0
ms-dotnettools.vscode-dotnet-runtime-2.0.0

vscode-eslint
dbaeumer
2.4.2
dbaeumer.vscode-eslint-2.4.2

vscode-nuget-gallery
patcx
0.0.24
patcx.vscode-nuget-gallery-0.0.24

vscodeintellicode-csharp
ms-dotnettools
0.1.26
ms-dotnettools.vscodeintellicode-csharp-0.1.26-linux-x64

Extension version: 2.12.19
VS Code version: Code 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:50:47.800Z)
OS version: Linux x64 6.2.0-37-generic snap
Modes:
Remote OS version: Linux x64 6.2.0-37-generic

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz (16 x 3820)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
62.52GB (48.24GB free)

Process Argv
--no-sandbox --force-user-env --unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
wayland

Item
Value

Remote
Dev Container: C# (.NET)

OS
Linux x64 6.2.0-37-generic

CPUs
11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz (16 x 3564)

Memory (System)
62.52GB (48.24GB free)

VM
0%

This might already be fixed with https://github.com/dotnet/roslyn/pull/70669; not sure if @Cosifne might want to look at this.
@AlexanderInova do you have a source generator running in this project? If you upgrade to 2.12.19 (which just released yesterday) is it fixed?

I configured "dotnet.server.trace" and then restarted VS Code, which seemed to automatically update the extension (at least it is now on 2.12.19). The last few ours the issue didn't appear. If the issue doesn't appear tomorrow, I'll let you know here and close the issue assuming you were right. Thank you so far.

The issue didn't appear again since the new update was installed. This might indicate that you were right and it is probably already fixed with https://github.com/dotnet/roslyn/pull/70669.
So it was fixed even before I actually reported it - that's quite fast. :)
Thank you very much!

So it was fixed even before I actually reported it - that's quite fast. :)

Our time machine is still in the shop, so in this case you just got lucky and we had other reports too. :smile:


🚧 Refactor initialize spotify

Description
The main purpose of this pull request is to enhance the process of initializing the Spotify module and auto-logging in the current user if they have a profile.
Test Plan
N/A
Motivation and Context
The general functionality of the app needs to be improved to be leaner and more performant overall. By refactoring not only the InitializeSpotify async thunk, but as well as the containers where the thunk is used, the process of initializing the module and getting the user logged in automatically can be improved upon.
How Has This Been Tested?
I've manually tested the performance of the app by going through the containers in question and seeing how the new implementation is performing.
Types of Changes

Documentation
Bug fix
[x] New feature
Breaking change

Checklist

[x] My code follows the code style of this project

 Initialize spotify thunk


Allow the ability to programmatically specify a proxy when using the library in node.

This implements the ability to specify a proxy programmatically when using jsforce on the server as a node module. This pull request piggybacks on the proxyUrl parameter and passes it through to the request module's proxy parameter. This also provides a fix for jsforce/jsforce#1.
The issue for us with using the HTTP_PROXY environment variable is that then all outgoing requests (at least using the request library) would be forced to use the proxy. This is because the request library also respects the HTTP_PROXY environment variable (https://github.com/request/request#controlling-proxy-behaviour-using-environment-variables). In our use case, we want to force all jsforce requests to go through a proxy but we don't necessarily want all other requests in that same node process to use the proxy.
Usage (as in jsforce/jsforce#1):
var jsforce = require('jsforce');
var conn = new jsforce.Connection({
    proxy: 'http://proxy_url:port'
});

Thanks for the feedback @stomita! Would you be more inclined to accept this if we create a new parameter proxy rather than overloading the proxyUrl parameter? If so I can make that change and you can review the updated pull request.

@aotimme I prefer "httpProxy" rather than "proxy".

Thanks @stomita. Are you okay with the updated implementation otherwise? Specifically, are you okay with the new ProxyServerTransport class? I can call it HttpProxyTransport to be more in-line with using the term "httpProxy" if that sounds better.

I just changed it to use the parameter "httpProxy" so the usage looks like:
var jsforce = require('jsforce');
var conn = new jsforce.Connection({
    httpProxy: 'http://proxy_url:port'
});

Let me know if there are any other changes you'd like to see.

Thanks @stomita! Do you know when you're likely to issue a new release of this module (to npm) to include this? No worries if you don't have an estimate at this time.

:+1: for releasing an update to npm


Refactor widgets towards factories

IWidgetType -> IWidgetFactory (think about proper name)
A factory should be responsible for:

[x] Instantiate of new widget
[ ] Installation of required widget class
[x] Qualification of a dashboard for ability to display this particular widget
[x] Population of "default" dashboards

To have one class much more convenient than have to: widget + factory.
So it was decided to stick to current implementation, but additional thing like "selector" was implemented.
Installation of widget class will be done when #35 will be implemented. So it's out of scope of this issue.


don't generate unused noise

Fixes #1888

Please check if anything around this needs to be clarified in the spec, @jonsneyers

The spec is OK here, this was a libjxl bug.
Note that the decoded images for conformance bitstreams using noise will need to be regenerated since the current ones will be wrong because of this bug.


Help to write correctly rule

A task
It is necessary for a particular URI. For a certain rule ID remove arg chek out.
Create a rule:
SecRule REQUEST_URI "@beginsWith  /wanted/vacancy/anketa/" "phase:1,t:none,pass,
nolog,ctl:ruleRemoveTargetById=973333;ARGS:work[company][]
Does not work.
ModSecurity: No action id present within the rule Wrapped exception
Written as a rule:
SecRule REQUEST_URI "@beginsWith  /wanted/vacancy/anketa/" "phase:2,chain,t:none,id:902212,nolog"
SecRuleUpdateTargetById 973333 !ARGS:work[company][]
Does not work/
ModSecurity: Execution phases can only be specified by chain starter rules
How to write a rule right?
Thank you

Hey @shrv, Rules have some requirements. All 'starting' rules must contain an ID. any rule that features the 'chain' action means that it expects another rule (without an id) to follow. If you have other questions please feel free to comment back

Thank you, rule with a "chain" i understood.
But that is not true in the first rule?
With CTL?

The first rule has no id. rules must contain an ID

Thank you


Unable to flick clock dial on Time Picker

I try to flick clock dial on Time Picker but I am unable to do so

I resolved this issue and created a pull request #41

#41  merged so the issues can be closed


Custom Decoder For Autoencoder methods

Can you add-on the ability for all AE models (maybe just update the AE class) to support user-supplied decoders (which can be problematic for decode_indices)? I'm getting poor results right now just trying to overfit to my original graph (I want my distribution of predictioned values for edges to separate so I can set a threshold to declare 0 or 1), and tried to devise this model, however, decode_indices does not work for reconstruction loss:
class VGAE2(VGAE):
    def __init__(self, encoder, p=2):
        super(VGAE2,self).__init__(encoder)
        self.latent_transform = nn.Linear(self.encoder.n_latent,self.encoder.n_latent)
        self.distance_operator = nn.PairwiseDistance(p=p)

    def decode(self, z, sigmoid=True, pdist=True):
        if pdist:
            print(z)
            adj = torch.exp(-self.distance_operator(z,z))
            print(adj)
        else:
            adj = torch.matmul(z, self.latent_transform(z).t())
        return torch.sigmoid(adj) if sigmoid else adj

Hi, I am not sure if I understand your problem correctly. For using your own decoder, you need to override both decode and decode_indices for dense and sparse decodings respectively. I do not see a reason that prevents your decoder from implementing decode_indices.

Sure. Why hack it through class inheritance? Wouldn't it be better for the user to input an instance of some decoder class that could serve as one of the input arguments to the GAE class, like the encoders and discriminators? Perhaps if there were a set of decoder classes from pytorch geometric that the user could select from?

This is a reasonable request. Working on it!

You can now pass a user-defined decoder module to the Autoencoder models. I'm closing this for now. Feel free to reopen if you encounter any issues.


Calendar is not showing iterations

I haven't found any settings for enable iterations in calendar

You need to check the boxes next to the iterations to select them.

If you do, they will appear in the calendar like so:

Yes, I've found it. Thank you.


Panic on request

I tried the simple-api demo as described. The api-key is turned of.
local espress = require 'espress'
local port = 80
local server = espress.createserver(port)
server:use("routes_auto.lc")
The nodemcu crashes with this message:
Request received: 	GET	/api/hello
routes/hello.get.lc
PANIC: unprotected error in call to Lua API (?:0: attempt to call a nil value)
Tested firmware is the newest version from http://nodemcu-build.com/ master build with modules: bit, crypto, file, gpio, net, node, tmr, uart, wifi

Hi @saeugetier , let me take a look on this issue right away.

If you want the hello to work, you need to upload it to your nodemcu as well (samples are not included in the upload script)

[Edit]
My bad. I misread your post. Seems there is a bug indeed.

I think the bug is in file askme.post.lua
 if req.params.question then
should be
if req.params["question"] not nil then
I seems that the code crashes if route is nil

Both should work from my previous tests.
However, I think you are right for the route. If route does not exist, it will crash.
I will implement a safe behaviour tonight.
Can you work with that for now?

Dear @saeugetier ,
I just released a new 0.3.0, with your fixes along with a lot of new improvements:
Changes:

Fixed body parsing bug
Fixed concurrent file reading bug
Added not_found as default behaviour
Added res:sendredirect method
Implemented global error handler
Implemented forms parsing
Added simple-website-with-forms sample
Added api with cjson sample
Added 500, 302 status codes to server default uploads
Updated docs and upload scripts

Thanks for helping with your issue, I hope you will like the new version.
N.B: There is a small API change when starting a server
Instead of
 espress:createserver(port)

 espress:use("plugin1.lc")
 -- ...

Write the following
 -- The server creation becomes lazy
 espress:createserver()

 espress:use("plugin1.lc")
 -- ...
 espress:listen(port)

Thank you for the fast fix. I will try this version and report back.


Webpack breaks on cached run

versions:

hard-source-webpack-plugin v0.3.4
webpack 2.2.0-rc.0 as well as 2.1.0-beta.25

When I run the project the first time, everything is fine, but on subsequent runs I get this crash until I clear the hard-source cache or disable the plugin:
/pathToProject/node_modules/webpack/lib/dependencies/HarmonyImportSpecifierDependency.js:31
	var importedModule = this.importDependency.module;
	                                          ^

TypeError: Cannot read property 'module' of undefined
    at HardHarmonyImportSpecifierDependency.HarmonyImportSpecifierDependency.getWarnings (/pathToProject/node_modules/webpack/lib/dependencies/HarmonyImportSpecifierDependency.js:31:44)
    at /pathToProject/node_modules/webpack/lib/Compilation.js:643:21
    at Array.forEach (native)
    at /pathToProject/node_modules/webpack/lib/Compilation.js:642:22
    at Array.forEach (native)
    at Compilation.reportDependencyWarnings (/pathToProject/node_modules/webpack/lib/Compilation.js:641:9)
    at Compilation.<anonymous> (/pathToProject/node_modules/webpack/lib/Compilation.js:507:8)
    at Array.forEach (native)
    at Compilation.finish (/pathToProject/node_modules/webpack/lib/Compilation.js:506:15)
    at /pathToProject/node_modules/webpack/lib/Compiler.js:471:16
    at /pathToProject/node_modules/tapable/lib/Tapable.js:189:11
    at Compilation.<anonymous> (/pathToProject/node_modules/webpack/lib/Compilation.js:454:10)
    at /pathToProject/node_modules/webpack/lib/Compilation.js:425:12
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Note that I run webpack with babel-register for the config files, not sure if that is related.

Hey @wmertens. I'm guessing you are doing something out of order. Are you using something imported before its imported? We had another bug like this related to exporting things before they were imported.
Do you have anything like
doSomething();
// ...
import {doSomething} from './some-file';

Just want to confirm so my regression test to go along with the fix keeps this from popping up again later.

No before-import use, eslint would have told me (I just double-checked).
I do use System.import(...).then(...), and there may be circular (but
finite) dependencies (not sure). Would any of those cause problems? Maybe
webpack itself?

I added a console.log(this) and this error happens for this this:
HardHarmonyImportSpecifierDependency {
  module: null,
  importDependency: undefined,
  importedVar: null,
  id: 'default',
  name: 'Promise',
  range: null,
  loc:
   { start: { line: 5, column: 102 },
     end: { line: 5, column: 109 } } }

I changed it to log for this.name === 'Promise' and it turns out that the first run, it has Bluebird for Promise, and the second time, it shows the above output. So maybe something is using Promise before I polyfill it with Bluebird?

All I needed to do was rename my bluebird import from Promise to something else and it stopped happening. Weird, because I never assigned to the Promise global, it seems that global name clashes are not appreciated?

Oh wow. I think the Promise detail is exactly the issue. The good news is my 0.3.5 change in #80 works for this too. I'm guessing its a detail of the acorn parser webpack uses hoisting global names or some other similar mechanism effectively causing the specifier to appear before the import in the dependencies of that module.
I'm not worried about System.import being the cause. Its basically a fancy require.ensure dependency that doesn't have to deal with the more complex situations harmony imports and exports deal with.
Thanks for helping me figure this out @wmertens!

🎉 can confirm it is fixed now!
Imports are hairy - I'm battling an issue where a Babel preset appears as an empty object unless I force-import it with node -r lib/myBabelPreset :-/


Not compatible with Node 0.6

I am getting messages about being unsupported when installing with npm.
node 0.6.0
npm 1.0.105
package.json refers to node 0.4
requires.path is also removed in favour of node_modules
When is an update likely?

I had a quick bash at this: https://github.com/rnewson/nodeload/tree/0.6-compatibility


Give better error message if record not found

If possible with the API, give a different error if a request is successful but a record is not in the database, versus if a request fails completely.

For instance, this is what we get from the API at the moment:
> artifact <- db$Artifact$get("KBW89Mf7IGcekja2hADu")
Error in `private$process_response()`:
! list index out of range

I'd say this is up to the API to do better error handling to provide more descriptive error messages.
For instance, in this case the API should be something like Artifact "KBW89Mf7IGcekja2hADu" not found.
We could implement functionality to map the most common error messages returned by the API into something that is understandable from the perspective of an R user, but at this point it seems the API is simply returning whatever goes wrong inside of the Python code, so there is no knowing which error messages the API might return.

I agree. @fredericenard can comment how fast this can be done.

I did this here for the get record endpoint.

@rcannood @lazappi it has been deployed in prod, you should now have a 404 error for non existing records.


Figure out how to deploy a web app to Azure App Service

For this research, I'll use the sample app https://github.com/Azure-Samples/msdocs-python-flask-webapp-quickstart from Microsoft.
Refs:

https://learn.microsoft.com/en-us/azure/app-service/quickstart-python

I am following the steps at https://learn.microsoft.com/en-us/azure/app-service/quickstart-python

Clone, set up, and run the web app my local machine:
git clone https://github.com/Azure-Samples/msdocs-python-flask-webapp-quickstart

cd msdocs-python-flask-webapp-quickstart

python -m venv venv

. venv/bin/activate

pip install -r requirements.txt

$ flask run
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: off
WARNING: This is a development server. Do not use it in a production deployment. Use a production WSGI server instead.
 * Running on http://<IP_ADDRESS>:5000
Press CTRL+C to quit

Run the app on Azure as an App Service:
az login --tenant <YOUR TENAND ID>

az webapp up --runtime PYTHON:3.9 --sku B1 --logs --location "Germany West Central" --name mbranca-app

The --logs is crucial for our purpose (app services los):

flag configures default logging required to enable viewing the log stream immediately after launching the webapp.

Pick the right region (especially if you want to use an event hub for logs delivery):
$ az appservice list-locations --sku B1
[
  {
    "name": "East US"
  },
  {
    "name": "North Europe"
  },
  {
    "name": "West Europe"
  },
  {
    "name": "Southeast Asia"
  },
  {
    "name": "East Asia"
  },
  {
    "name": "West US"
  },
  {
    "name": "Japan West"
  },
  {
    "name": "Japan East"
  },
  {
    "name": "East US 2"
  },
  {
    "name": "North Central US"
  },
  {
    "name": "South Central US"
  },
  {
    "name": "Brazil South"
  },
  {
    "name": "Australia East"
  },
  {
    "name": "Australia Southeast"
  },
  {
    "name": "Central US"
  },
  {
    "name": "Central India"
  },
  {
    "name": "West India"
  },
  {
    "name": "South India"
  },
  {
    "name": "Canada Central"
  },
  {
    "name": "Canada East"
  },
  {
    "name": "West Central US"
  },
  {
    "name": "West US 2"
  },
  {
    "name": "UK West"
  },
  {
    "name": "UK South"
  },
  {
    "name": "Korea South"
  },
  {
    "name": "Korea Central"
  },
  {
    "name": "France Central"
  },
  {
    "name": "Australia Central"
  },
  {
    "name": "South Africa North"
  },
  {
    "name": "Switzerland North"
  },
  {
    "name": "Germany West Central"
  },
  {
    "name": "UAE North"
  },
  {
    "name": "Norway East"
  },
  {
    "name": "West US 3"
  },
  {
    "name": "Sweden Central"
  },
  {
    "name": "Qatar Central"
  }
]

I am picking "Germany West Central" because I already have an event hub there. Using "East US" due to lack of SKU availability in "Germany West Central".

The app is up and running on Azure at https://mbranca-app.azurewebsites.net


configure is broken on Windows + Cygwin

configure is broken on Windows + Cygwin.
When I run
../configure --ide vs2017 or ../configure --ide qtcreator I get the following:
Loading required submodules...
Downloading GN from cloud storage...
Standard output:

Standard error:
CIPD selfupdate failed. Trying to bootstrap the CIPD client from scratch...
Error: open /home/YURA/Shaka/depot_tools/cipd_client_version: The system cannot find the path specified..
Bootstrap from scratch failed, something is seriously broken. Run the following commands to diagnose if this is repeating:
  export CIPD_HTTP_USER_AGENT_PREFIX=depot_tools/9db428f4f78ea6eebea783850f889a55d8f6a948/manual
  /home/YURA/Shaka/depot_tools/.cipd_client selfupdate -version-file /home/YURA/Shaka/depot_tools/cipd_client_version

Event with
export CIPD_HTTP_USER_AGENT_PREFIX=depot_tools/9db428f4f78ea6eebea783850f889a55d8f6a948/manual
I get the same output.
Additionally, executing
python tests/cipd_bootstrap_test.py
produces following output:
FF
======================================================================
FAIL: test_new_bootstrap (__main__.CipdBootstrapTest)
Bootstrapping the client from scratch.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/cipd_bootstrap_test.py", line 94, in test_new_bootstrap
    self.fail('Bootstrap from scratch failed:\n%s' % out)
AssertionError: Bootstrap from scratch failed:
CIPD selfupdate failed. Trying to bootstrap the CIPD client from scratch...
Error: open /tmp/tmphVnEojdepot_tools_cipd/cipd_client_version: The system cannot find the path specified..
Bootstrap from scratch failed, something is seriously broken. Run the following commands to diagnose if this is repeating:
  export CIPD_HTTP_USER_AGENT_PREFIX=depot_tools/???/manual
  /tmp/tmphVnEojdepot_tools_cipd/.cipd_client selfupdate -version-file /tmp/tmphVnEojdepot_tools_cipd/cipd_client_version

======================================================================
FAIL: test_self_update (__main__.CipdBootstrapTest)
Updating the existing client in-place.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/cipd_bootstrap_test.py", line 101, in test_self_update
    self.fail('Update to %s fails:\n%s' % (OLD_VERSION, out))
AssertionError: Update to git_revision:ea6c07cfcb596be6b63a1e6deb95bba79524b0c8 fails:
Platform windows-amd64 is not supported by the CIPD client bootstrap: there's no pinned SHA256 hash for it in the *.digests file.

----------------------------------------------------------------------
Ran 2 tests in 8.325s

FAILED (failures=2)

Maybe  windows-amd64 hash is missing on the remote server?
Also it looks like there are some issues with absolute paths.
This one fails
/home/YURA/Shaka/depot_tools/.cipd_client selfupdate -version-file /home/YURA/Shaka/depot_tools/cipd_client_version
Error: open /home/YURA/Shaka/depot_tools/cipd_client_version: The system cannot find the path specified..

while this one works
 cd ../../depot_tools/
 .cipd_client selfupdate -version-file ./cipd_client_version

Please address this issue.
Here are some screenshots:

We don't support Windows.  We only support building on Mac and Linux.
As an aside, CIPD is handled by Chrome build tools and we don't handle it.  Make sure to follow the install instructions to setup gclient and make sure to run gclient once from cmd first.

Thank you for your response.
It is a little bit misleading because
configure --help
displays the following
 --ide {eclipse,vs,vs2013,vs2015,vs2017,xcode,qtcreator,json}

I guess vs stands for Visual Studio. AFAIK, Visual Studio is only available on Windows.
Please consider removing vs* from the aforementioned list to avoid confusion.

@TheModMaker How much work would it take to add Windows support?
Are there any plans to add Windows support in near future?

It is hard to say since we haven't tried.  We need to update our build scripts and make sure that Chrome's CIPD and GN work correctly.  Then we need to update a bit of our code and several third-party configure steps.
We have no plans to add Windows since we only officially target iOS for the moment.


Fix i18n problems

I fixed some problems around i18n.

Privacy policy
Cover image for Chinese.
Language links changed to select form.

And improvement other small things.
The following problems are still pending.

Link of text.
Linefeed of text.

Please confirm it.
Regards.
notice
I changed package.
So, you have to run yarn .

@sot528 sorry for late to check pull request. You may leave privacy policy link but I haven't decided whether we need it or not. So please wait and tell you when I decided. LGTM!


Prevent making call to Asset Manager if missing file information

We're seeing issues in Sentry as follows:

URL: https://asset-manager.production.govuk-internal.digital/assets/5b115d34ed915d2cc681ab6f
Response body:
{"_response_info":{"status":["File can't be blank"]}}

An attempt was made in #5581 to catch this error
and provide more detail to help make the error actionable.
Unfortunately, the error condition coded in that PR doesn't
ever appear to happen, so we're still seeing the more obtuse
Sentry logs.
It is evident that new_attributes ISN'T empty, but the file
is considered 'blank' when attempting to send to Asset Manager.
This commit has an explicit check that the attachment_data.file
property exists before trying to update the asset.
This is a short/medium term approach which will give us a slightly
more meaningful error message, and prevent us from making the
faulty Asset Manager call in the first place. In the long term
we will still need to figure out how some assets get into this
state and prevent that from happening.
Trello: https://trello.com/c/FDu35RHS/1977-investigate-whitehall-sending-blank-file-when-updating-assets

I seem to remember digging into this error a little bit with Ed and the idea we came up with was that potentially it wasn't Whitehall that was doing anything wrong here and instead it could be that Asset Manager had data that was already invalid. Since AssetManager is using the ActiveModel approach of validating the model rather than the input this can result in validation errors being returned despite the input itself being valid (which is most frustrating).
I just popped onto an Asset Manager console to try verify with the Sentry link you provided:
irb(main):012:0> asset = Asset.find("5b115d34ed915d2cc681ab6f"); nil
=> nil
irb(main):013:0> asset.valid?
=> false
irb(main):014:0> asset.errors.to_a
=> ["File can't be blank"]

So I think you might find the approach to end this error is to look into Asset Manager to see how/why these assets are invalid.
To add a bit of extra confusion:
irb(main):019:0> asset.file.present?
=> false
irb(main):020:0> asset.file
=> #<AssetUploader:0x00007f871aa26360 @model=#<WhitehallAsset _id: 5b115d34ed915d2cc681ab6f, created_at: "2018-06-01T14:50:28.153Z", updated_at: "2020-02-14T12:54:48.712Z", replacement_id: nil, state: "clean", filename_history: [], uuid: "b69c56f6-eacd-4ac1-a970-b062729fa2c3", draft: false, redirect_url: nil, etag: "5b115d34-3d797", last_modified: "2018-06-01T14:50:28.150Z", md5_hexdigest: "ae028b7c4f20624a39dc763a45193d79", size: 251799, access_limited: [], access_limited_organisation_ids: [], auth_bypass_ids: [], parent_document_url: "https://www.gov.uk/guidance/applications-and-decisions-for-the-west-midlands", deleted_at: nil, file: "ad-2789-1-June-2018.pdf.pdf", _type: "WhitehallAsset", legacy_url_path: "/government/uploads/system/uploads/attachment_data/file/713227/ad-2789-1-June-2018.pdf.pdf", legacy_etag: nil, legacy_last_modified: nil>, @mounted_as=:file, @storage=#<CarrierWave::Storage::File:0x00007f871aa26248 @uploader=#<AssetUploader:0x00007f871aa26360 ...>>, @file=#<CarrierWave::SanitizedFile:0x00007f871aa2f398 @file="/var/apps/asset-manager/uploads/assets/11/5d/5b115d34ed915d2cc681ab6f/ad-2789-1-June-2018.pdf.pdf", @original_filename=nil, @content_type=nil>, @versions={}>

Thanks for the pointer, @kevindew! The conclusion I've come to is this:
The reason bad_asset.valid? is false is because of this line (validates :file, presence: true, unless: :uploaded?).
Its file is neither present*, nor is its state uploaded (bad_asset.state => "clean"), so it fails validation.
* I checked the S3 bucket on staging and it has not been uploaded. I also SSH'd into all Production backend machines to ls /var/apps/asset-manager/uploads/assets/11/5d/5b115d34ed915d2cc681ab6f/ad-2789-1-June-2018.pdf.pdf but it didn't exist on any of them.
Given that it has a state of clean (and not unscanned, which is the first state assets receive), it is reasonable to assume that the file was successfully uploaded to Asset Manager and was successfully scanned. It then somehow got deleted before it could be uploaded to S3.
The two ideas I'm investigating at the moment are:

A logic error in the code caused the file to get deleted before it was uploaded.
The machine the file was locally hosted on got restarted in the time after virus scanning but before deploying. The temporary file got lost in the process.


Copying HParams returns dict

Copying a previously created hparams instance creates a dict instead.
Minimal working example:
import lightning_trainable as lt
hparams = dict(
  batch_size = 1024,
  max_epochs = 20,
  n_dims = 2,
  optimizer = {
    "name": "Adam",
    "lr": 3.e-3,
  },
  gradient_clip = 10.0,
)
hparams = lt.TrainableHParams(hparams)
hparams2 = hparams.copy()
print(type(hparams2)

I am not sure if this is intended. If it is, it wasn't obvious to me that I should expect this reaction.

Not intentional. Fixed in the next minor version.


Vertical layout: English text overflow in the table cell

Source issue: https://github.com/metanorma/metanorma-jis/issues/226
Source adoc: https://github.com/metanorma/mn-samples-jis/tree/vertical_layout_8301/sources/jis-z-8301-2019
Page 102 in PDF:

Fixed.


Hide iconv functions on Apple devices behind a feature flag

See #2870 and #2944 for context. I believe the comment in the code explains the changes well enough, but if you'd like me to add a description to the commit as well (or adjust any wording etc.), let me know.

What would be the best path forward for libc-test? Should I just add the apple-iconv to its dependency on libc, or do something more complex?

Uhm, I hesitate to add a platform feature flag as we support a lot of platforms and the flag management can be complex easily 😱(we've done a similar thing to Rust versions and it introduces a ton of complexity, huh).
Perhaps we should have asked that those functions be separated into a platform-specific crate rather than added to the libc crate.
@rust-lang/crate-maintainers Any thoughts on this?
(By the way, this is a breaking change and I'd like to ship it in a major release.)

I still don't understand what the problem is with linking against iconv on apple targets. This library is guaranteed to always be present, so it shouldn't cause any issues?

@Amanieu See @thomcc's excellent comment here for some info.

:+1: for just dropping these functions entirely in a subsequent major release, if we ever make a major release.

@JohnTitor @joshtriplett Do you think there's any chance of a major release happening within, say, a few months? A year? Just curious, I appreciate your insights, thanks! 😊

We're going to make a major change to musl in https://github.com/rust-lang/libc/pull/3068 and I guess we could publish this along with that PR. It will be merged once FCP passes.
I cannot tell the exact date and time for that, as it requires agreement by team members, but it could happen within a few months (hopefully).

See @thomcc's excellent comment

That was about libstd's usage. I'd don't love that we link to it in libc, but there are reasons I haven't pursued this.

I still don't understand what the problem is with linking against iconv on apple targets. This library is guaranteed to always be present, so it shouldn't cause any issues?

It's mostly compelling since it's practically unused[^2], and linking with it means it must be available on the host[^host] breaks some minimalist (or perhaps naïve) cross-compilation setups, where a full SDK is not available, but a linker and a libSystem stub are (for example, this is the case with zig's cross compilation support, unless you bring along the .tbd files you need). IOW, the library is guaranteed to be present on the target, but not on the host, and it is needed in both places (as linking a mach-o requires the linker know which dylib provides each symbol).
Concretely, this probably is one of the first things hit by folks who try to cross compile rust to macOS by using zig {cc,ld} directly. I'm unsure how reasonable it is to expect cross compilation to work in such cases, and users will probably hit something after this anyway, and are likely better off with something like cargo-zigbuild anyway, which shaves off these kinds of sharp edges (it bundles libiconv.tbd for this reason).

In any case, I don't think we should removing the items (even if they're nearly unused), and I really don't think we should add a feature flag for this. I do think it's annoying that Rust users may have to cart around semi-vestigial libiconv.tbd in their cross compilation setup, even if they don't need it (but perhaps if it weren't that it might be something else[^3]). So, I have two thoughts here:

If we care about that kind of use case, then we should just change the PR to remove the #[link] attribute and deprecate the items (with a message explaining a fix). I don't know if this is technically breaking, but it seems like it unlikely to break anybody.

If we don't care, then we probably should wait until raw-dylib is supported on macOS (which sadly seems stalled). At that point, we could change the attribute to #[link(name = "iconv", kind = "raw-dylib")]. Then (IIUC) we'll generate the tbd automatically when compiling, avoiding such problems. That said, it may be quite a while before that is available on stable.

I do think the hassle this causes in practice is not nearly worth it given how few people (if any) depend on it, but I also think libc should not make the breaking API changes it does (altho I'm unsure how much the link removal would count...). So I don't know.
[^2]: virtually nobody on github use libc's declarations, and the code that does want to call iconv seems to explicitly link anyway. Of course, neither of those searches are absolute proof of anything, but it's a ballpark idea of usage.
[^3]: IOW, perhaps it's better that cross compilation does not work out of the box. I don't think we want to make any guarantees that it will (even in std), nor do we want the bug reports that could come if it worked and then we began linking to another system library.

That's insightful, thank you!

If we care about that kind of use case, then we should just change the PR to remove the #[link] attribute and deprecate the items (with a message explaining a fix).

Do you mean we should deprecate items first and then remove them? If so, how about deprecating on 0.2.x and removing on 0.3.x (along with time64 change)?

Given that it seems much more likely that we will be pushing out a 0.3 release sometime "soon" (highly relatively speaking): Yeah, we should probably instead do a deprecation and doc(hidden).


Snapshot is interpreting single pain windows weird

System Info
OS: Mac OSX 10.14.3
Muxed Version: MASTER @ b3a0703b58ebca912911595a5962726417bdef9a
Description
Snapshot is outputting config files where single pane windows are being written like multi-pane windows. They include active options (which seems like a bug), layout, and panes.
A hand written config like:
windows:
  - make: ''
  - vim: vim .
  - bash: ''

Will produce a snapshot config like:
windows:.
  -
    make:
      active: true
      layout: "5ad5,119x32,0,0,35"
      panes:
        - ""
  -
    vim:
      active: false
      layout: "5ad6,119x32,0,0,36"
      panes:
        - vim .
  -
    bash:.
      active: false
      layout: "5ad7,119x32,0,0,37"
      panes:
        - ""

This seems to lead into additional problems when executing the snapshot config. It may be trying to send commands to the wrong pane? I'm not entirely sure yet. But if a window only has 1 pain, we should avoid writing layout and pane information and use the shorthand syntax. (Although we still need a way to signify active for single pane windows.)
Questions

What to do about active?
Is a single pane still not considered the 0 indexed pane for a window?

Steps to reproduce

Open a tmux session with a single pane window
Execute a command in that window. tail -f /usr/logs/system.log
snapshot the session muxed snapshot -t 0 test
re-run the session muxed test

Expected
A single window session with a log tail running
Actual
A single window session with nothing running

Turns out this has nothing to do with single pane windows and everything to do with pane-base-index.
pane-base-index is no longer provided by tmux show-options -g and requires being accessed by show-window-options. We'll need to show both sets to properly load all configuration.