Spanner table.read() returns a list of column name-value pairs. Why not an object?

The response of calling table.read(query) is a list of rows, but the row isn't a native object, instead it's a key-value pair-style object:
[ [ { name: 'id', value: { [Number: 1] value: '1' } } ],
 [ { name: 'id', value: { [Number: 2] value: '2' } } ],
 [ { name: 'id', value: { [Number: 3] value: '3' } } ]

Why wouldn't we match these up as a proper object? Since we don't customers end up having to write a helper-snippet just so they can pass around a useful object... That is, it'd be nice if the rows returned were...
[ {id: '1'}, {'id': '2'}, {'id': '3'} ]

Another clash of idiomatic vs API "what-ifs". Call .toJSON() on a row to get the most JS-like thing we could do.
I lost access to the repos we developed in, so I can't link to the history of the decision.

@vkedia This seems... silly. What were the what-ifs so we can sort this out?

I believe it was mostly because columns aren't always named.
Solution: don't include them in the default response, make them available in the apiResponse argument
Problem: The default response shouldn't exclude data

Yes the issue is that columns might not have names or there might be
duplicate names. So it's not guaranteed that each spanner row can be
converted to a json object.
On Mar 17, 2017 9:38 AM, "Stephen Sawchuk"<EMAIL_ADDRESS>wrote:

I believe it was mostly because columns aren't always named.
Solution: don't include them in the default response, make them available
in the apiResponse argument
Problem: The default response shouldn't exclude data
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-node/issues/2103#issuecomment-287406133,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATdef_xv90p1bfpgznGfIObanj7jOIcDks5rmrcZgaJpZM4MgxAX
.

The guidelines of this library have always been: make the common case easy and the advanced case possible.
In this situation, it seems that we're making the common case annoying to make the advanced case possible, which doesn't make sense to me.
I believe there are a few reasons where we may have either missing names or ambiguous results, such as expressions (COUNT(*)) or duplicate names (due to interleaved tables). I think the right thing to do here is:

No matter what, the results should be addressable by their position in the query. That is, the results of SELECT a, b, c ... should always have the value of a as row[0] and b as row[1], etc.

If there is one and only one unambiguous value for a column with a name, that should be addressable by that name on the result object. That is, SELECT SingerName FROM Singers WHERE SingerId = 1 should return a row where I can get the result by addressing row.SingerName.

If there is ambiguity anywhere in the results (such as duplicate names), we should throw a helpful error saying:

"Sorry, you asked for $ColumnNameHere in the result, but there were multiple fields with that name. You should either use an alias in your query or address the result you want by its position."

How's that sound?
/cc @vkedia

/cc @stephenplusplus : I think the question then becomes about the best way to have a thing that acts partly like an array, but with a few addressable properties for things falling into item (2) above.

Yeah, this was my original solution as well. Wish I could link to the original discussion, hoping to get access back soon.

But basically:
row = {
  0: 'value-1',
  1: 'value-2',
  2: 'value-3', // does not have a col name

  namedProp1: 'value-1',
  namedProp2: 'value-2'
}

for (var prop in row) {
  // only iterates named properties
}

The hard part is how do we communicate the ambiguity error. If we throw or return it, that takes users down the non-success execution path of their code. Could we just use the docs to state "Here's how we return the response and here's why we do it" and leave it to the user to consider when building their query?

We also want:
row.forEach(function (column, i) {
  // i = 0, column = { name: ... , value: ...}
  // but we DON'T want i = namedProp1, column = value
});

Regarding the ambiguity, I see a couple different cases:

You're specifying the columns in a query and it results in having duplicate names.
Throwing an actual error here doesn't seem like a bad idea. You know your query, you'll see it and go "oh got it", and fix the error during development.

You're selecting * and it results in duplicates.
Again, throwing an error here seems reasonable. You can find this during development. It's not that scary to see "you are asking for something that is not clear... you should be more clear". Again, an actionable error message here is very important.

You're selecting * and right now there are no duplicates, but when you add a new column later to a table it results in duplicates.
Throwing an error here is a smidge scary because it means that production code may stop working due to the schema change. Then again, you're altering the schema on your live production instance while not being specific about what you want. I think the better thing to do here is to recommend that you be specific in how you address fields (via an index) or don't duplicate names (in the global docs we do this, by suggesting you avoid the vague name of Id and instead use specific names like SingerId).
Being totally honest here, I suspect that most people won't even think about this and will blindly take the results we have now and turn them into an object. In that case, the last value found (for a duplicate) would win, and people would go looking for why the value for their id column is wrong, not realizing that their little helper function blasted out the first one and swapped it with the second. If anything, throwing an error here is a very good idea unless Spanner gives us enough information to tell which table a field came from (but I don't believe it does).

So my vote still is for the "if you ask for a named property that is ambiguous, we throw an actionable error message saying so". We also document in the run() method that if you're using SELECT *, you should be careful because you may grab stuff that conflicts and that may cause an error if you ask for something by it's name and we don't know which one you mean.

So I went through the earlier discussion and @stephenplusplus had suggested the exact solution there as well. But my objection at that point was around duplicate column names. I am fine if column with duplicate names cannot be retrieved by name and we just document that fact. But I would also like to have some way for user to know those column names. So lets says columns 0 and 1 both have the name 'a'. I am ok if both of them are just addressable by index. But there should be some way for user to get the names for both these columns. It can be some sort of extra metadata property on the row object which just contains the names and types for each column.

But there should be some way for user to get the names for both these columns.

I'm confused what you mean by this. Can you explain?

So user should be able to know what the name of a column whose index is i is. It can be as some sort of metadata on the row object.

could the metadata only exist if there is a conflict? adding a metadata object to every row sounds really expensive...

This issue was moved to googleapis/nodejs-spanner#27


