Save Multiple Lora as Metadata

Hi there,
I'm having an issue where I'd wish to save multiple Loras that I'm stacking. I am not sure if the Image Saver node was meant to only save the checkpoint or if it could accept other types of models to populate Civitai resources. Here is my current workflow:

Currently all other metadata seems to be checked on Civitai :

I've tried to follow the json file format for json and adds commas between each Loras and add the main checkpoint at first. I am almost sure that this is where I would need to reword it so the modelname would accept it. I am not a fan of including all the <lora:Lora:0.8> inside the prompts, so that would be pretty useful:

I am currently testing this with XL, but will eventually tackle the Flux compatibility. Any help would be appreciated as I currently have to upload 1 image per resource used and this would be a game changer!
Thank you!

Hashes for LoRAs and embeddings are extracted from the prompt, in the format that you can find in the readme. I recommend using ImpactWildcardEncode to do this conveniently. It also has advantage that you can easily copy and reuse LoRA info just by copying the string, instead of having to copy a bunch of widget settings. And you have to do this for embeddings already.
For the moment this is the only supported way and is unlikely to change. If you want to use a stacker from the efficiency pack, I believe there is a node that can convert that to a compatible string or you can do so manually.

Ah I see now, that make sense.
I've now fixed this with a LoRA Stack to String converter node and combined the main checkpoint with a text concatenate. I'd thought you had to feed them all in the modelname, but I've Text Concatenate it into the main prompt and it worked.

While waiting, I as working on the 2nd node that I knew could also save metadata, the RvTools. I was successful with their node and liked their "loras" input that I could feed in instead of in the prompt. Knowing that I could feed the Loras into the prompt earlier and I would have sticked to Image Saver nodes.
I am currently trying to make a workflow for Flux, that would also capture the metadata just like SDXL, but I am seeing lots of image quality difference when using a KSampler instead of the SamplerCustomAdvanced node to be able to gather the info. The thing with the SamplerCustomAdvanced node is that there are unique inputs that aren't compatible with both of the Image Save nodes and RvTools. I tried feeding the Flux Guidance number as a CFG, but I think the Guider is what I am missing as it seem to additionally mix with the models once more.
Were you successful in making having the Image Saver compatible with Flux before while having the same quality?
Here's my current workflow, which I've unfortunately used the RvTools nodes, but are very similar to the Image Saver nodes. On the left is the SamplerCustomAdvanced and the right is the KSampler results:

Nevermind, I was overcomplicating myself. I just need to feed the CFG / Guidance number to the FluxGuidance and to the Image Saver node.
Everything works now!


[BEAM-2223] PipelineOptionsFactory: improve debuggability

Include the name of the offending interface in the error message
R: @lukecwik

LGTM

I'm guessing that you added a wrapping runtime exception instead of improving all the messages of the places we currently throw because the exception your getting is from some location we don't have control over?

@lukecwik See the stack trace in BEAM-2223.
It is plausible that we could wrap the ClassLoader invocations directly, but given that we get Error thrown in various places it seemed safer to have a catch-all at the front door.
Thoughts?

Its just that the message from the failure will not be the top exception so users will be:
failed to validate MyIface.class
...lines of stack trace...
actual validation failures
...some more lines of stack trace...

Coverage decreased (-0.0004%) to 70.537% when pulling 4ae133c0e05427016c4a6ab60d0b259bde6a1ad6 on dhalperi:pipeline-options-debugging into 23731fe7ab12ea5d3ebb7b3f16c788b83d0caf0b on apache:master.


NBabel benchmark - Ecological Impact of HPC ...

Did you read this paper published in Nature: The Ecological Impact of High-performance Computing in Astrophysics ?
One of the few figures show that Python-Numpy is very inefficient for a particular problem (N-Body). The code used should be available here https://www.nbabel.org/.
I played a bit on this problem (https://github.com/paugier/nbabel) since it would be interesting to show what can be done with Pythran on this case (published in Nature!) and some people in my lab work on the subject of the impact of computing on environment (@CyrilleBonamy).
The Pythran code is in this file https://github.com/paugier/nbabel/blob/master/py/bench.py Do you see some possible improvements in this code?
On my machine, the speedup compared to Python without acceleration is huge (~ x180) but it is still ~ 4 times slower than the Fortran and C++ implementations (which are not that crazy). Also, the 3 implementations do not give the same results :slightly_smiling_face: (the C++ implementation seems to be less accurate) and I don't understand why.
Moreover, I don't understand why -DUSE_XSIMD does not seem to give any speed up. Is it expected?

The description you give sounds super interesting, and I 100% agree with the goal. I'll try to have a look at your implementation soon :-)

Good news. I added an implementation of the most important function with loops (https://github.com/paugier/nbabel/pull/1). It's now faster than the C++ and Fortran implementations.
The elapsed times in second:

Py
C++
Fortran

29
67
51

Note that there are no tests for https://www.nbabel.org/ problem, so I'm not sure about the correctness.
I will try to parallelize with OpenMP.
@CyrilleBonamy, it would be interesting to measure the energy consumption for this algorithm too and to replot the Nature figure. I wonder if some people have thought about submitting a reply to this article.

That's great news! I commented in the PR to rip some extra perf but indeed, one need to be cautious with the actual result ;-)

BTW, you should compile fortran and C++ code with -Ofast too to get a fair comparison ;-)

When doing so, I get Fortran down to 39 which is still not as good as Pythran, but significantly closer.

@paugier: #1671 should make it possible to use
def compute_distance(vec):

    return sqrt(np.sum(vec ** 2)

instead of
def compute_distance(vec):
    tmp = 0.0
    for i in range(3):
        tmp += vec[i] ** 2
    return sqrt(tmp)

without much performance impact :-)

I confirm! Really nice!
Other thing about this code: Pythran gives wrong results (no modification of the arrays when doing inplace assignation) in two cases.

without .copy() in https://github.com/paugier/nbabel/blob/master/py/bench.py#L18
with float[:,3] instead of float[:,:]

A bad thing it that in both case, it runs without error but only the results are completely wrong.


Expiration Date and MFA-Device Filtering

Added functionality which allows for filtering over expiration dates. Also allowed filter for mfa-devices, but at the moment it seems as though this doesn't allow custodian to search for iam-users which have no mfa-devices.

this is going to need a unit test to get past the coverage status check

missing test data dir


VS 2013 with Update5 And Boost 1_61_0

1.在编译方案后,Pingpong并不能正常编译通过,需要自行加入Boost的System相关的lib,希望提高这方面的易用性,你们包好,无需使用者关心包含关系

我不说中文. 你说不说英文?

Here's a translation:

After I try to compile the program, Pingpong cannot pass the compiling unless I add a System library file from Boost. Hope you can improve the usability of your tool to completely take care of the issue like this so user does not need to worry about the dependency.

For Bond.Comm, you will have to link with some of the Boost libraries. We're using functionality that isn't available in just the Boost headers.
If you are defining BOOST_ALL_NO_LIB in your build rules, consider undefining it. This will enable Boost's auto-linking feature. If that doesn't work for you, you'll need to configure each target to link with the required libraries. (BOOST_ALL_NO_LIB is defined when we build Bond itself, but that shouldn't make it into your own projects.)
If you need more help with this, please let us know how you're building your projects, and we'll try to find you some more pointers for your particular environment.
We cannot ship a version of Boost with Bond: that prevents you from using the version most suitable for your environment.

My problem is solved, but Thank you very much. I love you for answer.


aio app dev not loading .env file on Mac Os

Expected Behaviour
I have built an AIO Action that relies on a .env variable for API keys.
The .env file exists in the root folder, the app.config.yaml file has been updated to include the variable reference in the inputs section
I expect that aio app dev will load the .env file and substitute the variable reference (e.g. $TEST_VARIABLE) with the value assigned in the .env file. Thus, my action would have the actual value at runtime.
Actual Behaviour
The variable in app.config.yaml is not being substituted during the build process. Therefore, the variable value is the literal variable name (e.g. $TEST_VARIABLE).
Reproduce Scenario (including but not limited to)
I have build a simple project that replicates the bug: https://git.corp.adobe.com/mboucher/aio-app-dev-bug-mac
Steps to Reproduce

Populate a .env file in the project root and fill it as shown below
run aio app dev at the command line
run curl https://localhost:9080/api/v1/web/aio-app-dev-bug-mac/env -k at the command line

The result will be {"envVariableValue":"$TEST_VARIABLE"} where the literal variable name "$TEST_VARIABLE" is returned instead of the actual value of the TEST_VARIABLE "HELLO_WORLD"
.env
You can generate this file using the command aio app use.
# This file must **not** be committed to source control

## please provide your Adobe I/O Runtime credentials
# AIO_RUNTIME_AUTH=
# AIO_RUNTIME_NAMESPACE=
TEST_VARIABLE=HELLO_WORLD

Environment Info
  System:
    OS: macOS 14.6.1
    CPU: (16) arm64 Apple M3 Max
    Memory: 417.95 MB / 48.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.14.0 - ~/.nvm/versions/node/v20.14.0/bin/node
    Yarn: 1.22.22 - ~/.nvm/versions/node/v20.14.0/bin/yarn
    npm: 10.7.0 - ~/.nvm/versions/node/v20.14.0/bin/npm
  Virtualization:
    Docker: 26.1.1 - /usr/local/bin/docker
  npmGlobalPackages:
    @adobe/aio-cli: 10.2.0

Sample Code that illustrates the problem
https://git.corp.adobe.com/mboucher/aio-app-dev-bug-mac
Logs taken while reproducing problem

Same behavior on M1 Max chip, node 20.
Validated app.config.yaml and action code are correct.
Marking as bug.

This was an old prerelease version of the dev command, uninstalling it resolved the issue.


Add warning "Results are incomplete and may change"

Hello software team,
For a competitor it can be very frustrating to find out they did not advance to the next round even though they had previously checked the live results and thought they qualified. This can happen for example when some results are entered late or when the number of advancing competitors is changed due to no-shows being removed (if the advancement criterion is set to a percentage of total competitors). Unfortunately the Disciplinary Committee is relatively often notified about this, either because a parent complains directly to us about the situation or because a delegate reports a heated discussion during a competition.
I think we could help mitigate this by adding a simple warning when the results are not completely entered. I could see this consist of two/three parts:

On the result page for a round, right below the name of the event, a warning, possibly highlighted in yellow or red: "Results for this round are incomplete. Rankings and qualification limits may change."
On the competitor's result page, a question mark next to rankings for incomplete rounds, showing the same message as above when clicked or tapped.
Same as the previous point on the mobile version when clicking/tapping on a competitor's name from the round results page.

Another idea is to not highlight any ranking in green before the results are complete. This is compatible, but not dependent on, the warning message proposal I explained above.
I hope you can consider implementing something like this soon, but I understand that you are probably busy with more important stuff. If you can think of a different way to solve the same problem feel free to implement that instead :)
Thank you,
Sebastiano

