FFmpegPostProcessorError: Filtering and streamcopy cannot be used together.

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.06.25. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.06.25

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
user@user:~/Videos/misc$ youtube-dl -v https://www.youtube.com/watch?v=Nyw4rTywyY0
[debug] System config: []
[debug] User config: [u'-x', u'-o', u'%(title)s.mp3', u'--postprocessor-args', u'-filter:a atempo=1.25']
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=Nyw4rTywyY0']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.06.25
[debug] Python version 2.7.12 - Linux-4.4.0-81-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: ffmpeg 2.8.11-0ubuntu<IP_ADDRESS>, ffprobe 2.8.11-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {}
[youtube] Nyw4rTywyY0: Downloading webpage
[youtube] Nyw4rTywyY0: Downloading video info webpage
[youtube] Nyw4rTywyY0: Extracting video information
[youtube] Nyw4rTywyY0: Downloading MPD manifest
[debug] Invoking downloader on u'https://r5---sn-cxaaj5o5q5-tt1l.googlevideo.com/videoplayback?id=372c38ad3cb0c98d&itag=140&source=youtube&requiressl=yes&mn=sn-cxaaj5o5q5-tt1l&pl=22&initcwndbps=1927500&mv=m&ms=au&mm=31&ei=wxJTWZUph_4NgaqF2Ac&ratebypass=yes&mime=audio/mp4&gir=yes&clen=116109407&lmt=1497979072572737&dur=7310.523&mt=1498616436&signature=2F1D9A6388701B8B81C1FDAE28B55DAEF293CB07.678FCF6B0105120EAD0AFE8AA5B946F0C157DF6D&key=dg_yt0&ip=<IP_ADDRESS>&ipbits=0&expire=1498638115&sparams=ip,ipbits,expire,id,itag,source,requiressl,mn,pl,initcwndbps,mv,ms,mm,ei,ratebypass,mime,gir,clen,lmt,dur'
[download] Destination: Strategy (1) - 12 principles for a 21st century conservatism.mp3
[download] 100% of 110.73MiB in 00:41
[ffmpeg] Correcting container in "Strategy (1) - 12 principles for a 21st century conservatism.mp3"
[debug] ffmpeg command line: ffmpeg -y -i 'file:Strategy (1) - 12 principles for a 21st century conservatism.mp3' -c copy -f mp4 '-filter:a' 'atempo=1.25' 'file:Strategy (1) - 12 principles for a 21st century conservatism.temp.mp3'
ERROR: Filtering and streamcopy cannot be used together.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1973, in post_process
    files_to_delete, info = pp.run(info)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/postprocessor/ffmpeg.py", line 530, in run
    self.run_ffmpeg(filename, temp_filename, options)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/postprocessor/ffmpeg.py", line 208, in run_ffmpeg
    self.run_ffmpeg_multiple_files([path], out_path, opts)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/postprocessor/ffmpeg.py", line 204, in run_ffmpeg_multiple_files
    raise FFmpegPostProcessorError(msg)
FFmpegPostProcessorError: Filtering and streamcopy cannot be used together.

Description of your issue, suggested solution and other information
FFmpeg doesn't like it when the copy command and the filter command are in the same line.
I don't know how important the -c copy -f mp4 flags are.
When I take them out, the following command works just fine.
user@user:~/Videos/misc$ ffmpeg -y -i 'file:Strategy (1) - 12 principles for a 21st century conservatism.mp3' '-filter:a' 'atempo=1.25' 'file:Strategy (1) - 12 principles for a 21st century conservatism.temp.mp3'

Configuration File:
#~/.config/youtube-dl/config
# Only Audio
-x

# Speed up audio
--postprocessor-args "-filter:a atempo=1.25"

OS:
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.2 LTS
Release:        16.04
Codename:       xenial

FFmpeg flags:
built with gcc 5.4.0 (Ubuntu 5.4.0-6ubuntu1~16.04.4) 20160609
  configuration: --prefix=/usr --extra-version=0ubuntu<IP_ADDRESS> --build-suffix=-ffmpeg --toolchain=hardened --libdir=/usr/lib/x86_64-linux-gnu --incdir=/usr/include/x86_64-linux-gnu --cc=cc --cxx=g++ --enable-gpl --enable-shared --disable-stripping --disable-decoder=libopenjpeg --disable-decoder=libschroedinger --enable-avresample --enable-avisynth --enable-gnutls --enable-ladspa --enable-libass --enable-libbluray --enable-libbs2b --enable-libcaca --enable-libcdio --enable-libflite --enable-libfontconfig --enable-libfreetype --enable-libfribidi --enable-libgme --enable-libgsm --enable-libmodplug --enable-libmp3lame --enable-libopenjpeg --enable-libopus --enable-libpulse --enable-librtmp --enable-libschroedinger --enable-libshine --enable-libsnappy --enable-libsoxr --enable-libspeex --enable-libssh --enable-libtheora --enable-libtwolame --enable-libvorbis --enable-libvpx --enable-libwavpack --enable-libwebp --enable-libx265 --enable-libxvid --enable-libzvbi --enable-openal --enable-opengl --enable-x11grab --enable-libdc1394 --enable-libiec61883 --enable-libzmq --enable-frei0r --enable-libx264 --enable-libopencv
  libavutil      54. 31.100 / 54. 31.100
  libavcodec     56. 60.100 / 56. 60.100
  libavformat    56. 40.101 / 56. 40.101
  libavdevice    56.  4.100 / 56.  4.100
  libavfilter     5. 40.101 /  5. 40.101
  libavresample   2.  1.  0 /  2.  1.  0
  libswscale      3.  1.101 /  3.  1.101
  libswresample   1.  2.101 /  1.  2.101
  libpostproc    53.  3.100 / 53.  3.100

-c copy is important.
Do not hardcode extension in output template.

That doesn't resolve the underlying issue.
See log below with output name changed to %(title).%(ext)s
$ youtube-dl -v https://www.youtube.com/watch?v=Nyw4rTywyY0
[debug] System config: []
[debug] User config: [u'-x', u'-o', u'%(title)s.%(ext)s', u'--postprocessor-args', u'-filter:a atempo=1.25']
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=Nyw4rTywyY0']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.06.25
[debug] Python version 2.7.12 - Linux-4.4.0-81-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: ffmpeg 2.8.11-0ubuntu<IP_ADDRESS>, ffprobe 2.8.11-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {}
[youtube] Nyw4rTywyY0: Downloading webpage
[youtube] Nyw4rTywyY0: Downloading video info webpage
[youtube] Nyw4rTywyY0: Extracting video information
WARNING: unable to extract uploader nickname
[youtube] Nyw4rTywyY0: Downloading MPD manifest
[debug] Invoking downloader on u'https://r5---sn-cxaaj5o5q5-tt1l.googlevideo.com/videoplayback?id=372c38ad3cb0c98d&itag=140&source=youtube&requiressl=yes&initcwndbps=2082500&mv=m&ei=ViJTWY_oFYeoDK3NnPAI&ms=au&mm=31&mn=sn-cxaaj5o5q5-tt1l&pl=22&ratebypass=yes&mime=audio/mp4&gir=yes&clen=116109407&lmt=1497979072572737&dur=7310.523&signature=2B118F73CF0FA805AD4BD5EF1EF1F95181ACB71C.4E47E46565989881622FDD5A0D009654EE999B11&key=dg_yt0&mt=1498620403&ip=<IP_ADDRESS>&ipbits=0&expire=1498642102&sparams=ip,ipbits,expire,id,itag,source,requiressl,initcwndbps,mv,ei,ms,mm,mn,pl,ratebypass,mime,gir,clen,lmt,dur'
[download] Destination: Strategy (1) - 12 principles for a 21st century conservatism.m4a
[download] 100% of 110.73MiB in 00:42
[ffmpeg] Correcting container in "Strategy (1) - 12 principles for a 21st century conservatism.m4a"
[debug] ffmpeg command line: ffmpeg -y -i 'file:Strategy (1) - 12 principles for a 21st century conservatism.m4a' -c copy -f mp4 '-filter:a' 'atempo=1.25' 'file:Strategy (1) - 12 principles for a 21st century conservatism.temp.m4a'
ERROR: Filtering and streamcopy cannot be used together.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1973, in post_process
    files_to_delete, info = pp.run(info)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/postprocessor/ffmpeg.py", line 530, in run
    self.run_ffmpeg(filename, temp_filename, options)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/postprocessor/ffmpeg.py", line 208, in run_ffmpeg
    self.run_ffmpeg_multiple_files([path], out_path, opts)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/postprocessor/ffmpeg.py", line 204, in run_ffmpeg_multiple_files
    raise FFmpegPostProcessorError(msg)
FFmpegPostProcessorError: Filtering and streamcopy cannot be used together.

I've found the source of the issue and came up with a workaround.
Workaround
Use the --exec flag to run ffmpeg after the fact.
#~/.config/youtube-dl/config
# Only Audio
-x

--audio-format "mp3"

-o "%(title)s.%(ext)s"

# Speed up audio
--exec "mv {} temp; ffmpeg -i temp -filter:a atempo=1.25 {}; rm temp"

Issue
When specifying the -x flag, --postprocessor-args appends the commands to the ffmpeg -y -i 'file:title.m4a' -c copy -f mp4 'file:title.temp.m4a line.
This is a streamcopy command. FFmpeg doesn't decode and re-encode the audio in this command. Conversely, the command I was trying to run (-filter:a atempo=1.25) is a filtergraph command, which does require re-encoding.
FFmpeg doesn't allow mixing of streamcopy and filtergraphs. With the current configuration --postprocessor-args must be streamcopy commands if the -x flag is set. It would be nice if this were documented.


#1: add support for horses in database initialization

Issue resolved successfully

Reviewed and confirmed by PM.

/paulo - can you please review this pull request?

/paulo - can you check now?


Attachments with incorrect base64 contentUrl in sendActivity fails silently

Versions
What package version of the SDK are you using: 4.0.8 (botbuilder, botbuilder-dialogs)
What nodejs version are you using:  v.8.12.0
What browser version are you using: Chrome v69
What os are you using: Windows 10
Describe the bug
When trying to create an attachment that is intentionally corrupt (via an incorrect base64 url), the bot fails silently without throwing an error.
To Reproduce

Create bot with command to trigger send activity with an attachment with the following code:

  try {
    console.log('Start');

    await context.sendActivity({
      type: 'message',
      attachments: [{
        contentType: 'text/plain',
        contentUrl: 'data:image/png;base64,abc\nabc',
        name: 'dot.png'
      }]
    });

    console.log('Succeeded');
  } catch (err) {
    console.log(err);
    throw err;
  }

