freeze requirements for djcelery, django-classification-banner

docker-compose up --build was failing.  Freezing requirements for django-celery and celery to last known stable version fixes the problem.

My suggestion is to freeze requirements for all the dependencies that you
care about, for reproducible builds.
On Tue, Nov 8, 2016 at 9:59 AM, Michael Van Veen<EMAIL_ADDRESS>wrote:

docker-compose up --build was failing. Freezing requirements for
django-celery and celery to last known stable version fixes the problem.
You can view, comment on, or merge this pull request online at:
https://github.com/terranodo/eventkit-cloud/pull/27
Commit Summary

freeze requirements for djcelery, django-classification-banner

File Changes

M requirements.txt
https://github.com/terranodo/eventkit-cloud/pull/27/files#diff-0 (6)

Patch Links:

https://github.com/terranodo/eventkit-cloud/pull/27.patch
https://github.com/terranodo/eventkit-cloud/pull/27.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/terranodo/eventkit-cloud/pull/27, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADW1_2LW8vVF5XycJBs9X4lLOC6qAIoks5q8I5mgaJpZM4KshQc
.

@ingenieroariel Yeah, thanks for that suggestion.  Agree there's some more to be done here around requirements freezing.  I tried to keep this change more or less specifically honed to the error I was experiencing in my build.
@jsvrcek @jj0hns0n : do you guys mind if I go ahead and freeze more of these packages?

Update: synced up with Joe we're gonna freeze all the things :)

/me is memesmerized by the enthusiasm
On Tue, Nov 8, 2016 at 10:28 AM, Michael Van Veen<EMAIL_ADDRESS>wrote:

Update: synced up with Joe we're gonna freeze all the things :)
https://camo.githubusercontent.com/7fc86c63607153f22bd1f3d54b025dc26d965559/68747470733a2f2f63646e2e6d656d652e616d2f63616368652f696e7374616e6365732f666f6c6465723138392f353030782f37323938383138392e6a7067
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/terranodo/eventkit-cloud/pull/27#issuecomment-259167338,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADW1wVya7Q2A0fd-Lzg7uLMGanXGtAlks5q8JUdgaJpZM4KshQc
.


serverless-webpack 3.0.0 is released

I just wanted to inform you, that I finally released serverless-webpack 3.0.0 and you can update your demo officially to 3.0.0.

In the meantime we're at 4.0.0 😄

Thanks for the reminder :) 4350f13122546386085530d8079c8efac829094f


limiting a numeral, numeralScale?

Like limiting a numeral before the point, is there a numeralScale?

Feature added :)
https://github.com/nosir/cleave.js/blob/master/doc/options.md#numeralintegerscale

Released in v0.7.20

thank you very much! @nosir


Removing Azure AD graph permissions and replacing with MS Graph

Removed the Directory.ReadAll Azure AD graph permission and replaced with the same permissions from the Microsoft Graph instead.
This is required because the Azure AD graph will shortly be no longer supported and will not receive security updates.
In addition, I have removed the User.Invite permission as this is not required and was leftover from a previous version of the app template.

@abbodh this has been tested and is ready to review/approve


Failure to set proxy during redirects

PR Checklist:

[x] I have run npm test locally and all tests are passing.
[x] I have added/updated tests for any new behavior.
[x] If this is a significant change, an issue has already been created where the problem / solution was discussed: [#2683]

PR Description
This change makes sure that the proxy setting is re-evaluated according to the redirect url

@mikeal - Anything blocking the merge of this PR? It looks well-written, complete with a test case and merging that would allow us to update the requests version in Yarn and fix a bug instead of having implementing a workaround ourselves.


open SimpleMDE vulnerability  issues

Hi,
Thanks for keeping this project alive!
I've seen on the original SimpleMDE repository that there are various open vulnerability  issues
My question: Have these been fixed? Or, what is their status?
Cheers.

https://snyk.io/test/npm/easymde/2.11.1-373.0

Vulnerabilities in the code or in the dependencies have all been fixed.
However the often-reported XSS vulnerability / sanitizing input is not something I believe should be handled by the editor, but by the developers using EasyMDE on their websites. Either by using the option from marked or processing the input before it is passed to the editor.

Thank you very much!


Empty entry point causes build failure

Take this config as an example:
var config = {
    ...
    entry: [
        "", // empty string
        "./client/index.js"
    ],
    ...
};

The empty string causes a build error, with the following message:
Module build failed: Error: Final loader didn't return a Buffer or String
I'm not sure if this behaviour is wanted, but a more helpful error message would be nice.

Is this still an issue on RC3? I just took a look and wanted to try and help here but it looks like the output has been improved.

Specifically last line:
* configuration.entry[0] should not be empty.

@TheLarkInn would this be a good issue for me since I am completely new to the repo?

@tarang9211 As far as I understood this issue is solved?
Check @johnstew's screenshot.

I'm closing this issue as it is no longer relevant.


[BUG] Metadata url extraction from Confluence Document Loader

Describe the bug
when extracting contents from a confluence page, the page url is saved as metadata.url instead of metadata.source into the database (pinecone here)
thus the "return source documents" in chats cannot return the correct link, and return a json popup instead, not really useful for end users
To Reproduce

import confluence data into pinecone using the confluence document uploader (split text on markdown)
see that embeddings in pinecone have the page confluence url saved as metadata.url
test a chat, with "return source documents" > will not output the correct pages links
in the database, rename some embedding field from "metdata.url" to "metadata.source"
the chat bot will now ouput the correct links for those embeddings

Expected behavior
the confluence document loaded should store pages urls as "metadata.source"
Screenshots
Flow
Setup

Flowise 1.6.2
OS,  Browser, Docker: not relevant

Thanks !

hey @lde-net2! langchain added as url - https://github.com/langchain-ai/langchainjs/blob/main/langchain/src/document_loaders/web/confluence.ts#L228

Thanks Henry
Sorry , i didn't catch it in Lanchain code.
can close this (not a bug) then
How would you suggest to handle this, as the chat module expects metadata.source ?
thanks !!

@lde-net2
Can you share your chatflow?
I will try help you!

i don't think this is related to the flow,  but instead a discrepedancy between the langchain metdata structure and what flowise expects.
i hacked the frontend chatmessage to use source.metada.url & source.metdadata.title instead of source.metadata.source
not clean..  but no way to get an advice or answer nowhere :( !

I think there is something bugged on Pinecone's end.
I had a similar bug where Pinecone was hard tagging the metadata "text" even if you tagged as "url" or anything else.
https://github.com/HonzaTuron/pinecone/issues/4#issuecomment-2018436559
For 2 weeks we were working on this problem so we quit.

no, as henry pointed out abovce, langchain is saving the page url as metadata.url
pinecone is just storing the data the way it receives it

you'll have to override the ConfluencePagesLoader createDocumentFromPage method to change the metadata, or you can submit a PR to langchainjs then we dont have to override in Flowise.
Since this is not a bug, I'll be closing it for now


feat: set arbitrary plugin config value at cli

Doing a bit of spit balling.
We previously addressed the need for supplying configuration to plugins with the --plugin-config flag to load a yaml file into settings for use by plugins. We've found this feature to be quite helpful, especially in the case of loading multiple plugins (i.e. resolvers) that each require their own configuration. However, we have also found that it would be handy to be able to set or modify these values from the command line. So instead of having to modify the plugin_config.yml you're using to load the universal resolver plugin to point it to a different resolver instance, we could have a command line argument like the following:
--plugin-config-value http_uniresolver.endpoint=http://localhost:3000

Or perhaps a short form of:
-o http_uniresolver.endpoint=http://localhost:3000

With such a command line argument, we propose that dotted key values act as a shorthand for nesting values. For example, a.b.c.d=value becomes {"a": {"b": {"c": {"d": "value"}}}}. We also propose that the value be loaded as yaml to avoid parsing ourselves while still allowing us to express values more complicated than just strings and numbers.
Open to thoughts :slightly_smiling_face:

Codecov Report

Merging #1347 (ca302c0) into main (1f8eeb0) will decrease coverage by 0.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main    #1347      +/-   ##
==========================================
- Coverage   95.43%   95.41%   -0.02%
==========================================
  Files         476      476
  Lines       28977    28986       +9
==========================================
+ Hits        27653    27656       +3
- Misses       1324     1330       +6

Seems reasonable to me, can you write something like --plugin-config-value "a.b.c=[one,two]" ?

Yep, [one, two], {inline: yaml_dict}, true, etc. all get parsed into the expected values


Setting Open/Close time from HA

Hello guys,
Thanks a lot for putting the effort in this integration!
I was wondering if setting the open/close time from HA is on your roadmap.
My scenario: leave the Omlet on time based automatic open/close and update the open/close time based on the sunrise/sunset from HA. This way, even if the Omlet goes offline (due to a bad wifi connection) it would still close on time.
Tillsy already proposed an workaround but it would be nice(r) if that would simply be a feature on the integration.
Thanks again!

Hi @tinutac,
That should already be there and working. As of v0.9.2, it is in the configuration section.

Let me know if it doesn't work, feel free to open a new ticket with some info (like version etc.) and screenshots.
Cheers,
Will


custom marker class / styling issue (1.17.1 regression)

http://plnkr.co/edit/Jcjl1LGTJlcGo8t3q3SN?p=preview
change the ng map version from 1.17.1 to 1.16.7

as you can see the styling of the custom markers changes.
I think it's related to this commit: https://github.com/allenhwkim/angularjs-google-maps/commit/101e9b1084e519d7bd79aa31dfc1df66872c4f9a

if you inspect the html of the first custom marker
v1.17.1
check the class(es) of the outer div

            blue

            marker1

v1.16.7
check the class(es) of the outer div

            blue

            marker1

Thanks for reporting it. I really appreciate it. I would be careful when I accept PR.

I have fixed this I believe with 1.17.2


improve queues, (one effect: improve code completion)

After updating to the recent 1.23.x versions I had severe performance issues on code completion.
I investigated that and made the following changes:

Completion, CompletionResolve, and AutoComplete should be equally prioritized (add to normal queue)
remove request from _waiting queue when cancelRequest is called

The second bullet point addresses a deferred queue that stays always full when requests are cancelled. A symptom for that are many log entries of the kind:
Problem invoking 'GetCodeActions' on OmniSharp server: Error: Pending request cancelled: /v2/getcodeactions

For me, especially the other change, calling dequeue when a request is canceled is most important. Please note the log entries that I entered to debug the behaviour in 954ce95. I found out, the deferred queue is considered full just when two requests are canceled but remain in the _waiting queue forever. Then the queue does no longer accept further requests. This especially had an impact on code completion in the last versions. But also, Unit-test classes loose the "RunTest" links, Code Structure gets lost in the outline panel and so on.
The code I am working on has lots of automapper statements which made trouble when calling codeactions couple of times that were subsequently canceled, some already _waiting. And suddenly all requests that are prioritized to deferred queue didn't reach out for the omnisharp server.
I am not sure, if adding the id to the requests is proper, but at least solves a quick handling in the dequeue method.
I think this fix addresses at least some of:
#4292
#4289
#4311
#4080
#4047
#4072
#3421
#4065

Sure. I needed to do these adjustments because I realized the OmnisharpWorkspaceSymbolProvider (ctrl-t) also needs request.onError to cancel the awaited task when it is in the waiting queue.
Essentially what changed in the last 4 commits are two things:

I added the OmnisharpServerRequestCancelled BaseEvent and did some small changes to other BaseEvents to get more logging for canceled requests in the output panel when debug=true in main.ts.
I moved the request.onError call one level higher to be called independent of the queueStatus the request is in. Originally, onError was only called for requests in the _pending queue.

Cheers.

Sure. I needed to do these adjustments because I realized the OmnisharpWorkspaceSymbolProvider (ctrl-t) also needs request.onError to cancel the awaited task when it is in the waiting queue.
Essentially what changed in the last 4 commits are two things:

I added the OmnisharpServerRequestCancelled BaseEvent and did some small changes to other BaseEvents to get more logging for canceled requests in the output panel when debug=true in main.ts.
I moved the request.onError call one level higher to be called independent of the queueStatus the request is in. Originally, onError was only called for requests in the _pending queue.

Cheers.


roccat-tools-{full,tyon,etc.} does not build

(They all build from the same source, just packaged differently)
Here is my error log: https://gist.github.com/ashkitten/722dd5ffbcd6bed5cf344a3dc166ae9b

All the dependencies appear to be satisfied otherwise `makepkg' wouldn't continue, however roccat-tool's cmake setup is looking for libgaminggear and failing to find it.
Does it build with plain `makepkg'?

oops, looks like it doesn't build with makepkg, my mistake. closing.

side note after closing: reinstalling libgaminggear seems to have fixed the issue, so there's that


Bugfix for logfile creation

This fixes the bug that no log files are created.

Works!

@yaron4u merge this


Implement coordinate blinding for EC_POINT (1.1.0)

Backport of #6501 to OpenSSL_1_1_0-stable.
This commit implements coordinate blinding, i.e., it randomizes the
representative of an elliptic curve point in its equivalence class, for
prime curves implemented through EC_GFp_simple_method,
EC_GFp_mont_method, and EC_GFp_nist_method.
This commit is derived from the patch
https://marc.info/?l=openssl-dev&m=131194808413635 by Billy Brumley.
Coordinate blinding is a generally useful side-channel countermeasure
and is (mostly) free. The function itself takes a few field
multiplicationss, but is usually only necessary at the beginning of a
scalar multiplication (as implemented in the patch). When used this way,
it makes the values that variables take (i.e., field elements in an
algorithm state) unpredictable.
For instance, this mitigates chosen EC point side-channel attacks for
settings such as ECDH and EC private key decryption, for the
aforementioned curves.
For EC_METHODs using different coordinate representations this commit
does nothing, but the corresponding coordinate blinding function can be
easily added in the future to extend these changes to such curves.
Co-authored-by: Nicola Tuveri<EMAIL_ADDRESS>Co-authored-by: Billy Brumley<EMAIL_ADDRESS>

We manually fixed the new error code for EC_F_EC_GFP_SIMPLE_BLIND_COORDINATES to match with the one in master.
Ping @mattcaswell @richsalz : is this the right way to do it ?

Hold the review for a second while we fix a logic error caused by the automatic cherry pick in ec_mult.c

Ok false alarm ;)
The cherry picking was fine.
Pending your feedback on the error code ( https://github.com/openssl/openssl/pull/6526#issuecomment-398465803 ), it should be ready for the review.

Rebased on Openssl_1_1_0_stable to fix the merge conflicts after fixing the formatting comments.
@dot-asm sorry it's been a while, but should be good to go now !

CHANGES need another re-base....

@dot-asm CHANGES re-based !

Ready for merge ?

I was hoping to have a green light from @dot-asm

is this ready for a merge or is there anything that still needs to be done?

It seems to me the only thing missing is either @dot-asm reapproval (after rebasing and fixing the CHANGES issue) or an approval from another @openssl/omc member.
I can take care of merging if @dot-asm is too busy!

Even with @dot-asm reapproval, it still seems to be missing an OMC approver.

Merging now!

Merged with 875ba8b. Thanks everyone!


Update role name

Fixes #34441

@Potapy4 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

Only Microsoft employees can sign off on PRs. Your PR will be routed for review.

@vladvino Bug filed for PRMerger issue: https://ceapex.visualstudio.com/SRE/_workitems/edit/103643
Merging the PR now.


pyasn1.error.PyAsn1Error: Duplicate name

for the following block of code
for (errorIndication,
     errorStatus,
     errorIndex,
     varBinds) in nextCmd(SnmpEngine(),
                          CommunityData('public'),
                          UdpTransportTarget(('<IP_ADDRESS>', 161)),
                          ContextData(),
                          ObjectType(
                              ObjectIdentity("SNMPv2-MIB", "sysObjectID").addAsn1MibSource('http://mibs.snmplabs.com/asn1/@mib@').loadMibs('ExaltComProducts')
                          ),
                          lexicographicMode=False):

    if errorIndication:
        print(errorIndication)
        break
    elif errorStatus:
        print('%s at %s' % (errorStatus.prettyPrint(),
                            errorIndex and varBinds[int(errorIndex) - 1][0] or '?'))
        break
    else:
        for varBind in varBinds:
            print(' = '.join([x.prettyPrint() for x in varBind]))

this error is produce:
'pyasn1.error.PyAsn1Error: Duplicate name mQPSK\n']caused by <class 'pyasn1.error.PyAsn1Error'>: Duplicate name mQPSK

Sounds like the ExaltComm MIB has an error - multiple mQPSK symbols here:
    AcmModulationT ::= TEXTUAL-CONVENTION
        STATUS current
        DESCRIPTION "These are the supported ACM Modulation types. When ACM is disabled or when telemetry is down, the modulation is -9999."
        SYNTAX INTEGER {
           notAvailable( -9999 ),
           mQPSK( 0 ),
           m8PSK( 7 ),
           m16QAM( 1 ),
           m32QAM( 5 ),
           m64QAM( 2 ),
           m128QAM( 3 ),
           m256QAM( 4 ),
           m512QAM( 6 ),
           mBPSK( 8 ),
           mQPSK( 9 ),
           m8PSK( 10 ),
           m32QAM( 11 ),
           m64QAM( 12 ),
           m128QAM( 13 ),
           m16QAM( 14 ),
           m512QAM( 15 ),
           mQPSK( 16 ),
           mBPSK( 17 )
        }

The latest (yet unreleased) pysmi has a workaround engineered into to handle such situation. So you may want to either use the latest version in your application or just use it one-time to compile required MIBs into Python (with mibdump tool).

Thanks a looooooooot :)