Indeed this confuses and frustrates competitors and parents very often.
To me, not highlight any ranking in green before the results are complete seems the way to go.

I think we should show the highlight, because it encodes relevant information when looking at live results.
For the percentage-based rules I think computing it based on the currently entered results can be slightly confusing, because that dynamically changes the number of advancing competitions. However, given the incidents you both mentioned I agree that it is a better approach as it prevents from false positives.

Closed in #111.


HERE not working – changes in REST API authentication

HERE REST API is no longer generating app codes for authentication. It now only generates API-keys. node-geocoder, on the other hand, throws an error ("You must specify appId and appCode to use Here Geocoder") if appId and appCode are not specified for HERE and apiKey is used instead.
Previously generatedapp_id and app_code for authentification still seem to work. Therefore, node-geocoder can't be used with HERE if you didn't generate the credentials in the past.
https://developer.here.com/documentation/authentication/dev_guide/topics/api-key-credentials.html

This is a working version:
https://github.com/nchaulet/node-geocoder/pull/291

Resolved by #302


Move PR logic to poli-baselines to simplify collaboration

Currently, the implementation of Probabilistic Reparametrization relies on a fork I have edited. This PR moves the logic to poli-baselines.
We are including a file from Meta and its license. I'll also add a clarification in the readme.

Ah, the problem goes deeper: they have a deprecated dependency on sklearn. I'll still point to my fork for installation, but the logic should be contained in this one.


Duo administrator/authentication logs