And on the BotFrameworkAdapter instance, register an onTurnError handler like so:
adapter.onTurnError = async (context, error) => {
  console.log(error);
  await context.sendActivity(`Sorry, ran into an error!\n  ${ error.message }`);
}

Run bot, send command utterance to send activity
Sending the command failed
Note that there is no error in the console, and the line console.log("Succeeded"); is never reached. Similarly, onTurnError is never triggered.

The message fails silently, and after timing out, the Web Chat instance logs a 502 Bad Gateway error.
Sending other messages pass through successfully.
Expected behavior
Since the contentUrl is invalid, the bot should try and catch an error when this command is initiated.
[bug]

@compulim resize once you have an idea of the scope of this issue.

moving to the DLJS project. cc @dandriscoll @ckkashyap this looks to be a platform problem.


Question: search depth limitation

Hi
I have noticed that no matter how fresh latest release is, release-please always checks last 500 (default) commits.
Is it mandatory behaviour? Or it is possible to fetch commits after latest release?
In my case run of release-please takes around 2 minutes no matter how big release is (even if it just one small fix added to one of the packages)

Are you letting release-please tag the releases? How is your repository configured (manifest or programatically via GitHub actions/app inputs)?
Release-please tries needs to find the commit SHA of the latest release. Using a manifest config is usually the most efficient, because the manifest tells release-please what the expected last version is -- it then needs to find the SHA of that release version. Without the manifest, we paginate backwards through commits to try and guess what the last release was.

I am using manifest, and have a "root" package with multiple packages underneath. Only root gets released via release-please but everything gets tagged and that release commit is captured by release-please according to logs.
But it still goes further away in history by some reasons.
Will check or at least do a reproduction repository to demo the issue.

For my release process I have 1 root release with tags @root-v1.2.3, @package-foo-v1.2.3, @package-bar-v1.2.3 and 0 packages github releases. And in the case it finds github release, it finds all the tags but does not go out from the iterator here
because needsBootstrap is true since it consider only github releases and not tags
Here is the small fix #1915


Resizing Images when being compared with DeepFace using FaceNet

I am working on a project that requires scanning faces from an image and cross-checking it with another image. However, the accuracy of deepface tends to be poor when I extract the vectors and compare and I'm thinking the issue with this is the images not being in the proper size. How do I resize the images to (160,160) which is the size required by Google Facenet before the vector embeddings are generated? I use Retina Face to extract the faces and then generate the vectors using DeepFace.represent() function.
Any idea how I could resize it beforehand?

img = cv2.resize(img,(160, 160))


No IntelliSense for collections.deque and parse.SplitResult

Environment data

VS Code version:1.25.1
Extension version (available under the Extensions sidebar): python2018.7.1
OS and version: ubuntu16.04
Python version (& distribution if applicable, e.g. Anaconda):  python3.6.5
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): virtualenv

use python language server with "python.jediEnabled": false, configured.
Actual behavior
import collections
q = collections.deque()
q.

q not show any method for deque
from urllib import parse
parsed_tuple = parse.urlparse("http://www.github.com/aaaa")
parsed_tuple.

parsed_tuple does not show any suggestion like scheme, netloc.

Oddly enough collections work in VS

I can't see parse in the urllib in typeshed. Did you mean
import urlparse

urlparse.urlparse().

https://github.com/Microsoft/PTVS/issues/4613

@MikhailArkhipov maybe because of python3.
In python2:
import urlparse

in python3
from urllib import parse

urllib.parse.urlparse() in Typeshed.

@brettcannon ,
however, it still cannot suggest any thing
from urllib import parse
parsed_url: parse.ParseResult = parse.urlparse("http://www.baidu.com")
parsed_url.

it seems that intellisense failed on namedtuple,
import collections
RequestStartLine = collections.namedtuple("RequestStartLine", "method uri version".split())
start_line = RequestStartLine("GET", "http://www.baidu.com", "HTTP/1.1")
start_line.

start_line also cannot suggest anything

@chen19901225 yep, my note was meant for Mikhail so he knew where to find the Typeshed reference.

@chen19901225 for namedtuple you will need to open a separate issue.

Related https://github.com/Microsoft/vscode-python/issues/2377

Collections issue fixed with typeshed by https://github.com/Microsoft/PTVS/pull/4628


Do not check ssl on server validate

should be parameterized somewhere but this is fix similar to the fix for disable ssl in electron itself.

@alkuzad Can you test it against this url - http://akashnimare.in . If we use strictSSL It will throw a 200 response even though the file https://akashnimare.in/static/audio/zulip.ogg does not exists. In this case this logic fails -
            if (!error && response.statusCode !== 404) {

                document.getElementById('main').innerHTML = 'Add'
                document.getElementById('urladded').innerHTML = newDomain + '  Added';
                db.push('/domain', domain);
                ipcRenderer.send('new-domain', domain);
            }
            else {
                document.getElementById('main').innerHTML = 'Add'
                document.getElementById('urladded').innerHTML = "Not a vaild Zulip server";
            }

@akashnimare right now I'm behind proxy so I can not test your case easily but I've changed zulip.ogg to zulip.oggnotexisting and it throws 404 as expected

@alkuzad I'm holding this PR as of now. Let me come up with a better solution to validate server.


Modernize EE dependencies

Update Java EE dependencies to Jakarta EE 9

Update dependencies in pom.xml
Update import statements (javax.* to jakarta.*)
Update jetty to 11 (very minor edit required in EmbeddedWebServer to
fix a variable's assigned type to match the constructed instance type)
Update jersey to 3
Update hibernate-validator to 7
Use bom for dependency management for jakartaee, jetty, jersey, hk2,
jackson, and jaxb
Remove unnecessary maven-dependency-plugin warning suppressions in
pom.xml
Minimize bundled jars, removing any that did not appear to be required
for functionality

Additionally, fix #1898 by updating LICENSE/NOTICE files while updating
dependencies:

Relocate ellipsis datatables plugin and jquery to subdirectories to
make it easier to reference in LICENSE and associate with distinct
upstream sources for future updates
Remove unused select2.css
Review all bundled external resources (js/css) for monitor and update
LICENSE and NOTICE files accordingly
Review all bundled jars in lib directory of binary tarball and update
LICENSE and NOTICE files accordingly
Standardize the structure of the LICENSE/NOTICE files to avoid
duplication of external license texts where possible (especially for
EPL-2.0, which is massive) and to use the Maven coordinates from
component.xml for easier maintenance

There is a bug with MonitorSslIT with this on my machine, that I'm trying to track down. It seems that there's a mismatch in the way the new SSL handling code on the server identifies the certificate to present and the way our http client code in the test initiates the request, that causes the client to pass SNI information (which the client sets to null) that prevents the test server from satisfying the request. I'm still tracking this down.

I created https://github.com/FasterXML/jackson-bom/issues/40 to deal with the fact that I couldn't use the jackson bom for managing some of the Jakarta EE variants of jackson libraries.


Multiple errors while deleting

Describe the bug
While trying to delete messages im getting multiple eroors. Messages do not start to delete at all.
screenshot provided

You need to use the Special Chrome Browser mentioned in the https://github.com/pishangujeniya/instagram-helper#initial-steps Initial Steps


[backport v1.0] fix: fix intermediate certificate validaiton

Fixes https://github.com/slsa-framework/slsa-github-generator/issues/762

I will trigger the 1.0.3 release after merge


css ids with spaces

Some of our css ids (and classes?) have spaces in them, which could lead to problems

Turns out I already fixed the ids, just some container classes left


"Use /connect to reconnect" doesn't

In v2.2.0, using /connect shows "Connected to the network", but about two minutes later gets disconnected again. No channels are joined.
Note: I am connecting to a ZNC, not directly to a network.

Ugh, new Android app for github, wrong button.

I actually just started experiencing this issue on 2.2.0 myself. I had no problems connecting to ZNC with any prior version.

Yep. This worked in 2.1.0, before I updated my instance to 2.2.0. Seems to be a regression.

When I loose my connection to my znc proxy I see Disconnected from the network, and will not reconnect. Use /connect to reconnect again. but using /connect doesn't seem to do anything for me (no log output from lounge running in the foreground of a terminal on localhost or in the server tab in the UI).
I'm not aware of automatic reconnection ever working for me (even since #18) so I'm not sure this is the same more-recent regression others are describing.

@rib I've never seen auto-reconnection work either, on my instance. It sounds like what you're experiencing is the same issue (same error message, same symptom), but perhaps it's not a regression—just a bug with ZNC specifically?

I'm having this issue too. I'm running git tag v2.2.0 in dev mode with nginx as a reverse proxy. I turned on the debug.ircFramework setting and forced a disconnect by null routing the server. Then I restored the route and ran /connect, and it failed to reconnect. Here's the TL server output and what I saw in the web app.

Thanks @drkitty!
@xPaw, is that any useful to you?

Looks like ping timeout is the initial trigger for the issue.

https://github.com/kiwiirc/irc-framework/issues/56

@xPaw Shouldn't you still be able to manually reconnect, though? Because that's what I was testing here. Or are auto and manual reconnect related somehow?

I'm thinking the quit is throwing it into some invalid state. cc @prawnsalad

Confirmed the bug, there's definitely some weirdness that doesn't do anything after issuing /connect.

@xPaw, do you think that would come from us or irc-framework?

I haven't dug into the code, but I expect it to be in the framework.

This should have been fixed in 2.2.2


Model transformer e2e tests

Description of changes

added e2e tests for model transformer V2
bug fixes to assure the e2e tests pass
implemented on top of #7935 (only the last commit is relevant for this set of changes)

Description of how you validated changes

ran yarn test
ran SearchableModelTransformerV2, IndexTransformer, and ModelTransformer e2e tests

Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@lazpavel this has merge conflicts.


parse error near `&&'

https://github.com/zsh-users/zsh-syntax-highlighting/blob/master/highlighters/main/main-highlighter.zsh#L197
With zsh 5.0.2, zsh installed via apt-get, in Ubuntu server 14.04 (64 bit) raise the following exception

XXXX/zsh-syntax-highlighting/highlighters/main//main-highlighter.zsh:197 parse error near `&&'
zsh-syntax-highlighting: 'main' highlighter should define both required functions '_zsh_highlight_main_highlighter' and '_zsh_highlight_main_highlighter_predicate' in XXX

Maybe ${interactive_comments+'set'} part?

Does adding -n fix it?  That is, make the condition read:
[[ -n ${interactive_comments+'set'} && $arg[1] == $histchars[3] ]]

Feel free to join #zsh-syntax-highlighting on freenode to iterate there.

Or, another option:
(( ${+interactive_comments} )) && [[ $arg[1] == $histchars[3] ]]

@danielshahaf Thanks for quick reply. Adding -n solve problem. Looking forward the next commit :-)