Hello,
Sorry for bothering you, but I would need some help, please.
I am dealing with this from a particular MIB:
Add:  bnameE8000ESU91SPUC01(**16748552**),    --Eudemon8000E-X,SU91SPUC01,40G ......
Add:  subbnameEudemon8000ECR52L1XX0(**16748552**),  --1-port 10GBase WAN/LAN XFP......

The problem is some MIBs have duplicated numbers. I tried to compile them with mibdump.py and that's ok, but at the moment of loading them, the error raises an exception like this:
File "/usr/local/lib/python3.9/site-packages/pyasn1/type/namedval.py", line 77, in init\n    raise error.PyAsn1Error('Duplicate number  %s=%s' % (name, number))\n', 'pyasn1.error.PyAsn1Error: Duplicate number  bnameUSG9100SU61SPUA=16748552\n']caused by <class 'pyasn1.error.PyAsn1Error'>: Duplicate number  bnameUSG9100SU61SPUA=16748552
Could you please give me a clue?
Thank you, I really appreciate it.
M


pack Instruction subclasses more tightly

The Itanium (Unix) C++ ABI has this interesting feature where if you have a class/struct NotFull:
struct NotFull {
  void *p;
  char c;
  // Seven bytes of unused space on 64-bit platforms.
};

and you inherit from NotFull, class members will, if possible, be allocated starting in the unused space immediately after c.
This situation is exactly the situation we have with cfg::Instruction: there are seven bytes after isSynthetic and, in many cases, we are not using those bytes to the fullest.  This patch rearranges class members to pack them into that empty space where possible.
It would be nice to pack VariableUseSite a little more tightly with other members in several places, but that is a much more involved change; this change merely shuffles members around and updates checks accordingly.
Motivation
Less space is more better.
Test plan
Existing automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build result here:
→ https://go/builds/bui_IQoY2KnqatoXgD


No module resizeimage

Advertising of monetized Frameworks is strictly forbidden and will be reported

[ ] The language must be English, if you are not fluent in English consider using: https://www.deepl.com/translator if it supports your native language

[ ] Please check if any other issue already adresses your concerns or if a recent PR is already fixing the mentioned issue

[ ] Please make sure to format your code, and to mark your code as such, with the provided tools from github (upper right of this box)

[ ] Please state your Operating System (inlcuding version), your Python Version, and your installation way (pip, from repo) and if you are running it inside a docker container

[ ] Note that the pip installation might be behind, so try to fix your issue with a pull from the repo and update to the most recent version