This collector template would be very helpful to have - there was at least one discussion on the community forum that discussed the particular HMAC requirements for the REST endpoints used with no clear answer.
I think a collect header starting with C.Crypto.createHmac('SHA1', `${C.Time.strftime(new Date().getTime()/1000, '%a, %d %b %Y %H:%M:%S %Z')} followed by the rest of the multi-line payload would probably work, but I'm unsure how to make sure the HTTP Date: header sent by Cribl matches exactly to that same value... can it also be assigned in the headers with some variable like {latest}?
Thanks!

Docs for the API endpoints and HMAC requirements are at https://duo.com/docs/adminapi

This feature is being tracked internally by CRIBL-9726. Setting this ticket as blocked.

This is now unblocked with the release of Cribl 4.8: https://docs.cribl.io/stream/hmac-functions/


npm package public version is still 13.2.6

Hey, could you please update the public version to latest 14.0.7?

latest: 16.0.0
will update soon to 17...


[BUG][opensearch] sudo: a password is required

Describe the bug
sudo: a password is required
Run Playbook
git clone https://github.com/opensearch-project/ansible-playbook.git opensearch-project
ansible-playbook -i host.ini opensearch-project/opensearch.yml -b

Inventory
master0 ansible_host=xxxxx ip=<IP_ADDRESS> roles=master,ingest
data0 ansible_host=xxxxx ip=<IP_ADDRESS> roles=data
data1 ansible_host=xxxxx ip=<IP_ADDRESS> roles=data
dashboard0 ansible_host=xxxxx ip=<IP_ADDRESS>

[os-cluster]
master0
data0
data1
dashboard0

[master]
master0

[dashboard]
dashboard0

[all:vars]
ansible_user=ubuntu
ansible_ssh_private_key_file=~/.ssh/id_rsa
domain_name=opensearch.local
os_download_url=https://artifacts.opensearch.org/releases/bundle/opensearch
os_version=1.3.0
os_user=opensearch
cluster_type=multi-node
os_cluster_name=opensearch
xms_value=8
xmx_value=8

error:
TASK [linux/opensearch : Security Plugin configuration | Create local temporary directory for certificates generation] *******************************************************************
Friday 01 April 2022  11:54:03 +0600 (0:00:01.821)       0:00:15.542 **********
fatal: [master0 -> localhost]: FAILED! => changed=false
  module_stderr: |-
    sudo: a password is required
  module_stdout: ''
  msg: |-
    MODULE FAILURE
    See stdout/stderr for the exact error
  rc: 1

Version
ansible [core 2.12.2]
  python version = 3.8.10 (default, Nov 26 2021, 20:14:08) [GCC 9.3.0]

Server where ansible
cat /etc/*release*
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.3 LTS"
NAME="Ubuntu"
VERSION="20.04.3 LTS (Focal Fossa)"

Remote server
cat /etc/*release*
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.3 LTS"
NAME="Ubuntu"
VERSION="20.04.3 LTS (Focal Fossa)"

@patsevanton As @peterzhuamazon said, we tested this playbook only with root user. I can see you ran with some user which have sudo privileges and faced some error, fixed issue. We will check and test the PR. Thanks


System.InvalidCastException: Specified cast is not valid.

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[] iOS
[x] Android
[] UWP
[] PCL

Version of OS(s) listed above with issue
6.0.1
Version of ACR UserDialogs Library
Affects 6.3.5, 6.3.4. The problem goes away after reverting to 6.3.3 (and earlier).
Expected Behaviour
The prompt dialog should appear as it has with 6.3.3 and earlier.
Actual Behavior
An InvalidCastException is thrown when the dialog is triggered.  The code to trigger the dialog is as follows:
            var startingOdometer = await UserDialogs.PromptAsync(new PromptConfig()
                .SetTitle("Starting Odometer")
                .SetMessage("Input the 6 digit starting odometer.")
                .SetInputMode(InputType.Number)
                .SetMaxLength(6)
                .SetOnTextChanged(args =>
                {
                    args.IsValid = args.Value?.Length == 6;
                }));

Here is the exception:
01-13 14:29:19.896 I/MonoDroid(14415): UNHANDLED EXCEPTION:
01-13 14:29:19.904 I/MonoDroid(14415): System.InvalidCastException: Specified cast is not valid.
01-13 14:29:19.905 I/MonoDroid(14415):   at Acr.UserDialogs.Builders.PromptBuilder+<>c__DisplayClass2_0.b__0 (System.Object sender, System.EventArgs args) [0x00011] in C:\dev\acr\userdialogs\src\Acr.UserDialogs.Android\Builders\PromptBuilder.cs:97
01-13 14:29:19.905 I/MonoDroid(14415):   at Android.Content.IDialogInterfaceOnShowListenerImplementor.OnShow (Android.Content.IDialogInterface dialog) [0x0000d] in /Users/builder/data/lanes/3511/501e63ce/source/monodroid/src/Mono.Android/platforms/android-24/src/generated/Android.Content.IDialogInterface.cs:782
01-13 14:29:19.905 I/MonoDroid(14415):   at Android.Content.IDialogInterfaceOnShowListenerInvoker.n_OnShow_Landroid_content_DialogInterface_ (System.IntPtr jnienv, System.IntPtr native__this, System.IntPtr native_dialog) [0x00011] in /Users/builder/data/lanes/3511/501e63ce/source/monodroid/src/Mono.Android/platforms/android-24/src/generated/Android.Content.IDialogInterface.cs:747
01-13 14:29:19.905 I/MonoDroid(14415):   at (wrapper dynamic-method) System.Object:38dec8fe-7c8f-40af-997e-1759eabc62e4 (intptr,intptr,intptr)
01-13 14:29:19.913 D/Mono    (14415): Assembly Ref addref HockeySDK.AndroidBindings[0x7f85315e00] -> System[0x7f6ff1ae80]: 20
Code of Conduct

No Library Update Requests
No Timeline Requests
Please read through the documentation and FAQs
Please be respectful of my time. I put a lot of effort into my libraries.  I try to support the issue requests within a day.  I'm not psychic, so please take the time to fill out the above form as complete as possible

@aritchie Is there an ETA on when the fix gets pushed to Nuget? It's still on 6.3.5.


README update request

Description
the README states the following quote: "All versions require Python >= 3.7 and Git installed on your system."
But, when I tried to use python 3.7.4, I encountered an error in IDA, at a line that uses python's walrus operator (the := operator) which is added only in python 3.8+
Steps to reproduce the bug
the error was in:
File "C:\Users\my_user_name\AppData\Local\Programs\Python\Python37\lib\site-packages\libbs\decompilers\ida\compat.py", line 818
    while member := ida_enum.get_next_enum_member(_enum, member, 0):
                  ^
SyntaxError: invalid syntax

Environment
No response
Additional context
Love what you do, since I'm new to this, I hope I'm writing this at the correct place! :D

LOL! This is why you don't use complicated language features :(, lol. I think the solution here is to see if I can quickly remove these 3.8 features and make it 3.7 compatible again.
Out of curiosity, what OS and version where you running on this? I'm mostly curious about what OS version comes with 3.7 still.

Love what you do, since I'm new to this, I hope I'm writing this at the correct place! :D

Thank you! The appreciation is also appreciated :)


minor adjustments  to up/down scripts

Looks good to me... I was confused, however, on why we needed two terminals in the Test Cases?


Removed order by from get_group_by in SQLCompiler

Ticket: #26434
I opened this pull request to discuss when collecting group by columns why are order by columns added into extensions.
For example:
Model.objects.values("col_a").annotate(max=Max("col_b")).order_by('col_c')
current query:
select col_a, MAX(col_b) as max
from Model
group by col_a, col_c
order by col_c

I think the expected behavior has should be like that:
django.db.utils.ProgrammingError: column "Model.col_c" must appear in the GROUP BY clause or be used in an aggregate function
Because this behavior changes my group by and result. if I'm lucky, my result doesn't change.
If we check test_annotation_with_value, 'name' column is being added into group_by because Book model has Meta.ordering.

If you want to discuss ideas then please open a new thread on the django-developers mailing list.


make column#limit and column#null work properly (Rebased version)

This is a rebase of #50 which hopefully resolves the conflicts properly.

Thanks @thedrow but it doesn't pass the tests (See travis link in this PR). I have managed to get the tests working on my branch (https://github.com/nulldb/nulldb/compare/master...nulldb:support_limit_and_null?expand=1)
I wonder if you can apply this change and see if that works okay for you, if so I think I can probably merge this in.
@erkki Any feedback (as the original author/requester of this change)

You can edit this PR's branch if you are a maintainer of this repository. It's a new feature of Github.

Thanks @thedrow I have now merged my branch - https://github.com/nulldb/nulldb/pull/72 - Please reference this in your code, if you're happy with the new master branch, I'm happy to cut a gem release.


Reverse Proxy examples

Would it be possible to provide some reverse proxy examples for e.g. nginx or similar.
I cannot get it running properly, it seems to sync if i configure and access it via LAN.
But if i configure it for external access it only syncs to my mobile phone, but not to other browsers.

There's nothing special about the reverse proxy setup.  I have SSL set up for both sync.mywebsite.com and emby.mywebsite.com using fairly standard HAproxy settings in http(s) mode, sent to the correct backend based on host header matching.

There's nothing special about the reverse proxy setup.  I have SSL set up for both sync.mywebsite.com and emby.mywebsite.com using fairly standard HAproxy settings in http(s) mode, sent to the correct backend based on host header matching.

Further, there's no particular reason one client would work better than another based on this.  Its using API commands to the emby server to control the clients, the connection between the actual playing client and the emby server is fully independent of this.


AST: remove dump for GSB

This relied on UB as per [meta.rqmts]p4
(http://eel.is/c++draft/meta#rqmts-4) and has been UB since C++11
20.9.2p1 has the same restriction.
MSVC 19.24 became more stringent and silently ignores this now,
preventing the Swift build.  Require the user to explicitly pass in
-DSWIFT_PERMIT_UNDEFINED_BEHAVIOR to get the dump functionality.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @DougGregor

Since dump() is only meant to be called from a debugger, can you leave it in? Or fix the UB?

@slavapestov, I did leave it in, just behind the macro since it will not even build.  The UB is the entire graph writing support, that needs to be completely rewritten I think.  The template specialization of std::is_pointer is not permitted by C++ ([meta.rqmts]p4).

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - b3e699c33363d0cf828c974fd0bf2cc59d125f11

Build failed
Swift Test OS X Platform
Git Sha - b3e699c33363d0cf828c974fd0bf2cc59d125f11


Python unittest tests do not work unless running within a workspace with the folder added

Python 'unittest' test discovery fails unless the user is using a properly configured workspace.
The unittest module could perhaps be updated to at least inform the user that a workspace is required.  Without having a workspace, no amount of time spent by the user attempting to configure the python.unittest  search path or test pattern, will succeed.

VSCode Version:1.20.1
OS Version: Linux
Extension name and version: ms-python.python-2018.1.0

Steps to Reproduce:

Install ms-python python extension if not installed
Launch VSCode with a folder of python sources (not a workspace)  eg run "code /path/to/python-code"
Modify user settings to enable python unittest module  (File->Preferences-Settings and add       "python.unitTest.unittestEnabled": true in the right panel
Open a python file
at this point, the automatic discovery of tests by the python unittest support from the ms-python module fails, and status bar shows "Test discovery failed"
Enable Output panel (View-> Output)
Display Python Test Log (select "Python Test Log" from the combo/dropdown at top right of output panel
The output panel shows the error message: "Test Discovery failed" and "TypeError: Cannot read property 'uri' of undefined

Does this issue occur when all extensions are disabled?: No, because the python module is required for this.
I have traced this back to the ms-python.python-2018.1.0 extension, file name out/client/unittests/common/managers/baseTestManager.js, function discoverTests, approximate line number 134.
The line of code that fails is:
const wkspace = vscode_1.workspace.getWorkspaceFolder(vscode.Uri.file(this.rootDirectory)).uri;
within this line, the code 'vscode.Uri.file(this.rootDirectory)' correctly returns a valid URI
(file:///path/to/the/python/folder)
However the  method
vscode_1.workspace.getWorkspaceFolder()
returns
undefined
This only happens when using VS Code without having a configured workspace.
The workaround for the issue is:

create a vscode workspace
use 'File->Add folder to workspace'   to add the python folder to the workspace
save the workspace

If requiring a workspace is intended, then a potential fix would be just to inform the user of that fact.
Likely related issues:
https://github.com/Microsoft/vscode/issues/36221
Issue 36221 is related to the operation of workspace.getWorkspaceFolder() changing between the 1.16 and 1.17 vsCode versions (apparently, this method used to work when vscode was opened with a python folder without having a configured workspace)

Additionally, I found that the fix of creating a vscode workspace and adding the python file to it, appears to work just one time.
Once vscode is closed and started again, it once again fails with the same symptoms (error message 'cannot read uri property in undefined'):
I was able to fix this by the following process:

Close vscode
Edit the vscode workspace settings file (eg example.code-workspace)
Modify the value for the 'path' property that specifies the python folder, so that it contains a full path  (by default it seems to contain the path relative to the code-workspace settings file).
Once the path is specified in full, the test discovery succeeds again.

I have tested with a workspace and folder not using any symbolic links, and the issue still occurs as long as the folder path within the workspace settings file is relative.  And the issue goes away (test discovery works) if the folder path is updated to an absolute path.

Update:
I believe I have traced this back entirely to an issue of using vscode with directory paths that use symbolic links.
I will close this issue and open another if necessary.
This test scenario fails:
a real directory:   /data/devel
a symbolic link:         /home/devel   pointing to /data/devel
a python folder:        /data/devel/example-python
a python file:            /data/devel/example-python/init.py
a python test file:     /data/devel/example-python/tests/test_a.py
workspace file:        /data/devel/example.code-workspace
( containing setting folders[0].path:  'example-python' )
The unit tests discovery works and runs fine if executed like this:
code /data/devel/example.code-workspace
The unit test discovery fails if executed like this:
code /home/devel/example.code-workspace


feedback: allow the user to partecipate with access to non-genAI resources

Awsome concept!
However, since many questions are based on mnemonic information, I would like to be able to declare and partecipate to the challenge with access to Google and my books, i.e., non-genAI resources.
It would be nice to have anyway some constraints such that the comparison remains fair, for example a 3-minutes timeframe to search the answer.

Hi Daniele, i.e. you'd like to use some other tools and declare that you used these tools?
We're currently considering such things for the next iterations so it is good to see that you share this interest!

Exactly, with the rational being to have the final score (that compares scientists and LLMs) to be less dependent on the retrieval of mnemonic (easy to access but hard to remember) info.

@danieleongari, if you want to participate in the next round, we have just created a new dataset for "testing humans" with and without tool use, and the app has also been adjusted to handle this.
If you are interested, you can also email/whatsapp me for details.


(PDB-2100) ssl setup fails on nonproduction environments

This changes our ssl-setup script to use "puppet agent" rather than "puppet
master" when gathering its hostname and config file locations.

reopening on stable


Improve privacy of web interface by reducing third party contents

Currently the web interface loads some third party contents. Maybe we can provide some of that locally.

The method du jour is to structure code as a module, rely on Node.JS for managing included libraries and bundle for distribution with webpack/browserify/parcel.


When used in a Navigator, RenderUiKitView tried to access its size without having a size

Fixes:

https://github.com/flutter/flutter/issues/94528
https://github.com/flutter/flutter/issues/83481

Pre-launch Checklist

[x ] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[ x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[ x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[ x] I signed the [CLA].
[ x] I listed at least one issue that this PR fixes in the description above.
[ x] I updated/added relevant documentation (doc comments with ///).
[ x] I added new tests to check the change I am making, or this PR is [test-exempt].
[ x] All existing and new tests are passing.

I committed with the wrong email adress. I will open a new request with the correct one. sorry


Add Improved Name Update Logic

Issue

You can see above here that right not the Update Name logic takes in both the new name that you would like to use, along with the id of the asset that you would like to have updated.
A great update here would be adding an actively selected snippet, or current asset, that tracks the asset that is currently selected based on the user input, with a default as well.
Proposed Solutions
The result of this would be a single input that allows you to put in a new name and update the currently selected asset.
Will need these things:

[ ] a 'currently active' state that is being checked and updates a value that references what the currently active assets ID is
[ ] updates to the renameAsset()

function renameAsset(_name: string, _id: String){

    new Pieces.AssetsApi().assetsSnapshot({}).then(_assetList => {
        for (let i = 0; i < _assetList.iterable.length; i++) {
            if (_assetList.iterable[i].id == _id) {

                let _asset = _assetList.iterable[i];

                _asset.name = _name;

                new Pieces.AssetApi().assetUpdate({asset: _asset}).then(_updated => {
                    console.log("updated:", _updated);
                })
            }
        }
    })
}

[ ] updates to renameAssetInput()

function RenameAssetInput() {
    const [name, setNameValue] = useState('');
    const [id, setIdValue] = useState('');

    const handleNameChange = (event) => {
        setNameValue(event.target.value);
    };

    const handleIdChange = (event) => {
        setIdValue(event.target.value);
    };

    return (
        <>

Name:

            <input value={name} style={{ width: '450px', verticalAlign: 'top' }} onChange={handleNameChange} />

ID:

            <input value={id} style={{ width: '450px', verticalAlign: 'top' }} onChange={handleIdChange} />
            <button style={{ marginTop: '10px', maxWidth: '200px' }} onClick={() => renameAsset(name, id)}>Rename Snippet</button>
        </>

    );
}

Hey @jordan-pieces I am opening a pull request to address this, I also modified the deleteAsset function to use the selected asset

@Pratere been a crazy day over here - but should have some time to go through this today/tomorrow morning ! Will let you know what i find, there were also some larger name updates that took place today so i will merge things and make sure the whole project is good to go before its on main !


Implement antctl agent-info

Issue #275
Output should be like this:
{
  "metadata": {
    "name": "antrea-k8s",
    "selfLink": "/apis/clusterinformation.antrea.tanzu.vmware.com/v1beta1/antreaagentinfos/",
    "uid": "11111111",
    "resourceVersion": "11111111",
    "generation": 36,
    "creationTimestamp": "2020-02-04T10:50:20Z"
  },
  "version": "v0.3.0-dev-4452e66.dirty",
  "podRef": {
    "kind": "Pod",
    "namespace": "kube-system",
    "name": "antrea-agent-0"
  },
  "nodeRef": {
    "kind": "Node",
    "name": "node-k8s1-1"
  },
  "nodeSubnet": [
    "<IP_ADDRESS>/24"
  ],
  "ovsInfo": {
    "version": "1.0",
    "bridgeName": "br-int",
    "flowTable": {
      "0": 5,
      "10": 7,
    }
  },
  "networkPolicyControllerInfo": {},
  "localPodNum": 2,
  "agentConditions": [
    {
      "type": "AgentHealthy",
      "status": "True",
      "lastHeartbeatTime": "2020-02-04T11:25:20Z"
    },
  ]
}

@jianjuns can we merge this local mode command ? I think the redesign is more about remote mode.

@jianjuns can we merge this local mode command ? I think the redesign is more about remote mode.
Remote mode can be implemented separately, and I assume wont impact much your current code. However, I would suggest you to consider changing to reuse the Agent monitor CRD struct and show CRD fields. Could you evaluate that?

Last, I hope after the Agent/Controller info commands, you guys can revisit the design, and decouple API server implementation from antctl, as (again and again) the API will be consumed by other clients besides antctl, and we need to make it generic. If we continue to add more and more commands based on the current framework, it will be become harder to change later.
@weiqiangt @tnqn

@jianjuns can we merge this local mode command ? I think the redesign is more about remote mode.
Remote mode can be implemented separately, and I assume wont impact much your current code. However, I would suggest you to consider changing to reuse the Agent monitor CRD struct and show CRD fields. Could you evaluate that?

Last, I hope after the Agent/Controller info commands, you guys can revisit the design, and decouple API server implementation from antctl, as (again and again) the API will be consumed by other clients besides antctl, and we need to make it generic. If we continue to add more and more commands based on the current framework, it will be become harder to change later.
@weiqiangt @tnqn

Hi @jianjuns, the refactoring is almost finished. I'm polishing the CL and will send it out later today.

/test-all

In the outputs, the "metadata" field looks strange, which should not be needed for CLI outputs, and it is also confusing like we are dumping a K8s resource. Do we have a way to exclude a field?
If not, maybe we should consider to define a new struct, almost copying the AgentInfo CRD but no the metadata field (sorry I suggested to reuse AgentInfo but I did not know we will end up print all fields).

In the outputs, the "metadata" field looks strange, which should not be needed for CLI outputs, and it is also confusing like we are dumping a K8s resource. Do we have a way to exclude a field?
If not, maybe we should consider to define a new struct, almost copying the AgentInfo CRD but no the metadata field (sorry I suggested to reuse AgentInfo but I did not know we will end up print all fields).

@jianjuns I think we'd better use a new struct. TypeMeta and ObjectMeta only have json omitempty tag but we have other output format like yaml, so leaving these value unset won't help.

@jianjuns I think we'd better use a new struct. TypeMeta and ObjectMeta only have json omitempty tag but we have other output format like yaml, so leaving these value unset won't help.

Ok. If no way to filter the fields, probably let us still create a new struct. Sorry for the confusion.

/test-all

@lzhecheng : do you plan to make any further changes for this PR? I hope it can get into 0.4.0.

@lzhecheng : do you plan to make any further changes for this PR? I hope it can get into 0.4.0.

@jianjuns I am ok to merge it. But there will be change after the antctl framework is refactored. If you want to have this command for 0.4.0 and then we can merge it.

/test-networkpolicy

It failed on #386. If it keeps failing, you can rebase on master.
/test-networkpolicy


audit:  --new-formula, Don't fail CI instead comment on PR about audit violations

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

WIP
For --new-formula , dont fail CI when there are audit violations. Instead, comment on formula's PR.
@BrewTestBot user could do the commenting part?
cc @MikeMcQuaid

This somehow would need to guard against making the same comments over and over.

Nice work on this so far!

@MikeMcQuaid Added a check to see if same comment already exists.

This is my bad, I'll look at this more.

@GauthamGoli @ilovezfs I've tested this and done some minor cleanup and it looks good to me. If you're 👍 I'll merge this when I can be around to babysit it for a bit.

Looks good to me.

I'd like to see the references to "new formulae" removed from the messaging, again to avoid planting the immediate-follow-up-PR-to-cheat-the-audit seed in contributors' minds.

Thanks again @GauthamGoli!

brew audit --new-formula doesn't error or have any output when run locally.

Hmm... we do need some way to run this locally and see the new formula audit errors.

Its possible. @new_formula_problems can be inspected

I mean from the CLI.

Oh. Yeah even that is possible , and if HOMEBREW_NEW_FORMULA_PULL_REQUEST_URL environment variable is not set, new formula audit errors are outputted normally.
@commitay  brew audit --new-formula works fine for me. Which audit specifically were you looking for ?

I've suspended all builds on Jenkins as @commitay indicates even regular audits seem not to be running.

e.g brew audit --new-formula imagemagick
No output currently.
Previously:
imagemagick:
  * C: 29: col 3: New Formula should not use `deprecated_option`
  * C: 40: col 3: dependency "freetype" (line 40) should be put before dependency "libtiff" (line 39)
  * C: 43: col 3: dependency "fontconfig" (line 43) should be put before dependency "x11" (line 42)
  * C: 44: col 3: dependency "little-cms" (line 44) should be put before dependency "x11" (line 42)
  * C: 45: col 3: dependency "little-cms2" (line 45) should be put before dependency "x11" (line 42)
  * C: 46: col 3: dependency "libde265" (line 46) should be put before dependency "x11" (line 42)
  * C: 47: col 3: dependency "libwmf" (line 47) should be put before dependency "x11" (line 42)
  * C: 48: col 3: dependency "librsvg" (line 48) should be put before dependency "x11" (line 42)
  * C: 49: col 3: dependency "liblqr" (line 49) should be put before dependency "x11" (line 42)
  * C: 50: col 3: dependency "openexr" (line 50) should be put before dependency "x11" (line 42)
  * C: 51: col 3: dependency "ghostscript" (line 51) should be put before dependency "x11" (line 42)
  * C: 52: col 3: dependency "webp" (line 52) should be put before dependency "x11" (line 42)
  * C: 53: col 3: dependency "openjpeg" (line 53) should be put before dependency "x11" (line 42)
  * C: 54: col 3: dependency "fftw" (line 54) should be put before dependency "x11" (line 42)
  * C: 55: col 3: dependency "pango" (line 55) should be put before dependency "x11" (line 42)
  * C: 56: col 3: dependency "perl" (line 56) should be put before dependency "x11" (line 42)
  * New formulae should not have a HEAD spec
Error: 17 problems in 1 formula

brew audit --new-formula wget worked fine , while brew audit --new-formula imagemagick strangely doesnt return. Am Looking into this right now.

Found the issue.
next if fa.problems.empty? should be replaced with
next if fa.problems.empty? && fa.new_formula_problems.empty?
Opening a PR soon.

brew audit --new-formula python is another with no output

@commitay This issue shouldn't be affecting regular audits. Can you tell me which non new-formula audit weren't getting reported ? So that I can triage

This build failed with a brew test error only.
https://jenkins.brew.sh/job/Homebrew Core Pull Requests/24451/version=high_sierra/console
It should have regular and new formula audit failures

The exit code wasn't being set to a non-zero value. Conditions have been fixed in the PR referenced above.

https://github.com/Homebrew/homebrew-core/pull/28141
This PR failed GitHub notability but the bot didn't comment on it?

https://github.com/Homebrew/brew/pull/4201

@MikeMcQuaid what was the issue..?

@GauthamGoli brew-test-bot wasn't setting the correct environment variable (https://github.com/Homebrew/homebrew-test-bot/commit/b03e66770b07465d851bdab42f9650c0e24e72d3). Also, the GitHub API failure exception message was never shown (
https://github.com/Homebrew/brew/pull/4538) and the API token needed updated to have the correct scopes. After that: 🎉

It's a hypothetical concern at the moment but having green CI when a formula has problems that should prevent it being merged might mean the comment doesn't get noticed and fixed in longer discussions?

@commitay Personally I still prefer the approach before this PR but deferred to @ilovezfs who’s has been requesting this for a while.

The idea is that we should be able to expect all PRs to pass CI, including new formula PRs. If, for example, a new formula must depend on a system-duplicate formula to prevent build failure, the approach before this PR meant that we would have to merge the new formula PR despite its failing CI.
The new approach may ultimately prove unworkable, but it's worth trying given the confusion the prior approach has caused for some contributors who think red means something must not be merged, as is the case in many projects (e.g. those with protected branches).


No easy way to skip some actors.

Actual behavior
There is no way to easily disable or enable actor. The only way to disable actor is to delete it. However, recent refactoring of leapp broke OpenStack CI as path of actor was changed from
/usr/share/leapp-repository/repositories/system_upgrade/el7toel8/actors/
to
/usr/share/leapp-repository/repositories/system_upgrade/common/actors/
I propose to introduce --skip-actors actor1,actor2 or introduce environment variable to be able to disable some actors for CI purposes. That will allow to use cli option no matter how leapp is organized or refactored underneath.
To Reproduce
Install leapp-0.12 and delete actor
Install leapp-0.13 and delete actor using same script.
Expected behavior
Consistent interface for disabling actors.

@holser The  proposal does not make sense from the design POV. With the option you would be able to skip particular actor, which could lead in various breaks / issues / crashes / unexpected outputs ... basically it is breaking the design and provides more problems than solutions. In the same time, it does not solve your problem, as the name of the actor can be changed as well - which happened several times already, even during the last year IIRC.
First, we are missing examples why some actors needs to be removed.

performance - that does not seem so much as a good reason; in case of CI runs, is LEAPP_DEVE_DATABASE_SYNC_OF  specified? usually the majority of time that is consumed by our actors includes operations that cannot be skipped
ignore specific inhibitors - that's a different RFE that is on our roadmap and it will require mark the run as unsupprted
something is not working as expected in case .... - expected solution is opened PR with the proper fixes, not skipping of actors;

Can you provide reasons why to skip some actors that are not included above?
btw,  the suggested feature is supposed to be reported in leapp-repository, as this is the thing of particular repositories (including leapp CLI commands included in them). I will transfer it.


Expose BaseAddress for configuration

RRConfiguration has a "BaseAddress" property, but it is not exposed in RequestReduceConfigSection.
I think it makes sense to expose BaseAddress, I know I would use it.
Thanks!

I actually created that for a "secret" test page I use for sucking down the
markup from other sites and pumping them through requestreduce. Typically
for self hosted scenarios, you would use the contentHost config to adjust
the base of transformed urls. I'm curious why you think this would be
useful.

From: "Craig"
<EMAIL_ADDRESS>b.com>
Sent: Tuesday, March 06, 2012 10:38 AM
To: "Matt Wrock"<EMAIL_ADDRESS>Subject: [RequestReduce] Expose BaseAddress for configuration (#153)
RRConfiguration has a "BaseAddress" property, but it is not exposed in
RequestReduceConfigSection.
I think it makes sense to expose BaseAddress, I know I would use it.
Thanks!

Reply to this email directly or view it on GitHub:
https://github.com/mwrock/RequestReduce/issues/153

I'm having a problem where Request Reduce is outputting root-relative URLs for CSS/JS (ex /app/js/something.js) and not absolute URLs (ex http://mysite/app/js/something.js). I was under the impression RR should always output absolute URLs. So now I'm looking for ways to get RR to output absolute URLs (root relative won't work due to a very unique server side include from another domain).
BaseAddress seemed to be a way to work around that issue, but perhaps it isn't the right way.
Not to derail this issue, but do you have any ideas on why RR might be writing root relative URLs?

That is by design. Currently, RR only creates absolute URLs if you specify a contentHost. So there are a couple ways you might be able to work around this:

Use a contentHost in your url that points to http://. This is the easiest but can be difficult to manage for multiple hosts or sites that have both http and https content.
Use RequestReduce.Api.Registry.Configuration.ContentHost to set this at run time. This should be set at app start since it is a singleton with application scope. So this will likely not work for sites with https and http content.

Will that work for you?

Ah, I didn't understand this.
This mostly works for me. To get the functionality I really wanted, I added this to Global.asax:
        private string handleRequestReduceUrls(string inputUrl)
        {
            string appDomainAppVirtualPath;
            if (HttpRuntime.AppDomainAppVirtualPath == "/")
                appDomainAppVirtualPath = "/";
            else
                appDomainAppVirtualPath = HttpRuntime.AppDomainAppVirtualPath + "/";
            if (inputUrl.StartsWith(appDomainAppVirtualPath, StringComparison.OrdinalIgnoreCase))
            {
                return "http://mysite.com" + inputUrl.Substring(appDomainAppVirtualPath.Length-1);
            }
            //return the input unmodified
            return inputUrl;
        }
        void Application_Start(object sender, EventArgs e)
        {
            RequestReduce.Api.Registry.UrlTransformer =
                (context, originalAbsoluteUrl, urlWithContentHost) =>
                    handleRequestReduceUrls(urlWithContentHost);
        }

Ah. That works. Glad you got it working.


Fix typos

Ⅰ. Describe what this PR did
Fix arugments to arguments
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

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

Project coverage is 44.35%. Comparing base (ef31e09) to head (5eaee4d).
Report is 93 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1300      +/-   ##
==========================================
+ Coverage   35.91%   44.35%   +8.44%
==========================================
  Files          69       75       +6
  Lines       11576     9823    -1753
==========================================
+ Hits         4157     4357     +200
+ Misses       7104     5140    -1964
- Partials      315      326      +11

see 90 files with indirect coverage changes


Add license information

Would you mind adding a LICENSE to this repository? In most jurisdictions, unlicensed software by default does not grant any privileges or protections to anyone besides the author.
Thanks!

@JoshDrake-minted No problem... fixed in 342fd541276d207e26525497b09262b0301ff5b3. ✌


Introduce the cranelift-bitset crate; use it for stack maps in both Cranelift and Wasmtime

@alexcrichton this should be ready for a second round of review now.


Add @dfinity/auth-client as peer dependency or even better, remove it.

It looks like you're just depending on LocalStorage implementation of @dfinity/auth-client. Would be nice to replace that. this pulls a big dependency in which is not required to us. I'm happy to provide a PR if you like.

Please update to 1.0.3


Omit 'begin' callbacks from the _finishAnimation call during childWil…

Thanks! I'll get a release up soon.

Awesome, glad I could help out :)

Shipped in 1.1.4. Thanks again!


How are are suggestions suppose to work ?

I was unable to get the search suggestions to work so I examined the code. It seems the suggestions array is ignoref when deciding to to show the suggestions (Only the the search history db item count is taken into considation)
Am I missing something here ?
https://github.com/lapism/SearchBar-SearchView/blob/6d8a29bd66db1af24f3ebb19f36ff132a6aa423d/src/main/java/com/lapism/searchview/widget/SearchView.java#L478

You need to add suggestions to the Adapter with method setSuggestions(). But it should work like Play Store implementation. That means if you do not have anything in History Database and EditText is empty then you have no visible items. If EditText is not empty then filtering (suggestions + DB) is going to start. But I have to check Play Store if something has changed.
https://github.com/lapism/SearchBar-SearchView/blob/389538c621e3b8568f8fd52a9e962b9ff4ec13a2/src/main/java/com/lapism/searchview/widget/SearchAdapter.java#L245


Screen view count with screen name decreased when used new sdk log screen view method

Step 0: Are you in the right place?

For issues or feature requests related to the code in this repository
file a Github issue.

If this is a feature request please use the Feature Request template.

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general (non-iOS) Firebase discussion, use the firebase-talk
google group.
For backend issues, console issues, and other non-SDK help that does not fall under one
of the above categories, reach out to
Firebase Support.
Once you've read this section and determined that your issue is appropriate for
this repository, please delete this section.

[REQUIRED] Step 1: Describe your environment

Xcode version: _____xcode 12
Firebase SDK version: ____ 6.30.0
Firebase Component: _____ Firebase analytics
Component version: _____6.30.0
Installation method: `CocoaPods

[REQUIRED] Step 2: Describe the problem
For the new sdk i have changed logging screen view to new method Analytics.logEvent(AnalyticsEventScreenView, parameters: [AnalyticsParameterScreenName: screenName]) previously i have used  Analytics.setScreenName(screenName, screenClass: nil)
Now iam experiencing a decrease in screen name count when used new log event method with updated sdk.
Also iam getting cannot log screen view event when app is not active debug log sometimes

I suspect this may be related to how automatic screen view logging detects view presentations. @allenktv anything obvious that might be causing this?

Hi @ChinjuJames12, could you give some code snippets on how you are calling the API? And if you have logs on when you call the log screen view would be very helpful as well.
As for logging screen screen views when the app is not active, the SDK does not allow the screen view events to be logged when the app is in the background. Are you perhaps attempting to log screen views then?

Hi, This are the cases I will call the api.
Firstly my app is news app...so I got tob bar with categories and subcategories like https://github.com/rechsteiner/Parchment so when user move in between this tabs i will be calling log screen view api Analytics.logEvent(AnalyticsEventScreenView, parameters: [AnalyticsParameterScreenName:"will be sending current category name over there like sports,Home etc"])
Secondly  when user select an article from the listing will push to a new viewcontroller  and will call log screen view method over its viewdidappear Analytics.logEvent(AnalyticsEventScreenView, parameters: [AnalyticsParameterScreenName:" Reading news"]).
As for logging screen screen views when the app is not active, the SDK does not allow the screen view events to be logged when the app is in the background. Are you perhaps attempting to log screen views then? - yaa this one I have seen not logging in debugging view of previous version of sdk but now only started showing error like cannot log screen events when app is not active in Xcode debugging console so this is not causing screen view difference

The usage you described sounds normal. It is possible if Analytics' automatic logging uses viewWillAppear that that may cause more screen_view events than viewDidAppear.
Can you share the app's debug logs when you try to log a screen view?

Attaching the logs getting in debug console
NewSdk.docx
The screen logging as in new sdk
2020-10-21 04:21:56.959368+0800 ChinaPress[19088:5979724] 6.30.0 - [Firebase/Analytics][I-ACS031029] Logging screen view with screen name and screen class: 抢鲜, LaunchScreenController
2020-10-21 04:21:56.959368+0800 ChinaPress[19088:5979724] 6.30.0 - [Firebase/Analytics][I-ACS031029] Logging screen view with screen name and screen class: 抢鲜, LaunchScreenController
2020-10-21 04:22:05.302610+0800 ChinaPress[19088:5979721] 6.30.0 - [Firebase/Analytics][I-ACS031029] Logging screen view with screen name and screen class: 阅读新闻, LaunchScreenController
OldSdk.docx
The screen logging as in old sdk
2020-10-21 04:12:11.416103+0800 ChinaPress[19074:5975982] 6.27.1 - [Firebase/Analytics][I-ACS031009] Set screen name and screen class: 抢鲜, LaunchScreenController
2020-10-21 04:12:11.430222+0800 ChinaPress[19074:5975998] 6.27.1 - [Firebase/Analytics][I-ACS031009] Set screen name and screen class: 主页, NewsListWithFlashNewsViewController
2020-10-21 04:12:23.775240+0800 ChinaPress[19074:5976016] 6.27.1 - [Firebase/Analytics][I-ACS031009] Set screen name and screen class: 阅读新闻, EMPageViewController
Here iam passing two of my categories name when open app also when click on listing will send 阅读新闻 .The word logs consist of this and when come pop back stopping it.

Also want to update some thing in my previous sdk i can see when on third click of article from listing saying duplicate view but i can see it successfully logged in debug view.
OldFirebaseSdk.docx
2020-10-21 04:37:11.132543+0800 ChinaPress[19106:5985383] 6.27.1 - [Firebase/Analytics][I-ACS031009] Set screen name and screen class: 抢鲜, LaunchScreenController
2020-10-21 04:37:11.145275+0800 ChinaPress[19106:5985383] 6.27.1 - [Firebase/Analytics][I-ACS031009] Set screen name and screen class: 主页, NewsListWithFlashNewsViewController
2020-10-21 04:37:15.715137+0800 ChinaPress[19106:5985379] 6.27.1 - [Firebase/Analytics][I-ACS031009] Set screen name and screen class: 阅读新闻, EMPageViewController
2020-10-21 04:37:20.130717+0800 ChinaPress[19106:5985376] 6.27.1 - [Firebase/Analytics][I-ACS031009] Set screen name and screen class: 阅读新闻, NewsListWithFlashNewsViewController
2020-10-21 04:37:25.348242+0800 ChinaPress[19106:5985586] 6.27.1 - [Firebase/Analytics][I-ACS031004] Ignoring duplicate screen name and screen class: 阅读新闻, NewsListWithFlashNewsViewController
NewFIREBASESDK.docx
But for new sdk not getting that warning
2020-10-21 04:38:12.033982+0800 ChinaPress[19123:5986613] 6.30.0 - [Firebase/Analytics][I-ACS031029] Logging screen view with screen name and screen class: 抢鲜, LaunchScreenController
2020-10-21 04:38:12.047076+0800 ChinaPress[19123:5986613] 6.30.0 - [Firebase/Analytics][I-ACS031029] Logging screen view with screen name and screen class: 主页, LaunchScreenController
2020-10-21 04:38:31.455803+0800 ChinaPress[19123:5986614] 6.30.0 - [Firebase/Analytics][I-ACS031029] Logging screen view with screen name and screen class: 阅读新闻, LaunchScreenController
2020-10-21 04:38:35.728241+0800 ChinaPress[19123:5986614] 6.30.0 - [Firebase/Analytics][I-ACS031029] Logging screen view with screen name and screen class: 阅读新闻, LaunchScreenController
2020-10-21 04:38:39.307432+0800 ChinaPress[19123:5986617] 6.30.0 - [Firebase/Analytics][I-ACS031029] Logging screen view with screen name and screen class: 阅读新闻, LaunchScreenController

Those logs look as expected. The reason why the old SDK has Ignoring duplicate screen name and screen class is due to the way setScreenName worked, where it automatically ignored screen view events with the same name and class even if they were set by the automatic screen tracking or by the API.

"But when it comes to screen_views total count per day on event console the not set value seems increased and one with screen name seems decreased"

When you mean not set, do you mean screen_name not set? If so, it could be due to the automatic screen views that occur when you switch screens. If you would like to decrease that so that you only have screens with names, I recommend following the blog and setting FirebaseAutomaticScreenReportingEnabled to NO in the info.plist

I mean (not set) Showing in screen_view increased and I can see a drop in one with screen names when used the new sdk

Do the new sdk with new api solved any duplicate screen names issue?

@ChinjuJames12 Yes, as mentioned above, by setting FirebaseAutomaticScreenReportingEnabled to NO, the automatic screen view events are not triggered and will remove (not set) screen names in the console, as well as remove duplicate screen views. The only time screen view events will be logged will be when the API is called manually

Do the new sdk with new api solved any duplicate screen names issue?
I have not changed any other codings in new version expect changed to new screen event method but the screen view with screen name decreased.
Do automatic collected screen views got screen names it will be marking as (not set) right

Yes. You can fix the (not set) screen names by adding the FirebaseAutomaticScreenReportingEnabled key to your app's Info.plist and setting the value to NO.

Is there chance when I turn of FirebaseAutomaticScreenReportingEnabled the one with screen names count will come same as previous version count itself

We haven't yet identified what's causing the decrease in total screen_view events for your app. If you're able to test with a sample application and reproduce the decrease in screen_view events by converting automatically logged screens to manually logged screens, that would be helpful.

@ChinjuJames12 When you say "decreased" screen views with screen name, are you saying the total count of screen_views with screen name has decreased? Or that the percentage of screen views with (not set) has increased?

what I mean is (not set) increased and one with screen names decreased

@morganchen12 reproduce the decrease in screen_view events by converting automatically logged screens to manually logged screens
Means what yaa do automatically logged screens also can have names like 阅读新闻, like 主页

https://github.com/firebase/firebase-ios-sdk/issues/1894 is this bug fix in anyway can affect the one with screen names in new sdk

I see. Please read the blog. The new API gives full control over screen views to the developer, and if the AnalyticsParameterScreenName is set on every call, you will not get any (not set) screen names in your console

@allenktv So in previous sdk the screen name saving don’t have full developer control it is mixed with automatic screen report also.
Please help yaa my client is asking why the count is decreased

I see. Please read the blog. The new API gives full control over screen views to the developer, and if the AnalyticsParameterScreenName is set on every call, you will not get any (not set) screen names in your console
Right now I have not turned off FirebaseAutomaticScreenReportingEnabled and Iam passing AnalyticsParameterScreenName with all api calls

I found the bug the sdk version used earlier  is Firebase/Analytics (6.27.1)
in which  can see the 阅读新闻 logged when press back button also...but from my code wise iam not sending it...
I cannot upload video over here if you give your mail id i will send video in which debug view logging it twice but from console sending it only one time

Since you say you are not setting FirebaseAutomaticScreenReportingEnabled to NO, the screen view event that is logged when you press the back button is from the automatic screen view reporting, and is not a bug.
I am closing this thread now since it has been established that the solution is to turn off automatic screen tracking.


Write a Scala program to add two matrices

Description
Write a Scala program to add two matrices
Matrix addition is the process of adding two matrices by adding the corresponding elements of the two matrices. Two matrices must have an equal number of rows and columns to be added.
Input  :
1 2 3
4 5 6
7 8 9

1 2 3
4 5 6
7 8 9

Output :
2 4 6
8 10 12
14 16 18

How to contribute

Save the solution in program/addTwoMatrices.scala file
Add addTwoMatrices.scala file in add-two-matrices folder

!assign

Hey @BrokenDesign,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)


