First Pass at Non-provable SelfProof Counting + Event

Co-authored by @yunus433
Heavily WIP: updates ZkProgram and the method decorator such that the number of aggregated proofs is tracked and then emitted within a standard event, of which block explorers and other such services could make use.

This currently results in a Cannot read properties of undefined (reading 'toFields') error.
It seems the MetadataEvent constructor has the static toFields method (given it inherits from the Struct factory call). It is present when logged out as well, so I suspect the offending toFields-lacking item is not the MetadataEvent. The complete error + stacktrace doesn't make clear where this offending item lives:
compile (trivial zkapp)
/Users/harrysolovay/Desktop/o1js/dist/node/bindings/compiled/_node_bindings/o1js_node.bc.cjs:6761
         throw err;
         ^

TypeError: Cannot read properties of undefined (reading 'toFields')
    at assertEqualImplicit (o1js/dist/node/lib/provable/provable.js:203:16)
    at Object.assertEqual (o1js/dist/node/lib/provable/provable.js:195:16)
    at PublicKey.assertEquals (o1js/dist/node/lib/provable/types/circuit-value.js:104:18)
    at Object.wrappedMethod [as value] (o1js/dist/node/lib/mina/zkapp.js:299:42)
    at async TrivialZkapp.wrappedMethod (o1js/dist/node/lib/mina/zkapp.js:144:38)
    at async file:///Users/harrysolovay/Desktop/o1js/dist/node/lib/mina/zkapp.js:853:38
    at async Object.constraintSystem (o1js/dist/node/lib/provable/core/provable-context.js:64:9)
    at async TrivialZkapp.analyzeMethods (o1js/dist/node/lib/mina/zkapp.js:851:60)
    at async TrivialZkapp.compile (o1js/dist/node/lib/mina/zkapp.js:464:27)
    at async file:///Users/harrysolovay/Desktop/o1js/dist/examples/zkapps/simple-zkapp-with-proof.js:65:51

Node.js v21.7.3

The complete error + stacktrace doesn't make clear where this offending item lives:

I get the following info from the stacktrace:

It comes from PublicKey.assertEquals(), so the offending item is a public key which is undefined
The assertEquals() is called inside wrappedMethod() in zkapp.ts. with source maps it would be easy to see on which line, but since there is only one assertEquals() on public keys in that file it's probably that one

Heavily WIP: updates ZkProgram and the method decorator such that the number of aggregated proofs is tracked and then emitted within a standard event, of which block explorers and other such services could make use.

The way I understood the original idea was that we wanted to track the number of times a proof was recursively verified? like, if there is a chain of proofs P0 (base proof) -> P1 (recursively verifies P1) -> P2 (recursively verifies P2), and P2 gets sent to the chain, we wanted to expose 3 as the number of aggregated proofs.
In the current approach, you're not doing that, as you're just counting SelfProof arguments, but a method with such an argument can be called on its own proof again & again.
Another problem I have with the WIP apprach is that you're modifying the default ZkProgram that everybody uses. Would be nicer to implement it as wrapper for ZkProgram so that it's optional to expose something that leaks private inputs.

@mitschabaude thank you for the feedback on this draft. It does seem that our brute solution neglects a wide range of considerations, such as the possibility of a recursive proof being created in and loaded in from an out-of-band process. Closing this draft for now.


Fix protect_against_docker_hang

Check not only for plugin-v2 but also for plugin-v1, as both create the same plugin sockets (weave.sock and weavemesh.sock).
Should fix flakiness in 80* smoke tests.