Thanks for the report @lambdalisue!

Thanks a lot @danielshahaf :smile:

I confirmed. Thanks again :-)

You're most welcome.  Feel free to yell at me directly on #zsh-syntax-highlighting on freenode, if I ever break it again :-).


waitUntilXyzDoesNotContainxxx / getWindowSize

waitUntilXyzDoesNotxxx
http://robotframework.org/Selenium2Library/doc/Selenium2Library.htm

contains waitUntilPageDoesNotContainElement --> mapping/redirecting to waitUntilPageNotContainsElement

contains waitUntilPageDoesNotContain --> mapping/redirecting to waitUntilPageNotContains

getWindowSize now marked as robotkeyword

Thanks for your work!


OOPS

A class is a blue print of a object; set of component with their functionalities
Object->object is an instance of a class;
Constructor->the constructor creates an object of its class and returns its;
->The constructor has no return Type;
->the constructor has the same name as a name of the class;( in python we call constructor as INIT).
->the default constructor takes as   no input  parameter and is provided by the compiler;(compiler automatically makes default constructor);
->Encapsulation->it is putting inside a capsule;
->we can achieve encapsulation by using -public, private, protected, package(default);
->Abstraction->concentrating on  using an object rather than on the implementation and other background detail;(learn how to use an object);
Polymorphism->Same command having different meanings. base on the context. and the object to which the command belongs; ex:-mouse with same click on different things can give different results;
Inheritance-> use to  maintain (backward compatibility) with some advancement;
->something add new
->Modify something existing;(cannot remove );
->Ex ;- television transition form black and White to color television;
Modularity-:
module should be separately editable and stored in separate files , separately compilable and separately tested;
public ->accessible everywhere;
private ->accessible in only class;
protected ->accessible in current class and all inherited classes;
Types on Inheritance:-
->Single Inheritance for ex:- every employee is a Person . (Person)(Parent)----->(Employee)(Child)
->Multiple Inheritance:-not in Java;
->Multilevel Inheritance:-(two times single Inheritance);
->hierarchical Inheritance:-multiple single Inheritance;
->hybrid Inheritance :- is not in Java;

To inherit a class, you simply incorporate the definition of one class into another by using the extends keyword.
class subclass-name extends superclass-name { // body of class
}
You can only specify one superclass for any subclass that you create. Java does not support the inheritance of
multiple superclasses into a single subclass. You can, as stated, create a hierarchy of inheritance in which a subclass
becomes a superclass of another subclass. However, no class can be a superclass of itself.
Although a subclass includes all of the members of its superclass, it cannot access those members of the superclass
that have been declared as private.
A Superclass Variable Can Reference a Subclass Object:
It is important to understand that it is the type of the reference variable—not the type of the object that it refers
to—that determines what members can be accessed.
When a reference to a subclass object is assigned to a superclass reference variable, you will have access only to
those parts of the object defined by the superclass.
plainbox      =  weightbox;
(superclass)     (subclass)
SUPERCLASS ref = new SUBCLASS();    // HERE ref can only access methods which are available in SUPERCLASS
Using super:
Whenever a subclass needs to refer to its immediate superclass, it can do so by use of the keyword super.
super has two general forms. The first calls the superclass’ constructor.
The second is used to access a member of the superclass that has been hidden by a member of a subclass.
BoxWeight(double w, double h, double d, double m) {
super(w, h, d); // call superclass constructor
weight = m;
}
Here, BoxWeight( ) calls super( ) with the arguments w, h, and d. This causes the Box constructor to be called,
which initializes width, height, and depth using these values. BoxWeight no longer initializes these values itself.
It only needs to initialize the value unique to it: weight. This leaves Box free to make these values private if desired.
Thus, super( ) always refers to the superclass immediately above the calling class.
This is true even in a multileveled hierarchy.
class Box {
private double width;
private double height;
private double depth;
 // construct clone of an object

 Box(Box ob) { // pass object to constructor
   width = ob.width;
   height = ob.height;
   depth = ob.depth;
 }

}
class BoxWeight extends Box {
double weight; // weight of box
 // construct clone of an object

 BoxWeight(BoxWeight ob) { // pass object to constructor
    super(ob);
    weight = ob.weight;
 }

}
Notice that super() is passed an object of type BoxWeight—not of type Box.This still invokes the constructor Box(Box ob).
NOTE: A superclass variable can be used to reference any object derived from that class.
Thus, we are able to pass a BoxWeight object to the Box constructor.Of course,Box only has knowledge of its own members.
A Second Use for super
The second form of super acts somewhat like this, except that it always refers to the superclass of the subclass in
which it is used.
super.member
Here, member can be either a method or an instance variable. This second form of super is most applicable to situations
in which member names of a subclass hide members by the same name in the superclass.
super( ) always refers to the constructor in the closest superclass. The super( ) in BoxPrice calls the constructor in
BoxWeight. The super( ) in BoxWeight calls the constructor in Box. In a class hierarchy, if a superclass constructor
requires parameters, then all subclasses must pass those parameters “up the line.” This is true whether or not a
subclass needs parameters of its own.
If you think about it, it makes sense that constructors complete their execution in order of derivation.
Because a superclass has no knowledge of any subclass, any initialization it needs to perform is separate from and
possibly prerequisite to any initialization performed by the subclass. Therefore, it must complete its execution first.
NOTE: If super( ) is not used in subclass' constructor, then the default or parameterless constructor of each superclass
will be executed.
Using final with Inheritance:
The keyword final has three uses:
First, it can be used to create the equivalent of a named constant.
Using final to Prevent Overriding:
To disallow a method from being overridden, specify final as a modifier at the start of its declaration.
Methods declared as final cannot be overridden.
Methods declared as final can sometimes provide a performance enhancement: The compiler is free to inline calls to them
because it “knows” they will not be overridden by a subclass. When a small final method is called, often the Java
compiler can copy the bytecode for the subroutine directly inline with the compiled code of the calling method, thus
eliminating the costly overhead associated with a method call. Inlining is an option only with final methods.
Normally, Java resolves calls to methods dynamically, at run time. This is called late binding. However, since final
methods cannot be overridden, a call to one can be resolved at compile time. This is called early binding.
Using final to Prevent Inheritance:
Sometimes you will want to prevent a class from being inherited. To do this, precede the class declaration with final.
NOTE: Declaring a class as final implicitly declares all of its methods as final, too.
As you might expect, it is illegal to declare a class as both abstract and final since an abstract class is incomplete
by itself & relies upon its subclasses to provide complete implementations.
NOTE: Although static methods can be inherited ,there is no point in overriding them in child classes because the
method in parent class will run always no matter from which object you call it. That is why static interface methods
cannot be inherited because these method will run from the parent interface and no matter if we were allowed to
override them, they will always run the method in parent interface.
That is why static interface method must have a body.
NOTE : Polymorphism does not apply to instance variables.


Display all transactions at one page for an account

Now, when I check on an account, I can see all the transactions of the account, the fund transferred, received and the transaction fees incurred. However, I can't see the rental fee of namespaces and mosaics.
To check on the rental fee, I will need to click on the namespace/mosaic, and then click on the block height to link me to the block info and find out about the rental fee incurred.
It would be good to have all the transaction that involved changing the balance of the account listed in one page, like a bank statement.

Well, In that case, I will add the Receipt's TransactionStatement data on the Account Detail page.


Grammarly does not load

Grammarly app does not load on the open-source version, works fine on the closed source one.


tongue squamous cell carcinoma

http://purl.obolibrary.org/obo/MONDO_0000500
The MESH and UMLS close matches can be removed as the exact matches are already there and correct.

Should be done next release
There will be general improvements with the skos calls in the next release, as we've done a lot more work on mesh/umls integration.

not quite done. The original thinking was to 're-annotate' xrefs that upstream sources may make, and to provide a clear interpretation of their semantics
On reflection it would be better to just filter noise as you suggest
I'm not sure this will be in the next release..

Fixed in the specific case, and I believe in the general case too
I believe @cbizon had the same question


Use Git "fresh" method

Git "clean" method keeps most files created by a previous build. Use
Git "fresh" method instead to ignores .gitignore rules and so remove
all generated files (like ".o" files): run a fresh build rather than
an incremental build.

cc @pablogsal @zware
See https://docs.buildbot.net/latest/manual/configuration/steps/source_git.html documentation.

This change should fix<EMAIL_ADDRESS>issue.
Copy of my message:
setup.py failed with:
"RuntimeError: subprocess not supported for isolated subinterpreters"
It seems like aarch64 Fedora Rawhide 3.x doesn't use a fresh build,
but incremental build. I get such error when I use incremental build.
How can I check if the build is incremental or not? How can it be changed?
Victor


Add option to move JS and CSS files to differenct directory than the dist folder

Command
build
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
It might be duplicate of https://github.com/angular/angular-cli/issues/2688. But there is no solution I found to move JS and CSS files to specific folder at the time of build. Even the deploy-url is deprecated. What's the robust solution to do this job without using any external node script to move the files.
Minimal Reproduction
I am using Angular 14 version for now o test this issue.
Exception or Error
No response
Your Environment
ng build

Anything else relevant?
No response

Hi, what is the use-case that you need to move the CSS and JS into a different directory?

Yes actually I am working on micro-frontend, the unauthorized like login/register are made with next js and the authorized like user/dashboard, user/profile are made in Angular. So the pointing to assets like js/css/fonts needs to be in a separate folder like dist/user/{.js|.css*.fonts) and in index.html the src/href should be updated accordingly.

This is rather a specific use-case. In this case it would be better to extend the Angular CLI 3rd party builders. Like https://www.npmjs.com/package/ngx-build-plus


Add support for emailing multiple email addresses

Example configuration:
define('EMAIL_ON_ERROR'<EMAIL_ADDRESS>'some-other@email.com']);