Update: VivaldiTechnologies.Vivaldi version 3.6.2165.40

Removing old versions to reduce clutter
Microsoft Reviewers: Open in CodeFlow

/azp run


fix tests for zig v0.12

Thanks again ! Also I'm curious, are you using this code for anything ?

Nope - just exploring and learning iouring & zig at the same time (probably should stick to one at a time :laughing:)!

Nope - just exploring and learning iouring & zig at the same time (probably should stick to one at a time 😆)!

Haha alright, this project served basically the same purpose for me at the time :)


Automatic PR from RestAPI-PR1756

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1756

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: AutorestCI sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1488   +/-   ##
=======================================
  Coverage   55.07%   55.07%
=======================================
  Files        3454     3454
  Lines       84233    84233
=======================================
  Hits        46395    46395
  Misses      37838    37838

Impacted Files
Coverage Δ

...h/models/compute_node_enable_scheduling_options.py
42.85% <ø> (ø)
:arrow_up:

...re-batch/azure/batch/models/pool_resize_options.py
27.27% <ø> (ø)
:arrow_up:

.../azure/batch/models/file_list_from_task_options.py
33.33% <ø> (ø)
:arrow_up:

...e/batch/models/compute_node_update_user_options.py
42.85% <ø> (ø)
:arrow_up:

