Invalid type class cache after a change in MetavarContext

Prerequisites
Please put an X between the brackets as you perform the following steps:

[x] Check that your issue is not already filed:
https://github.com/leanprover/lean4/issues
[x] Reduce the issue to a minimal, self-contained, reproducible test case.
Avoid dependencies to Mathlib or Batteries.
[x] Test your test case against the latest nightly release, for example on
https://live.lean-lang.org/#project=lean-nightly
(You can also use the settings there to switch to “Lean nightly”)

Description
It is possible for the type class synthesis cache to become invalid when the MetavarContext changes. This is usually no problem because the function instantiateMVars is run on the goal, so if the MetavarContext changes, then the goal also changes. However, if the metavariable is hidden inside a let variable, this becomes a problem.
Context
I was trying to better understand type class synthesis cache, and came up with this example.
Steps to Reproduce
Create a metavariable ?m, and define a let variable a := ?m. First let ?m := Nat, and synthesize Inhabited a, and then let ?m := Int and synthesize Inhabited a again. Lean then finds the instInhabitedNat instance in the cache, and so it fails to find Int.instInhabited. Here's the code:
import Lean

open Lean Meta

run_meta do
  let m ← mkFreshExprMVar (Expr.sort levelOne)
  withLetDecl `a (Expr.sort levelOne) m fun a => do
    withMCtx (← getMCtx) do
      logInfo m! "{← isDefEq m (.const ``Nat [])}; {← trySynthInstance (.app (.const ``Inhabited [levelOne]) a)}"
    withMCtx (← getMCtx) do
      logInfo m! "{← isDefEq m (.const ``Int [])}; {← trySynthInstance (.app (.const ``Inhabited [levelOne]) a)}"

Expected behavior: [Clear and concise description of what you expect to happen]
The logInfo should give
true; (some instInhabitedNat)
true; (some Int.instInhabited)

Actual behavior: [Clear and concise description of what actually happens]
The logInfo gives
true; (some instInhabitedNat)
true; none

Versions
4.9.0-rc1
Additional Information
Impact
I doubt this ever comes up in practice, so there is not much impact.
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

Possible fix: define a function instantiateVars which instantiates both metavariables and let variables by their definitions, and use this instead of instantiateMVars at the start of synthInstance?.


Java-language-server

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

🏷 Feature Type
New Addon
🔖 Feature description
title describes the feature.
✔ Solution
I can make a PR to setup java-language-server like other LSPs, have tested it adding it in my config and it works, just wanna know if i should go ahead with it as most people prefer jdtls but it much harder to setup lol, also with all the changes that might be made to how lsp config works, wondering if it would be the right time to do so
❓ Alternatives
waiting for a while until LSP changes are done and then making a PR
📝 Additional Context
idk anyone who likes java so there isn't any hurry for sure

We are always open to new features.
If you are confident with your implementation, and believe you an adequately document its quirks, go right ahead.
P.S. LSP changes are still a long time away, might be a good idea to PR now and I'll change them later when necessary.

I'll start working soon making it good enough and the documenting

It's not. If you can implement it, would be nice. If not, you may skip that.

nice, unfortunately it only supports LSPv2.0 so it gives one error when opening a Java file, the server stuff still works, auto complete suggestions and all, but there is an error, it doesn't support the client/registerCapability. I will give nvim-jdtls a fourth shot ig :sob:

I'll move to pr


Option to suppress console logging

Suppress console logging
I upgraded my Napari version and am now seeing loads of logging output on my launch console (not the built-in Napari console). This makes it difficult to view other information being printed. I tried various approaches to suppress the logging output from the module, but have not found any success. Logging suppression may already be a feature, but I cannot seem to find it in the documentation. Ideally, there would be a way to select which information is printed, e.g., deprecation warnings.
The "Console notification level" in Preferences seems to only affect the built-in console.
Motivation
Want to be able to view other printed output without sifting through loads of logging info 🙂

strange.  I don't think it's napari doing that... that is, I've never seen that while working on/with napari.  That could happen though if any of the packages imported in your environment modify the global python logger.  Have you already tried a new environment?  and could you perhaps give the output of conda list or pip list so we can see what's in the environment?  (even still, it does seem like it could also be settings-related for another package.  I do see a lot of matplotlib in there, which makes me wonder whether something is adjusting its logging level?)`
I also wonder what would happen if you manually set the python logging level with basicConfig:
import logging
logging.basicConfig(level=logging.CRITICAL)

import napari
viewer = napari.Viewer()

thanks for the env info!

but then that instance is missing all the features we added to the program, which I assumed was the cause of the problem ha.

can you tell me how you're starting napari then?  is it a custom script? something other than running napari on the command line or just creating a viewer in ipython, eg.  Any particular imports there that might be involved?

Yes! I have a launch script that creates an instance of a custom Controller object. Inside that object we instantiate a Viewer object—this is also where we start modifying some of the GUI. Then we run napari.run() at the end of the launch script. Please see below...
import napari
import logging

from controller import Controller

if __name__ == "__main__":
    logging.basicConfig(level=logging.CRITICAL)

    # execute main application loop
    well_controller = Controller(version="1.5.2")
    napari.run()

The reason I suspected this may be due to a change in Napari was that this logging output only started appearing after a recent upgrade of the Napari package, and all other packages were kept at their current version.

ok, I don't see controller in your pip list above, and when I tried pip install controller I get an object that lacks version param:
TypeError: Controller() got an unexpected keyword argument 'version'

so ... need more information.  What is controller and what version do you have

Sorry, controller is a local script that we import. However, if I run the script below, I still see the verbose logging.
import napari
import logging

from controller import Controller

if __name__ == "__main__":
    logging.basicConfig(level=logging.CRITICAL)

    # execute main application loop
    # well_controller = Controller(version="1.5.2")
    viewer = napari.Viewer()
    napari.run()

No, if that is commented out then the verbose logging stops. So that means it must be related to another package?

It at least means there's an interaction somehow.  The fact that it didn't happen on an earlier version of napari is interesting, but without knowing more about the source of the chatter it's hard to say why

We can absolutely continue this conversation here, but since the actually logging is coming from outside of napari, I'm going to close this as I'm not sure there's too much we can do besides help guide you in discovering and suppressing the source of the logging.  Free free to share more of your controller module if you'd like some help

Thanks for your help @tlambert03! I have no issue sharing my code, but it is also a massive project. I will try to first hunt down the module responsible 🙂


Add resolution range in documentation

Related with https://github.com/CartoDB/carto-vl/issues/838

Question 👼: in the torque documentation, it says:

-torque-resolution Spatial resolution in pixels. A resolution of 1 means no spatial aggregation of the data. Any other resolution of N results in spatial aggregation into cells of NxN pixels. The value N must be power of 2.

I changed the 4x4 example to the NxN example. Should we add that 'N' does not need to be power of 2?
cc @makella @davidmanzanares


Removing weird "unset" that are preventing proxy url setting

Hi folks,
This PR acts in removing a weird "unset" that was unsetting the proxy URL that was already setted.
I was trying to apply some proxy use with this lib for 2catpcha but the "proxy" URL are never sent in ApiClient, just "proxytype".
So looking in your source-code I could found some strange code:
$proxy = $params['proxy'];
$params['proxy'] = $proxy['uri'];
$params['proxytype'] = $proxy['type'];
unset($params['proxy']);

As you can see in the second line we are setting the proxy url as $params['proxy'] = $proxy['uri']; but in the last line unset($params['proxy']); the script are removing the proxy url that was setted some lines before.
If you aprove this PR, that will be needed to submit a new version to composer packagist.

The problem was caused by matching key names. Thank you @lucasferreira, we'll publish the updated version soon.


Publish Errors not shown in VScode

With the May Update when I publish an App and Error occurs, VScode doesn't do anything after Sending request to:
VScode-Output:
Success: The package is created.
[2018-05-27 09:38:21.21] Publishing package to tenant 'default'
[2018-05-27 09:38:21.26] Sending request to https://bcsandbox.westeurope.cloudapp.azure.com:7049/NAV/dev/apps?tenant=default&SchemaUpdateMode=recreate

In the Event Log of the Service Tier, I get something to work with:
[..]
Message Unexpected event subscriber failure -- SubscriberType:CodeUnit; SubscriberId:20; TargetType:Table; TargetId:17; SubscriberCLRMethod:OnInsertGLEntry; SubscriberALName:OnInsertGLEntry; TargetMethod:OnAfterInsertEvent; Error:Value cannot be null.
Parameter name: element
[..]

Would be nice to get this same error message in VScode.

Hi @DanielGoehler, I think you are facing two unrelated issues here.
The first one is that the webclient does not open automatically after publishing the app in VS Code which seems to be a bug. We also experienced it, but we couldn't get a stable repro for it yet. A workaround is to try to re-publish many times until it succeeds.
The error you can see in the Event Log is a runtime error from the base app because one subscriber is missing the element name property (in C/AL). In AL, this is something we verify at compile time so you shouldn't get into this situation. As this is a runtime error, the AL extension does not know about it.

@qutreson I also have the issue that the webclient can't open after publishing the app in VS Code and therefore, I cannot debug. I can reproduce this every time I try to publish in the master:us (but also in other master images).
I even used just AL:GO and then published the sample and am receiving the error after going with the latest master image. When I try to go back to a previous image (or also downgrade the AL extension to a previous build), it does not resolve the issue. So, this is a dead stop for debugging. Do you need this reported In a new issue?

@qutreson On May Update (13.0.22267.0 DE), which I create from "Daily Builds - Developing for a future release of Dynamics 365 Business Central" as Azure VM, I get this problem repeatedly every time I hit F5.
When I change the Size of that VM from D2_v3 (2 CPUs and 8 GB RAM) to D4_v3 (4 CPUs and 16 GB RAM) I get a proper error message:
[2018-05-29 11:53:12.64] Publishing package to tenant 'default'
[2018-05-29 11:53:12.65] Sending request to https://sfcxthuxppfcupx.westeurope.cloudapp.azure.com:7049/NAV/dev/apps?tenant=default&SchemaUpdateMode=recreate
[2018-05-29 11:56:51.85] The request for path /NAV/dev/apps?tenant=default&SchemaUpdateMode=recreate failed with code 422. Reason: Value cannot be null.
Parameter name: element