[ (test) actually evaluates the expression as a string of non-zero length:
#!/bin/sh
foo() {
	false
}

if [ ! foo ] && [ ! foo ] ; then
	echo "NOT foo"
else
	echo "foo"
fi

outputs: "foo" which is unexpected as the intent really is:
#!/bin/sh
foo() {
	false
}

if ! foo && ! foo ; then
	echo "NOT foo"
else
	echo "foo"
fi

which outputs "NOT foo" as expected.

Fixed in master in b46ccc0


I am interested in using your game on my website. Could you add a license file like the MIT one in this pull request so I know how you want your software used?

Thanks, for your interest. I am currently working on how to better it using LibGDX. So you could expect some additional features to the game very soon. especially a more fun 2D animation.

Cool. Thanks for sharing!


Fix Obstacle Distance Laser Scanner Node

When manually constructing a QoS profile with only the depth value, or
only passing a depth value to the QoS parameter when creating a
subscription, the subscription will not pick up LaserScan messages.
Using the System Default QoS profile works.

It seems like there may be a deeper ROS2 bug here, I haven't seen this for other messages only LaserScan, and it's strange how using a manually created QoS profile does not work.
Fixes #57
Testing

Tested running locally with simulation_ws and it previously did not pick up scan messages unless they were re-echo'd on the CLI (see ROS-2968 in SIM). After this change the scan messages are picked up correctly.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Changed to SENSOR_DATA as recommended by @emersonknapp as it's the best QoS policy for unreliable sensor data like this and has the least strict QoS policies. There's something in the default QoS policy that is too strict for this lidar scanner.


Command line execute SQL files

Executing command line SQL commands with configuration of connection which is defined in application.

Resolved


#558 Japan

Japan added new event chain.
Minor altering of previous existing events to fit in with the new event chain.
Minor altering of the buyback decision to work correct with new event chain.
Minor altering of province history to fit in with new event chain.
Added picture for use in new events.
Added localisation for new events.

merged manually,.


Fix namespace mismatch during installation

What this PR does / why we need it:
Using the SDK and pass namespace to installation does not take effect, helm show the ns is different to the real deployed ns.
namespace shoud be passed to kube.Client anyway.
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[x] this PR has been tested for backwards compatibility

Closing this as abandoned. If you can come back and address the change request, please reopen the PR.


Make it easier to insert numbered and bulleted lists

We should make it easier to insert correct formatting for numbered and bulleted lists.

Pressing "Enter" after a line with list formatting should insert the appropriate list formatting on the next line, i.e. the next number (e.g. "2. " or "2) ") for numbered lists, and "* ", "- " or "+" for bulleted lists. The formatting inserted should have a trailing space and match the previous line.
Pressing "Enter" on a line with list formatting when the current item is blank should remove the list formatting on the current line, and not insert a line break. You can see an example of this behavior on GitHub.
We should add buttons below the compose box for inserting formatting for numbered and bulleted lists.

Tooltip: Numbered list; Icon: https://fontawesome.com/v4.7/icon/list-ol
Tooltip: Bulleted list; Icon: https://fontawesome.com/v4.7/icon/list-ul

Clicking the button should insert the appropriate formatting. This is especially useful if multiple lines are selected, in which case each line should be given the appropriate list formatting.
Note that different apps behave somewhat differently if part of a line is selected or the cursor is in the middle of a line. Once this issue is being worked on, we can discuss the precise details of what to do in these situations on chat.zulip.org.
Related issue: #20302

@zulipbot claim

This issue is still being worked on by me.  I will soon post a PR with an implementation to be discussed.

@alya I have now implemented what I gathered from your issue. Please have a look at the WIP pull request and comment on the current implementation. One thing I have noticed is the way to handle numbered lists. The way I understand it based on the documentation (https://zulip.com/help/format-your-message-using-markdown), is that you number it by writing "1. " at the beginning of each row. This is different from what you stated in your issue.

Thanks for working on this, @isakhagg !
If you look at the documentation again, you'll notice that Markdown supports both increasing the numbers and writing 1. at the beginning of each row. For this issue, we want to increase the numbers (as described), as that will be more expected/helpful for users.

@alya Thanks for the comment! I will have a look at how to change the current implementation.

@zulipbot claim

@zulipbot claim

@zulipbot claim

Hello!
Progress Update: I have setup the environment, and am experimenting the code previously pushed for this issue.

Hello, I'm still working on the issue. I'm currently reading and rewriting portions of @isakhagg 's code.
I've found bugs for the bulleted and numbered points when enter-sends is enabled. I'm trying to fix that.

@zulipbot claim


!toggle_pass broken

!toggle_pass does not print to in-game chat and doesnt toggle game pass

!toggle_pass has been changed to !no_pass and !pass to enable with -t parameter to enable a timer. This hasn't been update in the wiki yet.


does it works with Djagno Server

Hi,
I was wondering whether it works with django server (a server bundled with Djagno). I have turned on settings
CORS_ORIGIN_ALLOW_ALL = True
CORS_ALLOW_CREDENTIALS = True

But I do't see any headers related cors on response

I don't see the see the headers neither.

CORS_ORIGIN_ALLOW_ALL is set to True but the header I got back is the following.  I don't see Access-Control-Allow-Origin in the header at all.
$ http --headers 'http://localhost:8000/api/post/?lat=37.7019713&lng=-122.420255'
HTTP/1.0 200 OK
Allow: GET, HEAD, OPTIONS
Content-Encoding: gzip
Content-Length: 2818
Content-Type: application/json
Date: Sat, 29 Aug 2015 22:01:49 GMT
Server: WSGIServer/0.2 CPython/3.4.3
Vary: Accept, Accept-Encoding, Cookie
X-Frame-Options: SAMEORIGIN

Looks like httpie doesn't automatically include Origin in the http request header.  When there's no Origin, Access-Control-Allow-Origin is not returned in the response header.  @s2krish you might want to check your request header to see if Origin is there.

Seems it works. It adds the header dynamically. For instance I setup a virtualhost in my local with name if cors-headr-test.lan. I made ajax request to http://<IP_ADDRESS>.1:800/my/api from http:// cors-headr-test.lan/, it has added cors header.
Access-Control-Allow-Credentials:true
Access-Control-Allow-Origin:ttp:// cors-headr-test.lan

So closing this.


Set up continuous integration

https://github.com/Igalia/snabbswitch/pull/111

Done!

Three cheers for @andywingo and @aperezdc . :)


defrecord in instarepl produces warnings

When I eval or Instarepl a defrecord in a .clj:
  (defrecord Test [])

I get a bunch of warnings in the console:
  WARNING: Symbol IHash is not a protocol at line 4
  WARNING: Symbol ILookup is not a protocol at line 4
  WARNING: Symbol IAssociative is not a protocol at line 4
  WARNING: Symbol IPrintWithWriter is not a protocol at line 4
  WARNING: Symbol IRecord is not a protocol at line 4
  WARNING: Symbol ICollection is not a protocol at line 4
  WARNING: Symbol ISeqable is not a protocol at line 4
  WARNING: Symbol ICounted is not a protocol at line 4
  WARNING: Symbol IEquiv is not a protocol at line 4
  WARNING: Symbol IWithMeta is not a protocol at line 4
  WARNING: Symbol IMeta is not a protocol at line 4
  WARNING: Symbol IMap is not a protocol at line 4

This isn't a critical issue, but since it fills the console every time I reevaluate the file it's a bit annoying.
This may be down to my limited understanding of records, protocols, and the warning system. I have not seen this behavior with other REPLs, though I suppose they may be suppressing the warning. I've looked for any information on this and no one else seems to have mentioned this exact issue.

This issue has moved to https://github.com/LightTable/ClojureInstarepl/issues/10 as the instarepl is now its own plugin

This issue has moved to https://github.com/LightTable/ClojureInstarepl/issues/10 as the instarepl is now its own plugin


Random 'j' characters inserted when renaming training samples.

@nebgnahz Any ideas? Have you seen this?

Good call... this was an keyboard output stream. Never mind.


Support Volume Type

The operations should be supported for starters:

http://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.Client.describe_volumes
http://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.Client.create_volume
http://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.Client.attach_volume
http://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.Client.delete_volume
http://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.Client.detach_volume

#113


Latest release in debian repository stuck at 1.0.5

Expected behavior
apt-get update gauge should update it to the most recent available version, which is 1.0.7 assuming it's supposed to be an official release.
Actual behavior
it doesn't update past 1.0.5
Steps to reproduce
$ date
Fri 18 Oct 2019 02:56:18 PM CEST
$ sudo apt-get update
Hit:1 http://ppa.launchpad.net/numix/ppa/ubuntu disco InRelease
Hit:2 http://ftp.nl.debian.org/debian buster InRelease
Ign:3 http://dl.google.com/linux/chrome/deb stable InRelease
Hit:4 http://security.debian.org/debian-security buster/updates InRelease
Hit:5 http://ftp.nl.debian.org/debian buster-updates InRelease
Hit:6 http://dl.google.com/linux/chrome/deb stable Release
Ign:7 https://dl.bintray.com/gauge/gauge-deb stable InRelease
Get:8 https://dl.bintray.com/gauge/gauge-deb stable Release [1,840 B]
Hit:9 https://download.docker.com/linux/debian buster InRelease
Fetched 1,840 B in 1s (2,706 B/s)
Reading package lists... Done
$ sudo apt-get upgrade gauge
Reading package lists... Done
Building dependency tree
Reading state information... Done
gauge is already the newest version (1.0.5).
Calculating upgrade... Done
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
$ apt-cache madison gauge
     gauge |      1.0.5 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      1.0.4 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      1.0.1 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      1.0.0 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.9.8 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.9.7 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.9.5 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.9.4 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.9.3 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.9.2 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.9.0 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.8.5 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.8.4 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.8.3 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.8.2 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.8.1 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.8.0 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.7.0 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.6.2 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.6.1 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.6.0 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages
     gauge |      0.5.0 | https://dl.bintray.com/gauge/gauge-deb stable/main amd64 Packages

Downloading it manually and installing with dpkg works:
$ sudo dpkg -i ~/Downloads/gauge-1.0.7-amd64.deb
(Reading database ... 221008 files and directories currently installed.)
Preparing to unpack .../gauge-1.0.7-amd64.deb ...
Unpacking gauge (1.0.7) over (1.0.5) ...
Setting up gauge (1.0.7) ...

Gauge version
$ gauge --version
Gauge version: 1.0.5
Commit Hash: 562f036
Plugins
html-report (4.0.8)
java (0.7.2)
screenshot (0.0.1)
xml-report (0.2.1)

@kimholan The latest released version of Gauge is 1.0.6. Can you please try updating it again and see if the package is available?

@nehashri : thanks a bunch. it updated from the debian repository now:
$ sudo apt-get upgrade gauge
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... Done
The following packages will be upgraded:
  gauge
1 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Need to get 4,427 kB of archives.
After this operation, 69.6 kB of additional disk space will be used.
Do you want to continue? [Y/n] Y
Get:1 https://dl.bintray.com/gauge/gauge-deb stable/main amd64 gauge amd64 1.0.6 [4,427 kB]
Fetched 4,427 kB in 0s (11.0 MB/s)
Reading changelogs... Done
(Reading database ... 205031 files and directories currently installed.)
Preparing to unpack .../archives/gauge_1.0.6_amd64.deb ...
Unpacking gauge (1.0.6) over (1.0.5) ...
Setting up gauge (1.0.6) ...


Unclear if/how implementation/usage of: Ad Hoc, Skippable, Ad Hoc Autostart, Ad Hoc Sub process

Description
The docs outline multiple references to the usage of

Ad Hoc,
Skippable,
Ad Hoc Autostart,
Ad Hoc Sub Processes

but details about implementation and actual API usage is missing.  For example, when applying skippable (or unchecking skippable) on a User Task, has what effect and how does a user interact with this aspect?  Same types of questions for the others.
Thanks!
Implementation ideas
No response

For 1,3,4 yes that doc exists but it misses the implementation part:
@fjtirado Skip is a phase as described here: https://docs.jboss.org/kogito/release/latest/html_single/#con-task-lifecycle_kogito-developing-process-services so initial interpretation of the modeler UI would suggest that if I disable/uncheck "Skippable" it would prevent the Skip Phase from being activated/used.  But your explanation suggests that is not the case.

@StephenOTT , the problem is the possible semantic interpretation of skipable flag. If we modify provided skip life cycle phase implementation to thow exception if skipable is equal to false, then  we will limit user margin of manoeuvre.
Imagine an user want to use skipable to flag those task which can be automatically closed after a certain time by a custom bot, but do not want prevent to explicitly skip them by an user with enough permission. He will have to override default skip life cycle.
I will prefer to actually remove skipable altogether rather than impose a particular interpretation

@fjtirado 100% agree... The skippable is a great concept.  But unless you want to implement it, it is just confusing and creates mental connections that don't correlate with actual implementation.

@Kris wdyt? Should we remove Skipable? Provide a Skip implementation that uses it? Leave things as they are and documment the current approach?


Active Endpoints Response should be in ascending order

The response to an active endpoints request should be in ascending order of endpoint id.
At the moment it's not in ascending order.

I cannot fix this problem (?) because it relies on the Zstack 1.2 implementation.

Thanks for the quick reply!
The problem here is that coordinators such as Hubitat and SmartThings (where I am planning on using devices with your firmware) tend to use the first endpoint returned in the list to set the "main" endpoint for a device.  This can make things a little screwy if it's not actually the first endpoint.
The spec does indicate that the list should be in ascending order - https://zigbeealliance.org/wp-content/uploads/2019/11/docs-05-3474-21-0csg-zigbee-specification.pdf
Perhaps the Zstack returns them in whatever order you are storing / adding them when the are configured in the configuration software?

I've implemented a workaround in the latest version related to this problem.


New command: Get information about the specific Microsoft Teams team channel

Gets information about the specific Microsoft Teams team channel

command: teams channel get -c|--channelId <channelId> -i|--teamId <teamId>
teamId: The ID of the team to which the channel belongs
channelId: The ID of the channel for which to retrieve more information

Hey @waldekmastykarz , please assign that to me. Thanks!

All yours! 💪


HTTP 5xx When trying to merge files

Hi,
I'm unable to get the webapp to work. I'm trying to merge ~7 .xopp files - and every time I hit merge the server returns a HTTP 5xx error. Usually it's an HTTP 503, although I've seen 502 and 520 as well. Response server is cloudflare.
Sample response/request headers:

Sorry for the rather uninformative post, I unfortunately don't have the time to investigate much further but wanted to at least let you know that there's an issue.

@EthanGraber how big are your files? I believe I put a limit on file sizes like you can probably send max 35MB worth of files currently I think.

HTTP 5xx error.

Hmm those are server errors. Maybe the server I am hosting on got down or something? Not sure.

@EthanGraber I am just curious does an Error popup appear and says "Unexpected Error occured" or something similar?

couple of kilobytes

That's great!

Yep, unexpected error occured

Thanks for the feedback ❤

I'll try again later

Thank you!

Alright, I gave it another try today. Still getting HTTP 503 errors when I try to merge. I tried merging a subset of the files just to see if that would work, and that gave me an HTTP 507 (Insufficient Storage) response. Odd that it didn't respond with that the first few times.
The files are ~8.1MB combined, well under the 35MB limit you mentioned. I also don't think they're actually being transferred to the server - I monitored the network tab while I was testing and my client only ended up transmitting a grand total of 1.3 kB.
If you're still interested in having the files themselves, I can send them over, I was able to combine them manually on my end with no problem though / I think this is a server related issue more than a file-specific issue.

That's bad to hear.

If you're still interested in having the files themselves, I can send them over

Yes that would be great for testing, thanks!

I think this is a server related issue more than a file-specific issue.

I'll look into it asap though currently I have exams going so I might not be able to check it this week


Truthy value not recognized by CFA as a constant condition in loops

Bug Report
🔎 Search Terms
while loop, infinite, cfa, control flow analysis
🕗 Version & Regression Information
This is also the 'Nightly' version in the playground: nightly TS playground

This is the behavior in every version I tried

⏯ Playground Link
Playground link with relevant code
💻 Code

type DigraphNode = {
  data: {
    key: string;
  };
  parent?: DigraphNode;
};

export function getStringPath(node: DigraphNode): string[] { // Function lacks ending return statement and return type does not include 'undefined'.(2366)
  let currentNode: DigraphNode = node;
  const path = [];

  while (currentNode) {
    // do not include root's key
    if (!currentNode.parent) {
      return path.reverse();
    }
    path.push(currentNode.data.key);
    currentNode = currentNode.parent;
  }
}

🙁 Actual behavior
I get an error:

Function lacks ending return statement and return type does not include 'undefined'.(2366)

🙂 Expected behavior
I shouldn't get an error because currentNode is always truthy, this should work in the same way as while (true) (TS playground).

Interesting case. Right now this check is almost entirely syntactic; it does consider exhaustive switches but otherwise doesn't use CFA types at all.
This is probably a Won't Fix unless the fix is extremely cheap, since the logic of "this is the same as a while (true)" strongly implies that the program should just be written that way.

Wouldn’t it just be the same check as the one that produces “this condition will always be true because…” diagnostics for if statements?

This is probably a Won't Fix unless the fix is extremely cheap, since the logic of "this is the same as a while (true)" strongly implies that the program should just be written that way.

I agree that this is how it should be written and I've rewritten it this way before opening this issue. However, I think that the compiler shouldn't rely on this - that's why I've created an issue. A diagnostic like the one mentioned by @fatcerberus would also be nice but not necessary.

Wouldn’t it just be the same check as the one that produces “this condition will always be true because…” diagnostics for if statements?

Interestingly, this particular situation doesn't produce such a diagnostic even with the if statement and it suffers from the same issue related to the return type:
type DigraphNode = {
  data: {
    key: string;
  };
  parent?: DigraphNode;
};

export function getStringPath(node: DigraphNode): string[] { // Function lacks ending return statement and return type does not include 'undefined'.(2366)

  let currentNode: DigraphNode = node;
  const path = [];

  if (currentNode) {
    path.push(currentNode.data.key);
    return path.reverse();
  }
}


Update CircleCi base image

Update to the latest base image, which has a newer version of Go, and a newer version of our build-go-binaries script. This supports building fetch for Darwin/ARM, which hopefully fixes #82.

Kicked off internal release to check if the build/release steps pass: https://github.com/gruntwork-io/fetch/releases/tag/v0.0.0-internal-test

OK, build passed:

Thanks for the review! Merging now.

https://github.com/gruntwork-io/fetch/releases/tag/v0.4.2


HillCipher feature added.

I have created a new folder named 'Hill Cipher' in the projects folder. It is created with Python and MySQL.
It contains the readme file which can be used to understand about the code written.

Hey Mrinank,
Just wanted to give you a heads up that I've made a pull request on issue
#456. It would be great if you could take a look at it when you get a
chance.
Also, I wanted to let you know that this PR is the last one I need to
complete my 4 PRs. Almost there!
Thanks in advance,
Payal
On Sat, Oct 7, 2023 at 9:56 PM github-actions[bot] @.***>
wrote:

Greetings! This is an automated message from GitHub Actions. 🤖 Your pull
request has been received and is awaiting for a review by the repository
owner or a maintainer. This may take some time, so please be patient.
While you wait, you can continue to work on other issues or pull requests,
or explore the project further. Or you can simply relax and enjoy your day.
Thank you for your contribution to this project! You are awesome! ⭐
—
Reply to this email directly, view it on GitHub
https://github.com/Mrinank-Bhowmick/python-beginner-projects/pull/456#issuecomment-1751750942,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT6EWI6KVNX4PPZJBYLVGVDX6F7DVAVCNFSM6AAAAAA5XA3YNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJRG42TAOJUGI
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


Refactor and add structure to browser handling

This adds some more structured handling of browser configuration, to prepare for the next PR which adds remote webdriver support. Progress towards https://github.com/Polymer/tachometer/issues/48
Also fixes https://github.com/Polymer/tachometer/issues/58 where only Chrome could be launched.
Also fixes a bug where exceptions wouldn't exit the process, because the server continued to listen indefinitely.
cc @sorvell

(Sorry for churn, pushed some more commits and updated description)


Deploy UI5 project stuck

HI,
I've very simple UI5 application which I build with the MTA Build tool as suggested in the readme,
I try to deploy it to SAP Cloud Foundry
The problem is after two min the deployment process stuck with starting the application and nothing happen.
Any idea what it could be ?
Btw, in this case could you improve the logs to reflect the status, in fact the application is not starting
New MTA version: 0.0.1
Deployed MTA version: 0.0.1
Building cloud deployment model...
Building cloud undeployment model...
Validating services to create...
Validating applications to deploy...
Adding domains...
Deleting unused reserved routes...
Creating or updating services...
Preparing applications deployment...
Checking application "ui5"...
Application "ui5" exists
Updating application "ui5"...
Application "ui5" is not modified and will not be updated
Uploading application "ui5"...
Matched files count: 17
Matched resources processed, total size is 7,155,484
Upload status: queued
Upload status: failed
Checking upload status of application "ui5"...
Scaling application "ui5"...
Actions that should be executed on application ui5: [STOP, STAGE, START]
Stopping application "ui5"...
Starting application "ui5"...
Best Regards,
Rayn

Can't tell you much without seeing the logs.
So can you send us deploy-service logs to make more detailed analyze of the problem ?
Regards,
Kristian

Hi Rayn,
Any updates from your side?
Regards,
Mihail


Fix condision for error when duplicate link titles

CheckForDuplicateTitles() returns true if there are duplicates.
Therefore, if it is not duplicated, an error occurred.

Given the recent schema updates/cleanup, the generator produces a correct client either way; there are currently no duplicates in the schema - yay!
With that said, as the author of this recent addition to the generator, and on reading it now a month or so removed from the change, I am less than comfortable with it readability-wise. If I may suggest, let's:

change the name of the method to something like AreTitleLinksUnique, and
change the return to something like return len(uniqueLinks) == len(s.Links)
leave your update above the same (except change the method being called)

That way we are stating exactly what we expect to be true and the code checks for a match with that expectation.
@theoden9014 @cyberdelia how doe that sound?

Thank you for your quick reply !
I agree with your idea, It's hard to understand what's returned if the method name is "CheckFor".

👍 LGTM


remove redundant property in required error message

For required properties, the error message is currently:
"data.name should have required property .name"
This works for other validations, where the property is present, but required is a special case that indicates the property is not there. The above message is somewhat confusing. I've simply updated the required errors to read:
"data should have required property .name"

I see the failing assertion.. if this change is acceptable, then I suppose that assertion can be removed for the required validation.

this is contradictory to the wish to have a separate error for each required property and to have missing property in dataPath (e.g. https://github.com/epoberezkin/ajv/issues/18). I think the better solution is to change the message to data.name is a required property, i.e. to change the message rather than the dataPath

Fair enough! All set - let me know if I've missed anything.

Thanks!
You've done it all correctly. I am just still thinking how to address this issue better... The old messages are better in case they are used separately (that's how I use them). In case they are connected using errorsText they are obviously wrong - that's what your fix is improving... To be honest I wasn't fully sure about the changes that addressed #11 and #18. But I didn't think about the errorsText at the time (not sure if it even existed).
Your first solution may be better and in this case missingProperty can be made available as some additional parameter to those who needs them. There already exists an additional property of the error object params that is added if i18n option is used - these params are used to construct internationalised messages. So maybe I should have dataPath pointing to the object rather than to the property and and rename i18n option to params - they can be useful for other use cases too.
Reverting dataPath to point to object for both required and additionalProperties keywords will be a change that breaks backward compatibility for some users, but probably it is worth it because it is more correct - for these keywords the error is in the object, not in the property.
What do you and others (@blakeembrey, @shachr, @cdmcnamara) think?

I had trouble parsing everything, so I might follow up, but some initial thoughts. I prefer required and additionalProperties and everything else to use the exact path to the issue (the property that is missing, not the object). I think it's more useful in generating error messages and parsing - in any case, a consumer can always split it and go backwards but it's trickier to get the path from the message itself.
I guess I'll have to relook into params now, I haven't updated to use the latest i18n code to evaluate it properly.

I merged the message change while I am thinking what to, message is wrong anyway in the current situation. I am thinking more and more that dataPath should point to the object (that's where the problem is) and some extra property should have the name of the missing (or additional) property.

@blakeembrey another argument for switching dataPath to point to object rather than to property is that at the moment dataPath and data in error point to different things (data - object, dataPath - property). In all cases apart from required and additionalProperties they point to the data currently validated... So I think in 2.0 I'll preserve existing behaviour with some option (for backwards compatibility) and the default behaviour will be to make dataPath point to the object and report the property in params.
Still not sure what's the better thing to do :)


Can't rebuild - version string is incorrectly marked as wrong

I'm trying to rebuild https://argus.scylladb.com/test/854158d6-db2f-4365-a140-553fac97e1b4/runs?additionalRuns[]=aaca5e42-b630-4640-a645-a5dec65a1143 and getting the following:

Has the version string been changed in SCT/scylla? I see there's a dash - now instead of ~, which iirc wasn't a thing before.

Actually, the test failed previously precisely because the version string is wrong - it should have tilde instead of a dash before rc marker.

Has the version string been changed in SCT/scylla? I see there's a dash - now instead of ~, which iirc wasn't a thing before.

That's a bit odd. Why would anyone use ~ sign?

Actually, the previous test failed precisely because the version string is wrong - it should have tilde instead of a dash before rc marker.

Nevermind, that is wrong - it seems that promotion in Azure didn't happen? Both - and ~ don't work.

Has the version string been changed in SCT/scylla? I see there's a dash - now instead of ~, which iirc wasn't a thing before.

That's a bit odd. Why would anyone use ~ sign?

Scylla reports it that way, but it looks like we do it both ways - I'll add - as a valid version separator then.

@yaronkaikov claimed promotion did take place.

For now you can switch to All parameters tab and build from there - validation doesn't happen on that tab. I'm trying to start it again with 2022.2.0-rc2

For now you can switch to All parameters tab and build from there - validation doesn't happen on that tab. I'm trying to start it again with 2022.2.0-rc2

Aand it doesn't work:
13:10:19  ValueError: Azure Image for scylla_version='2024.2.0-rc2' not found in eastus

For now you can switch to All parameters tab and build from there - validation doesn't happen on that tab. I'm trying to start it again with 2022.2.0-rc2

Aand it doesn't work:
13:10:19  ValueError: Azure Image for scylla_version='2024.2.0-rc2' not found in eastus

@yaronkaikov ?

@k0machi where are you trying to take the image? from the marketplace?

@k0machi where are you trying to take the image? from the marketplace?

I'm not sure, I don't think we are using marketplace to retrieve image for SCT. @fruch @soyacz

@yaronkaikov
yes we are taking it from  CommunityGalleries:
+---------------------------------------------------------------------------------------------------------------------------------------------------------+
|                                                             Azure Machine Images by version                                                             |
+---------+----------+---------------------------------------------------------------------------------------------------------------------+--------------+
| Backend | Name     | ImageId                                                                                | CreationDate |
+---------+----------+---------------------------------------------------------------------------------------------------------------------+--------------+
| Azure   | 2024.2.0 | /CommunityGalleries/scylladb-7e8d8a04-23db-487d-87ec-0e175c0615bb/Images/scylla-enterprise-2024.2/Versions/2024.2.0 | N/A          |
+---------+----------+---------------------------------------------------------------------------------------------------------------------+--------------+

but there's only 2024.2.0, no version named 2024.2.0~rc2

@yaronkaikov
yes we are taking it from CommunityGalleries:
+---------------------------------------------------------------------------------------------------------------------------------------------------------+
|                                                             Azure Machine Images by version                                                             |
+---------+----------+---------------------------------------------------------------------------------------------------------------------+--------------+
| Backend | Name     | ImageId                                                                                | CreationDate |
+---------+----------+---------------------------------------------------------------------------------------------------------------------+--------------+
| Azure   | 2024.2.0 | /CommunityGalleries/scylladb-7e8d8a04-23db-487d-87ec-0e175c0615bb/Images/scylla-enterprise-2024.2/Versions/2024.2.0 | N/A          |
+---------+----------+---------------------------------------------------------------------------------------------------------------------+--------------+

but there's only 2024.2.0, no version named 2024.2.0~rc2

yes, this is because Azure won't accept -rcX or ~rcX so until we have an official release, we override this so 2024.2.0 is now rc2


chore(release): Publish 6.1.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.321%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
109

Relevant Lines:
112

💛  - Coveralls


Make baselines faster by not writing out unneeded files

Two changes here
The first is to not write out baseline files to /local when the content is already identical to what's in /reference. Instead, we write out new files when they've changed, and write .delete files when the file exists in /reference but not in /local
The second is to update the baseline-accept tasks to simply copy with overwrite, plus treat .delete files specially so they delete the destination file. These are now basically "soft by default" so you can run arbitrary subsets of tests and baseline-accept without having to worry about nuking your reference folder.
Locally, these brought the total time to run the 'conformance' suite from 1m33s to 0m54s. Might even be a bigger gain on non-SSD machines (not sure what travis is using).

:bell:

:+1: if you're satisfied with it. I don't see any issues. We could probably do even better if we just wrote all the files-to-delete out to a single manifest file all at once (rather than a smattering of .delete files), but that needn't be considered in this PR.

👍

I have run jake rutests locally and cannot get diff by jake diff. It seems jake will always remove local baseline and no diff can be produced.


Why TrapKind::Host is commented out?

Hello,
why this line is commented out https://github.com/baidu/rust-sgx-sdk/blob/master/third_party/wasmi/src/lib.rs#L232?

Months ago when I was porting wasmi I ran into a problem with serialization of HostError. It alert me that if an enclave could say 'Hey, you make me an HostError instead of Interpreter Error' because it may leaks something. So I chose to comment it out.
I would like to say that it's a personal decision based on my understanding of SGX security model.
But I could implement HostError if it's highly desired...

Thank you very much. No need to implement, just being curious.


Comment about active users

Hi, we're an EDU tenant where students have Teams licences but it's not enabled at the tenant level for them. They are showing up in the active user report with 0 activity in all fields. We had around 800 of them so I emailed a sample to ask how they were using Teams. Replies where in two camps, those that could see the tile in the launcher and clicked it, and those who had a device with it preinstalled and it loaded on start up. All reported the same splash screen with words to the effect 'contact your administrator'. I did raised a ticket with premier support and they advised raising a user voice. I just wanted to share this information with you.

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

ID: 78ade1a4-092c-1132-44df-ee2c683fdd33
Version Independent ID: 396b1aee-2f40-9895-9b00-07e38d11d31b
Content: Microsoft Teams analytics and reporting - Microsoft Teams
Content Source: Teams/teams-analytics-and-reports/teams-reporting-reference.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Hi, I've done that already. Microsoft Premier Support advice (via a ticket) was to raise a Uservoice. And here's the link https://microsoftteams.uservoice.com/forums/555103-public/suggestions/39250318-teams-usage-report-shouldn-t-include-active-user


Balance set back to 0 when harvesting food

when harvesting food, the amount of food i had in my balance gets replaced by the harvest amount.
Here i had 1.000.000 fish, then only 84 after harvesting.

this happened to me

closing as this being changed


next Cartographer Open House Hangout

Dear @cschuet
Will there be another Cartographer Open House Hangout? If so, when?
Thanks :)

Any response on this? This repo seems to have gone silent, is it being maintained?

See https://github.com/cartographer-project/cartographer/issues/1675#issuecomment-710781559 for an update. We haven't discussed the open house format yet, but it might have a comeback.


Create a feature for suspending multiple instances in Google Cloud.

As the title suggests, this feature is very useful for automatically suspending several instances when they have not been used for a while. Additionally, this feature can leverage the multi-server BungeeCord functionality.

You mean use the ability of BungeeCord that enable switching between multiple servers?

Hi @yasridho - thanks for creating the issue and I'm sorry that I've not replied since. However I'm not actively working on this project anymore. If you would like to contribute a PR I would be very happy to accept :)


MAINT: remove Python 3.9 from release action

Summary by Sourcery
CI:

Remove Python 3.9 from the release workflow matrix.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.31%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
34116

Relevant Lines:
36958

💛  - Coveralls


Make kNN Imputer installable via pip

PR for some basic changes to make kNN imputer installable via pip

hi, @robinsonkwame , thanks for your work!
Have u already tested on your machine, still work? :)