...atch/azure/batch/models/certificate_get_options.py
37.5% <ø> (ø)
:arrow_up:

...h/azure/batch/models/task_list_subtasks_options.py
37.5% <ø> (ø)
:arrow_up:

...re/batch/models/pool_disable_auto_scale_options.py
42.85% <ø> (ø)
:arrow_up:

.../azure/batch/models/job_get_task_counts_options.py
42.85% <ø> (ø)
:arrow_up:

.../azure/batch/models/task_add_collection_options.py
42.85% <ø> (ø)
:arrow_up:

azure-batch/azure/batch/models/task_get_options.py
23.07% <ø> (ø)
:arrow_up:

... and 65 more

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 2fdfd7d...bdf5188. Read the comment docs.

Doc only


❓ [Question] Not enough inputs provided (runtime.RunCudaEngine)

❓ Question

i make a pressure test on my model compiled by torch-tensorrt, it will report errors after 5 minutes, the traceback as blow:
2022-09-09T09:16:01.618971735Z   File "/component/text_detector.py", line 135, in __call__
2022-09-09T09:16:01.618975181Z     outputs = self.net(inp)
2022-09-09T09:16:01.618978313Z   File "/miniconda/envs/python36/lib/python3.6/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
2022-09-09T09:16:01.618981965Z     return forward_call(*input, **kwargs)
2022-09-09T09:16:01.618985142Z RuntimeError: The following operation failed in the TorchScript interpreter.
2022-09-09T09:16:01.618988457Z Traceback of TorchScript, serialized code (most recent call last):
2022-09-09T09:16:01.618991980Z   File "code/__torch__.py", line 8, in forward
2022-09-09T09:16:01.618995305Z     input_0: Tensor) -> Tensor:
2022-09-09T09:16:01.618998495Z     __torch___ModelWrapper_trt_engine_ = self_1.__torch___ModelWrapper_trt_engine_
2022-09-09T09:16:01.619001820Z     _0 = ops.tensorrt.execute_engine([input_0], __torch___ModelWrapper_trt_engine_)
2022-09-09T09:16:01.619005168Z          ~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
2022-09-09T09:16:01.619008442Z     _1, = _0
2022-09-09T09:16:01.619011485Z     return _1
2022-09-09T09:16:01.619014563Z
2022-09-09T09:16:01.619017565Z Traceback of TorchScript, original code (most recent call last):
2022-09-09T09:16:01.619020865Z RuntimeError: [Error thrown at core/runtime/register_trt_op.cpp:101] Expected compiled_engine->exec_ctx->allInputDimensionsSpecified() to be true but got false
2022-09-09T09:16:01.619024625Z Not enough inputs provided (runtime.RunCudaEngine)

