CORE-1340 Change locator port 444 -> 443

Description
As part of protecting the platform from DDoS attacks, we will be moving locator (xavier) behind a Google Global Load Balancer. These GLBs only support serving TLS traffic on port 443. Locator already supports serving gRPC traffic on port 443, so we can make this patch immediately, and it will be a seamless transition when the DNS for locator starts pointing to the GLB (which exposes port 443) instead of our current load balancer (which exposes port 443 and 444).
Tests
I have manually confirmed that Xavier is serving on 443 and 444 just like all other base servers. I have also confirmed that no additional steps are required to serve gRPC TLS on HTTP TLS ports for our base servers.
I have also run the QA pipeline on this branch and successfully connected to the resulting deployment: https://buildkite.com/improbable/gdk-for-unity-release-qa/builds/75
Documentation
Internal release note in the changelog.
Primary reviewers
If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

Thanks for the reviews - I don't have write access so can't hit the merge button, please can someone land this for me?

Thanks for the reviews - I don't have write access so can't hit the merge button, please can someone land this for me?

Sure thing 😄


mosek.soltype.itr -> unknown but mosek.soltype.bas giving optimal

I've a case where, for verbose=True, Mosek 8.0 completes with
Interior-point solution summary
  Problem status  : UNKNOWN
  Solution status : UNKNOWN
  Primal.  obj: 5.0643588790e+05    nrm: 2e+08    Viol.  con: 3e-01    var: 0e+00
  Dual.    obj: 5.0415624857e+05    nrm: 2e+05    Viol.  con: 5e-02    var: 4e-04

Basic solution summary
  Problem status  : PRIMAL_AND_DUAL_FEASIBLE
  Solution status : OPTIMAL
  Primal.  obj: 4.9845334246e+05    nrm: 2e+08    Viol.  con: 2e-07    var: 0e+00
  Dual.    obj: 4.9845334246e+05    nrm: 8e-01    Viol.  con: 3e-05    var: 6e-17

Per the Mosek interface this causes task.getprosta(mosek.soltype.itr) to return unknown while task.getsolsta(mosek.soltype.bas) reports optimal.  I see that Mosek 7 has the same soltype enum-- I've not confirmed this outcome is version 8-specific but I doubt it given that documentation.
Is mosek.soltype.bas irrelevant or is not interrogating soltype.bas a shortcoming of the cvxpy interface?  Monkeying around a bit in the Python debugger suggests that I can walk through much of the remainder of that interface logic using bas.  That said, a simple s/itr/bas/g in mosek_intf.py does not permit running the front-page example at http://www.cvxpy.org/en/latest/.
Gross hackery in mosek_intf.py like...
        try:
            task.getprosta(mosek.soltype.bas)  # unused
            solsta = task.getsolsta(mosek.soltype.bas)
            soltype = mosek.soltype.bas
        except:
            task.getprosta(mosek.soltype.itr)  # unused
            solsta = task.getsolsta(mosek.soltype.itr)
            soltype = mosek.soltype.itr

...along with s/mosek.soltype.itr/soltype/g in the remainder of the file permits both the front page example as well as my problem to run.  That try/except causes Mosek to spit out MOSEK error 1265: The solution with code 1 is not defined. which is fair.  I've not deduced how to pre-query which solution types are defined.
Recommendations for how to proceed much appreciated.

Thanks for pointing out this! Did you find any definite instruction on how to extract the solution type, in the python-mosek documentation? (I believe lack of that information is the reason why we didn't dig further.)

Something like
(Pdb) task.solutiondef(mosek.soltype.bas)
1
(Pdb) task.solutiondef(mosek.soltype.itr)
1
(Pdb) task.solutiondef(mosek.soltype.itg)
0

informs whether or not a solution has been attempted with a given type.  Using that would be more sensible than the try/except as with solve(..., verbose=True) Mosek logs a scary looking 1265 warning.
Mosek support indicated that the interior-point optimizer provides both by default, with most folks preferring the interior-point variant.  I'm unsure of the right pattern for falling back to the basic solution for cvxpy's purposes, or of how to make cvxpy specify which solution is desired.

I suspect, but have not confirmed, that looks like monkeying wit hhttp://docs.mosek.com/7.0/capi/MSK_IPAR_BI_CLEAN_OPTIMIZER.html.

An easy option would be, as @RhysU says, to check which solutions are available with solutiondef and then pick the one with higher status (prefer optimal over unknown etc.)
For example imagine the scenario when the user passes a MOSEK parameter like mosek.iparam.optimizer: mosek.optimizertype.primal_simplex explitcitly asking only for the basic solution. Now this crashes cvxpy.

@aszekMosek's solution seems reasonable. I'm happy to merge such a PR.

Closing per the merge.  Please re-open if you disagree.


can't remove node label

Describe the bug
Node label can't be removed by setting label: undefined. Setting to an empty string does work but this is not a good workaround because if there is a label background, then this is still visible for an empty string.
Your Example Website or App
https://stackblitz.com/edit/react-spflig-szup5p?file=index.js
Steps to Reproduce the Bug or Issue

try to remove node label with graph.updateItem(node, { label: undefined })
the node label is not removed

Expected behavior
I expect to be able to remove the label by setting it to undefined, or null.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: chrome
Version: 112

Additional context
No response

Thanks for your PR! 4.8.11 has been published last month


Improved error log with failed transactions

When a migration fails with
Error: The query was not executed due to a failed transaction

Caused by:
    The query was not executed due to a failed transaction

I'd like to see a more verbose error message what the reason was for the failed transaction. Would it be possible to display this information?

Yes, I suppose we can go inside the surrealdb query response and extract the error message.
Great idea.

That would be fantastic!

I have released the v1.0.1 crate which should now display the list of error messages when a migration fails.

I tried to compile v1.0.1. create with
cargo install<EMAIL_ADDRESS>but ran into a compile error
error[E0658]: use of unstable library feature 'build_hasher_simple_hash_one'
    --> /home/stephan/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ordered-multimap-0.7.1/src/list_ordered_multimap.rs:3288:34
     |
3288 |     let hash = self.build_hasher.hash_one(key);
     |                                  ^^^^^^^^
     |
     = note: see issue #86161 <https://github.com/rust-lang/rust/issues/86161> for more information

For more information about this error, try `rustc --explain E0658`.
error: could not compile `ordered-multimap` (lib) due to 27 previous errors

Any idea how to fix this?

No problem on my end. What is your version of Rust? Can you try to upgrade to that latest version of Rust and see if the error disappear?

Updating Rust fixed the error. Thanks.


You got firecracker support, great, what about image repo for it ?

Description of problem
Sorry for my ignorance here, but everybody keeps going on about how wonderful firecracker is. Fair play for implementing its support, but I think the whole solution is quite early and it's missing its killer feature - a firecracked image repo.
Expected result
In a similar fashion like docker, it'd be really useful to have an image repository - that'd greatly improve portability and image building process. Are there any efforts to address this yet, even from anybody else ?

@sokoow closing this issue. Please reopen if the details provided were insufficient.


refactor(iroh-net): Make derp_map not an option in MagicEndpoint

Description
This removes the Option from MagicEndpoint::derp_map.  An empty
DerpMap is treated the same as a missing DerpMap in all the code so
this is just a noisy indirection.

Notes & open questions
Is this change important enough to break the public API of
MagicEndpoint?  Should the Option<DerpMap> also disappear from
that API?
In favour of not changing that public API is that this way it is an
implementation detail that an empty DerpMap is treated the same as no
DerpMap.  Though in that case maybe we should make it an error to use
an empty DerpMap?

Change checklist

[ ] Self-review.
[ ] Documentation updates if relevant.
[ ] Tests if relevant.

Allowing None gives us the ability to force non-derp connections only and directly through the provided tickets.

Allowing None gives us the ability to force non-derp connections only and to talk directly through the provided addresses in tickets.

You can still do this by using an empty DerpMap.  Though there should probably be a DerpMap::empty() constructor to make this easy and obvious.

Allowing None gives us the ability to force non-derp connections only and to talk directly through the provided addresses in tickets.

You can still do this by using an empty DerpMap. Though there should probably be a DerpMap::empty() constructor to make this easy and obvious.

oh wait, the public API still has it as an option, but with a TODO to consider this.
So yes the question is both about the internal and external api.  I think internal it makes sense.  External maybe too and I could like a DerpMap::emtpy() for this.

I would like to see sth like this
enum Derp {
  Enabled(DerpMap), // Constructor checks that this is not empty and the node checks that at least the local region id is available
  Disabled,
}

I would like to see sth like this
enum Derp {
  Enabled(DerpMap), // Constructor checks that this is not empty and the node checks that at least the local region id is available
  Disabled,
}

So the only really public API is the MagicEndpointBuilder::derp_map(mut self, derp_map: Option<DerpMap) call.  Are you saying this for there or for internal as well?
This PR started as just acknowledging that internally the code does not ever distinguish a None DerpMap from an empty one, it does eventually end up as an empty one.  So it just peeled back some layers and simplified this.  As this is internal I'm largely fine with that.
I think switching internally to this Disabled enum will be a lot more work.  Maybe the code will end up being clearer, maybe not.  Hard to say ahead of time.

for the public api I would do sth like disable_derp() and enable_derp(map) which forces map to not be empty

I've now updated the public API to be MagicEndpointBuilder::enable_derp(derp_map) and .disable_derp().  This touches a surprising number of places.
I've moved validation of the DerpMap into the constructors itself, the constructor is basically DerpMap::from_regions which is now fallible.  This again touches a lot of places but ensures that if the type exists we have a valid one, which was nicer than having a separate validation function.
(I tried having this constructor in TryFrom<T: Into<Iterarator<Item = DerpRegion>>> since the old one was in such a From, but I lost that fight with the compiler)

can you update the Node::builder api as well, to match this new public api?

can you update the Node::builder api as well, to match this new public api?

oh yes, good call.  sorry to gloss over that at first.
Note that I also changed the default DerpMap used in iroh::Node::builder: this was also empty originally and now uses defaults::default_derp_map.


Cryptographic key import section could provide the details of how to read RSA PCKS1 key

It would be good to just provide simple example how to transform PCKS1 certificate content to actually import it.
Generate key using openssl utility
openssl genrsa -f4 -out private.txt 4096
var keyPath = ".\private.txt"; string keyBytesAsText = System.IO.File.ReadAllText(keyPath); string keyBytesWithoutPEM = keyBytesAsText                 .Replace("-----BEGIN RSA PRIVATE KEY-----", "")                 .Replace("-----END RSA PRIVATE KEY-----", ""); var keyBytes = Convert.FromBase64String(keyBytesWithoutPEM); using var rsa = RSA.Create(); rsa.ImportRSAPrivateKey(keyBytes, out int readBytes); Console.WriteLine($"Read {keyBytesAsText.Length} chars, key bytes {keyBytes.Length}"); var parameters = rsa.ExportParameters(true); Console.WriteLine($"{BitConverter.ToString(parameters.D)}");
This sample describes nicely how to import crypto keys
https://docs.hidglobal.com/auth-service/Content/pages/BuildingApps/Csharp/Read_different_certificate_key_file_formats_with_C_.htm

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

ID: d458030b-3efb-197f-6ac7-9fd4e2296218
Version Independent ID: 6d01409e-fca6-10a3-2def-13ce28c9b210
Content: What's new in .NET Core 3.0
Content Source: docs/core/whats-new/dotnet-core-3-0.md
Product: dotnet-core
GitHub Login: @adegeo
Microsoft Alias: adegeo

@mulawam Thanks for the suggestion.
@bartonjs Should we link to the suggested URL in the 3.0 What's new, and  if so, are there other docs that should also have the link?

@mulawam Thanks for the suggestion.
@bartonjs Should we link to the suggested URL in the 3.0 What's new, and  if so, are there other docs that should also have the link?

I wouldn't link to that, no.   Partially because I don't know how often we link out to third-party content, but definitely because their code samples use RSACryptoServiceProvider instead of RSA.Create().
With .NET 5 the code is simpler:
string keyBytesAsText = System.IO.File.ReadAllText(keyPath);
RSA rsa = RSA.Create();
rsa.ImportFromPem(keyBytesAsText);
return rsa;

I wouldn't link to that, no.   Partially because I don't know how often we link out to third-party content, but definitely because their code samples use RSACryptoServiceProvider instead of RSA.Create().
With .NET 5 the code is simpler:
string keyBytesAsText = System.IO.File.ReadAllText(keyPath);
RSA rsa = RSA.Create();
rsa.ImportFromPem(keyBytesAsText);
return rsa;


Add examples in Scala

No longer relevant to this repository as the sample content has been migrated to the main Apache Kudu repository.


Add Waifu Command

owo i make waifu command for you papi shivaco. 100% tested and ready to use uwu.
changes besides command creation:

pull aiohttp back in
uwu

love tyler :)

