changed CHICKEN_REPOSITORY_PATH to $HOME/lib/chicken/11

so that the user won't have trouble installing packages with chicken-install ( the permission denied error)

Motivation for this change
I keep getting permission denied when trying to install eggs with chicken-install
Things done
changed the CHICKEN_REPOSITORY_PATH to  user directory

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

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

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

Notify maintainers
cc @

Is this still relevant? Because I dont understand the change request here. that setup hook is always being run in by the nix-daemon and not by the user.

closing for now. Feel free to reopen if this is still relevant.


Export number of classes to Torch models

Allow networks to select desired number of outputs depending on
number of classes in (classification) dataset.
Stop automatically adjusting number of outputs in Lua wrappers.
This is less confusing and this will make it easier to enable
multi-GPU training with Torch in DIGITS.

Great idea! Works for me. I wonder if we could do something similar with Caffe? It'd have to be something that gets replaced before the protobuf is parsed, since you can't have variables in protobuf.
There's a merge conflict - let's address that and then I think this is good to to.

Thanks Luke, I have rebased my patch on the tip of the master branch.


Level 2 is done

This is basically as done as level 2 can be. Also fixed some music related bugs.

Closes #65


[Security Plugin] Update tenant header identifiers

Description
Follow up need since the copy on these pages was modified in https://github.com/opensearch-project/security-dashboards-plugin/pull/1881
Issues Resolved

Resolves https://github.com/opensearch-project/security-dashboards-plugin/issues/1893

Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@derek-ho Can you double check?

@ruanyl can you help me get this merged, it should fix our broken 2.14 integration tests.

@CCongWang I'm not sure what is happening with this failure, are the gradle checks broken, can you help me troubleshoot?
 FATAL  Error: Cannot find module '../../public/utils/constants'
Require stack:
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/plugins/notificationsDashboards/server/routes/configRoutes.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/plugins/notificationsDashboards/server/routes/index.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/plugins/notificationsDashboards/server/plugin.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/plugins/notificationsDashboards/server/index.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/src/core/server/plugins/plugin.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/src/core/server/plugins/discovery/plugins_discovery.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/src/core/server/plugins/discovery/index.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/src/core/server/plugins/plugins_service.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/src/core/server/plugins/index.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/src/core/server/index.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/src/cli/serve/serve.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/src/cli/cli.js
- /home/runner/work/opensearch-dashboards-functional-test/opensearch-dashboards-functional-test/opensearch-dashboards-3.0.0/src/cli/dist.js

@peternied A known issue by changes from dashboards-notification and should have been solved.


OpenVPN: added block-outside-dns - ref. issue #4422

https://github.com/opnsense/core/issues/4422#issuecomment-730555859

@juliocbc looks good at a first glance, will test drive and pull it in later

@juliocbc thanks! added https://github.com/opnsense/core/commit/19a39dc8b3b2c9cbab86da00fb02a33bfe025af0 to hide values when not applicable (not a fix for your commit, just something I noticed while pulling it in)


Incomplete python wrapping of enum class objects

Description
In the most recent set of failing ITKExamples tests, #130, the listed tests are unsolvable unless changed from the source. In the latest release of the ITKPythonPackage 5.1b.1 errors occur when attempting to access python wrapped enumerations.
The issue arises because some enumerations are no longer wrapped after the
#1061 pull request (In ITK). In #1061, they were changed from 'c' style enumerations to enum classes. During this transition a number of new enums were moved outside their original classes.
Steps to Reproduce

Clone the latest version of ITKExamples from Github:

git clone -b master https://github.com/InsightSoftwareConsortium/ITKExamples.git

Make build directory:  mkdir ITKExamples-Superbuild

Allocate to build directory:  cd ITKExamples-Superbuild

Run:  cmake ../ITKExamples/Superbuild

Configure CMake with:  ITKExamples_USE_WRAP_PYTHON = ON

Run: make

Allocate to ITKEx-build directory: cd ITKEx-build

Test: ctest -V

Expected behavior
100% Tests passed.
Actual behavior
98% tests passed, 12 tests failed out of 589
248 - ApplyAColormapToAnImageTestBaselineComparisonPython (Failed)
252 - ApplyAColormapToAnImageTestPython (Failed)
568 - SegmentWithWatershedImageFilterTest01BaselineComparisonPython (Failed)
570 - SegmentWithWatershedImageFilterTest02BaselineComparisonPython (Failed)
572 - SegmentWithWatershedImageFilterTest03BaselineComparisonPython (Failed)
574 - SegmentWithWatershedImageFilterTest04BaselineComparisonPython (Failed)
576 - SegmentWithWatershedImageFilterTest05BaselineComparisonPython (Failed)
583 - SegmentWithWatershedImageFilterTest01Python (Failed)
584 - SegmentWithWatershedImageFilterTest02Python (Failed)
585 - SegmentWithWatershedImageFilterTest03Python (Failed)
586 - SegmentWithWatershedImageFilterTest04Python (Failed)
587 - SegmentWithWatershedImageFilterTest05Python (Failed)

Reproducibility
The tests fail 100% of the time when the ITKPythonPackage 5.1b.1 is used.
When ITKPythonPackage 5.0.1 is used, 100% of tests pass.
Versions and Environment

Done on macOS

Python 3.7.3

Latest ITKExamples, ITK, versions pulled from Github.

ITKPythonPackage version 5.1b.1

Possible Solution
Lets examine an enumeration not wrapped: RGBColormapFilterEnumType
Path: /ITK/Modules/Filtering/Colormap/include/itkScalarToRGBColormapImageFilter.h
In the previous instance of this class, the class contained an instance variable that was a c-style enum. After being changed to a enum class, the new enum was placed outside the class (Line: 30). We can see the difference of the 2 classes when built as a python package.
ITKPythonPackage 5.0.1 (older)
Download the python package separate of your ITKExamples-Superbuild.
Allocate to: itk/itkScalarToRGBColormapImageFilterPython.py
As you can see on lines: 280-293, the enum values are specifically stated:
(280)Red=_itkScalarToRGBColormapImageFilterPython.itkScalarToRGBColormapImageFilterIF2IRGBAUC2_Red
ITKPythonPackage 5.b1.1 (new)
Allocate to ITKExamples-Superbuild/itkpython/lib/python3.7/site-packages/itk/itkScalarToRGBColormapImageFilterPython.py
When inspecting the file, NO enum values are set, however it could be located outside the class .py file since it was moved out in the Source ITK.
When running the following steps to see if RGBColormapFilterEnumType is an attribute of itk we can see NO attribute RGBColormapFilterEnumType is listed.
Steps:

Enter in terminal:  python (opens up python interface)

Enter:  import itk

Enter: dir(itk)

The output will be a list of all attributes and methods for itk. As expected RGBColormapFilterEnumType is not listed, hence not wrapped.
Reason for not being wrapped
This is a bit past my understanding of the full architecture surrounding itk, but in the .py file: ITK/Wrapping/Generators/SwigInterface/igenerator.py we have a list of the enumerations to be wrapped.
On Line: 53, we see a list of enumerations set not to be wrapped and as expected RGBColormapFilterEnumType and many others are listed. I suspect modifying this file to begin wrapping enumerations listed from lines 90-98 will solve the issue. With these changes, users will be able to use the latest ITKPythonPackage with all of its functionality.  Please let me know of any questions as this is a larger issue with many components.
@hjmjohnson

@mseng10 well done on the detailed and well-researched description!
Until we have the enum's wrapping in ITK's 5.1 Python wrapping, I propose we do not install the pre-release Python packages in the Superbuild here:
https://github.com/InsightSoftwareConsortium/ITKExamples/pull/146

@thewtex  Did # 1353 address this completely?

#1353 addressed the enum issues. We can get the tests passing after 5.1b2 binaries are published.

Fixed in #156. Closing now.


2.3.x release breaks Chef 11.x clients

We're seeing failures on chef 11.x clients when using build-essential 2.3.x.  This was unexpected, as we thought that 2.x.y releases would maintain compatibility with chef-client 11.x.
       ================================================================================
       Recipe Compile Error in /tmp/kitchen/cache/cookbooks/build-essential/providers/msys_archive.rb
       ================================================================================

       NoMethodError
       -------------
       undefined method `provides' for #<Class:0x000000027792f8>

       Cookbook Trace:
       ---------------
         /tmp/kitchen/cache/cookbooks/build-essential/providers/msys_archive.rb:22:in `class_from_file'

Fixed in 2.3.1.  My mistake - thanks @tas50!


perf: merge several style syntax linters

Does this improve performance? To be determined!

Opening just for benchmarking purposes. Would need clean-up before it can be merged.

Benchmarking this, once CI is done, would be welcome.

!bench

I don't know what happened to the bench summary bot: locally, it prints

File
Instructions
%

build
-253.872⬝10⁹
(-0.17%)

Mathlib.Tactic.Linter.Style
+1.584⬝10⁹
(+23.57%)

!bench


Integrate Parker into the build process

i.e. https://github.com/katiefenn/parker
By diffing a PR's CSS stats against the previous stats, we can get a warning if something causes the number of entities to explode.
See also: mdo's jQuery SF talk where he mentions similar tooling and graphs at GitHub
https://speakerdeck.com/mdo/at-mdo-ular-css?slide=41
https://speakerdeck.com/mdo/at-mdo-ular-css?slide=42

Or perhaps whatever generates http://www.basscss.com/stats/