then i get an error about cuda memory illegal access:
2022-09-13T02:32:46.621963863Z   File "/component/text_detector.py", line 136, in __call__
2022-09-13T02:32:46.621966267Z     inp = inp.cuda()
2022-09-13T02:32:46.621968419Z RuntimeError: CUDA error: an illegal memory access was encountered

What you have already tried

I have tried upgrade the pytorch version from 1.10.0 to 1.10.2, also tried upgrade torch to 1.11.0 python 3.7, but it didn't works.
Environment

Build information about Torch-TensorRT can be found by turning on debug messages

PyTorch Version (e.g., 1.0): 1.10.2
CPU Architecture: x86
OS (e.g., Linux): centos 7
How you installed PyTorch (conda, pip, libtorch, source): pip
Build command you used (if compiling from source): /
Are you using local sources or building from archives: no
Python version: 3.6
CUDA version: 11.3
GPU models and configuration:  gpu is nvidia-T4 with 16G memory
Any other relevant information:

Additional context

Is there any repro script you can share here?


typo

Codecov Report

Merging #275 (d154499) into main (d24f2d9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #275   +/-   ##
=======================================
  Coverage   65.41%   65.41%
=======================================
  Files          33       33
  Lines        3233     3233
=======================================
  Hits         2115     2115
  Misses        806      806
  Partials      312      312

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 d24f2d9...d154499. Read the comment docs.

Thanks 🙏


Fix deprecations that showed up in Flutter 3

Flutter 3, the latest stable version, deprecated some APIs that are used in super_editor. Go through and refactor those uses, as needed.

cc @matthew-carroll this can be closed now that #595 was merged


Suggest a project

Project
SquadSwap
Description
SquadSwap is a DEX created by the Pancake Squad NFT community. You can easily swap, provide liquidity and earn fees. SquadSwap is powered by it’s native token, SQUAD.
Please provide the project scope.
Safe Apps
Select a primary category of your project.
DeFi
Select secondary categories of your project.
DEX, DeFi
Project logo
https://drive.google.com/file/d/15qYqwW3M6CdW-BUf1UtbLBJ53bKpz5pP/view?usp=sharing
Value proposition
No response
Contact
https://t.me/Oxfforest https://t.me/Steve_Cake007
Project URL
https://squadswap.com/
Github / Dev Docs
No response
Twitter
https://twitter.com/squad_swap
Primary integration
Safe Apps SDK
Packages
safe-apps-wagmi
Modules / Guards
No response
Smart Accounts
No response
Networks
BNB
Can you create a Safe via the interface?
None
Can you import an existing Safe via the interface?
None

hey @eyyubmermer - have you followed our process (https://docs.safe.global/safe-apps-sdk/release) to be listed as a Safe App on our Safe Wallet interface? This is a prerequisite to be added to the Eco page.

I apologize for the late response. I waited for Andrei from your team to review our audit reports. We want to list our app on gnosis. He approved it and said we could continue with the submission. We integrated Gnosis safe SDK into our project. We use Wagmi. We also have a manifest.json file. Who can we get support on if we are missing something to be listed?


Fix: IAM snippets

on Windows, IAM snippets doesn't correctly resolve the module folder(s) name(s), mostly 'cause file paths on windows contains backslashes \, example: C:\projects\my-project\backend\modules\my-module.
This PR resolves that by supporting both slashes.

Yes, right. Thanks for the fix. I will merge that.


chore(deps): group dependencies for easier updates

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
959

Relevant Lines:
2081

💛  - Coveralls


Clean option no works properly

Describe the bug
Visual bug in django-select2 option, when I try to clean the content in the box, removing all items, but if I select a new, I get a blank space. I use django-select2==7.2.0 (lastest version)
Exception & Traceback
If I click 2 times in the "x", the content is cleaned completely
Code Snippet
forms.py

views.py
especialidadesFormset = CentroInvestigacionEspecialidadForm(prefix='especialidades')

reg.html
<div class="col-12">
       {{ especialidadesFormset.management_form }}
        {% for form in especialidadesFormset %}
              {{ especialidadesFormset }}
        {% endfor %}
  </div>

To Reproduce
Steps to reproduce the behavior:

Go to the view with the select2 field
Add/select one or more options
Remove all items, clicking in "x"
Select another item of the list
Will see a blank selection before the last selected

Expected behavior
It must not show that blank space, with my selection
Screenshots

This repo has moved, please address your issue at https://github.com/codingjoe/django-select2 Thanks!


Table 5054 "Contact Business Relation" Primary Key

I need a change in the primary key of table 5054 "Contact Business Relation"
"Contact No.","Business Relation Code" as:
"Contact No.","Business Relation Code","No."
Thank you.

We do not provide changing of primary keys, this is not possible.


add initial content for release consolidation

This PR combines the content of the website and tractus-x-release repository into sig-release.
It thereby fixes the Issue #572
See:

https://github.com/eclipse-tractusx/tractus-x-release/pull/26
https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/pull/852

Superseeds https://github.com/eclipse-tractusx/sig-release/pull/519

Like this PR and the description. Since the lifecycle management changed a little (Maybe not for Tractus-X, but I would like to check) and also the standards are not provided on the “expected” URL any more, I would like to review this PR ... but ... not this week. It would be planned for next week.

@Grand-Thibault maybe we can review this together because some parts have to match the working model (at least the open-source part :))