The bot is being mean >:( i no make bad code


Plugins V1

As I've been hacking on a POC of the WASM+WASI plugin system, it seems quite clear that the level of effort is still quite high, and WASI and other standards (WIT comes to mind), especially other parts of the WASI protocol such as the sockets API, are not stable at all yet, and especially not in a reasonably useable state. Let's punt on the WASM+WASI plugin system for a future v2.
In the immediate term, it makes sense to build upon what has been introduced in #221 (the hooks system). Not as a replacement, but as a sibling pattern. A history of much more discussion on this topic can be found in #104.
Architecture

Every plugin is simply a binary which can be downloaded by Trunk. We can define standards around how plugins should be archived for Trunk to be able to unpack them properly, along with how to include additional files and the like.
Plugins are invoked by trunk and are passed a protobuf payload on stdin. All plugin behavior should be based upon the protobuf payload.
Plugins are expected to return a protobuf payload on stdout. Stderr can be used by the plugin for diagnostics.
Plugin output is expected to declare the name of any files created, deleted &c.

Declaring Plugins

Plugins are declared in the Trunk.toml under a [[plugin]] section.
The location of the plugin on the internet must be declared. We'll do some exploration on how to support private repos/artifacts.
The "rel" type to be used in the index.html to refer to this plugin for use.

Using Plugins

Plugins are declared for use the same way that other Trunk built-in pipelines are, and their "rel" type must match its declaration in the Trunk.toml.
Attributes declared on the plugin's tag in the index.html will be passed to the plugin as part of the protobuf payload when invoked.
Plugins can be executed (for now) in one of three stages: pre-build, build, post-build. This primarily governs the files which will be visible to the plugin. EG, plugins invoked in the post-build stage will have visibility of all files produces by the built-in pipelines as well as the output of other build stage plugins.
We could also allow plugins to declare custom stages, as well as a dependencies comma separated list of other stage names. Such a stage would only be executed after all of its dependencies have been completed. Basic workflow orchestration.

Misc

This will be an opportunity for us to build the "manifest" system, which could be used by #9 .
Other folks have requested a manifest for other uses. Such uses could probably just be plugins.
The current built-in pipelines will be adapted (pretty minimal) to operate within this framework (just not protobuf or external calls needed in the built-in pipelines case).

How will this work with different platforms? Is trunk expected to install the correct binary for the platform it is currently running on? What if there isn't a binary for the platform?
I like the simplicity of this design but I'm afraid there will be problems across platforms (especially for Windows and Macs with M1 chips)
Possibly related to #212

@lukechu10 the standard should be that a plugin should provide a binary for the same platforms as Trunk. Any "official" plugins which are part of the Trunk org will adhere to the same standard.
If the plugin does not offer binaries for some OS/Arch, but it can be compiled on that arch, then we should have a mechanism to allow for the plugin to be "discovered" as a system installation, which is exactly what the current download system does for wasm-opt & wasm-bindgen.


Using it as a standalone app

Can I use it as a standalone app?
What database is it using and how to configure it?

@ugintl
You can use it as a standalone app. With Electron, maybe?
But That's an implementation detail we don't care about.
It's just a JavaScript application, so it manages events(or schedules) as JavaScript objects. When the window refreshes, all the data will be gone.
So you might need to apply any persistent layer to your application and sync with the calendar. Such as IndexedDB, or other backend servers connected with any database.
You can sync your events between the calendar and the database in beforeCreateSchedule events or beforeUpdateSchedule events.


Python asposewordscloud PostExecuteTemplate API throws ApiException though the response is 200OK

I am using Aspose Python SDKs (Words and Storage) to render word documents within my application.
I installed the SDK using pip:
Aspose Storage v1.0.0 - pip install asposestoragecloud
Aspose Words v1.0.0 - pip install asposewordscloud
I followed the example documented to invoke the Aspose words API,
    try:
        response = wordsApi.PostExecuteTemplate(name=template_name,
                                                file=xml_file_name,
                                                filename=dest_file_name,
                                                withRegions=True)
    except ApiException as e:
        LOG.error("Aspose PostExecuteTemplate ApiException:, Code: {}, "
                  "Message: {}".format(e.code, e.message))

The PostExecuteTemplate is throwing ApiException. The Execption code is 200 and the message OK.  There is no sign of what the error could be.
I did execute the request using CURL and the response looks good.
I assume it is a python PostExecuteTemplate issue.
I also tried upgrading to v1.0.1 (Assuming this bug is fixed in the latest version) using
pip install --upgrade asposewordscloud==1.0.1

But I see these errors: (the same for asposestoragecloud)
Collecting asposewordscloud==1.0.1
Could not find a version that satisfies the requirement asposewordscloud==1.0.1 (from versions: 1.0.0)
No matching distribution found for asposewordscloud==1.0.1
I was only able to install v1.0.1 for both words and storage by cloning the repository and building it.
However, even with this version (v1.0.1) I still face the same issue. Please find attached my curl request and response.
I am using this workaround to overcome the issue:
try:
        response = wordsApi.PostExecuteTemplate(name=template_name,
                                                file=xml_file_name,
                                                filename=dest_file_name,
                                                withRegions=True)
except ApiException as e:
        if e.code != 200:
            message = "Aspose PostExecuteTemplate ApiException:, Code: {}\n" \
                      "Message: {}\n".format(e.code, e.message)
            LOG.error(message)
            raise Exception(message)

Is there anything wrong with the API invocation ?
aspose_curl_request_response.txt

Dear Vidyarani,
Thank you for getting in touch with us. We are looking into the issue and will keep you updated about the results.
Kind Regards,
Mateen Sajjad

Dear Vidyarani,
We have tried to reproduce the error you are facing and unfortunately we have been unable to do. Here is the code sample we are using
``
#set input file name
filename = "SampleInvoiceTemplate.doc"
destfilename = "updated-" + filename
#set input file data name
filedataname = "SampleInvoiceTemplateData.txt"
#upload file to aspose cloud storage
storageApi.PutCreate(Path=filename, file=data_folder + filename)
try:
#invoke Aspose.Words Cloud SDK API to execute mail merge and populate a word document from XML data
response = wordsApi.PostDocumentExecuteMailMerge(name=filename, withRegions=True, file=data_folder + filedataname, filename=destfilename)
if response.Status == "OK":
    print ("mail merge template has been executed successfully")
    #download updated document from storage server
    response = storageApi.GetDownload(Path=destfilename)
    outfilename = "./" + destfilename
    with open(outfilename, 'wb') as f:
                for chunk in response.InputStream:
                    f.write(chunk)

except ApiException as ex:
print ("ApiException:")
print ("Code:" + str(ex.code))
print ("Message:" + ex.message)
``
I have also attached the sample files and the output.  Please let me know if you continue facing this issue.
Kind Regards,
Mateen Sajjad
input-files.zip

Thanks for reverting back to me. Yes, I do still face the issue with this API.
Due to the sensitive nature of the information, I cannot share my files here. Is there a way I can share my template files and keys as a private message ?

Hello,
Kindly post your files on our [https://www.aspose.com/community/forums/aspose.words-product-family/75/showforum.aspx](Support Forums). Don't forget to mark the post as private. We will get back to you as soon as we analyzed the issue you are facing.
Thank You.

We have also uploaded 1.0.1 to PyPI.

I have raised the issue again along with the files in the aspose communtiy forum.
Thanks for the update, I will try upgrading my aspose package to 1.0.1.


guide problems and errors (collective thread)

Used: VS 2017
Library: Intercept <IP_ADDRESS>0 - Static
1. thing
intercept::sqf::disable_ai(follower_unit, intercept::sqf::ai_behaviour_types::TARGET);
ai_behaviour_types is undefined...
-> because the release was outdated and the guide used not the master
2. thing
Also the DllMain is in the default project already in the dllmain.cpp file and is not needed in the Hello_Intercept.cpp (https://github.com/intercept/intercept/wiki/Windows-Environment-Setup-and-Sample-Client-Installation#hello_interceptcpp):
// Normal Windows DLL junk...
BOOL APIENTRY DllMain(HMODULE hModule,
	DWORD  ul_reason_for_call,
	LPVOID lpReserved
)
{
	switch (ul_reason_for_call)
	{
	case DLL_PROCESS_ATTACH:
		break;
	case DLL_THREAD_ATTACH:
	case DLL_THREAD_DETACH:
	case DLL_PROCESS_DETACH:
		break;
	}
	return TRUE;
}

3. thing
VS2017 Enterprise has CMake version 3.7 and for making CMAKE on intercept you need 3.8 so the: https://github.com/intercept/intercept/wiki/Building-and-installing-Intercept-from-source
Information to make that with VS2017 is not valid atm.
-> update 2 of vs2017 fix this
4. thing
In this guide (https://github.com/intercept/intercept/wiki/Windows-Environment-Setup-and-Sample-Client-Installation#create-the-intercept-client-addon) there are some ; missing at all classes after class Intercept (3 ; after })
-> fixed
5. thing
in the guide (https://github.com/intercept/intercept/wiki/Registered-Functions#examples) is the thing outdated:
intercept::types::__internal::GameDataType::STRING
must be:
intercept::types::GameDataType::STRING
under unary and binary is (https://github.com/intercept/intercept/wiki/Registered-Functions#examples):
types::GameDataType::STRING
but need to be (develop branch):
intercept::types::GameDataType::STRING
6. thing
in the guide (https://github.com/intercept/intercept/wiki/Registered-Functions#examples) is the old register function:
intercept::client::host::functions.register_sqf_function_nular
must be:
intercept::client::host::registerFunction
now it is under (https://github.com/intercept/intercept/wiki/Registered-Functions#examples):
intercept::client::host::register_sqf_function(
but it need to be (develop branch):
intercept::client::host::registerFunction(

You need to be on the develop branch to use intercept::sqf::ai_behaviour_types for the moment.

Can you tell us exactly where that double DllMain is?

In the default project there is the DllMain in the dllmain.cpp.
In the guide there is the DllMain in the Hello_Intercept.cpp.
So my VS2017 said me that this is a double definition.

Use an older revision of the wiki page. Here you can find the code that is compatible with what you have downloaded:
https://github.com/intercept/intercept/wiki/Windows-Environment-Setup-and-Sample-Client-Installation/787aea698bf6b5cab73234202f8b0daba85785d1#hello_interceptcpp
In general, I'd say that the wiki pages should pertain to the official releases and not development branches. I realize that everyone will now jump on me and say that it's not an officially official release, but it's still the "oficiallest" official release that is available so I still think that the wiki pages should be modified :P

edited main

In the guide there is the DllMain in the Hello_Intercept.cpp.
That's what I meant. What guide?

mini thing added

4-6 fixed
3. VS 2017 Update 2 has updated CMake.

changed... but 5 and 6 are still not 100% fixed

Public API should be snake_case. So this is an error in the Source not the wiki

fixed.

also to 2. you don't even need to define dllmain in a dll apparently. I didn't know that but it seems to work ^^
5. is also fixed.

In section "Windows Environment Setup and Sample Client Installation" the tutorial from begining to the end of visual studio project setup is very exact. But when it comes to building an addon it starts to be chaotic: in category "Create the Intercept Client Addon", fourth point ask to create file in folder @Hello_Intercept but it hasn't been created.
Also there has to be another problem because even if I assume that @Hello_Intercept is the root folder of the addon it still doesn't work in arma. (I think it might be a problem with naming of the folder where dll has to be put)


Rename lib/$data.js and lib/refs/$data.json without the "$"

Using ajv-5.1.5
We use ajv as a dependency to webpack in the development of single page web applications. Our corporate build servers are configured (by design) to not be able to access Internet resources, including npmjs.com. Since we do not have an internal npm server, we are forced to check-in our dependencies so our build servers can build our web sites. The issue here is that ajx had two files that cannot be checked into TFS -  $data.js and $data.json. This is the first such issue we've encountered in a nodejs type library.
In order to maintain compatibility with the TFS system that is still popular in corporate environments, it would be helpful to rename the $data.* files to simply data.* to allow them to be checked in to TFS. TFS restricts filenames from including the following characters - "/ \ [ ] : | < > + = ; ? * https://www.visualstudio.com/en-us/docs/reference/naming-restrictions
I would create a pull request with a fix; however unfortunately I am forbidden by my employer from contributing to such projects.

What can be the new file name ?

I would suppose that data.js is fine. TFS is refusing to accept file names that starts with a $ for some reason. In other words: It is the leading $ that is the problem.

@Kirtlander if you do a PR please make it to the branch "beta", I can release it as a beta version.

Any progress on this?
I really need to add this files to TFS and the $ is a complication.

what i have done in the mean time is replace all the "$" to "dollar" in this module while waiting for the 6.0

Feel free to submit PR to "beta" branch. "$" can be just dropped.

I am having issues with getting Pending Changes in Team Explorer on Eclipse (on a Mac) due to this issue. My Pending Changes list is always empty because it detects the $ and silently fails. What can I do in the meantime to resolve this?

@GunslingerBara as I wrote above, you can submit PR to "beta" branch (not to master as it is a breaking change), I will then release version 6-beta.

@epoberezkin I'm running into this myself, so I could submit a pull request over the weekend. Would you mind giving me a quick primer on why this is a breaking change?

I'm not sure why this would be considered a breaking change; I was able to
patch this locally by simply renaming the file and the reference, removing
the leading "$" symbol.
Ideally, I would like to install the package unpatched in the event I need
to refresh my node_modules in my project prior to checking them into TFS.
Our corporate build servers don't have access to the Internet; therefore
cannot run npm install on the fly, necessitating checking in all the
dependencies with the project.
On Fri, Oct 20, 2017 at 4:33 PM Kevin Partington<EMAIL_ADDRESS>wrote:

@epoberezkin https://github.com/epoberezkin I'm running into this
myself, so I could submit a pull request over the weekend. Would you mind
giving me a quick primer on why this is a breaking change?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/epoberezkin/ajv/issues/526#issuecomment-338315387,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASVYZta0U_nFuzsFbrW1_73seCbs0PwZks5suQOMgaJpZM4OFdWS
.

why this is a breaking change?

Because this is a public schema URI that depends on the file name: https://raw.githubusercontent.com/epoberezkin/ajv/master/lib/refs/$data.json#

I would like to install the package unpatched

Once the change is released you will be able to install<EMAIL_ADDRESS>without any patches

in 6.0.0-beta.0

Thanks @epoberezkin, and sorry I couldn't get to this in a timely fashion.
Anything I can subscribe to for watching for the official 6.0.0 release? Thanks!

No problem. Probably just watch the repo? Or gitter. Also, if you use greenkeeper it will tell.


Disable pretty print on debug

Environment
Mac
Knex version: 0.15.2
Database + version: Postgresql
Feature discussion / request

I Wish to enable the knex debug log and send the result to elastic search

Since knex outputs the debug data in a pretty-print format log shippers and appenders split the output to multiline and index it wrong.

I suggest allowing the user to supply a "debugFunction" that will print the output in the user requested format (for example, stringify the object)

I found out the "log" key in the config object.
thank you!


Enhance lora tests with more layer and rank variations

Enhance lora tests with more layer and rank variations.

On a high level, I think these tests would be much better understandable if it was a single test that evaluates the reference and then tests against the target, instead of having two tests and hardcoding the taget.
I also find it very irritating that we only test correctness of the first token for each sequence. Is there a way to test the rest of the sequence too and still have it be robust? Other tests do similar things, right? Maybe instead of testing the predicted tokens, you can test that the logits are numerically close.

can we merge?


Log time delta of scheduler terminate_child call

This is for debugging, remove later.

Nah, will just hotpatch this for debugging


slickGoTo not working when responsive

Go to http://jsfiddle.net/wzL4ztLg/
When you run the demo in a full sized screen, slickGoTo works as expected. When you shrink the frame down and hit run, slickGoTo doesn't seem to work.
I've also found if you keep the frame small but remove the responsive settings, slickGoTo works again - http://jsfiddle.net/wzL4ztLg/1/

I've encountered the bug as well. But it seems as it will only break the slider if you run it directly on init.
I added a timeout off 300ms and that "solved" the problem for me.
Like so:
$('.the-slider').on('init', function(){
setTimeout(function() {
$('.the-slider').slick('slickGoTo', 1);
}, 300)
});

can we have some news on it ?

Nearly two years later and still no news?? Here's a code pen.


Better copilot introduction needed

Describtion
So right now, when you go to the chats in the webapp and open one of them, you'll see a popup with information about the new copilot feature:

but the problem is, that it uses the same card style thingy like you can find on the signin page, which is in my opinion not the greatest.
Describe the solution
I think a much better way to intruduce the copilot would be, to make some kind of a tooltip that would appear right on the checkbox  when you first load the chat (based on the localStorage CapacitorStorage.copilotInstructionsSeen property).

Related w/ #753


Flaky test_inverse_indices_shape

https://ci.preferred.jp/cupy.linux.cuda110/97390/
E           hypothesis.errors.Flaky: Hypothesis test_inverse_indices_shape(func=unique_all, x=Array([False], dtype=bool)) produces unreliable results: Falsified on the first call but did not on a subsequent one

I think this will be fixed by merging #6573


Replicating the results in papers -- Evaluation Details?

Dear authors, thanks for releasing the code and congrats on the RA-L acceptance! We are trying to replicate the results listed in Table II (goal-conditioned manipulation) because we want to test our model using the exact same pipeline you used. May I ask the details of evaluations you ran? For example, we are running the test_quantitative.py as-is with the default parameters (100 trials for each category). Does this suffice to replicate the results reported?
Another thing is, how were the test pickle files in test_data generated? How are the sampled object pose and joint states distributed?
Thanks.

The previous test_quantitative .py has a small bug, which has been fixed in the latest version. The default parameters are sufficient to replicate the results reported in the paper.
If scene_state =None, the reset() function in sim.py will randomly sample a scene. The scene state (e.g., object pose, joint state) is then achieved by get_scene_state() and stored in a pickle file.

Object position is fixed (center of the workspace), but the orientation about the z-axis will be randomly sampled.
The camera pose is fixed. Other camera parameters can be found here.
Our test data is generated by the reset() function exactly.

Thanks. Closing this for now.


add nan checks

Detailed Description
Add checks on nwp and satellite if there are any nans
Context

good to check data is validate
should we log a warning, or raise an assertion?

Possible Implementation

could check when we load the data, and when we have made the batches
we should check for, nans, if all the values are exactly the same

@jacobbieker would you be able to take this?

Yep, I'll start working on it today

Thanks @jacobbieker

Finished with #236


[Event Request] codeunit 80 "Sales-Post" - OnCheckCustBlockageAfterResetTempLines

In codeunit 80 "Sales-Post" please add the following event:
    local procedure CheckCustBlockage(SalesHeader: Record "Sales Header"; CustCode: Code[20]; ExecuteDocCheck: Boolean)
    var
        Cust: Record Customer;
        TempSalesLine: Record "Sales Line" temporary;
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeCheckCustBlockage(SalesHeader, CustCode, ExecuteDocCheck, IsHandled, TempSalesLineGlobal);
        if IsHandled then
            exit;

        with SalesHeader do begin
            Cust.Get(CustCode);
            if Receive then
                Cust.CheckBlockedCustOnDocs(Cust, "Document Type", false, true)
            else begin
                if Ship and CheckDocumentType(SalesHeader, ExecuteDocCheck) then begin
                    ResetTempLines(TempSalesLine);
                    TempSalesLine.SetFilter("Qty. to Ship", '<>0');
                    TempSalesLine.SetRange("Shipment No.", '');
                    // New event
                    OnCheckCustBlockageAfterResetTempLines(SalesHeader, TempSalesLine);
                    // New event
                    if not TempSalesLine.IsEmpty() then
                        Cust.CheckBlockedCustOnDocs(Cust, "Document Type", true, true);
                end else
                    Cust.CheckBlockedCustOnDocs(Cust, "Document Type", false, true);
            end;
        end;
    end;

    [IntegrationEvent(false, false)]
    local procedure OnCheckCustBlockageAfterResetTempLines(SalesHeader: Record "Sales Header"; var TempSalesLine: Record "Sales Line" temporary);
    begin
    end;

Thank you

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. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[BUG] Create a 'Jobs' Page

On clicking on 'Jobs', nothing happens, instead it should route me to Jobs Page
Would like to create a 'Job' Page with demo jobs for now
Request you to assign this task to me under GSSoC '24

Thanks for letting know to work on Job page !
Please take reference from UI made in readme file
Good to go !

Thanks for letting know to work on Job page ! Please take reference from UI made in readme file Good to go !

hi sahil, may i use tailwindCSS or should i go with normal CSS? because using tailwindCSS would be more effecient

Yes go with normal css because tailwind is effecting to all over responsive side

I would like to work on this issue @yashmandi

Raise the New issue @Pranali3103 for this
Thanks


FEATURE: add draining nodes metric

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


When you encode something as Option[Option[String]] it can not properly decode

Expected: Right(VacancyProfileF(1683177f-5edb-4a9c-8f41-ca1b9e9ac631,Some(PartTime),Some(None),Some(O)))
Recieved: Right(VacancyProfileF(1683177f-5edb-4a9c-8f41-ca1b9e9ac631,Some(PartTime),None,Some(O)))

final case class VacancyProfileF[F[_]](
  id: VacancyProfileId,
  vacancyType: F[VacancyType],
  title: F[Option[NonEmptyString]],
  description: F[NonEmptyString]
)

Where F is set op Option
As you encode Some(None) and expect that.. but it's isomorphic to None which you get back
The use case is, when F is set to cats.Id it's an insert when it's set to Option it's an update where you choose to update specific fields by supplying a Some of something. When there is a Option inside this data structure, like title you'll get a Option[Option[NonEmptyString]]. Setting the title to nothing is Some(None)

This question comes up occasionally, and [this approach] is something I've used and recommended in the past. The issue isn't really circe-golden-specific, or even circe-testing-specific—it just becomes more obvious when you're doing this kind of laws checking.

Good point, thanks for the approach on fixing that. I see no reason to keep this issue open


iOS  15+ save password prompt dismiss is pushing UI up and down

Steps to Reproduce

Launch a flutter application with a login form (email/pw).
Fill out the form, submit and navigate to another page.
On iOS (I observed this on iOS 15+), dismiss the prompt.
** On the machine I ran, it currently has flutter 2 however, when upgraded to flutter 3, the same behavior is present. And appears only to affect iOS.

Expected results: The prompt should dismiss from the widget leaving the underlying page unmodified.
Actual results: After being dismissed, the contents on the widget is being pushed up and down.

Code sample
Simplified widgets to show the issue:
import 'package:exampleapp/widgets/simple_page.dart';
import 'package:flutter/material.dart';

class AutoFillPwWork extends StatefulWidget {
  const AutoFillPwWork({Key? key}) : super(key: key);

  @override
  State<AutoFillPwWork> createState() => _AutoFillPwWorkState();
}

class _AutoFillPwWorkState extends State<AutoFillPwWork> {
  final TextEditingController email = TextEditingController();
  final TextEditingController password = TextEditingController();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(
          title: const Text('Save PW iOS'),
        ),
        body: ListView(
          children: <Widget>[
            const Text('Login'),
            AutofillGroup(
              child: Column(
                children: <Widget>[
                  TextField(
                    controller: email,
                    autofillHints: const <String>[AutofillHints.email],
                  ),
                  TextField(
                    controller: password,
                    autofillHints: const <String>[AutofillHints.password],
                  ),
                ],
              ),
            ),
            ElevatedButton(
              onPressed: () {
                Navigator.push(
                  context,
                  MaterialPageRoute(builder: (context) => const SimplePage()),
                );
              },
              child: const Text('Push SimplePage'),
            ),
          ],
        ));
  }
}

import 'package:flutter/material.dart';

class SimplePage extends StatelessWidget {
  const SimplePage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.red,
      appBar: AppBar(
        title: const Text('Simple Page'),
      ),
      body: Center(
        child: Container(
          color: Colors.blue,
          child: const Text('Simple Page'),
        ),
      ),
    );
  }
}

Flutter (Channel unknown, 2.10.1, on macOS 11.6.8 20G730 darwin-x64, locale en-US)
    • Flutter version 2.10.1 at /Users/user/Development/flutter
    • Upstream repository unknown
    • Framework revision db747aa133 (8 months ago), 2022-02-09 13:57:35 -0600
    • Engine revision ab46186b24
    • Dart version 2.16.1
    • DevTools version 2.9.2
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
• Android SDK at /Users/user/Library/Android/sdk
• Platform android-31, build-tools 31.0.0
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2020.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
[✓] VS Code (version 1.71.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.48.0
[✓] Connected device (2 available)
• iPhone (mobile) • ios            • iOS 15.7 19H12
• Chrome (web)           • chrome                    • web-javascript • Google Chrome 105.0.5195.125
[✓] HTTP Host Availability
• All required HTTP hosts are available

https://user-images.githubusercontent.com/66581623/192004581-06dc62c9-1b97-43ac-9073-c94a578dbb8d.MP4

Thanks for the report @rsp-84
I think there's an animation taking place when after entering password and you directly tap on push simplePage button, with virtual keyboard still open which might be impacting the UI as you described.
Can you try, after entering password, tap on done button from virtual keyboard and then tap on push simplePage and see if you still get same UI behavior on next page or not ?

https://user-images.githubusercontent.com/66581623/192310655-8310d224-cbec-456f-a0c2-215962f8fc74.MP4

@rsp-84
I am unable to replicate it on iPhone 6s running on 15.3.1 using latest master version.

3. ** On the machine I ran, it currently has flutter 2 however, when upgraded to flutter 3, the same behavior is present. And appears only to affect iOS.

Can you provide updated flutter doctor -v along with device details and also try on latest master to see if same behavior is present ?

@darshankawar –
[✓] Flutter (Channel unknown, 3.3.0, on macOS 11.6.8 20G730 darwin-x64, locale en-US)
    • Flutter version 3.3.0 at /Users/user/Development/flutter
    • Upstream repository unknown
    • Framework revision ffccd96b62 (4 weeks ago), 2022-08-29 17:28:57 -0700
    • Engine revision 5e9e0e0aa8
    • Dart version 2.18.0
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/user/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.71.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (3 available)
    • iPhone (mobile)  • ios            • iOS 15.7 19H12
    • macOS (desktop)        • macos                     • darwin-x64     • macOS 11.6.8 20G730 darwin-x64
    • Chrome (web)           • chrome                    • web-javascript • Google Chrome 105.0.5195.125

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Device: It is an iPhone X on iOS 15.7

Thanks for the update @rsp-84. For some reason, I am unable to see the save password option on next screen as shown below even though I have autofill passwords and icloud keychain settings enabled.
https://user-images.githubusercontent.com/67046386/193201018-9cf0614d-c8bb-47dd-a055-e0bcd7e3c006.MP4
One of my colleagues is also seeing same behavior on different iOS device:
https://user-images.githubusercontent.com/67046386/193201220-cd0d46f5-aec4-42a2-be8b-2fc6d224d9b4.MP4
I am going ahead and keeping this issue open based on the report.

/cc @LongCatIsLooong

@darshankawar I believe in order for it to work on iOS and receive the prompt, you need to have an Assoicated Domains set in your Xcode project.

Thanks for the tip @rsp-84. Yes, that was missing from my project. Adding it and then running the code sample, on latest stable and master, with which I am now able to replicate the behavior.
https://user-images.githubusercontent.com/67046386/193522188-ebe7ec7e-68e8-404d-97ea-826bc37c2e98.MP4

 stable, master flutter doctor -v

[✓] Flutter (Channel stable, 3.3.3, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.3.3 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18a827f393 (2 days ago), 2022-09-28 10:03:14 -0700
    • Engine revision 5c984c26eb
    • Dart version 2.18.2
    • DevTools version 2.15.0

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 3.4.0-36.0.pre.3, on macOS 12.2.1 21D62 darwin-x64,
    locale en-GB)
    • Flutter version 3.4.0-36.0.pre.3 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5cd15fd224 (4 hours ago), 2022-10-02 21:21:39 -0400
    • Engine revision a7785c48cd
    • Dart version 2.19.0 (build 2.19.0-265.0.dev)
    • DevTools version 2.17.0

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

we are facing the same issue too

@darshankawar is there any update regarding this issue?

This is still happening in the latest versions of Flutter and iOS

I'm also facing this with Flutter 3.10.1 and iOS 16.4.1 in all situations where AutofillGroup is used, making the app look really buggy.

Any changes it can be prioritized soon?

I'm also facing this with Flutter 3.10.6 and iOS 16.4. in all situations where AutofillGroup is used.

@darshankawar  ^

Also seeing this...

It's a good thing ios and flutter aren't both popular. Oh wait a second..

I'm still seeing this issue.

Same here...

Same here... but testing on simulator, if save password prompt is called by hitting enter on physical keyboard with soft keyboard hidden, it does not happen.
Flutter 3.10.5 iOS 16.4, 15... on iOS 14 it does not happen at all
additionally, is it possible to only show the prompt if successfully logged?
from tests I have done, the prompt is displayed whenever the soft keyboard is dismissed... tried TextInput.finishAutofillContext() but it either save(show prompt) or ignore if shouldSave is false...

Are you find solution for this issue?

In case someone needs a workaround I found something but it's kind of specific for my scenario.
I'm designing a login form and I want to save credentials only on success. I achieved this but when going to another screen this bug was causing the jumpy animation. I was able to "solve" it by using provider (I guess any state manager will do).
Steps I did:

the AutofillGroup will always AutofillContextAction.cancel unless a state variable is true (in my case my login flag)
the related clients (TextInputFields) will call TextInput.finishAutofillContext(shouldSave: false); when losing focus unless the login flag is true
the keyboard of the inputs will be set to TextInputType.none if the login flag is true
before navigating, the login flag is set to true (changes the above mentioned behaviours)
I request the focus of an element (seems to recreate the autofillgroup sate) but keyboard won't show (is set to none)
I navigate after a microtask which triggers the prompt without the jumpy animation

More info on this thread of SO:
https://stackoverflow.com/questions/77273376/flutter-autofillgroup-and-provider-on-a-login-form-how-to-save-password-on-lo/77278658#77278658

Is any soulution for this annoing issue ?

The bug / glitch is still reproducable with Flutter 3.16.8 and iOS 17.3.

Hi everyone!
I've made the following observations:

This can be reproduced without even navigating to another page.
The UI up and down glitch is produced the Scaffold widget resizing the scaffold body. When I replace Scaffold widget with Material widget (this is required), the issue goes away. This confirm the `

To reproduce enter some text in both email and password fields and press "done" on the keyboard in the second field.
Code sample

expand to view the code sample
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      debugShowCheckedModeBanner: false,
      home: HomePage(),
    );
  }
}

class HomePage extends StatefulWidget {
  const HomePage({super.key});

  @override
  State<HomePage> createState() => _HomePageState();
}

class _HomePageState extends State<HomePage> {
  final TextEditingController controller1 = TextEditingController();
  final TextEditingController controller2 = TextEditingController();

  @override
  void dispose() {
    controller1.dispose();
    controller2.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold( // Replace Scaffold with Material to fix glitch.
      body: Center(
        child: Column(
          mainAxisSize: MainAxisSize.min,
          children: <Widget>[
            const Text('Login (Without Scaffold)'),
            AutofillGroup(
              child: Column(
                children: <Widget>[
                  TextField(
                    controller: controller1,
                    autofillHints: const <String>[AutofillHints.username],
                  ),
                  TextField(
                    controller: controller2,
                    autofillHints: const <String>[AutofillHints.password],
                  ),
                ],
              ),
            ),
          ],
        ),
      ),
    );
  }
}

Without Scaffold
https://github.com/flutter/flutter/assets/48603081/70ff48ed-92ee-430b-b3e4-059dc382e016
With Scaffold
https://github.com/flutter/flutter/assets/48603081/e5ab28a1-0636-4def-a53e-6627d0b74780

resizeToAvoidBottomInset seems to be the cause of this issue. When the Scaffold or CupertinoPageScaffold is allowed  to resize to avoid bottom inset, iOS popup produces a 300 pixels of bottom padding after being dismissed,
flutter: MediaQuery.of(context).viewInsets.bottom:<PHONE_NUMBER>35732
flutter: MediaQuery.of(context).viewInsets.bottom: 0.0

This regressed in https://github.com/flutter/engine/pull/29281. I'm working on a fix.

Unfortunately, the fix for this issue is reverted in https://github.com/flutter/engine/pull/50760 as it is causing a regression for keyboard created by a third-party package.

@TahaTesser Hello! Could you please provide more explanation about issues with third-parties? I think now if anyone want to fix this, they can't be sure that it won't break something again.


Parse Server as of release 2.2.15 creates initial _User and _Index collections that prevent migration from facebook.

Parse Server as of release 2.2.15 creates initial _User and _Index collections that prevent migration from facebook.
in order for migrations to be available the mongoDb must be empty.
the code was first introduced here:
https://github.com/ParsePlatform/parse-server/blob/d599fdd4feccf93be56acd3fa5a3307aac187803/src/ParseServer.js#L200
and now lives here:
https://github.com/ParsePlatform/parse-server/blob/master/src/Controllers/DatabaseController.js#L895
is this change really required?

If you're planning to do a migration, you should have an empty database, you're right, so you should shut down your parse-server or point it to another DB, run your migration and then point back to the right database. Does that help?

not really. we are trying to stand up a hosted service, and since 2.2.15 this makes things very difficult.

The parse-server should only connect to mongoDB AFTER the migration is completed. Is that really something you can't do?

our flow is:
user signs up, gets redirected to dashboard.
user creates app from dashboard.
dashboard talks to app (at this point we're broken because mongo is populated).
User finds mongo url in dashboard settings and migrates.
all very clean, within the dashboard, and very similar with the historical parse flow.

I believe you'll need to find another flow, work around that or provide a PR that will still ensure people that deploy to their own infrastructure the functionality that is currently in place.


[Need help]BetterDiscord Custom CSS

can someone tell me how i can rempove this public and users online on the top left corner. if yes can you pls help me. Ty

Not the place to be, you should join the support servers instead.

Settings > Core > Public Servers


ValueError: x and y must have same first dimension, but have shapes (118,) and (134,)

Tried to create a run chart using two lists - but held back due to sample data returned in second query is not consistent -
result = spark.sql("select temperature from washing where temperature is not null")
print(result.count())
result_array = result.rdd.map(lambda row : row.temperature).sample(False,0.1).collect()
len(result_array)
**printed below as expected -
1342
134
result_ts = spark.sql("select ts from washing where temperature is not null")
print(result_ts)
result_array_ts = result_ts.rdd.map(lambda row : row.ts).sample(False,0.1).collect()
len(result_array_ts)
** Was able to verify result_ts as '1342'  but  sample is not giving 10% of given data,  i.e. result_array_ts returned 118 records
plt.plot(result_array_ts,result_array)
plt.xlabel("time")
plt.ylabel("temperature")
plt.show()
** failed to create run chart with the error mentioned
"ValueError: x and y must have same first dimension, but have shapes (118,) and (134,)"

please use the coursera forum, thanks a lot, this is for errors in my code, not yours :)


Compatibility with Vapor 4 implementation

Hi there!
I've been working on a server reference implementation based in the Swagger file. I'd love your help with testing and keeping it compatible moving forward.
It can be found here: https://github.com/kevinrmblr/traceprivately-server
There's some open ends, such as limits on updating daily keys and updating the related data, but mostly it should be ready:

It saves the form fields (as plain text) alongside the batch
Saves the risk level
Calls support both plan JSON and MessagePack

I also left room to submit data in the auth call, so a token can be added, like a PushToken or a DeviceId token. Depending on the app setup, it'll either require this or also accept an empty body (which is currently the default to be compatible with the iOS app), an auth body could look like this:
{
	"strategy": "IOS_DEVICE_ID",
	"token": "SOMETOKEN"
}

Let me know your thoughts!

Hey @kevinrmblr - this is great! I will link to it from the README.
In regards to auth - the auth call in the iOS app can submit data as required. The protocol KeyServerAuthentication allows for this (with a couple of example implementations).
Any yea, it submits in the same way as you've described. Please see see KeyServerAuthentication.swift.

Hi there, thanks for the info. As far as I can see, it can either post a body with a key of either:'receipt' or 'token'. The former would be a push token and the latter a DeviceID token.
Would you up for adding a strategy field (or some other name)? Then it'll be a bit easier to identify which authentication method is used, especially if a server allows more than one. Then both push-token and deviceid-token can share the same token key.

It’s loosely defined because I think there’s still scope for this to change somewhat. Realistically the publisher or the app and server will customise this as necessary.
Perhaps the prebuilt Auth modules can include a strategy field rather than requiring it. I think that’s worthwhile doing, especially to help debug/setup the app and server.

@kevinrmblr I've added a t parameter to the requests which indicates the method used for authentication. It's defined by the specific auth module and is optional, so any custom implementations don't need to use it.
See 0a436786f3b09c9d77e5da3a924ca38862370677

Hi @HendX, thanks!
I see there's an identifier t, that currently can be either dc or asr, ill update the server to mimic that!


V/A Lag (1 Sec Delay)

Hi,
I know you've had some issues with this last year and I read a lot of the posts about it but couldn't find an actual solution or whether it was an on-going issue (wasn't shown in issues).
As the title states whenever I download a VOD (whether it be 1 or 2 hours, 30fps or 60fps) I get a V/A delay of around 1 second from the MP4 file produced by your software. Haven't had this issue before yesterday but now I always have the delay. Provided the log below. Any help would be grateful.
Download Log:
Download task has been started!
TWITCH LEECHER INFO
Version: 1.4.3
VOD INFO
VOD ID: 267733324
Selected Quality: 1920x1080@60fps (Source)
Download Url: https://www.twitch.tv/videos/267733324
Crop Start: Yes (00:00:00)
Crop End: Yes (02:06:06)
OUTPUT INFO
Output File: D:\nmp\20180531_267733324_Fortnite.mp4
FFMPEG Path: D:\Twitch Leecher\ffmpeg_x64.exe
Temporary Download Folder: C:\Users\lewis\AppData\Local\Temp\TL_004c61c0-bb48-4071-a6b2-962ef37cb281
ACCESS INFO
Token: %7B%22authorization%22%3A%7B%22forbidden%22%3Afalse%2C%22reason%22%3A%22%22%7D%2C%22chansub%22%3A%7B%22restricted_bitrates%22%3A%5B%5D%7D%2C%22device_id%22%3Anull%2C%22expires%22%3A1527913049%2C%22https_required%22%3Afalse%2C%22privileged%22%3Afalse%2C%22user_id%22%3Anull%2C%22version%22%3A2%2C%22vod_id%22%3A267733324%7D
Signature: 5395fae8427a138302dd2eaae6b38e9e05217d69
Sub-Only: No
Privileged: No
Creating directory 'C:\Users\lewis\AppData\Local\Temp\TL_004c61c0-bb48-4071-a6b2-962ef37cb281'... done!
Retrieving m3u8 playlist urls for all VOD qualities... done!
https://vod-pop-secure.twitch.tv/b5829e318561eb53ce82_nmplol_28907184336_875369185/chunked/index-dvr.m3u8
https://vod-pop-secure.twitch.tv/b5829e318561eb53ce82_nmplol_28907184336_875369185/720p60/index-dvr.m3u8
https://vod-pop-secure.twitch.tv/b5829e318561eb53ce82_nmplol_28907184336_875369185/720p30/index-dvr.m3u8
https://vod-pop-secure.twitch.tv/b5829e318561eb53ce82_nmplol_28907184336_875369185/480p30/index-dvr.m3u8
https://vod-pop-secure.twitch.tv/b5829e318561eb53ce82_nmplol_28907184336_875369185/audio_only/index-dvr.m3u8
https://vod-pop-secure.twitch.tv/b5829e318561eb53ce82_nmplol_28907184336_875369185/360p30/index-dvr.m3u8
https://vod-pop-secure.twitch.tv/b5829e318561eb53ce82_nmplol_28907184336_875369185/160p30/index-dvr.m3u8
Playlist url for selected quality 1920x1080@60fps (Source) is https://vod-pop-secure.twitch.tv/b5829e318561eb53ce82_nmplol_28907184336_875369185/chunked/index-dvr.m3u8
Retrieving playlist... done!
Parsing playlist... done!
Number of video chunks: 1262
Starting parallel video chunk download
Number of video chunks to download: 1262
Maximum connection count: 10
Parallel video chunk download is running...
Download of all video chunks complete!
Creating local m3u8 playlist for FFMPEG... done!
Writing playlist to 'C:\Users\lewis\AppData\Local\Temp\TL_004c61c0-bb48-4071-a6b2-962ef37cb281\vod.m3u8'... done!
Executing 'D:\Twitch Leecher\ffmpeg_x64.exe' on local playlist...
Command line arguments: -y -ss 0 -i "C:\Users\lewis\AppData\Local\Temp\TL_004c61c0-bb48-4071-a6b2-962ef37cb281\vod.m3u8" -analyzeduration<PHONE_NUMBER> -probesize<PHONE_NUMBER> -c:v copy -c:a copy -bsf:a aac_adtstoasc -t 7566 "D:\nmp\20180531_267733324_Fortnite.mp4"
Processing complete!
Starting temporary download folder cleanup!
Deleting directory 'C:\Users\lewis\AppData\Local\Temp\TL_004c61c0-bb48-4071-a6b2-962ef37cb281'... done!
Download task ended successfully!

As indicated in other issues TL is not capable of adding A/V desync since it does not re-encode the video but only merges the source streams. The mp4 is a direct result of the data from Twitch without any processing. I have tried downloading a part of your given video and everything is exactly the same as the source stream in the Twitch web player.
You say you haven't had this issue before yesterday... how is it possible that TL has something to do with it? TL does not decide to function differently from one day to another ;)
There are 2 options:

Your video player messes up the A/V sync on playback (happens with Media Player for example). Use VLC to verify.
You import it into a video software and re-encode it. Adobe Premiere is one of the miserable tools that cannot handle Twitch streams the way they are.

Like I said often times before: TL does not do anything with the source material except of merging files together so it is not even capable of adding or removing A/V desync.

Yup, I think it's the video player. Didn't mean to blame your software if it came across that way, thought it was a genuine issue.
Sorry for any wasted time.


[HW] MONTOUT

Name: Jean-Marc MONTOUT
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/jean-marc-montout/
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:
Did not finish the tuto yet 😕
https://github.com/jmarcm/stargate-tik-tok.git

Hey Jean, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/Nh65nMcpT0yBR61JGfjLVg


Axios dependency

Hi there, cool library! I would love to use this in Vue 3, got it on display also but it complains about axios not being defined. I provided the full API path in the "server" tag. Do I have to initialize something in axios?

@caveofjulian
Hello,
i get the same error. I do not know how to fix it but to temporarily bypass it you can do some kind of global import of axios.
Check here https://stackoverflow.com/questions/51374367/axios-is-not-defined-in-vue-js-cli .

Since i am testing the package on a Laravel project, i don't get this error.
I just made a new commit, where i tried to fix that .
Please make sure that you are using the last version (1.0.4 for now) and tell me if this problem still occurs to you.
Thanks for your feedback!


added dialog flow agent with hackernews intents

added dialog flow agent that includes the hackernews intents, as part of the pr here: https://github.com/intuit/benten/pull/37


[intel-mkl] build failure using GitHub Actions (MSBuild based project)

Host Environment

OS: Windows via GitHub Action (msvc_build.yml)
env:
  BUILD_CONFIGURATION: Release
  ARCH: x64

jobs:
  build:
    runs-on: windows-latest
...

Compiler: MSBuild (supplied by GitHub Actions virtual environments)
steps:
...

- name: Add MSBuild to PATH
  uses<EMAIL_ADDRESS>

...

To Reproduce
Steps to reproduce the behavior:
- name: Install vcpkg prerequisites
  shell: cmd
  if: steps.cache.outputs.cache-hit != 'true'
  run: |
    vcpkg update
    vcpkg install intel-mkl --triplet=${{env.ARCH}}-windows
    vcpkg ...
    vcpkg integrate install
    vcpkg list

  ...

Failure logs

See full log here.

Run vcpkg update
  vcpkg update
  vcpkg install intel-mkl --triplet=x64-windows
  shell: C:\Windows\system32\cmd.EXE /D /E:ON /V:OFF /S /C "CALL "{0}""
  env:
    SOLUTION_FILE_PATH: ./dynadjust/dynadjust_1_01_00.sln
    BUILD_CONFIGURATION: Release
    ARCH: x64
    VCPKG_INSTALL_FOLDER: c:/vcpkg/installed
    CS_XSD_INSTALL_FOLDER: xsd/xsd-4.0.0-i686-windows/libxsd
Using local portfile versions. To update the local portfiles, use `git pull`.
No packages need updating.
Computing installation plan...
The following packages will be built and installed:
    intel-mkl[core]:x64-windows -> 2020.0.0
Detecting compiler hash for triplet x64-windows...
Could not locate cached archive: C:\Users\runneradmin\AppData\Local\vcpkg\archives\f8\f8e99c4db6108860ec37bb79c9bbdbbc05a34887cc655ec8fcd3a7400f38d72b.zip
Starting package 1/1: intel-mkl:x64-windows
Building package intel-mkl[core]:x64-windows...
CMake Error at ports/intel-mkl/portfile.cmake:15 (message):
  Could not find MKL.  Before continuing, please download and install MKL
  (20200000 or higher) from:

      https://registrationcenter.intel.com/en/products/download/3178/

  Also ensure vcpkg has been rebuilt with the latest version (v0.0.104 or
  later)
Call Stack (most recent call first):
  scripts/ports.cmake:139 (include)

Error: Building package intel-mkl:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: intel-mkl:x64-windows
  Vcpkg version: 2021-05-05-b110959e13f562c2a17d7bd9e352e2dd8d6dc603

Additionally, attach any relevant sections from the log files above.
Error: Process completed with exit code 1.

Additional context

Successful build of the repo requires other packages (e.g. boost-*, 7zip, xerces-c, etc.).   Installation of these packages (e.g. vcpkg install boost-process ... --triplet=${{env.ARCH}}-windows) is successful.
Caching was originally applied, then removed, but neither cases were successful.
Full yaml here.

What is the question?
CMake Error at ports/intel-mkl/portfile.cmake:15 (message):
  Could not find MKL.  Before continuing, please download and install MKL
  (20200000 or higher) from:

      https://registrationcenter.intel.com/en/products/download/3178/

https://github.com/oneapi-src/oneapi-ci

Thanks @autoantwort.
As this yaml is for a Windows-based workflow using MSBuild, the expected behaviour (as per docs) is:

vcpkg automatically downloads and builds intel-mkl using VC20xx (via vcpkg install intel-mkl ...)
vcpkg automatically adds intel-mkl include directories, link directories and libraries to VC20xx  (via vcpkg integrate install)

Looking at the vcpkg error message, the issue seems to be that vcpkg cannot find/build/install intel-mkl, yet it is a supported port.  For example, the following yaml:
run: |
  vcpkg update
  vcpkg search intel-mkl

gives:
No packages need updating.
intel-mkl                2020.0.0         Intel® Math Kernel Library (Intel® MKL) accelerates math processing rout...

As this is a GitHub Action, there is no previous installation of intel-mkl (as per #3322 and #13050).
The question is - Why is vcpkg unable to automatically download, build and install intel-mkl using MSBuild on a Windows-based workflow?
Thanks again.

The first line of the file ports/intel-mkl/portfile.cmake says:
# Due to the complexity involved, this package doesn't install MKL. It instead verifies that MKL is installed.

But the descriptions should definitely mention that.

Thanks @autoantwort. If there is no future plan for this package to install MKL, I'm happy for this issue to be closed.
Yes, a description that notifyies users of vcpkg ports that aren't automatically installed would be very beneficial for future users.
Thanks again.

Hi @rogerfraser
Thanks for posting this issue.
For interl-mkl, vcpkg just detects if MKL is installed. So you need to install MKL manually, then rebuild this.

Since there are also other ports have the same cases with intel-mkl. And there is also the information described in portfile.cmake:
# Due to the complexity involved, this package doesn't install MKL. It instead verifies that MKL is installed.
So I think it's unnecessary to mention this in description field.

And there is also the information described in portfile.cmake:
# Due to the complexity involved, this package doesn't install MKL. It instead verifies that MKL is installed.
So I think it's unnecessary to mention this in description field.

But nobody reads the portfile.cmake. And why should you as normal user?

I think adding the information to description field is fine. @NancyLi1013, could you please submit a PR to add that information?


Remove SnapshotProposals directory

Fixes https://github.com/SetProtocol/index-ui/issues/372
Follow up to https://github.com/SetProtocol/index-ui/pull/367
Description
Delete SnapshotProposals and parent components that are no longer used
Testing
yarn start works

Tagging @tytarma and @0xModene as reviewers


Feed for blog

Check for existing issues

[X] Completed

Describe the feature
The Zed blog should have a RSS/Atom/JSON feed, for people to follow in their news aggregators.
If applicable, add mockups / screenshots to help present your vision of the feature
No response

Also for the releases page please!

Just tried to subscribe to your the blog's RSS feed, and found nothing. All Blogs should have feeds. Developer focused blogs doubly so.

Yes, especially for things that are in active development. If a planned feature is currently blocking me from using Zed, a simple RSS feed will hopefully tell me when I can start using Zed! 😸


Logger assumes string

Description
In
https://github.com/vikejs/vike/blob/b272e325909bcc0a012ba5c9d893992671656f88/vike/node/plugin/shared/loggerVite.ts#L17-L29
msg is assumed to be string, and trimmed.
Using vavite and ran into this line:
https://github.com/cyco130/vavite/blob/3347bfbbf92127cb1a220ce02d65761967e1add5/packages/reloader/src/index.ts#L288
Passing a TypeError object produces another error, and the original one is 'silenced' in the output.
Arguably this is a bug on vavite's side, but ideally Vike's logger plays it safe too.

Makes sense. Up for a PR?

Going ahead and closing as it's more of a vavite issue, but PR welcome!


Custom Controller

Laravel Version: 5.4
Voyager Version: last
PHP Version: 7
Database Driver & Version: MYSQL

Description:
I can edit controller of some bread?
For example, i want create order_by or show date by some colum of my table...
I try create with voyager:controllers but i dont understand  the error: "namespace must start with application namespace.." What is application? My folder of laravel?
Thanks...
Steps To Reproduce:

Inside config/voyager.php I changed this:
'controllers' => [
    'namespace' => 'TCG\\Voyager\\Http\\Controllers',
],

To this:
'controllers' => [
    'namespace' => 'App\\Http\\Controllers\\Voyager',
],

Then I ran:
php artisan voyager:controllers
And it worked just fine.

Please don't run artisan voyager:controllers unless you NEED to override ALL controllers.  Instead, you should subclass the controller and override only what you NEED to, then change the controller through the BREAD configuration.

That should be mentioned in the documentation.

I agree completely @zietbukuel.  We're always open to pull requests.

Thanks guys.

A complete example of this please

Hi, I have a multiple input(checkbox) with the same name in array, but voyager saved only the first value on the input. I want to overwrite only the save and update methods in Controller but I don't know what is the best approach to do. I red the documentation, but it seems incomplete.

Labels: documentation, discussion, question, help wanted, open for suggestions

php artisan voyager:controllers copies all base controllers to whatever namespace provided, re running is not unlinking old files, need more documention and commands for it

We recommend against using that command for that very reason.  Yes, The docs need to be updated.

What if you want to override the VoyagerAuthController? How can one go about this?

I would like to know why you'd want to do that.  Off the top of my head, I'm not sure how you'd do that, but I'm sure there's a way (likely just extending it with your own, then overriding the appropriate route(s))

okay, thanks for the lead, I want to extend the Voyager AuthController and override some methods.

Yes I understood that much.  But what functionality are you looking to override?  I ask because there may be an easier way.

@fletch3555 I want to override the postLogin and register methods, from one of your replies above, I was able to do that yesterday, thanks for the help.

Go to
vendor/tcg/voyager/src/Commands/ControllersCommand.php
And comment
//        if (!starts_with($namespace, $appNamespace)) {
//            return $this->error('The controllers namespace must start with your application namespace: '.$appNamespace);
//        }
them change
$location = str_replace('\', DIRECTORY_SEPARATOR, substr($namespace, strlen($appNamespace)));
to
$location = '\Http\Controllers\Voyager';
After that run the command
artisan voyager:controllers
Finally change de Namespace correctly in all the new child controllers to
namespace App\Http\Controllers\Voyager;

Please don't recommend people change files under vendor.
Also, this has been closed for over a year and was for a MUCH older version that's no longer supported.  Please avoid commenting on old/closed issues.


475 add different downsampling methods to patch gan discriminator

Added features according to issue #475 leaving the default behaviour the same. According to my testing it should be as fast as the previous implementation.
Additionally, I fixed the 2D_spade_VAE.py tutorial, as it didn't work on my Windows laptop.

@StijnvWijn thanks for fixing the tutorial. Since the tutorial file has undergone a lot of changes, could you create a different issue and PR for this? It's a different issue from the one treated in this one, so we would like to treat them separately.
Just copy-pasting the tutorial of this branch into the new PR would suffice.
Thanks.

In addition, could you add use cases to test test_patch_gan.py that uses the available pooling methods? No additional test is needed, just additional configurations in TEST_3D and TEST_2D.

Things work,  I ran the autofix script and committed these changes.


Hamburgers are not vertically aligned to center of button

My hamburger settings:
$hamburger-padding-x: 3px;
$hamburger-padding-y: 0;
$hamburger-layer-width: 24px;
$hamburger-layer-height: 2px;
$hamburger-layer-spacing: 5px;

With Normalize.css, a style with
font-family: sans-serif;
font-size: 100%;
line-height: 1.15;
margin: 0;

will be applied to the <button> element causing the button to have a height of 24px whereas my hamburger will only have 16px height with the settings above.

As per the screenshot, you can see the hamburger does not align with the button's height vertically.
First thing I tried
Adding vertical-align: middle to .hamburger-box.

But it still doesn't seem vertically centered.
Second (which hopefully is a better solution for hamburgers)
Giving .hamburger a width and height property, and position centering .hamburger-box in relative to .hamburger as such.
.hamburger {
   width: 32px; // can be overwritten by button css
   height: 32px; // can be overwritten by button css
   position: relative;
}

.hamburger-box {
   position: absolute;
   top: 50%;
   left: 50%;
   transform: translate(-50%, -50%);
}

Giving us this.

I can create a PR for this if you'd like @jonsuh
Cheers

@xavianaxw I ran into this as well. A simpler solution is to explicitly set a line-height value of 1 on the base hamburger class:
.hamburger: {
    line-height: 1;
}

@xavianaxw I ran into this as well. A simpler solution is to explicitly set a line-height value of 1 on the base hamburger class:
.hamburger: {
    line-height: 1;
}

And to set the hamburger-box element to display block:
.hamburger-box {
    display: block;
}

I just did it, can confirm it's pixel perfect.

The reason is display: inline-block on hamburger-box, it's considered as a text node by browser and it tries to align it with baseline (vertical-align value by default).
2 possible solutions:

add vertical-align: middle; on .hamburger-box class (pull-request maybe?);
add font-size: 0; on .hamburger, but if you want too use hamburger with label with .hamburger-label, you would need to specify font-size explicitly for .hamurger-label.

PR https://github.com/jonsuh/hamburgers/pull/88


Deployment with Windows bootstrap node: Docker version is outdated

The Docker version present on the agents is the one shipped by default via the Azure marketplace image, which is 17.06.2-ee-13.
We need a version >= 18.xx, since it includes a patch that will let us use the flag com.docker.network.windowsshim.disable_gatewaydns=true, needed for dcos-net dns functionality to work reliably.
To upgrade, we need to run this as part of provisioning:
Install-Module DockerMsftProvider -Force
Install-Package -Name docker -ProviderName DockerMsftProvider -Force -RequiredVersion 18.03.1-ee-1
Restart-Service Docker

Fixed by https://github.com/Azure/dcos-engine/pull/49


Inject Manifest - Should we write to a variable for flexibility?

There are a number of issues where developers would like to intercept / manipulate the array of precached assets. Would it make sense to ask the developer to define an array with a unique string instead of a specific function allows flexibility over precache() vs precacheAndRoute() or manipulation of the assets based on client side or runtime hints?
For example, if the inject regex looked for [/*** workbox-inject-manifest ***/], the following would be all valid:
const myPrecacheList = [/*** workbox-inject-manifest ***/];

workbox.precaching.precache([/*** workbox-inject-manifest ***/]);

workbox.precaching.precacheAndRoute([/*** workbox-inject-manifest ***/]);

const precacheController = new workbox.precaching.PrecacheController();
precacheController.addToCacheList([/*** workbox-inject-manifest ***/]);

There is the custom inject regex that should enable this.


fromJust shouldn't exist in polished code

Related to https://mail.haskell.org/pipermail/libraries/2015-February/025010.html
% cat testi.hs
import Data.Maybe

foo :: Maybe Int -> Int
foo x = if isJust x then fromJust x else 0

bar :: Maybe Int -> Int
bar x = 1 + fromJust x

% cabal run testi.hs
Preprocessing executable 'hlint' for hlint-1.9.16...
Running hlint...
testi.hs:4:9: Error: Use fromMaybe
Found:
  if isJust x then fromJust x else 0
Why not:
  fromMaybe 0 x

testi.hs:4:26: Error: Use fromMaybe
Found:
  fromJust
Why not:
  fromMaybe (error "Impossible! This should never happen.")

testi.hs:7:13: Error: Use fromMaybe
Found:
  fromJust
Why not:
  fromMaybe (error "Impossible! This should never happen.")

3 suggestions

The implementation of fromJust is fromMaybe (error "Maybe.fromJust: Nothing"), which I think is better than fromMaybe (error "Impossible! This should never happen.").

@watashi the behind Impossible! this should never happen. is that is controllable by end-programmer, he could specify the message more precisely. Generic Maybe.fromJust: Nothing doesn't help, as it doesn't tell who called it.
Yet @ndmitchell discussed this on mailing list already, he didn't like this PR. Maybe if we could have separate rule databases, than this would be ok-ish.

You definitely don't want to suggest they replace fromJust with fromMaybe (error "bad fromJust") since after they've made the replacement there is no call to fromJust at all so it's very misleading. I'll come back to the multiple rule databases sometime in the near future, with any luck.

See https://github.com/ndmitchell/hlint/issues/420 - we can now, so this is now fine by me.


[misc] Removed spacejam dependency

Phantomjs is an unmaintained project. Removed spacejam from dev-dependencies as it uses phantomjs internally.

Coverage remained the same at 88.41% when pulling 93c5b7dc07965c3af3fa4f37ef9cbd6720e3c348 on ossdev07:moment_aarch64 into 13a61b285c095bda7ea8e33156090ea5ccfeaef1 on moment:develop.

Coverage remained the same at 88.41% when pulling 93c5b7dc07965c3af3fa4f37ef9cbd6720e3c348 on ossdev07:moment_aarch64 into 13a61b285c095bda7ea8e33156090ea5ccfeaef1 on moment:develop.

Ping @marwahaha Please can you have a look at this PR.

@mgorabbani , @marwahaha : I would like to see PhantomJS removed. As well as being unmaintained, it is not available for AArch64.

Ping @ashsearle could you please have a look at this PR and share your opinion.

@ossdev07 Looks fine to me, but I'm not a maintainer: I don't know what tools are needed for building a release, and whether spacejam needs to be replaced with a suitable alternative.

Thanks for the quick response @ashsearle, if you are aware, could you please let me know who are the maintainers so that I can ping them.

Based on commit history, the most active maintainer is marwahaha.

@marwahaha, @mgorabbani I would like to see PhantomJS removed. Please, could you consider this PR for the merge?

Hi @marwahaha , @mgorabbani ,
Can you please have a look at above PR.
Thanks.

If you want to remove spacejam you should also remove all code that depends on it: Gruntfile.js.

Hi @ichernev, Done as you suggested.
Can you please have a look at the changes?
Thanks.

Hi @ichernev , Done the changes suggested.
Please have a look at above changes.
Thanks

Removing the dependency was done in afa66633e7f1584919657962269dab68c91b2d76


Overhaul Arch instructions

Move to alphabetical order under Linux heading
Remove confusing wording about official/community since Ghostty's perspective and Arch's perspective on this (or any distro for that matter) use the same terms to mean different things.
Simplify/clarify where the package is and how to get it.
Stop promoting the AUR package as a better alternative to the official stable package, and don't promote a specific AUR helper instead linking to relevant Arch docs.

Fixes up remnants of #135, see also #141.

Looks good. I still want to make it clear what packages are and are not maintained by the Ghostty project, but I'll avoid the use of the word "community" in this context.

Fixed a quick typo too!


installation fails in snow leopard

As a note, in snow leopard because of its outdated curl, brew doesn't even install you have to do some funky workarounds: https://apple.stackexchange.com/questions/280203/install-old-version-of-homebrew-on-snow-leopard
Not sure if it's worth anything but thought I'd mention it.

[x ] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[n/a ] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[n/a ] ran brew update and can still reproduce the problem?
[ n/a] ran brew doctor, fixed all issues and can still reproduce the problem?
[ n/a] ran brew config and brew doctor and included their output with your issue?

To help us debug your issue please explain:

What you were trying to do (and why)
Install home brew
What happened (include command output)

rogers-macbook:~ rdp$ /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
usage: sudo [-n] -h | -K | -k | -L | -V | -v
usage: sudo -l[l] [-AnS] [-g groupname|#gid] [-U username] [-u username|#uid]
[-g groupname|#gid] [command]
usage: sudo [-AbEHnPS] [-C fd] [-g groupname|#gid] [-p prompt] [-u
username|#uid] [-g groupname|#gid] [VAR=value] [-i|-s] []
usage: sudo -e [-AnS] [-C fd] [-g groupname|#gid] [-p prompt] [-u
username|#uid] file ...
==> This script will install:
/usr/local/bin/brew
/usr/local/share/doc/homebrew
/usr/local/share/man/man1/brew.1
/usr/local/share/zsh/site-functions/_brew
/usr/local/etc/bash_completion.d/brew
/usr/local/Homebrew
Press RETURN to continue or any other key to abort
==> Downloading and installing Homebrew...
curl: (35) error:1407742E:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert protocol version
Failed during: /bin/bash -o pipefail -c '/usr/bin/curl -fsSL https://github.com/Homebrew/brew/tarball/master | /usr/bin/tar xz -m --strip 1'

What you expected to happen

It work :)

Step-by-step reproduction instructions (by running brew commands)

/usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"

We no longer support 10.6 I'm afraid so we'll accept PRs to fix this but won't be working on it ourselves.


Problem with InlineKeyboardMarkup

Steps to reproduce
I used below example and i clicked on button but nothing happened
TelegramBotClient Bot = new TelegramBotClient("<Bot Token>");

Bot.OnMessage += async (seder, e) =>
{
    var keyboard = new InlineKeyboardMarkup(
        new InlineKeyboardButton[][]
        {
        // First row
        new [] {
            // First column
            new InlineKeyboardButton("1.1", "Data1"),

            // Second column
            new InlineKeyboardButton("1.2", "Data2")
        },
        // Second row
        new [] {
            // First column
            new InlineKeyboardButton("2.1", "Data3"),

            // Second column
            new InlineKeyboardButton("2.2", "Data4")
        }
        }
    );

    await Bot.SendTextMessageAsync(e.Message.Chat.Id, "tap on the keyboard", replyMarkup: keyboard);
};

Bot.OnCallbackQuery += async (sender, e) =>
{
    await Bot.AnswerCallbackQueryAsync(e.CallbackQuery.Id, $"You have choosen {e.CallbackQuery.Data}");
};

Bot.StartReceiving();
Console.ReadKey();
Bot.StopReceiving();

Expected  behavior
Bot must response to clicked button without any problems
Actual behavior
When i clicked on the button nothing happened:

Environment data
Telegram.Bot Version: Latest version
.NET Version: 4.5
IDE: VS 2017 Enterprise
App: Desktop
OS: Windows 10 Enterprise

Hi,
You can try this version of the BotOnCallbackQueryReceived procedure:
Bot.OnCallbackQuery += async (sender, e) =>
{
await Bot.AnswerCallbackQueryAsync(e.CallbackQuery.Id, $"You have choosen {e.CallbackQuery.Data.ToString()}");
};
please note the call to ToString() method;
This is the piece of my code that had worked for me:
 private static async void BotOnCallbackQueryReceived(object sender, CallbackQueryEventArgs callbackQueryEventArgs)
    {
        await Bot.AnswerCallbackQueryAsync(callbackQueryEventArgs.CallbackQuery.Id,
            @"Received " + callbackQueryEventArgs.CallbackQuery.Data.ToString() );
    }

It works, thank you


examples/mesh/ip_internal_network: creation of corrupt mesh_netif_driver (IDFGH-13311)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
In mesh_netif.c, the data structure mesh_netif_driver attempts to represent the ESP-WIFI-MESH driver control block.  In fact, the declaration of mesh_netif_driver is in fatal conflict with the declaration of wifi_netif_driver in wifi_netif.c.  Specifically, the fields wifi_if and sta_mac_addr occupy the same memory location in the control block.
Because ESP-WIFI-MESH 'netif' structures are often passed through esp_wifi APIs, these conflicting field declarations result in the first 4 octets of the ESP32's MAC address being used as the wifi_if index, leading in disastrous runtime results (which have been observed during testing of the ip_internal_network).
While it would at first appear that adding wifi_if to the mesh_netif_driver structure would resolve the conflict, the result is still not correct.  In fact, there is no public declaration of the actual ESP-WIFI-MESH driver control block, so any creation or manipulation of the structure within mesh_netif.c can not be assured to be conflict-free with other fields within the actual structure.
For WiFi purposes, not only is the full declaration of the driver control block visible, but the function esp_wifi_create_if_driver() is available to reliably create and initialize the structure.  No such public declarations or utility functions appear available to safely create an ESP-WIFI-MESH driver control block.

@vjgriswold Thanks for report, we will check it ASAP.

@vjgriswold
In the mesh_netif.c file, although the driver parameter passed to esp_netif_attach() function is of type mesh_netif_driver_t, only the base part of mesh_netif_driver_t is read by esp_netif_attach(). Therefore, the sta_mac_addr field does not affect the wifi_if field.

In the ip_internal_network example, the purpose of the mesh_netif.c is to redefine the transmit function of the netif driver, thus the nodes can communicate using low level mesh send/receive API to exchange data. However, in reality the ESP-WIFI-MESH software stack is built atop the Wi-Fi Driver. So there is no need to public the mesh_netif.c like wifi_netif.c
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/network/esp-wifi-mesh.html#esp-wifi-mesh-programming-model

The problem is not with esp_netif_attach(), which knows nothing about WiFi STA vs AP configuration, but with all the other functions which do distinguish between STA vs AP and use the wifi_if field to make the distinction.  These functions include:  wifi_transmit(), wifi_transmit_wrap(), esp_wifi_create_if_driver(), esp_wifi_destroy_if_driver(), esp_wifi_get_if_mac(),  esp_wifi_is_if_ready_when_started(), and esp_wifi_register_if_rxb().
Each of these functions is subject to invalid memory access faults when the field within the device control block is used for any other purpose besides wifi_if.  This has been observed numerous times during operation of the ESP-WIFI-MESH sample program.
If the Mesh device control block is absolutely identical to the WiFi device control block, the example code must still be adjusted so that sta_mac_addr does not sit on top of the same memory location as does wifi_if.


fix(config): fix order of precedence in option processing

Default values were overriding values set at higher levels of
precedence, namely command line args (which have the highest
precedence!).
Patch includes specific tests cases, which could be expanded to include
ini and keyring sources of config options. (The tests only cover
defaults, env, and command line sources of config params.)

Nice catch @larsbutler!
Is there a test case for

For example, if the defaults
define a value for the option foo, and args contains foo=None,
the args should override the default.

?
And I left a comment demonstrating my confusion about argparse.SUPPRESS and the help/usage output. Maybe you could include one test that asserts that the --help output includes/doesn't-include what you expect?