Unable to run tutorial

I'm attempting to follow this tutorial and got the following error when I ran
npm start on the project for the first time after updating the API keys as mentioned in the video around the 20:45 mark :
><EMAIL_ADDRESS>start
> npm run develop

><EMAIL_ADDRESS>develop
> gatsby develop

info We're shipping new features! For final testing, we're rolling them out first to a small % of Gatsby users
and your site was automatically chosen as one of them. With your help, we'll then release them to everyone in the next minor release.

We greatly appreciate your help testing the change. Please report any feedback good or bad in the umbrella issue. If you do encounter problems, please disable the flag by setting it to false in your gatsby-config.js
like:

flags: {
  THE_FLAG: false
}

The following flags were automatically enabled on your site:
- DEV_SSR · (Umbrella Issue (https://gatsby.dev/dev-ssr-feedback)) · Server Side Render (SSR) pages on full reloads during develop. Helps you detect SSR bugs and fix them without needing to do full builds.

success open and validate gatsby-configs - 0.253s
success load plugins - 3.118s
success onPreInit - 0.032s
success initialize cache - 0.007s

 ERROR

Unable to copy site files to .cache Source and destination must not be the same.

  Error: Source and destination must not be the same.

  - stat.js:80
    [agility-gatsby-starter]/[fs-extra]/lib/util/stat.js:80:17

  - stat.js:38
    [agility-gatsby-starter]/[fs-extra]/lib/util/stat.js:38:16

  - polyfills.js:295 callback
    [agility-gatsby-starter]/[graceful-fs]/polyfills.js:295:20

not finished copy gatsby files - 2.616s

Running gatsby clean then npm run develop seemed to fix it for me. Hope this helps someone else.


Download

This is more or less a proof of concept. Needs a vignette and some more error handling but we'll call it done for now.

Coverage decreased (-0.08%) to 96.708% when pulling fdbd8002850952ab6ed8209faf9ca269485cc30b on download into 77e8462b86b78addcad6bf4cc3f6a9d88851b72e on master.


Add automatic test project support for forks

Now the buildTestProject gradle task will actually generate a proper test project based on what has changed between your local changes and the upstream / main.  Previously, this was only working with branches off the main repository.  A significant oversight on my part 🤦

@bjhham I don't think I have enough context here, can you explain in a little detail what happens here?


Snake Chrome Extension

Snake-Chrome-extension
Very basic chrome extension that let's you play snake with arrow keys (up, down, left, right) in a popup activated by clicking the extension icon.
Highscore gets saved in chrome.storage.sync, which means it's stored in the cloud (maintained between different chrome instances logged into same google account).
Game is reset by closing and reopening the popup and no pause function is implemented (yet).
Written with Pure JavaScript, CSS and HTML snake
Game loop:

check for collisions (wall, food, snake itself)

if collision with wall - end game
if collision with food - add new node, spawn new food
if collision with itself - end game

advance snake (delete last node, add new)
check if end of game (spawned enough food to end the game)

#Screenshots

#Video Demo
https://user-images.githubusercontent.com/81406458/159215333-ddb85e5f-3496-4d8a-a711-25c72c11101c.mp4

@Krytax-24 good work
Just do one thing add pictures and video in readme file also

@Krytax-24 good work Just do one thing add pictures and video in readme file also

Done!

Great, Merging the PR 🚀
Thanks for Contributing 💙

Kindly tell the issue number

Kindly tell the issue number

208


Get remote builder with organization

@billyb2 This seems like a lot of extra info to pull for every GetApp(). I'm not sure what the performance implications would be for that. It might be better to have an explicit GetRemoteBuilderApp() call.


DO NOT MERGE: Fix policy loading during rcheckin

Error loading policies when using rcheckin that do not occur when using checkintool. Avoiding use of Assembly.LoadFrom, as recommended by the Managed Debugging Assistant, fixes the problem.
Example of errors:
$ git tfs version
git-tfs version <IP_ADDRESS> (TFS client library <IP_ADDRESS> (MS)) (64-bit)
C:\Program Files\GitTfs\git-tfs.exe

Note: If you want to force git-tfs to use another version of the tfs client library,
set the environment variable `GIT_TFS_CLIENT` with the wished version (ie: '2013' for Visual Studio 2013,...)

$ git tfs rcheckin -w12345 -c "Mr. Reviewer"
Working with tfs remote: default => $/Project/Example
Fetching changes from TFS to minimize possibility of late conflict...
Commit was authored by git user John Doe<EMAIL_ADDRESS>(DOMAIN\jdoe)
Starting checkin of 53cf92c4 'Example commit'
 edit .git\~w\Test.cpp
[ERROR] Policy: Internal error in Changeset Comments Policy. Error loading the Changeset Comments Policy policy (The policy assembly 'Microsoft.TeamFoundation.PowerTools.CheckinPolicies.ChangesetComments, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' is not registered.). Installation instructions: To install this policy, follow the instructions in CheckForComments.cs.
[ERROR] Policy: Internal error in Work Items. Error loading the Work Items policy (The policy assembly 'Microsoft.TeamFoundation.VersionControl.Controls, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' is not registered.). Installation instructions:
No changes checked in.

During debugging with break on exceptions the following information was provided:
Managed Debugging Assistant 'LoadFromContext' has detected a problem in 'C:\git\git-tfs\GitTfs\bin\Debug\git-tfs.vshost.exe'.

Additional Information: The assembly named 'Microsoft.TeamFoundation.Client' was loaded from 'file:///C:/git/git-tfs/GitTfs/bin/Debug/GitTfs.Vs2015/Microsoft.TeamFoundation.Client.dll' using the LoadFrom context. The use of this context can result in unexpected behavior for serialization, casting and dependency resolution. In almost all cases, it is recommended that the LoadFrom context be avoided. This can be done by installing assemblies in the Global Assembly Cache or in the ApplicationBase directory and using Assembly.Load when explicitly loading assemblies.

If there is a handler for this exception, the program may be safely continued.

After getting the build from AppVeyor I tried using it for a checkin but got the following error:
$ git tfs ct
Working with tfs remote: default => $/QC-CAT/Main
 edit .git\~w\Test.cpp
Constructor on type 'Microsoft.TeamFoundation.VersionControl.ControlAdapter.CheckinDialog' not found.

@kgybels - thanks for doing some extra QA around this. :)

Constructor on type 'Microsoft.TeamFoundation.VersionControl.ControlAdapter.CheckinDialog' not found.

There's another Assembly.LoadFrom in the middle of the code that builds a checkin dialog. I don't know if that's related or not. Also, there are some TFS DLLs bundled with git-tfs these days. I wonder if those are related?

@spraints I will try updating to the latest version of the Microsoft Team Foundation Server Client package tomorrow, maybe that fixes the problem with the policies.

Updating the TFS client libraries fixes the problem. This is just #959 that came back because of Visual Studio 2015 Update 3.


[CLOSED] Use phase correctly

 Issue by tommyp
Wednesday 02 September 2015 at 16:22 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/553

With the changes in https://github.com/alphagov/finder-frontend/pull/230
we now use the phase attribute from the top level of the Content Item
hash. This commit modifies the check to merge this if it is present in
the metadata along with adding a test to test this behaviour.

tommyp included the following code: https://github.com/alphagov/specialist-publisher/pull/553/commits

 Comment by mikejustdoit
Thursday 03 September 2015 at 10:35 GMT

Other than the minor comment, looks good!
Nice to see some context in the commit message :+1:


[medViewContainer] scene, allow to keep patient info

From this issue https://github.com/Inria-Asclepios/music/issues/399 ("Imported scene has anonymous data")
It allows to keep patient name, patient ID, birthdate, gender, and create a dedicated "Scene" study description. If you save this loaded scene, it will be stored in the original patient tree.
Example:
<!DOCTYPE xml>
<scene>
 <layeredView path="view{81bfdeb6-aabd-4747-9630-d95bd6fdc056}/mapping.xml" id="medVtkView"/>
 <patient gender="M" patientID="cc6405a1-5f69-416c-a13d-f21b36514d19" birthDate="19500929" studyDescription="Scene" patientName="Belzebuth"/>
</scene>

I save the patient information in the globalMapping.xml file, as an independent QDomElement. @LoicCadour we'll have to check if it's ok on the Web Viewer.
Works on scene, and from the "Export to Viewer" button in pipelines.
:m:

I actually tried this one to test on the Viewer and I got a crash when loading the new scene.
I couldn't debug much because I am on VS.
It loads everything then, when entering the new code, one file is renamed ( the others seem to be removed from the db ..??) and it crashes.

oki, thanks, we'll have to look at it

@LoicCadour It should be fine with scenes exported from pipelines and imported in music :)

What was the pb in the end?

There is indeed a question about that issue/PR.
Do we consider a (previously "Scene") batch of vtk and xml  from the pipelines, exported through "Export to Viewer" as a real Scene? I mean, if the user wants a proper Scene at the end of the pipeline, there is the Scene button in the settings button of the view/container.
In real Scenes, the metadata are kept since few months in our code. This issue was only relevant for pipelines. So, if we do not want anymore that the users import "pipelines results files for the Viewer" in MUSIC, this PR is not relevant.
An other solution would be to keep that PR but not the medViewContainer::loadScene() changes. In this situation, even xml/vtk files exported in Pipelines through "Export to Viewer" would have metadata in the xml, which could be used in the Web Viewer, or something like that. It could be interesting.

OK, maybe let's put this PR on standby and let's talk about that at the next meeting.

We talked about this PR in the last meeting. The idea is indeed that these files exported at the end of a pipeline to use with the Viewer are not really "Scenes". However, it could be interesting to add metadata in the xml as done. I'll adapt this PR

Done, this PR allows to saves some patient metadata in the xml files, which could be used later in the Viewer.

As said on meetings yesterday, to validate a PR we'll need a validation from a "core" dev (the old ones ^^, @fcollot and I) and the other dev can review it if they want to.
I merge this one


Bug in TypeScript Templates

There are some issues concerning the provided service app in TypeScript template:

post-deploy.ts and pre-undeploy.ts should start with
import * as dotenv from 'dotenv'; dotenv.config();
in the switch statement of the service.controller.ts, the cases should be payment and order instead of payments and orders
the dependency "@tsconfig/node16" is missing from package.json so that "extends"<EMAIL_ADDRESS>from tsconfig.json works

Hi @mbe1987, thank you for opening this issue.
Regarding the first tick on the issue list, what is your motivation to use .dotenv library in the post-deploy.js script?
Thanks,
Rui

Thank you @mbe1987 for reporting those issues. We have made the fixes in https://github.com/commercetools/connect-application-kit/pull/57.


Moist Held-Suarez is qualitatively different between CPU and GPU

See stand-alone Moist held-suarez atmos runs here:
https://buildkite.com/clima/climacoupler-longruns/builds/514
Running with this script, which mimics the atmos driver.
Some plots (100d, i.e. 10 10-day averages):
Zonal mean wind

Zonal mean temperature

1st level temperature

Instantaneous 100d rhoe_tot (first level)