[ ] Follow the Stackoverflow Guidelines (https://stackoverflow.com/help/how-to-ask) when in need of help

[ ] Consider posting your question on Stackoverflow if it is a general programming question

[ ] There will be no support for spamming, or other mischief

[ ] There will be no support for finding professionals

python3.6 bot.py
Traceback (most recent call last):
File "bot.py", line 2, in
from webwhatsapi import WhatsAPIDriver
File "/usr/local/lib/python3.6/site-packages/webwhatsapi-2.0.3-py3.6.egg/webwhatsapi/init.py", line 22, in
from resizeimage import resizeimage
Any suggestions?
pip know nothing about it.
Is project alive?

Any solutions ????

python3 -m pip install python-resize-image


Base URL Not Working

Deployed three websites using "hugo-theme-learn", all those three themes having an issue with base URL.Base URL working with local server(http://localhost:1313) but not working in production server URL. I tried to change base URL in the configuration file, which also not working.

Closing due to old age. Feel free to open a new issue if you are still having difficulties


QuadtreePrimitive Sandcastle example Rectangle exception

With master, browse to http://localhost:8080/Apps/Sandcastle/index.html?src=development/QuadtreePrimitive.html&label=Development
Then zoom in, and switch to the console tab:

Is this due to recent changes to the Rectangle error checking?

Duplicate of #3263

Ah, I searched for "QuadtreePrimitive", not "Quadtree Primitive"


fix: also link piece to derived file if inserting

When we insert the source file (because there's no derived file) we should still link it to the piece otherwise Themis & Valvas break.

Tested on ACC (because @MathiasZabot noticed the bug there) and it works. Merging and deploying.


add options wsGetAuthentication hook for authentication

... based on upgrade request info (e.g. cookies, headers)

Coverage increased (+0.01%) to 96.481% when pulling 30e759a878d46628fb64978c673d0882fca7c646 on add-websocket-authentication into 8f1314cf9138c5793dec039b22707bcb191ef4b3 on master.

Coverage increased (+0.01%) to 96.481% when pulling 30e759a878d46628fb64978c673d0882fca7c646 on add-websocket-authentication into 8f1314cf9138c5793dec039b22707bcb191ef4b3 on master.

Coverage increased (+0.01%) to 96.481% when pulling c8f8d972a4ef019a14512db3bafadca620208b6a on add-websocket-authentication into 8f1314cf9138c5793dec039b22707bcb191ef4b3 on master.


Peer review process switch on

Preview environment images deployed!
The preview environment has been updated:
client image tag: preview-1984-6ea8b19a46b869b8e363a4eaf58c7e26578aebca-20240613.2024
Client URL: https://pr-1984--epp.elifesciences.org
Note: Updated images can take a few minutes to roll out.

Preview environment created!
The preview environment has been deployed to the cluster.
Client URL: https://pr-1984--epp.elifesciences.org
Note: If this is a new environment, it can take a few minutes for DNS to update.


store: fix to 'go vet' errors

This check has been introduced to go tip [1]. Just fixes by calling
the Index method.
1: https://github.com/golang/go/commit/0f89efa255a46eb6528d27c920030721ae68b507

lgtm

thanks


Support for Zero-Shot Learning models

It would be great to have ZeroShot learning models support. Given a business case, where I would like to understand a rough direction of a given dataset, this model usage at scale would be highly helpful.

Applies to Hugging Face


Check TypeScript errors during build

npm run typegen (and npm run build) now will check TypeScript errors.
I migrated jsconfig.json to tsconfig.json because tsc --build can only recognize tsconfig.json. tsc --build is faster than tsc because tsc --build has cache, so it's better to use tsc --build.

Since it reveals all typing errors now, I need to address all of them to make the build pass.

Most of them are just missing types in configs, like the one below:
this.config.image_token_index;
            ~~~~~~~~~~~~~~~~~
            ^ Property 'image_token_index' does not exist on type 'PretrainedConfig'.

For those errors, I simply add @ts-expect-error comment to let TypeScript ignore it. I use @ts-expect-error instead of @ts-ignore because we can remove those comments once we add missing types in the future.

I fixed some other types issues that I can fix.

One thing I noticed is that MgpstrProcessor.batch_decode has different shape compared to the method in the parent class. This might not be a good idea.

Thanks for this! I'll do a full review soon, but just to address your third point: https://github.com/huggingface/transformers/blob/c8c8dffbe45ebef0a8dba4a51024e5e5e498596b/src/transformers/models/mgp_str/processing_mgp_str.py#L102 has a similar issue where MgpstrProcessor has a non-standard batch_decode function. This is because the model returns three tensors that all need to be used in the decode method. Not too sure how to make it fit the parent class type though.

I would consider this as an issue in the original Python transformers lib. At least its docstring is wrong.
I don't know any context about MgpstrProcessor, but maybe renaming this method to something else (e.g. just decode()) could improve it.

Apologies for the delay - reviewing now 👍

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Fix release build and artifact publications to local maven repository.

It was broken with the refactor of our plugin into a settings plugin

Great, circleci need to be restored in following PR


allowJs option with namespaces

I am currently trying to put the compiler option "allowJs" to use. My goal is to convert an existing JavaScript application into TypeScript step by step. The application does not use (external) modules, so I'm converting the JavaScript namespaces into real TypeScript namespaces, but I'm running into "duplicate identifier" errors as outlined below. Shouldn't it be possible to combine TypeScript and JavaScript using the same namespace? Maybe I'm doing something wrong or this is not intended, but I expected this to work.
TypeScript Version:  1.8.10
Code
File 1:
namespace myapp {
    export var c1 = 5;
}

File 2:
var myapp;
(function (myapp) {
    myapp.c2 = 4;
})(myapp || (myapp = {}));

tsconfig.json
{
    "compilerOptions": {
        "allowJs": true,
        "outFile": "dist/appBundle.js"
    }
}

Expected behavior:
Bundling the compiled code of file 1 and the code of file 2 without modifications into a single JavaScript file.
Actual behavior:
The compilation is done, but the compiler throws an error: "error TS2300: Duplicate identifier 'myapp'."

Duplicate identifier 'myapp'

Personal opinino: Recommend using modules please (https://basarat.gitbooks.io/typescript/content/docs/tips/outFile.html) :rose:

I agree that modules are a better solution in many cases, but as I am working on an existing application, I want to do the conversion one step at a time.
Regardless, I still think that the option "allowJs" should at least somehow work when using namespaces.

@sandersn this is would be the next step after https://github.com/Microsoft/TypeScript/issues/16489. If we track the property assignment to a target that is a parameter of a an IIFE, we should redirect the declaration to be on the argument of the IIFE instead..


Update init.py

Cleanup:

Unused import
Unused assignment.
Incorrectly named use of var. (read_preference != read_pref)

Thanks!

This is released in version 0.3.1 -- Thanks!


66: add ability to use polymorphic types

Here is solution for #66

Current coverage is 100% (diff: 100%)

Merging #67 into master will not change coverage

@@             master   #67   diff @@
=====================================
  Files            27    27
  Lines           357   357
  Methods           0     0
  Messages          0     0
  Branches         79    79
=====================================
  Hits            357   357
  Misses            0     0
  Partials          0     0

Powered by Codecov. Last update e2b0297...cee0945

Thank you


if "latest" for revision, grab latest deployment available for proxy

This PR includes a change that allows for users to set revision="latest" in their .tf files. During the Update stage, terraform will fetch the latest valid revision from the api proxy and deploy that revision.

Nice.  This looks like a step in the direction of fixing issue #8 too.  Does that make sense?

Yep that is exactly what this does. If the revision is 'latest' terraform will grab the most recent deployed revision for some api proxy and use that as the target revision for the deployment resource. It's also nice if you dont want to depend on the apgiee_api_proxy resource

I took a look at the issue you opened https://github.com/hashicorp/terraform/issues/15857 and this is exactly what we were running into. This is a bit hacky, but gets the job done.

Nice!  Thanks for the contribution.

@zambien great, mind giving it an approval then ill push out a new release?

you got it!

hello,
we still need to double apply to deploy the last revision.


Text Description

https://github.com/Saipraneet/iitdthesis/blob/12e477ee0c542d65ce9bb211bb29b979bdc3db62/iitddiss.cls#L194
Was just going through the code, is this an expected or an incorrect text in this line? If so, I can send a PR to correct.

 Please feel free to send a PR :)


Update access-instances.md

Additional edits are needed, but the changes made herein are needed ASAP to reduce confusion associated with the Tier 1 deployment option changes.  This page is linked in the email notification sent to customers about the Tier 1 changes.

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.


Limits the recoverable attributes in case of sorting

Describe the bug
I'm using a dataset of 24M hacker news posts. Here is an example of a document:
{
    "by": "perler",
    "descendants": 0,
    "id": 5,
    "kids": [454414],
    "score": 7,
    "time":<PHONE_NUMBER>,
    "title": "Google, YouTube acquisition announcement could come tonight",
    "type": "story",
    "url": "http://www.techcrunch.com/2006/10/09/google-youtube-sign-more-separate-deals/"
}

I created an index with the following configuration:
{
    "displayedAttributes": [
        "by",
        "id",
        "score",
        "time",
        "title",
        "type",
        "url"
    ],
    "searchableAttributes": [
        "title"
    ],
    "filterableAttributes": [
        "by",
        "type"
    ],
    "sortableAttributes": [
        "score",
        "time"
    ],
    "rankingRules": [
        "words",
        "typo",
        "proximity",
        "attribute",
        "sort",
        "exactness"
    ],
    "stopWords": [],
    "synonyms": {},
    "distinctAttribute": null
}

When I try to search without sorting, here is the result. I have the full document (following my settings).

But if I add a sort, I only have a few fields returned.

Meilisearch version: [e.g. v0.25.2] (Cloud)
Additional context
If you need access to my server to test, I can share my URL and my key with you, don't hesitate to contact me.

 Can you check that the document with id 11017550 does contain the fields you thought it should? We never had this issue before.

Indeed this is a mistake on my part, I had checked the 1101755 instead... From ID 10000000 onwards the deleted elements no longer have the metadata and as I didn't know about the deleted field I didn't realize that this could happen. Good catch @Kerollmops!


Use custom resource exports

Since https://github.com/godotengine/godot-proposals/issues/18 is included in 4.0, we can use it in various places where resource properties are exported.

Implemented in #14


Virtual attribute accessor methods no longer getting called.

With now having collection specific classes, the virtual
attribute accessor methods are no longer seen and called.
This is causing an incorrect call to content when
evaluating that virtual attribute for service dialogs.
Example call: get service_templates/1/service_dialogs --expand=resources --attributes=content
This commit simply moves the method around so it's seen by
the renderer, would need a better solution where it can
lives with the collection specific class.

/cc @imtayadeway

@gmcculloug can you verify this fixes the issue ?

@abellotti LGTM, unless we can easily add an expectation that proves that this fixes it?

I agree with adding a spec if possible.

:thumbsup: LGTM :balloon:

Verified this change corrects the issue.  👍


Flickr url regex fix

This module was rejecting certain urls, for example: https://www.flickr.com/photos/jrfrancis42/13113373634
It assumed it would be followed by "/in/photostream" or at least a slash.
Also made the photo id matching only digits so that it doesn't match something
like 'https://www.flickr.com/photos/12037949754@N01/sets/'

Thanks for the pull request!


how to set SIGNING_KEY in local.properties

hello, thanks for the template for the library publishing.
but I have one question, how to set SIGNING_KEY at local.properties. export key base64 is had a lot of line keys and how to set it to local.properties.
thanks.

I just have it as an env variable.
https://github.com/warting/Perfect-android-library-template/blob/main/.github/workflows/publish.yml#L57


iOS Project Template Updated to Support .NET 6

PR Details

The iOS .csproj template does not support .NET Core, this results in various exceptions when generating iOS projects with Stride 4.1.
Description

The template file was updated to support .NET Core/6.0, GenerateAssemblyInfo is false to avoid build conflicts with the Stride generated AssemblyInfo file.
Related Issue
https://github.com/stride3d/stride/issues/1414
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

iOS projects generated from the Stride source code do not currently build, I believe this is unrelated to the iOS csproj as the issues are documented elsewhere for the Android platform.
https://github.com/stride3d/stride/issues/1565#issuecomment-1327363220
https://github.com/stride3d/stride/pull/1456#issuecomment-1326702698

Thanks !


Maximized panel should stay maximized after a zoom

Steps to Reproduce:

Maximize panel, maximizes the panel
Zoom out, the panel is no longer maximized but still much larger than usual
Maximize panel, maximizes the panel

Fixed by #18820


Use DSPA custom ca cert on MLMD and Persistence Agent clients

Signed-off-by: Helber Belmiro<EMAIL_ADDRESS>(cherry picked from commit 40ca5edae1a86e7460e9c86a0de4de15664c95d4)

/test all

A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-739
An OCP cluster where you are logged in as cluster admin is required.
To use this image run the following:
cd $(mktemp -d)
git clone git@github.com:opendatahub-io/data-science-pipelines-operator.git
cd data-science-pipelines-operator/
git fetch origin pull/739/head
git checkout -b pullrequest 9d06a9f8341cc9c80b2328c6d001bc53a870aa75
oc new-project opendatahub
make deploy IMG="quay.io/opendatahub/data-science-pipelines-operator:pr-739"

More instructions here on how to deploy and test a Data Science Pipelines Application.


mount host /proc in the kind cluster to resolve pod name

Why do we need this PR:
On my Kind cluster, some pod names weren't being resolved
Actually the errors were related to no read access to /proc/pid
This is because when we create a kind cluster, the host directory /proc is not mounted in the kind worker node container
What this PR does:
We mount the host directory /proc in the kind container as /proc-host
Unfortunately, we can't use the same directory name /proc -> /proc
So since the host proc directory has a different name, we also updated the kepler manifest to mount the correct directory
Signed by: Marcelo Amaral<EMAIL_ADDRESS>

This is indirectly related to #264

I understand the purpose is to instead of use /proc, but /proc-host to compare the cgroup but seems still not working ..
need print more logs to see why the pod is filtered
root@jitest39:~/kepler# docker exec -it 0fcc639ede5d ls /proc-host
1      1428    1994     3149870  482651  518     81
10     1429    1995     3149871  482657  52      82
100    1430    1998     3149872  482661  53      83

# kubectl get daemonset kepler-exporter -n kepler -o yaml | grep 'path: /proc-host' -C 2
        name: tracing
      - hostPath:
          path: /proc-host
          type: Directory
        name: proc

I0930 08:58:23.471502       1 reader.go:227] Kubelet Read: map[kepler/kepler-exporter-kdwnv:7.638958 kube-system/coredns-6d4b75cb6d-t5wrg:1.897575 kube-system/coredns-6d4b75cb6d-zcktr:1.848968 kube-system/etcd-kind-control-plane:10.738694 kube-system/kindnet-4h2s8:0.765241 kube-system/kube-apiserver-kind-control-plane:33.895684 kube-system/kube-controller-manager-kind-control-plane:12.056751 kube-system/kube-proxy-xk5g8:1.074178 kube-system/kube-scheduler-kind-control-plane:3.984606 local-path-storage/local-path-provisioner-9cd9bd544-j52s6:0.602095 system/system_processes:85.26562799999999], map[kepler/kepler-exporter-kdwnv:1.49794816e+08 kube-system/coredns-6d4b75cb6d-t5wrg:1.77152e+07 kube-system/coredns-6d4b75cb6d-zcktr:1.7563648e+07 kube-system/etcd-kind-control-plane:3.516416e+07 kube-system/kindnet-4h2s8:1.3963264e+07 kube-system/kube-apiserver-kind-control-plane:3.16407808e+08 kube-system/kube-controller-manager-kind-control-plane:4.8308224e+07 kube-system/kube-proxy-xk5g8:1.8366464e+07 kube-system/kube-scheduler-kind-control-plane:2.1745664e+07 local-path-storage/local-path-provisioner-9cd9bd544-j52s6:9.453568e+06 system/system_processes:4.63466496e+08]
I0930 08:58:23.471576       1 reader.go:238] Aggr value overflow 463466496 < 463704064
I0930 08:58:23.471607       1 node_energy.go:95] node energy stat core 53 (5333) dram 703 (71327) uncore 0 (0) pkg 53 (5333) gpu 0 sensor 0 (0)
I0930 08:58:23.471684       1 estimate.go:142] dial error: dial unix /tmp/estimator.sock: connect: no such file or directory
I0930 08:58:23.471706       1 reader.go:352] energy from pod (157 processes): name: system_processes namespace: system
        cgrouppid: 0 pid: 688 comm: multipathd
        ePkg (mJ): 53 (5223) (eCore: 53 (5223) eDram: 703 (69852) eUncore: 0 (0)) eGPU (mJ): 0 (0) eOther (mJ): 0 (0)
        eDyn (mJ): 0 (0)
        avgFreq: 0.00
        CPUTime:  204435<PHONE_NUMBER>30906)
        counters: map[cache_miss:0 (0) cpu_cycles:0 (0) cpu_instr:0 (0)]
        cgroupfs: map[]
        kubelets: map[container_cpu_usage_seconds_total:0 (85) container_memory_working_set_bytes:2387968 (463466496)]