@bwanglzu I've tested my fork master several times and it works as expected for me, let me know if it doesn't for you...


densities for DHF and GHF objects

Currently no rdm for DHF and GHF objects is implemented and the program crashes.
I would prefer if either an error message was printed telling the user that it has not yet been implemented, or (much better) it was implemented.

Sorry, it seems the issue is with pyscf.dft.numint.eval_rho, which crashes. It there a way to make it handle DHF or GHF density matrices?

Sorry, it seems the issue is with pyscf.dft.numint.eval_rho, which crashes. It there a way to make it handle DHF or GHF density matrices?

A possible solution for GHF objects might be something like this:
dm = np.asarray(dm)
nso = dm.shape[-1]
nao = nso // 2
dm_a = dm[...,:nao,:nao].real
dm_b = dm[...,nao:,nao:].real
dm = dm_a+dm_b
in the numint.eval_rho function.

A possible solution for GHF objects might be something like this:
dm = np.asarray(dm)
nso = dm.shape[-1]
nao = nso // 2
dm_a = dm[...,:nao,:nao].real
dm_b = dm[...,nao:,nao:].real
dm = dm_a+dm_b
in the numint.eval_rho function.

The density in the GHF case is a spinor. So while that is creating a kind
of density it is not the actual 1-particle density, so this is not
the right code to put into pyscf itself, although if for your purposes this
is what you want, you should use it.
As should be clear, PySCF does not currently support GKS, DKS. This is
because the exchange-correlation functional gradient term is more
complicated for spinors and has not been implemented.
On Thu, Jan 21, 2021 at 7:05 AM AStegm<EMAIL_ADDRESS>wrote:

