Fully unlocking epics for unlockpets.js and fixing level hack for changelevel.js

Missing parts
For unlocking all pets, it doesn't fully unlock epics. You can use them, but you cannot do the battles in the Epics area. For the level hack, after a team titan battle, it shows your real level.
Suggestion for unlockpets.js
Maybe add Mythical Epics?

Yes I am aware that the pack hack doesn’t do that but the level hack is supposed to reload the page so then it saves the level, let me fix that now.

Ok I don’t see a problem with the level hack so I’ll look at it later but yeah I’ll look into the mythical pet idea

I did the mythical hack i’m releasing in a bit if I don’t find any bugs

Added your request.


version 0.4.0

Mapper register mixins no declarative extensions.
This allow for a proper sandbox of metadata.
Backward incompatible

#release-note


One  per block

@picklesrus @NeilFraser @rachel-fenichel @thisandagain
How much work do we think it would take to experiment with doing one SVG per block?
The <g class="blocklyDraggable"> becomes a <div>, the <path> becomes <svg><path>... and most fields get folded into that SVG. Connected blocks get placed in the draggable div as fields do now.
First this would require splitting the blocklyBlockCanvas out of the main SVG, as we've already discussed for panning performance.
I think this change would have a few benefits:

First, I think it would give us a huge performance gain. By moving the blocks to the DOM, we could avoid repainting during block operations. Currently when you reshuffle the blocks (or, for example, insert or remove ghosts - see #200), the entire SVG is repainted by the browser. Compare to this impressive prototype: https://nathan.github.io/pixie/. Compared to Blockly, this is doing many, many less paints.
Second, it would let us create fields in a much cleaner way. We could avoid the WidgetDiv hack and also avoid foreign object - simply by putting HTML inputs positioned inside the drag div. This would greatly simplify doing animations. The current way we've pulled off animations (redrawing the entire field as an HTML element only when we want to edit it) is hacky and it makes it very difficult to implement new fields. It also means to do more than one animation at a time, we'd need to implement multiple WidgetDivs. By using one <input> per field, we could also immediately solve #231 without creating more of a performance headache.

Any thoughts? Why did Blockly choose not to do it this way? How much work would it take to change? Is this even possible?

It feels like we're abandoning this idea, since it would require significant refactoring and we've pretty much determined it won't provide much performance benefit beyond the existing plans in #154.


Include necessary utilities (or dnf) in Gateway POD

As part of reducing the size of Gateway Engine, we have removed dnf. But this is going to make debugging issues a lot more difficult as we do not have necessary utils (like tcpdump, conntrack etc) and cannot install them. For a production environment, it makes sense to strip off everything that is not necessary, but for our day-to-day installations of Submariner, it might be worth to include dnf or the necessary utils.

I think most debugging scenarios (checking the networking setup, iptables rules etc.) are better addressed by starting a separate network debugging container alongside the engine (e.g. based on netshoot). See also Using sidecars to analyze and debug network traffic in OpenShift and Kubernetes pods.
In other scenarios, tools such as oc-inject can be used to inject arbitrary binaries into a running container; see Installing debugging tools into a Red Hat OpenShift container with oc-inject for details.
Similar approaches are possible in non-OpenShift Kubernetes setups.

Thanks @skitt for your inputs and possible alternatives.
In the meantime, can someone share how much improvement (in size) we see by removing this from the Gateway Pods?
@mkolesnik @mangelajo @dfarrell07

We recently changed the gateway container images to keep microdnf installed, so I’m not sure this is still relevant.

We recently changed the gateway container images to keep microdnf installed, so I’m not sure this is still relevant.

Cool. Yeah, I see microdnf in the Gateway/Globalnet pods. Thanks.


tests matrices factors

OS
python versions
pydantic versions
optional dependecies or not
pymongo/motor versions

Absorbed by #52


lezer-promql: fix missing types export in package.json

NPM packages that declare themselves as es modules ("type": "module) in their package.json must declare an exports.types property for typescript to successfully resolve the types when the tsconfig moduleResolution is set to node16 or above. Arethetypeswrong can be used to assert this.

Without this change projects that attempt to set moduleResolution to something other than node results in TS errors like:
Type-checking in progress...
ERROR in ./packages/grafana-prometheus/src/add_label_to_query.ts:2:40
TS7016: Could not find a declaration file for module '@prometheus-io/lezer-promql'. <EMAIL_ADDRESS>implicitly has an 'any' type.
  There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@prometheus-io/lezer-promql' library may need to update its package.json or typings.
    1 | // Core Grafana history https://github.com/grafana/grafana/blob/v11.0.0-preview/public/app/plugins/datasource/prometheus/add_label_to_query.ts
  > 2 | import { parser, VectorSelector } from '@prometheus-io/lezer-promql';
      |                                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      ```

@jackw Thanks, it looks good to me, although I am not an expert in this at all. Happy to merge unless @Nexucis has any objections :)

sorry I didn't see this PR. It's good to me as well, and thank you for fixing that !!

Appreciate you taking the time to review and merge. 👍

@Nexucis @juliusv I'm sorry to bother you but we just updated @prometheus-io/lezer-promql to 0.55.1 but it doesn't contain this change.  Is there something I can do to help get this into the next public release?

@jackw Sorry again for delay to replay. I don't know if you already noticed, but it is available under the tag 0.300.0-rc.0.  Hope it will help you


Feature Request: Site blocking/parental control

It would be nice if users had the option to block sites that they don't want accessible on their phone (e.g. sites that store information, are inappropriate, etc).
Perhaps this could be an option in settings -- a user could add sites they want blocked to a list. Then if the user attempts to access one of these sites, they would be redirected to a page that tells them they are blocked, for example.
Thanks!

Thanks for the feature request, really interesting. I will share it internally with the team.

Friendly reminder that we are a small team with a big backlog of features. We try to prioritize all our work based on our users' needs and the impact of each feature. We keep issues open so other users can show interest too, so thanks again for sharing it here.

If you want to block some sites that aren't family-friendly you could just set a child-safe ip in your device dns settings, like for example https://support.opendns.com/hc/en-us/articles/228009007-Android-Configuration-instructions-for-OpenDNS
The nice thing about this is that this would apply also to other browsers / apps that are installed on the same device.
However that obviously only works as long the child doesn't figure out how to reset that again :-)

Tracking this feature request: https://app.asana.com/0/0/1199514257586724/f

Tracking this feature request: https://app.asana.com/0/0/1199514257586724/f

Hello @christophergeiger3.
I did a small demo project of this will look like in our app, but we've agreed that there's no plan in the foreseeable future to support this request. Should this change in the future, we'll update this conversation appropriately.


[macOS Ventura, Xcode 14.2, M1 Pro] Get an error when install SwagGen

Hi @yonaskolb!
Thank you for this lib!
I tried to install it and got an error for all types like below. Do you have any ideas?)
brew
brew install SwagGen
==> Fetching yonaskolb/swaggen/SwagGen
==> Downloading https://github.com/yonaskolb/SwagGen/archive/4.7.0.tar.gz
Already downloaded: /Users/a27/Library/Caches/Homebrew/downloads/f36d827f276ed453e352be8214b3efac05cbee29f9659c070075bafe4221bebe--SwagGen-4.7.0.tar.gz
==> Installing SwagGen from yonaskolb/swaggen
==> make install PREFIX=/opt/homebrew/Cellar/SwagGen/4.7.0
Last 15 lines from /Users/a27/Library/Logs/Homebrew/SwagGen/01.make:
2023-01-05 18:25:41 +0000

make
install
PREFIX=/opt/homebrew/Cellar/SwagGen/4.7.0

swift build --disable-sandbox -c release --arch arm64 --arch x86_64
error: xcbuild executable at '/Library/Developer/SharedFrameworks/XCBuild.framework/Versions/A/Support/xcbuild' does not exist or is not executable
make: *** [build] Error 1

Do not report this issue to Homebrew/brew or Homebrew/core!

/opt/homebrew/Library/Homebrew/utils/github/api.rb:310:in `raise_error': Validation Failed: [{"message"=>"The listed users and repositories cannot be searched either because the resources do not exist or you do not have permission to view them.", "resource"=>"Search", "field"=>"q", "code"=>"invalid"}] (GitHub::API::ValidationFailedError)
	from /opt/homebrew/Library/Homebrew/utils/github/api.rb:238:in `open_rest'
	from /opt/homebrew/Library/Homebrew/utils/github.rb:179:in `search'
	from /opt/homebrew/Library/Homebrew/utils/github.rb:40:in `search_issues'
	from /opt/homebrew/Library/Homebrew/utils/github.rb:73:in `issues_for_formula'
	from /opt/homebrew/Library/Homebrew/exceptions.rb:498:in `fetch_issues'
	from /opt/homebrew/Library/Homebrew/exceptions.rb:493:in `issues'
	from /opt/homebrew/Library/Homebrew/exceptions.rb:549:in `dump'
	from /opt/homebrew/Library/Homebrew/brew.rb:154:in `rescue in <main>'
	from /opt/homebrew/Library/Homebrew/brew.rb:142:in `<main>'
/opt/homebrew/Library/Homebrew/formula.rb:2526:in `block in system': Failed executing: make install PREFIX=/opt/homebrew/Cellar/SwagGen/4.7.0 (BuildError)
	from /opt/homebrew/Library/Homebrew/formula.rb:2462:in `open'
	from /opt/homebrew/Library/Homebrew/formula.rb:2462:in `system'
	from /opt/homebrew/Library/Taps/yonaskolb/homebrew-swaggen/Formula/SwagGen.rb:34:in `install'
	from /opt/homebrew/Library/Homebrew/build.rb:177:in `block (3 levels) in install'
	from /opt/homebrew/Library/Homebrew/utils.rb:605:in `with_env'
	from /opt/homebrew/Library/Homebrew/build.rb:139:in `block (2 levels) in install'
	from /opt/homebrew/Library/Homebrew/formula.rb:1326:in `block in brew'
	from /opt/homebrew/Library/Homebrew/formula.rb:2692:in `block (2 levels) in stage'
	from /opt/homebrew/Library/Homebrew/utils.rb:605:in `with_env'
	from /opt/homebrew/Library/Homebrew/formula.rb:2691:in `block in stage'
	from /opt/homebrew/Library/Homebrew/resource.rb:158:in `block (2 levels) in unpack'
	from /opt/homebrew/Library/Homebrew/download_strategy.rb:116:in `chdir'
	from /opt/homebrew/Library/Homebrew/download_strategy.rb:116:in `chdir'
	from /opt/homebrew/Library/Homebrew/download_strategy.rb:103:in `stage'
	from /opt/homebrew/Library/Homebrew/resource.rb:154:in `block in unpack'
	from /opt/homebrew/Library/Homebrew/mktemp.rb:77:in `block in run'
	from /opt/homebrew/Library/Homebrew/mktemp.rb:77:in `chdir'
	from /opt/homebrew/Library/Homebrew/mktemp.rb:77:in `run'
	from /opt/homebrew/Library/Homebrew/resource.rb:266:in `stage_resource'
	from /opt/homebrew/Library/Homebrew/resource.rb:153:in `unpack'
	from /opt/homebrew/Library/Homebrew/resource.rb:127:in `stage'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/forwardable.rb:230:in `stage'
	from /opt/homebrew/Library/Homebrew/formula.rb:2671:in `stage'
	from /opt/homebrew/Library/Homebrew/formula.rb:1319:in `brew'
	from /opt/homebrew/Library/Homebrew/build.rb:133:in `block in install'
	from /opt/homebrew/Library/Homebrew/utils.rb:605:in `with_env'
	from /opt/homebrew/Library/Homebrew/build.rb:128:in `install'
	from /opt/homebrew/Library/Homebrew/build.rb:229:in `<main>'

mint
mint install yonaskolb/SwagGen
🌱 Finding latest version of SwagGen
🌱 Cloning SwagGen 4.7.0
🌱 Resolving package
error: 'github.com_yonaskolb_swaggen': Invalid manifest
/private/var/folders/s1/m32136tj39bgvc264bjg_89w0000gq/T/mint/github.com_yonaskolb_SwagGen/Package.swift:2:8: error: no such module 'PackageDescription'
import PackageDescription
       ^
🌱 Encountered error during "swift package resolve". Use --verbose to see full output
🌱  Failed to resolve SwagGen 4.7.0 with SPM

make
make install
swift build --disable-sandbox -c release --arch arm64 --arch x86_64
error: xcbuild executable at '/Library/Developer/SharedFrameworks/XCBuild.framework/Versions/A/Support/xcbuild' does not exist or is not executable
make: *** [build] Error 1

Fixed by this: https://github.com/yonaskolb/SwagGen/issues/311


refactor formatter (#2)

if file opened whilst unsaved changes in previous file, the saved indicator remained on unsaved.

fixed #2, closes #2


wip

Packages impacted by this PR
Issues associated with this PR
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

/azp run js - event-hubs - tests

/azp run js - data-tables - tests


Fixing terrible camera code with less suspicious maths

Fixed issue where the wrong thing was transposed (the invert matrix function needed to be transposed) Fixed issue where transposing the matrix with the source and destination as the same would lead to incorrect answer.

Whoops, need to pull into broken-mouse-test, not what-the-heck.


Django database transaction duration has 0 duration

How do you use Sentry?
Sentry Saas (sentry.io)
Version
1.5.8
Steps to Reproduce

Use sentry 1.5.8 with latest django 3.2
Enable django integration and transactions
Create a transaction of one request with database queries

Expected Result
Queries last anything from 1ms to 20ms depending on whatever the query is
This is what happens in 1.5.7
Actual Result
everything takes 0.01ms to 0.02ms

@fredgido I'd need some sort of minimal reproduction to take a look at this.

@fredgido I'd need some sort of minimal reproduction to take a look at this.

https://gitlab.com/fred.fernandes/minimal_sentry/-/tree/master/app
Built a simple example here

1.5.8

1.57
do you need my traces ids?

uhh thanks for the report @fredgido, this is a regression caused by https://github.com/getsentry/sentry-python/pull/1368 because we got rid of the yield in the context manager so the time is measured incorrectly now.
will try to fix soon.

Hey @fredgido !
@sl0thentr0py has created a fix for this and we will release this on Monday.
Thanks again for reporting this. If you send me your mail address to anton [dot] pirker [at] sentry {dot} io I can send you a couple of stickers as a small "Thank you" for your contribution!

@fredgido should be fixed in 1.5.9, can you give it a try and reopen if persists?


normalise luatex missing character message

luatex uses a different format for missing character messages
! Missing character: There is no ^^c5 in font cmr10!
! Missing character: There is no ^^c5 (U+00C5) in font cmr10!

This seems to fix it (just do two ^^ version that could also be in etex.)
$ svn diff l3build.lua
Index: l3build.lua
===================================================================
--- l3build.lua (revision 5568)
+++ l3build.lua (working copy)
@@ -593,6 +593,11 @@
     line = string.gsub (
         line, "I''m going to assume", "I'm going to assume"
       )
+    -- LuaTeX Missing Character message includes U+ notation.
+    line = string.gsub (
+        line, "Missing character: There is no (%^%^..) %(U%+(....)%) in font",
+              "Missing character: There is no %1 in font"
+      )
     return line
   end
   local newlog = ""

Now what we want here is a pull request ;-)
Seriously, will add in a second.

Fixed by SVN r5573.


Native Support For 64bit ARM in fabric

With the introduction of the M1 ARM chip for Apple Macbooks plus the increasing interest in ARM as an alternative to Intel it would be good if fabric supported and published native ARM binaries
M1 Mac users at the moment can make use of Rosetta 2 to use fabric (although the bootstrap script currently fails to pull down binaries and images as it looks for a non-existent ARM architecture in the image name or binary package)
On top of Mac book support it would also facilitate being able to run fabric nodes on 64bit raspberry pi 4 distros which seems to be quite popular for IOT projects

Guys, I find a solutions, refer to https://jira.hyperledger.org/browse/FAB-18389?page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel&showAll=true

Note that a fix has been merged in bootstrap script to pull down amd64 when using M1 Mac. Got confirmation that this approach works.

Confirmed that with current (v2.4) Fabric release and current fabric-samples, these things work:

Fabric install using install script.
Create test-network
Deploy Go chaincode
Run Go, Node and Java client applications

What doesn't work is deploying Node and Java chaincode using the test-network/network.sh script, with suitable nodeenv and javaenv Docker images not found:
Error: chaincode install failed with status: 500 - failed to invoke backing implementation of 'InstallChaincode': could not build chaincode: docker build failed: docker image build failed: docker build failed: Failed to pull hyperledger/fabric-nodeenv:2.4: no matching manifest for linux/arm64/v8 in the manifest list entries

Error: chaincode install failed with status: 500 - failed to invoke backing implementation of 'InstallChaincode': could not build chaincode: docker build failed: docker image build failed: docker build failed: Failed to pull hyperledger/fabric-javaenv:2.4: no matching manifest for linux/arm64/v8 in the manifest list entries

I already solve this question on my mac m1.
https://juejin.cn/post/7175001573045567548/

I already solve this question on my mac m1.
https://juejin.cn/post/7175001573045567548/

I'm not sure the arm64 Fabric Docker images you are using are a good solution for current Fabric versions, given that they were published three years ago and include this message:
"NOTE: These images were created as part of a project at Linköping University, they have not been tested in detail and will not be maintained."

All set!  Additional details and discussion thread at #3876
Release pipeline is now set up to generate arm64 and amd64 builds for binaries and docker images.  We should also be OK extending with additional arch specs (ppc, etc.) with minimal updates, or the next time Steve Jobs reaches out from the grave to remove a few decibels from a laptop fan.
Really looking forward to seeing this used for IoT, cel, and broadcom/Pi devices.
Happy hacking.


Use OSError, not IOError

Description
Use OSError, not IOError
OSError is the builtin error type used for exceptions that relate to the
operating system.
In Python 3.3, a variety of other exceptions, like WindowsError were
aliased to OSError. These aliases remain in place for compatibility with
older versions of Python, but may be removed in future versions.
Prefer using OSError directly, as it is more idiomatic and future-proof.
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

/lgtm

/approve


Fixed annotations docs

Describe your changes :
#1056 #1057. Changed the annotation documentation to the appropriate entities.

Type of change :

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[x] I have tagged my reviewers below.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
@sureshms @harshach

@sureshms RIght, sorry. I've fixed it or DashboardServiceResource.java and DatabaseServiceResource.java.

Thank you @ReyhanPatria for the contribution!


Reserve Driver Management

Allow full time drivers the ability to say they won't be driving for a race so we can fill with reserves.

It would be awesome to be able to mark which days in the future somebody will be out (kind of when you declare OOO in a google calendar) so it will automatically flag the driver as "not racing".
For example, I already know that next season I will miss probably 2 races because of vacations.


Remove waitForMappingUpdateOnAll

All dynamic mapping updates are done in a dedicated atomic action and is a blocking operation, we don't need this method.
For waiting for pending tasks we have waitNoPendingTasksOnAll()
See also https://github.com/crate/crate/pull/14936

Did you give this a couple of iterations?
The atomic update + acknowledge only ensures the other nodes got the message, but we afaik still have the Schemas/DocTableInfo cache and invalidation might lag a bit.
I'm not sure if that's actually a problem, but just to be sure we aren't making things flaky with this. Would be nice if we can remove it.

Did you give this a couple of iterations?

Locally it was fine, added more iterations (will revert before merging) to some tests (but not SqlTypeMappingTest which is already flaky), passed on CI as well.

The atomic update + acknowledge only ensures the other nodes got the message, but we afaik still have the Schemas/DocTableInfo cache and invalidation might lag a bit.

Thanks for pointing it out. I'm now thinking if we should invalidate cache in general whenever a node gets index update ==> whenever it acks somewhere in IndicesClusterStateService.updateIndices. Afaik that's an entry point for receiving an update from the master and maybe it could help with LR/SqlTypeMappingTest flakiness). Wdyt?

Thanks for pointing it out. I'm now thinking if we should invalidate cache in general whenever a node gets index update ==> whenever it acks. We can do it somewhere in IndicesClusterStateService.updateIndices. Afaik that's an entry point for receiving an update from the master and maybe it could help with logical-replication-mapping-related-tests/SqlTypeMappingTest flakiness). Wdyt?