[BUG] Chart not working or it is... but the colour is always black (so I can't see it!) ?

Describe the bug
I've taken your chart data example and implemented within a NodeRed function so that I can validate the chart function working.  The code is as below (using the "quote" option to embed within this description).

function processChart(dataMessage) {
// Correctly access values, yAxisTicks, and yAxisLabel directly from dataMessage
var values = dataMessage.values;
var yAxisTicks = dataMessage.yAxisTicks;
var yAxisLabel = dataMessage.yAxisLabel;
// Construct the chart object
var chart = {
    data: values,
    yAxisTicks: yAxisTicks,
    yAxisLabel: yAxisLabel
};

return chart;

}
// Your exampleDataMessage structure is correct as per the revised function
let exampleDataMessage = {
"values": [
{
"value": 10,
"label": "Jul"
},
{
"value": 20,
"label": "Aug"
},
{
"value": 30,
"label": "Sep"
},
{
"value": 40,
"label": "Oct"
},
{
"value": 50,
"label": "Nov"
}
],
"yAxisTicks": [10, 50, 100],
"yAxisLabel": "Amount"
};
let processedData = processChart(exampleDataMessage);
// This will correctly structure your return object based on the processed data
return {
topic: 'data',
payload: processedData
};

Nothing appears on my panel.
NOTE:  Within the "chart" node, I note that the colour is black.  If you change it to another colour and click "Done" and then go back into the "chart" node configuration, its reverted back to black again.
To Reproduce
Steps to reproduce the behavior:

Wire up a NodeRed function with the above code connected to the "chart" node.
Instigate the function using a standard inject node
Check what is dislayed on the panel

Expected behavior
A simple chart appears
Version information:

node-red-contrib-nspanel-lui Version: 0.0.1-alpha.52.1
Panel type: uk

Hey,
please note, that values are expected to be in msg.payload.values.
The code processing the data you posted puts them into msg.payload.data.
You could modify it like this:
// Construct the chart object
var chart = {
    values,
    yAxisTicks: yAxisTicks,
    yAxisLabel: yAxisLabel
};

return chart;

Have a nice day!

Thank you @laluz742 .
There still is a bug (as highlighted above) where the colour of the chart remains black even after its colour has been changed.

Hey, I cannot reproduce your color issue.
Which commands do you send to the panel exactly?

This is within the Chart node itself, e.g. go into Edit Chart mode and change the colour there.  Once changed, click done.  If you go back into the colour editor, its reverted back to black.

Ok, now I got you.
For a quick workaround you could change file nodes/nspanel-page-chart.html line 61 like so:
<input type="color" id="node-input-chartColor" style="width: 42px" />

Thank you!


Website Redesign

This epic will encompass the redesign of our website with an improved user experience and website performance so folks are able to "get involved" (Join, Donate, Mentor, Partner) and more concisely learn the benefits of getting involved from our homepage. Also, improved will be our code schools page, scholarship pages, and blog.

Going to track this in a project: https://github.com/OperationCode/operationcode/projects/2


strengthened test requirements

Signed-off-by: Sam Markowitz<EMAIL_ADDRESS>

Is this what you had in mind @orius123 for #74?

yes, Fixes #74


Caption text encoding

I have problem with charset encoding (caption_text_display param from stories). If someone uses a special character or emoji I received something like this: Ã³w ð. How should I set encoding?

Use utf8_decode().
http://php.net/manual/en/function.utf8-decode.php


SQL Exception CategoryTranslation at Form flush

I have a form for the locale de and a shadow page de_at and de_ch with the shadowLocale de. Saving the form is working for de and de_at, but not for de_ch -> there's following error during flushing:
request.CRITICAL: Uncaught PHP Exception Doctrine\DBAL\Exception\NotNullConstraintViolationException: "An exception occurred while executing 'INSERT INTO ca_category_translations (translation, locale, idCategories) VALUES (?, ?, ?)' with params [null, "de_ch", 4]:  SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'translation' cannot be null" at /var/data/websites/www.candulor.com/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractMySQLDriver.php line 112 {"exception":"[object] (Doctrine\\DBAL\\Exception\\NotNullConstraintViolationException(code: 0): An exception occurred while executing 'INSERT INTO ca_category_translations (translation, locale, idCategories) VALUES (?, ?, ?)' with params [null, \"de_ch\", 4]:\n\nSQLSTATE[23000]: Integrity constraint violation: 1048 Column 'translation' cannot be null at /var/data/websites/www.candulor.com/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractMySQLDriver.php:112, Doctrine\\DBAL\\Driver\\PDOException(code: 23000): SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'translation' cannot be null at /var/data/websites/www.candulor.com/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php:93, PDOException(code: 23000): SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'translation' cannot be null at /var/data/websites/www.candulor.com/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php:91)"} []

Our temporary solution is to clear the entityManager before persisting and flushing my data.
We don't know why it want's to insert a new categoryTranslation but it seems like the error is from Api/Category.php
public function getTranslation()
    {
        foreach ($this->entity->getTranslations() as $translation) {
            if ($translation->getLocale() == $this->locale) {
                return $translation;
            }
        }
        $translation = new CategoryTranslation();
        $translation->setLocale($this->locale);
        $translation->setCategory($this->entity);

        $this->entity->addTranslation($translation);

        return $translation;
    }

If no translation exists, it creates a new one without a title.

hmm the question is why do you have no translation? do you use this outside the API?

No - that's why it's so weird.
We can take a look at this together with #1546

@danrot do you know if we can delete this lines? hmm maybe we have to check everytime if there is a translation!

@wachterjohannes I think there was a reason to add these lines, IIRC. I guess it's not as easy as simply deleting these lines, that would probably break other stuff.


Disable front-facing /back-facing button

I am using this library for taking photos.
Is it possible to disable either the front-facing or back-facing camera to force a particular camera.

As far as I can tell you cannot force front or back facing if both are available. The only control related to those would be which one is the default and what icons are used.

There is this option: defaultToFrontFacing.
I would like to use that option and also have another option which disables the button that allows the customer to change camera. That way they are 'forced' to front-facing camera.
It definitely is a common use case for chat/telephone based apps.

There is this option: defaultToFrontFacing.
I would like to use that option and also have another option which disables the button that allows the customer to change camera. That way they are 'forced' to front-facing camera.
It definitely is a common use case for chat/telephone based apps.

Hey @pjebs: Your explanation of the feature makes sense, and I don't think it would be particularly difficult to add to the API. However, would you mind explaining the chat/telephone app use case a bit more? The chat apps I use don't seem to force a particular orientation so I'm just trying to get a better feel for whether other people would be looking for this feature as well.
Thanks!

I'm actually an ios developer who has just started learning Android in an attempt to port my app.
In iOS, video-chat apps like Facetime and skype only activate the front-facing camera since it doesn't make sense to use the back camera when you are supposed to look at the screen to see the other person and the other person is meant to see you.
For my app, that is a definite requirement but I don't know enough about Android to fork this library and figure out how to make the modifications.
I'm pretty sure it'd be quite easy to implement and I hope I have convinced you of a usecase that justifies it.

Does this library work with API16. I noticed that minSdkVersion in gradle says 14, but does that mean it just won't crash (and/or use a reduced feature set) or will it work as intended on API 16.

Hey @pjebs, does your app need to be streaming the video? If so this library might not be for you, as it returns the video after it is done recording. This library should work back to 14, however lower APIs are tested less frequently as the devices are harder to come by. I usually test the new versions on my Razr Maxx which is on API 17 and it usually works well. I can look in to adding this feature when i have some free time as a branch or part of a release -- but I'm a full time student so I can't guarantee a time frame.

I'm actually just using this library for snaps hitting their face to create profile pic. Thanks @tylerkrupicka

Ah. Sounds good. Actually, @sugarmanz was able to get it working earlier so he will be uploading a branch soon.

https://github.com/afollestad/material-camera/commit/ab60cbe1e86b74b8c5bee9368619bee820a5af1f

Will I need to make changes to gradle dependency to use new commit?

I will have to publish a release to jCenter before you can access it.

@pjebs yeah its not in gradle yet. We'll do some testing on our devices to make sure it doesn't break anything and then merge it in to master. In the meantime if you want to test it you can download the material camera repo, switch to the branch, and run the sample app. Someone will post here when we merge it in.

I've finished up testing on my devices:

Device
Manufacturer
OS
Switch Lock

Nexus 6p
Huewei
7.0
✓

Nexus 5
LG
6.0.1
✓

Droid Razr Maxx
Motorolla
4.4.2
✓

One M8
HTC
5.0.5
✓

No real gotchas, didn't have any problems toggling it for each device. It's worth noting that the API is allowChangeCamera(false); not allowCameraChange as is stated in the commit. @sugarmanz we need to make sure this gets added to the README.
Also, @pjebs turns out my Maxx is actually API 16 and material camera is working well.

Thanks guys for the amazing work!
Will trump winning have any impact on the performance of the library?


Stimulus IE 11 .contain() method doesn't exists

Hello my problem is connected with multiple targets connected with controller.
Here's example which works on every browser - besides our beloved IE 11 (I'm not looking for previous versions) example it's on my subdomain since I'd like to reproduce as much as possible from my original project so webpack build etc
index.ts
import "@stimulus/polyfills";

import { Application, Controller } from "../node_modules/stimulus";

const application = Application.start();

application.register(
  "api-index-animation",
  class extends Controller {
    static targets = [
      "upperRightDot",
      "upperLeftDot",
      "lowerRightDot",
      "lowerLeftDot",
      "centerRightDot",
      "centerLeftDot",
      "centerDot"
    ];
    private upperRightDotTarget;

    connect() {
      console.log(this.upperRightDotTarget);
    }
  }
);

index.html
<!doctype html>
<html>

<head>
    <title>Getting Started</title>
</head>

<body>

    <svg class="api-index-animation"
        data-action="mouseover->api-index-animation#restartAnimationForMouseover mouseleave->api-index-animation#resetMouseoverParameter"
        data-controller="api-index-animation" height="100px" style="margin-bottom: -3.2rem" version="1.1"
        viewBox="0 0 800 700" width="200px" xmlns="http://www.w3.org/2000/svg">
        <circle class="api-index-animation__dot api-index-animation__dot--hidden" cx="400" cy="300" data-start="50"
            data-target="api-index-animation.centerDot" r="55" style="fill: #31e24c"></circle>
        <circle class="api-index-animation__dot api-index-animation__dot--hidden" cx="660" cy="150"
            data-target="api-index-animation.upperRightDot" r="55" style="fill: #3196e2"></circle>
        <circle class="api-index-animation__dot api-index-animation__dot--hidden" cx="800" cy="300"
            data-target="api-index-animation.centerRightDot" r="55" style="fill: #3196e2"></circle>
        <circle class="api-index-animation__dot api-index-animation__dot--hidden" cx="660" cy="450"
            data-target="api-index-animation.lowerRightDot" r="55" style="fill: #3196e2"></circle>
        <circle class="api-index-animation__dot api-index-animation__dot--hidden" cx="140" cy="450"
            data-target="api-index-animation.lowerLeftDot" r="55" style="fill: #3196e2"></circle>
        <circle class="api-index-animation__dot api-index-animation__dot--hidden" cx="0" cy="300"
            data-target="api-index-animation.centerLeftDot" r="55" style="fill: #3196e2"></circle>
        <circle class="api-index-animation__dot api-index-animation__dot--hidden" cx="140" cy="150"
            data-target="api-index-animation.upperLeftDot" r="55" style="fill: #3196e2"></circle>
        <path d="M455 300 L 745 300" data-duration="80" data-start="50" id="cr_line"
            style="stroke: #D4D4D4; stroke-width: 10px; opacity: 0.7; stroke-dasharray: 1000; stroke-dashoffset: 0">
        </path>
        <path d="M345 300 L 55 300" data-duration="80" data-start="50" id="cl_line"
            style="stroke: #D4D4D4; stroke-width: 10px; opacity: 0.7; stroke-dasharray: 1000; stroke-dashoffset: 0">
        </path>
        <path d="M420 250 C 420 250, 420 150, 520 150 M520 150 L 520 150, 610 150" data-duration="70" data-start="50"
            fill="none" id="ur_line"
            style="stroke: #D4D4D4; stroke-width: 10px; opacity: 0.7; stroke-dasharray: 1000; stroke-dashoffset: 0">
        </path>
        <path d="M380 250 C 380 250, 380 150, 280 150 M280 150 L 280 150, 190 150" data-duration="70" data-start="50"
            fill="none" id="ul_line"
            style="stroke: #D4D4D4; stroke-width: 10px; opacity: 0.7; stroke-dasharray: 1000; stroke-dashoffset: 0">
        </path>
        <path d="M380 350 C 380 350, 380 450, 300 450 M300 450 L 300 450, 190 450" data-duration="70" data-start="50"
            fill="none" id="ll_line"
            style="stroke: #D4D4D4; stroke-width: 10px; opacity: 0.7; stroke-dasharray: 1000; stroke-dashoffset: 0">
        </path>
        <path d="M420 350 C 420 350, 420 450, 520 450 M520 450 L 520 450, 610 450" data-duration="70" data-start="50"
            fill="none" id="lr_line"
            style="stroke: #D4D4D4; stroke-width: 10px; opacity: 0.7; stroke-dasharray: 1000; stroke-dashoffset: 0">
        </path>
    </svg>

    <script src="main.js"></script>
</body>

</html>

Error: Object does not support property or method "contains"

Testing -> http://stimulus.szymondziewonski.com/project.zip
If you are not feeling comfortable to download zip
here's files needed to recreate it
dist/index.html (already included)
src/index.ts (already included)
root
webpack.config.js
tsconfig.json
package.json
install packages and run yarn build
Any help would be much appreciated