Closing a subscription should not close other subscriptions.

If you have multiple subscriptions open for the same stream then calling the close method should close each subscription, not all subscriptions on the client.

Subscriptions are single threaded and each subscriber should have their own subscription which they are responsible for closing. This is different from publications that are shared and ref counted.

Apologies, the title and text on the original issue had a typo in. I meant "subscription" where I wrote "subscriber". Does that make more sense now?

The subscriptions are independent and should not be shared. I cannot recreate. Can you provide more details?

Does this still happen with latest changes?

I've not seen this issue now, so I'm going to close it.  If it crops up again then I'll re-open.


Recover the data in the USB drive

Hi all,
I used the etcher to flash an os image to my USB drive but I forgot to copy all the original files in the USB drive before I did the action. Then, I lost all my files......
Is it possible to recover the USB drive? How can I do that?
Thanks

Not likely
On Wed, May 30, 2018, 8:48 PM seevider<EMAIL_ADDRESS>wrote:

Hi all,
I used the etcher to flash an os image to my USB drive but I forgot to
copy all the original files in the USB drive before I did the action. Then,
I lost all my files......
Is it possible to recover the USB drive? How can I do that?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/resin-io/etcher/issues/2378, or mute the thread
https://github.com/notifications/unsubscribe-auth/AF79fDj8_IgK9Isy7iD0MqBp4Egvc2gsks5t3z29gaJpZM4UUPDa
.

Sorry to hear that @seevider, unfortunately that's bad news. There might be a possibility to recover some files, depending on large the image you flashed was, and some other factors. But I'm afraid we can't do much, apart from suggesting recovery software that helped some people before (have to go look them up again)

Thank you for your information @jhermsmeier. I formatted my USB driver and the storage space went back to original size. I will try the recovery software to recover the data.

Unfortunately formatting your drive first may make the recovery process less successful :-(


Clustering, skipping find

Improves the performance if CLUSTER_SERVERS are used, especially if there are multiple backend servers.

On the first request to any backend node a pool of worker threads gets created
Everything that can be offloaded from the request handling threads or that can be parallelized is sent to the worker pool via queues
The request handling thread and the worker threads synchronize via locks and queues

In my benchmarks it performs better than both master and 0.9.15

Codecov Report

Merging #1818 into master will decrease coverage by -0.62%.

@@            Coverage Diff             @@
##           master    #1818      +/-   ##
==========================================
- Coverage   65.88%   65.27%   -0.62%
==========================================
  Files          55       56       +1
  Lines        6308     6574     +266
  Branches     1248     1300      +52
==========================================
+ Hits         4156     4291     +135
- Misses       1938     2060     +122
- Partials      214      223       +9

Impacted Files
Coverage Δ

webapp/graphite/settings.py
75.15% <100%> (+0.96%)
:white_check_mark:

webapp/graphite/util.py
76.92% <100%> (+0.76%)
:white_check_mark:

webapp/graphite/finders/standard.py
67.05% <100%> (-0.39%)
:x:

webapp/graphite/node.py
88.88% <100%> (+1.93%)
:white_check_mark:

webapp/graphite/render/attime.py
100% <100%> (ø)
:white_check_mark:

webapp/graphite/remote_storage.py
34.15% <28.27%> (-4.7%)
:x:

webapp/graphite/render/evaluator.py
61.53% <33.33%> (-10.88%)
:x:

webapp/graphite/render/datalib.py
57.06% <35.65%> (-14.84%)
:x:

webapp/graphite/render/views.py
46.32% <50%> (-0.52%)
:x:

webapp/graphite/intervals.py
37.8% <50%> (+0.3%)
:white_check_mark:

... and 5 more

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 8e61ee2...7c47c90. Read the comment docs.

Wow! Will try to review this beast on the weekend. Or it's too early?

@deniszh definitely still very early, you're welcome to take a look though :)

There is still an issue when using prefetch:
2017-02-10,21:26:45.422 :: Got an exception when fetching data! Try: 1 of 2. Root cause:
Traceback (most recent call last):
  File "/home/dcech/graphite-project/graphite-web-master/webapp/graphite/render/datalib.py", line 170, in fetchData
    seriesList = _fetchData(pathExpr, startTime, endTime, requestContext, seriesList)
  File "/home/dcech/graphite-project/graphite-web-master/webapp/graphite/render/datalib.py", line 240, in _fetchData
    for path, results in result_queue:
  File "/home/dcech/graphite-project/graphite-web-master/webapp/graphite/render/datalib.py", line 211, in result_queue_generator
    for key, fetch in fetches.iteritems():
RuntimeError: dictionary changed size during iteration
Traceback (most recent call last):
  File "/home/dcech/graphite-project/graphite-web-master/webapp/graphite/render/datalib.py", line 170, in fetchData
    seriesList = _fetchData(pathExpr, startTime, endTime, requestContext, seriesList)
  File "/home/dcech/graphite-project/graphite-web-master/webapp/graphite/render/datalib.py", line 240, in _fetchData
    for path, results in result_queue:
  File "/home/dcech/graphite-project/graphite-web-master/webapp/graphite/render/datalib.py", line 211, in result_queue_generator
    for key, fetch in fetches.iteritems():
RuntimeError: dictionary changed size during iteration

@deniszh @obfuscurity I think this is ready for some more testing now 🤞

Looks very nice at the first glance, indeed!
Let's test that! 🥇

This may help dramatically with #1690.  I'm going to build this week and report my findings.

@DanCech do you have an approximate ETA for this ? Just checking if we should start tagging an RC without it or not.

Let me see if I can carve out some time today

@iksaif @deniszh I think this is ready for another round of feedback
@reyjrar did you get a chance to give this a workout?

looks good ! added a few comments, nothing big :)

Who's brave to merge this?!
Let's do that and proceed to 1.0.0rc1 :shipit:


Copilot causes vscode scaling to change

Issue Type: Bug
Just click the code prompt of copilot, and the zoom of vscode will change
VS Code version: Code 1.62.2 (3a6960b964327f0e3882ce18fcebd07ed191b316, 2021-11-11T20:56:38.428Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

@zhang-hongwei I can not reproduce. Can you reproduce using Visual Studio Code Insiders https://code.visualstudio.com/insiders/
fyi @hediet

/duplicate #137512


Use read_file macro in Crystal::Config.version

Followup to #6967. If accepted, needs to be merged after 0.27.1 release.

@Sija do you want to update this with the  || raise "..." idiom?

@bcardiff first I'd like to discuss adding a raising variant and I'll do it after that if there won't be a consensus.

@bcardiff I don't think it's settled since as I wrote in https://github.com/crystal-lang/crystal/pull/6967#issuecomment-439232353 I believe having raising one is still more beneficial - raising outside of read_file will obscure the actual error - which might be other than missing file.

I began work on a variant of read_file that raises here: #7094.

@Sija would you mind pushing a rebase on master?

@bcardiff Sure, done.


option to remove full screen button

I think this should provide an option to remove the full screen button in the top bar

Ah I can't believe that I missed this PR - thanks for opening it! This was way out of date with master, so I rebased and cleaned up the errors, and merged it in here: https://github.com/executablebooks/sphinx-book-theme/pull/328
thanks very much for the contribution!


Add support for IORING_OP_URING_CMD

This PR adds optional support for the following IO_URING features:

SQE128
128-byte submission queue entries were introduced in Linux 5.19. This PR enables 128-byte submission queue entries when the squeue-entry128 feature is enabled.

CQE32
32-byte completion queue entries were introduced in Linux 5.19. This PR enables 32-byte completion queue entries when the cqueue-entry32 feature is enabled.

IORING_OP_URING_CMD
The uring-cmd command introduced in Linux 5.19 allows device specific commands to be passed through io_uring. There are a variety of devices that support and / or requireuring-cmd including, but not limited to, /dev/null, nvme devices, ublk devices.  This PR enables the uring-cmd operation to be used when the uring-cmd feature is enabled.

Testing
The Ubuntu Github Action runner is not running a new enough version of Linux to test the squeue-entry128, cqueue-entry32, or uring-cmd features. These can be tested on a sufficiently recent version of Linux (I used 6.1). Each of the 3 features can be enabled independently.
cargo test --features sqe128
cargo test --features cqe32
cargo test --features uring-cmd
cargo test --features sqe128,uring-cmd

I have been testing this on a Linux 6.1 QEMU VM on the “/dev/ublk-control” device created by the ublk module.
in general, IORING_OP_URING_CMD is used for device specific operations, very similar to ioctl. In addition to ublk, I believe that NVMe devices and /dev/null also support IORING_OP_URING_CMD. I will add some tests using /dev/null since it doesn’t require any specific devices.

I have a VM with an NVMe root partition but I don't know if that can be used for these URING_CMD operations. I'll have to read some more soon.

Do the feature flags mean one can't have both forms of sqe and cqe in the same app? I always thought we would end up with the main driver being the smaller version and when a user wanted the larger version, there would be a secondary ring created but I believe Noah was leaning towards supporting multiple threads, each with their own uring potentially. The uring interface doesn't require all urings from one app be the same size so we probably shouldn't be enforcing something like that either. But @Noah-Kennedy needs to get involved and say what he wants in this feature.

I have a VM with an NVMe root partition but I don't know if that can be used for these URING_CMD operations. I'll have to read some more soon.

Hmm, you I have not personally played around with uring-cmd + NVMe drives, but https://github.com/espoal/uring_examples/blob/main/examples/nvme/src/main.rs seems to be a good example for anyone interested. You could probably get away with some read-only operations to your root partition... but you'd be playing a dangerous game.

Thanks for the link. I'll figure out why it didn't work in the morning. Also, my VM that is supposed to have an NVMe root doesn't show up as NVMe so I admit I may have spoken too soon. Maybe through virtio it doesn't give the kernel an NVMe interface anyway.

Thanks for the link. I'll figure out why it didn't work in the morning. Also, my VM that is supposed to have an NVMe root doesn't show up as NVMe so I admit I may have spoken too soon. Maybe through virtio it doesn't give the kernel an NVMe interface anyway.

If you are using plain QEMU, I have personally found this feature good for emulating NVMe devices.

Why is there a binary file?

Why is there a binary file?

Good catch. I totally missed that when looking through the diffs.

Good catch @Noah-Kennedy! Removed. Perhaps a cargo cross artifact? Struggling to run CI pipeline on macOS 😄

Why is the command behind a feature flag? We have other commands that don't work in older kernels.

Good point, the uring-cmd operation doesn't need its own feature flag... the sqe128 flag should suffice. As for the sqe128 and cqe32 flags... I will take another look at adding them as generic arguments.

Hmm @FrankReh ... so I took a look at making the runtime generic w.r.t SQE and CQE size. The current holdup is that the thread_local CONTEXT variable cannot be made generic. I don't believe that there is any zero-cost way to make the runtime generic. Rather than add a bunch of boxed traits, the best move IMO is probably just to keep SQE and CQE size as a feature flag.

Also, the uring_cmd tests will fail unless run on a sufficiently recent version of Linux. What is the established standard way to include version specific tests if not feature flags?

Also, the uring_cmd tests will fail unless run on a sufficiently recent version of Linux. What is the established standard way to include version specific tests if not feature flags?

I guess the test needs to start by checking the feature or command is supported at runtime and quit the test gracefully if the most rudimentary aspect fails or can't be supported. Maybe leave a comment in the test. I had to put safeguards into the io-uring crate for this but don't recall having to do that for tokio-uring yet. Whatever you come up with is fine.

@Noah-Kennedy Once this passes all CI tests, even if it means the tests were commented out, I'm inclined to approve something like this as it doesn't appear to change what any of us are currently doing and for the those that want the new feature, they can have it immediately - and maybe with a little experience, someone will figure out a good way of having both rings - I know other rust project(s) have already done that.
Also this is a good time to mention that something going in now might be changed for a more general solution down the road. And it's too early to be holding anyone to benchmark results from one release to the next.

Ugh sorry... should have checked all of the CI after making my changes. Should be good now.

Thanks for your great work:)
We are planning to implement a ulbk server in rust, and this definitely help us much!

