FREME NER is not spotting entities

This is the file with the content. test.txt
Some information about the file:

cUrl command
→ curl -v -d @test.txt "http://api-dev.freme-project.eu/current/e-entity/freme-ner/documents/?informat=text&outformat=json-ld&language=en&dataset=dbpedia&enrichement=dbpedia-categories" -H "Content-Type:"

I tried something similar with the opened issue https://github.com/freme-project/freme-ner/issues/46 by removing the first paragraph from the content and I get one entity only.
{
    "@id" : "http://freme-project.eu/#char=10,17",
    "@type" : [ "nif:Phrase", "nif:Word", "nif:RFC5147String", "nif:String" ],
    "nif:anchorOf" : "Germany",
    "beginIndex" : "10",
    "endIndex" : "17",
    "referenceContext" : "http://freme-project.eu/#char=0,2889",
    "taClassRef" : "http://nerd.eurecom.fr/ontology#Location",
    "itsrdf:taConfidence" : 0.5323128034518319,
    "taIdentRef" : "dbpedia:Germany"
  }

Germany/German is mentioned 13 times in the content so this entity is OK. China/China’s is mentioned 4 times and was not spotted as a entity.
This is the link from where I get the content. http://www.ft.com/intl/cms/s/0/7786936c-73fc-11e5-bdb1-e6e4767162cc.html#axzz3p07brijW
Same content in Opencalais

I noticed the same issue while I was testing FREME NER by using very simple text.
For example no entities are spot in the text Welcome to Dublin and only Dublin (but not Ireland) is recognized as an entity in the text Dublin is in Ireland. The e-Entity service with DBpedia Spotlight engine properly recognizes all entities.
First curl request
C:\Users\Martab\curl\nossl>curl -X POST --header "Content-Type: "  "http://api-dev.freme-project.eu/current/e-entity/freme-ner/documents?input=Welcome
%20to%20Dublin&informat=text&outformat=turtle&language=en&dataset=dbpedia"

Response
@prefix dbpedia-fr: <http://fr.dbpedia.org/resource/> .
@prefix dbc:   <http://dbpedia.org/resource/Category:> .
@prefix dbpedia-es: <http://es.dbpedia.org/resource/> .
@prefix xsd:   <http://www.w3.org/2001/XMLSchema#> .
@prefix itsrdf: <http://www.w3.org/2005/11/its/rdf#> .
@prefix dbpedia: <http://dbpedia.org/resource/> .
@prefix rdfs:  <http://www.w3.org/2000/01/rdf-schema#> .
@prefix nif:   <http://persistence.uni-leipzig.org/nlp2rdf/ontologies/nif-core#> .
@prefix dbpedia-de: <http://de.dbpedia.org/resource/> .
@prefix dbpedia-ru: <http://ru.dbpedia.org/resource/> .
@prefix freme-onto: <http://freme-project.eu/ns#> .
@prefix dbpedia-nl: <http://nl.dbpedia.org/resource/> .
@prefix dcterms: <http://purl.org/dc/terms/> .
@prefix dbpedia-it: <http://it.dbpedia.org/resource/> .

<http://freme-project.eu/#char=0,17>
        a               nif:String , nif:Context , nif:RFC5147String ;
        nif:beginIndex  "0"^^xsd:int ;
        nif:endIndex    "17"^^xsd:int ;
        nif:isString    "Welcome to Dublin"^^xsd:string .

Second curl request
curl -X POST --header "Content-Type: " "http://api-dev.freme-project.eu/current/e-entity/freme-ner/documents?input=Dublin%2
0is%20in%20Ireland&informat=text&outformat=turtle&language=en&dataset=dbpedia"

Response
@prefix dbpedia-fr: <http://fr.dbpedia.org/resource/> .
@prefix dbc:   <http://dbpedia.org/resource/Category:> .
@prefix dbpedia-es: <http://es.dbpedia.org/resource/> .
@prefix xsd:   <http://www.w3.org/2001/XMLSchema#> .
@prefix itsrdf: <http://www.w3.org/2005/11/its/rdf#> .
@prefix dbpedia: <http://dbpedia.org/resource/> .
@prefix rdfs:  <http://www.w3.org/2000/01/rdf-schema#> .
@prefix nif:   <http://persistence.uni-leipzig.org/nlp2rdf/ontologies/nif-core#> .
@prefix dbpedia-de: <http://de.dbpedia.org/resource/> .
@prefix dbpedia-ru: <http://ru.dbpedia.org/resource/> .
@prefix freme-onto: <http://freme-project.eu/ns#> .
@prefix dbpedia-nl: <http://nl.dbpedia.org/resource/> .
@prefix dcterms: <http://purl.org/dc/terms/> .
@prefix dbpedia-it: <http://it.dbpedia.org/resource/> .

<http://freme-project.eu/#char=0,20>
        a               nif:String , nif:Context , nif:RFC5147String ;
        nif:beginIndex  "0"^^xsd:int ;
        nif:endIndex    "20"^^xsd:int ;
        nif:isString    "Dublin is in Ireland"^^xsd:string .

<http://freme-project.eu/#char=0,6>
        a                     nif:RFC5147String , nif:Word , nif:String , nif:Phrase ;
        nif:anchorOf          "Dublin"^^xsd:string ;
        nif:beginIndex        "0"^^xsd:int ;
        nif:endIndex          "6"^^xsd:int ;
        nif:referenceContext  <http://freme-project.eu/#char=0,20> ;
        itsrdf:taClassRef     <http://nerd.eurecom.fr/ontology#Location> ;
        itsrdf:taConfidence   "0.9885517011780088"^^xsd:double ;
        itsrdf:taIdentRef     dbpedia:Dublin .

only Dublin (but not Ireland) is recognized as an entity in the text Dublin is in Ireland. The e-Entity service with DBpedia Spotlight engine properly recognizes all entities.

This should be solved with https://github.com/freme-project/freme-ner/commit/eae252d765f680c7622dcda953e6cf69371cd3ac
I try and both, Dublin and Ireland are spotted. See http://api-dev.freme-project.eu/current/e-entity/freme-ner/documents/?informat=text&input=Dublin is in Ireland.&outformat=turtle&language=en&dataset=dbpedia

@xFran

Germany/German is mentioned 13 times in the content so this entity is OK. China/China’s is mentioned 4 times and was not spotted as a entity.

This should be fixed with https://github.com/freme-project/freme-ner/commit/eae252d765f680c7622dcda953e6cf69371cd3ac
I tried, and got China spotted.

I tried, and got China spotted.

How is that? Did you fixed something? My cUrl was/is wrong?
Please send me your text document along side with the cUrl request so I can test it.
We should close the issue than?

yes, it was fixed yesterday. That was bug.

I will test it again and if is ok I will close the issue.


chore: add clone to DictStateReader for Mempool testing

this is required for the TestStateReaderFactory

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.81%. Comparing base (e7b88c5) to head (5256b58).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1899   +/-   ##
=======================================
  Coverage   77.81%   77.81%
=======================================
  Files          60       60
  Lines        8722     8722
  Branches     8722     8722
=======================================
  Hits         6787     6787
  Misses       1496     1496
  Partials      439      439

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


go get goimports fails on go 1.4.2

What version of Go are you using (go version)?
1.4.2

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOCHAR="6"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/travis/gopath"
GORACE=""
GOROOT="/home/travis/.gimme/versions/go1.4.2.linux.amd64"
GOTOOLDIR="/home/travis/.gimme/versions/go1.4.2.linux.amd64/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?
go get golang.org/x/tools/cmd/goimports

What did you expect to see?
successful install

What did you see instead?
package runtime/trace: unrecognized import path "runtime/trace"

It appears to be caused by https://go-review.googlesource.com/#/c/25001/
I know that 1.4.2 is out of date.  We ran into this while doing CI on the gonum/internal package, and we are planning on dropping support for it entirely.  However, goimports is integrated into some IDEs and this might lead to some breakage, so I figured I would point it out.

Dup of #16434.
Also, gonum itself no longer supports Go1.4: https://github.com/gonum/internal/pull/33/commits/518f259eb2a573d42d591c29afb87b39c14f3845

Oops, sorry.


create_mgmt_root_dir wrong use of API

