Change Button ripple color

How can I change the ripple color ?
It's gray.
The button is the color of ColorAccent, it's ok but for the ripple ?
Thank you

https://github.com/afollestad/material-dialogs#selectors

I have to create selector, I can not just pass the color?

Yeah, changing the color that only applies above Lollipop wasn't a
necessary feature IMO.
You may want to look into the theme attribute that ripples get their color
from. You can change it globally in your app if you wanted to.
On Wed, Sep 23, 2015, 10:06 AM Anth06ny<EMAIL_ADDRESS>wrote:

I have to create selector, I can not just pass the color?
—
Reply to this email directly or view it on GitHub
https://github.com/afollestad/material-dialogs/issues/709#issuecomment-142632641
.

--
Regards,
Aidan Follestad
http://aidanfollestad.com
651-528-1662

I already change the ripple color in all application

But it's not apply on your library

In the next release, it will correctly use the colorControlHighlight value for ripples.
I'll actually just add a way to manually set it, too. Thanks for the suggestion.

When I publish the next release, it will respect the controlColorHighlight attribute from your Activity theme.
You will also be able to use a new md_btn_ripple_color attribute specifically for this library, and you'll be able to set it from the Builder (buttonRippleColor(int), buttonRippleColorRes(int), buttonRippleColorAttr(int)).

https://github.com/afollestad/material-dialogs/releases/tag/<IP_ADDRESS>

Thanks for your reactivity.

Be carreful it's "colorControlHighlight" and not "controlColorHighlight" in your release text.
It's working great job


forbid-prop-types and require-default-props doesn't pick up variable defs

At Airbnb, we use the following convention:
const propTypes = {
   ...
};
class MyComponent extends Component {...}
MyComponent.propTypes = propTypes;

The propTypes rule doesn't pick up this convention.
@leland-richardson @ljharb

@lelandrichardson

This overlaps with #997.

I can take a look at this one.

Edit: It looks like there is already a passing test for this case, @gpeal would you mind providing a code sample or failing test?


InTopKV2 cannot use k as a tensor

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux Ubuntu 16.04
TensorFlow installed from (source or binary):
binary
TensorFlow version (use command below):
TF 1.0

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
I try to use k as a tensor in InTopKV2 however, it tells me the k must be a scalar.
This is my code
input = tf.constant([[0.1, 0.2, 0.3, 0.4], [0.5, 0.6, 0.7, 0.8]])  k = tf.constant([2, 3])  output = gen_nn_ops.in_top_kv2(input,[2,2],k)

Describe the expected behavior
K can be used as 1-D tensor
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@593618638
Looks like code is incomplete.Please, provide simple standalone code, then it is easy for localizing the issue faster. Thanks !

@593618638
Please, let us know any update on this issue. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


MATRIX SEARCH

STAR THIS REPO!
Given an n x m matrix, where every row and column is sorted in increasing order, and a number x . Find if element x is present in the matrix or not.
Input Format
First line consists of two space separated integers N and M, denoting the number of element in a row and column respectively. Second line of each test case consists of N*M space separated integers denoting the elements in the matrix in row major order. Third line of each test case contains a single integer x, the element to be searched.
Constraints
1 <= N,M <= 30 0 <= A[i] <= 100000
Output Format
Print 1 if the element is present in the matrix, else 0.
Sample Input
3 3
3 30 38
44 52 54
57 60 69
62
Sample Output
0

ADD IN SEARCHING SORTING REO

can you assign this to me as a part of hacktoberfest 2020

ok

should I contribute

Yes you can

i will contribute for hacktoberfest 2021. So please accept

please assign me

Hello Akshita, hope everything is good.
If this one is still available then please assign it to me, I'll finish this in few hrs.


Clip planes defines perf

Revert v5 factorization changes as they seem to be harmful to the V8 optimizer.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13742/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13742/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13742/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13742/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13742/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13742/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13742/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13742/merge#BCU1XR#0


non-writable observables should not have a setter

Thanks for the great work! :+1:  This is definitely a hidden gem in the Knockout universe.

thanks! :)


change orderBy asc to desc in findFirst

In findFirst description it stated to fetch the largest ID as it is the most recent but in orderBy it was using asc i.e. lowest first
so changed the orderBy to desc

Nice catch!


Settings Editor: Parent Null error when keyboard navigating settings

Insiders master 8/1/18
Windows
Error Cannot read parent of 'null' when navigating the settings editor with keyboard
Steps to reproduce:
Open new settings
starts in search bar
navigate down with keyboard to files autosave delay input box
navigate back up the list
error usually occurs just before header

Thanks!

FYI
I think you can get this with a variety of Tabs and arrow keys as well


RelativeRect.fromLTRB() bottom value has no effect

Steps to reproduce
CustomElevatedButton(
icon: Icons.more_vert,
onPressed: () {
showMenu(
context: context,
position: const RelativeRect.fromLTRB(double.infinity, double.infinity, 20, 20),
items: const [
PopupMenuItem(
value: 'Añadir',
child: Text('Añadir'),
),
PopupMenuItem(
value: 'Filtrar',
child: Text('Filtrar'),
),
],
elevation: 8.0,
).then((choice) {
if (choice != null) {
if (choice == 'Añadir') {
showDialogMethod();
} else if (choice == 'Filtrar') {
}
}
});
},
)
Expected results
It should separate menu 20 from bottom, as happens with menu 20 from right but it has no effect
Actual results
The menu is at the bottom right, separated 20 from the right but stuck below
Code sample
CustomElevatedButton(
icon: Icons.more_vert,
onPressed: () {
showMenu(
context: context,
position: const RelativeRect.fromLTRB(double.infinity, double.infinity, 20, 20),
items: const [
PopupMenuItem(
value: 'Añadir',
child: Text('Añadir'),
),
PopupMenuItem(
value: 'Filtrar',
child: Text('Filtrar'),
),
],
elevation: 8.0,
).then((choice) {
if (choice != null) {
if (choice == 'Añadir') {
showDialogMethod();
} else if (choice == 'Filtrar') {
}
}
});
},
)
Screenshots or Video
No response
Logs
No response
Flutter Doctor output

_It looks like you're trying to position a PopupMenu in Flutter using showMenu with a RelativeRect. However, the positioning you've described might not work as expected because RelativeRect.fromLTRB sets the left, top, right, and bottom offsets of the rectangle relative to the parent widget's edges, but it doesn't take into account the menu's size.
To achieve the desired separation from the bottom right corner, you can calculate the position dynamically based on the menu's size. Here's an updated version of your code that should position the menu 20 units from the bottom and right edges:_
CustomElevatedButton(
icon: Icons.more_vert,
onPressed: () {
final RenderBox button = context.findRenderObject() as RenderBox;
final Offset buttonPosition = button.localToGlobal(Offset.zero);
final double screenHeight = MediaQuery.of(context).size.height;
final double screenWidth = MediaQuery.of(context).size.width;

final double dx = screenWidth - buttonPosition.dx - 20;
final double dy = screenHeight - buttonPosition.dy - 20;

showMenu(
  context: context,
  position: RelativeRect.fromLTRB(dx, dy, 0, 0),
  items: const [
    PopupMenuItem(
      value: 'Añadir',
      child: Text('Añadir'),
    ),
    PopupMenuItem(
      value: 'Filtrar',
      child: Text('Filtrar'),
    ),
  ],
  elevation: 8.0,
).then((choice) {
  if (choice != null) {
    if (choice == 'Añadir') {
      showDialogMethod();
    } else if (choice == 'Filtrar') {
      // Handle 'Filtrar' choice
    }
  }
});

},
)
This code calculates the position of the button relative to the screen and then adjusts the menu's position accordingly to ensure it is 20 units from the bottom and right edges.
``

You're right!! The menu have been placed 20 units from the bottom edges. The 20 units of the right edge have not worked in this way, what I have done is to collect both options for it to work correctly:
CustomElevatedButton(
icon: Icons.more_vert,
onPressed: () {
final RenderBox button = context.findRenderObject() as RenderBox;
final Offset buttonPosition = button.localToGlobal(Offset.zero);
              final double screenHeight = MediaQuery.of(context).size.height;
              final double screenWidth = MediaQuery.of(context).size.width;

              final double dx = screenWidth - buttonPosition.dx;
              final double dy = screenHeight - buttonPosition.dy - 150;

              showMenu(
                context: context,
                position: RelativeRect.fromLTRB(dx, dy, 20, 0),
                items: const [
                  PopupMenuItem(
                    value: 'Añadir',
                    child: Text('Añadir'),
                  ),
                  PopupMenuItem(
                    value: 'Filtrar',
                    child: Text('Filtrar'),
                  ),
                ],
                elevation: 8.0,
              ).then((choice) {
                if (choice != null) {
                  if (choice == 'Añadir') {
                    showDialogMethod();
                  } else if (choice == 'Filtrar') {
                    // Handle 'Filtrar' choice
                  }
                }
              });
            },
          )

Thank you very much!!!!

Hi @Rafapeerez, has your issue been resolved?


[Broken] TV2 extractor fails on all videos

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running yt-dlp version 2021.08.10
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

