ISPN-12402 + ISPN-12184 Multiple single-port endpoints and explicit/implicit fixes

This PR introduces the ability to have multiple single-port endpoints. It also allows disabling admin features (CLI/Console) per-endpoint. Additionally it fixes an issue where it was not really respecting explicit authentication mechanisms.
https://issues.redhat.com/browse/ISPN-12184
https://issues.redhat.com/browse/ISPN-12402

17:06:52,033 INFO  (ForkJoinPool.commonPool-worker-3) [org.infinispan.SERVER] ISPN080018: Protocol HotRod (internal)
17:06:52,115 INFO  (main) [org.infinispan.SERVER] ISPN080018: Protocol REST (internal)
17:06:52,285 INFO  (main) [org.infinispan.SERVER] ISPN080004: Protocol SINGLE_PORT listening on <IP_ADDRESS>:11222
17:06:52,285 INFO  (main) [org.infinispan.SERVER] ISPN080034: Server 'pedro-desktop-17255' listening on http://<IP_ADDRESS>:11222
17:06:52,302 INFO  (ForkJoinPool.commonPool-worker-3) [org.infinispan.SERVER] ISPN080018: Protocol HotRod (internal)
17:06:52,335 INFO  (main) [org.infinispan.SERVER] ISPN080018: Protocol REST (internal)
17:06:52,664 INFO  (main) [org.infinispan.SERVER] ISPN080004: Protocol SINGLE_PORT listening on <IP_ADDRESS>:13333
17:06:52,664 INFO  (main) [org.infinispan.SERVER] ISPN080034: Server 'pedro-desktop-17255' listening on http://<IP_ADDRESS>:13333
17:06:52,664 INFO  (main) [org.infinispan.SERVER] ISPN080001: Infinispan Server 12.0.0-SNAPSHOT started in 5329ms

I think there is a verb missing in  ISPN080018 message
ISPN080004: Protocol SINGLE_PORT  IMO, it would be nice to add the socket-binding name to the log message, like
Protocol SINGLE_PORT (<socket-binding>) listening on <IP_ADDRESS>:11222

@tristantarrant there are test failures, probably related:

RestServerResource (NPE)
CliIT