A possible solution for GHF objects might be something like this:
dm = np.asarray(dm)
nso = dm.shape[-1]
nao = nso // 2
dm_a = dm[...,:nao,:nao].real
dm_b = dm[...,nao:,nao:].real
dm = dm_a+dm_b
in the numint.eval_rho function.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/801#issuecomment-764706380, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN5NRBQNG6WIDKX7OJV3WDS3A7FFANCNFSM4WMZ47PA
.

The density in the GHF case is a spinor. So while that is creating a kind
of density it is not the actual 1-particle density, so this is not
the right code to put into pyscf itself, although if for your purposes this
is what you want, you should use it.
As should be clear, PySCF does not currently support GKS, DKS. This is
because the exchange-correlation functional gradient term is more
complicated for spinors and has not been implemented.
On Thu, Jan 21, 2021 at 7:05 AM AStegm<EMAIL_ADDRESS>wrote:

A possible solution for GHF objects might be something like this:
dm = np.asarray(dm)
nso = dm.shape[-1]
nao = nso // 2
dm_a = dm[...,:nao,:nao].real
dm_b = dm[...,nao:,nao:].real
dm = dm_a+dm_b
in the numint.eval_rho function.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/801#issuecomment-764706380, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN5NRBQNG6WIDKX7OJV3WDS3A7FFANCNFSM4WMZ47PA
.

Thank you for your answer.
Yes, that is true, that is only a very rough approximation for the one particle density matrix. It is actually used in pyscf.dft.gks.get_veff.
True, GKS and DKS is not supported at the moment. However, expected behaviour should be that density output should be supported for all methods (that includes DHF and GHF). Currently the go-to method for that is numint.eval_rho(mol, ao, dm) which is also what is used to generate density cube files.
So shouldn't there be a warning when eval_rho is called with a GHF or DHF density instead of a crash? Or should eval_rho be extended to allow plotting of these arguably valid densities?

