Release ruby-style 1.24.0

set required ruby version to 2.4
set rubocop dependency to ~> 0.74.0

Commits since previous releasecommit d9c276f3b286f4a5225439222a86b1d11a5622c6
Author: Graham Paye
Date:   Thu Aug 8 14:26:57 2019 -0700
bump required ruby version to 2.4 (#17)

* bump required ruby version to 2.4
* bump rubocop dependency to ~> 0.74.0

commit 0eb4adad7e98cc4018ba83c0b1564691a28131af
Author: Daniel Azuma<EMAIL_ADDRESS>Date:   Thu Jun 13 13:14:39 2019 -0700
Bump version to 0.3.1 (#16)

Code changes since previous release
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 03700f4..b81f8b3 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Release History

+### 0.3.1 / 2019-06-12
+
+* Fix file permissions
+
 ### 0.3.0 / 2019-05-06

 * Enforce expanded for Style/EmptyMethod
diff --git a/README.md b/README.md
index 731af02..69105b8 100644
--- a/README.md
+++ b/README.md
@@ -51,11 +51,12 @@ This library is licensed under Apache 2.0. Full license text is available in

 ## Supported Ruby Versions

-These libraries are currently supported on Ruby 2.3+.
+These libraries are currently supported on Ruby 2.4+.

 Google provides official support for Ruby versions that are actively supported
 by Ruby Core—that is, Ruby versions that are either in normal maintenance or
-in security maintenance, and not end of life. Currently, this means Ruby 2.3
-and later. Older versions of Ruby _may_ still work, but are unsupported and not
-recommended. See https://www.ruby-lang.org/en/downloads/branches/ for details
-about the Ruby support schedule.
+in security maintenance, and not end of life. Currently, this means Ruby 2.4
+and later. Starting with Ruby 2.4, minor releases of the google-style gem under
+`1.x.x` will correspond with the required Ruby version, with `1.24.x` requiring
+Ruby 2.4.
+See https://www.ruby-lang.org/en/downloads/branches/ for details about the Ruby support schedule.
diff --git a/google-style-1.24.0.gem b/google-style-1.24.0.gem
new file mode 100644
index 0000000..adf25c9
Binary files /dev/null and b/google-style-1.24.0.gem differ
diff --git a/google-style.gemspec b/google-style.gemspec
index 56ad6be..7b8ea7a 100644
--- a/google-style.gemspec
+++ b/google-style.gemspec
@@ -14,7 +14,7 @@

 Gem::Specification.new do |gem|
   gem.name          = "google-style"
-  gem.version       = "0.3.0"
+  gem.version       = "1.24.0"

   gem.authors       = ["Graham Paye"]
   gem.email         =<EMAIL_ADDRESS>@@ -26,9 +26,9 @@ Gem::Specification.new do |gem|
   gem.files         = ["CONTRIBUTING.md", "CODE_OF_CONDUCT.md", "LICENSE",
                        "README.md", "google-style.yml"]

-  gem.required_ruby_version = ">= 2.3.0"
+  gem.required_ruby_version = ">= 2.4.0"

-  gem.add_dependency "rubocop", "~> 0.64.0"
-  gem.add_development_dependency "bundler", "~> 1.17"
+  gem.add_dependency "rubocop", "~> 0.74.0"
+  gem.add_development_dependency "bundler", "~> 2.0"
   gem.add_development_dependency "rake", "~> 12.3"
 end
diff --git a/google-style.yml b/google-style.yml
index 44452df..3404f91 100644
--- a/google-style.yml
+++ b/google-style.yml
@@ -13,7 +13,7 @@
 # limitations under the License.

 AllCops:
-  TargetRubyVersion: 2.3
+  TargetRubyVersion: 2.4
 Layout/AlignHash:
   EnforcedHashRocketStyle: table
   EnforcedColonStyle: table

This pull request was generated using releasetool.

I'm evaluating the recent changes.

We might want to add a section to README discussing the shifted semver nature of this gem. How to declare a dependency, etc...

@blowmage are you good with the updated README?

Yes, that looks good.
I would remove the google-style-1.24.0.gem file.

this PR is removing it, accidentally added it in the previous PR

Release is at https://github.com/googleapis/ruby-style/releases/tag/ruby-style/v1.24.0


Feature Request: Force unlock workflow/pipeline

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Inspiration: https://github.com/terraform-azurerm-examples/terraform-enterprisescale-starter/blob/main/.azure-pipelines/templates/tf-force-unlock.yml
Describe the solution you'd like

Additional context

Hi, I would like to try this (would be my first issue). Anything I need to be aware of/need to know outside of the contribution guidelines?


mountinfo should ignore EINFO_QUIET from its initial environment

If mountinfo is run with EINFO_QUIET=YES in the the  environment it will blindly obey it.  If you run
oppenrc -q shutdown

EINFO_QUIET=YES is set and so when do_unmount queries the mounts with
set -- $(mountinfo "$@")

it gets nothing, as mountinfo is quiet.  As the result the root fs is not remounted read-only on shutdown and is dirty on reboot.
Reproducible: Always
Steps to Reproduce:

Arrange for shutdown runlevel to run quietly, so that EINFO_QUIET=YES is set
Arrange for the kernel to be verbose on the next boot
Observe it complaining about the dirty filesystem:

[    1.497147] EXT4-fs (sda2): INFO: recovery required on readonly filesystem

Or just:
$ /lib/rc/bin/mountinfo
/tmp
/boot
...
$ EINFO_QUIET=YES /lib/rc/bin/mountinfo
$

A few programs already do
unsetenv("EINFO_QUIET");

on startup, mountinfo should too.

while i tend to agree, this seems to be somewhat on purpose ... see commit 978dbe028414fc44603d3be70f2a4b226eda9916 from https://bugs.gentoo.org/439010

Note that $ grep -r mountinfo sh seems to indicate that all existing "test" invocations of mountinfo supply the -q explicitly (handled by case_RC_COMMON_GETOPT) and only the two $() invocations where its output is required do not use it, for obvious reasons, and that's where this bug bytes.
I would say that applying EINFO_QUIET (how rc as a whole should be, if I may paraphrase) to mountinfo feels like a bit of non-sequitur.  Kinda like making that variable to apply -q to all calls to grep.  But that might be just my uninformed impression...


Fix error when action called before initialized

Fixes #1112.
As a bonus, since I was editing that part of the code, I removed the problematic "shift" and all its even uglier workarounds.

Closing all old PR against master.


Add sniffer callback

SnifferCallback allows callers to intercept in the sniffing process and decide which (healthy) nodes are considered for receiving requests.
When the sniffing process finds a healthy node, the callback is invoked with the NodesInfoNode struct passed into it. The callback can inspect that struct and return false when the node should be ignored. No requests are routed to that node by the client.
Example:
func MySnifferCallback(node *elastic.NodeInfoNode) bool {
        // This sniffer simply ignores node with name "node1"
        if node.Name == "node1" {
                return false
        }
        return true
}

...

client, err := elastic.NewClient(
        elastic.SetSnifferCallback(MySnifferCallback))
if err != nil {
        log.Fatal(err)
}

The official clients have a similar feature, e.g. the nodesToHostCallback in the Node.js
client.
See #388 for further discussion.

test doublecheck for masnec

test doublecheck for masnec

test doublecheck for masnec

test doublecheck for masnec

test doublecheck for masnec

test doublecheck for masnec

test doublecheck for masnec

test doublecheck for masnec

test doublecheck for masnec


discussion: Allow each comittee to set admission period outside of the main admission period

Background: For some Comitees the main admission period is not enough. For example does SL (Studentlekene) want a continous admission period after the main admission period, which results in them going back to use Google Forms (https:ntnui.no/sok.sl).
Where some commitees might have a strict deadline for applicants, others might not (at the same time).
Questions for discussion:

Should the application system support admissions outside of the main admission period?
If so, should each comittee be able to set their own deadline (or no deadline)?

If deadlines are different, can they be displayed to the applicants in a good way, or will this create confusion? Maybe the admission system should not show the deadline in this case?
Maybe the main board can set a common deadline for the main admission periods? Or change the comittees deadlines?

Good point, and I guess it is nice that committees can use the admission system outside of regular admission periods in case they need more people.
An option here is to always show the form as long as there are any committees that accept new applications. In case there is no current admission period, the form could say "The main admission period is over, but some committees still consider applicants"


go routine leak in k8s tests

goleak: Errors on successful test run: found unexpected goroutines:
[Goroutine 660 in state chan receive, with k8s.io/client-go/util/workqueue.(*Type).updateUnfinishedWorkLoop on top of the stack:
goroutine 660 [chan receive]:
k8s.io/client-go/util/workqueue.(*Type).updateUnfinishedWorkLoop(0xc0003f8cc0)
	/home/circleci/project/vendor/k8s.io/client-go/util/workqueue/queue.go:198 +0x149
created by k8s.io/client-go/util/workqueue.newQueue
	/home/circleci/project/vendor/k8s.io/client-go/util/workqueue/queue.go:58 +0x28c

 Goroutine 658 in state chan receive, with k8s.io/client-go/util/workqueue.(*Type).updateUnfinishedWorkLoop on top of the stack:
goroutine 658 [chan receive]:
k8s.io/client-go/util/workqueue.(*Type).updateUnfinishedWorkLoop(0xc0003f8c60)
	/home/circleci/project/vendor/k8s.io/client-go/util/workqueue/queue.go:198 +0x149
created by k8s.io/client-go/util/workqueue.newQueue
	/home/circleci/project/vendor/k8s.io/client-go/util/workqueue/queue.go:58 +0x28c
]
FAIL	github.com/prometheus/prometheus/discovery/kubernetes	14.195s

In: master

I want to fix this bug.
could you give more info about this issues？
Q1: How to get this error？
Q2: Which branch is used？


Upgrade Php-Parser from 4 to 5

Right now psalm blocks this.

I successfully used https://github.com/bamarni/composer-bin-plugin to separate dev-dependencies (like Psalm) from production dependencies and unblock production dependency upgrades. Maybe it could be used here as well.


Allow schema title to be a function

It is usefull when you want to compute data in the label. For instance I need to do: Price ($) where the currency is variable. And the variable may not be available when the model is instantiated, though it will when the form is rendered. So I cannot just do title: 'text'+variable

Needs better doc's rather than this fix?

Closing to clean my list of PRs


Provide image for easy Gaia hub setup

This issue represents an epic in ZenHub for tracking work towards the provision of an image for easy Gaia hub setup.

Closing in favor of https://github.com/blockstack/gaia/issues/163


Add test file needed for testing CaC content

Add required test file so we can test if datastream can be parsed correctly by parser.

the e2e will fail because of https://github.com/ComplianceAsCode/content/pull/11614

/hold for test

/hold for test

this will be internal for testing only changes

/unhold

the e2e failure will be fix with https://github.com/ComplianceAsCode/compliance-operator/pull/495

Looks like this is ready for a rebase to pickup the fix in #495 .


KeyMapper

Hi,
I want to use your library with a swift project. I know that you said that you'll be doing it later but swift is above beta now. I'm finding it difficult to use KeyMapper with swift, something on this would be great.
Got something from a stackoverflow question but nothing of much help.
http://stackoverflow.com/questions/26896508/override-objective-c-class-method-in-swift

Just in case anyone wants to use this in swift. if you have arrayJson in response Simply use.
.arrayOfModelsFromDictionaries for array to object array
For simple ObjectJson response use
(dictionary: dictionary) for single object


Add me! Also I wanna help you! - Null#6235

I added you on discord, also I wanna help you code it better and update it with you. Null#6235 - I wanna make it look nice etc. I've been experimenting with it making it look better you know. Alright, just hit me up whenever if you want. Btw I don't fully know how to code, but I've done it enough to understand mostly what's going on and code shit etc.

Sorry I decided not to update it for now. I've moved on to other projects, which appeal more interesting to me.

Sorry I decided not to update it for now. I've moved on to other projects, which appeal more interesting to me.


Handle window maximization better

Platform: Linux (Debian)
Version: 3.0.0-alpha.1
Currently, when the window is maximized, it creates a weird white space on the right side. It should either re-flow the UI to use the additional space or not allow the user to maximize the window.

Yes this is a good suggestion. I say we disable window maximisation and possibly also set some constraints on the maximum and minimum window size. @elioqoshi what do you think?


Create Javadocs

Would be great if you could create javadocs host them on github

A direct download can be found in the README. There you also find the JavaDoc. Here is a direct link to the latest JavaDoc jar (unzip the content): http://search.maven.org/remotecontent?filepath=com/evernote/android-job/1.0.2/android-job-1.0.2-javadoc.jar

Here is a cool script that will automate javadoc and github hosting https://github.com/JakeWharton/butterknife/blob/master/deploy_website.sh


Use Redoc to generate API docs

Preview link: https://deploy-preview-10503--cockroachdb-docs.netlify.app/docs/api/cluster/v2.html

This change is

Unintended merge, sorry


Update dev dependency karma-chrome-launcher to latest

I don't see any breaking changes in either the release notes or source diff:
https://github.com/karma-runner/karma-chrome-launcher/blob/master/CHANGELOG.md
https://github.com/karma-runner/karma-chrome-launcher/compare/v2.2.0...v3.0.0

/azp run js - storage-queue - tests

/azp run js - servicebus - tests

/azp run js - keyvault-keys - tests


update Win CI req. #122

Fixing bug with Appveyor CI  #122
it failed because we were installing 1.1.0 directly but in requirements.txt is now required 1.2.0
Seems that the fix is
pip install -r requirements.txt -f https://download.pytorch.org/whl/torch_stable.html

you shall change the badge for yours...


drivers: led_strip: ws2812-rpi_pico-pio: Split the dts definition for LED and controller

The dts definition of rpi_pico-pio is divided so that ws2812 can be described with a common dts definition, and the implementation also follows.
Initially, there was some discussion, but with the introduction of #xxx, the situation has changed to the point where it is more desirable to be able to reuse the ws2812 DTS.
bff is one example, and in line with this modification, the definition has been changed to one that can be used commonly by the ws2812-gpio and ws2812-rpi_pico-pio drivers.

@simonguinot
First, I would like to hear your opinion.


fix(swc-helpers): import _typeof in _decorate

Description:
fix _typeof import in _decorate.mjs.

BREAKING CHANGE:

Related issue (if exists):

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.何庚坤 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


"memory.hugepages" / "cpu.affinity" have invalid values

I am using the provider to make a full clone of a template.  Everything works correctly the first run, and the vm works as expected.
However, the next time I run a preview I am met with the following errors:

error: proxmoxve:VM/virtualMachine:VirtualMachine resource 'dev' has a problem: expected hugepages to be one of ["1024" "2" "any"], got . Examine values at 'dev.memory.hugepages'.
error: proxmoxve:VM/virtualMachine:VirtualMachine resource 'dev' has a problem: invalid value for affinity (must contain numbers or number ranges separated by ','). Examine values at 'dev.cpu.affinity'.

I've tried ignoring memory.hugepages and cpu.affinity, but that doesn't help.  I am not setting any properties in memory or cpu at all.
Is this a problem with the pulumi provider or the terraform provider?

generally speaking, this looks like an upstream issue to me where the retrieved values are not mapped properly (anymore). i raised one such issue upstream in the past because it broke the pulumi provider. could you please raise this issue upstream as well?
however, there was a property which raised a similar issue for me once. my workaround was to set the property in pulumi to its default value. did you try if this would solve your issue intermediary?

I'm having the same bug. No matter what value I set, it gives me back the same error


How to use MUI icons in header title?

Hello, I'm trying to add Icons in the calender entry titles, but I'm not sure if react-scheduler supports this? If so how can it be done.
Thanks

have to use event render prop thanks!


Development With ServiceWorkers

In the ReadMe it is not clear what the development cycle is with the default ServiceWorker setup. After the first page load everything seems to be cached and a page refresh does not reflect any changes in the components.
After trying to figure it out I have resorted to just removing the ServiceWorker setup. I figure there must be a better way than removing the configs for development and restoring it for production. Any chance someone could help me in figuring it out and maybe we can add it to the ReadMe?

Hey ootoovak, thanks for submitting this bug!
Could you tell me which browser you're using, and if you're using the v3.0.0 or the master branch? On master, the problem is the AppCache that's very persistent in its caching, which the ServiceWorker shouldn't be. This is solved in v3, which you can get by running git checkout v3.0.0 in your clone repo folder.

Note:
Only loading the ServiceWorker in production would be as easy as wrapping the navigator.serviceworker.register in the app.js in this:
if (process.env.NODE_ENV === "production")

which, to be honest, sounds like a good idea, so I'll implement that! Thanks for the tip!

Ah, I was using the master branch. I'll be sure to checkout v3.0.0 instead. :+1:
Also, thanks for the changes to the code and ReadMe and all the work you have done on this project so far. :smile:

@ootoovak Thank Samson, that is very much appreciated. I hope you enjoy v3.0.0, I'm going to assume this problem is fixed there. If it reappears on v3.0.0, feel free to reopen this issue!


[Suggestion] How to fix people from using this as a forum.

I been on reddit for some time and here reading articles and progress. This section has been miss used quite a bit. This is meant for issues with the program and not so much talking. A while back i purchased a domain. Over the past week i have been working to improve it. I added a forum.  Would the staff want to make their own thread or section so people can talk and not open up issues when its just general chat?
Anyways if interested everything is fresh. http://pokemongohelp.net
Let me know if its something you guys would be interested in.
Cheers!

To add to @zunox, there are already reddit, slack, and discord threads up that are constantly populated. As such, perhaps we can have someone moderate the issues board and simply close out non-issues?

Slack is supposed to be it !

We have slack, no forums needed.


501 Error in debug mode

I created a project which would use phantomjs. It runs well in common mode. But when I want to debug it, 501 error occurs.

What's common mode? How do you use phantomjs? In PATH or set --phantomjs-proxy parameter?
To use phantomjs in webui, webui should able to connect to phantomjs-proxy.

I put the binary in /usr/local/bin and using the below commands to run the pyspider
pyspider -c ./config/pyspider.json scheduler >> ./log/pyspider.log 2>&1 &
pyspider -c ./config/pyspider.json --phantomjs-proxy="localhost:25555" fetcher >> ./log/pyspider.log 2>&1 &
pyspider -c ./config/pyspider.json processor >> ./log/pyspider.log 2>&1 &
pyspider -c ./config/pyspider.json result_worker >> ./log/pyspider.log 2>&1 &
pyspider -c ./config/pyspider.json webui >> ./log/pyspider.log 2>&1 &

pyspider -c ./config/pyspider.json --phantomjs-proxy="localhost:25555" webui >> ./log/pyspider.log 2>&1 &

There is another problem. When I just restart phantomjs, I can crawl a js page normally. But when I tried to crawl (click 'run' again) the page again, it causes a error. Only the first run after restarting phantomjs doesn't cause any error. But this problme doesn't happens in demo.pyspider.org.
The error: 'HTTPError: Protocol "about" is unknown'

Does it happen in debug mode by running multiple times?

It happens in both modes.
The info below is the output of phantomjs when I run the task twice. The first time succeed and the second time failed. The URLs are same.
Web server running on port 25555
Request error: #18 [301=Protocol "about" is unknown]about:blank
[200] http://xxxxxxxx 1.416
Request error: #1 [301=Protocol "about" is unknown]about:blank
[599] http://xxxxxxxx 1.331

Can you tell me the version of phantomjs in demo.pyspider.org?

demo.pyspider.org is running with docker image binux/pyspider based on cmfatih/phantomjs phantomjs 1.9.7

I think it's the problem caused by phantomjs. I replaced phantomjs2.1.1 with phantomjs1.9.7. The problem solved.


Compatibility Issue

I run a spigot server 1.7.10 version. I downloaded the latest beta version of TerrainControl 2.6.1 which sais its for CB 1.7.9-R<IP_ADDRESS>. When i am trying to install the Folder that is created miss almost everything and its not properly installed and its not working in game (/tc) . I assume its the version compatibility. But i thought it could work with 1.7.10 as it hasnt many differences with 1.7.9.
Can you verify that 2.6.1 version isnt compatible with 1.7.10? Cause if it does, then there is something else wrong and i cant install TerrainControl.
If that is the problem, when are you going to release a beta 1.7.10?

Closing this issue, as I do not believe this is a bug that needs to be fixed.
By the way, biome group builds can be downloaded from the build server. The default settings are getting really close to vanilla. :smiley:


Extracting isosurfaces of Mip-NeRF

Hello!
Thanks for sharing this awesome work! :)
I'm curious, have you thought about the correct way of extracting the isosurfaces from the trained implicit function?
For vanilla NeRF model it is possible to extract the level surface at two scales using separately trained coarse and fine networks. Here, as far as I understand, it is possible to extract the level surface at an arbitrary scale, and for that I could just query the network with a positional encoding obtained for a desired point x and some manually selected variance, which determines the scale.
Does this approach makes sense to you, or are there some reasons why it could fail?

You can definitely query a mipnerf at any desired point x and covariance matrix Sigma. I'm not sure what this has to do with isosurfaces though.


Error installing Sharp on FreeBSD v12.0 + Node 10 and 12

Description
Received fatal error: 'vips/vips8' file not found running npm install manually after gridsome create app. Auto module installation also failed.
Steps to reproduce

In FreeBSD v12.0 with Nodejs v10
Exec gridsome create myapp (message will appear asking to install manually)
Exec npm install in myapp/
Error message shows up trying to build Sharp 0.22.1

Expected result
Installation completes susccessfully.
Actual result
Error log
><EMAIL_ADDRESS>install /usr/home/app/ihazcodez3/node_modules/sharp
> (node install/libvips && node install/dll-copy && prebuild-install) || (node-gyp rebuild && node install/dll-copy)

ERR! sharp BSD/SunOS systems require manual installation of libvips >= 8.7.4
info sharp Attempting to build from source via node-gyp but this may fail due to the above error
info sharp Please see https://sharp.pixelplumbing.com/page/install for required dependencies
gmake: Entering directory '/usr/home/app/ihazcodez3/node_modules/sharp/build'
  TOUCH Release/obj.target/libvips-cpp.stamp
  CXX(target) Release/obj.target/sharp/src/common.o
warning: unknown warning option '-Wno-cast-function-type'; did you mean
      '-Wno-bad-function-cast'? [-Wunknown-warning-option]
../src/common.cc:25:10: fatal error: 'vips/vips8' file not found
#include <vips/vips8>
         ^~~~~~~~~~~~
1 warning and 1 error generated.

Environment
 System:
    OS: freebsd
    CPU: (4) x64 Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz
  Binaries:
    Node: 10.16.3 - /usr/local/bin/node
    npm: 6.11.3 - /usr/local/bin/npm

Hello @jhfoo, on the third line of output in your error log it looks like it may be presenting a solution.
ERR! sharp BSD/SunOS systems require manual installation of libvips >= 8.7.4
Have you attempted to manually install libvips?

Yeah the package is installed
app@dev1 ~ %pkg info | grep vips
vips-8.7.4_9                   Free image processing system

Looking at the logs
ERR! sharp BSD/SunOS systems require manual installation of libvips >= 8.7.4
info sharp Attempting to build from source via node-gyp but this may fail due to the above error

It looks like somehow the installed vips package isn't recognized.

Hey, I have two idears for you!
Looking at the documentation for the library, I see there's a bug that affects FreeBSD's install process, perhaps this is the issue?
This isn't a solution but more of a stopgap. I'm seeing a similar issue over here in the sharp repo so perhaps using an older version of gridsome would use an older version of sharp and fix the issue!

Hey, I have two idears for you!
Looking at the documentation for the library, I see there's a bug that affects FreeBSD's install process, perhaps this is the issue?

Yeah I tried that and didn't work. The issues seem to be rather old.

This isn't a solution but more of a stopgap. I'm seeing a similar issue over here in the sharp repo so perhaps using an older version of gridsome would use an older version of sharp and fix the issue!

Holy cow installing<EMAIL_ADDRESS>on node 10 (not 12) was successful! I note the vips detection log entry:
info sharp Detected globally-installed libvips v8.7.4

So the latest shap version that works is 0.22.1. The versions after that (0.23.0 onwards) do not detect libvips.
Thanks @zacnomore !


Push release graph to graphstore

branch off dev_eco
The releases created need to be pushed to Blazegraph, and an 'op' needs to be added
Suggested steps:

use postman to figure out how to push a 'release' file to Blazegraph.
add method to earthcube_utilites/graph/manageGraph... there may already be a method in the code
code operation in schedule template, preferably using the manageGraph so that we maintain one codebase.

do we need a script in earthcube utilites to do this from the command line?

https://graph.geocodes.ncsa.illinois.edu/blazegraph/#update
Use the dev tools to see what it pushes up for a small release file.

post to:
https://graph.geocodes-aws-dev.earthcube.org/blazegraph/namespace/test/sparql?uri=https://oss.geocodes-aws.earthcube.org/gleaner/graphs/latest/summonediris_2023-03-13-15-54-37_release.nq
easy, but nabu needs to be changed to output to latest in a simpler name form, no time, no summoned


nodePackages.typescript-language-server: typescript dependecy as fallback

Make possible to use a different typescript version.
If there is already a typescript binary on your PATH, probably you want
to use that. This way it's possible to use a different typescript
version (with nix shell, direnv, npm, ...), but still have a fallback
version, so the lsp server doesn't fail to start
Related to #73119 and #108235
ping @kira-bruneau @SuperSandro2000

Motivation for this change
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

This looks good to me!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/should-we-support-encoding-runtime-dependencies-without-requiring-a-specific-implementation/21031/6


Top level ARG support

The ARG parameter comes in two shapes. Either it is declared after a FROM statement, and has its scope limited to that stage, or it is declared before any FROM statement, and then it will be global. Because dockerfile-x can interleave the content of several files, some ARG may migrate from top level to stage scope. The answer here is to move every ARG which is local in a single file to the top of the resulting Dockerfile.

Thanks for pointing this, could you give me a full example, so I'll be sure to correctly understand your point (I will try to fix it soon)

I think I understand (tell me if it's not), you can handle this by putting ARG on top on root Dockerfile before any include, so this will let the possibility to globalize an ARG or not

I cannot move the ARG to the top root Dockerfile because that root Dockerfile may be used as the root for different projects, and the ARG would make sense only in one of those projects. In other words, that breaks the include mechanism goal of serving as a way to model common dependencies.

OK, understood, thanks for your feedback, I will add this to the INCLUDE (but not the include feature of FROM and COPY)

OK, understood, thanks for your feedback
I think the problem is what if there are default ARG in one local stage and another default for same named ARG in another stage, what should be the global default, eg:
FROM debian AS builder
ARG FOO=BAR

FROM debian AS server
ARG FOO=BAZ

but I see nothing that is opposed to redeclare the ARG key globally, eg
ARG FOO

FROM debian AS builder
ARG FOO=BAR

FROM debian AS server
ARG FOO=BAZ

I will implement this

It's implemented and released just now, ready to use ;-)
@bonitao / @davireis

I think this fix is causing #16

the approach was not good and caused a regression: https://github.com/devthefuture-org/dockerfile-x/issues/16
the new approach is to only get ARG on top if the ARG is on top of the included Dockerfile (before FROM) and when the Dockerfile is included using FROM or --from, not INCLUDE (to let it agnostic of this).
Example:
# syntax = devthefuture/dockerfile-x
FROM ubuntu.dockerfile

ubuntu.dockerfile
ARG UBUNTU_VERSION=22.04
FROM ubuntu:$UBUNTU_VERSION

Let me try the latest version and give some feedbacks later. 👀


Fixed order and mob IDs

Hm I think maybe we need to leave the old ones in there as well for pre-1.13 versions?


How Use Dynamic permission

hi, how used dynamic permission in this repo.?
please handle with a example.
my second question is how get payload from jwt in client?
i use blazor wasm an my code is:
private IEnumerable ParseTokenClaims(string access_token)
{
    return Jose.JWT.Payload<Dictionary<string, object>>(access_token)
        .Select(keyValue => new Claim(keyValue.Key, keyValue.Value.ToString() ?? string.Empty))
        .ToArray();
}

but dose not work .
tanx a lot

I will develop an example for it in the new branch. keep checking for further updates...

Token is encrypted and cannot be parsed like that (that's why it's called JWE. Use User.Identity.FindFirstValue(string claimName) instead

how i can decrypt this jwe in blazor?
is it possible ?

how i can decrypt this jwe in blazor?
is it possible ?

Nope and I would not recommend that you decrypt a JWT token in client application
just create an action ( GetUserInfo for example) and decorate it with
[Authorize] attribute. and use the User property in the action. there is also an extension for this usage
for example for getting user email you can use User.Identity.FindFristValue(ClaimTypes.Email)

For adding more claims to the access token , modify the configuration in the file below

Example Added
There are a lot of changes in new update
so for dynamic permission example look at this files:
RoleManagerController
RoleManagerService
Related Role Command and query handlers
REMEMBER: each controller that is decorated with [Authorize(ConstantPolicies.DynamicPermission)] attribute will be considered an auth controller that needs special permission or admin role to be accessed. Example


Wheezy: boot warning

/usr/local/sbin/oc-fetch-ssh-keys: line 13: [: ==: unary operator expected

Fixed here: https://github.com/scaleway/image-tools/pull/72


htlcswitch+routing+routerrpc: error source as index and expose decrypt failure

This PR modifies the switch behavior to return an integer indicating the error source instead of a pub key. This allows for a node occurring in the path multiple times. It also changes routerrpc.SendToRoute to return the failure source as an integer.
A second change introduces the ErrUnreadableFailure result for payments. This is a preparation for this case to be processed properly in router. Also an unknown failure (this is different from an unreadable failure) is now distinguishable.
Builds onto https://github.com/lightningnetwork/lightning-onion/pull/37
This PR prepares for persistent mission control #3164

@halseth ptal

@roasbeef ptal

@cfromknecht ptal

The itest single_hop_invoice was failing because of a subtle bug caused by mixing values and pointers for failure messages. Added commit to fix that.

PR rebased after failure reason changes were merged in #3156


Explicitly add docs-csm-reviewers

ssi-reviewers is constantly the only team being queried for reviews.

/backport release/1.4 relaese/1.3


Two options not three

Issue in ./help/using/building-journeys/wait-activity.md
If you want to wait before executing the next activity in the path, you can use a Wait activity. It allows you to define the moment when the next activity will be executed. Three options are available:
Duration
Custom
Suggested edit:
If you want to wait before executing the next activity in the path, you can use a Wait activity. It allows you to define the moment when the next activity will be executed. Two options are available:
Duration
Custom

Thanks for your feedback.


Target languages field fails to accept one language

Extension name: firestore-translate-text
Configuration values (redact info where appropriate):

Target langs: ar

Steps to reproduce:

Use the console or CLI to install the firestore-translate-text extension.
In the target languages field list only one language for example ar

Expected result
I except for the target languages field to be able to accept ONE target languages
Actual result
The target languages field only accepts MORE than one language

OR

Follow up:
It is also possible to input the same language twice:

@dahliasalem Thanks for reporting this. Should be fixed in #46

@dahliasalem thanks once again for reporting this. It has been fixed in v.0.1.1 of the extension. You can update using the Firebase Console or the CLI by running:
firebase ext:update firestore-translate-text --project <projectId>


C# Language Support Request

If you have the time and inclination, I would use your plugin if you added support for C#.

I currently develop an alternative to DocBlockr called DoxyDoxygen
This language is supported.
See Compare Doxygen to DocBlockr for details
Perhaps you may try it.
I am interested by any feedback

I would definitely use this for C# if it were supported. I'm sure many others would as well.


Perf report

PLEASE DON'T MERGE
This PR adds a performance report section to the SidecarDevTools.
It's a table that tells you the average render time of fields/views/layouts on the page, and the number of times they were rendered to detect potential flaws.

@yvan33 This is helpful. Good job! Thanks.
:thumbsup:


Screen Not Reload

Hello there,
I connected via phone and computer and tried to upload a file from my cell phone, the screen went down on me like in the video, but it did not come again, only sound and black screen but when I look at peercalls.com, this area is constantly renewed, how do you do this? It's just enough for me if I find out how you did it.
Chrome WebRTC Log line:
STUN allocate request timed out.
OR
TURN allocate request timed out.
deleted vıdeo links...

There is no problem in peercalls.com v3

case STREAM_TRACK_ADD:
return addStreamTrack(state, action.payload)
not working.
thx.

Hi @ahmetozalp, are you still experiencing the issue? I haven't had the chance to see the video

I live STREAM TRACK ADD does not trigger at all

Proje hatası video bağlantım: https://youtu.be/mn8Ftk1ssJo
Peer Calls video bağlantısı: https://youtu.be/s3hVOF7IT8k

have you tried peercalls.com/beta ?

Can you reproduce this on https://peercalls.com/beta ?

thank you problem fix.
`handleTrack = (track: MediaStreamTrack, stream: MediaStream) => {
const { user, dispatch } = this
const userId = user.id
debug('peer: %s, track', userId)
// Listen to mute event to know when a track was removed
// https://github.com/feross/simple-peer/issues/512
track.onmute = () => {
debug(
'peer: %s, track mute (id: %s, stream.id: %s)',
userId, track.id, stream.id)
dispatch(StreamActions.removeTrack({
userId,
stream,
track,
}))
}
function addTrack() {
  debug(
    'peer: %s, track unmute (id: %s, stream.id: %s)',
    userId, track.id, stream.id)
  dispatch(StreamActions.addTrack({
    userId,
    stream,
    track,
  }))
}

if (!track.muted) {
  addTrack()
}
track.onunmute = addTrack`


Configuring Various Themes in kickstart.nvim Using rebelot/kanagawa.nvim

Description
I am currently facing an issue while trying to configure different themes within the kickstart.nvim project using the rebelot/kanagawa.nvim plugin. I am seeking assistance in resolving this matter.
Problem
I have started my journey with nvim and opted to begin by setting up the kickstart.nvim project. To switch between themes, the kickstart project utilizes the lazy.nvim plugin. I initially tried using the "axiom" theme along with the themes from kanagawa, but I am only able to get the light mode to work.
Specific Issue
The main problem I'm encountering is that I cannot seem to get other themes, such as "dragon," to function as expected. Despite following the configuration provided below, I am unable to apply the desired themes:
{
    -- Kanagawa Theme
    "rebelot/kanagawa.nvim",
    priority = 1000,
    config = function()
      vim.cmd("colorscheme kanagawa-dragon")
    end,
  }

Request for Assistance
I am reaching out to the community for guidance and support. Could you kindly review my configuration and let me know if I'm missing any crucial steps? Your expertise would be invaluable in helping me overcome this roadblock.
Did i Read correctly The Friendly Manuel  ?
Thank you in advance for your time and assistance.
Best regards,
BBA

Set the bg option to dark
:set bg=dark
And see if it works, if so
vim.o.bg="dark"
Will work when you past in your config
If it doesn't work try
:KanagawaCompile

Thank you :+1:  @pooseethestroyer it's working !

But why is it working?


AllFlagsState to get all flag value for a specific user

Description

Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking changes (change that is not backward-compatible and/or changes current functionality)

Closes issue(s)

Resolve #
Checklist

[ ] I have tested this code
[ ] I have added unit test to cover this code
[ ] I have updated the documentation (README.md and /docs)
[ ] I have followed the contributing guide

Coverage decreased (-2.4%) to 94.087% when pulling e3a952fa3981dd419941c1c77d5d6918e6973ea8 on feat/user-all-flags into 057088a9cc7d0f6f5da30bf6c3f7af7b97e0bf68 on main.


Add missing colours to jsx code blocks

Changed the markdown language on JSX code blocks.
In reference to the following ticket: https://github.com/microsoft/TypeScript-Website/issues/396
Before

After

This is my first time contributing, if I've got the wrong end of the stick please let me know!

Nah, this looks perfect 👍


Can the FlaUI get the css-selector in IE or Chrome

I can get the xPath in IE,but it can not get the point with Text .However i can get the point in chrome ,but flaui can not get the xPath.
So i think FlaUI can get the css-selector both on ie and chrome? the css-selector can let me find the same element .
If you have some idea ,please response to me as soon as possible.

Hi,
It's not possible. You have to use Selenium or any other tool that's designed for testing web apps. Now you're trying to drive a nail using scissors.
Regards,
PM

Thank you for your reply，i will try to use Selenium if it can conforms to my request @pedroNew

FlaUI automates Windows Applications and uses what the Windows Automation API provides. Some browsers convert some web content into Windows Controls and therefore provide limited support for FlaUI and other Windows Automation API based products. If you want to fully test web applications, using something like Selenium is definitely the better way.

Thank you for your reply，I know the FlaUI is based on the uiautomation, I just see it can get the xPath in IE，So I wondered if FlaUI can also get the xPath in other browsers too.
Thank you for your suggestion， I am trying to use Selenium in browsers @Roemer


Add travisCI

It would be a good idea to include TravisCI or CircleCI to automate some testing when a PR is issued. for example: Someone issues a new plugin PR, travis checks if the new plugin matches the structure defined in the readme.

Can I work on this?

@wupewu sure!


[master] Add support for map -> record assignment

Purpose

This PR adds support for assigning compatible map values to record typed variables.
Fixes #17202

Samples
import ballerina/io;

public function main() {
    map<string> m = {"a":"aaa", "b":"bbb"};
    record {| string...; |} r = m;

    io:println(r);
}

Remarks

Came across #22276 while writing tests for this.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #22292 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #22292   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 87ade9f...919793a. Read the comment docs.


Switch to npm5

This should make installation time (and therefor builds) much faster and make lockfiles way more reliable, especially in regard to handling of optionalDependencies like fsevents.
I just ran scripts/npm install with npm5, which upgrades all the shrinkwrap files, then renamed then.
Additionally updated npm version in Travis and adapted gulpfiles.
I recommend merging this without squashing so the renames are recorded and not a huge delete+add.

Install step on master: 350.53s
Install step on this branch: 143.84s

Awesome numbers. Is the idea that people stop using shrinkwrap altogether?
Did you actually update to the latest versions of our runtime dependencies? That is something we wouldn't take very lightly as it's bitten us in the past - odd issues here and there. Can we somehow do this without actually updating our runtime dependencies versions?

For projects, yes. shrinkwrap still serves a purpose for locking the dependency versions of published packages, e.g. if you install gulp and gulp had an npm-shrinkwrap.json, the dependency tree of gulp's dependencies would be locked. You generally don't want this. npm-shrinkwrap.json and package-lock.json have an identical schema, but the former is published to npm, the latter not. package-lock.json is the default for new projects.
I did not update any versions - npm migrates the schema automatically, and I then renamed all the files. All dependencies should still locked to the same version (you can see this if you look at https://github.com/Microsoft/vscode/pull/29576/commits/cc38d99ac4d542113c052f069e94554edbe600ed). Also, devDependencies are now included by default.

What happens if we leave the shrinkwrap files in the repo as well? IF we remove them we force everyone to update to npm@5 at once, which is not ideal.

I wouldn't keep them both, because then we risk them getting out of sync. It also wouldn't be of any use because the schema was upgraded and is not backwards-compatible (i.e. npm5 can upgrade npm4 shrinkwraps, but npm4 can't read upgraded npm5 shrinkwraps/package-locks)

Exactly, so it needs to be a coordinated event across the whole dev team, internal and external build definitions and documentation.

yep, just like on a Node version update.
As this PR is very prone to merge conflicts and a merge conflict fix means redoing the whole PR, it would be nice if this could get merged quickly - wdyt about adding a check to npm.sh that checks the npm version and prints something like "Please run npm -g npm to update to npm 5"? That would make the fix obvious and easy to any dev who's still running npm 4 :)

I added a check to both npm.sh and npm.bat and tested on mac and Windows
✘-1 ~/git/vscode [master|✚ 1⚑ 5]
17:22 $ scripts/npm.sh i
VS Code requires npm v5
Please run npm i -g npm to update

@alexandrudima Pointed out another source of issues: our internal OSS tracking tool, which currently gets its input from npm-shrinkwrap.json files. It shouldn't be tough to upgrade though.

@felixfbecker I'd put the npm checks in here instead: https://github.com/Microsoft/vscode/blob/master/build/npm/preinstall.js#L6
We use the preinstall.js file for catching issues when launching npm. It's usually only executed during install, but that should be fine. I just want to keep the batch scripts as small as possible.

Done.

@alexandrudima Pointed out another source of issues: our internal OSS tracking tool, which currently gets its input from npm-shrinkwrap.json files. It shouldn't be tough to upgrade though.

Depending on what properties it uses, it might not have to change at all

Oh boy, shrinkwraps got updated in master...

Rebased and re-migrated all changed shrinkwrap files.

@felixfbecker No worries about keeping this up to date with master. I can take it from here. I plan on merging this next week, after endgame.

Okay, cool.

Spent the whole afternoon on this. It's not as trivial at it looks.

npm 5.0.4 has issues, npm install fails with that version. So make sure you use 5.0.3.
Some dependencies seem to be missing from the top level package-lock.json, especially the platform dependent, optional ones, like windows-mutex. Need to look into that.
The package-lock.json inside build/lib/watch is broken, so one needs to delete it to make this work.

I'll pick it up tomorrow, but I might just trash this PR and start over. Sorry for that @felixfbecker.

Some dependencies seem to be missing from the top level package-lock.json, especially the platform dependent, optional ones, like windows-mutex. Need to look into that.

I think with shrinkwrap it only saved what you had installed, so if you run shrinkwrap on a mac, it would save fsevents, if you run it on a Windows machine, it would save windows-mutex - never optionalDependencies not installed. If you tried to install from a shrinkwrap that included platform-incompatible optionalDependencies, npm would error. I think npm 5 now saves all optional dependencies, but only installs those that can be installed.
But because I just renamed the files, the needed ones might not be in there.
Another problem I saw on this branch is that there seems to be some manual modifications in the npm-shrinkwrap.json to replace transitive dependencies with a fork - this seems to fail for npm5 with an EINTEGRITY error because the forked version's hash doesn't match the hash in the shrinkwrap (just a theory, could be different issue).
No problem if you start over, I'm happy if npm5 makes it into master in any way :)
I hope this PR still served a purpose as a proof of concept or draft.

And as I suspected: npm i will always update package-lock.json with whatever it really installed. So... fsevents won't install on Windows, so running npm i on Windows updates the package-lock.json and removes it from there.

We can't really take this right now since we have optional dependencies which are platform specific. We'd have constant changes in these files all the time.
I see some activity going on in npm/npm#16839, which seems related, let's see if that fixes it.
I've created #30134 so we remember to do this eventually.


tml: init at 0.6.0

https://github.com/liamg/tml
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 234489 run on x86_64-linux 1

1 package built:

tml

the tag doesn't have a changelog, but ok

the tag doesn't have a changelog, but ok

Upstream's details only qualifies partially and sometimes as changelog.


Prelaunch + optional splash screen

I couldn't improve performance of lists because code not clean

I'm unsure what prelaunch here does. If you enable prelaunch without doing anything it wouldn't help much perf wise I would say.


Fix the feedback form

# What’s been improved:
The form has a centered, white background with a slight shadow to make it pop.
Larger and more readable input fields.
The submit button is now more visible and responsive with hover effects.

This PR is for the Issuse for #3

@GarimaSingh0109
i am unavailable to set the lable . can you help me?

Thank you label the PR


Update Typeahead to work with special characters

Fixes #50
@wxsms

Coverage increased (+0.005%) to 95.356% when pulling cc09e8a83c8032d3b98dd0b6a35ed2ce971e0b91 on AdrianLungu:master into 723ce6eaf473b5571c680deedfe49c88a4aae23a on wxsms:master.

Thx!


refactor: inline svg loader

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Refactoring

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #9716
inline-svg loader
async import svg file

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

Copilot Summary
copilot:summary
copilot:poem

Farcaster icon seems a bit smaller than the others on footer, other than that looks good

Farcaster icon seems a bit smaller than the others on footer, other than that looks good

Okay, I have adjusted it to be as big as before.

😍 Perfect, I’ve sent the payout
💵 $30 @ 11.75 USD/DOT ~ 2.553 $DOT
🧗 16SjUbGKSdjCdWTy3NNT3JxbRVGGqD4mwkHpc6BD9U2Rp29Z
🔗 0x3d76380868c7fad3589538b0385ac85b450dd9a0b7211ff2ac83a8a5cfa8bee1
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Unable to create document from custom content type, using document template

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
not applicable
Developer environment
None
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
No response
Describe the bug / error
If we create a content type in the Content Type Gallery and specify a document template URL, the content type cannot be used to create a new document in a library using the "New" menu (after pulling the content type in to the libary). The users have read access to the document template.
When selecting the content type from the "New" menu, a tab is opened and then immediately closed. Upon inspection, the following error is found in the console:

The document template is stored in a different site, to the site in which the content type is being used e.g.
Template URL: /sites/Templates/Shared Documents/Short document template.dotx
Library: /sites/Testteam/Shared Documents
The content type is pulled from the Content Type Gallery, with the document template URL property already filled in.
This also occurs if you upload the document template to the content type within the Content Type Gallery and then pull that content type down to the site. This leads to a synchronisation error which can be found in the Content type publishing error log:

When using the "New" menu it seems that the GetListUsingPath API call, is unable to handle a server relative URL that refers to a document template in another SharePoint site.
The alternative of uploading the document template to the content type via the gallery, doesn't seem to work because the file is not synchronised to sites when the content type is pulled down.
Steps to reproduce

Create a custom content type in the Content Type Gallery, inheriting from the Document content type.
Open the content type in the Gallery and click Settings and then Advanced settings
Enter a URL of document template stored in a SharePoint site in your tenant. Ensure that you have access to this site and document template.
Navigate to a library in a separate site to where your template is stored.
Click the "+Add column", scroll to the bottom and click "Add a content type", then click Next.
Choose the content type you created in step 1 and apply.
Use the New menu and attempt to create a document using the content type.

Expected behavior
After clicking the content type from the "New" menu, a new document will be created using the document template for that content type.

Thank you for your submission. As explained in our wiki (Issue List: What doesn't belong in the issue list), this issue list is for SharePoint developer/development issues. All capability question/discussion questions, or topics related to SharePoint administration & end-user topics should be reported through the support user interface available in the tenant admin settings. You can also have a discussion and ask questions at the SharePoint TechCommunity forum. You can learn more about this in our wiki: type:invalid-not-dev-issue


Renaming of "community" Repo to ".github"

It would help to rename the community repo to .github, which is a special repo name, and thus would be displayed on all repos of the whole organization.

Just did the rename, so this is done.


Documentation | Guide - Add Icons

Reference:
https://geist-ui.dev/en-us/components/icons

Please give details over the issue

Hello @maddy020 !
Regarding the Icons, as you may have noticed we use a third party library for those called react-icons, I've indicated what we are referring to just above!
As for #70 you just need to create a documentation in ./docs/pages/docs/guides/icons.mdx from ./examples/next-typescript/pages/button.tsx without forgetting to mention react-icons or even to specify that it is possible to use another library or just an svg as long as it is an svg


Feature/dig 53247 file upload frontend

Big redesign of ffe-file-upload-react including new styling package ffe-file-upload, new design can be seen here (requires access): https://www.figma.com/file/cn3VFs107XnyxeLo4Du4ny/Fil-Opplasting---Master?node-id=0%3A1
Some examples of the new design:

These are breaking changes.

Travis doesn't seem happy, https://travis-ci.org/SpareBank1/designsystem/builds/582730542#L837, but other than that it looks good to me.

Travis doesn't seem happy, https://travis-ci.org/SpareBank1/designsystem/builds/582730542#L837, but other than that it looks good to me.

Yeah, this is due to the role="presentation" I removed on the 'drag and drop' receiver element based on comment on the other PR. Unsure what is preferred here :)

Then I would suggest the region along with aria-label describing it as a droppable area.

Then I would suggest the region role along with aria-label describing it as a droppable area.

Based on conversation in PM, we decided to add the role="presentation" back and we'll create new case issue to deal with potential screen-reader problems

Coverage decreased (-0.5%) to 81.441% when pulling 15465ca9957adfbbbf067e4ae15d5b2d306555c6 on jafeld:feature/DIG-53247-file-upload-frontend into d2748fc1d4b6cad0bf7168475f377f0342742310 on SpareBank1:develop.

@henrikhermansen @kwltrs your requested changes is what's stopping the merge right now. I'm not sure how to deal with them.


Minor Text Fix

Very minor, added a space to correctly render text

@BickNooth : Thanks for your contribution! The author, @stevestein, has been notified to review your proposed change.

Merging typo fix


Feature request about using SpringBoot ELK Kafka

1.As we know,Pinpoint is a SpringMVC project,does the Pinpoint Team have plan to convert SpringMVC to SpringBoot?
2.Pinpoint now is using Hbase to stroe,I have met a problem that using Hbase has not been able to satisfy the multi-condition multi-dimensional query,I think ElaticSearch is good way to solve the problem.
3.I think agent->kafka->collect->web is a good way to reduce the server stress.
I really think Pinpoint is very good APM product,I think it will be better.

@emeroad
What do you think? :)

Could you tell me about this feature request ?

There is a plan. But the priority is very low
There is no plan to support ELK
I do not intend to place kafka in front of the collector.
I am thinking about placing Kafka behind the collector. This feature is low priority


Improve modeling of annotations in Ion Schema

Problem
The annotations constraint has some confusing interactions between the modifiers (ordered::, closed::, required::, and optional::); and without any modifiers, the annotations constraint actually does nothing, which is a really confusing default behavior. The annotations constraint is also unable to model things like:

Annotations should be any lower-case string
There should be only one annotation, but it can be anything.
There should be no duplicate annotations
First annotation should be one of N values, second annotation should be one of M values.
There should be exactly one annotation that looks like a Java canonical-form class name.

Possible Solution
We could model annotations (approximately) as a list of symbol tokens, and allows the annotations constraint to take a <TYPE_REF> as its argument.
For example:
type::{
  name: annotated_type_1,
  // max 10 annotations
  // annotations must start with a lower case letter and consist only of lowercase letters and numbers
  // no duplicates annotations allowed
  annotations: {
    container_length: range::[min, 10],
    element: distinct::{ // <---------- As proposed in ion-schema#43
      regex: "[a-z][a-z0-9]*"
    }
  }
}

type::{
  name: annotated_type_2,
  // Has 1 of M media types followed by 1 of N encodings
  annotations: {
    ordered_elements: [
      { valid_values: [ image_jpeg, image_gif, image_png, text_plain, application_json ] },
      { valid_values: [ gzip, deflate, zstd ] },
    ]
  }
}

type::{
  name: annotated_type_3,
  // Must have exactly 1 annotation that looks like a fully-qualified java class name
  annotations: {
    container_length: 1,
    element: { regex: "([\p{L}_$][\p{L}\p{N}_$]*\.)*[\p{L}_$][\p{L}\p{N}_$]*" }
  }
}

This can be done in a backwards compatible way that preserves the existing syntax because all <TYPE_REF> are either a symbol or a struct, which can be clearly distinguished from the existing syntax that is a list of symbols.

In the course of modeling PartiQL test cases in ion text, one proposal for a way to model the test cases was something like:
'literal'::[
  'int'::parse:: "5",
  'null'::parse:: "null",
  // ...
  'nested empty struct'::parse:: "{'a':{}}"
]

In attempting to model this in Ion Schema, it doesn't currently seem possible to:

require an annotation on the list (e.g., ‘literal’) in order to give the test namespace a name
require an additional annotation on the type that already requires the parse annotation, where that additional annotation would be the test name

Proposal
The annotations constraint will be expanded to define the type and/or constraints for all annotations of a value. Annotations are symbols tokens but will be represented as a non-null list of non-null, un-annotated symbol values for the purpose of validation.
The existing annotations syntax shall remain, since it is the most concise and readable syntax for use cases such as required::closed::[one_required_annotation].
Syntax
<ANNOTATION> ::= <SYMBOL>
               | required::<SYMBOL>
               | optional::<SYMBOL>

<ANNOTATIONS_MODIFIER> ::= required::
                         | ordered::
                         | closed::

<ANNOTATIONS> ::= annotations: <ANNOTATIONS_MODIFIER>... [ <ANNOTATION>... ]
                | annotations: <TYPE_REFERENCE>

Evaluation

if the constraint argument is an Ion list, use the Ion Schema 1.0 behavior.
if the constraint argument is not a list, convert the annotations to a non-null list of non-null, unannotated symbols, and validate that list against the given type reference.

Sample Implementation
This is an example of one way of implementing the constraint to serve as a reference for the correct behavior. There is no requirement that it must be implemented this way. This example covers only the new logic. Actual implementations should also include the existing annotations logic.
private fun test(value: IonValue, typeRef: Type): Boolean {
    val annotationsList = value.typeAnnotations.mapTo(ionSystem.newEmptyList()) { ionSystem.newSymbol(it) }
    return typeRef.isValid(annotationsList)
}

Alternatives Considered
It would be possible to completely eliminate the existing annotations syntax since this is going into a new major version of Ion Schema and therefore is not required to be backwards compatible. However, the reasons for keeping the old syntax are twofold

A common use case is to require or allow only one specific annotation. The new syntax may be (subjectively) less readable than the old syntax. For example:

Old Syntax
New Syntax

annotations: closed::required::['com.fooservice.Request']
annotations: { valid_values: [['com.fooservice.Request']] } or annotations: { contains: ['com.fooservice.Request'], container_length: 1 } or annotations: { ordered_elements: { valid_values: ['com.fooservice.Request'] } }

annotations: required::['com.fooservice.Request']
annotations: { contains: ['com.fooservice.Request'] }

annotations: closed::['com.fooservice.Request']
annotations: { valid_values: [[], ['com.fooservice.Request']] } or annotations: { ordered_elements: { occurs: optional, valid_values: ['com.fooservice.Request'] } }or annotations: { element: { valid_values: ['com.fooservice.Request'] }, container_length: range::[0, 1] }

Backwards compatibility will make it easier for users to upgrade from Ion Schema 1.0 to 2.0, though this could be mitigated with a schema upgrade tool.

The behavior of some of the ISL 1.0 syntax is unexpected and/or not useful, so perhaps only part of the old syntax should be retained. By removing ordered:: and by requiring at least one of closed:: or required::, we can eliminate all of the potentially confusing or useless cases mentioned in https://github.com/amzn/ion-schema-kotlin/issues/150, while also preserving the succinct syntax for the simplest use cases.

Resolved by #76 in Ion Schema 2.0


SFDX unable to retrieve metadata from org using package xml

Summary
Hi team, I faced this issue a couple of times already and no idea how to work around this. I was attempting to retrieve the metadata from my sandbox through the sfdx command and via the "SFDX: Retrieve Source in Manifest from Org" option found when you right-click the package.xml file. I wasn't to able retrieve the metadata either way, so I am stuck. When I did run the sfdx command through the terminal, it returned an error, with ErrorId:<PHONE_NUMBER>-218606 (796380918)
Hope that ErrorId has some use. I noticed there were other issues similar to mine, and in one of the issue thread, someone requested to run the sfdx retrieve command with the "--dev-debug" keywords at the end of the command and I managed to find out what the error is, attached in the screenshot below. Please let me know if you need any other information, thank you for your help in advance!
Steps To Reproduce:

Create a new project with SFDX: Create a Project with Manifest.
Authorize any org (Dev Org/Sandbox/Production) doesn't matter
Try to retrieve the code from Package.xml (Package xml contains a whole list of metadata)
You'll find the "Unknown Exception" in the Output Panel of the VSCode

Expected result
The code should get retrieved which is defined in the package.xml
Actual result
Received an UNKNOWN_EXCEPTION: UNKNOWN_EXCEPTION: An unexpected error occurred. Please include this ErrorId if you contact support:<PHONE_NUMBER>-242753 (796380918) error message
Additional information

Salesforce Extension Version in VS Code: v54.5.0
SFDX CLI Version: sfdx-cli/7.140.0 darwin-x64 node-v16.14.0
OS and version: macOS Monterrey

Hi @hreethwan, is this issue consistently reproducible? The please include this ErrorId... message is coming from the server so I don't believe it's anything related to the client-side CLI. Have you tried converting it to mdapi-format and using force:mdapi:retrieve? That'll tell us if it's a client-side or server-side issue

Hi @WillieRuemmele thank you for following up. I was attempting to retrieve the metadata from the org again today, using sfdx force:source:retrieve. The Error Id I received is somewhat similar --> UNKNOWN_EXCEPTION: An unexpected error occurred. Please include this ErrorId if you contact support: 511912618-81534 (796380918)
I attempted the metadata retrieval using the mdapi-format using this command -->sfdx force:mdapi:retrieve -u marsuat -k manifest/package.xml -r /Users/hreethwan/Desktop
This time, I am getting another error: Element {http://soap.sforce.com/2006/04/metadata}type invalid at this location in type Package
I noticed that this "http://soap.sforce.com/2006/04/metadata" url was part of parameter in my package.xml, but even after deleting it, I received the same error. Any advice on what I should do?

Thanks for trying that, did you run sfdx force:source:convert and then the mdapi:deploy?


java.io.IOException: Decompression error: Unknown frame descriptor

Hi,
sorry to bother again, but unfortunately I bumped into an issue where I'm actually not quite sure what is causing this and how to fix it. Even though the issue is reproducible (in production) and always points a specific piece of code I'm currently unable to tell what is causing this behaviour.
In a nutshell what I'm currently doing is that I create a writer with closeFrameOnFlush=true and a reader with continuous=true. Reader and writer might run in parallel threads. The writer-Thread flushes every x-Elements (e.g. every 100 elements) and notifies the reader that he can start reading. Since the zstd-Frame is closed at a flush on the writer side and continuous-Mode is enabled on reader side this should technically just give the reader a complete frame to read and should not cause an Exception.
Implementation wise the reader and writer are created in the following way (quite similar to what I outlined in https://github.com/luben/zstd-jni/issues/52 just with threads)
            private static final int ZSTD_CLEVEL_DEFAULT = 3;
            // create writer
            final BufferedWriter writer = new BufferedWriter(
                    new OutputStreamWriter(new ZstdOutputStream(
                            Files.newOutputStream(compressedFile.toPath()),
                            ZSTD_CLEVEL_DEFAULT, true),
                            StandardCharsets.UTF_8));
            // create reader
            final BufferedReader reader = new BufferedReader(
                    new InputStreamReader(
                            new ZstdInputStream(
                                Files.newInputStream(compressedFile.toPath())
                            ).setContinuous(true),
                            StandardCharsets.UTF_8));

The Exception that currently pops up is the following:
java.io.IOException: Decompression error: Unknown frame descriptor
	at com.github.luben.zstd.ZstdInputStream.read(ZstdInputStream.java:111)
	at sun.nio.cs.StreamDecoder.readBytes(StreamDecoder.java:284)
	at sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:326)
	at sun.nio.cs.StreamDecoder.read(StreamDecoder.java:178)
	at java.io.InputStreamReader.read(InputStreamReader.java:184)
	at java.io.BufferedReader.fill(BufferedReader.java:161)
	at java.io.BufferedReader.readLine(BufferedReader.java:324)
	at java.io.BufferedReader.readLine(BufferedReader.java:389)

I currently can't reproduce the issue with testcases (so the tests and code seems to work fine), but the "production" code is producing this exception reliably. Cleaning up any produced files and have them regenerated on-the-fly does not fix the issue.
I can reproduce the issue with 1.3.3-4 and 1.3.4-1
When looking at the zstd-project the issue seems to be related with "Support for Legacy format". Since I can also reproduce the issue with a clean work environment the files generated are not "old" hence I don't see any relation.
Just a side note:
I can open the files with the zstd command-line tools without any error-messages.
Any help on how to troubleshoot this further or what might go wrong is appreciated.

Since everything seems to work fine with 1.3.4-2 I'll close the ticket and consider it as fixed by
https://github.com/luben/zstd-jni/commit/162661e867f0a279859da2c6eb9cdc7793355bbc
Thanks again for your help and the quick fix :-)

Thanks for reporting and testing the fix.


HIVE 20958: clean code at Hive common module

https://issues.apache.org/jira/browse/HIVE-20958

@kgyrtkirk thanks for taking a look, i will add some check style rules about import order. Current code base has no unique style which is bad IMO some files start with java.* and some other java.* is grouped to the end. I think most of the new projects and IDE prefer grouping JDK stuff at the bottom with a different group for static imports. What you think ?


Fix/case

fix duplicate file issue related to case sensitivity

Fixing issue with case sensitivity, squashing commits.


Fix issue with scroll when camera is being moved

Fix for issue #644 by decreasing the camera's z velocity when the camera is moving.

Merged. Thanks!

Package release in progress. Should be out there in about 20 minutes.


Use fully-qualified syntax in triple! for calling from_str

This way the caller doesn't need to import FromStr.

Thanks!

This is now released in 0.12.10.


Full length comments not supported. Issue with Underlying ufw not with pyufw.

Hey I noticed that if you try to use quotes to get full length comments it isn't supported. You can modify the parser.py file provided from ufw by changing the code to:
if "'" in comment:
#combine all the comment strings into 1
comment = ' '.join(argv[comment_idx+1:len(argv)])
comment = comment.replace("'","")
#delete comment group arguments from argv
argv = argv[:argv.index('comment')]

Still waiting 🥺


Improve / redefine Result::resultPoints

I'd like to propose a change regarding the Result::resultPoints contents and name. Currently the contents of this std::vector is basically undefined. The documentation says it contains points of interest like e.g. the position of detection markers. So the number of points in this array and their meaning change from barcode format to barcode format.
I assume that almost no one makes use of this information and if so, they assume the points form a convex quadrilateral which contains the area of the barcode. That is e.g. true for the DataMatrix reader but not quite for the QRCode reader and apparently not at all for the PDF417 reader. In the 1D case the behavior is also inconsistent across the different formats.
I suggest to deprecate this member and instead introduce a new one called Result::position. And this should provide a struct containing excatly 4 2D points describing the above mentioned quadrilateral. The struct should have a means to identify which point is which, like Point topLeft(), etc. methods.
An open question for me is whether those points should contain int or float coordinates. There is potentially sub-pixel precision information available but I highly doubt their usefulness on the client side. And they have the downside that you would need to communicate and understand

what exactly a position of e.g. (0.5, 0.5) means (center or corner of the first pixel)
what exactly the position of each corner means (center of the first pixel of the barcode or edge
between background and barcode)

To make it most strait forward and user friendly, I'd go for integer precision and have it mean "the quadrilateral that covers all the area that went into the reading of this code". For 1D codes this will mean it contains only one line.
Again, I'm not interested in creating more problems than solving, so if @huycn, @vkrause, @bimusiek or others have any feedback, please let me know.

We aren't using that feature at all AFAIK, but the proposed changes sound reasonable to me.


PermissionStore.{function} Callback not called

I can't get my callback function to be called on this code:
PermissionStore.definePermission('admin', function (stateParams) { console.log("ok");     });
Nothing happens to the console.
Context:
var app = angular.module('MyApp', ['ui.router', 'permission', ...]);

app.run(function (PermissionStore) {
    var permissions = ["Admin", "Super"];

    PermissionStore.defineManyPermissions(arrayOfPermissionNames,
        function (stateParams, permissionName) {
            console.log("ok");
    });

});

Never mind. The function is only called when we ask for the validation.
I have the idea of roles of the and permissions of an resource. And I need to load and store that info at start time of the app.


Grid unresponsive with large datasets when columns are set to autoSizeAll

Here is a demo app to reproduce:

import dash_ag_grid as dag
import dash
from dash import Input, Output, html, dcc
import pandas as pd

app = dash.Dash(__name__)

df = pd.read_csv(
    "https://raw.githubusercontent.com/plotly/datasets/master/liquor_iowa_2021.csv"
)

app.layout = html.Div(
    [
        dcc.Markdown(
            "Switch between autosize and size to fit to see the columns respond. Columns can also be resized by dragging at their edge."
        ),
        dcc.RadioItems(
            id="columnSizing",
            options=[
                {"label": i, "value": j}
                for i, j in [
                    ("Auto size", "autoSizeAll"),
                    ("Size to fit", "sizeToFit"),
                ]
            ],
            value="autoSizeAll",
        ),
        dag.AgGrid(
            id="input",
            columnDefs=[{"field": i} for i in df.columns],
            rowData=df.to_dict("records"),
            columnSize="autoSizeAll",
            defaultColDef=dict(
                resizable=True,
                sortable=True,
                filter=True,
                minWidth=100
            ),
        ),
    ]
)

@app.callback(Output("input", "columnSize"), Input("columnSizing", "value"))
def column_sizing(size_type):
    return size_type

if __name__ == "__main__":
    app.run_server(debug=True)

In the meantime, passing columnSize=None and then using a callback to set autoSizeAllColumns with True will perform the action when the grid is ready to receive updates and auto sizes the columns accordingly.

Fixed in #28 :tada:


universal filter generates RGB image/urf for the monochrome printer

This is derivative from the following bug reported against CUPS: https://github.com/OpenPrinting/cups/issues/804
Although hotfix for this problem exist (https://github.com/OpenPrinting/cups/pull/805), the actual problem is in the cups-filters (according to @michaelrsweet, and I agree with him).
With the PPD file, autogenerated by CUPS for the Pantum BM5100ADW Series printer (see attachment), the universal filter generates RGB image/urf output instead of the Grayscale one. Printer rejects this image, which effectively breaks driverless printing on this device (and probably on many other devices with similar behaviour).

Pantum_BM5100ADW_Series_2DFFA2.ppd.txt -- the problematic PPD, auto-generated by CUPS

Adding the following line to the PPD file fixes the situation:
*cupsUrfSupported: "V1.4,W8,IS1,CP99,PQ4,OB10,RS600,DM1"

However even without this line the generated PPD should produce monochrome image. According to @michaelrsweet:

...but the bug is in cups-filters - the ColorModel option code, when executed, specified sGray as the colorspace. And the PPD specifies ColorDevice: False...

I'm willing to spend some time investigating this problem in cups-filters, so for now I assign this issue to myself

Hi @alexpevzner ,
cups-filters nowadays is a place for filter binaries and backends for CUPS 2.x, and all logic was moved into libcupsfilters and libppd, so IMO the issue will be somewhere in those libraries.
I've reproduced the issue by running the filter by hand and the output file is in color - I see there are logs about ghostscript filter using srgb.icc profile, which might cause the issue - I will try to dig deeper once I have time.

Hi @zdohnal,
my first suspicion was ppdFilterLoadPPDFile() from the libppd - it reconstructs a set of IPP printer attributes from the supplied PPD.
Though the itself idea perform double conversion (IPP->PPD->IPP) sounds, as for me, a little bit quirky, the reconstructed attributes looks correct. So my next suspicion is the libcupsfilters, where parameters for Ghostscript are generated from the printer attributes.
From the filter log, Ghostscript is called with incorrect parameter -dcupsColorSpace=19, must be 18 for the grayscale image.


Add missing const_cast<>.

Known failures: #22523
The ALTS interop failure is a transient problem fetching external deps.


Variable size encoding of draw objects

This patch switches to a variable size encoding of draw objects.
In addition to the CPU-side scene encoding, it changes the representation of intermediate per draw object state from the Annotated struct to a variable "info" encoding. In addition, the bounding boxes are moved to a separate array (for a more "structure of "arrays" approach). Data that's unchanged from the scene encoding is not copied. Rather, downstream stages can access the data from the scene buffer (reducing allocation and copying).
Prefix sums, computed in DrawMonoid track the offset of both scene and intermediate data. The tags for the CPU-side encoding have been split into their own stream (again a change from AoS to SoA style).
This is not necessarily the final form. There's some stuff (including at least one piet-gpu-derive type) that can be deleted. In addition, the linewidth field should probably move from the info to path-specific. Also, the 1:1 correspondence between draw object and path has not yet been broken.
Closes #152

Agree we should improve the magic constants and consolidate types and reader/writer functions into header files. Will try to roll that up into future updates.


Submit additional form data using codeigniter

I got jQuery File Upload (blueimp-jQuery-File-Upload-a747af3) working with Codeigniter v2.1.0.  Now I'm trying to add a text input for users to include a comment with the upload.  After reading https://github.com/blueimp/jQuery-File-Upload/wiki/How-to-submit-additional-Form-Data  it works great with a standalone install, but with Codeigniter I get an Internal Server Error message, and in Firebug Console I see "The action you have requested is not allowed".  I can't seem to figure out where to go from here.
Any suggestions would be greatly appreciated.
Mark

To be a little more clear, the data is being posted, but the file is not being processed/saved.
From Firebug:
Source
-----------------------------28617237579832 Content-Disposition: form-data; name="title" Title goes here -----------------------------28617237579832 Content-Disposition: form-data; name="comment" Comment Goes Here -----------------------------28617237579832 Content-Disposition: form-data; name="userfile"; filename="IMG_20070610_1806.JPG" Content-Type: image/jpeg ÿØÿá%Exif��II*����
Again, the changes I've made come directly from "Setting formData on upload start for each individual file upload" on https://github.com/blueimp/jQuery-File-Upload/wiki/How-to-submit-additional-Form-Data

Hi Mark,
It's most probably because you have enabled csrf in codeigniter:$config['csrf_protection'] = TRUE;
$config['csrf_token_name'] = 'ctn';
$config['csrf_cookie_name'] = 'ccn';
$config['csrf_expire'] = 7200;
You can disable csrf, or:
To fix this, you have to add a hidden input into your form with the name of csrf_token_name and value of csrf_cookie_name. This input has to be passed in the POST data along with the other info.
Tested & worked.
Let me know if it does for you, too.

Thanks for the reply.  After checking, I found CSRF to not have been enabled.  I have no idea what happened, but as it turns out it was only an issue with Firefox on one machine.  I found it working on my other machines after I took my laptop outside with me to have a cigarette.  After removing/reinstalling FF, everything seems to be working great.


Refs #25589 -- Fixed admin_scripts test failure on Windows.

Traceback (most recent call last):
File "tests\admin_scripts\tests.py", line 646, in test_startapp_unicode_name
content = f.read()
File "lib\encodings\cp1252.py", line 23, in decode
return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x81 in position 46: character maps to

buildbot, test on windows.


JDK necessary in builder image?

Seeing "Installing adwaita-cursor-theme-3.36.1-1.fc32.noarch" scroll past as create_container.sh was running, I looked into where that dependency came from. It's java-11-openjdk:
https://github.com/monogon-dev/monogon/blob/6ef7f9bb94890748cc7c635f187fce7c5f497fe3/build/ci/Dockerfile#L34
I then removed this package from build/ci/Dockerfile and re-ran create_container.sh, scripts/bin/bazel clean && scripts/bin/bazel test //... and tried the IntelliJ plugin, and everything appeared to continue to work as expected.
My recommendation would be:

If the included JDK is indeed unnecessary, it can be removed -- no need to include it and its dependencies in the builder image!
If the JDK is necessary, it seems likely that java-11-openjdk-headless would suffice. This would avoid a few themes, fonts and Xorg dependencies from being pulled in, which are almost certainly unused.

Thanks for the report - you're right, we probably don't need it since Bazel brings its own JRE anyway.
Want to submit a Gerrit CL to remove it?

Want to submit a Gerrit CL to remove it?

Sure, I'll give that a go!

I've never used Gerrit before. There seems to be an authentication error. Here's what I've done so far.
commit-msg hook: works
I set up the Gerrit commit-msg hook as follows:
f="$(git rev-parse --git-dir)/hooks/commit-msg" && \
  mkdir -p "$(dirname "$f")" && \
  curl -Lo "$f" https://review.monogon.dev/tools/hooks/commit-msg && \
  chmod +x "$f"

This hook works -- I can see that a line with "Change-Id: [...]" was added to my commit message.
git push: fails
I'm following the walkthrough, which says:
$ git commit
[master 3cc9e62] Change to a proper, yeast based pizza dough.
 1 file changed, 10 insertions(+), 5 deletions(-)

$ git push origin HEAD:refs/for/master
[... pushes successfully ...]

origin in the example refers to the Gerrit server, not GitHub. So I set up the Gerrit remote as follows:
$ git remote add gerrit https://review.monogon.dev/monogon

$ git fetch gerrit
[... successful ...]

$ git remote show gerrit
* remote gerrit
  Fetch URL: https://review.monogon.dev/monogon
  Push  URL: https://review.monogon.dev/monogon
  HEAD branch: main
  Remote branch:
    main new (next fetch will store in remotes/gerrit)
  Local ref configured for 'git push':
    main pushes to main (fast-forwardable)

But then when I try to push, there's an authentication step:
$ git push gerrit HEAD:refs/for/master
Username for 'https://review.monogon.dev': [waits for input]

So it looks like I need an account on the Gerrit instance. Is that correct?

Easiest way is to log into Gerrit with a Google account, add your SSH key there and use Git over SSH to push.

Here's a tutorial for Wormhole, which also applies here (except it's a different Gerrit instance): https://github.com/certusone/wormhole/blob/dev.v2/CONTRIBUTING.md#quickstart

Thanks for the pointers! Here's the Gerrit review link: https://review.monogon.dev/c/monogon/+/413

Fixed in https://review.monogon.dev/c/monogon/+/413 / https://github.com/monogon-dev/monogon/commit/070ec4eb5f6853185209494455a9a7b751cf32e1.

Actually I think the Jenkins agent may need it 😅

Actually I think the Jenkins agent may need it sweat_smile

Oops! That may be a new personal record in "minimal number of contributions before the first build breakage".

I'll take a stab at testing the CI part to see if the JRE/headless JDK is good enough.

Sounds good to me. If that's too much hassle, please go ahead and revert my commit if that's the quickest way to fix CI!

https://review.monogon.dev/c/monogon/+/414 should fix it for good

To bring this issue to a close: yes, the JDK is necessary for the Jenkins agent, but the GUI / Xorg dependencies are probably not required, so headless will probably do.


Wire Keyboard: unable to type certain characters in German layout

Wire keyboards with the German layout cannot write '<', '>' and '|'. The Memory output simply stays 0.

no u

you wont fix

XDDD


Add code and config for small players holding other players, and held players holding players

Fixed this in my branch, will push soon


⚠ Acceso a Mercados has degraded performance

In 29b23ec, Acceso a Mercados ($URL_AAM) experienced degraded performance:

HTTP code: 503
Response time: 20986 ms

Resolved: Acceso a Mercados performance has improved in 1c32eb8 after 11 minutes.


Fix #306: Custom elements in page containers

[x] By placing an X in the preceding checkbox, I verify that I am a PayPal employee or
I have signed the Contributor License Agreement

Fix #306
Support custom elements in page object containers
Also unrelated change in parent POM to remove
eclipse lifecycle mapping plugin.

Is there a test that can be added to ensure the bug doesn't resurface?

merged this to 1.2.0


softether: 4.38 -> 4.41

Description of changes
Update softether package to latest stable release.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Will take a look at this soon.

Could you also get rid of the with lib; in the meta attr please ?

I think you forgot this.
Also, in src, use rev = "refs/tags/${version}"; to prevent branch/tag mismatch.

Also should get rid of rec and use finalAttrs.

I think you forgot this.

yes sorry issed that

Could you also get rid of the with lib; in the meta attr please ?

well now the build fails ... I can use the full path for e.g. maintainers ...

It looks like there is an issue with your change though.
They are reported to not be parseable.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 343981

x86_64-linux

:white_check_mark: 1 package built:

softether

aarch64-linux

x86_64-darwin

aarch64-darwin

:point_up: :heavy_check_mark:  that gif


Change the "got it tooltip" location

The "got it" tooltip crashed on Linux (but, the link opened). It seems to be an error because the tooltip is attached to the file. The point is that it should not be attached to the file (even if it works).

[ ] fixed

test or close


Lists (tables)

I do not want to say tables but kinda like tables.

Will help and submit own PR's and should be labeled as feature/enhancement.

Looks great.


Use CompletableFuture from package for lower API level support

According to CompletableFuture CompletableFuture  is Added in [API level 24]
If app imported this  package build with minSdkVersion less than [API level 24], app would run into this error
Fatal Exception: java.lang.NoClassDefFoundError
Failed resolution of: Ljava/util/concurrent/CompletableFuture;

Caused by java.lang.ClassNotFoundException
Didn't find class "java.util.concurrent.CompletableFuture"

So this PR is needed to support lower API level, which referred from the flutter official plugin
https://github.com/flutter/plugins/commit/0bd38d2a2cf367e2418334bd32d362c1880704be

@alexmercerind

According to CompletableFuture CompletableFuture is Added in [API level 24]
If app imported this package build with minSdkVersion less than [API level 24], app would run into this error
Fatal Exception: java.lang.NoClassDefFoundError
Failed resolution of: Ljava/util/concurrent/CompletableFuture;

Caused by java.lang.ClassNotFoundException
Didn't find class "java.util.concurrent.CompletableFuture"

So this PR is needed to support lower API level, which referred from the flutter official plugin flutter/plugins@0bd38d2

I have merged the PR, I'm not exactly a Java or Android developer. I know more about Windows or Linux plugins.
Thanks for the contribution.


NER and RE only

Hi,
I want to experiment with this architecture for NER and RE only. What changes do I need to make?

Hi @Mahmedturk ,
If you want to experiment with only two tasks (NER and RE), the fastest way (and the easiest way) would be to modify the training configuration file (https://github.com/huggingface/hmtl/blob/master/configs/emd_relation.json) and make sure that the EMD task is replaced by NER.
Victor

Closing it since there doesn't seem to be any activity.
Feel free to re-open it if necessary!
Victor

Hi there - I'm trying to do the same thing and so followed your advice. I also changed the model type to ner_relation but hit the following error "ner_relation not in acceptable choices for model.type: ['bcn', 'constituency_parser', 'biaffine_parser', 'coref', 'crf_tagger', 'decomposable_attention', 'event2mind', 'simple_seq2seq', 'bidaf', 'bidaf-ensemble', 'dialog_qa', 'nlvr_coverage_parser', 'nlvr_direct_parser', 'quarel_parser', 'wikitables_mml_parser', 'wikitables_erm_parser', 'atis_parser', 'srl', 'simple_tagger', 'esim', 'bimpm', 'graph_parser', 'relation_extractor', 'ner', 'relation', 'coref_custom', 'ner_emd', 'emd_relation', 'emd_coref', 'ner_emd_coref', 'ner_emd_relation', 'hmtl']" - I've just kicked off my training again using "emd_relation" in the hope that magically works, but if not do you have any suggestions?

Hey @syramarshall
Yes, you should keep "type": "emd_relation" in the training config file. It just means that the EMD layer will be composed of a BiLSTM+CRF, but it is agnostic to whether you are actually doing MD with this layer. So just change the data in data_params for your NER dataset and the other model hyper-parameters if you need.
Victor