CSSStats (from mdo's jQuery SF talk) is interesting. The report for Bootstrap is here.


Create Campaign Preview should match published page

I've turned the HTML in the publish page into a partial view, which I'm using in both the published page and the admin preview tab.
Resolves #2213

@dracan This looks like a good change, but there are conflicts. Are you able to resolve so that we can merge?

@MisterJames Yep, but probably not for a day or so - got a shed load of stuff to do unfortunately. It's on my list though - so I won't forget ;)

@MisterJames Done :)


"path" regex doesn't accept strings on Python 3

Using a string for the path regex argument to route() under Python leads to the following exception:
Traceback (most recent call last):
  File "/Users/ulo/.pythonz/pythons/CPython-3.4.3/lib/python3.4/threading.py", line 920, in _bootstrap_inner
    self.run()
  File "/Users/ulo/Envs/dali/lib/python3.4/site-packages/channels/management/commands/runserver.py", line 152, in run
    worker.run()
  File "/Users/ulo/Envs/dali/lib/python3.4/site-packages/channels/worker.py", line 69, in run
    match = self.channel_layer.router.match(message)
  File "/Users/ulo/Envs/dali/lib/python3.4/site-packages/channels/routing.py", line 50, in match
    match = route.match(message)
  File "/Users/ulo/Envs/dali/lib/python3.4/site-packages/channels/routing.py", line 133, in match
    match = re.match(value, message[name])
  File "/Users/ulo/Envs/dali/lib/python3.4/re.py", line 160, in match
    return _compile(pattern, flags).match(string)
TypeError: can't use a string pattern on a bytes-like object

Changing the value of path to bytes appears to fix the problem.
Edit: This also leads to any captured keyword arguments becoming bytes as well.
Versions used:

Python 3.4.3
django 1.9.4
channels 0.10.0
asgi-redis 0.9.0

Fix for this is now released in channels 0.10.1.


Unable to loop set method for RecycledLoader

Does anyone know why I am unable to loop through the setting of my RecycledLoader?
From what I understand from the test output, it appears that my code seems to always forgo the if condition and go straight to the else condition.  When I print out the list of loaders to check, the existing RecycledLoaders dont seem to be modified while everything else is fine. I have faced no issues with the generating of new RecycledLoaders if required. Is there anything I am missing out on? Does it have to do with the way my RecycledLoader class is implemented?
A snippet of my code is as follows, the only difference between this and level 5 is the addition of an if condition to decide whether to set a new RecycledLoader or Loader:

My test output:

Line 13 of your output gives “Loader 3” instead of “Recycled Loader 3”. When more Loaders are needed, are you creating them all as Loader objects? I think you can directly create a RecycledLoader object when a new Loader is needed and when its identifier is divisible by 3.

Line 13 of your output gives “Loader 3” instead of “Recycled Loader 3”. When more Loaders are needed, are you creating them all as Loader objects? I think you can directly create a RecycledLoader object when a new Loader is needed and when its identifier is divisible by 3.

Hi, to clarify my question, I am able to create the new RecycledLoader Object with the correct conditions. However, when it comes to using a pre-existing RecycledLoader that is already created (i.e. there are enough Loaders/RecycledLoaders already in my loaders ImList), I am unable to set it to its new Cruise.

Hmm I think your if-else code is ok… Did you override the serve method in RecycledLoader such that it returns a RecycledLoader instead of a Loader?

Hmm I think your if-else code is ok… Did you override the serve method in RecycledLoader such that it returns a RecycledLoader instead of a Loader?

This is the problem I faced during my coding too, need to override the serve method to make it return RecycledLoader else it will convert the type back to Loader

Hi, based on the snippet of code, could there be a possibility where you're printing before the set method is called, hence resulting in the old object being printed out first?

Hi, I faced this issue too! You got to overwrite ur serve method in the recycledloader class else the java will return loader instead based on the code tracing

Hi there, did you @Override your serve() function in the RecycledLoader class?


make cluster-internal non-tls setup work, with proper token authentication

removed http2 - we use https://github.com/teamhephy/registry-proxy to get around the "insecure registry" stuff (registries running on localhost don't need TLS), and somehow nothing works when listening as http2

we need to set REGISTRY_AUTH_TOKEN_REALM to the actual node IP since the docker running on the nodes can't resolve the k8s DNS name

normally, this helm chart uses the same certificate for HTTPS (to the outside world) and for token authentication (only between the actual registry and portus).
However, token auth itself actually also works with self-signed certs. So this splits it up into portus.tls.enabled and portus.tokenauth.enabled. Now we can have proper authentication for the registry without needing to setup TLS.

@rwos Wonder if we should use an ingress for Traefik and then expose it (via MetalLB), then we get SSL for free?

@till yeah but this is still needed then, because one has to manually enter the cert in the values.yaml otherwise


Updated build.xml to include a RUN target

Thanks for the PR.  Currently our build/run setup is a weird mix of command shell and Ant.  The run stuff is all currently in the refine / refine.bat scripts.  Is there a specific need to move this piece from the shell scripts to the Ant file?  Note also, that the shell script does a bunch of additional processing.
While I'm not opposed to merging this, I'd prefer to make a single concerted cleanup pass through everything (which might also mean switching to Maven instead of Ant).
As an aside, it's preferable to have the PR on it's own branch (e.g. thadguidry-patch-1) as opposed to master, so that it doesn't accidentally get polluted with later changes that are unrelated.

The specific reason was getting the Netbeans environment playing nicely.  I have some other PRs coming for that later.  Good point, I'll create my own branch and then ask for a PR there. Thanks Tom! and welcome back ! :)