Thank you for your answer.
Yes, that is true, that is only a very rough approximation for the one particle density matrix. It is actually used in pyscf.dft.gks.get_veff.
True, GKS and DKS is not supported at the moment. However, expected behaviour should be that density output should be supported for all methods (that includes DHF and GHF). Currently the go-to method for that is numint.eval_rho(mol, ao, dm) which is also what is used to generate density cube files.
So shouldn't there be a warning when eval_rho is called with a GHF or DHF density instead of a crash? Or should eval_rho be extended to allow plotting of these arguably valid densities?

For DHF objects, a density can be used via:
dm = hf.make_rdm1(ao_repr=True)
n = dm.shape[1]
dm_a = dm[0:n//2:2,0:n//2:2].real
dm_b = dm[1:n//2:2,1:n//2:2].real
dm = np.array([dm_a,dm_b])
Is this correct?
There is another issue, however:
Some densities are not normalized. I performed manual integration on single atoms and found densities that were not normalized to the total number of electrons for some atoms (for example Z=5). The issue persisted for UHF-X2C and DHF densities. If required, I'll post and example skript.

Here is an example for the density of GKS and DKS at
https://github.com/pyscf/pyscf/blob/master/examples/dft/31-xc_value_on_grid.py


Logs not displayed

Hi I set up the provider, and after trying to create a resource this is what the pod logs looked like:

Please let me know if I need to do something different or if it's a bug.

Mmm i have never seen this before. What version on XP are you running? It might be smart also to check at the Crossplane Slack community.

oh could be that, thanks


Closest Named Color app doesn't work when ran locally

I was trying to reverse-engineer this great app for my own purposes, so I downloaded the main color.js branch's source code. But even when unmodified, running the index.html inside the "apps/named" folder results in this:

Am I doing something wrong?

It's hard to figure out what's going on from that description. Does the console include any error messages? Are you running it from localhost or file://?

You can't just open the HTML. For best results, you need to run a simple  HTML server to get the site up and running locally.
Try installing a simple HTTP server:
 $ npm install http-server -g

And then run the server:
$ http-server

You'll be able to navigate the whole site as seen below, though you could configure the port to something else if you want:
http://localhost:8080/

Where do I run these commands? I'm doing them in the colorjs main folder and I keep getting a module not found error, even though I have node.js/npm and color.js installed in that folder, I believe. I'm using cmd for Windows 10 x64 - the shortcut that Node.js installs for you - if that helps.

@P-Star7 I do them right in the main directory of the repository.
npm install http-server -g should install the http-server module globally. Then it should be accessible anywhere.
I will admit though that I don't normally do this in Windows (I'm on macOS) so I'm not sure if there are some Windows specific considerations that I'm missing.
I'm also not sure if the authors of color.js use a different way to run and test their site, this is just what I do on the occasions when I contribute to this project.

Aha! It now works, I forgot to install it globally before running http-server. I installed it globally afterwards, but never tried http-server again, only "node http-server". Just running http-server did the trick. Thanks!

That's odd... I spoke too soon. It indeed runs on localhost now, but still with the result in the OP. All the src dependencies in this demo's index.js (index.js and keywords.js) are where they SHOULD be.

Okay, now it does. Augh...

I just enabled Discussions on this repo, since it's much more suitable for these kinds of things (troubleshooting as opposed to a bug report). I will transfer this to that section now.


docs(FAQ): failed to load modules

Failed to load textlint's rule module: "${packageName}" is not found.

See FAQ: https://github.com/textlint/textlint/blob/master/docs/faq/failed-to-load-textlints-module.md

@homu r+
FAQ document is inspired by deku v1.

:pushpin: Commit b6746ab has been approved by azu

:zap: Test exempted - status


looks like API_BASE_URL and API_URL environment variables doesn't work

I trying to up docker-compose.yaml
Looks like API_BASE_URL is hardcoded in frontent and can't be override from .env file
i create working copy
git clone https://github.com/BloopAI/bloop ./BloopAI/bloop
cd BloopAI/bloop
git checkout oss

and BloopAI/bloop/.env with
API_URL=http://<my-ip>:7878

run
docker compose build bloop-app
docker compose up -d

and see frontend application always try to use http://localhost:7878
did i miss something?

no


chore: remove --container-runtime from AL2 user data

Fixes #N/A
Description
In light of --container-runtime being ignored by the EKS bootstrap script and v1beta1 APIs removing the option to set container runtime, this PR drops the field from the AL2 AMI.
How was this change tested?

make test
/karpenter snapshot

Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 82.527%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
4945

Relevant Lines:
5992

💛  - Coveralls


Dependency Map?

Hi,
Is there a "Dependency Map" export option in the ts compiler. I am working on a fairly large UI framework with many classes and modules. Applications using this framework are not going to use all the classes all the time. Next to that, many of these classes need their corresponding css files, for example: "My.Button" class needs "My.Button.css". It would be very useful to see which classes are "actually used" in the project so we can bundle only those classes/files when building the final .js file.
Any thoughts?
Regards,
Gevik.

The compiler does not provide this as a feature out-of-the box. there are tools that can do that for you. for instance @basarat's https://atom.io/packages/atom-typescript has a dependency graph view.

You probably want to +1 this issue https://github.com/Microsoft/TypeScript/issues/1479

Note : alm has this as well : https://basarat.gitbooks.io/alm/content/features/dependency.html :rose:


fix warnings with VS

It's now /W2 clean.
We can't use \u with VS because the encoding is dependent on the locale. With the \xXX encoding we ensure it's UTF-8.

It's not much of a difference, but can't we use u8"..." literals?

It's not much of a difference, but can't we use u8"..." literals?

In VS, yes. I thought it was not used due to some compatibility issue with older compilers?

merged


Published flag

Catalogs, Views and Tasks should have a published flag, to hide them when they are not completed yet.

Added in e9e95ab10f45daae99879e3af8064ed04183a173.


"WebRTC stack" image in README contains a broken footnote

UDP*

I expect that * to lead a phrase like "Typically UDP, but can use TCP as well if needed", but in reality it just dangles.

Just remove * to remove the confusion.

Just remove * to remove the confusion.


Fix to allow multiple /'s in application image

Signed-off-by: Jason Yong<EMAIL_ADDRESS>What this PR does / why we need it?:

Allows an application image name to have 3 "/"

Fixes #325

@halim-lee I've merged this fix into main. Please pull into 0.8.x and also include with OLO. Thank you.


Update README.md

We created a logo to help the clarify the difference between js-ipfs and js-ipfs-api. Would you like to add to your client library as well? You can see how it looks here: https://github.com/ipfs/js-ipfs-api
This design is from @akrych 🌟

Coverage remained the same at 100.0% when pulling 9a0eb4808d59a656e28287f58225fdd205ca14dd on diasdavid:patch-1 into 96a890f4518665a56581a2a52311eaa65928eac8 on vasild:master.

Thanks!


Link to tinygrad-tensor-puzzles in docs

I implemented Sasha's Tensor Puzzles using tinygrad, and saw that it could be useful to new users of tensor libraries. Thus, I thought I could link my solutions in tinygrad docs. I'm unsure how desired this is though, but here is a PR ..

I don't think solutions are the right thing to link, but rather perhaps a very clean fork of the puzzles themselves


What happens to enabled Enterprise features like Credential Guard on Downgrade?

In the documentation above it states:
"The Windows 10 Enterprise device reverts seamlessly to Windows 10 Pro edition, after a grace period of up to 90 days"
If we have policies that turn on Enterprise features like Credential Guard, what happens when the PC downgrades to Pro?  Will the credentials continue to be stored in Credential Guard, or will something happen the will force the representation of the credentials to be stored outside of the TPM?  This is our primary concern with enabling some of the Enterprise features since only some of our environment is covered by W10 Enterprise subscriptions.

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

ID: dd18b4c7-1c71-916d-cb5c-8322903e8ed5
Version Independent ID: 7b4c1663-1b1a-2952-7bf8-b0cb724013af
Content: Windows 10 Subscription Activation
Content Source: windows/deployment/windows-10-enterprise-subscription-activation.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@officedocsbot assign @mypil

@PiKappZ746 - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@PiKappZ746 Credential guard does not store any credentials itself, it uses VBS (Virtualization based security) to protect and isolate the LSASS secrets (password hashes and Kerberos TGTs) into a separate container.
So by reverting back to Windows 10 Pro and turning off Credential Guard, you will no longer be protected, and the LSASS secrets can be dumped again very easy.
I recommend this article for deeper insights into this matter: https://www.blackhat.com/docs/us-15/materials/us-15-Moore-Defeating Pass-the-Hash-Separation-Of-Powers-wp.pdf

@PiKappZ746 - From our understanding, the issue has been resolved but it may take a few days for the merged content to appear in the article. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


Optional dependencies are not checked in vuln checks

Describe the bug
When checking for vulnerabilities or yanked crates, optional dependencies are not checked.
While the current behavior might be reasonable to some use-case, we have some crates in which we gate various crates behind a bunch of optional features. This means cargo deny only ends up checking a subset of our tree. Having a --all-features flag/config option would allow us to properly check the whole tree for vulnerabilities across all our build configurations.
To Reproduce
[package]
name = "python-stuff"
version = "0.1.0"
authors = ["me"]
edition = "2018"
publish = false

[dependencies]
pyo3 = { version = "0.12.3", optional = true }

Lockfile (forces use of yanked version of pyo3):
[[package]]
name = "pyo3"
version = "0.12.3"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "a9b90d637542bbf29b140fdd38fa308424073fd2cdf641a5680aed8020145e3c"
dependencies = [
 "ctor",
 "indoc",
 "inventory",
 "libc",
 "parking_lot",
 "paste",
 "pyo3cls",
 "unindent",
]

deny.toml:
[advisories]
yanked = "deny"

Running cargo deny check advisories will print advisories ok, despite an optional dependency being yanked (and having a RUSTSEC advisory opened).
Expected behavior
I expect either cargo-deny to run on optional feature, or to provide a flag/config option to let the user chose to either test the default config or all features.
Device:

OS: linux
Version: cargo-deny 0.8.1

The default behavior used to always check with --all-features, but this was changed in 0.7.0, you can use --all-features on the command line to check all optional dependencies.

Ah, thanks. I see the option now, I must have checked too hastily. Is there a similar flag I can set in the deny.toml to make checking all features the default?

No, but it would be easy to add, as we already support eg target specification tor filtering crates in the config file.

I've opened #314 as a feature request for this functionality.


feat: Enable support for Postgresql 16

Enhancement:
enable using PostgreSQL 16 via this role
Reason:
The new PostgreSQL stream will be available in c9s (RHEL9) and c8s (RHEL8)

Thanks, README.md should be fixed now


Documentation is unclear on how to go forward after molo scaffold ...

@bruskiza got stuck on trying to run tests for the molo repo, not the newly scaffolded project.

@bruskiza please review

Getting there... when scaffolding the base app the new app doesn't have any tests.
I can now run the tests with the following output:
bru:myapp bruce$ ./manage.py test
Creating test database for alias 'default'...

Ran 0 tests in 0.000s
OK
Destroying test database for alias 'default'...
bru:myapp bruce$
Is it up to the user or the scaffolded app to write the new tests?
If it's the user, we can point them to how to write tests, if it's the scaffolded app, then the app should have at least an initial test surely?

Considering a scaffolded application is largely empty I'm not entirely sure what tests to add.
I'll give it some thought.

@bruskiza the commit you made now conflicts with the same change

sorry hit enter too early:
@bruskiza the commit you made now conflicts with the same change made in #23. Generally what we do in these kinds of situations is merge in #23 into develop when reviewed, then merge develop in this feature branch to pull in the latest changes. That way you avoid the need to make the changes twice (and as a result, the conflict).

@bruskiza I'm going to leave it without tests, our expectation is that developers should know about tests, it's standard Django stuff and Django's got good documentation on how to write tests.

No problem. Thanks.

:+1:

http://molo.readthedocs.org/ done.


Documentation of base widgets

Add both undocumented and inherited members to the autodoc references for widgets. This will expose the base widget methods and the un-docstringed methods
Add some explanation to the basic widgets and a more detailed description to the box reference
Add an intro to the app class

Still needs #101 to be fixed before most of this will show on RTD.

How does an optioncontainer relate to a select widget?
It looks kind of obvious, a list of tuples (label, selection) are sent to an option container?

Another round of docstrings, docs and updates. I have a few questions about some of the specific containers. Like what is the difference between optioncontainer and selection? @freakboy3742

this is done now. So much for 1 change per PR!

updated to reflect suggested changes @freakboy3742


fix usage of proxy with https (#845)

Fixes #845, a regression introduced in e81a47. I feel this is not the best way to fix this, but it at least points out where the issue is. Perhaps someone more familiar with the code base could rewrite the patch. I'm also not sure how to write a regression test for this, but would be happy to do so if pointed in the right direction.

Current coverage is 98.78% (diff: 100%)

Merging #1366 into master will increase coverage by <.01%

@@             master      #1366   diff @@
==========================================
  Files            29         29
  Lines          6762       6763     +1
  Methods           0          0
  Messages          0          0
  Branches       1123       1123
==========================================
+ Hits           6680       6681     +1
  Misses           38         38
  Partials         44         44

Powered by Codecov. Last update 071098d...2223f69

Very thank you.
You've found a real source problem.
.path hacks should be replaced with better way but I've got your idea.
Will create new PR soon.
Unfortunately we have not yet proper functional tests for proxies so I'll ask you to manually check upcoming PR.

Fixed

@asvetlov Thanks! I'll test ASAP, but it may be a week or so.


Adding LCViz tutorial notebook

Basic example of using LCViz with data retrieval from S3. Apparently the data retrieval, which works locally for me, doesn't actually work on TIKE, so this will need a little debugging.

after running .flatten in the code cell, the plot looks strange:

bizarrely, if I open the "flatten" option in the side menu, the points re-appear:

This is a known bug that we are trying to get fixed upstream in bqplot.  If we can't get that in in time, perhaps we should disable normalization in the notebook?

As Kyle said, this is an upstream bug that I'm hoping gets resolved in the next week or two (i.e., before this notebook would be merged). I should have mentioned it when I opened this PR - if for some reason it's not fixed by the end of November I'll remove that section.

I added a note about the display bug and an edit to avoid it (by not normalizing the flattened curve), as well as one fix that I think was needed due to the new Astropy release.

last thing and then I'll merge: if we're using a literature reference, we should cite it. It's not clear to be which of the three papers on HAT-P-11 authored by Brett in 2017 is the correct one

@bmorris3 - any preference?

FYI the upstream bug with displaying the flattened light curve was fixed, so I removed the workaround.


Nested navigation

It would be very useful to have nested navigation pages.

Thanks for the suggestion.
Everyone: to vote this feature up/down, please use 👍 and 👎 on the comment above.

:+1:

Definitely want this!


Add darwin arm64 target

Go 1.16 was released today with support for the new MacOS platform, darwin/arm64.

I think this might take a little time to get working. Firstly Fyne itself needs to support compiling in the Apple M1, see https://github.com/fyne-io/fyne/issues/1739. There is a PR open that fixes it but it is not done and might require more thought to make sure that compiling for iOS still is compatible with older Go releases.
Then there is the issue with the licensing for the Apple SDK. I think it might require a new version and we cannot quite distribute that.
It will also of course require an update to Go 1.16 which has a couple points to consider before doing so. We are still on Go 1.14 as Go 1.15 drops support for one more MacOS release. The SDK that we are using is for the version that is being dropped and I don’t know if it will compile with newer Go versions. That requires testing, but assuming that it does work, upgrading should only be a matter of the team being okay with a MacOS version being dropped (as Go 1.16 does not drop any more versions).
I have perhaps painted a bit of a dark picture here, but it should all be solvable one way or another. It just requires some clever thinking and agreement with the team 🙂 We might end up having the user download the Apple SDK manually on to their computer (due to distribution issues with licensing).
I am curious as to what @lucor has to say about it. At the end of the day, he is the main developer and professional when it comes to fyne-cross 🙂

I might also want to specify that I am personally very interested in this per https://github.com/Jacalz/wormhole-gui/issues/7.

Just as a bump, https://github.com/fyne-io/fyne/issues/1739 was closed today with a fix in the develop branch.

I'll be spinning up a test image using Go 1.16 later today for some testing. Will update here with any developments.

Looks like darwin builds indeed can't compile with an updated Go version. I think we need to figure out something better.
cc @lucor

Did I misunderstand this message, or is it true that Go 1.16 does not work for darwin?

Did I misunderstand this message, or is it true that Go 1.16 does not work for darwin?

That's correct. Our macOS SDK is for 10.11 and later but Go 1.15 only support 10.12 and later, thus it cannot compile for Darwin on amd64. We'd need to update the SDK.

I can double check Go 1.15 tomorrow to make sure that it wasn't a bug in Go 1.16.

It works perfectly well with Go 1.15.
The macOS SDK is part of fyne package not part of the go compiler, so upgrading Go should not have changed anything.

I’m not saying that Go 1.15 doesn’t work on MacOS, I am saying that it doesn’t work with the SDK that we have in the fyne-cross docker image. That SDK is for 10.11 and I could not get fyne-cross to build using Go 1.16. As it was an error about linking to C-code  I assumed that it was due to 10.11 no longer being supported as of Go 1.15 and later. Will check Go 1.15 in fyne-cross tomorrow and see if I find the same error happening there as well.

Aaah! That all makes sense :)

@Jacalz has reported all the issues regarding this activity, so no more to add there. I'd try just to recap the action items in order to figure out a possible way to get this sorted...

[x] darwin arm64 support in fyne: landed in develop with fyne-io/fyne#1739
[ ] update fyne-cross to use  fyne-io/fyne#1739
[ ] add the arm64 arch to the supported arch for darwin in fyne-cross
[ ] update the base docker image to go v1.16
[ ] update to the latest macOS SDK version supported by go v1.16

As reported above the major issue is with the last point due to the SDK licensing that does not allow to redistribuite the SDK into a docker image.
As discussed previously into the slack channel, one possible to solution is to have the final user to build itself a local copy of the macOS docker image.
First ideal comes in mind is to have a fyne-cross command that given a valid Apple developer account will download the SDK and build the docker image.
Since handling the auth on the Apple dev site could require some effort, probably as first step a command (or a bash script / make task) that builds the image using a local copy of the SDK already download by the user could be good enough :-)