I0930 08:58:23.471732       1 reader.go:354] node energy (mJ):
        ePkg: 53 (eCore: 53 eDram: 703 eUncore: 0) eGPU: 0 eOther: 0

@jichenjc right, you problem is different then this one here

@marceloamaral actually , I mean I used your patch manifest and kind config file , got the result above ..
so I doubt whether the fix can solve the problem that kind can't see pod info ..

@marceloamaral @jichenjc can you share your environment (VM? OS? cgroup?)

mounting /proc is consistent with the deployment manifest, let's merge it.

My environment is Kind on top of Baremetal nodes

My environment is Kind on top of Baremetal nodes

ok, this might do the job
at least it doesn't work on BM => VM => kind
so i think this PR solved the BM => kind which is good


[Feature] Cross-namespace mariaDbRef support

Describe the solution you'd like
Being able to refer to MariaDB instances in different namespaces allowing to have shared cross-namespace MariaDB instances. This implies changing our MariaDBRef object to use a ObjectReference instead of a LocalObjectReference to be able to specify a namespace.
Describe alternatives you've considered
Use a per-namespace MariaDB instance as we currently support.
Additional context
This feature may introduce security concerns:

By default, tenants will be able to connect, create users, grants and even execue sql scripts in MariaDB instances living in different namespaces

This will need to be mitigated by:

Locking down RBAC to restrict object permissions i.e. a tenant owned by platform teams will be able to create Backups and Restores whereas an application will only be able to create Connections to its logical MariaDB database
Introduce a policy engine such as Kyverno to define policies on the mariadb-operator CRDs
Create additional policy CRDs managed by mariadb-operator to restrict access across namespaces. cert-manager and the Gateway API have already solved this problem:

cert-manager's CertificateRequestPolicy
Gateway API's ReferenceGrant

We need this for propagating user's password secret. Is there a good way?

@outductor could you provide more details? I'm guessing that you are willing to get the connection string to a MariaDB in a different namespace maybe? Something like:
apiVersion: mariadb.mmontes.io/v1alpha1
kind: Connection
metadata:
  name: connection
  namespace: app
spec:
  mariaDbRef:
    name: mariadb
    namespace: mariadb
 ....

The aim of this feature is allowing  spec.mariaDbRef to provide cross-namespace references in all the CRDs: Connection, Backup, Restore ... etc
If this is not what you want, please open a different issue and let's discuss it there 🙏🏻


ReferenceError: Can't find variable: Modernizr

I get the following error when pre-rendering an Angular app:
2017-04-20T06:16:18.045Z starting worker thread #0
2017-04-20T06:16:18.055Z starting worker thread #1
2017-04-20T06:16:18.196Z starting phantom...
2017-04-20T06:16:18.203Z Server running on port 3000
2017-04-20T06:16:18.228Z starting phantom...
2017-04-20T06:16:18.236Z Server running on port 3000
2017-04-20T06:16:18.318Z started phantom
2017-04-20T06:16:18.339Z started phantom
2017-04-20T06:16:22.726Z getting http://app.mydomain.com/
ReferenceError: Can't find variable: Modernizr

  http://app.mydomain.com/:83 in global code
2017-04-20T06:16:24.596Z got 200 in 1870ms for http://app.mydomain.com/

The page that is pre-rendered stops at the line that contains this:
  <script>
    if (!Modernizr.cors) {
      window.location.pathname = "/error-modern.html";
    }
  </script>

Hmm, can you send an email to us at<EMAIL_ADDRESS>with the URL you are testing? I'm thinking that there could be another issue causing that. I'd be happy to run some tests on that URL.

Hey Todd,
Thanks for looking into this.
I’m trying to get prerender running in a docker container on AWS ElasticBeanstalk to render this site: http://poc-website-s3.pgs.io (hosted on S3, and served through CloudFront).
It's currently not being rendered fully: http://poc.publicgood.com/about/?escaped_fragment=
I’ve attached the middleware configuration we’re using for nginx.
André
...aob
Any Other Business - Leave The Technology To Us
http://aob.com.jm
On 20 Apr 2017, 9:42 AM -0500, Todd Hooper<EMAIL_ADDRESS>wrote:

Hmm, can you send an email to us at<EMAIL_ADDRESS>with the URL you are testing? I'm thinking that there could be another issue causing that. I'd be happy to run some tests on that URL.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Schema Check is Unable to Work when Variable is passed

I am facing an issue when defining variables for my check. The issue is that the variable works fine in all the checks other than Schema Check  where is raise the following error:
"value": [{"columnName": c["name"], "sourceDataType": c["type"]} for c in self.value], TypeError: 'Undefined' object is not iterable.
My checks.yml looks like this:
variables:
table: 'customers'
checks for ${table}:
 - schema:
fail:
when required column missing:
- id
- name
What could be the possible reason for this error.? Or is it a bug related to variable usage in Schema Check

SODA-1197

Closed via #1628

this issue appears again after the last commit that modifies schema_query.py


Need  "Remap" OP

Hi,  Openppl
I need to use op "remap" on aarch64.   Could you provide this op.
Thanks.

Okay, I will add this OP in the near future.


Properly handle array outputs

Currently arrays are transformed in objects with the array indices as (string) keys through the spread operator.
With this pull request arrays are properly returned as arrays, and additionally the validation now properly works for non object ZodTypes.