The project.zip link you provided 404s for me. I'm also unable to reproduce the issue using a bare bones Stimulus application: https://glitch.com/~classic-yellowhorn.

Thanks for quick reply,
Glitch link is indeed working but my example didn't work.
But since you use glitch and I want you to see problem I prepared example for you on glitch.
https://recondite-flare.glitch.me/
This example has additional data-action="" on svg  with mouseover and mouseleave events, svg animation is triggered at start and on mouseover restarted

Aha! My example didn't have any data-action attributes.
Until the next release is out, you can fix this by adding the code from aa76e25f104ed165a8985f9c93e25c7912ed218f to your application: https://github.com/stimulusjs/stimulus/blob/aa76e25f104ed165a8985f9c93e25c7912ed218f/packages/%40stimulus/polyfills/index.js#L12-L16
Thanks for the report!

@javan thank you for quick resolving issue, its working :)


Upgrading to 0.8.0 will break imports from outside of the test's directory

If you are reporting a bug, please fill in below. Otherwise feel free to remove this template entirely.
Can you reproduce the problem with latest npm?
Yes. npm 4.0.3
Description
After upgrading to react-scripts 0.8.0 tests that import files or utilities from outside the current testing directory return 'test-file-stub' as the default (and undefined, of course, for any destructured  imports).
Expected behavior
In the prior version of react-scripts, importing utilities into tests worked as expected. If you had a utils directory in your src and you imported it into a test running in another directory in src, you would have access to to the imported code. For example:
src
├── utils
│   └── range.js
└── parsers
    └── books.js
    └── books.test.js

Where in the test file books.test.js:
import parser from './book';
import range from '../utils/range';

console.log(parser);
console.log(range);

const generateFakeBook = (n) => ({title: `Hello ${n}`, author: 'Who Ever'});

it('should parse the book', () => {
  for(let n in range(10)) {
    expect(
      parser(generateFakeBook(n))
    ).toEqual({
      title: '',
      author: ''
    });
  }
});

The test would run and if you logged out the range function, you would see that it was the function you expected. In the prior release, you could make use of your utility functions in your tests as well as import other items for assisting in your testing.
Actual behavior
If you log out range, you'll see test-file-stub. This happens with any import outside of the  directory with the test.
Environment
Run these commands in the project folder and fill in their results:

npm ls react-scripts (if you haven’t ejected):

<EMAIL_ADDRESS>

node -v:

v7.2.0

npm -v:

4.0.3

Then, specify:

Operating system:
Mac OS X 10.11.6

Browser and version:
Firefox 50

Reproducible Demo
I've created 2 versions to show the behavior in 0.7.0 and 0.8.0 and included them in this repo: https://github.com/adregan/create-react-app-issue

I should note that if you remove this line:
https://github.com/facebookincubator/create-react-app/blob/6e0081363265c055075777af77363a38a0111417/packages/react-scripts/utils/createJestConfig.js#L23
the test will pass.

This appears to be related to #1147

The problem seems to be that the moduleNameMapper regex in Jest is matched before mapping the module name to a file path. So even though the regex added in #1077 correctly mirrors the webpack loader setup we have, Jest only sees ../utils/range and matches that, instead of /absolute/path/src/utils/range.js, which would not be matched because it has the .js extension.
Now it would be great if Jest allowed matching on the full path instead of just the module name, but since it doesn't seem to be possible at the moment, it looks like we need to revert the moduleNameMapper  regex to the previous whitelist. We can't tell if a module is a JS module by just looking at the module name.

I've just published react-scripts 0.8.1 with a hot fix for this issue.
I'll reopen #667 to discuss how to get the same behavior as our webpack loaders have (any unknown file extensions treated as a file resource) back to the Jest configuration.

Thanks for getting the hot fix out so quickly. 👍

Yes, thank you very much! Really appreciated!


Broke rules after optimization

There are two independent files:
.class
{
    border: 2px solid #fc0;
    border-right-width: 0;
}

and
.another-class
{
    border: 2px solid #fc0;
}

The files can used independently and together. And when it together, result is broken rules:
.class{border-right-width:0}
.another-class,.class{border:2px solid #fc0}

First occurs setting of sub-rule border-right-width but then occurs settings of border and it override sub-rule.
It must be as follows:
.another-class,.class{border:2px solid #fc0}
.class{border-right-width:0}

See #619 as well.

This is a bug :+1:

:+1:

The fix is on 3.3 branch so you can give it a try.
If it's all good I'll release it on Tuesday when back from a short break.

Wow, thanks! We are going to try)

It's released as 3.3.7 now.


Doc request: Mention that asconfigc itself is transpiled to JS/Nodes

Hi Josh,
probably it's obvious for some people but I think it is worth to mention in the readme that asconfigc itself is transpiled to JS/Node by using Falcon and also captures packages like jsen and minimist.
So this is a real world example for an awesome working AS3/JS app! ;-)
Although I spend some time with FlexJS it took some time until I noticed that *lol
Thanks,
Olaf

That's a good idea!


[Table] selectable and reorderable columns

if we have more columns than are visible, we will have the table horizontally scroll inside the table container.  Is there an option for selecting fixed columns, if so how does the user select a fixed column versus non-fixed column?  How does an consumer of the table indicate a fixed column versus a non-fixed column?    @bryancboyd we will make a separate issue to horizontal scroll and fixed column

@cgirani please test this function with up to 200 columns and make sure the UX works, @Cynzhang can you please think about this usecase and see if you want to make a UX change based on it? Thanks!

<EMAIL_ADDRESS>

@JordanWSmith15 @stuckless

:tada: This issue has been resolved in version 1.27.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.27.0 :tada:
The release is available on npm package (@release-1.x dist-tag)
Your semantic-release bot :package::rocket:


permissions incorrect for '/mnt/state/config/current/remote.json'

$ /usr/share/admin-scripts/appgate-autoscale.py upscale \
>     ${controller_dns} \
>     --port 8443 \
>     --cacert /tmp/cacert.pem \
>     --username $USERNAME \
>     --site ${site_id} \
>     --share-client-hostname  \
>     --password-path /tmp/password-executable > /tmp/seed.json
Traceback (most recent call last):
  File "/usr/share/admin-scripts/appgate-autoscale.py", line 932, in <module>
    main()
  File "/usr/share/admin-scripts/appgate-autoscale.py", line 922, in main
    action_fn(sys.argv[2:])
  File "/usr/share/admin-scripts/appgate-autoscale.py", line 530, in upscale
    admin = _admin(args, peer_api_version=args.peer_api_version or get_current_version())
  File "/usr/share/admin-scripts/appgate-autoscale.py", line 515, in get_current_version
    with open('/mnt/state/config/current/remote.json') as f:
PermissionError: [Errno 13] Permission denied: '/mnt/state/config/current/remote.json'

$ sudo ls -la /mnt/state/config/current/
total 32
drwx------ 3 root root 4096 May 19 15:53 .
drwxr-xr-x 8 root root 4096 May 19 15:53 ..
drwx------ 4 root root 4096 May 19 15:53 compiled
-rw------- 1 root root 8951 May 19 15:53 local.json
-rw------- 1 root root 4613 May 19 15:53 remote.json

fix:
$ sudo chmod 0777 /mnt/state/config/current
$ sudo chmod 0777 /mnt/state/config/current/remote.json

appgate-autoscale.py assumes its running as root on an appgate sdp instance.
Which is the case when run from a startup script in a cloud instance.
So you shouldn't need to change file permissions.

you're absolutely right about this


Obsolete System.Drawing.Common dependencies

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

28 of 34 (82.35%)  changed or added relevant lines in 3 files are covered.
33 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-2.5%) to 82.306%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/EasyPDF/PageSize.cs
2
8
25.0%

Files with Coverage Reduction
New Missed Lines
%

src/EasyPDF/ClassEasyPDF.cs
4
57.28%

src/EasyPDF/Font.cs
29
53.41%

Totals

Change from base Build<PHONE_NUMBER>:
-2.5%

Covered Lines:
934

Relevant Lines:
1118

💛  - Coveralls


added cbz unpacking functionality when adding cbz files using directoryButton

New functionality works the same as adding .cbz files one by one using "Add file" button,
added 3 necessary functions which make this process possible in comic2ebook.py (listFilesS, listCbzFilesS, countCbzFilesS) and 1 in KCC_gui.py (displayProgressMessage)

https://github.com/ciromattia/kcc/issues/695

Why do they all end with S?
def listFilesS(source):

Also, use this syntax in first post to link PR to issue
- #695

Also, code doesn't run if 7z isn't installed and only using tar.

Isn't 7zip required to work properly?
README

This time I took different approach with adding .cbz files via add directory button and decided .cbz files to appear exactly the same as adding files by drag and drop feature or selecting files after clicking add file button.

Added code of conduct to repo by the way. It's (probably) good practice to have it.

Closing based on discussion in #612, reopen as needed. That feature probably affects how you want to approach this.


After wait_for_frames timeout, get_frame_timestamp gives values between -3 and 4

Sometimes when I start my program, the wait_for_frames function gives a timeout error. I implemented a try...catch handler. In this case I stop the the camera and start it again (dev->stop(); dev->start();). Then the wait_for_frames works, but when I want to give out the timestamps, it gives me values between -3 and 4. The framenumber instead counts regularly.

Required Info

Camera Model
F200

Firmware Version
2.60.00

Operating System & Version
Windows 10

This is rather unusual. If you enable logging and reduce log level (rs::log_to_console) do you see anything suspicious in the log?
F200 camera is passing hardware time of capture using first couple of pixels of the image. (this rather odd method is used in a lack of better alternative before UVC frame metadata) You can usually see these pixels flickering with a naked eye. If they don't, the timestamp will be calculated from whatever pixel values that happened to be in the left-top corner. It is possible this is why you are getting -3 and 4.