yt-dlp -v https://www.tv2.no/v/1676667/
[debug] Command-line config: ['-v', 'https://www.tv2.no/v/1676667/']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] yt-dlp version 2021.08.10 (exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg git-2020-06-28-4cfcfb3, ffprobe git-2020-06-28-4cfcfb3
[debug] Optional libraries: mutagen, pycryptodome, sqlite, websockets
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (NO) as X-Forwarded-For
[debug] [TV2] Extracting URL: https://www.tv2.no/v/1676667/
[TV2] 1676667: Downloading metadata JSON
[TV2] 1676667: Downloading play JSON
ERROR: Unable to download JSON metadata: HTTP Error 503: Service Unavailable (caused by <HTTPError 503: 'Service Unavailable'>); please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
  File "yt_dlp\extractor\common.py", line 682, in _request_webpage
  File "yt_dlp\YoutubeDL.py", line 3190, in urlopen
  File "urllib\request.py", line 531, in open
  File "urllib\request.py", line 640, in http_response
  File "urllib\request.py", line 569, in error
  File "urllib\request.py", line 502, in _call_chain
  File "urllib\request.py", line 649, in http_error_default

Description

All tv2.no videos currently fail. The API version used by yt-dlp/youtube-dl seems to have been shut down.
Looking at the page in a desktop browser, the metadata API url now seems to be https://rest.tv2.no/player-metadata-api/v3/asset/[video-id], and the structure of the data returned has changed slightly. Also, that endpoint only returns .ism files now, so there's a separate POST request to https://api.sumo.tv2.no/play/[video-id]?stream=DASH to get at the dash stream.

Yeah it's broken. I'll make a PR fixing this soon.

@vonstring Does the site no longer has live videos?
If they do then send an example link

@vonstring Does the site no longer has live videos?
If they do then send an example link

It does, here's an example: https://www.tv2.no/v/1153546/


update react-scripts

Click tested and everything looks good. Can you please add some documentation of why this is high risk, and list the pros and cons?

@zarsky-broad how would I test this? What should I be on the lookout for?

@katevoss literally everything. But basically, if all the parts still load on the PR site, we're good


Spring Kick banish management

Like you, I agree that this looks like other banishers and should work.
Do you have any sense that this code works as expected (from local build and run)?  Would you consider expanding your skill set and trying to write a test?  That might be as simple as capturing HTML and doing a copy/past from some other test.

I can give a test a try, but it might take me a long time.  I know very little about how any of this works in detail.
I'll look through other commits and see if I can find something to copy maybe.

Sounds good.  If you give up let me know and I will either try and help or just approve.  But since I don't have the item you might have to be the one that captures fight html with a banish.

I was hoping to find a similar test from a past banish commit, but it looks like all the recent additions (monkey slap. crimbo23 stuff, asol, Roar like a Lion) did not include any tests.
I can capture and share html if that helps.

The maximizer fails to equip is sucking up all of my cycles.   Maybe someone else will chime in and assist with a test?  Otherwise I'll get to it eventually or just submit.  Thanks.

I look casually at FightRequestTest.  commerceGhostIncrementsByOneOnFight seems to parse a combat and then check for a preference to change.  You could capture HTML from a fight with a spring banish, parse it as part of the test and just verify that the preference changed to include the banished monster.  I would expect that to work as a test although I am not in a position to try it for you.

look at e.g. FightRequestTest.canDetectEagleScreech. That's phyla instead of monster, but the idea is the same.

I looked at that FightRequestTest... most of that went way over my head.  I could spend some time trying to hack together a test, but I fear there is an equal change of getting a good test as there is at just messing up the code and breaking it.
It would probably be easier to try and figure out how to compile Mafia and run some test adventures, but I don't know how to do that either or if it counts.

no problem, if you're up for it I can make a test and add it to your branch myself, and we can join a video call over Discord so you can see what I do live, if you'd like.
But the long of it is:

enter a fight with spring boots equipped
command debug on
use the banish
command debug off
you now have some HTML in the Mafia root directory. You're going to put this in test/root/request/ and make a test looking something like

  @Test
  public void canDetectSpringBoots() {
    var cleanups =
        new Cleanups(
            withFight(),
            withProperty("banishedMonsters"));

    try (cleanups) {
      parseCombatData(
          "request/test_fight_spring_boots_banish.html", "fight.php?action=skill&whichskill=7501");

      assertThat("banishedMonsters", hasStringValue(startsWith("fluffy bunny:Spring Kick:")));
    }
  }

at the bottom of FightRequestTest.java.
As to "how do I compile Mafia myself", I will assume you've already got the source downloaded and the jdk installed. From there, go to the root directory of the source, and run ./gradlew :shadowJar. This will make a jar file in dist/. Copy it to wherever or just run in straight from there.

Also, it probably /should/ be turn-free, because it doesn't end the combat. I think this is the first banish that's true for?

@midgleyc That helps.  I will capture some fight html and use your code snippet.  I also need to install a JDK (I have JRE only now) to try that compile.
Do we need to check that spring shoes are equipped in the 'Cleanups' section?
When is this check done?  Is this part of the PR check, or does that test happen at compile?
I am happy to jump on a call and learn more about this if you have time.  I am on the west coast of the US, and you look like you are in the UK, so timing might be tricky.
As for "isTurnFree" I don't know what it should be as it works different than all the other banishes that end combat.  I would expect this to be false based on the fact that it cannot act as a free runaway or delay burn, which was my line of thinking when I made this update.

Hey look!  I added a test... which failed... I'm not sure if this is because the code is wrong or the test is wrong.

Yeah, the test was wrong: I forgot that banishedMonsters needs special handling, because it reads from a static variable to populate it.
I've fixed it.


Add previous namespace mismatch warning

For now we can add at least warning for this issue #3037
What do you think? May be we can change target namespace if previous release was deleted without purge flag.

I'll take a look at this in the beginning of next week

@Lookyan So I tried to get this warning to print, but I couldn't get it. --namespace is only used if you are in install, in which case the release would fail anyway because it already exists. Am I forgetting a use case with this?

@thomastaylor312 Yes, --namespace flag works only with --install. And this warning also appears only in case we have --install flag and namespaces are not equal. This is the main check for --namespace functionality which we have for now.
If we want to have an ability to use this flag without --install, algo can be smth like this:

Get previous namespace
Check whether previous release is in deleted phase or not
Compare previous and current namespaces. If they are equal we continue deploying. If they are not equal and previous release is not in deleted phase, we throw warning and exit. And last case: if previous release was deleted, we deploy new release to namespace from --namespace flag.

But for now --namespace is used only for installation and at least we should have warning from this PR.
So, I can implement this algo if you think it's OK

So I don't think we need to add more to the algorithm, but I was wondering how to trigger this case. I tried doing a helm upgrade --install --namespace a-different-one ... and the error didn't print.

@thomastaylor312 you need to install, delete the release and then try to install it again using helm upgrade --install flag.

Yes, to get this warning you should do smth like this:
helm upgrade --install --namespace first myrelease .
helm upgrade --install --namespace second myrelease .
Second command will have this warning because namespace still will be "first" for deployment and "second" won't be applied.


New package: Mozilla.SOPS version 3.7.3

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

And another one similar to #95634

@wingetbot waivers Add Validation-Executable-Error


Help: How to assign a nodeSelector to the service

Please report any bugs or feature requests specific to the PGO Examples that are in this repository. This includes anything around the examples for Kustomize and Helm.
For any bugs or feature request related to PGO itself, please visit https://github.com/CrunchyData/postgres-operator

Closing since this one is missing a valid description.


Image content for phonegap app on android

I m trying to use addImage in a phonegap application. All the code works perfectly well on a desktop browser but results in a corrupted pdf file on android.
toDataURL returns a png image string instead of a requested jpeg image string.
I used the JPEGencoder framework which can be found in this repo https://github.com/owencm/javascript-jpeg-encoder.git to fix this.
There is a copy of the code  I use at the end of this post. The code prepares data for addImage and does pretty much what addImage does at the beginning when passing an Image object.
Once the document is written I can observe a /Image object in the pdf script code but it differs from what I obtain on a desktop computer (JPEG algo probably differs). what bother me is that Acrobat reader complains there is an error in the pdf file and no image is displayed.
Here is the piece of code I used
	var getJpegImageDataURI= function(img,w,h)
	{
		var canvas    	= document.createElement("canvas"),
			ctx   		= canvas.getContext("2d"),
			url			= undefined;

		if( w === undefined )w=img.width;
		if( h === undefined )h=img.height;

		if(ctx!==undefined)
		{
			canvas.width  = w;
			canvas.height = h;

			ctx.drawImage(img,0,0,w,h,0,0,w,h);

			var toDataURLFailed=false;
			try
			{
				url=canvas.toDataURL("image/jpeg",1.);
			}
			catch(e)
			{
				toDataURLFailed=true; // android may generate png
				alert("toDataURL failed try JPEGEncoder if type deos not match jpeg: " + e);
			}

			if( toDataURLFailed || (url.slice(0,"data:image/jpeg".length) !== "data:image/jpeg") ) // check not jpeg type
			{
					// fallback generate jpeg data
					try
					{
						var encoder = new JPEGEncoder();
						url= encoder.encode(ctx.getImageData(0,0,w,h), 100);
					}
					catch(e){
						alert("JPEGEncoder failed"+e);
					}
			}
		}
		else
		{
			console.error("[IZIWATT-CONTOLLER] 2d canvas not supported");
		}
		if(url===undefined)alert("getJpegImageDataURI return undefined");
		return url;
	};

WOW its work @pascalPimaia !!!!!!!! THX A LOT MAN !!!

Thanks @pascalPimaia !!


How can i resize an icon button

For my application i need to resize an icon button. Basically i need to make it a little smaller. I can make icons smaller but button seem to be of the same size irrespective of the size of icons.
Any ideas how to resize the buttons.
Thanks

You can indeed change the size, but the ripples will get messed up. the best way to achieve this is:

set width and height to the size you want.
use negative values for margin on the icon menu to bring the neighboring elements closer.

Your best bet it flexbox. it was very easy to position the button correctly for me using flexbox.


New Loss function and Metric to monitor best weights

Hello I am still exploring keras in the past few months so I am new here. I like it so much thanks for @fchollet  . I just have 3 problem now.

How we can create new loss function?[better with example]
In here for example I have 4 pairs of coordinate p = x1,y1,x2,y2,x3,y3,x4,y4 which allocated in 1x8 array as the output of network. I want to create euclidean loos function which compare y_pred and y_true so I will have 4 distance as the result comparing every point in the array. The problem is I just know that I must use Keras Backend operation. Now the proble is how can I iterate through the tensor? for example I am taking the mean of all 4 distance:
def euclidean(y_true, y_pred):
    ktotal = K.zeros((1,4))
    i = 0
    j = 0
    while(i<7):
        ktotal[j] = K.sqrt(K.square(y_true[i]-y_pred[i]) + K.square(y_true[i+1]-y_pred[i+1]))
        j = j + 1
        i= i + 2
    return K.means(ktotal)
It still have error, I know maybe because of tensor we must use backend operation, but I dont know what to do.
How can we create new metrics, for example we define our custom metric of accuracy? like in my case acc = 1-(max distance/mean(euclidean)
How we monitor using new metrics and save the best weight using our custom metrics?
Any suggestion is welcomed. Thank you.

Here you find an example of how a custom dice function is as loss function:
https://github.com/EdwardTyantov/ultrasound-nerve-segmentation/search?utf8=✓&q=dice_coef&type=

@Danielhiversen Yes I have check it before post this question. From that post I know that it must using Keras Backend. But I just ask how about we need iterate through index?, because in my case I need taking it by indexevery 2 elements like d1 = sqrt((predx1-truex1)^2+(predy1-truey1)^2) then compute d2, d3, d4. For example if I just take one by one not per 2 elements in array, I think I just simply can do like in that example.


Support rgb/rgba and hsl/hsla colors in parameters, fixes #39

Changes
This PR adds support for rgb and hsl color values in parameters:
test-sprite-with-rgb-param {
	background-image: url("sprite-camera.svg" param(--color rgb(0, 0, 255)));
}

test-sprite-with-rgba-param {
	background-image: url("sprite-camera.svg" param(--color rgba(0, 0, 255, 0.5)));
}

test-sprite-with-hsl-param {
	background-image: url("sprite-camera.svg" param(--color hsl(240, 100%, 50%)));
}

test-sprite-with-hsla-param {
	background-image: url("sprite-camera.svg" param(--color hsla(240, 100%, 50%, 0.5)));
}

Tests are included, and the output has been tested in a live project.
package.lock.json has been committed as well.

Looks good. Now I’m reviewing this to see what happens currently before the changes.

@jonathantneal thanks! Currently, isFilledParam returns false as the node type of the value is function, not word, so the default value gets substituted in (if provided). It does not error.

@lazd, thanks for informing me of this bug, and going the extra awesome mile to make a PR for it.
I will accept the PR as is, but I would prefer a smaller change. If you’re okay with this, I’d prefer to work with you and let you make the change I’m suggesting.
The real bug appears to be that we should support non-word params. That can be down by removing the last type check in isFilledParam on L121:
return node.type === 'function' && node.value === 'param' && node.nodes.length === 3 && node.nodes[0].type === 'word';

Then, because we’re still allowing the param value to be any value node, we should just stringify that node on L110:
params[node.nodes[0].value] = parser.stringify(node.nodes[2]);

And that’s it! That should solve your problem, and not limit us to only currently known functions or node types. What do you think?

Well, I can just amend your commit so you can still get credit :)

That sounds perfect, I'll go ahead and insolent that shortly. Thanks for the guidance!

On Jun 23, 2017, at 12:53 PM, Jonathan Neal<EMAIL_ADDRESS>wrote:
@lazd, thanks for informing me of this bug, and going the extra awesome mile to make a PR for it.
I will accept the PR as is, but I would prefer a smaller change. If you’re okay with this, I’d prefer to work with you and let you make the change I’m suggesting.
The real bug appears to be that we should support non-word params. That can be down by removing the last type check in isFilledParam on L121:
return node.type === 'function' && node.value === 'param' && node.nodes.length === 3 && node.nodes[0].type === 'word';
Then, because we’re still allowing the param value to be any value node, we should just stringify that node on L110:
params[node.nodes[0].value] = parser.stringify(node.nodes[2]);
And that’s it! That should solve your problem, and not limit us to only currently known functions or node types. What do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Released as 2.1.0!

@jonathantneal you're too fast, thanks!


Pulling release caused my HASS to crash.

Problem/Motivation

(Why the issue was filed)

Expected behavior

(What you expected to happen)

Actual behavior

(What actually happened)

Steps to reproduce

(How can someone else make/see it happen)

Proposed changes

(If you have a proposed change, workaround or fix,
describe the rationale behind it)

Just an FYI.  After months of using Z-Wave JS to MQTT with zero issues.  A few days ago my HASSIO VM started crashing.  It would last anywhere from 1 minute to 25 minutes before suddenly powering off.  I looked everywhere for a cause, from VCenter to HASS Logs but there was no culprit.
Suddenly I noticed that both Z-Wave JS and Z-Wave JS to MQTT were installed and running.  Every time I would stop Z-Wave JS  and even uninstall it.  After the VM powered back on it was back.  Then I also noticed that Z-Wave JS to MQTT had a generic puzzle icon instead of the normal Z-Wave icon.  Finally out of desperation I uninstalled Z-Wave JS to MQTT. Only to find it was no longer listed in the Supervisor add-on store.    I tried in vain to figure out where Z-Wave JS to MQTT went, but it was completely gone from the store.  So I reconfigured my Z-Wave to use the built-in Z-Wave JS.  Today I checked and it's listed in the store again, and I noticed in the changelog that it says "The previous release was withdrawn"  I don't know if that means you withdrew it or the HASS team did, but it seems like somehow pulling the release broke my container enough to force HASS to install the default Z-Wave JS, and having both enabled caused instability and crash.
I am not complaining or pointing fingers, and I greatly appreciate the products you and others create for the community.  But I wanted you to know about my experience.  Especially if it happened to others too.  And if it was caused by pulling a release, maybe in the future you could just push a replacement instead.
Brady

Moved to discussions


Network INT8 calibration failed.

Q1:
When I run command line CUDA_VISIBLE_DEVICES=0 retinanet export retinanet_rn50fpn.pth model_int8.plan   --batch 2 --int8 --calibration-images /data/calib/ --calibration-batches 10 --calibration-table model_calibration_table --size 1280, I got the following error log
Loading model from retinanet_rn50fpn.pth...
     model: RetinaNet
  backbone: ResNet50FPN
   classes: 1, anchors: 9
Exporting to ONNX...
Building INT8 core model...
Building accelerated plugins...
Applying optimizations and building TRT CUDA engine...
Half2 support requested on hardware without native FP16 support, performance will be negatively affected.
Tensor (Unnamed ITensor* 295) copy is uniformly zero; network calibration failed.
../builder/cudnnBuilder2.cpp (1508) - Misc Error in buildEngine: -1 (Could not find tensor x in tensorScales.)
../builder/cudnnBuilder2.cpp (1508) - Misc Error in buildEngine: -1 (Could not find tensor x in tensorScales.)
Segmentation fault (core dumped)

Q2:
Considering that tensorrt inference only occupies 1.6G cuda memory with batch 2, it's necessary to increase max batchsize of tensorrt engine. However, exporting trained pytorch model to tensorrt occupied too much cuda memory, and it's hard to increase max batchsize of tensorrt. Is there any solution?

Hi Ricardozzf,
Q1: Can I ask which GPU you are trying to use? We have been able to successfully use the INT8 calibration process on Volta and Turing GPUs. I suspect your GPU may not have the proper HW support needed for fast INT8 (or FP16 according to the TRT error).
Q2: The issue you are describing is mostly likely happening during the PyTorch -> ONNX export before the TensorRT engine is built. We just added a small change to lower the GPU memory footprint during export, please give it a try and let us know if that helps you export models with a larger batch size.
Some other potential ideas: you could try using a smaller input resolution (you can add --size x y to your retinanet export command above to specify a different input resolution). Alternatively, TensorRT allows you to use create multiple execution contexts for a single TensorRT engine, which allows one set of weights to be reused for multiple inference tasks. You can get some more information here: https://docs.nvidia.com/deeplearning/sdk/tensorrt-developer-guide/index.html#perform_inference_c

Thanks for your reply @pkashinkunti:
Q1:
The GPU I used is TITAN X (Pascal), as far as I know, TITAN X do not support FP16, but it should support INT8. I have run FP32 tensorrt engine successfully.
Q2:
I am glad to inform that the small change did work. Now I could use TITAN X (Pascal) to generate tensorrt engines with a larger batch size 2->12, 14, 16 ,32. However, the  *.plan files whose batch size is bigger than 12 are unusable. The log shows:
root@ee13bae5e0e3:/workspace/retinanet/TensorRT# CUDA_VISIBLE_DEVICES=0 retinanet infer model_fp32_16.plan --images /data --annotations /data/Test.json --full-precision --batch 16
Loading CUDA engine from model_fp32_16.plan...
Preparing dataset...
    loader: pytorch
    resize: 800, max: 800
   backend: tensorrt
    device: 1 gpu
     batch: 16, precision: unknown
Running inference...
THCudaCheck FAIL file=../aten/src/THC/THCCachingHostAllocator.cpp line=265 error=77 : an illegal memory access was encountered
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/opt/conda/lib/python3.6/site-packages/torch/utils/data/_utils/pin_memory.py", line 40, in _pin_memory_loop
    batch = pin_memory_batch(batch)
  File "/opt/conda/lib/python3.6/site-packages/torch/utils/data/_utils/pin_memory.py", line 55, in pin_memory_batch
    return [pin_memory_batch(sample) for sample in batch]
  File "/opt/conda/lib/python3.6/site-packages/torch/utils/data/_utils/pin_memory.py", line 55, in <listcomp>
    return [pin_memory_batch(sample) for sample in batch]
  File "/opt/conda/lib/python3.6/site-packages/torch/utils/data/_utils/pin_memory.py", line 49, in pin_memory_batch
    return batch.pin_memory()
RuntimeError: cuda runtime error (77) : an illegal memory access was encountered at ../aten/src/THC/THCCachingHostAllocator.cpp:265

I have tried to use a smaller input resolution 800 with 16-batch plan file , it did not work. But, I have run 12-batch plan file with input resolution 1280 successfully.

Q1: I just tried to use your command on a similar GPU (P40, which is also GP102 based like TitanX) and did not see the same failure. I used images from Pascal VOC2012 to calibrate the engine. Are you using a custom dataset for INT8 calibration? It may be the case that you need to use more batches for calibration (or perhaps a different dataset).
Q2: Glad that change helped a bit: we are looking into this error with large batch plan files.

Regarding Q2, I merged https://github.com/NVIDIA/retinanet-examples/pull/7 which should fix the issue. Thanks for flagging it!

Q2 has been fixed. :)
Regarding Q1, I have tried to use more batch and different dataset to calibrate model, but the error still exists. I find that someone has met similar issue https://devtalk.nvidia.com/default/topic/1046066/int8-calibration-problem/, so I doubt that my model is the cause of Q1. I will tried it with a new model.