Hi @DoubleMalt
Thank you for your contribution! Please add a changeset by running pnpm changeset so that I can give credit to you in the release note and changelog.

Click here to learn what changesets are, and how to add one.


Clarify environment-variable definitions

Reword environment variable description and include an example of defining more than one variable.
In response to https://github.com/dotnet/msbuild/issues/5308#issuecomment-679239219
@ghogen

@rainersigwald : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rainersigwald : Thanks for your contribution! The author(s) have been notified to review your proposed change.


filterPanel checkbox has unexpected value and behavior

Issue Type
bug report
Description
The filterPanel checkbox at the bottom is not change filter state.
Steps to Reproduce
Click to 'status' filter button(☱), select any one.
Try click to filterPanel checkbox below - the checkbox is not change.
Current Behavior
Before clicking In html: <input type="hidden" value="true">.
After clicking: <input type="hidden" value="[object Object]">
Expected Behavior
The checkbox and filter are change state properly.
Demo
https://stackblitz.com/edit/react-c5ztsy?file=App.js
Environment Details
The behavior is repeated on any version of devextreme-react.

devextreme version: 18.1.6
devextreme-react version: 18.1.6-alpha.11
react version: 16.5.2

Hi,
When you set the filterPanel option, it is assumed to be controlled. This means making it a part of the state, subscribing to the option change event, handling it, and updating the state.

If you want to control it - use the onOptionChanged option (e.name should be filterPanel).
If you don't need this - you should use an appropriate property with the default prefix. In your case, it should be defaultFilterPanel, but it's not available at the moment. I suggest you use a nested configuration component.
<DataGrid
    dataSource={columnsData}
    ...
>
    <FilterPanel visible="true" />
    ...
</DataGrid>

In this case filterPanel.filterEnabled will be uncontrolled (as not set directly).

We will update the issue when we add defaultFilterPanel.

I have changed the code:
https://stackblitz.com/edit/react-c5ztsy?file=App.js
The question is why e.value has object type (equal filterPanel configuration)? It should be boolean instead.
How to control this value?

Sorry, thanks. I understood how to control


misc(prettier) Add schemas directory to prettier format

What kind of change does this PR introduce?
Refactoring, Added schemas directory to prettier format script.
Did you add tests for your changes?
No
Summary
schemas folder was missing from prettier formatting, it should be part since it is also part of webpack source code.
It was never added in the original PR https://github.com/webpack/webpack/pull/6081

Yesterday I had to delete my fork from this repo in order to create a PR against @xtuc repo when helping him merge master to his PR #6433, after restoring my original fork GitHub failed to reference it again, so I had to create a new PR https://github.com/webpack/webpack/pull/6766.
Sorry for the mess.


Russian language support

My native language is Russian and I made localization of the blog taking into account the grammar of the Russian language. Translated from the English version.
Added new parameters for i18n "RU".
After launching the site I made sure that nowhere wrong:
https://blog.kerbs.dev/
Github repository with blog:
https://github.com/KerbsOpenSource/KerbsOpenSource.github.io
I am going to use Gmeek and was happy to help with Russian language support in the future.

Added a transliteration from Russian to English so that the links were beautiful and there was better SEO optimization
Added new parameters for urlMode "ru_translit".
Example, new : https://blog.kerbs.dev/post/pwdgenz---CLI-programma-dlja-generatsii-parolej.html

@KerbsOpenSource
Thank you very much for your support of Gmeek's Russian language. Welcome more Russian friends to use it.


Release fails - decryption operation failed

Agent Version and Platform
Azure Pipelines --> windows-latest
Azure DevOps Type and Version
dev.azure.com (formerly visualstudio.com)
What's not working?
Have a release that fails with a message in the logs that mentions decryption. This only started happening today [24/11/2021]. Build was done yesterday [23/11/2021] and worked but release today [24/11/2021] did not work. We believe the error is happening within the Deployment group job.
Agent and Worker's Diagnostic Logs
2021-11-24T09:15:11.7177558Z Downloading task: DownloadBuildArtifacts (0.194.0)
2021-11-24T09:15:51.6773186Z ##[warning]Failed to download task 'DownloadBuildArtifacts'. Error The decryption operation failed, see inner exception.
2021-11-24T09:15:51.6833430Z ##[warning]Inner Exception: {ex.InnerException.Message}
2021-11-24T09:15:51.6853916Z ##[warning]Back off 17.743 seconds before retry.
2021-11-24T09:16:16.7916447Z ##[warning]Failed to download task 'DownloadBuildArtifacts'. Error The decryption operation failed, see inner exception.
2021-11-24T09:16:16.7920737Z ##[warning]Inner Exception: {ex.InnerException.Message}
2021-11-24T09:16:16.7938695Z ##[warning]Back off 18.429 seconds before retry.
2021-11-24T09:16:53.2192492Z ##[error]The decryption operation failed, see inner exception.
2021-11-24T09:16:53.2251087Z ##[section]Finishing: Initialize job

Hi @k-sek, we aren't able to reproduce your issue currently. I assume that the error is caused by some configuration issues on your side, e.g. it could be related to some certificate issues or network issues (like FireWall or proxy configurations). Could you please reply to some questions to allow us to investigate your issue future?

Is this issue still persist for you?
Do you have FireWall or proxy in your network?
If you use proxy or FireWall please recheck that there were no changes in its politicks in the days when the issues appeared.
Please attach full agent and worker logs for a failed run, if the issue still persists.

Hi @EzzhevNikita, thank you for looking into this.
Our hosting provider corrected the heartbeat HA configuration for the hosts on the firewall level and turned off IPv6 on the firewalls. That seemed to have resolved our issue. Cheers.


Fix /MT /MD flags for Cmake 3.15+

The /MT /MD flags are not set correctly in CMake 3.15+. Maybe you want to do this in a different place, but this works.
See: https://cmake.org/cmake/help/latest/variable/CMAKE_MSVC_RUNTIME_LIBRARY.html#variable:CMAKE_MSVC_RUNTIME_LIBRARY

...also happens on my Windows box with VS2019 and cmake 3.22.2
There's a weird "additional option":

Reverting to the last commit fixes the issue, so I'll do that. From looking at your changes I have no idea how that would happen though... hmm.

Ok I have undone the merge commit for now.
Any idea what the problem could be?

Ah, found the problem, note the little "d" here at the end:
https://github.com/floooh/fips/blob/85b7904bd15f2bdfaf53cb6ffe3b1b893848b08b/cmake-toolchains/windows.cmake#L60

ok, now I understand, the 'd' is the problem, sorry about that!

should I do a new PR or you want to fix it?

Would be nice if you could provide a new PR :)
I think all uses of the FIPS_VS_CRT_FLAGS variable must be moved up into the "cmake version < 3.15" block.


Fixes a bug where a missing DocBlock caused errors

Hi,
This will fix the bug that has occurred since Laravel removed the DocBlock. This fix ensures that all expected mixin entries are present.
Fixes #623

Trying to fix it upstream https://github.com/laravel/framework/pull/23111


composer.lock must be commited

anyone who sets up the project will download the exact same version of the dependencies

from GetComposer

Yes, this is true for projects, but not packages. We've had this discussion before, I just can't remember where.

Further reading.


Use the typescript server from the current project

Currently LSP-volar has a hard-coded the path to the tsserver:
https://github.com/sublimelsp/LSP-volar/blob/2d01fd3504d58c52ac2bdf08c0be00a329f64129/LSP-volar.sublime-settings#L4-L7
LSP-volar assumes that the node_modules will be at the root of the opened folder, which is not always the case.
Proposal,
find the tsserver path, even when the node_modules is not at the root of the project,
and if it cannot find one fallback to a bundled tsserver path. (which requires typescript as a new dependency in the package.json file)

I don't really see why the volar server itself should not do it. It's pretty common for servers to do it (vetur, typescript-language-server, ...). Maybe just try creating an issue for it in volar's repo?

I know. I did a PR to volar that should add just that:
https://github.com/johnsoncodehk/volar/pull/401
The biggest concern was security. The maintainer shifted the security responsibility to the client... But after a while of thinking I still thing that this logic should be on server.
To be more precise. The server would have the logic to find the tsserver, but if the security is a concern the server should not be started by the client in the first place.
It is kind of a pain to find the tsserver path with python.

I don't think you should priority use workspace typescript module. If the workspace has no install node_modules yet, or user have no add typescript to project dependencies (This is very common, especially for js projects.), language server can't working at all.
If Sublime has built-in typescript module you should use it first, otherwise you need to add typescript as @volar/server peer dependency in https://github.com/sublimelsp/LSP-volar/blob/master/server/package.json#L5, and config typescript.serverPath as typescript/lib/tsserverlibrary.js.

It's pretty common for servers to do it (vetur, typescript-language-server, ...).

They do not mean it is correct to do so. Yes this is convenient for the language client, but it also urge language client to give up security considerations.
But before considers security, I suggest to consider whether it can work in js projects.
This is a example: https://github.com/vitejs/vite/tree/main/packages/create-vite/template-vue

I don't use sublime, so I don't know, but I'm sure there are ways to deal with this, such as referring to the angualr (LSP-angular) plugin. https://github.com/sublimelsp/LSP-angular/blob/master/server/package.json#L6
https://github.com/sublimelsp/LSP-angular/blob/master/LSP-angular.sublime-settings

Thanks for dropping by and stating your opinion. :)
I don't exactly agree with all your points though so let's discuss.

I don't think you should priority use workspace typescript module. If the workspace has no install node_modules yet, or user have no add typescript to project dependencies (This is very common, especially for js projects.), language server can't working at all.

That isn't a valid argument against this request because there normally is a fallback to a typescript version bundled with the extension, editor or a global one. So that's not an issue.
Security considerations aside for now, the ideal case IMO is that the workspace typescript is used (when available) since otherwise there can be various incompatibilities between what the project expects and what the Volar expects (typescript introduces breaking changes all the time, even in minor releases). This is especially important for projects that rely on typescript for compilation, not so much on pure JS projects that just rely on typescript for code intelligence of course.

If Sublime has built-in typescript module you should use it first, otherwise you need to add typescript as @volar/server peer dependency in https://github.com/sublimelsp/LSP-volar/blob/master/server/package.json#L5, and config typescript.serverPath as typescript/lib/tsserverlibrary.js.

Sublime uses Python for its extension API, not TS/JS like VSCode. So it has no business in having typescript anywhere already. That said, your suggestion to add typescript as a peer dependency (or in fact just a normal dependency in that case) is how we normally do it with those LSP-* packages and we have no problem with that. The issue is that the Volar itself won't fall back to it automatically. We would have to create some wrapper JS script (or have that logic in Python) that does that which IMO shouldn't be necessary.