@tfmorris @jackyq2015 @magdmartin @MattBlissett
OK, I am now working on setting up our new Gradle build.  (and with the Gradle Wrapper, so you won't even have to install Gradle beforehand !)  I'm about halfway done with the new organizational changes, but from a high level it will be treated as a Gradle multi-project build - A multi-project build in gradle consists of one root project, and one or more subprojects that may also have subprojects.:

/openrefine

/client

build.gradle

/server

build.gradle

/shared

build.gradle
settings.gradle

I will maintain it.
This NEW build way will be pushed into my repo later for eveyone to review once I am done, Then later we can clone it under openrefine/gradle_build at some point in the future once everyone is happy. (this way I don't slow anyone down,and we get an easier build and even easier distributions !! in the future)
(@tfmorris - And, we will not even have to lose the Eclipse .project settings etc. apply plugin: 'eclipse'
I looked at the current state of affairs with Maven and SBT and I still was not pleased enough - most of the major players are now going Gradle and for good reason, its also defacto standard added now in Eclipse Mars !! - no plugin needed.  We also use it at Ericsson)

I don't need this anymore.


Monolog handler

initial implementation of a logs bridge api for opentelemetry logging

Would it make sense to record context, extra and channel as separate attributes to prevent overriding data?

I've put them all into their own attributes (and set the default formatter to NormalizerFormatter so that exceptions can be logged). I think let's get a 0.0.1 release out, and see whether the log record structure is useful when people actually use it.


chore: Add stale bot (box/box-codegen#600)

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

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

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
10560

Relevant Lines:
11428

💛  - Coveralls


gnome3.libchamplain: 0.12.19 -> 0.12.20

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libchamplain/versions.
meta.description for gnome3.libchamplain is: '"C library providing a ClutterActor to display maps"'.
meta.homepage for gnome3.libchamplain is: '"https://wiki.gnome.org/Projects/libchamplain"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

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

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

found 0.12.20 with grep in /nix/store/s8sxic2vab2cw0ngi8jmgqbqi9s548hc-libchamplain-0.12.20

directory tree listing: https://gist.github.com/d8c7fa40164fb1b2db7f202d68fa8001

du listing: https://gist.github.com/afd63e7200205bedbc072327ae861d37

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

45 total rebuild path(s)
15 package rebuild(s)
15 x86_64-linux rebuild(s)
15 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
15 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
empathy
geeqie
gnome3.empathy
gnome3.gnome-contacts
gnome3.gnome-maps
gnome3.libchamplain
gnome3.shotwell
gpx-viewer
gthumb
libchamplain
pantheon.elementary-calendar
pantheon.elementary-greeter
pantheon.wingpanel-indicator-datetime
pantheon.wingpanel-with-indicators
shotwell

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s8sxic2vab2cw0ngi8jmgqbqi9s548hc-libchamplain-0.12.20 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A gnome3.libchamplain https://github.com/r-ryantm/nixpkgs/archive/09c0a8518b875ce182dd6d4a313984b299ea7e66.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s8sxic2vab2cw0ngi8jmgqbqi9s548hc-libchamplain-0.12.20
ls -la /nix/store/s8sxic2vab2cw0ngi8jmgqbqi9s548hc-libchamplain-0.12.20/bin

cc @lethalman @jtojnar @hedning @worldofpeace for testing.

$ ~/Projects/what-changed/changelogs.py libchamplain 0.12.19 0.12.20
libchamplain 0.12.20 (2019-10-28)
=================================

Stable release

Changes:

* Remove autotools-based build and all the related files (Jiří Techet)
* Fix gtk-doc module name (Mart Raudsepp)
* Add missing gdk required dependency (Mart Raudsepp)
* Make the build reproducible (Chris Lamb)

Tested GNOME Maps.


Tutorial: changes to the schedule

Just gathering up the changes I proposed to the schedule. The original comments are here and below. These have been updated in light of the big meeting we had after I posted those comments.

Give Cider 5 mins.

Right now we have

10am-10.05am | MrXL: A map-reduce frontend
10.05am-11am | Compiling MrXL to Calyx

And I think we could instead have

10am-10.15am | MrXL: A map-reduce frontend
10.15am-11am | Adding map to MrXL.

That is, in the first 15 mins, we'll walk them through the tutorial as written. We will gloss over the parts where the tutorial shows how to compile MrXL using the eDSL, since that's really best for working through yourself. Indeed, they'll spend 45 mins adding map to the DSL.

Speaking of later on, I propose that we change

11.25am-12.25pm | Hardware Performance 101 / MrXL competition

into three tracks.

Reduction trees, for those interested in speedup.
Arbitration and multi-banking, for those interested in that.
Office hours, for those just wanting to backfill about what the heck is going on.

@rachitnigam if this is okay, I can make the changes. I can also move fud up, if that's of interest.

I think moving fud up might be good since folks will have just finished the language tutorial? Maybe we swap Pollen and fud?

 Frontloading all the not-MrXL stuff is awesome and gives folks contextual continuity for independent work time. Totally in favor :D

One issue is that it violates one of our original principles: we now talk at them for over an hour before they get cracking. If we wanted to soothe that, I suppose we could move Cider (and even fud) to the end of the day, just before closing.

@rachitnigam, now that we are leaning on the Docker a good bit more, do you think you could squeeze the "setting up" stuff down by a few minutes and yield that time to map implementation?

So, the reason "setting up" is that long is because it might take people a while to download the docker image and we would need to help people who run into problem.

Ah yes, both very fair! I've adjusted the proposed schedule above with these changes, see what you think?

LGTM! Merge the changes on the website!


ExtractClaims Helper Method to aide in access JWT claims

I initially went back and forth on how to implement this:
func ExtractClaimsMethod1(request *rest.Request) (map[string]interface{}, error) {
  if request.Env["JWT_PAYLOAD"] == nil {
    return nil, fmt.Errorf("No JWT_PLAYLOAD")
  }
  jwt_claims := request.Env["JWT_PAYLOAD"].(map[string]interface{})
  return jwt_claims, nil
}
func ExtractClaimsMethod2(request *rest.Request) map[string]interface{} {
  if request.Env["JWT_PAYLOAD"] == nil {
    empty_claims := make(map[string]interface{})
    return empty_claims
  }
  jwt_claims := request.Env["JWT_PAYLOAD"].(map[string]interface{})
  return jwt_claims
}

In the end I settled on a combo of the two (as seen in the PR) which returns an
empty map[string]interface{} that allows you to ignore the returned err unless
you really need to know that the claims were not set.
If you don't need to know that the claims were not set that allows a more
"every day" type of approach to be taken:
endpoint := func(w rest.ResponseWriter, r *rest.Request) {
  claims, _ := jwt.ExtractClaims(r)
  if claims["isAdmin"]==true {
    log.Println("You sir, are an admin")
  }
  w.WriteJson(...)
}

I also updated the TestClaimsDuringAuthorization test submitted in #4 to use
the new ExtractClaims method to provide  some debugging info if the test
fails due to missing claims.  Notice the first line of output.
$ go test
2015/05/20 13:08:32 JWT Claims Extraction failed during Authorization: No JWT_PLAYLOAD
--- FAIL: TestClaimsDuringAuthorization (0.00s)
        util.go:41: Code 200 expected, got: 401
FAIL
exit status 1
FAIL    github.com/StephanDollberg/go-json-rest-middleware-jwt  0.009s

Thanks for the quick follow up.
I am wondering whether we need the error handling at all as request.Env["JWT_PAYLOAD"] actually will never be nil.
The only case I can imagine is if the user would somehow try to access it in an earlier middleware which I would claim to be a logic error that should crash as early  as possible.
Cheers and Thanks,
Stephan

That does seem to be the case right now; however it is not hurting anything.  If we leave it in then we can make the future gaurentee

That does seem to be the case right now; however it is not hurting anything. If we leave it in then we can make the future guarantee that any calls to ExtractClaims will be panic free, whereas we can not make that guarantee right now because of the type assertion.

It hurts in the sense that it produces an extra unneeded return value.
ExtractClaims is panic free. If the type assertion fails we use the jwt-go in a wrong way and have to fix it.

Sorry for the delay... real life and junk.
PR Updated, ExtractClaims no longer returns an error object

No problem, I am also quite busy these days. Will have a look at it sometime next week.

Merged. Thanks again!


[nextcloud] Redis server went away

Hi,
if have upgraded my docker container to 11.0 and I'm not able to log into my nextcloud account. I searched the log and I found an exception which told me that the "Redis server went away".
I'm not very into Redis and I did't get a clue whats going wrong.
Ich attached the log file because the output is to big to post it within the post.
nextcloud.log.txt
I hope you can help me with this.

https://github.com/Wonderfall/dockerfiles/issues/128#issuecomment-281509540
I like helping but try to search a bit more next time :)


[MM-20903] Category collapsing and unread filter tests

Summary
Adds E2E tests that test the functionality of collapsing categories, and the unread filter of the new experimental sidebar.
NOTE: Make sure you are running the latest master server code, which contains the code necessary to enable the experimental sidebar.
Ticket Link
https://mattermost.atlassian.net/browse/MM-20903

/update-branch


Backend Integration unable to upgrade since v2.4.4

Describe the bug
Objects do not have the same structure after v2.4.4. I spent some time reviewing the relevant commit log, and recently tried with the latest version (v2.9.9)  and the result is consistent.
Thank you for creating the stackblitz setup - I am able to reproduce the issue here as it appears on my machine.

<= v.2.4.4 (working setup) - https://stackblitz.com/edit/vitejs-vite-knwzlj
v2.5.0+ - https://stackblitz.com/edit/vitejs-vite-gy51da

After reviewing the code to see what it does, please open a browser console and click the link. You will see two different logs, which I believe should be identical. Any guidance for backend integration changes to make would be greatly appreciated.
Thank you all for your work on this delightful project.
Reproduction
https://stackblitz.com/edit/vitejs-vite-gy51da
System Info
System:
    OS: macOS 11.6.5
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
    Memory: 57.79 MB / 16.00 GB
    Shell: 3.4.1 - /usr/local/bin/fish
  Binaries:
    Node: 14.18.1 - ~/.volta/tools/image/node/14.18.1/bin/node
    npm: 7.24.2 - ~/.volta/tools/image/npm/7.24.2/bin/npm
  Browsers:
    Chrome: 101.0.4951.64
    Firefox: 100.0.2
    Safari: 15.4
  npmPackages:
    vite: 2.4.4 => 2.4.4

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This change (#4279) is affecting it.
Set useDefineForClassFields in tsconfig.json to false to get the previous behavior.
Or change HelloController to
class HelloController extends Controller {
  static targets = ['header'];

  // headerTarget!: HTMLElement;
  declare headerTarget: HTMLElement;

  handleClick(evt: MouseEvent) {
    evt.preventDefault();
    console.log('clicked', this.headerTarget);
  }
}

@sapphi-red thank you for reviewing this. I will update the tsconfig.json. What potential impacts would setting this value to false have as it relates to the usage of Vite for future releases as you understand it today?

From typescript 4.3.2, the default value of useDefineForClassFields is true and some libraries may expect true.
https://vitejs.dev/guide/features.html#typescript-compiler-options


draw_cable meander feature

Hi,
I'm missing one feature in draw_cable(). I'd like to be able to adjust the length of the segments as indicated in the picture below.

If you use '-1' as the number of meanders to put the maximal number of meanders (depending on fillet radius) then you cannot do so. You can trick the code by indicating a given number of meanders. Then, the dimension you are looking for can be computing accordingly.


agent: support debug console

Start a debug console if the agent.debug_console option is part of the kernel
command line and a shell (bash or sh) is installed in the rootfs.
The debug console is useful when the developer wants to debug the agent process
or any other processes (even containers). The current way to do this is by
modifying a systemd target + creating a new system unit, the problem with this
approach is that it's not user-friendly and we depend on systemd, since it must
be installed in the rootfs.
fixes #546
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

/test

cc @clarklee92 @jodh-intel @sboeuf @jcvenegas

https://github.com/kata-containers/osbuilder/issues/96 is already open for one avenue to test ;)

@grahamwhaley @jodh-intel changes applied, thanks

/test

/test

/test

@devimc - guess we should check why 18.04-initrd and sles CIs failed?

@grahamwhaley
intird
+ sudo docker run --runtime=kata-runtime -d --name kata-VDABum busybox tail -f /dev/null
5f4b7c55efe19567b657992359855cae5bb1c72197f25f73085c3472120bb7fa
docker: Error response from daemon: OCI runtime create failed: rpc error: code = Unavailable desc = transport is closing: unknown.
Makefile:126: recipe for target 'vm-factory' failed
make: *** [vm-factory] Error 125
Build step 'Execute shell' marked build as failure
Performing Post build task...
Match found for :.* : True

sles
• Failure [30.093 seconds]
[Serial Test] Update number of CPUs
/tmp/jenkins/workspace/kata-containers-agent-sles-12-PR/go/src/github.com/kata-containers/tests/integration/docker/cpu_test.go:228
  Update CPU constraint
  /tmp/jenkins/workspace/kata-containers-agent-sles-12-PR/go/src/github.com/kata-containers/tests/vendor/github.com/onsi/ginkgo/extensions/table/table.go:92
    should have 2 CPUs [It]
    /tmp/jenkins/workspace/kata-containers-agent-sles-12-PR/go/src/github.com/kata-containers/tests/vendor/github.com/onsi/ginkgo/extensions/table/table_entry.go:46

    Expected
        <int>: 1
    to be zero-valued

    /tmp/jenkins/workspace/kata-containers-agent-sles-12-PR/go/src/github.com/kata-containers/tests/integration/docker/cpu_test.go:299
------------------------------
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS

Summarizing 1 Failure:

[Fail] [Serial Test] Update number of CPUs Update CPU constraint [It] should have 2 CPUs

nothing related to this pr, restarting CIs


Error on require on react-native 0.16.0-rc

I've just upgraded react-native to 0.16.0-rc and now the barcode scanner is throwing an error on require:
'BarcodeScannerView' has no propType for native prop 'RNBarcodeScannerView.renderToHardwareTextureAndroid' of native type 'boolean'.

+1

fixed it by replacing the last part of BarcodeScanner.js to
BarcodeScannerView.propTypes = {
viewFinderBackgroundColor: PropTypes.string,
viewFinderBorderColor: PropTypes.string,
viewFinderBorderWidth: PropTypes.number,
viewFinderBorderLength: PropTypes.number,
viewFinderDrawLaser: PropTypes.bool,
viewFinderLaserColor: PropTypes.string,
torchMode: PropTypes.string,
cameraType: PropTypes.string,
onBarCodeRead: PropTypes.func,
rotation: PropTypes.number,
scaleX: PropTypes.number,
scaleY: PropTypes.number,
translateX: PropTypes.number,
translateY: PropTypes.number,
importantForAccessibility: PropTypes.string,
accessibilityLabel: PropTypes.string,
testID: PropTypes.string,
renderToHardwareTextureAndroid: PropTypes.string,
onLayout: PropTypes.bool
};
var RNBarcodeScannerView = requireNativeComponent('RNBarcodeScannerView', BarcodeScannerView, {
nativeOnly: {
onChange: true,
accessibilityLiveRegion: 'none',
accessibilityComponentType: 'button'
}
});

@NorthFoxz Thank you so much! Also I can confirm that this bug is still present with the latest 0.16.0

Same error on react-native  0.17.0

@NorthFoxz Your code works, thanks. :+1:

I can confirm that @NorthFoxz solution fixes the issue on React Native 0.17 it would be great to get this fixed in master.

With @cpunions's merged pull request the issue should now be fixed.


Not an Issue

Just an idea to run on raspberry, and utilize "remote control" feature from app to be able to add no go zones and whatnot to lower end models or others with no root avail like the new e4. Raspberry is running valecloudo, and interfacing with vacuum through remote control feature, basically creating a route with a previous map (i guess) and then it it follows route turn for turn with "remote control"
Does that make sense?

Sorry i didnt read the first line apparently (stand alone binary)  what I want is to emulate the vacuum with the pi and automate mi home app remote controle feature

Sorry didn't rtfm


uv_sync_flags and tox 4.21 native toml configuration

I tested passing groups to uv_sync_flags with this configuration (with the new tox 4.21 native toml configuration support), as suggested in #113, but without success at the moment:
[tool.tox.env.doc]
runner = "uv-venv-lock-runner"
uv_sync_flags = ["--group=doc"]
commands = [
    ["sphinx-build", "--builder", "html", "doc", "build/sphinx/html"],
    ["sphinx-build", "--builder", "man", "doc", "build/sphinx/html"],
]

It fails with TypeError: '--group=doc' is not list.
When passing a simple string like uv_sync_flags = "--group=doc" it also fails with the same error message.
Originally posted by @azmeuk in https://github.com/tox-dev/tox-uv/discussions/114#discussioncomment-11084270

https://github.com/tox-dev/tox-uv/releases/tag/1.16.0 also now have support for dedicated field https://github.com/tox-dev/tox-uv#dependency_groups to be more unified with upstream tox

Thank you for the quick fix!


Add acquisition function penalties

Codecov Report

Merging #31 (b5f299a) into main (5d19003) will increase coverage by 0.01%.
Report is 2 commits behind head on main.
The diff coverage is 96.46%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
+ Coverage   94.25%   94.27%   +0.01%
==========================================
  Files          36       37       +1
  Lines        2734     2813      +79
==========================================
+ Hits         2577     2652      +75
- Misses        157      161       +4

Flag
Coverage Δ

unittests
94.27% <96.46%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

gpax/acquisition/penalties.py
92.59% <92.59%> (ø)

tests/test_acq.py
98.27% <97.14%> (-1.73%)
:arrow_down:

gpax/acquisition/acquisition.py
68.86% <100.00%> (-0.51%)
:arrow_down:

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


extract_levels does not preserve bounds when using interpolation

Preprocessor function extract_levels does not preserve bounds when using interpolation. We could add the bounds in some cases, e.g. when the source of the pressure levels is another dataset we can simply copy the bounds from there. Also for the CMIP6 CMOR coordinates bounds are available.

Doesn't this belong to ESMValCore?


MMM-ImageSlideshow not all images are loaded

Good Day all 🙂 ,
Does anyone having issues with the MMM-ImageSlideshow showing images from NW drive. I have MM setup on a Raspberry PI3 and the following MMM-ImageSlideshow module configuration.
        {
        module: 'MMM-ImageSlideshow',
        position: 'bottom_center',
        config: {
        delayUntilRestart: 6000,
        randomizeImageOrder: true,
        imagePaths: ['modules/MMM-ImageSlideshow/exampleImages/shared_media'],
        treatAllPathsAsOne: true,
        fixedImageWidth: 600,
        fixedImageHeight: 800,
        slideshowSpeed: 60000
                }
        },

The issue that I have is that the Module does not load all images within this shared_media folder. When I check the Browser Console the error is 404 and it is not finding the image even though it is there. I can also see that it is adding an ‘_’ infront of some images hence not finding them in the folder and not sure how this can be fixed.
Console error:
GET
http://<IP_ADDRESS>:8080/modules/MMM-ImageSlideshow/exampleImages/shared_media/.__f1726418696.jpg
Status
404
Not Found
VersionHTTP/1.1
Transferred701 B (211 B size)
Referrer Policyno-referrerg
Thank you in advance,

Hi, compliments of the season.
Just a fellow user on what's a quiet part of Github..
I saw in this thread a comment on 'local' folders
https://github.com/AdamMoses-GitHub/MMM-ImageSlideshow/issues/1
The comment made was:
" the code DEPENDS on the path being part of the MM folder tree.."

Hi, compliments of the season.
Just a fellow user on what's a quiet part of Github..
I saw in this thread a comment on 'local' folders
https://github.com/AdamMoses-GitHub/MMM-ImageSlideshow/issues/1
The comment made was:
" the code DEPENDS on the path being part of the MM folder tree.."

Hey, Thank you for the feedback, but the Shared folder is within the MMM module folder tree and it is still not showing all images. This is quite frustrating problem :)

Hi,
did you catch the point that there is a limit on the number of images that can be displayed ~ 2000?
see #6
From what I recall its as the code reads in the directory listing and allocates memory space. I have not had the opportunity to dig further into the specifics.

Hi again, yes I saw that but i have separated all my photos in subfolders with 100 items and it is still not showing all of those ....


Changing jshint tool and created a rake task to wrap tests

The task was to create a rake task and wrap jasmine and jshint tests, but I figure out the older jshint wasn't checking all important things, also there was no config to set what we want and what we don't want to check.
Now the config lives in: spec/config_jshint.yml
We can keep tuning and improving the checks as we want.

:+1:

:mag:


Build not generating expected output - missing some build files

Expected Behavior
Running yarn build should produce all the build artifacts required for a release as stated in the README.md instructions for production
Actual Behavior
yarn build is not producing a build that is missing some required files.
Missing are newtab.html, options.html  and the following js files that should be in the build/chrome/js folder:
background.bundle.js
key.js
login.bundle.js
logout.bundle.js
newtab.bundle.js
options.bundle.js

Steps to Reproduce the Problem

Run yarn build
Run ls -R build/chrome

Specifications

Version: <IP_ADDRESS>
Browser: Chrome
Operating System: macOS Mojave

I was able to reproduce this fairly consistently in macOS Mojave 10.14.4


Explore what's in MVT files

Can we extract building and area polygons from them, and use for hit-testing when we thicken roads?
Can we feed the roads into route-snapper (ignoring bridges/tunnels)?

https://github.com/lukasmartinelli/mapbox-gl-inspect
https://github.com/wangyoucao577/vt2geojson
https://github.com/mapbox/vector-tile-js

Where are road names in there?


Remove filter loader

Remove filter loader
Make use of Zend\ServiceManager\MutableCreationOptionsInterface.
For example:
namespace HtImgModule\Imagine\Filter\Factory;

use Zend\ServiceManager\ServiceLocatorInterface;
use Zend\ServiceManager\FactoryInterface;
use Zend\ServiceManager\MutableCreationOptionsInterface;

class BackgroundFactory implements FactoryInterface, MutableCreationOptionsInterface
{
     public function setCreationOptions(array $options)
     {

     }

     public function createService(ServiceLocatorInterface $filters)
     {

     }
}

This produces BC break.

MutableCreationOptionsInterface has been removed. Options can be passed directly through Factory now.


Eye tracking not working

As reported, eye tracking is not working with the Droolon eye tracker on Pimax HMD-wrong port.

Not a PimaxXR issue. That code is in OpenXR Toolkit.


Feature: Count Unique Users

New Stats schema entity to hold uniqueUsers value which is incremented every time a new user registers
The stats entity is a singleton and we create it if not exists

See https://subgraph-staging.openformat.tech/subgraphs/name/open-format/mumbai/graphql?query=query+MyQuery+{
++stats(id%3A+"STATS_SINGLETON")+{
++++id
++++uniqueUsers
++}
}

See https://subgraph-staging.openformat.tech/subgraphs/name/open-format/mumbai/graphql?query=query+MyQuery+{
++stats(id%3A+"STATS_SINGLETON")+{
++++id
++++uniqueUsers
++}
}