Thank you for your advice. I did enable logging and attached the output. In one case the waitForFrames worked from the beginning and the timestamps in this case are right. In the other case, waitForFrames timed out, this was handled with dev->stop(); dev->start(); and the next waitForFrames worked but then the timestamps are wrong.
I would estimate that in 90% of the cases the waitForFrames works from the beginning. In the other 10 one restart is needed and then it works but with the wrong timestamps.
restartNeeded.txt
workedDireclty.txt

In my opinion, if this problem repeats it is a hardware malfunction. It seems both UVC and the raw USB endpoints get stalled. Please consider reaching out to RealSense support.


Strange description of "userPrincipalName" capability

The userPrincipalName restricted capability allows apps to modify and access the thumbnail cache from photos. - this doesn't seem right

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

ID: fdfe599a-af5b-acbc-1405-d1030d9f6661
Version Independent ID: 85857337-8448-af63-b543-f01ef482ea09
Content: App capability declarations - Windows UWP applications
Content Source: windows-apps-src/packaging/app-capability-declarations.md
Product: uwp
Technology: deployment
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Thanks for catching this. The description for this capability has been fixed.


Add support for allocation with given alignment

Currently all allocations are aligned at least to 8 bytes boundaries. For some use cases (e.g. vectorized instructions) it's necessary to align to 16 bytes boundaries. One might also want to align to cache line size boundary for parallel workloads.
To support this we can extend allocate method to take alignment as an additional implicit parameter. Alignment would be just a simple single-field value class holding the alignment value.

Once we have that we can also try aligning data structures by the alignment of their biggest field to reduce padding waste.

Apart from extra parameter for allocate it would also be a good idea to add new @align(n) annotation that can be used to annotate off-heap classes and their fields to have specific alignment.

@align(n) idea deserves a separate issue https://github.com/densh/scala-offheap/issues/89.


Number Theory

Description 📜

Domain of Contribution 📊

[ ] Android Dev (Flutter, Java, Kotlin)
[ ] Backend Dev (Java, .NET, Python)
[ ] C/CPP
[ ] Competitive Programming
[ ] Cyber Security
[ ] DSA
[ ] Database
[ ] Datascience with Python
[ ] Datascience with R
[ ] Frontend Dev HTML/CSS/JS
[ ] Frontend Dev React/Angular/Vue
[ ] Golang
[ ] Interview Prep
[ ] Java_Domain
[ ] JavaScript
[ ] MERN
[ ] Machine Learning
[ ] Open Source
[ ] Python_Domain
[ ] Rust
[ ] Statistics
[ ] UI/UX

/assign

Wait... What is this??? Why is every domain tagged?

@Perpetual-Incantation  @Safwan310
Delete all other domains from your comment.. like mentioned in PR template directives
Kindly follow the issue template from next time ... & preach this to your fellow contributors

@Perpetual-Incantation  Perfect ... Preach this to your peers too :)

Hey, I have edited this before. You can check it out also.

\assign Can I work on this issue because I have been working on Number theory for Competitive programming

@Suvraneel Please review my code.

@Rishabh510 Please review my code.


ISSUE: Unable to connect GMail account in 'Devices' menu

Describe the bug
I installed 'Mail and Packages' through HACS, then I went ahead to 'Devices' and tried to add integration to my Home Assistant. After filling up all 4 fields with correct gmail information, as well as my email address along with app password that I created for my email, it starts loading for 7-10 minutes meanwhile my whole home assistant gets frozen, unable to navigate anywhere except for my Dashboard with old values displayed. After waiting for pretty long time, I'm getting "Invalid flow specified" error on the same panel where I entered all 4 values with credentials etc.
Environment (please complete the following information):

OS: HassOS
Type of device: Windows
Home Assistant version: 10.1
Hassio
Component version: Unknown

Logs
Enabled debug mode, no logs found in home-assistant.log
Screenshots

Additional context
Enabled IMAP in Google account settings. Using Windows machine, under which I'm running VM Virtual box - HassOS. Connection to internet is stable.

Seems like a bad download, uninstall/reinstall from HACS.

same problem been ongoing for a while, author is of no help keeps saying wants log but it does not get to the point of writing a log so he does not help pretty convenient way of getting out of helping


Wasm: clear shadow stack slots in case of call to RhCollect

As part of fixing #8171, its apparent that there is a bug in the call to GC.Collect which prevents some objects from being collected.   This PR fixes this.
What happens is that in RhCollect and System_Runtime_InternalCalls__RhpCollect 3 slots are allocated on the shadow stack but are not all set before the call to RhpCollect.  If these slots have object references in them, from previous calls which used more of the shadow stack, then these references will still be there, and if they are the only references left to live objects, will prevent those objects from being collected.
Not a huge deal in the long term as at some point they would probably have gotten overwritten or collected, but a definitely a problem when testing.

Realised that there may be another case
    private static object TestMethod()
    {
        GC.Collect();
        return null;
    }

This will create a spill slot and as there are no object parameters it could uncover an otherwise dead object so I've put this to WIP to check.

I can't reproduce an error with the code above so will release this as is.


Add a PathPicker

Please Describe The Problem To Be Solved
Please, create a "PathPicker" because the "FilePicker" requires a file obligatory to take the path, or add an option to the "FilePicker" to not use a file to select a path.

You can already do that by using
file_picker =  ft.FilePicker()
file_picker.get_directory_path()

What Flutter package or library does solve that?


HMR with webpack

Description
HMR does not work.
Expected behavior
import React from 'react';
import ReactDOM from 'react-dom';
import App from './components/App';
import './index.css';

ReactDOM.render(
  <App />,
  document.getElementById('root')
);

if (module.hot) {
  module.hot.accept('./components/App', () => {
    ReactDOM.render(
      <App />,
      document.getElementById('root')
    );
  });
}

Hot updates should refresh page.
Actual behavior
Nothing happens.
see https://github.com/webpack/webpack-dev-server/issues/206
https://github.com/facebookincubator/create-react-app/blob/88c15d0988fcb456fc976cbe15823ed518470126/packages/react-scripts/scripts/start.js#L221 uses hot option and https://github.com/facebookincubator/create-react-app/blob/88c15d0988fcb456fc976cbe15823ed518470126/packages/react-scripts/config/webpack.config.dev.js#L206 uses the plugin.
Removing either solved it for me.

Removing either solved it for me.

I don't think it "solved" the problem for you, it just refreshed the page.
Your example doesn't work because it is written for Webpack 2. We still use Webpack 1. Here's the version that works with Webpack 1:
import React from 'react';
import ReactDOM from 'react-dom';
import App from './components/App';
import './index.css';

ReactDOM.render(
  <App />,
  document.getElementById('root')
);

if (module.hot) {
  module.hot.accept('./components/App', () => {
    var NextApp = require('./components/App').default;
    ReactDOM.render(
      <NextApp />,
      document.getElementById('root')
    );
  });
}

Note that this resets internal state of components. Create React App doesn't currently support preserving component state on hot reloading.

@gaearon Thanks for clarifying this. You're right I'm using webpack 2 and yes, the state is lost. I'm currently working on adding react-hot-loader 3 into the mix. The problem was the refresh wasn't working. Thanks!

Even if there's something that's unstable is there any way to support preserving component state on a HMR @gaearon?

As CRA now uses Webpack 2, is there any change to the preferred way of doing this?

The examples you see are still the recommended way to do this.
However, no stable release depends on webpack 2. Please use the webpack 1 example for now. It should continue to work when we switch to 2.


Terminal suggest: Completions on inputs with multiple lines show and hide on -

Testing #210394
Typing - flickers the suggest widget here:

ctrl+space shows it:

Partially fixed, now the suggestions will show on -, but hide after every keypress. Ctrl+space brings it back up correctly.

The problem might be in the filtering/suggest widget side, not prompt input model.

Lots has changed since this, closing as we haven't tackled multi line input much anyway


Will there be a support to work with GitHub and GitHub enterprise instead of Azure DevOps?

If I want to duplicate repo, actions, workflow, projects etc. in GitHub or GitHub enterprise, will I be able to use this tool to do that?

Hi @harrchen  - Let's start an email discussion on this.  It already supports the repos which means much of the configurations and Actions are  already supported.  Projects and Issues and a few other things would still need work.  Ping me via email with your requirements.  Thanks!


ICAP TIMEOUT

i don´t speak english
Hi, first thanks for share it
but i can´t run this, i send a simple text but doesn´t occur until the icap 408 error.

i attach wireshark captures.

Thanks !
Regars ;)

Thanks for posting!
Which version did you try? Java or C#?

Java

I have set up a C-ICAP server with the "echo" service, but I have not been able to replicate the error.
The server could be stuck somewhere. Have you looked at the server logs?
A wild guess might be to insert a line with "HTTP/1.0 200 OK\r\n" before the line with "Content-Length". The server you use, might be expecting a more "complete" HTTP header.
Just so you can see how it looks to me:
C-ICAP client

Java client

these ejecution(JAVA) are using exactly the same code shared in github? or u was  has to modify some lines of code?
And whats mind "User-Agent : IT-Kartellet ICAP Client/1.1"?

I am running the exact code, which is on GitHub right now.
As you can see form the "User-Agent", I ran the C-ICAP client as a test.
The User-Agent is just an identifier to show what client version is connecting.

can you help me with a capture of java ejecution in wireshark?
thanks so much

I'm not sure what you mean? You already have done that in your first post.
I don't know what you want me to send?
I don't see any way to proceed, without you looking at the ICAP server to see if it stops before answering back. Until proven otherwise, that is the only logical conclusion I can draw from the information you have shown.

I change the header and it work but it are not considers the file to scan.