They do not mean it is correct to do so. Yes this is convenient for the language client, but it also urge language client to give up security considerations.

That's another topic but there is really no security in the node ecosystem and we should not pretend otherwise. Just doing npm i alone can do all kinds of bad things on your system. Having a language server be targeted as an attack vector when there are much easier ways to screw someone up, seems of little relevance.
What VSCode's "workspace trust" solution is IMO, is just a way for MS to say "it's not our fault" when something bad happens. It doesn't really provide any meaningful protection since the user who would fall for a security issue like that will likely not be able to properly judge and audit the suspicious project to prevent himself from getting in trouble.
And also it should be editor's choice IMO whether such mitigations should be provided and how.

Edit: I see that you plan to fall back to bundle's typescript when can't find typescript in workspace. If priority use bundle's typescript, and support user config specific typescript path if user want to use workspace typescript, you don't need to implement typescript path finding.

IMO that's not an ideal flow.
As I've said above, the priority should be:

workspace
bundled

I don't use sublime, so I don't know, but I'm sure there are ways to deal with this, such as referring to the angualr (LSP-angular) plugin. https://github.com/sublimelsp/LSP-angular/blob/master/server/package.json#L6
https://github.com/sublimelsp/LSP-angular/blob/master/plugin.py#L13-L16
https://github.com/sublimelsp/LSP-angular/blob/master/LSP-angular.sublime-settings

As said above, there is no problem with this plugin (package) including typescript as a dependency in the same way. The problem is with Volar not having a logic picking the most relavant typescript versions.

Security considerations aside for now, the ideal case IMO is that the workspace typescript is used (when available) since otherwise there can be various incompatibilities between what the project expects and what the Volar expects (typescript introduces breaking changes all the time, even in minor releases). This is especially important for projects that rely on typescript for compilation, not so much on pure JS projects that just rely on typescript for code intelligence of course.

In my opinion, this is the reason why bundle's typescript should be used first. :S

What VSCode's "workspace trust" solution is IMO, is just a way for MS to say "it's not our fault" when something bad happens. It doesn't really provide any meaningful protection since the user who would fall for a security issue like that will likely not be able to properly judge and audit the suspicious project to prevent himself from getting in trouble.

Just because it can't protect doesn't mean it doesn't need to be done. I agree that its greater role is to prevent a lawyer's letter from being sent to you when the company suffers a loss. But for users who always set the trust scheme correctly, this feature is useful.

And also it should be editor's choice IMO whether such mitigations should be provided and how.

If implemented typescript finding in language server, editor's can't. They can only disable entire language server if not trust.

IMO that's not an ideal flow.
As I've said above, the priority should be:

workspace
bundled

This implies the main problem, we have different UX designs for IDE support, so the implementation is different. This is why I put the logic in the language client. You expect that the logic implemented in the language server actually leaks IDE-specific logic to the LSP, and this is a bug for other IDE plugins because it does not conform UX design of other IDE plugins.
Anyway, I have no intention of deciding on your UX design, I just explain the reason why this logic is not implemented in language server.

Security considerations aside for now, the ideal case IMO is that the workspace typescript is used (when available) since otherwise there can be various incompatibilities between what the project expects and what the Volar expects (typescript introduces breaking changes all the time, even in minor releases). This is especially important for projects that rely on typescript for compilation, not so much on pure JS projects that just rely on typescript for code intelligence of course.

In my opinion, this is the reason why bundle's typescript should be used first. :S

No because that can end up with a situation where Volar reports an error while the project itself builds just fine. Or the other way around - project building fails while Volar doesn't report the issue.
We could have a cake and eat it too if Volar would provide an option to "useWorkspaceDependencies". Which is something that Vetur actually provides (and disabled by default).

Even if implemented in language server, it may not be suitable for you. In our implementation, only the typescript in the workspace will be searched. If the workspace does not exist, the search will fail. But what you need seems to be if you can't find it in the workspace, first try to fall back to the global typescript module. I may also can't agree with this design.
I think you can use a one-line node script to solve your needs:
// resolve_module.js
console.log(require.resolve(process.argv[3], { paths: [process.argv[2]] }));

import subprocess
proc = subprocess.Popen(['node', 'resolve_module.js', 'WORKSPACE_DIR', 'typescript/lib/tsserverlibrary.js'], stdout=subprocess.PIPE)
ts_path = proc.stdout.read()
print(ts_path)

Thank you for your thoughts @rchl, @johnsoncodehk and @yaegassy.
Although I mostly have the same point of view as Rafal(@rchl),
I can also see that different people have different expectations to what should be the client responsibility and the servers responsibility. (and it is all debatable)
For now I made a PR to implemented the finding of the ts server on the client. :)


Clear Lap button does not work when timer is paused

@avinash201199 Please assign this issue to me.


Allow setting of the client interface access modifier based on the model access modifier

This PR allows changing the client interface access modifier.  This PR is to address #1363.
Changes:

When run from the command line, the model type TypeAccessModifier is used. When TypeAccessModifier is set in the OpenApiToCSharpClientCommand, it sets the settings ClientInterfaceAccessModifier to the same value. This is required because TypeAccessModifier is found in the Newtonsoft library and not in CSharpClientGeneratorSettings.
CSharpClientGeneratorSettings defaults ClientInterfaceAccessModifier to public
Updates the C# Client.Interface.liquid to use the new  ClientInterfaceAccessModifier setting
Add unit tests to ensure changing the access modifier is correctly changed in the generated C#.

This PR does not address valid combination of setting the generated C# class and model to public, but the interface internal. This would require additional work to expose additional arguments to the command line tool and changes to the UI tools. Someone would need to justify the use case that make this additional combination valid and the effort to implement.
Valid in the table means it is valid C# and will compile.  This PR really addresses the last item in the table.

valid
class
model
interface
Handled?

Yes
public
public
public
Yes - worked this way before

Yes
public
public
internal
No - use case?

No
public
internal
public
No - not valid C#

No
public
internal
internal
No - not valid C#

Yes
internal
public
public
Yes - worked this way before

Yes
internal
public
internal
No - edge case?

No
internal
internal
public
No - not valid C#

Yes
internal
internal
internal
Yes - This PR fixes this specific combination

The failing unit tests pass locally. Is this something that I need to investigate or could it be related to the test runner setup?

The failing unit tests pass locally. Is this something that I need to investigate or could it be related to the test runner setup?

Looks like the same failure I got in #4785 - I wondered if the test timeout I mentioned at https://github.com/RicoSuter/NSwag/pull/4785#issuecomment-1998612818 simply isn't long enough for the macOS CI runner, though I don't have a Mac to test it locally with

Awesome, thanks for your work @pbolduc !

MacOS build failures should be fixed by #4896

https://github.com/RicoSuter/NSwag/issues/1363#issuecomment-2163715770
Now that #4896 has been merged,  @pbolduc could you please re-run the Macos build?
Latest run is from March 17th.
Hoping this PR could be merged soon.
Thanks

MacOS build failures should be fixed by #4896

#1363 (comment)
Now that #4896 has been merged, @pbolduc could you please re-run the Macos build? Latest run is from March 17th.
Hoping this PR could be merged soon.
Thanks

I am not a maintainer on this Repo. I am not clear on how I would "re-run the Macos build". Usually a maintainer needs to re-run the github actions.

MacOS build failures should be fixed by #4896

#1363 (comment)
Now that #4896 has been merged, @pbolduc could you please re-run the Macos build? Latest run is from March 17th.
Hoping this PR could be merged soon.
Thanks

I am not a maintainer on this Repo. I am not clear on how I would "re-run the Macos build". Usually a maintainer needs to re-run the github actions.

Ok thanks for your answer. I thought maybe the creator of the PR could manually re-run the build without having to push a new commit.
I guess we just have to wait for a maintainer then.


Some additives

It would be nice if the player could see his Nick in color.
And colors if someone is an administrator to.

Open seperate issues for this, and im not going to add both nick and rp name


[SPARK-6027][SPARK-5546] Fixed --jar not working for KafkaUtils and improved error message

The problem with SPARK-6027 in short is that JARs like the kafka-assembly.jar does not work in python as the added JAR is not visible in the classloader used by Py4J. Py4J uses Class.forName(), which does not uses the systemclassloader, but the JARs are only visible in the Thread's contextclassloader. So this back uses the context class loader to create the KafkaUtils dstream object.
Also improves the error message. This PR assumes that #4754 is going to make --packages work for pyspark (which is what is referred to in the error message as 1. ).
@davies

Test build #27980 has started for   PR 4779 at commit 7b88be8.

This patch merges cleanly.

Test build #27980 has finished for   PR 4779 at commit 7b88be8.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtilsPythonHelper

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27980/
Test FAILed.

 The existing message was incorrect, @davies and I missed it in the original PR because we probably never tested in distributed mode. Instead, --jars is supposed to be a single solution for adding any jar to both driver and executor. It works for Scala/Java but does not work correctly for Python, hence this workaround.

Test build #27984 has started for   PR 4779 at commit c1fdf35.

This patch merges cleanly.

LGTM, the helper class actually simplify things.

Test build #27987 has started for   PR 4779 at commit fb16b04.

This patch merges cleanly.

Test build #27984 has finished for   PR 4779 at commit c1fdf35.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtilsPythonHelper

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27984/
Test PASSed.

Test build #27987 has finished for   PR 4779 at commit fb16b04.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtilsPythonHelper

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27987/
Test PASSed.

LGTM as discussed with @tdas offline we're gonna change it to private. This is going into master and 1.3.


Fix caching on databroker build

There was a typo that prevents full caching of build artifacts for databroker build (CLI was ok already). Fixed now.
Example build with no cache (so probably also the build here, as changing cache contents invalidates cache)

Subsequent build satisfying cache requirements

(see here https://github.com/boschglobal/kuksa-databroker/actions?query=event%3Apull_request+branch%3Atest%2Fcaching for various test builds)
I also checked where time is mainly used now

20+   sec is always used pulling the cross images (crossb uilds using docker). the cross cargo package is cached and not reinstalled but the docker images are not. I do not intend to look into this/fix this in this PR. Just a hint, if somebody feels bored. I am however not sure if there is even a feasible solution as those images are around 2 GB each and I think there are strict limitations on how much cache can be used before things get evicted)
Another large chunk is basically the "linking" so I think, this has to be accepted.

This seems to fix caching for the build step.
The overall build is still slower than it was since caching of the unit test step was also broken in the original PR.

I suppose always having the target dir under target/ would make it easier to get this right, i.e. using --target-dir "target/cross-$target" in order to work around the issue with building for different systems under cross.