Is that number correct? 1181 feels like a lot.

See https://subgraph-staging.openformat.tech/subgraphs/name/open-format/mumbai/graphql?query=query+MyQuery+{
++stats(id%3A+"STATS_SINGLETON")+{
++++id
++++uniqueUsers
++}
}

Is that number correct? 1181 feels like a lot.

New commit to check for empty user with Zero address before incrementing uniqueUsers, as proposed by @tinypell3ts


Incorporate experiment tracking

Is your feature request related to a problem? Please describe.
The model development process has been, at times, slow, and this has been made worse by having a less defined monitoring/tracking process.
Describe the solution you'd like
Using MLOps tools like DVC, Kubeflow, or MLflow could be used for tracking experiments, as well as offering other potential benefits like version controlling data and, later down the line, easing deployment.

I've done some initial work to implement DVC, largely to test the process and see if it is a good fit. For the moment it seems worth continuing to explore DVC, but there are some issues deciding where Tidymodels should control the process and what belongs under the DVC umbrella.


keepAlive msg have to load from database, fectch the properties when …

…there is no sessionCache

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

I am not following. The data should either cached with latest or loaded.

I am not following. The data should either cached with latest or loaded.

https://github.com/apache/skywalking/issues/10796#issuecomment-1556611342
please see this link