offtopic: while testing "admin" and "metric-auth", it looks like the disconnect is broken. connection = null is missing in org.infinispan.cli.impl.ContextImpl#disconnect(). I guess it can be fixed here :) it is just one line.
pedro-desktop-53750@cluster//containers/default]> disconnect
[pedro-desktop-53750@cluster//containers/default]> disconnect
[pedro-desktop-53750@cluster//containers/default]> disconnect
[pedro-desktop-53750@cluster//containers/default]> disconnect -h
Usage: disconnect [<options>]
Disconnects from a remote server

Options:
  -h, --help

[pedro-desktop-53750@cluster//containers/default]> disconnect
[pedro-desktop-53750@cluster//containers/default]> ls
Could not connect to server: java.io.InterruptedIOException: executor rejected
[pedro-desktop-53750@cluster//containers/default]> ls
Could not connect to server: java.io.InterruptedIOException: executor rejected
[pedro-desktop-53750@cluster//containers/default]> cd
The name of the subsystem/item is required for this command.
[pedro-desktop-53750@cluster//containers/default]> ls
Could not connect to server: java.io.InterruptedIOException: executor rejected
[pedro-desktop-53750@cluster//containers/default]> cd /
[pedro-desktop-53750@cluster/]> ls
containers
cluster
server
[pedro-desktop-53750@cluster/]> cd containers
[pedro-desktop-53750@cluster//containers]> ls
default
[pedro-desktop-53750@cluster//containers]> cd default
Could not connect to server: java.io.InterruptedIOException: executor rejected

@pruivo I've addressed all issues I believe

integrated! thanks @tristantarrant !


Multiple body parameters are not allowed in swagger

In GitLab by @guyingyan on Apr 17, 2018, 15:42

In GitLab by @sokril on Nov 8, 2019, 07:49
fix issue with pr#1206

In GitLab by @sokril on Nov 8, 2019, 08:21
closed

In GitLab by @sokril on Nov 8, 2019, 08:21
assigned to @luotian-github


Annotations: move operator list addition logic to src/core/document.js

Ideally, the Annotation class should not have anything to do with the page's operator list. How annotations are added to the page's operator list is logic that belongs in src/core/document.js instead where the operator list is constructed.
Moreover, some comments have been added to clarify the intent of the code.

/botio-linux preview

/botio test

My initial reaction was that this feels like a lot of code to be putting in the Page.getOperatorList method. However, I'm not sure that moving this annotation code to a helper method in the Page "class" would be much better, considering the number of parameters that you'd still need to pass around then.

I thought about that as well and came to the same conclusion as you did. In the new patch I managed to remove one more line that wasn't really necessary. In total we're adding around 15 lines to Page.getOperatorList, which in my opinion is okay, especially since some of them are just comments.

Would it now perhaps make sense to add a helper function for checking the viewability/printability of annotations (e.g. like below)?

It most certainly does! It has now been added to the new patch. This makes the patch size +27 -30, which is also nice to have.

/botio-linux preview

/botio-linux preview

/botio test


[BUG] - Serialization rules aren't followed for arrays of nested objects

Describe the bug
If you have an array of nested objects, the @SerializeProperty rules for those objects are not followed.
To Reproduce
Here's at test I added to serialize_property_tests.ts:
test({
  name: "Serialize arrays of nested objects",
  fn() {
    class Test1 extends Serializable {
      @SerializeProperty("nested_property")
      serializeMe = 999;
    }
    class Test2 extends Serializable {
      @SerializeProperty({
        serializedKey: "outer_property",
      })
      nested: Test1[] = [new Test1()];
    }
    const testObj = new Test2();

    assertEquals(
      testObj.toJson(),
      `{"outer_property":[{"nested_property":999}}]`,
    );
  },
});

Output:
failures:

Serialize arrays of nested objects
AssertionError: Values are not equal:

    [Diff] Actual / Expected

-   "{\"outer_property\":[{\"serializeMe\":999}]}"
+   "{\"outer_property\":[{\"nested_property\":999}}]"

Expected behaviour
If you have an array of objects with serialization information, it should be used in toJson() / toPojo().

Thanks for the report.


no accepting anonymous connection to mqtt brocker

Hello,
Before PIN implementation everything was working great.
now, i don't now why, empty user do not work :/
any idea ?
logs :
sudo docker logs -f --tail 10 tydom2mqtt
MQTT Disconnected !
##################################
cat  ../tydom2mqtt/options.json
{
"TYDOM_MAC":"XXXXXXXX",
"TYDOM_IP": "XXXXXX",
"TYDOM_PASSWORD":"XXXXXX",
"MQTT_HOST": "XXXXX",
"MQTT_USER": "",
"MQTT_PORT": 1883 ,
"MQTT_PASSWORD": "",
"MQTT_SSL": false,
"TYDOM_ALARM_PIN": "",
"TYDOM_ALARM_HOME_ZONE": "",

restarting tydom/mqtt/hass was requiered ! :)


[WIP - need Windows help] Get Linux and Mac working to download stanc3

This is a PR to switch over CmdStan to stanc3. We discussed a few strategies in the meeting last week
It seemed like the option everyone favored (including a bunch of CmdStan users and devs) was to switch it over all at once and basically be on call for the next month until the release with bugfixes to develop, and after that as well. CmdStan users will be able to help us iron out any remaining issues before being integrated into RStan for 2.21 a few months later.
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

@rok-cesnovar would you mind taking a look at this and helping me adapt it for Windows?

Also it seems that this is updating the Stan submodule?

That sounds like a decent strategy to me.  I don't know how we could switch it in pieces.

Two questions/comments that came up in a discussion with Sean:

which version of the compiler will make build download
if we decide that make build downloads a "stable version", do we want to add a make build-nightly that downloads a nightly version?

Is there any simple way of a user to retrieve the "which version of the compiler is this" info? That would be helpful for support purposes.

Woops, I'll fix the stan submodule thing.
re: versioning - I was thinking CmdStan release versions like 2.21.0 would download a corresponding stanc3 2.21.0, while on develop it would download stanc3 nightly. Then we could add an additional make build-nightly target as you suggest to get the latest compiler :)

Is there any simple way of a user to retrieve the "which version of the compiler is this" info? That would be helpful for support purposes.

Yep, bin/stanc --version. Though right now that's not returning the right version number, woops! Also all of the generated files will have a version number.

Turns out curl be tricky on Windows. Yes I know, I know, you are all shocked.
Running curl in the normal command line (cmd.exe) uses curl that is equivalent to the one on Linux (curl.exe is located in C:/Windows/System32). But curl in powershell is actually an alias for another command called Invoke-WebRequest, which does not support TLS 1.2 by default and thus needs additionally trickery. Not to mention that Invoke-WebRequest has different arguments than the regular curl.
I am genuinely interested in what is the reasoning behind having an alias to a function that doesnt match in arguments :)
I am looking for a solution that would be agnostic to the command line but I am striking out badly :/

Can we use http instead of https?
On Wed, Sep 25, 2019 at 11:47 Rok Češnovar<EMAIL_ADDRESS>wrote:

Turns out curl be tricky on Windows. Yes I know, I know, you are all
shocked.
Running curl in the normal command line (cmd.exe) uses curl that is
equivalent to the one on Linux (curl.exe is located in C:/Windows/System32).
But curl in powershell is actually an alias for another command called
Invoke-WebRequest, which does not support TLS 1.2 by default and thus needs
additionally trickery. Not to mention that Invoke-WebRequest has different
arguments than the regular curl.
I am genuinely interested in what is the reasoning behind having an alias
to a function that doesnt match in arguments :)
I am looking for a solution that would be agnostic to the command line but
I am striking out badly :/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/cmdstan/pull/737?email_source=notifications&email_token=AAGET3B47OT6R5AJCID5MPDQLOBXFA5CNFSM4I2BBLU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7SMGUY#issuecomment-535085907,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGET3FLQXZHTB6G7KDANWDQLOBXFANCNFSM4I2BBLUQ
.

Invoke-WebRequest http://github.com/stan-dev/stanc3/releases/download/nightly/windows-stanc -o bin/stanc
Invoke-WebRequest : The request was aborted: Could not create SSL/TLS secure channel.

I think this access some Github API endpoints that are still under https.
The additional trickery is not that bad but we would need to add a powershell script file. If you feel that is fine, I can add it to the branch.

I found a solution after struggling with this for too much time. Sorry this took so long.
The solution is superdumb but this works in both powershell and cmd.exe (the old command line):
bin/stanc$(EXE) :
	@mkdir -p $(dir $@)
ifeq ($(OS_TAG),windows)
	$(shell echo "curl -L https://github.com/stan-dev/stanc3/releases/download/nightly/$(OS_TAG)-stanc -o bin/stanc$(EXE)")
else
	curl -L https://github.com/stan-dev/stanc3/releases/download/nightly/$(OS_TAG)-stanc -o bin/stanc
	chmod +x bin/stanc
endif

so echoing the command. I cant even...

Sent to fast. If you are fine with the solution I can push it on the branch if you want.

Hah, looks good to me and seems better than powershell. Kind of crazy that this works.
Which versions of Windows (we can assume RTools is installed) will this work with?

This should work on all Windows 10 that have been updated in the last 2 years plus on systems that have git for windows. See here for a detailed answer.
Also tagging @t4c1 because I know he does most if not all of his development on Windows.
Would you mind checking the stanc3-switch branch on cmdstan and running
make clean-all
make build
and double check that you can compile a model. You should see something downloading in the make build step.

I tested in both cmd and powershell on windows 10.
make build works
make examples/bernoulli/bernoulli.exe works
make examples\bernoulli\bernoulli.exe does not work (It also does not work under old compiler)

@t4c1 thanks! Can you try to merge recent develop, there was a fix for that merged after this PR was open? It should fix the \ bug.

Recent changes on develop do fix this bug for both compilers for basic case. However, I also tried compiling with OpenCL. That made everything break down again:
PS F:\Users\tadej\Programming\bstatcomp\cmdstan> make .\examples\bernoulli\bernoulli.exe

--- Translating Stan model to C++ code ---
bin/stanc.exe  --o=./examples/bernoulli/bernoulli.hpp ./examples/bernoulli/bernoulli.stan
Model name=bernoulli_model
Input file=./examples/bernoulli/bernoulli.stan
Output file=./examples/bernoulli/bernoulli.hpp

--- Compiling, linking C++ code ---
g++.exe -std=c++1y -m64 -D_REENTRANT -Wall -Wno-unused-function -Wno-uninitialized -Wno-unused-but-set-variable -Wno-unused-variable -Wno-sign-compare -Wno-unused-local-typedefs   -I stan/lib/stan_math/lib/opencl_2.1.0  -O3 -I src -I stan/src -I lib/rapidjson_1.1.0/ -I stan/lib/stan_math/ -I stan/lib/stan_math/lib/eigen_3.3.3 -I stan/lib/stan_math/lib/boost_1.69.0 -I stan/lib/stan_math/lib/sundials_4.1.0/include  -D_USE_MATH_DEFINES  -DBOOST_DISABLE_ASSERTS  -DSTAN_OPENCL -DOPENCL_DEVICE_ID=0 -DOPENCL_PLATFORM_ID=0 -DCL_HPP_TARGET_OPENCL_VERSION=120 -DCL_HPP_MINIMUM_OPENCL_VERSION=120 -DCL_HPP_ENABLE_EXCEPTIONS -Wno-ignored-attributes   -c  -x c++ -o .\examples\bernoulli\bernoulli.o .\examples\bernoulli\bernoulli.hpp
g++.exe -std=c++1y -m64 -D_REENTRANT -Wall -Wno-unused-function -Wno-uninitialized -Wno-unused-but-set-variable -Wno-unused-variable -Wno-sign-compare -Wno-unused-local-typedefs   -I stan/lib/stan_math/lib/opencl_2.1.0  -O3 -I src -I stan/src -I lib/rapidjson_1.1.0/ -I stan/lib/stan_math/ -I stan/lib/stan_math/lib/eigen_3.3.3 -I stan/lib/stan_math/lib/boost_1.69.0 -I stan/lib/stan_math/lib/sundials_4.1.0/include  -D_USE_MATH_DEFINES  -DBOOST_DISABLE_ASSERTS  -DSTAN_OPENCL -DOPENCL_DEVICE_ID=0 -DOPENCL_PLATFORM_ID=0 -DCL_HPP_TARGET_OPENCL_VERSION=120 -DCL_HPP_MINIMUM_OPENCL_VERSION=120 -DCL_HPP_ENABLE_EXCEPTIONS -Wno-ignored-attributes       -L"C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v9.0\lib\x64" -lOpenCL   .\examples\bernoulli\bernoulli.o src/cmdstan/main.o  -static-libgcc -static-libstdc++    -lOpenCL  stan/lib/stan_math/lib/sundials_4.1.0/lib/libsundials_nvecserial.a stan/lib/stan_math/lib/sundials_4.1.0/lib/libsundials_cvodes.a stan/lib/stan_math/lib/sundials_4.1.0/lib/libsundials_idas.a stan/lib/stan_math/lib/sundials_4.1.0/lib/libsundials_kinsol.a  -o .\examples\bernoulli\bernoulli.exe
g++.exe: error: .examplesbernoullibernoulli.o: No such file or directory
make: *** [make/program:41: .\examples\bernoulli\bernoulli.exe] Error 1

Running last command manually (without make) works. I have no idea what could be the cause for this.

We fixed this separate issue with @t4c1 . PR is here: https://github.com/stan-dev/cmdstan/pull/739

Hmm, that fix is in but looks like there's still something failing with MPI that's probably stanc3 specific. I'll have to dig into that...

Oh wait, I think this is the same as your stanc3 ticket here: https://github.com/stan-dev/stanc3/issues/309 woops, didn't see the connection before. Thanks!

Yeah, that is how I "found" it. I should have mentioned it in the stanc3 issue.
This fix fixed something that was not caught on any of our tests, just something @t4c1 found.

I think this actually might just need to be changed over to use stanc instead of libstanc to compile all of these test models..

I think this is ready to review! Tests were passing but I just set off new ones because I needed to fix the accidentally checked in stan submodule 😖

The following test is failing:
TEST(StanUiCommand, zero_init_domain_fail) {
  std::string expected_message
    = "Rejecting initial value";

  std::vector<std::string> model_path;
  model_path.push_back("src");
  model_path.push_back("test");
  model_path.push_back("test-models");
  model_path.push_back("domain_fail");

  std::string command = convert_model_path(model_path) + " sample init=0 output file=test/output.csv";

  run_command_output out = run_command(command);
  EXPECT_EQ(int(stan::services::error_codes::SOFTWARE), out.err_code);

  EXPECT_TRUE(out.header.length() > 0U);
  EXPECT_TRUE(out.body.length() > 0U);

  EXPECT_EQ(1, count_matches(expected_message, out.body))
    << "Failed running: " << out.command;
}

The last test fails, the others are OK. Meaning that it probably cant find "Rejecting initial value"
The model it runs: https://github.com/stan-dev/cmdstan/blob/develop/src/test/test-models/domain_fail.stan
The other failed test is of the same nature on the same model. Does stanc3 handle this differently?

Oh woops, the Jenkins tests didn't pass last time, just the circle ci ones and github didn't notice. I'll dig into it, thanks!

This PR gets stuck on the Linux tests for some reason.

What's left to do for TBB? Is that almost done? If not let's just go ahead and merge, they shouldn't interact much. Feature freeze is fast approaching on Friday!

Travis bug was resolved, so we are just waiting for 1376 on math to finish. Yeah its fine to merge. Its probably the PTSD from "waiting for various test to finish" talking. Sorry. Lets merge.

I’m happy to wait a few hours haha
On Wed, Oct 9, 2019 at 08:41 Rok Češnovar<EMAIL_ADDRESS>wrote:

Travis bug was resolved, so we are just waiting for 1376 on math to
finish. Yeah its fine to merge. Its probably the PTSD from "waiting for
various test to finish" talking. Sorry. Lets merge.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/cmdstan/pull/737?email_source=notifications&email_token=AAGET3CACKAGKUIXC3TE6Q3QNXGRFA5CNFSM4I2BBLU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAXXZ5Q#issuecomment-539983094,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGET3ASKNL6RBVUHEBCEC3QNXGRFANCNFSM4I2BBLUQ
.


Group by item display - card view

s it possible to hide the group selection
Ive list view with group by item and im trying to display card view inside the group and try to avoid the group selection option.

When you created the issue, there was a template added to the form. Without any of the information provided that we requested, there's no context about your question. Please feel free to resubmit with some context.


Question on class/method/property location data when it has decorator(s)

I find it a little unintuitive that if you apply a decorator to a method (or other target construct such as a class), the method's location data does not include the decorator, given that the decorators: [] array is a property of the node.
Both the TypeScript and Flow parsers (the only others I believe support decorators) behave differently to Babylon here, their relevant method node location includes the decorator content.
I could not find any guidance on location data within https://github.com/tc39/proposal-decorators
Was this behaviour explicitly discussed and decided on in Babylon? Are there other precedents of nested nodes "overflowing" their parents?
Example Code

class Other {
    @foo({ baz: true })
    static get bar() { return this._bar; }
}

Live example: http://astexplorer.net/#/gist/c7692dd145ef9aaa2c9600f976257a2a/bf55ed4537b7520a6c19045d7e9fa4e6f3f11d14
...take a look at the highlighted method for Bablyon vs TypeScript and Flow (and the location data in the AST of course).

Hey @JamesHenry! We really appreciate you taking the time to report an issue. The collaborators
on this project attempt to help as many people as possible, but we're a limited number of volunteers,
so it's possible this won't be addressed swiftly.
If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack
community that typically always has someone willing to help. You can sign-up here
for an invite.

@JamesHenry IIRC this is fixed in decorators2

Indeed it is, thanks, @existentialism!
Updating my parser config to use decorators2 instead of decorators, however it clashes quite badly with the TypeScript support, because TS has more options for things like parameter decorators than decorators2 allows (cc @andy-ms)
Is there any reason not to apply the location fix to the decorators plugin as well? That works perfectly with TypeScript as far as I can tell.

(Updated the description a bit as this does not just apply to method decorators)

@JamesHenry not sure how others feel re: making changes to old decorator plugin, but I'm fine with it... and nice side effect: it'll help remove some comment-related hacking in prettier as well.

Yep, and I'm sure @vjeux will be happy about that 😄
@hzoo @danez How do you feel about me adding the location fix to the original decorators plugin?

(Other than test output) This is all that would need to change AFAICT: https://github.com/babel/babylon/compare/master...JamesHenry:target-decorators-location?expand=1#diff-71f5ec1ce64b229d1ebbcf5d9a486d8a

I'd be super happy indeed :p

Go ahead.

Thanks @jridgewell, PR opened here https://github.com/babel/babylon/pull/699

The PR was merged! Thanks all


Provisioning page with list or document library webpart fails

When I use Apply-PnPProvisioningTemplate, the standard webparts for document library and list no longer gets installed on the page. Other webparts work fine.
This behaviour seems to have begin very recently - last couple of days.
To reproduce:

Create a page with just a document library webpart.
Set the page to be the websites start page.
Connect to the site: Connect-PnPOnline -Url https://...
Get the template: Get-PnPProvisioningTemplate -Out c:\temp\page.xml -Handlers PageContents
Apply the same template the site: Apply-PnPProvisioningTemplate -Path C:\temp\page.xml

The page is now just a blank page - without the document library webpart. Same thing happens for list webparts, but for example text webparts work fine.
If I look into the trace log (Set-PnPTraceLog -On -Level Debug) I get this error message: Could not find the control with either ID = f92bf067-bc19-489e-a556-7fe95f508
720 or Name =  to install. Did you install your client side web part to this site?
It is a standard out-of-the-box webpart, so I have not "installed it".
If am running version 3.25.2009.1 of PnP Powershell.
The contents of the template is this:
<?xml version="1.0"?>
<pnp:Provisioning xmlns:pnp="http://schemas.dev.office.com/PnP/2020/02/ProvisioningSchema">
  <pnp:Preferences Generator="OfficeDevPnP.Core, Version=3.25.2009.1, Culture=neutral, PublicKeyToken=5e633289e95c321a" />
  <pnp:Templates ID="CONTAINER-TEMPLATE-0584A4AAAE374171AB9A224A4435A375">
    <pnp:ProvisioningTemplate ID="TEMPLATE-0584A4AAAE374171AB9A224A4435A375" Version="1" BaseSiteTemplate="GROUP#0" Scope="RootSite">
      <pnp:WebSettings NoCrawl="false" WelcomePage="SitePages/LFR.aspx" CommentsOnSitePagesDisabled="false" QuickLaunchEnabled="false" SearchScope="DefaultScope" SearchBoxInNavBar="Inherit" />
      <pnp:ClientSidePages>
        <pnp:ClientSidePage PromoteAsNewsArticle="false" PromoteAsTemplate="false" Overwrite="true" EnableComments="false" Title="LFR" ThumbnailUrl="" PageName="LFR.aspx">
          <pnp:Header Type="Custom" ServerRelativeImageUrl="/_LAYOUTS/IMAGES/SLEEKTEMPLATEIMAGETILE.JPG" LayoutType="NoImage" TextAlignment="Center" ShowTopicHeader="false" ShowPublishDate="false" TopicHeader="" AlternativeText="" Authors="[
  {
    "id": "i:0#.f|membership|lfr@mth.dk",
    "upn": "lfr@mth.dk",
    "email": "lfr@mth.dk",
    "name": "Leif Frederiksen",
    "role": "IT-udvikler, SharePoint"
  }
]" AuthorByLine="[
  "i:0#.f|membership|lfr@mth.dk"
]" AuthorByLineId="-1" />
          <pnp:Sections>
            <pnp:Section Order="1" Type="OneColumn">
              <pnp:Controls>
                <pnp:CanvasControl WebPartType="List" JsonControlData="{"id": "f92bf067-bc19-489e-a556-7fe95f508720", "instanceId": "93c78a74-ca09-48dd-962c-a00bb5e8c9a4", "n": "Dokumentbibliotek", "description": "Tilføj et dokumentbibliotek", "dataVersion": "1.0", "properties": {"isDocumentLibrary":true,"filterBy":{}}, "serverProcessedContent": {}, "dynamicDataPaths": {}, "dynamicDataValues": {}}" ControlId="f92bf067-bc19-489e-a556-7fe95f508720" Order="1" Column="1" />
              </pnp:Controls>
            </pnp:Section>
          </pnp:Sections>
        </pnp:ClientSidePage>
      </pnp:ClientSidePages>
    </pnp:ProvisioningTemplate>
  </pnp:Templates>
</pnp:Provisioning>

Same here. In a daily provisioning task I notice this appearing just this morning 1am CET.
Update of PnP PowerShell does not help.
Assuming something changed on the IDs or Configuration on the SharePoint Side I extracted a newly created SitePage and applied the PnP-Template which showed the same behaviour.
I don't see the List WebPart in Get-PnPAvailableClientSideComponents. But I don't know if it ever appeared in this command.

Same here, missing list web part from page

Update: https://twitter.com/LukusMuller/status/1311286890151149569

Just tested and can confirm that it is now working at my end.

I can also confirm that this is working again - thank  you for looking into this :-)


Fix for issue #25 - Creating welcome page

I have made the welcome page for the website.
The only issue left is that website does not directly go to the GitHub Dashboard application if user is already logged in.
The user will first land on the welcome page, after clicking the "Login to Github" button, user is redirected to the dashboard.

Honestly I think we could just make a separate issue for the 'Route to home if you're logged in' feature. People can still do development work with the welcome page as it is.
There have been some recent changes made to master, they'll have to be merged into this branch before I can merge it. After that it should be good to go!

@jkmdev ok I can create a new issue for the rest of the development. I will update this branch and push the final code today.


demo data missing

We can either re-upload or make an example that uses astroquery.
This needs to be fixed in the example notebook

fixed in 0ede1e4
Also removed deprecated teff model in configuration.yaml


Use appassembler

Also, update the corresponding documentation.

CC: @mziccard

LGTM. Will merge when tests pass.


Application doesn't auto restart after update.

After installing update and clicking re-launch app the app updates and then closes instead of restarting automatically.

Hi, thanks for creating an issue. Can you please provide some information about:
🌍  Your Environment

OS:  e.g. Win7 x64
Version:  e.g. 0.6.9 portable
PoE:  e.g. Steam 3.10.0/e English

OS: Win10 x64
Version: 0.6.4-9
PoE: Standalone 3.10.0E English.

This seems related to #411. I'll close this issue.


Got application/json as default response content type

Hi,
I'm having a problem with http server in the master branch. That is:
As stated in following function's description: httpd_resp_send, httpd_resp_send_chunk, httpd_resp_set_type; If no status code and content-type were set, the default content type is 'text/html'.
But when I send response without setting the content type as below:
httpd_resp_send(req, buf, buf_len);
The content type I get is application/json
I found that there may be a mistake here in file httpd_parse.c, line 570:
ra->content_type = (char *)HTTPD_TYPE_JSON;
I don't know if anyone having the same problem like me.
Or could any one confirm about the issue.
Thank you,

According to docs it should be text/html
https://docs.espressif.com/projects/esp-idf/en/latest/api-reference/protocols/http_server.html#_CPPv219httpd_resp_set_typeP11httpd_req_tPKc

Yes , that seems like a mistake. Line 570 should read
ra->content_type = (char *)HTTPD_TYPE_TEXT;

Thanks for letting us know. Will fix this issue ASAP


S3 Improvements

Proposed fix for #731 and #628

Adds generic testing for Files Adapters
Adds support for content type setting when uploading to S3.
Adds ability to enable S3 adapter tests with environment variables S3_ACCESS_KEY, S3_SECRET_KEY

Looks good! Can we move towards making this an external repo? Then FileAdapter implementers can run the tests without depending on Parse Server

I wrote the test to make it easier to move to another repo. We should merge that, first, then move all adapters to 3rd party repos.
@gfosco had some thoughts on that too


display:block anchors don't seem to work on mobile devices (touch events) when embedded in Salvattore layouts

We're using Salvattore to develop a new learning portal for our products: http://learn.pimoroni.com/
Each item in the list is a block level anchor which works great in desktop browsers but not on iPhone/iPad. As soon as I disable the Salvattore layout functionality it works fine (though obviously we lose our lovely responsive grid!).
I can't see how the way Salvatorre modifies the DOM would be causing this.

OK, I'm dumb. The problem wasn't even remotely connected to Salvatorre I'd just convinced myself it was. Apologies - case closed.


XML12 -> XML15

Description
Changed XML12 -> XML15 in config.py.template to update webtools URL used
Fixes #634
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[ ] Within a week
[x] When possible

Type of changes
Please select a relevant option:

[x] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[x] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[x] My changes generate no new warnings.
[x] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

@AnubhavG1999, along with each PR, please update the CHANGELOG too. Please see an example here: https://github.com/rokwire/events-manager/pull/631/files. Please let us know if you have any questions. Thanks.

@sandeep-ps Will do!


Error upon initialization. Deprecated method.

Kai@Frank MINGW64 /c/app/aina/var.www/html $ yo jhipster-primeng
WARNING! This method is deprecated. Use getAllJhipsterConfig
Error jhipster-primeng
ERROR! Can't read .yo-rc.json
Kai@Frank MINGW64 /c/app/aina/var.www/html $

What is the JHipster version you are trying. I didn't find this issue with latest version. Let me check my logs one more time

brand spanking new, directly from the website.

could it be beacuse i'm using windows? i'm still an amateur programmer. i'm a system design expert, but learning programming, which is completely abstractions made up of 100,000 peoples ideas of how the system should work, and hodge-podging it all together is where the true talent i feel lies in java.. making stuff that shouldn't work together somehow magically work, and 20 times faster, at 1/50th the size, while looking cooler. java... special kind of torture but can't argue with results.


Improve docgo generation in Makefile

Description
Minor update on Makefile to avoid modification of go.mod and go.sum when running docgo
Fixes #147
Type of change

Refactor (refactoring code, removing useless files)
Documentation update

Testing steps
Regular CI

Codecov Report

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

@@           Coverage Diff           @@
##           master     #148   +/-   ##
=======================================
  Coverage   49.31%   49.31%
=======================================
  Files          12       12
  Lines        1022     1022
=======================================
  Hits          504      504
  Misses        460      460
  Partials       58       58

Continue to review full report at Codecov.

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


Mcpe for intel processors

i am not aware of a mcpocket edition made specially for intel processors,either it's the program detecting it wrong,as i can run mcpe on my zenfone 2 deluxe unmodded,or this still has not been fixed.
p.s. i know,i am using latest beta.
please help,really liked modded mcpe when using snapdragn cpu,never thought cpu would be a problem.

Download the one from the play store.

Can't i use amazon version? Worked before,but not this device....

Yea it should work if it got updated for intel.


Unable to run serve

Dear Team,
I am unable to run gulp serve.
npm i and npm i -g gulp run perfectly. I am new to create a webpart
Attched the error:

Looks like you are in the wrong folder. You need to move to the folder that you want to run.

Were to move
Get Outlook for Androidhttps://aka.ms/ghei36
Sheik Naveed
شيك نافيد
Academic Systems Administrator
مسؤول النظم الأكاديمية
Smart Servicesالخدمات الذكية
Tel: 800- MBRU (6278)
Direct +9714 383 8606
<EMAIL_ADDRESS>www.mbru.ac.ae
P.O. Box 505055, Dubai,  UAE

From: Ashik Paul<EMAIL_ADDRESS>Sent: Wednesday, July 8, 2020 6:36:39 PM
To: OlivierCC/spfx-40-fantastics<EMAIL_ADDRESS>Cc: Sheik Naveed<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [OlivierCC/spfx-40-fantastics] Unable to run serve (#124)
This message is from an external sender. Please take care when responding, clicking links or opening attachments.
Looks like you are in the wrong folder. You need to move to the folder that you want to run.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOlivierCC%2Fspfx-40-fantastics%2Fissues%2F124%23issuecomment-655559909&data=02|01|naveed.sheik%40mbru.ac.ae|88e442bc73fe4279636208d8234c5464|c9a341c74b6f4e56b8d9f2325c282fda|0|0|637298158032112430&sdata=UV2xh5lgmVhw8R%2FmcOS7kTepU9oOnbLa87ea6ESaC0s%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOKZL5JJ45FXD3LHZB4R47DR2R77PANCNFSM4MRGY6MA&data=02|01|naveed.sheik%40mbru.ac.ae|88e442bc73fe4279636208d8234c5464|c9a341c74b6f4e56b8d9f2325c282fda|0|0|637298158032112430&sdata=8XwLAZvnUZQhVUmx%2FZtTznctVCi8EuP8W0tBlQ04VDY%3D&reserved=0.


Download Demo

Is the demo previewed on https://unitylist.com/p/mib/Unity-Gameplay-Ability-System still avaliable to download?

That's a very old release. There have been 2 complete rewrites since then.
The main GitHub page has the latest release.
I haven't released a demo for the latest iteration, but feel free to
download from the latest branch to check out the progress.
I'm hoping to complete documentation this month or next for the core
concepts as well as a demo/tutorial.
On Tue, 6 Apr 2021, 12:13 am Abdallah Akrab, @.***>
wrote:

Is the demo previewed on
https://unitylist.com/p/mib/Unity-Gameplay-Ability-System still avaliable
to download?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sjai013/UnityGameplayAbilitySystem/issues/17, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB7VQS4OME5INRLGWEZIHRTTHHARVANCNFSM42M334EQ
.

Thank you for your response.
I think I will wait for the tutorial , for me , as a beginner it is not very clear how to use the framework in existing project ex: unity 3D Kit

@abdallahakrab Give this a go now - the Wiki has some useful information as well as the main repository README.  The documentation is still in progress, but hopefully this will give you a starting point.  If you have any issues, raise a new issue and I'll try to sort it out.

Thank you


Feature Request: Allow developer to use JavaScript or TypeScript

I know I'm not alone in this when I say that I'm a JavaScript developer that has never touched TypeScript. It would be great if there was a way to allow the developer to use JavaScript instead of TypeScript.
Additionally in previous versions of Visual Studio (if I remember correctly) Microsoft allowed for extensions to extend the One ASP.NET dialog to provide your own options when creating a new project. I'm not sure if this changed with recent versions of Visual Studio but it would be great if when selecting the project type if the developer could choose between JavaScript and TypeScript.

Hello @tylerbhughes
Thanks for your request. That's a nice idea.
Well, I will think about that in my spare time.

JavaScript version of the React Core Boilerplate is successfully ported.


launch.json and tasks.json not added when moving from project.json to .csproj

Environment data
dotnet --info output:
.NET Command Line Tools (1.0.0-preview4-004110)

Product Information:
 Version:            1.0.0-preview4-004110
 Commit SHA-1 hash:  740a7fe3fd

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.11
 OS Platform: Darwin
 RID:         osx.10.11-x64
 Base Path:   /usr/local/share/dotnet/sdk/1.0.0-preview4-004110

VS Code version: 1.6.1
C# Extension version: 1.5.3
Steps to reproduce

Download the latest dotnet cli tools (preview 4)
mkdir foobar
cd foobar
dotnet new
code .

Expected  behavior
I expect that Visual Studio Code would prompt me to add missing assets (launch.json and tasks.json).
Actual behavior
Visual Studio Code does not prompt me to add missing assets. I'm guessing this is because whatever was adding those assets was keying off the presence of project.json, which no longer exists in the new project model.
Modifying sdk.version in global.json and changing the versions back and forth between the old and new models seems to confirm this behavior.

This is a duplicate of https://github.com/OmniSharp/omnisharp-vscode/issues/767.


when unlinking files do not remove but return to upload folder

After taking a look at this issue, we have 3 options: (1) rename and change the remove_filepath methods within the rawdata so they move the files to the upload folder, (2) add a flag (move_to_upload_folder=True) to those methods so the files are not removed but moved, or (3) leave those methods as they are and create new ones with the requested functionality. @josenavas @wasade what do you think?

My vote goes for (1), but we should be able to delete files from the upload folder.

OK, thanks. Agree about the upload folder but that's a separate issue.

:+1:

@josenavas After a second look I think it will be better to write a new methods to replace purge_filepaths to move_filepaths_to_upload_folder, mainly because the raw data will still remove the file paths but at the end we can simply replace that function so the files get moved vs. purged.


warning if conflicting licenses get detected

What would be really great (in my opinion) would be if legally could emit warnings if it detects that you are about to use packages that have a license which is not "compatible" with your own project license ... The best resource I could find so far is the page of the free software foundation about licenses compatibility (the is more in their wiki and on their FAQ page). I also found a chart on wikipedia comparing open source licenses and showing which one are compatible with the GPLv3 and which not.
You can close this after reading, as I don't think this is doable right now and without the help of a bunch of legal advisors :( but as stated on the apache foundation GPL-compatibility page, maybe the software freedom law center might be able to help out!?

This would be nice but I do not plan on implementing this for the complexity but feel free to send a PR with this functionality.

Also, this is a duplicate, check https://github.com/franciscop/legally/issues/3


lib/options: multi-line plugin default strings

Detect whether a plugin default is multiline using hasInfix "\n".
This works fairly well, but without trimming the default string we may run into some false-positives. We could borrow the implementation from https://github.com/NixOS/nixpkgs/pull/315411 or wait and see if/when that gets merged. Personally, I don't see this FIXME as a blocker, unless we find it affects a significant portion of the docs.
I also noticed some default strings seem to have excessive indentation, IDK how easily we could remove that or if we should even bother. I left a TODO comment.
Here's a screenshot of docs built with this change:
Screenshot

Note how 2/3 options have inline plugin defaults, while the first option has a multiline plugin default.

I have checked locally and it looks great !


UnicodeEncodeError: 'latin-1' codec can't encode characters in position 42-45: ordinal not in range(256)

I've cloned the repo and installed it using the virualenv method.
When running the app without providing --language and/or --mnemonic_language, I get an exception.
After adding print statement to debug the error, it originated from the app attempting to print non-unicode characters as shown below.
I've solved the problem by manually passing those parameters.
Debug output:
b"Please choose your mnemonic language ['1. \xe7\xae\x80\xe4\xbd\x93\xe4\xb8\xad\xe6\x96\x87', '2. \xe7\xb9\x81\xe9\xab\x94\xe4\xb8\xad\xe6\x96\x87', '3. \xc4\x8de\xc5\xa1
tina', '4. English', '5. Italiano', '6. \xed\x95\x9c\xea\xb5\xad\xec\x96\xb4', '7. Portugu\xc3\xaas', '8. Espa\xc3\xb1ol']:  [english]: "

Stack Trace
Traceback (most recent call last):
  File "/root/staking-deposit-cli/./staking_deposit/deposit.py", line 61, in <module>
    cli()
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/core.py", line 1257, in invoke
    sub_ctx = cmd.make_context(cmd_name, args, parent=ctx)
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/core.py", line 700, in make_context
    self.parse_args(ctx, args)
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/core.py", line 1048, in parse_args
    value, args = param.handle_parse_result(ctx, opts, args)
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/core.py", line 1623, in handle_parse_result
    value = self.full_process_value(ctx, value)
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/core.py", line 1964, in full_process_value
    return self.prompt_for_value(ctx)
  File "/root/staking-deposit-cli/staking_deposit/utils/click.py", line 49, in prompt_for_value
    return super().prompt_for_value(ctx)
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/core.py", line 1933, in prompt_for_value
    return prompt(
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/termui.py", line 147, in prompt
    value = prompt_func(prompt)
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/termui.py", line 128, in prompt_func
    echo(text, nl=False, err=err)
  File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/utils.py", line 273, in echo
    file.write(message)
UnicodeEncodeError: 'latin-1' codec can't encode characters in position 42-45: ordinal not in range(256)

Hey this has been open for a while I see and I’ve just dug into a portion of that code. These aren’t Latin-1, there’s Arabic in there for example. This should render as utf-8 no problem. This sounds like an issue with your environment. Make sure it’s Python 3 not 2, and a recent version of Python 3 at that. “It works here” - you just got to figure out what’s different in your environment.

Hi Thanks for taking the time to look into it.
In the traceback you can see lines like this:
File "/root/staking-deposit-cli/env/lib/python3.9/site-packages/click/core.py", line 1964, in full_process_value

Here it clearly shows that not only am I using python3 but a fairly recent version too.
I think either the app drop support for non-latin languages or use utf8 instead of latin1 as you've suggested.
My environment is English but even if it was Arabic, this error shouldn't have happened (I think)

Yeah not sure. I build in an English environment, straight Ubuntu, and have not had this issue. The app indeed uses utf-8. Maybe it's an environment configuration thing?

Is it possible to use a different language (say Arabic) in --language ?  If that works fine we can close this ticket until another person face same problem. If that raises an exception similar to the one I reported, we see how to proceed after.
I don't have the VPS/environment anymore that I tested on otherwise I would tried it

.\deposit.exe existing-mnemonic --help
Using the tool on an offline and secure device is highly recommended to keep your mnemonic safe.
Please choose your language ['1. العربية', '2. ελληνικά', '3. English', '4. Français', '5. Bahasa melayu', '6. Italiano', '7. 日本語', '8. 한국어', '9. Português do Brasil', '10. român', '11. Türkçe', '12. 简体中文']:  [English]: 7
Traceback (most recent call last):
File "staking_deposit\deposit.py", line 63, in
File "click\core.py", line 829, in call
File "click\core.py", line 782, in main
File "click\core.py", line 1257, in invoke
File "click\core.py", line 700, in make_context
File "click\core.py", line 1048, in parse_args
File "click\core.py", line 1630, in handle_parse_result
File "click\core.py", line 123, in invoke_param_callback
File "click\core.py", line 950, in show_help
File "click\core.py", line 570, in get_help
File "click\core.py", line 975, in get_help
File "click\core.py", line 1003, in format_help
File "click\core.py", line 1024, in format_options
File "staking_deposit\utils\click.py", line 56, in get_help_record
File "staking_deposit\utils\click.py", line 19, in value_of
File "staking_deposit\cli\existing_mnemonic.py", line 34, in
File "staking_deposit\utils\intl.py", line 68, in load_text
File "json_init.py", line 293, in load
UnicodeDecodeError: 'cp950' codec can't decode byte 0xe6 in position 64: illegal multibyte sequence
[49636] Failed to execute script 'deposit' due to unhandled exception!


Update UserGuide.md and AboutUs.md

Fixes #1 Fixes #2

Used wrong branch. Reverted changes to branch and created new PRs from another branch


Changes lost after reloading

Hi,
thank you for providing this great exension to Trix. I am trying to add some extensions like the ones provided by your code to an app, but need to run it within a Rails 4 environment. I am using the vanilla Trix JS&CSS with your trix-extensions added.
They seem to work fine - I can apply your custom attributes exactly like I would expect - and the changes are persisted correctly in the backend. However - when I reload the content on the same page, the custom formatting is lost, presumably removed by some sanitising code in Trix.
Do you have any suggestions on how this could be fixed ? I've been pulling my hair out for a while on this issue
Here's a visual of what happens:
https://www.dropbox.com/s/y0fsnxa31s4ymaf/trix-editor-custom-attributes-not-persisting.mov?dl=0
This is what is saved to the backend - and then loaded again:

Bold is working:

The quick brown fox jumps over the fence. The slow fox waits and ponders.

Any help in getting this last step to work would be greatly appreciated Lázaro !

And...after 8 months I finally figured this out for myself. I had 'deferred' the loading of the trix JS....once I loaded things in the right order they worked perfectly.

@patsch I have the exact same problem and I've also concluded it's due to JS loading issues. What was your solution?
I'm using a different setup (Rails 6, Rails Admin, webpacker), but maybe your solution can give me a clue.

The key problem was that I was loading the trix JS explicitely on those pages where it was needed via
<%= javascript_include_tag "trix.js", defer: true %>
<%= stylesheet_link_tag 'trix.css', media: 'all' %>

The defer: true was what caused all the problems; I didn't want to load trix for all pages and include it in application.js, but that's what I ended up doing; changing the defer to false on its own also fixed it for me.

Ok thanks for your response!

I'm having the same issue here, it's been months trying to solve it,
and I couldn't! I really don't know is happening with this editor.
I tried all the solutions out there
@patsch could you give me some help?

I'm having the same issue here, it's been months trying to solve it,
and I couldn't! I really don't know is happening with this editor.
I tried all the solutions out there
@patsch could you give me some help?

@Hussam3bd Do you have a publicly accessible webpage that exhibits the problem ?

@Hussam3bd Do you have a publicly accessible webpage that exhibits the problem ?


⚠ Bofrost.si has degraded performance

In 548b324, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 4550 ms

Resolved: Bofrost.si performance has improved in cc24e69.


Copy-Paste in some editors

Windows Terminal version
1.13.10336.0
Windows build number
10.0.22000.0
Other Software
micro:
Version: 2.0.10
Commit hash: b9763856
Compiled on August 07, 2021

far:
Far Manager, version 3.0.5961.0 x64
Copyright © 1996-2000 Eugene Roshal, Copyright © 2000-2022 Far Group

Steps to reproduce
Paste copied text that contains leading spaces in an editor.
Expected Behavior
Text should be pasted as is, without corrupted formatting
Actual Behavior
Leading spaces grows with each line. The behavior is the same in both editors (  far -e micro  )
This is not observed in any other terminal where launched editor.

I can't replicate the issue with Far Manager 3.0.5961.0 x64.
micro indeed produces the behavior from the screenshot above. Curiously however this only happens when run natively under Windows. If you run micro with WSL2 inside Windows Terminal everything works as expected, even though we're still the exact same terminal. Could this perhaps be a bug in micro? 🤔

I can't replicate the issue with Far Manager 3.0.5961.0 x64

I turned off autoindent in far:config. And autoindent set to false in micro's settings.json  And this solve the problem in both editors. I do not know why the terminal affects autoindent.

relevant: #7041, #9034. I'd bet it's related to bracketed paste mode.


Adding the B-Tree algorithm in the T-Tree folder

Related Issue
#51
Proposed Changes
Change 1 = Added B-Tree Algorithm with comments and procedure.
Change 2 = Added Readme for B-Tree Algorithm.
Checklist

[x] ✅ My code follows the code style of this project.
[x] 📝 My change requires a change to the documentation.
[x] 🎀 I have updated the documentation accordingly.
[x] 👀 I have read the CONTRIBUTING document.
[x] ✨ I have added tests to cover my changes.
[x] 🚩 All new and existing tests passed.
[x] 🌟 ed the repo
[] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[] Breaking change (fix or feature that would cause existing functionality to change)

Output Screenshots

Test Case

Code Snippet

Hey @ayushi1210  , make sure your Pull Request is descriptive and complete the checklist .
I would recommend you to go through this tutorial video of the project here ,thanks.
For contribution and more guidelines i would recommend you to go through this
tutorial video of the project here, thanks.
Please make sure your file conventions and extensions are correct .

T-Tree/B-BTree/BTrees.cpp

Let me show you how to check the check list , there should be no space inside brackets like [x]
[x]
Checklist
Or you can just check them from outside without doing any of these
✨ I have added tests to cover my changes.
🚩 All new and existing tests passed.
🌟 ed the repo
Starr the repo if you haven't
Have you watched this video

Now I have corrected the checklist and make the readme as per the video and full description of the algorithm I have provided in the readme and the in code as comments.

Please review this PR.

Remove ticks from these 2

I dont think your cpp file is correct..it has not any cpp extension

Add more points on readme->

Description
image
algo
time&Space complexity
Advantages & Disadvantages of using B-Tree

I did all the changes suggested and as mentioned in the issue I have to submit c file only.

I did all the changes suggested and as mentioned in the issue I have to submit c file only.

It's CPP Project add C++ Code

All the recommended changes are done.

All the recommended changes are done.

We cannot take a PR with these much commits ...create a Fresh PR and You have changed 3 files
You need to follow particular guidelines for your README , please follow
these guidelines click here
also you can see how your README should look like see here
for more please go through these here


Add Italian

I've split this PR into two commits. The second just involved pulling the translations and running destructure.js. The first sets up i18n for Italian, which was just a few lines. I'll continue updating translations as they're modified and will force-push changes to the second commit. However, this PR should be ready for review.


Help with CUDA error: invalid device ordinal

I'm encountering an error in my code that says "RuntimeError: CUDA error: invalid device ordinal
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
". I've checked that I have a compatible GPU and that it is properly installed on my machine, and I've installed the correct version of CUDA and the NVIDIA driver.
However, I'm still encountering this error and I'm not sure what to do next.
I would appreciate any help or advice you can provide on how to solve this issue. Please let me know if you need any more information or if there are any specific steps I should take to troubleshoot the problem.

please paste the complete error message (stacktrace).

in the torchrun command, I changed --nproc_per_node=4 to --nproc_per_node=1 and that fixed this error for me.

in the torchrun command, I changed --nproc_per_node=4 to --nproc_per_node=1 and that fixed this error for me.

Thanks, this really helped.

@GooDRomka where you able to train the model on single A100?


BUG: Custom model missing config.json when launching

Describe the bug
A clear and concise description of what the bug is.
  File "/home/codingl2k1/anaconda3/lib/python3.11/site-packages/transformers/configuration_utils.py", line 620, in get_config_dict
    config_dict, kwargs = cls._get_config_dict(pretrained_model_name_or_path, **kwargs)
    ^^^^^^^^^^^^^^^^^
  File "/home/codingl2k1/anaconda3/lib/python3.11/site-packages/transformers/configuration_utils.py", line 675, in _get_config_dict
    resolved_config_file = cached_file(
    ^^^^^^^^^^^^^^^^^
  File "/home/codingl2k1/anaconda3/lib/python3.11/site-packages/transformers/utils/hub.py", line 400, in cached_file
    raise EnvironmentError(
    ^^^^^^^^^^^^^^^^^
OSError: [address=<IP_ADDRESS>:43560, pid=16055] /home/codingl2k1/.xinference/cache/qwen-chat-lora-pytorch-14b does not appear to have a file named config.json. Checkout 'https://huggingface.co//home/codingl2k1/.xinference/cache/qwen-chat-lora-pytorch-14b/None' for available files.

To Reproduce
To help us to reproduce this bug, please provide information below:

Your Python version.
The version of xinference you use.
Versions of crucial packages.
Full stack of the error.
Minimized code to reproduce the error.

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

still face this problem in version 0.11.0


feat(amplify-cli-core): visually hide "immutable files" for VSCode users

I'm proposing a developer experience improvement to Amplify, specifically targeting new users.
Consider the following scenario:
A new user decide is trying out Amplify for the first time.
Forgetting to use the CLI, they attempt to modify the metadata files directly.
They didn't read the docs, and certainly the not the small section at the end explain the Reference > Files and Folders section.
So the only outcome is they get stuck. This causes frustration and a bad new user experience.
Good thing it's totally avoidable. This change visually hides those metadata files so new users aren't tempted to fiddle with them.
With basic workspace configuration settings, we can provide a better user experience for new users by directing their energies and focus towards things in their control, like the templates and source code.
Of course, these defaults are easily opted out of by updating the settings.json file.
Issue #, if available:
Description of changes:
If a user selects VSCode during amplify init, create or safely update their .vscode/settings.json file to exclude immutable files.
Based on Files and Folders where manual edits are not allowed
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Feedback and open dialog encouraged! I discussed this feature with a few other devs, but would love to get a wider range of opinions on it. Thanks and looking forward to hearing your thoughts.

Pasting in my message from Discord:
I'm actually curious what the other devs in this channel think about it?
I see arguments for both sides: 1. "Hide all the files that I'm not supposed to modify anyway!" or 2. "I want to see everything that's going on in my project. This isn't just a big dependency dump folder like node_modules. I don't want to accidentally modify a folder thinking that there's nothing inside"
cc @sw-yx @dabit3 @aspittel any opinions on this from your experience?

To provide an example:
With excludes:

Without excludes:

I'm actually pretty for it. I've taught create-react-app a ton of times and it always terrifies new developers to see all the files (even though I tell them right away they don't need to touch most of them). I'm pro as few visible files as possible, it feels simpler.

For me I'd say that if I'm likely to never touch it ever, then I don't want to see it (i.e. maybe #currentcloudbackend & any others that are similar)

:tada:

Just retested this in in a few configurations locally and everything looks good.
As a note for monorepo projects with multiple amplify apps in different packages. This will create and apply .vscode settings files for each package, but they will only take effect if you open vscode with that package as the workspace route. I think that's the best user experience anyhow.


Charge only does 25 damage instead of expected 306

Hurm
01:19:03.340 FAILED TO APPLY MELEE DAMAGE TO ATTACKER!
01:19:03.340 Object reference not set to an instance of an object
01:19:03.340   at (wrapper dynamic-method) BattleTech.Weapon.WillFireAtTargetFromPosition_Patch2(object,BattleTech.ICombatant,UnityEngine.Vector3,UnityEngine.Quaternion)
at BattleTech.AttackDirector+AttackSequence..ctor (BattleTech.AttackDirector director, BattleTech.AbstractActor attacker, BattleTech.ICombatant target, UnityEngine.Vector3 attackPosition, UnityEngine.Quaternion attackRotation, System.Int32 attackSequenceIdx, System.Collections.Generic.List1[T] selectedWeapons, BattleTech.MeleeAttackType meleeAttackType, System.Int32 calledShotLocation, System.Boolean isMoraleAttack, System.Int32 id, System.Int32 stackItemUID) [0x00111] in <4184af8dbeb44635831353f4d349631c>:0    at (wrapper dynamic-method) BattleTech.AttackDirector.CreateAttackSequence_Patch2(object,int,BattleTech.AbstractActor,BattleTech.ICombatant,UnityEngine.Vector3,UnityEngine.Quaternion,int,System.Collections.Generic.List1<BattleTech.Weapon>,BattleTech.MeleeAttackType,int,bool)
at CBTBehaviorsEnhanced.Helper.AttackHelper.CreateImaginaryAttack (BattleTech.Mech attacker, BattleTech.Weapon attackWeapon, BattleTech.ICombatant target, System.Int32 weaponHitInfoStackItemUID, System.Single[] damageClusters, BattleTech.DamageType damageType, BattleTech.MeleeAttackType attackType) [0x001cd] in :0
at CBTBehaviorsEnhanced.Melee.MechMeleeSequence_OnMeleeComplete.Prefix (BattleTech.MechMeleeSequence __instance, MessageCenterMessage message, BattleTech.AttackStackSequence ___meleeSequence) [0x00285] in :0
01:19:03.340FAILED TO APPLY MELEE DAMAGE TO ATTACKER!

BTALogs-20210908_212400.zip

This was fixed with corrections to imaginary weapon


other areadetector file plugins need warmup() method

A warmup() method was implemented for the HDF5 plugin: https://github.com/bluesky/ophyd/blob/19269911c64594ca85e27c01dd0bfd86122b8400/ophyd/areadetector/plugins.py#L889-L903
As noted in https://github.com/bluesky/ophyd/issues/598#issuecomment-420327292 (merged in #599), other file-saving plugins need this capability, too.  We've encountered this a couple times at APS (thanks to @KedoKudo for reporting this) with a TIFF plugin and also with a custom file-saving plugin.  Should be the existing warmup() method move to the FilePlugin class?

There has to be a better way to do this.

Does anyone have time to find one? If not, I think we should move forward with this as a stop gap.

I think the warmup() method should check the port flow map (if there is one) to make sure that it can actually get an image from the camera plugin.  There could be situations where the file plugin is pointing to another processing plugin, which happen to be disconnected from the camera.


Corrects usage of refresh token in jwt client.

Fixes #28

Coverage remained the same at 92.21% when pulling 758adcb2f4e71708b28de34db376ca418ef2f1c7 on tbetbetbe:nodejs-auth-correct-refresh-token into 7601f5eb2b68450247d04b538c9de19b0d529852 on google:master.


Mark Ubuntu as recommended on the recommended list

From mjfet<PHONE_NUMBER>49961789):
On the supported distro list, Ubuntu LTS is set as Supported, but not recommended, which seeing the change for the recommended list could mean for a change to make it Recommended as well.

From V.XLII Senhara | she/her<PHONE_NUMBER>36010755):
Agreed, I'll ping Syretia about it as I can't modify it myself


Views being supported?

Discussed in https://github.com/9tigerio/db2rest/discussions/204

Originally posted by thadguidry January 27, 2024
Also, good to support:

We need to have a view that has 2 owners, and where we do a delete operation by 1 owner, and then a get operation by the other owner to verify the row was deleted.
And supporting materialized views additionally. https://www.postgresql.org/docs/16/view-pg-matviews.html  and https://www.postgresql.org/docs/16/rules-materializedviews.html
because of:

While access to the data stored in a materialized view is often much faster than accessing the underlying tables directly or through a view, the data is not always current; yet sometimes current data is not needed.
(edited)

Views can also sometimes allow INSERT, DELETE, UPDATE

A view can also have other kinds of DO INSTEAD rules, allowing INSERT, UPDATE, or DELETE commands to be performed on the view despite its lack of underlying storage. This is discussed further below, in Section 41.2.4.

https://www.postgresql.org/docs/16/rules-views.html (edited)

@thadguidry - View and materialized views already supported - has to be documented.
Explain this in details with example
We need to have a view that has 2 owners, and where we do a delete operation by 1 owner, and then a get operation by the other owner to verify the row was deleted.


[Bug report] Sidebar heading links path missing

Description
The URL path is missing for links to headings in the sidebar. This is also occurring on the docs of the default theme. This issue can be seen when hovering over any of the links to headings on the provided reproduction page.
Reproduction
https://ecosystem.vuejs.press/themes/default/config.html
Used Package Manager
npm
System Info
-

so what's them problem?


ASMO-4754 [ Samhammer.Logging] Update github action

This is only used for HD team so we dont merge and leave it for HD team to review :)


bump version 0.1.12.dev

Just a reminder that we should tag v0.1.12 in the git history and make a PyPI release that does not include these latest commits.


Don't force lowercase for claim names and descriptions

Certain URLs require that uppercase/lowercase URLs be understandable. For example, I was not able to check this google docs link because the app made all uppercase letters lowercase:

We need to ensure that the "force lowercase" rule DOES NOT apply to bounty claim NFT names or descriptions.

https://github.com/picsoritdidnthappen/poidh-app/blob/8f93ac9c5a6f0de0e77bde1816b2efee589d4790/src/components/bounty/ProofItem.tsx#L142
edit line 144 and 145
{title}
{description}

Pull Request Created


Update version

@JuliaRegistrator register()

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

@JuliaRegistrator register()

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

@JuliaRegistrator register()

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

@JuliaRegistrator register()

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

@JuliaRegistrator register()

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

Registration pull request created: JuliaRegistries/General/96267
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.8.5 -m "<description of version>" bd62b1237b384c22855ed17a6650fa83b3e3ae88
git push origin v0.8.5

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/96267
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.8.5 -m "<description of version>" 564fd03280fab2dcf7acff04b6edda17de314db4
git push origin v0.8.5

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/96291
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.8.6 -m "<description of version>" 78326a4955275a6b6172749377fe9ab7660337f0
git push origin v0.8.6

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/97548
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.8.7 -m "<description of version>" 2e667a5c851416b538c5ca2e03da8de2f3f9cb53
git push origin v0.8.7

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/102632
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.9.0 -m "<description of version>" 4091d6d925049bdeeac8be44c3942c4a8a2e443f
git push origin v0.9.0


Change state case for rtorrent

Description:
Fixed the case of sensor.rtorrent
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

This requires a paragraph for the release notes as it's a breaking change.

@fabaff where I can provide them?

As comment.

Changed the returned states of sensor.rtorrent from Up/Down,
Downloading and Seeding to up_down, downloading and seeding to
reflect the guidelines. Hence by automations relying on the state of this
sensor should be updated.
On Thu, Nov 29, 2018, 16:25 Fabian Affolter<EMAIL_ADDRESS>wrote:

As comment.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/18778#issuecomment-442873566,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH2OZxoM2CDX5CSeAjsMzLQe3wqNAWc2ks5uz_xMgaJpZM4Y5avE
.


why the arrow position offset seems wrong

You see? the red arrow seems offset +1 right

maybe you just need to change your terminal font


Upgrade TiCDC from v6.2 to v6.3 takes a long time

What did you do?
Upgrade v6.2 TiCDC cluster to v6.3, it takes about 100s for every TiCDC instance.
What did you expect to see?
Should be with in 10s.
What did you see instead?
Takes about 100s to shutdown a TiCDC instance.
Versions of the cluster
TiCDC version (execute cdc version):
v6.2.0

/label affects-6.2


V5 audioplayer2

🤺

Here's a fork of Cookbook changed to this player, just for an example. Uses an extension to support the schedule methods.
https://github.com/ryanfrancesconi/Cookbook/tree/audioplayer2

Looks good to me. Are you planning to do fade in/out in a subsequent change?

Definitely could. I'm just not sure what things are desired and what aren't. I don't want to overdo it like i did last time.

I think you can go ahead and make this be "AudioPlayer" right in this PR. Involves deleting some files, renaming some, etc.

I think you can go ahead and make this be "AudioPlayer" right in this PR. Involves deleting some files, renaming some, etc.

you want me to remove the old AudioPlayer?


Not all docs being included in report

Ember-cli recently started using inch-ci, and we noticed that several files aren't being included in the report somehow, even though they do have docs. This one, for instance is probably one of the more heavily documented files in the project, but nothing shows up for it in the report: http://inch-ci.org/github/ember-cli/ember-cli/branch/master
I was curious if maybe something in the docs was tripping up the worker parsing the files, or something to that effect. Please let me know if you need any more info!

Hi there,
I fixed the problem with the missing objects (I did not properly detect members of modules tagged via @module). The result is a better report, but it still has its flaws.
For example, I need to fix this object that is tagged as @private but its members show up in my report as "undocumented".
Thanks for the bug report! :+1:

Awesome, thank you for fixing that, our report looks MUCH better! Please let me know if we can do anything from our end to help with improving this!


Request to delete

URL: https://pub.dartlang.org/packages/flutter_morse
Email is public, name is misleading, it's been mocked on r/FlutterDev and I'd rather keep the package on GitHub.

URL: https://pub.dartlang.org/packages/install_apk
I published it by mistake

I've sent uploaders an email, waiting for confirmation.

Request have been confirmed...
btw, I think it's a pretty cool project :)
Not that I have clue what to use it for... hehe.. maybe writing an app :)

package flutter_morse has been delete per request.
@epireyn, please file a issue bug for install_apk.


[Feature Request]: Get it to work on macos via crossover?

Would be nice for this to work via crossover but as of now it will just reinstall the entire game when i try to launch it

This is probably not something I can really help with. I don't have any access to Mac hardware so I wouldn't be able to test anything. Apologies!


How to update relationships

Hi there,
thanks so much for writing vuex-orm, it's great!
I think the documentation is lacking a little bit information (and that relates to my question):
How can one update relationships?
E.g. my Usecase is the following:
export default class Container extends Model {
  static entity = 'containers'

  static primaryKey = 'uid'

  static fields() {
    return {
      uid: this.attr(null), // is actually the binding.id but does not matter

      binding: this.hasOne(Binding, 'container_uid'),
    }
  }
}

export default class Binding extends Model {
  static entity = 'bindings'

  static fields() {
    return {
      "id": this.number(),
      "container_uid": this.attr(),

      "container": this.belongsTo(Container, 'container_uid')
    }
  }
}

I hope this is correct like this.
When I want to update now one container and replace a binding with another binding
// newBinding is an instance of Binding
// container is an instance of Container, it has a container.binding already set
// i like to modify the state in vuex-orm: container.binding === newBinding
swap({dispatch}, {container, newBinding}) {
  dispatch('entities/????/update', {
    ????
  })
}

That's all unclear to me:

Which side of the relationship do I have to update? binding or container?
Do i need to update the container_uid foreign key?
Do i have to update the binding value in container?
Do I have to update the binding.container value in container (as well)?
Do I have to manage the inverse side as well?
Can I use the model here directly? Or do I need to "reduce" them to uid/id?

All examples in documentation for insert/update/insertOrUpdate are with json data, but I have the real models here.
Thank you so much!

okay, i found my solution here - but some questions are still open
https://github.com/vuex-orm/vuex-orm-examples/blob/master/src/components/TodosAssignee.vue

@pscheit Hi! Thank you for the feedback!
You should update the record you want to update. Currently Veux ORM doesn't offer any sugar syntax to update the relationship. The only value you have to update is the container_uid at Binding model in this case (because that is the key connecting two models).
So, it should be something like,
swap({ dispatch }, { container, newBinding }) {
  // Delete old binding.
  dispatch('entities/bindings/delete', container.binding.id, { root: true })

  // Create new binding with correct container_uid.
  newBinding.container_uid = container.uid

  dispatch('entities/bindings/insert', {
    data: newBinding
  }, { root: true })
}

In this code is newBinding a plain object or an instance of Model? (is both valid?)

I think it's both valid (unless you set the property to readonly).

thank you :)

👍 🎉


Community fab edit location

Reworked on community_fab_editLocation button.
-Tapping on "Search Location" no longer builds a new page, but open a overlay
-Added TextController for passing of data in "Search location"

suggest to mark as draft. buttons are not functional yet in current implementation, but they do look good.


Update yEd to <IP_ADDRESS>

Fixes caskroom/homebrew-cask#19843

Thank you for the contribution. It was merged directly as d98bb5514ff58c96ba2744e25124e0d8dea015a5 to keep commit history cleaner. Your contribution is still credited to you.


No recieve from Callbacks

Dear,
I want to create a secure server to receive data from an app I created to an esp32. The pin-code was accepted in my android. But something strange happens when the code is accepted.  It doesn't confirm the code in the callbacks. I tryed too make a SecurityCallback class but there ware too many arguments...?? Also the CharacteristicCallbacks doesn't appear when I send data. Does anyone have suggestions on that?
#include <NimBLEDevice.h>
#define SERVICE_UUID        "4fafc201-1fb5-459e-8fcc-c5c9c331914b"
#define CHARACTERISTIC_UUID "beb5483e-36e1-4688-b7f5-ea07361b26a8"
#define PASSKEY 123456
NimBLEServer* pServer = NULL;
NimBLECharacteristic* pCharacteristic = NULL;
bool deviceConnected = false;
bool oldDeviceConnected = false;
String StrRecieved;
/**  None of these are required as they will be handled by the library with defaults. **
 **                       Remove as you see fit for your needs                        */
class MyServerCallbacks : public BLEServerCallbacks {
	void onConnect(BLEServer* pServer) {
		deviceConnected = true;
		BLEDevice::startAdvertising();
	};
	void onDisconnect(BLEServer* pServer) {
		deviceConnected = false;
	}
	/***************** New - Security handled here ********************
	****** Note: these are the same return values as defaults ********/
	uint32_t onPassKeyRequest() {
		Serial.println("Server PassKeyRequest");
		return PASSKEY;
		//return 000000;
	}
	bool onConfirmPIN(uint32_t pass_key) {
		Serial.print("The passkey YES/NO number: "); Serial.println(pass_key);
		return true;
	}
	void onAuthenticationComplete(ble_gap_conn_desc desc) {
		Serial.println("Starting BLE work!");
	}
	/*******************************************************************/
};
//class SecurityCallback : public BLESecurityCallbacks {
//
//	uint32_t onPassKeyRequest() {
//		return 000000;
//	}
//
//	void onPassKeyNotify(uint32_t pass_key) {}
//
//	bool onConfirmPIN(uint32_t pass_key) {
//		vTaskDelay(5000);
//		return true;
//	}
//
//	bool onSecurityRequest() {
//		return true;
//	}
//
//	void onAuthenticationComplete(esp_ble_auth_cmpl_t cmpl) {
//		if (cmpl.success) {
//			Serial.println("   - SecurityCallback - Authentication Success");
//		}
//		else {
//			Serial.println("   - SecurityCallback - Authentication Failure*");
//			pServer->removePeerDevice(pServer->getConnId(), true);
//		}
//		BLEDevice::startAdvertising();
//	}
//};

/** Handler class for characteristic actions */
class CharacteristicCallbacks : public NimBLECharacteristicCallbacks {
	void onWrite(NimBLECharacteristic* pCharacteristic) {
		std::string value = pCharacteristic->getValue();
		Serial.println("here");
		if (value.length() > 0) {
			StrRecieved = "";
			for (int i = 0; i < value.length(); i++) {
				// Serial.print(value[i]);
				StrRecieved = StrRecieved + value[i];
			}
			Serial.println("*********");
			Serial.print("Recieved = ");
			Serial.println(StrRecieved);
		}
	}
};
void setup() {
	Serial.begin(115200);
	Serial.println("Starting NimBLE Server");
	/** sets device name */
	NimBLEDevice::init("NimBLE-Arduino");
	/** Optional: set the transmit power, default is 3db */
	NimBLEDevice::setPower(ESP_PWR_LVL_P9); /** +9db */
	NimBLEDevice::setSecurityAuth(true, true, true);
	//NimBLEDevice::setSecurityAuth(false, false, true);
	NimBLEDevice::setSecurityPasskey(PASSKEY);
	//NimBLEDevice::setSecurityPasskey(PASSKEY);
	NimBLEDevice::setSecurityIOCap(BLE_HS_IO_DISPLAY_ONLY);
	//NimBLEServer* pServer = NimBLEDevice::createServer();
	//BLEDevice::setSecurityCallbacks(new SecurityCallback());
	pServer = NimBLEDevice::createServer();
	pServer->setCallbacks(new MyServerCallbacks());
	NimBLEService* pService = pServer->createService(SERVICE_UUID);
	NimBLECharacteristic* pCharacteristic = pService->createCharacteristic(CHARACTERISTIC_UUID, NIMBLE_PROPERTY::READ | NIMBLE_PROPERTY::READ_ENC | NIMBLE_PROPERTY::READ_AUTHEN);
	pCharacteristic->setCallbacks(new CharacteristicCallbacks());
        pService->start();
	NimBLEAdvertising* pAdvertising = NimBLEDevice::getAdvertising();
	pAdvertising->addServiceUUID(SERVICE_UUID);
	//pAdvertising->setScanResponse(true);
	pAdvertising->start();
	Serial.println("Advertising Started");
	////NimBLEDevice::setMTU(128);//128bytes
	//Serial.println("recieving size MTU");
	//Serial.println(NimBLEDevice::getMTU());
}
void loop() {}

Hi @podaen the issue with the authentication complete callback is that the parameters are not correct in your code;
void onAuthenticationComplete(ble_gap_conn_desc desc)  should be
void onAuthenticationComplete(ble_gap_conn_desc *desc)  as the callback takes a pointer.
Also your characteristic callback does not match the properties of the characteristic, if you are wanting to write values to this you should change all the READ_X properties to WRITE_X

It works. I adapted it like you specified
void onAuthenticationComplete(ble_gap_conn_desc* desc) {
		Serial.println("Starting BLE work!");
	}

and
NimBLECharacteristic* pCharacteristic = pService->createCharacteristic(CHARACTERISTIC_UUID, NIMBLE_PROPERTY::WRITE | NIMBLE_PROPERTY::WRITE_ENC | NIMBLE_PROPERTY::WRITE_AUTHEN);
I'm now recieving data, but only 20bytes... I checked the recieving size MTU and it gives me 255 like config in the nimconfig.h. How come?

hmmm. I have the same app working on the hug BLE library by setting the MTU. I have looked in what I can do in the app and I just can make an MTU request. I have tried it but with the same results.

Hard to say without seeing a log from either device. I know it works fine with nRFConnect and other esp32 clients. The flow should be connect > exchange mtu > read/write characteristics. If you are doing this in your app already please post a log from the server.

 I forgot... By doing so I did some changes in the app. I found out something went wrong after an update of the bluetoothLE extension 2.0.


./patch.sh not working on Ubuntu

RUN "./patch.sh"
Directory:

Output:
ramdisk.img: 1 file pushed, 0 skipped. 636.0 MB/s (1520108 bytes in 0.002s)
./patch.sh: 5: [[: not found
./patch.sh: 6: [[: not found
busybox: 1 file pushed, 0 skipped. 117.4 MB/s (2074240 bytes in 0.017s)
process.sh: 1 file pushed, 0 skipped. 101.5 MB/s (11525 bytes in 0.000s)
***** Grab magisk.zip or magisk.apk first ! *****
adb: error: failed to stat remote object '/data/local/tmp/ramdisk.img': No such file or directory

use bash instead of sh "bash patch.sh", not "./patch.sh"


Request to either remove the pause button when playing a video from a preview or move it to the bottom left corner

In this instance the pause button didn't go away after the video finished playing

In this instance the pause button didn't go away after the video finished playing

I'm closing this. i dont think this is an issue anymore


allow dev comments to be enabled from cli, opt in

What does this PR accomplish?

🩹 Bug Fix
🦚 Feature

Closes #132 .
Changes proposed by this PR:
A minor simplification attempt on cli args, and threads a bool to the places that need it.
📜 Checklist

[x] Works on the ./demo sub directory
[x] Test coverage is excellent and passes
[x] Documentation is thorough

I agree it's not pretty, especially having it in traverse is just wrong from the logic partitioning. Needs a refactor, but I want to get this feature out the door with the next alpha, and refactor during the alpha / beta releases.

I agree it's not pretty, especially having it in traverse is just wrong from the logic partitioning. Needs a refactor, but I want to get this feature out the door with the next alpha, and refactor during the alpha / beta releases.


Latest tweets and retweets buttons broken again

It's broken again, tried it with the chromium extension and userscript.
Latest tweets button is missing, only retweet is visible.

Twitter seems to have reverted the change which broke these last time - currently adding support for both variants until they've made their mind up

Fixed in v.2.15.4


Error while Building the Project

Hi,
I was trying to build the Project, and I was trying to run this command to derive the PCR Value in Read Mode:
go run pcr_utils/main.go --mode=read --pcr=23 -v 10 -alsologtostderr
But, I get the following error while trying to build it:
F0313 15:06:45.723237   58900 main.go:42] can't open TPM %!q(*string=0xc000050f60): stat /dev/tpm0: no such file or directory goroutine 1 [running]: github.com/golang/glog.stacks(0x0) 	/Users/deepanshu/go/pkg/mod/github.com/golang/glog@v0.0.0-20160126235308-23def4e6c14b/glog.go:769 +0x89 github.com/golang/glog.(*loggingT).output(0x1005585e0, 0x3, 0xc00014e3f0, {0x1003dcde5?, 0xc00011bec8?}, 0x2?, 0x0) 	/Users/deepanshu/go/pkg/mod/github.com/golang/glog@v0.0.0-20160126235308-23def4e6c14b/glog.go:720 +0x46d github.com/golang/glog.(*loggingT).printf(0xc000050f90?, 0x1?, {0x1002fe32d, 0x15}, {0xc00011bec8, 0x2, 0x2}) 	/Users/deepanshu/go/pkg/mod/github.com/golang/glog@v0.0.0-20160126235308-23def4e6c14b/glog.go:655 +0x10f github.com/golang/glog.Fatalf(...) 	/Users/deepanshu/go/pkg/mod/github.com/golang/glog@v0.0.0-20160126235308-23def4e6c14b/glog.go:1148 main.main() 	/Users/deepanshu/Desktop/gcp_tpm_sealed_keys/pcr_utils/main.go:42 +0x1bf exit status 255
Your help will be appreciated on this
Thanks !!

The tutorial assumes the tpm device is available at /dev/tpm0 (which is the path for GCP Shielded VMs)
I'm guessing your trying it out on a mac which i don't really know how to access the TPM on that os.

slightly.
the repo shows a way to seal some data on your laptop and only get it unsealed on a remote VM that has a TPM.
your laptop doesn't need a TPM but it does need the endorsement public key of the remove VM.  How your laptop gets that public key isn't specified here in this repo; the fact that the key was distributed to your laptop by other means is assumed.
once your laptop has the endorsement key, it can seal data such that it can only get decrypted by that same TPM.
the encryption part on yoru laptop can also contain a set of PCR values you expect to be present on the target TPM.  If the PCR values don't match, you can't decrypt.
the command from your original post is
go run pcr_utils/main.go --mode=read

is intended to be just read the state of the PCR values on the VM that has the TPM

slightly.
the repo shows a way to seal some data on your laptop and only get it unsealed on a remote VM that has a TPM.
your laptop doesn't need a TPM but it does need the endorsement public key of the remove VM. How your laptop gets that public key isn't specified here in this repo; the fact that the key was distributed to your laptop by other means is assumed.
once your laptop has the endorsement key, it can seal data such that it can only get decrypted by that same TPM.
the encryption part on yoru laptop can also contain a set of PCR values you expect to be present on the target TPM. If the PCR values don't match, you can't decrypt.
the command from your original post is
go run pcr_utils/main.go --mode=read

is intended to be just read the state of the PCR values on the VM that has the TPM

One Further Question:
Question 2:
When you say that the Command is intended to just read the State of the PCR Values on the VM that has the TPM. I am assuming you mean to refer the VM as the Laptop(The Local Machine) ?

no, you read the state of the PCR  banks only on the system where the TPM is present.   (when i mentioned VM, i'm talking just about a gcp vm here that has a TPM)
for a GCP Shielded VM, PCR0 happens to be 24af52a4f429b71a3184a6d64cddad17e54ea030e2aa6576bf3a5a3d8bd3328f

https://github.com/salrashid123/tpm_key_distribution#server

Then here with the Command that I am executing, why the Build process fails. Because I am able to get the Public Endorsement Key in the First Step and if the read operation is happening from the PCR Bank of the GCP VM, then why it fails ?

Thanks for your Prompt Response
With your Repo, we can access the TPM on Windows/Linux ?

@salrashid123 Also when we are performing the extend operation on the Remote VM, by updating the PCR 23 Value. On the Console, it shows that the value has been extended successfully and it shows us the new PCR Value for Index 23
But When I visit the Logs for the Remote VM on the console, under the measurements, it does not show me any measurement with the PCR[23] Indice

Also please correct me If I am wrong here but is the -v flag used in the Command used to extend the new value that we want into the New PCR Indice ?
I have noticed that, with both the read and the extend operation you are using -v flag with the 10 value here. So in essence what does this -v flag signify ?

@salrashid123 if you could provide Intel about the -v flag and it's significance? Why are we using it with both read and extend operation.
I am assuming that v flag is being used to extend a value let's say 10 into a PCR Index . Is this the case of am I misinterpreting ?


It would be useful to support #reply messages

Ardour prefixes reply messages with #reply, although it does have an option to change that to /reply.
Would it be possible to support #reply?
If you dont have time I would be happy to work on it if you can give me some suggestions.

I haven't tried interfacing this with Ardour before, but it's definitely worth making sure that the library has whatever capabilities it needs to work with something like Ardour, and it sounds like maybe it's not possible to right now.
Can you explain a little more what #reply looks like? Right now, a message includes an address, and then a sequence of arguments. If you serialize a struct where the address is set to "#reply", does that accomplish what you need?

I haven't tried interfacing this with Ardour before, but it's definitely worth making sure that the library has whatever capabilities it needs to work with something like Ardour, and it sounds like maybe it's not possible to right now.
Can you explain a little more what #reply looks like? Right now, a message includes an address, and then a sequence of arguments. If you serialize a struct where the address is set to "#reply", does that accomplish what you need?


Deal with Idents/Names efficiently

It may be the case that identifiers or names should be interned, or use Text, but in any case the current situation should change.

Might be worth considering an Invalid constructor given the number of times Rust relies on that idea. Either that, or adjust the AST to get rid of this pattern.
data Ident = Invalid | Ident String -- or some ID in a global symbol table

For now, Ident stayed the same, and instead the AST was adjusted to avoid having to fill Ident holes with an empty/filler/invalid Ident. Still not ruling out the possibility of adding the Invalid :: Ident variant, especially since it would match more closely the Rust AST.

Reopening this for the more general problem of how to efficiently deal with Ident and Name.

This commit removed the Name data type and just changed it to be a synonym for String. After a couple more simplifications, things ended up pretty nice. This issue is now repurposed for the question of whether it is worth trying to compact strings?
For example, have something like
import Data.IntMap

mkIdent :: IntMap Int Ident -> String -> (IntMap Int Ident, Ident)
mkIdent s cache = case lookup h cache of
                           Just ident@(Ident s' _) | s == s' -> (cache, ident)
                           _ -> let ident = Ident s h in  (insert h ident cache, ident)
     where h = hash s

The cost of the IntMap probably dwarfs any benefits of using it though... Still something to keep in mind.

There are really two ideas to keep track of here:

Compacting common Name's (like the previous comment)
Representing Name's in a format that saves bytes and pointer indirection (like Text)

I should benchmark this before doing anything.


ToolkitRegistry() should initialize an empty registry

@jchodera, the current default constructor for ToolkitRegistry will attempt to initialize OE, Amber, and RDK toolkits. The command to initialize an empty ToolkitRegistry is ToolkitRegistry(toolkit_precedence=[]). Revisiting this months later, this is really unintuitive, and I think we should change it (ToolkitRegistry() should initialize an empty registry). Opening a new issue.
Originally posted by @j-wags in https://github.com/openforcefield/openforcefield/issues/346#issuecomment-502410801

If the API is breaking the "principle of least surprise", then the default behavior should definitely be changed.
I originally thought the least surprising default behavior was "do the right thing automatically unless I tell you not to"---that is, "find all the toolkits that are installed and use those". I expected that, if someone wanted fine-grained control over toolkits, they would initialize with specific toolkits or an empty set and add them one at a time, since this is the minority use case.
But I clearly got the API wrong in my debugging in https://github.com/openforcefield/openforcefield/issues/346#issuecomment-502054952 :)

I agree that ToolkitRegistry() should give me an instance with no wrappers registered into it. I find this type of behavior quite surprising when I ran into it:
In [1]: from openforcefield.utils.toolkits import *

In [2]: reg = ToolkitRegistry()

In [3]: [reg.register_toolkit(tk) for tk in [OpenEyeToolkitWrapper, RDKitToolkitWrapper, AmberToolsToolkitWrapper]]
Out[3]: [None, None, None]

In [4]: reg.registered_toolkits
Out[4]:
[<openforcefield.utils.toolkits.OpenEyeToolkitWrapper at 0x14071f6d0>,
 <openforcefield.utils.toolkits.RDKitToolkitWrapper at 0x144349550>,
 <openforcefield.utils.toolkits.AmberToolsToolkitWrapper at 0x144349910>,
 <openforcefield.utils.toolkits.BuiltInToolkitWrapper at 0x144349a50>,
 <openforcefield.utils.toolkits.OpenEyeToolkitWrapper at 0x1443758d0>,
 <openforcefield.utils.toolkits.RDKitToolkitWrapper at 0x14527e110>,
 <openforcefield.utils.toolkits.AmberToolsToolkitWrapper at 0x14527e050>]

(this is surprising for two reasons, one being the non-empty default registry and also that .register_toolkit should probably check to see if one is already registered)
I wouldn't expect this to be a hugely problematic API break since most functionality silently uses GLOBAL_TOOLKIT_REGISTRY behind the scenes and initializing a custom ToolkitRegistry() often comes with a populated toolkit_precedence argument. Is there a compelling reason to maintain this behavior that I'm not considering?