I wouldn't spend any time on this now but instead tackle it as part of https://github.com/crate/crate/issues/11939 in a more holistic way. The Table representations we use could be more tightly integrated into the ClusterState instead of being a decoded view onto it that lags behind.


Allow lossy for PNG / automatic selection of lossy/lossless

See https://wordpress.org/support/topic/lossy-webp-option/#post-11292849

lossless now defaults "auto" for png, but false for jpeg


Ppx_annot

This PR adds a PPX for extracting the string-typed annotations on fields of a record type definition. For example,
type t =
  {a: Foo.t (** doc-comment *) [@annot "hello"]; b: Bar.t} [@@deriving ann]

will create a value t_annots that maps from the field name to the list of annotations.
This PPX is meant to be used for pulling annotation data into OCaml user-space for data-type generic programming (like the derivers in #10132 )

Can the PR comment describe how this ppx is meant to be used?

Can we also pull in all nullary annotations on the fields too? If we end up going with the more domain-aware version of this tool for derivers: [@skip] and [@deprecated] would be the two that we want.

Tweaked the syntax and behaviour slightly. See the tests for the new usage and output.
In particular, the returned values are now t_toplevel_annots : unit -> (string * string option) and t_fields_annots : string -> (string * string option). This option is None for annotations without a string argument.


Map of Maps is incorrectly parsed

When given original.txt to picojson and serialized to string again, and manually curated to easily see the problem, we obtain reserialized.txt, which lacks information of the original JSON.
This issue is not problem of the serialization, but the parsing, as the object lacks this information too.
original.txt
reserialized.txt

Sorry, as seen the JSON contains a trailing comma that is producing the error. Closing the issue.


[top / ast] added ast documentation

@arnonsha
hey ziv, looks like there's a swp file that was also accidentally committed, can you remove it? (just do git rm on that file and commit).
Also, you have some CI failures, but these are because when you commit, you need to do a git commit -s to sign it with an author string.  Arnon I think also ran into similar issues, so feel free to check with him also.

Done.
BTW, I ran with '-s' last time. I don't know what caused the CI failure.

@zi-v if you click on the Check tab above, it will tell you what you're doing wrong.
For example this is the error right now.
Commit 55d4ebb9f8b3b3a9c2b53b086c693084e8596e7c: The commit author name "Ziv" contains no space. Use "git config user.name 'Johnny English'" to set your real name, and update the commit with "git rebase -i " and/or "git commit --amend --reset-author".

Hmm. Thinking about this, I wonder whether we need to fix this check: "Johnny English" is brilliantly anglo-centric (it's even in the name!) and we might have some contributors who genuinely have just a single name. I'll file an issue to track this.

thanks Rupert!
On Tue, Sep 22, 2020 at 1:28 AM Rupert Swarbrick<EMAIL_ADDRESS>wrote:

Hmm. Thinking about this, I wonder whether we need to fix this check:
"Johnny English" is brilliantly anglo-centric (it's even in the name!) and
we might have some contributors who genuinely have just a single name. I'll
file an issue to track this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/2997#issuecomment-696581276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSQT5XWYTYSIA676JATSHBNY7ANCNFSM4PKSYYUA
.

For anyone following in the PR and getting very confused: my first version of https://github.com/lowRISC/opentitan/pull/2997#issuecomment-696581276 (quoted in Tim's reply) was when I thought the messages about "Johnny English" caused errors. I did a bit of digging, and that's not true (so I updated the comment).
I'm convinced that this warning makes sense, because it catches cases where someone has their unix username as their "name" in Git. This is quite common, especially with people who have used Subversion in the past. And should be stamped out! :-D

Hi @zi-v, for OpenTitan we ensure all commits have a Signed-off-by line (which should use your full name) and we try to keep a clean commit history, meaning intermediate fix-up commits that occur during review are squashed. We have some documentation on this here.
To go from the current commit to a ready-to-merge (once reviewed) PR, the aim would be to squash your 3 commits into one and adjust the commit message appropriately, ensuring it contains a Signed-off-by: Ziv Hershman<EMAIL_ADDRESS>line at the end. This can be done by (from a local checkout of your ast_doc branch):

git rebase -i HEAD~3
(this opens a text editor with the last 3 commits. Change the "pick" on the second and third lines to "squash" or "s"
(a text editor now opens with the draft commit message for the squashed commit. Clean this up so it reads as a normal commit message and ensure it ends with the proper Signed-off-by line)
git push -f to update the PR

Do git config user.name 'Ziv Hershman' to ensure future git commit -s puts the Signed-off-by line in the preferred form.
I hope that helps - any questions or problems please let us know.

@asb , thanks!
I force pushed an updated version. I hope it is cleaner.

looks good @ziv! all the self checks have passed.
Is there anything else you want to change? If not we should work on merging this.

No pending changes

Looks like the rebase didn't fully work out according to plan: there's a second commit in there with some RTL changes (and images which are deleted and re-added later, but that's more of a cosmetic problem).
@msfschaffner I'm not sure where those other changes came from, can you have a look if they are OK to be merged as they are now, or if we should back them out?

The diff of the first commit is... weird. The files have already been present in the tree (added about 3 months ago), so I don't get why they show up as being added in that commit.

Oh I see now why. This is a white-space only change which introduces Windows newlines in all those files. We should revert that.


Replaced Firestore Writes with Firebase Cloud Function REST Endpoint

@madim awesome work! Builds perfectly, tested on LG G6 and iPhone 11 Pro Max works no problems.

@madim can you disable the second two PR requests and merge this. Unfortunately, I cannot disable them for you.


Enable the TinyMCE 'lists' plugin as this functionality was dropped from the core

TinyMCE 4.5 removed bullist and numlist from the core, meaning that these buttons disappeared from the editor in the product edit dashboard view (see https://github.com/tinymce/tinymce/issues/3342). This patch enables the lists plugin so that these buttons become available again.

Current coverage is 84.24% (diff: 100%)

Merging #2184 into master will not change coverage

@@             master      #2184   diff @@
==========================================
  Files           304        304
  Lines         15091      15091
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits          12713      12713
  Misses         2378       2378
  Partials          0          0

Powered by Codecov. Last update b161403...84ccb19


Document how to get MAC address of device from dbus

 Front conversations

Adding it to patch/beginner considering MAC addresses are now available on the dashboard, so maybe the process to get MAC address from DBUS might be simpler if not easier.

this now can be retrieved from the supervisor API: https://www.balena.io/docs/reference/supervisor/supervisor-api/#get-v1device. can we just close this?


next_token changed to pagination_token

Code returned only the first page of followers and get an error 400 on the second page request.
Changing params to currently used pagination_token fixed an issue.

I have read the CLA Document and I hereby sign the CLA


Add config option for custom ntp servers

For my current client we discussed having the ntp servers configured at boot time due to a perceived security issue with public ntp pools.  This can be done after the install, but adding here saves the extra steps.
https://github.com/RedHatOfficial/ocp4-vsphere-upi-automation/pull/32

https://github.com/RedHatOfficial/ocp4-vsphere-upi-automation/pull/33


i18n Formatter::asRelativeTime() not working

I have a problem with method asRelativeTime().
I try:
<?php
echo "PHP: " . PHP_VERSION . "\n";
echo "ICU: " . INTL_ICU_VERSION . "\n";
echo Yii::$app->formatter->asDate('2014-01-01');
echo Yii::$app->formatter->asRelativeTime('2014-01-01');
?>

and get:
PHP: 5.6.11
ICU: 54.1

1 янв. 2014 г.
a year ago

Why does the "asRelativeTime" does not return an answer in Russian?

Do you have sourceLanguage as default and language as ru_RU?

yes,
        'language' => 'ru-RU',
        'sourceLanguage' => 'ru-RU',
        'formatter' => [
            'class' => 'yii\i18n\Formatter',
            'nullDisplay' => ''
        ]

Well, you have both source and target language the same which means translation won't happen.

no,
        'language' => 'ru_RU',
        'sourceLanguage' => 'en_US',
        'formatter' => [
            'class' => 'yii\i18n\Formatter',
            'nullDisplay' => ''
        ]

also won't work

Do you have file with translated string?

OK. It works well for me producing
01 янв. 2014 г.
год назад

My language is set to ru-RU. Source language is left as default.

It's not connected to ICU version for sure.

Aha, you've used ru_RU while it should be ru-RU.

no) I try to use ru-RU, its not works for me.
for this method need a translation file with translate string?
eg:
Yii::t('yii', 'in {delta, plural, =1{a year} other{# years}}', ['delta' => $interval->y], $this->locale)

I thought this method works as asDate()?

It uses yii category for which translations are bundled with the framework. If you haven't changed message source for it, it should work.

I have a clean install yii2 from composer. (Windows, XAMPP)
Nothing changes, the translation files are not created. The config file contains rows:
        'language' => 'ru-RU',
        'sourceLanguage' => 'en-US',
        'formatter' => [
            'class' => 'yii\i18n\Formatter',
            'nullDisplay' => ''
        ]

asDate() working, but asRelativeTime() not.
What could be the reason? Any ideas?

Maybe this method requires the translation files?
The file \yii\i18n\Formatter have this method, and it has a line, maybe they need to be translated? Or all the same method should work without translation files? As a method asDate?

I have just tried to reproduce this, no luck. Works fine for me... Translations are provided by yii so there should not be anything to add from your side...

what have you set in $app->i18n?

Yii::$app->i18n a big object.
What particularly interested in?

if you empty it, will it translate?

how to clean it? he is read-only

dont set it in config

yes! its working

sorry, that's my fault

:+1:


Update to work with expressive 3

Update to https://github.com/zendframework/zend-expressive/releases

seems to have landed in master, thanks

:fireworks:


Same line of code written twice

Same line of code is written twice in Main Activity. Here is a screen shot of it.

I would like you take this issue and remove the extra line

@ahsan44411 Yes, please work on this.


Open right page when callback was clicked

Expected Behaviour
When user click in some button on notification, open the right page for this callback
Actual Behaviour
The notification handled with
push.on('callback').subscribe((notiication) =>
    this.router.navigate(['/flightSearchResults');
 })

Reproduce Scenario (including but not limited to)
When app is closed and user click on the button, the app doesn't open.
When app is open and user click on the button, the page doesn't change.
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android 6.3.0
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Xiaomi Mi6
Cordova CLI version and cordova platform version
cordova --version

8.0.0
Plugin version
cordova plugin version | grep phonegap-plugin-push

2.2.2
Sample Push Data Payload
{
    "to": "xxx",

    "content_available": true,
    "data": {
        "title": "Sua chance de viajar é de 92%",
        "message": "Gostaria de alterar sua jogada?",
        "click_action": "FCM_Acao_principal",
        "actions": [
          { "icon": "emailGuests", "title": "Sim", "callback": "app.emailGuests", "foreground": true},
          { "icon": "snooze", "title": "Comprar Agora", "callback": "app.comprarAgora", "foreground": false},
          { "icon": "snoozez", "title": "Melhorar Jogada", "callback": "app.melhorarJogada", "foreground": false}
    ]
  }
}

Sample Code that illustrates the problem
Logs taken while reproducing problem

@fredgalvao  (Inhaê menn 🥇)
The callback value I put as an example, I am capturing a push and it is yes calling this event.

So, if it's indeed calling the event you want, what's the issue then?

@ttibau the action button for app.comprarAgora is set to foreground false so it won't bring the app to the foreground. Try setting that to true if you want the app to open and navigate to the correct page.

@macdonst Thanks! It's Works.  ❤
When I set true in foreground the action that's works!
{ "icon": "snoozez", "title": "Comprar agora", "callback": "app.comprarAgora", "foreground": true}


Add new admin services to Glossary author view

Add new admin services to Glossary author view
1- addBulkTerms. this will be an API that will take an object of shape:
{
glossaryGuid: {glossaryguid}
terms: [ {name:{name}, description:{description}, ... ]
}
It will return an object which is the same as the request, but each Term will have addition

status of success, error
an optional msg, which is filled in on error
the guid of the created term.

2- Add a rest call that will take a payload of, getBreadCrumb
{ glossaryGuid: {glossaryGuid},
categoryGuid : { categoryGuid},
termGuid : { TermGuid} }
The termGuid field is optional.
This rest API returns the breadCrumbPath of the termGuid if specified or the categoryGuid. something like :
[
{type:glossary, guid:aaa-bbb, name="main glossary"},
{type:category, guid:ccc-bbb, name="cat1"},
{type:category, guid:ddd-eee, name="cat2"},
{type:term, guid:ddd-esee, name="term1"},
]

I am developing the addBulkTerms call in the view service. I realised that I wanted to track the responses of each Term  create and also return a response object in case the overall course grained API failed.
I started looking at an API Shape of
`   @PostMapping(path = "/{guid}/terms")
public SubjectAreaOMASAPIResponse<SubjectAreaOMASAPIResponse> createBulkTermsInAGlossary(@PathVariable String serverName,
@PathVariable String userId,
@PathVariable String glossaryGUID,
@RequestBody  Terms[] terms
                                                                                ) {
  ....
}`

The server does not accept any calls to this. @immmus  Any thoughts on what I am doing wrong? The alternative is that I do not reuse SubjectAreaOMASAPIResponse in the response, but create other objects maybe a TermResponses object containing TermResponse elements. With TermResponse containing Term and a response status.

@immmus it looks like a typo guid does not match glossaryGUID!

@immmus it looks like a typo guid does not match glossaryGUID! Working now.

Yes, problem with parameter name (glossaryGUID). You need to change name to "guid'.

@immmus it looks like a typo guid does not match glossaryGUID! Working now.

Yes, problem with parameter name (glossaryGUID). You need to change name to "guid'.

Thanks - as soon as I put up the question I spotted the issue! I hope you are ok to review the PR; it might be a pattern that we could use more widely.

Separating the issues out so this issue just does addMultipleterms


Please add support for environment to cloudflare_worker_route

Current Terraform and Cloudflare provider version
Terraform v1.3.7
on windows_amd64

provider registry.terraform.io/cloudflare/cloudflare v4.6.0

Description
Please add support for selecting worker environment to cloudflare_worker_route
Use cases
We have one domain with one zone, but it serves 2 environments, so testing workers separately would be nice.
Eg. www.staging.domain.com and www.domain.com
Potential Terraform configuration
resource "cloudflare_worker_route" "staging_route" {
  zone_id = var.cloudflare_zone_id
  pattern = "www.staging.domain.com/*"
  environment = "staging"
  script_name = var.script_name
}

resource "cloudflare_worker_route" "prod_route" {
  zone_id = var.cloudflare_zone_id
  pattern = "www.domain.com/*"
  environment = "production"
  script_name = var.script_name
}

References
No response

Service environments were deprecated a long time ago and basically removed at this point. Environments in Workers are back to appending the env on to the Worker name
Are you talking about the service environments or the Worker name ones?
I'd personally be hesitant to add an environment field until we properly reimplement actual environments.

I think I mean Environments in Workers. The one you can specify in the Dashboard when adding a Worker Route.
Can that be done by adding a special suffix to the worker name?

Yeah, that's service environments, you aren't actually able to create new environments anymore and they're being removed (slowly).

I see, that explains it, thank you


Migrate cleanup script for old npm packages from graphql to rest

Description
Migrate the script that cleans up old npm packages from the graphql API to the rest API of Github. The graphql API is deprecated for packages (See: https://github.blog/changelog/2022-08-18-deprecation-notice-graphql-for-packages/).
Self Check:

[ ] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation

Processing this pull request

Failed to merge changes into iso4 due to merge conflict. Please open a pull request for these branches separately by cherry-picking the commit that was made on the branch master (git cherry-pick 2ee3f189f4deaf31ac1d97c557ab7c51cce1fee5).

Not closing this pull request due to previously commented issues for some of the destination branches. Please open a separate pull request for those branches by cherry-picking the relevant commit. You can safely close this pull request and delete the source branch.

This branch was not deleted as it seems to still be in use.


[FIX-7732][fix] fix column 'is_directory' of table t_ds_resources type error in PG database

Purpose of the pull request
fix column 'is_directory' of table t_ds_resources type error in PG database

Brief change log

alter column from int to boolean. In the origin Resource.java ,isDirectory has a default value, so in the init sql set column default false
Format class Resource.java

can close #7732

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #7898 (bb1187f) into dev (57f000e) will decrease coverage by 0.00%.
The diff coverage is 47.56%.

@@             Coverage Diff              @@
##                dev    #7898      +/-   ##
============================================
- Coverage     40.98%   40.98%   -0.01%
+ Complexity     3694     3693       -1
============================================
  Files           637      637
  Lines         27058    27058
  Branches       3073     3073
============================================
- Hits          11090    11089       -1
  Misses        14899    14899
- Partials       1069     1070       +1

Impacted Files
Coverage Δ

...g/apache/dolphinscheduler/dao/entity/Resource.java
47.56% <47.56%> (ø)

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

Continue to review full report at Codecov.

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


Remove React Native 57+ ListView deprecation warnings

fixes https://github.com/APSL/react-native-keyboard-aware-scroll-view/issues/316

Could this get merged and a new version released?

https://github.com/APSL/react-native-keyboard-aware-scroll-view/releases/tag/v0.8.0


Add a few root files to lint-spell

Files in root were not linted, add them. No new violations.

Should we spellcheck the changelog or do we consider it "immutable"? I'd say yes, we can change it.

I'd say yes, we can change it.

I agree

Applied to changelog as well now and ran make lint-spell-fix. Also the wildcard is removed, which will give better Makefile performance.

Ideally there would be a way to also lint commit messages during PR checks, not sure how to do it though, at least not without a temp file 😆.

Ideally there would be a way to also lint the PR title during PR checks, not sure how to do it though, at least not without a temp file and then there's the issue that CI does not run after commit title change.

New @GiteaBot functionality? 🤔

Ideally there would be a way to also lint the PR title during PR checks, not sure how to do it though, at least not without a temp file and then there's the issue that CI does not run after commit title change.

New @GiteaBot functionality? 🤔

Likely will be hard to run the misspell go spellchecker in the bot which is typescript. If there is a similar or stricter spellcheck tool that the bot could use, then yes, it would likely be rather easy to implement.
cc: @yardenshoham.

hmmmm, maybe with https://www.npmjs.com/package/cspell

I think I may actually exclude changelog again because I don't want to put the burden onto the releaser to fix typos that they haven't made.

hmmmm, maybe with https://www.npmjs.com/package/cspell

On topic: https://github.com/crate-ci/typos looks interesting, but it's rust which we have not integrated yet as tooling language.

CHANGELOG.md is excluded again. I think we can at least keep the typo fixes in it, so it's a one-time correction basically. The wildcard does not seem to have any impact on Makefile performance, so I kept it.


Porting Fixes to Ansible installer on release-1.0 branches

Hello, everyone
This PR port back to release-1.0, some issues addressed

Porting #7735 on release-1.0 branch
Also exposing routes to Grafana, Istio-Ingress Gateway and Prometheus (Openshift related)

This PR fixes #8730
Best Regards,
Guilherme Baufaker Rêgo

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: andraxylia
If they are not already assigned, you can assign the PR to them by writing /assign @andraxylia in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

cc @sdake

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gyliu513
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [gyliu513]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


EM and F1 scores

What were your EM and F1 scores on SQuAD 2.0?

I run 7 epoch but the F1 score is higher than 70%. I wonder if the model maybe wrong


Is it possible to deploy to github pages?

Wondering how I can deploy a nextra site to github pages, I know it can deploy to vercel, but I am looking to deploy html pages to github pages.

@FriendlyUser Nextra converts the code used in the repo to html pages.
Why would you want to deploy html pages?
But to answer your question, just create a Github Action to do that or browse through the Github Actions marketplace

can you let me know which folder hosts the static build that can be deployed to github pages? I couldnt find it.

 The action workflow file should be stored in the .github/workflows directory to work.
Your deployed static website can be found on Vercel if you're using Vercel to deploy.
To locate it, just visit the latest deployment, then navigate to the Source tab and look through. I haven't figured a way to download the source so you'll have to copy and paste everything.

I am trying to run https://nextjs.org/docs/advanced-features/static-html-export#next-export
unless nextra automatically makes a set of static html pages, seems to be it just produces a next production build which can be deployed to vercel, but requires a next server to run.
I can run next run build manually, getting the errors above when trying to produce html pages with next export.

Are you deploying from your local machine/PC?

So I can deploy from my computer or in a github action, I do not want to deploy to vercel as there is a custom domain I want to use thats tied to github already.

@FriendlyUser Just create an action then.
Below are links to some resources you can use:

https://dev.to/franciscojarceo/deploying-a-next-js-site-using-github-actions-1g1e
https://franciscojavierarceo.github.io/post/github-actions (Practically same as above)
https://itnext.io/deploy-next-js-apps-using-github-actions-6322261757bc (You could ignore useless details)
https://fedeantuna.github.io/article/deploy-nextjs-app-to-github-pages

Hope this helps

See goes back to point 1, next export throw an error as I cannot configure disabling image optimization.

So I can deploy from my computer or in a github action, I do not want to deploy to vercel as there is a custom domain I want to use thats tied to github already.

You can deploy from your machine or Github Actions

Can you provide a link to an example repo?

https://github.com/FriendlyUser/latex-diagrams-docs/blob/main/next.config.js

Try this code and see if it works
const withNextra = require('nextra')({
  theme: 'nextra-theme-docs',
  themeConfig: './theme.config.tsx',
})

module.exports = {
  ...withNextra(),
  images: {
    unoptimized: true,
  },
};

Paste it in your next.config.js file

seems to work, thanks for your help.


.currentTime() returns 0 at all times

/** [p5.sound]  Version: 0.3.12 - 2020-01-06 */
This might be related to the following issue (As it showed up in my code at the same time): https://github.com/processing/p5.js-sound/issues/506
When calling .currentTime() on a loaded sound (.loadSound()), the .currentTime() method always returns 0. This began occurring either yesterday or the day before and I could not find anyone else who has submitted this as a bug.
Seems to happen on all current/stable versions of major browsers (Tested on Chrome and Firefox).
I'm testing on Chrome Version 84.0.4147.105 (Official Build) (64-bit)

@BrewHog  thanks for the patience, will look into the matter soon!

I'm having this issue, too. Though, I think it may be a race condition because 1 out of 20 times currentTime() behaves correctly for me. Looks like this._lastPos is always set to 0 in the currentTime implementation, and it appears the callback that sets this._lastPos is not firing.
https://github.com/processing/p5.js-sound/blob/master/src/soundfile.js#L1353-L1364

Figured it out! Here's a pull request to fix the issue, @endurance21
https://github.com/processing/p5.js-sound/pull/542

thanks again, @MrRoboman !

and thank you for reporting, @BrewHog

I am still having this issue, both with the sound library downloaded with from the main P5 site, as well as the more recent 1.0.0 version downloaded directly from GitHub. Why is this issue listed as 'closed'?

When I use FireFox the issue seems to go away, I only experience it when using Chrome.

Still returning zero on Brave browser


Add "Fracas" to game list

We just released a demo for our game using GodotSteam!
It would be awesome to have it on the documentation

Fantastic!  Thank you so much for adding your game!  I will also put it in our shout-list for socials!


MySQL time type not working

SQL
CREATE TABLE `User` (
	id integer PRIMARY KEY,
	time time,
);

Prisma Schema
model User {
  id        Int       @id
  time      DateTime?
}

Code
const date = new Date('2015-01-01T00:00:00Z')

const data: UserCreateInput = {
	id: 1,
	date: date,
	datetime: date,
	time: date,
	timestamp: date,
}

await client.user.create({
	data,
})

reproduction available at https://github.com/prisma/datetime-experiments
ConnectorError(ConnectorError { user_facing_error: None, kind: ConversionError(Custom { kind: InvalidData, error: "DateTime value not stored as datetime, int or text" }

   0: backtrace::backtrace::trace
   1: backtrace::capture::Backtrace::new_unresolved
   2: failure::backtrace::internal::InternalBacktrace::new
   3: <failure::backtrace::Backtrace as core::default::Default>::default
   4: sql_query_connector::row::row_value_to_prisma_value
   5: <quaint::connector::result_set::result_row::ResultRow as sql_query_connector::row::ToSqlRow>::to_sql_row
   6: <std::future::GenFuture<T> as core::future::future::Future>::poll
   7: <std::future::GenFuture<T> as core::future::future::Future>::poll
   8: std::future::poll_with_tls_context
   9: <std::future::GenFuture<T> as core::future::future::Future>::poll
  10: <std::future::GenFuture<T> as core::future::future::Future>::poll
  11: <std::future::GenFuture<T> as core::future::future::Future>::poll
  12: <std::future::GenFuture<T> as core::future::future::Future>::poll
  13: <std::future::GenFuture<T> as core::future::future::Future>::poll
  14: <std::future::GenFuture<T> as core::future::future::Future>::poll
  15: <std::future::GenFuture<T> as core::future::future::Future>::poll
  16: <std::future::GenFuture<T> as core::future::future::Future>::poll
  17: std::future::poll_with_tls_context
  18: <std::future::GenFuture<T> as core::future::future::Future>::poll
  19: <std::future::GenFuture<T> as core::future::future::Future>::poll
  20: std::panicking::try::do_call
  21: __rust_maybe_catch_panic
  22: <std::future::GenFuture<T> as core::future::future::Future>::poll
  23: tokio::task::core::Core<T>::poll
  24: std::panicking::try::do_call
  25: __rust_maybe_catch_panic
  26: tokio::task::harness::Harness<T,S>::poll
  27: tokio::runtime::thread_pool::worker::GenerationGuard::run_task
  28: tokio::runtime::thread_pool::worker::GenerationGuard::run
  29: std::thread::local::LocalKey<T>::with
  30: tokio::runtime::thread_pool::worker::Worker::run
  31: tokio::task::core::Core<T>::poll
  32: std::panicking::try::do_call
  33: __rust_maybe_catch_panic
  34: tokio::task::harness::Harness<T,S>::poll
  35: tokio::runtime::context::enter
  36: std::sys_common::backtrace::__rust_begin_short_backtrace
  37: std::panicking::try::do_call
  38: __rust_maybe_catch_panic
  39: core::ops::function::FnOnce::call_once{{vtable.shim}}
  40: <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once
  41: std::sys::unix::thread::Thread::new::thread_start
  42: _pthread_start
) })

I have same issue.
Any updates ?

Is it exactly the same error @ghyath5 ? We did some work on this, and time won't work in all cases because MySQL is ambiguous about what this type represents, it can be a duration or a time of the day. Prisma is currently opinionated and treats it as a time of the day, so negative values or values greater than 24 hours won't work.

@tomhoule
I'm using Postgres database
I'm trying to select some fields from prisma like:
   const select = new PrismaSelect(info).value
let data = await ctx.prisma.users.findMany({       ...args,       ...select     })
  return data
Then this error show me: DateTime value not stored as datetime, int or text
Knowing that when I was using includes instead of select there were no errors

Hmm this is a different bug then, we should look into it. Can you open an issue? :pray:

Well, I will

Unable to reproduce this crash with 2.21.0-dev.37 but problem reported in https://github.com/prisma/prisma/issues/2734 is reproducible so closing in favour of that.


create custome css style

This custom style follows basic resume norms with clean formatting.

@surapuramakhil Would you like to evaluate this new style?

@mhmoussatche, can you attach generated sample resume with this style


trt_inf.py tensorrt推理加速出现orig_target_sizes不匹配的错误无法成功推理

我在训练的时候，将图像大小由640改为了1280，修改了图像大小相关的适配为1280,其他的没改,成功训练完毕
训练好之后保存为best_stg2.pth，然后用官方的代码和命令转成best_stg2.onnx
再将onnx用官方的命令转成engine文件，trtexec --onnx="best_stg2.onnx" --saveEngine="best_stg2.engine" --fp16
最后运行官方的trt推理文件python tools/inference/trt_inf.py --trt best_stg2.engine --input image.jpg
结果就报错了，如下错误，Traceback (most recent call last):
File "/localdata/lj/code/D-FINE/tools/inference/trt_inf.py", line 226, in
process_image(m, file_path, args.device)
File "/localdata/lj/code/D-FINE/tools/inference/trt_inf.py", line 151, in process_image
output = m(blob)
File "/localdata/lj/code/D-FINE/tools/inference/trt_inf.py", line 109, in call
return self.run_torch(blob)
File "/localdata/lj/code/D-FINE/tools/inference/trt_inf.py", line 99, in run_torch
assert self.bindings[n].data.dtype == blob[n].dtype, '{} dtype mismatch'.format(n)
AssertionError: orig_target_sizes dtype mismatch
我用tools/inference/torch_inf.py推理是成功的，而且效果相当不错（感谢作者们的工作）

torch_inf.py 140-143 resize the input image to 640*640.

thanks, but I run with trt_inf.py not torch_inf.py,torch_inf.py is good for me.

trt_inf.py 140-143 resize the input image to 640640.if you want infer 12801280,please check it.

trt_inf.py 140-143 resize the input image to 640_640.if you want infer 1280_1280,please check it.

yes, I changed the input to 1280, but whether I set it to 1280 or 640, I still get the same error. I suspect the issue might be related to a mismatch in data types, such as int32 or float32, during the model conversion process. All the conversion steps were followed strictly according to the README, but I am not very familiar with this project and am unsure how to troubleshoot.

you can change trt_inf.py 138 to  orig_size = torch.tensor([w, h], dtype=torch.int32)[None].to(device)

https://github.com/Peterande/D-FINE/pull/78/files

Thanks for thie PR, it works!


bug: 1.3.0 broke line comment toggle

Did you check docs and existing issues?

[X] I have read all the vscode-neovim docs
[X] I have searched the existing issues of vscode-neovim
[X] I can reproduce the issue with an empty init.vim/init.lua and no other vscode extensions (when applicable)

Neovim version (nvim -v)
v0.9.1
Operating system/version
Ubuntu 22.04.3 LTS
Describe the bug
Before 1.3.0, I could use C-/ to toggle line comment like vscode does without neovim. Version 1.3.0 opens the forward search instead, as if I typed / without ctrl. This affects the normal, visual and select modes. The insert mode still works.
This also happens on 1.4.0 but I didn't show it in the screencast as I had to install the vsix manually
https://github.com/vscode-neovim/vscode-neovim/assets/19656762/46a7ef1c-2a02-48cb-94b8-48d4415564fc
Steps To Reproduce

Use vscode and vscode neovim 1.3.0
open a source file like markdown, html, js...
use keyboard shortcut C-/
forward search is opened instead of line getting commented

Expected Behavior

Use vscode and vscode neovim 1.2.0
open a source file like markdown, html, js...
use keyboard shortcut C-/
line is commented

Show me

Please turn on neovimClean or search '' in your nvim config

I turned on Neovim Clean in the vscode extension options and reloaded and still have this behaviour with extension version 1.3.0 or 1.5.0. My ~/.config/nvim/init.vim only has this:
source ~/.vimrc

and ~/.vimrc
syntax on

" Highlights searches
set hls
set is

" Add line numbers
set number
" Relative line numbers
set rnu

" Set up tabs as spaces
set tabstop=4
set shiftwidth=4
set expandtab

" Auto indent on new line
set ai

" Hightlight trailing whitespace
highlight TrailingWhitespace ctermbg=red
:autocmd Syntax * syn match TrailingWhitespace /\s\+$/

I'll try replicating it in a completely clean VM to be really sure there's no config involved

@ohemelaar try map gcc

:map gcc shows No mapping found, both in VSCode or nvim with extension version 1.5.0.
With version 1.2.0 it shows this (sorry for the screenshot I can't select it)

@ohemelaar No idea what's happening.

Well I tried in a VM and noticed weird stuff with the neovim PPA. I switched to the AppImage install instructions and this works well with any extension version. Sorry for the really weird bug, and thanks for answering those questions :pray:

I have the same bug. I have neovim 0.9.1-1 installed from Prebuilt MPR


Dev

improve type-stability
add precompile commands to improve FFTX
more robust and clean API

Codecov Report

Merging #42 (1d0d7bb) into master (778f633) will decrease coverage by 1.20%.
The diff coverage is 75.72%.

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
- Coverage   64.60%   63.39%   -1.21%
==========================================
  Files          29       29
  Lines        1862     1795      -67
==========================================
- Hits         1203     1138      -65
+ Misses        659      657       -2

Impacted Files
Coverage Δ

src/FeynmanDiagram.jl
41.66% <ø> (ø)

src/strong_coupling_expansion_builder/Gc.jl
0.00% <0.00%> (ø)

src/strong_coupling_expansion_builder/Gn.jl
0.00% <0.00%> (ø)

src/strong_coupling_expansion_builder/common.jl
0.00% <ø> (ø)

src/diagram_tree/operation.jl
59.64% <56.75%> (-4.74%)
:arrow_down:

src/diagram_tree/tree.jl
55.12% <57.74%> (-34.16%)
:arrow_down:

src/parquet_builder/common.jl
56.45% <58.33%> (-0.27%)
:arrow_down:

src/diagram_tree/traits.jl
38.09% <66.66%> (-0.55%)
:arrow_down:

src/expression_tree/eval.jl
65.78% <66.66%> (-2.51%)
:arrow_down:

src/common.jl
60.95% <73.52%> (-1.91%)
:arrow_down:

... and 40 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


use tcmalloc explicitly without overriding system malloc/free

In our application, we want malloc/free calls using system ones(it's glibc here), and when we want to use tcmalloc, we can call tc_malloc/tc_free explicitly. However, after we linked to tcmalloc with -ltcmalloc_minimal, it seems all malloc/free have been replaced.
I searched with google but nothing useful found, is there any way to achieve this?
EDIT:
We are using a Linux box with kernel version 3.10, glibc version 2.17, gcc version 4.8.5, and tcmalloc is compiled with --enable-minimal flag, and is statically linked into our program.

I had a look at the code, and found the __attribute__(alias(...)) thing, and tried commenting out the following lines in tcmalloc.cc, and it now does not implicitly replace system malloc/free anymore:
// ...

using STL_NAMESPACE::vector;

// #include "libc_override.h"  // <--- commented out this line

using tcmalloc::AlignmentForSize;

// ...

TCMallocGuard::TCMallocGuard() {
  if (tcmallocguard_refcount++ == 0) {
    // ReplaceSystemAlloc();    // defined in libc_override_*.h // <--- commented out this line
    tc_free(tc_malloc(1));

// ...

However, I think this is just a DIRTY HACK, I still want to know if there is a more elegant solution.

Yes, we don't officially support this. But because our windows bits already add necessary ifdef-s, I think it should be easy to provide compile-time option to disable aliasing to operator new/delete & malloc/free/etc. So feel free to just propose patch for that.


Create dataset button Is not clicked

Describe the bug
Create dataset(dataflow) button Is not clicked
To Reproduce

Go to the list of dataset
Click on 'generate new dataset.'

Unable to click the button
Desktop (please complete the following information):

OS: all
Browser all
Version all

Additional context

Other modules can also cause design problems


BUILD: polish jenkins test script

Test PASSed.
See http://hpc-master.lab.mtl.com:8080/job/hpc-libverbs-tests-SimX-pr/20/ for details (Mellanox internal link).

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ibverbs-tests-pr/55/ for details. (Mellanox internal link)


JUST TO TEST: Revert "HARP-7808 Fallback has render order. (#1038)"

This reverts commit 27ef9afb62ebe32ab3681d0979704e6f041fd31d.

Codecov Report

Merging #1074 into master will decrease coverage by 0.02%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #1074      +/-   ##
==========================================
- Coverage   45.86%   45.84%   -0.03%
==========================================
  Files         238      238
  Lines       22174    22158      -16
  Branches     4983     4978       -5
==========================================
- Hits        10170    10158      -12
+ Misses      12004    12000       -4

Impacted Files
Coverage Δ

@here/harp-mapview/lib/MapView.ts
57% <ø> (-0.14%)
:arrow_down:

@here/harp-mapview/lib/DataSource.ts
81.01% <ø> (-0.24%)
:arrow_down:

@here/harp-omv-datasource/lib/OmvDataSource.ts
33.84% <ø> (-1.98%)
:arrow_down:

@here/harp-mapview/lib/VisibleTileSet.ts
69.97% <ø> (+0.02%)
:arrow_up:

@here/harp-mapview/lib/Tile.ts
63.23% <ø> (-0.13%)
:arrow_down:

@here/harp-mapview/lib/BackgroundDataSource.ts
97.56% <100%> (ø)
:arrow_up:

...e/harp-mapview/lib/geometry/TileGeometryCreator.ts
11.2% <80%> (-0.12%)
:arrow_down:

Continue to review full report at Codecov.

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


Feature/more main screen UI

Fixes: https://aeternity-blockchain.atlassian.net/browse/SW-61
UI Ref: https://www.figma.com/proto/3oGLWzSH0oJljo4RETZtur/Superhero-Wallet-UI-(✔Updated)?node-id=13969%3A315682&scaling=min-zoom&page-id=10439%3A144915&starting-point-node-id=10641%3A135489

Tested, LGTM.
Maybe this should be fixed (view of bell button):

Idk if this related to this pr:
I was not able to do a retip on superhero.com:

When I switched to the testnet and then to the mainnetwork, the wallet automatically switched me to testnet by default , which is incorrect.


Feature/single file structure

What?

Added basic structure for single file implementations.
Added requirements and dockerfile.

Why?

How?

Extra
Closes https://github.com/instadeepai/Mava/issues/885 .

Thanks for this @KaleabTessera, looks great!
Just one thing I would change is the filename ddpg to iddpg, just so it's not confused with a single agent version and to keep it consistent with ippo and idqn, but we can do this in our implementation smile

Thanks for the review @sash-a ! I am happy for the people doing the implementation to rename it. I just haven't seen iddpg anywhere, so maybe when implementing, people can research the name.

Thanks @KaleabTessera , looking forward to working on this. Just a quick thing though. I don't think we need both idqn and maqdn. They usually refer to the same algorithm in literature as far as I'm aware.

So madqn, I was thinking DQN with joint action value function across all agents, but that is not quite common or part of ctde, so I changed it to rdqn (recurrent dqn).


winget-source: rename index file after syncing is done

Try fixing the issue mentioned in https://github.com/ustclug/discussions/issues/435#issuecomment-2197544426.
cc @stevapple

Given that .bak is usually an older copy (backup), maybe we can use .source.msix instead?

I'm worrying about the confusion caused by the convention of .bak files and what this file actually is.
Any alternatives, perhaps .tmp?


Create method for transforming scenarios

Say I want to import a CSV of survey responses but use the header row (of question texts) as scenarios in addition to / paired with scenarios for the responses. Existing method ScenarioList.from_csv() will automatically use the header row as the scenario keys, which is not desired (and also creates a formatting issue where keys are sentences).
It would be convenient to have a method / option for transforming scenarios that have been auto-generated in this way -- i.e., a way of specifying that (1) the original keys should become values for a new key and (2) the original values should be assigned a new key. The new keys would be identical across the values, by default, e.g., "question" and "response":
scenarios = ScenarioList.from_csv("filename.csv")

transformed_scenarios = []
for s in scenarios["scenarios"]:
    for question, response in scenario.items():
        transformed_scenarios.append({"question": question, "response": response})

Need to think of how to handle potential additional columns that may or may not also be used, eg respondent ids.

This may make the most sense to build as Conjure methods, however, there could be other types of scenarios that a user may want to construct in this way that are not survey responses.

What I think we really want here is some optional parameters in from_ScenarioList that let you skip rows, re-name, come up with automatic valid indentifiers and so on. Right now, there's very little: https://github.com/expectedparrot/edsl/blob/88448ddbd6af402a605893e877e6dc8c57fec047/edsl/scenarios/ScenarioList.py#L693
@rbyh want to give some examples in a NB with CSVs you'd like to do something different with and maybe some proposed syntax?

This is covered in new ScenarioList method unpivot.


chore: add storybook example

Related issue (if exists)

Summary

🤖 Generated by Copilot at 1458ef6
This pull request adds a new example of using rspack with React and Storybook. It creates a react-storybook folder under the examples folder and adds several files to configure and run the example. The example demonstrates how to use rspack to bundle a React app and a Storybook UI, how to use Storybook addons and docs, how to write stories and documentation in JS, TS, and MDX, and how to style and test the components. The example also shows how to use the HTML builtins plugin and the webpack alias feature of rspack.

Walkthrough

🤖 Generated by Copilot at 1458ef6

Add a React Storybook example with various components, stories, and documentation (link-link)
Configure the ESLint rules for the example with some overrides for the ./src/**/* files (link)
Configure the Storybook settings for the example with the framework, addons, docs, and typescript options (link)
Configure the Storybook preview for the example with the parameters for the actions and controls addons (link)
Customize the webpack configuration for the example with an alias for the react and react-dom modules (link)
Define the metadata and dependencies for the example in the package.json file (link)
Configure the rspack CLI for the example with the context, entry, builtins, and module fields (link)
Define some global styles for the example in the index.css file (link)
Bootstrap the React app for the example in the main.jsx file (link)
Define the main App component for the example in the App.jsx file (link)
Define some styles for the App component in the App.css file (link)
Serve the index.html file as the template for the HTML builtins plugin of rspack (link)
Define a documentation page for the example in the Introduction.mdx file (link)
Define the Button component for the example in the Button.tsx file (link)
Define some styles for the Button component in the button.css file (link)
Define some stories for the Button component in the Button.stories.js file (link)
Define the Header component for the example in the Header.jsx file (link)
Define some styles for the Header component in the header.css file (link)
Define some stories for the Header component in the Header.stories.js file (link)
Define the Page component for the example in the Page.jsx file (link)
Define some stories for the Page component in the Page.stories.js file (link)

Hi @hardfist
I found a bug in storybook-builder-rspack package causing forever spinning screen. I was unable to track down the original repo to submit PR with fix. Are you the author and if so, do you accept PRs?

cc @JSerFeng is the author


Consistent XML error when parsing.

Hey, I'm just trying to parse many conversations and see the stats (basic stuff first) but I keep getting thrown this error every time I try at the same point, no matter what command I issue:
"The streaming parser crashed due to malformed XML. Falling back to the less strict/efficient python html.parser. It may take a while before you see output..."
It then kills the process.
Thoughts?

Hi, thanks for reporting this.
Could you give me more details? Example: operating system, python version, library version (newest is 0.7.post8. etc)

Sure! It's Ubuntu 14.04.2 LTS, python 2.7.6 and the newest library version.

Okay cool, I can reproduce that environment perfectly. Out of curiosity, how large is your messages.htm file? This sounds like it may be the OS unexpectedly reaping the process due to running out of memory.
I'm working on getting the primary streaming parser to stop being too sensitive to avoid this issue.

It's about 5 years worth of messages so it might very well be a size issue.

it's about 100mb.

Yeah, I have another report of memory running out with an 80 MB messages.htm file. I'm going to try swapping for a different XML parser and see if that helps. I'll post back here when it's ready!

Hi there, I think I've fixed the issue. Please try version 0.7.post10 and let me know if that fixes it.

Will do, thanks =)

It didn't throw back that error this time, it just kept going until it stopped and said 'Killed'. Not sure if that's a printout when it's actually done or it just killed itself during the process.

Hmm, was there progress text still being displayed? (i.e. Discovered chat thread with [...])
It could be that there is simply too much data. You could try using the -t switch (see the main page for examples) to just extract specific threads, rather than trying to do the whole thing in one go.
I may try doing something with temp files to get around the fact that most people's chat logs are huge somewhere down the line.

Yeah no progress text, it's probably the same issue. I've been playing with -t and it seems to do the trick anyway. Thanks!

Do you never see progress text? It should be printing out either way :/

Oh, I see the progress text, I mean not once it's killed. There's text until then. It was sort of an unnecessary statement on my end - obviously once it's killed there's no text. Ha

Ah okay, good to know. Looks like it's just a memory issue then. I'll try to think of a way to deal with that for the next release. Thanks!


Expanding Makefile/build system to create inconsistent fat binaries

I'm currently looking at the Makefile to see what our best approach is to creating inconsistent fat binaries (i.e., the kind that will trigger the new behavior in #55).
In order to lipo together two inconsistent binaries of different architectures, we need a fat-only target that can generically depend on two single-arch-only targets. That's not possible with the current layout of the Makefile, and probably won't be without some significant changes.
For the time being, it might be worthwhile to externalize this process to a shell script, one that's called by the makefile at the very end of the all: chain. With a few minor tweaks, I can provide that script with the information it needs to access the correct single-arch directories and drop the lipo'd results in the correct fat-arch directories.
cc @UniqMartin

The closest I've come is doing some voodoo with filter-out and splitting ALL_DIRS into SINGLE_DIRS and FAT_DIRS and adding special fat-only targets, but this hasn't led me anywhere meaningful.
(I think the easiest approach is just to use the libhello.dylib from one arch and the libextrahello.dylib from another, since we already guarantee that libextrahello.dylib has additional dylib load commands that libhello.dylib cannot have.)

https://github.com/Homebrew/ruby-macho/commit/a34fed9b994046f75bb3115358aa8a69b7e81f66 on ruby-macho/inconsistent-dylib-tests shows what I had in mind re: an external script called by the Makefile.

I'm pretty sure this can be done with a Makefile but I'm perfectly happy with the solution you came up with. This can always be cleaned up and optimized later, and the shell script gets the job done nicely and allows us to move forward. :+1: Closing in favor of #61.


Expose password policy on GraphQL

I'm not sure how best to do this since AIUI the polices are dictated by rego files. However, it would be good if we could expose to the UI what the policy is on passwords so real time feedback can be given.

We don't use the OPA policy for passwords anymore, but rather the zxcvbn algorithm, and the minimal complexity is exposed through the GraphQL APi:
query {
  siteConfig {
    minimumPasswordComplexity
  }
}

Note that @reivilibre did some work so that zxcvbn-ts gave the same result as the Rust crate we use by exporting the dictionaries: https://github.com/matrix-org/matrix-authentication-service/blob/main/frontend/src/utils/password_complexity/index.ts

Aha, this is great thank you!


[4508] Data migration to remove duplicate degrees belonging to HESA trainees

Context
https://trello.com/c/egpS9hjF/4508-duplicate-degrees-on-some-hesa-imported-trainees
Recent code fixes should prevent duplicates from been created in the future.
Changes proposed in this pull request

Data migration to remove duplicate degrees

Note
Takes about 3 minutes to run. Approximately 1400 trainees from last year's academic cycle have their degrees removed and recreated using data from the hesa_students table. Trial runs demonstrate that the code works as expected.
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Duplicate degrees on some HESA-imported trainees


Update tooltip to include * alleles (overlapping deletion)

Requirements
Update of tooltip to cover * allele (alleles missing due to overlapping deletions)
Description
VCFs can include * alleles (allele missing due to overlapping deletion)
Tooltip updated to include these alleles
Views affected
Population Genetics view - info button. When allele is in VCF but not one of alleles of variant definition
See GEM-J frequency, click info buttons
http://ves-hx2-76.ebi.ac.uk:5050/Homo_sapiens/Variation/Population?db=core;r=1:8044972-8045972;v=rs1293885450;vdb=variation;vf=507101856#gem-j_anchor
Possible complications
None, a documentation change
Merge conflicts
Tested on sandbox, branched from master
Related JIRA Issues (EBI developers only)
ENSVAR-3769 - Update Web frequency tooltip for variant alleles only in VCF

@at7  please could you review. Thanks


handle gzip encoded content

Basically this is an updated version of this pull request: https://github.com/intridea/oauth2/pull/159
We are still having issues with this in 2.2.5, which causes us to do some very dirty hacks around that.

Coverage decreased (-0.4%) to 96.117% when pulling 0ec98196cb25bf2ac410dd4f0ab8c1484fde367f on ChaosSteffen:handle-gzip-encoded-content into 7691a2c851a14f08c6f33356a0041d015963de01 on intridea:master.


project version was not updated to match released version

nuget and github released versions do not match project released versions


Traffic light bug at one way train track

Hello,
I am not sure if this is a problem with TM:PE or a problem with "One-Way Train Tracks [New version]", or maybe both.
I basically tried to help someone build a longer section with a one-way train track.
since the game by default does not support semaphores for such situations, i tried to make a workaround with timed traffic lights to ensure the there is never a deadlock.
The setup goes like this:
' = ' are two one way tracks ... the default one of the game
' - ' is a two way single track ... the one from the mod
(R) red light
(G) green light
step1: ====(G)------(R)====
step2: ====(R)------(R)====
step3: ====(R)------(G)====
step4: ====(R)------(R)====
Step 2 and step 4 are the most important ones.
These steps must be timed carefully such, that if a train enters the single track, it must get enough time to leave the single track again before the train from the opposite direction can enter.
I expected the traffic light to be relevant only for the train entering the single track.
But the problem is that the train stops and waits at the end of the single track, no matter what the traffic light says, and no matter if there is another train waiting for the opposite direction or not.
====(R)-------->BLOCKEDTRAIN>(G)====
Happens only if I play with traffic lights though. Without any setup everything flows until the deadlock occurs.
Also there is a minor glitch for such an intersection.
If I change the mode of the traffic light, there seem to be two traffic lights overlapping each other.
It would be awesome if such setup could actually work :)
It should in theory.
I opened the issue in both repos.
https://github.com/earalov/Skylines-ElevatedTrainStationTrack/issues/14
Regards
Egi

Fixed and updated right now!


Office hours inconsistent

The times at https://github.com/uit-no/hpc.uit.no/blob/594400e1053364ae9489d82a3ee9e20cfd98f901/content/_index.md?plain=1#L48
are inconsistent with
https://github.com/uit-no/hpc.uit.no/blob/594400e1053364ae9489d82a3ee9e20cfd98f901/content/_index.md?plain=1#L51
The RSE website also states 13:00-15:00. Which times are correct?

Thanks for noticing. That happens when you do copy and paste and forget to read it again.


Change argument --port to --listen in nix-serve.nix

According to nix-serve manual, it is recommended to use --listen instead.

Tested locally, seems to be working just fine.


Add tests with numba.jit for Series

Fixes #256
For new tests for Series you should:

use self.jit(...) instead of hpat.jit(...)
Add test name to hpat.tests.test_series._numba_not_supported_tests only if it does not support numba.jit

Also, old Series tests should be revised for reasons why they do not support numba.jit. See _numba_not_supported_tests list.

I will add numba.jit(nopython=True, parallel=True).


Unable to start django server if celery instance is defined according to cookbook 1.0 Windows 8.1

I use django 1.8.8, celery 3.1.20 and django-configurations 1.0. Windows 8.1
If I define celery instance according to cookbook http://django-configurations.readthedocs.org/en/latest/cookbook/#id4 I am unable to start django dev server.
It just freezes, nothing happens
Here is my celery.py
from __future__ import absolute_import
import os

from celery import Celery
from django.conf import settings

os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'common.settings')
os.environ.setdefault('DJANGO_CONFIGURATION', 'Dev')

import configurations
configurations.setup()

celery_broker_connection = 'redis://{0}:{1}/{2}'.format(
    settings.REDIS['host'], settings.REDIS['port'], settings.REDIS['db'])
celery_app = Celery('jobs', backend=celery_broker_connection, broker=celery_broker_connection)

celery_app.config_from_object('django.conf:settings')
celery_app.autodiscover_tasks(lambda: settings.INSTALLED_APPS)

Then if I use celery cookbook ver 0.8 http://django-configurations.readthedocs.org/en/0.8/cookbook/#id4
...
from configurations import importer
importer.install()
...

django server starts correctly but I get endless stream of errors from celery worker like this
[2016-01-28 11:42:43,256: ERROR/MainProcess] Process 'Worker-8' pid:13656 exited with 'exitcode 1'
Perhaps these two issues are not related to each other.
Also the second issue is being reproduced on django 1.7 and django-configurations 0.8 on Windows. (on Ubuntu 14.04 everything is fine). If do not use django-configurations then django server and celery worker start and work correctly. So I decided that it's related to django-configuration.
Does anybody know what it can be? Thanks in advance!

See https://github.com/jazzband/django-configurations/issues/196 for some followup.


Fix append!!(::Table, ::SingletonVector)

xfail: https://travis-ci.com/tkf/BangBang.jl/builds/141120575

Codecov Report

Merging #67 into master will decrease coverage by 0.94%.
The diff coverage is 33.33%.

@@            Coverage Diff            @@
##           master     #67      +/-   ##
=========================================
- Coverage   92.75%   91.8%   -0.95%
=========================================
  Files          18      19       +1
  Lines         290     293       +3
=========================================
  Hits          269     269
- Misses         21      24       +3

Impacted Files
Coverage Δ

src/typedtables.jl
0% <0%> (ø)

src/BangBang.jl
100% <100%> (ø)
:arrow_up:

src/NoBang/base.jl
94.44% <0%> (-1.39%)
:arrow_down:

Continue to review full report at Codecov.

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


Explicitly sets IsolationLevel

Description
This is to explicitly set IsolationLevel.
Default IsolationLevel is ReadCommitted.
If for any transaction we need to set different IsolationLevel, we can pass that in the SqlConnectionWrapper.
E.g. We need to set IsolationLevel.Snapshot in SqlServerSearchService
Related issues
Addresses #70783
Testing
Existing tests continue to pass.

It is no longer required as mentioned in the #AB70783


Slices backed by AWS Athena load sequentially, not in parallel.

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
Versions: 0.22.1, 0.23, 0.25.6
Expected results
Slices should load simultaneously. Athena logs should also show queries to be called at the same time.
Actual results
Slices load one after the other, making for poor user experience. The Athena logs also shows a query is called only once the previous one finishes.
Athena log timestamps:
2018/06/25 13:53:43 UTC+-2 Query...
2018/06/25 13:53:41 UTC+-2 Query...
2018/06/25 13:53:37 UTC+-2 Query...
2018/06/25 13:53:36 UTC+-2 Query...
2018/06/25 13:53:36 UTC+-2 Query...
2018/06/25 13:53:34 UTC+-2 Query...
2018/06/25 13:53:31 UTC+-2 Query...

Steps to reproduce

Build a dashboard with multiple slices, each built on it's own Athena query.

Use the below gunicorn command line arguments:
GUNICORN_CMD_ARGS "-w 2 -k gevent --threads 12 --timeout 120 -b  <IP_ADDRESS>:8088 --limit-request-line 0  --limit-request-field_size 0 --proxy-protocol true --proxy-allow-from * --forwarded-allow-ips * --keep-alive 75"

I changed my configuration to what you suggested. I tried both with 8 workers and 16 workers and no --threads argument. Neither of these worked.
I can still see queries running one after the other in the Athena logs.

Looks like Athena is the thing limiting concurrency somehow.

@mistercrunch I'm busy doing some more digging around and found something interesting. I am logging out the timestamp when the Ajax from a slice hits the explore_json endpoint, and I'm logging out when the query() method is called to execute the query. Even though I have 16 workers, only 4 Ajax calls are handled at first. After that some are staggered and some are handled two by two.
When it comes to executing the queries, they are being run in sequence. It is clear from the logs that one is waiting for another to finish before running. Is it possible that the issue is with the JDBC driver or the way in which that connection is made?
Also, any idea why only 4 Ajax requests would be handled even when 16 workers are used?

What's your -w argument?

The Celery workers are mostly there to deal with long running queries and heavy workloads.
create_engine being called for each query is right, we do not use connection pooling for external databases, only for the connections to the metadata database.

@mistercrunch It seems like the issue is the running of the JDBC driver jar. A new JVM gets created for each connection, instead of threads being added to one running JVM. I think it has something to do with SQLAlchemy wanting one thread per process.

Hi @sjmeyer Is there a way to config PyAthena to send multiple queries to Athena so that Superset can have better performance on dashboard which have more than 10 slices. Everytime I apply a filters, it takes 45s to fully load visualization for all slices

Hi @sjmeyer @LeoDiep: Is this issue resolved? same issue I am experiencing even after enabling Global Async queries. Opened a new ticket as this ticket is marked as completed
https://github.com/apache/superset/issues/25827

Hi @sjmeyer @LeoDiep: Is this issue resolved? same issue I am experiencing even after enabling Global Async queries. Opened a new ticket as this ticket is marked as completed #25827

Hi, I solved my problem by increasing the number of gunicorn workers from 1 (default) to 6 workers, now 45s decrease to 6s

@LeoDiep : That's what I ended up doing. Thanks for suggesting it. Although I don't know why async queries are not working. I hope someone from superset team reviews by request in https://github.com/apache/superset/issues/25827

@LeoDiep : That's what I ended up doing. Thanks for suggesting it. Although I don't know why async queries are not working. I hope someone from superset team reviews by request in #25827

I guess async queries are only applied to long response time queries (more than 10s each) so that the sql lab doesn't get stuck when querying other queries


responsive-footer

Responsiveness problems were corrected, the footer was added, an icon and name were added to the window tab, and the background of the product images was removed.

New adjustments were made to the responsiveness and new designs were added to the slider that provide more visibility to the e-commerce.


Update to incorporate new features in drake

Development drake now tracks the history and provenance of targets over time (CRAN ETA mid-July) and the additions in this PR describe how it works.
drake history is similar to MLflow tracking except:

You do not have to tell drake which parameters to track. drake_history() just analyzes commands automatically and detects named length-one atomic arguments to function calls in the plan.
drake does not automatically create downstream summaries such as performance metrics. Users still need to create special targets for those.

As of https://github.com/ropensci/drake/pull/977, we no longer need to worry about the performance issue I mentioned in https://github.com/richfitz/storr/issues/77#issuecomment-476275570. With target(format = "keras"), drake can now save and load Keras models directly in HDF5 format instead of a storr RDS file. The workaround from https://github.com/sol-eng/tensorflow-w-r/pull/9 is obsolete, so I removed it in https://github.com/sol-eng/tensorflow-w-r/pull/11/commits/b982bf9221714ed146b7563e1402e2cb2f9c226f when I added target(format = "keras") just now.
Related: drake also supports fst storage for large data frames. It does not immediately work for the data in this example because it is an object of class rsplit, but a little refactoring might help us work around this little technicality.
library(drake)
n <- 1e8 # Each target is 1.6 GB in memory.
plan <- drake_plan(
  data_fst = target(
    data.frame(x = runif(n), y = runif(n)),
    format = "fst"
  ),
  data_old = data.frame(x = runif(n), y = runif(n))
)
make(plan)
#> target data_fst
#> target data_old
build_times(type = "build")
#> # A tibble: 2 x 4
#>   target   elapsed              user                 system
#>   <chr>    <Duration>           <Duration>           <Duration>
#> 1 data_fst 13.93s               37.562s              7.954s
#> 2 data_old 184s (~3.07 minutes) 177s (~2.95 minutes) 4.157s

Created on 2019-08-05 by the reprex package (v0.3.0)


VS2015 project and solution files, more German translations

I have added VS2015 project and solution files because VS2015 Community Edition (free) is my main build environment for windows software. Unfortunately CrashExplorer can not be compiled directly from the IDE since Boost is downloaded only via the build script. Can we add Boost like zlib to the source tree?

Awesome! Thank you! VS 2015 is my default IDE also. So, thanks for this!
Re: Boost
I'm not sure if adding the entire boost source is a good solution. Boost is very huge from what I recall. Here are a few ideas worth investigating:

BCP boost dependency copy: http://www.boost.org/doc/libs/1_59_0/tools/bcp/doc/html/index.html All we need is boost regex right? Perhaps we could just copy over the compiled libs for just boot regex and it's dependence. If there within a few MB in total, I think it might be okay.
Install, build locally, and have a batch script that can be modified that sets environment variables to point to the local build of boost.
Submodule checkout.

Let me know your thoughts.
Thanks,
Brian

Just wanted to comment that AppVeyor already has boost installed on it's Build Agent in C:\Libraries\boost so it isn't downloaded.  I believe the only dependency that is downloaded and installed is the Mulitbyte MFC dependency.


Allow .css files in paths

Due to this line it's currently impossible to provide .css files to extract selectors from.
https://github.com/FullHuman/purgecss-webpack-plugin/blob/6a981502497969c21f1ecefae16fee0554c03502/src/index.js#L38
I want to pass my hand-written css files to an extractor so that css never gets purged, I only want to purge unused vendor/framework css.
Would it be possible to allow css files in paths?

I'm not sure it would be the right way to do it.
I think you can separate your vendor files and your own with webpack.
What you want to do would have the same result as the populating the whitelist option with the css. whitelist docs
The idea would be to do something like this:
const Purgecss = require('purgecss')
const purgecss = new Purgecss({
    css: [],
    content: []
}

and use the purgecss method to get the selector
/**
     * Extract the selectors present in the files using a purgecss extractor
     * @param {array} files Array of files path or glob pattern
     * @param {array} extractors Array of extractors
     */
    extractFileSelector(files: Array<string>, extractors?: Array<ExtractorsObj>): Set<string> {

const myCssSelectors = purgecss.extractFileSelector(['mycss/*.css'])
// return a set of selectors

I think it's better if the plugin provides a way to select css files that should not be purged. I will try to add this option.

I think it's better if the plugin provides a way to select css files that should not be purged. I will try to add this option.

This doesn't work in my case since app css and vendor css end up in the same file, and I only need to purge the vendor stuff...

Hi,
This plugin purges the files before they are written to disk, If you would be able to select css files that should not be purged like @Ffloriel said then you can only purge your vendor.css and the app.css will not be touched.
It all depends on how you concat. the two into one file.
If you are using some webpack plugin then you should be sure that the order of the plugins is as follows: purge, concat, extract-text.
If you are doing it post-build then it does not really matter

On second thought, I think being able to ignore an entry for purging would be good enough for me

@Ffloriel Trying your suggestion above, but keep getting Error: No content provided.. Here's the code I'm using:
const Purgecss = require('purgecss')
const purgecss = new Purgecss({
  css: [],
  content: []
})
const selectors = purgecss.extractFileSelector(['node_modules/typer-js/typer.css'])
console.log('SELECTORS:', selectors)

Was hoping to see what the value of selectors was. Didn't know that there was a way to extract a list of selectors from a file. This will help me because right now I'm manually passing in a whiteList array that I've had to create myself. This isn't a big problem at the minute since there's only like 7 selectors in that list. But I could imagine if I was dealing with multiple files and a ton of selectors I wanted whitelisted, that extractFileSelector would come in handy. Just can't get it to work.


Properly encode image content.

Image content was not being properly encoded before being JSON serialized for the request.
Fixes #2525.

@tseaver, I moved the imports and constants for the other test I updated for this.
LMKWYT


Run '{TestObjectName} will not work unless cursor is at start of line of class definition

To reproduce

find the test shortcut to run test (in this case Ctrl + Shift + F10)

2. pressing test shortcut will not work.  pressing test shortcut  inside test body will not work, either.

3. pressing test shortcut will work.
it's frustrating because of constant cursor moving.

IntelliJs RunLineMarkerContributor only allows us to contribute this info for a single PSI element, and the action is attached to that element. There's no such element that encompasses both the keyword and identifier (and supertype), so we have to settle for either having it on class/object or the name itself.


Automated app updates through "update-operator"

User story: As an end user developer, I want to be able to specify how updated I want my app to be within certain major / minor boundaries and have the app be automatically updated according to those specifications.
We don't want to encourage this until configurations are safe. Currently, there is no migration path for configuration values.

@giantswarm/team-batman , @pipo02mix commented in Slack from customer,
"At the same time they spend time and resources in updating the managed apps. Ideally, we offer the same experience as clusters and we push and maintain those up to date. They would like to trust the versioning and allow upgrade apps automatically for minor and patches. There is a roadmap ticket #124 but not sure how far we are with this."
Now that we have more apps, would it be a good time to revisit?

@cokiengchiara Yes, I think we should be aiming for the same experience as clusters. However this is a complex feature especially if we need to have maintenance windows etc.
I think an intermediate step could be to have a dashboard showing all optional apps that have an upgrade available. We could do this as part of https://github.com/giantswarm/roadmap/issues/322
We could add this quite easily to the metrics generated by app-exporter and use the AppCatalogEntry CRs to check if there is an upgrade.

That would be awesome so we can track customer apps in a board and ask them to update old versions or perform he updates for them

And for the auto-upgrades themselves: I think we can't really do that before we have automated upgrade testing.

I agree with @pipo02mix and there is a ticket for this: #322 - I would actually put this on a backlog somewhere further in the back and mark it as not to be done now.

closed in favor of https://github.com/giantswarm/giantswarm/issues/20641