And another curiosity is only work sending REQMOD ,but in the options, this return RESPMOD.
`String clientIP = "<IP_ADDRESS>";
        String httpRequest  =
        		"GET http://"+ clientIP+"/"+"eicar.com.txt"+ " HTTP/1.1\r\n"
        		+"Host: icap.heal.check\r\n\r\n";

        String httpResponse =
        		"HTTP/1.1 200 OK\r\n"
        		+"Transfer-Encoding: chunked\r\n\r\n";

        String icapRequest=
                "REQMOD icap://"+serverIP+":"+port+"/"+icapService+" ICAP/"+VERSION+"\r\n"
                +"Allow: 204\r\n"
                +"Connection: close\r\n"
                +"Host: "+serverIP+"\r\n"
        		+"Encapsulated: req-hdr=0"
        		+ ", res-hdr="+httpRequest.length()
        		+ ", res-body="+(httpRequest.length()+httpResponse.length())+"\r\n\r\n";

        sendString(icapRequest+httpRequest+httpResponse);`

could guide me why the file is not consider ?
Thanks!

I can't really tell if your data is malformed, but something seems wrong with the GET and "200 OK" lines. I think you should only have one of them.
Apart from that, it looks like you have "\r\n\r\n" right after "44" in the Wireshark dump. It should only be "\r\n".

The request should look something like this instead:
REQMOD icap://.../RESPMOD ICAP/1.0
Allow: 204
Connection: close
Host: ...
Encapsulated: ...

GET http://.../eicar.com.txt HTTP/1.1
Host: icap.heal.check
Transfer-Encoding: chunked

44
X5O!P%@AP[4\PZX54(P^)7CC)7}$EICAR-STANDARD-ANTIVIRUS-TEST-FILE!$H+H*
0; ioef

And now don´t recognize the test malware file.

@Baekalfen Thanks so much for your time and help me patiently!
Finally i get a response from my icap server!

have a nice day

Great to hear!
Did you end up changing the code? If so, I would very much like to get a pull request with the improvements.

I will send a pull request,but keep in mind each ICAP server work different apparently.
Best Regards


Update policy enforcement docs with fresh Policy API info

What does this PR do?
Mostly, adds some missing links and a sample GS PolicyException
What does it look like?
(If this is more than a textual change, a screenshot can help in some cases to speed up the review process.)
Any background context you can provide?
This is one of several updates with requested improvements to these docs to support the PSS transition.
What is needed from the reviewers?
Have you maintained the front matter?
Front matter is still current, will bump the review on next update.

CI failure is unrelated to the content change


ReportIncident save button always rendered

🐛 Bug Report
In ReportIncident.test.tsx there is a test implying that the save button should only be rendered after all required inputs are filled out, however the save button is always rendered regardless of input values, so the test will always fail. It may be because the Date of Incident input is prefilled with the current date/time, but in the staging app if you empty that input the page crashes.
https://github.com/HospitalRun/hospitalrun-frontend/pull/2516#discussion_r557076908
To Reproduce
Uncomment the test and run npm t
Expected behavior
Test passes

Thanks for this @codyarose

In ReportIncident.test.tsx there is a test implying that the save button should only be rendered after all required inputs are filled out, however the save button is always rendered regardless of input values, so the test will always fail.

Agreed, the test will always fail. I am not sure if that test is valid since the 'save' buttons in all forms are currently always rendered regardless of whether or not the fields are filled in
@jackcmeyer thoughts?

in the staging app if you empty that input the page crashes

This is a bug that needs addressing, would you like to take a shot at it @codyarose ?

Thanks for this @codyarose

In ReportIncident.test.tsx there is a test implying that the save button should only be rendered after all required inputs are filled out, however the save button is always rendered regardless of input values, so the test will always fail.

Agreed, the test will always fail. I am not sure if that test is valid since the 'save' buttons in all forms are currently always rendered regardless of whether or not the fields are filled in
@jackcmeyer thoughts?

in the staging app if you empty that input the page crashes

This is a bug that needs addressing, would you like to take a shot at it @codyarose ?

@blestab I would like to take this up

@Adarsh710 sure

@tehKapa If the input is empty and has focus then the user can enter a date and if not, then previous date will appear, is this a good flow?

@tehKapa If the input is empty and has focus then the user can enter a date and if not, then previous date will appear, is this a good flow?

If initial value was empty, how do you want to populate it? With the current date?

@tehKapa If the input is empty and has focus then the user can enter a date and if not, then previous date will appear, is this a good flow?

If initial value was empty, how do you want to populate it? With the current date?

Yes, btw it is already populated with current date on initial render

So let's fallback on that

@blestab, @tehKapa please assign reviewers to the above PR because I'm not able to do that


Feature/decl impl

@pbrunet : i'll finish the rebase, there a few conflicts

looks good to travis!

Can I just merge it? (they are my commits so ....)

On Sun, Apr 26, 2015 at 07:13:56AM -0700, pbrunet wrote:

Can I just merge it? (they are my commits so ....)

yes :-)

Changes Unknown when pulling 01a2686c5bb56c1e7d8d903174d51d62567ae421 on feature/decl-impl into ** on master**.


[FIZZ-201] Publishing Google Classroom on pypi

Many files had to change for this pull request, primarily because of name changes.

In order to push the GCP extractor to PyPi, the shared library needed to be pushed to PyPi first.
In doing so, the shared library name was enhanced for clarity, the directory was moved.
Google Classroom extractor's packages names were also changed, introducing a prefix of "edfi_".
In both cases, twine was installed as a dev dependency for publishing. It is preferable to using poetry or pip because it communicates over https instead of http.
Version numbers were bumped to 1.0.0-alpha0 (or as 1.0.0a0 as represented in PyPi), using the poetry version premajor command.
To publish, you must have a username and password on PyPi, and the ed-fi-alliance account needs to run the initial publishing step. Once that occurs, that account can grant other accounts "collaborator" access if needed. The best way to provide those credentials to twine is through environment variables TWINE_USERNAME and TWINE_PASSWORD.
Publish using build.py:python .\eng\build.py publish google-classroom-extractor

One near-term concern is the package setup for the shared library. It's probably not going to work long-term to need to publish changes to pypi during development just to pick up a file edit to the shared library.

So far I'm not seeing any good alternatives other than manually copying the shared library into the virtual environment (temporarily).

So far I'm not seeing any good alternatives other than manually copying the shared library into the virtual environment (temporarily).

This gives me an interesting idea.  What if you were to create a symlink in the virtual enviroment back to the package...


Sean/codegen patterns experiment

This is really experimental and rough. I tried adapting the pattern machinery from the constraint solver to generate code. (This was generally an experiment on bundling codegen with rules/patterns; it was easier to start generating type declarations than functions.) I think there are some good ideas in here, but it definitely needs to be substantially reworked.

@bladyjoker
I think this might be ready for you to take a look at. Bear in mind that it's still very first-draft-ey and the details of everything can change. Having said that, I think this might be a good idea once refined.
At the moment the Gen stuff is capable (at a very basic / primitive level) of generating Rust type definitions from patterns. It's a bit ugly, and there's a lot of stuff it doesn't support, but it works for basic structs and enums containing primitive types, and can translate (something like) the Source type you drafted a while ago into a pattern form.
Common.Types contains the ADT for patterns (and a bunch of pattern synonyms) + some utils and translation functions. Start there.
Resolve.Rules contains the ADTs to represent instances and classes, and Resolve.Solver contains the basic constraint solver.
Common.Match contains the pattern matcher (technically it checks whether the second arg is a substitution instance of the first). This is used by both the codegen and the constraint solver.
Gen.Generator contains the basic type (Gen) and functions for code generation. It's extremely simple. Like, it'll take you 5 minutes to understand it tops.
Gen.Rust contains the primitive rust code generators. This is probably what you want to spend most of the time on b/c it'll give you an idea of whether you think my approach is worth using. You can try it out on the test types in Common.SourceTy by using genFromDefAndPrint from the Gen module.
Notes (for both of us, I guess):

It doesn't generate typeclasses (well traits I guess) yet but I'm pretty sure I can just bundle generators with instance rules and make it do that fairly easily. A Map Instance Gen and when you solve a constraint you run the generator for it. (This is how we can just derive every intermediate instance for every component type.)

We need to figure out imports. We should actually be capable of gathering all the necessary imports before we start generating code, I think? But we don't seem to have any place for users to define the package in which a foreign module  is located, so we can't actually generate any kind of language-specific package manager stuff until we do that.

I need to generate some kind of pretty printer library's Doc type instead of Text. As much as I hate them, they're necessary here; it's a bit too difficult to make things look as pretty as they should with just Text output.

Almost all of this discussion is outdated and significant portions of this have been split out. Closing in favor of #23 / #24 / #8 to refocus discussion and attention.


Move fa-transV to the new perf-kernels dir

This PR creates a dir named perf-kernels and we'll put customized/performant kernels there.
As the first step, the tranV version of FA is moved there.

Should we use the name rocm-perf-kernels for the folder?

It's expected to live in our fork anyway.
@jayfurmanek any thoughts on the dir name?

It's expected to live in our fork anyway. @jayfurmanek any thoughts on the dir name?

It sounds good to me.


Create 805. SplitArrayWithSameAverage.cpp

Added code for problem 805 in C++
Issue no.(must) - #560
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[x ] This issue was assigned to me if not it will be marked as spam.
[x ] My file is in the proper folder
[ x] I am following clean code and Documentation.
[ x] I have added the title and what program will do first in the file
[ x] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

Pls check the issue.


Fix -Wrestrict in vtkFSSurfaceAnnotationReader