I think that the first thing that we should do in narrowing down this issue, is making sure that we get the same result within machine precision between CPU and GPU. A few places where we could have differences is where the order of operations could differ between the two implementations, in particular, reductions and DSS.

Thanks, @charleskawczynski , this sounds like a good way forward.
@juliasloan25 has already done some machine precision tests of atmos states, and she's seen departures after the first timestep (https://github.com/CliMA/ClimaCoupler.jl/pull/614). I don't think we've looked any deeper yet though. Do we already have any CPU-GPU consistency tests in ClimaCore? I thought someone's mentioned them recently but I'm not totally sure where to look.

Here is one place in ClimaCore where I'd be surprised to see bitwise equality:
function reduce_cuda_blocks_kernel!(
    reduce_cuda::AbstractArray{T, 2},
    op,
    ::Val{shmemsize},
) where {T, shmemsize}
    blksize = blockDim().x
    fidx = blockIdx().x
    tidx = threadIdx().x
    nitems = size(reduce_cuda, 1)
    nloads = cld(nitems, blksize) - 1
    reduction = CUDA.CuStaticSharedArray(T, shmemsize)

    reduction[tidx] = reduce_cuda[tidx, fidx]

    for i in 1:nloads
        idx = tidx + blksize * i
        if idx ≤ nitems
            reduction[tidx] = op(reduction[tidx], reduce_cuda[idx, fidx])
        end
    end

    blksize > 32 && sync_threads()
    _cuda_intrablock_reduce!(op, reduction, tidx, blksize)

    tidx == 1 && (reduce_cuda[1, fidx] = reduction[1])
    return nothing
end


Cherry-pick Do not match non vars in inner plan's target for LASJ_NOTIN etc

Include single node test modify.
Fixes #ISSUE_Number
What does this PR do?

Type of Change

[ ] Bug fix (non-breaking change)
[ ] New feature (non-breaking change)
[ ] Breaking change (fix or feature with breaking changes)
[ ] Documentation update

Breaking Changes

Test Plan

[ ] Unit tests added/updated
[ ] Integration tests added/updated
[ ] Passed make installcheck
[ ] Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

[ ] Followed contribution guide
[ ] Added/updated documentation
[ ] Reviewed code for security implications
[ ] Requested review from cloudberry committers

Additional Context

CI Skip Instructions

Better not.  In principle, we should ensure that every commit  pipeline passes.


Use slog in a backward compatible way

Go SDK will be used in other projects, thus it makes sense to use standard structured logging instead of our own one.
https://github.com/livekit/protocol/pull/668 added support for slog to the protocol's logger package. This change propagates it to the SDK.
For our own projects, calling logger.SetLogger will automatically set slog logger as well, making this change backward-compatible.
Also, add SetLogger to the most important structs of the package, allowing to associate additional metadata with RTC logs.

Did we have to change all of the logger usage internally to slog?

Only in the SDK, yes.

It breaks a certain consistency within our Go codebase. Everywhere else we are using protocol logger

This depends on if we will adopt it long term. It could be just a temporary inconsistency.
In practice (if I understand correctly), we mostly use Zap under the hood, which has slog adapter as well. So in general we could decide to move to slog as a main log interface eventually. That would remove the need to maintain our own logger interface. There are too many loggers in the wild already :sweat_smile:
But that's just as an option, it's not directly related to the PR.

We would lose features, particularly with WithValues, WithComponent, etc

WithValues is supported: slog.Logger.With. WithComponent could be a wrapper for With("some-predefined-key", name+component) in our logger package. I will make sure it plays nicely with Zap if we decide to switch.
We do lose Zap-specific things like WithItemSampler. But I think it could also be a wrapper in logger package that asserts that the slog handler is Zap and reconfigures it accordingly. Something along the lines of:
package logger

func WithItemSampler(log *slog.Logger) *slog.Logger

If the underlying logger is not Zap it will just return the same logger with no changes. This will work for Logr, for example, but also for any other log package out there.
So it's definitely worth a separate discussion about the long term plans. I could make a separate DD/proposal with all the details that we can discuss.
But for now this PR only changes client-facing SDK to use the logger from Go stdlib. I think eventually most libraries will converge to using that interface anyway, so I wanted to see how invasive it would be to switch. Turns out it's not.


Fix memory leak issue in linearize

This PR adds an argument detach to linearize and make_empirical_fisher_vp to avoid an odd memory leak when gradients are not detached in make_empirical_fisher_vp. In case people would like to autodiff through linearize (e.g., for higher-order influence function computations), then detach should be set to False.
Resolves #516.

@agrawalraj , I'm not seeing a test that confirms that this addresses the memory leak issue. Could you please add a test that shows: (i) when using detach=False the memory footprint increases for two successive calls to influence_fn and (ii) when using detach=True the memory footprint does not increase for two successive calls?

@SamWitty I added test_memory_leak.py to track how memory changes when detach is True or False. I didn't make it a formal test because this issue might be specific to torch.func and could be resolved in a future release of torch.func. The memory leak when detach = False is weird because I'm not saving any variables in the loop...

@SamWitty I added test_memory_leak.py to track how memory changes when detach is True or False. I didn't make it a formal test because this issue might be specific to torch.func and could be resolved in a future release of torch.func. The memory leak when detach = False is weird because I'm not saving any variables in the loop...

Fair enough. I think a good approach then is to just add a version with detach = True to the tests, making sure that it doesn't accumulate a memory footprint. This way we'll be able to confirm that this actually addresses the memory bug, while having the tests be robust to upstream changes in torch.func.

Can this be addressed using the torch.no_grad() context manager without adding a flag to the API?

Can this be addressed using the torch.no_grad() context manager without adding a flag to the API?

I tried adding this flag and that works! Should we just close this PR then? My only concern though would be people might not know to use torch.no_grad() and then might run into this memory issue.

@SamWitty I added test_memory_leak.py to track how memory changes when detach is True or False. I didn't make it a formal test because this issue might be specific to torch.func and could be resolved in a future release of torch.func. The memory leak when detach = False is weird because I'm not saving any variables in the loop...

Fair enough. I think a good approach then is to just add a version with detach = True to the tests, making sure that it doesn't accumulate a memory footprint. This way we'll be able to confirm that this actually addresses the memory bug, while having the tests be robust to upstream changes in torch.func.

Good idea. I changed the file to explicitly test with detach=True. However, we might abandon this PR b/c you can avoid this memory issue w/ torch.no_grad()

Closing this PR in favor of #519.


fixed nltk.grammar.cfg_demo() Nonterminal division error

from __future__ import division
makes slash ('/') division of non-terminals (and hence cfg_demo()) break.

@aetilley: thanks for the fix. Unfortunately, we need to find another way to solve this, to preserve the slash notation of categorial grammar. I wonder if we can drop the from future import __division__ line instead. Would you mind updating your PR, and also including a regression test, and we can see if this works?

@stevenbird
I think removing the future import of division would be risky since it might break other numerical operations for python 2 users.
I have found that including a truediv operation which uses div will allow the slash operator to work correctly.  I will submit a pull request.

@stevenbird:  Wrt above, see https://github.com/nltk/nltk/pull/1660

Thanks @aetilley – much appreciated!


DHT22

Hi,,
I couldn't find the configuration for the binary sensor DHT22.
How can i configure my DHT22 which is connected to GPIO 04 in Raspberry pi?

This replaced the GPIO integration, the DHT22 is a different integration. For now I try to focus on hardware that I have and can test, it would be better to wait for a developer which use DHT22 to take maintain of the DHT22 integration.

thanks for the response,, well I'm connecting (wiring) the DHT22 through raspberry pi GPIO.
So will this work for now, or we still need this to be developed?

It will not work unless a developer picks it up and add it as a custom integration.