It would be nice if instead of the publishing suddenly stops, I get an error if RAM is insufficient (see also #1632). Also it would be nice if the details of the error would be shown (Message Unexpected event subscriber failure -- SubscriberType:CodeUnit; SubscriberId:20; TargetType:Table; TargetId:17; SubscriberCLRMethod:OnInsertGLEntry; SubscriberALName:OnInsertGLEntry; TargetMethod:OnAfterInsertEvent; Error:Value cannot be null. Parameter name: element).

Hi all, have the same issue on my D365BC Trial sandbox.
When trying to load the .app file on the "production" tenant of the trial, i get a deployment error with this detail info:
Deployment failed. Errors: App ID : <myAppID> Message : { The server "MS" was unable to process the request. The application will close. } - Job Id : 468330fa-ce95-4b8e-941e-053ae25f386c

This might be related to duplicate event subscriber names. If you're still facing this issue please try to rename your subscribers to ensure uniqueness.  Otherwise, we would need more specific repro to find the root cause.

@StanislawStempin Would it be possible to see which event subscriber that are? I have so many in my converted ISV solution.

Are you still encountering the issue on latest bcsandbox-master?  There were several issues fixed in this area and hopefully your scenario should work now.


update location and status tag

update location and status tag

forgot the update the status tag on the component, now that it is in use in the app.


Unable to use NSFWJS with File interface

I'm building a feature where users can upload files via input type file and then I want to validate images before they're uploaded to the backend.
To do that I created input and am listening to changes (selected files)
      <input
        type="file"
        onChange={handleChange}
      />

However, I noticed that the library works with the HTMLImageElement interface, while the input type file returns the File interface.
So I started playing around, looking for workarounds.
I tried to take it by creating an invisible image
      <img
        crossOrigin="anonymous"
        id="img"
        alt="img"
      />

And to put the source in it manually.
    const  selectedPicture = event.target.files[0]
    const reader = new FileReader()

    const imgTag = document.getElementById('img') as HTMLImageElement
    imgTag.src = URL.createObjectURL(selectedPicture)
    imgTag.title = selectedPicture.name

    const model = await nsfwjs.load()
    const predictions = await model.classify(imgTag)
    console.log('Predictions: ', predictions)

Unfortunately, it doesn't work as I expected. The predictions log always return a dummy response, saying everything is neutral.
Is it possible to make nsfwjs work with a file that comes from input?

You can use the FileReader API to locally load the selected image: https://stackoverflow.com/a/3814285

@MirzaLeka I'm going to close this issue for now because it's been some time since some possible fixes were provided. If you're continuing to experience this error, we'll gladly reopen this issue! Thanks.


Fix most of the clippy warnings

Some warnings haven't fixed because it breaks html! macro

bors r+

bors r+

bors r+

bors r+

bors r+


OMP thread option

What would you like to see added in fMRIPrep?
It would be a courtesy to those of us who have the environment variable OMP_NUMTHREADS firmly imprinted to make --omp-numthreads an alias for --omp-nthreads.  The error message I was getting when I used --omp-numthreads was about missing paths and not about an invalid option name.  I think the bad option was messing up positional argument processing, so the error would depend what precedes/follows the incorrect form.
If the alias isn't viable, perhaps adding a check for that and giving a better error message would help some other poor soul in the future.
I saw that a discussion had been started on the topic of threads versus procs and memory by Chris, but no discussion had started.
I'd be interested in getting some discussion going, especially to help people who run fmriprep on clusters where they have to request cores, threads, and memory and incur some kind of charge for what they request.
Thanks for considering the request re numthreads v nthreads.
Do you have any interest in helping implement the feature?
Yes, but I would need guidance
Additional information / screenshots
No response

Could you share the error you got with --omp-numthreads? I would expect it to be an argument parsing error.
That said, I'm also fine with making --omp-numthreads an alias for --omp-nthreads.

@effigies That was the problem -- there was no error message directly related to the OMP threads option.  Here is what it says.
$ fmriprep --anat-only --fs-license-file /home/bennet/freesurfer_license.txt --nprocs 2 --omp-numthreads 1 participant -w
usage: fmriprep [-h] [--version] [--skip_bids_validation]
                [--participant-label PARTICIPANT_LABEL [PARTICIPANT_LABEL ...]]
[ help text elided ]
fmriprep: error: Path does not exist: <1>.

I think it is taking the 1 intended for --omp-numthreads  as the value for bids_dir?
Any solution that results swallowing both the invalid option and its argument is fine with me.  I suspect there might be others who have or will trip on prior ingrained habit.  ;-)
Thanks for anything you can do.

Ah, yes there are parsing ambiguities when using both positional and flagged arguments. For this reason, I always place the positional arguments first and flags after.

That isn't the order that is shown by fmriprep --help, though, which lists the options in the order,
$ fmriprep --help
usage: fmriprep [-h] [--version] [--skip_bids_validation]
                [--participant-label PARTICIPANT_LABEL [PARTICIPANT_LABEL ...]]
. . . .
                [--debug {compcor,fieldmaps,all} [{compcor,fieldmaps,all} ...]]
                [--sloppy]
                bids_dir output_dir {participant}

I totally agree that mixing positional and flagged leads to..., well, this situation.
I tried a few permutations, and it doesn't seem to matter where --omp-numthreads 1 appears, fmriprep still throws the fmriprep: error: Path does not exist: <1>. error.  The issue is that all trailing arguments that haven't been parsed are assumed to be participants and have a directory corresponding to them, and since there can be more than one, there isn't really a way to catch them.
Would changing the error message slightly so it reads
fmriprep: error: Participant directory '1' does not exist

help?  At least then people would be looking for how a 1 got interpreted as a participant.
Ideally, I guess, adding an unknown option would raise an 'unknown command option' error and stop processing, maybe?

Yes, unfortunately we have limited control over how the help text is rendered or the order in which options are processed. We're using the standard Python argparse module.
I suspect that what's happening is that the positional arguments are processed first, so the error is being raised before flags are checked.

Something has to eat the options and their arguments first in order for --omp-numthreads to be removed but the numeral to remain, doesn't it?  I thought argparse could catch illegal options.  The example at https://docs.python.org/3/howto/argparse.html seems to do it.  Perhaps its that there is a variable-length list of positional arguments that is complicating things.
Might this have something to help here?
https://stackoverflow.com/questions/20165843/argparse-how-to-handle-variable-number-of-arguments-nargs
I suspect this situation would also arise in any simple typo in any other option name, as well, so might be worth trying to find a general recourse?

I'm happy to review a PR to improve error messaging. I don't foresee time to work on this myself, however.

One question, isn't the variable name OMP_NUM_THREADS ? At least that's what numpy (with openblas) will expect, for instance.

Yes, you're correct.  My mistake.  So, --omp_num_threads=?


Add support for async stream.button() in copilot

It would be handy if the stream.button() could wait for the attached command to be resolved.
When generating a participant answer for a particular command, we might encounter that a user is not connected yet. We want to expand the command palette, so users can select one of the existing connections, and then the extension would proceed with the answer generation.
Currently, we do it without the button, and just open the command palette directly, but this has proven to be confusing and difficult to notice for users.
I tried to pass vscode.ChatRequest and vscode.ChatResponseStream to my connect function and after connecting call the this._participant.requestHandler() with the same request and response. But it looks like the stream lives within one iteration user prompt -> the handler response and when I invoke the handler again the stream is already closed.
if (!isConnected()) {
  stream.markdown(
    "Looks like you aren't currently connected, first let's get you connected to the cluster we'd like to create this query to run against.\n\n"
  );
  stream.button({
    command: EXTENSION_COMMANDS.MDB_CHANGE_ACTIVE_CONNECTION_FROM_COPILOT,
    title: vscode.l10n.t('Connect'),
    arguments: [request, context, stream, token]
  });
  return { metadata: { command: '' } };
}

I also tried to solve it with await vscode.commands.executeCommand('workbench.action.chat.open', { query: chatHandlerArgs.request.prompt }), but it prints the initial user prompt the second time to the chat, what is expected, but not ideal experience for our use case. Something like await vscode.commands.executeCommand('workbench.action.chat.open', { response }) would do the job, so we could print messages to the chat at any time, without requiring a user prompt.
Please let me know if there is an existing way to handle this. Thank you!

cc @isidorn

Sounds similar to https://github.com/microsoft/vscode/issues/211136

Do you want the current request to wait on the user clicking the button, or send a new request to call back to the participant? You can do either.
You can wait on the command yourself by setting up some callback to do await somePromiseThatIsResolvedWhenTheCommandIsInvoked inside your chat handler, after sending the button.
Or, you can do the workbench.action.chat.open trick, with some other query that you want to show, even an empty string. I'm guessing the first is what you'd prefer though.

For workbench.action.chat.open we would need to return the command that we want to return control to, right? Like:
@MongoDB /query

Otherwise, it won't know which action to take.
To await somePromiseThatIsResolvedWhenTheCommandIsInvoked, I thought about setInterval with a condition as a workaround. That would work, but wanted to enquire if there is no better more native copilot way to do so, e.g. awaing stream buttons.

I'd recommend awaiting a Promise, not polling with a setInterval, especially because the user might never click the button. But yeah, either of those work, but we don't have anything better for you right now.

After experimenting a bit more, I see how this would be useful if you supported the following command:
await vscode.commands.executeCommand('workbench.action.chat.requestHandler', { command: 'query' })

This would call the required handler directly, without printing the command itself to the chat.
Here is the use case:

A user calls @MongoDB /query find all docs by name lena
We see that there is no connection or namespace specified
We show a markdown with a selection on click

Now we need somehow to tell the copilot to proceed with query generation. With workbench.action.chat.open, we would need to print @MongoDB /query find all docs by name lena again.
With workbench.action.chat.requestHandler it would not print anything and just returned a generated query.
I find it better than awaiting a promise, because as you said, what if the user didn't click anything, what if they want to call another command, what will happen with all unresolved promises? We kind of block the flow while awaiting. With workbench.action.chat.requestHandler, the user could also scroll the history and click those links again to change the selection.
What do you think?

Another command that could be useful is something like:
await vscode.commands.executeCommand('workbench.action.chat.userPrompt', { query: '@MongoDB /query' })

This could be similar to workbench.action.chat.open, with the difference that it won't send the message to chat, but only insert the text into the input, suggesting to the user the next step.

with the difference that it won't send the message to chat, but only insert the text into the input, suggesting to the user the next step.

I like this suggestion and see the need for this. Especially since we want to put the user in control to review the query before it is run.
@roblourens could it be an argument sent to the open command - send: boolean? And maybe even false is the default?

You can use the isPartialQuery argument to fill in the input but not send the query.
I wonder whether you actually want the confirmation API though, this shows a confirmation box inline, the response ends, and it makes another request when the user picks an option: https://github.com/microsoft/vscode/blob/24da11d618d76e825966709b3dd8d40de399a3d5/src/vscode-dts/vscode.proposed.chatParticipantAdditions.d.ts#L151