when there is no sessionCache for InstanceTraffic
1 、the skywalking service just started
2、the sessionCache was expired
then get a keepalive metrics from http interface or kafka consumer
private Metrics requireInitialization(Metrics metrics) {
        final Metrics cached = sessionCache.get(metrics);

        // All cached metrics, it at least had been written once.
        if (cached != null) {
            return cached;
        }

        // If the metrics do not have time bucket relative ID
        // it is treated as high dimensionality metrics.
        // Policy, always try load from the database when miss in the cache.
        if (!model.isTimeRelativeID()) {
            return null;
        }

        // When
        // (1) the time bucket of the server's latest stability status is provided
        //     1.1 the OAP has booted successfully
        //     1.2 the current dimensionality is in minute
        //     1.3 the OAP cluster is rebalanced due to scaling
        // (2) the metrics are from the time after the timeOfLatestStabilitySts
        // (3) the metrics don't exist in the cache
        // the kernel should NOT try to load it from the database.
        //
        // Notice, about condition (2),
        // for the specific minute of booted successfully, the metrics are expected to load from database when
        // it doesn't exist in the cache.
        if (timeOfLatestStabilitySts > 0 &&
            metrics.getTimeBucket() > timeOfLatestStabilitySts
            && cached == null) {
            // Return metrics as input to avoid reading from database.
            return metrics;
        }

        return null;
    }

requireInitialization will return metrics,  and do not load  from storage, do not call combine() ,then call  prepareBatchInsert() ,  then the properties is null
this is the pr https://github.com/apache/skywalking/pull/10841  for simple fix

I saw that, but properties are reported periodically too. There may be several minutes latency, but eventually, it would be overrided back to normal.
We don't know there would be properties or nor, we don't want to risk reloading everytime if therw is no property for some agents.

1、in java-agent, the kafka producter send keepalive and register msg , use the different key, so the two msg will be consumed by different consumer.
2、for http interface, we use  a domain name (nginx )for the skywalking cluster,  after the balancer,  the two msg may be handled by different skywalking service.
so the properties is often null
when "it would be overrided back to normal.", after one or two minutes ,  the properties will by null again.
i agree with "We don't know there would be properties or nor, we don't want to risk reloading everytime if therw is no property for some agents."
the agent writed by unofficial  is uncontrollable

thank you !   I will check the cluster config . Thank you very much for guidance


Move position from last to first of DOMParser arg or implemented "flipped" variants

The current API has DOMParser as the last argument; this seems a bit inconvenient for what I imagine to be most uses of partial application (in fact, even if concurrency were an issue for DOMParser, afaik, JS is still single threaded afaik, so it can't be an issue - so I can't at the moment imagine ever needing more than one DOMParser).
The same applies to XMLSerializer as currently implemented in #4, but will plan to make an isolated release with #4 for those preferring the current order.
If there are conceivably other uses for the current API, now or in the future, then we go with flipped: what's a good alternative to name the flipped functions - appending Flipped to the end makes long function names that much longer, so how about just a '?

I'd suggest just letting people use flip if they want a flipped version; the fewer functions we export, the better, I think. I think flipped versions are only really needed if you want to give them operator aliases.

☝ hah I was just writing a reply too.
The convention for putting "the thing at the end" still works better here than the alternative I think, since it allows this kind of usage:
something = do
  result <- parseHTMLFromString "foo" =<< makeDOMParser
  ?x result

Which is probably a pretty common pattern over reusing a DOMParser value multiple times.

Both good points I hadn't thought of, thanks!


Use of undeclared type 'CLAccuracyAuthorization'

Hey!  Sorry, I'm new to Swift and I know this must be right in front of my eyes but I can't seem to figure out this error running all through the source code.  Particularly in the GPSLocationOptions.swift file I keep seeing the error above.
I'm running XCODE 11.3 and Swift 5.1.3.
I've included swiftLocation in my podfile like so:
use_frameworks!
pod 'SwiftLocation/Core'
I've also tried to run the demo playground and am getting the same exact errors.
Any help would be greatly appreciated!
Thanks

Figured it out!  You must have XCODE 12.  Just FYI


Fix dynamic options change

Fixes #7

:tada: This PR is included in version 1.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Snyk] Security upgrade python from 3.11.6-slim to 3.13.0a3-slim

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

Dockerfile

We recommend upgrading to python:3.13.0a3-slim, as this image has only 45 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

436
CVE-2023-6779 SNYK-DEBIAN12-GLIBC-6210088
No Known Exploit

436
CVE-2023-6779 SNYK-DEBIAN12-GLIBC-6210088
No Known Exploit

514
Information Exposure SNYK-DEBIAN12-GNUTLS28-6062100
No Known Exploit

614
Out-of-bounds Write SNYK-DEBIAN12-PERL-6085273
No Known Exploit

500
Integer Overflow or Wraparound SNYK-DEBIAN12-ZLIB-6008963
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Base image was updated to  python-3.11.7-slim in #804  to minimise vulnerabilities while still staying on the python 3.11.x branch.

not compatible with Python 3.13


Introduce context.Context to resource.Builder

I don't see a way to provide a context.Context to the resource.Builder for the purposes of cancellation/timeout

/lifecycle frozen

is this issue available to take up?


ERROR: Config file is missing when trying to start trino-coordinator_1 container

Attempting to follow this README. After a fresh clone of this repo, I hit the following errors when trying to run docker-compose up. Perhaps I am missing something? Thanks!
trino-coordinator_1  | ERROR: Config file is missing: /etc/trino/config.properties
hive-metastore_1     | Exception in thread "main" java.lang.RuntimeException: java.io.FileNotFoundException: /opt/apache-hive-metastore-3.0.0-bin/conf/metastore-site.xml (Permission denied)

I just tried this scenario locally and couldn't reproduce it.
You should rather see different names for the containers.
I have the following:
➜  trino-getting-started git:(main) ✗ docker ps
CONTAINER ID   IMAGE                                 COMMAND                  CREATED         STATUS                   PORTS                              NAMES
1cde2f9fd2d1   bitsondatadev/hive-metastore:latest   "sh -c /entrypoint.sh"   2 minutes ago   Up 2 minutes             <IP_ADDRESS>:9083->9083/tcp             trino-minio-hive-metastore-1
c7080b18a104   trinodb/trino:latest                  "/usr/lib/trino/bin/…"   2 minutes ago   Up 2 minutes (healthy)   <IP_ADDRESS>:8080->8080/tcp             trino-minio-trino-coordinator-1
a80cf2a6f32b   mariadb:latest                        "docker-entrypoint.s…"   2 minutes ago   Up 2 minutes             <IP_ADDRESS>:3306->3306/tcp             trino-minio-mariadb-1
0b461dd71bff   minio/minio:latest                    "/usr/bin/docker-ent…"   2 minutes ago   Up 2 minutes             <IP_ADDRESS>:9000-9001->9000-9001/tcp   minio

They differ from trino-coordinator_1 and hive-metastore_1.
Please add (as file attachment) the output of:

docker ps
docker logs trino-coordinator_1
docker logs hive-metastore_1


[Usage]: Request for vllm project to support OpenAI's /v1/models endpoint

Your current environment
I would like to request that the vllm project support OpenAI’s /v1/models endpoint. When using the Open WebUI framework, this endpoint is required to view the available models, but the vllm project currently does not provide support for it.
How would you like to use vllm
I would like to request that the vllm project support OpenAI’s /v1/models endpoint. When using the Open WebUI framework, this endpoint is required to view the available models, but the vllm project currently does not provide support for it.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

We do have a /v1/models/ endpoint here: https://github.com/vllm-project/vllm/blob/main/vllm/entrypoints/openai/api_server.py#L316

Thank you for the response. It seems I made a mistake in my operations yesterday, which caused the interface not to display. I retried it, and now it works.


"header not found x-ms-request-charge" error when deleting from cosmos