Hi,
The create_mgmt_root_dir Ansible role utilizes the following action:
ibmsecurity.isam.web.reverse_proxy.management_root.directory.create
This function has the following signature:
create(isamAppliance, instance_id, id, name, check_mode=False, force=False)
Upon executing a task using this role, I received the following trace:
[DEBUG] [ibmsecurity.appliance.ibmappliance] [_process_response():64] Text: [{\"name\":\"ISAM Base Appliance\",\"description\":\"IBM Security Access Manager Base Appliance\",\"id\":\"wga\",\"enabled\":\"True\"}]\n", "msg": "Error> action does not have the right set of arguments or there is a code bug! Options: isamAppliance=isam_server, force=False, instance_id=\"rp-demotest\", id=\"management/C\", filename=\"dirname\"", "name": "ibmsecurity.isam.web.reverse_proxy.management_root.directory.create"}
This is due to an incorrect mapping in the actual task: see here.
Which according to the API should use "name" instead of "filename".
I'll open a Pull Request in order to resolve this issue.
Kind regards,
Dries Eestermans
P.S.: A restart of the reverse proxy instance should also be issued, this will be added via the PR as well.

Please note that Reverse Proxy does NOT need to be restarted for changes to the file taking effect. So I deliberately left out the handler call to restart reverse proxy. I realize that reverse proxy does get flagged as requiring a restart - which I would like to submit a RFE.
Let me know if you need the restart handler - will hold off on merging the pull request.

As suggested, the restart handler has been removed.

Looks like I merged in the wrong order - I will submit another change with the handler removed.


Updated CLI tool for DASH

Work on the original DASH standalone application has been discontinued, and we are now exclusively using the DASH website to generate seeds. The latest CLI tool is a script that should be run with NodeJS which generates seeds directly from the website. Recommended NodeJS version is 18.14.0 LTS, but the script should theoretically work with 17.5+ due to the use of fetch.
Current presets supported by the CLI tool: mm and full
Per discussions with Synack, I have not committed the dash.cli.js file. It is hosted on the DASH website here: https://dashrando.github.io/app/dash.cli.js

Thank you for the pull request.


SentinelHub package

I tried to follow the tutorial (workshop) and when I executed the following:
from sentinelhub import BBoxSplitter, BBox, CRS, DataSource, transform_bbox, GeopediaFeatureIterator
I got this error:
ImportError                               Traceback (most recent call last)
 in
21 #sentinelhub imports
22
---> 23 from sentinelhub import BBoxSplitter, BBox, CRS, DataSource, transform_bbox, GeopediaFeatureIterator
ImportError: cannot import name 'transform_bbox' from 'sentinelhub' (C:\Users\ASUS\Anaconda3\envs\ASV\lib\site-packages\sentinelhub_init_.py)

Although I am not sure which particular tutorial that is, I am sure it was not updated for some time. the transform_bbox method has been removed, as the same can be achieved by
bbox.transform(to_crs)

Just remove the transform_bbox from the imports, and replace the transform_bbox(bbox, crs) with the code above.
We'd be grateful if you could pinpoint the exact tutorial for this, or even fix it and create a pull request.


fix(build): add publish config

: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:


arrows hide

the arrows no show in the demo.
How can i show the arrows ?
i tread in css arrows.svg, but i can't show.

oh, sorry must have overlooked this one.
are you sure you copied them to the right place? how are you loading the demo. It should be served by some kind of web server, so that all the paths are right.
I hope this still helps somehow..


Add survey link in footer.html

Purpose/implementation Section
What changes are being implemented in this Pull Request?
We realized that Coursera doesn't have an easy to find feedback form, so we are adding one to the footer.
This will automatically get incorporated on all pages upon rendering.

There's some weird capitalization inconsistencies I couldn't get rid of either and not sure if they matter.


do we have any method or attribute to indicate that page has closed?

Searched issues and could not find any hints, so I opened this. If I missed a issue already has this, please close this. Thanks in advance.
I read the  api doc and see currently pptr only have 'close' event and found _target._isClosedPromise is a promise value which could not be used as a state indicator. Do we have any method or attribute to indicate that the page has been closed after invoke page.close() ?
Feel free to correct me if I misunderstand sth.

@aslushnikov
In most cases, that's enough for usage. But currently i am working on a testing tool and in some scenario, page is kind of used in global scope and i need to judge the page is closed in a afterAll callback to see if i need to call page.close() again.

@Rvtea I see. Feel free to add the method to send a PR.


Question related to auto-encoding

Hi,
I wonder, since the proposed one-step decoder uses full-length encoded representation as well, so to pretrain with autoencoding, is it possible that the decoder just copy the original sequence from the encoder side and uses it directly as outputs?
Thanks

well...i forgot its masked


Enable Java Gradle Precommit in Jenkins to gather data using Java as a benchmark.

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Seed Job

Run Seed Job

FAILURE
--none--

R: @tgroh

I'm purposely killing the precommit runs since this change does not impact those builds.


[SMALLFIX] Replace explicit type argument with lamda

Del "AlluxioURI" in "Status.RUNNING"

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2648/
Test PASSed.

alluxio-bot, merge this please.


rasa shell nlu ouput shows unicode code not character string

Rasa version:
1.1.4
Python version:
Python 3.6
Operating system (windows, osx, ...):
Windows 10 1903
Issue:
(rasa) D:\OneDrive\Work\rasabot>rasa shell nlu
NLU model loaded. Type a message and press enter to parse it.
Next message:
陪我聊聊天
{
  "intent": {
    "name": "chitchat",
    "confidence": 0.9645294547080994
  },
  "entities": [],
  "intent_ranking": [
    {
      "name": "chitchat",
      "confidence": 0.9645294547080994
    },
......
    {
      "name": "whoami",
      "confidence": 0.010188285261392593
    },
    {
      "name": "define",
      "confidence": 0.0
    },
    {
      "name": "affirm",
      "confidence": 0.0
    },
    {
      "name": "exit",
      "confidence": 0.0
    }
  ],
  "text": "\u966a\u6211\u804a\u804a\u5929"
}
Next message:

The output show unicode like #3564.

Thanks for raising this issue, @samwill get back to you about it soon.


python3 support

More of a feature request. OpenBMC builds the croserver into it's image. OpenBMC uses the upstream yocto distribution as it's base and they have removed all support for python2 and now only support python3.  We'll need tools like eCMD that we pull in to move as well.
Any chance this is on the list for things like config.py? Sometimes it's as easy as just changing the first line to "#!/usr/bin/env python3"

Does yocto not have a python->python3 link?  We hit a similar issue when fedora made python3 the default.  We updated the script to be compatible for both so the current #!/usr/bin/env python will work regardless of the python install.

The direction I've seen is to not do that because you may end up with some weird undefined behavior. In general, python scripts are supposed to explicitly move to python3 to show they support it (and have tested it). I'm def not the expert though. Are you thinking eCMD is going to have to support both for a while? With python2 susnet (https://www.python.org/doc/sunset-python-2/), yocto just put the big hammer down and removed it completely.
We could probably put a little workaround in our croserver recipe and just replace the python with python3 in the config.py script if you feel like the support is there for it.

Ahh, shoot, I ran into an issue with my workaround. config.py is using a deprecated python2 only option with platform:
| Establishing eCMD build variables..
| Determining host and distro..
| Traceback (most recent call last):
|   File "/home/andrewg/Code/openbmc/build/tmp/work/arm1176jzs-openbmc-linux-gnueabi/croserver/git-r0/git/config.py", line 373, in <module>
|     dtuple = platform.linux_distribution()
| AttributeError: module 'platform' has no attribute 'linux_distribution'

See https://docs.python.org/2/library/platform.html

eCMD still supports building on el6, which barely has python3 support.  So it's been a balancing act.  While the env python method isn't ideal, it has provided the most compatibility for users.
That said, you could try pulling this branch: https://github.com/thejta/eCMD/tree/update-config
That has both a hardcoded python3 call and updated logic to handle linux_distribution being pulled from python3.8.  The specific changeset is: https://github.com/thejta/eCMD/commit/28e306fb777681ffb9cfd6de293e3b29a174448a
It does require the distro module be available in the build environment, since that is the replacement for the deprecated function.
Adding @mklight and @kahnevan as they ultimately decide what goes in.  I just had a chance to hack this together.

Thanks for the patch @thejta . Looks like yocto does not have a recipe for distro yet :(  I'll see what it takes to get one done.

ok, I put a upstream recipe in via https://github.com/openembedded/meta-openembedded/pull/213 to get the Distro package.
@mklight or @kahnevan can we get the code from @thejta into master?

I put it up the branch a PR#361 for @mklight and @kahnevan to review/test

Can we close this based on #361 merge?

Yep, thanks for all the help guys, looks good!


Can't show all database names when DBbeaver connects to PG database?

System information:

Operating system (distribution) and version
DBeaver version 21.2.0
Additional extensions

Connection specification:

Database name and version PostgreSQL
Driver name org.postgresql.Driver
Do you use tunnels or proxies (SSH, SOCKS, etc)?  no

Describe the problem you're observing:
I showed 8 databases using the PG Admin tool, but Dbeaver didn't show up.
picture
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

need to configure the connection to display a non-default database.


title for Amazon did not work as expected

12:56:41 <user> http://www.amazon.com/gp/product/B002NSCHY4
12:56:41 <bot> Robot Check

Could this be avoided by spoofing a user agent?

Looks like it's just you. Works on my machine™
[13:27]  <dosman711> http://www.amazon.com/gp/product/B002NSCHY4
[13:27]  <dosman-bot> Amazon.com: Skin-TacTM Adhesive Barrier Wipes 50 count: Health & Personal Care


[hotfix][flink-avro] Removed unused variable

What is the purpose of the change

The variable actualSchema is declared but not used.

Brief change log

Removed unused variable.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@leonardBang @wuchong - Can you please review this PR and let me know your feedback ?  Thanks

@leonardBang  @wuchong  - Thanks for your advice.   Please let me know if there is anything else required.  Thanks.


Resolve CLIMATE-407 Dataset select does not queue empty datasets.

Addresses the issue CLIMATE-407. Please have a look at it.

Can one of the admins verify this patch?

@lewismc please take a look at this patch.

:+1: this looks great to me. I'll merge this tonight when I get home unless someone beats me to it. Thanks for poking on it @Omkar20895!

Thank you :D @MJJoyce


Save playlist sometimes not working

Seems that some users are having issues with saving the playlist. This could be due to browser requests being blocked, or due to user error.
If issue #5 doesn't resolve these issues we may need to look deeper into this.

Duplicate  issue with #4


Add riscv-64 to the supported architectures of the install script

Thanks for filing a feature request! Please fill out the TODOs below.
Feature
TODO: Brief description of the feature/improvement you'd like to see in
Cranelift/Wasmtime.
The install script currently doesn't allow me to install a riscv-64 program
Benefit
TODO: What is the value of adding this in Cranelift/Wasmtime? What problems does
it solve?
Ease of installation for riscv devices
Implementation
TODO: Do you have an implementation plan, and/or ideas for data structures or
algorithms to use?
I'm unable to locate the install script curl https://wasmtime.dev/install.sh -sSf | bash
Alternatives
TODO: What are the alternative implementation approaches or alternative ways to
solve the problem that this feature would solve? How do these alternatives
compare to this proposal?

I started working on this. Updating the script was fairly easy. However I've ran into an issue with the arch name being different in the releases vs what the OS identifies locally.
Locally I'm running Debian 13 (trixie), and uname -m identifies the architecture as riscv64, but on our release binaries we tag them as riscv64gc, which is the target triple name for our target.
I'm not entirely sure where we should fix this. Do we update our release names, or should I try to fix this up in the script? (cc: @alexcrichton)

Hi, thanks for making that change, where is the script located to make these changes?

https://github.com/bytecodealliance/wasmtime.dev/blob/gh-pages/install.sh


[ Feature request ] Include build number in version.

Any chance to include last commit hash in --version

No. The final release versions are unique enough. You can always track the commits based on the Github tags.

thanks for adding it @dweidenfeld


Run C++ TOPI tests in Jenkins build

Currently the Jenkins build doesn't run the tests for C++ TOPI ops and schedules. This PR adds these to the Jenkinsfile to avoid unnoticed regressions.

Not sure why the test is failing - seems to be failing on the python log op in test_topi_math.py

for things like log, sqrt, make sure that input is positive(take abs after you do uniform),

Ok just done some digging with this test - the one that was failing was the python one (not the c++ one) which was already taking abs after uniform. The problem was it wasn't setting the atol parameter in assert_allclose, so it was defaulting to 0. I've fixed this and also set the uniform so it will never output 0 which may cause log to fail.


Update README for NextJS Apps

I added this in the Discussions but adding here as well for extra visibility
Hello! I am using the react95 package in a NextJS app and was getting an error because it wasn't able to properly load the fonts front the package. I think it's worth to update the README with the following instructions:

Add next-transpile-modules and next-fonts dependencies:

  npm install --save-dev next-transpile-modules next-fonts
  // or
  yarn add next-transpile-modules next-fonts

Edit your next.config.js file to something like:

const withFonts = require('next-fonts');
const withTM = require('next-transpile-modules')([
  'react95',
]);
module.exports = withTM(withFonts());

hey! i was having a similar problem, and you can also solve it with nextjs support for local fonts: https://nextjs.org/docs/app/building-your-application/optimizing/fonts in your layout.tsx file


InfoWindow DOM element leak

Hi!
It seems that InfoWindow is creating DOM element, but it doesnt remove it on unmount cycle.
You can see this on screenshot from this example

Having same issue @Coobaha have you found a fix or a way around it?

We're going to look into this shortly. Close for now.
We're also looking for maintainers. Involve in #266 to help strengthen our community!


[Doc] Add sample requests to create secondary userstores with UserStoreConfigAdminService

'addUserStore()' section in 'Managing User Stores with APIs' doc [1] explains about how to add a userstore with a sample request, input parameters and etc. But the provided content needs to be update with complete set of parameters (specially parameters such as username, password, drivername) that will required to create a secondary userstore. Please update the documentation with sample requests and input parameters for following user stores,

JDBCUserStoreManager
ActiveDirectoryUserStoreManager
ReadOnlyLDAPUserStoreManager
ReadWriteLDAPUserStoreManager
CarbonRemoteUserStoreManager

[1] https://docs.wso2.com/display/IS5xx/Managing+User+Stores+with+APIs#ManagingUserStoreswithAPIs-addUserStore()

Fixed in 5.5.0 - 5.8.0 [1]
[1] https://docs.wso2.com/display/IS550/Managing+User+Stores+with+APIs#2ea45d71a2b140b897c651718b99bf0d


[FEEDBACK] Using webp instead of png and/or use avif | Use woff2 instead of ttf

Heyo!
I saw your lovely project and i'm definitly going to switch using this instead of heimdall.
While setting up, i saw a few none-urgent warnings in the logs when compiling the project.
I did read up on: #63 so i know these aren't even used.
-  Building for production...
WARN  A new version of sass-loader is available. Please upgrade for best experience.
WARNING  Compiled with 2 warnings9:37:02 AM
warning
asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
This can impact web performance.
Assets:
fonts/Raleway-Variable.3b5635ac.ttf (302 KiB)
js/chunk-vendors.a6a3e10c.js (5.64 MiB)
js/dashy.b54b8e16.js (497 KiB)
web-icons/screenshots/dashy-scrsht-1.png (367 KiB)
web-icons/windows10/LargeTile.scale-400.png (307 KiB)
web-icons/windows10/SplashScreen.scale-400.png (307 KiB)
warning
entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.
Entrypoints:
dashy (6.36 MiB)
css/chunk-vendors.d8067ad8.css
js/chunk-vendors.a6a3e10c.js
css/dashy.660d27ff.css
js/dashy.b54b8e16.js

Without any problems, you can use woff2 instead of ttf:
woff2 support: https://caniuse.com/woff2 (95%, rip IE users)
ttf2 support: https://caniuse.com/?search=ttf (97%)
The size would go from: 302kB to 128kB. Ofcours we're talking about small differences so i tottaly understand if you decline this suggestion!
For images the 367kB would go to 188kB or even 54kB for avif. I did notice converting the biggest png image gave a few weird artefacts/coloring issues. I would advice to use this project to convert the images from png to webp/avif (believe i even have a github workflow to use this project to make the alternative images automatically) https://github.com/funbox/optimizt
png (98%)
webp (94% rip ie)
avif 69% (nice) (also rip edge and safari)
For images, i personally would advice to have avif and webp as fallback. Because i'm more a backend guy and only thinkerd with frontend trough statically typing all the html/css. I can only say that using the  would be perfect for this.

Again, i know that my suggestions wouldn't yield megabytes levels, and fixing those warnings isn't effecting Dashy directly. But maybe you can use these suggestions for your other projects :) feel free to instantly close this after reading!

   WOFF 2.0 - Web Open Font Format | Can I use... Support tables for HTML5, CSS3, etc      "ttf" | Can I use... Support tables for HTML5, CSS3, etc       GitHub - funbox/optimizt: CLI image optimization tool   CLI image optimization tool. Contribute to funbox/optimizt development by creating an account on GitHub.

I totally missed this ticket, sorry! These are really good points, and thanks for doing the math here :) I'm going to reopen, and try and get your changes implemented when I have a moment.

sure thing and no worries! I thought i should've closed it after a while, but thank you so much :) <3


i.MX RT1060 support

Is there any plan to expand the support to other NXP boards?
In particular I'm interested in the 1060, which has very similar specs to the 1050, already supported.

Not at the moment. Let's keep this open and see what kind of traction we get from the community.
If this is a commercial application you can contact Picovoice directly and they might be willing to add the support as part of the engagement.

closing due to insufficient traction


Build failure when including a node_module in main.scss

Submit a feature request or bug report

[x] I've read the guidelines for Contributing to Roots Projects
[x] This is a bug report
[x] This request isn't a duplicate of an existing issue
[x] I've read the docs and NPM Debugging Guidelines post and followed them (if applicable)
[x] This is not a personal support request that should be posted on the Roots Discourse forums

What is the current behavior?
When I am including a node module in main.scss that references another installed module the build fails (I was attempting to install Material Design Web Components).
What is the expected or desired behavior?
Any module that is called should be able to include other installed in the node_modules folder if required.

Bug report
Please provide steps to reproduce, including full log output:

After theme setup install material components: yarn add material-components-web

In resources/assets/styles/main.scss reference the sass file by adding the following line: @import "~material-components-web/material-components-web";

Run yarn start and the build will fail

Please describe your local environment:
WordPress version: 4.8.2
OS: MAC OS High Sierra 10.13
NPM/Node version: NPM: 5.4.2 | Node: 8.1.2
Where did the bug happen? Development or remote servers?
Dev server locally

Other relevant information:
X

looks like you'll need to modify the webpack config, see https://github.com/material-components/material-components-web/issues/351
please use https://discourse.roots.io/ for further support


Why all models in multiclass mode failed to persist and raise ValueError: multiclass format is not supported

I have trained models on multiclass mode as below:

But in training stage, all models crashed due to the mentioned error. What I did wrong here? I can not find any example about multiclass problems so I try it myself.

I think it's because you set the evaluation metric = roc_auc which is only supported for binary not multiclass classification. Does your code work if you omit the eval_metric argument?

Yes, I remove 'roc_auc' and the code works well. Thanks for suggestion


 in logback-spring.xml not working with StackDriverJsonLayout

@meltsufin
Im trying to add a custom json to get some custom data added in the console appender with StackDriverJsonLayout. Below is the code from logback-spring.xml:
<if condition='property("LOG_APPENDER").contains("CONSOLE_IN_SD_FORMAT")'>
				<then>
					<encoder
						class="ch.qos.logback.core.encoder.LayoutWrappingEncoder">
					<layout
						class="org.springframework.cloud.gcp.logging.StackdriverJsonLayout">
						<customJson>{"product_name":"${PRODUCT_NAME}"</customJson>
					</layout>
					</encoder>
				</then>
				<else>
				    <encoder>
							<pattern>${LOG_PATTERN}</pattern>
							<charset>utf8</charset>
					</encoder>
				</else>
			</if>

This works in local. But when the application is deployed to GKE and the console logs are checked the custom json field does not appear.
I tried to override the addCustomDataToJsonMap method in a custom class which extends the JsonLayout. But how should this custom class be referred in logback-spring.xml. Also please confirm if this step of overriding the addCustomDataToJsonMap method is really required or not.
Please share an example if possible

You shouldn't have to override addCustomDataToJsonMap(). Note that product_name is not supposed to appear as a top level attribute of the LogEntry in the Cloud Logging console. It should appear under jsonPayload. See: https://cloud.google.com/logging/docs/agent/configuration#process-payload.
Can you share the LogEntry as it appears in the Logging console?

Closing due to inactivity from OP.


validation netwerk connectivity

We need to define and validate the network connectivity rules.
Basins need some connector nodetypes inbetween, some non-basin nodetypes are not connectors. Could be related to additional characterization needs such as #199
Some inpiration: a flexible schema definition from the network editor of Ribasim 8:
NetworkValidationRules.zip
http://fews.wldelft.nl/schemas/version1.0/topology-editor-schemas/networkValidationRules.xsd

The place in the code has been reserved :)
https://github.com/Deltares/Ribasim/blob/6a8183d439000c75914113b342a6c07574337af0/core/src/solve.jl#L17-L30

Also include the information network (control)

These will be hardcoded rules in Julia, no external file parsing/generation yet.


Domoticz gives multiple Connection reset messages

Domoticz Log gives multiple random "Connection reset!" messages.

Not sure what the trigger is for it to happen.
The OTGW for sure does not reboot.
Because if that happens the Domoticz Log gives much more info.
Also the OTGW "Nr. Reboots" is on 4 for a long time. (And does not increase.)

I recently installed the new software on the old OTGW.
Alle devices are automaticly found by Domoticz via MQTT.  Monitoring is perfect but control is not. Not able to set the override temperature.
Meanwhile als the error 19 appears about once ever 6 minutes: Connection lost , restarting.

Interesting. Your OTGW should not just reset. Have you looked at the reset count in the webUI? It might just be a mqtt reconnect.

Robert, no reboot but disconnect <> connect which imho is something else.

The firmware does auto reconnect when it loses the MQTT hub connection for any reason. The 6 minute cycle is too constant to be random, so it must be something else.
However I do not run Domoticz. So I cannot recreate this issue.
Closing unless someone help to reproduce.


Reload account list after an account is created

Currently the accounts are not reloaded automatically.

I think this is fixed


User creation and permissions - Readme?

I'm creating an API for a mobile app and would like to know the logic behind the currently implemented user model.
I'd like to know a little bit more about the intended user registration flow of the boilerplate code.
From my perspective I have two options:

Create an admin user for the mobile app to be able to create users. (Then I'd have to store some credentials on the mobile device, not ideal)
Make the registration endpoint public and allow users to create themselves.

The user's endpoints of the boilerplate are just an example.
In an API for a mobile app you should make the registration endpoint public, so anyone can register.
The flow is simple:

User send email and password
You try to save the user credentials into the database
The pre-save event of mongoose is triggered, so it can hash the password before saving into the database for security reasons
The api generates a JWT access token and returns it

user model in this example allows saving duplicates emails, is it normal?

user model allows saving duplicates emails, is it normal?

src/api/user/controller.js:
export const create = ({ bodymen: { body } }, res, next) =>
  User.create(body)
    .then((user) => user.view(true))
    .then(success(res, 201))
    .catch((err) => {
      /* istanbul ignore else */
      if (err.name === 'MongoError' && err.code === 11000) {
        res.status(409).json({
          valid: false,
          param: 'email',
          message: 'Email taken'
        })
      } else {
        next(err)
      }
    })

src/api/user/index.js:
router.post('/',
  body({ email, password, name, picture, role }),
  create)

Should be enough.


new /ui path broken in HA mode: non-leaders must not follow 302s from leader

Reproduction steps

Start Marathon cluster of 3 instances
Request / from the leader, you'll get a redirect to /ui:

air@megatron:~/dev$ curl -i http://srv2.foo.com:8080/
HTTP/1.1 302 Found
Date: Wed, 09 Sep 2015 23:09:03 GMT
X-Marathon-Leader: http://srv2.foo.com:8080
Access-Control-Allow-Origin: *
Access-Control-Allow-Methods: GET, HEAD, OPTIONS
Access-Control-Max-Age: 86400
Cache-Control: no-cache, no-store, must-revalidate
Pragma: no-cache
Expires: 0
Location: http://srv2.foo.com:8080/ui/
Content-Length: 0
Server: Jetty(9.3.z-SNAPSHOT)

Now try requesting the same from a non-leader. The request is forwarded to the leader, but the 302 from the leader seems interpreted instead of being relayed back as-is to the browser:

air@megatron:~/dev$ curl -i http://srv3.foo.com:8080/
HTTP/1.1 200 OK
Date: Wed, 09 Sep 2015 23:10:04 GMT
Server: Jetty(9.3.z-SNAPSHOT)
Cache-Control: no-cache, no-store, must-revalidate
Access-Control-Allow-Origin: *
Access-Control-Allow-Methods: GET, HEAD, OPTIONS
X-Marathon-Leader: http://srv2.foo.com:8080
Expires: 0
Pragma: no-cache
Access-Control-Max-Age: 86400
Date: Wed, 09 Sep 2015 23:10:04 GMT
Content-Type: text/html
X-Marathon-Via: 1.1 srv3.foo.com:8080
Content-Length: 395

<!DOCTYPE html>

    <meta charset="utf-8">
    <title>Marathon</title>
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <link rel="icon" type="image/x-icon" href="img/marathon-favicon.ico">
    <link rel="stylesheet" href="main.css">

Second action, can we add a regression test for this. e.g. Stand up Marathon in HA mode (N instances) and verify that leaders/non-leaders return the same results.


Plan rights model

Still a work in progress.
We want to have the option to parse plan instruction pdf and store associated data.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: Tomer Laor:x: boris-pTomer Laor 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.


COVID Figures: Light Level

Similar to this issue, we want to see how light levels might have changed because participants started waking up earlier or later before and after Spring Break.
Figure
The ideas for this figure are similar to the sleep figures:

Individual and population-level histogram that shows the minute difference when morning and evening light levels hit 50 lux or some other threshold.
Individual-level heatmap that shows the light measurements for each day stacked on each other so you can see how the sleep schedule changed.

Started in beacon summary notebook

Light levels not a reliable measure


[0.8] Binding to "id" is not working

in Polymer 0.8 RC4,
Binding to "id" (input) and "for" (label) is not working:
Please check the below example: (click HTML tab)
http://howjs.com/sTrT/2
Thanks.

Interesting case.
I managed to get this working by using attribute binding for id (id$="{{uuid}}"), and property binding for the for value (html-for="{{uuid}}"). This appears to be contrary to our advice in the docs (in the case of html-for), so I'm going to ask @kevinpschaaf or @sorvell to weigh in on whether this is working as intended.
(See the "Some attributes are special" note in https://www.polymer-project.org/0.8/docs/devguide/data-binding.html#property-binding -- that's the doc that appears to be wrong, because it advises using for$ (which doesn't seem to work) rather than html-for (which does).
I also initialized the value for uuid in the properties object. I'm not sure whether that's required, but it seemed a little cleaner.
Note that binding to id like this appears to prevent you from using this.$ to access local DOM elements, which may be a consideration. Nesting the input inside the label would be a simpler approach
(but may not give you as much flexibility):
    <label>
        <input id="input" type="radio" name="{{name}}" title="{{title}}" value="{{value}}" checked="{{checked}}">
        <content></content>
    </label>

Working version using data binding on id:
http://plnkr.co/edit/MtYDzf6IgI1P4kVULMmZ?p=preview
Using composition instead:
http://plnkr.co/edit/j31xhq802EGT9I0CPqzj?p=preview
(BTW, I'd never tried howjs before. It went crazy when I tried to open devtools, so I moved the sample to Plunker.)

Thanks arthurevans. That's interesting indeed.
I haven't seen "html-for" before. Is that a Polymer's attribute or HTML's ? I have tried google but couldn't find it online. Could you ping me the Doc for it ?
Anyway, it'd be great to have for$ binding fixed.

htmlFor is a standard property on HTMLLabelElement that reflects the for attribute:
https://developer.mozilla.org/en-US/docs/Web/API/HTMLLabelElement
Because of the way we map attribute names to property names, you can bind the htmlFor property using html-for="{{value}}".
https://www.polymer-project.org/0.8/docs/devguide/properties.html#property-name-mapping

Fixed in https://github.com/Polymer/polymer/commit/12513df2e19b457e25167750893cc379d5dab7dd.


modulus with policy and primitive test

@saeb-faraji-gargari I added minor comments to your code. It starts to look fine. Just a few edits and the PR is good to go.


Update toggl-track extension

Description
Add CMD + D shortcut to Create Similar Time Entry
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

looks good to me!


The WORLD_SIZE environment variable for elastic policy is not getting set correctly

According to this PR the nProcPerNode in elastic policy is deprecated and it is suggested to use   nprocPerNode in the spec.
However after trying the above, the world size is getting set to the number of replicas when using elastic mode and not replicas * nprocs_per_node.
The following yaml is not working, instead the commented line , when uncommented works.
apiVersion: "kubeflow.org/v1"
kind: "PyTorchJob"
metadata:
  name: "torchrun-test"
  namespace: training
spec:
  nprocPerNode: "2"
  elasticPolicy:
    rdzvBackend: c10d
    # nProcPerNode: 2
  pytorchReplicaSpecs:
    worker:
      replicas: 2
      restartPolicy: OnFailure
      template:
        metadata:
          labels:
            kind: "deepanker2"
          annotations:
            test_vesrion: "1"
            sidecar.istio.io/inject: "false"
        spec:
          containers:
            - name: pytorch
              image: quay.io/deepanker_gupta/kubeflow_training:torchrun
              imagePullPolicy: Always
              args: ["torchrun","/workspace/exp/training.py"]
              resources:
                limits:
                  nvidia.com/gpu: 2
                  cpu: 10
                  memory: '10Gi'

@kuizhiqing @tenzen-y

We can close this? @kuikuikuizzZ

We can close this? @kuizhiqing

Yes, I think so. @deepanker13 Can you confirm with the fix.

If the uncommented yaml works as you said, it should work with my fix. I've confirm the equivalence of the two version.
I suggest to verify it with either following way, first of all, pull the latest code with my fix, then

rebuild an image to update training-operator deployment
or delete the deployment(set the replicas to 0 is OK), build the binary locally with go build -o bin/manager cmd/training-operator.v1/main.go, if your kubectl is work(or you should set KUBECONFIG), run the bin/manager locally
finally, , restart your job.

@kuizhiqing I am getting the following error in the training operator pod when I rebuild the image.
deepanker@W3X1P2XTV2 ~ % kubectl logs training-operator-7475584c5c-fl4c9 -n kubeflow
exec /manager: exec format error

Yes, it is working as expected. Thanks @kuizhiqing , we can close this issue.
Ps - Building image using golang 1.17+ on M1 chip Mac is failing when target platform is set to linux/amd64. I had to build the image on a linux VM.

Yes, it is working as expected. Thanks @kuizhiqing , we can close this issue.
Ps - Building image using golang 1.17+ on M1 chip Mac is failing when target platform is set to linux/amd64. I had to build the image on a linux VM.


OPENUI5 Diagnostic popup opens while typing polish letter Ś

OpenUI5 version: 1.44.6
Browser/version (+device/version): Chrome 55.0.2883.87
Any other tested browsers/devices(OK/FAIL):
IE 11 (11.321.14393.0) - FAIL
FireFox 51.0.1 - FAIL
URL (minimal example if possible): https://openui5.hana.ondemand.com/#content/Controls/index.html
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:

Change your keyboard's settings to Polish (programmer's)
Select any input field
Press LShift + RAlt + S
Letter Ś should appear, as well as sapui5 diagnostic dialog.

What is the expected result?
Sapui5 diagnostic dialog should not be visible (documented shortcut contains also ctrl key)
What happens instead?
Any other information? (attach screenshot if possible)

Hi @webmajstr , thanks for reporting the issue we're gonna have a look at it, an internal incident has been opened with id :<PHONE_NUMBER>
Thanks,
Ted.

Hi @webmajstr,
we limited the key combination for opening technical info to "ctrl" + "shift" + "left alt" + s. The change is already merged into master and will be available from nightly build from tomorrow.
Do you need the change to be downported to 1.46 and 1.44?
Best regards,
Jiawei


ability to restart automatically right after a crash.

this is gonna be something related to linux since the process will die off, and it's not gonna restart via python...

I did it in a batch file

update: bot crashes and the startup command doesn't work. this issue also affects Ruby Rose, although it has a boot loop command and I don't. Either way, same effect.

It's been done, after a few months.


MoveFile not working and also not returning error

FTP Server OS:
Unknown
*FTP Server Type:
Unknown (3rd party integration)
Client Computer OS:
Windows
FluentFTP Version:
48.0.3
Framework:
.NET 6
When performing call MoveFile, the file is not being moved yet no error or exception happens in the C# code. We are converting an application to .NET 6 and this is currently blocking us until we get this figured out.  The file is 1.5mb's in size
  _ftpClient.MoveFile(srcFile, destFile);

Logs :
I will attempt to get these.

Looks like using the synchronous ftpclient inside an async handler is what is causing the ftpclient issue because ftpclient is being disposed.


Fix GH actions

Fixes #73

ah! good catch-- let's wait for the CI to complete

ah! good catch-- let's wait for the CI to complete

It works :D

It works :D

@alexfoias can you fix it for the single subj as well?

@alexfoias can you fix it for the single subj as well?

done here: https://github.com/spine-generic/data-single-subject/pull/20

done here: https://github.com/spine-generic/data-single-subject/pull/20


Add third-party plugin for jq

Adds my plugin for jq to the third-party plugins.

@A-D-E-A Just need to fix the format.


Validate proxy ip only if using BIND type

solve #196
When using proxies in docker container, feature to using hostname for http proxy will be needed.
Ipv4 validation seems to be effective only for BIND ips configuration.
So I moved it.
How about this?

This is also required for some proxy providers such as trusted proxies who provide a hostname to connect to their services.

Thanks for the PR, but not sure it will work out of the box, I don't know if the proxy host resolution will be done by the http client, it is possible it expect just an IP. I have to check that.


Error while geocoding

It was working, but now I'm receiving this error:
message: Error from the server while geocoding. The received datas are in the error's 'origin' field. Check it for more informations.
  code: 4
  origin:Object
    plus_code:Object
      global_code:"4CW22222+22"
  results:Array(0)
    length:0
  status:"ZERO_RESULTS"

Perhaps it's an error on Google's side?

5 seconds after submiting this issue (I've been trying to figure it out for almost 2 hours) I realized that the emulator had some random stupid coords that were actually in the middle of the ocean... lol
No problems at all. I apologize.

Hi @mdelclaro
How do you solve this?
I'm running in my iOS phone and also end up in the middle of ocean, but the code itself is working in Android.

@darvinhuangg are you running on the emulator or a physical device? I was running on the Android Studio emulator and there you can change the device's location. I'm not sure about iOS, sorry...

@darvinhuangg are you running on the emulator or a physical device? I was running on the Android Studio emulator and there you can change the device's location. I'm not sure about iOS, sorry...

I'm running both platform on physical device. I've also checked the location in my iOS phone using maps and it gets exact coordinate location.


Allow import language.experimental.captureChecking in the REPL

Attempt to fix #16250 with @rjolly during the issue spree.
Two things currently seem to prevent import language.experimental.captureChecking to be used in the REPL.
1. needsCaptureChecking flag preservation across compilation units
The needsCaptureChecking is defined here:
https://github.com/scala/scala3/blob/b5941e4c55c45b8e3ba2f23ef9e82dfc76629a2f/compiler/src/dotty/tools/dotc/CompilationUnit.scala#L57
And is set from handleGlobalLanguageImport:
https://github.com/scala/scala3/blob/b5941e4c55c45b8e3ba2f23ef9e82dfc76629a2f/compiler/src/dotty/tools/dotc/config/Feature.scala#L153
To preserve it, we tried calling handleGlobalLanguageImport in setImportInfo (4702968c5ea1c29cc6b07b0f49c06e9e526cdae8).
2. outermost syntax errror
Currently, a syntax error is emitted if the import is not at the top level:
https://github.com/scala/scala3/blob/b5941e4c55c45b8e3ba2f23ef9e82dfc76629a2f/compiler/src/dotty/tools/dotc/parsing/Parsers.scala#L3312-L3313
Outermost is originally set here:
https://github.com/scala/scala3/blob/b5941e4c55c45b8e3ba2f23ef9e82dfc76629a2f/compiler/src/dotty/tools/dotc/parsing/Parsers.scala#L764
We are not sure yet if and how we should special-case this flag so that it works with wrapper objects generated by the REPL.

Closing in favor of the new #21725.


feat: Updating badges to not be tags

Summary of Changes
Updated queries to get Badges rather than Tags with badge type on neo4j.
Tests
N/A
Documentation
N/A
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test

lgtm, but I now recall that we have csv file for quick start to show the badge usage ( https://github.com/amundsen-io/amundsendatabuilder/blob/1e01e8c7300088619c8f19f25945eddc5d9433ec/example/sample_data/sample_tags.csv) . I think we need to update that quick start file otherwise it will be broken. It could be done in a different pr, but just FYI.


Feat multiple slack notifier

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

93 of 153 (60.78%)  changed or added relevant lines in 10 files are covered.
20 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.4%) to 73.959%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

plugins/providers/dataplex/client.go
0
4
0.0%

domain/provider.go
0
6
0.0%

plugins/notifiers/client.go
35
43
81.4%

plugins/providers/bigquery/provider.go
14
22
63.64%

internal/store/postgres/store.go
6
16
37.5%

plugins/notifiers/slack/client.go
34
58
58.62%

Files with Coverage Reduction
New Missed Lines
%

plugins/providers/dataplex/client.go
1
3.7%

plugins/notifiers/slack/client.go
19
54.09%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
8671

Relevant Lines:
11724

💛  - Coveralls


Update Vercel deployment guide

The current guide for Vercel deployment contains outdated build commands. I've updated them to their proper versions I suppose.
However it would be great to have more complete guide on how to deploy the entire storefront + checkout stack from start to the end

Superseded by https://github.com/saleor/react-storefront/pull/391


Add getParent method to BranchDataProvider interface

Previously, the Resource Groups Tree Data Provider was caching child -> parent relationships for all items so it could do two things:

generate a path based id for an item and attach it to the vscode.TreeItem in getTreeItem.
implement getParent

This change breaks up this responsibility so that whatever ___DataProvider owns the item is also responsible for implementing getParent and generating TreeItem ids. Meaning the RGs TDP is responsible for caching the base items subscription items, grouping items, and application resource items). And any items beneath application resources in the tree will need to be cached (or not) by their respective BranchDataProvider.
To accomplish this, I've written a CachedBranchDataProvider class, which wraps BranchDataProvider and implements getParent and sets TreeItem.id by caching getChildren results.
In the future, client extensions can share an implementation of a cache that allows for similar features that AzExtTreeDataProvider had.

I wasn't super happy with how these changes had to be implemented due to how we were handling caching in RGs. So in #434 I simplified the cache, and I've re-implemented the changes (the changes are a lot smaller now) in #435.


Converts the Manage Operational Layers settings to a popover

@philium I fixed the bug on this you sent in slack.

@pgruenler Can you re-review when you get a chance?


docs(i18n-id): translate downlevelIteration.md and several config into Indonesia

Overview
this is part of #938
Changes
translate the following files

[x] downlevelIteration.md

LGTM


Change span name for AspNetCoreDiagnosticObserver

Changes the span name for the diagnostic observer to match OpenTelemetry pattern "HTTP <METHOD>" if no "friendly" path is available.
There were two previous PRs that used only "<METHOD>", I will revisit those to use this OTel pattern instead.

Also found other options here: https://github.com/VerifyTests/Verify#snapshot-management


typo fix:in iocAPI externalisable->externalizable

fix - issue #184
typo fix: externalisable->externalizable
Under API in iocAPI.

@chaitanyassr i think it is fine with 's' please have a look @justion_o

We have contributors from various parts through out the world, so we get documentation and etc with different localized spellings. I don't think we've declared a standardization on which locale to use for spelling. So at this point, I don't think it's something that needs to be changed, but maybe something to think about for the future.

We have contributors from various parts through out the world, so we get documentation and etc with different localized spellings. I don't think we've declared a standardization on which locale to use for spelling. So at this point, I don't think it's something that needs to be changed, but maybe something to think about for the future.

Okay no problem, I would be closing down this PR for now.


fix: do not set pinnedframeurl when initializing group object

If the group was created before this property was added, we will throw an error. Disabling setting the pinned frame until we potentially update libxmtp to return an empty string when the field is missing.

You could probably just remove it entirely from that wrapper. Unless you want it there with a comment so future us knows why 😅

@nplasterer Think we might re-add it after this change in libxmtp - https://github.com/xmtp/libxmtp/issues/936


[COST-4417] Include token URL for stage override

Update token url override env in clowder

Same test instructions as https://github.com/project-koku/nise-populator/pull/91

/retest


Searching for T#'s doesn't give results

On Chrome

Hi @clr2of8 , you are not seeing any results because you have no matrix selected on the far right (Enterprise, ICS, Mobile). Once you select Enterprise, the technique will show up.
This is a good catch, though, and we can add a feature request to the backlog to have a default of Enterprise when no matrices are selected.


Add FreeBSD download URLs

This does not currently work because the Makefile it calls requires GNU make (ie. it does not work with BSD make). We can fix this by having FreeBSD call gmake instead of make, but I haven't figured out how to do that yet (bash is not my specialty).
So if anybody would like to do that or help me with it, we can get FreeBSD support happening!

cabal-install downloads are missing afais

There is already one for <IP_ADDRESS>, which I successfully used.

We can fix this by having FreeBSD call gmake instead of make, but I haven't figured out how to do that yet (bash is not my specialty).

You can try the following
# @FUNCTION: emake
# @USAGE: [arguments]
# @DESCRIPTION:
# Wrapper around 'make', may call 'gmake' if it exists.
emake() {
	if command_exists gmake ; then
		gmake "$@"
	else
		make "$@"
	fi
}

And then replace all our make calls in ghcup with emake.

I've marked this "ready for review". Note that the binaries don't work very well on FreeBSD12 (which I'm running). @hvr is trying to find the relevant issue, which relates to ABI differences between FreeBSD 11 and 12

IIrc the ABI issues are due to  https://lists.freebsd.org/pipermail/freebsd-fs/2017-April/024684.html which affects syscalls such as stat(2) at the ABI level. Basically we need to be mindful to use the appropriate bindists for fbsd11 and fbsd12 respectively

Perhaps I should mark these as freebsd=11 rather than just freebsd. Thoughts?

Perhaps I should mark these as freebsd=11 rather than just freebsd. Thoughts?

That would enable us to merge the current PR I guess.

It does now! On FreeBSD 11, I built and ran tests of lens on all major ghc releases that ghcup supports. The tests all pass except for the doctests.
For 8.0.2, I've put FreeBSD 9 or 10 or 11, because the release notes claim that build will work on those versions. For later versions I've been more conservative.

By the way, I can smash this all into one or two commits if you'd prefer.


Migrate to poetry

Why Poetry?  Namely so that you don't have to maintain 60+ individual dependencies when the project only directly uses 6.
How to use Poetry
Install/Docs: https://python-poetry.org/docs/
Run project: poetry run streamlit run Home.py --server.enableCORS false --server.enableXsrfProtection false
How to bump versions: poetry update
Major updates for versions, modify the version number in pyproject.toml when needed (for example 1.39 to 2.0).  Generally this can be done every 6-12 months (or as needed for supply chain vulnerabilities).  Typically a version bump (using poetry update), letting Poetry handle figuring out compatible versioning is more than enough.
Lock file (with all dependency chains) is included to install specific versions when built (through the devcontainer.json). (So deployments use the locked versions, locally you can test bumped versions.)
If using an IDE with Poetry support (such as PyCharm), an interpreter/environment can be configured that will automagically sort it out (and use the Poetry commands for you if you add/change dependencies).
I've tested this locally, no issues other than a stack trace in Post2002.py, likely due to me running on a Windows box.  Easily fixed by specifying an encoding (shouldn't cause any problems with *nix as UTF-8 is quite standard).  I've included this fix in this PR as well.

One other thought on using Poetry, if you want to publish this as a package to PyPi (to let other folks use this), Poetry makes that a whole lot easier. :)


Add transforms for pretrained models in pytorchvideo

Signed-off-by: Jael Gu<EMAIL_ADDRESS>Add utils to preprocess video with transforms, configs are setup in script by model name.
Apply this utils to get video data ready for pre-trained models in torch hub:

slow_r50
slowfast_r50
slowfast_r101
x3d_xs
x3d_s
x3d_m

Codecov Report

Merging #829 (2044d79) into main (af732b8) will increase coverage by 0.28%.
The diff coverage is 100.00%.

:exclamation: Current head 2044d79 differs from pull request most recent head 3ebaca9. Consider uploading reports for the commit 3ebaca9 to get more accurate results

@@            Coverage Diff             @@
##             main     #829      +/-   ##
==========================================
+ Coverage   71.08%   71.37%   +0.28%
==========================================
  Files         175      181       +6
  Lines        7938     8063     +125
  Branches     1303     1319      +16
==========================================
+ Hits         5643     5755     +112
- Misses       1908     1912       +4
- Partials      387      396       +9

Impacted Files
Coverage Δ

towhee/models/utils/video_transforms.py
100.00% <100.00%> (ø)

towhee/functional/option.py
77.77% <0.00%> (ø)

towhee/models/layers/swin_transformer_block3d.py
83.07% <0.00%> (ø)

towhee/models/utils/window_reverse.py
100.00% <0.00%> (ø)

towhee/models/utils/get_window_size.py
41.66% <0.00%> (ø)

towhee/models/utils/window_partition.py
100.00% <0.00%> (ø)

towhee/functional/mixins/__init__.py
100.00% <0.00%> (ø)

towhee/functional/data_collection.py
92.77% <0.00%> (+2.56%)
:arrow_up:

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 af732b8...3ebaca9. Read the comment docs.

/lgtm
/approve


don't prefix unshared class initializer parameters with "_$"

Currently makePrivate() prepends _$ to avoid conflicts with Ceylon identifiers that already begin with _. Instead, we should:

Not use makePrivate() on class initializer parameters, and
Prefix Ceylon parameters that begin with _ with $, since Dart parameters should begin with _.

Test case:
shared class AvoidNonUniqueNames(String myString, shared String _myString) {}

Done. Dart constructors parameters for Ceylon class initializer parameters now resemble the Ceylon identifiers.


Process data from MQTT

On the server side we need am MQTT client to process data coming from the satellite devices via MQTT
Requirements:

[x] Connect to a MQTT broker using TLS connection with username and password
[x] Subscribe to MQTT topics
[x] Read data from subscribed topics
[x] Write data to subscribed topics
[x] Multi-threaded operation to avoid the blocking loop

Test cases need update with the new disconnect function.

Test cases need update with the new disconnect function.


Significant camera stutter on mouse left click

Godot version
3.5-stable, 3.4.5-stable, 3.3-stable, 4.0 beta 2
System information
macOS Monterey Version 12.5, 13-inch M1 2020
Issue description
The camera stutters noticeably when clicking mouse left click.
Example video 1 (from minimal reproducible example):
https://user-images.githubusercontent.com/34614301/190055892-e9b1fc1f-e6f1-45e5-8a91-22c5a7363409.mov
Example video 2 (from First Person Starter template):
https://user-images.githubusercontent.com/34614301/190056250-57988ac6-b80a-44a3-9e3b-3d68402f3fb7.mov
I now notice in the first video that my cursor also appears to be stuttering on left click (and the camera movement is controlled by InputEventMouseMotion), if that is any hint at what the issue might be.
I'm guessing that this is somehow specific to my environment/hardware, as it presumably would have been reported by now with how common this camera style is.
Steps to reproduce

Download either the minimum reproducible example or First Person Starter template above.
Look around, while clicking left click.

Minimal reproduction project
godot-left-click-freeze.zip

Is your mouse one of the fancy gaming mice that poll at very high Hz, like 100 or 120 Hz? If your monitor 120 Hz or more? I seem to recall similar issues being reported...

Hi yes I have a Razer Mamba Elite mouse which per online specs looks to have a 1000hz refresh rate. And no, my monitor on my mac only goes up to 100hz, where I see this same behavior as I did on 60hz. I do have a Windows machine I could set up and test it on, where my monitor should be able to reach 144hz, if you think that would be useful to try out.
Also, if it useful, this behavior only occurs on left click. Right click, and every other mouse button doesn't cause any stutter at all.

This is likely caused by trackpad palm rejection, which seems to be enabled on mouse input for some reason. Look into disabling it in the macOS system settings if possible.

Darn, after going through all (generally unanswered) threads on "disable mac palm rejection" that I could find, all of the exposed trackpad/mouse settings in System Preferences, and all the trackpad related plist options in the defaults configuration, I have found no indication that a setting exists that can configure it.
FWIW I did try enabling "Ignore built-in trackpad when mouse or wireless trackpad is present", and the stutter behavior still exists.

Note: I'm in the process of migrating my project, and this behavior is still present in Godot 4.0 beta 2.

Does this occur with other mouse buttons, and when V-Sync is disabled in the project settings?

Hi thanks for the suggestions. Issue persists with V-Sync both enabled/disabled, as well as with and without set_use_accumulated_input(false).
This only happens with the Left mouse button. RMB, Scroll wheel presses, and side buttons have no visible effect for me.
Also maybe worth noting here that I'm on forward_plus rendering method.


Generate minified versions

Would it be possible to provide minified versions of combined files (in addition to existing ones in case some people want readable files)? Such as Items.min.json.
I believe there are many libraries that can do it, or a manual tab/space/linebreak remover is probably fine too.

i see what ya mean, ill look into it and see what i can do

alright, i just pushed a change that implements minified versions. src/combined contains the json file and the minified version, similar to what you suggested. bin/minify.sh generates the minified version and then saves the minified version to the module, then the program does what it needs to do etc.

ill leave the issue open for a bit to see if ya have any more ideas or suggestions regarding minifying the files


ASCII versus Binary detection and fallback

An outstanding issue is some corruption/errors in an ASCII STL file will cause numpy-stl to fallback to a binary load, which can lead to the MAX_COUNT overrun exception. The ASCII STL errors that cause this include:

An extra vertex in a facet
A missing vertex in a facet

Strictly speaking, the smallest complete ASCII STL file is, I believe, 186 bytes. This file would include the following keywords:
solid facet normal outer loop vertex endloop endfacet endsolid
along with the normal and vertex values.
I think that the following decision tree should be included, for a strict ASCII/Binary determination:

File includes all keywords from above in standard UTF-8 ASCII outside of the header/solid name

ASCII

File includes some but not all of the above keywords

Corruption warning, or perhaps some extra checks.

File size < 196 bytes

Extra validation, or default to Binary

Else

Binary

There would be no fallback to Binary after an ASCII parsing error. While it is technically true that a Binary file could contain all of the above keywords, the odds are low that it would be a valid STL file (I haven't checked this) and that this would occur from an actual STL file not designed to break numpy-stl.
We can then remove the MAX_COUNT arbitrary cutoff. The loading process, I think, would be greatly simplified, and some better exceptions can be thrown (more descriptive; can be recoverable/unrecoverable; can add a loading mode of "strict"/"flexible"; etc.).
As an example, PyCam did it this way when they implemented their own STL loader.

It's been a while, but I believe this might be fixed now.
It should be noted that as long as your file handle is seekable (which most file handlers except for stdin are), it can still recover. So I think it should be versatile enough to fall back now since it defaults to ascii.


NewMeetingRecordingExpirationDays Example

Added one more example to show how an admin can get current TMR expiration date
Related PR:
https://github.com/MicrosoftDocs/office-docs-powershell/pull/8455
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/pull/8009

Docs Build status updates of commit 6efcc0b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit d06bbde:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@get-itips the reason being hesitant on adding markdown is to make sure these examples look good not only on the web, but also in command prompt

Last directive about that is that it needs to be present.

Docs Build status updates of commit 07c9570:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hey @dariomws , do we really need to surround cmdlets in markdown? I have tried examples provides by @get-itips https://docs.microsoft.com/en-us/powershell/scripting/community/contributing/powershell-style-guide?view=powershell-7.1#executable-examples and it looks like copy does not work really well. It copies everything including header

Is it allowing you to do this?

@tiburd Please merge this PR. Thanks


setSwipeOffset and the delegate

Is there any way of not triggering the willStart/didEnd delegate callbacks when using setSwipeOffset? I need to tell if the user initiated the swipe or it's the programmatic setSwipeOffset(). Any pointers to doing this?

Create a property that you set to true when you set the swipe offset. Then check if that property is true/false on the delegate.


Chopsticks support

related to #394

Submission checklist:
Layout

[ ] Change inspected in the desktop web ui
[ ] Change inspected in the mobile web ui

Compatibility

[ ] Functionality of change validated with a connected account with multisig
[ ] Applicable elements hidden / disabled for watched multisigs / pure
[ ] Looks good for solo multisig
[ ] Looks good for multisig with proxy

Chopsticks needs arrow squid, and I've been making changes in both branches. Merging this into the underlying one to avoid headaches.


Implement conservation of mass

https://darwin3.readthedocs.io/en/latest/phys_pkgs/darwin_cons.html

Focus mass tracer on parameters which are conserved (i.e. NOT C)


Apply headline capitalization on front page

Experimental PR, might not merge

Doesn't look good, decided against it


 It's a problem I'm starting to open a dialog with the ocs-ci team about. For now, that specific test lane doesn't offer us much value unfortunately.
Our own functest e2e job is pretty solid though, which at least provides some level of confidence.

@davidvossel Welcome! :)

I'll try not to screw things up too horribly, no promises though :D


This pull request addresses the task assigned to me to add the World Population Analysis project to the repository.

Pull Request: Adding World Population Analysis Project  Description: This pull request addresses the task assigned to me to add the World Population Analysis project to the repository. The project aims to analyze and visualize world population data using Python and Power BI, with a focus on identifying population trends across continents and highlighting the top countries with the highest and lowest populations.  Changes Made: - Added a new section, "World Population Analysis," containing detailed project information. - Created a Power BI Dashboard section to cover dashboard-related tasks and visualizations. - Added an "Import Libraries" section to provide information about necessary libraries. - Included "About Dataset" section with a summary of the dataset used in the analysis. - Added "Exploratory Data Analysis" section to cover initial data exploration. - Included "Data Visualization" section with subsections for various population visualizations.  Contributor: @nasir-hussain-swat  Closes #1

Merge pull request #3 from main
Merged changes:

Added World Population Analysis Project
Updated Power BI Dashboard
Improved Data Visualization

Details:
This merge includes the following changes:

Added the World Population Analysis project to the repository, focusing on analyzing and visualizing world population data.
Updated the Power BI Dashboard section to include more informative visualizations.
Improved data visualization in the Data Visualization section to enhance insights into population trends.

Contributor(s):
@nasir-hussain-swat
@FirstNet-Systems-UK
Reviewers:
@FirstNet-Systems-UK
Closes #1


Task - Give each application and applicant profile a unique url

The main motivation for this is allowing admins to view/edit specific applications and profiles.

Replaced with #3109.
So, after taking a closer at the existing routes, it turns out the applicant profile already has a unique url that a manager can visit (/profile/{applicantId}/about) and though it's impossible for admins to edit an application, they can view applications at the unique url /applications/{applicationId}.
While building the new Application builder, we should remember to index the urls by {applicationId} instead of {jobPosterId}, but for now I think it would be most useful to close this issue in favour of allowing Admins to find the applicant and application ids they need to view existing pages.


how to use it on gcp?

it can be used on aws but not on gcp
info. about the gcp VM: Intel Broadwell x86/64 [debian-11-bullseye-v20230206]
here are the logs
Swap already enabled
Filename                                Type            Size            Used            Priority
/swapfile                               file            8048636         0               -2
total        used        free      shared  buff/cache   available
Mem:           3.8Gi       133Mi       3.4Gi       0.0Ki       363Mi       3.5Gi
Swap:          7.7Gi          0B       7.7Gi
Installing docker
Executing docker install script, commit:<PHONE_NUMBER>7a96caa0a25be56051ff8b726a1b28

sh -c 'apt-get update -qq >/dev/null'
W: An error occurred during the signature verification. The repository is not updated and the previous index files will be used. GPG error: http://deb.debian.org/debian bullseye InRelease: Splitting up /var/lib/apt/lists/deb.debian.org_debian_dists_bullseye_InRelease into data and signature failed
W: An error occurred during the signature verification. The repository is not updated and the previous index files will be used. GPG error: http://deb.debian.org/debian bullseye-updates InRelease: Splitting up /var/lib/apt/lists/deb.debian.org_debian_dists_bullseye-updates_InRelease into data and signature failed
W: An error occurred during the signature verification. The repository is not updated and the previous index files will be used. GPG error: http://deb.debian.org/debian bullseye-backports InRelease: Splitting up /var/lib/apt/lists/deb.debian.org_debian_dists_bullseye-backports_InRelease into data and signature failed
W: An error occurred during the signature verification. The repository is not updated and the previous index files will be used. GPG error: http://packages.cloud.google.com/apt google-compute-engine-bullseye-stable InRelease: Splitting up /var/lib/apt/lists/packages.cloud.google.com_apt_dists_google-compute-engine-bullseye-stable_InRelease into data and signature failed
W: An error occurred during the signature verification. The repository is not updated and the previous index files will be used. GPG error: http://packages.cloud.google.com/apt cloud-sdk-bullseye InRelease: Splitting up /var/lib/apt/lists/packages.cloud.google.com_apt_dists_cloud-sdk-bullseye_InRelease into data and signature failed
W: Failed to fetch http://deb.debian.org/debian/dists/bullseye/InRelease  Splitting up /var/lib/apt/lists/deb.debian.org_debian_dists_bullseye_InRelease into data and signature failed
W: Failed to fetch http://security.debian.org/debian-security/dists/bullseye-security/InRelease  Error writing to file - write (28: No space left on device) [IP: <IP_ADDRESS> 80]
W: Failed to fetch http://deb.debian.org/debian/dists/bullseye-updates/InRelease  Splitting up /var/lib/apt/lists/deb.debian.org_debian_dists_bullseye-updates_InRelease into data and signature failed
W: Failed to fetch http://deb.debian.org/debian/dists/bullseye-backports/InRelease  Splitting up /var/lib/apt/lists/deb.debian.org_debian_dists_bullseye-backports_InRelease into data and signature failed
W: Failed to fetch http://packages.cloud.google.com/apt/dists/google-compute-engine-bullseye-stable/InRelease  Splitting up /var/lib/apt/lists/packages.cloud.google.com_apt_dists_google-compute-engine-bullseye-stable_InRelease into data and signature failed
W: Failed to fetch http://packages.cloud.google.com/apt/dists/cloud-sdk-bullseye/InRelease  Splitting up /var/lib/apt/lists/packages.cloud.google.com_apt_dists_cloud-sdk-bullseye_InRelease into data and signature failed
W: Some index files failed to download. They have been ignored, or old ones used instead.
sh -c 'DEBIAN_FRONTEND=noninteractive apt-get install -y -qq apt-transport-https ca-certificates curl >/dev/null'
E: You don't have enough free space in /var/cache/apt/archives/.
Failed to enable unit: Unit file docker.service does not exist.
Failed to start docker.service: Unit docker.service not found.
docker installation failed, please check your environment

Failed to install docker due to error E: You don't have enough free space in /var/cache/apt/archives/.

Failed to install docker due to error E: You don't have enough free space in /var/cache/apt/archives/.

但这是一个新是实例，我是直接运行交互命令的，在他创建了Swap之后就会填满vm的硬盘空间，所以创建docker会提示空间不足，但是在亚马逊云上就没有这个问题。

Fixed in bb1ec7d0fc826d4bdb49d522a84e4c22ff3d25ae
新创建一台，重新试一下

Fixed in bb1ec7d
新创建一台，重新试一下

done. it works on gcp now.


Add files via upload

Work in progress code for creating the yaml files for the authz policy performance tests.

@yangminzhu


Issue with LDAP Auth: Spaces in LDAP Base DN

I'm having issues getting the LDAP authentication working with my Active Directory server. I think the issue may be that we have a space in our base DN.
OU=User Accounts,DC=ourdomain,DC=com
When Shipyard starts up, it looks like it's not getting the entire base DN that I've configured, it's cut off right at that space.
INFO[0000] shipyard version 3.0.1
INFO[0000] Using LDAP authentication: server=dc.ourdomain.com port=389 basedn='OU=User
INFO[0000] checking database
INFO[0000] controller listening on :8080
WARN[0044] unauthorized request for /containers/json from <IP_ADDRESS>:50722
ERRO[0049] error during login for myself from <IP_ADDRESS>:50722: LDAP Result Code 49
"Invalid Credentials": 80090308: LdapErr: DSID-0C0903C5,
comment: AcceptSecurityContext error, data 52e, v2580

This might just be an issue with the way the data is displayed, but looking at the relevant Go code has me suspecting otherwise. Escaping the space with \ doesn't appear to make a difference.
Thank you!

Hav eyou tried using quotes on the flag?
On Fri, Oct 16, 2015 at 12:26 PM, Christopher Miles <
<EMAIL_ADDRESS>wrote:

I'm having issues getting the LDAP authentication working with my Active
Directory server. I think the issue may be that we have a space in our base
DN.
OU=User Accounts,DC=ourdomain,DC=com
When Shipyard starts up, it looks like it's not getting the entire base DN
that I've configured, it's cut off right at that space.
INFO[0000] shipyard version 3.0.1
INFO[0000] Using LDAP authentication: server=dc.ourdomain.com port=389 basedn='OU=User
INFO[0000] checking database
INFO[0000] controller listening on :8080
WARN[0044] unauthorized request for /containers/json from <IP_ADDRESS>:50722
ERRO[0049] error during login for myself from <IP_ADDRESS>:50722: LDAP Result Code 49
"Invalid Credentials": 80090308: LdapErr: DSID-0C0903C5,
comment: AcceptSecurityContext error, data 52e, v2580
This might just be an issue with the way the data is displayed, but
looking at the relevant Go code has me suspecting otherwise. Escaping the
space with \ doesn't appear to make a difference.
Thank you!
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/637.

Yep, I'm using double quotes on the parameters. I also tried changing it to just "DC=ourdomain,DC=com" thinking it will just search the whole tree (this works for our Nexus server) but that gives the same error. It muddies the water some, perhaps it's an Active Directory issue after all.
We use Nexus and Gitlab, both use AD for authentication but require a domain user's credentials in order to function, in addition to the DC hostname and base DN. Is there any particular switch or setting on the AD side that Shipyard may be expecting?

Hmm shouldn't be. I tested on both AD and OpenLDAP and they are fine. I
will see if I can duplicate. is there anything in the logs of AD that would
give more info?
On Friday, October 16, 2015, Christopher Miles<EMAIL_ADDRESS>wrote:

Yep, I'm using double quotes on the parameters. I also tried changing it
to just "DC=ourdomain,DC=com" thinking it will just search the whole tree
(this works for our Nexus server) but that gives the same error. It muddies
the water some, perhaps it's an Active Directory issue after all.
We use Nexus and Gitlab, both use AD for authentication but require a
domain user's credentials in order to function, in addition to the DC
hostname and base DN. Is there any particular switch or setting on the AD
side that Shipyard may be expecting?
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/637#issuecomment-148876393.

I have the same issue,
curl -sSL https://shipyard-project.com/deploy | SHIPYARD_ARGS="--ldap-server=<IP_ADDRESS> --ldap-base-dn=cn=linux_admins,OU=Grupos Segurança,DC=company,DC=local --ldap-autocreate-users --auth-whitelist-cidr=admin" bash -s
I miss some parameters: LDAP Filters, LDAP BIND User and Password, LDAP, bypass certificate, type of search (sub, one), convert user names to lowercase, attribute of LDAP account(sAMAccountName)...

Ok I don't think this will work but could you try:
curl -sSL https://shipyard-project.com/deploy | SHIPYARD_ARGS="--ldap-server=<IP_ADDRESS> --ldap-base-dn=\"cn=linux_admins,OU=Grupos Segurança,DC=company,DC=local\" --ldap-autocreate-users --auth-whitelist-cidr=admin" bash -s

I executed the command above, but no sucess.
Look the dokcer inspect of container:
 "Cmd": [
            "--debug",
            "server",
            "--listen",
            ":8080",
            "-d",
            "tcp://swarm:3375",
            "--ldap-server=<IP_ADDRESS>",
            "--ldap-base-dn=\"cn=linux_admins,OU=Grupos",
            "Segurança,DC=company,DC=local\"",
            "--ldap-autocreate-users",
            "--auth-whitelist-cidr=admin"

there was a line break.

@darioajr Maybe try escaping the space?
curl -sSL https://shipyard-project.com/deploy | SHIPYARD_ARGS="--ldap-server=<IP_ADDRESS> --ldap-base-dn=\"cn=linux_admins,OU=Grupos\ Segurança,DC=company,DC=local\" --ldap-autocreate-users --auth-whitelist-cidr=admin" bash -s

@pirogoeth Same problem
 "Cmd": [
            "--debug",
            "server",
            "--listen",
            ":8080",
            "-d",
            "tcp://swarm:3375",
            "--ldap-server=<IP_ADDRESS>",
            "--ldap-base-dn=\"cn=linux_admins,OU=Grupos\\",
            "Segurança,DC=company,DC=local\"",
            "--ldap-autocreate-users",
            "--auth-whitelist-cidr=admin"

+1

:thumbsup:

:thumbsup:

Hi,
Ldap does not support anonymous search,How to specify the linux_admins password?


Do not require a profile in "dbt deps" (#2231)

resolves #2231
Description
In "dbt deps", if a profile cannot be found/loaded, use invalid values instead. The generated rendering context for projects in this case will still contain target, but it will be empty.
This works by overriding all the profile-ish elements of RuntimeConfig if a real profile can't be found. If a real profile is found, deps will use a full RuntimeConfig like previously.
Checklist

[X] I have signed the CLA
[X] I have run this code in development and it appears to resolve the stated issue
[X] This PR includes tests, or tests are not required/relevant for this PR
[X] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

how would a user actually trigger the attribute error shown in this diff

I don't think a user could. I think a code change inside dbt could inadvertently do so, and I'm interested in that not happening silently.


Stop longpress propagation?

Hi,
Great project, I just stumbled on it while looking for longpress action implementations.
I basically found 3 and had made one myself years ago, they all look relatively similar but none stop propagation of the mouseclick event.
I tried a few approach like stopImmediatePropagation and preventDefault but I'm probably doing it wrong.
As this repo seems the most up to date project to ship such an action it would be nice to fix it here.
What do you think?
Thanks
simplified reproduction
On release of longpress here, onclick is triggered
 <button
      on:click={() =>  console.log(`Tool Clicked`)}
      on:longpress={() => console.log(`Context Menu Triggered`); }
      use:longpress
      class="tool-icon"
    >
	ToolA
</button>

For completeness here is my version:
import type { Action } from "svelte/action";

interface Attributes {
  "on:long"?: (event: CustomEvent) => void;
  "on:long_released"?: (event: CustomEvent) => void;
}

export const longpress: Action<HTMLElement, number | undefined, Attributes> = (
  node: HTMLElement,
  ms = undefined
) => {
  const defaultMS = 500;
  ms = ms || defaultMS;
  let timeoutPtr: number;
  function handleMouseDown(e: MouseEvent) {
    e.stopImmediatePropagation();

    timeoutPtr = window.setTimeout(() => {
      node.dispatchEvent(new CustomEvent("long"));
    }, ms);
  }

  function handleMouseUp() {
    node.dispatchEvent(new CustomEvent("long_released"));
    window.clearTimeout(timeoutPtr);
  }
  node.addEventListener("mousedown", handleMouseDown);
  node.addEventListener("mouseup", handleMouseUp);

  return {
    update(newMs: number | undefined) {
      window.clearTimeout(timeoutPtr);
      ms = newMs || defaultMS;
    },
    destroy: () => {
      window.clearTimeout(timeoutPtr);
      node.removeEventListener("mousedown", handleMouseDown);
      node.removeEventListener("mouseup", handleMouseUp);
    },
  };
};

I never thought of this use case, but it sounds reasonable to me. I’ve been
(and still am) onboarding at a new employer so probably be a few days /
week till I can get all caught up on my notifications and review items, but
I would very much appreciate a PR and will try to review soon. Adding an
example of how this stops the propagation to the docs would be great too.
My current focus ATM is getting ready to ship the big theme / intl release,
but could see this making the release, or following up quickly afterwards.
On Thu, Jan 25, 2024 at 1:45 PM Mel Massadian @.***>
wrote:

For completeness here is my version:
import type { Action } from "svelte/action";
interface Attributes {
"on:long"?: (event: CustomEvent) => void;
"on:long_released"?: (event: CustomEvent) => void;}
export const longpress: Action<HTMLElement, number | undefined, Attributes> = (
node: HTMLElement,
ms = undefined) => {
const defaultMS = 500;
ms = ms || defaultMS;
let timeoutPtr: number;
function handleMouseDown(e: MouseEvent) {
e.stopImmediatePropagation();
timeoutPtr = window.setTimeout(() => {
  node.dispatchEvent(new CustomEvent("long"));
}, ms);

}
function handleMouseUp() {
node.dispatchEvent(new CustomEvent("long_released"));
window.clearTimeout(timeoutPtr);
}
node.addEventListener("mousedown", handleMouseDown);
node.addEventListener("mouseup", handleMouseUp);
return {
update(newMs: number | undefined) {
window.clearTimeout(timeoutPtr);
ms = newMs || defaultMS;
},
destroy: () => {
window.clearTimeout(timeoutPtr);
node.removeEventListener("mousedown", handleMouseDown);
node.removeEventListener("mouseup", handleMouseUp);
},
};};
—
Reply to this email directly, view it on GitHub
https://github.com/techniq/svelte-ux/issues/221#issuecomment-1910786081,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABLKRBIUAKKHFRAXGL3RJTYQKR47AVCNFSM6AAAAABCK7QSBCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJQG44DMMBYGE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Rename the attribute domain to hostname for in config of Git services

In the Git service-related sections in the .ini config, we are using the attribute domain to obtain the hostnames for Git services. This may not be the most accurate terminology in accordance with RFC 3986 and Python documentation for URL parsing.
Current ini config:
[git_service.github]
domain = github.com

[git_service.gitlab.publicly_hosted]
domain = gitlab.com

It is better to have:
[git_service.github]
hostname = github.com

[git_service.gitlab.publicly_hosted]
hostname = gitlab.com

Note: When using the urlparse/urlsplit function from the urllib.parse package, we get the following:
>>> from urllib.parse import urlsplit
>>> parse_result = urlsplit("https://github.com/owner/repo.git")
>>> parse_result
SplitResult(scheme='https', netloc='github.com', path='/owner/repo.git', query='', fragment='')
>>> parse_result.hostname
'github.com'

Therefore, it is probably best to use the name hostname as it is consistent with the naming of the library.

Closed in #453.


quantize llama lm_head using ParallelLMHead

Usage of habana_next branch is deprecated and we do not accept any further contributions to it. Please target habana_main.


Does not complete for node.js, or, how to set tern config globally?

I'd really love for tern to complete node.js symbols for me because I am seriously tired of having the node docs open in my browser.
I found that Tern has a node plugin, introduced here, but it seems like it's not enabled by tern_for_vim.
How to enable it? Even cooler would be only to enable it if the file is detected as a node.js script.
Edit: Looks like I need to use a .tern-project file to configure this. That will work but I wonder if this is is just too much of a pain. I am starting to write node scripts rather than writing shell scripts, and so they will literally start getting littered across my system, I would not be interested in having to also write little .tern-project files to support them as well. I would really prefer to be able to set a global setting (getting node completions in a browser js file is no big deal. the key is being able to get those completions)

is this still valid?

Yes.


Chore/contact button amendments

Zenhub: 695

@Sepehr-Sobhani there's some conflicts that need to be resolved, can you rebase this?

@Sepehr-Sobhani there's some conflicts that need to be resolved, can you rebase this?

@matthieu-foucault On it


[hi] Add content/hi/docs/tutorials/kubernetes-basics/scale/scale-intro.html

This PR is a part of ongoing hi localization effort here #29353
/language hi

Thanks for suggesting the changes @anubha-v-ardhan

/assign verma-kunal

Hey @anubha-v-ardhan , is this PR good to go? should i squash the commits?

@verma-kunal You can squash the commits.
LGTM

/assign
/lgtm
/approve
Thank you @verma-kunal :)


New highlight function on view pdf

I think it could be useful to add an highlight function on view pdf. Can be added?

Sorry this is a 3rd party tool from pdf.js I suggest raising a ticket there!


adding CRI integration test for containerd

What this PR does / why we need it:
Test the creation, correct configuration on the host, and deletion of a worker pool with CRI - containerd enabled.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Can you also rebase to latest master to make ci/cd happy?


V1.6.0

This PR contains backward incompatible changes
Removed support for Django<4.0;
Removed support for django-smart-media < 0.4.0;
Sorl 12.11.0 has been released and drop support of Django<5.0: https://github.com/jazzband/sorl-thumbnail/blob/master/CHANGES.rst#12110
django-smart-media  V0.4.0 which is a dependency for this project should fix this issue by dropping compatibility with Django < 4.0.

We can not allow this change on Accordion container that would not have any title, i'm going to reject this PR.