/tmp/SlicerFreeSurfer/FreeSurfer/vtkFSSurfaceAnnotationReader.cxx: In member function ‘int vtkFSSurfaceAnnotationReader::ReadFSAnnotation()’:
/tmp/SlicerFreeSurfer/FreeSurfer/vtkFSSurfaceAnnotationReader.cxx:353:26: warning: passing argument 1 to restrict-qualified parameter aliases with argument 3 [-Wrestrict]
  353 |           sprintf (this->NamesList, "%s%3d {%s} ", this->NamesList,
      |                    ~~~~~~^~~~~~~~~

If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.

Source: https://stackoverflow.com/a/1973595/1539918
And answer from ChatGPT:

This warning suggests that there is a potential issue with the use of the restrict keyword in the function argument, and it is causing an aliasing problem.
The restrict keyword is a C99 feature that tells the compiler that a particular pointer argument is not aliased by any other pointer in the same scope. In other words, if you use the restrict keyword, you're telling the compiler that the pointer passed as an argument does not point to the same memory location as any other pointer passed to the same function.
In your case, passing this->NamesList as the first argument to sprintf with %s format specifier is causing the warning. The %s format specifier expects a null-terminated string, and since the sprintf function can write to the memory location pointed to by this->NamesList, it is possible that it can also write to memory locations pointed to by other pointers. This violates the restrict keyword, which is meant to prevent such aliasing.
To fix this warning, you should consider using a different approach to write to this->NamesList. You can either remove the restrict keyword from the function argument, or you can use a temporary buffer to store the formatted string and then copy it to this->NamesList using strcpy or strncpy. Here is an example of the second approach:
char tempBuffer[256]; // temporary buffer to store the formatted string
sprintf(tempBuffer, "%s%3d {%s} ", this->NamesList, someNumber, someString);
strncpy(this->NamesList, tempBuffer, sizeof(this->NamesList));
this->NamesList[sizeof(this->NamesList)-1] = '\0'; // make sure the string is null-terminated

In this example, we use sprintf to format the string into tempBuffer, and then we use strncpy to copy the formatted string to this->NamesList. We also make sure that the string is null-terminated by setting the last character to '\0'. This approach avoids the aliasing issue and should fix the warning.

This freesurfer file reader is all very old, low quality code. We just moved it out of Slicer core but did not review or rework it. If we touch it then we should modernize it a bit to use STL strings and streams.


Typescript Error when running npm run build out of the box

Steps followed:

Cloned from master branch
run npm install
run npm run build

The build fails with the following error:
><EMAIL_ADDRESS>build C:\Users\caabreut.REDMOND\Desktop\ApplicationInsights-JS-Mezzurite
> npm run clean && tsc --p tsconfig.json && rollup -c && typedoc --out Doc

><EMAIL_ADDRESS>clean C:\Users\caabreut.REDMOND\Desktop\ApplicationInsights-JS-Mezzurite
> shx rm -rf _bundles dist dist-esm

src/Mezzurite.ts:114:7 - error TS2322: Type '{ name: string; url: string; }' is not assignable to type 'IPageViewPerformanceTelemetry'.
  Object literal may only specify known properties, and 'url' does not exist in type 'IPageViewPerformanceTelemetry'.

114       url:  url
          ~~~~~~~~~

Found 1 error.

npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>build: `npm run clean && tsc --p tsconfig.json && rollup -c && typedoc --out Doc`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\caabreut.REDMOND\AppData\Roaming\npm-cache\_logs\2019-05-29T17_41_27_124Z-debug.log

I am unsure if this was caused by the revving of the AI SDK version or some other previous commit. Nonetheless, a broken build out of the box is not acceptable and we should have CICD in place to have caught it.

This is referring to the current 1.x implementation


Simplify Raiden Smart Contract version bump

Problem
Right now every new version of the Raiden Smart Contracts require and update of the requirements.txt file as well as writing a new class in raiden-installer/tokens.py.
Todo

Find a way to "simplify" the handling of new smart contract versions.

How to update the raiden smart contracts right now
There are several things that have to be done manually atm to update to a newer version of the raiden smart contracts. (This is necessary because the raiden client, the raiden service bundle and the raiden smart contracts have to be compatible).

Update the raiden version in the requirements and reinstall the requirements afterwards.
Add a new Token class like this one
--> Note: The "RDN" Token is called Service Token (SVT) on all testnets!
Add new Token class to the "list of tokens"


Web console: fix lookup edit dialog, allow column renaming

This PR fixes an issue introduced in https://github.com/apache/druid/pull/10360/files#diff-bf9dd2e02fb9a363490ddc495577810dR60 where the lookup edit dialog does not function correctly.
The fundamental issue is that the types here https://github.com/apache/druid/compare/master...implydata:edit-fix?expand=1#diff-ba8558b9ae27105139ba0090510c5395R79 were too loose
This PR also adds a convenient way to rename output columns in a SQL query output.

(first) CI failure was legit... empty scss blocks are not allowed by the linter. TY for the review!


callPackage_i686 is counter-intuitive with parameter overrides

When one uses
foopkg = callPackage_i686 ../foopkg {
  barpkg = barpkg_baz;
}

the barpkg is a 64-bit version. Currently I don't see how to improve it.

barpkg_baz is resolved in the package set that contains callPackage_i686, not in the one that callPackage_i686 constructs to resolve implicit arguments. It's not easy to remedy that without change the functions signature, IMHO.

Yes, I thought there might be some different way to have 32-bit overrides.

The function could pass the 32-bit package set as an argument to another function that returns the override -- like ghcWithPackages does.

(triage) status?

(triage) status?

I wouldn't expect any changes.  Let me close it.  It would be rather hard to remedy and the improvement wouldn't be that much useful anyway.


Taxonomy picker for enterprise

Category

[x ] Enhancement
[ ] Bug
[ ] Question

Version
Please specify what version of the library you are using: [1.6.0]
Expected / Desired Behavior / Question
When opening the term picker list, today the complete tree is expanded.
This does not work for comprehensive trees of taxonomy. We intend to use it for our organisation tree with about 3700 nodes in the tree. This loads extremely slow and picking the right term is a nightmare with a huge vertical scrollbar and bad performing scrolling (especially in IE).
Enhancement would be:

Having the option to expand and collapse the nodes (not leaf nodes) in the taxonomy tree (just as in good old SharePoint ;). The tree should be collapsed as default and only the immediate children of the root should be visible.
Asynchronous load of subtrees. The subtrees should not be loaded until the user actually expand the node to avoid pre-loading of extreme trees. The best would be that the level shown is loaded +1 to maximize usability (e.g. load is always one level "ahead" of user).

Use case: We have our (quite comprehensive) organisation tree synchronized as a termset (manually as the flat list provided by SharePoint in predefined termset "Department" is useless when it is flat).  On pages we have developed (looking forward to the announced "Page details") a property pane where we use this control to pick an organisation unit, the page belongs to. The we use that property to:

Roll-up content (based on search) from specific organisation (sub)trees
Roll-up targeted content to user based on their profile data

Steps to Reproduce
Create a huge termset and use it in this control, then you will quickly see the bad usability of working with the control.

Hi @kristianbkjaer this is good feedback this will require to rewrite the whole component. A good thing is that the PnP team recently released the @pnp/sp-taxonomy for the PnPJS library https://pnp.github.io/pnpjs/sp-taxonomy/index.html. This allows us to make Managed Metadata calls a lot easier and we could use it in our components.

Sounds great. I really hope for this feature in an upcoming release, but understand that it might be good to wait for the api update. I guess asynchronous load of subtrees become easier then?
Keep crossing my fingers :)

@kristianbkjaer no API update, it is a new part of the PnP JS Core library.

Yes, that was also what I meant (but did not write ;)) But the rest of my comment makes sense right?

Yes indeed @kristianbkjaer (y)

Any news on this? Is it assigned to some roadmap or being worked on?
cross my fingers here :)

At the moment it's not assigned to anyone.


Fail to Scrap data

Hi I would like to scrap twitter data based on keyword but someow on my CLI

Based on the error, you need to install Golang to have it running
Please also consider that this tool is not working anymore, #12


Code about fsod  ？？？

I found that there is a lot of code content in your repository that is not related to fsod's paper, or even more like rewritten detectron. Importantly, there are no detailed steps on how to perform fosd's few shot detection using the code in it. For example, when training fsod, parameter configuration, backbone network selection, evaluation method, input and output selection and so on. How are these selected and implemented? It looks more like a repository of how to use detectron to complete faster rcnn, mask rcnn, keypoint detection on the coco dataset. I suggest that you should organize a pure set of fsod code, rather than other existing facts.

Thank you for your suggestion. I will reorganize it.

The Readme is updated and useless configs are removed.

Does it currently support other no-few shot detection datasets (coco,voc), or can be used for mask rcnn, keypoint and other few shot methods?

It is shown in above fig, The model is Mask rcnn， Not faster rcnn，which is different from your paper.
And other dataset such as coco , voc can be used in your repo ?


[Question]What is the difference between preconfer and preconferandproposer status

https://github.com/NethermindEth/Taiko-Preconf-AVS/blob/ec553f79333a3a22494a596c4f0d2e11cad6da82/Node/src/ethereum_l1/execution_layer.rs#L265
Why do the first three transactions need to be sent to the execution layer's memory pool? It seems that on the fourth time, all four transactions will be included in the block through mev-boost's force inclusion.

We send transactions to the memory pool so that they can be included earlier. At the fourth time (the last L2 slot, 3s each), we force all transactions that have not been included up to that point to be included.

Please check: https://github.com/NethermindEth/Taiko-Preconf-AVS/blob/master/Docs/design-doc.md


Hash partition keys for transaction metadata

This distributes the load evenly across Cassandra machines when dealing with transaction metadata.