Fixed. I choose to fix the cache definition in workflow instead of changing dirs in script.
(I was thinking if somebody during dev is using plain "cargo build/run" and also in between places with the build scripts, the structure inside the default target folder might seem confusing. Also... easier/faster to test for me just now)


Doesn’t seem to work with Hyper 2

Hej @zachflower,
thanks for that awesome plugin.
Did I configure it wrong, or doesn’t it work with Hyper 2?

Edit: I’m on Mac OS 10.12.6 btw.

Confirm that it isn't working under Hyper 2.0.0

Validated this on my end. It seems that the Electron before-quit event in Hyper v2.0.0 is executed after all windows are closed, rather than before (which was the v1.* behavior).

Just curious if you've had any luck getting this working in Hyper 2.0.0? Thanks so much!

Did this fix itself? Because it works fine for me on macOS 10.13.6.

I think this might have been a Windows issue. Thanks!


[Bug] kompile - lines still wrapped after giving --no-exc-wrap

K Version
$ kompile --version
K version:    5.1.0
Build date:   Tue May 25 12:04:28 EEST 2021

Description
Running kompile with --no-exc-wrap doesn't prevent error lines from being wrapped.
Input Files
File: demo.k
module DEMO
  imports DOMAINS
  configuration <k> .K </k>
endmodule

File: rhs-vars-broken-spec.k
// test error messages

requires "demo.k"

module VERIFICATION
    imports DEMO
endmodule

module RHS-VARS-BROKEN-SPEC
    imports VERIFICATION

   claim <k> .K => _ </k>

   claim <k> .K => A </k>
endmodule

Reproduction Steps
The error lines given by kprove are wrapped (at 75 chars from what I counted).
andrei@q:~/Desktop/issue$ kompile --syntax-module DEMO --no-exc-wrap --backend java  demo.k -d .
andrei@q:~/Desktop/issue$ kprove rhs-vars-broken-spec.k --def-module VERIFICATION  -d .
[Error] Compiler: Found variable _ on right hand side of rule, not bound on
left hand side. Did you mean "?_"?
	Source(/home/andrei/Desktop/issue/././rhs-vars-broken-spec.k)
	Location(12,20,12,21)
[Error] Compiler: Found variable A on right hand side of rule, not bound on
left hand side. Did you mean "?A"?
	Source(/home/andrei/Desktop/issue/././rhs-vars-broken-spec.k)
	Location(14,20,14,21)
[Warning] Compiler: Variable 'A' defined but not used. Prefix variable name
with underscore if this is intentional.
	Source(/home/andrei/Desktop/issue/././rhs-vars-broken-spec.k)
	Location(14,20,14,21)
[Error] Compiler: Had 2 structural errors.

Expected Behavior
--no-exc-wrap should keep the lines long, like
andrei@q:~/Desktop/issue$ kprove rhs-vars-broken-spec.k --def-module VERIFICATION  -d .
[Error] Compiler: Found variable _ on right hand side of rule, not bound on left hand side. Did you mean "?_"?
	Source(/home/andrei/Desktop/issue/././rhs-vars-broken-spec.k)
	Location(12,20,12,21)
...

@andreiburdusa your call to kprove does not have the --no-exc-wrap option, just your call to kompile. The error you get looks like it comes from the kompiler (and indeed it does), but you need to still provide --no-exc-wrap to kprove.
Please close if that fixes it.


sendMessage Duplicate

Adding channel ID to sendMessage as option sends two messages:
One into the channel the message was last sent
Second is into the specified channel ID.

i will test that later

fixed in v1.0.5


#277 Fixed incorrect parse result from plain scalars starting with special values

As reported in #277, the parser generate a incorrect YAML node tree from plain scalars which start with special string values (such as null or True).
This PR has fixed the issue by changing the timing where such special values are detected.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

coverage: 100.0%. remained the same
when pulling 90f23a90db938dba72f3b14674e6eb83b1f89ca5 on bugfix/277_parse_plain_scalars_starting_with_special_values
into 6e1f78a04fbcb1e7eda5693513097b356ac57324 on develop.


Is gating applied to target p ?

Hi,
First of all, big congrats for the paper, it's really fun to read!
I have a minor question regarding how the similarity function is implemented in general.
Eq. (1) in your paper (Sec. 2.2) states that the gating function is applied to both sides, i.e. query and target. However, at Sec 3.2, you wrote that SCE loss is applied to $V_q(q) \odot G_q(q)$ and $V_p(p)$, which is confirmed by the pseudocode in Fig 2.
May I kindly ask your clarification about it?
Thanks in advance!

Hi, @oezyurty . Thank you for your interest in our work.
To clarify:

For inference, we apply gating to both side ( i.e., $(V_q(q) \odot G_q(q)) \cdot (V_p(p) \odot G_p(p)) $).
For training, specially, we fully activate representation on one side (e.g., $V_q(q)  \cdot (V_p(p) \odot G_p(p)) $) to guarantee sufficient co-activation, as we mentioned this concern in Sec 3.3 (a).

We appreciate your feedback and will aim to clarify this further in our final manuscript!

Hi @jzhoubu, thanks a lot for the swift reply!
Thanks for the clarification. Yes, now I saw in 3.3 (a) that you state "we fully activate $V_p(p)$ while sparsify $V_q(q)$". Honestly, I couldn't directly associate this statement with the gating functions, my mistake :)
I think such clarification would definitely help your readers in the future.
Best regards,


Extended UserPrincipal with methods to fetch specific claims easier

Summary
Extended UserPrincipal with methods to fetch upn, unique name, name and claim via claim name
Issue Type

New Feature

Starter Names

active directory spring boot starter

Additional Information

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


Opbeat integration

Despite the fact that Opbeat does not support the Vue, we can use it for VanillaJS\Django perfomance monitoring

Looks kinda useless, maybe later


Get image blob data(or source data, png or jpg) from cornerstone image data

Hello there,
When I call cornerstone.loadImage(imageId) then I get image data;
Is there any way to get image blob data(or source data, png or jpg) from the image data. to use it in image tag.
I know after drawing(with cornerstone.displayImage) I can get the image blob from canvas.
I want to get image blob data without drawing the image.
I found image.getPixelData(). Can I use this function to get image blob data?
Could anyone help me with that issue, please?
Thank you in advance.

I am facing a similar issue too.
I'm looking to send the image blob to a post request.
Did you find any solution?

According to cornerstone's image documentation, there's a function called getImage() that describes the same thing you want. I also tried it, but I get an error that there's no such function in existence.

The solution I found is using image.getPixelData() function. I can get raw data of pixels. I should convert it to 0~255 RGB data. I'm now sure whether this is the best solution.

If you're running orthanc for your database, they have an API to get the image png if you give them an instance id. I'm not sure at the moment of how you can access

any updates on this? really want to load the image into png..

Hello! If you want to get the blob, all you have to do is
const imageBlob = image.getCanvas().toDataURL('image/png');

This won't work unless the image is a color image

This won't work unless the image is a color image


Send image

Hi guys,
how can i send local images?
thanks

currently api able to sending text messages only


Update tutorial links

Changes
We recently updated the introductory tutorial on the documentation
website. That tutorial covers the same content as the one here. To avoid
duplicated efforts and content drift, I'm linking that doc here and
replacing the existing content.
Additionally, this removes references to PipelineResources, addressing #3705 .
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Docs included if any changes are user facing
[ ] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in
(if there are no user facing changes, use release note "NONE")

Release Notes
NONE

I'll update the links at the end of "Getting Started with Pipelines to more useful resources".
For the content not covered in the current "Getting Started" (i.e. credentials and more complex pipelines) I'm currently working on a Build and Push tutorial. Currently it's very much work in progress here.

/lgtm
/test pull-tekton-pipeline-alpha-integration-tests


Debug configuration select box looks broken when opened for the first time

This is just a "cosmetic" bug.

VSCode Version: 1.12.2
OS Version: Windows 10 Version 1703 (OS Build 15063.332)

Steps to Reproduce:

Start VSCode
Open Debug configuration select box

When you open select box for the first time it will look broken (although still functional):

If you open it again it will look ok:

Dup of https://github.com/Microsoft/vscode/issues/25700


Update

Hi Please can you update this to the latest version as its now patched for the Log4j Vulnerability

Hi, i had the same issue with the missing update. For my personal use i created an updated fork with Serviio 2.2.1 and FFmpeg 5.0.
Feel free to use it.
You can find the Docker file here
And the Docker image at Docker Hub here

former developer abandoned this project ? :(

The migration at unRAID from this Docker image to my new Docker image is simple:

stop the old Docker image
install my new Docker image (via Apps Tab.)
mapp the same paths (lib and media) from the old Docker to the new Docker
start the new Docker

That's all folks ;-)

(optional) remove the old Docker image

so i added yours, copy/pasted values / path / port from Former Serviio docker
Seems its working now ;) Thanks!
BTW : how hard is it to make it available in unRaid apps with prefilled values? = easier for everyone

I need to read the publishing guidelines (code signing, registering etc.) for unRaid apps.
Perhaps i find some time til the end of the month ;-)

sure , i did install it already and works...
But this way your docker gets more popular...;)

The migration at unRAID from this Docker image to my new Docker image is simple:

stop the old Docker image
install my new Docker image (via Apps Tab.)
mapp the same paths (lib and media) from the old Docker to the new Docker
start the new Docker

That's all folks ;-)

(optional) remove the old Docker image

Thanks.  as its just a fork I edited the existing template I had and just changed the Repository really simple

so i added yours, copy/pasted values / path / port from Former Serviio docker
Seems its working now ;) Thanks!
BTW : how hard is it to make it available in unRaid apps with prefilled values? = easier for everyone

You can now find my Version at Unraid Community Apps as "gucky79 Serviio"

