Changing placeholder in file upload map

Currently with multi file upload file number is always added at the eng (e.g. file.0, file.), and I have an input for a mutation like this:
input FileInput {
  file: Upload!!
}

input MyInput {
  files: [FileInput!]!
}

Which would require me to have keys input.files.{Placeholder Here}.file, which doesn't seem currently possible or simply isn't documented.
It works if I construct that string myself, but then I obviously can't use multi file upload with altair client
Describe the solution you'd like
I think some sort of placeholder could be added, like {} or $ which would be replaced by the file number
Additional context
Example with manually mapping files:

@ThirVondukr is what you are describing something that is supported by the GraphQL multipart request spec?
Assuming we added this placeholder approach to Altair, let's consider how it would work:
Single mode means there is a single named file variable involved (i.e. it is not a list, so no number in the variable name/path) and the GraphQL type for the variable is not a list type
Multiple mode means there is a list of files assigned to one named file variable, which means each individual file is referenced by a number index (corresponding to the position of the file in the file list)
Using the example you have above for the placeholder, input.files.$.file, the $ placeholder is meant to correspond to the position of the file in the file list, which means for each expanded entry (input.files.0.file, input.files.1.file, etc), there can only be one file, so this can only be single mode (although you might be considering input.files as the "multiple" part but seems like a different thing from what you're describing) (or can you imagine how this can be multiple mode, with e.g. input.files.0.file having a [Upload!]! list type?)
While this will be a quality-of-life improvement over what we have today, it doesn't enable any new use cases. Does that summarize this properly?

Honestly I'm not sure if multipart spec explicitly supports that, but you can add/pass files into nested inputs.
I don't think placeholder makes a lot of sense sense to use with single file inputs.
Since I created the issue some time ago I don't remember a lot of details, but placeholders aren't something that should be explicitly supported by the multipart spec since it's purely client side, for example that's how I've done it in my python test client:
response = await authenticated_graphql_client.query(
    QUERY,
    variables={"input": input},
    files={
        "input.arts.$.image": [
            GraphQLFile(
                name=f"{i}.png",
                buffer=create_dummy_image(),
                content_type="image/png",
            )
            for i in range(arts_count)
        ],
    },
)

internally it just unwraps it into a list of inputs with names such as input.arts.0.image, etc:
for i, file in enumerate(file_list):
    id_ = str(uuid.uuid4())
    if placeholder in key:
        file_key = f"variables.{key.replace(placeholder, str(i))}"
    else:
        file_key = f"variables.{key}.{i}"

Looks like it should be supported by the spec: https://github.com/jaydenseric/graphql-multipart-request-spec?tab=readme-ov-file#curl-request-2

You can now use the $$ placeholder to achieve this. https://altairgraphql.dev/docs/features/file-upload


Fixed go example in readme file

Fixes #87

Codecov Report

Merging #88 into master will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #88      +/-   ##
==========================================
- Coverage   64.59%   64.48%   -0.11%
==========================================
  Files          22       22
  Lines        1881     1881
==========================================
- Hits         1215     1213       -2
- Misses        535      537       +2
  Partials      131      131

Impacted Files
Coverage Δ

scheduler/scheduler.go
74.08% <0%> (-0.73%)
:arrow_down:

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 f29515f...79eee96. Read the comment docs.


Adds test case for http with missing protocol

And header_names keyword in rules
See https://github.com/OISF/libhtp/pull/195

Ready for merge after https://github.com/OISF/libhtp/pull/236 got merged
Than, we can close https://redmine.openinfosecfoundation.org/issues/2619

Merged, thanks!


Update FAQ

remove "soon" from open source
add link to creator plugin
add link to GitHub

Hi @ewoerner ,
Hope you are doing well! I can work on this issue for you. Please assign it to me when you get a moment, so I can get started.
Thanks!


[Docs] Update of UPGRADE file and installation instruction

Continuation of #184

Thank you, Łukasz! :tada:


GroupBy alerts

This is still in progress, but it's a pretty big PR so it would be good to get some eyes whenever.
Implements #742
TODO:

Add enter and scrolling support for auto-complete
Clean up all the filtering and business logic in ui/app/src/Views/AlertList/Views.elm, add tests for it
More testing in general, probably

MAYBE:

Highlight portions of matching text? Don't think this adds a ton to the experience, though ..

@stuartnelson3 This looks very cool! If it's ok with you I will delay my review, as this is not part of the 0.7 milestone which I would like to focus on first to get the new UI into master as soon as possible.

@stuartnelson3 I added keyboard controls for the autocomplete, and changed the style a little so more items fit on the screen.

@mxinden sure thing. But maybe it'll make it if I get some free time to finish this :)

This looks really good. Small Todo: When clicking on an autocomplete element it should be added like when pressing enter.

@stuartnelson3 I had to rebase because of the conflicts.

I synchronised the group bar with the location and implemented the default grouping by alert name. I also updated the design of the alerts page, it looks a little cleaner now.

@stuartnelson3 @mxinden I'm satisfied with the result now, I think this is ready for the final review. Would really like this to be merged into master.

@w0rm @stuartnelson3 Cool, sounds good. I will have a look at it tomorrow morning. Maybe we can merge this into ui-rewrite before merging #789. Whatever you guys prefer.
Sorry for all the spamming through changing the destiny branch. Forgot that #789 is not yet merged.

@mxinden I told a few colleagues about the UI rewrite and they were pretty excited to try it.

Would love to get this in, but there's a bug that needs fixing first.
I would also like to make the styling on silences and alerts a bit more similar before we release. I like the direction @w0rm is going in, but remembering to keep it compact is important. I think we're on the right track, and a lot of the requests for a "more compact" view are really just because there was no good filtering previously, but making sure it stays easy to scan over a large number of alerts is still important.
Also -- it seems like the silence view page lost the ability to edit or expire? What was the motivation for this? I think it's important to be able to hold on a link to a silence so that a user can expire/modify it without having to search through the whole silence list. Not sure where it was introduced, just noticing it now. There's also an inconsistency in displaying matchers (instance="ip-1-2-3-4.com") and labels (instance=ip-1-2-3-4.com).

@stuartnelson3 Not being able to expire/edit a silence on the SilenceView page was introduced by me. Sorry about that. I have created an issue to get that back in (#794).

@stuartnelson3 I think it's  pretty compact and scannable, but also clean and lightweight. The default grouping pulls out the alert names, but even if this grouping is removed, they are the first in the matchers list

@brancz @fabxc liked the new design of the AlertList view as well. Only thing we would suggest visually separating each alert more e.g. with a line in between?
Other than that only blocking issue that I see right now is that the list of alerts is not refreshed when removing a filter, like described by @stuartnelson3 in this comment.
It would be great if you could style the SilenceList view accordingly as well. But I would suggest to do this in a separate PR as this one is already pretty big.

@mxinden sgtm

@stuartnelson3 I changed the code to always fetch the alerts, but I want to keep the Initial state, because of my experience we may need it in the future. This is NotAsked state from http://blog.jenkster.com/2016/06/how-elm-slays-a-ui-antipattern.html

@w0rm As far as I can tell the silenced url query parameter is not yet implemented in the /alerts endpoint. Could we hide the checkbox for now and unhide it as soon as this is implemented on backend side?

I would prefer to keep the silenced checkbox, since this is already implemented in the client side, and then we can just swap it out when we get to implementing it on the server side.

Big thanks to @w0rm for pushing this through, I've been busy lately, and lack his awesome experience with elm and front-end design/ux. kudos!

And: It's not a huge burden to have Initial, and I remember removing that from our implementation of remote data because I had no reason to differentiate that state from Loading, but it's fine to leave it.
So, from my side, as soon as we get the silences page to look like the Alerts page, and maybe adding a light-gray 1px line between alerts, we're good to go!

@stuartnelson3 I restored showSilenced checkbox

@mxinden @stuartnelson3 I addressed your comments (one was contradicting, so I chose to keep the checkbox) and made the ci green (had to bump my local elm-format, and compiled the bindata using the docker image instead of running make assets cause they produced different results).


Customizing the format of needs_incoming

I looked though the documentation but couldn't find an answer so was wondering if anyone can help.
When linking a need with another need in the jinja templates using :need_incoming:id or  :need_outgoing:id by default you will get the following format:
Title (id)
Is it possible to remove the title and only display the id, if so how?

I figured out that I should reference the need with :need: and use the option 'needs_role_need_template' to customize the reference.


OneDrive for Business unsupported for CI

Hi @snobu, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@dariac - Please review this contribution from @snobu to your article in the public repository.

Anyone monitoring this PR?


Error for wd:Q149998

TypeError: Cannot read property 'P1813' of undefined
    at getPlace (/packages/plugin-wikidata/lib/prop.js:61:50)
    at parseProp (/packages/plugin-wikidata/lib/prop.js:117:14)
    at parseEntity (/packages/plugin-wikidata/lib/entity.js:67:42)
    at Array.map (<anonymous>)
    at Object.<anonymous> (/packages/plugin-wikidata/lib/entity.js:135:50)
    at Generator.next (<anonymous>)
    at asyncGeneratorStep (/packages/plugin-wikidata/lib/entity.js:20:103)
    at _next (/packages/plugin-wikidata/lib/entity.js:22:194)
    at processTicksAndRejections (internal/process/next_tick.js:81:5)

Underlying problem: it has already fetched Q17 (country) (due to a currently unavoidable problem) and decides not to fetch it again, which makes sense. However, this does not immediately fill in the info in the place it's needed.


Remove invalid pytest markers and turn on strict mode

I found pytest.mark.skipIf in test code but it is not working (it should be pytest.mark.skipif).
Removed the decorator as it seems unneeded, and also enable checkign unused markers.

/test mini


correct some misspellings to make Go Report more happy

Signed-off-by: Helen<EMAIL_ADDRESS>

Codecov Report

Merging #1217 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1217   +/-   ##
======================================
  Coverage    28.1%   28.1%
======================================
  Files          28      28
  Lines        2832    2832
======================================
  Hits          796     796
  Misses       1887    1887
  Partials      149     149

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 a850427...d3d1987. Read the comment docs.


Added HTML encoding

Hi! Thanks for making the tool! I wanted to use it to get a custom neovim cheat sheet printed out. As I was preparing my template I realized the strings are not encoded for HTML (I couldn't figure out why my keybinds where missing and then realized things like e.x. <C-w> were being treated as HTML), so I added a part encoding every string that's going into the HTML. I'm very new to Go so do let me know if this could be handled better in the language!

Hey! Thanks for the PR, that's a great addition and necessary one! But there is much simpler way to handle this - you can simply modify escape html characters by adding html filter in the template where the strings are being rendered, e.g. {{ $item.Term }} -> {{ $item.Term | html }} and you should achieve exactly the same result. I think this solution is much simpler. That being said, I don't want to take credit for finding this away from you, so if you're ok with that feel free to modify the pull request and I'll merge it :)

Thanks a lot, I made the changes


READ If You Have Login Issues

As of recently new logins may get stuck at a challenge_required error. At the time of writing there's no viable solution for this problem. Please don't create any new issues asking me how to solve this error.
Instagram requires you to verify your login as your login attempt is considered suspicious by their systems.
The API dependency currently does not support challenge solving therefore there's nothing I can do.
A potential workaround is present here: https://github.com/notcammy/PyInstaLive/issues/57#issuecomment-524650248
Thanks.

At the time of writing there's no viable solution for this problem.

I see. It's a big issue though, the purpose of PyInstaLive is to download livestreams which it doesn't do anymore. It's not just a minor issue but rather something major that prevents this program from... working.
Do you have any idea what the cause of this is?

Do you have any idea what the cause of this is?

Probably some internal changes at Instagram.

With instaloader I had similar issues but solved it by specifying the exact same useragent I also used when signing up & using the Desktop Version on my computer. So that it’s always using the same and already known/logged useragent instead of the instaloader (/pyinstalive) specific useragent.
I assume you have already considered this?

With instaloader I had similar issues but solved it by specifying the exact same useragent I also used when signing up & using the Desktop Version on my computer. So that it’s always using the same and already known/logged useragent instead of the instaloader (/pyinstalive) specific useragent.
I assume you have already considered this?

The problem has suddenly gone away for me personally a week after I stickied this so I haven't done any further research, but i'm confident it was a location/IP problem in my case.

Thanks! That’s good to know. I’ll investigate further in case it happens for me too.

If you create a new account on the same device you're running pyinstalive and don't verify the email, it works

If you create a new account on the same device you're running pyinstalive and don't verify the email, it works

i try it....and this is work for me. why i have problems with my different accounts? :(

@notcammy I can't install this at all with my version of pip using the command you gave or whatever is causing the issue, I don't know. Error says "pip no such option process-dependency-links" is there a quick fix for this? Another project I installed had requirements.txt and that worked for me. Thanks for any suggestions :)

@notcammy I can't install this at all with my version of pip using the command you gave or whatever is causing the issue, I don't know. Error says "pip no such option process-dependency-links" is there a quick fix for this? Another project I installed had requirements.txt and that worked for me. Thanks for any suggestions :)

See https://github.com/notcammy/PyInstaLive/issues/46#issuecomment-487759180
I'll look into adding a requirements.txt file some time later this week

I got arround the "pip no such option process-dependency-links" changing some things (sorry for the poor fix with no version check, I don't know how to do that):
I changed, on setup.py:
install_requires=[
        'instagram_private_api>=%(api)s' % {'api': _api_version},
        'instagram_private_api_extensions>=%(ext)s' % {'ext': _api_extensions_version},
        'argparse',
        'configparser'
    ],
    dependency_links=[
        'https://github.com/ping/instagram_private_api/archive/%(api)s.tar.gz'
        '#egg=instagram_private_api-%(api)s' % {'api': _api_version},
        'https://github.com/ping/instagram_private_api_extensions/archive/%(ext)s.tar.gz'
        '#egg=instagram_private_api_extensions-%(ext)s' % {'ext': _api_extensions_version}
    ],
to:
    install_requires=[
        'instagram_private_api @ https://github.com/ping/instagram_private_api/archive/master.tar.gz',
        'instagram_private_api_extensions @ https://github.com/ping/instagram_private_api_extensions/archive/master.tar.gz',
        'argparse',
        'configparser'
    ],
I ran python setup.py sdist to build the tar.gz and then installed by pip.
pip output here: https://pastebin.com/8XBHYCL0
Creating a new account solved the "challenge" problem as stated before
Doing all this I could download a replay
I used this to change the dependecies: https://stackoverflow.com/questions/12518499/pip-ignores-dependency-links-in-setup-py
Sorry for the big text and poor english. Thanks for maintaining this project!

@notcammy Can the readme be updated to address this or somehow work with the current version of pip (19.x) where the --process-dependency-links option has been removed, and will similarly remain inaccessible going forward? @FlyingWolFox's fix might work if I were to clone the repo and modify the source, but I'm still not sure how to install the script simply from the command line (and installing an old version of pip via Homebrew doesn't seem easy/possible either). Addition of a requirements.txt sounds like it'd be great addition/fix, from my understanding.

@FlyingWolFox @frozenpandaman I've updated setup.py to no longer use dependency_links and added a requirements.txt file as well just in case, hopefully that should resolve any problems you may have installing PyInstaLive with newer pip versions.
For the user not much has changed regarding the installation process, you simply no longer have to pass --process-dependency-links when running the install command. setup.py should take care of the required dependencies.

@notcammy Works great! Thank you!

Is there any way to run the command via a cron job instead of a loop? Thanks!

@greenzwiz

Is there any way to run the command via a cron job instead of a loop? Thanks!

I don't see a reason why this shouldn't be working. there is enough information on the internet about cron jobs (or Task Scheduler if you use Windows) and plenty of guides on how to set one up so feel free to try

I saw the fork from https://github.com/kn0wm4d/instagram_private_api which included an interactive way to verify your account, but the cookie wasn't written, so I readded it back: https://github.com/RomanKarwacik/instagram_private_api . Now it works just fine, no need to create a new account.

I saw the fork from https://github.com/kn0wm4d/instagram_private_api/tree/checkpoint which included an interactive way to verify your account, but the cookie wasn't written, so I readded it back: https://github.com/RomanKarwacik/instagram_private_api/tree/patch-1 . Now it works just fine, no need to create a new account.

Thanks for the head's up. I will not be implementing this into my own project as it does not originate from ping's private api library until it gets merged by ping himself. If any login problems occur then this is more than likely the result of suspicious behaviour (e.g. spam) which goes against the intentions of my own project.

I saw the fork from https://github.com/kn0wm4d/instagram_private_api/tree/checkpoint which included an interactive way to verify your account, but the cookie wasn't written, so I readded it back: https://github.com/RomanKarwacik/instagram_private_api/tree/patch-1 . Now it works just fine, no need to create a new account.

@Romern could you please describe, how to use your version of instagram_private_api with pyinstalive?

@kautaleksei
This should work.
pip install git+https://github.com/Romern/instagram_private_api.git@patch-1

When logging in in pyinstalive you will then be asked which verification method you want to use, then for the pin and then pyinstalive should continue as usual.

@Romern thank you, it's working.
In my case I have installed instagram-private-api version from pyinstalive package, so need to do next steps:

uninstall installation of instagram-private-api via command:
pip uninstall instagram-private-api
install your version as you wrote above:
pip install git+https://github.com/Romern/instagram_private_api.git@patch-1

Then when challenge appear it's asked for code from SMS or Email:
Choose a challenge mode (0 - SMS, 1 - Email): 0 Enter code received:
Thank you for your help.

I have followed these steps:
pip uninstall instagram-private-api
pip install git+https://github.com/Romern/instagram_private_api.git@patch-1
but I am still unable to use it.
'''
PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\Scripts\pyinstalive-script.py", line 11, in
load_entry_point('pyinstalive==3.2.2', 'console_scripts', 'pyinstalive')()
PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\pyinstalive\startup.py", line 313, in run
pil.ig_api = auth.authenticate(username=pil.ig_user, password=pil.ig_pass)
PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\pyinstalive\auth.py", line 113, in authenticate
datetime.datetime.fromtimestamp(cookie_expiry).strftime('%Y-%m-%d at %I:%M:%S %p')))
TypeError: an integer is required (got type NoneType)
'''
Any idea about how to fix this?

@jozephbrasil Seems to be a problem with the API dependency trying to retrieve cookie information
Please delete any existing cookie files that exist and try again. If error persists let me know

If you create a new account on the same device you're running pyinstalive and don't verify the email, it works

It didn't work for me!
I run the script on an Ubuntu Server.
I installed OpenVPN on my server then I used this VPN to create an Instagram account, this means that I use the exact same IP address on the server to register a new Instagram account.
When I try to download a stream it says:
Could not login: {"message": "Sorry, there was a problem with your request.", "status": "fail"}

@jozephbrasil Seems to be a problem with the API dependency trying to retrieve cookie information
Please delete any existing cookie files that exist and try again. If error persists let me know

Thank fyour for work help. But I am not finding where the cookies are installed in to delete it

@dvingerh I have uninstalled everything and installed again. The problem persists. Do you have some idea?

@dvingerh Thank you for helping. But I am not finding where the cookies are installed in to delete it because apparently there is no cookie files in directory. I have uninstalled and installed again and the problem persists.
[I] Overriding FFmpeg binary path: C:\ffmpeg\bin\ffmpeg.exe [W] Unable to find cookie file: codeuser.json [I] Creating a new one. overrided login() unhandled exception 'NoneType' object has no attribute 'get' [I] Successfully logged into account: None Traceback (most recent call last): File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.8_3.8.1008.0_x64__qbz5n2kfra8p0\lib\runpy.py", line 194, in _run_module_as_main return _run_code(code, main_globals, None, File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.8_3.8.1008.0_x64__qbz5n2kfra8p0\lib\runpy.py", line 87, in _run_code exec(code, run_globals)

Im troubled by the cookie file too many time. I create THE new account on the same device / IP Address / location the application but it didn‘t work. Always show the text like:
PYINSTALIVE (SCRIPT V3.2.3 - PYTHON V3.8.3) - 06-02-2020 02:51:16 PM
Unable to find cookie file: shohuko39.json
Creating a new one.
Could not login:

can someone create a dummy instagram login or send me an example of how a working cookie file looks like .

When I had this challenge_required and login_required error I bypassed it by redownloading the source code and run my scripts again. I am not sure how and why that worked but since I have two IG accounts my 2nd throwaway account got stuck on login_required and I didn't pay much attention since it's a dummy account. Then my main account got stuck on challenge_required today and I had to do something.
Being stuck on that means that the actual IG finds a suspicious activity on your account and you have to go through some verification to re-enable it. They send you some kind of a code to your phone or mail or something (they even ask you to change your pass) and you can log back in. But PyInstalive still gets stuck on challenge_required. I guess the old cookie keeps that happening.
So I redownloaded the latest PyInstalive and gone through the setup process by setting up the .ini file etc etc and it works fine now for both of my accounts.
Maybe that will help you, I don't know.

I started to log the output to a file to see if I can find how long it takes for a challenge_required prompt. I was running my delays randomly between 30 to 60 seconds using a batch script. Note that for some reason the random method within the batch script sometimes would return values between 20-80. I do not know the occurrence of that. I am using PyInstaLive to download livestreams from users I am following rather than passing in a specific user.
I started PyInstaLive nearing 4 AM. At 7:48 AM an Instagram user I was following went live and the "[daemon process] started successfully". They ended their livestream at around 8:48 AM. I didn't see any other log between that time frame when typically IIRC there would be a message saying that a lock file has already been found for that user. The next output I received was at 8:49 AM and that was when the account was flagged with challenge_required.
I have reason to believe that accounts are being flagged with challenge_required when they run PyInstaLive with the download-following argument. Perhaps it's because they flag when we attempt to fetch other livestreams while already watching (downloading) another? Of course, this is solved by re-verifying the account, deleting the login file, and running PyInstaLive again. However, the issue is reoccurring.
Has anyone experienced something similar and can confirm this?

@Infinitay
I guess it's not the download-following argument. My last script checking only 1 account for running lives every 65 seconds got my account "challenge required" stuck aswell.
2 months ago i could run a script 24/7 for like 4 days until I got "challenge required" stuck.
Recently I was only able to run the script for a couple hours until I got stuck.
I confirmed my accounts with code by phone numerous times. And in process I probably needed to change password 20 times,
Now: Confirmed account by code and it said: We check your account in the next 24 hours.
Result: Account disabled by instagram
They disabled 5 of my accounts including my "main account" where I actually put work into. I needed to send them a photo of myself with me holding a sign with a 5 digit code on it and name etc. That was 1 week ago. I hope my main account will be unbanned at some point :D . I guess using my ip (and same phone number for challenge) with pyinsta will get me banned now no matter what. Just doing the challenge isn't working anymore.
I wonder whether using specific android device id and sessionid etc. makes a difference. But I can't find any info on that and sadly there is no support/maintenance around this code anymore.

@Infinitay
I guess it's not the download-following argument. My last script checking only 1 account for running lives every 65 seconds got my account "challenge required" stuck aswell.
2 months ago i could run a script 24/7 for like 4 days until I got "challenge required" stuck.
Recently I was only able to run the script for a couple hours until I got stuck.
I confirmed my accounts with code by phone numerous times. And in process I probably needed to change password 20 times,

Do you remember if by any chance you also downloaded comments? I think a few days ago I witnessed something where in the middle of the livestream I got soft banned and got "challenge required" for comments when reading my download log for a sepcific IG Live. However from what I can tell it was still downloading the stream just fine.

@Infinitay
I guess it's not the download-following argument. My last script checking only 1 account for running lives every 65 seconds got my account "challenge required" stuck aswell.
2 months ago i could run a script 24/7 for like 4 days until I got "challenge required" stuck.
Recently I was only able to run the script for a couple hours until I got stuck.
I confirmed my accounts with code by phone numerous times. And in process I probably needed to change password 20 times,

Do you remember if by any chance you also downloaded comments? I think a few days ago I witnessed something where in the middle of the livestream I got soft banned and got "challenge required" for comments when reading my download log for a sepcific IG Live. However from what I can tell it was still downloading the stream just fine.

I always download the comments. Sometimes there is a mistake and the comment downloader stops, but not because I got "soft banned". At least I don't know about that, because I don't really pay much attention to the comment downloader.
And when i get "challenge required stuck" while downloaing a stream, it will finish downloading the entire video stream.
Btw.: I got my main acc back, after 2 weeks and sending 2 e-mails to ig.
I won't run a "24/7" script for a while now and I'm going to test whether there is a difference to use same android device_id like my phone. Maybe it's worth changing session_id etc., but like i said, I'm clueless about programing etc.


flyway clean is not dropping Oracle scheduler jobs or programs

I recently added a scheduler job and program to my development schema. When I tried to refresh the schema, I did a flyway clean, and then a flyway migrate.
I got the following error:
ERROR: Found non-empty schema "TESTDATA" without metadata table! Use init() or set initOnMigrate to true to initialize the metadata table.
When I dropped the job and program by hand, I was then able to run migrate again.
Note: I see the option -initOnMigrate, but this causes me two problems:

I have a lot of batch files which would be sensitive to trying to add another runline option.
I use flyway both to update existing schemas and to refresh schemas from scratch. If I need to modify the job or program, I could only include initOnMigrate (and have it bomb on the update), or not include it, and have it bomb on refresh (my current problem).

StackOverflow link for this problem:
http://stackoverflow.com/questions/23767341/flyway-clean-is-not-dropping-scheduler-jobs-or-programs
Thank you
Update:
Statements used to create the stored procedure
CREATE OR REPLACE PACKAGE CLEANUP AS
PROCEDURE DONIGHTLY;
END CLEANUP;
CREATE OR REPLACE PACKAGE BODY CLEANUP AS
PROCEDURE DONIGHTLY IS
BEGIN
-- blah blah blah
NULL;
END;
END CLEANUP;
Statement used to create the program:
BEGIN
SYS.DBMS_SCHEDULER.CREATE_PROGRAM
(
program_name         => 'DO_NIGHTLY'
,program_type         => 'STORED_PROCEDURE'
,program_action       => 'CLEANUP.DONIGHTLY'
,number_of_arguments  => 0
,enabled              => FALSE
,comments             => NULL
);
SYS.DBMS_SCHEDULER.ENABLE
(name                  => 'DO_NIGHTLY');
END;
Statement used to create the job:
BEGIN
SYS.DBMS_SCHEDULER.CREATE_JOB
(
job_name        => 'TEST_CLEANUP'
,schedule_name   => 'SYS.DAILY_PURGE_SCHEDULE'
,program_name    => 'DO_NIGHTLY'
,comments        => NULL
);
SYS.DBMS_SCHEDULER.SET_ATTRIBUTE
( name      => 'TEST_CLEANUP'
,attribute => 'RESTARTABLE'
,value     => FALSE);
SYS.DBMS_SCHEDULER.SET_ATTRIBUTE
( name      => 'TEST_CLEANUP'
,attribute => 'LOGGING_LEVEL'
,value     => SYS.DBMS_SCHEDULER.LOGGING_OFF);
SYS.DBMS_SCHEDULER.SET_ATTRIBUTE_NULL
( name      => 'TEST_CLEANUP'
,attribute => 'MAX_FAILURES');
SYS.DBMS_SCHEDULER.SET_ATTRIBUTE_NULL
( name      => 'TEST_CLEANUP'
,attribute => 'MAX_RUNS');
BEGIN
SYS.DBMS_SCHEDULER.SET_ATTRIBUTE
( name      => 'TEST_CLEANUP'
,attribute => 'STOP_ON_WINDOW_CLOSE'
,value     => FALSE);
EXCEPTION
-- could fail if program is of type EXECUTABLE...
WHEN OTHERS THEN
NULL;
END;
SYS.DBMS_SCHEDULER.SET_ATTRIBUTE
( name      => 'TEST_CLEANUP'
,attribute => 'JOB_PRIORITY'
,value     => 3);
SYS.DBMS_SCHEDULER.SET_ATTRIBUTE_NULL
( name      => 'TEST_CLEANUP'
,attribute => 'SCHEDULE_LIMIT');
SYS.DBMS_SCHEDULER.SET_ATTRIBUTE
( name      => 'TEST_CLEANUP'
,attribute => 'AUTO_DROP'
,value     => FALSE);
SYS.DBMS_SCHEDULER.ENABLE
(name                  => 'TEST_CLEANUP');
END;
These are the statements I use to clean up the jobs by hand:
BEGIN
SYS.DBMS_SCHEDULER.DROP_JOB
(job_name  => 'TEST_CLEANUP');
END;
BEGIN
DBMS_SCHEDULER.DROP_PROGRAM
(program_name          => 'DO_NIGHTLY');
END;

Looks like this problem still exists in flyway 4.0.1 when pointing to Oracle 10G (<IP_ADDRESS>) as we are on one of the dev teams. (Same flyway scripts work fine on 11G)

@johnlmorgan If the problem persists, please open a new issue containing clear steps how to reproduce as well as a possible solution. Pull request with test case welcome!


fix Kicked for exceeding packet rate limit

Kicked for exceeding packet rate limit

Lower the amount of packets you are sending to the server.


click audio episode button close #41

This patch fixes issue #41
I've test this fix on my local machine

Before merging I suggest it being tested in GitHub actions

As discussed on the discord before merging I will wait for @TheJoin95 to review the changes


Link Broken: https://www.visualstudio.com/support/privacy-policy-vs

Found in docs/articles/team-services-security-whitepaper.md

This link is fixed.


import orientation file no longer an option in pmag_gui

we need to be able to import orientation.txt files with orientation_magic.py
also this should be upgraded to 3.0

This option is now available as step 2 in Pmag GUI.  I believe it is upgraded to work in 3.0, but that should be tested.  @ltauxe, let me know if this seems to work!

I am working on some more updates to how orientation magic works in Pmag GUI, but in the meantime I am closing this issue.


wip: added jetty http client instrumentation

What does this PR do?

closes #1567
Checklist

[ ] This is an enhancement of existing features, or a new feature in existing plugins

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Added an API method or config option? Document in which version this will be introduced
[ ] I have made corresponding changes to the documentation

[ ] This is a bugfix

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that would fail without this fix

[ ] This is a new plugin

[ ] I have updated CHANGELOG.asciidoc
[ ] My code follows the style guidelines of this project
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have updated supported-technologies.asciidoc
[ ] Added an API method or config option? Document in which version this will be introduced
[ ] Added an instrumentation plugin? Describe how you made sure that old, non-supported versions are not instrumented by accident.

[ ] This is something else

[ ] I have updated CHANGELOG.asciidoc

:grey_exclamation: Build Aborted

The PR is not allowed to run in the CI yet

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Reason: The PR is not allowed to run in the CI yet

Start Time: 2021-08-21T17:51:24.672+0000

Duration: 3 min 8 sec

Commit: 349890e56e6e8283aebb5a09e8b9b887859616c1

Trends :test_tube:

Steps errors
Expand to view the steps failures

Load a resource file from a shared library

Took 0 min 0 sec . View more details on here
Description: approval-list/elastic/apm-agent-java.yml

Error signal

Took 0 min 0 sec . View more details on here
Description: githubPrCheckApproved: The PR is not allowed to run in the CI yet. (Only users with write permissions can do so.)

Log output
Expand to view the last 100 lines of log output

[2021-08-21T17:54:03.957Z]  > git checkout -f 349890e56e6e8283aebb5a09e8b9b887859616c1 # timeout=100
[2021-08-21T17:54:04.211Z]  > git remote # timeout=10
[2021-08-21T17:54:04.215Z]  > git config --get remote.origin.url # timeout=10
[2021-08-21T17:54:04.225Z] using GIT_ASKPASS to set credentials GitHub user @apmmachine User + Personal Access Token
[2021-08-21T17:54:04.227Z]  > git merge 849b1f55429ef6189307483f1d1ff47e19b535b0 # timeout=10
[2021-08-21T17:54:04.236Z]  > git rev-parse HEAD^{commit} # timeout=10
[2021-08-21T17:54:04.245Z]  > git config core.sparsecheckout # timeout=10
[2021-08-21T17:54:04.250Z]  > git checkout -f 349890e56e6e8283aebb5a09e8b9b887859616c1 # timeout=100
[2021-08-21T17:54:08.819Z] Commit message: "delete import"
[2021-08-21T17:54:08.819Z] First time build. Skipping changelog.
[2021-08-21T17:54:09.618Z] Masking supported pattern matches of $JOB_GCS_BUCKET or $NOTIFY_TO
[2021-08-21T17:54:09.688Z] Timeout set to expire in 1 hr 30 min
[2021-08-21T17:54:09.717Z] The timestamps step is unnecessary when timestamps are enabled for all Pipeline builds.
[2021-08-21T17:54:10.065Z] [INFO] Number of builds to be searched 10
[2021-08-21T17:54:10.433Z] [INFO] 'shallow' is forced to be disabled when running on PullRequests
[2021-08-21T17:54:10.456Z] Running in /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-2072/src/github.com/elastic/apm-agent-java
[2021-08-21T17:54:10.496Z] [INFO] gitCheckout: Checkout SCM PR-2072 with some customisation.
[2021-08-21T17:54:10.536Z] [INFO] Override default checkout
[2021-08-21T17:54:10.600Z] Sleeping for 10 sec
[2021-08-21T17:54:20.621Z] The recommended git tool is: git
[2021-08-21T17:54:20.914Z] using credential 2a9602aa-ab9f-4e52-baf3-b71ca88469c7-UserAndToken
[2021-08-21T17:54:20.929Z] Cloning the remote Git repository
[2021-08-21T17:54:20.987Z] Cloning repository https://github.com/elastic/apm-agent-java.git
[2021-08-21T17:54:21.053Z]  > git init /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-2072/src/github.com/elastic/apm-agent-java # timeout=10
[2021-08-21T17:54:21.059Z] Using reference repository: /var/lib/jenkins/.git-references/apm-agent-java.git
[2021-08-21T17:54:21.059Z] Fetching upstream changes from https://github.com/elastic/apm-agent-java.git
[2021-08-21T17:54:21.059Z]  > git --version # timeout=10
[2021-08-21T17:54:21.064Z]  > git --version # 'git version 2.17.1'
[2021-08-21T17:54:21.064Z] using GIT_ASKPASS to set credentials GitHub user @apmmachine User + Personal Access Token
[2021-08-21T17:54:21.065Z]  > git fetch --tags --progress -- https://github.com/elastic/apm-agent-java.git +refs/heads/*:refs/remotes/origin/* # timeout=10
[2021-08-21T17:54:21.618Z]  > git config remote.origin.url https://github.com/elastic/apm-agent-java.git # timeout=10
[2021-08-21T17:54:21.621Z]  > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # timeout=10
[2021-08-21T17:54:22.116Z] Checking out Revision 349890e56e6e8283aebb5a09e8b9b887859616c1 (origin/PR-2072)
[2021-08-21T17:54:22.309Z] Commit message: "delete import"
[2021-08-21T17:54:21.636Z]  > git config remote.origin.url https://github.com/elastic/apm-agent-java.git # timeout=10
[2021-08-21T17:54:21.643Z] Fetching upstream changes from https://github.com/elastic/apm-agent-java.git
[2021-08-21T17:54:21.643Z] using GIT_ASKPASS to set credentials GitHub user @apmmachine User + Personal Access Token
[2021-08-21T17:54:21.644Z]  > git fetch --tags --progress -- https://github.com/elastic/apm-agent-java.git +refs/pull/2072/head:refs/remotes/origin/PR-2072 +refs/heads/master:refs/remotes/origin/master # timeout=10
[2021-08-21T17:54:22.111Z]  > git rev-parse origin/PR-2072^{commit} # timeout=10
[2021-08-21T17:54:22.118Z]  > git config core.sparsecheckout # timeout=10
[2021-08-21T17:54:22.122Z]  > git checkout -f 349890e56e6e8283aebb5a09e8b9b887859616c1 # timeout=10
[2021-08-21T17:54:23.229Z] Masking supported pattern matches of $GIT_USERNAME or $GIT_PASSWORD
[2021-08-21T17:54:24.001Z] + git fetch<EMAIL_ADDRESS>+refs/pull/*/head:refs/remotes/origin/pr/*
[2021-08-21T17:54:24.691Z] Running in /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-2072/src/github.com/elastic/apm-agent-java/.git
[2021-08-21T17:54:24.804Z] Archiving artifacts
[2021-08-21T17:54:25.996Z] + git rev-parse HEAD
[2021-08-21T17:54:26.379Z] + git rev-parse HEAD
[2021-08-21T17:54:26.690Z] + git rev-parse origin/pr/2072
[2021-08-21T17:54:26.745Z] [INFO] githubEnv: Found Git Build Cause: pr
[2021-08-21T17:54:27.092Z] Masking supported pattern matches of $GITHUB_TOKEN
[2021-08-21T17:54:28.068Z] [WARN] githubApiCall: The REST API call https://api.github.com/repos/elastic/apm-agent-java/pulls/2072/reviews return 0 elements
[2021-08-21T17:54:28.136Z] [INFO] githubPrCheckApproved: Title: wip: added jetty http client instrumentation - User: kananindzya - Author Association: CONTRIBUTOR
[2021-08-21T17:54:28.463Z] ERROR: githubPrCheckApproved: The PR is not allowed to run in the CI yet
[2021-08-21T17:54:28.463Z] ERROR: githubPrCheckApproved: The PR is not allowed to run in the CI yet. (Only users with write permissions can do so.)
[2021-08-21T17:54:28.528Z] [INFO] Let's stop build #6. The PR is not allowed to run in the CI yet
[2021-08-21T17:54:28.551Z] Sleeping for 5 sec
[2021-08-21T17:54:29.549Z] Stage "Build" skipped due to earlier failure(s)
[2021-08-21T17:54:29.742Z] Stage "Tests" skipped due to earlier failure(s)
[2021-08-21T17:54:29.920Z] Stage "Unit Tests" skipped due to earlier failure(s)
[2021-08-21T17:54:29.930Z] Stage "Smoke Tests 01" skipped due to earlier failure(s)
[2021-08-21T17:54:29.941Z] Stage "Smoke Tests 02" skipped due to earlier failure(s)
[2021-08-21T17:54:29.951Z] Stage "Benchmarks" skipped due to earlier failure(s)
[2021-08-21T17:54:29.961Z] Stage "Javadoc" skipped due to earlier failure(s)
[2021-08-21T17:54:30.125Z] Failed in branch Unit Tests
[2021-08-21T17:54:30.132Z] Failed in branch Smoke Tests 01
[2021-08-21T17:54:30.140Z] Failed in branch Smoke Tests 02
[2021-08-21T17:54:30.147Z] Failed in branch Benchmarks
[2021-08-21T17:54:30.154Z] Failed in branch Javadoc
[2021-08-21T17:54:30.396Z] Stage "Integration Tests" skipped due to earlier failure(s)
[2021-08-21T17:54:30.481Z] Stage "Unit Tests" skipped due to earlier failure(s)
[2021-08-21T17:54:30.688Z] Stage "Matrix - JAVA_VERSION = 'java12'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.693Z] Stage "Matrix - JAVA_VERSION = 'openjdk12'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.698Z] Stage "Matrix - JAVA_VERSION = 'openjdk13'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.703Z] Stage "Matrix - JAVA_VERSION = 'openjdk14'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.708Z] Stage "Matrix - JAVA_VERSION = 'openjdk15'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.713Z] Stage "Matrix - JAVA_VERSION = 'openjdk16'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.824Z] Stage "Matrix - JAVA_VERSION = 'java12'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.833Z] Stage "Matrix - JAVA_VERSION = 'openjdk12'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.838Z] Stage "Matrix - JAVA_VERSION = 'openjdk13'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.843Z] Stage "Matrix - JAVA_VERSION = 'openjdk14'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.848Z] Stage "Matrix - JAVA_VERSION = 'openjdk15'" skipped due to earlier failure(s)
[2021-08-21T17:54:30.853Z] Stage "Matrix - JAVA_VERSION = 'openjdk16'" skipped due to earlier failure(s)
[2021-08-21T17:54:31.231Z] Failed in branch Matrix - JAVA_VERSION = 'java12'
[2021-08-21T17:54:31.238Z] Failed in branch Matrix - JAVA_VERSION = 'openjdk12'
[2021-08-21T17:54:31.246Z] Failed in branch Matrix - JAVA_VERSION = 'openjdk13'
[2021-08-21T17:54:31.252Z] Failed in branch Matrix - JAVA_VERSION = 'openjdk14'
[2021-08-21T17:54:31.259Z] Failed in branch Matrix - JAVA_VERSION = 'openjdk15'
[2021-08-21T17:54:31.267Z] Failed in branch Matrix - JAVA_VERSION = 'openjdk16'
[2021-08-21T17:54:31.533Z] Stage "Stable" skipped due to earlier failure(s)
[2021-08-21T17:54:31.623Z] Stage "AfterRelease" skipped due to earlier failure(s)
[2021-08-21T17:54:31.661Z] Stage "AfterRelease" skipped due to earlier failure(s)
[2021-08-21T17:54:32.079Z] Running on Jenkins in /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-2072
[2021-08-21T17:54:32.345Z] [INFO] getVaultSecret: Getting secrets
[2021-08-21T17:54:32.406Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2021-08-21T17:54:33.315Z] + chmod 755 generate-build-data.sh
[2021-08-21T17:54:33.315Z] + ./generate-build-data.sh https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-2072/ https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-2072/runs/6 ABORTED 188368
[2021-08-21T17:54:33.315Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-2072/runs/6/steps/?limit=10000 -o steps-info.json
[2021-08-21T17:54:33.315Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-2072/runs/6/tests/?status=FAILED -o tests-errors.json
[2021-08-21T17:54:33.565Z] Retry 1/3 exited 22, retrying in 1 seconds...
[2021-08-21T17:54:34.476Z] Retry 2/3 exited 22, retrying in 2 seconds...


Fix throwing nil exceptions.

The following is perfectly legal on Mac OS X:
@try
{
  @throw(nil);
}
@catch (id obj)
{
  printf("Caught: %p", obj);
}

It doesn't work presently because we were trying to dereference the object pointer to determine the class.

What's the correct behaviour for, throwing nil and catching (for example)NSObject *?

It is only caught by @catch(id) and @catch(...), not by any other interface type. I've added a few more tests for those cases, but I'm running into some weird behaviour here: The standalone test case passes, but similar code, linked against gnustep-base doesn't work (i.e. @catch(NSObject*) catches nil as well…

Should nil also be caught by void* catches in Objective-C++ (this case used to crash the Apple runtime, not sure if they've fixed it now)?

I've checked and it's not caught by a void* handler on 10.10.5 (it doesn't crash either, fwiw).

The stand-alone code may be interacting oddly with the CXXException.mm stuff in -base, which attempts to box C++ exceptions as Objective-C objects.  We should probably remove that now - it was largely a work-around for the fact that we didn't have working Objective-C++.


Filter can be array or lambda.

Hi @wconrad! I'm bringing this back from the dead as I do some refactoring. It got me thinking about the filter_proc method, most of which you wrote.
def filter_proc(filter)
      if filter.respond_to?(:to_a)
        filter_procs = Array(filter).map(&method(:filter_proc))
        ->(word) {
          filter_procs.any? { |p| p.call(word) }
        }
      elsif filter.respond_to?(:to_str)
        f = filter.split.collect { |word| word.downcase } # <---- THIS LINE HERE
        ->(w) { f.include?(w.downcase) }
      elsif Regexp.try_convert(filter)
        filter = Regexp.try_convert(filter)
        Proc.new { |w| w =~ filter }
      elsif filter.respond_to?(:to_proc)
        filter.to_proc
      else
        raise ArgumentError, "Incorrect filter type"
      end
    end

I understand whats going on here, mostly because I went back and reread your code review (it makes so much sense now that I actually have some good experience under my belt).
There is one line I want to enquire about.
filter_procs = Array(filter).map(&method(:filter_proc))

If I understand this correctly, we're using recursion? If an array is passed, then this line will run and iterate over the array. It will call the method filter_proc for each item, passing in the item as an argument. The time the other conditions should catch it, be it a string, a regexp, or a proc. The return value will always be a callable object, which bubbles back up the stack. Does this make sense? This is pretty genius because you can pass in an array of any combination of filters in an array.

Yes, you've got it: it is recursive.  The only twist is that the recursion does not happen at the time filter_proc is called.  It happens, later, at the time that the lambda returned by filter_proc is called.
The "pass in an array of any combination of filters" is exactly what the code means to achieve, so you've interpreted it exactly right.

No, wait, i was wrong.  The recursion does happen at the time the method is called.

@wconrad thanks! That makes sense. I should write some tests and document this feature. :smile:

I forgot to ask one last thing.
def filter_proc(filter)
  if filter.respond_to?(:to_a)
    # ...
  elsif regexp_filter = Regexp.try_convert(filter)
    Proc.new { |word| word =~ regexp_filter }
  else
  # ...

In the case of a regexp, is a reason we're using a proc and not a lamba? In my tests ->(word) { word =~ regexp_filter } words just as well.


fixing dead link in references

fixing dead link due rename for Azure Hybrid Benefit for Azure Stack HCI

Due to important changes from January 2024, I would be grateful for a quick review, test and merge.

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Update README.md

Corrected npm install command - package name
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x ] Other... Please describe: Readme update - install instruction

What is the current behavior?
Readme file had instruction to install class-validator package, which is not nestjs fork. Now it is correct package npm i @nestjs/class-validator
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[ x] No

Other information

LGTM


Import tweedledum failing on MacOS

Describe the bug

Whenever I'm trying to import something from the tweedledum module, it throws an ImportError.
e.g.
from tweedledum.classical import simulate
or
import tweedledum
fail with the following stacktrace-
>>> from tweedledum.classical import simulate
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/arihant/opt/anaconda3/envs/qiskit/lib/python3.8/site-packages/tweedledum/__init__.py", line 5, in <module>
    from ._tweedledum import *
ImportError: dlopen(/Users/arihant/opt/anaconda3/envs/qiskit/lib/python3.8/site-packages/tweedledum/_tweedledum.cpython-38-darwin.so, 2): Symbol not found: _aligned_alloc
  Referenced from: /Users/arihant/opt/anaconda3/envs/qiskit/lib/python3.8/site-packages/tweedledum/_tweedledum.cpython-38-darwin.so
  Expected in: /usr/lib/libSystem.B.dylib
 in /Users/arihant/opt/anaconda3/envs/qiskit/lib/python3.8/site-packages/tweedledum/_tweedledum.cpython-38-darwin.so

Expected behavior
The libraries get imported.
Information

tweedledum version: 1.1.0
Python version:  3.8.10
Operating system: MacOS Mojave Version 10.14.6 (18G8022)

Additional context
I'm running it from a conda environment - conda 4.10.1.(which I have tried re-installing to try fixing this)
EDIT: The same works on my windows machine, so might be a MacOS specific issue.

Same here, MacOS 10.14.5, Apple LLVM version 10.0.1 (clang-1<IP_ADDRESS>). Tested with pip install tweedledum for Python 3.8.11 in a clean venv setup and Python 3.7.9.
I also failed to install tweedledum==1.1.0 via pip install tweedledum==1.1.0 --no-binary tweedledum.
Installation of version 1.0.0 seems to work fine.

pip install tweedledum==1.0.0 fixed the problem for me

Seems like the problem is that I'm building on MacOS 10.15 (Catalina), and using the MacOS 10.9 SDK while building. Furthermore, I'm using GCC 10 while linking against libstdc++, which Apples seems to discourage in favor of libc++.
There might be some problem with my CI build script, I will try to fix it. Unfortunately, I don't have means to test a possible solution on my side.

@boschmitt one way we can test this is to have the ci job on the fix PR and upload the wheel to artifacts and then ask @eileen-kuehn or @Arihant-Joshi (or any other users experiencing the issue) to test that wheel file locally. That's what I normally do in these situations where there are platform specific issues and I don't have access to the platform.

@boschmitt, I am happy to do some tests for you as soon as you let me know :)

Hi all, I've been facing the same issue on High Sierra 10.13.6 and conda 4.10.3. Unfortunately downgrading to tweedledum 1.0.0 didn't do the trick for me:
ImportError: dlopen(/Users/Caco/anaconda3/envs/NEW_QISKIT/lib/python3.8/site-packages/tweedledum/_tweedledum.cpython-38-darwin.so, 2): Symbol not found: _aligned_alloc   Referenced from: /Users/Caco/anaconda3/envs/NEW_QISKIT/lib/python3.8/site-packages/tweedledum/_tweedledum.cpython-38-darwin.so   Expected in: /usr/lib/libSystem.B.dylib  in /Users/Caco/anaconda3/envs/NEW_QISKIT/lib/python3.8/site-packages/tweedledum/_tweedledum.cpython-38-darwin.so

@boschmitt, I am happy to do some tests for you as soon as you let me know :)

Hi @eileen-kuehn, Thanks. Could you try installing tweedledum-dev and see if it works. (You will need to uninstall tweedledum or install it in a new env)

@boschmitt, the tweedledum-dev package is working great for me 👍
I cannot see any issues anymore.

After much thought, I decided to drop support for macOS High Sierra (10.13) and macOS Mojave (10.14).  From now on, I will support both systems on a best-effort basis. Meaning that some I won't build wheels for all releases. The reason is simple: I was not able to successfully build these wheels using a GitHub workflow.
The workaround is to install tweedledum directly from source:
CC=gcc-10 CXX=g++-10 CXXFLAGS="-static-libgcc -static-libstdc++" pip install tweedledum --no-binary :all:

Note that this requires a gcc that has been build for the specific macOS you are using. (This should work with any version with C++17 support, but I only tested with gcc10 and gcc11.) Indeed, the CI box has various gcc versions installed, but they were all built for macOS 10.15. Currently, even if I set the deployment target to be 10.9 (or 10.13) and statically link against standard libraries, the compiler/linker still assumes that the aligned_alloc function will be provided by some other library.

The workaround is to install tweedledum directly from source:
CC=gcc-10 CXX=g++-10 CXXFLAGS="-static-libgcc -static-libstdc++" pip install tweedledum --no-binary :all:

In my case, the workaround is perfectly fine 👍 and I am happy to install tweedledum like this. Thanks a lot for your efforts!


Could not get it working in Linux: dial unix /tmp/mpvsocket: connect: connection refused

Hi there
I could not get it working in my Linux machine. I did everything according to instructions, but it doesn't seem to work. The log is like:
[discord] mpv-discord v1.6.0 by tnychn
[discord] (mpv-ipc): /tmp/mpvsocket
[discord] launched subprocess
[discord] dial unix /tmp/mpvsocket: connect: connection refused
.....
[discord] aborted subprocess

The problem probably occurs when connection is refused. I couldn't find when you log this, but I thikn this is from mpv-discord module(the backend).
My configurations and stuff is all fine. Please tell me if you need any other information.

I have read/write permissions in /tmp. Altough, I changed it to ~/.local/share/mpv/socket, It still says connection refused.
Changing use_static_socket_path to no, resulted(i just changed socket_path too)
dial unix /home/murad/.cache/mpv/socket/mpv-discord-2998: connect: no such file or directory


Cross-platform support

This project in its current state works exclusively on Windows. To make it more accessible, we need to support other platforms, starting with Linux and potentially extending to MacOS. This will likely require minor code modifications to accommodate OS-specific APIs and headers. Also, the Makefile provided is configured for Windows (MSYS2/MinGW) only - it needs to be extended to other platforms.

Make sure you can test build on the platforms you're adding support for.

Use conditional compilation (i.e with  #ifdef  etc) for OS specific stuff.
You need to link to different libraries depending on the platform, eg. -lGLU -lGL -lm is required on Linux.
The libraries provided in /lib were built for x64 Windows in an MSYS2 environment. You will need to obtain equivalent binaries for other platforms.
Support for Linux is essential, bonus points if you can get it to work on MacOS.
Helpful snippet for OS detection in a Makefile:ifeq ($(OS),Windows_NT) # OS is a preexisting environment variable on Windows
    OS = windows
else
    UNAME := $(shell uname -s)
    ifeq ($(UNAME),Darwin)
  	  OS = macos
    else ifeq ($(UNAME),Linux)
  	  OS = linux
    else
  	  $(error OS not supported by this Makefile)
    endif
endif

can i get assigned

can you assign me this issue @roh1th-s

!assign @Viprakshi

can you assign me this issue @roh1th-s

!assign @Viprakshi

@Viprakshi if you want to work on this go ahead, the bot seems to be having issues rn

can you please deassign me here

!deassign

@Viprakshi I'm not sure if the bot is doing anything since it's not replying. If you're trying to work on another issue and they're not able to assign you its probably because of the bot.

can I get this?

@srijan-nodes  withdraw from [this issue[(https://github.com/acmpesuecc/BrainSpark/issues/3) first

done...that has been completed

!assign @srijan-nodes

made a pr


Add crash-reporter to get feedback from Users

-> http://electron.atom.io/docs/v0.30.0/api/crash-reporter/

:+1:

In a certain way added now. 😄


Fail the tool if any analysis fails

Motivation is https://github.com/Smaug123/fsharp-prattparser/actions/runs/7619221208/job/20751901249 , which succeeded despite the output being as follows:
info: Running in verbose mode
info: Treat as Hints: []
info: Treat as Info: []
info: Treat as Warning: []
info: Treat as Error: [GRA-IMMUTABLECOLLECTIONEQUALITY-001, GRA-INTERPOLATED-001, GRA-JSONOPTS-001, GRA-LOGARGFUNCFULLAPP-001, GRA-STRING-001, GRA-STRING-002, GRA-STRING-003, GRA-TYPE-ANNOTATE-001, GRA-UNIONCASE-001, GRA-VIRTUALCALL-001]
info: Exclude Files: []
info: Include Files: []
info: Loading analyzers from /home/runner/work/fsharp-prattparser/fsharp-prattparser/.analyzerpackages/g-research.fsharp.analyzers/0.7.0/
info: Registered 12 analyzers from 1 dlls
error: Checking of file /home/runner/work/fsharp-prattparser/fsharp-prattparser/PrattParser/obj/Debug/net8.0/.NETCoreApp,Version=v8.0.AssemblyAttributes.fs aborted
error: Checking of file /home/runner/work/fsharp-prattparser/fsharp-prattparser/PrattParser/obj/Debug/net8.0/PrattParser.AssemblyInfo.fs aborted
error: Checking of file /home/runner/work/fsharp-prattparser/fsharp-prattparser/PrattParser/obj/Debug/net8.0/PrattParser.Version.fs aborted
error: Checking of file /home/runner/work/fsharp-prattparser/fsharp-prattparser/PrattParser/Parser.fs aborted
info: No messages found from the analyzer(s)

In order not to have a heap of primitive types like _ list option, I pulled out the failure case into a single-case DU. That way, the semantics are much clearer if you're only looking at the type signatures.

Personally I'm less concerned about documenting exit codes because we already log on all these error cases.

Personally I'm less concerned about documenting exit codes because we already log on all these error cases.

Thanks for the work. We can document the exit codes in another PR, then.

@Smaug123 would you like a new release with all your latest changes?

That would be great, thanks!

Thanks - working perfectly (https://github.com/Smaug123/fsharp-prattparser/actions/runs/7708627409/job/21008123060?pr=2 now loudly fails rather than silently failing).


Add live migration support to LXD

This commit adds live migration support to LXD. As you can see from the diff,
there are several TODOs, notably:

we need to transport the container config too (more robust config support
will be arriving shortly, so I'll use that when it's here)
we need to use protobufs for the control socket protocol so others can
interact with us
we need to use p.haul and other similar optimizations
we need to figure out what to do with criu log files so that people have a
nice debugging experience

To set up a container for migration, its configuration needs to have the
following entries:
lxc.console = none
lxc.tty = 0
lxc.cgroup.devices.deny = c 5:1 rwm

Which can be acheived by using raw.lxc for now. You also (of course) need to
have a relatively recent version of criu installed. For this patch, I tested
with 1.4.
Signed-off-by: Tycho Andersen<EMAIL_ADDRESS>

Ah, hm. Looks like I need to add a blurb to the spec about how exactly the multiple channels work. Other than that this should be good.

@tych0, can you please explain the fisrt entry of TODOs list in this page. Dose "container config" means the config file of the container? What's " config support"?

There is no config file in LXD, this was about making sure we transferred the container Config map over to the target, and it's been done quite a while ago.


Draft: Focus the commandline when a command is entered

Closes: #95
This is currently blocked: see #95

No longer necessary due to #111


luci-mod-dashboard: custom.css: Add color variables

Add color variable references from luci-theme-boostrap to facilitate light and dark color scheme switching by default. Fallback to original colors when variables are not declared by theme.

[x] This PR is not from my main or master branch :poop:, but a separate branch :white_check_mark:
[x] Each commit has a valid :black_nib: Signed-off-by<EMAIL_ADDRESS>row (via git commit --signoff)
[x] Each commit and PR title has a valid :memo: <package name>: title first line subject for packages
[x] Incremented :up: any PKG_VERSION in the Makefile
[x] Tested on: (architecture, openwrt version, browser) :white_check_mark:
[ ] ( Preferred ) Mention: @ the original code author for feedback
[x] ( Preferred ) Screenshot or mp4 of changes:
[ ] ( Optional ) Closes: e.g. openwrt/luci#issue-number
[ ] ( Optional ) Depends on: e.g. openwrt/packages#pr-number in sister repo
[x] Description: (describe the changes proposed in this PR)

So.. @stokito @dannil does my memory serve me correctly when I think we actually got rid of 'variables' a while back?

So.. @stokito @dannil does my memory serve me correctly when I think we actually got rid of 'variables' a while back?

No, they're still there. I double-checked the variables as well, they're present for the Bootstrap theme (see @stokito's link above).

It's unclear to me why this request was closed. Would anyone be able to elaborate? @systemcrash @stokito @dannil


The blobs in the virtual folder are not counted at the folder level with 'Blob versioning' enabled

Storage Explorer Version: 1.16.0
Build Number: 20201203.3
Branch: main
Platform/OS: Windows 10/ CentOS 7.6.1810 (Core) / MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to Reproduce

Select one GPV2 or Blob storage account with 'Blob versioning' enabled.
Expand the selected storage account -> Blob Containers.
Create a blob container -> Create a virtual folder and upload one blob to it.
Right click the folder -> Click 'Selection Statistics 2'.
Observe the activity log -> Check whether the blob is counted or not.

Expected Experience
The blob is counted.
Actual Experience
The blob is not counted.
The log for the activity 'Selection Statistics 2':

The log for the activity 'Selection Statistics':

Additional Context
This issue doesn't reproduce when 'Blob versioning' is disabled.

Fix merged in build #20201204.3 and later.


Add Vue support

I'm not a user of Vue, but if anyone wants to PR vue and vue-ui packages based on the react and react-ui packages, I'd be happy to accept them.

there should be a vue and svelte version of this....

PRs are welcome @goldcoders

Would like a vue adapter as well please.

I don't use Vue or Svelte, but I'm happy to accept contributions to support them. I opened this issue to encourage contribution, so please open a PR if this is something you'd like to see. 🙇♂

We need that wallet-adapter-vue please

Hi there 👋
I've managed to use the wallet adapter using a Vue3 application.
Here's the code: https://github.com/lorisleiva/solana-vue3-counter/tree/9321e7c584906b3844d1c3eb1b69321f3ba8343d/app/src
It's a simple counter application but I've gathered most of the logic in useWallet and useWorkspace files.
I hope this helps. 🍀

That looks nice @lorisleiva, are you willing to put a PR for it?

I could certainly try. 🙂
I've got two freelance jobs on the go at the moment so I might be a bit slow though.
Is it okay for me to use TailwindCSS for the UI?

I'd recommend doing the adapter first and then discuss the UI part, just so the PRs are a bit easier to digest.
I don't think there's a hurry, if somebody really needs it, they can do it themselves. Thanks a lot for stepping up and offer the solution you're using.
Regarding TailwindCSS, I'm really a fan of it but that means more dependencies. Since I'm not the code owner we may want to wait for @jordansexton feedback.

Hey, thats awesome @lorisleiva! Great Job

That's a good shout, I'll get started with the core adapter first to provide a more digestible PR.
I see what you mean about the extra dependency but I don't think Vue has any official UI libraries so there would be a need for extra dependencies anyway. With Tailwind I can make sure to purge any unused classes to make the final minified CSS super tiny. Anyway, that's my case for Tailwind but I'm more than happy to integrate it with something else. 😅

I'm not familiar with Vue tbh, I don't know the UI alternatives. But I love TW and if it's already a dependency I'd use it for the angular support. The thing is that it's not up to me to decide.

That's a good shout, I'll get started with the core adapter first to provide a more digestible PR.

Awesome! Yeah, I think having a core package for the state management expected by Vue (Vuex?) users is a good start and will help frame how we think about structuring and maintaining the UI components.

Regarding TailwindCSS, I'm really a fan of it but that means more dependencies.

If there's a core/vue package and a ui/vue-ui package, it could include a CSS framework, but I might like to see something closer to the ui/react-ui package, which has few dependencies, the styles are easy to override, and it's extensible if you choose to fork it.
Tailwind is a pretty opinionated framework. I think whether we want to use Tailwind depends on how common/idiomatic it is for Vue apps to use Tailwind versus some other CSS/CSS-in-JS solution, and whether we need a framework at all for the small number of components we expect.

I ended up here for the exact reason of wanting a Vue version.  Glad to see this moving!

it could include a CSS framework, but I might like to see something closer to the ui/react-ui package, which has few dependencies, the styles are easy to override, and it's extensible if you choose to fork it.

I have my vote for this approach. Leaves the freedom for users to style on their own.  Ideally a default style can be scoped to just the component and class props (whether TW or otherwise) can be still be passed to override the default.  While I enjoy using TW, I like the freedom to change if needed.

So I've made some progress by translating the app files to TypeScript but I'm having issues when compiling due to Vue 3 and the --isolatedModules flag even though the const enums mentioned below are not actually being used by what I am tree shaking.
../../../node_modules/@vue/runtime-core/dist/runtime-core.d.ts:1187:6 - error TS2748: Cannot access ambient const enums when the '--isolatedModules' flag is provided.

1187     [BooleanFlags.shouldCast]?: boolean;
          ~~~~~~~~~~~~

../../../node_modules/@vue/runtime-core/dist/runtime-core.d.ts:1188:6 - error TS2748: Cannot access ambient const enums when the '--isolatedModules' flag is provided.

1188     [BooleanFlags.shouldCastTrue]?: boolean;
          ~~~~~~~~~~~~

../../../node_modules/@vue/runtime-core/dist/runtime-core.d.ts:1722:10 - error TS2748: Cannot access ambient const enums when the '--isolatedModules' flag is provided.

1722 export { TrackOpTypes }
              ~~~~~~~~~~~~

../../../node_modules/@vue/runtime-core/dist/runtime-core.d.ts:1751:10 - error TS2748: Cannot access ambient const enums when the '--isolatedModules' flag is provided.

1751 export { TriggerOpTypes }
              ~~~~~~~~~~~~~~

Found 4 errors.

I can confirm that the error comes from this import line:
import { ref, computed, Ref, watch, watchEffect } from 'vue';

More precisely, the error comes from @vue/runtime-core.
import { watch, watchEffect } from '@vue/runtime-core';

Since I can get the other values from @vue/reactivity without any issue.
import { ref, computed, Ref } from '@vue/reactivity';

I've looked everywhere online and searched the Vue 3 issues and there's nothing that mentions this so my guess is I must be doing something wrong. I'm fairly new to TypeScript so I wouldn't be surprised.
Any help would be much appreciated. You can find the repo I'm working on here. 🙏

Did you try overriding isolatedModules flag here? I'm not sure how this would affect your code but here's the documentation about isolatedModule flag

Thanks Daniel, setting this flag to false on the vue core package did the trick! 🙇♂ Now I can continue on my journey. 😊

On top of that first core PR, I've managed to recreate the UI for Vue using the exact same stylesheet as the React UI for consistency (so not using TailwindCSS in the end).
So far this is not typed or extracted to a library but simply lives inside a dummy app I'm playing with. When/if that first PR is merged I can extract that on a ui/vue-ui PR that will make use of core/vue.
In the meantime, if anyone wants to check the code of that dummy app, you can find it here. Here's a quick demo below using the wallet-adapter UI for Vue.
https://user-images.githubusercontent.com/3642397/135502773-aea2dd5f-0899-4d20-b724-5a766891f398.mp4

Do you have a non typescript version? Cheers


feat: compatibility with commonjs

Fixes #174

Any ETA on when this might be merged? Thanks.

@claycephas merged. Sorry for the delay.


Plugin: Scheduled Notifications

Flutter should have an easy to use scheduled notification API. (UILocalNotification in iOS, AlarmManager + Notification in Android)

Note: These are similar in spirit to push notifications, but are different in that they are scheduled locally on a timer, and do not require a network connection.
Also looks like iOS is moving to https://developer.apple.com/reference/usernotifications/unnotificationrequest.

I've been digging through issues because I'm interested in this feature, and it looks to me like this ticket is fully encapsulated by https://github.com/flutter/flutter/issues/2731 and https://github.com/flutter/flutter/issues/6192 / the AlarmManager plugin.

@assumptionsoup, yes between those two issues, and the plugin https://pub.dartlang.org/packages/scheduled_notifications, I believe this ticket cap be closed. Thanks for the note!

This issue should not be closed. Where is the iOS version? Thanks.

Hi @sgon00 we are tracking that in https://github.com/flutter/flutter/issues/6192

@mit-mit ok, got it. sorry about that.


Number of trainable parameters

I logged out the number of trainable parameters here and I received 45,901,914 params.
Using this function
np.sum([np.prod(v.get_shape().as_list()) for v in tf.trainable_variables()])
But when I look at the number of trainable parameters in the wideresnet from their original paper: https://arxiv.org/abs/1605.07146 I see 36.5M. Why is yours so much more?

Also, is there a pytorch version of this network? I noticed you referred to the Robustness platform, but I don't see an actual implementation of the exact same network mentioned here in that repository here: https://github.com/MadryLab/robustness/blob/master/robustness/cifar_models/resnet.py I only see a ResNet 18 wide but that's it.
Thanks,

You are right, the architecture here is not exactly the same as a typical "wide ResNet". We started from a standard ResNet architecture from the Tensorflow tutorial and modified it to be 10x wider for most blocks.
In https://github.com/MadryLab/robustness you can find a pre-trained robust ResNet50. It is not the exact same architecture as the model in this repo, but it turns out to actually be more robust.


Google Firebase API examples

Added Firebase API examples for developers

Seems like new examples should be in https://github.com/google/google-api-objectivec-client-for-rest, instead as the JSON-RPC version here is being phased out (see the note in the README.md and on all the wiki pages).

Closing since things should be moving to GTLR.


custom command on options

This minor modification allows specify a custom command via the options object:
netstat({
    command:{
          cmd: 'netstat',
          args: ['-v', '-p', 'tcp']
    },
    filter: {
        pid: 4123,
        protocol: 'tcp'
    },
    limit: 5
}, function (data) {
    // a single line of data read from netstat
});

Thanks for the submission and sorry for the delay. Before I merge, what would be the use case for sending a custom command in with the call? I would think if you changed the arguments or command, you would need to also create a custom parser and at that point you'd want to use the netstat.commands and netstat.parsers fields and specify a custom option.platform when making the call:
netstat.commands.custom1 = { cmd: 'netstat', args: ['-v', '-p', 'tcp'] };

netstat.parsers.custom1 = function (line, callback) {
    // perform custom parsing
};

// and then when calling netstat, specify the platform "custom1" to match the name of the parser and command
netstat({
    platform: 'custom1',
    filter: {
        pid: 4123,
        protocol: 'tcp'
    },
    limit: 5
}, function (data) {
    // ....
});

The proposed functionality is already supported using the aforementioned netstat.commands.custom1 usage. If you feel this usage is clunky or cumbersome (and I'd agree with you to some extent), then we can look at a refactor.


Feat/sortable bug fix: duplication and overwrite issue

Attempt at addressing duplication issue raised here: https://github.com/ObaidUrRehman/ng2-drag-drop/issues/75
Needs further testing as this issue is inconsistent and difficult to duplicate.

Thanks, @cah-alexsukhodolsky. I'll test this.


Implemented disjunction (or) via '|| ' for prefix filters.

All other types of filter disjunctions will currently lead to parse errors since they are harder to properly evaluate and to estimate during planning.

@joka921 Awesome that this feature is available now, thank you!
I just tested it and found an awkward bug. I hope it's easy to fix. Finding a minimal example was not that easy. Consider the following query:
PREFIX wdt: <http://www.wikidata.org/prop/direct/>
PREFIX wd: <http://www.wikidata.org/entity/>
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
PREFIX skos: <http://www.w3.org/2004/02/skos/core#>
SELECT ?predicate ?name ?alias WHERE {
  ?continent wdt:P31 wd:Q5107 .
  OPTIONAL { ?continent @en@rdfs:label ?name }
  OPTIONAL { ?continent @en@skos:altLabel ?alias }
  FILTER (regex(?name, "^\"Afr") || regex(?alias, "^\"Old"))
}

Without the FILTER, this shows all continents, and their names and aliases.
With the FILTER above, it should show two continents (name = Africa) and Europe (alias = Old Continent). However, the result is empty :-(
With only the first regex in the FILTER, the result is correct: one line is shown (Africa).
With only the second regex in the FILTER, the result is wrong: the full result is shown, as if there were no FILTER.
If the first OPTIONAL is omitted (only the keyword, not the triple), one line is shown (Africa), but not both (Africa and Europe) as they should. The same happens, when both OPTIONAL keywords are ommitted.
I checked the corresponding queries on the Wikidata Query Service (without the " after the ^ and with language filters instead of @en@) and the results were correct and as expected in all cases.

@joka921 I'm afraid there is another (hopefully small) bug in the implementation of the new || feature for prefix filters. Consider the following query (it took me quite some time to find a minimal example, the original query was huge and I had no idea at first what went wrong):
PREFIX wdt: <http://www.wikidata.org/prop/direct/>
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
PREFIX wd: <http://www.wikidata.org/entity/>
SELECT ?type ?name WHERE {
  wd:Q873 wdt:P31 ?type . ?type @en@rdfs:label ?name .
  FILTER (regex(?type, "^\"hu") || regex(?type, "^hu") || regex(?name, "^\"hu"))
}

Without the FILTER, this should produce a result with one row: Q5, human.
With the FILTER the result is empty. If you leave out the first regex, it works. If you leave out the second regex, it works. But for some mysterious reason, the first two regexes in combination (although they don't actually filter anything out) lead to a wrong result.


Feat/ethan/getcurrencyinfo

This PR continues from this PR
https://github.com/Permissionless-Software-Foundation/bch-api/pull/166

Removed v4 endpoint per feedback
Moved v5 endpoint from util.js to price.js
Added unit tests to test/v5/
Added integration tests to test/v5/integration

All unit tests pass, which is good. But when those tests are run against eCash infrastructure (full node, Fulcrum indexer, & SLPDB), I get the following errors. I suspect the issue may be with the use of nock. I think it might be messing up the network connections of downstream tests.
413 passing (45s)
  2 pending
  19 failing

  1) #PriceRouter
       #getCurrencyInfo
         should get the currency settings of the full node:

      AssertionError: expected { Object (error) } to have keys 'ticker', 'satoshisperunit', and 'decimals'
      + expected - actual

       [
      -  "error"
      +  "decimals"
      +  "satoshisperunit"
      +  "ticker"
       ]

      at Context.<anonymous> (test/v5/price.js:382:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  2) #Raw-Transactions
       decodeRawTransactionSingle()
         should GET /decodeRawTransaction:

      AssertionError: expected { Object (error) } to have keys 'txid', 'hash', 'size', 'version', 'locktime', 'vin', or 'vout'
      + expected - actual

       [
      -  "error"
      +  "hash"
      +  "locktime"
      +  "size"
      +  "txid"
      +  "version"
      +  "vin"
      +  "vout"
       ]

      at Context.<anonymous> (test/v5/raw-transactions.js:194:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  3) #Raw-Transactions
       decodeRawTransactionBulk()
         should decode an array with a single hex:
     AssertionError: expected { Object (error) } to be an array
      at Context.<anonymous> (test/v5/raw-transactions.js:305:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  4) #Raw-Transactions
       decodeRawTransactionBulk()
         should decode an array with multiple hexes:
     AssertionError: expected { Object (error) } to be an array
      at Context.<anonymous> (test/v5/raw-transactions.js:335:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  5) #Raw-Transactions
       decodeScriptSingle()
         should GET /decodeScriptSingle:

      AssertionError: expected { Object (error) } to have keys 'asm', 'type', and 'p2sh'
      + expected - actual

       [
      -  "error"
      +  "asm"
      +  "p2sh"
      +  "type"
       ]

      at Context.<anonymous> (test/v5/raw-transactions.js:447:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  6) #Raw-Transactions
       decodeScriptBulk()
         should decode an array with a single hex:
     AssertionError: expected { Object (error) } to be an array
      at Context.<anonymous> (test/v5/raw-transactions.js:565:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  7) #Raw-Transactions
       decodeScriptBulk()
         should decode an array with a multiple hexes:
     AssertionError: expected { Object (error) } to be an array
      at Context.<anonymous> (test/v5/raw-transactions.js:585:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  8) #Raw-Transactions
       getRawTransactionBulk()
         should get concise transaction data:
     AssertionError: expected { Object (error) } to be an array
      at Context.<anonymous> (test/v5/raw-transactions.js:708:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  9) #Raw-Transactions
       getRawTransactionBulk()
         should get verbose transaction data:
     AssertionError: expected { Object (error) } to be an array
      at Context.<anonymous> (test/v5/raw-transactions.js:728:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  10) #Raw-Transactions
       getRawTransactionSingle()
         should get concise transaction data:
     AssertionError: expected { Object (error) } to be a string
      at Context.<anonymous> (test/v5/raw-transactions.js:840:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  11) #Raw-Transactions
       getRawTransactionSingle()
         should get verbose transaction data:

      AssertionError: expected { Object (error) } to have keys 'hex', 'txid', 'hash', 'size', 'version', 'locktime', 'vin', 'vout', 'blockhash', 'confirmations', 'time', or 'blocktime'
      + expected - actual

       [
      -  "error"
      +  "blockhash"
      +  "blocktime"
      +  "confirmations"
      +  "hash"
      +  "hex"
      +  "locktime"
      +  "size"
      +  "time"
      +  "txid"
      +  "version"
      +  "vin"
      +  "vout"
       ]

      at Context.<anonymous> (test/v5/raw-transactions.js:858:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  12) #Raw-Transactions
       sendRawTransactionBulk()
         should throw 400 error if hex is invalid:

      HTTP status code 400 expected.
      + expected - actual

      -503
      +400

      at Context.<anonymous> (test/v5/raw-transactions.js:925:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  13) #Raw-Transactions
       sendRawTransactionBulk()
         should submit hex encoded transaction:
     AssertionError: expected 'Network error: Could not communicate with full node or other external service.' to include 'bad-txns-inputs-missingorspent'
      at Context.<anonymous> (test/v5/raw-transactions.js:1013:18)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  14) #Raw-Transactions
       sendRawTransactionSingle()
         should throw an error for invalid hex:

      HTTP status code 400 expected.
      + expected - actual

      -503
      +400

      at Context.<anonymous> (test/v5/raw-transactions.js:1090:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  15) #Raw-Transactions
       sendRawTransactionSingle()
         should GET /sendRawTransaction/:hex:
     AssertionError: expected 'Network error: Could not communicate with full node or other external service.' to include 'bad-txns-inputs-missingorspent'
      at Context.<anonymous> (test/v5/raw-transactions.js:1176:18)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  16) #SLP
       txDetails()
         should get tx details with token info:

      AssertionError: expected { Object (error) } to have keys 'tokenIsValid', or 'tokenInfo'
      + expected - actual

       [
      -  "error"
      +  "tokenInfo"
      +  "tokenIsValid"
       ]

      at Context.<anonymous> (test/v5/slp.js:1166:16)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  17) #Util
       #validateAddressSingle
         should validate address:

      AssertionError: expected { Object (error) } to have keys 'isvalid', 'address', 'scriptPubKey', 'ismine', 'iswatchonly', or 'isscript'
      + expected - actual

       [
      -  "error"
      +  "address"
      +  "ismine"
      +  "isscript"
      +  "isvalid"
      +  "iswatchonly"
      +  "scriptPubKey"
       ]

      at Context.<anonymous> (test/v5/util.js:164:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  18) #Util
       #validateAddressBulk
         should validate a single address:
     AssertionError: expected { Object (error) } to be an array
      at Context.<anonymous> (test/v5/util.js:286:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)

  19) #Util
       #validateAddressBulk
         should validate a multiple addresses:
     AssertionError: expected { Object (error) } to be an array
      at Context.<anonymous> (test/v5/util.js:314:14)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)


uProxy icons should look "active" when you are proxying

NTS to create animated gifs for these

Updated retina Chrome/Firefox extension icons can be found here: https://drive.google.com/open?id=0B4hBIX40_CeZMnZxOEVPS1QyMGM&authuser=0
Preview of "active" animated gifs:

And here are icons for the UI
Getting access: https://drive.google.com/open?id=0B4hBIX40_CeZRFlEajdYd3habG8&authuser=0
Sharing access: https://drive.google.com/open?id=0B4hBIX40_CeZWGoyZkhPV3BtWnM&authuser=0

@lucyhe / @dborkan would you be able to swap these out in the UI, and then we can critique things like speed, color, curves, etc. in context?

NICE!
On Thu, Feb 5, 2015 at 9:10 PM, Izzie Zahorian<EMAIL_ADDRESS>wrote:

And here are icons for the UI
Getting access:
https://drive.google.com/open?id=0B4hBIX40_CeZRFlEajdYd3habG8&authuser=0
Sharing access:
https://drive.google.com/open?id=0B4hBIX40_CeZWGoyZkhPV3BtWnM&authuser=0
[image: gettingaccess_ui_40]
https://cloud.githubusercontent.com/assets/945845/6073354/187f14ea-ad62-11e4-849d-214892f3234f.gif [image:
sharingaccess_ui_40]
https://cloud.githubusercontent.com/assets/945845/6073358/1e153128-ad62-11e4-94e7-1f4024233e9d.gif
@lucyhe https://github.com/lucyhe / @dborkan
https://github.com/dborkan would you be able to swap these out in the
UI, and then we can critique things like speed, color, curves, etc. in
context?
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/849#issuecomment-73170330.

--
Lucas Dixon | Google Ideas

@gwpenn has mentioned this a few times but just wanted to make sure it's noted somewhere: if the user sets a Chrome theme (ex. below) we want to make sure the icons are still visible.

Thanks, Lucy! I'll plan on iterating on these during the next 3 week sprint to try variations with a bg color.

Awesome :) I'm guessing not a huge priority, but thanks for getting to it when you can!
Update on animations:

extension icons in Chrome can't be animated, but it's looking good in FF
I think the blinking giving/getting UI icons make proxying more clear! Love it!

Do we also want the blinking icon in the black status bar? If yes, would it be remove the tiny white border around the icon:

Now using the animated icons, done here: https://github.com/uProxy/uproxy/pull/897
While the animated gifs work in Chrome popups, unfortunately they are not supported for the browser icon. FF browser icon is animated, though.


Roberta multiple choice is failing deterministically on Grayskull

models/experimental/roberta/tests/test_roberta_for_multiple_choice.py::test_roberta_for_multiple_choice is failing deterministically on Grayskull. We need to look into it
cc: @TT-billteng

This test started failing after commit 4b39362.
Previously computed logits were PyTorch [0.1566, 0.1597] and TT [0.157227, 0.161133]. After commit we have PyTorch [-0.0451, -0.0456] and TT [-0.0371094, -0.0390625]. Values still match but with slightly higher difference. PR #5493 to raise ATOL from 0.008 to 0.0081 will make this test pass.

@vtangTT can you confirm if this is true or not

@boris-drazic do we no longer see this in scheduled pipelines?

No longer seeing this failure on pipelines.


Fix incorrect c3 instance types bandwidth accordingly to GCP document…

…ation https://cloud.google.com/compute/docs/general-purpose-machines#c3-standard
First off, thanks for taking the time to contribute!
Please Complete the Following

[ ] I read CONTRIBUTING.md
[ ] I used tabs to indent
[ ] I have tested my change

Notes
Feel free to put whatever you want here.

Thank you 😊


Support the FNT file format

Additional data: Ascendancy Font Editor v1.1+, which contains the format documentation. daumiller/ascendancy seems to have font support, too.

Documentation from the Ascendancy Font Editor: ASFNTEDT.TXT


Add support for OpenShift Route's

Maybe auto-detect if it is openshift? Or, just use a flag to enable it similar to nginx ingress?

Meeting notes: It may be possible to install the router operator into Kind to enable an easy method for testing. We would like for this to be implemented by the end of 1Q 2021 to support our integration use-cases if at all possible.

@SaaldjorMike Any update on this one?

@SaaldjorMike Any update on this one?

@zach-robinson Unfortunately not.

@zach-robinson Unfortunately not.


WIP: Travis CI: Add JDK 10 tests

not sure yet which version is supported by Travis ATM

Both oracle and openjdk still seem to be unsupported:
https://github.com/travis-ci/travis-ci/issues/9368

It works now!


fix(manage-merge-queue): remove PRs from queue properly

:pencil: Description

Fixes an issue where PRs were being removed with stale pending statuses

:tada: This PR is included in version 1.59.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Support for std::string_view in sf::String

Description
Revision of https://github.com/SFML/SFML/pull/2451
This is in a new branch and up to date with the origin's master as of this post.
This PR is related to the issue #2445
Tasks

[ ] Tested on Linux
[x] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
Test driver (modified to match changes in sf::Text):
#include <SFML/Main.hpp>
#include <SFML/Graphics.hpp>
#include <SFML/Window.hpp>

#include <string_view>

int main() {
  sf::RenderWindow window(sf::VideoMode(sf::Vector2u(800, 600)), "SFML Testbed");
  window.setFramerateLimit(60);

  sf::Font font;
  bool derp = font.loadFromFile("c:/Windows/Fonts/arial.ttf");

  const char data[] = "derpdurr";
  const wchar_t wdata[] = L"herpboing";

  std::string str(data + 4, 4);
  std::string_view sv(data, 4);
  std::wstring wstr(wdata, 4);
  std::wstring_view wsv(wdata + 4);

  std::vector<sf::Text> texts;
  texts.emplace_back(font, wstr);
  texts.back().setPosition(sf::Vector2f(10, 0));
  texts.emplace_back(font, sv);
  texts.back().setPosition(sf::Vector2f(10, 40));
  texts.emplace_back(font, str);
  texts.back().setPosition(sf::Vector2f(10, 80));
  texts.emplace_back(font, wsv);
  texts.back().setPosition(sf::Vector2f(10, 120));

  for(bool run = true; run;) {
    window.clear();
    for(auto& text : texts) {
      window.draw(text);
    }
    window.display();

    sf::Event event;
    while(window.pollEvent(event)) {
      if(event.type == sf::Event::Closed) { run = false; break; }
    }
  }
}

I'll be more careful in the future.
Thank you for your patience.

The current implementation in origin:master has constructors for
string/wstring/u32string. I added string-view/wstring_view/u32string_view
to match those. I didn't see anything relevant for u16.
On Wed, Apr 12, 2023, 9:57 PM Chris Thrasher @.***>
wrote:

@.**** commented on this pull request.
In src/SFML/System/String.cpp
https://github.com/SFML/SFML/pull/2517#discussion_r1164999620:

@@ -118,7 +129,13 @@ String::String(const char32_t* utf32String)

////////////////////////////////////////////////////////////
-String::String(const std::u32string& utf32String) : m_string(utf32String)
+String::String(std::u32string_view utf32String) : m_string(utf32String)
What about the u16string overloads?
—
Reply to this email directly, view it on GitHub
https://github.com/SFML/SFML/pull/2517#pullrequestreview-1382629599, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACXDSQC3ZBOQVFLFFA5WHI3XA6BTXANCNFSM6AAAAAAW4QUKMY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I always thought it was used for large character sets like Hangul or Mandarin.

You probably mean Unicode in general, but most modern code uses either UTF-8 or UTF-32 to represent them, which can be encoded in various types in C++ (e.g. char, std::uint8_t, std::char8_t for UTF-8).
wchar_t on the other hand is a legacy type, mostly to deal with UCS-2/UTF-16 encoding at the time. See also StackOverflow or Wikipedia. You generally don't see modern APIs use it except for interop; newer proramming languages don't even have such a type.
I'm not saying sf::String should not provide interop for those, but I think their usefulness is overstated. And in my experience, these legacy APIs tend to be low-level (i.e. pointer/size pairs rather than C++17 wstring_view). But maybe it's worth providing just for consistency.


Macro causes problem when used in the middle of a statement

a &&
#ifdef ALLOW_QUICK_THROW
{b1} &&
#else
{b2} &&
#endif
{c}

    [1,2]:error:can't interpret statement (missing ;?): <Variable(a)> <Keyword(&&)>
    [2,5]:error:can't interpret statement (missing ;?): <Code({b1})> <Keyword(&&)>
    [3,5]:error:can't interpret statement (missing ;?): <Code({b2})> <Keyword(&&)>

Not easily solved because the parser cannot decide, without knowing about ALLOW_QUICK_THROW´, whether a && ` is syntactically wrong or not.
In principle, this could be solved by evaluating both branches, a && {b1} && {c} and a && {b2} && {c}, but this is challenging because they could cause completely different things (e.g. one branch is error, the other not).
I am doing that in some sense to evaluate if-then, but this is at the level of the parser, not the analyzer, so it is a bit harder to tackle.

It is rare, maybe have it as a warning rather than error?

Fixed in 7ac5ec9052d0ca34e5afe815f2436fd8be83661c
Essentially, the parser now parses #ifdef and friends, taking the branch whose #defined variables are in the script (i.e. #defines that are #included are currently not taken into account).

if (x1) then {
    systemChat "A";
#ifdef DEBUG_MODE_FULL
} else {
    systemChat "B";
#endif
};

will crash with:
Traceback (most recent call last):
  File "V:\sqf\sqflint.py", line 98, in <module>
    _main()
  File "V:\sqf\sqflint.py", line 94, in _main
    main(sys.argv[1:])
  File "V:\sqf\sqflint.py", line 86, in main
    analyze(code, writer)
  File "V:\sqf\sqflint.py", line 20, in analyze
    result = parse(code)
  File "V:\sqf\sqf\parser.py", line 543, in parse
    result = parse_block(tokens + [EndOfFile()], _analyze_tokens)[0]
  File "V:\sqf\sqf\parser.py", line 428, in parse_block
    expression, size = parse_block(all_tokens, analyze_tokens, i + 1, lvls, stop_statement, defines=defines)
  File "V:\sqf\sqf\parser.py", line 476, in parse_block
    assert(isinstance(expression, IfDefStatement))
AssertionError

Yeah, I am aware of that case (and that ACE has one file with that :) ). Between parenthesis is still not solved (nor unit-tested)

@PabstMirror, I pushed a fix that, while does not solve the problem entirely, it does avoid that crash and just raises an SQFParserError saying that parenthesis are not balanced or something.

This is still being problematic for some cases - https://github.com/IDI-Systems/acre2/issues/346#issuecomment-307136574