https://ocd.appscale.com:8080/job/Daily Build/2454/
Migration: https://ocd.appscale.com:8080/job/Migration/832/
Euca-Faststart: https://ocd.appscale.com:8080/job/Euca-Faststart/509/

@cdonati do you see any possible upgrade issue that our migration test may not be able to catch?

@obino No, the only schema change involves ephemeral data, so there's no potential data loss. There's just a new table that will be created upon an 'up'.


[iOS] Video Recording Max Time

I think it would be nice if we could limit the video recording max time

PR are the best way to offers a new feature ! :)

option durationLimit have been added with video picking in Android
Version 0.17.0
Not available on iOS for the moment

@yfuks awesome thanks :)
I'll see if I can maybe take a shot at submitting a PR for this.


chore(main): release 0.9.1

:robot: I have created a release beep boop
0.9.1 (2024-11-16)
Bug Fixes

Fix path for symlinked tools in .vscode (816a2a3)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.9.1
:sunflower:


Improve script around use of @-sign and rerun formatter

There seems to have been a misunderstanding of why we use the "@" sign.
We only need it when we run the tests, not when we create the file.
@iachettifederico I think this will need a little bit of re-shooting.

@iachettifederico we can merge this, right?


React group project: set up your Kanban board

React group project: set up your Kanban board
Hi good reviewer,

These This is the setup of the kanban board for the Space-Travelers project (3 people group):
Project requirements

link to the project.
There are three people in our team.

Hi team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

great job following the instructions.

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


[Python][C++] Casting a list of struct array with null field results in invalid result

Describe the bug, including details regarding any error messages, version, and platform.
OS: MacOS Sonoma 14.6.1
Python: 3.10.14
pyarrow: 17.0.0
Snippet to reproduce:
import pyarrow as pa

fields = [('b', pa.null()),('g', pa.string())]
dtype = pa.list_(pa.struct(fields))
arr  = pa.array([[{'b': None, 'g': None}, {'b': None, 'g': 'moo'}]], type=dtype)
carr = pa.chunked_array(arr)
ext_arr = carr.cast(dtype, safe=True)
print(ext_arr) # prints Invalid array but no exception
print(len(ext_arr)) # Len is 1
print(ext_arr[0]) # Raises ArrowIndexError

Component(s)
Python

@Angel-212 thanks for the report!
The issue already happens with the plain (non-chunked) array as well. And the repr also shows it being invalid (which is then the reason that a getitem operation errors):
In [24]: arr.cast(dtype)
Out[24]:
<pyarrow.lib.ListArray object at 0x7f0536bbd480>
<Invalid array: List child array invalid: Invalid: Struct child array #0 has length smaller than expected for struct array (1 < 2)>


Detailed description for recursion in Fibonacci

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hey, could you please reopen this PR?

Sorry to trouble you, but I tried resolving conflicts and I am still getting "Changes requested". Did I go wrong somewhere?

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


Try to fix the passing variable problem

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

@dc7290
I tried the solution I mentioned in #16 (using customized includer function).
And things seem to work without problem(I am not very sure if this will cause any side effect or problem, 'cause I still don't have time for a full-test.)
but THIS did work:

result:

And bc  it is gonna be a big change(if you think this solution is ok), I guess it should be checked carefully.
fix #16
Type of change

Please delete options that are not relevant.

[V ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Bug fix I guess.
How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

I forked this repo, change index.ts then run build, then copy paste all the built library to the node_modules in my testing repo.
Checklist:

[V] My code follows the style guidelines of this project
[V] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[V] My changes generate no new warnings

@mizok
#16 can include, but updating the file will not be reflected.
Is that correct?

@dc7290  I guess you missed this?
devServer: {
watchFiles: ['*.html', 'src/template/*.html', '*.ejs', 'src/template/*.ejs','src/template/**/*.ejs'],  // this is important

},
You will have to watch all your file if that is going to be dependency

@mizok Did I understand #16 correctly that you can include ejs files, but they are not recognized as webpack dependencies? If so, does this solution update the webpack dependencies?

You can check my change , I thought I did add
this.addDependency(parsedPath);

I guess it's line 93 or some what :/

@mizok

If so, does this solution update the webpack dependencies?

Yap ,it does.

@mizok
I'm impressed!
The idea of using customIncluder as a way to add dependencies to webpack was an eye opener.
As for the code formatting, I would like to see a slight modification, so please just do that.

@mizok
Try running the following command again.
yarn

Then husky will be installed and will automatically lint and format the file before committing it.

@mizok Try running the following command again.
yarn

Then husky will be installed and will automatically lint and format the file before committing it.

ok sure :D

@mizok Try running the following command again.
yarn

Then husky will be installed and will automatically lint and format the file before committing it.

Cool, I have

@dc7290
OK, I guess the lint things are done already, just check it.

@mizok
Nice!


fix: Dot-containing paths do not correctly receive extension completion.

fixes: https://github.com/honojs/hono/pull/2236
Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

This partially resolves this issue, and allows the "." except for the extension completion accuracy.
https://github.com/honojs/hono/issues/2239

@usualoma
Can I have it reviewed?

@watany-dev Thank you!
First, in response to your comment in #2239, I think your point below is spot on.

For those who want to use extensions like .html.html, they can simply specify it explicitly in the Route :)

We can write as follows
app.get(
  '/shops/:id.html',.
  async (c) => { ... }
)

Thus, I think the question I raised in #2239 will be answered by this PR.
I have no objection to this PR test case, but I think .endsWith() would be sufficient for getPathWithExpectedExtension(). No?
const generateFilePath = (routePath: string, outDir: string, mimeType: string) => {
  const extension = determineExtension(mimeType)

  if (routePath.endsWith(`.${extension}`)) {
    return joinPaths(outDir, routePath)
  }

  if (routePath === '/') {
    return joinPaths(outDir, `index.${extension}`)
  }
  if (routePath.endsWith('/')) {
    return joinPaths(outDir, routePath, `index.${extension}`)
  }
  return joinPaths(outDir, `${routePath}.${extension}`)
}

Thanks for reviewing this!
I didn't know there was an endWith in the standard API. This one is great.
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/String/endsWith

Oh, sorry, not quite right, the following doesn't work good. It would be a variable id.html
app.get(
  '/shops/:id.html',.
  async (c) => { ... }
)

Well, okay, that's a separate issue, so let's consider it in other issue 🙇

I have no objection to this PR test case, but I think .endsWith() would be sufficient for getPathWithExpectedExtension(). No?
This has been fixed.


Might be user error...

Thanks for all the work on this. I'm trying to get my pico w to blink via tinygo.
However,  going into the blinky directory and running
tinygo flash -target=pico main.go
followed by tinygo monitor gives me the following output

Connected to /dev/cu.usbmodem142401. Press Ctrl-C to exit.
panic: runtime error at 0x10006709: nil pointer dereference

I suspect user error, and was hoping for some insight in how to make the blink work.

Does this happen with the DHCP example too?

I haven't tried it with that. I will try it tomorrow and report

I get this output:
tinygo flash -monitor -target=pico ./examples/dhcp/. Connected to /dev/cu.usbmodem141401. Press Ctrl-C to exit. starting program INFO Init:start slog.Level=DEBUG DEBUG read back bus ctl got=177 DEBUG current bus ctl00010030writing:000300b1 got:feedbead DEBUG flashing firmware chip_id=43439 fwlen=230321 DEBUG bp_write addr=0 last16=442030312d39353536366436612d30f0 DEBUG bp_write:done status=no status DEBUG flashing nvram DEBUG bp_write addr=523540 last16=006274635f6d6f64653d31000000636f DEBUG bp_write:done status=no status DEBUG core up CY43 hndarm_armr addr: 0x18003000, cr4_idx: 0 CY43 000000.001  CY43 RTE (SDIO-CDC) 7.95.62 (b03806e CY) on BCM43439 r5 @ 37.4/81.6/81.6MHz CY43 000000.003 sdpcmdcdc0: Broadcom SDPCMD CDC driver CY43 000000.008 reclaim section 0: Returned 46156 bytes to the heap CY43 000000.012 wlc_bmac_info_init: host_enab 1 CY43 000000.037 wl0: wlc_ampdu_tx_set: AGG Mode = MAC+Ucode txmaxpkts 64 txmaxpkts_agg 0 CY43 000000.048 wl0: wlc_channels_commit: no valid channel for "#n" nbands 1 bandlocked 0 CY43 000000.053 wl0: Broadcom BCM43439 802.11 Wireless Controller 7.95.62 (b03806e CY) CY43 000000.055 TCAM: 256 used: 77 exceed:0 CY43 000000.058 reclaim section 1: Returned 86104 bytes to the heap DEBUG base init done DEBUG initControl clm_len=4752 DEBUG sendIoctl kind=2 cmd=SET_VAR len=1044 DEBUG rx len=256 hdr=asyncev D rx ?256 hdr=asyncev DEBUG sendn?@uC?C!xv

I've run into this issue many times since, it's always because I forget to add the -stack-size flag to the compiler. This then results in a runtime stack overflow. Use 8kB of stack size like so:
tinygo flash -target=pico -stack-size=8kb main.go

Success! Thanks very much! Clearly I ran into the same issue.


refactor: [M3-6316, M3-6317] MUI v5 Migration - Components > DateTimeDisplay, DebouncedSearchTextField

Description 📝
What does this PR do?

Ran the tss-react codemod on DebouncedSearchTextField, but then replaced with styled since it was such a small change
No styles to run codemod on DateTimeDisplay
Modernized both components by using named exports and better prop interface names

Preview 📷
DebouncedSearchTextField:

DateTimeDisplay:

How to test 🧪
What are the steps to reproduce the issue or verify the changes?

Check that there have been no changes visually or in functionality for the DebouncedSearchTextFields throughout the app

e.g. http://localhost:3000/stackscripts/community