Thanks!
BTW can you somehow update JAVA in the docker?
As older version was fine, but newer from some reason is not "promoting" on the network. Devices are not able to see it.
https://forum.serviio.org/viewtopic.php?t=23783
But i forgot how i got new version inside the docker :(
Thanks!

Java 8 is a dependency from the Serviio Server (see the Minimal requirements )
But feel free to test a newer Java Version. At the Unraid Docker Tab, you can click on the Serviio Container Icon and then on "Console". By this way you have access in to the running alpine Linux.
The current Java Version is:
/ # java -version
openjdk version "1.8.0_302"
OpenJDK Runtime Environment (IcedTea 3.20.0) (Alpine 8.302.08-r2)
OpenJDK 64-Bit Server VM (build 25.302-b08, mixed mode)

Thanks , but how to install newer version inside docker?
Found this is suggested : Recommended Version 8 Update 321, Release date: January 18, 2022

I think the easiest way is to add the alpine "edge" branch.
The newest Java Version at alpine edge is Update 312 -> https://pkgs.alpinelinux.org/packages?name=openjdk8-jre&branch=edge

To switch to alpine edge branch:

edit /etc/apk/repositories and replace 3.15 with edge
run apk update && apk upgrade

trying to find how to edit, as there is no "nano" or "vim" :(

apk add nano
;-)
I will add the alpine "edge" brache with the next Docker Image Update

getting there :D sry Windows user...
so replace those URLS:
https://dl-cdn.alpinelinux.org/alpine/v3.15/main
https://dl-cdn.alpinelinux.org/alpine/v3.15/community
http://dl-cdn.alpinelinux.org/alpine/v3.15/community
http://dl-cdn.alpinelinux.org/alpine/v3.15/main
---with this?
https://pkgs.alpinelinux.org/package/edge/community/x86/openjdk8-jre

i was able to upgrade it, but from some reason is still not promoting :(
So not really usable for me at the moment, no clue where is the issue...
I try on unraid board.
Thanks for fast action / help ! ;)

What is still not promoting? If you talk about DLNA promoting, then it's not an JAVA issue

Hi @killeriq have you check that you use UDP for Port 1900. DLNA use UDP and not TCP

Hi, i had the same issue with the missing update. For my personal use i created an updated fork with Serviio 2.2.1 and FFmpeg 5.0. Feel free to use it. You can find the Docker file here And the Docker image at Docker Hub here
If you mount path /opt/serviio/config to unraid, serviio start or no ? For me serviio don't start. This is the autor mistake?

Hi @ssersh ,
sounds like that the target path (on the host) is empty (error message "Profiles definition is not present.").
You can fix this by inital copy the missing files from the docker container to your host target path.
Here you find some infos to the docker cp command: https://docs.docker.com/engine/reference/commandline/cp/

@soerentsch, thank you so much, it works! :)


Link nekogram, + vivaldi browser & exteragram.

https://github.com/LawnchairLauncher/lawnicons/issues/169
https://github.com/LawnchairLauncher/lawnicons/issues/203
https://github.com/LawnchairLauncher/lawnicons/issues/198
https://github.com/LawnchairLauncher/lawnicons/issues/193

Thanks, @arun54321!


Mishandling of connect followed by a send request

code snippet:
// Inside app-container.component => componentDidMount()
eosclient.connect()

...

// inside page => componentDidMount()
this.listener = eosclient.getTransactionLifecycle(this.props.match.params.id)

this.listener.onMessage((message: InboundMessage<{ lifecycle: TransactionLifecycle }>) => {
    if (message.type === InboundMessageType.TRANSACTION_LIFECYCLE) {
         this.setState({ lifecycle: message.data.lifecycle })
    }
})

and I get the following error:
Unhandled Rejection (InvalidStateError): Failed to execute ‘send’ on ‘WebSocket’: Still in CONNECTING state.

See https://github.com/dfuse-io/eosws-js/tree/stress-testing

Ok, problem is usage of multiple calls to connect which is not handled correctly. To solve:

In connect, if a connectionPromise already exists, returns it right away
In connect, we need to handle reconnection flow, this is more tricky I think, will need to investigate further.


Add initial DAI support

Implements part of https://github.com/google/shaka-player/issues/2367 and https://github.com/google/shaka-player/issues/2365

Should I directly reproduce the URL or expose it to the application?

@joeyparrish @ismena , Could you accept this PR without integration with the Demo page?

@avelad We most definitely will! :)
Give me some time, I need to set up a Google Ad Manager account, so we can test your code.
Please elaborate on your url question, I'm not sure what you mean yet.
I haven't looked too closely at the PR, but on the first glance it's looking really good! We were all super exited today to see a PR for a such a large feature :)

I'll answer the question on Monday, now I'm not in the office
Please, note that it’s a work in progress. It’s not finalized yet.

I think that almost is done.
There is an error that sometimes events are not thrown correctly, and you don't know if there are ads, but I didn't found yet where the error is.
To test we can use:

The examples are public, but are HLS (only works in Safari due https://github.com/google/shaka-player/issues/1335). I tested with DASH stream but are private.
@ismena , about the URL question:
Should the SS manager load the url automatically (player.load()) or raise a event and the app should load the url by self?
If the SS manager load the url.. Where do I get the starttime? Do you think the startTime should be a configuration such as audio / text preference?
@joeyparrish, do you think it is possible to include this change in v2.6.0? When do you plan to release this version?

@joeyparrish, do you think it is possible to include this change in v2.6.0? When do you plan to release this version?

I'm trying to finish the major manifest refactor for v2.6 before the end of February.  If the ads APIs are stable by then, they will be included in the release.
Otherwise, I'll need to revert them in the v2.6 release branch.  If that happens, we could release v2.7 as soon as they are ready.
We plan to make more frequent feature releases after v2.6 is out, so it shouldn't be a major delay.

@avelad I'll check out your PR and take a close look at it by the end of this week.
Thanks for taking on such a major contribution.
For the url/startTime thing, my opinion might change when I take a closer look at the code, but my initial though is, it should be an optional param to the loadServerSideStream() functions on the ad manager. If it was provided, it goes into player.load().

Okay, I'll make the change tomorrow to admit startime as optional parameter. Thanks!

@avelad Do you think you might have a stream or two to share with me for testing (privately)?
I tried IMA's sample streams, but I'm not getting any ad events from them except for the initial 'loaded' one, but their streams are kind of flaky and I'm not sure if this is an issue with our code vs the streams themselves.
As soon as I can verify the code, I should be able to go ahead and merge the PR. There's still might be some work to hook it into our ad UI properly, but I can work on that as a follow up.

Alright, I finally got to test the code and a big chunk of it works with a few minor tweaks!
I'm gonna go ahead and approve the PR, given it goes through the build bot alright and then start to incrementally get the rest of it to work/add more features.

HERE WE GO!
Thanks for the contribution, Alvaro!
Let me know, if you're ever looking for a new job :p
I will follow up with integrating ad UI. Also, I don't think snapback logic works quite yet, I'll look into it next.
Live stream ad UI integration might take more work as, it seems, we need to be able to parse ID-3 tags for that.


NBS: Prevent concurrent loading of the same table index

By adding per-entry locking to indexCache, this patch allows both NBS
storage backends to ensure that only one goroutine is loading a given
table index at a time. Previoudly, if multiple readers tried to open
the same table at the same time, it was possible for both to get a
cache-miss and then read in the table index at the same time, which
wastes memory.  Now, one will win the race, load the index, and then
the other will see a cache hit.
Fixes #3581

PTAL


missing namespace xml header.

I feel like something obvious is wrong, but I cannot figure it out.  it seems l'm missing xmlns attributes in the response.
getting:
<ListBucketResult>...

expected:
<?xml version="1.0" encoding="UTF-8"?>
<ListBucketResult xmlns="http://s3.amazonaws.com/doc/2006-03-01/">...

@toppk S3Mock does indeed not always set the namespace in responses.
Not sure if S3 itself always does this either.
Do you need the namespace for anything? All S3 SDKs work without it...

it's been a while that I understand xml norms, but this tiny python library I'm using requires it.  (aiohttp-s3-client), which doesn't use a common s3 sdk.
I'm also a bit surprised that s3mock doesn't even send the <?xml> header.   I've tested minio/aws and they both send the namespace & xml header.   That being said, I'll look at this little library i'm using and see if it can be changed there.

S3Mock is implemented using the Spring Framework, which heavily favors the Jackson library.
Historically, Jackson is good for JSON and works OK for XML.
For example, namespace declarations on the root element only (resulting in XML namespace defaulting) do not work:
https://github.com/FasterXML/jackson-dataformat-xml/issues/355
I'll look into the suggested workarounds.
Looks like adding the xml prolog just needs some configuration for Jackson's XmlMapper...

@toppk I released 3.6.0 yesterday which adds the XML prolog and namespace declarations to all responses. :)


Template & styles: pagination & sorting

testing notes
designs
initial
desktop / tablet / mobile
stuck to top
desktop / tablet / mobile
this should look basically the same on both the books list and members list pages - try both so that you can see the different values that show up for the page labels (alphabetical for members, numeric for books). some notes:

the mockups show a "go" button; this will only be present if javascript is turned off. if you want to see what that looks like, follow these instructions for chrome. you don't need to click "go" if js is turned on (the default); just choose a new page and you should go there immediately.
the sizing of the dropdowns will be different from the design. unfortunately we can't auto-scale them based on the text, so we have to set a constant size for them.
the pagination should "stick" to the top of the screen when you scroll past it. when this happens, it should have the slight shadow that's in the designs. if it unsticks, the shadow should go away.
make sure the right labels are visible: on desktop, you should see the "DISPLAYING / SORT BY" text next to the dropdowns. on tablet and mobile, the labels should be above them and disappear when the controls are "stuck" to the top.
check the labels on the "next"/"previous" arrows. they shouldn't show up on mobile, to save space.
it's important that you can still interact with the main menu and home buttons when the page controls are "stuck" up next to them. make sure you can click the home button and/or open the main menu while the pagination is stuck to the top, on all screen sizes.

dev notes
todo

[x] move DISPLAYING/SORT BY labels up out of sticky area on mobile/tablet
[x] fix mobile safari not displaying gradients/fade out effect 😠
[x] make fade effect smaller to avoid a weird optical illusion with chrome input outlines

todo:
[ ] move DISPLAYING/SORT BY labels up out of sticky area on mobile/tablet
[ ] fix mobile safari not displaying gradients/fade out effect 😠
[ ] make fade effect smaller to avoid a weird optical illusion with chrome input outlines

*Using Chrome and Safari on a Mac

The shadow for the pagination stuck to top (scrolled down) does not appear in mobile-mode but is there in desktop and tablet screens: the shadow falls only under the SORTED BY/DISPLAYING tiles.

I disabled js on my chrome (with Devtools) but was not able to see the "go" button on any screen.

Everything else is good and functional.

@wafaisfahany thanks for pointing these out...I'm realizing there are a bunch of things I should have been clearer about!
the location of the shadow under the pagination is determined at the time the page loads, which shouldn't be an issue except in the case where somebody (like you) is testing by resizing their browser window. this means that when you resize it or go into mobile mode, you have to refresh the page first so that it can "realize" the new size of the window and adjust the shadow location accordingly. give this a try and see if it gets it correct.
for disabling js, you also need to refresh right after checking the box. hopefully the article mentioned this, but if not give that a try and see if it changes things. on chrome, refreshing showed me the "go" button after I turned off js.
last thing - the shadow for the pagination won't show up at all when js is off, because js is needed to detect when the pagination gets "stuck" to the top. this is normal. if you turn js back on and refresh, it should work again.

Tested! all good -