@lucor Would it be possible to load the SDK from the computer at runtime without having to recompile the docker images? Compiling the images a takes a lot of time and it would be great if it could be avoided.

@Jacalz we could try something like that, but the great part of the time is spent to create the osx toolchain starting from the xcode package :(
Further details can be found at the osxcross project (the tool used to create the macOS Cross toolchain for Linux and *BSD) but basically the steps needed and that takes time are:

download xcode (~11Gb)
extract the macOS SDK
install dependencies required by osxcross
build OSXCross toolchain

Some time could be saved providing a base image with the desired version of Go and the required dependencies but IMO it will save only a small fraction of the total time...
I do not like the build all locally way too, especially for users that does not have a fast internet connection...  but unless it is not possible to host the image or rely on a third-party one no other solutions comes in mind.

The PR #34 add support for darwin arm64 along with some other new stuff like Go v1.16.
Help with testing on that architecture is welcome :-)

ideally, the darwin arm64 support would lead right into darwin universal.  "native" apple apps don't require the user to have knowledge of which processor architecture they are on.

ideally, the darwin arm64 support would lead right into darwin universal. "native" apple apps don't require the user to have knowledge of which processor architecture they are on.

I’m not so sure about that. We are using a compiled language. If we want universal apps, we should just use interpreted languages like Python. Either way, I don’t think it’s anything that we can do anything about unless Go supports it.