clear,,
so nothing else to do now except waiting for someone to get this developed... :(
thanks.

same problem here, any news

same problem here, any news

News will be when a developer picks it up, I am only maintaining the GPIO integration: https://www.home-assistant.io/integrations/rpi_gpio/


Add support for C#/HTML CompletionList.Data and CommitCharacters

Update LSP protocol version and add support for CompletionList.Data wrapping in our HTMLC# completion handler.

Part of dotnet/aspnetcore#31852
Part of dotnet/aspnetcore#31883
Part of dotnet/aspnetcore#31886
/cc @ToddGrun @jimmylewis Down for another dual insertion to consume the SumType/CompletionList optimizations?

/cc @ToddGrun @jimmylewis Down for another dual insertion to consume the SumType/CompletionList optimizations?

Sure, when are you looking to insert?

Sure, when are you looking to insert?

Was thinking Monday


KE-10001 ci test

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

retest this please

retest this please


icons resources for licenses are missing

it would only work for Creative Commons, and even in that case it's not working anymore
STR

create a new public dataset
asign a license in the metadata (i.e. CC by attribution alone)

expected behaviour

license icon to appear, related to the license chosen, as in the capture below

actual behaviour

no icon appears, not for CC, not for anyone else, as a compromise solution we can make the icon default to CC, but ideally it should relate to the license or display a more generic one

Can you provide more background on this please?

sure, on public datasets with license set in the metadata, the icon should be related to the license in each case (see attached an example with MIT which defaults to CC icon)

we could also use a generic icon for all licenses, otherwise it would require producing a new icon for each license

What about this icon? As a generic one:

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.

this is still happening

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.

this is still happening


 Also, if you select something completely different in the Tekla window than to what was saved then those objects are sent instead. It seeems the selected objects are not saved and/or not loaded when sending using the saved sender unlike in Revit.
Speckle for TeklaStructures 2020 & 2021 v2.8.0-alpha2

@Reynold-Chan is there a PR associated with this issue? The status is set to Needs Testing, but I don't see where the issue has been referenced in a PR.


Keep old schemas available?

Recent changes have removed the older schemas from the master branch. They're also not hosted on http://schemas.elite-markets.net/eddn. This makes finding an authoritative copy of the older schemas much harder than it should be for when tools transmit data using the older schemas. It would be preferable if the older schemas remained available. I understand the desire to push producers to new schemas, but consumers still have to deal with older data. At present, I have to look back through the commit history and hope I find the schemas as were published. Assistance appreciated.

The network no longer allows messages sent in these old schemas, so they're not useful.
Or do you have some other need for them?

I host an archive of the messages. The older messages refer to the older schemas. I can also start archiving the schemas but that wasn't really anticipated.

I take your point. I'm not sure why we started deleting obsolete schemas.
Here's the last version of the commodity/1 schema: https://github.com/jamesremuscat/EDDN/tree/8fa93fbb409a91eb5d7dee42f6cf86e940e4aa51/schemas
And of the commodity/2, outfitting/1 and shipyard/1 schemas: https://github.com/jamesremuscat/EDDN/tree/2d6e7d9f3827fb65a5a969ed5b1cb141c3078ff2/schemas

Just a note that commodity/1, commodity/2 and shipyard/1 are available at their canonical locations - that is, the location the schemas claim to be from, e.g. http://schemas.elite-markets.net/eddn/_schema_/_version_.
Newer versions and other schemas aren't there because, well, nobody's asked me to put them there...

I recall having asked for that, but the email stayed without any response,
GitHub should suffice to see them anyway.


message for dataset not in external repo

Describe the improvement
Instead of "If your dataset is not yet ..., you can upload your dataset to OSF.", include "If your dataset is not yet in an external repository, head to the RDM website for instructions on data sharing.
This url needs to be updated to point to the specific section on data repositories.
Screenshots

I would expect the link to go to some actionable practical steps, is there such a page? Way too much other information on that page.

The documentation on this page will be updated and then we can point to a more specific section of this page. I will provide a new link as soon as it is available.


Hyperparameter tuning :  Training script command job runs indefinitely

Module: Perform hyperparameter tuning with Azure Machine Learning
Lab/Demo: 09/ Hyperparameter tuning.ipynb
Task: Tune hyperparameters with a sweep job
Step: Configure and run a command job in Hyperparameter tuning.ipynb
Description of issue
Commnad job runs indefinitely - remains in Preparing status, does not transition to running status, nor complete status
Repro steps:

perform all steps given here: https://microsoftlearning.github.io/mslearn-azure-ml/Instructions/09-Hyperparameter-tuning.html
Issue will be encountered in the following section of notebook Configure and run a command job, prior to defining sweep and running sweep job.

I also get stuck in this notebook. The compute cluster never scales up even if the sweep job is submitted.

Hi,
I re-ran the notebook successfully.  The train job ran for about 5 min and the sweep job ran for about 10 min.
This issue should be closed.
Thanks,
orod


FR : A way to disable runtime errors in OJS for R functions defined in webr

I've discovered the webr/ojs interactivity possibilities. It's nice. I've tested something like that :
```{webr}
#| edit: false
#| output: false
#| define:
#|   - ok_reponse
library(htmltools)
ok_reponse <- function(reponse, n) {
  if (is.na(reponse)) HTML("")
  else if (reponse == n) div(HTML("Bonne réponse ✓"), style = "color: green")
  else div(HTML("Mauvaise réponse ✗"), style = "color: red")
}
```

```{ojs}
//| echo: false
viewof reponse = Inputs.radio(
new Map([
["On n'utilise que des procédures (des fonctions sans valeur de retour).", 1],
["On n'utilise que des fonctions dites pures, c'est-à-dire des fonctions purement combinatoires.", 2],
["On n'utilise que des fonctions, pures ou non.", 3],
["Différentes étapes successives qu'on appelle fonctions. On les applique sur des objets qu'elles modifient progressivement.", 4]
])
);
ok_reponse(reponse, 2);
```

This is a funny way to enable easy interactivity with R. However before the loading, the R functions lead to OJS runtime errors. Is there a way to disable these ones ?
So far  I've been able to get the intended result by writing an additional ojs cell.
```{ojs}
function ok_reponse(reponse, n) {
  return html``;
}
```

This writes nothing and gets replaced when webr is loaded. Hence avoiding runtime errors.
However, it looks like an ugly patch. Is there, or could it be, a good practice to do the same thing ? Like a default value for a definition ?

However before the loading, the R functions lead to OJS runtime errors. Is there a way to disable these ones ?

No, but it is likely that in the future we'll try to replace the error message with something more informative, letting the user know that the WebAssembly engine is still loading and to wait.

This writes nothing and gets replaced when webr is loaded. Hence avoiding runtime errors. However, it looks like an ugly patch.

I know it seems strange, but I can't think of a better way to handle this with Observable itself. IIUC even Quarto has to make some tweaks to hide these kind of error messages as the page loads.
It does seem like this is the idiomatic way to do this in OJS. See, for example, the example notebook about how to catch errors: https://observablehq.com/d/43fb4c8c2f600135

Here is how I would implement it in Quarto Live, by adding an extra mutable variable to the first line of the block:
```{ojs}
//| echo: false
mutable ok_reponse = (reponse, n) => { return html`Loading...` };
viewof reponse = Inputs.radio(
  new Map([
  ["On n'utilise que des procédures (des fonctions sans valeur de retour).", 1],
  ["On n'utilise que des fonctions dites pures, c'est-à-dire des fonctions purement combinatoires.", 2],
  ["On n'utilise que des fonctions, pures ou non.", 3],
  ["Différentes étapes successives qu'on appelle fonctions. On les applique sur des objets qu'elles modifient progressivement.", 4]
  ])
);
ok_reponse(reponse, 2);
```

I think the above could possibly even be implemented automatically as part of the quarto-live filter, I will have a think about how best to do so.


Create a configuration schema for dotnet-monitor

I took an initial stab at it by basing it off the ASP.NET schema. Try:
{
  "$schema": "https://gist.githubusercontent.com/shirhatti/880c71e47dc95d9099df1b240119850f/raw/d72b3d32d6554d9a9ce88ed0dbef06b894476cbe/settings.json"
}

https://aka.ms/dotnet-monitor-schema

I just realized this schema doesn't include ASP.NET Core schema. No completions for any kestrel server options 😭

I just realized this schema doesn't include ASP.NET Core schema. No completions for any kestrel server options 😭

I wonder if we can do something like:
{
  "allOf": [
    { "$ref": "<address of asp.net schema so we don't have to do it ourselves>" },
    { <dotnet-monitor specific schema> }
  ]
}

for the dotnet-monitor schema.

I looked at https://github.com/SchemaStore/schemastore/blob/master/src/schemas/json/appsettings.json as an example, and it's got a lot more than we'd want (also 3rd party properties) and yet doesn't have enough of the more standard properties, such as console logging and event logging objects. So maybe we do need to maintain our own self-contained schema with the more common ASP.NET Core properties.

I think we're approaching this the wrong way; us maintaining or referencing asp.net schema shouldn't be necessary. We should:

Rely on tools to support multiple schemas. (And probably make sure we are not restrictive in our schema if users want to combine settings)
Possibly support more than 1 settings file, to make it easier to configure disparate settings.

Sample multi-schema in vs code settings:
"json.schemas": [
{
"fileMatch": [
"settings.json"
],
"url": "https://aka.ms/dotnet-monitor-schema",
},
{
    "fileMatch": [
        "settings.json"
    ],
    "url": "https://json.schemastore.org/appsettings"
}
],

I tried searching a bit for how to do something similar in Visual Studio (not Code) and all I've been able to find is specifying $schema in the json file.

Close this and make a new issue to track the additional kestrel schema work. @wiktork


Exclude Shift from keydown events that terminate selection drags

Fixes #16063
Follows up on #16060 by including Shift in the modifiers that are exempt from terminating a drag operation.
/cc @50Wliu @as-cii

Thanks!

I compared Atom to ST3 selection cancel keys:
Atom doesn't cancel on: ctrl, shift (soon), alt
ST3 doesn't cancel on: ctrl, shift, alt, caps lock, num lock
I can't figure out any specific reason to cancel or not on the lock keys.
I can't test meta or scroll lock because those keys don't work on my keyboard.

I can't figure out any specific reason to cancel or not on the lock keys.

I would guess that they're doing some kind of check for non-printing keys.


updated spec

removed binary format, don't think it makes sense
added some optional/mandatory parameters
example on how to calculate the x, y in pixels to render

cc @andrewxhill


Add note about apxs, Fix old name, use lower/equal sign

I've added a note about where to find apxs when building from sources and a link to the documentation (there is no other mention of it). Beside that, there is one fix for the old name and one for ≤ being used instead of \< (hopefully that was the original meaning).

Thanks @jajik !


SIVT does not fetch vSphere Datastore full path correctly

Bug description
IHAC using Storage Folders in vCenter, and SIVT does not cope with folder path, when configuring TKGm (jinja templates for the key VSPHERE_DATASTORE)

I found 2 places in the code where this is done while testing and this fixed my issue, it's a harcoded fix obviously we need to fetch this properly, and store it.
https://github.com/vmware-tanzu/service-installer-for-vmware-tanzu/blob/main/src/vsphere/managementConfig/vsphere_management_config.py#L2067
https://github.com/vmware-tanzu/service-installer-for-vmware-tanzu/blob/main/src/vsphere/managementConfig/vsphere_management_config.py#L2212
my dirty fix to unstuck us
datastore_path = "/" + datacenter + "/datastore/vSAN Storages/" + datastoreName
Affected product modules (please put an X in all that apply)

[x] SIVT APIs
[x] SIVT UI
[X] SIVT CLI
[ ] Docs

Expected behavior
When using the UI, and being at the vCenter Connection step/credentials:

It will load all the required information, but for vSphere Datastore it will only keep the datastore name assuming it's at the root level in vCenter.

Steps to reproduce the bug

Put a vSphere Datastore in any folder
arcas deployment
arcas should deploy AVI Controller correctly
arcas will fail deploying TKGm vSphere VM with an error stating it cannot find the Datastore.

Version (include the SHA if the version is not obvious)
NA
Environment where the bug was observed (vSphere+VMC, vSphere+DVS, vSphere+NSXt, etc)
vSphere+DVS

SIVT version: 1.2.0
vSphere version: 6.7u3
vCenter version: 6.7u3
Kubernetes version: (use kubectl version): from SIVT-1.2.0
Kubernetes installer & version: from SIVT-1.2.0
Cloud provider or hardware configuration: NA
OS (e.g. from /etc/os-release): from SIVT-1.2.0
Sonobuoy tarball (which contains * below)

Relevant Debug Output (Logs, manifests, etc)

The fix has been made and will be available in SIVT 1.3 release


:robot: Add internal-monologue.vm

Automated PR from https://github.com/mandiant/VM-Packages/issues/592 :rocket:

@nos3curity issue with CLA fixed, should we merge this PR?

@mandiant/commando-vm :eyes:

@Ana06 LGTM. Category checked to be Credential Access

VM-Install-From-Github has been replaced byVM-Install-From-Zip in https://github.com/mandiant/VM-Packages/pull/988 @emtuls will fix this PR :smile:

Good to go @Ana06 !


CSIMigrationvSphere doc update

Documentation update for CSIMigrationvSphere feature gate.
KEP - https://github.com/kubernetes/enhancements/issues/1491
Changes Reviewed at https://github.com/kubernetes/website/pull/37701

/lgtm

/assign @sftim @tengqm

Hi @sftim @tengqm, can we get this PR merged?

/cc @krol3
/approve


feat: add bigint to is_numeric_dtype

This allows the profiler to run against dbt-spark / dbt-databricks profiler
Description & motivation
add bigint to is_numeric_dtype
Checklist

[x] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered):

[ ] Postgres
[ ] BigQuery
[ ] Snowflake
[ ] Redshift
[ ] SQL Server
[x] Databricks

[ ] I have written tests for new macros (either as dbt schema tests in integration_tests/models, dbt data tests in integration_tests/tests or integration tests in the CI workflow)
[ ] I have updated the README.md (if applicable)

@jomach Everything looks good here! Thanks for your contribution :)


CLI: lb_service resolver does not have vault setter

