folder share with edit permission - error with WOPI

oCIS 5.0.0
Steps to reproduce:

create a folder
create a txt and a docx/pptx/xlsx document in that folder
share that folder with edit permissions
the share receiver can edit the txt document
the share receiver can not edit the docx/pptx/xlsx document
the share receiver can not create a new docx/pptx/xlsx document
the share receiver can create a new txt document
the share receiver can not upload a docx/pptx/xlsx document
the share receiver can not upload a txt document

https://github.com/owncloud/ocis/assets/115982880/8020c358-d9f5-43c1-bc02-e2a384716e0e

I'm not sure if we run the WOPI validator on public links in the meantime (https://github.com/owncloud/ocis/issues/5111, https://github.com/owncloud/ocis/issues/8712)

Public link?

Public link?
yes

Duplicate of #8691 and https://github.com/owncloud/enterprise/issues/6558
We are already on it.
Thanks for reporting.
EE issue from customer has Sev3, P3. Aligning.

I'm not sure if we run the WOPI validator on public links in the meantime (#5111, #8712)

Not related to WOPI.
WOPI testing plans are here https://github.com/owncloud/ocis/issues/8712 for the new collaboration service.

New finding: Also upload of office documents does not work. upload of text files work fine.

New finding: Also upload of office documents throughs an error after upload.

Not reproducable on the vanilla product.

@ScharfViktor moved to "needs tests"

@ScharfViktor moved to "needs tests"

thanks. we'll create test for that
already tested it manualy. works fine
same issue: https://github.com/owncloud/ocis/issues/8691


pull request for e2e test at 01/19/2017 17:56:16

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Send along the agent with Segment identify calls

This will help us get segmentation between Android and Web set up properly in Amplitude.

          1 Warning

:warning:
Please provide a linked tracker story for this PR in the description

  Generated by :no_entry_sign: danger


Factory for automatically identifying RSS vs. Atom

It's fairly easy to determine if a feed is Atom vs. RSS by examining the name of the wrapping element. If the feed is Atom, the root will be a <feed> element. If RSS, the root is <rss>. Given that there's a common XmlFeedReader, I propose a factory method that can return a XmlFeedReader of the appropriate type (at least for XML-based feeds) by examining the root element of the document.
The easiest implementation could look like:
public abstract class XmlFeedReader : ISyndicationFeedReader
{
    // ..

    public static XmlFeedReader GetReader(XmlReader reader)
    {
        reader.MoveToContent();
        if(reader.Name == "feed")
        {
            return new AtomFeedReader(reader);
        }
        if(reader.Name == "rss")
        {
            return new RssFeedReader(reader);
        }
        throw new ArgumentException("The feed type was not recognized.");
    }

    // ..
}

This would require the XmlFeedReader to know about both the reader implementations as well as have a rudimentary understanding of each feed specification to know what root element name to look for (as well as any other conditions to identify the feed if future feed specifications use more than the element name to differentiate). I think that's fine for such a small universe of potential readers - it's not like new feed types are being added regularly.
Would you accept a PR for this change? Does the design need to be iterated?

I understand the thinking. I do think it's unfortunate because the library provides a nice set of feed-agnostic abstractions. If the consumer needs to know the type of feed beforehand (whether by sniffing or just knowing), it lessens the utility of the provided abstractions.


$ npm install => Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.

So I properly logged in as suggested in #67
then:
$ rm -rf ./composition
$ git clone git@github.com:nteract/composition.git
$ cd composition
$ npm install

And... surprise surprise:

npm-debug.log.zip

Exciting! This is because we introduced our core binary dependency.
Here are some instructions we should port over: https://github.com/nteract/enchannel-zmq-backend#zeromq-dependency
If you're using homebrew on your Mac, brew install pkg-config zeromq. You may run into other node-gyp fun times.
Long term solution: prebuilt versions of zmq for everyone: #1

This should be going ok now and upstream work is ongoing for zmq and node (as well as the zmq-prebuilt library that @minrk has been working on).


Improve the error when failing to submit a Private Marker Transaction

When using privacy without defining the --privacy-marker-transaction-signing-key-file an auto-generated key (account) is used.
On a network with a non-zero gas price, when a private transaction is submitted with the from account being well funded, the error Upfront cost exceeds account balance is encountered (because the account signing the PMT is not funded).
While the error is technically correct, it is fair to assume that a user (who submits the private transaction) will infer that the error relates to the transaction they have submitted, and the insufficient funds relate to the account they submitted in the from field.
Let's do something to improve this user experience, either something in the JSON-RPC response, or at a minimum additional logging to help people diagnose the problem (i.e. identify the PMT failed).
Raised by @CjHare in Jira

I am getting the same issue..
Where to define this marker tag ??

How to disable privacy ?

@amitcz - privacy must be explicitly enabled and configured as described in this tutorial https://besu.hyperledger.org/en/stable/Tutorials/Privacy/Configuring-Privacy/
Are you using privacy and getting this error message?  Or getting this error message with public transactions?


why this prompt? tell it You are ChatGPT?

https://github.com/waylaidwanderer/node-chatgpt-api/blob/d2cdb5831bdf8c653dec4d2600dceb333cfa2d30/src/ChatGPTClient.js#L137

I explained it in the code comments. It's the same prompt that OpenAI gives to ChatGPT when you start a chat with it. It's just hidden to us.


Using an empty OIDC configuration

Hi,
I was told to ask here by @Tratcher in https://github.com/aspnet/AspNetCore/issues/17616#issuecomment-567545682.
I've written som authorization tests against my app startup for API protected by bearer tokens using JWT bearer tokens. In the test I wanted a way of overriding  hitting the OIDC Discovery document (.well-known/openid-configuration) to avoid network traffic, so I tried using a static configuration. Just by experimenting, I tried the empty OIDC configuration:
var webHostBuilder = new WebHostBuilder()
    .UseStartup<Startup>()
    .ConfigureTestServices(s =>
    {
        s.Configure<JwtBearerOptions>(o =>
        {
            o.Configuration = OpenIdConnectConfiguration.Create("");
        });
    });

What surprised me, is that this made the test pass. I would have expected it to crash with "empty configuration", or similar.

Is setting an empty OIDC configuration a valid scenario?
Will setting an empty OIDC configuration bypass all jwt verification (issuer, signer, audience)?

@johnkors using a static configuration is a fine way to avoid the http traffic, most of our test use this.
However, I would expect a token validation test to fail because no issuer, keys were found, so this surprises me.
Can you share a bit about your tests?
Audience is not obtained from configuration, that is a relying parting setting, usually the clientid.


flaky tests

Hello!
Could it be that the tests are flaky?
On a loaded machine, they seem to sometimes pass and sometimes fail. Does the project contain timing-dependent tests?
For example:
--- FAIL: TestNoRaceRoutedQueueAutoUnsubscribe (10.23s)
	norace_test.go:176: Did not receive all 250 queue messages, received 250 for 'bar' and 238 for 'baz'

Could be. I am pretty sure I fixed this on on trust branch since I ran into it as well.

Actually I fixed another one, but same issue. Will fix this one.


the documentation website some content can not show

for example https://material-ui.com/style/typography/

its like these hrefs  can not show  its all  "An unexpected error has occurred."

now i don know where to see the doc  with out "An unexpected error has occurred."

Duplicate of #13512


Detailed Revenue Metrics

Suggestion on future schema for revenue - instead of doing all the math on the backend and having one revenue number on the schema, can we have an array of each different rev type?
Example
TotalRevenue: [
{name: FeeRevenue,
AmountUSD: 10000},
{name: LiquidationRevenue,
AmountUSD: 1000}
]
This way the consuming app/users more clearly knows what’s in the rev number and can have more flexibility in displaying rev (example being us generating financial statements for quarterlies where you would want to show the breakout of revenues across product lines and types like swap fees, withdraw fees, liquidation etc)

@bye43 what protocols have liquidation revenue?

@dmelotik From my quick methodology scan:

AAVE V3
Abracadabra
MakerDAO
Liquity
QiDAO
Synthetix
trueFI
Vesta
Moola Market

Couple notes here though. The methodology for Compound and its associated forks says that liquidations are currently note included, so potentially may want to take a look at that being included if its relevant. Also, we should consider flashloan revenue too.

Maybe we can keep the revenue as is, but create a new field for revenue by type and have an array with revenue types. Going to see what people say, but something we could include in the 3.0.0 schema

Being fixed in new schema

Being addressed here: https://github.com/messari/subgraphs/issues/1081


locationAlwaysAndWhenInUse returns didAllow when selecting 'Only While Using the App'

Either this is a bug, or intended and I don't understand why.
While using the locations permissions, the  locationAlwaysAndWhenInUse SPPermissionType will return as didAllow if you select the 'Only While Using the App' option. It should only return true if 'Always Allow' is selected.  Otherwise, what's the point of having  locationWhenInUse and locationAlwaysAndWhenInUse types?

Sorry for long response. Today I am release new version 3.0.6, please, update. In this version bug fixed. Thanks!


How can i use our own chessboard pics to get the results?

Hi,I would like to ask if it is possible to input the checkerboard pictures taken by myself so as to obtain calibration results in your method. I can't find the model file. I don't know how to run it, sorry, looking forward to your reply.

For better accuracy, a specific model trained on synthetic images which are close to your own data and calibration environment is better, though time consumption is large.
So if you pursue accuracy, you should train your own model.
If you just want to test calibration, you can use the RANSAC-based calibration algorithm in our code.
We will upload a pre-trained corner detection model later, you can use that to process your image.
For distortion correction, our method is sensitive to different chessboards, so a specifically-trained model is better.


Update to use subsectionID instead of full-link in "Windows containers in Kubernetes" page.

Related Issue : #39616

Some links are accessible only with subsection ID, not full link.
They exist in same page.

/lgtm
Thanks!


EF-Core filter not working

I've problems in filtering the output of serilog in an asp.net core 3.1 application.
Here is a part of the serilog configuration in appsettings.json
["Using":] ["Serilog.Sinks.Console", "Serilog.Sinks.File", "Serilog.Sinks.Async", "Serilog.Sinks.MSSqlServer",
  "Serilog.Filters.Expressions"],
"MinimumLevel": "Debug",
"Enrich": [
  "FromLogContext"
],
"WriteTo:SubloggerConsole": {
  "Name": "Console"
},
"WriteTo:SubloggerFile": {
  "Name": "Async",
  "Args": {
    "configure": [
      {
        "Name": "File",
        "Args": {
          "restrictedToMinimumLevel": "Information",
          "path": "%TEMP%/Logs/DbCommands.Log",
          "rollingIntervall": "Day",
          "retainedFileCountLimit": 2
        },
        "Filter": [
          {
            "Name": "DbLoggerCategory.Database.Command.Name"
          },
          {
            "Name": "ByIncludingOnly",
            "Args": {
              "expression": "Contains(@Message, 'UPDATE') or Contains(@Message, 'INSERT')"
            }
          }
        ]
      }
    ]
  }
}