cross-compiling is for darwin arm64 target is now available for testing in develop (#39)


python38Packages.databricks-connect: 7.3.9 -> 7.3.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.databricks-connect is: "Client for connecting to remote Databricks clusters"
meta.homepage for python38Packages.databricks-connect is: "https://pypi.org/project/databricks-connect"
meta.changelog for python38Packages.databricks-connect is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 7.3.10 with grep in /nix/store/j3adf7pw35l0ahvrj80mk3b7nn4q6dwa-python3.8-databricks-connect-7.3.10

directory tree listing: https://gist.github.com/95b7cce423d6d2284f23121c41b05c05

du listing: https://gist.github.com/88b991ee95ebaa599fe5be0f05a703c2

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.databricks-connect
python39Packages.databricks-connect

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/j3adf7pw35l0ahvrj80mk3b7nn4q6dwa-python3.8-databricks-connect-7.3.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.databricks-connect https://github.com/r-ryantm/nixpkgs/archive/cc61bcb16edbe1c71b536a4c13526211db420499.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/j3adf7pw35l0ahvrj80mk3b7nn4q6dwa-python3.8-databricks-connect-7.3.10
ls -la /nix/store/j3adf7pw35l0ahvrj80mk3b7nn4q6dwa-python3.8-databricks-connect-7.3.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python38Packages.databricks-connect
python39Packages.databricks-connect

Maintainer pings
cc @kfollesdal for testing.

Result of nixpkgs-review pr 116057 at cc61bcb1 run on aarch64-linux 1

2 packages built successfully:

python38Packages.databricks-connect
python39Packages.databricks-connect


Allow easy overriding of Filter.field_class from FilterSet

FilterSet can currently override a lot of things about its filters - but it can't override the form field class. This makes it difficult to do things like use django-filter with floppyforms, or more generally to use any custom field that improves UI on your site.
This should be fairly easy to implement - just allow field_class to be passed in on instantiation, and people can use filter_overrides to do the rest.
For full similarity with Django, you could also add support for passing in a choice form field as well as a normal one. See also: model Field.formfield.

I'm going to close this as Out of Scope for the moment. Happy to consider documented, tested pull requests. We may have capacity to reconsider in the future.


Identify + fix BLE re-connection issue

Whenever there is a disconnect to OW device that isn't done by the user - for example, turning off the OW without hitting the "disconnect" button, the app has problems trying to reconnect. The workaround could be either fully closing and restarting the app and/or turning on and off bluetooth in the device settings.
I'm not sure what is going on, logs aren't showing much, so adding verbose logs in the BluetoothImpl may shed light. If there is a solid open sourced + maintained library that deals with the Android BLE buggy af stack I'm all about moving to it...

I have a friend working on one and has investigated all of them he can find. He says there is sadly no silver bullet.

Sad panda.
On Mon, Mar 12, 2018, 10:58 AM Erik Babel<EMAIL_ADDRESS>wrote:

I have a friend working on one and has investigated all of them he can
find. He says there is sadly no silver bullet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ponewheel/android-ponewheel/issues/63#issuecomment-372405405,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0LrePDpLfin_VkAFMStmL1xRTsI_e_ks5tdrczgaJpZM4SnGZ1
.

As @ebabel mention, It isn't easy to get bluetooth right on Android, but this might help...
In your BluetoothUtilImpl:
https://github.com/ponewheel/android-ponewheel/blob/bbc9cd217252aeb8ef5b313ce923b3934c64b180/app/src/main/java/net/kwatts/powtools/util/BluetoothUtilImpl.java#L328-L345
Upon disconnect you reconnect using BluetoothDevice.connectGatt:
https://github.com/ponewheel/android-ponewheel/blob/bbc9cd217252aeb8ef5b313ce923b3934c64b180/app/src/main/java/net/kwatts/powtools/util/BluetoothUtilImpl.java#L315-L318
When using BluetoothDevice.connectGatt you should clean up the previous connection using BluetoothGatt.close, or use BluetoothGatt.connect to reconnect.

Thanks Travis, I'll give this a try!
On Tue, Mar 13, 2018, 12:09 AM Travis Wyatt<EMAIL_ADDRESS>wrote:

As @ebabel https://github.com/ebabel mention, It isn't easy to get
bluetooth right on Android, but this might help...
In your BluetoothUtilImpl:
https://github.com/ponewheel/android-ponewheel/blob/bbc9cd217252aeb8ef5b313ce923b3934c64b180/app/src/main/java/net/kwatts/powtools/util/BluetoothUtilImpl.java#L328-L345
Upon disconnect you reconnect using BluetoothDevice.connectGatt:
https://github.com/ponewheel/android-ponewheel/blob/bbc9cd217252aeb8ef5b313ce923b3934c64b180/app/src/main/java/net/kwatts/powtools/util/BluetoothUtilImpl.java#L315-L318
When using BluetoothDevice.connectGatt
https://developer.android.com/reference/android/bluetooth/BluetoothDevice.html#connectGatt(android.content.Context, boolean, android.bluetooth.BluetoothGattCallback)
you should clean up the previous connection using BluetoothGatt.close
https://developer.android.com/reference/android/bluetooth/BluetoothGatt.html#close(),
or use BluetoothGatt.connect
https://developer.android.com/reference/android/bluetooth/BluetoothGatt.html#connect()
to reconnect.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ponewheel/android-ponewheel/issues/63#issuecomment-372567768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0LrQNOdf2fUJ70E7qnvQkiOlLw4fxWks5td3CrgaJpZM4SnGZ1
.

@Travis that did the trick. Now onto the next bug after fixing that one,
the board reconnects but the UI thread doesn't. That should be easy
compared to this one, big thanks again for spotting it.
On Tue, Mar 13, 2018, 8:44 AM Kevin Watkins<EMAIL_ADDRESS>wrote:

Thanks Travis, I'll give this a try!
On Tue, Mar 13, 2018, 12:09 AM Travis Wyatt<EMAIL_ADDRESS>wrote:

As @ebabel https://github.com/ebabel mention, It isn't easy to get
bluetooth right on Android, but this might help...
In your BluetoothUtilImpl:
https://github.com/ponewheel/android-ponewheel/blob/bbc9cd217252aeb8ef5b313ce923b3934c64b180/app/src/main/java/net/kwatts/powtools/util/BluetoothUtilImpl.java#L328-L345
Upon disconnect you reconnect using BluetoothDevice.connectGatt:
https://github.com/ponewheel/android-ponewheel/blob/bbc9cd217252aeb8ef5b313ce923b3934c64b180/app/src/main/java/net/kwatts/powtools/util/BluetoothUtilImpl.java#L315-L318
When using BluetoothDevice.connectGatt
https://developer.android.com/reference/android/bluetooth/BluetoothDevice.html#connectGatt(android.content.Context, boolean, android.bluetooth.BluetoothGattCallback)
you should clean up the previous connection using BluetoothGatt.close
https://developer.android.com/reference/android/bluetooth/BluetoothGatt.html#close(),
or use BluetoothGatt.connect
https://developer.android.com/reference/android/bluetooth/BluetoothGatt.html#connect()
to reconnect.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ponewheel/android-ponewheel/issues/63#issuecomment-372567768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0LrQNOdf2fUJ70E7qnvQkiOlLw4fxWks5td3CrgaJpZM4SnGZ1
.

This is fixed, thanks again. This actually made the last release the most solid yet,


[4.1][Idea] SoftDelete operation

@mgralikowski mentioned using soft deletes with Backpack, and letting the admin soft delete / hard delete / restore entries - which I think is a pretty cool use case, though rare. Usually you wouldn't let the admin hard delete something, that's the whole point. But you could let him restore. Food for thought. I'll dump by thoughts below, in response to PR #1761
--
I think we could have a different operation that provides full support for soft deletes. Maybe called SoftDeleteOperation, used interchangeably with Delete. In my opinion, on top of this, an additional button to "Hard delete" would also be good. But then... that would require two more actions, each with its own route and controller method:

restore
hard delete

Interface
Default:

Delete button (if it has softdelete trait a dropdown shows with the "Hard Delete" option)

If soft delete button enabled:

Delete button disabled, dropdown shows Restore and Hard Delete options

Routes

/id/destroy (soft delete)
/id/destroy?type=hard (hard delete)
/id/destroy?type=restore
/id/destroy?type=soft (soft delete, just like no parameter set)

Thoughts? :-)

I can dig it. We are using Backpack to build an internal application which has a lot of tools for our development teams which have associated CRUD pages. Effectively, we use Backpack as our frontend/dashboard for this application rather than as a traditional "Admin" dashboard. Because of this, I could see Soft delete functionality being really useful to us.
@tabacitu I'd be interested in helping with this if needed, once the community has weighed in

@tabacitu from your comment on #2282
I am not really fan of neither options.
There is the option to keep only one button for delete, and add a button if entry is softdeleted to restore! Might change the text from Delete to Force Delete or something.
1º When going to list 'usually' you will not mix trashed with active items. Maybe have a filter to show the trashed, but i don't see why would anyone want to show a mix of active/trashed items.
If they mix, the restore button will appear near the delete, then you know, delete is delete for GOOD and restore will restore the item.
2º For a developer it might make sense to have the delete -> hard delete -> restore  etc etc.
I think we just need to add a new permission for crud that is HardDeletes, the user can delete an item, can view hard deleteds (like a filter eg.) but can/can't HardDelete.
I am trying to not over-engineer here. Let me know your thoughts.
Best,
Pedro

Thank you for your input @pxpm . Let's move the conversation to #2282 then since it's newer, and has the proposed solutions there. I no longer agree with the first proposal in this thread, and #2282 is more up to date.
I'm afraid I don't quite understand how your proposition is different from (Solution 1) there, seems to me like we're saying the same thing. Please explain/rephrase in #2282 what the differences are from Solution 1.

1º When going to list 'usually' you will not mix trashed with active items. Maybe have a filter to show the trashed, but i don't see why would anyone want to show a mix of active/trashed items.
If they mix, the restore button will appear near the delete, then you know, delete is delete for GOOD and restore will restore the item.