variables:
  lb_service:
    type: string
    required: false
    from:
      vault: ${STACK}_var_lb_service
      service_link:
        prompt: Pick a loadbalancer
        image: kontena/lb
        name: lb
    to:
      vault: ${STACK}_var_lb_service

 [error] 404 : Setter 'vault' for 'lb_service' : Not found (/v1/secrets/asdf/my-kontena-kuristin_var_lb_service)

Looks like your stack name has a slash, it will probably cause other problems too.
Did you do something like kontena stack install -n asdf/my-kontena-kuristin?

It's something else. The setters don't care where the value came from.
Are you perhaps using a nonexisting grid?

Yeah, I think this happened because the master was down?
Anyway, isn't the error message a bit misleading?

Ran into this bug today, the error message is indeed very misleading.


How to append   while sending a request

1 'Scenario: xml containing DTD complex
2    * def xml =
3   """
4
5
6
7     666
8
9    """
10    * match xml == 666'
Hi team
Suppose above xml is this payload
if print is given the it only prints 666
line 4 is missing to print
When the xml is passed as the request body ->
the whole xmlbody should be pass but it only pass  666
So how to send the below line in the request body

please ask questions like this on stackoverflow, and see: https://stackoverflow.com/a/62868837/143475
most teams don't need this - if you do, do consider contributing code to karate


SaveStateBulk and executeStateTransaction serialization difference

In what area(s)?

/area runtime

What version of Dapr?

1.13.x RC

Expected Behavior
Saving state to state store should serialize the payload in the same way no matter if SaveState, SaveStateBulk or executeStateTransaction are used.
Actual Behavior
When using SaveStateBulk and executeStateTransaction to store JSON content in Redis, the payloads are serialized differently, breaking the Query API.
SaveStateBulk

executeStateTransaction

Steps to Reproduce the Problem
Save the same JSON content using  SaveStateBulk and executeStateTransaction and see the different results in Redis.
Release Note

RELEASE NOTE:

@JoshVanL here is a reproducer with a README https://github.com/salaboy/reproducers/tree/main/json-dapr-transactions


Bad interaction with cdef property and function annotation

@scoder The code below fails to compile. After renaming the property, compilation succeed. I tried hard to figure out where the problem originated, but I failed miserably :disappointed:.
#cython: embedsignature=True
#cython: annotation_typing=False

cdef class memory:
    pass

cdef class Foo:

    def bar(self) -> memory:
        return memory()

    property memory:
        def __get__(self):
            return memory()

Issue no longer present in master.


Bumping to v2:  Terribly slow

Only after bumping to v2.0 and deploying, and we saw huge latency spikes and CPU consumption, did we note this part of the README:  100,000 iterations of PBKDF2 password hardening
Our bad, no doubt.
I'm just noting here that you should call out just how slow this makes your code:
cipher = Gibberish::AES.new('p4ssw0rd')
Benchmark.bmbm do |x|
  x.report('encrypt'){ 1_000.times{cipher.encrypt("some secret text")}}
end

=>
              user     system      total        real
encrypt 197.830000   0.000000 197.830000 (197.830117)

That's 200ms per simple encryption.  Useful in some cases, but I think it deserves an explicit warning for those of us using gibberish in web applications.

I'll update the docs on this. Right now it defaults to a very high number of iterations to harden the password, but it's customizable.
cipher = Gibberish::AES.new('p4ssw0rd', {iter:1000});
Find a balance between security and speed by adjusting the number of iterations.

:+1:

 > cipher = Gibberish::AES.new('p4ssw0rd', {iter:1000});
RuntimeError: unsupported cipher algorithm (aes-{:iter=>1000}-cbc)

Make sure this is 2.0 of Gibberish and now the old 1.x version. The method signature is now initialize(password, size, mode) whereas 2.x is initialize(password,options) which makes me think that error is the old version trying to make {iter:1000} into the size param, where it would normally be "aes-256-cbc"

I'm sorry:  I had my version locked.  Thanks.


How to configure the menu from config.toml

Using this theme I can successfully add a weight in the page front-matter.
Usually the menu for a Hugo site can also be configured from the config.tom file by adding entries to [[menu.main]], see https://gohugo.io/extras/menus/
But, for this theme this does not work, is there another way to configure the menu from config.toml?
Hugo version:  0.21

Hello @kamar535 , this theme does not use Hugo menus at all. The menu is automatically generated from sections.
In the FrontMatter of your pages, the Title parameter will be used as label of the menu. The weight will be used to order sections in the menu.
https://matcornic.github.io/hugo-learn-doc/cont/pages/#types

Tanks for the answer.
Maybe the theme documentation could mention https://gohugo.io/content-management/sections/ and that the menu can only be configured from the pages front matter.

Actually, the theme is currently on a rewrite to use latest features of Hugo: nested sections. In the future release, menu will be also generated automatically but infinite level of sections will be possible!
Therefore, the documentation will be updated as well.
Follow #51 for more information.

This is an old thread but hope it's ok to chip in?  I like Learn a lot and just want to simplify the (side) menu, eliminating the parts not generated from my content i.e. everything under 'More' and the search box.  Must be an easy way... I get a bit lost in the docs.

This is an old thread but hope it's ok to chip in? I like Learn a lot and just want to simplify the (side) menu, eliminating the parts not generated from my content i.e. everything under 'More' and the search box. Must be an easy way... I get a bit lost in the docs.

Hi Amberalpha, you've probably found the solution. Just delete any [[menu.shortcuts]] in config.toml
My own question to @matcornic : firstly a great theme, and I've used 4 different ones on live sites, I'll contribute. Question is can I change the "MORE" wording in menu to "Latest Addition" ? Been searching for ages but cannot find. Thanks

Ignore last question - found "More" is set in the i18n folder under en.toml


Implement comment/uncomment for CSS files

From<EMAIL_ADDRESS>on 2013-06-25T01:18:00Z
It will be convenient to have this comment/uncomment functionality for CSS. CSS supports only /* */ syntax, so the rules should be simple.
Some notes for myself if I ever get to it.

no selection - work with current line only
find first 2 non-space chars in line
if chars == /*
find last 2 non-space chard
if last == */
strip
else:
insert
# TODO - check if we're inside comment already - do nothing in this case

selection - hmm, the same stuff applies, but for selected text instead of line

Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1457

From ccordoba12 on 2015-01-01T16:21:56Z
Hard to do it and kind of out of scope.
Status: WontFix


[Android] Fix VerticalContentAlignment for elements in root

Related Issue
Fixes #3549
Description
Fixes the height for scenarios when the height has been set through min Height for containers growing in the root of a card
How Verified
This fix was tested manually and this is how the card looks now with the middle text aligning to the center and the last one aligning to the bottom

Microsoft Reviewers: Open in CodeFlow

/azp run


add api data to product items

Linked API data to frontend.

Product list for one of the category can be viewed locally at localhost:8000/weightloss
weightloss page being a category for Diet Plan.


Fantom bridge

There are comments regarding a genesis file, but currently we have to use the default genesis and changed eth keys, later we can revert the second commit

Do not merge this with master, I am doing this for the CI. Needs to be its own branch or repo


feat: add tool to generate resolvo snapshots

Adds a tool to create a resolvo snapshot from a conda channel.

Awesome stuff


Weinberg Link

Hompage. The link should be in red under the pciture (as in german version). No box given that there is no connection to form.

Ixed, just needed to change index.doc in the other languages


WIP: update to parity-scale-codec

Also update all parity codec dependency to codec
TODO:

[x] release scale-codec: https://github.com/paritytech/parity-scale-codec/pull/144
actually we might want to wait to EncodeLike so it is done for all dependencies (even if it is not a breaking change its good if dependency implement EncodeLike for self type as well.)
[ ] primitive-types
[ ] grandpa
[ ] rhododendron
[ ] trie (trie-bench)

cargo +nightly test --all is successful locally

hmm I currently have this issue:
thread 'tests::voter_catches_up_to_latest_round_when_behind' panicked at 'assertion failed: set_state.read().last_completed_round().number >= 4', core/finality-grandpa/src/tests.rs:1501:5

so comment adressed and also I put version to 1.0.0 for scale-codec as it is fine with this version

why is CI not answering though ?


Build our own cmake 3.3.2 in CI

especially with homebrew one has to go with whatever version they
choose, which is not testing our minimum version and can cause unrelated
failures.

Actually I'm going to switch to cmake binary packages


[2] Deleting subnamespaces removes resources in the wrong order

Describe the bug:
As an application developer, deleting subnamespaces that has resources with finalizers remaining will prevent me from being able to remove the finalizers, because the workload-admin rolebinding in the namespaces gets removed as the namespaces terminates.
Also, networkpolicies should be deleted after pods because if a pod is stuck in terminating state because of finalizers, patching the pod is not allowed by gatekeeper constraints, because the pod is no longer attached to a networkpolicy.
To Reproduce:
Steps to reproduce the behaviour:
As an application developer, so not cluster-admin privileges

Create a subnamespace
Create some resource e.g. a pod and add some finalizer to it.
Delete the subnamespaceanchor
See that the namespace gets stuck in "terminating" state and you no longer have permissions to remove the finalizers.

Expected behaviour:
As an application developer, I should be able to clean up finalizers after trying to delete a subnamespace.
Version (add all relevant versions):

Compliant Kubernetes apps version v0.31

Also, networkpolicies should be deleted after pods because if a pod is stuck in terminating state because of finalizers, patching the pod is not allowed by gatekeeper constraints, because the pod is no longer attached to a networkpolicy.

This sounds to me like a separate issue. Could you confirm if that is the case or if this strictly has to do with subnamespace deletion?

As an application developer, deleting subnamespaces that has resources with finalizers remaining will prevent me from being able to remove the finalizers, because the workload-admin rolebinding in the namespaces gets removed as the namespaces terminates.

If I understand the issue correctly this also doesn't sound like a problem specifically related to subnamespaces but RBAC in general. If the a rolebinding is created in a namespace and that rolebinding is necessary to remove the finalizers you have set yourself up for a deadlock if you're not careful when you delete that namespace. Please correct me if I'm wrong though.

Related upstream issue: https://github.com/kubernetes/kubernetes/issues/115070

Moving this issue to Blocked/stale as it's not clear to me that this is something we could fix other than informing users to be more careful when deleting namespaces, i.e. make sure to delete all the workloads first.

Closing this since AFAICT there is no difference between the deletion order of normal namespaces and subnamespaces. Please track the upstream issue linked above if you want to follow the progress of this. Feel free to reopen if you disagree.


Validate bluetooth and tilt integration

I need help to validate this function (i dont own a tilt). Bluetooth needs to be imported via USB into the container to see if this works properly.