I don't think we should do that, because if there's a response in chat, there should be a request that shows you why that response is showing up.

I see your point. Feel free to close the issue. I will be printing to chat then with workbench.action.chat.open and keep an eye on new features that you deliver. Maybe in the future, there will be something to improve this flow.
And thank you so much for your quick and detailed responses. They’ve been incredibly helpful in moving our participant's progress forward!

Thanks, I appreciate the feedback!


[Feature] Add support datacache.partition_duration for mv

Why I'm doing:
Materialized view doesn't support to set datacache.partition_duration so far.
What I'm doing:
Add support datacache.partition_duration property for materialized view to
control the validity time of its datacache.
Fixes #issue
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[x] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

@wyb PTAL

@Mergifyio backport branch-3.2

@Mergifyio backport branch-3.1


OpenAPI

Hey,
is there a plan to output API routes / responses to Open API format so it can be easily imported to Postman or used for generating the client typescript bindings?

For now there are no plans for adding OpenAPI spec generation support out of the box.
I may reconsider it in the future since it was asked already several times (https://github.com/pocketbase/pocketbase/issues/414, https://github.com/pocketbase/pocketbase/discussions/209), but there are already a lot of other tasks piled in the roadmap and for now this will remain out of the scope for v1.0.0.
For TS types generation you may want to check - https://github.com/pocketbase/pocketbase/discussions/773.

I think this would definitely be a worthwhile investment. As you already indicated in one of the other issues parts of the documentation are already auto-documented and adapting this to a different output format should not be too hard (also there seem to be some projects which auto generate them for echo servers but I did not use any of those yet).
The advantage is that this makes it soo much easier to generate bindings for all kinds of languages which currently do not have an SDK, get automatic TS types including collections etc.


[Mutator] Capture the Beer

According to the readme, the authors are Cynical Modder and Tomcat

URL: https://unrealarchive.org/mutators/unreal-tournament/C/capture-the-beer_199b8942.html
Hash: 199b894292762d53cd5dd6124b25a4094a50ffed
Current name: Capture the Beer

Updated


se fields service.spec.externalTrafficPolicy and service.spec.healthCheckNodePort instead

**Beta annotations service.beta.kubernetes.io/external-traffic and service.beta.kubernetes.io/healthcheck-nodeport have been removed. Please use fields service.spec.externalTrafficPolicy and service.spec.healthCheckNodePort instead. **
From 1.8 notes

Fixed


issue:3445890 Add port regular expression for BF port

for port number/name like "Port B4D0F0/0/0"

Regular expression update. No functionality changes.


[CIT-598] Update Package Plugin API doc

Signed-off-by: Rina Fujino<EMAIL_ADDRESS>Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to thin-edge.io?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[x] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report

Merging #458 (5d68ba8) into main (f463f01) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #458   +/-   ##
=======================================
  Coverage   63.02%   63.02%
=======================================
  Files         112      112
  Lines        7007     7007
=======================================
  Hits         4416     4416
  Misses       2591     2591

Continue to review full report at Codecov.

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


Update README.md

LGTM!

Made a spelling mistake in the youtube link

LGTM!

What is the first comment? Is that his face?

What is the first comment? Is that his face?

Wow that's interesting...

What is the first comment? Is that his face?

I accidentally dragged and dropped from the github profile, undid it in the second commit

What is the first comment? Is that his face?

I accidentally dragged and dropped from the github profile, undid it in the second commit

ok?

?

?

what is this?

Why did you send this?

@Dellamoresteven what is this guy looking at?


Some states not working

Some states are not working:
The engine state does not change, though the api returns the correct changed state, when the button is pressed:
engineOn: false -> engineOn: true
electricOn: false -> electricOn: true
Same for the parking brake:
parkBrakeOn: true-> parkBrakeOn: false
Other states are working so I don't assume that the plugin can't communicate with the server.
To be specific, only the "If ... is" does not work:

This is working:

I am using the current version of ETS2, the current version of the plugin and the standard german page layout.

Can you try to create a new Page and create a test button to see if the state in general does nothing or only on the German edition?

It is indeed working on a new page. Did a simple test:


Change Charset Encoding

Hi,
When I try to change charsetName from windows-1252 in ThermalPrinterCordovvaPlugin.java
charsetEncoding = new EscPosCharsetEncoding(  charsetEncodingData.optString("charsetName","CP737"),charsetEncodingData.optInt("charsetId", 16));
(same applies for every other charsetName, always returns windows-1252)
The printer settings are setup to CP737 and are working properly when printing test-page.
Running getEncoding still returns windows-1252
Any ideas whay might causing this?

Hello Chris. Did you manage to solve this?
I also need to use this plugin to print in Greek so i may need your help!

Hi Aris,
Yes, All I did was i.e in printFormattedTextAndCut:
printFormattedTextAndCut(
{
type: "tcp",
mmFeedPaper: 150,
address: #######,
port: 9100,
charsetEncoding: {
charsetName: "CP737",
charsetId: 12,
},
id: ######,
printerDpi: 203,
printerWidthMM: 72,
printerNbrCharactersPerLine: 40,
},
Take a note that CP737 might also need configuration on the printer level as well depending on the model.

Thanks for your response Chris!
for my printer it was "WCP1253" and charsetId:45 (POS 80 printer).

Hi @arisSiozos, how you get the charsetId?

Hi @arisSiozos, how you get the charsetId?

By reading printer's manual. I think every printer has its own pair for charset names and Id's. In my case for most European languages  the Id is 45.

Hi @arisSiozos, how you get the charsetId?

By reading printer's manual. I think every printer has its own pair for charset names and Id's. In my case for most European languages the Id is 45.

Thanks for your reply. Too bad the printer manual doesn't have anything about the charsetId.
The model I am using is Imin POS D4. The printer is built-in and the documentation here .
Possible I can send some command or anyway to print the charset table?

Sorry for taking too long to reply. I couldnt find anything for your printer sadly. Today i tried to print Greek with another printer(POS80C), whose manual had charsetId 45 for Greek letters, but i had to use charsetId:"15" in order to work (Makes no sense).

I'm attaching this here https://github.com/DantSu/ESCPOS-ThermalPrinter-Android/issues/31
It's a list of charset ids


read_gef() with bin_size = 1 fails

When I try to read in my gef file with a bin size of 1, I get the following error message:

[2024-03-19 11:12:23][Stereo][2860691][MainThread][22949910382400][reader][1070][INFO]: read_gef begin ...
Traceback (most recent call last)
in read_gef(file_path, bin_type, bin_size, is_sparse, gene_list, region, gene_name_index)
1223 cell_names = gef.get_cell_names()
1224 cell_num = gef.get_cell_num()
-> 1225 gene_names, gene_id = gef.get_gene_names()
1226 gene_num = gef.get_gene_num()
1227 data.position = np.array(list(
1228     (zip(np.right_shift(cell_names, 32), np.bitwise_and(cell_names, 0xffffffff))))).astype('uint32')
File gefpy/bgef_reader_cy.pyx:87, in gefpy.bgef_reader_cy.BgefR.get_gene_names()
File stringsource:61, in vector.to_py.__pyx_convert_vector_to_py_std_3a__3a_string()
File stringsource:38, in string.to_py.__pyx_convert_PyUnicode_string_to_py_std__in_string()
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 0: invalid start byte

I have no problems reading in the file with any other bin size. Thanks for your help.

Upgrade gefpy to latest to clear this error.


feat/update-routing-hierarcgy

Please review before merging.

:tada: This PR is included in version 1.70.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Error in some views

There are some marbles that throw:  Invalid marble parent or parent width, which prevents them from moving sometimes. Happens not in the first place but after navigating trough some views.

Fixed in v1.2.1


Editor doesn't properly handle removal of geometry (deletions or generating a simpler mesh)

c6e1ce71dc60e3315bb4186e5e03f1b85998789f fixed for edges & faces but not vertexes


added support for new chars in number input

Added support for entering ( ) . symbols in number input. These symbols are used in USA and Canada phone numbers.

@skmsd98  Yeah as the original owner has not responded to any of the requested features. I took it upon myself to update the package and add some new features in the meantime. Including your pull request.
https://github.com/MillieOfzo/ion-intl-tel-input/commit/9a48f7075a08e6efbc47f3e3031c08162ec69b32


Refactor Testworkflow structure...

...to provide a compute.Image and a compute.Service instead of image name
Use these for determining architecture and supported OS features instead of curated lists of names when possible
Retain the partial URL attribute (renamed) as this is needed by daisy and not readily available anywhere in the compute.Image object

/test all

/cc @zmarano
Zach could you take a look at this? I'm not looking for LGTM yet, it has gofmt problems, needs to check GVNIC in more places, and I want a more comprehensive CIT presubmit before I make large changes.
I just want to get feedback on whether this is a good direction/too large to change without discussion first/etc.
I'm trying to make the Right Thing mentioned in this PR a long time ago easier to do.

Generally I think this is a great change. It also facilitates image families. It is worth updating the README or even adding a section on the ways to reference and use images, features, and architectures: https://github.com/GoogleCloudPlatform/guest-test-infra/blob/master/imagetest/README.md

/citpresubmit

/test all

/retest

/hold
This is ready for review and it's generating workflows fine but I want to make sure they are running to completion before removing the hold.

/unhold

/retest


[BUG] MSNotificationHub.setUserId exception

Describe the bug
When calling MSNotificationHub.setUserId("(userId)"), I get date1 cannot be nil Future exception.
Exception or Stack Trace
2021-01-18 15:44:35.946674+0900 SampleNHAppSwiftUI[40719:3241842] *** -[NSCalendar isDate:equalToDate:toUnitGranularity:]: date1 cannot be nil
Future exception.
A few of these errors are going to be reported with this complaint, then further violations will simply be ignored.
Here is the backtrace where this occurred this time (some frames may be missing due to compiler optimizations):
(
	0   CoreFoundation                      0x00000001a994c168 935533F2-35EE-314E-A760-E74521F68435 + 1278312
	1   SampleNHAppSwiftUI                  0x000000010429ca5c -[MSInstallation isEqualToMSInstallation:] + 332
	2   SampleNHAppSwiftUI                  0x000000010429cd6c -[MSInstallation isEqual:] + 100
	3   SampleNHAppSwiftUI                  0x00000001042ab77c -[MSDebounceInstallationManager execute] + 96
	4   Foundation                          0x00000001aacd0b20 __NSFireTimer + 68
	5   CoreFoundation                      0x00000001a98b5fa0 935533F2-35EE-314E-A760-E74521F68435 + 663456
	6   CoreFoundation                      0x00000001a98b5ba0 935533F2-35EE-314E-A760-E74521F68435 + 662432
	7   CoreFoundation                      0x00000001a98b4ffc 935533F2-35EE-314E-A760-E74521F68435 + 659452
	8   CoreFoundation                      0x00000001a98aeee4 935533F2-35EE-314E-A760-E74521F68435 + 634596
	9   CoreFoundation                      0x00000001a98ae21c CFRunLoopRunSpecific + 600
	10  GraphicsServices                    0x00000001c13b2784 GSEventRunModal + 164
	11  UIKitCore                           0x00000001ac2ecfe0 23311071-8AD7-3E8F-836A-BE5DC8A8B0DF + 12357600
	12  UIKitCore                           0x00000001ac2f2854 UIApplicationMain + 168
	13  SampleNHAppSwiftUI                  0x00000001042842d0 main + 88
	14  libdyld.dylib                       0x00000001a956e6b0 785BEE7F-AC2C-388D-865C-36279D1B3DD1 + 5808

To Reproduce
Call MSNotificationHub.setUserId("(userId)") after or before MSNotificationHub.start(connectionString: connectionString!, hubName: hubName!) initialization.
Code Snippet
func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {

        if let path = Bundle.main.path(forResource: "DevSettings", ofType: "plist") {
            if let configValues = NSDictionary(contentsOfFile: path) {
                connectionString = configValues["CONNECTION_STRING"] as? String
                hubName = configValues["HUB_NAME"] as? String

                if (!(connectionString ?? "").isEmpty && !(hubName ?? "").isEmpty)
                {
                    UNUserNotificationCenter.current().delegate = self;
                    MSNotificationHub.setDelegate(self)
                    MSNotificationHub.start(connectionString: connectionString!, hubName: hubName!)
                    MSNotificationHub.setUserId("myUserId")
                    addTags()

                    return true
                }
            }
        }

        NSLog("Please setup CONNECTION_STRING and HUB_NAME in DevSettings.plist and restart application")

        exit(-1)
    }

Expected behavior
I would expect some success return message from NotificationHub that my userId is set successfully.
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: [iOS14]
IDE : [Xcode 12.3]
Version of the Library used: v3.1.2

Additional context
Add any other context about the problem here.
I have tried moving setUserId before and after the NotificationHub initialization but had no luck.
There must be something that I missed or the instruction has missing part to set the userId correctly.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

Closed by #108

Closed by #108


fix: don't shadow remoteIPPolicies

What
The linter surfaced this problem - remoteIPPolicies was being shadowed
in the if block, so it would always be empty when DeepEqual was
called.
Not sure what this was breaking or what it fixes, but seems more correct
now. :shrug:

Current dependencies on/for this PR:

main

PR #227

PR #229

PR #230   👈

PR #228

This comment was auto-generated by Graphite.

@alex-bezek Yeah, opening a new PR for it #231


velociraptor instoling prablome

finished the whole process of installing the virtual machines through vagrent lab, and when I ran the \post_build_checks.ps1 I got the following message:
[*] Verifying that Velociraptor is reachable...
Error occured on webrequest: Exception calling "DownloadString" with "1" argument(s): "Unable to connect to the remote server"
[!] Velociraptor was unreachable and may not have installed correctly.
whet sholde i do?

Facing the same challenge here.


Tune image display on Databricks notebook again

Fix #357

[x] Tested on Databricks Notebook
[x] Tested on Jupyter Notebook

Test Cases for Databricks Notebook

Test Cases for Jupyter Notebook

from rikai.types.vision import Image

Image("s3://bucket/darcy/image.png")

This case on local jupyter notebook does not work.
It is another issue that we need to fix later at https://github.com/eto-ai/rikai/issues/435.

@da-tubi please add tests for jupyter. these are the most direct form of "user experience" features so it's important we test these for jupyter notebooks at least. ping me on slack/wechat if you want to discuss how to do it.

OK, I've mark it pull request as draft. Let me learn about jupyter first.

https://github.com/ipython/ipython/blob/f162b26d9158838e1d8051f790d877e85c7737b1/IPython/core/display.py#L991
I think I should test _repr_html_ .

The main idea of the first approach is that you have to implement special
display methods when you define your class, one for each representation you
want to use. Here is a list of the names of the special methods and the
values they must return:
- _repr_html_: return raw HTML as a string, or a tuple (see below).
- _repr_json_: return a JSONable dict, or a tuple (see below).
- _repr_jpeg_: return raw JPEG data, or a tuple (see below).
- _repr_png_: return raw PNG data, or a tuple (see below).
- _repr_svg_: return raw SVG data as a string, or a tuple (see below).
- _repr_latex_: return LaTeX commands in a string surrounded by "$",
or a tuple (see below).
- _repr_mimebundle_: return a full mimebundle containing the mapping
from all mimetypes to data.
Use this for any mime-type not listed above.

https://github.com/ipython/ipython/blob/f162b26d9158838e1d8051f790d877e85c7737b1/IPython/core/formatters.py#L925-L976
_repr_mimebundle_ is the function which is actually used.

@changhiskhan @eddyxu  ready to be reviewed


is verified issue getting overridden when user goes through applicatio…

is verified issue getting overridden when user goes through application flow
Title
ECER-{###}: Brief description of the changes
Description

issue with isVerified flag getting set to false when user saves their profile information.

Checklist

[ ] I have tested these changes locally.
[ ] Changes to backend endpoints are covered by passing integration tests.
[ ] I have added or updated the necessary documentation.

Screenshots (if applicable)
Include screenshots to demonstrate the changes visually.
Additional Comments (optional)
Any additional context or information that might be helpful for the reviewer.

We also can return IsVerified flag there if needed.

We also can return IsVerified flag there in endpoint if needed.

oh I don't think we need to in this instance. We were just setting profile information based on data from the wizard, not the endpoint.


dssat47: init at <IP_ADDRESS>

Description of changes
DSSAT is a famous tool used for crop modeling. The first release was in 1989 but this is still in use today. It's in Fortran.
https://en.wikipedia.org/wiki/DSSAT
This package is only for the CLI version.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Notes
This is for DSSAT47. DSSAT 48 also exists. Instead of naming the package dssat, I decided to call it dssat47 as this is common to have both dssat47 and dssat48 on the same system right now as some model files only work for dssat47.
I don't have yet a working package for DSSAT48 (yet).
What is the recommended way to deal with this situation? Should I have 2 folders dssat47 and dssat48 in pkgs/by-name/dss/ and use dssat47.override{} in the dssat48 package?
Or should I maybe have a dssat package pointing to latest (48), and a separate dssat47 package?

Thank you @paparodeo for the review.
That's very good points, I applied your changes. :+1:
Somehow I had issues originally just doing the default cmake configure. Must have been because of something else. It seems to be working fine now.

one thing i noticed is all the files are installed into bin which seems undesirable

Yes I completely agree. Thing is this software is super old and working in strange ways, it's extremely sensitive to where the files are.
I tried to put the files in the $out/shared folder and originally it failed, but after changing pname = "dssat47" to name = "dssat47" it works... which is really strange.
When I have pname = "dssat47", for instance it gives:
 File not found. Please check file name or create file.
File: ///nix/store/38zb9hx0mz0n051h26x57cl8   Line:     0   Error key: CO2VAL

Additional information may be available in WARNING.OUT file.

The WARNING.OUT file contains:
 CO2 input file does not exist or has invalid format.
///nix/store/38zb9hx0mz0n051h26x57cl8qkb2ik7a-dssat47-<IP_ADDRESS>/bin/Standard/CO
 Line number    0
 Program will stop.

 CO2VAL  YEAR DOY = 2022 125
 File not found. Please check file name or create file.
 File: ///nix/store/zry2zmdnndm44ci05arvnfcn
    Line:     0
    Error key: CO2VAL
 Simulations terminated.

but it works fine when I have name = "dssat47" in my package... It seems that there is an issue with the resulting path being too long maybe? :thinking:
I put the MR as draft until I investigate more.

The original program had a char path limit of 80 chars. Which was quickly an issue with the nix store paths of course.
I added a patch for it to work. This seems to be working fine on my machine.
I'm trying to merge the patch upstream, but it may take months.
Notes:

This is now DSSAT <IP_ADDRESS>, I gave up on trying to have both 47 and 48 because of the path issue and not wanting to maintain both. This is likely fine. Package is then now called dssat only.

I tried to directly use patches = [] to apply the patch, but it wasn't working because github returns patch files in unix format, while the code is formatted with CLRF (windows devs it seems). Just trying to apply with --binary was only working after the patch file was unix2dos'ed.

Result of nixpkgs-review pr 315025 run on x86_64-linux 1

1 package built:

dssat

Thanks @OPNA2608 for the review.  I've addressed all the suggested changes. Would you be willing to revisit and consider merging this one? :pray:


Can't reconnect local reader on iOS when going background on enter pin screen

Describe the bug
We are using EXPO to run the app. During the tests, we noticed that when the phone is going into the background on the enter pin screen we are not able to re-connect to the reader. We check both ^0.0.1-beta.13 and ^0.0.1-beta.14.
On ^0.0.1-beta.13 we are getting Could not execute connectReader because the SDK is busy with another command: processPayment.
On ^0.0.1-beta.14 connect to reader method hangs forever.
To Reproduce
Steps to reproduce the behavior:

Type *.03 amount (test stripe mode)
Start the TTP payment process which follows the example app
Tap the card
Go into the background when the pin screen is visible

Expected behavior
SDK should clear the state so we can easily reconnect when the app goes back active
Stripe Terminal React Native SDK version

^0.0.1-beta.14
^0.0.1-beta.13

Smartphone (please complete the following information):

Device: iPhone XS
OS: 16.6

Additional context
It's working fine when going into the background on any other scenario, like on the tap the card, etc. Also, I was trying to find any method that could cancel processPayment (version 13) or confirmPaymentIntend (version 14) but was not able to find any.

hey @dakkor71 did you get a chance to check this behavior on beta.15? what are you experiencing for that?

We were able to track this down as an issue with the underlying Terminal iOS SDK. Unfortunately it's not possible to fix this within the RN SDK, and will need to wait for an update to the iOS SDK. I'll update this ticket when that happens

@nazli-stripe No, I didn't test it on beta.15. I'm glad that you found the root problem and hope you will fix this issue soon. Thanks!

hey @dakkor71 can you test this on beta.16? this issue was fixed in iOS 3.4.0, which is the version beta.16 is built with

Hey @nazli-stripe!
Need to update EXPO to 50 and will let you know ;)
Cheers!

@dakkor71 are you running into this issue on beta.17?

@nazli-stripe Thanks for releasing beta.17. Now we are able to proceed with the EXPO update. It could take some time because we have to adjust our app to a new version but I will let you know ASAP.
Cheers

It's still not working fine or maybe I'm doing something wrong:
I can see in the logs:
 LOG  [Stripe terminal]: didRequestReaderInput ["tapCard"]
 LOG  [Stripe terminal]: didChangePaymentStatus ready
 LOG  [Stripe terminal]: didChangePaymentStatus processing

 I can see enter pin screen and then I'm clicking side button and going into background

 LOG  [Stripe terminal]: didChangePaymentStatus notReady
 LOG  [Stripe terminal]: didChangeConnectionStatus notConnected

Then going back into active state and I can still see the discoveredReaders and connectedReader returned from the useStripeTerminal hook.

If I try to retry payment using the same connectedReader it hangs on collectPaymentMethod
If I will try to connectLocalMobileReader using reader from discoveredReaders it hangs
If I try calling disconnectReader it hangs

hey @dakkor71 sorry for the delay here, and thanks for reporting it. We still have it identified as an issue with the native iOS SDK and are tentatively aiming to have it fixed in the 3.9.0 release

This has now been fixed with the release of beta.23


"Incomplete" color schemes are rejected as if they don't exist (possibly only if in fragment)

Windows Terminal version (or Windows build number)
1.11.2421.0, 1.10.2383.0
Other Software
No response
Steps to reproduce
Use a color scheme that is "incomplete" in a fragment, use the scheme name's in your default profile.
Example:
{
  "schemes": [
    {
      "name": "Test",
      "background": "#000000",
      "foreground": "#000000",
      "black": "#002b36",
      "red": "#6a4d53",
      "green": "#468666",
      "yellow": "#75896c",
      "blue": "#002b36",
      "magenta": "#6a4d53",
      "cyan": "#468666",
      "white": "#75896c",
      "brightBlack": "#002b36",
      "brightRed": "#002b36",
      "brightGreen": "#002b36",
      "brightYellow": "#002b36",
      "brightBlue": "#002b36",
      "brightMagenta": "#002b36",
      "brightCyan": "#002b36",
      "brightWhite": "#002b36"
    }
  ]
}

Drop in %LOCALAPPDATA\Microsoft\Windows Terminal\Fragments\Test\Test.json, and change settings.json's defaults: colorScheme to Test. Launch WT, get greatly misleading warning.
Notice the use of #11456 here. Else, everything seems to be listed as per the documentation on schemes.
Change "Magenta" to "Purple" on both lines, and watch the fragment suddenly be discoverable and work.
Also note: the colors in this scheme are garbage on purpose, if they're ugly, its working.
Expected Behavior
Either working behavior (with the missing colors replaced with Campbell's) and/or a warning specifically telling me what is wrong with the scheme. At no point should WT tell the user that the scheme doesn't exist without any other warnings that specifically state the scheme was found, but is incomplete.
Actual Behavior
On launch with incomplete scheme in fragment, but specified in settings.json

Notice the lack of saying the color scheme exists, but misses elements; it just claims it never existed ever, which leads to a day long wild goose chase.

If no one is assigned, maybe I look into this.

@SufyaanKhateeb Go right ahead ☺

While building the application I am running into
  D:\Windows Kits\10\Include\10.0.22621.0\shared\TraceLoggingActivity.h(439,1): fatal  error C1060: compiler is out of heap space (compiling source file Ne
wTabMenuTests.cpp) [C:\Users\sufya\OneDrive\Desktop\Sufyaan\windows-terminal\src\cascadia\LocalTests_SettingsModel\SettingsModel.LocalTests.vcxproj]
  D:\Windows Kits\10\Include\10.0.22621.0\shared\TraceLoggingActivity.h(847,1): fatal  error C1060: compiler is out of heap space (compiling source file Se
rializationTests.cpp) [C:\Users\sufya\OneDrive\Desktop\Sufyaan\windows-terminal\src\cascadia\LocalTests_SettingsModel\SettingsModel.LocalTests.vcxproj]

How much memory space does this require for building?

Like, at least 4GB. The precompiled header files for XAML are outrageously big, and they need to be entirely stuffed into memory all at once.

First-time contributor here - I'm interested in working on this issue. I'll try to get the environment set up - perhaps the error should be expanded to include what sections of which schemes are missing?

Yes, you can go ahead. I didn't get time to check it out.

I've set up my development environment (including an installation of the Windows 10.0.22621.0 SDK), but am still getting this error on launch. I'm not able to find the "retarget solution" option for the OpenConsole file and have tried a few workarounds, to no success - any aid would be appreciated.

I was able to solve the above issue by un/reinstalling my SDKs and performing a repair on Visual Studio. However, i'm now getting a different error when attempting to build with powershell:

Attempting to build with the command prompt gives the powershell version error as well.
As before, any assistance is appreciated.


Skip cleanup on deployment

Codecov Report

Merging #26 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #26   +/-   ##
=======================================
  Coverage   94.09%   94.09%
=======================================
  Files          28       28
  Lines        1084     1084
=======================================
  Hits         1020     1020
  Misses         64       64

Continue to review full report at Codecov.

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


Removed a redundant half sentence in the cold start section

Removed the statements in brackets in the following paragraph from the provision concurrency section.
Functions with provisioned concurrency have a maximum rate of 10 requests per second per provisioned concurrency. [[For example, a function configured with 100 provisioned concurrency.]] For example, a function configured with 100 provisioned concurrency can handle 1,000 requests per second. If the invocation rate exceeds 1,000 requests per second, some cold starts can occur.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for correcting the error.


Update font theme

Use new user_host_prompt to add more configurability to theme
Disable default clock being shown

Thanks!


Ecto 3.3

Hello
How to fix deps ?
Failed to use "phoenix_ecto" (versions 3.2.0 to 3.3.0) because
  ex_admin (version 0.8.2) requires ~> 3.0.0
  mix.exs specifies ~> 3.2

https://hexdocs.pm/mix/Mix.Tasks.Deps.html#module-dependency-definition-options
Specify a version of ecto as a dep in mix.exs and set override: true, i.e.:
{:ecto, ">= 3.2.0", override: true}

@yurgon have you found solution for this?
both
{:ecto, ">= 3.2.0", override: true}

and
{:phoenix_ecto, ">= 3.2.0", override: true}

cannot make it to work..
any other idea @aphillipo ?

hmm i can make it work using (latest?):
{:ex_admin, github: "smpallen99/ex_admin"},

and after that, faced this issue https://github.com/smpallen99/ex_admin/issues/400#issuecomment-332940314 with that fix

found more reliable solution here https://github.com/smpallen99/ex_admin/issues/399

Ecto is not in v3.0 guys,
Use Ecto 2.X

Ecto is not in v3.0 guys,
Use Ecto 2.X

Now it is. :D


Fil not found error in Node project

Get the following error:
fs.js:439
return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
^
Error: ENOENT, no such file or directory 'test_spurs_data.xlsx'
at Object.fs.openSync (fs.js:439:18)
at Object.fs.readFileSync (fs.js:290:15)
at readSync (/home/ubuntu/workspace/node_modules/xlsx/xlsx.js:11387:67)
at Object.readFileSync (/home/ubuntu/workspace/node_modules/xlsx/xlsx.js:11403:9)
when I require the module that uses xlsx. Lots of other people seem to be having this issue but I can't find a solution anywhere. Any help appreciated.

That error is telling you that XLSX is looking at the wrong place.  If you want to see which directory the library is attempting to read from, try writing a test file using fs.writeFileSync just before the XLSX readFile function and see where that test file appears.  I'm guessing it won't be in the directory with your desired file.

@jacka92 how you solved this issue. I am having same thing.

Sorry, used the awesome answer by @SheetJSDev and it fixed the issue. Thanks guys

Try this
XLSX.readFile(path.join(__dirname, 'test_spurs_data.xlsx'));


Add in fuzz testing/monte carlo simulation for OOM Retry

This is far from perfect. There is no documentation for the code yet. There are no nightly tests to verify that we didn't break anything.  It just is a program that you can run. I wrote it so that we can do different experiments without having to get Spark all the way updated to deal with retry. Also it should be a lot faster than trying to run thousands of spark jobs.
I am happy to add more docs/etc to this. But I thought I would put it up here anyways.

build


Pub, service invocation methods support structs without serialization

In addition to the current []byte as an input type, provide a way to pass raw structs

Assuming this would default to JSON serialization. Any objections?


SyntaxError: Cannot use import statement outside a module

I got this error after I update the version from v1.0.6 to the latest one.
It's fine if I downgrade back to v1.0.6 but wondering what causes this error.
..\node_modules\react-py\dist\index.js:1
import { PythonProvider } from './providers/PythonProvider';
^^^^^^

SyntaxError: Cannot use import statement outside a module

Can you please provide an example of how you're importing it or provide a link to a reproduction?
We're always using the latest version internally on the docs site, so wouldn't expect this issue when upgrading to the latest version.

It isn't my importation. The error seems happening in the node_modules.
node_modules\react-py\dist\index.js
For me, I'm importing it in the _app.tsx using Nextjs.
import { PythonProvider } from "react-py";
//...
<PythonProvider>
  // ...
  <Component {...pageProps} />
</PythonProvider>

Thanks for the additional info. You will see this error if you attempt to import newer versions of react-py without dynamically loading the package on the client side. react-py is not Server-Side Rendering (SSR) friendly, due to client only APIs such as web workers.
I've added a new documentation page with the required steps, Usage with Next.js. This should get you up and running with newer versions of react-py.
If this does not solve your issue, or you run into additional problems, please let me know.

That works! You're awesome!


Unable to connect and read/send messages.

The Connected event doesn't write anything to the console after starting. I changed client.ClientPrivateInformation.email and client.ClientPrivateInformation.password to client.ClientPrivateInformation.Email and client.ClientPrivateInformation.Password.  It used to work just fine, not sure what changed. I have version <IP_ADDRESS>-rc1 according to NuGet.

Try latest Dev branch build

Yup, that fixed it! Thanks!


Make strict-transport-security rule take into consideration TLD-level HSTS

From https://security.googleblog.com/2017/09/broadening-hsts-to-secure-more-of-web.html

...
The use of TLD-level HSTS allows such namespaces to be secure by default. Registrants receive guaranteed protection for themselves and their users simply by choosing a secure TLD for their website and configuring an SSL certificate, without having to add individual domains or subdomains to the HSTS preload list. Moreover, since it typically takes months between adding a domain name to the list and browser upgrades reaching a majority of users, using an already-secured TLD provides immediate protection rather than eventual protection. Adding an entire TLD to the HSTS preload list is also more efficient, as it secures all domains under that TLD without the overhead of having to include all those domains individually.
We hope to make some of these secure TLDs available for registration soon, and would like to see TLD-wide HSTS become the security standard for new TLDs.

Copying @alrra's comment from #699:

From https://textslashplain.com/2017/12/05/strict-transport-security-for-dev/:

Chrome 63, shipping to the stable channel in the coming days, things have changed. Chrome has added .dev to the HSTS Preload list (along with the .foo, .page, .app, and .chrome TLDs)

See also: https://cs.chromium.org/chromium/src/net/http/transport_security_state_static.json?maxsize=9259190&l=278-286


slogdet timing differences

Hello!
I was wondering why there is a timing difference (gpu) between jit(slogdet) and the plain slogdet expression by one order of magnitude.
From the source code, it seems that slogdet is decorated with jit already link to code.
Therefore, I would have expected no difference. Though, this is what I get.
import jax.numpy as np
from jax import random, jit

key = random.PRNGKey(0)
a = random.normal(key, (2, 2))

jit_slogdet = jit(np.linalg.slogdet)
slogdet = np.linalg.slogdet

# initial run
jit_slogdet(a)
slogdet(a)

# timeit
%timeit -n 10 jit_slogdet(a)[1].block_until_ready()
%timeit -n 10 slogdet(a)[1].block_until_ready()

Output
148 µs ± 16.4 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)
68.6 ms ± 437 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

What am I missing?
jax-version: 0.1.55
Cheers
Christian

It looks like using @custom_tranforms in addition to jit is slowing down raw slogdet. @mattjj is this an issue you're already aware of?

I wasn’t aware of this but it sounds plausible. We need to rewrite custom_transforms and this will go on the list of fixes. I might not have time until after the holidays though.
Thanks for raising this!

Yeah, it looks like custom_transforms needs to be inside a jit to avoid recompilation. With slogdet, the custom_transforms decorator is outside jit, so the inner jit is re-traced into a new xla_call in a new jaxpr each time slogdet is called, and that new xla_call becomes a new XLA compilation.

For now probably best to use another jit like you have in your code. You can use
from jax import config
config.FLAGS.jax_log_compiles=True

to see when JAX is unexpectedly compiling things.

My guess is that this eval_jaxpr in the custom_transforms logic is getting cache misses every time, for the reason outlined in #1829. However, I had to roll back #1829 because of one internal test failure (not a JAX test) that I didn't understand.
My long-promised still-vaporware rewrite of custom_transforms would attempt to avoid eval_jaxpr, so two fix options for this issue are (1) try to roll-forward #1829, (2) just wait for a custom_transforms rewrite that fixes all problems and brings about world peace.

I think this is fixed now the custom_transforms rewrite has landed!
On a V100 GPU, I get:
10 loops, best of 3: 182 µs per loop
10 loops, best of 3: 289 µs per loop


Include comments in AST

Cool! Can you please include the minified version of app.js? I.e. the one generated by npm run build.

@fkling Are you planning on making babel the default on github pages (http://felix-kling.de/esprima_ast_explorer/)? I do see #4

@hzoo: Good question. I'm planning to create a new fork of the tool which would allow selecting different parsers / languages, over here: https://github.com/fkling/ast-explorer. Alas, I haven't found the time to get around it yet.
However, as a first step, I could simply move the babel branch over there. I don't really want to make Babel the default here, since after all this is named "esprima_ast_explorer" ;) (and renaming it might be confusing).


Connection pool exhaustion from delivery thread serializing reports

Describe the bug
Bugsnag's delivery thread can checkout an ActiveRecord database connection that is never returned to the pool. This causes subsequent connections to time out waiting for a connection from the pool.
Steps to reproduce

An ActiveJob job receives an ActiveRecord instance as a job argument
The job raises an exception
Bugsnag creates a report that contains the serialized job arguments
The ActiveRecord class overrides to_s which is called when it gets serialized by bugsnag. The to_s method contains code that queries the database.
Bugsnag's delivery thread checks out a database connection from the connection pool when the to_s method is invoked by Bugsnag::Cleaner#traverse_object
The connection pool has now permanently lost a connection to the bugsnag delivery thread

Environment

Bugsnag version: 6.26.0
Ruby version: 3.0.5
Bundle version: 2.3.26
Integration framework version:

Que:
Rack 2.2.8
Rails: 6.1.6
Rake:
Sidekiq: 7.1.2
Other:

Example code snippet
# Run a single instance of sidekiq worker with concurrency 2

class Foo < ApplicationRecord
  def to_s
    Bar.first # Database query triggered during bugsnag report delivery
    "foo"
  end
end

class Bar < ApplicationRecord
end

class MyJob < ApplicationJob
  def perform(foo)
     Bar.first # Query will time out waiting for connection
     sleep 10
     raise "Fail"
  end
end

Foo.create
Bar.create
10.times { MyJob.perform_later(Foo.first) }

Error messages:

Hi @evanlok
Apologies for the delay in response, and thanks for your patience.
In order for us to investigate this further, can you please confirm whether our understanding of what you are doing in the example is correct?

You're running Sidekiq supporting 2 concurrent jobs
You're enqueueing 10 jobs to run, passing in the reloaded Foo instance
The jobs are sleeping for 10 seconds, then raising a failure
When the failure is raised, BugSnag serializes the argument, which is a Foo
Serializing the argument calls to_s on it
This makes further network requests, and doesn’t return the connection to the pool

Please elaborate on anything here that may not be exactly correct.

Yes, that is correct.

Hi @evanlok
Can you please elaborate on why you are making a database query in to_s ? We generally wouldn't recommend making a database call in such a way as to_s is a standard method in Ruby support to cast the thing you're looking at to a string representation. You wouldn't really expect this to do anything other than represent the in-memory object.

The to_s method was querying for related configuration to determine the string output. We removed the database call from the method to resolve the issue but it's possible this can happen again in the future. It was quite difficult to debug initially so if there's a way bugsnag can prevent or warn when this occurs it would be helpful.


Deprecated: Use UINib .. instead

Hello @mac-cain13 Ma
Version: 5.0.0.alpha.2
I got a lot of warning on my project like this
'MyCell' is deprecated: Use UINib(resource: R.nib.myCell) instead

Thank you so much.

You're welcome!

@mac-cain13: Can you update your code in the library to fixing this warning?

I think you should update your code according to the warning. Could you post the code you have in your project that gives this warning? You probably need to replace it with the suggested new initializer.

This is my code which always show warnning " 'myCell' is deprecated: Use UINib(resource: R.nib.myCell) instead":
 tbv.register(R.nib.myCell(), forCellReuseIdentifier: cellID)

I try to use this but some cases not work:

tbv.register(UINib(resource: R.nib. myCell), forCellReuseIdentifier: cellID) => no warnning and work as well
tbv.register(R.nib.myCell(), forCellReuseIdentifier: cellID) => get warnning
Follow into Documentation :  tbv.register(R.nib.myCell ) => error  " Cannot invoke 'register' with an argument list of type '(_R.nib.myCell)'"

Tks.
@mac-cain13

+1

The correct way to register a cell for reuse, as the documentation states, is: tableView.register(R.nib.myCell)
If you get the error Cannot invoke 'register' with an argument list of type '(_R.nib.myCell)', that means you haven't configured a reuseIdentifier in you nib.
I agree that this isn't a very useful error message from the Swift compiler. However, I'm not sure if we can do something about that.


AzCopy V10.0.7 Cannot Bind parameter 'NewerThan' error

Azcopy V10.0.7 - Win 7 64 bit and win 10 64 bit
When using AZCOPY SYNC from BLOB to LOCAL i keep getting this strange date time error

ERROR: Cannot bind parameter 'NewerThan'. Cannot convert value "True" to type "System.DateTime". Error: Invalid cast from "Boolean' to "datetime'."

Any Ideas?

This really seems like an AzCopy v8 issue; not a v10 issue.

Jeffrey Richter (watch Architecting Distributed Cloud Appshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fazure.microsoft.com%2Ftraining%2Fdistributed-cloud-apps%2F&data=02|01|jeffreyr%40microsoft.com|948cf3366acf45f6ef8d08d62e3b3b00|72f988bf86f141af91ab2d7cd011db47|1|0|636747229263047172&sdata=sg%2FtzHWpscWQDr7foOSsaNrKDP5vNSHE3b72HJG8swU%3D&reserved=0)

From: Kapanther<EMAIL_ADDRESS>Sent: Thursday, February 14, 2019 3:13 PM
To: Azure/azure-storage-azcopy<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [Azure/azure-storage-azcopy] AzCopy V10.0.7 Cannot Bind parameter 'NewerThan' error (#213)
Azcopy V10.0.7 - Win 7 64 bit and win 10 64 bit
When downloading from BLOB to LOCAL i keep getting this stange date time error
ERROR: Cannot bind parameter 'NewerThan'. Cannot convert value "True" to type "System.DateTime". Error: Invalid cast from "Boolean' to "datetime'."
[image]https://user-images.githubusercontent.com/12637127/52823524-61653480-3109-11e9-9dc7-998274bb4ed0.png
Any Ideas?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-storage-azcopy/issues/213, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACK0v1rFYwvcj7haiXLSPYbnn-Kd9EVbks5vNe1pgaJpZM4a8n5R.

Hi @Kapanther, thanks for reaching out! Could you please share the full command (please scrub away your account info)? That error indeed looks a bit strange.
On a side note, I've rewritten the sync command, to address the various issues reported on Github. It will be released soon.

How embarrassing.. was a faulty subroutine in power shell script that was called just before azcopy ran.. Please delete this thread or abuse me accordingly for wasting your time! Promise ill check more thoroughly before posting.

Hi @Kapanther, no worries! Thanks for letting us know!


re-add EPI rule.

The EPI rule was removed at some point, then re-added but not added to the config file so it wasn't being run. https://github.com/data-to-insight/quality-lac-data-beta-validator/issues/654#issuecomment-1641573773
The changes to this pull request add it back into the config file.
tweaks are made to the offline_debug.py file to make its outputs comparable to that of the lac_validator (refactored version). At the point of this writing, both the new and old validators have been seen to produce the exact same result.


Zoom Integration?

On the docs under integration and zoom plugin, it says:

This plugin requires chartjs-plugin-zoom 1.x.

And Installing the current version of chartjs-plugin-zoom doesn't work (well, it does, but the custom limit properties do not work, which I need as the default zoom limits don't do anything with this plugin). v1.3.0 of the zoom plugin is deprecated but doesn't work either anyways, and v1.2.1 is not compatible with chart.js 4.x.

I'm sorry but I'm not the maintainer of the original package. I just fork it for my own use.
Maybe you could do the same thing?

Thanks for the quick response, I'll see what I can do in a fork


Implement WriteBatchTagged

Hmm seems a prop test is failing:
--- FAIL: TestCommitLogReadWrite (0.21s)
	<autogenerated>:1:

	Error Trace:	read_write_prop_test.go:111

	Error:		Not equal: "aoszmkGGedebafoikLM0wxdxvzzt6lop0Io8whdyumed9jhhazos6taQkhGbmctBd3c" (expected)
			        != "henejf60Jtnqrxn1bFztwppgVlwdpanfiomtdsmflo6jwjxuOcvkjfjqp" (actual)

@robskillington scarily enough, that's in the commit log code path (un-changed in this PR). Opened https://github.com/m3db/m3db/issues/497 to investigate

LGTM other than minor nit checking wtype twice in write attempt call stack.


Toggle button not fully visible on Firefox

Hi! I'm Andrew and I'm new to contributing and coding in general so please let me know if I have missed some obvious or common detail. Thank you.

Bug description below:

When using the toggle button feature on Mozilla Firefox, the toggle button is partly hidden along the left/right padding at page-load. Once the user clicks on the toggle button, the full button becomes visible.

Partly hidden - Firefox 87 (desktop)
Partly hidden - Firefox 88 (android)
Normal - Brave (android)
Normal - Chrome (android & desktop)

Expected Behavior:

The toggle initial view should be same on both Firefox and Chrome.

Screenshots:

Here is a JS Fiddle showing the problem: LINK
Overview:

While inspecting element:

Additional:

I tried setting the padding from 10px to 0px (in the css input), but this caused the white middle button to be incorrectly positioned.
I am thinking maybe the way to fix this is to look in the .less file, adjust the padding rules for the "input checkbox", adjust the white button animations, recompile, and test in various browsers.
Do you guys have any thoughts? Or opinions on how to approach this?
I'd like to work on this, but not cause extra problems... this would be my first real contribution to any project.

Hello Andrew. Sorry for the late delay and thank you for your interest in contributing to sButtons!
That was a nice catch. I'll assign you to this issue. Let me know if you need any help while working on this.

Hi shahed!
No worries at all. I've been working on the issue this week and I've been getting some trouble from the lessc compiler (maybe prettier too)
I planned to make 3 changes to 2 files, in toggle.less and variables.less
When I run npm start, my changes to the .less files are not compiling as expected.  I have tried running npm start before making the changes, but then prettier will loop repeatedly without stopping.
My main problems are:

lessc is causing unexpected changes in slides 4 & 6 to the hover in the snake-btn, and to the font-face src version.

after running prettier, my git shows that 89/91 files have been modified, but no text inside the files have actually been modified.

Again, I've put the screenshots in the link below, and if you can offer any advice, I'd be very grateful.
SCREENSHOTS HERE: google slides link


Tabular Editor 3 Support?

I really like the Mini Model builder (used with TE2) but get script errors when attempting to use this in Tabular Editor 3.  Is there any Tabular Editor 3 support for this currently, and if not is any planned?  Thanks.

Ah, this is great, thanks.  I just wanted to be sure I wouldn't have to change my workflow in order to upgrade to TE3.  The mini model builder is a really great way to focus on who can access what data without duplicating measures across models, thanks for creating this!


Configure RSEs and CRIC in RUCIO

CRIC can be used as a library for RSEs configuration details (ref. https://github.com/cern-vre/project/issues/15). With a script running in a cronjob, this information can then be pulled from CRIC and imported into rucio.

[ ] Create RSEs in a new CRIC instance
[ ] Implement this script as a cronjob: https://github.com/ESCAPE-WP2/Utilities-and-Operations-Scripts/tree/master/cric-rucio-sync

@garciagenrique maybe for testing we deploy this: https://github.com/goseind/k8s-tutorial/blob/master/xrd.yaml

Two RSEs added to the CRE-VRE cluster: EULAKE-1 and LAPP-DCACHE.
Followed this documentation: https://github.com/cern-vre/cern-vre/wiki/Components#adding-rses-to-the-rucio-instance

Uploads to EULAKE-1 are working fine, we receive a warning that we don't know why it's appearing. RUCIO team contacted.
/root/envs-virtualenv/rucio-cli/lib/python3.6/site-packages/urllib3/connectionpool.py:1050: InsecureRequestWarning: Unverified HTTPS request is being made to host 'vre-rucio.cern.ch'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  InsecureRequestWarning,

It seems that the above Warning was also appearing in the escape-gitops cluster.

@egazzarr  configured CNAF-STROM in the VRE.

CESNET-S3 gives the following problem after the DB was upgraded:

$ rucio-admin rse info CESNET-S3
Database exception.
Details: An unknown Database Exception has ocurred.
Rucio exited with an unexpected/unknown error, please provide the traceback below to the developers.
Traceback (most recent call last):
  File "/root/envs-virtualenv/rucio-cli/bin/rucio-admin", line 97, in new_funct
    return function(*args, **kwargs)
  File "/root/envs-virtualenv/rucio-cli/bin/rucio-admin", line 530, in info_rse
    rseinfo = client.get_rse(rse=args.rse)
  File "/root/envs-virtualenv/rucio-cli/lib/python3.6/site-packages/rucio/client/rseclient.py", line 50, in get_rse
    raise exc_cls(exc_msg)
rucio.common.exception.DatabaseException: Database exception.
Details: An unknown Database Exception has ocurred.

Erasing all replicas and files within the RSE:

$ rucio list-rules --account=<ACCOUNT_NAME>
$ rucio delete-rule <RULE_ID>

However, some files are still there

# Connect to the PSQL DB
rucio=> SELECT * FROM replicas WHERE rse_id = '<RSE_ID>';

# To get the rse-id: (f.ex:)
$ rucio list-rse-usage <RSE_NAME>

Issue being investigated

Current status:

Investigation of participation and functioning of current/old RSEs: https://docs.google.com/spreadsheets/d/1tp3osag7GrH_8TNd3R_LkfPOdAe71LwZd82L-UHM7WI/edit?usp=sharing
Request of new endpoints for these RSEs to use with the new Rucio instance

IMO, I will take out this script because it might lead to wrong RSE configuration.
If everybody agrees we could just add a section on the Wiki explaining the main attributes and points to be configured, but without proposing a template.

CRIC will not be used atm. Ne RSE endpoints will be added as they come in.


[browserified-demo] Fails on Safari and IE

IE gives error about base64 object
Safari gives strange error at all.

Fixed most of the places. Now it works on IE9.
Still have an error in browsers:

IE6, IE7, IE8: First comes error from Codemirror, then JS-YAML fails with an error 'anchor' is null or not an object
All Safaries: Has different errors on 4, 5.x versions


GH-41005: [CI] HDFS and skyhook tests require docker compose usage because they require multiple containers

Rationale for this change
We are currently using docker run but we require linked containers to be also started.
What changes are included in this PR?
Use docker compose instead of docker
Are these changes tested?
Via archery
Are there any user-facing changes?
No

GitHub Issue: #41005

@github-actions crossbow submit test-conda-python-3.10-hdfs-*

@github-actions crossbow submit test-skyhook-integration

@github-actions crossbow submit test-conda-python-3.10-hdfs-*


[Messagesmusic] Remove unnecessary line of code

... making the app a little more reliable. Specifically while testing @rigrig's rewrite of  the Messages app.
The functionality is intact for the official Messages app.

Thanks!


Community: mention open-source communication option(s)

Feature request
Current Behavior
The only options listed are FPChat (Slack) and a Discord guild on Discord's servers. These are not open source, are hostile to alternative clients, and require community member agree to a terms of service that could change at any point behind for-profit companies. (Can read more here from Drew DeVault).
Desired Behavior
At least mention open-source alternatives as unofficial places to meet for discussions so people know where to go.
Suggested Solution
I'm not asking to take down the current options, but a mention to say #fp-ts on Libera.Chat or a Matrix Space on Matrix.org would be nice. Currently they do not exist.
Who does this impact? Who is this for?
Users that oppose the closed-source options: open-source proponents, people with low-end hardware that can't run Electron or web apps, people with disabilities that need alternative clients, people with privacy/security concerns, people in countries that these servers are censored from (e.g. US sanctions), or anti-capitalists not wanting to participate in the data-harvesting economy more than necessary.
Describe alternatives you've considered
I went to #fp-ts on Libera.Chat and saw no one. I search Matrix.org and there were no results.

This issue unfortunately transcends fp-ts. The Haskell Foundation has settled on Slack as well, although understandably their rationale was that they need to pick their battles as a community. For what it's worth I'd participate in an #fp-ts channel on IRC.
You could make a similar case against GitHub. It's created its own workflows that keep you locked into the platform rather than working in tandem with Git's native features a la Sourcehut. I believe there have been geopolitical concerns in the past as well.
Likewise copyleft licensing as opposed to MIT if you're politically aligned as I am.
Alas like the Haskell Foundation I'm picking my battles. I use Sourcehut and copyleft by default but fp-ts-std is on GitHub under MIT because on balance I'm more interested in seeing functional programming succeed.

I joined your chat as I'm a fan of irc myself. But fwiw, I'm against spreading the userbase more.

I'm in the IRC room now from the bouncer provider by Sourcehut. I don't believe spreading a userbase is a problem though, because certain people, like myself, will not join these closed communities unless absolutely necessary. Expecting it to be vibrant and lots of chatter and official support is a bit much, but even an unofficial endorsement could go a long way for the people that would participate but don't have an option that meets their needs.


Update .Net Version in Readme, Add 393.4 to GameVersion enum

I really should start deleting old branches before commits

Not your fault. I pushed that like 5 minutes ago =).

You can probably just change the readme yourself. I'll delete this PR


Prim’s Minimum Spanning Tree Algorithm | C++

Title - Prim’s Minimum Spanning Tree Algorithm
what will change - Adding prim's.cpp
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[ ] Improving Code
[ ] Improving Documentation
[ ] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[ ] Python
[x] C++
[ ] Java
[ ] C
[ ] Go
[ ] Other language

Self Check

Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Happy Coding,

Please assign this to me


[Fasthttp] panic: BUG: net.Listener returned (nil, nil)

Describe the bug
Trying to integrate netpoll with an existing fasthttp project.
To Reproduce
With fasthttp the following code got panic: BUG: net.Listener returned (nil, nil) on startup.
Minimal code to reproduce:
package main

import (
	"fmt"
	"log"

	"github.com/cloudwego/netpoll"
	"github.com/valyala/fasthttp"
)

func fastHTTPHandler(ctx *fasthttp.RequestCtx) {
	fmt.Fprintf(ctx, "Hi there! RequestURI is %q", ctx.RequestURI())
}

func main() {
	server := &fasthttp.Server{
		Handler: fastHTTPHandler,
	}

	const addr = ":12567"

	ln, err := netpoll.CreateListener("tcp4", addr)
	if err != nil {
		log.Fatalf("Create listener failed: %s\n", err)
	}

	log.Printf("Listening on %s\n", addr)
	if err := server.Serve(ln); err != nil {
		log.Fatalf("Error in ListenAndServe: %s\n", err)
	}
}

Expected behavior
Should be ok for both net and netpool.
Screenshots
If applicable, add screenshots to help explain your problem.
 With latest commit
github.com/cloudwego/netpoll v0.3.3-0.20230301065315-6704628e5d13

Hi, now netpoll cannot use like such way. As a server,  you should use NewEventLoop API to integrate your protocol layer. https://github.com/cloudwego/netpoll-examples/blob/main/server.go#L30

Thanks! Is this a intended behavior or will be improved in the foreseeable future?

@joeky888 netpoll is a trade-off between compatibility and performance. go net pefer compatibility and netpoll perfer performance. That's why it maybe cannot used simply by fasthttp or otherthing like that. But we now still commit oneself to improve the compatibility.
So, for answer your question, yes, we will let netpoll more easy to use in the future.


ntriples format does not work on version 3

Description
The dump can't be generated as N-TRIPLES format.
Steps to Reproduce
Execute ontop (version 3) like in the following example:
$ ontop materialize -m my_mapping.r2rml.ttl --format "ntriples" -o my_dump.nt-p my_configuration.properties

Expected behavior:
the file my_dump.nt should be created in ntriples format
Actual behavior:
the execution returns an error Unknown output format: ntriples
Reproduces how often:
it happens all the time
Attached material
this an example of the returned exception:
it.unibz.inf.ontop.answering.reformulation.impl.QuestQueryProcessor - Ontop has completed the setup and it is ready for query answering!
java.lang.RuntimeException: java.lang.RuntimeException: Unknown output format: ntriples
        at it.unibz.inf.ontop.cli.OntopMaterialize.runWithSingleFile(OntopMaterialize.java:175)
        at it.unibz.inf.ontop.cli.OntopMaterialize.run(OntopMaterialize.java:124)
        at it.unibz.inf.ontop.cli.Ontop.main(Ontop.java:18)
Caused by: java.lang.RuntimeException: Unknown output format: ntriples
        at it.unibz.inf.ontop.cli.OntopMaterialize$OutputSpec.createRDFHandler(OntopMaterialize.java:338)
...

Versions
Version 3
Additional Information
We should be able to add a configuration here:
SQL99DialectAdapter#L42-L56
in order to control this specific behaviour

Dear Alfredo,
Thank you for notifying the bug.
We will fix this asap.
Regards,
Julien

Hi Julien thanks very much!
we look forward for the next release then!


Ability to set orientation for yAxis added.

Ability to set orientation for yAxis of AreaChart added with this PR. This will give us possibility to make graph AreaChart with yAxis on left or right side of chart.

Thank you for the pull request! I will merge and try to make a new release tomorrow. If you have any other ideas you want to discuss or add feel free to send another pullreq!


Implement User Authentication

Develop user registration and login functionality using JWT authentication.

This is working now

Yep, I think it's working!


fix: remove upper limit for required terraform version

/run pipeline

:tada: This PR is included in version 4.13.3 :tada:
The release is available on:

GitHub release
v4.13.3

Your semantic-release bot :package::rocket:


Subdependency issue when using with Nuxt 3.8.0

Describe the bug
I am facing the following exceptions when using Vue Datepicker with Nuxt 3.8.0:
[vite-node] [ERR_LOAD_URL] date-fns/parse/index
Cannot find module '../_lib/defaultLocale/index.js' Require stack: - S:\Zytro\customer-dashboard\frontend\date-fns\parse\index
The first one happens when I try to run the project with Vue Datepicker. The second one occurred once I installed the<EMAIL_ADDRESS>on my project (as dev and not dev dependency).
My Vite version is 4.4.9.
I am using SSR.
To Reproduce
Steps to reproduce the behavior:

Start a Nuxt 3.8.0 project
Install Vue Datepicker
pnpm dev
See error

Expected behavior
I was expecting it to not throw an exception on Vue Datepicker dependency.
Desktop & mobile (please complete the following information):

Windows 11, Brave
7.2.1

I really appreciate using Vue Datepicker. I am also not sure if that is a problem related to Vue Datepicker, or if should I configure something else on Vite.
Thank you for your time.

I workaround it by downgrading Vue Datepicker to 7.1.0

7.2.0 is the latest working version.. its the 7.2.1 changes that brought this bug


u() Vs unicode()

There are a few places in the code (https://github.com/CivicKnowledge/ambry/blob/ambry1.0/ambry/bundle/files.py#L223) where:
return u(',').join(u(e).replace(',', '\,') for e in v)

The u(e) is really required because e can be either a string or an int, and if it is a string, the replace needs to be run. But in python2, u() does it's own replacement first, which fails when e is an int.
What should actually be there is something that is equivalent to uniccode() in python2, and str() in python3

What about:
from six impor u
temp = lambda v: u(',').join(u('{}').format(e).replace(',', '\,') for e in v)

Some test:
python2
# list of strings
>>> temp(['a,b']), type(temp(['a,b']))
(u'a\\,b', <type 'unicode'>)

# list of ints
>>>  temp([1, 2]), type(temp([1, 2]))
(u'1,2', <type 'unicode'>)

python3
# list of strings
>>> temp(['a,b']), type(temp(['a,b']))
('a\\,b', <class 'str'>)

# list of ints
>>>  temp([1, 2]), type(temp([1, 2]))
('1,2', <class 'str'>)

Is it required behaviour? This method is described here and is 2to3 compatible (that means code formatted this way will not be changed by 2to3 util.)

On Sep 15, 2015, at 2:06 AM, Kazbek<EMAIL_ADDRESS>wrote:
What about:
from
six impor u
temp
= lambda v: u(',').join(u('{}').format(e).replace(',', ',') for e in v)

Ah, yes, format() would do fine.
eric.

Eric Busboom<PHONE_NUMBER>
<EMAIL_ADDRESS>http://www.busboom.org


wasabi.sh and container.sh require bash 4.x

When running ./bin/wasabi.sh start:cassandra,mysql (or any other start sequence) I get syntax errors:
./bin/wasabi.sh: line 317: syntax error near unexpected token `&'
./bin/wasabi.sh: line 317: `    start) command="start:cassandra,mysql,wasabi";&'

I am using zsh (version 5.2) but also tried to run it with the vanilla OSX bash shell (version 3.2.57(1)-release from 2007), same error applies. Neither bash 3.2 nor zsh 5.0.8 support ;& as fall through, only bash 4(.3 linked).
It's not only wasabi.sh but also in container.sh.
The problem is also described on stackoverflow. To make the script more portable, consider portability and POSIX compliance.
For now I will be using an older script (git checkout origin/feature/experiment_pagination -- ./bin/wasabi.sh ./bin/container.sh).
Update: To use the script with zsh one has to do it explicitly: zsh ./bin/wasabi.sh thanks to the shebang line. However == still causes problems then. So it is probably the best idea to get bash 3.2 upgraded automatically.
Update 2: After posting this issue to the zsh mailing list Bart Schäfer recommended in a private mail to use the following: ARGV0=sh zsh wasabi.sh, which seems to solve the problems. I will now alias this in my rc file.
However that doesn't solve the bash 3.2 issue, so I will leave this open for now.

After the recent rollback, this is no longer a real issue. But the scripts are under surveillance.


Only Test and ITCase suffixes

a package with tests may only have classes with Test or ITCase suffixes that map one-to-one to live classes and nothing else.

Maybe we need to create an issue for that check.

@volodya-lombrozo 2 puzzles #215, #216 are still not solved.

@volodya-lombrozo 2 puzzles #215, #218 are still not solved; solved: #216.

@volodya-lombrozo the puzzle #218 is still not solved; solved: #215, #216.

@volodya-lombrozo all 3 puzzles are solved here: #215, #216, #218.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.2 to before v5.10.85",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ae6d9ce05691bf79694074db7c7da980080548af",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=db12d95085367de8b0223929d1332731024441f1"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ae6d9ce05691bf79694074db7c7da980080548af",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "db12d95085367de8b0223929d1332731024441f1",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "iio: mma8452: Fix trigger reference couting\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.85 by commit db12d95085367de8b0223929d1332731024441f1, it was introduced in version v4.2 by commit ae6d9ce05691bf79694074db7c7da980080548af. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002519


[Feature Request] Set an "Idle Timer" to cancel out the "Ctrl + Hover" shortcut

It would be great if an idle timer for the mouse cursor could be set to deactivate the Ctrl + Hover shortcut to open any kinds of links.
It happens quite often that I navigate around a document using Ctrl + Arrow Keys and if the mouse cursor just happens to be hovering a link somewhere a hover editor window pops open. Worst case then is an accidental edit of another file, as I usually open hover editor in edit mode.
Cheers :)

Have you tried turning off auto-focus in Hover Editor's settings?

I actually prefer using hover editor with auto focus turned on, as a quick way to edit reference links and embeds.

The idle timer is an idea worth considering, though it still has the problem of "how long is too long", as well as the problem of what if you just bumped the mouse or you just scrolled a link under the cursor.  That is, even if the mouse stayed in place, scrolling content underneath it will still register as hovering.  (There are ways to counteract that as well, of course, but this is all very complex and the code is already quite complex.)
What I would suggest in the meantime is that you assign an additional hotkey to Pane Relief's "go to last pane/tab" command, e.g. ctrl+space or some such, and then turn off auto-focus.  Then, when you want to focus on a new hover editor, hitting that ctrl-space will do the trick (or of course clicking), and your keys will never go to an inadvertently-opened hover editor.