This is already supported, due to PHPs mail function accepting multiple recipients in the $to parameter. Do it like this:
define('EMAIL_ON_ERROR'<EMAIL_ADDRESS>second@example.com');

@markomarkovic Oh, I missed that part in the docs. Thanks!


“assert len(self.hr_set) == len(self.lr_set)" error

Hi Authors,
When I run train.py, it prompts an error as below.
Traceback (most recent call last):
  File "/workspace/aero-main/train.py", line 135, in main
    _main(args)
  File "/workspace/aero-main/train.py", line 127, in _main
    run(args)
  File "/workspace/aero-main/train.py", line 54, in run
    tr_dataset = LrHrSet(args.dset.train, args.experiment.lr_sr, args.experiment.hr_sr,
  File "/home/wsd/workspace/aero-main/src/data/datasets.py", line 136, in __init__
    assert len(self.hr_set) == len(self.lr_set)
AssertionError
ERROR conda.cli.main_run:execute(49): `conda run python /workspace/aero-main/train.py dset=12-48 experiment=aero_12-48_512_256` failed.

prompt error :  assert len(self.hr_set) == len(self.lr_set),then I print the length of both :
len(self.lr_set): 64349
len(self.hr_set): 64353
I  read the code about the length of segment,The logic is to segment the audio based on different sampling rates.
class Audioset:
    def __init__(self, files=None, length=None, stride=None,
                 pad=True, with_path=False, sample_rate=None,
                 channels=None):
        """
        files should be a list [(file, length)]
        """
        self.files = files
        self.num_examples = []
        self.length = length
        self.stride = stride or length
        self.with_path = with_path
        self.sample_rate = sample_rate
        self.channels = channels

        for file, file_length in self.files:
            if length is None:
                examples = 1
            elif file_length < length:
                examples = 1 if pad else 0
            elif pad:
                examples = int(math.ceil((file_length - self.length) / self.stride) + 1)
            else:
                examples = (file_length - self.length) // self.stride + 1
            self.num_examples.append(examples)

I am using the same dataset as the author, which is VCTK (excluding p315 and s5).
can you tell me How should this situation be handled?

Hi,
I need some more info, usually this happens when the sampling rates in the conf file don't match the actual data. Did you change the sample rates in your experiment conf file to be 12000 (for LR) and 48000 (for HR)?
By the way, usually the speakers removed are p280 and p315 - as there were technical issues with their recording as mentioned here: "(However, two speakers, p280 and p315 had technical issues of the audio recordings using MKH 800)."

Thank you for your reply.
Since there is no 12-48 conf file provided in the GitHub repository, I created a 12-48 conf file based on the 4-16 conf file and modified “lr_sr: 12000, hr_sr: 48000, nfft: 512, hop_length: 256”. I have uploaded the aero_12-48_512_256.yaml file as an attachment（Since GitHub does not allow uploading files with the .yaml extension, I added the .txt extension.）.
aero_12-48_512_256.yaml.txt
Also, thank you for reminding me that usually the speakers removed are p280 and p315. Since create_meta_files.py defines
TOTAL_N_SPEAKERS=108
TRAIN_N_SPEAKERS=100
TEST_N_SPEAKERS=8

, there are a total of 108 speakers, so I kept the data for p280 . Is it correct to delete p280 and p315 and keep s5?

After testing, I found that the number of segments produced by the following four files is different, so I deleted these files and it can run normally and pass.
p244_153_mic1.wav
p250_393_mic1.wav
p254_320_mic1.wav
p263_258_mic1.wav

Glad it worked out!


Save document to filesystem support

It often happens in my workflow that some app opens a pdf which opens it in PdfViewer which is the only pdf viewer I have installed. Then I can't save the document in my filesystem because PdfViewer lacks this feature.

I came here to ask for the same feature. A simple "save at..." option would be much appreciated.

Implemented in v14


Replace calls to print to calls to logging

It'd be great if all the print statements about initializing modules are converted into calls to logging (especially for when running MMC chains!). See for instance https://github.com/DarkQuestCosmology/dark_emulator_public/blob/main/dark_emulator/darkemu/sigmaM.py#L12

Thanks. This is now implemented.


Parking card

Summary
Add parking card using webview, as per Issue #771
Changelog
[General] [Add] - Add parking card
Test Plan
Verify card with parking data API at https://4pefyt8qv7.execute-api.us-west-2.amazonaws.com/dev/parking/v1.1/status/
Test on various low tier devices for latency issues.

I have added the feature for persistent userPreferences selections, and have updated the branch with most recent changes from upstream as of now.


Error Saving Model Due to Incorrect Relative Import

Reminder

[X] I have read the README and searched the existing issues.

System Info

llamafactory version: 0.9.2.dev0
Platform: Linux-5.15.0-1064-azure-x86_64-with-glibc2.31
Python version: 3.10.14
PyTorch version: 2.5.1+cu124 (GPU)
Transformers version: 4.46.1
Datasets version: 3.1.0
Accelerate version: 1.0.1
PEFT version: 0.12.0
TRL version: 0.9.6
GPU type: NVIDIA A100-SXM4-80GB
DeepSpeed version: 0.14.4
vLLM version: 0.6.4.post1

Reproduction
command:
llamafactory-cli train phi3.yaml

phi3.yaml:
### model
model_name_or_path: ../../models/Phi-3-mini-4k-instruct

### method
stage: sft
do_train: true
finetuning_type: full

### ddp
ddp_timeout: 180000000
deepspeed: examples/deepspeed/ds_z3_config.json

### dataset
dataset: wildfeedback-gpt4o-sft
template: phi
cutoff_len: 4096
max_samples: 10
overwrite_cache: true
preprocessing_num_workers: 16
mask_history: true

### output
output_dir: ../../models/wildfeedback-december/phi-wildfeedback-gpt4o-sft
logging_steps: 10
save_steps: 50000
plot_loss: true
overwrite_output_dir: true

### train
per_device_train_batch_size: 2
gradient_accumulation_steps: 8
learning_rate: 5.0e-6
num_train_epochs: 1
lr_scheduler_type: cosine
warmup_ratio: 0.1
fp16: true

### eval
val_size: 0.005
per_device_eval_batch_size: 8
eval_strategy: steps
eval_steps: 40

# report
report_to: wandb
run_name: phi-wildfeedback-gpt4o-sft

After training, when saving the model, I got the following error. It seems that the relative path is not constructed correctly.
100%███████████████████████████████████████████| 1/1 [00:05<00:00,  5.78s/it][INFO|trainer.py:3801] 2024-12-19 20:23:29,329 >> Saving model checkpoint to ../../models/wildfeedback-december/phi-wildfeedback-gpt4o-sft/checkpoint-1
Traceback (most recent call last):
  File "/app/src/llamafactory/launcher.py", line 23, in <module>
    launch()
  File "/app/src/llamafactory/launcher.py", line 19, in launch
    run_exp()
  File "/app/src/llamafactory/train/tuner.py", line 50, in run_exp
    run_sft(model_args, data_args, training_args, finetuning_args, generating_args, callbacks)
  File "/app/src/llamafactory/train/sft/workflow.py", line 101, in run_sft
    train_result = trainer.train(resume_from_checkpoint=training_args.resume_from_checkpoint)
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/trainer.py", line 2122, in train
    return inner_training_loop(
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/trainer.py", line 2541, in _inner_training_loop
    self._maybe_log_save_evaluate(tr_loss, grad_norm, model, trial, epoch, ignore_keys_for_eval)
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/trainer.py", line 3000, in _maybe_log_save_evaluate
    self._save_checkpoint(model, trial, metrics=metrics)
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/trainer.py", line 3090, in _save_checkpoint
    self.save_model(output_dir, _internal_call=True)
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/trainer.py", line 3706, in save_model
    self._save(output_dir, state_dict=state_dict)
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/trainer.py", line 3823, in _save
    self.model.save_pretrained(
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/modeling_utils.py", line 2809, in save_pretrained
    custom_object_save(self, save_directory, config=self.config)
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/dynamic_module_utils.py", line 623, in custom_object_save
    for needed_file in get_relative_import_files(object_file):
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/dynamic_module_utils.py", line 128, in get_relative_import_files
    new_imports.extend(get_relative_imports(f))
  File "/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/dynamic_module_utils.py", line 97, in get_relative_imports
    with open(module_file, "r", encoding="utf-8") as f:
FileNotFoundError: [Errno 2] No such file or directory: '/opt/conda/envs/ptca/lib/python3.10/site-packages/transformers/models/phi3/..generation.py'

Expected behavior
No response
Others
I used the same yaml config to train llama and qwen but received no error. I can also save the model using the following code, which supposedly uses the same save_pretrained method.
from transformers import AutoModel, AutoTokenizer

# Path to your model
model_path = "../../models/Phi-3-mini-4k-instruct"  # Update with your path
output_path = "../../models/wildfeedback-december/phi-wildfeedback-gpt4o-sft"  # Directory to save the model

try:
    # Load the model and tokenizer
    model = AutoModel.from_pretrained(model_path)
    tokenizer = AutoTokenizer.from_pretrained(model_path)

    print("Model and tokenizer loaded successfully.")

    # Save the model and tokenizer
    model.save_pretrained(output_path)
    tokenizer.save_pretrained(output_path)

    print(f"Model and tokenizer saved successfully to {output_path}.")
except Exception as e:
    print(f"An error occurred: {e}")

Could anyone please help? Thank you!

Similar issue is found here #6210, but my model path name is correct.

use absolute path


You know how to send images in the message?

it's posibble using the javascript api send images ?
I am interested in helping with the project, if you want we get in touch
I don't know put question label, sry.

Hello! I am interested too. I want send to my friend in WhatsApp not only messages, but also pictures. When you willbe ready to give some version with the feature?

I need help to send pictures


don't generate discovery when discoverable is false

Codecov Report

Merging #143 into master will increase coverage by 0.37%.
The diff coverage is 72.72%.

@@             Coverage Diff              @@
##             master     #143      +/-   ##
============================================
+ Coverage     80.02%   80.39%   +0.37%
- Complexity     1677     1707      +30
============================================
  Files           156      156
  Lines          5591     5779     +188
  Branches        729      803      +74
============================================
+ Hits           4474     4646     +172
- Misses          840      852      +12
- Partials        277      281       +4

Impacted Files
Coverage Δ
Complexity Δ

...i/server/spi/discovery/LocalDiscoveryProvider.java
85.29% <ø> (ø)
10 <0> (ø)
:arrow_down:

...e/api/server/spi/discovery/DiscoveryGenerator.java
91.07% <72.72%> (+2.44%)
74 <0> (+13)
:arrow_up:

...ava/com/google/api/server/spi/auth/GoogleAuth.java
84.61% <0%> (-1.1%)
48% <0%> (+15%)

...api/server/spi/auth/GoogleOAuth2Authenticator.java
86.66% <0%> (-0.44%)
11% <0%> (+1%)

...le/api/server/spi/auth/EndpointsAuthenticator.java
100% <0%> (ø)
6% <0%> (ø)
:arrow_down:

.../server/spi/auth/GoogleAppEngineAuthenticator.java
86.07% <0%> (+0.62%)
17% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


INetworkPrefabInstanceHandler not called on server

Describe the bug
Prefab handlers are never called on the server. The HandleNetworkPrefabDestroy method is never being run. This is the case because the INetworkPrefabInstanceHandler expects that HandleNetworkPrefabSpawn was called on the object which is not the case on the server where the object has to be created manually.
This just silently fails and does not throw an exception.
To Reproduce
Steps to reproduce the behavior:

Run any scene which uses a prefab handler such as a pooling script as a server
Set a breakpoint on HandleNetworkPrefabDestroy.
Breakpoint gets never hit. When the object gets despawned it doesn't run destroy or the prefab handler just nothing happens

Actual outcome
Silent fail.
Expected outcome
Prefab handlers work on the server.
Environment (please complete the following information):

OS: [e.g. Windows 10]
Unity Version: [e.g. 2019.1]
MLAPI Version: develop
MLAPI Commit: c25821d23b4ade30848a90e54d5ec0b579df8296

Additional context
The problem is the following on NetworkPrefabHandler:

The m_PrefabInstanceToPrefabAsset gets propagated in HandleNetworkPrefabSpawn. On the server when I manually take an object from the pool and call spawn on it this function is never run.
I think to avoid this in the future this should be:
if (m_PrefabInstanceToPrefabAsset.ContainsKey(networkObjectInstanceHash))
{
// whatever
}
else
{
    throw new InvalidOperationException("PrefabHandler: Failed to despawn NetworkObject, this should never happen.")
}

This is currently in review and Luke could you close this when you verify that PR-977 addresses this issue?

Fixed by #977


Comparison Operators reveal trouble handling negative denominators

Comparison Operators tests fail when a fraction has a negative denominator, or a negative both parts.
However, we don't want to dissallow negative denominators, it would be great is callers could still use negative denominators in their math.
So what's the solution to this?

The easiest way to fix the comparison operators is to make it so that just the numerators can represent negatives.
But then we lose that data and the user might not see a negative denominator when, e.g., printing to text.

tests for all the comparison operators are having trouble.

We need to maintain component signs. Doing so will make math much easier.
So we need to solve this just for comparisons.
One option is, in each comparison, roll both parts signs into a new numerator and deal with a signed numerator and a positive denominator. For example:
BigInteger newNumeratorSign = olderNumeratorSign ^ oldDenominatorSign;
In this way, if both are negative, both are worked with as positive, if either one is negative, the numerator will be negative. Then we just conditionally flip the denominator sign.

This is solved with pull request https://github.com/jacob-bauer/PerfectDecimal/pull/42


Headers are not included for GET requests of Prometheus client

What did you do?
While integrating Perses to our product, I noticed that HTTP headers are not included in Prometheus GET requests. Because our Prometheus server requires authentication, this leads to failure of get requests for metric name completion while editing panels.
What did you expect to see?
Headers are included in GET requests.
What did you see instead? Under which circumstances?
No headers are included.
System information
No response
Perses version
No response
Perses configuration file
No response
Logs
No response

We are working on a fix for this. Just need some time to verify on our setup.


Initializer supports dtype option and initialization from shape

It seems that you changed many parts of __call__ methods in all inherited classes of Initializer, but is this really necessary? This implementation looks somewhat complicated due to the branching of array is None case and array is not None case. I think this can be simplified.
Suppose that we add a private method to Initializer, say, _set_value, which corresponds to the previous __call__ method. So _set_value just destructively changes the value of given array.
_set_value a method to implement in each inherited class.
And then, suppose we change the implementation of Initializer.__call__ so that it just calls _set_value if array is not None, and otherwise it first initializes an array appropriately according to shape and xp then calls _set_value.
What we should do is just to rename current __call__ method in each inherited class to _set_value.
I may overlook something, but please consider this way to simplify the implementation.

Thank you for your review.
I changed the implementation more simple.
Please review it.

Thank you for your review.
I fixed.

@ir5 Thank you for your review. I fixed.

LGTM except one small comment

LGTM!


Fixed #24200 -- Made introspection bypass statement cache

https://code.djangoproject.com/ticket/24200
Oracle will no longer close connections when table columns have changed. Busting the cache with an increasing counter for get_table_description solves the stale table description problem.
The following tests fail when commenting out the line that increases the counter:

test_add_field (migrations.test_operations.OperationTests)
test_alter_order_with_respect_to (migrations.test_operations.OperationTests)
test_remove_field (migrations.test_operations.OperationTests)
test_remove_fk (migrations.test_operations.OperationTests)
test_alter (schema.tests.SchemaTests)

Yep. Much better. I'm not saying it just because that's what I suggested in the first place :smiling_imp:
(I just searched the web for any uses of connection_persists_old_columns that isn't in core. There was one, in an old MSSQL-jdbc backend, and it was commented out; I think it is safe to remove the flag, but with this fix, we can discuss and do that separately)

Yes it is better. Once I realised that get_table_description was the only query susceptible to stale queries it made sense.
django-mssql also has the feature commented out, so I thought that at some point it may have been necessary. I didn't do an exhaustive search of other backends though.
Happy to commit and backport this?

On Tuesday 10 February 2015 02:30:08 Josh Smeaton wrote:

Happy to commit and backport this?

I would be, but it's 2:40 am here. Not tonight, sorry.

I meant are you happy with me to do it, so I'll take that as a yes. I'll give it a few hours incase anyone else has comments, then I'll merge.

This looks way better than the approach taken in #4090.

Not sure the docs are really needed (this seems more of a cleanup/optimization than a "feature"), but if you think someone might care about it, it looks okay.

1fbe8a2de334bfec5e9b77e36f8a3c1cf2cd70be [master]
3518d51697f1f2e0a516cd82dfc6e05f9f686dab [1.8]


Make sure the important different data points in the current search.ts data prop are ported

Things like geoFence, tags should be available. For the table I need to refer to tags for example.

We've added tags to the SearchStats object (addressing #149):
https://github.com/gravwell/js-client/blob/eb1edda0ad22570030b8647f7e67b7783e2e4f15/src/models/search/search-stats.ts#L38


desk [name] without the dot

Really like the idea and implementation, there's only that one thing with the dot when loading a desk.
I'm sure it's there for a reason, but is there a way to do a workaround for it not to be there, doing a desk [deskname] is so much more elegant than desk . [deskname].

I'll consider this one because there've been multiple requests for it, but how do you feel about doing something like alias d.='desk .' in the meantime? This is what I do personally.

To be honest, my preference is to use explicit commands, so I'd even prefer desk load [deskname], but desk [deskname] would be ideal.

Unsure how desk load foo differs materially from desk go foo or desk . foo in terms of being explicit.

Wasn't aware you could use desk go as an alternative. load feels more natural but go is good as well. The dot simply has no meaning whatsoever.


Issue #5114 warn about history prop in Router-derived components

Adding a warning as suggested in issue #5114.

@pshrmn when I tried just https://github.com/ReactTraining/react-router/pull/5151/commits/99d971676e7131e7832ffa727c611b161fde906b, the test failed because console.error was called 0 times + because there was some spill-over of 2 additional console-errors for unrelated test in HashRouter:
lerna ERR! execute   A <BrowserRouter>
lerna ERR! execute     × warns when passed a history prop
lerna ERR! execute       Chrome 58.0.3029 (Windows 10 0.0.0)
lerna ERR! execute     Error: Expected 0 to be 1
...
lerna ERR! execute   A <HashRouter>
lerna ERR! execute     × warns when passed a history prop
lerna ERR! execute       Chrome 58.0.3029 (Windows 10 0.0.0)
lerna ERR! execute     Error: Expected 3 to be 1

the additional 2 HashRouter warnings are following:
lerna ERR! execute   A <HashRouter>
lerna ERR! execute     × warns when passed a history prop
lerna ERR! execute       Chrome 58.0.3029 (Windows 10 0.0.0)
lerna ERR! execute     Error: Expected 'Warning: setState(...): Cannot update during an existing state transition (such as within `render` or another component\'s constructor). Render methods should be a pure function of props and state; constructor side-effects are an anti-pattern, but can be moved to `componentWillMount`.' to match /<HashRouter.*import \{Router}/

I have better things to do that trying to debug how to test this :(
I would be happy to add spaces in imports if you think this can be merged without warning module and/or someone can figure out the tests :)

when I tried just https://github.com/ReactTraining/react-router/pull/5151/commits/99d971676e7131e7832ffa727c611b161fde906b, the test failed because console.error was called 0 times + because there was some spill-over of 2 additional console-errors for unrelated test in HashRouter:
lerna ERR! execute   A <BrowserRouter>
lerna ERR! execute     × warns when passed a history prop
lerna ERR! execute       Chrome 58.0.3029 (Windows 10 0.0.0)
lerna ERR! execute     Error: Expected 0 to be 1
...
lerna ERR! execute   A <HashRouter>
lerna ERR! execute     × warns when passed a history prop
lerna ERR! execute       Chrome 58.0.3029 (Windows 10 0.0.0)
lerna ERR! execute     Error: Expected 3 to be 1

the additional 2 HashRouter warnings are following:
lerna ERR! execute   A <HashRouter>
lerna ERR! execute     × warns when passed a history prop
lerna ERR! execute       Chrome 58.0.3029 (Windows 10 0.0.0)
lerna ERR! execute     Error: Expected 'Warning: setState(...): Cannot update during an existing state transition (such as within `render` or another component\'s constructor). Render methods should be a pure function of props and state; constructor side-effects are an anti-pattern, but can be moved to `componentWillMount`.' to match /<HashRouter.*import \{Router}/

@pshrmn I have better things to do that trying to debug how to test this :(
I would be happy to add spaces in imports if you think this can be merged without warning module and/or someone can figure out the tests :)

I used wrong syntax in previous change, please disregard. warning(this.props.history, ...) instead of  warning(!this.props.history, ...)

The code looks good to me now, thanks!
warning (and invariant) are always weird to use because they work when the value is falsy instead of truthy.

OK, cleaned up the messaging a bit. Thanks!


Edit command UI different order

Status message shows email before address whereas the person's information shows email after address

This is a bug that exists in AB3 I think its small enough that it does not matter if we have extra time can consider fixing it.

Noted, thanks!


feat: clarify why marker genes are unavailable

Reason for Change
https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/gh/chanzuckerberg/single-cell-data-portal/5608
Changes
shows the info icon for all cell types, and implements the copy changes as described in the figma. there are now 3 "empty states" that we show users:

if the tissue type is blood, then we explain that we don't show marker genes for blood since the cell types are closely related
if there are < 25 cells for the cell type, then we explain that we don't show marker genes with such a low cell count
if there are no marker genes, we retain the existing copy we have

Testing steps
blood cells

cell types with < 25 cells

no marker genes at all

i also updated the geneUtils.ts test to verify that the info icon always shows up
Checklist 🛎

[x] Add product, design, and eng as reviewers for rdev review

[x] For UI changes, add screenshots/videos, so the reviewers know what you expect them to see

[x] For UI changes, add e2e tests to prevent regressions

Notes for Reviewer

LGTM!


provide api to product REST API - /rest/product

I'm working on a project to make a yeoman generator to file bug reports in bugzilla, and I need to get product information from bugzilla so I can auto fill fields like product, component and version. I've implemented the code in my own project to query and process the /rest/product API. Would you consider a PR which adds methods to the BugzillaClient class to get this product information?

Yes.

So here's the thing - you've only changed the built files, eg build/node/index.js, whereas to make changes what you really need to do is edit src/index.js and then run gulp to build the new files.
Also, src/index.js is written using ES6 syntax, which allows you to use es6 promises - there should be no need to include / use the Q library.

I've refactored my PR so they are in src/index.js and also use native Promises. I closed the original PR and opened a new one https://github.com/canuckistani/bz.js/pull/34


[cinder-csi-plugin] Add clusterID Helm value

What this PR does / why we need it:
cinder-csi doesn't currently allow to set the cluster ID via its helm chart. It's hardcoded to kubernetes. This PR adds a Helm value clusterID so that it's configurable.
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

[cinder-csi-plugin] Add clusterID Helm chart value

/lgtm

I don't remember correctly, we should update the version in Charts.yaml for such change?

@jichenjc no, chart version is incremented only on new releases now (you can see that the helm linter in CI doesn't report an error either).

/lgtm
/approve
ok , got it :)


Handle entity resolvers in the n+1 checker

Current implementation is checking only the fields while calculating n+1 issue. But after implementing https://github.com/tailcallhq/tailcall/issues/2639 resolvers could now be defined on types as well.
When considering entity resolver the entity resolver itself should be batched to prevent n+1 issue and that should be applied to any nested fields.
Handle resolvers on types while executing n+1 check

/bounty 150


Running from container cannot use config file.

I have this as a docker-compose file:
version: '3'

networks:
  ext_network:

services:
  fetcher:
    image: ghcr.io/nanos/fedifetcher:latest
    container_name: fedifetcher
    restart: always
    command:
      - "-c /app/artifacts/config.json"
    networks:
      - ext_network
    volumes:
      - artifacts:/app/artifacts

volumes:
  artifacts:
    driver_opts:
      type: none
      device: /data/fedifetcher
      o: bind

When I run it, I keep getting this message:
fedifetcher  | 2023-07-22 22:24:13.993367 UTC: Starting FediFetcher
fedifetcher  | 2023-07-22 22:24:13.993506 UTC: Config file  /app/artifacts/config.json doesn't exist
fedifetcher exited with code 0

The file does exist:
$ ls /data/fedifetcher/
config.json

I can run the container by including all configuration variables in the "command" block, but I'd rather embed the config file in a locked down volume on the server rather than on my workstation where I keep all my compose files.

Hm. I must admit that I’m not really familiar with docker, so other than are you sure you are mounting the correct directory in the correct place, I’m not sure.
The directory should also contain a lot of artifacts, so the fact that it contains only the config file, is a bit of a red flag.

When I run with the config specified in the compose file, I get all the artifact files appearing in that folder.

@politas you'll have to bind mount the config file directly.  Here's my working config:
fedifetcher:
    image: ghcr.io/nanos/fedifetcher:latest
    container_name: fedifetcher
    hostname: fedifetcher
...
    command: -c /app/config.json
    volumes:
****      - /opt/fedifetcher/config.json:/app/config.json****
      - /opt/fedifetcher/app/artifacts:/app/artifacts

@th0mcat , Bind mounting the config the same as your setup did not work for me. I still get a Config file /app/config.json doesn't exist message.

I was able to get the docker-compose file working by changing the command property.
Like this:
version: '3'

networks:
  ext_network:

services:
  fetcher:
    image: ghcr.io/nanos/fedifetcher:latest
    container_name: fedifetcher
    restart: always
    command: " -c=/app/artifacts/config.json"
    networks:
      - ext_network
    volumes:
      - artifacts:/app/artifacts

volumes:
  artifacts:
    driver_opts:
      type: none
      device: /opt/test/artifacts
      o: bind

The device path on the volume shouldn't have an influence since it's handled by the Docker engine.
I just changed so it's easier to clean up :P
I don't know why this works, I just saw that the -c flag was defined like this in the readme.
Maybe Python handles it differently?

Just doing a bit of a clean up. I believe this is no longer relevant. Please feel free to re-open if I've mis-judged this.


Benchmarking

The performance of the client should be compared with other official clients.

Closing temporarily. Not much requirement.


Allow custom scan delay option in config

It would be nice if we had a way to specify how long we want the threads to sleep for in the config file, similar to what is seen in other maps (ie scan delay = 12 seconds)

Agreed. With Niantic's never-seem-to-end policy of increasing time bewteen scans, this would be great even to avoid unnecessary updates.


Fix Bugsnag integration

Apisonator seems to be reporting to Bugsnag only in production, but not in development or staging. That's because nor staging or development are accepted release stages in Bugsnag for Apisonator, the only accepted ones are production and preview (code). However, we stopped using preview as RACK_ENV when we migrated SaaS to OCP, we now use staging instead, so it must be added to the list of accepted release stages.
This PR also introduces some changes I had to make for the TLS PR, and I think this is a good opportunity to move them to master and reduce that PR size. Those changes are:

Remove the integration with Airbrake because it's dead code today.
Update the Bugsnag gem.
Remove the old custom integration between Bugsnag and Resque. The newer Bugsnag release already includes its own, so we can remove ours.

Let's remove airbrake mention from here.
Also, can we get rid of this patched resque, I wonder? :thinking: https://github.com/jlledom/apisonator/blob/fix-bugsnag/Gemfile#L59-L60
We can keep the same version to avoid any changes in behavior (except the airbrake integration) for now, but this way we'll be able to upgrade it later.

Let's remove airbrake mention from here.
Also, can we get rid of this patched resque, I wonder? 🤔 https://github.com/jlledom/apisonator/blob/fix-bugsnag/Gemfile#L59-L60
We can keep the same version to avoid any changes in behavior (except the airbrake integration) for now, but this way we'll be able to upgrade it later.

That makes sense, I'll work on this. We are already removing that fork (and upgrading the resque gem) in the TLS branch anyway

Let's remove airbrake mention from here.
Also, can we get rid of this patched resque, I wonder? 🤔 https://github.com/jlledom/apisonator/blob/fix-bugsnag/Gemfile#L59-L60

Done


include license in wheels

Currently the LICENSE file isn't included in the distributed wheels.

I couldn't find a .stickler.yml file in this repository. I can make one for you, or you can create one by following the documentation.

Coverage remained the same at 96.566% when pulling 48a1e0fbfb76bb299a7f3b9b9553fcaf878bd58f on dougalsutherland:patch-1 into 27e210e066bb3096bf6f112f5e792b95c6b7dc4d on peterbrittain:master.

Coverage remained the same at 96.566% when pulling 48a1e0fbfb76bb299a7f3b9b9553fcaf878bd58f on dougalsutherland:patch-1 into 27e210e066bb3096bf6f112f5e792b95c6b7dc4d on peterbrittain:master.

Coverage remained the same at 96.566% when pulling 48a1e0fbfb76bb299a7f3b9b9553fcaf878bd58f on dougalsutherland:patch-1 into 27e210e066bb3096bf6f112f5e792b95c6b7dc4d on peterbrittain:master.

Coverage remained the same at 96.566% when pulling 48a1e0fbfb76bb299a7f3b9b9553fcaf878bd58f on dougalsutherland:patch-1 into 27e210e066bb3096bf6f112f5e792b95c6b7dc4d on peterbrittain:master.


Computed property does not bind to input field

The issue is that my date property does not bind to datepicker input when my component renders.
For demonstration purpose I added the display of actual bound property:
{{filter.value}}
{{bootstrap-datepicker value=filter.value autoclose=true class="form-control rt-input"}}

This template renders like this - input doesn't have the value displayed above, datepicker also doesn't have selected day marked:

solution: Date object

@soulim - can this be added to the docs? This is not obvious.


Subscription Trial Days not Passed into subscription_data

Bug/Feature? report

Please close one if this is considered a duplicate: https://github.com/stripe-samples/firebase-subscription-payments/issues/17

Additional context

I can probably do a PR adding that field later today.. but not sure if the team still needs to consider updating documentation/other things around trial days, as it's not working in its current state.

It's working as expected on the demo: https://stripe-subs-ext.web.app/ (the premium yearly plan has a 1 day trial)
Please double check that you've configured everything correctly.

trial period days should not be set on the client side as it means a user could extend their trial by modifying it. That's why we use trial from plan.

Duplicate of https://github.com/stripe/stripe-firebase-extensions/issues/105

Thanks! Makes sense. Can't believe I didn't think of that lol I will check my configuration and report back 😀

@thorwebdev  I've double checked everything, and nothing looks off. The extension is working flawlessly, aside from trials. I can't figure out what could be the issue. It's just a matter of setting the Trial Days field in the Price for a Product right?
Also, did you temporarily add the 1 day trial on that demo app? Because it's not showing up for me. Thanks

@thorwebdev I misunderstood the trial_from_plan field, I thought stripe would automatically handle not giving the trial again...!


config

what is the best skycolor_time_change_needed_for_update and lightmap_time_change_needed_for_update value for better performance?

So you know how based on the in-game time the sky and blocks darken?

The optimization from this mod is to cache it-the higher it is, the less often it'll update (so slightly better perf) but it will also make the sky and lighting update slower.

yes what about them

so I should just leave it to the default value?

You could-I don't remember the exact values but increasing them wouldn't impact perf that much.

lightmap_time_change_needed_for_update: 80
skycolor_time_change_needed_for_update: 3
btw thanks!


Terminate crashpad_handler as soon as we finish with Gradle invocation during instrumentation

Workaround for https://github.com/ReactiveCircus/android-emulator-runner/issues/373
Also contains workaround for https://github.com/ReactiveCircus/android-emulator-runner/issues/391

@TWiStErRob I'm trying to use the same workarounds that you had success with but unfortunately I can't get it to work somehow. I'll be very grateful if you could take a pass and offer any suggestions you might have 🙏🏻 https://github.com/airbnb/Showkase/actions/runs/9476982031/job/26110795032?pr=393


Fix for #3: Support tests being split across multiple files

Calling the Prolog file using the -s flag instead of -f allows to check for prolog_load_context(script, true) in the end_of_file term expansion. This way the main/0 is created only once for the directly called test definition file. So it will catch all tests defined in previously consulted files, too.
This is a fix for issue #3. Please not that this is a breaking, i.e. MAJOR change according to semver.

This looks good.  Thanks.  Sorry that it took me so long to review this pull request.

Please not that this is a breaking, i.e. MAJOR change according to semver.

Fortunately we still have major version 0 so breaking changes are allowed.

Thank you for merging :)


Use crosssbram_channel.

Codecov Report

Merging #215 into develop will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #215      +/-   ##
===========================================
+ Coverage    49.21%   49.26%   +0.04%
===========================================
  Files          115      115
  Lines        17229    17240      +11
===========================================
+ Hits          8480     8493      +13
+ Misses        8749     8747       -2

Impacted Files
Coverage Δ

engine/src/lib.rs
50% <ø> (ø)
:arrow_up:

jsonrpc-types/src/rpctypes/transaction.rs
0% <0%> (ø)
:arrow_up:

jsonrpc-proto/src/transaction.rs
86.25% <0%> (+0.88%)
:arrow_up:

jsonrpc-types/src/macros.rs
75.6% <0%> (+14.89%)
:arrow_up:

Continue to review full report at Codecov.

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


重构，将过时的方法改为最新的实现。

What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

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

增加xml，以便单元测试通过

<EMAIL_ADDRESS>


fix(Comment): spread image props on CommentAvatar

Fixes #3250.

Before

After

Codecov Report

Merging #3260 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #3260   +/-   ##
=======================================
  Coverage   99.92%   99.92%
=======================================
  Files         169      169
  Lines        2802     2802
=======================================
  Hits         2800     2800
  Misses          2        2

Impacted Files
Coverage Δ

src/views/Comment/CommentAvatar.js
100% <100%> (ø)
:arrow_up:

src/elements/Image/Image.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


How to optionally match?

Coming from Scala's pattern matching, I'm a bit confused what the library is doing. In Scala, pattern matching performs validation, and whether something matches is optional. Picking up on the email example, how can I actually validate whether the input type satisfies the criteria for unwrapping the values?
match("not a valid email string"):
  email(name, domain):
    echo &"Valid email: {name}@{domain}"
  _:
    echo "not a valid email"

Specifically, how should I express that s.split('@', 1) may or may not work? Is it only possible to unwrap things unconditionally with this library or is there a mechanism for expression optional matches?
My expectation from Scala were to write an unapply function (what is called unpackers here), which can return either

a bool for direct value matching
an Option[T] for unwrapping a single value, or
an Option[(T, ...)] for unwrapping multiple values.
For matching an email for instance, the unapply implementation would get an input string, and return some((name, domain)) only if s.split('@', 1) is valid.

i agree this might make more sense, i think i intended to use e.g. empty seq / invalid values for cases when i want to make sure the unpacker doesnt match, but it's not a good solution indeed.
i still find Option[<T: having length/index>] simpler than having both Option[T] and Option[(T, ..)] but i suspect the single value case might be a sensible optimisation..


Feature request: iOS language support

It would be nice to have support for Swift and Objective-C scanning.
Maybe through running https://mobsf.github.io/docs/#/ as a local server in the docker container.

@donnie-jp Good suggestion. I remember seeing mobsf a while ago and decided against bundling it with scan. The tool is a nice static + dynamic analyzer. The static analysis tool, however, uses basic rules with regular expressions - https://github.com/MobSF/Mobile-Security-Framework-MobSF/tree/master/StaticAnalyzer/views/ios/rules
Other than JavaScript which uses a grepping tool (and about to be removed), all the tools bundled with scan have undergone a level of testing to ensure the findings are accurate with less false positives. The tools should at a minimum use a combination of AST (Abstract syntax trees) with some amount of CFG (Control Flow Graph) to arrive at the findings. If you find such a tool for ios swift please let me know so that we can do some testing collaboratively with sample vulnerable repos.

Maybe you can add support through the new (I think) https://github.com/MobSF/mobsfscan which doesn't require a server/container - or is that more of an alternative to sast-scan?

@donnie-jp Thank you for sharing this link. I looked at mobsfscan now, and it looks like a grep based tool and therefore can hardly work. Let's take an example -  deserialization
https://github.com/MobSF/mobsfscan/blob/main/mobsfscan/rules/semgrep/deserialization/jackson_deserialization.yaml
Deserialization occurs when an attacker controlled input is passed to a method without any validation or sanitization. In case of jackson databind there are methods such as readValue. There are other read methods too and there are other libraries such as XStream that can offer deserialization capability.
The rule used by mobsfscan is very rudimentary. It is looking for an import of ObjectMapper and then a method called enableDefaultTyping.
import com.fasterxml.jackson.databind.ObjectMapper;
              ...
              $Z.enableDefaultTyping();

This is a wrong assumption that deserialization is only possible after invoking enableDefaultTyping. There are many other enable methods and many other forms of performing insecure deserialization. So, the tool has both a high false negative and false positive rate.
This is where my dilemma starts. Should I include the only oss tool available for a given language despite knowing its limitations, or wait for a better tool to be developed? I think I would rather wait, but please don't hesitate to use multiple tools.


Allow backdating created_at

Issue Summary
When publishing content via the Ghost admin API, if the created_at key is set and the date is in the past, the created_at in Ghost is set to (now). Is this intentional?
When looking through the draft posts everything is listed at the time it was uploaded to ghost via the API rather than the created_at time.
Thanks.
Steps to Reproduce
Make a new post using the Admin API, set the created_at key to an rfc3339 timestamp that is in the past.
Ghost Version
CLI: 1.24.2 Ghost: 5.56.0
Node.js Version
v18.16.1
How did you install Ghost?
Manually on Ubuntu headless
Database type
MySQL 8
Browser & OS version
No response
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

I'm lead to believe this is intentional. The created_at of a post is when it was created. If you want to indicate a different date, you probably want the published_at date


Add Gym pictures

Add pictures to show the school's gym and athletics action shots.

pictures taken, processed, sent to Principal for selection

DSC_0132_s.jpg was selected


Release Provisioner Exception Bug Fixes

Hello,
First off, thanks for writing this plugin for Vagrant. It was the deciding factor to start using Azure. :+1:
I am using the Chef provisioner from Vagrant, but it is throwing the following exception:
.vagrant.d/gems/gems/vagrant-azure-1.1.1/lib/vagrant-azure/action/provision.rb:28:in `run_provisioner': uninitialized constant VagrantPlugins::WinAzure::Provisioner (NameError)
I looked around the commit history and found a PR that appears to fix this issue. It looks like these commits didn't make into the last release in February.
Would you mind cutting a new release that includes the commits for this bug fix? I am sure you guys already have a plan for the next release, but I just thought I'd give you a nudge. ;)
Thanks again for the great plugin!

This has been merged into master and will be distributed in the next release. Please verify that master fixes your problem.

v1.2.0 released


Wrap clinica run deep-learning-prepare-data

clinicadl extract should perform the same operation than clinica run deep-learning-prepare-data with the same arguments (currently this is not the case).
Then clinicadl extract should call clinica run deep-learning-prepare-data to prevent unwanted differences.

Closed by PR #103


Application Insights and Business Continuity

Look like the Application Insights can only export the telemetry continuously to a storage account and not to another application insight. Also, not seeing an option to import the telemetry in the destination application insights . Any clarification on this would be very helpful.
https://docs.microsoft.com/en-us/azure/azure-monitor/app/export-telemetry

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6bf9eec4-f13c-55e0-351a-84c1ea8f5a77
Version Independent ID: 6cd22d70-e588-24b2-c1ac-2a3eded96fac
Content: Business continuity plan - QnA Maker - Azure Cognitive Services
Content Source: articles/cognitive-services/QnAMaker/How-To/business-continuity-plan.md
Service: cognitive-services
GitHub Login: @tulasim88
Microsoft Alias: tulasim

@sowsan Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@tulasim88 Hi, could you please help to check this issue and update the document to make it clearer? Thank you.

@YutongTie-MSFT While the Qna Maker documentation mentions the use of Application Insights' feature of continuous export, the question/issue from @sowsan should be directed to the Application Insights docs/service team.

@diberry Thank you Dina, I will close this. We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


Got "Cannot allocate memory" in Go 1.8, after strip app, run successfully.

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.8 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/rm/pwgcqbl91wl2kj768xl2967h0000gn/T/go-build524964552=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
What did you do?
I upgrade my mac from 10.10 (Yosemite) to 10.12.1 (macOS Sierra).
Then I cannot run binary which go build outputted, and no errors found in /var/log/system.log.
But after I strip it, I can run it successfully.

Note:

I can run my project before I upgraded my macos.
I also can run simple hello_world.go, seem the binary is too big to load into memory.

Simplified Makefile
run:
    go build -o app
    ./app

then, I  run followings
$ make run
make: ./app: Cannot allocate memory
make: *** [run] Error 1

$ ./app
Killed: 9

$ ls -l app
-rwxr-xr-x   1 subchen  wheel  5902620 Apr 15 14:08 app

$ strip app

$ ls -l app
-rwxr-xr-x   1 subchen  wheel  5092444 Apr 15 14:13 app

$ ./app
Listening https://<IP_ADDRESS>:8080/v1/ ...

I believe this is a known issue (see #19734). There is also one comment in that thread noting that stripping the binary seems to fix it.

Yes, this is likely #19734. Please upgrade to go1.8.1, that should fix it.
Closing. Comment here if you can still reproduce with go1.8.1. and we'll re-open.

I upgrade to go1.8.1, this issue is gone.
Thanks for your comments.

 If yes, please open a new issue with details (a way to reproduce would be great, if you are allowed to share the code).

Silly me.
Now I found the problem.
Sorry for the noise.

No problem, thanks for checking.


Summary

implementation of queueStats (rpc: summary)

I like the idea of making each mode a sub-field of the summary object.  I think this makes it a little easier to cache.
I made a commit attempting to implement this approach here: https://github.com/ajhyndman/riot-graphql-api/commit/8a87017408dfb41f2590d40eab1dd48bb6fe41bd

I like the idea of making each mode a sub-field of the summary object.  I think this makes it a little easier to cache.
I made a commit attempting to implement this approach here: https://github.com/ajhyndman/riot-graphql-api/commit/8a87017408dfb41f2590d40eab1dd48bb6fe41bd

Superceded by #15


Navigating to CFIs results in incorrect jump location

This bug #121 is still not fully fixed for PR #210
It is harder to reproduce but still exists.
Further investigation is needed.

Let's start a discussion of how we can address this issue.  I believe that Daniel, Juan and Jonah will all be working on it.  Are there other developers that would like to participate?  The next order of business will be to decide on which branch of readium-shared-js we should all work in.  Should we use intel-cfi-improvements or create a new branch to work in?
Once we have decided who and where we can address next steps.

@bluefirepatrick Thanks for starting this thread!
Two items:

Let's stick with the "intel-cfi-improvements" branch
We need the simplest possible test that shows the failure.  I'm thinking an EPUB with one spine item with a reflowable element.  Some simple navigation ("to the beginning, to the end, to the second paragraph", etc.)
If we can then demonstrate the error, then we're in business.


BUG: Unable to Build Docker Image for Xorbits Development Environment: Dockerfile Not Found

Describe the bug
When setting up the environment, following the code on the [Creating a development environment — Xorbits 0.1.2 ] encountered a problem with option 3: using Docker in step 2. The command provided, "docker build -t xorbits-dev ." , seems problematic, as it does not fail to find the Docker file. According to Docker's official documentation, Dockerfile files are stored in subdirectories and the source files cannot be found when compiling.
docker build -t xorbits-dev .

[+] Building 0.0s (2/2) FINISHED
 => [internal] load .dockerignore                                                         0.0s
 => => transferring context: 213B                                                         0.0s
 => [internal] load build definition from Dockerfile                                      0.0s
 => => transferring dockerfile: 2B                                                        0.0s
ERROR: failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount968795007/Dockerfile: no such file or directory

To Reproduce
To help us to reproduce this bug, please provide information below:

Your Python version

Python 3.11.3

The version of Xorbits you use

latest

Versions of crucial packages, such as numpy, scipy and pandas

1.5.3

Full stack of the error.

Minimized code to reproduce the error.
# Build the image
docker build -t xorbits-dev .

Expected behavior

Successfully built

Additional context
Add any other context about the problem here.

Hi, @YELineee .

If you simply want a complete environment with xorbits, you can directly pull the corresponding image from our Dockerhub repository.
Ref: https://hub.docker.com/repository/docker/xprobe/xorbits/general

If you indeed need to manually build a Docker image, for example, to add some additional Python libraries, you can follow these steps:
Make sure your working directory is at the top-level "xorbits" directory, then execute the following command

$ docker build -t xprobe/xorbits:dev --progress=plain -f python/xorbits/deploy/docker/Dockerfile  . --build-arg PYTHON_VERSION=<your_python_version>

Replace <your_python_version> with the Python version you need. Currently, we support versions 3.8, 3.9, 3.10, and 3.11.
There are indeed errors in our documentation, and I will promptly submit a pull request to make the necessary modifications.


Add flagx.File type

This change introduces a new flag type flagx.File that preserves the file name during flag handling and should eventually replace flagx.FileBytes.
In my own experience and two recent PRs (https://github.com/m-lab/measure-saver/pull/7 & https://github.com/m-lab/alertmanager-github-receiver/pull/44 ) it would be more helpful for the file name to be displayed by the flag processing logic, rather than the file contents. The flagx.FileBytesArray introduced this idea. This change adds a new flag type that exposes both the File.Bytes and the File.Name.
This change introduces a new type rather than modifying FileBytes because the shorter name better represents the flag purpose, and prevents the naming stutter of FileBytes.Bytes, as well, this preserves compatibility with existing users of the flagx.FileBytes type until they can be migrated to this replacement.

This change is

Pull Request Test Coverage Report for Build 897

17 of 17 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 85.371%

Totals

Change from base Build 890:
0.2%

Covered Lines:
1313

Relevant Lines:
1538

💛  - Coveralls


Component not working anymore

The component is not working anymore. I tried to fix the errors from the HomeAssistant console, but its still not working. If anyone is interested in further investigating, here is my fork: https://github.com/hanneshier/HA-Vestel-Component?tab

Hello! The problem seems to be that previously working pyvesteltv library (and more specifically SSDP discovery) does not work correctly anymore with the newest Home Assistant versions.
I decided to write new code which uses only TCP port 1986 to communicate with TV and detect whether it is turned on. It keeps things relatively simple as it removes SSDP, some HTTP requests and Websockets but also removes some functionality (ability to start/stop Netflix/Youtube for example).
Please see code here: https://github.com/T3m3z/HA-Vestel-Component/tree/master/custom_components/vestel2
Example of minimal configuration:
media_player:
  - platform: vestel2
    host: !secret vestelhost
    name: "Procaster"
    use_headphone_volume: false
    sources:
      - TV
      - HDMI1
      - HDMI2
      - HDMI3
      - VGA/PC


Feature request: move paragraphs with drag & drop

Hi - great component :)
I'm working on a Ruby on Rails plugin to use it as front-end content editor.
It would be nice to be able to reorder paragraphs using drag & drop (for example with an icon on the left of a row, outside the editing region).
Or if you can provide some info to make a Tool to move up / move down the current paragraph (better if in plain JS), it would be great.

Hi @blocknotes - this functionality is already supported :)
To move a paragraph click and hold for a second or 2 until the drag helper appears. In addition you can move list items and table rows this way, and if you hold longer on lists or tables you can move the entire list or table.


feat(@vtmn/css): add snackbar component

Pull Request checklist
🚨 Please review the guidelines for contributing to this repository.

[ ] Make sure you are requesting to pull a topic/feature/bugfix branch. Don't request your main!
[ ] Check the commit's or even all commits' message styles matches our requested structure.
[ ] Check your code additions will fail neither code linting checks nor unit test.
[ ] Check PR name, it must follow the https://www.conventionalcommits.org pattern
[ ] If it's a new component in CSS, ask for design review

Description
Please describe your Pull Request.
Does this introduce a breaking change?

Yes
No

Other information

❤ Thank you!

I removed the frontend-community review as PR is not ready yet

@GaspardMathon great job! :tada:


Fix Java statuscode usage

This PR adds small changes to the java runtimes to handle status codes differently:

Unauthorized requests result in a 401 code (instead of 500 prior)
Setting the statuscode in RuntimeResponse now correctly sets the code on the http response

Related: #108

Thank you so much for the PR 🤩. We're adding the hacktoberfest-accepted label to ensure this PR counts towards your Hacktoberfest contributions count. With that said, please stay active on this PR to address any comments once you receive a review. Happy Hacktoberfest! 🎃

THANK YOU! All changes merged 🥳
Please reach out to me on our Discord server if you would like to claim your Appwrite swags! As a way of saying thank you, we would also love to invite you to join the Appwrite organization on GitHub. Please share your GitHub username with us on Discord.
You can accept the invite by visiting https://github.com/orgs/appwrite/invitation. By joining our team, you will officially be an Appwrite maintainer on GitHub.
You can change your membership visibility settings, so your new Appwrite team membership badge will show up on your personal GitHub profile.
Please feel free to look for more PRs you might be interested in helping with on our long list of Hacktoberfest friendly issues and help make Appwrite better :)


allow to optionally use file.relative instead of file.path when matching

In the code here, the matching is done against file.path. I'd like to be able to choose to match against file.relative.
Do you have any ideas on how we could configure this? Maybe an optional function that's passed through to Layer that's used there.
By doing this it will allow us to write middleware handlers like this:
app.onLoad('posts/:post/:index*', (view, params) => {
  console.log(params);

  //=> file path => /path/to/files/posts/post-1/index.hbs
  //=> { post: 'post-1', index: 'index.hbs' }

  //=> file path => /path/to/files/posts/create.hbs
  //=> { post: 'create.hbs', index: undefined }
});

By doing this it will allow us to write middleware handlers like this:

Versus what? What does it do now?

fwiw, to do what you want all you have to do is:
router.onLoad('(.*)/posts/:post/:index*', (file, params) => {
  console.log(params) //=> { '0': '/path/to/files', post: 'post-1', index: 'index.hbs' }
});

The patterns that you point out aren't something I thought of. That will work instead of adding additional support for something else. Thanks.


UID of Ambassador image

Some Kube platforms such as OpenShift by default have restricted UID ranges for any new project or namespace, thus necessitating this Kubeflow troubleshooting note.
It would be great if the Ambassador image was modified similar to what the radanalytics.io project has done for some of its images.
/cc @mhausenblas
/cc @tmckayus
/cc @willb

Hey @pdmack, are you on the Ambassador Gitter channel? Seems like it might be useful to chat about this there...

(Meant to comment, not close-and-comment. Sigh.)

coming...

Short version of the discussion we had on Gitter: for this to work, Ambassador needs to work when run as an arbitrary UID with GID 0. More information is here: https://blog.openshift.com/jupyter-on-openshift-part-6-running-as-an-assigned-user-id/ but the short version is that one can rely on group write permissions (how exactly it's "better" to rely on a known group than on a known user I'm not exactly sure ;) ).
As an added bonus, fixing this should also fix #199.


getting errors in console

getting such error in console  when trying to export to xls file.
no file as a result
[876:32944:0808/181101.280:ERROR:device_event_log_impl.cc(214)] [18:11:01.280] USB: usb_device_handle_win.cc:1048 Failed to read descriptor from node connection: A device attached to the system is not functioning. (0x1F)

Исправил ошибку.
Скачайте программу заново и попробуйте еще раз


Cmd+K for searching hotkeys is already reserved in Firefox

... and because of that, it opens the Firefox search engine search, instead of Hotkys' search field.
https://github.com/user-attachments/assets/45e89af8-5117-4301-a259-bd4dd781d2bb
Some thoughts:

I could of course edit the Firefox shortcut, but that is not a sustainable solution. It would still be broken for all other Firefox users.
Check if this Firefox behavior is not preventable, with other words, if we are allowed to override some Firefox shortcuts. Maybe there is a preventDefault() or similar missing somewhere? Of course, it's still a question whether it's desirable to override a default Firefox function.
We can detect Firefox (for example using https://uaparser.dev/); and change the Hotkys shortcut in that case only to an alternative that isn't taken yet by FF.
We can change the Hotkys search shortcut to a safer option by default.

Oh, I didn't know about this. Can you check this site? https://developers.raycast.com/
Does CMD+K work for you here?

Good idea. Cmd+K works fine there (doesn't lose focus). So presumably it is possible to override the Firefox behavior 🙂

Yeah, I wanted to migrate to https://github.com/pacocoursey/cmdk for this as I wanted to get "Raycast" feeling from the web

@th0rgall please check. I think I've fixed this

@solomkinmv confirmed fixed! Thanks!


Remove hard coded OpenStackClient storageClass

Adds OpenStackClientStorageClass parameter to the controlplane
CRD to be able to controll the storage class the pvc for the
openstackclient gets created.

/lgtm