I'm going to leave this for @Noah-Kennedy to comment on. It's not the direction I was hoping for but also it's not next in line for me to figure out. The project is at an interesting point. The APIs are still in flux and there's no corporate sponsorship of course. Do we put something in that we most likely will change later or do we hold off putting anything about it in until we think we know what we want in the end.

Thanks for your great work:)
We are planning to implement a ulbk server in rust, and this definitely help us much!

Very cool! I actually have an early stage (but working) ublk server implementation that I will probably make public in the next day or so. If this is for an open source project, I would be happy to contribute.

Feel free to close out this MR. It turns out that I need a bit more manual control then tokio-uring provides at the moment. I am going this project moving forward. It essentially takes the guts of tokio-uring and creates a IoUringAsync device that can run directly on top of normal single-threaded Tokio runtime. It could be interesting to integrate something like this into tokio-uring in the future. Let me know what you think.

@thomasbarrett what additional manual control do you need? We're working on providing more manual control of a number of things to users.

Hey @Noah-Kennedy. It would be great if 3rd party libraries had the ability to send manual io-uring commands to the Runtime. This would allow libraries to be built w/ support for io-uring features that are not yet supported by this runtime. I have started using this project for my own work (it essentially just wraps the runtime portion of tokio-uring). I have started using this repository for my own work, but I would love to contribute some of these ideas back into tokio-uring.
https://github.com/thomasbarrett/io-uring-async


Rename downloaded video

Does youtube-viewer have the ability to save the file as something in particular when I download it? Something other than the title of the youtube video?
If not, I'd like to open this as a feature request.

By adding the following entry inside the video_players in configuration file, you can download videos with wget with a custom name, by executing youtube-viewer --player=wget:
   wget => {
	 # download video with wget with a custom name (title-id.format)
         cmd => "wget -O *TITLE*-*ID*.*FORMAT* *VIDEO*",
    },

Now quite what you are asking for, but pretty close. I will try, in the future, to add an easier way for doing this.

That's a great idea. Thanks.

An easier way has been added here: https://github.com/trizen/youtube-viewer/commit/ac0fed3476c3e40021bd9874b90fedaffb905b76
Usage:
youtube-viewer --filename '*FTITLE*-*ID*.*FORMAT*' -d [URL]
For more special tokens, see: youtube-viewer -T


Fix wrong function call, caught by assert on Windows.

Wow, the OSX GL implementation totally doesn't catch that in glGetError(). On the other hand... not really surprising ;) Thanks!


[1.19.2] Apotheosis Bosses Spawn in "no-hostile" biomes

Using version 6.0.1, it appears that bosses spawn on Mushroom Islands. Checked the changelogs for future version and didn't see a change for this particular issue. A biome blacklist in addition to a dimension blacklist would be nice.

Can't reproduce.  Boss spawns utilize the SpecialSpawn event as a trigger (with a natural spawn source), which fires only on successful mob spawns.  Since mobs don't spawn in the mushroom fields by default, this effectively prevents boss spawns.  If you have normal mobs spawning there for some reason, then bosses could spawn, but something else would have to be enabling spawns there first.


Add Article APIs

This change adds the Articles APIs to the Zendesk Go library. Additionally, this pr includes apis for adding categories, sections and permission groups. These other resources were mainly added to allow for writing the end-to-end tests.
Our team is planning on displaying resource articles inside our app, so by adding the article API, we will be able to fetch the article HTML from Zendesk.
I also added a go mod file to this project, it points to the github.com/jredl-va/go-zendesk repo since that is the repository being vendored, if we wanted to contribute back to MEDIGO, this could be changed back to their repo url.
@jredl-va

I have decided to fork this repo into Vendasta and will make the updates in there.


Drag handler causes content to be pushed down

Drag handler enabled: (see the red border)

Drag handler hidden: (see the red border)

Hi @CarterSnich, yes, that seems like a good approach to me. What exact behaviour do you want to achieve, see if I can help you with that

As you can see, some part of the View is not visible if the drag handle is enabled. I want to keep the drag handle and the all parts of the View visible.

So you want the drag handle overlayed on the content?

Try setting the View to 90%

I added this on the source code (inside node_modules: bottom-sheet/src/components/bottomSheet/index.tsx) and it fixes it.
style={[sepStyles?.paddingStyles, {
  borderWidth: 1,
  borderColor : "red",
  flex: 1
}]}

Why it wasn't designed like that? Would it cause some unwanted issues?


Required Pick

Hello community! There is a way to set a property in SlotPicker to make it required in a form? I just want to prevent the user to pick one slot as mandatory.
Thank you in advanced.

Its done manually, here is an example
 const data = useState({
  // other states
  slot: null
 })

const handleSubmit = () => {
 if(data.slot == null)
  // show an error
}

Or if you use formik, which is the best library for handling forms in reactjs (for me at least), it would much easier to handle validation, here is an example on how to set custom value with formik
const formik = useFormik({
 initalValues: { slot: null }
})

// ... some code
<SlotPicker
 onSelectTime={(slot) => formik.setFieldValue("slot", slot)}
/>

check out all the available methods here
formik won't submit the form until all the state values are not empty, you can learn more in their docs

Thank you, I'm using React Hook Form, It's similar to Formik but I will check if I had no issues with that library.
Have a good 2023! :) and thank you for the quick answer!


fix: travis file

Coverage remained the same at 78.716% when pulling 508700ce8e91d0b541b10958011bad8c27b7246e on JamilOmar:master into e852e115899bef2b5e08e1a4b02d90089c6a17e5 on LabShare:master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix the failed UT on update branch

Bug description
The following tests FAILED:
217 - velox_type_test (Failed)
225 - velox_expression_test (Failed)
227 - velox_dwio_common_test (Failed)
262 - velox_dwrf_e2e_filter_test (Failed)
282 - velox_functions_test (Failed)
284 - velox_functions_spark_aggregates_test (Failed)
288 - velox_hive_connector_test (Failed)
292 - velox_exec_test (Failed)
System information
n/a
Relevant logs
No response

@zhouyuan I do see a few other test suites failing
254 - velox_dwio_parquet_reader_test (SEGFAULT)
256 - velox_parquet_e2e_filter_test (SEGFAULT)
258 - velox_dwio_parquet_table_scan_test (Child aborted)

@zhouyuan I do see a few other test suites failing 254 - velox_dwio_parquet_reader_test (SEGFAULT) 256 - velox_parquet_e2e_filter_test (SEGFAULT) 258 - velox_dwio_parquet_table_scan_test (Child aborted)

@surnaik thanks for reporting! indeed the failed the UTs are changing in daily rebase work.

The failures are due to patch & patch "conflicts" - as we merged 29 pending patches, however for each individual patch, the unit tests are actually passing.
In gluten we do make sure all Spark unit tests are passing w/ the daily rebase velox
as Meta team gradually merging the pending patches, all the unit tests should be "fixed" naturally

I'm still discussing with team to check for the best solution. Will keep update here.


Button interaction internal builder misused

Hello,
I recently started using the library and wanted to use the button interaction API but when I wanted to handle the button I got an error:
Unhandled exception:
type 'Null' is not a subtype of type 'String'
#0      new ButtonInteraction.from (package:mineral/src/api/interactions/button_interaction.dart:37:14)
#1      InteractionCreate._executeButtonInteraction (package:mineral/src/internal/websockets/packets/interaction_create.dart:141:61)
<asynchronous suspension>

So to address this issue I think that we should just change this constructor from
 return ButtonInteraction(
      payload['id'],
      payload['application_id'],
      payload['version'],
      payload['type'],
      payload['token'],
      payload['member']?['user']?['id'],
      payload['guild_id'],
      payload['message_id'],
      payload['data']['custom_id'],
      payload['channel_id'],
    );

to
 return ButtonInteraction(
      payload['id'],
      payload['application_id'],
      payload['version'],
      payload['type'],
      payload['token'],
      payload['member']?['user']?['id'],
      payload['guild_id'],
      payload['id'],
      payload['data']['custom_id'],
      payload['channel_id'],
    );

Hello,
We fixed this some time ago, it will be available in a future release. Meanwhile, you can replace payload['message_id'] with payload['message']?['id'].

Thanks for the information !


feat: Drop zone for save files

Found it cumbersome to continually navigate to my save file location and instead made a drop zone so I can keep the folder open and just drop the save file rather than deal with the navigator.

PS: Please target dev branch

The border in the video is just compression weirdness indeed. As the screenshot shows, it looks fine normally.
There does, in fact, seem to be some issue with dropzone not properly accepting .sav files. I'll have to check mime types here. It still uploads the save files and rejects other types but let me see what I can do to fix it.

Switched to a custom validator logic since the accept prop technically worked but would throw warnings on console. The dropzone still throws a "reject" on dragOver because the browser doesn't handle these types of custom files well but once the file is dropped, it will check the file extension and only accept .sav files. All other file types are rejected.

Errors will be displayed under the dropzone. Let me know if you want any styling changes.