Check that there have been no visual changes for the DateTimeDisplay throughout the app

e.g. http://localhost:3000/events

Check that Storybook is still rendering correctly: http://localhost:6006/?path=/docs/elements-typography-date-time-display--docs

How do I run relevant unit or e2e tests?
yarn test DateTimeDisplay

@bnussman-akamai Thanks - did that clean up in c058760.


CVE-2022-42966: exponential ReDoS

Hi,
A vulnerability was published for cleo in [0] and exposed in Debian [1][2].
| An exponential ReDoS (Regular Expression Denial of Service) can be
| triggered in the cleo PyPI package, when an attacker is able to supply
| arbitrary input to the Table.set_rows method
I didn't check it yet, but I'm going to do it.
[0] https://research.jfrog.com/vulnerabilities/cleo-redos-xray-257186/
[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1024018
[2] https://security-tracker.debian.org/tracker/CVE-2022-42966

Seems that the issue is in v1.0.0* versions.


Add UI to activate login with eventyay ticket

This PR closes/references issue #XX. It does so by:

Add SSO form under organizer config.

How has this been tested?

Checklist

[x] I have added tests to cover my changes.

Summary by Sourcery
This pull request introduces a new SSO feature allowing users to log in with Eventyay tickets. It includes a new form in the organizer configuration, enhancements to the OAuth2 adapter and provider classes, and updates to the login templates to support this functionality.

New Features:

Added a Single Sign-On (SSO) form under the organizer configuration to enable login with Eventyay tickets.

Enhancements:

Enhanced the EventyayTicketOAuth2Adapter to dynamically set the provider ID based on the session or default to 'eventyay'.
Updated the CustomSocialAccountAdapter to support subproviders by looking up by provider ID.
Modified the EventyayProvider to initialize with the appropriate SocialApp based on the event or session data.

Tests:

Added tests to cover the new SSO functionality.

There's a SSO client config form in organizer setting, if this key is filled, 'Login with Eventyay-ticker' will be shown in login page.

@sourcery-ai review

Hi @hongquan, could you help me to review again, some comment can not be fixed because I customize the lib allauth for our service, using same socialapp for app organization

As part of this issue on the login screen we also need to change the following text.

You do not need an account to view the event, or submit feedback, or receive schedule updates. You’ll only need an account if you participate in the event as speaker or as an organiser.

Change to:
If you already created a proposal for a different event on this server, you can re-use your account to log in for this event.
You do not need an account to view the event, submit feedback, or receive schedule updates. An account is only necessary if you want to create your own personalized schedule, or if you are participating in the event as a speaker or organizer.

As part of this issue on the login screen we also need to change the following text.

You do not need an account to view the event, or submit feedback, or receive schedule updates. You’ll only need an account if you participate in the event as speaker or as an organiser.

Change to: If you already created a proposal for a different event on this server, you can re-use your account to log in for this event. You do not need an account to view the event, submit feedback, or receive schedule updates. An account is only necessary if you want to create your own personalized schedule, or if you are participating in the event as a speaker or organizer.

Hi @mariobehling, message is changed


Update jrobot-remote-server feature

remove embeding of nonbundle dependencies
update feature bundle dependencies

Signed-off-by: Martin Mihálek<EMAIL_ADDRESS>

Coverage remained the same at 83.972% when pulling b5d7257b9368c16b70982ee89b1e90d69fc6e957 on feature/dependency-cleanup into 1357fb1733cf5aed4ddb7c675b10944d028334e4 on master.


refactor: monorepo

Current dependencies on/for this PR:

main

PR #1   👈

This comment was auto-generated by Graphite.


Infrequent .NET Error on load of <IP_ADDRESS>...

Unhandled exception has occured in your application. If you click continue, the application will ifgnore this error and attempt to continue. If you click Quit, the application will close immediately.
Access to the path C:\WINDOWS\system32\Favourites.xml is denied.
-------- Details ----------
See the end of this message for details on invoking just-in-time (JIT) debugging instead of this dialog box.
************** Exception Text ************** System.UnauthorizedAccessException: Access to the path 'C:\WINDOWS\system32\Favourites.xml' is denied. at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath) at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost) at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy) at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, Boolean useAsync) at System.Xml.XmlWriterSettings.CreateWriter(String outputFileName) at System.Xml.XmlWriter.Create(String outputFileName, XmlWriterSettings settings) at Reddit_Wallpaper_Changer.RWC.CreateXML() at Reddit_Wallpaper_Changer.RWC.RWC_Load(Object sender, EventArgs e) at System.Windows.Forms.Form.OnLoad(EventArgs e) at System.Windows.Forms.Form.OnCreateControl() at System.Windows.Forms.Control.CreateControl(Boolean fIgnoreVisible) at System.Windows.Forms.Control.CreateControl() at System.Windows.Forms.Control.WmShowWindow(Message& m) at System.Windows.Forms.Control.WndProc(Message& m) at System.Windows.Forms.ScrollableControl.WndProc(Message& m) at System.Windows.Forms.Form.WmShowWindow(Message& m) at System.Windows.Forms.Form.WndProc(Message& m) at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m) at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m) at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
************** Loaded Assemblies ************** mscorlib Assembly Version: <IP_ADDRESS> Win32 Version: 4.6.1586.0 built by: NETFXREL2
CodeBase: file:///C:/Windows/Microsoft.NET/Framework/v4.0.30319/mscorlib.dll
Reddit Wallpaper Changer Assembly Version: <IP_ADDRESS> Win32 Version: <IP_ADDRESS>
CodeBase: file:///C:/Program%20Files%20(x86)/Reddit%20Wallpaper%20Changer/Reddit%20Wallpaper%20Changer.exe
System.Windows.Forms Assembly Version: <IP_ADDRESS> Win32 Version: 4.6.1586.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GACMSIL/System.Windows.Forms/v4.0_<IP_ADDRESS>_b77a5c561934e089/System.Windows.Forms.dll
System Assembly Version: <IP_ADDRESS> Win32 Version: 4.6.1586.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GACMSIL/System/v4.0_<IP_ADDRESS>_b77a5c561934e089/System.dll
System.Drawing Assembly Version: <IP_ADDRESS> Win32 Version: 4.6.1586.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GACMSIL/System.Drawing/v4.0_<IP_ADDRESS>_b03f5f7f11d50a3a/System.Drawing.dll
System.Configuration Assembly Version: <IP_ADDRESS> Win32 Version: 4.6.1586.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GACMSIL/System.Configuration/v4.0_<IP_ADDRESS>_b03f5f7f11d50a3a/System.Configuration.dll
System.Core Assembly Version: <IP_ADDRESS> Win32 Version: 4.6.1586.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GACMSIL/System.Core/v4.0_<IP_ADDRESS>_b77a5c561934e089/System.Core.dll
System.Xml Assembly Version: <IP_ADDRESS> Win32 Version: 4.6.1586.0 built by: NETFXREL2
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GACMSIL/System.Xml/v4.0_<IP_ADDRESS>_b77a5c561934e089/System.Xml.dll
************** JIT Debugging ************** To enable just-in-time (JIT) debugging, the .config file for this application or computer (machine.config) must have the jitDebugging value set in the system.windows.forms section. The application must also be compiled with debugging enabled.
For example:
 <system.windows.forms jitDebugging="true" />
When JIT debugging is enabled, any unhandled exception will be sent to the JIT debugger registered on the computer rather than be handled by this dialog box.

Modified code to specifically reference Favorites & Blacklist XML files to App directory. Published <IP_ADDRESS>


Native promises, server URL in dashboard, more config options, and tests

Hey there DasKeyboard.  I've made significant updates to the Server Ping applet in the past couple weeks.  I made a PR to the parent repo and it was approved/merged, however there seems to be no movement toward merging the most-recent changes into the marketplace repo.  Therefore I'd like to propose a merge from the stable master branch of the parent repo into the marketplace repo, as the parent repo contains the most-recent stable version of the Server Ping applet.

I should also add, that @chbakouras has requested a merge here in Issue #14 on the q-marketplace repo but it hasn't gotten any attention which is why I'm submitting this PR here in the first place

@Mubramaj Since you were the one who merged my last update on the ICMP applet, I wanted to call you out here and ask, might you guys be able to take a look at this one?  I know you guys are busy over there making daskeyboard awesome, but it would be fantastic to see this update be released into the marketplace sooner than later.

Hi @Conroman16 I am sorry for the delay. Looking into it right now

Hey apologies, hadn't seen that you got back to me there.  Anyways, I made the changes and put in this PR for @chbakouras to merge and then we should be good

@Conroman16 PR is merged, so we are good to go! @Mubramaj could you please review the requested changes?


resumeSession should also set isLoggingIn to true?

For example: the app is loaded, and called resumeSession  - the tokens were fetched from the storage and now it being validated, during this time, does isLoggingIn should be set to true?

Yes it should be. In fact I think isLoggingIn should be more aptly named isLoading


fees: support max functionality for multi-asset fees

References https://github.com/penumbra-zone/web/issues/1313.
this only support max spends

Actually, sorry, one big-picture question: is it actually correct to set the spends instead of the outputs for a send transaction? I'm not sure what that would do... is that being handled by your code changes to planner.rs?

Actually, sorry, one big-picture question: is it actually correct to set the spends instead of the outputs for a send transaction? I'm not sure what that would do... is that being handled by your code changes to planner.rs?

We modified the protobuf definition to add spend actions as a special user intent to signify max functionality. In a regular send transaction, the planner constructs an output request. However, for send transactions where the user attempts to spend their entire balance, we track additional internal state to identify this intent and then branch on that logic to construct a spend request. The spend action is handled differently by the planner.

@jessepinho switched auctions to default to using the native staking token rather than supporting multi-asset fees, which will implemented separately to reduce scope.