Part of program.cs
public static IHostBuilder CreateHostBuilder(string[] args) =>
    Host.CreateDefaultBuilder(args)
        .ConfigureWebHostDefaults(webBuilder => {
          webBuilder
          .UseStartup<Startup>()
          .CaptureStartupErrors(true)
          .UseSetting("detailedErrors", "true")
          .ConfigureAppConfiguration(config => {
            config.AddJsonFile("appsettings.Local.json", optional: true);
          });
        }).UseSerilog((hostingContext, loggerConfiguration) => loggerConfiguration
          .ReadFrom.Configuration(hostingContext.Configuration)
        );

Part of startup.cs
public void Configure(IApplicationBuilder app, IWebHostEnvironment env, IServiceProvider serviceProvider, ILoggerFactory loggerFactory) {
  if (env.IsDevelopment()) {
    app.UseDeveloperExceptionPage();
  } else {
    app.UseExceptionHandler("/Home/Error");
    // The default HSTS value is 30 days. You may want to change this for production scenarios, see https://aka.ms/aspnetcore-hsts.
  }
  app.UseHsts();

  loggerFactory.AddSerilog();

Part of context file
public class FristentoolContext : DbContext {
private readonly ILoggerFactory _loggerFactory;

public FristentoolContext(DbContextOptions<FristentoolContext> options, ILoggerFactory loggerFactory)
    : base(options) {
  _loggerFactory = loggerFactory;
}

protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder) {
  optionsBuilder.UseLoggerFactory(_loggerFactory);
}

Logging to the file is working but without any filter.
Any help will be highly appreciated
Thanks in advance
Matt

This might not help but if you turn on the Serilog debug logging its invaluable with regards telling you what is going on, and more importantly what is failing: -
var file = File.CreateText("c:\\temp\\Serilog-SelfLog.txt");
Serilog.Debugging.SelfLog.Enable(TextWriter.Synchronized(file));

Hi - thanks for your question. Unfortunately the maintainers are short of time to dig in further, but if you are able to post it to Stack Overflow you should get the right eyes on it. HTH!


Expose the rightFacing parameter of the doors

This parameter can flip vertically the texture of doors. The game supports it, it's used in some official levels (Choice on the top of my head). But the editor doesn't support the feature.
https://github.com/getkey/bombhopperio-level-tools/blob/e9ce9beaa02b9f79349da370b623a9b9253b8571/src/entity.schema.json#L192-L194

hello!
may i work on this?

Yes gladly!
To get you started, there is a very similar feature that you can have a look at and copy https://github.com/getkey/ble/blob/8ad099879d85ecc295beeb91f3f0b364ecf3ab48/src/models/StaticParams.ts and https://github.com/getkey/ble/blob/8ad099879d85ecc295beeb91f3f0b364ecf3ab48/src/components/StaticParam.tsx
You'll need to replace the model here https://github.com/getkey/ble/blob/8ad099879d85ecc295beeb91f3f0b364ecf3ab48/src/models/Door.ts#L18-L20 and you can hook up the UI here https://github.com/getkey/ble/blob/8ad099879d85ecc295beeb91f3f0b364ecf3ab48/src/components/ParamsBox.tsx
Lemme know if you need more guidance :wink:


reduced a little ts and js icons

They look a little bit bigger than I expected.

Hi,
is it possible to bring back the orgiginal icons of js and ts, that were introduced in #509? They looked so much better than the current ones.
Thanks,
Thomas

Hi @tmueller, some people have been complaining of the official icons. They were claiming they were to much obstrusive so we have decided to make a rollback for the moment while we're preparing a user customization tool. We're going to deliver it in phases so it will be improving. First version will be have some presets so you can decide if you want to use the official ones or not, among other things. See #537, #514

Thanks for your quick reply. It's a pity other users found them to obstrusive. Boils down to personal likes I guess. Looking forward to the costumization tool.

I've have a very rough version. I guess I will be able to provide something during the week. 😉

I want the new icons back…

@robertohuertasm FACEPALM


hintText is not working hasFloatingPlaceholder: true

Fix issue: https://github.com/maheshmnj/passwordfield-flutter-package/issues/3#issue-664979680

Looks Good to me Thanks for your time


Will Azure Linux replace CBL-Mariner form 3.0?

Will AzureLinux replace CBL-Mariner form 3.0? Or will CBL-Mariner will also be upgraded to 3.0 later to support node 20? AzureLinux has been upgraded to 3.0 for a while but CBL-Mariner is still on the old version.

AzureLinux (aka CBL-Mariner) is the successor for CBL-Mariner.
We have renamed CBL-Mariner to AzureLinux. We will only keep on supporting AzureLinux 2.0 for security related fixes till its deprecation timeline.


Parallelize IPP with rayon

Each commit is a different configuration of the calculations for G, H.
On my own laptop, results for make_ipp_64 are:
7c811db: unoptimized: 17,958,887 ns/iter (+/- 2,645,491)
1654069: x-axis parallelization: 10,427,160 ns/iter (+/- 321,816) (updated)
655ab83: y-axis parallelization: 17,827,652 ns/iter (+/- 2,091,841)
e230f13: x-axis and y-axis parallelization: 10,461,115 ns/iter (+/- 711,309) (updated)
x-axis = parallelizing the calculation of each index of the vectors G and H.
y-axis = parallelizing the calculation of G and H, with a join().
Next steps:

get zoomzoom github access so we can run the benchmarks there
pick one of the 4 (or potentially more) options to use

I think this branch was accidentally overwritten; closing this PR since it's stale


Issue with Dillinger docs

When we try to use expressions like this one, E[X], inside a LaTeX environment ($$ $$), Dillinger understands that we want a task list instead of actually that expression.
Is there any workaround to fix this behavior?

Hey @4r7hu3, this repository is for the Markdown Guide, not Dillinger specifically. Please open an issue in the Dillinger repository if you're having problems with Dillinger. Thanks!

Thank you for taking the time to close this mistake for me


convert all projects except ui to Netstandard 2.0

As the project is about architecture, I might suggest to convert all projects except UI and Tests to netstandard 2.0 to make them reusable on all compatible platforms.

tommes, I could be mistaken but i believe that Persistence will not be able to update to netstandard. this is because of the dependency on EntityFrameworkCore.
<PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="2.2.1" />
<PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="2.2.1" />

A core dll can reference a netstandard but not the reverse. You will see this error in the package manager console when attempting to create or apply a migration.
As for updating the others Domain, Common, Integration etc to netStd, +1. In fact i did this in my spin off project.
Simmo

Hoopie, EF Core IS netstandard, works on classic .NET 4.6.1 and higher, see also https://docs.microsoft.com/en-us/ef/core/platforms/
I didn't get any errors when restoring and compiling.
I do a lot of work with EF Core targeting .NET full framework and that works.

Sweet. I think i got confused with a package manager issue I got in that you need to target an assembly that is either core or .net in order to get the code first migrations running.
I'm going to change my persistence proj over.. Thanks for the fact check tommes!

Good call, thanks for the contribution. 👍


lepton: Update to 1.8.0 (manually)

Lepton is a Electron based app, and Lepton-Setup-1.8.0.exe can be extracted by 7z as other NSIS installer. Since native hash existed for installer, I suggest using following manifest.

Fixed manifest
{
    "homepage": "https://hackjutsu.com/Lepton/",
    "description": "A lean code snippet manager based on GitHub Gist",
    "license": "MIT",
    "version": "1.8.0",
    "url": "https://github.com/hackjutsu/Lepton/releases/download/v1.8.0/Lepton-Setup-1.8.0.exe#/dl.7z",
    "hash": "sha512:8aaf1324d9f4061033880cd5238f87d4c53cb434aefcc745f0914dfa0bebb401d453d05e55d0af02bc37d5c49f5e31f8e558f9576bb4f3ee764585d6a5ea6700",
    "architecture": {
        "64bit": {
            "installer": {
                "script": [
                    "Expand-7zipArchive \"$dir\\`$PLUGINSDIR\\app-64.7z\" \"$dir\"",
                    "Remove-Item \"$dir\\`$PLUGINSDIR\", \"$dir\\Uninstall*\" -Force -Recurse"
                ]
            }
        },
        "32bit": {
            "installer": {
                "script": [
                    "Expand-7zipArchive \"$dir\\`$PLUGINSDIR\\app-32.7z\" \"$dir\"",
                    "Remove-Item \"$dir\\`$PLUGINSDIR\", \"$dir\\Uninstall*\" -Force -Recurse"
                ]
            }
        }
    },
    "shortcuts": [
        [
            "Lepton.exe",
            "Lepton"
        ]
    ],
    "checkver": {
        "github": "https://github.com/hackjutsu/Lepton"
    },
    "autoupdate": {
        "url": "https://github.com/hackjutsu/Lepton/releases/download/v$version/Lepton-Setup-$version.exe#/dl.7z",
        "hash": {
            "url": "$baseurl/latest.yml",
            "regex": "sha512: $base64"
        }
    }
}

Why add bin? This app shouldn't start from console, 'cos it doesn't support it, and will produce nonsense output while starting from console.

@rasa Yeah that is correct. I was tired and missed that there is not extract_dir = $PLUGINSDIR
"script": "Expand-7zipArchive \"$dir\\$PLUGINSDIR/app-64.7z""will be extracted to$dir\$PLUGINSDIR` not $dir


Apply profiling conditions when building the ZIP archive with the modified files set

Feedback from one of our end users:
       We would like to be able to export new/changed files that are specific
       to a selected root map, with profiling condition filters taken into account.

My question:
        If a common topic has inside it content belonging to Product A and Product B (for example two paragraphs, one which appears only in A and one which appears only in B), for Product A do you want to send that common topic to the translator with one of those paragraphs filtered out?

Their answer:
       Not necessarily. The complete topic with all of the conditional content could be translated at once, as long as it’s ready.

So when creating the ZIP besides selecting the milestone file the end user would like to also apply a profiling condition set to filter out additional resources.

About the same thing was asked here:
https://www.oxygenxml.com/forum/viewtopic.php?f=20&t=14376


EOS transact error

Version of EOSJS
Not sure how to check the version.
Describe the bug
I had code that sends token transfer transactions via the api.transact method which worked fine yesterday. I am not sure what changed but today I get UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'get_info' of undefined (seems like rpc is undefined) error.
I am following the syntax given in the documentations. It behaves as if rpc is undefined in the API object but I have declared and fetched it. I can also make rpc calls on it .
Here is how I declare and fetch the objects and make the transaction:
const signatureProvider1 = new JsSignatureProvider(node1Key);
const rpc1 = new JsonRpc('http://localhost:8011', {  fetch });
const api1 = new Api({rpc1, signatureProvider1,chainId: chainID, textDecoder: new TextDecoder(), textEncoder: new TextEncoder()
});
const transactionOptions = {
chainId: chainID,
blocksBehind: 3,
expireSeconds: 3600,
broadcast: true,
sign: true,
};
const trans = generateTransaction('node1','node2','1.0000 ET',' asd' );
api1.transact(trans,transactionOptions);

This seems like it is due to new property shorthand syntax (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Object_initializer), though I can't explain why it would be working yesterday and not today if you have the exact same code.
Two ways to solve the initialization:

Specify rpc: rpc1

const rpc1 = new JsonRpc('http://localhost:8011', { fetch });
const api1 = new Api({rpc: rpc1, signatureProvider1,chainId: chainID, textDecoder: new TextDecoder(), textEncoder: new TextEncoder()
});

Update rpc1 name to rpc

const rpc = new JsonRpc('http://localhost:8011', { fetch });
const api1 = new Api({rpc, signatureProvider1,chainId: chainID, textDecoder: new TextDecoder(), textEncoder: new TextEncoder()
});

@jlamarr22 To be fair I change the code a bit but then it stopped working and i reverted back the changes. I used to have the object declared as rpc rather than rpc1 but I did not think that would matter that's why I did not mention it. I wanted to create multiple connections (rpc and api objects) because my test should involve sending transactions from multiple nodes.

@alexy97 That's why it was working before with object shorthand syntax (your variable was named rpc). If you want to pass in Rpcs named differently, then you need to specify what it's mapping to. Here are three scenarios:
// ES6 implicitly maps `rpc` to the `rpc` object value. (notice there is no `rpc: xxx`)
const rpc = new JsonRpc('http://localhost:8011', { fetch });
const api = new Api({rpc, signatureProvider1,chainId: chainID, textDecoder: new TextDecoder(), textEncoder: new TextEncoder()
});

// Explicitly says that you want `rpc1` to map to the `rpc` object value. (notice there IS `rpc: xxx`)
const rpc1 = new JsonRpc('http://localhost:8011', { fetch });
const api1 = new Api({rpc: rpc1, signatureProvider1,chainId: chainID, textDecoder: new TextDecoder(), textEncoder: new TextEncoder()
});

// Explicitly says that you want `rpc2` to map to the `rpc` object value. (notice there IS `rpc: xxx`)
const rpc2 = new JsonRpc('http://localhost:8011', { fetch });
const api2 = new Api({rpc: rpc2, signatureProvider1,chainId: chainID, textDecoder: new TextDecoder(), textEncoder: new TextEncoder()
});

@alexy97 That's why it was working before with object shorthand syntax (see link above for more info on that) -- your variable was named rpc. If you want to pass in Rpcs named differently, then you need to specify what it's mapping to. Here are three scenarios:
// ES6 implicitly maps `rpc` to the `rpc` object value. (notice there is NO `rpc: xxx`)
const rpc = new JsonRpc('http://localhost:8011', { fetch });
const api = new Api({rpc, signatureProvider1,chainId: chainID, textDecoder: new TextDecoder(), textEncoder: new TextEncoder()
});

// Explicitly says that you want `rpc1` to map to the `rpc` object value. (notice there IS `rpc: xxx`)
const rpc1 = new JsonRpc('http://localhost:8011', { fetch });
const api1 = new Api({rpc: rpc1, signatureProvider1,chainId: chainID, textDecoder: new TextDecoder(), textEncoder: new TextEncoder()
});

// Explicitly says that you want `whateverYourVariableNameIs` to map to the `rpc` object value. (notice there IS `rpc: xxx`)
const whateverYourVariableNameIs = new JsonRpc('http://localhost:8011', { fetch });
const api2 = new Api({rpc: whateverYourVariableNameIs, signatureProvider1,chainId: chainID, textDecoder: new TextDecoder(), textEncoder: new TextEncoder()
});

Does this make sense?

Yes I understand now and it works. I haven't used javascript a lot that's why I was confused as of why the varbiable name can break the code. Thanks a lot

No worries -- tend to be little nuances/"magic" in every language when you first get in to it. Happy to help!


Hhh 7009 4.0

Create dialect for Postgres 8.2 and later which supports "if exists"
in drop statements. Update resolver to return the correct dialect
version.

Pushed to master.
Thanks!
Gail


Is this project dead?

Found this project while research ETL in .NET Core.  It appears that there has only been one major update in the last 4 years.  Plus there are some pull requests out there (for what appears months) that would make this project .NET Core compatible.
Is this project dead?
Just trying to decide if I want to take the time to either use this project as-is, fork it and maintain my own version, or just move on.  It looks like a good library, so I am trying not to dismiss out-right.

I am afraid it is not very alive. For the full upgrade to .NET Core some classes are needed that should appear with .NET Core 3. However, if I were to start again I would consider using https://github.com/madhon/ReactiveETL instead.


Invite to view another members tree with privacy

Hi @JoshKisb @curtisdelicata  Please don't be cheated by this Scammer.
He will never pay you more than he pays for an Indian guy who works for $5 per hour. He always promise $30 or more per hour, but will never send money once you submit all your works. Don't push your updates before you receive the payment.
Otherwise, he will block you anywhere you have contact with him, don't believe him ever.


Workaround for jitpack.io maven repo returning 522 http status

According to https://github.com/jitpack/jitpack.io/issues/3973

Have you tried the most recent version?

No change after updating to the 2019.2.3

I wasn't able to properly test out this fix locally because of gradle not detecting plugin in mavenLocal. If you have working setup, i'd appreciate if you could test it. But according to linked issue, this should fix the 522 status


Add "referrer_url" parameter Server Event Data

Would it be possible if you add the "referrer_url" parameter to Server Event Data section? I am missing this one in the template.
Here's the documentation where this value is referenced (at the bottom): https://developers.facebook.com/docs/marketing-api/conversions-api/parameters/server-event

Sorry, just saw the latest commit: https://github.com/stape-io/facebook-tag/commit/c3fbe0854fe803bc769d568a9a77e89bc7495d94


[AUTHZ] Fix NoSuchElementException when listing database in CatalogImpl in Spark 3.4

Why are the changes needed?
Fix #4902
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Good job ! Thanks for the efforts. All the tests pass with Spark 3.4 now, with this PR applied.

Rebase onto the latest commit of the master branch. The commit causing failed test of DropTable is reverted. Sorry for that.

Codecov Report

Merging #4903 (8bf3e13) into master (6b301b0) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master   #4903   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         557     557
  Lines       30695   30700    +5
  Branches     3994    3995    +1
======================================
- Misses      30695   30700    +5

Impacted Files
Coverage Δ

...park/authz/ranger/FilterDataSourceV2Strategy.scala
0.00% <0.00%> (ø)

...gin/spark/authz/util/ObjectFilterPlaceHolder.scala
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks, merged to master.


End-to-end DAG testing

Overview
This PR adds an end-to-end dag tester to Airflow, located in tests_dags/dag_tester.py, as well as some DAG tests located in tests_dags/tests/bash_operator_dag_tests.py
These tests require a connection to a concurrent db (i.e. not SQLite) in the sql_alchemy_conn field of your unittests.cfg. The tests will be skipped otherwise.
How to use

The test author creates one or several DAG files and store them in tests_dags/dags.

The test author provides one or several units test for the dag(s), (cf examples in tests_dags/tests/bash_operator_dag_tests.py). This test should provide:

get_dag_file_names to specify which dag files are involved in this test
get_context (optional): a dictionary of Variable to be provided to the DAG at test time
if we want to test the dags with a BackfillJob, inherit from EndToEndBackfillJobTest and  implement get_backfill_params to specify the constructor parameters of BackfillJob.
if we want to test the dags with a SchedulerJob, inherit from EndToEndSchedulerJobTest and  implement get_schedulerjob_params to specify the constructor parameters of SchedulerJob
implement post_check with a set of post-execution validation steps

The DAG created by the test author may access the name of their runtime temporary folder with Variable.get("unit_test_tmp_dir") as well as any other context parameter provided by the test. For example, if the test implements this:
def get_context(self):
    return {"country": "India",
            "product_sample_size": 10000}

then the DAG will be able to access those values at test time via:
Variable.get(key="country", deserialize_json=True)
Variable.get(key="product_sample_size", deserialize_json=True)

The design of the test should keep in mind that the executors are ultimately starting child processes via some airflow run blah commands. This is particularly important to design the interactions between the test initialization, its running, and the validation of post-conditions, since they do not all run within the same python process.

How it works
The DAG Tester will:

create a temporary folder for each test and copy all the tested DAGs in a subfolder dags of that folder
copy an airflow configuration file to that temporary folder, adapts its dags_folder value according to the above.
write in Variables the location of the temporary folder in unit_test_tmp_dir as well as all the values provided by the test in get_context. (If any Variables would have been overwritten in the process, their values are saved and restored during cleanup)
depending on what is implemented by the test, build a BackfillJob and/or a SchedulerJob with the tested dags and run it
after each run, execute the test post_check

Future possible improvements

we currently only run the test with a BackfillJob with a SequencialExecutor and/or with a SchedulerJob with a LocalExcecutor. We could add more executors and combinations
each test is executed with its own config file, though it's almost the same for every test => we could extend the framework to validate different kinds of configurations per test
all tests provided so far are validating the BashOperator, though adding more kinds of operator should be straightforward.
we could extend it to run the job several times, maybe thus allowing to validate restartability of a DAG (e.g. a whole DAG that fails twice before succeeding), idempotency (a DAG that is executed several times with the same execution_date, with a clear between each run, should not produce corrupted output) or simulating addition/removal/update of DAG files between successive executions.
we could try to extend to execution of several jobs in parallel, thus improving the validation of race condition (although test of parallel code is always tricky)

Limitations

The num_runs parameter for SchedulerJob tests is artificially high in some cases, we do not always understand why more DagRuns than the theoretical number are needed. This has an impact on the execution time.
The execution time of the 7 DAG tests with both kinds of executors over a period of up to 10 days requires about 20 minutes to execute on Travis => we might need to discuss when we want execute them.

Hey guys! Any feedback just yet? I've just rebased to fix conflicts. @mistercrunch

This looks solid. I'm unclear on how much travis can take in terms of compute time. I like to keep it down to make it quick when trying to patch or hotfix.
Maybe we could have this run only on a subset of the build matrix. More parallelism would be positive, maybe with a scheduler based approach as opposed to a backfill based approach, but I'm unclear on whether it would conflict with the multi-config support....

Hey @mistercrunch, thanks for the feedback !
Indeed, computing time can become a barrier for quick fixes, and it's only expected to grow as people add more dag tests.
We also agree that it would be difficult to run several tests in parallel as each test configures airflow with a specific dag folder and each test uses the same database (e.g. we at least know that they share the same Variables to parametrize the dags).
We've identified several possibilities :

Paralellize the dag tests on existing builds:

Run the unit tests on the full build matrix, for each PR
Spread the dag tests on existing builds to "parallelize" them, for each PR
(e.g. backfill tests on one build, scheduler tests on another)

=> Simple but only pushes the problem a bit further as people will add more tests

Trigger dag tests nightly through the travis api:

Run the unit tests on the full build matrix, for each PR
Run the dag tests on the full build matrix, nightly, triggered through the travis api

=> Requires external service that triggers travis every night
=> Some bugs might be caught after being merged to master (although before going into airbnb_prod)

Trigger dag tests through a staging branch mechanism:

Create a staging branch that acts as the trunk, keep master for validated changes through dag tests
Run the unit tests on the full build matrix for each PR on trunk
Run the dag tests on the full build matrix, for each PR on master

=> Heavily complexifies the process with extra PRs
We think the second option makes more sense, as it doesn't slow down the PRs while still protecting the airbnb_prod with dags tests (or integration tests).
I you agree, we only need to update the travis configuration and find that extra service to trigger the nightly builds. Would you have a way of automating the travis trigger nightly from inside airbnb ? Through airflow ? ;D

Hey @mistercrunch, any comment ?

Hey guys, I'm still rebasing / fixing conflicts. Any update on when this could be integrated ? @mistercrunch

@mistercrunch are you still considering this?

I would like to be able to do e2e tests this way. Currently we are configuring airflow with environment variables to allow us to run the dag at least once without touching production directories. However it is not possible to run tests like this from a CI server. I'm sure this PR would help resolve this.

This would help us a lot! what's needed to get this into airflow?

Let me get other committers to get their thoughts here.

This looks awesome.  @thoralf-gutierrez how long does it take to run?

One way to handle this is to do nightlies/hourlies on a fixed schedule, rather than on every commit. Most larger OS projects have something similar to this for integration tests like this. I'm not sure if Travis supports anything like this. I did find:

https://nightli.es/
https://github.com/travis-ci/travis-ci/issues/582

Cool ! It takes about 30-35mim : https://travis-ci.org/airbnb/airflow/builds/115027370
You could make the travis.yml to only run the normal unit tests by default, and use nightli.es to override that default travis.yml to also run these dag tests on nightly builds.
Travis has that feature to override the default config through its triggering API, but I'm not sure if nightli.es integrated it yet though.

@mistercrunch what do you think? nightlies?

Coverage increased (+0.8%) to 67.562% when pulling 47cfa0b30293d64be522b18aa2ca4624df853548 on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into 9d6b6d52b1ca57d19e280593c2fb23d6fd919f70 on airbnb:master.

Coverage increased (+0.6%) to 67.442% when pulling 47cfa0b30293d64be522b18aa2ca4624df853548 on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into 9d6b6d52b1ca57d19e280593c2fb23d6fd919f70 on airbnb:master.

Changes Unknown when pulling 46c36be464719ec19be38cd3a2dea8a06cfe32e1 on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into ** on airbnb:master**.

Coverage decreased (-0.2%) to 66.923% when pulling 46c36be464719ec19be38cd3a2dea8a06cfe32e1 on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into 0d9c0ac77effcff1103293ad59e92b4d1fd0cce8 on airbnb:master.

Coverage increased (+0.5%) to 67.6% when pulling 03699f93f0d924fed9a696ee9434008b94aa01a4 on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into 0d9c0ac77effcff1103293ad59e92b4d1fd0cce8 on airbnb:master.

Coverage increased (+0.5%) to 67.543% when pulling b9adc9f6b734484dc3524966791818ef7ba2b2ed on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into 154dec0f747d899d8fa8c25c27e104cdaee9d137 on airbnb:master.

Coverage increased (+0.6%) to 67.595% when pulling 3b0a1c83cb8f4af6a40fb3fd69d5aea09002b54f on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into 310182543a15f636a3af2ce740bae37feee5c1da on airbnb:master.

I've just rebased + fixed conflicts.
@criccomini : I also tried Nightlies on my airflow fork, and it doesn't notify the build that it is a special nightlies build yet, so we don't have a way to enable or disable these longer tests yet. See https://github.com/coderanger/nightlies/issues/12. Don't hesitate to +1

Coverage increased (+0.3%) to 68.06% when pulling 2555f649cdd2ba48756c46bdc43c4ab50e248274 on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into 559a98fb472ea1915d4f0ad15d196a88701a3a3a on airbnb:master.

Hey guys ! @mistercrunch @criccomini
I've just rebased and fixed conflicts, but it looks like Travis CI is having some random problems communicating with coveralls.
I've investigated nightlies a bit more and I see two solutions for this PR to move forward :

We use nightlies to trigger Travis builds on master every night. It is not super straightforward to actually figure out whether that build is a nightlies build or not (there's no env variable, see https://github.com/coderanger/nightlies/issues/12#issuecomment-208080613). The build scripts could connect to the Travis API through the Travis Client to get some more information about the current build (itself). We could then check whether the commit message has been overwritten by nightlies or not, and launch the dag tests accordingly.

Travis actually already automatically sets a TRAVIS_PULL_REQUEST env variable if the build is triggered from a PR, and a TRAVIS_BRANCH env variable to identify what branch it is building. We could decide that we launch these dag tests only on commits to the master branch.

I prefer the second option which is simpler and does not add a dependency on an extra external service. Would you guys agree to only launch these dag tests for commits to the master branch ? If so, I'll work on adding that condition in the build scripts.
I'd really like to merge this PR asap, it's starting to take significant time to rebase and fix conflicts.

@mistercrunch Is there any progress on this? It would be really helpful to have some end-to-end tests in place.

Coverage increased (+0.3%) to 67.421% when pulling 11f70f44ed50b6020c8f133cc0cd7b4cde8faf47 on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into 7da6a94ebc77b6b763485c5f9072a8ebce5af8d7 on airbnb:master.

Coverage increased (+0.2%) to 67.384% when pulling 11f70f44ed50b6020c8f133cc0cd7b4cde8faf47 on RealImpactAnalytics:svv_dag_tester_with_schedule_job_v6 into 7da6a94ebc77b6b763485c5f9072a8ebce5af8d7 on airbnb:master.

Hey guys, here's the JIRA issue. https://issues.apache.org/jira/browse/AIRFLOW-79

@jlowin, can you remove the label ?

Would you guys agree to only launch these dag tests for commits to the master branch

I think that's a good solution

@thoralf-gutierrez are you still looking to get this PR merged?

@thoralf-gutierrez unfortunately as part of the cold-case PR cleanup we will close this if you don't respond in 5 days. I think this is a great PR to have and I would be sad to see it go.

I'm not sure what's the status here. Have end to end test been added in another way to the airflow project ? If not, I still think it's a good idea to work on merging this PR.
Last time we discussed this, there were two outstanding issues if I recall correctly :

We weren't sure where to have these end-to-end tests run. Since they take some time, we didn't want to have them run for each PR but there should be ways to at least run them nightly. I haven't kept up with the build and testing process since the project got into the Apache incubator so I can't comment yet here.
@mistercrunch wanted to have configuration rewritten to avoid "hacking" how configuration works for this PR. I haven't heard of whether this has been done or not in the mean time. If it has, I could spend some time trying to rebase this PR. @mistercrunch can you give us an update ?

I think most people will be OK with having them run on each commit to master, or running nightly. Let's ping the dev mailing list to check.
The hacks to conf do look a bit scary but I haven't reviewed them in detail yet

I'm not sure what's the status here. Have end to end tests been added in another way to the airflow project ? If not, I still think it's a good idea to work on merging this PR.

We still don't have end to end tests; thanks for continuing to spend time on this

@thoralf-gutierrez has been very patient! @zodiac I agree this will be good to have and great if you can champion it. We might just need a little scripting magic to switch out (or re-symlink) the airflow.cfg? Not sure.

nightly pls
-s
On Mon, Nov 21, 2016 at 10:36 PM Li Xuanji<EMAIL_ADDRESS>wrote:

I think most people will be OK with having them run on each commit to
master, or running nightly. Let's ping the dev mailing list to check.
The hacks to conf do look a bit scary but I haven't reviewed them in
detail yet
I'm not sure what's the status here. Have end to end tests been added in
another way to the airflow project ? If not, I still think it's a good idea
to work on merging this PR.
We still don't have end to end tests; thanks for continuing to spend time
on this
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-airflow/pull/748#issuecomment-262159781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjgZo_n-NL3ViA2d0O-qwB87wpytUpvks5rAo1ygaJpZM4Gy-_F
.


Calling functions in a template results in extra parameters being added

Given this template
{> import some_module

{{ some_module.function() }}

The generated function takes an argument called some_module, thinking that it was a parameter. Perhaps it could be made that parameters have to be specified up front using the with syntax?

Interesting point. Definitely worth handling. I guess you can work around it by passing that value in instead but that isn't ideal.
Perhaps if we parse the import list more careful then we can not make parameters for anything that is imported? Or would there be issues with that?

That sounds good, it reminds me of Scala Play's templates a bit.
https://www.playframework.com/documentation/2.8.x/ScalaTemplates
See the bit about template parameters

I guess it is a bit complicated by the fact that Gleam does support variables and modules with the same name? I think that came up? So you could technically have a variable called some_module and a module called some_module and it would be hard to decide just from the import of some_module that there shouldn't be a variable of the same name?
I guess we could go further and say that if there is an import of that name and not with statement to state the type of the variable with that name and the variable uses dot-access of some kind then we can assume it must be a useful of the module?
Or we introduce some other syntax for value blocks that don't introduce function arguments maybe?
{> import some_module

{= some_module.function() =}

Or something. (Not sure I love that but can't think of anything obvious right now.)

I personally think requiring all parameters to be declared up-front would be reasonable, to avoid complexity, and let the Gleam compiler figure out the right thing when there are ambiguities

I think I would also prefer all parameters to be explicit, I think there's probably quite a lot of tricky edge cases such as the module situation Michael describes which would be challenging to code. It's much simpler to implement the explicit system and I don't see it at being an inferior user experience. It might even help catch typos in some cases as we'll get an undefined variable error.

So we'd avoid inferring anything and assume that all parameters are stated with "with" statements at the top of the template file?

Yes, I think that would be the most straightforward.

I've put together #14 if you have the time to check it out and assess it at some point.
I suspect it means we can roll back the {[ name ]} syntax for StringBuilders as they will be identified by the type given in the corresponding with statement (or maybe in a for ... as ....)
Or we could drop the requirement for as String and as StringBuilder and infer it from the usage syntax but I imagine that is less explicit than we're aiming for here.
I'll probably remove the {[ name ]} before merging if I find time over the weekend.

Or we could drop the requirement for as String and as StringBuilder and infer it from the usage syntax but I imagine that is less explicit than we're aiming for here.

If I had a vote, I would vote for that (so making parameter types optional, even if that means keeping separate StringBuilder insert syntax)

If the string builder interpolation syntax is removed does this mean that the inner field of this type can no longer be interpolated? I don't understand how it would know the type of the inner field.
import gleam/string_builder.{StringBuilder}

pub type SomeArgument {
  SomeArgument(inner: StringBuilder)
}

Good point! I don't seem to be the best at thinking through the use cases :) We can leave it in.
@lpil are you in favour of making the as Type part of the with optional? Or leave it as required?

Either or sounds good to me.

I'll leave it required for the moment and we can revisit in the future.

This has been resolved in the latest release, I believe.


crash on startup

It was working just fine then started crashing as the mod menu started to initialize the mod
re2_framework_log.txt
reframework_crash.dmp
I am running the most recent version which is 1.53 I think
I am running windows 11, a RTX 4060, and an i7 as a CPU along with a metaquest 3 headset

Disable medal.tv
Make sure you are using the correct version of RE7 mod for your game. If you are using the very latest version of RE7, you need to use the "RE7.zip". If you are using the old, pre-RT version, you must use RE7_TDB49.zip

I've pinned down a crash to the medal.tv software. It's just not compatible with REFramework.


Clangd error

Opening vim I get this below  error, I'm using the git clone https://github.com/yegappan/lsp method to install
Error: LSP server /usr/local/bin/clangd is not found

Opening vim I get this below error, I'm using the git clone https://github.com/yegappan/lsp method to install Error: LSP server /usr/local/bin/clangd is not found

Execute
which clangd
and recheck the path to clangd in your system (and whether it is installed).
For example in Slackware current (full install) the path is /usr/bin/clangd and not /usr/local/bin/clangd as shown in the instructions.

To use the clangd language server, you need to install the clangd server. After installing, you need to provide
the correct path to clangd in the LSP server options.


Dynamic FormControlLabel Radio inside RadioGroup is not getting checked

I'm using a RadioGroup component to display a dynamic list of Radio options using FormControlLabel. The dynamic radio options are successfully getting displayed and I'm able to retrieve the selected radio option using onChange in RadioGroup. However, when I check a radio option, that particular option does not appear "checked".

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔

It's supposed to check the selected radio option.
Current Behavior 😯

Selected option is not getting checked. Basically the "checked" class is not being added only when the list of FormControlLabel is being generated as a result of the map function. But if I write a FormControlLabel manually, it gets checked.
Steps to Reproduce 🕹

The code is quite straightforward, hence I'm creating a codesandbox.io link. Please let me know if that is required and I'll create one. Here's the code, though:
export default class Book extends Component {
  state = {
    slot: null,
  };
  ...
  onChange = (e) => {
    this.setState({ [e.target.name]: e.target.value });
  };
  ...
  <FormControl component="fieldset" className={classes.formControl}>
    <FormLabel component="legend">Select availability slot</FormLabel>
    <RadioGroup
      aria-label="Slot"
      name="slot"
      className={classes.group}
      value={this.state.slot}
      onChange={this.onChange}
    >
      {this.props.experience.availability !== null ? (
        this.props.experience.availability.map(single => (
          <FormControlLabel
            key={single.id}
            value={single.id}
            control={<Radio color="primary" />}
            label={single.title}
          />
        ))
      ) : ""}
      // Some manual options in addition to the above dynamic list
      <FormControlLabel
        value="female"
        control={<Radio color="primary" />}
        label="Female"
      />
      <FormControlLabel
        value="male"
        control={<Radio color="primary" />}
        label="Male"
      />
    </RadioGroup>
  </FormControl>
  ...
}

Context 🔦

In the above code, this.props.experience.availability is an array of objects that I'm getting from a call I'm making to the backend. The call is being made in componentDidMount(). I use Redux, which makes the result available as a prop.
Now, if I manually add a few FormControlLabel components in the same RadioGroup, I can see that it's checked after selecting that option.
I've taken two screenshots - this is when the manual FormControlLabel is selected: https://ibb.co/pL5Fy6L and this is when one of my dynamic option is selected: https://ibb.co/Jq7mLN4
You can see that in the second one, the "Female" option gets unchecked but the option that I selected (20-06-2019) does not appear to be checked.
Your Environment 🌎

Tech
Version

Material-UI
v3.9.3

React
v16.8.6

Redux
v4.0.1

@ashishsanjayrao Could you provide a live minimal reproduction (e.g. CodeSandbox is great) using v4? Thanks.

Hi @oliviertassinari, while I was creating the CodeSandbox instance, I realised my mistake and found the solution to this. However, I've anyway created it and here's the URL: https://codesandbox.io/s/dydqr using v3.9.3 (because that's the version I'm using)
The issue was that I was passing an integer value to the value prop instead of string like: value={single.id} so I added toString() to convert it to string like this: value={single.id.toString()} and it's working fine now.
I'll be closing this issue now. Thanks a lot for your time!

Thanks for sharing the solution.

I still experience the problem, when I click a radio button, the input should get checked, but it does not.
I was looking at the provided CodeSandbox example, and when I click any of the radio buttons, for me nothing even gets selected/checked (the radio buttons are unresponsive).
So I created my own example, using v4 here: CodeSandbox example

@balazsorban44 The problem seems to be that no change event is triggered:
  it.only('should be ok', () => {
    function App() {
      return (
        <div>
          <form>
            <label>
              Product 1 <input type="radio" onChange={console.log} name="product" value="product-1" />
            </label>
            <label>
              Product 2 <input type="radio" onChange={console.log} name="product" value="product-2" />
            </label>
          </form>
        </div>
      );
    }

    const { getByLabelText, debug } = render(<App />);
    const radio1 = getByLabelText(/Product 1/);
    const radio2 = getByLabelText(/Product 2/);

    console.log('1', radio1.value)
    console.log('1', radio2.value)
    fireEvent.change(radio1, {
      target: { value: 'product-2' },
    });

    console.log('2', radio1.value)
    console.log('2', radio2.value)
    debug(radio1);
  });

Could it be a testing library question?

The test already crashes when rendering the <App />. Stack trace begins in jss.
@oliviertassinari The change event would be fired on the radio2 if I remember correctly. The values wouldn't change but the checked state. Could you create a sandbox that renders the same in the browser and the test environment so that we can compare with the actual behavior in the browser?

Thanks for sharing the solution. Been struggling for hours

Any chance to enhance the type of value here something else other than unknown? FormControlLabelProps.value?: unknown


[Dokka] Better support for versioned docs

It seems that Dokka already has a concept for providing versioned docs with a better UX which would also solve the problem with the version being part of the URL: https://kotlin.github.io/dokka/1.5.0/user_guide/versioning/versioning/
We should investigate if this could replace our current approach where we put docs in this structure:
/0.2.0/
/0.3.0/
/0.4.0/
/latest/

See https://github.com/Kotlin/dokka/issues/2022
/cc @nathan-contino-mongo

I think this makes a lot of sense since it's a Dokka feature -- I can't tell from the linked docs, but I assume this plugin creates a version selector on each page that the user can choose from? Seems like a pretty minimal change from our perspective, so let's give it a shot.
CCing @cbush since this might have some consequences for YokedOx going forward.


[SPARK-42140][CORE] Handle null string values in ApplicationEnvironmentInfoWrapper/ApplicationInfoWrapper

What changes were proposed in this pull request?
Similar to #39666, this PR handles null string values in ApplicationEnvironmentInfoWrapper/ApplicationInfoWrapper
Why are the changes needed?
Properly handles null string values in the protobuf serializer.
Does this PR introduce any user-facing change?
No
How was this patch tested?
New UTs

cc @gengliangwang

Yeah, this one GA passed

Thanks, merging to master

thanks @gengliangwang


feat: change radio buttons to check box and user selection

Description

Change the user selection option to check boxes.
User selection functionality
Display selected users as a horizontal scroll

Type of Change

[x] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read and ran all relevant commands as specififed in the Running Tests section of the Contributor Guide.
[ ] The title of the PR follows the Conventional Commits guideline
[ ] My local branch follows the naming standards in the Deepsource Branch Naming Convention or Biodiversity Branch Naming Convention
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy,
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is [test-exempt].
[ ] All existing and new tests are passing.

Yo @DesmondTambe there are some conflicts here. Please resolve them and update the PR so I can merge it.
I think we should gitignore all the generated files like app_route.gr.dart, injection.config.dart, etc. or simply not pushing them anymore.
Please what to do think about this @fonkamloic ?


Issue triage process updates

Overview/Summary
Updated/defined the issue triaging description for all issue types.
This PR fixes/adds/changes/removes

Module proposal
Orphaned module process
standard issues/feedback/questions

Breaking Changes
n/a
As part of this Pull Request I have

[x] Read the Contribution Guide and ensured this PR is compliant with the guide
[x] Checked for duplicate Pull Requests
[ ] Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Ensured PR tests are passing
[x] Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@PmeshramPM, @jeanchg, @jtracey93, @prjelesi, please have review/comment/approve. Thanks!

As agreed.


Expose Script Code Editor's edit menu

Add new method in ScriptEditor to get edit menu in Script Code Editor.
This can be accessed by plugins to add menu buttons here

Update
doc/classes/ScriptEditor.xml
editor/plugins/script_editor_plugin.h
editor/plugins/script_editor_plugin.cpp

Closing. This is not working correctly

Why go with "edit menu?" Just "menu" wouldn't be enough?


uv tree should support --show-version-specifiers like uv pip tree

I was surprised to find this is not supported.

It's not possible to support right now. We don't store this information in the lockfile.

Hm. Yeah it seems important to be able to show this for debugging things.
We could at least show them for direct dependencies and read from the pyproject.toml

We could show them for a subset of the tree, yeah.

I think it will be a non-trivial increase to the lockfile though, maybe like 1.5x the size?

How bad would it be to resolve from the lockfile and create an in-memory lock with the additional information for cases like this?

I think I'd rather just add them to the lock honestly. They're useful for debugging and they'll almost certainly be required by the PEP.


Carousel Animation Broken

Latest version of Angular UI Bootstrap (0.13.0) does not have smooth transitioning with the carousel. This is broken because the transitioning works on 0.12.0

Are you using Angular 1.4? Probably related to #3811. 0.13 does not support Angular 1.4 yet.

It's also broken with 1.3.13
See plunker  http://plnkr.co/edit/KixcEb1uDNNgdXEzTDCe?p=preview

@gytisgreitai The demo Plunk is broken. You have to add ngAnimate to your angular module initialization. Please check this modified Plunk:
http://plnkr.co/edit/OyihVYocklUSMboUpENL?p=preview

@jrief is correct - animations are now optional, so you have to specify the ngAnimate dependency manually.


how to do secrets issue

https://github.com/sparrell/TwinklyMaHa/pull/24

this is resolved that we will use a combination of secrets file and environmental variables.


fix(contracts): Improve readability of cosmwasm error messages

Description
Currently the error messages from a failing cosmwasm transaction are displayed as-is, but it contains a little bit of fluff at the start that could be removed from the terminal output (but still be present in the DEBUG output)
Example:
Query failed with (6): rpc error: code = Unknown desc = failed to execute message; message index: 0: Error parsing into type dao_core::msg::MigrateMsg: Invalid type:
    migrate wasm contract failed [CosmWasm/wasmd@v0.32.0/x/wasm/keeper/keeper.go:436] With gas wanted: '300000000' and gas used: '121070' : unknown request

Since the return format of CosmWasm error messages is out of the CLI's control, we should keep them intact as they might contain important information for the developer.


html tag issue

Vue.js version
2.0.2
Reproduction Link

~~https://jsfiddle.net/baturYe/x4m8w2xv/2/~~
https://jsfiddle.net/baturYe/x4m8w2xv/5/
Steps to reproduce
What is Expected?
template displays correctly
What is actually happening?
I am playing #5097 and found that when put \> in the template string, the result is not expected.

you can't add content to the same named slot multiple times.
your child component doesn't output the slot content anywhere
the child template has invalid html in line 15

Please explain in more detail what you actually expect. It seems you think that you can concatenate slot contents. You can't, that's not a functionality Vue provides.

Sorry, I paste the wrong link, I know it's invalid to use like that. Should vue complain about it when it is coded like this?

Well, that's because the backslash is an escape character in Javascript strings. using \ leads to the expected result.


Add some simple, step-by-step tutorial examples.

This is a first cut at some 'step by step' tutorial examples which I will write docs around and comment extensively in a follow-up PR.
This PR also fixes some typing issues with DocsQA which Justin's PR also attempts to address. I also fix the bakeoff DocsQA example's typing.

PTAL

PTAL

PTAL


Legacy Update saying that I have to be logged in as admin even though I am

Hello. I use Legacy Update on my Windows XP computer, however, I tried running it today and it won't let me saying that I have to be logged on as an administrator. I tried it on the actual Administator account and it still said the same thing. It worked fine yesterday, so I don't know whats happening. I even tried rebooting, but it still didn't work.

I'm getting this exact same issue and so are at least 2 other people on Discord, I think it's server-side?

Sorry, yes this is a server issue. My fault. Working on it now.

Fixed now. Thanks heaps for posting/commenting here and on the Discord. (If it still doesn’t work, try again in about 15 mins.)

thanks!


multipackage monorepo base configuration

Monorepo basic configuration in reference to #403
Changes made to directory structure
|- packages (new root directory)
|-- create-react-app (was parent directory)
|-- react-scripts (was global-cli)
Testing
To test as you would use npm i create-react-app -g
cd packages/create-react-app && npm i . -g to install locally or npm link
create-react-app my-app should work as before
To test the build process
cd packages/react-scripts && npm i && npm run build
Notes
This is just the directory structure changes, perhaps a little discussion around @hzoo's comment regarding Lerna Independant is warranted?
# or global?
npm install lerna --save-dev
./node_modules/.bin/lerna init (add lerna.json)
./node_modules/.bin/lerna bootstrap (npm installs packages, links)
# when publish is necessary
./node_modules/.bin/lerna publish --independent

I'll fix that travis explosion

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@gaearon I'm having a little trouble with the e2e, did I mess something up something obvious I am missing?

Try running sh tasks/e2e.sh locally and see why React doesn’t end up in node_modules?

I think I have it pegged, sorry about the delay, thanks for the help!
On Aug 8, 2016 5:52 PM, "Dan Abramov"<EMAIL_ADDRESS>wrote:

Try running sh tasks/e2e.sh locally and see why React doesn’t end up in
node_modules?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebookincubator/create-react-app/pull/405#issuecomment-238402168,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACifKGN8iULTMarLOx3y1gT5k90HeK_ks5qd7MzgaJpZM4JfbzM
.


Release 2.0

What problem does this PR solve?
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has configuration change
Has HTTP API interfaces change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to update the tidb-ansible repository
Need to be included in the release notes

Hi @jefftsai2050 I think you created this by accident. Closing it.


x/mobile/webview: cross iOS/android web view

This is a dup of https://github.com/golang/go/issues/10247, it was autoclosed.

From the original:
From the original:

Embedding a web browser is a common strategy for apps that are not heavily focused on UI, on both Android and iOS. The web also provides a very easy way to use other UI systems in a Go application.

Is there enough commonality between the embedded web views on Android and iOS to build an API usable entirely from Go? The Android WebView has a wide interface and its not clear what subset lines up with iOS. Still, this would be very nice to have.

cc @hajimehoshi @eliasnaur


Add optional .wat module support to Module::new

A test-only wat2wasm function has been implemented several times in the Wasmi codebase, especially in tests which is an indicator that this functionality was crucially missing from Wasmi's module parsing.
Thus we added this feature optionally behind the new wat crate feature in Wasmi.
This also improves Wasmi's Wasmtime API mirroring because Wasmtime also provides .wat parsing support in its Module::new API.
We explicitly do not support .wat parsing in Module::new_streaming since there is no streaming parsing support for .wat files via the used wat crate.

The miri job fails due to timeout, so not because of this PR.


send_only_records as config parameter

Hi,
I had some difficulties using this plugin to collect metrics into Blueflood, so, I had to modify the plugin and add a new config_param that seems to be a good feature to add to the project.
Basically, Blueflood is waiting for a list of records, but http_buffered sends [[tag,time,record]], which was a problem.
To solve that, I add a new config_param, send_only_records, that can be enabled to send only an array of records.
Thanks.

Coverage decreased (-43.6%) to 50.877% when pulling 8d4614488dd2388f7f4d54d1afc0a077145c0613 on bmandrade:only_records_feature into b133a8662675764b1a15affee21af58d4357187a on ablagoev:master.

Forgot to update the tests, on the way.

Coverage decreased (-43.6%) to 50.877% when pulling dbc6dcad3f33c5c45ed2c2927b0513460902086c on bmandrade:only_records_feature into b133a8662675764b1a15affee21af58d4357187a on ablagoev:master.

I believe the errors I'm getting in Travis, are not related to the changes I've made to the code.
Can you confirm this, @ablagoev ?

bump....
Anything on this @ablagoev ?

Closing this PR due to lack of response.


Use redux-storage to persist state

Closes #156
Summary:
I recently learned that we'd run into problems when updating redux reducers. For more context, see #156.
The solution is to use redux-storage, with @mathieudutour's migration decorator. This way, we can simply specify how to transform redux state from one version to the next.
This change lays the groundwork for that, by shifting away from our custom redux-persistence and adding all the stuff we need for our first migration.
One change is that our custom persistence layer was able to mutate the tasks to remove properties that aren't needed. While this is undoubtedly possible with redux-storage, it's non-trivial, so instead I just used the blacklist feature to omit tasks altogether. This does mean that there's an awkward flash when the app loads, but I have plans to deal with this in a future PR.

Thanks for the review, @mathieudutour!

Another thing: we should do a refreshProject() on blur, but that's for another PR

Not sure what you mean; do you mean if the window is unfocused? What blur event?

Oh sorry focus event, not blur. When the user unfocused the window, they might do some changes to the package.json, or delete a project. So when the window is focused again, we should refresh the project

Gotcha, yeah, makes sense. Will tackle when I rework refreshProjects to use a saga


Nested references are not resolved for local files

If #454 testcases are changed to use a relative file reference, the nested definitions are not resolved.
tags:
- name: admins
  description: Secured Admin-only calls
- name: developers
  description: Operations available to regular developers
paths:
  /inventory:
    get:
      summary: searches inventory test
      operationId: searchInventory
      responses:
        200:
          description: search results matching criteria
        default:
          $ref: './issue-454-file-components.yaml#/responses/GeneralError'
definitions:
  ModelWithNestedProperties:
    properties:
      remoteProperty:
        $ref: './issue-454-file-components.yaml#/definitions/RemoteComponent'

Test case:
    @Test
    public void testNestedFileRefs() {
        Swagger swagger = new SwaggerParser().read("src/test/resources/nested-file-references/issue-454-file.yaml",
                null, true);

        assertNotNull(swagger.getDefinitions().get("ErrorModel"));
        assertNotNull(swagger.getDefinitions().get("ModelWithNestedProperties"));

        ModelImpl model = (ModelImpl)swagger.getDefinitions().get("ModelWithNestedProperties");
        Property property = model.getProperties().get("remoteProperty");
        assertNotNull(property);
        assertTrue(property instanceof RefProperty);
        RefProperty ref = (RefProperty) property;
        assertEquals(ref.get$ref(), "#/definitions/RemoteComponent");
        assertNotNull(swagger.getDefinitions().get("NestedProperty"));

        ModelImpl nestedModel = (ModelImpl)swagger.getDefinitions().get("NestedProperty");
        assertNotNull(nestedModel);
        assertNotNull(nestedModel.getProperties().get("name"));
        assertTrue(nestedModel.getProperties().get("name") instanceof StringProperty);
    }

The attached tar ball contains a testcase and test data to reproduce the issue.
test.tar.gz

Note that issue-454-file-components.yaml is missing in your tarball.
But I can confirm that I'm having the same issue.
Current workaround:
referencing to local definitions as remote ones.
e.g. in issue-454-file-components.yaml
responses:
  GeneralError:
    description: General Error
    schema:
      $ref: '`./issue-454-file-components.yaml#/definitions/ErrorModel'

Note that the HEAD now also resolves references relative to the input file instead of from the file in which the reference is located. This isn't the case with v2.0.1.
So references in referenced files need to be relative to the input file, which seems quite broken to me.

Hi @NathanTI8M please send the spec again, I have tried to decompressed the file but it throws an error. Or if you preffer send the spec here. Thanks. we have a branch for this issue, maybe you can test your case against it. fix-relative-resolving in this PR https://github.com/swagger-api/swagger-parser/pull/725

Hi @gracekarina
The content of the file us the same as the existing issue-454-components.yaml
issue-454-components.zip
I forgot to mention that. Sorry for the inconvenience.


TypeError: Interval() takes no arguments

Seems to be a python version incompatibility: with a Python 3.9.7 env
import annotated_types
annotated_types.Interval(gt=0)

gives TypeError: Interval() takes no arguments.
If I try with Python 3.9.16 or Python 3.10.11, no errors.

This seems like it's a bug in < 3.9.X. There's nothing special going on with this class and it works on other Python versions. Sadly I don't think there's anything we can do about this.

I encountered this issue recently, and found out this is a bug only exists in 3.9.7, here is the bug report.


test/keccak-normalize-table: unittest for lookup tables

This PR verifies the properties of the lookup tables used in Keccak. Also minor refactoring for testability.

the degree is the limit of gate poly degree, not related to the circuit height at all..


Can't create two auth client instances

Describe the issue:
The getInstance API doesn't create multiple instances.
How to reproduce:
const authClient = AsgardeoSPAClient.getInstance();
const fooAuthClient = AsgardeoSPAClient.getInstance("foo");
// initialize both clients (set configs to use session storage)
// sign in to both clients

on session storage, only one auth client instance can be seen.
Expected behavior:
Both auth client instances should be available on the stoarage (in this case, the session storage)
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: react sdk 1.0.4
OS:  ubutnu 20.02
Browser: chrome

Are you trying to create multiple instances in React SDK or SPA SDK?


Feat #69/mark for deletion

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

What is the new behavior?

Other information

Looks very good! One last thing, just please run yarn format because some files were not formatted correctly and you can merge the pull request :)


julia_18-bin: add support for aarch64-linux

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I do use aarch64-linux but I generally care about «general-purpose» part so I am never sure how the linear-algebra part working should even look like

As long as the upstream testcases pass, I would see it as fine, there's little we can do with a binary tarball.

@ofborg eval

I am on unstable channel, I updated the channel and added julia_18-bin to my configuration, but I get:
error: Package ‘julia-bin-1.8.1’ in /nix/store/k5bkr229krm4fbxkamh8sp20xz34jsg9-nixos/nixos/pkgs/development/compilers/julia/1.8-bin.nix:52 is not supported on ‘aarch64-linux’, refusing to evaluate.

       a) To temporarily allow packages that are unsupported for this system, you can use an environment variable
          for a single invocation of the nix tools.

            $ export NIXPKGS_ALLOW_UNSUPPORTED_SYSTEM=1

        Note: For `nix shell`, `nix build`, `nix develop` or any other Nix 2.4+
        (Flake) command, `--impure` must be passed in order to read this
        environment variable.

       b) For `nixos-rebuild` you can set
         { nixpkgs.config.allowUnsupportedSystem = true; }
       in configuration.nix to override this.

       c) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
         { allowUnsupportedSystem = true; }
       to ~/.config/nixpkgs/config.nix.
(use '--show-trace' to show detailed location information)

Is this because the channel is not up-to-date with master? How soon can I expect it to be there?

Usually within one or two days, depending on the queue length on hydra.


Variable Explorer cannot show the "_"

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
When a variable name with "_", such as "a_a", you can just see "a a" in the column rather than "a_a"

What steps reproduce the problem?

import pandas as pd
a=pd.DataFrame()
a['a_a']=1
Open the variable explorer and check the column name of "a_a"

What is the expected output? What do you see instead?
See the "a_a" instead.
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version: 4.2.3 （since 4.1.x , I forgot when, and it still happends in 5.0.0）
Python version: 3.8.8
Qt version:
PyQt version:
Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

Hey @mIgLLL, thanks for reporting. We'll take a look at it for our next version.


Don't depend on ? affecting type inference in weird ways

This is likely to stop working in the future versions of Rust, see https://github.com/rust-lang/rust/pull/122412#issuecomment-2037813927.

Merging this more as a code cleanup. For this breaking in the future, I'm assuming that is non-urgent as I'm assuming a lot needs to be worked out to uphold stability expectations before that will matter.
@bors r+

:pushpin: Commit 898969e4a4bd78be8334c7ddd201b571e7c7e0a7 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 898969e4a4bd78be8334c7ddd201b571e7c7e0a7 with merge 157066631b6571c8b7ed5e41c75be38d7367eb1b...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 157066631b6571c8b7ed5e41c75be38d7367eb1b to master...

@epage yes, this is non urgent (although this needs to happen before the change, so that rustc's CI can pass), so I just made a PR since this is a simple change & is indeed a little cleanup.
@weihanglo this is required for the crater run on #122412, although I think I can workaround that.

Thanks. I could do a cargo submodule update earlier if needed. Usually it happens on Tuesday and Friday.


How can i lock a worksheet?

tried this, password removed when open the file with excel or numbers ( mac app )

@randym where did i miss?


Final project

Added a progressing bar, added more arguments, restyled readme, updated requirements.txt, obliged to move apk_decompiler to the main directory due to import errors

Ho salvato il mio path all'interno del file config


misc: Add toString() to OutputBuffer stats

Sample output:
[bufferedBytes: 0B, bufferedPages: 0, totalBytesSent: 6.69KB, totalRowsSent: 400, totalPagesSent: 4, averageBufferTimeMs: 0, numTopBuffers: 4]
  D0: [finished: false, bytesBuffered: 0B, rowsBuffered: 0, pagesBuffered: 0, bytesSent: 1.67KB, rowsSent: 100, pagesSent:1]
  D1: [finished: false, bytesBuffered: 0B, rowsBuffered: 0, pagesBuffered: 0, bytesSent: 1.67KB, rowsSent: 100, pagesSent:1]
  D2: [finished: false, bytesBuffered: 0B, rowsBuffered: 0, pagesBuffered: 0, bytesSent: 1.67KB, rowsSent: 100, pagesSent:1]
  D3: [finished: false, bytesBuffered: 0B, rowsBuffered: 0, pagesBuffered: 0, bytesSent: 1.67KB, rowsSent: 100, pagesSent:1]

@xiaoxmeng Thanks for reviewing! I haven't finished the test yet, but here's the output what it looks like:
[bufferedBytes: 0B, bufferedPages: 0, totalBytesSent: 6.69KB, totalRowsSent: 400, totalPagesSent: 4, averageBufferTimeMs: 0, numTopBuffers: 4]
  D0: [finished: false, bytesBuffered: 0B, rowsBuffered: 0, pagesBuffered: 0, bytesSent: 1.67KB, rowsSent: 100, pagesSent:1]
  D1: [finished: false, bytesBuffered: 0B, rowsBuffered: 0, pagesBuffered: 0, bytesSent: 1.67KB, rowsSent: 100, pagesSent:1]
  D2: [finished: false, bytesBuffered: 0B, rowsBuffered: 0, pagesBuffered: 0, bytesSent: 1.67KB, rowsSent: 100, pagesSent:1]
  D3: [finished: false, bytesBuffered: 0B, rowsBuffered: 0, pagesBuffered: 0, bytesSent: 1.67KB, rowsSent: 100, pagesSent:1]

Let me know if it looks ok.

@assignUser @pedroerp @xiaoxmeng The tests are green now, could you please review again? The current CMakelists.txt looks like this
velox_add_library(velox_test_util ScopedTestTime.cpp TestValue.cpp)
velox_link_libraries(velox_test_util PUBLIC velox_exception)

velox_add_library(velox_test_output_matcher OutputMatcher.cpp)
velox_link_libraries(velox_test_output_matcher PUBLIC Folly::folly GTest::gtest re2::re2)

Thank you!

@xiaoxmeng I've addressed all comments and added test. The tests all pass now. Do you want to review again? Many thanks!

CI pass dropped to 37% with this change.


Gradio can not work

After correctly configuring the NExT-GPT-Legacy demo (using pre-trained weights and following the instructions in NExT-GPT-Legacy/README.md, completing the bash app.sh script), the Gradio web demo interface does not respond. After entering a prompt, the demo interface does not display the corresponding inference results. Additionally, there are errors in the input for audio, video, etc., as shown in the image:
input:

output:

console:

As can be seen, my backend is displaying the prompt and the corresponding text modality inference results. What should I do to ensure that normal inference works on the web interface?

正确配置NExT-GPT-Legacy演示后（使用预训练权重并按照中的说明NExT-GPT-Legacy/README.md完成bash app.sh脚本），Gradio 网页演示界面没有响应。输入提示后，演示界面没有显示相应的推理结果。此外，音频、视频等输入也有错误，如图所示：
输入：
输出：
安慰：
可以看到我的后端正在显示提示和对应的文本模态推理结果。我应该怎么做才能确保在 Web 界面上进行正常推理？

I'm not sure why, but the issue is resolved. However, the code for this demo doesn't seem to support generation tasks. The author mentioned that the diffusion model used for generation tasks will automatically be downloaded. I'm curious how I can modify the code to support generation tasks.


Sample Recorder is not reliable

The Sample Recorder program is saving data that can't be decoded back.
Something is not right with it and need investigation.
The Sample Fix program was created to resynchronize samples created on the first (buggy) version of the recorder.
The fix app can't decode all the data recorded, so the internal "chunk" format seems to not be respected by the recorder.
Example:
Input: FM7_CarDash_2019-04-25_Xbox.bin
Output: FM7_CarDash_2019-04-25_Xbox_TS-fix.bin
Fix: timestamp
Can't decode more chunks, the rest of the source is ignored
Chunks: 16334 read, 16334 written (diff: 0)
Size: 9499739 bytes on input, 5275882 bytes on output, (diff: -4223857)

During that execution, 16334 chunks were decoded, but still 4223857 bytes were ignored at the end of the file, because of this issue.

I've made a big mistake missing the remarks about Stopwatch.ElapsedTicks in the documentation.
The first implementation of the recorder was not starting the Stopwatch, and the second one is writing the ElapsedTicks without any conversion to a TimeSpan compatible value.
This needs an overhaul.


Show ammo HUD for weapons that can be fired multiple times

We need an ammo count in the HUD for weapons that can fired multiple times (such as Shotgun).
This could be screen UI somewhere on the screen or world UI near the Grub. Feel free to just experiment and we can reach UI consensus together.

https://github.com/apetavern/grubs/assets/43252311/b6ba1cc9-0479-4e35-a77c-5ae1b735c4b6


Dates along the bottom of the histogram are not showing.

On Firefox 37.0.2 on win 2003 the dates selection (axis/scale) at the bottom is cutoff the histogram.
In Win 7 same version of Firefox it displays correctly.
A minor problem.

I see the same effect but I can constantly reproduce how to make them to appear:

copy the share link and paste it into the url line
the same graphics appears but with the dates below
Thats somehow a strange effect

When searching and pasting the link it works now on both.
http://loklak.net:3001/statistics?q=youtube&since=2015-05-01_06%3A00&until=2015-05-06_10%3A00
A few times the results page just had the search, start and end date and nothing below but it is all working now.
Should the Browser sync disconnect banner and then sync reconnect banner comes again when asking for results?

I believe this is solved, my latest test was successful. If anybody sees the effect again, please report the browser type you used.


FEAT: Upgrade langchain (0.3.x) and associated packages including langchain-astradb

This upgrades our langchain dependency to 0.3.x, adds ~= requirements to associated packages, and allows the use of the latest langchain-astradb as a result.

Did you encounter any issues with Pydantic when installing and running the backend? @erichare

Did you encounter any issues with Pydantic when installing and running the backend of Langflow after the upgrade to 0.3.x? @erichare

I admit i haven't extensively tested it, would love some help from others, but i was able to build and start the backend service without problems. I was also able to run my existing flows i had saved. I do see depcration warnings about pydantic:
Starting Langflow v1.0.19...
/Users/erichare/GitHub/langflow/src/backend/base/langflow/utils/validate.py:224: LangChainDeprecationWarning: As of langchain-core 0.3.0, LangChain uses pydantic v2 internally. The langchain.pydantic_v1 module was a compatibility shim for pydantic v1, and should no longer be used. Please update the code to import from Pydantic directly.

But it doesn't appear to block the startup. question is whether we would like to go through and remove references to pydantic_v1, or defer that for later?

Did you encounter any issues with Pydantic when installing and running the backend of Langflow after the upgrade to 0.3.x? @erichare

I admit i haven't extensively tested it, would love some help from others, but i was able to build and start the backend service without problems. I was also able to run my existing flows i had saved. I do see deprecation warnings about pydantic:
Starting Langflow v1.0.19...
/Users/erichare/GitHub/langflow/src/backend/base/langflow/utils/validate.py:224: LangChainDeprecationWarning: As of langchain-core 0.3.0, LangChain uses pydantic v2 internally. The langchain.pydantic_v1 module was a compatibility shim for pydantic v1, and should no longer be used. Please update the code to import from Pydantic directly.

But it doesn't appear to block the startup. question is whether we would like to go through and remove references to pydantic_v1, or defer that for later?

Thats cool. Earlier when I tried the upgrade I faced some Pydantic upgrade error.
Let me review it and shall let you know soon.

Tested. Looks good to me.
The SecretStr error I observed earlier seems to be managed. Perhaps, as a follow-up pull request, we can add a chore to upgrade Pydantic from version 2.0 to 3.0 in some functionalities.

@erichare  can you resolve the conflicts in uv.lock?


No computed baseline value set in /camera_info topic in the recorded rosbag file for right infrared sensor

Required Info

Camera Model
D400 (D435i)

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
Ubuntu 18.04

Kernel Version (Linux Only)
5.4.0-121

Platform
PC

librealsense
2.51.1 RELEASE

Language
C++

Segment
Others

Issue Description
Hi,
I am trying to record a ros bag file of both Left and Right Infrared images using the rs-record utility. So to enable the infrared streams I modified the rs2::config cfg  as follows to suit my needs:
   cfg.enable_stream(RS2_STREAM_INFRARED, 1,848,480, RS2_FORMAT_Y8, 15);
   cfg.enable_stream(RS2_STREAM_INFRARED,2,848,480, RS2_FORMAT_Y8, 15);
   cfg.enable_record_to_file(out_file.getValue());

As mentioned in the following issue in the realsense-ros repo, the Projection matrix should have the P[0,3] = -fx * B in the Camera Info topic (according to sensor_msgs/CameraInfo )
I recorded the bag file using the modified rs-record and there is no corresponding value for the computed -baseline*fx value in the Right Infrared Sensor Camera Info topic. The values are same for both left and right infrared camera topics. I even cross-checked with the realsense-viewer to record a rosbag file, and it also has the same issue for the Camera Info topic.
Although, the baseline value can be queried in the ´/device_0/sensor_0/option/Stereo_Baseline/value´ topic, it makes the bag file recorded by rs-record difficult to use with some SLAM algorithms which depend on the Camera_Info topic to query the baseline.
Is there any parameter in the rs::config I can change so that the camera_info topic value will be correctly set in stereo recording setup?

Hi @vanguard478  I believe that this issue has been discussed on the RealSense ROS forum over several different cases - including https://github.com/IntelRealSense/realsense-ros/issues/1310 that you quoted - and so it can be a difficult subject to follow due to one case leading to another and then another.
It may therefore be best to link to a pull request at https://github.com/IntelRealSense/realsense-ros/pull/1242 that was eventually created and merged into the RealSense ROS wrapper at the end of the lengthy discussions.
The comments in this discussion remark that infra2 used the frame_id of infra1.  This is because having the right camera frame_id equal to that of the left camera is required by StereoCameraModel from image_geometry and stereo_image_proc
This subject has been further discussed at https://github.com/IntelRealSense/realsense-ros/issues/2220#issuecomment-1010720769 where suggestions for dealing with it were offered.
The information applies to the RealSense ROS wrapper though.  I am not aware of a method by which the issue could be addressed through librealsense SDK scripting, unfortunately.

Hi @vanguard478  Do you require further assistance with this case, please?  Thanks!

Hi,
I referred to the issues which you pointed out and I think Camera Info matrix are properly set by the ROS wrapper. The issue with rs-record is still there. As of now I am just modifying the published message as per my requirement.

Thanks very much @vanguard478 for the update

Hi @vanguard478  Can this case be closed now or do you have further questions, please?  Thanks!

Hi @MartyG-RealSense, I don't have any further question regarding this. I just hope this issue with the Camera Parameters being incorrectly set in the bag file recorded by the rs-record utility is fixed in an future release.


Slurm runtime error due to deprecated slurm syntax

Hi,
When i run the code in the vignette "supervised_marker_map = MarkerMap(
adata.X.shape[1],
hidden_layer_size,
z_size,
len(adata.obs[group_by].unique()),
k,
loss_tradeoff=0,
)
train_model(supervised_marker_map, train_dataloader, val_dataloader)" i get following error:

We think that the error is coming from the lightning_fabric package. The slurm plugin is, somehow, using a submit script with deprecated slurm parameters (slurm has been recently uprgaded to version 23). Is there a way to somehow, tell the plugin to use nstask-per-node instead?
Your help is greatly appreciated, thanks!

Hi @linderss,
Thank you for submitting an issue! Unfortunately, I was not able to reproduce it.
To my knowledge, MarkerMap does not automatically make use of slurm or specify a particular number of ntasks anywhere, so this must be something particular to your setup. train_model does create a pl.Trainer which might be where this is starting?
If you post a longer stack trace I might be able to help, in particular what is the last line of the MarkerMap package in the stack trace.
If this is an old version issue, it could be an old pytorch/pytorch-lightning issue? What versions of those are you currently using?


custom templates?

Like .jade?

If it would usually work with plain PurifyCSS - it'll work here too. For instance, PurifyCSS recognizes constraints like class="foo" or className="foo" and some other. So if you really want to make sure your jade templates work, try to convert them to html using html-loader. Just an idea, not guaranteed - I don't use Jade myself. I have used EJS, but I switched to JSX - and that works just fine.

Usually they look like:
p#elementIdName.classOne.classTwo my text here

Sometimes the classnames are dynamic variables, it would work just fine if this plugin can target emitted files, then the jade have been converted to html files.

The plugin scans over the files to be emitted and uses the captured ones as input to PurifyCSS. You could play around with providing it with extensions to use to pick up. Because with the jade format itself, PurifyCSS likely won’t work as it wouldn’t know where the CSS names start or end (class names, ids, other specifiers that are valid constraints to create selectors, etc…). So it’ll likely use the finalized HTML.

That's interesting, so if emitted files with paths:
index.html
subfolder/test.html

Can it be scanned with this config?
new PurifyCSSPlugin({
  paths: [
     'index.html',
     'subfolder/test.html'
  ]
})

Or maybe **/*.html? I wouldn't want the plugin to scan the whole node_modules tree.

@beckend I'm working on a new version that should achieve just that (now path management is on the user side) although the current stable version might work too.


Cookie based session doesnt work in the 'App'

The sessions of an ajax call and static http request (e.g. img src="http://samedomain.com/theImage.jpg") are different. However, when it's built in phonegap, it's working fine.

Hi @mwbrooks I see the proxy thing works. but in my app now, I'm using passport to login with socialnetworks(fb, ig, twitter and etc). and I tried adding "/proxy/" before the url but it's throwing broken urls, specially when it's going to redirect to social network login. do you have idea, how to implement proxy with passport js ?
thanks

Hi @rlaurente,
The latest PG Dev App / CLI now renders apps from the file:/// rather than http://. This should allow you to access external services without the proxy workaround.
Let me know if you continue to have issues!

I'm going to close this issue. Please let me know if you have any issues.


Error on startup

Hello, I'm seeing the following error on start up:

packer.nvim: Error running config for nvim-surround: ...ck/packer/opt/nvim-surround/lua/nvim-surround/config.lua:64: bad argument #1 to 'pairs' (table expected, got nil)

I'm using the following initalization in my Packer init.
  use {
    "kylechui/nvim-surround",
    config = function() require("nvim-surround").setup({}) end,
    event = "BufEnter",
  }

Interestingly, I dont get the error if I remove the BufEnter event.

Maybe try updating the plugin? I am currently using the same thing (I think) and it appears to be working just fine:
    use({
        "kylechui/nvim-surround",
        config = function()
            require("nvim-surround").setup({})
        end,
        event = "BufEnter",
    })

Closing this issue due to inactivity for an entire week. If the issue still exists, feel free to re-open this issue