The operation succeeds, but an error is returned. I have tested various versions of the library, including the newest. (The error seems similar to an earlier issue that was solved: https://github.com/Azure/azure-sdk-for-rust/issues/1022 )
[dependencies]
tokio = { version = "1.1.0", features = ["rt-multi-thread", "macros", "time"] }
azure_core = "0.8.0"
azure_data_cosmos = "0.9.0"

use azure_data_cosmos::prelude::*;
use std::error::Error;

const COSMOS_ACCOUNT: &str = "xxx";
const COSMOS_MASTER_KEY: &str = "xxx";
const COSMOS_DATABASE: &str = "xxx";

#[tokio::main]
async fn main() -> Result<(), Box<dyn Error + Send + Sync>> {
    let collection_name = "xxx";
    let document_id = "xxx";
    let partition_key = "xxx";

    let authorization_token = AuthorizationToken::primary_from_base64(&COSMOS_MASTER_KEY)?;
    let client = CosmosClient::new(COSMOS_ACCOUNT.to_string(), authorization_token);
    let database = client.database_client(COSMOS_DATABASE);
    let collection_client = database.collection_client(collection_name.to_string());
    let document_client = collection_client.document_client(document_id, &partition_key)?;

    document_client.delete_document().into_future().await?;

    Ok(())
}

Result:
Error: Error { context: Message { kind: DataConversion, message: "header not found x-ms-request-charge" } }

This issue has hit all our production API. I believe the issue started a few days ago, possibly the 9th. (We started receiving reports from users around that time).

The error occurs here: https://github.com/Azure/azure-sdk-for-rust/blob/356952d4e10a8d93e0cc782eb01a53f95604ec68/sdk/data_cosmos/src/operations/delete_document.rs#L64

An update on this. We got the following from Azure support today: "We got update from our product team, they are currently working on a hotfix for this issue. they will update with a deployment time after finalizing the changes."
So it seems this is not an issue with the SDK, but rather an issue in Azure that will be fixed.

Thank you for the update @ayngling. Will close this, but keep us posted.


Torrentio down

Add-on is stuck loading. Is there any indication of issues on the backend?

Having the same issue

Yup,it down yet very slow loading.....

What's considered "the backend" here?

Same over here.

seems to be up again :)

Another outage right now.

Another outage right now. Are you able to look? @TheBeastLT

Same here was just watching stremio and reloaded the app and now its not working again
I keep getting 2 addons are still loading issue which is annoying

@TheBeastLT
If this is a matter of money, give a donation link and my monthly donations you shall have!

#199


Spacing Issues across Daytona.io

None of the spacings seem to be consist and very few are correct.
Please refer to Figma for correct spacing

Also make sure to correct this on dotfiles, Documentation, Contact, Changelog, Definitions...
@mikimih

I think I fixed some issues with spacing inside markdown content.
The problem is that inside figma I couldn't find spacing numbers for the headings h3, h4, etc. Also the markdown uses ul and ol and I couldn't find spacing for that elements.
If the changes I did in the PR are not good enough we need to discuss above problems

Hey @Swearengen can you reuse spacing settings from https://fly.io/docs/? as i feel the spacing, especially ul and ol are still a bit large.
Thanks!


Don't use docker cache on main

As discussed on the discord. Haven't tested this against main since it'd need to be merged to do that 😅 So will need to see how this handles over the next while.

These docker tags look a bit healthier now 👍


Do not error when feed action is invoke with namespace _

Have the feed action go get the value for namespace if the one pass as parameter is _
We are instructing users to unset the namespace here:
https://console.ng.bluemix.net/openwhisk/learn/cli
wsk property unset --namespace

The feed action will need to handle when user uses the new auth key method and no namespace is configured in the CLI.
Options:

Have the feed action do a a OW API call to get the namespace for the authkey
Have the runtime provide the namespace string associated with the authkey being used to invoke the feedaction. Today there is a namespace environment variable but it doesn't have the value that we need it has the value of the owner of the action not the namespace associated with the authkey.

@jasonpet @johnsolm
It looks like there was a bug that is now fixed in PR https://github.com/openwhisk/openwhisk/pull/1665 pending review and merge.
The bug was that __OW_NAMESPACE was always returning whisk.system instead of the namespace of the user invoking the shared action.
This means we have a single solution regardless if using old key, new key, we can always retrieve the valid namespace from process.env['__OW_NAMESPACE']

This was fixed as part of https://github.com/openwhisk/openwhisk-package-cloudant/pull/48


Add mcrypt

Thanks for this awesome tap! It would be great if you could add the mcrypt (php 5.6) extension.

On PHP5.6, mcrypt should be available by default in shivammathur/php tap
Test: https://github.com/shivammathur/test-setup-php/runs/2380520227?check_suite_focus=true#step:8:29

@shivammathur thanks you for your fast reply, how I can install that? or what I need put in php.ini to enable mcrypt?


[BUG] Errors occured when execute flag install step

Observed behavior
I follow the step in section Go binary , links 👉🏻 https://flagd.dev/installation/#summary

But some errors occured:

Here is the original errors:
go install github.com/open-feature/flagd/flagd@latest

# github.com/open-feature/flagd/flagd/pkg/service/flag-evaluation/ofrep
.gvm/pkgsets/go1.22.5/global/pkg/mod/github.com/open-feature/flagd/flagd@v0.11.0/pkg/service/flag-evaluation/ofrep/handler.go:75:54: undefined: ofrep.BulkEvaluationContextError
.gvm/pkgsets/go1.22.5/global/pkg/mod/github.com/open-feature/flagd/flagd@v0.11.0/pkg/service/flag-evaluation/ofrep/handler.go:80:22: assignment mismatch: 2 variables but h.evaluator.ResolveAllValues returns 1 value
.gvm/pkgsets/go1.22.5/global/pkg/mod/github.com/open-feature/flagd/flagd@v0.11.0/pkg/service/flag-evaluation/ofrep/handler.go:84:47: too many arguments in call to ofrep.BulkEvaluationContextErrorFrom
	have (string, string)
	want ()
# github.com/open-feature/flagd/flagd/pkg/service/flag-sync
.gvm/pkgsets/go1.22.5/global/pkg/mod/github.com/open-feature/flagd/flagd@v0.11.0/pkg/service/flag-sync/sync-multiplexer.go:166:14: assignment mismatch: 2 variables but r.store.GetAll returns 1 value
# github.com/open-feature/flagd/flagd/pkg/service/flag-evaluation
.gvm/pkgsets/go1.22.5/global/pkg/mod/github.com/open-feature/flagd/flagd@v0.11.0/pkg/service/flag-evaluation/flag_evaluator.go:73:17: assignment mismatch: 2 variables but s.eval.ResolveAllValues returns 1 value
.gvm/pkgsets/go1.22.5/global/pkg/mod/github.com/open-feature/flagd/flagd@v0.11.0/pkg/service/flag-evaluation/flag_evaluator.go:336:28: undefined: model.GetErrorMessage
.gvm/pkgsets/go1.22.5/global/pkg/mod/github.com/open-feature/flagd/flagd@v0.11.0/pkg/service/flag-evaluation/flag_evaluator_v2.go:71:17: assignment mismatch: 2 variables but s.eval.ResolveAllValues returns 1 value

Expected Behavior
the flagd binary can be installed in the bin fold for go
Steps to reproduce
execute go install github.com/open-feature/flagd/flagd@latest
BTW, after diving into the code for the latest tag, which is<EMAIL_ADDRESS>
the flagd use ofrep.BulkEvaluationContextError() in core pkg

But the core pkg doesn't contains ofrep.BulkEvaluationContextError() for version core/v0.9.3

ofrep.BulkEvaluationContextError() doesn't occurs in core pkg until for core/v0.10.0
So the main branch is right, but still now the lastet tag for github.com/open-feature/flagd/flagd is wrong

Oh thanks for bringing this to our attention! I think the docker container is the most popular means of using flagd, but we certainly want the bin install to work as well.

So the main branch is right, but still now the lastet tag for github.com/open-feature/flagd/flagd is wrong

Then seems like a new release will fix this: https://github.com/open-feature/flagd/pull/1341
cc @Kavindu-Dodan

Unfortunately, this is due to the specific go install usage, where flagd module's go.mod file is considered for dependencies. Our image builds use go workspaces, so all submodules of core package (flagd & proxy) use latest core components.
@toddbaert  yeah the release will fix this. But we may need to revisit and release core and flagd independently if we wants to promote binary installations 🤔

go install github.com/open-feature/flagd/flagd@latest

sorry late for the reply, with the latest version ,it works on my desktop:

thanks~


endpoint: Register Account UUID

Should be able to register unique identifier e.g. User or UUID or did - for now we can use UUIDv4 generation
POST /users
The server should generate UUIDv4 it returns back - the accounts service does not handle naming / validation.
Accounts server manages solely attested unique reference point that ties in other identifiers e.g. via NNS in future

Close! Users don't have externally-facing IDs; they have DIDs.


Set upper bound for numpy version

Hiya,
I run into an error with newer numpy versions (1.24.0):
File ~/work/graphein/graphein/graphein/molecule/features/graph/molecule.py:67, in <dictcomp>(.0)
     62     descriptors = {
     63         k: v for k, v in descriptors.items() if k in descriptor_list
     64     }
     66 # Compute descriptors
---> 67 desc = {d: descriptors[d](mol) for d in descriptors}
     69 # Process Outformat
     70 if return_array:

File /usr/share/miniconda/envs/__setup_conda/lib/python3.8/site-packages/rdkit/Chem/GraphDescriptors.py:124, in Ipc(mol, avg, dMat, forceDMat)
    121       mol._adjMat = dMat
    123 adjMat = numpy.equal(dMat, 1)