Found a nice tool to simulate tilt over bluetooth (https://github.com/spouliot/tilt-sim).
So far I have concluded that one change will be needed in the docker image. Give the permissions to python for accessing bluetooth. The fermentrack troubleshooting wizard give the following suggestion.
sudo setcap cap_net_raw,cap_net_admin+eip /usr/bin/python3.8
However my docker environment (DSM) is currently running a too old linux kernel that does not support extended attributes for the filesystem AUFS. Apparently this is part of kernel 4.x. So i will need to wait for DSM 7.0 to be released in order to test this correctly.

Found a nice tool to simulate tilt over bluetooth (https://github.com/spouliot/tilt-sim).
So far I have concluded that one change will be needed in the docker image. Give the permissions to python for accessing bluetooth. The fermentrack troubleshooting wizard give the following suggestion.
sudo setcap cap_net_raw,cap_net_admin+eip /usr/bin/python3.8
However my docker environment (DSM) is currently running a too old linux kernel that does not support extended attributes for the filesystem AUFS. Apparently this is part of kernel 4.x. So i will need to wait for DSM 7.0 to be released in order to test this correctly.


TODO: Maximum number of players in a team

Add a player limit that can be assigned to a team, either 2 or 4.

Changes were made to master by accident, validations and teast already made and done so it shouldn't be an issue, will avoid this from now on by creating new branches for every issue.


Support undo/redo

Ideas:

client-side undo
server-side history

client-side redo seems hard and possible not needed.
server-side history seems like a good idea potentially

Not hard. We can just keep json history. Undo is good because people make mistakes a lot and undo helps them explore features worry-free so they have a better grasp of the controls.

Closing this for now.. maybe re-open eventually


Update 2.0 after release of 1.6.1

Update dev-2.0 with current status of master.
I was hoping there would be a way to make this a reviewable diff, but no. Even the merge commit, https://github.com/openpathsampling/openpathsampling/commit/35327d3749e0cdfa4b15b707e53cd7e514664867,  where I fixed merge conflicts, is probably too much to review. I will mention that while fixing merge conflicts, I went ahead and removed the PyEMMAFeaturizerCV (which is deprecated and slated for removal to 2.0).

This is ready for review and comment. Without review, I will merge this after at least 24 hours, no sooner than Wed 01 May 02:00 GMT (Tue 30 Apr 21:00 my local).
@sroet, review of the last 3 commits isn't too tedious. I hoped that the merge commit would have been possible, but 🤷♂. The change on CI scripts was that I ran diff against the normal CI script and kept the stuff I considered (potentially) relevant.


How to submit a run action?

Hi, thanks for the work on this library.
How do I submit a run tool output after my RunStatus becomes RequiresAction?

https://platform.openai.com/docs/api-reference/runs/submitToolOutputs this endpoint still has to be added


Option to check if file already exists before downloading

Such an option would check if a file with the name the outputfile would have, before starting to download.
It would be useful for playlist downloading. Especially because the download of some files fails sometimes.
With this feature you could just run the same command multiple times to be sure all files got downloaded.

https://github.com/ytdl-org/youtube-dl#how-do-i-download-only-new-videos-from-a-playlist


Documentation for #define macros

For some packages there are many constants that are nice to have documented. For example, in this format:
/*!
 * Indicate end of burst for transmit or receive.
 * For write, end of burst if set by the caller.
 * For read, end of burst is set by the driver.
 */
#define END_BURST (1 << 1)

is output as:
const SOAPY_SDR_END_BURST = 1 << 1

I am attempting the changes for this but I am having difficulty tracking down where the comment is dropped from the parser.

Libclang does not provide these information.
From https://github.com/llvm/llvm-project/blob/8f972cb0fd0cf2513a1134a5f37b3c059376ca47/clang/tools/libclang/CIndex.cpp#L8429-L8431 and https://github.com/llvm/llvm-project/blob/8f972cb0fd0cf2513a1134a5f37b3c059376ca47/clang/tools/libclang/CXComment.cpp#L28-L32, we can see that libclang only provides comment information for declarations, while a macro #define is, unfortunately, a MacroDefinition.


Added the service in the url, close #164

So you just had to add the service to get a valid answer... This was also a bug that I discovered long before, because sometimes you have a bug where you are not redirected correctly...
I will need to rewrite to be more on your style code but just tell me what to change

IssueHunt Summary

Referenced issues
This pull request has been submitted to:

#164: Couldn't use ENT Occitanie Educonnect

I cannot reproduce the bug, I used a venv, python 3.7 and same commands of the workflow

Why are you doing that this is useless. I give the URL in all functions and the functions use the param or not

Hi, cool that it works just with https://. The unit tests aren't failing because of your changes... LGTM.

No The unit test is not exhausting its normal that it's work

We must revert this PR @bain3

I think the failing test has nothing to do with this PR.
$ python -m pronotepy.ent.test_ent
DEBUG:pronotepy.ent.generic_func:[ENT https://cas.ent.auvergnerhonealpes.fr/login?selection=CLERMONT-ATS_parent_eleve] Logging in with username
E
======================================================================
ERROR: test_functions (__main__.TestENT)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/bain/repos/pronotepy/pronotepy/ent/test_ent.py", line 22, in test_functions
    self.assertRaises(pronotepy.ENTLoginError, func[1], "username", "password")
  File "/usr/lib/python3.10/unittest/case.py", line 738, in assertRaises
    return context.handle('assertRaises', args, kwargs)
  File "/usr/lib/python3.10/unittest/case.py", line 201, in handle
    callable_obj(*args, **kwargs)
  File "/home/bain/repos/pronotepy/pronotepy/ent/generic_func.py", line 158, in _cas
    for input_ in form.findAll("input"):
AttributeError: 'NoneType' object has no attribute 'findAll'

----------------------------------------------------------------------
Ran 1 test in 40.580s

FAILED (errors=1)

We can see that the test is failing because of an error in the _cas function. Probably during the ac_clermont_ferrand call.
This PR only changes the URL in occitanie_montpellier_educonnect.

Just to be sure, I ran the unit tests from fc2931a, and got the same error:
$ git checkout fc2931a
HEAD is now at fc2931a Merge pull request #170 from bain3/dev
$ python -m pronotepy.ent.test_ent
DEBUG:pronotepy.ent.generic_func:[ENT https://cas.ent.auvergnerhonealpes.fr/login?selection=CLERMONT-ATS_parent_eleve] Logging in with username
E
======================================================================
ERROR: test_functions (__main__.TestENT)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/bain/repos/pronotepy/pronotepy/ent/test_ent.py", line 22, in test_functions
    self.assertRaises(pronotepy.ENTLoginError, func[1], "username", "password")
  File "/usr/lib/python3.10/unittest/case.py", line 738, in assertRaises
    return context.handle('assertRaises', args, kwargs)
  File "/usr/lib/python3.10/unittest/case.py", line 201, in handle
    callable_obj(*args, **kwargs)
  File "/home/bain/repos/pronotepy/pronotepy/ent/generic_func.py", line 158, in _cas
    for input_ in form.findAll("input"):
AttributeError: 'NoneType' object has no attribute 'findAll'

----------------------------------------------------------------------
Ran 1 test in 40.320s

FAILED (errors=1)

I will try to fix it during the weekend


Unsubscribe is dead

Hi,
The unsubscribe link in the incident mail is dead. It links to /unsubscribe but i get a : Symfony \ Component \ HttpKernel \ Exception \ NotFoundHttpException

In the routes i can see that there is a route for /unsubscribe/{code}, but the e-mail is not attaching the code to the URL!

Should be fixed now.

Still not working... link text is now "Unsuscribe from email updates." but the link is still "/unsubscribe" without an token!

My bad, this should be fixed now :)


update postgresql-wale blueprint to refer the secret as an "Object" instead of reference name

In postgresql blueprint we are referring the secret with name "postgresql" viz.:
secretNames:
- postgresql

Objective of this task:
Refer the secret as an "Object" instead of reference name.

I am working on it

https://github.com/kanisterio/kanister/pull/562


feat: support events is zero for uv_poll

As the poll() manual says that events field may be specified as zero, in which case the only events that can be returned in revents are POLLHUP, POLLERR, and POLLNVAL.  This patch allows luv to support this call method with events equal zero. When the parameters in the lua is an empty string, it means that events is zero.

LGTM, please @squeek502 give suggestion

I'm not sure the changes in this PR would have the intended effect judging by the Libuv implementation of uv_poll_start. In both win and unix implementations, events == 0 is an early return:
https://github.com/libuv/libuv/blob/d4ab6fbba4669935a6bc23645372dfe4ac29ab39/src/win/poll.c#L498-L501
https://github.com/libuv/libuv/blob/d4ab6fbba4669935a6bc23645372dfe4ac29ab39/src/unix/poll.c#L137-L138

Other minor things:

Not sure why the CI was failing, will likely want to rebase the PR
This API currently doesn't have any tests in Luv, would be nice to add at least one (not a blocker for this PR, though)

Please rebase, wait CI pass to merge the PR.

Following the libuv definitions:
For windows, passing events=0 is the canonical behavior of uv_poll_stop.
For unix, passing events=0 is the canonical behavior of uv_poll_stop, with an additional check that the watcher already exists.
I'm definitely leaning towards this being not desirable behavior, if you want events=0 you should call poll_stop.


Problem with accents in a music name

When I try to download music with accents in the name, these are recorded on the computer with odd characters.
Example:
-> Music : Kalash Criminel - Mélanger ft. KeBlack
-> Fichier : Kalash_Criminel_-_MÃ©langer_ft._KeBlack.mp3
However, the tags are correct. It's juste the file name.

Yeah, I face this too but only on windows (seems to work okay for me on linux) but I am not sure how to fix this..

Any progress on this? I would love to help but I have no Windows machine anymore to reproduce - Linux only here :grin:

I can't reproduce this issue anymore since we shifted to python 3. I'll leave this open for a while in case somebody still faces this problem.

Ah, that was a short while. Glad we don't face this anymore 😊


Adding syslog functions, constants and structs

This commit adds the functions:-

syslog
openlog
closelog
setlogmask

It adds LOG_* constants.
It adds the CODE struct used by the #define definitions prioritynames and facilitynames.
It does not add:-

the function vsyslog; this is an extension to POSIX and uses va_list macros;
the #defines prioritynames and facilitynames, as usage is not common
rust functions mirroring the macros:-

LOG_PRI
LOG_MAKEPRI
LOG_MASK
LOG_UPTO
LOG_FAC

CODE is included in case a third-party unsafe C function returns or takes it.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Looks like a header for verification may not be included? You may need to modify libc-test/build.rs to add some headers here or there.

Let me know if you want the commits squashed, although done this way, I feel it's a little clearer the level of changes required.
By the way, is there a simple way I can use cfg checks to determine if a function is present in libc before useing it, or do I have to mirror the cfg checks libc uses in my consuming code? The latter seems like a subtle way to introduce breakages, eg if libc ever gained support for AIX or HP-UX (although given how they're slowly dying, probably not).

Looks like android CI is failing?

Let me know if you want the commits squashed

Nah whatever's fine by me :)

By the way, is there a simple way I can use cfg checks to determine if a function is present in libc before useing it, or do I have to mirror the cfg checks libc uses in my consuming code?

Unfortunately the only way to do so right now is to mirror the checks that libc has :(

I've not got an Android set up to test this on, so it's not a big surprise. A quick check at what I think is the Android bionic libc headers on GitHub suggests LOG_NFACILITIES  isn't defined on that platform... I'll try again....
 May be one day when I win the lottery I'll fund a team to re-write libc et al in rust...

@bors: r+ 828766fd23c1b052b7594f61d122cdd10e412458

:hourglass: Testing commit 828766f with merge 4fc03fe...

:sunny: Test successful - status-appveyor, travis

Thank you.


there is no 12213 jdbc config, where can i get it ?

can i use the follow config for 12213:
docker-images/OracleWebLogic/samples/1221-domain-with-resources/

In the process of removing the unsupported samples I will create new ones based on 12213.

yes you can use this sample just make sure you extend (FROM) the <IP_ADDRESS> image

i got issues, maybe i should change wlst to wlst.sh
[root@localhost 1221-domain-with-resources]# sh build.sh
Sending build context to Docker daemon  15.36kB
Step 1/4 : FROM oracle/weblogic:<IP_ADDRESS>-generic
 ---> 3b7136ac09fa
Step 2/4 : MAINTAINER Monica Riccelli<EMAIL_ADDRESS> ---> Running in fc1804d7f217
Removing intermediate container fc1804d7f217
 ---> ee02563578a6
Step 3/4 : COPY container-scripts/* /u01/oracle/
 ---> 837e05bfc01a
Step 4/4 : RUN wlst -loadProperties /u01/oracle/datasource.properties /u01/oracle/ds-deploy.py &&     wlst /u01/oracle/jms-deploy.py
 ---> Running in a2e68a41002c
/bin/sh: wlst: command not found
The command '/bin/sh -c wlst -loadProperties /u01/oracle/datasource.properties /u01/oracle/ds-deploy.py &&     wlst /u01/oracle/jms-deploy.py' returned a non-zero code: 127

then i get error again
[root@localhost 1221-domain-with-resources]# sh build.sh
Sending build context to Docker daemon  15.36kB
Step 1/4 : FROM oracle/weblogic:<IP_ADDRESS>-generic
 ---> 3b7136ac09fa
Step 2/4 : MAINTAINER Monica Riccelli<EMAIL_ADDRESS> ---> Using cache
 ---> ee02563578a6
Step 3/4 : COPY container-scripts/* /u01/oracle/
 ---> Using cache
 ---> 837e05bfc01a
Step 4/4 : RUN wlst.sh -loadProperties /u01/oracle/datasource.properties /u01/oracle/ds-deploy.py &&     wlst.sh /u01/oracle/jms-deploy.py
 ---> Running in 2022fb0ca3bf

Initializing WebLogic Scripting Tool (WLST) ...

Jython scans all the jar files it can find at first startup. Depending on the system, this process may take a few minutes to complete, and WLST may not return a prompt right away.

Welcome to WebLogic Server Administration Scripting Shell

Type help() for help on available commands

Error: runCmd() failed. Do dumpStack() to see details.
Problem invoking WLST - Traceback (innermost last):
  File "/u01/oracle/ds-deploy.py", line 19, in ?
  File "/tmp/WLSTOfflineIni6611239058251446014.py", line 131, in readDomain
  File "/tmp/WLSTOfflineIni6611239058251446014.py", line 19, in command
The domain directory must exist and contain config/config.xml
        at com.oracle.cie.domain.script.jython.CommandExceptionHandler.handleException(CommandExceptionHandler.java:69)
        at com.oracle.cie.domain.script.jython.WLScriptContext.handleException(WLScriptContext.java:2983)
        at com.oracle.cie.domain.script.jython.WLScriptContext.runCmd(WLScriptContext.java:735)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:498)

com.oracle.cie.domain.script.jython.WLSTException: com.oracle.cie.domain.script.jython.WLSTException: Error reading domain:
The domain directory must exist and contain config/config.xml

The command '/bin/sh -c wlst.sh -loadProperties /u01/oracle/datasource.properties /u01/oracle/ds-deploy.py &&     wlst.sh /u01/oracle/jms-deploy.py' returned a non-zero code: 1
[root@localhost 1221-domain-with-resources]#

i build <IP_ADDRESS> use the follow command,and i  access the console success
sh buildDockerImage.sh -v <IP_ADDRESS> -g

is it that when i create jdbc ,i should keep  the weblogic server in running  state?
or is there a script for me to create jdbc，i want the jdbc be ready before i start the container ,because i want use the images in cicd

Building the <IP_ADDRESS> WebLogic image only creates the binary image of WebLogic.  What kind of domain do you need  we have 2 samples the "single server domain" and the domain with an admin server, wls cluster, and x number of managed servers.  Which model do you need?  If it is the second then the next step is to create the WebLogic domain configuration.  We support 2 models of where the domain home is persisted 1) inside of an image 2) in a host volume.  Which of these two models are you interested in?

i just want a single adminserver and a jdbc config

What you need to do is change the wlst script in https://github.com/oracle/docker-images/blob/master/OracleWebLogic/dockerfiles/<IP_ADDRESS>/container-scripts/create-wls-domain.py to include the data source creation code in https://github.com/oracle/docker-images/blob/master/OracleWebLogic/samples/1221-domain-with-resources/container-scripts/ds-deploy.py make sure to set the properties like in https://github.com/oracle/docker-images/blob/master/OracleWebLogic/samples/1221-domain-with-resources/container-scripts/datasource.properties adjusted to the database you will connect to.

i find i must build domain and datasource together, is there have a way divide this two step?
as above ,you will get the error "The domain directory must exist and contain config/config.xml"

Yes you can use one WLST script or use two one to configure the domain alone and then one to configure the datasource.  After you have the store/oracle/weblogic:<IP_ADDRESS>  extend it and write a WLST script that configures the datasource with WLST online during the Docker build.  Look at the sample https://github.com/oracle/docker-images/tree/master/OracleWebLogic/samples/12212-domain-online-config.
Monica

@willzhang Do you still have questions or issues?

yes, before i deploy my app, sometime i just need one jndi datasource ,but sometimes i must run two or more jndi datasource for the app, how can i do that when i don't want to build another docker images?

The Operator doesn’t provide a way to add new resource modules (such as Datasources or JMS Modules); instead, you can create 'skeleton' modules in your original configuration and then use overrides to alter the existing modules.
For example, you can configure a skeleton datasource with a min-pool-size to 0 and its driver set to valid values, but its other values invalid - such as username and password.  This allows the datasource to start without errors at runtime even if no overrides are specified, but ensures the datasource cannot be accidentally used at runtime without overrides that are correct for the current environment.
When you are ready to use the datasource you can do a config override to set the valid values.

please take a look at the newly created sample OracleWebLogic/samples/12213-deploy-application.

thanks,the file help me:
docker-images/OracleWebLogic/samples/1221-domain-with-resources/container-scripts/ds-deploy.py

close the issue，thanks


Typings don't work for redux 4

After updating redux to v4.0.0-beta.1, compilation results in following error:
$ tsc
node_modules/redux-thunk/index.d.ts(4,47): error TS2314: Generic type 'Dispatch<D, S>' requires 2 type argument(s).
node_modules/redux-thunk/index.d.ts(8,20): error TS2428: All declarations of 'Dispatch' must have identical type parameters.
node_modules/redux/index.d.ts(103,18): error TS2428: All declarations of 'Dispatch' must have identical type parameters.
node_modules/redux/index.d.ts(150,13): error TS2314: Generic type 'Dispatch<D, S>' requires 2 type argument(s).
node_modules/redux/index.d.ts(271,13): error TS2314: Generic type 'Dispatch<D, S>' requires 2 type argument(s).
node_modules/redux/index.d.ts(285,59): error TS2314: Generic type 'Dispatch<D, S>' requires 2 type argument(s).
node_modules/redux/index.d.ts(285,75): error TS2314: Generic type 'Dispatch<D, S>' requires 2 type argument(s).
node_modules/redux/index.d.ts(357,95): error TS2314: Generic type 'Dispatch<D, S>' requires 2 type argument(s).
node_modules/redux/index.d.ts(362,33): error TS2314: Generic type 'Dispatch<D, S>' requires 2 type argument(s).
node_modules/redux/index.d.ts(364,106): error TS2314: Generic type 'Dispatch<D, S>' requires 2 type argument(s).
node_modules/redux/index.d.ts(369,34): error TS2314: Generic type 'Dispatch<D, S>' requires 2 type argument(s).

My dependencies:
"dependencies": {
		"react": "^16.2.0",
		"react-redux": "^5.0.6",
		"redux": "4.0.0-beta.1",
		"redux-thunk": "^2.2.0"
},
"devDependencies": {
                "@types/react": "^16.0.29",
		"@types/react-native": "^0.51.2",
		"@types/react-redux": "^5.0.14",
		"tslint": "^5.8.0",
		"typescript": "^2.6.2"
}

When I comment following lines from redux-thunk/index.d.ts, everything works fine:
declare module "redux" {
  export interface Dispatch<S> {
    <R, E>(asyncAction: ThunkAction<R, S, E>): R;
  }
}

It is not fixed all cases, moreover, you effectively pass Action instead of State to thunk action defined couple lines earlier:
export type ThunkAction<R, S, E> = (dispatch: Dispatch<S>, getState: () => S,
                                    extraArgument: E) => R;

Also I don't know right solution still... Will try to find.
PS. Author really need comaintainers here... (((

Typings for Redux v4 are not yet final.
Will update redux-thunk typings after v4 is released.

OK, so I'm just getting into redux-thunk. Just want to make sure I understand this. In Redux 3, my thunk action type currently should look something like this, right?
// using third parameter for environment
export type MyThunk = ThunkAction<void, any, "env1" | "env2">

But with Redux 4, it will look (possibly) something like this (stricter typing possible), right?
export type MyThunk = ThunkAction<void, MyThunkResultAction, "env1" | "env2">

Which then restricts my dispatch() function to dispatch MyThunkResultAction, correct?
If this is the case (which I hope it is), can we do something like this instead:
// third type is the thunk's return, is optional, and defaults to void if not specified
export type MyThunk = ThunkAction<MyThunkResultAction, "env1" | "env2", void>
// so you can do this
export type MyThunk = ThunkAction<MyThunkResultAction, "env1" | "env2">

How about something like this - seems to match what you want.
import { Middleware, Dispatch, Action, AnyAction } from 'redux';

export type ThunkAction<R, S, A extends Action = AnyAction, E = {}> = {
    (dispatch: Dispatch<A, S>, getState: () => S, extraArgument: E): R
}

declare module 'redux' {
    export interface Dispatch<A extends Action = AnyAction, S = {}> {
        <R, E>(thunk: ThunkAction<R, S, A, E>): R
    }
}

declare const thunk: Middleware & {
    withExtraArgument(extraArgument: any): Middleware;
};

export default thunk;

Changes the contract a little bit, but then again it's a major version of Redux so that's probably not the end of the world.
Middleware also takes generics now (yay) so that the thunk export might need to be extended if that makes sense for better typing support there.

Would a contributor be willing to release a beta version of redux-thunk that targets "4.0.0-beta.2" if I raised a pull request?
Apart from TypeScript I doubt we'll see any changes that break the thunk middleware functionality (https://github.com/reactjs/redux/issues/1342)

@NathanNZ ok, let me try.

Just created a PR. Please help review. Thank you guys.

Redux v4 is out. 🎉

So this basically makes the library unusable, as long as this is not fixed...

I just realized, that all sort of this problems exposed by library that is 14 lines of code )))))
I has a plans to return to this problem near weekend and try convert one project... will see how it behave now...

Another simple hack that doesn't require messing with messy types or declaration files, I had to use to be able to use react-redux's dispatch function which doesn't have an extension API built in:
interface IFetchBooksThunk {
  type: 'FETCH_BOOKS_THUNK'
}

export type MyAction = ISetTopic | ISetDisplayMode | IFetchStart |
                        IFetchComplete | IFetchFailed | IFetchBooksThunk

const actionify =
  <AT, A extends Action, S, E, R>(actionType: AT, thunk: ThunkAction<A,S,E,R>)
    :Action<AT> & ThunkAction<A,S,E,R> =>
      Object.assign(thunk, {type: actionType})

const fetchCurrentTopic: FetchBooksThunk = actionify<'FETCH_BOOKS_THUNK', MyAction, IMyStore, undefined, void>('FETCH_BOOKS_THUNK', (dispatch, getState) => {
  dispatch(fetchStart())
  const { topic } = getState()
  fetch(URL + topic)
    .then(res => {
      return res.json()
    })
    .then((json: IAPIResponse) => {
      if(json.error) {
        dispatch(fetchFailed(json.error))
      }
      else {
        dispatch(fetchComplete(json))
      }
    })
    .catch(error => {
      dispatch(fetchFailed(error))
    })
})

// elsewhere
store.dispatch(fetchCurrentTopic)
// No errors because it satisfies the action constraint via having a type
// field / property

@danlugo92 do you think you could make a PR?

I just made a pull request that fixes the issue while leaving the interface basically intact.
Pull request: https://github.com/reduxjs/redux-thunk/pull/196


MySQL - DriverError { Statement takes 4 parameters but 2 was supplied }

I've got the following piece of code which compiles:
pub fn add_user(&self, data: RegistrationForm) -> bool {
            let mut conn = self.conn.get_conn().unwrap();

            let execute_result = conn.exec_batch("
            INSERT INTO users (
                username, email, birthdate, password
            ) VALUES (
                :username, :email, :birthdate, :password
            )", params! {
                "username" => &data.username,
                "email" => data.email,
                "birthdate" => data.birthdate,
                "password" => data.password
            });

            match execute_result {
                Ok(_) => {
                    println!("User {} added!", data.username);
                    true
                }
                Err(e) => {
                    eprintln!("Failed to add user {}: {}", data.username, e.to_string());
                    false
                }
            }
        }

I'm sure that data.username, data.email, data.birthdate, data.password ain't empty. For some reason I get the following error at runtime;
Failed to add user test: DriverError { Statement takes 4 parameters but 2 was supplied }

I guess this has something to do with the prepared statement failing. But I don't see how it could fail?

Thanks for report. I'll look into it.
But could you please try to wrap params! { .. } into vec![] and see if it still errors?

The purpose of exec_batch is to execute a statement repeatedly with different sets of parameters:
conn.exec_batch(query, vec![params1, params2, params3, ..]);

Turns out that the result of params! invocation itself satisfies the bound on exec_batch's second parameter (IntoIter<Item: Into<Params>>). So calls like the following one are possible, yet semantically incorrect:
conn.exec_batch(query, params! { .. });

Thanks for report. I'll fix this.
Regarding your code. Since there is only one set of parameters and the query result is ignored I suggest you use exec_drop.

Okay thanks. That solved it. Is there a way to get the insert id?

Found it.
self.conn.get_conn().unwrap().last_insert_id();

Thx


Shorten container ID in events

Log only first 12 characters of container ID in events.
#9661

@brendandburns I've tested it locally. But I can not view the shippable results due to the reason mentioned in #9502. I tried to run shippable build for my own fork but encountered some problem. Is it possible to allow users not in GCP group to view the shippable result? Without logs it's hard to track the error.

@dchen1107 Can you post the shippable error message here? I've tested locally and used make release, all worked fine.

I think shippable failure is caused by flaky test, not by your code. I kicked out shippable for you.

LGTM


docs: Update README.md

google kubernetes autopilot is very strict in terms of networking capabilities. I tried wireguard-operator on it and it didn't work

Hi @Matthew-Beckett , can I get a review on this

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

v1.2.1
GitHub release

Your semantic-release bot :package::rocket:


Small UX improvements

Hi,
So here's the issue #18's feedback (old repository issue):

Inspecting variable words on mouse enter is working perfectly

Hiding / Showing atom bugs with crtl+shift+D works great, though it could also be nice to have this shortcut available through the menu Packages > Atom Bugs > Toggle

The current line highlight is not yet what I was expected (I may have not been crystal clear about it, sorry). The breakpoint line is highlighted on enter, yes, but it is not moving to the current line on each step over / into / out, as it does in Chrome (displaying the current debugged like is very useful)

Other considerations:

I would love to be able to move things around (I don't care if its the default display, though), a large console at the bottom of the screen, watch, call stack, scope and break points to the right, plus, the start / stop buttons within this panel, at the bottom of it (displaying it at the top of Atom moves the tabs down and I love tabs to be at the top of everything). Something like this:

With this display, it would be great if the watch, call stack, scope and break points have little arrows to toggle one or several of them, like in Chrome

Errors I encountered:

console.log(1, 2, 3) is printed on 3 lines

I was forced to move to from node 4.x to 6.x to make things work, may you consider supporting node 4.x?

On starting, it's necessary to click on Resume a first time for the server to really start — even if there ain't any breakpoint on server root. May you consider removing that need?

The remote debugging says Error: Parse Error

Like the last time, :champagne:, this tool you're creating is a missing must have for any JavaScript developer. Thanks so much.

Hi , @yamsellem Thank you the feedback you helping me a lot to improve this package. I'll add those updates this week.

Hello @willyelm, I'm happy to modestly contribute, you're doing something so great! I'll keep looking, and keep giving you some feedback ;-). It's awesome to get your feedback so quickly, as well!
I wish I could code with you, but I'm kinda busy at the moment. I'm also dreaming of a good mocha / ava atom integration, but it's another story.

Some fixes I have take care of so far in the latest version:

[x] Remote debugging connection fixed.
[x] Highlight current frame line.
[x] Only pause on step move and or breakpoint hint.


[Bug]: SessionLoad crashes

Your minimal.lua config
local root = "/tmp/persisted"

-- Set stdpaths to use root dir
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- Bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "--single-branch",
    "https://github.com/folke/lazy.nvim.git",
    lazypath,
  })
end
vim.opt.runtimepath:prepend(lazypath)

vim.opt.sessionoptions = "buffers,curdir,folds,globals,tabpages,winpos,winsize" -- Session options to store in the session

-- Install plugins
local plugins = {
  {
    "olimorris/persisted.nvim",
    opts = {
      silent = true, -- silent nvim message when sourcing session file
      use_git_branch = true, -- create session files based on the branch of the git enabled repository
      autoload = true, -- automatically load the session for the cwd on Neovim startup
      follow_cwd = true, -- change session file name to match current working directory if it changes
      allowed_dirs = { '/home/musinux/projects/front', '/home/musinux/projects/front-client' }, -- table of dirs that the plugin will auto-save and auto-load from
      ignored_dirs = { '/home/musinux/projects/front/node_modules', '/home/musinux/projects/front-client/node_modules' }, -- table of dirs that are ignored when auto-saving and auto-loading
      telescope = { -- options for the telescope extension
        reset_prompt = true, -- whether to reset prompt after session deleted
      },
    }
  },
  -- Put any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

Error messages
E5108: Error executing lua ...persisted/plugins/persisted.nvim/lua/persisted/utils.lua:58: missing '[' after '%f' in pattern
stack traceback:
        [C]: in function 'gsub'
        ...persisted/plugins/persisted.nvim/lua/persisted/utils.lua:58: in function 'make_session_data'
        ...persisted/plugins/persisted.nvim/lua/persisted/utils.lua:133: in function 'load_session'
        .../persisted/plugins/persisted.nvim/lua/persisted/init.lua:215: in function 'load'
        [string ":lua"]:1: in main chunk

Describe the bug
I expect :SessionLoad to function properly and not fail
However the error message doesn't seem to prevent sessions to actually load

Reproduce the bug

nvim --clean -u ./minimal.lua
:SessionSave
:SessionLoad -> this should trigger the error

Final notes: This bug has been there for multiple months for me, and I'm always following the latest dev release of nvim. I'm currently on NVIM v0.10.0-dev-2976+g208852126
Final checks

[X] I have made sure this issue exists in the latest version of the plugin
[X] I have tested with the minimal.lua config file above and still get the issue
[X] I have used SessionSave to save the session before restarting Neovim and using SessionLoad
[X] I have made sure this is not a duplicate issue

I've tried this with your minimal.lua file and can't recreate it. Is there something missing from your config?

Thanks for looking into it. I honestly don't know. Here's a session file that triggers the error consistenly. Maybe there's something in there that can help you?
The file is named: %home%musinux%projects%front-client@@branchprefix%the-branch-name.vim
let SessionLoad = 1
let s:so_save = &g:so | let s:siso_save = &g:siso | setg so=0 siso=0 | setl so=-1 siso=-1
let v:this_session=expand("<sfile>:p")
silent only
silent tabonly
cd ~/projects/front-client
if expand('%') == '' && !&modified && line('$') <= 1 && getline(1) == ''
  let s:wipebuf = bufnr('%')
endif
let s:shortmess_save = &shortmess
if &shortmess =~ 'A'
  set shortmess=aoOA
else
  set shortmess=aoO
endif
argglobal
%argdel
argglobal
enew
setlocal fdm=syntax
setlocal fde=0
setlocal fmr={{{,}}}
setlocal fdi=#
setlocal fdl=20
setlocal fml=1
setlocal fdn=20
setlocal fen
tabnext 1
if exists('s:wipebuf') && len(win_findbuf(s:wipebuf)) == 0 && getbufvar(s:wipebuf, '&buftype') isnot# 'terminal'
  silent exe 'bwipe ' . s:wipebuf
endif
unlet! s:wipebuf
set winheight=1 winwidth=20
let &shortmess = s:shortmess_save
let s:sx = expand("<sfile>:p:r")."x.vim"
if filereadable(s:sx)
  exe "source " . fnameescape(s:sx)
endif
let &g:so = s:so_save | let &g:siso = s:siso_save
set hlsearch
nohlsearch
doautoall SessionLoadPost
unlet SessionLoad
" vim: set ft=vim :

I have minimal knowledge of what a session file should look like unfortunately.
I wonder if the name is the problem:  %home%musinux%projects%front-client@@branchprefix%the-branch-name.vim
Most of mine are named %Users%Oli%.dotfiles@@main.vim.

The error comes from
https://github.com/olimorris/persisted.nvim/blob/ca9900c31ee6e254a0ba7011ba49f48ebf4c8db2/lua/persisted/utils.lua#L57-L58
And indeed, my folder is [...]%front-client[...] so and I see this stackoverflow post so maybe there's a pattern recognition thing going on with it ?

Could you share the full path to the session? Of course change any words for confidentiality reasons.

Yes, here's one:
/home/louis-cherel/.local/share/nvim/sessions/%home%louis-cherel%projects%front@@louis%fix-analytics-export-null-values.vim

I'm also having this same problem:
E5108: Error executing lua: vim/_editor.lua:0: nvim_exec2(): Vim(lua):E5108: Error executing lua ...l/share/nvim/lazy/persisted.nvim/lua/persisted/utils.lua:58: missing '[' after '%f' in pattern
stack traceback:
	[C]: in function 'gsub'
	...l/share/nvim/lazy/persisted.nvim/lua/persisted/utils.lua:58: in function 'make_session_data'
	...l/share/nvim/lazy/persisted.nvim/lua/persisted/utils.lua:133: in function 'load_session'
	...al/share/nvim/lazy/persisted.nvim/lua/persisted/init.lua:215: in function 'load'
	[string ":lua"]:1: in main chunk
	[C]: in function 'nvim_exec2'
	vim/_editor.lua: in function 'cmd'
	...re/nvim/lazy/dashboard-nvim/lua/dashboard/theme/doom.lua:24: in function <...re/nvim/lazy/dashboard-nvim/lua/dashboard/theme/doom.lua:20>
stack traceback:
	[C]: in function 'nvim_exec2'
	vim/_editor.lua: in function 'cmd'
	...re/nvim/lazy/dashboard-nvim/lua/dashboard/theme/doom.lua:24: in function <...re/nvim/lazy/dashboard-nvim/lua/dashboard/theme/doom.lua:20>

This only happen on this specific folder... The session file is /home/gabe/.local/share/nvim/sessions/%home%gabe%[REDACTED]%[REDACTED]%[REDACTED]@@[REDACTED]%fix-[REDACTED].vim, and the file contents are:
let SessionLoad = 1
let s:so_save = &g:so | let s:siso_save = &g:siso | setg so=0 siso=0 | setl so=-1 siso=-1
let v:this_session=expand("<sfile>:p")
silent only
silent tabonly
cd [REDACTED]
if expand('%') == '' && !&modified && line('$') <= 1 && getline(1) == ''
  let s:wipebuf = bufnr('%')
endif
let s:shortmess_save = &shortmess
if &shortmess =~ 'A'
  set shortmess=aoOA
else
  set shortmess=aoO
endif
argglobal
%argdel
tabnext 1
if exists('s:wipebuf') && len(win_findbuf(s:wipebuf)) == 0 && getbufvar(s:wipebuf, '&buftype') isnot# 'terminal'
  silent exe 'bwipe ' . s:wipebuf
endif
unlet! s:wipebuf
set winheight=1 winwidth=20
let &shortmess = s:shortmess_save
let s:sx = expand("<sfile>:p:r")."x.vim"
if filereadable(s:sx)
  exe "source " . fnameescape(s:sx)
endif
let &g:so = s:so_save | let &g:siso = s:siso_save
set hlsearch
nohlsearch
doautoall SessionLoadPost
unlet SessionLoad
" vim: set ft=vim :

On my session filename, I also have a %f on %fix (branch name is like something/fix-something), so I assume this is also the cause...

Thanks for flagging.
Any support you both could offer for this would be great. I don't have capacity right now.

Now that we have a good lead, I'll try to dig in the following weeks

I think I've solved it...can you guys try this PR: https://github.com/olimorris/persisted.nvim/pull/123

Thank you! That fixed it for me :)
Small note: I'd prefer not having my name in your test files, so if you can just twist your test string that would mean a lot to me


Fix relative path loading of chemiscope jsons

More fixing of paths...

Last time I merge like this promised


Add multiplayer support

This PR close sthe following user story: https://trello.com/c/bO6jZVgm/24-8-as-robert-i-want-to-play-with-other-distant-players.

Some more UI comments:

I would like the title of the page to show what step I am at, instead of displaying 'Home' all the time. E.g. 'Sign In / Sign Up' or 'Create a new game' or 'Lobby'
I would expect the UI to display the link I need to send to the other players to join in
The game lobby should also auto-refresh because otherwise I am not notified when another user has joined the game
The player color should be displayed as a colored circle instead of the color name
It's a bit confusing to see the target of the other player when it's not your turn, same goes with the player's name and color, which are actually the ones for the current player, but not necessarily mine. I would prefer to see my info all the time and only see which player's turn it is.

'Create a new game' or 'Lobby'
I would expect the UI to display the link I need to send to the other players to join in
The game lobby should also auto-refresh because otherwise I am not notified when another user has joined the game
The player color should be displayed as a colored circle instead of the color name
It's a bit confusing to see the target of the other player when it's not your turn, same goes with the player's name and color, which are actually the ones for the current player, but not necessarily mine. I would prefer to see my info all the time and only see which player's turn it is.

Will do in next PR (UI polish)

One more thing (to be done in next PR): you should mention in the docs that it is necessary to run make run twice when you start the project for the 1st time


Fixes 204: $.fetch() upload events

As per discussion in #204.
I was a little bit unsure if env.xhr.upload would always be present. I think it is, and in my tests (latest versions of Chrome, Firefox, Edge, Safari + IE11) it always was (even for GET-requests with no body), but I still felt uneasy not checking.
There is no parameter validation besides checking that upload is an object. The intention was just to provide access to attaching events to xhr.upload via $.fetch()s option params, similar to how it can be done with download events today. Not taking responsibility for how people use it. Unlike the xhr-object, xhr.upload doesn't have any read only-properties, so try...catch shouldn't be necessary (at least for that reason) unless the xhr standard changes.

I'm not sure why travis can't build? The tests work for me locally.


info in python's API reference doesn't look up to date

I noticed the python library API reference doc doesn't mention read_airr and friends.

Might also want to add the deprecation message within the description for the function so it shows in this doc, unless there's a special way to mark deprecated stuff...

Both are now fixed in #633.
(There is indeed a special way to mark deprecation.)


replace mentions of "master" in the proto

@parthea, there are few mentions of "master" in the proto files. Namely:
https://github.com/googleapis/python-documentai/blob/main/google/cloud/documentai_v1/types/document.py#L677-L688
and
https://github.com/googleapis/python-documentai/blob/main/google/cloud/documentai_v1beta3/types/document.py#L681-L692.
Could you fix these up in the proto, or assign someone to handle it?

These look like comments that include links to  https://github.com/googleapis/googleapis/blob/master/google/type/money.proto etc. and that repo is still using master as their main branch (line references have changed slightly but I still see 4 instances in each file).

I'm going to transfer this issue to the google-cloud-python repository as we are preparing to move the code for google-cloud-documentai to that repository in the next 1-2 weeks.


Is this a bug? removeStream does not remove streams

In my case, there is an initiator and a broadcaster who can see and hear each other. I want to delete the local stream of broadcaster from him and from the initiator

For example, I try to remove a local video stream broadcaster with the command:

connection.removeStream({
	video: true,
	local: true
}, true);

But I get a message in his console:

RTCMultiConnection.js: 5188 No such stream exist. {video: true, local: true}

2.Next, I check if there is a local stream broadcaster? Stream parameters I get like this::
connection.attachStreams.forEach (function (localStream) {
      console.log (localStream);
});

MediaStream {streamid: "ECi3I6TigcgB0XyLMFEpt4kqcFTO5DkfQeup", idInstance: "{" audio ": {" optional ": [{" sourceId ":" default "}, {" ech ... ameRate ": 5," minAspectRatio ": 1.333 , isVideo: 1, isAudio: false, type: "local", ...}

This means that the broadster stream has not been deleted locally.

Well, maybe the initiator has changed something, after applying the removeStream command?
I check if the initiator has deleted the flow of broadsters?

connection.peers [remoteUserId] .peer.getRemoteStreams () [0]

MediaStream {isAudio: false, isVideo: true, isScreen: false, streamid: "ECi3I6TigcgB0XyLMFEpt4kqcFTO5DkfQeup", type: "remote", ...}

There is a media stream. So removeStream did not work.

I try to delete the stream of broadkaster from the initiator console:

connection.streamEvents.selectAll({
	userid: remoteUserId
}).forEach(function(remoteStreamEvent) {
	console.log(remoteStreamEvent);
	connection.removeStream (remoteStreamEvent)
})

The same effect:

RTCMultiConnection.js: 5188: "No such stream exist".

I am trying to use a different method. In the broadcaster console I type the command:

connection.getAllParticipants().forEach(function(remoteUserId) {
	var user = connection.peers[remoteUserId];
	user.peer.getLocalStreams().forEach(function(localStream) {
		user.peer.removeStream(localStream);
	});
});

Broadcast has not changed anything. BUT! On the page of the initiator of the video, the stream Broadcast is stopped. This means the stream is transmitted by the broadcaster and is received by the initiator.
It would be convenient for me to use the connection.removeStream function declared in the API https://www.rtcmulticonnection.org/docs/removeStream/
Tell me, please, what I described is a bug, or did I not understand the API?

Temporary solution.

Temporary solution.
I`m use this:
connection.attachStreams.forEach(function(stream) { stream.stop(); });
Or an analogue of this:
connection.streamEvents.selectAll({
	local: true
}).forEach(function(streamEvent) {
	streamEvent.stream.getAudioTracks()[0].stop();
	streamEvent.stream.getVideoTracks()[0].stop();
});

When local streams are stopped, the connection.onstreamended event fires and removes the media container with the video tag from both the broadcaster and the initiator.
But, unfortunately, this is not a complete solution.
In another topic, I will try to explain, as it seems to me, a related problem.

Another way to remove a stream is to exit a user from a room.
connection.deletePeer (remoreUserId);
But then you can not apply the new settings without breaking the connection with all users.


Presence and Friends List

The progress so far on Presence and Friends List system. Presence is completely done, while Friends List is mostly done. There are still a few things left to do for friend's list:

Handle when accepting a user as a friend, update friends list.
Handle friend chat based on the friend requesting system.
Retrieve "pending" friend requests from history in case the user wasn't online.
Some more UI works needs to be done for Friends List (Friends Button, and adding a friend). It was added quickly for testing purposes.
Based on time, will not move friending system away from client-side. I will be creating a ticket to work on this in the future.

Please take a look at each of the commit summaries if you are curious as to what was done. Presence, Friends List, and UI bugs were fixed. I am creating this pull request so I am not blocking anyone's work tomorrow - it is up to you if you want to merge the changes or not. Although Friends List is not complete, it will not break existing functionality or hinder other people's workflows.

Looks good, I had a couple of issues when testing but nothing to block the merge & I'll also communicate these over Slack:

In my IDE, if I go to create a game, I can’t cancel the UI (because presumably the chat window is on top)
“The Referenced Script on this behavior (Game Object ‘LocalizationDropdown’) is missing error is seen.  Also some other ‘The referenced script (Unknown) on this Behaviour is missing - Looks to be coming from the Settings Window

Thank you for the review! The UI and referenced script behavior issues, will be included in the next PR - thanks!