I think I have found the cause of  the error. I trained a new model based on COCO2014 and successfully exported it to INT8-Plan, in this training, I used different dataset custom dataset -> COCO2014 and class-num(1->80), so I doubt one of diferent dataset and different class-num caused this error.
If custom dataset is the cause, it may make .pth model generate some odd value that interrupted  TRT's optimizing, and if class-num is, perhaps some code segment contains variable class-num interrupted optimizing.
For the above two cases, are there  any corresponding solutions?

Would you mind listing the different steps/commands used in this process so we can better understand the issue and answer your questions?

I used command CUDA_VISIBLE_DEVICES=0,1,2,3 retinanet train retinanet_rn50fpn.pth --backbone ResNet50FPN --images /coco/train --annotations /data/CocoTrain.json --val-images /coco/val --val-annotation /data/CocoVal.json  --batch 8 --logdir log --classes 80 which is different from CUDA_VISIBLE_DEVICES=0,1,2,3 retinanet train retinanet_rn50fpn.pth --backbone ResNet50FPN --images /data --annotations /data/SensingTrain.json --val-images /data --val-annotation /data/SensingTest.json --classes 1 --batch 8 --logdir log

Is the loss converging in the second case? You might have to tweak the learning rate to get good results there.

@florischabert Yes,the loss of the second case is convergent, and my model works correctly in fp32 mode. I have tried increase calibration-batches to 150, replacing Layer ReLU with Layer LeakyReLU, and changing calibration datasets, but these did not work.