--> 124 cPoly = abs(Graphs.CharacteristicPolynomial(mol,adjMat))
    125 if avg:
    126   return entropy.InfoEntropy(cPoly)

File /usr/share/miniconda/envs/__setup_conda/lib/python3.8/site-packages/rdkit/Chem/Graphs.py:43, in CharacteristicPolynomial(mol, mat)
     41 I = 1. * numpy.identity(nAtoms)
     42 An = A
---> 43 res = numpy.zeros(nAtoms + 1, numpy.float)
     44 res[0] = 1.0
     45 for n in range(1, nAtoms + 1):

File /usr/share/miniconda/envs/__setup_conda/lib/python3.8/site-packages/numpy/__init__.py:284, in __getattr__(attr)
    2[81](https://github.com/a-r-j/graphein/actions/runs/3680239676/jobs/6320741907#step:8:82)     from .testing import Tester
    2[82](https://github.com/a-r-j/graphein/actions/runs/3680239676/jobs/6320741907#step:8:83)     return Tester
--> 284 raise AttributeError("module {!r} has no attribute "
    285                      "{!r}".format(__name__, attr))

AttributeError: module 'numpy' has no attribute 'float'

From this article it appears some aliases (like np.float) have been deprecated. As a result, I think the supported NumPy versions should be constrained or the underlying code refactored to support new versions.

Hi. This was already fixed in the RDKit repository (https://github.com/rdkit/rdkit/pull/5897). The next RDKit PyPI release should work Numpy >=1.24.  In the meantime, please use an older numpy version. Hope this works for you.
Please leave this issue open until the next version is released.

Should be fixed in the 2022.9.4 release.


Android Version

Is there an Android version for this library.

There isn't, but take a look at our new initiative https://github.com/batoulapps/Adhan
Adhan doesn't use the ITL C library for calculations so we will be doing full ports to multiple languages, including Java.


Put the special character at the end

not the beginning

Hi, as you see I've closed the issue because I moved over the special character. I didn't move it to the end because it looked harder to memorize because the special character can split up thew number from the word.
Thank you for reporting,
nikhilhex


Add sticker.yml

Does what it says.

I think, this PR should be merged first, then #37 and then #40 - there might be a rebase neccessary… Let's see. I've prepared a new branch if needed: master_rebased_fix_issue_34


How to block WebRtc?

How to block WebRtc or block all UDP traffic?

headless true
without extensions

Page.addScriptToEvaluateOnNewDocument
'navigator.mediaDevices.getUserMedia = navigator.webkitGetUserMedia = navigator.mozGetUserMedia = navigator.getUserMedia = webkitRTCPeerConnection = RTCPeerConnection = MediaStreamTrack = undefined;'
https://github.com/puppeteer/puppeteer/issues/4149#issuecomment-550333435

It is better to create a small extension
manifest.json
{     "name": "private",    "background": {       "scripts": ["./background.js"],       "persistent": false    },    "version": "1.0",    "manifest_version": 2,    "permissions": [ "privacy" ] }
background.js
chrome.privacy.network.webRTCIPHandlingPolicy.set({   value: 'disable_non_proxied_udp' });

@liteCarma With new headless puppeteer + proxies I am not able to hide IP address. Any suggestions? I tested on pixelscan but it shows location is being spoofed or IP adddress leaked on webrtc test page.

@liteCarma With new headless puppeteer + proxies I am not able to hide IP address. Any suggestions? I tested on pixelscan but it shows location is being spoofed or IP adddress leaked on webrtc test page.

the same problem

@liteCarma With new headless puppeteer + proxies I am not able to hide IP address. Any suggestions? I tested on pixelscan but it shows location is being spoofed or IP adddress leaked on webrtc test page.

Me too, I have been searching for a fix for this on headless browser for ages. Nothing seems to work


Adds Discord Notifications

This pull request adds the ability to send build status updates to Discord via webhooks.
The configuration format looks like the following:
notifications:
  discord:
    channels:
      - [webhook_id]:[webhook_token]
      - [webhook_id_2]:[webhook_token_2]
    pull_request_template:
      - 'Build [#%{build_number}](%{build_url}) ([%{commit}](%{compare_url})) of %{repository}@%{branch} in PR [#%{pull_request_number}](%{pull_request_url}) by %{author} %{result} in %{duration}'
    branch_template:
      - 'Build [#%{build_number}](%{build_url}) ([%{commit}](%{compare_url})) of %{repository}@%{branch} by %{author} %{result} in %{duration}'

Related PRs:
https://github.com/travis-ci/travis-core/issues/562
https://github.com/travis-ci/travis-hub/issues/84

Thanks for opening this PR. We'll get back to you soon.

any updates? soon might have been a misnomer, lol.

Usefull PR ! 👍

Waiting warmly for this PR to be merged.

@BanzaiMan any feedback or updates?

Useful PR! +1 to night!

+1

@BanzaiMan sorry for the ping, but any updates? it's been a while and this would be amazing to get working.

I doubt we'll ever get a reply to this, but I think they've already made up their minds and have settled on a "no" to this integration, but are too afraid to say it. I mainly did this for hacktoberfest swag anyways, so it doesn't really irk me. But it woulda been kinda cool to see this be integrated. ¯\(ツ)/¯

have you seen any workarounds with existing integrations like their slack integration to force it to send to a webhook with slack structure? since they have similar structure, it's what i've done on sentry for some projects.

@night, would love to see this added! Maybe they haven't merged it because of a build failure here: https://github.com/travis-ci/travis-core/issues/562

The failure there is a dependency version issue, the dependency requires 2.1, bit it's trying to run against 2.0, @night. is that the only version of net-http-persistent that works?

@aliasfalse that test failing is not my fault nor my responsibility to fix, and there's already an open PR to fix the build, but it's unmerged as of November of last year.. the failing test is not a valid reason why this is not yet merged either.

Tasks should be core-less at this point.

Tested in staging (with my added logging and conflict resolution):
2017-02-20T02:53:02.282202+00:00 app[tasks.1]: I queue=discord notifier=discord build=542711 repo=BanzaiMan/travis_staging_test  time=0.001 jid=d2ecd4560a4c61671350ed84 event=push

If you want to test this now, please set up a repository on https://staging.travis-ci.org (go to https://staging/travis-ci.org/profile/USER; sync and enable a repo; push a commit to the repo).

Also, please open a documentation PR for https://github.com/travis-ci/docs-travis-ci-com.

Will be implemented when? @night @BanzaiMan Since this seems like a great feature.

Yup, this would be awesome to have!

+1

Hello. Sorry about the delay in responding. After some further internal discussion, we have put this PR on hold.

I figured I would add this here, I emailed support wondering if there was a way to send a slack webhook to a custom URL, rather than putting in the slack code and token, and this was the response I got (about 10 minutes before the reply above, from the same representative)

Hi, Tony,
Thanks for the email.
At the moment, it is not possible to pass URL to the Slack notifier, as the endpoint is hardcoded.
We have an outstanding PR https://github.com/travis-ci/travis-tasks/pull/71 that implements support for Discord. As indicated there, however, we have unfortunately decided to put it on hold. You are welcome to subscribe to the PR for any future updates, though.
It might be possible to write a small service that takes our webhook payload and POSTs data to the discord endpoint after appropriate transformation.
Thank you.

Figured I would post this here as I took BanzaiMan's suggestion and made a small php script that forwards a travis-ci webhook on to discord. It looks like:

The repo is here: https://github.com/Techtony96/TravisCI-Webhooks/

I have created a PR for documentation at https://github.com/travis-ci/docs-travis-ci-com/pull/1133, which should unblock this should I be the blocker.

@night might want to fix the conflict

Hmm, I'm currently working around this by simply doing this.
after_deploy:
- "curl -v -H 'Authorization: <token>' -H "User-Agent: bot" -H "Content-Type: application/json" -d '{"content":"Deployed!"}' https://discordapp.com/api/webhooks/<channel ID>/<token>"

It sucks. It would be nice to get some sort of date since the code and documentation already looks complete.

@fartbagxp you shouldn't be passing the webhook token as an authorization header. wherever you saw that documented is wrong

I believe he's passing a bot token, not a webhook token, in which case his
method of authentication is correct.
On Mon, Aug 7, 2017 at 1:37 AM Zack<EMAIL_ADDRESS>wrote:

@fartbagxp https://github.com/fartbagxp you shouldn't be passing the
webhook token as an authorization header. wherever you saw that documented
is wrong
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/travis-ci/travis-tasks/pull/71#issuecomment-320573064,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMYVaCO7El81-b1xVBqSlnKwkP7INc1tks5sVqKtgaJpZM4KSQdE
.

@serebit

He's clearly using the same token for the authorization header and the webhook token, and his variable is named as such.

Webhooks require no authorization header since the token is in the URL itself.

In that case, yes, his method of authentication is incorrect.
On Mon, Aug 7, 2017 at 12:06 PM Zack<EMAIL_ADDRESS>wrote:

@serebit https://github.com/serebit

He's clearly using the same token for the authorization header and the
webhook token, and his variable is named as such.
2.
Webhooks require no authorization header since the token is in the URL
itself.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/travis-ci/travis-tasks/pull/71#issuecomment-320706680,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMYVaCvNGTdRb69deFATiRI_iMImwVYhks5sVzYLgaJpZM4KSQdE
.

Ah, thank you. Fixed it.
I think I was copying somebody else's example while creating my own and put that in there.
Didn't know what I was thinking. Thanks for the help.

A few months/years late, but figured I'd put this out there anyway.
For those of us too scared to dabble with PHP, or can't deploy it for whatever reason, I've made ravendisruptor/travis-discord-webhook with Flask. It's already set up so that deploying to Heroku can be done with ease, and will only take a few seconds.
Any contributions welcome, the code is a bit eh, but it works I guess. ¯\_(ツ)_/¯
Thanks, @Techtony96, for the code. I might've nicked a few sections when making this.
I've tried to style it more like GitHub's message when you use its webhook, so it looks like the following:

Is there something we can do to unblock this, @BanzaiMan?
I too have a shim in place to transform your webhook format into my desired discord message, but that rather defeats the purpose of having all my CI infrastructure in the cloud :)

@BanzaiMan any updates? I'm practically done with a PR to implement native Twilio support but I'm worried it's going to go untouched indefinitely.

I really don't understand why this has not been acted on or even meaningfully responded to in over a year.

@msgerbs I too would like to know.
For now, I use the script written by @Techtony96 here

Tbh it'd be awesome to have Discord support.

For those of you that don't have a webserver for the php solution, I made a couple of scripts that run on the build server: https://github.com/Erlite/travis-ci-discord-webhooks

So, why is it on hold?

@BanzaiMan Are we going to get any updates here? It would be helpful for the whole community if you shared your reasons for why this isn't being merged. That way, we can resolve that and move on

It's been more than a year, any updates yet?
In the meanwhile, for those who are still looking for a workaround, here's a simple guide to get Travis CI build status notifications via webhooks in Discord: https://github.com/snkrsnkampa/travis-ci-discord-webhook
And you don't need any server for this.
Thanks @Erlite for the heads up.

Asking for updates on a frequent basis is annoying and unhelpful, give the kind people at Travis some time to reply to the request for updates, a couple of days is not enough time for a response

The submission date was October 2016, not 2016.

Yes, but many have already @ mentioned or otherwise the people at travis already, any further attention seeking is annoying and uneeded.

Just trying to get answers @Coding-Doctors . Not really unneeded. Going over a year without an actual reason to why this hasn't been merged or even discussed is "annoying and unneeded".
After all, the squeeky wheel gets the grease.


UI is resized when using this package

I experienced a visual bug as soon as I started using this package, this is what my UI look like without it:

And now, after using the JSONEditorWidget in my admin.py

Basically, the UI looks resized, smaller, as if I had zoomed out. Probably a bad CSS rule applying undesired styles on the whole page instead of a sub element or something.
Using Python 3.6, Django 2.1.7

Did you ever find a fix for this?

I never looked into it, and I'm not using this plugin anymore so I won't.

This should be fixed in master

And now in 0.2.2, closing.


Enforce create udp4 socket

Right now when I create udp socket there is no option to specify which ip version to use, I've got an issue on one of my servers statd clients sends udp packets over udp6
#
U ::1:13896 -> ::1:8125
environment.p007.p007_1.p007_1.redis.connection.active:1596|g

But statsd just drops these packets, only when I send them over ipv4
echo "sample.gauge:14|g" | nc -u -w0 <IP_ADDRESS> 8125
it is ok. Is there any way to specify ip version of client or it needs pull request?

not sure I follow... can you not specify a udp4 address in the constructor?
statsd.NewStatsdClient(udp4_address_here, ...)


DataViewModel should support groupBy

Could we add a groupBy property to the DataViewModel so that the DataView grid can render the sorted field according to groups (or at least a single, specified group)?
Use case for Hoist Roadmap, to group by quarterly phases:

👍 Thanks All!


invalid licensee "LGPL-2.1-or-later"

Describe the bug
I added [licenses] allow = ["LGPL-2.1-or-later"] to deny.toml. I expect it to be accepted, since it's listed on https://spdx.org/licenses/. Instead, it gives a parse error:
; cargo deny check licenses
error: invalid licensee
   ┌─ /Users/jyn/src/redacted/deny.toml:16:14
   │
16 │     "LGPL-2.1-or-later",
   │              ^^^^^^^^^ expected a `<bare-gnu-license>` here

2023-04-26 14:23:53 [ERROR] failed to validate configuration file deny.toml

To Reproduce
Steps to reproduce the behavior:

echo 'licenses.allow = ["LGPL-2.1-or-later"]' > deny.toml
cargo deny check licenses

Device:

OS: MacOS 13.3.1, Darwin 22.4.0
Browser: N/A
Version: cargo-deny 0.13.9

You need to leave off the -or-later. https://embarkstudios.github.io/cargo-deny/checks/licenses/cfg.html#note-on-gnu-licenses

When you release source code with a GPL2 or GPL3 license, you can choose whether you want people with access to that source code to use it under the terms of only that license or if they can use it under the terms of a newer license.
This is correctly handled by cargo deny in that if I have a "GPL-2.0-only" dependency and deny.toml allows "GPL-3.0" then the license check will fail, but if I have a "GPL-2.0-or-later" dependency then cargo deny will correctly accept it in this case.
But! If I have a "GPL-3.0-only" dependency and my project is "GPL-3.0-or-later" then deny.toml doesn't allow me to prevent this scenario. If I have a "GPL-3.0-only" dependency in a "GPL-3.0-or-later" project then my project is mis-licensed, the project license has to be as restrictive as all other licenses combined, "-only" is more restrictive than "-or-later" and so my project must be licensed as "GPL-3.0-only".
Therefore, the difference between allowing "GPL-3.0" and "GPL-3.0-or-later" would be a useful distinction in these circumstances. As by allowing "GPL-3.0-or-later" instead of "GPL-3.0" I can correctly encode the requirement that I do not want "GPL-3.0-only" dependencies.

Yah, I suppose the behavior could be improved for this kind of scenario. However, this would be extremely low priority for me since it's (IMO) a pretty niche problem.
If you are interested, the spdx crate is the one responsible for parsing and evaluating license expressions,


fix clangd wrapper

get libc-cflags and libcxx-cxxflags from the canonical locations &
the correct compiler.

fix the order of those for C++ (libc-cflags are -idirafter for a
reason).

make it easier to build custom clangd wrappers for different stdenvs
by way of overriding the "cc" attribute (not tested).

Motivation for this change
clangd wasn't wrapped completely properly
Things done

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

[v ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 120229 at 172b55fc run on aarch64-linux 1

2 packages marked as broken and skipped:

python38Packages.tiledb
python39Packages.tiledb

5 packages built successfully:

bat-extras.prettybat
clang-tools
cloudcompare
pdal
tiledb

The advantage here is that the compiler from the current environment is used rather what was used at compile time.
This should even work with compilers not coming from nixpkgs...

That could probably work too.  It still (at least) carries the "one stdenv, one cc" assumption about user's project, but is otherwise more robust.
But the thing is, what we are trying to paper over in the wrapper is really a clangd bug -- it has full command lines in its compile_commands.json and should really just do the -v /dev/null -o /dev/null thing itself, per each compiler driver it encounters.  I don't know whether complicating the wrapper any further to get a little closer is worth it.

But the thing is, what we are trying to paper over in the wrapper is really a clangd bug -- it has full command lines in its compile_commands.json and should really just do the -v /dev/null -o /dev/null thing itself, per each compiler driver it encounters. I don't know whether complicating the wrapper any further to get a little closer is worth it.

Clang has tones of logic to detect include directories on various platforms/architectures, that's why they don't bother with it.
I don't think this will change any time soon.

everyone's busy with the upcoming release, I gather?

Not to be a nag, but what is the actual status of this PR?
(This is mostly academic interest on my part at this point, but still an interest!  I mean, how are things supposed to work in the nixpkgs land, exactly?)
I see The System (whatever logic that was) assigned @aherrmann as the reviewer (and, I guess, potential committer?)  Is there an expiration on this assignment, seeing how @ahermann hasn't so far gotten around to saying anything here (I'm sure he has higher priorities and other good reasons for not getting around to this -- it is really really not my intention to nudge any particular individuals, doubly so for people volunteering their spare time!)?
Or is this PR somehow not mergeable now because @Mic92 suggested another possible solution (which I don't consider an improvement, see above)?  That's also fine, but is there anything I can do on my end?
Put another way: is there any actual working process to get a PR merged?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/get-clangd-to-find-standard-headers-in-nix-shell/11268/14


Region flag not returning expected (us-1,us-2, eu-1)

% export FALCON_CLOUD_ENV=$( ./falcon-container-sensor-pull.sh -t falcon-imageanalyzer --region )
% echo $FALCON_CLOUD_ENV
Using the following settings:
Falcon Region:   api.us-2.crowdstrike.com
Falcon Registry: registry.crowdstrike.com
1.0.9: Pulling from falcon-imageanalyzer/us-2/release/falcon-imageanalyzer
Digest: sha256:e84a675c5092bf79acdd0055d892d925393e94a2a7f2efe7aa41391ffd7aed9a
Status: Image is up to date for registry.crowdstrike.com/falcon-imageanalyzer/us-2/release/falcon-imageanalyzer:1.0.9
registry.crowdstrike.com/falcon-imageanalyzer/us-2/release/falcon-imageanalyzer:1.0.9

@jmckenzie-cs - The -r, --region flag is intended for setting the region value rather than retrieving it. This flag allows you to specify the region when pulling the image, but it does not return the region value.

If you would like to request additional functionality in the given script, please open up a new issue requesting such. Otherwise, I'm going to close this as there is no action necessary since it's just using the flag incorrectly.