Totally agree. By default CRUDs show non-deleted items. Trashed items are only visible with a filter. You click "Trash" and you get to see the items in the trash. Only then do you get to see items that have the "Hard Delete" or "Recover" buttons, when you're in the trash bin.

2º For a developer it might make sense to have the delete -> hard delete -> restore etc etc.

I think that's a great way to put it. We should do it just as it's done in Windows/Mac/Linux, since that's what the user is used to. And it sounds to me like Solution 1 is the one that best copies the flow. Maybe we just need to rename the buttons to "Delete" (soft delete), "Recover", "Remove from Trash" (Hard Delete). With a filter called "Trash" it would make sense to:

Go to Trash.
Inside the Trash, either recover items or remove them entirely.

I think we just need to add a new permission for crud that is HardDeletes, the user can delete an item, can view hard deleteds (like a filter eg.) but can/can't HardDelete.

I agree, in addition to "delete", there should be a separate "hard-delete" permission. And while we're at it, a "recover" permission. So that developers can easily choose for each CRUD who would be able to perform these actions.

Hi guys, has this been implemented?

Hi guys, has this been implemented?

@pierluromani sorry, no, we still don't have a built-in solution for this (or an addon). It's totally doable using custom operations but not something that's provided by default.
Note - since the links above broke, here's the current open issue about this - https://github.com/Laravel-Backpack/ideas/issues/38

@pierluromani sorry, no, we still don't have a built-in solution for this (or an addon). It's totally doable using custom operations but not something that's provided by default.
Note - since the links above broke, here's the current open issue about this - https://github.com/Laravel-Backpack/ideas/issues/38

Ok, no problem, after all that's not a major feature at the moment.
By the way keep up the good work! I've been digging Backpack for a few months now and it's just amazing.

Ok, no problem, after all that's not a major feature at the moment.
By the way keep up the good work! I've been digging Backpack for a few months now and it's just amazing.

Nah sorry, just closed it because there's another issue open for it here - https://github.com/Laravel-Backpack/ideas/issues/38
For now I say... roll your own! 😀


importing state 'AS' has different header order than other states.

=IMPORTDATA("http://covidtracking.com/api/states/daily.csv?state=AS") gives:
date | state | positive | negative | hash | dateChecked | total | totalTestResults | posNeg | fips | deathIncrease | hospitalizedIncrease | negativeIncrease | positiveIncrease | totalTestResultsIncrease | pending | hospitalizedCurrently | hospitalizedCumulative | inIcuCurrently | inIcuCumulative | onVentilatorCurrently | onVentilatorCumulative | recovered | death | hospitalized |
All other states give this header order:
=IMPORTDATA("http://covidtracking.com/api/states/daily.csv?state=AK") gives:
date | state | positive | negative | pending | hospitalizedCurrently | hospitalizedCumulative | inIcuCurrently | inIcuCumulative | onVentilatorCurrently | onVentilatorCumulative | recovered | hash | dateChecked | death | hospitalized | total | totalTestResults | posNeg | fips | deathIncrease | hospitalizedIncrease | negativeIncrease | positiveIncrease | totalTestResultsIncrease |

Hi there, Please use the column names rather than order for data extraction whenever possible


chore(docs): fix typos in README.md

Short options are IMHO mistakenly shown with two dashes.

Indeed, good catch! Thanks for your contribution ❤ .


Drop test_layout bindgen tests from stored versions

Note that this was done manually by cutting out all #[test] functions
named test_layout, rather than the automated re-generation step
that produced 53b1b598cb5ca63befa043a05905b681e8890dc7.
Closes: #722

To fix at least Clippy and Docs, you should merge the official master branch in your own branch.

See https://github.com/rusqlite/rusqlite/blob/4d8b8e71cd42fb58c2845e2d09a7075972e561e9/libsqlite3-sys/build.rs#L618


Large generated schematics generating rendering fails

Description of Issue:
When you imported that in your world and trying to look at it from above it might and shortly will skip chunks on loading phase. In result you will see your creation with miltiple full-height holes. Raising camera above precision rendering point (to trigger simplifying textures) and going back can fix that, but it will start again. Probably if you gonna do it 10-20-whatever times, it will be fully loaded. But anything nearby (same imported areas) will act same. Looks like memory-on-loading-chunks issue or saved-chunks issue. Also already rendered chunks after climbing always will disappear and be rendered in simple mode again. Most of them is good with normal view. Also that starts mostly far from starting position (after map is opened in editor). (rarely it just becomes stuck and wont render anything more with normal textures)
MCEdit Version:
<IP_ADDRESS> and pre<IP_ADDRESS>
Minecraft Version the world was last opened in:
1.12.2
Operating System:
Win 7 x64
A screenshot or full text of the error from the console:

How to reproduce:
Generate large schematic from common mesh (obj or something) via poly2vox.exe + aux filter for MCEdit, import it into any existing world and start to look whole affected area from above.

Also catched another problem

Alongside of common LT tile entities there is saved workbench (basically same with scripted LT's feature) and 2 chest with recipes and ready LTte constructions

Seems to be just MCEdit is unable to read all its content due to large bits amount in all of that LTtes saved inside of chest.

Any help? I can't work here with this bug...

Somehow got normal working (after some events with other games with BSODs)But now all the same, directly caused by Save process here.

Unstable, but semi-working, will close for now.

So, it is consistent on latest, mostly on no-chunks-area where I placing parts. Often goes stuck completely after 10 ок something minutes of work. Might go on another test runs but all basic info is here already.

Any idea?

Any idea?


coq: also install ocaml and findlib

Motivation for this change
Fix #34657.
native_compute does not work without OCaml and findlib being installed in the user environment.
By installing more packages, this may create installation conflicts, but I argued (in #34657) that these conflicts are fine because if a user has an incompatible OCaml version in scope, they may face even more surprising issues when trying to use native_compute or to build a plugin.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Does this change the closure size?

Closure-size after:
$ nix path-info . -S coq
/nix/store/a8zxmd6fn0mbnp5lafikk78yryw4nlz4-coq-8.11.2<PHONE_NUMBER>

Before:
$ nix path-info . -S coq
/nix/store/xsjvvm0y8gswllhb6z8h4w366h0xfwq4-coq-8.11.2<PHONE_NUMBER>

@GrahamcOfBorg build framac satallax

As this is a fix, should it be ported to 20.09? (I can do it).

It is a fix, yes, but it introduces a slight incompatibility, so I would say no. (It is easy to workaround the problem anyway.)

OK, thanks!

I would like to revert this.
Use case: develop a program in OCaml with some modules extracted from a Coq development. You cannot have in the same (nix) environment the tools to develop both Coq & OCaml parts due to conflicting versions of OCaml: the one propagated from Coq (that you don’t want to change because of the binary cache) and the one for your local development (that you want to change, e.g., to test compatibility with different versions, to enable flamdba or other goodies).


add Cycling ‘74 Max v7.0.3

There is already an cycling74-max.rb file but this points to the old 6.x Max Installer.
The new Version 7 is now a Simple App no Installer is needed anymore.
I think making a seperate Package for the new 7.x Version of Max is a good solution to separating this 2 Different types of Installation methods.

This should then replace the present formula, and the old one should be moved to caskroom/versions.


⚠ Visual-Crew_Builder has degraded performance

In 04207ca, Visual-Crew_Builder (https://visual-crew-builder.ai) experienced degraded performance:

HTTP code: 200
Response time: 1320 ms

Resolved: Visual-Crew_Builder performance has improved in 28fa04d after 6 minutes.


Is this project still actively maintained or dead?

What is the status of this project? Is it recommended for use in production?

Hi, this project doesn't have much movement cause its completed and 100% unit tested.
It is used in production by many people (including myself).
Hope that clarifies your doubts :)


Update index.js for webpack 2 compatibility

@alexfedoseev Please confirm. This LGTM!

I published 1.0.1 to npm!

@justin808 cheers! Thanks for that!


Unhandled Exception: HasuraRequestError: unexpected variables in variableValues: name

I couldn't move further. Everything is configured as mention.

Could you show the code of your query?

Now it's working perfectly. I am previously tried the general Hasura which hosted on heroku, once i replaced the hasura cloud2.0 query it asks for header validation to process secure API request.
But how to handle the json caching in hasura connect?

U can create ur own implementation of https://pub.dev/packages/hasura_cache_interceptor or use https://pub.dev/packages/shared_preferences_cache_interceptor


Fix link to home pages in 404.html

The link to the home pages in English and French don't point to the right page when the URL has a directory.
Example: https://canada-ca.github.io/ore-ero/1/2/3/4/pasdepage.html
We need the ./ore-ero so that is can work all the time.
Use the baseurl variable from _config.yml.  It can be changed because it's not being used.

#69


Download Playlist File

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question
Is there a way to download raw playlist files (m3u8, etc.) rather than the videos which those playlists point to?

--write-pages.


Adds spanish (es) translations

Summary
Adds spanish (es) translations, issue related #48

Awesome. Thanks again!

No problem 👍


Create migration file when blueprint updated for data type stored in DB

Obviously a luxury-end feature request, but it'd be really handy!
At the moment, if a data type (eg users) is moved from static files to the database, and then the blueprint is updated (eg to add a job title field) — then you need to also manually create a database migration file and type in the lines to create each new column in the database table.
In an ideal world, I'd love it if Statamic created migration files whenever the blueprint was updated for a datatype that is stored in the database, and then prompted you to run the file with the correct command.

Yeah agreed this would be nice, but unfortunately there are a lot of edge cases to think about (as an example how do you know if a field has been renamed versus it being deleted and a new one added).
Something to think about anyway.


Angular 11 example?

Could some one pls share Angular 11+ example?
Not able to install with Angular 11+ version, expecting to update Angular CDK.
Thanks

@suriyaJaay I have forked this and released a new package to support ng 9+.
See: https://www.npmjs.com/package/@danielmoncada/angular-datetime-picker