Hard to tell what the issue is with a specific dataset. You can try to calibrate using you test dataset and see if it works, maybe your calibration data is not a good representation of your test data?

Closing this, if you still have trouble with int8 please open a new issue with specific details allowing us to reproduce the problem. Thank you!

@Ricardozzf
Hi, did you solve the problem? I have faced the same error. If you have solved, could you tell me more information about it? Thank you.


fix(app): Re-enable change pipette and pipette settings

overview
This PR serves as a bug report and fix. A recent API update returns both a name and model for attached pipettes. The AttatchedInstrumentsCard and InstrumentInfo were using name for robot.name which caused a conflict resulting in broken routes for change pipette and pipette settings.
This PR changes name (robot) prop to robotName which fixes the pipette modal routes.
changelog

fix(app): Re-enable change pipette and pipette settings (robotName prop)

review requests
Please test on a robot with edge installed and one with an older version of the API!

[ ] attached pipettes card renders as expected
[ ] clicking [change/attach] opens change pipette modal (please confirm change pipette walkthrough still works as expected)
[ ]clicking [settings] for an attached pipettes pops up pipette config modal. (please confirm settings are still updatable.

Files that are not affected by this PR but should be double checked if possible:

[ ] file info page pipettes section is unaffected
[ ] tip probe pages are unaffected

Found another issue (separate maybe?):
When I am in the attach pipettes flow and I have detached one pipette in order to attach another, if I select which pipette to attach the modal disappears completely and I cannot finish the flow.


HHH-10762 - Implemented left joins for relation traversion in audit queries

... by leveraging the new HQL feature to join unrelated entities.
Furthermore, the implementation of inner joins have been improved by
using the same new HQL feature. The audit query API has been extended to
support criterias where two properties of different entities are
disjuncted, conjuncted or directly compared to each other.

Thanks for the PR, @ffeisst.

Applied upstream.


[WIP] Refs #32508 -- Raise proper exceptions instead of using 'assert' in django.db

[WIP] Currently worked on djagno.db.models only

This is great--thanks for taking this on. I see you've marked this WIP, so I hope this is a helpful set of notes for when you decide to return to this (rather than just being an annoyance!) Best, Jacob

Hey @jacobtylerwalls !
My pleasure! I'm sorry for being inactive. Giving my time to #14437. But I'd like to finish this up too! I'll add a set of notes in the description.

@abbasidaniyal If you don't time to keep working on this, I can push final edits to your PR and someone else can fix the remaining cases (if any).

@felixxm I'm sorry I couldn't make any progress as I had my college exams going on. I believe major occurences were tackled. If it is urgent, it'll be great is somebody else can take it up!


The information on environment construction is incorrect

Hello,
It is presumed that the content of "Let's get the environment ready" described in the following document is incorrect.

https://github.com/Azure-Samples/Synapse/blob/master/Notebooks/PySpark/Synapse Link for Cosmos DB samples/MongoDB/spark-notebooks/pyspark/01-CosmosDBSynapseMongoDB.ipynb

Creating requirements.txt as instructed and applying it to the Spark pool does not install pymongo.
Also, there was no mention of pymongo in the output result of the libraries described in the document.
I'm asking a similar question to Microsoft Q & A and I'm currently investigating the cause.

https://docs.microsoft.com/en-us/answers/questions/105473/spark-pool-does-not-reflect-the-contents-of-requir.html

Please confirm and investigate.
Thanks,

Hello Reviewers,
I modified the Readme and .ipynb files to add to the content of the pull request I have already sent.
Please confirm and merge.

Fixed an issue where samples could not be run according to the content https://github.com/Azure-Samples/Synapse/pull/53

Thanks,

Hello @ymasaoka
We published a new version. Do you want to check it out?

Hello @Rodrigossz ,
Of course I will! If you give me a few days, I will comment on the results I checked here again! Thank you for responding!

Hello @Rodrigossz ,
Sorry I made you wait. I checked the contents.
I have some concerns, can you check it?

Insert another dataset, but this time using the MongoSpark connector.

Do you use MongoSpark connector? For the two data inserts, both appear to be using db of the MongoClient.

Create a collection named HTAP with a partition key called item.

The name Shard key is used on the Cosmos DB Data Explorer. The name Shard key is also used for MongoDB.
I was worried because there is no sense of unity in this part.
Thanks,

Good catches.  WIll fix it. TKS

Also, some of the PySpark syntax is incorrect, could you please correct it?
cell 12
# df.groupBy(df.item.string).sum().show() # incorrect
df.groupBy(df['item']).sum().show() # correct

cell 13 & cell 18
# df.printSchema # We can confirm it, but the syntax is not accurate.
df.printSchema() # correct

I was trying out the content right now, but it seems that I got caught in a bug in Azure Synapse Link for Azure Cosmos DB.
I have not yet confirmed the part where the schema information of the timestamp of the new content is updated.
However, I have confirmed that this part is the same as the previous content, and since I was able to confirm the operation last time, I think that there is no problem.

I fixed the typos and changed the schema command. It was not wrong as you mentioned, but for sure your suggestion returns a more elegant view. About aggregations, what you wrote that the existing command is incorrect and suggested another one. But that's not true, the existing syntax is the correct one. But based on your mistake, I added an explanation about it.
The PR is here: https://github.com/Azure-Samples/Synapse/pull/58


Improve the table generated in pdf

Add different styles for the pdf table to be clear to the user his
strengths, weaknesses, threats and opportunities.

That's getting good, just change the colors to the ones used in /myplan. :+1:

Okey

Check again please, I change the colors


Crashes on 27.1

Hello
Memento-more throws a stack trace on my Emacs:
Debugger entered--Lisp error: (void-variable _)
  memento-mori-birth-time()
  memento-mori-age()
  memento-mori-update()
  memento-mori-mode()
  (progn (memento-mori-mode))
  eval((progn (memento-mori-mode)) t)
  elisp--eval-last-sexp(nil)
  eval-last-sexp(nil)
  funcall-interactively(eval-last-sexp nil)
  #<subr call-interactively>(eval-last-sexp nil nil)
  apply(#<subr call-interactively> eval-last-sexp (nil nil))
  call-interactively@ido-cr+-record-current-command(#<subr call-interactively> eval-last-sexp nil nil)
  apply(call-interactively@ido-cr+-record-current-command #<subr call-interactively> (eval-last-sexp nil nil))
  call-interactively(eval-last-sexp nil nil)
  command-execute(eval-last-sexp)

memento-mori-birth-date is a string like 1111-11-11

I have the same issue on emacs 28

Looked into this a bit. Seems to have to do with byte-compilation, but don't know what the root cause is. Manually calling the defun of the memento-mori-birth-time function apparently resolves the error, but debugging it after it loads on startup I found that the cl-destructuring-bind macro creates a compiled function that yields this error.
I could work around this by disabling byte compilation for this package, which is pretty straight-forward with straight.el. I wouldn't know how to do it with other package managers though.

Missing (require 'cl-lib)?

The error is most likely coming from the (cl-destructuring-bind ...) macro call. The macro probably is not defined at compile time unless we have (require 'cl-lib) at the beginning of the soucre file.


ParticleSet Class Hierarchy

Implementing the design from the UML class diagram as discussed in #894

Quick summary of what we discussed:

The ParticleSetIterator as it currently exists will be removed. Its functionality will be merged into the ParticleAccessor.
The ParticleIterator interface from the UML diagram will not be a separate class, but will be directly included in the BaseParticleAccessor.

Quick summary of what we discussed:
* The ParticleSetIterator as it currently exists will be removed. Its functionality will be merged into the ParticleAccessor.

* The ParticleIterator interface from the UML diagram will not be a separate class, but will be directly included in the BaseParticleAccessor.

We try to keep the UML updated to reflect this class structure for ease of discussions

General comment: I will now start taking over the pieces of the ParticleSetSOA init procedure over to the ParticleCollectionSOA. So, we are then gonna work on the same file (soa.py), but on different classes within it, minimising merge conflicts.

FULL REJECT of the changes - they hamper performance, they make the whole code once more less-generalisable, and - most importantly - they are right now unnecessary to do.

Proposal: if you really want to make that drastic change, we may wanna have a talk first ;-)

re-introducing the accessors is nice, but in the end, those functions actually need to modify the particle states themselves - or, in an SoA-case, the states of the related variables array. Otherwise, those modifications won't be visible in the kernel in the next run, which may cause trouble.

Maybe I am misunderstanding your point, but the functions that are defined on the Accessor (in the BaseParticleAccessor class) do exactly this, right? They do depend on the correct implementation of setattr and getattr in the subclass for this, however. In the SoA implementation, these read from/write to the arrays in the ParticleCollection. So this should work.

re-introducing the accessors is nice, but in the end, those functions actually need to modify the particle states themselves - or, in an SoA-case, the states of the related variables array. Otherwise, those modifications won't be visible in the kernel in the next run, which may cause trouble.

Maybe I am misunderstanding your point, but the functions that are defined on the Accessor (in the BaseParticleAccessor class) do exactly this, right? They do depend on the correct implementation of setattr and getattr in the subclass for this, however. In the SoA implementation, these read from/write to the arrays in the ParticleCollection. So this should work.

hm, yes, you're right in that argument. Well, we do have ParticleSet tests that change particle properties via the ParticleSet (before kernel execution) and change values in the kernel - if those test all succeed, the implementation is apparently correct.

I just read the diff-compare cab3c18..0f6755b - looks all fine for now, so I give an approve on the current state of the PR. The new test file looks also good.
Looks good.


undefined is not an object (evaluating 'cognitoClient.initWithOptions')

Trying to test this out but running into an issue. At face value it appears that the NativeModules.AWSRNCognitoCredentials is not loading... but not sure if symptom or cause. Ideas?
Note I am running android simulation, and here is the abbrev. code:
...
import {AWSLambda} from 'aws-sdk-react-native-lambda';
import {AWSCognitoCredentials} from 'aws-sdk-react-native-core';
var functionName = "sampleFunction";
var cognitoRegion = "us-east-1";
var identity_pool_id = "us-east-1:xxxx";
var serviceRegion = "us-east-1";
...
async Setup() : Promise {
Promise.all([AWSCognitoCredentials.initWithOptions({"region":cognitoRegion,"identity_pool_id":identity_pool_id})]).then(()=>{
AWSLambda.initWithOptions({"region":serviceRegion});
});
return;
}
async _invokeRequest() {
await this.Setup();
var shouldResolve = false;
var req = {
"FunctionName" : function,
"InvocationType" : "RequestResponse",
"LogType" : "None",
"ClientContext" : this.encode(JSON.stringify({"System":"android"})),
"Payload" : JSON.stringify({"email":"testing1234"})
};
try{
console.log("testing...");
var response = AWSLambda.Invoke(req);
if(response.LogResult){
console.error("response.LogResult should not be present");
shouldResolve = false;
return shouldResolve;
}
if(!response.Payload){
console.error("response.Payload is not present");
shouldResolve = false;
return shouldResolve;
}
if(response.Payload.Length === 0){
console.error("response.Payload.Length is 0. Response payload length: " + response.Payload.Length);
shouldResolve = false;
return shouldResolve;
}
if(response.StatusCode === 0){
console.error("response.StatusCode is 0. Response code: " + response.StatusCode);
shouldResolve = false;
return shouldResolve;
}
 } catch(e) {
  console.error(e);
  shouldResolve = false;
  return shouldResolve;
 }

}
...

I have the same problem. Android. Samsung S6.
Are there things missing from the README. Do we need to add external libraries (the Android AWS  SDK) to make this work?

This fixed the issue (a bad installation more likely) for Android.
Add this line
compile 'com.amazonaws:aws-android-sdk-core:2.2.+'
in the dependencies of android/app/build.gradle if it's not there.
[How I got there:
I went to my node_modules/aws-sdk-react-native-core/android/build.gradle
saw this:
dependencies {
compile 'com.facebook.react:react-native:+'
compile 'com.amazonaws:aws-android-sdk-core:+'
}
From there, I got to this
http://docs.aws.amazon.com/mobile/sdkforandroid/developerguide/setup.html
used option 1 (my IDE is not Android Studio but I do use it) and voilà! :)]

Thanks for the tip, although it didn't work for me. I added the entry in the build.gradle and even saw it download the dependency in the build... but still get undefined is not an object (evaluating 'cognitoClient.initWithOptions') when running.
I'm using Deco for IDE but I even tried running it in the terminal and still no good. Any other ideas?

I'd suggest may be looking into your own node_modules/aws-sdk-react-native-core/ (may be the lambda one too) and see if there are any other dependencies out there. Right now, I moved on trying the other services (meaning I am not using the lambda service yet. if you are, you'd probably have to add its dependencies too I guess)
Sorry no other ideas. Good luck!

(you may want to go with the jars too [Option 2 in the link]. Though I tried it earlier and it generated an error about how the class AmazonClientException was coming from different sources)

Back to my code, I realised I did a number of things when I first got the error. Here is what I recall doing manually. (I hope it helps but general advice is to track down the dependencies.)
___android/settings.gradle
include ':aws-sdk-react-native-core'
project(':aws-sdk-react-native-core').projectDir = new File(settingsDir, '../node_modules/aws-sdk-react-native-core/android')
___in my MainApplication.java
import com.amazonaws.reactnative.core.AWSRNCorePackage;
@Override
protected List<ReactPackage> getPackages() {
  return Arrays.<ReactPackage>asList(
      ...
      , new AWSRNCorePackage()
  );
}

That last part worked! Importing and loading explicitly in the MainApp class works... like you said must be something wrong with the install. For now this will do. Thanks again for the help!

I'm having this same issue under iOS. I suspect it's something to do with the code in the node-modules folders not compiling against the SDK framework files I've included in the Xcode project for my app.
Can anyone suggest how to get past this roadblock?


No option to edit and delete tweets

It was available previously

Thanks @diwash007 for letting us know, it's now fixed.


Test startup arguments -T (which sets 'term') and -x (encryption)

This PR adds tests for the command line option -T (which sets the 'term' option) and -x (encryption).
These were not yet tested according to codecov:
https://codecov.io/gh/vim/vim/src/0353f56ddb379e7f1a68172fa4743355e04df21e/src/main.c#L2447
https://codecov.io/gh/vim/vim/src/0353f56ddb379e7f1a68172fa4743355e04df21e/src/main.c#L2297

I see that Test_T_arg which I added fails on Windows.
I'll fix that tomorrow.

@dpelle wrote:

I see that Test_T_arg which I added fails on Windows.
I'll fix that tomorrow.

That was fixed. PR is ready to merge.


splitNode() needs to take into account exact prefix match

Current implementation mixes exact prefix match with partial prefix match. This results in creating a data node with a nil key. Whether to call this a bug is arguable, but I have determined that it is in RadixDB. Therefore handle exact prefix match as a dedicated case, where the value lives with the split node.

On second thought, this is not an issue of splitNode(). This needs to be handled before splitting is considered. The nuance is closer to swap(), such that the branch needs to be restructured to accommodate a new node/record that fully matches the longest common prefix.


add jstree-grid to the official list of jstree plugins

add jstree-grid to the official list of jstree plugins
http://www.jstree.com/plugins/

In favour of it... but not much to do here. Issue is open at jstree


[EN] Feedback for: /web/updates/2015/10/chrome-47-webrtc?hl=en

Recording Sample Donwload 2 Error.
L66 - Uncaught TypeError: Cannot read property 'size' of null
L127 - a.style = 'display: none';
Bye~!

cc @samdutton

Thanks for reporting.
I can reproduce this in Chrome 47.0.2526.8 on Mac OS (but not in Canary 49.0.2591.0).
I've created an issue on the WebRTC samples repo: https://github.com/webrtc/samples/issues/715.
I'll do a patch.

Fixed in https://github.com/webrtc/samples/pull/716.
Please let us know if you still have any problems with this demo.


Clients should send the difference of their weights

Now, they send the weights to the server but this should be only the difference

With #11 this is implemented in the FedAvg class. Another client implementation is needed as this difference calculation should happen in the client.


Board testing results

This is related to the testing part of issue #128. I have written a Python script that tries to import all the defined Platform classes in nmigen-boards and then checks if building blinky using the corresponding toolchain works. If the build succeeds, it writes the checksum of the board file and the filename to a file, such that the test can be skipped if the board file hasn't been modified when running the tests again. The only thing that is currently missing is to try and specify different toolchains by setting the PATH (e.g. Intel Quartus Lite 20.1 doesn't support Cyclone III and Quartus II Web Edition 13.1 is required to test the board instead). Once I have that part figured out, I can share a gist of the Python script, because it might be useful for automation, but the amount of disk storage required to have the toolchains installed is perhaps a bit problematic.
I have the following toolchains installed:

Quartus II Web Edition 13.1
Quartus Lite Edition 20.1
Xilinx Vivado WebPACK 2020.1
Xilinx ISE WebPACK 14.7 (nothing seems to depend on this)
yosys, nextpnr, icestorm, prjtrellis

I have the following programming tools installed:

mercpcl (Mercury board) (Git)
OpenFPGAloader (Git)
OpenOCD (Git)
xc3sprog (Git)

The following boards currently fail this test:

Atlys
Genesys2Platform
KC705
KCU105
MachXO3SK (depends on Lattice Diamond)
Mercury (some issue in the generated Verilog)
MicroZedZ010, MicroZedZ020 (only defines the package + two connectors)
Numato Mimas (no toolchain_program)
Quickfeather
SK XC6SLX9 (no toolchain_program)
TinyFPGA AX1 and AX2 (only defines the package + one connector)
ZTurnLiteZ007S and ZTurnLiteZ010 (only defines the package + one connector)

All other boards that are currently in the repository (commit 068d9fe) pass this test. If nothing is mentioned between parentheses after the board name, then I am not entirely sure what causes it to fail.

I worked out the PATH environment variable. The test script that I am using can be found at [nmigen-boards-test](https://github.com/StephanvanSchaik/nmigen-boards-test]. Finally, I also updated the list of boards as the pull requests got merged.

Thank you for doing this work! I've been planning to do it for a while, but other things were more important.
The aim was always to have something like this integrated in the upstream repository, though it might still take a while to figure out all the details.

I am glad to help as I really like the nmigen project myself.

Thank you for doing this work! I've been planning to do it for a while, but other things were more important.
The aim was always to have something like this integrated in the upstream repository, though it might still take a while to figure out all the details.

I am glad I could help out here. Yes, this is still pretty basic and there are a lot of details that we have to figure out to have it upstream.
One issue is the amount of disk storage required to have the toolchains installed: 17G for Quartus Lite 20.1, 32G for Vivado 2020.1, 18G for Xilinx ISE 14.7, 24G for Quartus II WebPACK 13.1, which totals up to 91G for the proprietary toolchains. The FOSS toolchain (yosys + nextpnr + icestorm + prjtrellis) sits at about 550M, which is much more reasonable.
Another issue is that, at least for Intel/Altera FPGAs, blinky does not generate all the constraints, but just those are needed for blinky, which makes it great as an initial test to check whether things work, but it would be nice if we can get better coverage.
One idea I had for next weekend is to implement a parser for the QSF (Intel/Altera) and XDC (Xilinx) constraint files (and in the future others). As most FPGAs boards either come with a project template to start from or with multiple examples, this could be useful to generate the Python file for nmigen-boards from the constraints, given that the pin naming is somewhat sensible, and in addition this could be helpful to ensure the board definitions in nmigen-boards have the right attributes set, especially in terms of I/O standards. The generated board files would not be an immediate replacement to doing it by hand, but it would simply be a helpful tool to get started much quicker, as I would rather focus on making sure the details are right, rather than making sure I typed over the pin assignments correctly.
For newcomers, this could also lower the barrier to use nmigen if their board is currently not supported, as they don't have to figure out how to write the board definition from scratch.


the graceTime don't work

[MBProgressHUD appearance].contentColor = [UIColor greenColor];
MBProgressHUD *hud = [MBProgressHUD showHUDAddedTo:self.navigationController.view animated:YES];
hud.graceTime = 5.0f;
hud.completionBlock = ^{
NSLog(@"------------completionBlock");
};
when you create  MBProgressHUD instance and show ,the graceTime always zero,because you set graceTime hud already show and don't change the graceTime value

The graceTime needs to be set before the hud is shown. You thus can't use showHUDAddedTo:animated:, but instead need to alloc / init the hud, configure the grace time and than call show manually.
I'll improve the documentation for this.

Could you provide an example code please. I do:
 let hud = MBProgressHUD(view: view)
hud.graceTime = 2
hud.removeFromSuperViewOnHide = true
hud.animationType = .zoom
hud.backgroundView.style = .blur
hud.bezelView.color = UIColor.white
hud.bezelView.style = .blur
hud.show(animated: true)

but it doesn't work.

See https://github.com/jdg/MBProgressHUD/blob/master/Demo/HudTests/HudTests.m#L268

Addressed in d416673

@NikKovIos  I know is a super old thread but just ran into the same problem.
add view.addSubview(hud) to your code and it should work


Add Cancellation Token support and async timeout

Since there is no way to cancel an async Read on a Socket, this PR adds a way to cancel the read via CancellationTokens. Even so it does pass the CancellationToken to the Async Read, the Socket doesn't care about it at all. That's why we catch the CT right before the ReadAsync and link it to a new tokensource which is triggered based on ReadTimeout.
If that tokensource triggers it closes the stream and catches the Exception that might occure. Depending on the tokensource that caused the exception either a TimeoutException or a CanceldException is thrown (or if the error is not caused by a CTS, then the exception is rethrown.
The main Timeout handling code is here:
https://github.com/robinrodricks/FluentFTP/pull/377/files#diff-0ee804b19786cb587081edeb1a65dc72R504
Also this PR adds CancellationToken support for everything (I hope I didn't miss anything).
Fixes #262
This also fixes sync calls on .NET Core by not using the async read function:
https://github.com/robinrodricks/FluentFTP/pull/377/files#diff-0ee804b19786cb587081edeb1a65dc72R478

Wow. What an epic PR! Thanks for all the hard work and effort you put into this. Much appreciated, bro.

Released in FluentFTP 23. Thanks again!


Ogg files inside eggs cause an assertion failure

Eggs can contain ogg files, but it looks like the audio engine doesn't load them appropriately, and instead causes an assertion failure. This is reproducible on both Windows and Linux.

Aborted

Ok. This is actually caused by a missing res/around-the-corner.ogg in the stock game.egg file. Still, we probably shouldn't crash when an asset isn't found. It'd be better to provide a helpful error message instead.

Should be fixed now.


Can we collapse the gist?

I think it might be an option for user to choose.

@wujysh Yes, by default expanded, but can be collapsed.

affc30e0418b7b1060f60631888a1a2800ab73b5
4e5eead4a8d6a79fa9a1766c23ec5155fe981adf
affc30e0418b7b1060f60631888a1a2800ab73b5


Change Makefile so it uses operator-sdk commands

Instead of running go commands directly; lets use operator-sdk as that's
what we're using to build this.

Can you squash in this change:
diff --git a/Makefile b/Makefile
index 4873278..681477b 100644
--- a/Makefile
+++ b/Makefile
@@ -7,7 +7,7 @@ GOBUILD=$(GO) build
 BUILD_GOPATH=$(TARGET_DIR):$(CURPATH)/cmd

 export APP_NAME=compliance-operator
-APP_REPO=github.com/jhrozek/$(APP_NAME)
+APP_REPO=quay.io/jhrozek/$(APP_NAME)

The app_repo variable is used as a tag for docker build and it should probably be the container hub path, not the repo.

quay.io

fixed.

Thanks, merging


Export all files for use in code

Could you expose all the opf schema's? Currently, only a limited amount is exported and thus can be referenced in code.
For example: you export epub_metadata_creator.dart but not epub_metadata_identifier.dart
Is this on purpose? Looking forward to your reply.

I just noticed commit #11 which does exactly this, could you publish this version to pub.dev?

I've created pull request #19 with the required changes for this


Update main.yml

@jph00 this is ready to be merged.  Much faster with the docker container

How do I update the docker container if there's new versions of stuff? What does the image contain?

@jph00 I added a comment in this PR https://github.com/fastai/fastai2/pull/462 to try to eliminate confusion HTH


[TiKV-BR] checksum mismatch after backup

Bug Report
1. Describe the bug
Backup report checksum failure if one tikv-server crash.
2. Minimal reproduce step (Required)
Load 2kw kv data to tikv cluster with more than 3 tikv-server nodes.
Run tikv-br backup raw
During backup running, kill one tikv-server process.
Wait for backup finish.
3. What did you see instead (Required)
backup finish with success but checksum report failure.
4. What did you expect to see? (Required)
backup finish with success and checksum succeed.
5. What is your migration tool and TiKV version? (Required)

TiKV BR:

Currently, backup use file respsonse to calc checksum  including (crc64xor, totalkvs, totalbytes). If one or more tikv-server crash, tikv-br may request another backup one restored tikv-server node or new region, the response may be duplicated. This is proved by the log file. The fix is to use the final backupmeta to do checksum. This should be correct way as backupmeta is the final result saved to storage instead of using the middle result BackupResponse.

Fixed in commit https://github.com/tikv/migration/pull/103/commits/15dbfdeb93847fa29b8e617c81fef6fedd97a894

fixed in pr #103


绑定云盘出现错误

绑定office E5账号onedrive时出现错误
因为v站回复你的主题要注册1000天,,,只好在这提一下了
Internal Server Error
The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

现在支持 E5 账号了，麻烦你再试试，哈哈

现在支持 E5 账号了，麻烦你再试试，哈哈

现在支持 E5 账号了，麻烦你再试试，哈哈
On Mon, Jan 4, 2021 at 9:12 AM xiyuesaves<EMAIL_ADDRESS>wrote:

绑定office E5账号onedrive时出现错误
因为v站回复你的主题要注册1000天,,,只好在这提一下了
Internal Server Error
The server encountered an internal error and was unable to complete your
request. Either the server is overloaded or there is an error in the
application.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PoweredByCloud/feedback/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHNXXNRXG7Q4OFXPIWZ5KDSYEIYLANCNFSM4VSKPWBA
.

现在支持 E5 账号了，麻烦你再试试，哈哈
On Mon, Jan 4, 2021 at 9:12 AM xiyuesaves<EMAIL_ADDRESS>wrote:

绑定office E5账号onedrive时出现错误
因为v站回复你的主题要注册1000天,,,只好在这提一下了
Internal Server Error
The server encountered an internal error and was unable to complete your
request. Either the server is overloaded or there is an error in the
application.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PoweredByCloud/feedback/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHNXXNRXG7Q4OFXPIWZ5KDSYEIYLANCNFSM4VSKPWBA
.

现在支持 E5 账号了，麻烦你再试试，哈哈
…
On Mon, Jan 4, 2021 at 9:12 AM xiyuesaves @.***> wrote: 绑定office E5账号onedrive时出现错误 因为v站回复你的主题要注册1000天,,,只好在这提一下了 Internal Server Error The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#2>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHNXXNRXG7Q4OFXPIWZ5KDSYEIYLANCNFSM4VSKPWBA .

emmmm还是会报错诶
Internal Server Error
The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

现在支持 E5 账号了，麻烦你再试试，哈哈
…
On Mon, Jan 4, 2021 at 9:12 AM xiyuesaves @.***> wrote: 绑定office E5账号onedrive时出现错误 因为v站回复你的主题要注册1000天,,,只好在这提一下了 Internal Server Error The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#2>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHNXXNRXG7Q4OFXPIWZ5KDSYEIYLANCNFSM4VSKPWBA .

emmmm还是会报错诶
Internal Server Error
The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

我没有 E5 账号，不过我还是想麻烦你再试一下，我抓一下日志，看看是哪个地方权限不对。

我没有 E5 账号，不过我还是想麻烦你再试一下，我抓一下日志，看看是哪个地方权限不对。

我没有 E5 账号，不过我还是想麻烦你再试一下，我抓一下日志，看看是哪个地方权限不对。
…

刚刚试了,还是不行,,我可以分配一个账户给你测试一下,留个邮箱吧

我没有 E5 账号，不过我还是想麻烦你再试一下，我抓一下日志，看看是哪个地方权限不对。
…

刚刚试了,还是不行,,我可以分配一个账户给你测试一下,留个邮箱吧

<EMAIL_ADDRESS>多谢🙏
On Mon, Jan 4, 2021 at 3:11 PM xiyuesaves<EMAIL_ADDRESS>wrote:

我没有 E5 账号，不过我还是想麻烦你再试一下，我抓一下日志，看看是哪个地方权限不对。
… <#m_-7337969685211491408_>
刚刚试了,还是不行,,我可以分配一个账户给你测试一下,留个邮箱吧
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PoweredByCloud/feedback/issues/2#issuecomment-753800572,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHNXXNUWU6UK5MNZ4OK7L3SYFSZFANCNFSM4VSKPWBA
.

<EMAIL_ADDRESS>多谢🙏
On Mon, Jan 4, 2021 at 3:11 PM xiyuesaves<EMAIL_ADDRESS>wrote:

我没有 E5 账号，不过我还是想麻烦你再试一下，我抓一下日志，看看是哪个地方权限不对。
… <#m_-7337969685211491408_>
刚刚试了,还是不行,,我可以分配一个账户给你测试一下,留个邮箱吧
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PoweredByCloud/feedback/issues/2#issuecomment-753800572,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHNXXNUWU6UK5MNZ4OK7L3SYFSZFANCNFSM4VSKPWBA
.

已发送
BeanYoung<EMAIL_ADDRESS>于2021年1月4日周一 下午3:32写道：

<EMAIL_ADDRESS>多谢🙏
On Mon, Jan 4, 2021 at 3:11 PM xiyuesaves<EMAIL_ADDRESS>wrote:

我没有 E5 账号，不过我还是想麻烦你再试一下，我抓一下日志，看看是哪个地方权限不对。
… <#m_-7337969685211491408_>
刚刚试了,还是不行,,我可以分配一个账户给你测试一下,留个邮箱吧
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/PoweredByCloud/feedback/issues/2#issuecomment-753800572>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAHNXXNUWU6UK5MNZ4OK7L3SYFSZFANCNFSM4VSKPWBA>

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PoweredByCloud/feedback/issues/2#issuecomment-753808997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGC4G6LDVZBYOB3EKO2AGPLSYFVH5ANCNFSM4VSKPWBA
.

已发送
BeanYoung<EMAIL_ADDRESS>于2021年1月4日周一 下午3:32写道：

<EMAIL_ADDRESS>多谢🙏
On Mon, Jan 4, 2021 at 3:11 PM xiyuesaves<EMAIL_ADDRESS>wrote:

我没有 E5 账号，不过我还是想麻烦你再试一下，我抓一下日志，看看是哪个地方权限不对。
… <#m_-7337969685211491408_>
刚刚试了,还是不行,,我可以分配一个账户给你测试一下,留个邮箱吧
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/PoweredByCloud/feedback/issues/2#issuecomment-753800572>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAHNXXNUWU6UK5MNZ4OK7L3SYFSZFANCNFSM4VSKPWBA>

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PoweredByCloud/feedback/issues/2#issuecomment-753808997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGC4G6LDVZBYOB3EKO2AGPLSYFVH5ANCNFSM4VSKPWBA
.


feat: CE-936 Add privacy indication field to complaint

Description
Adds a new checkbox field to the create and edit complaint forms. This new value also needs to be made available in the exported complaint.
Story # CE-936
Acceptance Criteria

[x] update model:

[x] drop privacy_request_ind
[x] add privacy_request

[x] update interfaces to add new privacy property
[x] update redux to use new privacy property
[x] Add privacy request dropdown to create complaint
[x] Add privacy request dropdown to edit complaint
[x] make new privacy filed available for only ceeb
[x] update export complaint template to use new field

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

[ ] Test A
[ ] Test B

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Further comments

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

Changes all look good but there is something not working on the display (and maybe the export I didn't test that)

export is working, but the thing that was off was the when I changed the name of the property vscode didn't change it everywhere, so it wasn't showing up in the ux. I've updated the interfaces to force this change back to working again.


Remove orphan instances provided by base-orphans-0.5

In a similar vein as #337, this commit consolidates some orphan instances that were introduced in GHC 8.0/base-4.9 using base-orphans-0.5. Specifically:

The Functor/Applicative instances for Sum/Product
The Monad instance for (,)

Thanks Ryan, merged.


sending an Accept header when using the REST API fails with a 406

It seems the REST API fails if an Accept header is passed.
e.g. this works fine:
curl -vk -u myuser:mypwd http://localhost:3000/api/v1/user/repos

however this fails with a 406
curl -vk -H "Accept: application/json" -u myuser:mypwd http://localhost:3000/api/v1/user/repos
...
< HTTP/1.1 406 Not Acceptable

it'd be nice to be able to pass an Accept header along too (so its easier to invoke the REST API via a browser in AngularJS ;)

Thanks for the report! I'm not quite sure what's going on with "Accept: application/json", but will take a look.

Many thanks! I've been trying to figure out how to use $http in angularjs and not send any Accept header at all and so far no luck :)

Hmm... it's strange, based on current master version <IP_ADDRESS>4,

curl -vk -H "Accept: application/json" -u myuser:mypwd http://localhost:3000/api/v1/user/repos

returns correct JSON results.

Close for now, but open for continuing discussion.


Flatten nested object

Addresses Issue: #228

[x] Running yarn lint does not trigger any linter errors
[x] The test of the method you have fixed is passing
[x] You have written a new skeleton method for someone else to work on!
[x] You have written tests to accompany your skeleton method

Nice @marcinwysocki !! Glad we finally got a curry skeleton! I might snag that one ;)
So this is kind of a generalized question, but you have written several utility functions while resolving the flattenNestedObject function: getKeyValuePairs, isObject, and hasNestedProperties. All of which I could see being useful outside of the context of this function.
I don't want to make you full fledged methods + accompanying tests for each of these, so I'm thinking I'm going to merge this in as is and create an issue for someone to "Refactor flattenNestedObject" - part of the acceptance criteria being that they must write a full test suite for each function you created in here.
Do you have any thoughts on this?

I think a refactor issue is a good idea.

Great. Thanks!!


Horizontally collapse support

I noticed Bootstrap added horizontal collapse support by adding the class width to the existing collapse, starting with v3.0.
Is there a way to do the same with Angular UI Bootstrap?
I found a post on stackoverflow: http://stackoverflow.com/questions/22654933/angularjs-bootstrap-collapse-horizontaly
The idea was to copy-paste the directive to make a custom one, and changing the direction...
Also found this one that suggest to upgrade the css : http://stackoverflow.com/questions/12498476/twitter-bootstrap-collapse-plugin-direction
It could be great to have this widthclass working!
Thanks,

I have modified the collapse directive and submitted PR #3995
I am also going to see about implementing a new ng-collapse directive that would give more options for collapse animations (ex. collapse from a specific corner)


Update Swagger definition

Requirements

Filling out the template is required.

All new code requires tests to ensure against regressions.

However, if your PR contains zero code changes, feel free to select the checkmark below to indicate so.

[ ] Have you ran tests against this code?

[x] This PR contains zero code changes.

Description of the Change

Update OAuth scopes
Fix api key auth
Update / description and name
Bump version number

Closing due to being superseded (and now hopefully fully automated)


Active record and graphql

Is there any plan or discussions to add graphql support to ActiveRecord ?
I know GraphQL is still a moving target but I wonder if there is any plan about it.

GraphQL home: http://graphql.org/
GraphQL working draft: http://facebook.github.io/graphql/

No, there's no plan implementing it. You can do it and release as an extension.

very rude. :(

https://github.com/tsingsun/yii2-graphql there is an extension already.

@TerraSkye Too bad: the README is in Chinese or Japanese or a language which I can't read...


Exploration of more intuitive .save designs.

We've gotten some initial feedback from alpha users that in the demo (https://github.com/LineaLabs/lineapy/blob/main/examples/Demo_1_Preprocessing.ipynb), the line

... # some work omitted

cleaned_data.filter(
    regex="Neighborhood=.|Gr_Liv_Area|Garage_Area|SalePrice"
).to_csv("outputs/cleaned_data_housing.csv", index=False)

artifact = lineapy.save(lineapy.file_system, "cleaned_data_housing")

Specifically, lineapy.file_system is not intuitive.
I speculate that this is because when the user is thinking about saving the "cleaned_data_housing.csv" from cleaned_data, they have to map that activity to a new concept, lineapy.file_system.
We initially went with lineapy.file_system because it's the most technically succinct way of describing the desired capture mechanism: via side-effects (this is similar to the requirement for asserts https://github.com/LineaLabs/lineapy/issues/449). Note the semantic difference as compared to lineapy.save(cleaned_data, ....), where we save the value of cleaned_data, and the slice would end with the final line that last changed cleaned_data (specifically in the notebook, cleaned_data = cleaned_data.drop(columns=Neighborhood_cats[0])). If we were to slice this into Airflow, the job would be a no-op since no change is persisted or passed to another job.
There are a few different options---all of them will require some form of additional annotation beyond lineapy.save(cleaned_data, ....).

lineapy.save(cleaned_data, include_side_effecs=True). Here, we'll include all the code that has side_effects that uses cleaned_data.

We can even set include_side_effecs to True by default to further reduce friction for this use case. It's hard to imagine when the user would want to exclude side effects. Though one scenario I can imagine is writing a sample to disk, and then writing the whole thing to a SQL database (but it's unlikely?).

Another option is to have a different API that saves the actual value of a something (and implicitly extracts out a process) vs. something that just extracts the process. So something like lineapy.save_pipeline(cleaned_data, "") and lineapy.save_value(cleaned_data, ""). My sense is that this option might be too indirect and confusing.

The benefit here would be that the .get would be clearer---we cannot do a .get on side effects but can on saved values.

We can also save the call, to_csv, and we can do so as a decorator to the line (@linea.save). But I'm not a fan because the user would have do the decoration before the invocation of the call, which doesn't go with our philosophy of deployment after the initial invocations.

Another consideration is fine-grained addressability: the downside of options 1 and 2 is that they don't allow us to now do more fine grained side_effect slicing, where as with lineapy.file_system, we can change it to something like lineapy.file_system("outputs/cleaned_data_housing.csv") or lineapy.file_system(cleaned_data) to be more fine grained. Option 3 allows the fine grained annotation directly.
These are just initial thoughts, please help brainstorm! cc @dorx per your request.

@yifanwu Maybe one more thought:
From my point of view as user it is very important to define not only how to save something, but also how to load it in the next pipeline step. For example, when a dataframe is saved to csv, how do you know how the dataframe is loaded again in the next step?  The same argument is valid for saving and loading ml models of different frameworks (pytorch, onnx, tensorflow, etc.).
A decorator for this feature would be fine for me.


Personal/yazanal/schema manager logging

Description

Switches most Console logging statements to ILogger, except for the ones that format table for output when running as a console app.
Refactors client logic out to a class library.

Related issues
Addresses [issue #].
Testing
Manually tested available, current, and apply commands on a local fhir service

Should we add logger to the BaseSchemaRunner as well?


You need another step to make tis work on MacOS with Docker

I only got the links working after changing the hosts file.
The tip came from this page: https://dev.to/stuartcreed/how-to-make-spatie-ray-work-with-laravel-sail-992

Thanks!


Add permissions request #15

Issue #15

Hi, (belated) thanks for this PR, much appreciated!
Since things have moved a little since your code was pushed I've had to manually apply your changes to the repo.


Is it possible to display a preloader?

Before images have loaded is it possible to show a div which can contain a preloading gif for example?

Yes, this is possible and documented. You can use the lazyloading / lazyload / lazloaded classes for this:
.lazyload {
    opacity: 0;
}

.lazyloading {
    opacity: 1;
    transition: opacity 300ms;
    background: #f7f7f7 url(loader.gif) no-repeat center;
}
.lazyloaded {
    opacity: 1;
    transition: opacity 9ms;
}

Here is a small demo. Please close this issue, if this helped.

I apologise, I missed that in the documentation. That's great, thanks you for your help.


users can now select projects with a toggle button and filter based on columns

use angular grid like henderson said

completed in task #95

[t:150]