NameError: name 'LlamaConfig' is not defined

Hey @MorganCThomas,
Thanks for all the updates especially adding llama2 model there! I tried but got the below error, can you check when you get a chance? I have transformers library installed btw. Thanks!
Traceback (most recent call last):
  File "/home/softwares/acegen-open_latest/scripts/reinvent/reinvent.py", line 118, in main
    run_reinvent(cfg, task)
  File "/home/softwares/acegen-open_latest/scripts/reinvent/reinvent.py", line 175, in run_reinvent
    actor_training, actor_inference = create_actor(vocabulary_size=len(vocabulary))
  File "/home/softwares/mambaforge3/envs/acegen_latest/lib/python3.10/site-packages/acegen/models/llama2.py", line 101, in create_llama2_actor
    config = define_llama2_configuration(
  File "/home/softwares/mambaforge3/envs/acegen_latest/lib/python3.10/site-packages/acegen/models/llama2.py", line 76, in define_llama2_configuration
    config = LlamaConfig()
NameError: name 'LlamaConfig' is not defined

Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.
Monitor may not have opened/closed properly: 'bool' object has no attribute 'pid'
[2024-06-18 10:55:23,500][molscore][ERROR] - Monitor may not have opened/closed properly: 'bool' object has no attribute 'pid'
Server killed
[2024-06-18 10:55:23,500][base][INFO] - Server killed
Calculating summary metrics
Exception ignored in atexit callback: <bound method MolScoreBenchmark._summarize of <molscore.manager.MolScoreBenchmark object at 0x7ff4ce4cfb50>>
Traceback (most recent call last):
  File "/home/softwares/mambaforge3/envs/acegen_latest/lib/python3.10/site-packages/molscore/manager.py", line 1237, in _summarize
    results = self.summarize()
  File "/home/softwares/mambaforge3/envs/acegen_latest/lib/python3.10/site-packages/molscore/manager.py", line 1205, in summarize
    print(f"Skipping summary of {MS.configs['task']} as no results found")
AttributeError: 'MolScore' object has no attribute 'configs'

This seems like it's still an environment issue. I suspect you might need a more recent version of the transformers library. Which version do you have installed? You can check with something like: pip list | grep transformers
What happens when you run the following line with you environment loaded?
python -c "from transformers import LlamaConfig"

I have transformers 4.24.0 and from the python command I got
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: cannot import name 'LlamaConfig' from 'transformers' (/home/softwares/mambaforge3/envs/acegen_latest/lib/python3.10/site-packages/transformers/__init__.py)

I can install the latest one to try, what version are you suggesting?

It seems Llama was integrated in version 4.28.0.
So at least this model, I will make sure to add a more informative message now we know what version is required.
@11carlesnavarro which version of transformers should be used for Llama2?

I tried transforms 4.28.0, I can import LlamaConfig from transformers but still get error
Error executing job with overrides: []
Traceback (most recent call last):
  File "/home/softwares/acegen-open_latest/scripts/reinvent/reinvent.py", line 105, in main
    run_reinvent(cfg, task)
  File "/home/softwares/acegen-open_latest/scripts/reinvent/reinvent.py", line 177, in run_reinvent
    adapt_state_dict(ckpt, actor_inference.state_dict())
  File "/home/softwares/mambaforge3/envs/acegen_latest/lib/python3.10/site-packages/acegen/models/utils.py", line 18, in adapt_state_dict
    raise ValueError(
ValueError: The source and target state dicts don't have the same number of parameters.

Hello!
I had the same problem when testing the code. Try installing the latest version of the transformers lib via
pip install git+https://github.com/huggingface/transformers
Eventually, with the next version release if transformers should be solved, do it like this for now. This should install version  4.42.0.dev0, which works fine

Hey @MorganCThomas,
On a related topic but regarding molscore, seems the syntax for including molscore changed. Another observation is, when using single mode with my own json file as the molscore_task, the speed is much slower than the provided which is benchmark plus the LibINVENT_Exp1 as the molscore_task, is this normal and what mode for molscore_mode are you suggesting to run for my own designs? Thanks!

Yes we changed the syntax to make it cleaner for the latest version of MolScore which now includes curriculum learning.
To answer that I would need to know what your own json file configures, the benchmark mode is just an iterator over single mode so it has no reason to be faster.

I might be confused by the spped between different models e.g, gru vsv llama2 my bad.
Regarding the molscore_mode single, benchmark and curriculum, seems I can only run single with my own json file, with benchmark or curriculum, I need to have something there in presets , maybe here MolScore/molscore/configs/ ?
This is the error i got AssertionError: Preset /home/test.json not found in presets

Yes that is exactly how it is designed. If you are running your own custom objective use single mode. Set molscore_task: to your config path.
Alternatively, if you have multiple objectives you want to run "back to back" like a benchmark, you can pass the directory containing to the parameter custom_benchmark: (Note you should leave molscore_task: null here). And of course switch to molscore_mode: benchmark. Note you could run only one individual task from a benchmark group of tasks by adding it to include: [my_task].
There is no need to put anything in MolScore/molscore/configs.
See the tutorials here

Got it, thanks for explaining!

Improved error messages in #41


Support adding CustomMeters to MeterRegistry

This will parallel the CustomCollector concept in Prometheus.
Proposal:

Add register(MeterReporter) to MetricRegistry
MeterReporter's interface will return a list of metrics that will be converted to the underlying MeterRegistry's metrics. The important thing to keep in mind is that a MeterReporter will be able to report multiple meters.

This feature solution will be a better all around solution to #29 (though I suspect the binder approach will still be useful elsewhere)

I'm going to work this now

Still a ways to go. Need a facility to make it easier to create Measurements each of which generally inherit the name and tags of its parent and add another tag like statistic.


[feat] UX - modify paddings on records list to better use space on mobile

We must keep the attachment file name shortened to not expand out the record item width


Markdown

Add markdown support

Use this? https://pypi.org/project/Markdown/

This proves to be somewhat hard. If you want to try your hand at it, go ahead!


Implement patch-based source-map concat strategy

Ready for review.
Similar to https://github.com/broccolijs/broccoli-concat/pull/102, but this time with support for source-maps.
TODO:

[x] Support concatenation of sourcemaps (assimilate them)
[x] Add caching of sourcemap encoder/decoder (will do in follow up)
[x] Add additional tests

[x] Unit tests for new SourceMapConcat class
[x] Header/Footer multiline scenarios in existing tests
[x] Additional rebuild tests (not needed, covered by unit tests above)

[x] Cleanup code
[x] Add any new documentation
[x] Run benchmarks

Ran some benchmarks:
FILE_COUNT=1000 DEPTH=3 yarn run bench

# new approach
Average time for build over 10 iterations: 426.2ms
Average time for rebuild over 10 iterations: 101.1ms

# old approach
Average time for build over 10 iterations: 353.1ms
Average time for rebuild over 10 iterations: 332.5ms

FILE_COUNT=10 LARGE_FILES=true yarn run bench

# new approach
Average time for build over 10 iterations: 227.4ms
Average time for rebuild over 10 iterations: 196.4ms

# old approach
Average time for build over 10 iterations: 174.9ms
Average time for rebuild over 10 iterations: 163.3ms

So similarly to SimpleConcat, initial build is slightly slower, but rebuilds are faster for most cases. However, when dealing with large files, there is pretty much no improvement. This is due to a lot of time being spent in creating the sourcemaps. I'm going to run this against our app to see what a real world example looks like.

LGTM, let me know once your real world app tests are confirmed working, and we can land + release.

Tested this in our app and Travis-Web. Improvements aren't as good as expected, but the pending change-tracking work will make this much better.
That said, I did catch some bugs during my investigation, so I need to write some additional tests and land them prior to these changes.

Added some additional rebuild tests. This should be good to land whenever, though we should note that integration with Ember-CLI apps is likely to not see a huge win until change-tracking is supported.

@trentmwillis should i merge and release?

Unsure. The last I remember we were waiting for a change-tracking approach for Broccoli to be agreed upon. This current implementation isn't a huge improvement, but it would be able to take advantage of change-tracking if it is implemented.

This seems to have stalled. I'm going to close, but I'll leave the branch intact in case it is decided to pick this up again in the future.


Add Improv WiFi Serial Protocol Support

Long time listener, first time caller! Love the project, wondering if you'd like to include this thing I put together...
I haven't written any docs here since it's an unsolicited PR and I didn't want to do a lot of writing if you decided it wasn't a good fit for the project or want major changes; happy to write some docs if this is something you think is worth including.
The Problem:
I've been playing around with rednblkx's HomeKey-ESP32 project based on HomeSpan, which has a bit of an arduous setup process including installing and trying to troubleshoot a VSCode extension.
Lots of ESP projects like WLED use browser-based installers based on ESP Web Tools, which lets you flash and set up WiFi directly from the browser using web serial. I started a repo for handling building the binary for that project, but then I wanted it to be able to confirm the install and configure WiFi, which requires implementing the Improv Wi-Fi standard to contribute to either HomeSpan itself or to that project. I figured I'd try my luck here to see if you're interested.
The Solution:
I have a pass here at implementing Improv Wi-Fi serial support (sorry, my C++ is... rusty at best, it was never that great in the first place). Improv works by sending serial commands that start with "IMPROV1", so I tapped into the main loop serial command reader to detect those and pass them off to the Improv.cpp file for processing.
These classes handle:

Responding to requests for device status (important when it comes to confirming install, or when you load up an ESP Web Tools instance)
Returning the list of scanned WiFi networks to show in the ESP Web Tools UI
Processing WiFi credentials and connecting

The End Product:
Once you have an image with the Improv stuff installed, you can go to any ESP Web Tools page (e.g. their official demo) and configure/change WiFi.
I made a proof-of-concept HomeSpan ESP Web Tools installer demo (repo here) that you can load up in a browser, flash a binary made using GitHub Actions, and configure WiFi straight from the browser.
That demo currently runs the 05-WorkingLED example, compiled in GitHub Actions and pushed to GitHub pages. It compiles it, merges it the way web tools requires into a single binary, and serves it up for install directly from the browser.

Thing I Got Stuck On:
Logging from within the improv class - LOG2 (etc) aren't defined on that scope, and apparently I can't just ::LOG2() them based on how they're implemented? I can investigate more to get that debug logging in, right now it's just commented out.
Future Idea:
When you finish installing and setting up WiFi, ESP Web Tools allows you to return a URL, which it shows as a button on the ESP Web Tools screen. I was looking at setting this to link to a QR code generated so the user can just scan it to add to HomeKit, but that required touching some more stuff than I wanted to with this PR. But I do think that would make for a pretty slick setup process for projects.

Thanks for this PR.  Though the functionality looks quite interesting, a web-based mechanism for loading sketches is outside the scope of what I intended for the project (at least at this time).  Your use of GitHub actions is really cool - I have never explored that aspect of GitHub.

Yes, thanks for the additional color.  However, I don't think adding this type of functionality is within the intended scope of HomeSpan at this juncture.

Got it, thank you. I'll see if there's another path forward on it externally.
I've seen so many cool HomeSpan-based projects out there gated behind "install Arduino IDE, download this archive, choose your board, flash it, use serial monitor or edit this config file or connect to the AP to set your WiFi credentials..." processes.
Appreciate all your thorough work on this project, just incredibly thorough documentation and all.


Add audio skipping icon to reading screen and remove settings icon

[ ] Remove settings icon from reading screen sidebar
[ ] New icon always appears on the reading screen sidebar
[ ] If reader clicks on this icon, they can then click on any word to begin playing audio from that word
[ ] If the audio was not on, then this icon will turn the audio on when they select the portion of the text they want to start from
[ ] If the audio was already on, then this should just change where the audio is playing once the reader selects the location
[ ] When the reader clicks the icon, the background of the icon should change color to indicate that they are using this feature; if the icon is clicked again while the feature is activated, then it should deactivate this feature and the icon should go back to being white (this allows the reader to cancel trying to skip through the audio if they accidentally click it or just realize they don't want to)

Please add your planning poker estimate with ZenHub @Fattimo


[README] Update README Community Section

Description
The Community section in the README says "Access the Community Drive". There is a new procedure of having this automated based on the individual filling in a community member form
Expected Behavior
Update "Access Community Drive" to "To Access Community Drive, fill Community Member Form"
Screenshots

Resources
Refer to the Contributing guide
Enviroment:

OS: [e.g. Ubuntu]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]
Device: [e.g. laptop, iPhone 8]

When I built the Meshery.io site, I noticed similar language on the bottom of the community page.

// @Ruth-ikegah


Use of-watchdog 0.9.3 to suppress custom health check logging messages

Signed-off-by: Nitishkumar Singh<EMAIL_ADDRESS>
Description

Use of-watchdog 0.9.3 to suppress custom health check logging messages
Motivation and Context

[x] I have raised an issue to propose this change (required)
[x] My issue has received approval from the maintainers or lead with the design/approved label

Closes #51
How Has This Been Tested?

Made changes into template
Created new function with from local templates using faas-cli new python3-flask-test --lang python3-flask
Build function faas-cli build -f ./python3-flask-test.yml
Deploy function faas-cli deploy -f ./python3-flask-test.yml
Invoke function

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

/close: changes incorporated in #56


How to use with laravel Events ?

Hi, thanks for the libaray !
I am to send messages and everything works but I want to ask how to send SSE event when some laravel events happens such as when a user logs in or when a new record of certain model is created.
I tried below code to send SSE event when a user logged in but it doesn't work:
class MyEvent extends SSEEvent
{
    public function check()
    {
        // send login notification
        Event::listen('auth.login', function(){
           return true;
        });

        // default
        return false;
    }

    public function update()
    {
        return 'A new user just logged in.';
    }
}

If I return true directly from check method, I can see the A new user just logged in coming in browser, however I want to be able to do so only when a user logs in on the auth.login event of laravel.
I also tried for my custom model events, they also don't work:
class MyEvent extends SSEEvent
{
    public function check()
    {
        // send message notification
        Message::created(function(){
           return true;
        });

        // default
        return false;
    }

    public function update()
    {
        return 'You have received a new message';
    }
}

Question:
How to make this library work with Laravel Events or how to handle laravel events logic in the check method ?
Thanks

hi @sarfraznawaz2005 its been a long time since I worked on this repo, if you are still interested in the solution lemme know.

@jrganfal I have stopped using it because of no solution and switched to websockets instead. However you can put up a solution/readme of how to do that in laravel for someone else who can use it.
Thanks

@jrganfal I work with it, and I wonder what is the solution?! Thx.

Hi @atefBB thanks for your comment, I'll take a look if it needs an update in the logic of the package and update you here. I haven't maintained this package for a long time but I think I should.

@MAnfal This is a very useful package! Thank you Bro.
جزاك الله خيرا.


Allow ecc_compact construction from a non-compact/compressed key

Returns an error if the bytes generate a key that is not compactable

Codecov Report

Merging #8 (ca6cd7c) into main (f19b4d8) will decrease coverage by 2.56%.
The diff coverage is 35.71%.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   66.20%   63.63%   -2.57%
==========================================
  Files           6        6
  Lines         287      297      +10
==========================================
- Hits          190      189       -1
- Misses         97      108      +11

Impacted Files
Coverage Δ

src/public_key.rs
54.79% <0.00%> (-6.08%)
:arrow_down:

src/ecc_compact/mod.rs
69.51% <50.00%> (-4.18%)
:arrow_down:

Continue to review full report at Codecov.

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


Keyboard shortcuts/settings context menu consistency

In the settings UI editor, right clicking on a setting to open the context menu will give two options: Copy Setting ID and Copy Setting as JSON.
For consistency, it seems like it'd make sense to name those options similarly in the Keyboard Shortcuts UI editor when right clicking on a keyboard rule, changing Copy to Copy Command as JSON and changing Copy Command to Copy Command ID.

Related: #79747 and #79783

I would just change Copy Command to Copy Command ID. Copy commands copies the complete entry as JSON. I do not think Copy as JSON looks good and Copy is  intuitive enough.


RGB plushies will sometimes not shift colors at all and stay black

Seems client side, as I've had plushies work just fine on my end but other people see it as pitch black.

duplicate of #20698


New term request "Familial hemophagocytic lymphohistiocytosis"

Another example:
ORPHA:540 "Familial hemophagocytic lymphohistiocytosis" is currently mapped to MONDO:0009974 "familial hemophagocytic lymphohistiocytosis type 1"
The Orpha:540 term maps to OMIM: 267700 , 603552 , 603553 , 608898 , 613101
ON MONDO:0009974, there is only 267700. Suggest making a new term for ORPHA:540 "Familial hemophagocytic lymphohistiocytosis", then setting the MIM subtypes as children of the orphaNet term.
Originally posted by @kanems in https://github.com/monarch-initiative/mondo/issues/2693#issuecomment-805888503

Orphanet:540 should have it's own term. and I think MIM number for phenotypic series: PS267700 would map to the same concept.

The term 'familial hemophagocytic lymphohistiocytosis type 1' (MONDO:0009974) needs to be split to(i.e obsoleted and replaced by ):

familial hemophagocytic lymphohistiocytosis  - MONDO:0700058 (referring to Orphanet:540)
familial hemophagocytic lymphohistiocytosis type 1 - MONDO:0700059 (OMIM:267700)

Hello @kanems
I'm Sabrina, a new curator who recently joined the Mondo team.
We will address your issue as soon as possible. Thank you for your patience.
If this is a high priority, please let me know and we can prioritize it. Thank you!

Hi @sabrinatoro, I see this was closed recently, but the terms are not on the browser yet... will these new terms go out in the August release?

Hi @kanems
Yes, these terms will be in the upcoming release, at the beginning of August. You will be able to see the terms in your browser as soon as it has updated (I know that OLS takes about a week to update after the release is out).


Update riscvassembler to 1.6.1

Updates com.carlosedp:riscvassembler from 1.6.0 to 1.6.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.carlosedp", artifactId = "riscvassembler" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.carlosedp", artifactId = "riscvassembler" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #43.


Multipleselect: deletion of one element is not possible when select box is higher than browser window

Prerequisites

[x] I have searched for similar issues in both open and closed tickets and cannot find a duplicate
[x] The issue still exists against the latest master branch of Select2
[x] This is not a usage question (Those should be directed to the community)
[x] I have attempted to find the simplest possible steos to reproduce the issue
[ ] I have included a failing test as a pull request (Optional)

Steps to reproduce the issue

generate a long multiselect with all selected items
make window height greater than select box
try to delete any selected item

Example see here: https://jsfiddle.net/vwsg4tn5/
Expected behavior and actual behavior
I was expecting that clicking on any items x would delete that item
Environment
Browsers

[x] Google Chrome
[ ] Mozilla Firefox
[ ] Internet Explorer

Operating System

[ ] Windows
[x] Mac OS X
[ ] Linux
[ ] Mobile

Libraries

jQuery version: 3.1.1
Select2 version: 4.0.3

Isolating the problem

[x] This bug happens on the examples page
[ ] The bug happens consistently across all tested browsers
[x] This bug happens when using Select2 without other pluigns
[x] I can reproduce this bug in a jsfiddle

I was able to reproduce the issue. As noted elsewhere, it happens because we are listening for the mousedown event to handle transitioning the focus and the click event (which is triggered after) to handle deleting options.

@kevin-brown This is exactly what I changed in my proposed PR #4644 - click to mousedown


Training the SDXL text encoder with sdxl_train.py adds a pink / purple color to output images

Hi there,
I'm using the recently fixed SDXL text encoder training support to train the text encoder with some new terms. This works, and I can successfully use the trained terms to generate matching images.
However, I am finding that whenever I train the text encoder, my generated images (in A1111) have a very strong pink / purple color across all of the image. If I disable text encoder training, and only train the UNet, I do not see the pink / purple tint.
The issue is especially noticeable if I only train the text encoder, and not the UNet, with sdxl_train.py. If I also train the UNet, it looks as though the image learning from the UNet "undoes" some of the purple effect. However, it still does not go away completely.
What kind of issue might cause this color tinting, and is there any way to work around it?

Does your dataset have many pink or purple images? I noticed when training that colors will bleed from concepts if the dataset is biased towards images with a color.
Try training with --debiased_estimation_loss on the dev branch. Using --color_aug works too but it's much slower.

With the new fix, I've found that training the text encoder at 1e-8 works well.  I think it's so sensitive compared to the unet because the unet has a huge number of pixels to play with, but there are very few words that go into a prompt.  So any big changes to the text encoder will destroy it.
And the thing is, I don't think the text encoder needs much training at all to learn a new phrase. As long as it's output almost any signal at all, the unet can recognise that signal, and amplify it.  So that's why barely any training is needed.
My tip would be to train the text encoder at 1e-8 for a while, and then switch off text encoder training entirely, and continue to train with the unet only, just like you did.  As long as the text encoder is continuing to be trained, I see tiny distortions appearing in the sample output, and those go away again (assuming you've not overtrained the text encoder badly like you were suggesting you might have) when you train just the unet.


Added the icon for chrome://extensions

I could have just pushed this to master, but wanted to see if there was some reason it hadn't been done yet.

:+1: lgtm


Utilize cache

Hi! I like this project and the whole concept of playground for doing NuGet package development in a "right way" via GitHub.
What's your opinion on using the GitHub Action Cache for NuGet packages used by the project? The cache has pros and cons.
As a maintainer of several projects of variable size I should note that using the cache (if possible) usually reduces build errors that happen due to network failures at the package restore step.
On the other hand, cache requires maintenance on its own, because typical modern projects may easily require several gigabytes of cache to be preserved between builds, and GitHub (to my knowledge) limits the cache size for free repositories to 2 GiB today. And if you exceed this size, then the whole cache experience quickly becomes clunky (slower uploads, slower downloads, cache failures).

It sounds like a good idea for such a small project (not many expected dependencies)! Feel free to make a PR - it will be a good example for library MVP.


Possibility to select category

Currently, you can put feeds in categories and select feeds individually. I think it should be possible to select the whole category in which all recent news from feeds are displayed.

Sure, will look into it.


Replace flutter run tooling for iOS

In https://github.com/flutter/flutter/pull/131865 we had to replace the previous ios-deploy-based code path for flutter run with Xcode automation, since ios-deploy doesn't work with (physical) iOS 17 devices, and we didn't have a better replacement. Once the new communication protocol for device communication has been reverse engineered (as was done originally for earlier versions for ios-deploy) we should restore a codepath that doesn't require the unfortunate UX (and increased fragility since it's automating UI) that we are currently having to use. That will likely require us building and bundling a new tool that's downloaded by flutter.

(/cc @vashworth FYI. Filing for tracking since we'll likely see issues that are going to depend back to this.)

since ios-deploy doesn't work with (physical) iOS 17 devices, and we didn't have a better replacement

Did you check go-ios? I mentioned it to the team here, some time ago.

@stuartmorgan I had already filed a bug for this: https://github.com/flutter/flutter/issues/133465
Feel free to close whichever.

🤦🏻 I'm not sure how I had it in my head that we hadn't filed something yet. Closing as a duplicate.


[PoC] Change the default pagesize on aarch64 from 4k to 16k to support apple hardware

This is an attempt at hotfixing https://github.com/Solo5/solo5/issues/529 for 16k aarch64 hardware as I need to have a working solo5 to test https://github.com/mirage/ocaml-solo5/pull/122 on my work laptop (apple M1 hardware).
I don't think this should be merged as-is as I think sysconf(_SC_PAGESIZE) should be used instead but the goal is to show which parts of the code needs to change for solo5 to work on 16k pagesize hardware.
As of afa7fdf2d09b55acdb955b5783e41f92585e2ee0: only solo5-spt works. solo5-hvt infinit loops for some reason, so something else might be needed.

For solo5-hvt, modification of aarch64_setup_memory_mapping [1] and TCR_TG_FLAGS [2] will be needed at least.
The current implementation creates a page table having mixed 1GB/2MB/4KB blocks. However, a page table having mixed 32MB/16KB blocks needs to be created to support Apple hardware.
(Check [3] for details)
In addition, TG0 and TG1 bits of TCR_EL1 needs to be configured to use the 16KB granule size.
(Check [4] for details)
[1] https://github.com/Solo5/solo5/blob/master/tenders/hvt/hvt_cpu_aarch64.c#L38
[2] https://github.com/Solo5/solo5/blob/master/tenders/hvt/hvt_kvm_aarch64.c#L131
[3] https://developer.arm.com/documentation/den0024/a/The-Memory-Management-Unit/Translation-tables-in-ARMv8-A/Effect-of-granule-sizes-on-translation-tables
[4] https://developer.arm.com/documentation/ddi0595/2020-12/AArch64-Registers/TCR-EL1--Translation-Control-Register--EL1-
16KB page size support requires slightly complicated work. 😢


Stereo label (E) saves in .smi file

Affected Version: Standalone
Steps to Reproduce:

Create any structure (for example, chain) with 2 sprouted stereo bonds and double bond between them. Or open 'aromatic_with_stereolabels.mol' file from templates.
Click 'Calculate CIP' button.
Click 'Save As...' button and save as .smi file. (ketcher.smi)
Clean canvas.
Open ketcher.smi file.

Expected Result:
Structure appears on canvas without stereo labels and stereo bonds.
Actual Result:
Structure appears without stereo bonds but with stereo label near double bond (E).

Cannot reproduce

Release/2.3 The issue is closed as non-reproducible.


Update NuGet package to use SharpDX >= 3

Right now the NuGet package references SharpDX 2.6.3, which is a pain when the current version is 3.0.2.

The SharpDX references in BulletSharp have already been updated to 3.0.2 in the repository. There will be a release of BulletSharp in August with the SharpDX update, maybe even today if all goes to plan. :)

Awesome, thanks!

Here it is: https://github.com/AndresTraks/BulletSharp/releases


Change "login with twitter" authentication model to read only

based on feedback from hardmaru https://twitter.com/hardmaru.

This has been implemented as it makes sense. It was only there as we´re planing on a "tweet this" button down the line. this can be achieved with out intrusive auth models though.  thanks for the feedback!


[BUG] : SIGNUP / LOGIN page link to Privacy Policy and Terms of Service MISSING

this is the current signup/login page

a direct link needs to be provided to Privacy policy and Terms of Service at the bottom , for enhancing the user experience
please assign me this task @Tejashri-Taral under Gssoc-extd24 , Hacktoberfest24 at level 1

@Tejashri-Taral there is no privacy policy page , needs to be made new ?

@Tejashri-Taral  please review PR #546

@Tejashri-Taral please review the PR

#546


Just Emu Tarkov - Installer Compile Failed

-=[ EscapeFromTarkov-Trainer Universal Installer - https://github.com/sailro ]=-

Target EscapeFromTarkov (<IP_ADDRESS>88) in C:\Users\xbox_\Desktop\Folders\- Offline Tarkov\Client.
>> Try #1 Compilation failed for master branch.
>> Try #2 Branch dev-<IP_ADDRESS>88 not found.
Trying to disable faulting feature(s): Hits, AutomaticGun, Aimbot, GameState, WallShoot, LootItems, Health, Quests.
>> Try #3 Compilation failed for master branch.
Unable to compile trainer for version <IP_ADDRESS>88. Please file an issue here :
https://github.com/sailro/EscapeFromTarkov-Trainer/issues

Press a key to exit...

This was a fresh install of JustEmuTarkov.
I have tried running it in Admin, same issue, I have not attempted manual installation yet

Yes, that's because <IP_ADDRESS>88 is a very old EFT version (released in 2020 perhaps). So we have no plan to support it sorry.


Add env variable to curator

Adds an environment variable to a curator script that needs to be set for curator jobs. Will not impact anything else.

Finished testing. Fixes as expected.


Save 21.4MB in published crate by adding an include directive to manifest

…fest
Please let me know if there is any files you did indeed want to add to
the source crate itself and I can make adjustments.
Here is the output of cargo diet with the files that would not be
included anymore in future releases.
➜  pdbtbx git:(master) cargo diet
┌────────────────────────────┬─────────────┐
│ File                       │ Size (Byte) │
├────────────────────────────┼─────────────┤
│ .gitignore                 │          13 │
│ .github/workflows/rust.yml │         940 │
│ tests/waterbox.rs          │        1393 │
│ tests/read_write_pdbs.rs   │        2635 │
│ CODE_OF_CONDUCT.md         │        3347 │
│ example-pdbs/1ubq.pdb      │      105204 │
│ example-pdbs/liquid.pdb    │      113291 │
│ example-pdbs/3b5j.pdb      │      216351 │
│ example-pdbs/1yyf.pdb      │      850905 │
│ example-pdbs/pTLS-6484.pdb │    20127847 │
└────────────────────────────┴─────────────┘
Saved 97% or 21.4 MB in 10 files (of 22.1 MB and 37 files in entire crate)

Thanks! That is indeed a good suggestion.

Thanks! That is indeed a good suggestion.


fix dns resolution issues due to static linking and cgo

Running into issues on ubuntu 14.04 due to name resolution:
dial tcp: lookup xx.redacted.xxx: no such host
This was resolved by using netgo, this has been seen in:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63731 and https://github.com/docker/docker/issues/9449
gcc (Ubuntu 4.8.4-2ubuntu1~14.04.3) 4.8.4

@jdamick if you look at the Dockerfile I set GODEBUG=netdns=go to force the pure Go resolver. Is this not working as expected?
see https://golang.org/pkg/net/#hdr-Name_Resolution
export GODEBUG=netdns=go    # force pure Go resolver
export GODEBUG=netdns=cgo   # force cgo resolver

yeah that does work - missed it in the dockerfile, was just running it from cmd line..

I have the same problem. But when i run a docker image after i build docker image from source , the error message show below:
docker: Error response from daemon: oci runtime error: exec: "/drone": permission denied.

I have the same problem

the fix to this problem is to start the docker container with  -e "GODEBUG=netdns=go"

docker: Error response from daemon: oci runtime error: exec: "/drone": permission denied.

this error can happens when the binary doesn't exist or wasn't correctly compiled. You first need to make sure you compile drone using the proper commands as defined in the .drone.yml to create the binary.
then you need to make sure you are building the image on a linux machine or inside a linux virtual machine. Drone uses CGO which means it needs to be compiled on its target architecture (linux) when creating a linux docker image. If you try to compile drone on osx and then build the Docker image you will see errors.
note that for local testing purposes you can run drone directly on osx or windows. You don't need to run drone itself in a container.

the fix to this problem is to start the docker container with -e "GODEBUG=netdns=go"
Can i add this environment variable GODEBUG=netdns=go to dronerc file?

@stevenjlho the dronerc is really just a docker environment variable file and provides a way to load environment variables from a file instead of with -e
  -e, --env value                   Set environment variables (default [])
      --env-file value              Read in a file of environment variables (default [])

so yes, any environment variables you add to the file loaded by --env-file will get added to the docker container when it is created.

And I am  compile drone from source in Ubuntu, and use command docker build --rm -t drone/drone . to build docker image, then run a docker container according to the document.
REMOTE_DRIVER=gogs
REMOTE_CONFIG=http://gogs.test.amwares.com?open=false
GODEBUG=netdns=go

This is my dronerc content.

@stevenjlho please use the support channel for general usage questions
https://gitter.im/drone/drone


Inserting at the End of the Linked List

Pull Request Template
What have you Changed(must)
Write here what you have changed in the codebase
Issue no.(must) -
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[ ] This issue was assigned to me if not it will be marked as spam.
[ ] My file is in the proper folder
[ ] I am following clean code and Documentation.
[ ] I have added the title and what program will do first in the file
[ ] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

@Ryuk6666 you have to mention the issue number in the PR Body.


[dashboard] fix duplicated chart fetching when dashboard has default filters

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[x] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
When dashboard has default filters, I saw duplicated query requests from those charts applicable for the default filters:

I believe this issue was introduced during building filter indicators or filter scope selector features. The root cause is the initialization value for filter state is wrong, which cause componentDidUpdate trigger one extra query when dashboards having default filters.
TEST PLAN
CI and manual test.
REVIEWERS
@etr2460 @mistercrunch

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8829   +/-   ##
=======================================
  Coverage   65.88%   65.88%
=======================================
  Files         483      483
  Lines       24178    24178
  Branches     2778     2779    +1
=======================================
  Hits        15930    15930
  Misses       8070     8070
  Partials      178      178

Impacted Files
Coverage Δ

...rset/assets/src/dashboard/components/Dashboard.jsx
85% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


AssertionError

I'm running zipformer under aishell dataset with this problem, would like to get an answer thanks!
The specific errors are as follows:
Traceback (most recent call last):
File "zipformer/train.py", line 1350, in
main()
File "zipformer/train.py", line 1343, in main
run(rank=0, world_size=1, args=args)
File "zipformer/train.py", line 1219, in run
train_one_epoch(
File "zipformer/train.py", line 927, in train_one_epoch
scaler.step(optimizer)
File "/home/weiyangjie/anaconda3/envs/icefall/lib/python3.8/site-packages/torch/cuda/amp/grad_scaler.py", line 313, in step
return optimizer.step(*args, **kwargs)
File "/home/weiyangjie/anaconda3/envs/icefall/lib/python3.8/site-packages/torch/optim/optimizer.py", line 140, in wrapper
out = func(*args, **kwargs)
File "/home/weiyangjie/anaconda3/envs/icefall/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "/newdisk0630/wbb/Project/icefall/egs/aishell/ASR/zipformer/optim.py", line 339, in step
clipping_scale = self._get_clipping_scale(group, batches)
File "/newdisk0630/wbb/Project/icefall/egs/aishell/ASR/zipformer/optim.py", line 493, in _get_clipping_scale
self._show_gradient_dominating_parameter(tuples, tot_sumsq)
File "/newdisk0630/wbb/Project/icefall/egs/aishell/ASR/zipformer/optim.py", line 538, in _show_gradient_dominating_parameter
assert torch.isclose(
AssertionError

hi, there's a recent update on the optim.py, have you tried it yet?

hi, there's a recent update on the optim.py, have you tried it yet?

I haven't tried it yet, I'll give it a try thanks.

Are you having parameters in your model that don't have gradient?

Successful. Thanks for the answer.


voice commands not working

i installed kodi on raspberry.. i get my iptv channels working.
installed kodi connect and get it connected with my kodiconnect credentials. i linked the skill with my kodiconnect account.. alexa echo dot discover one new device called kodi.
till now everything is ok.
i tried several video voice commands but not working. only turn on and off are recognized but nothong happened on kodi.
some help please.

What content country do you have set up in your Amazon account?

in aws console i set us virginia in region.
in amazon.com i just changed it to US.
Le sam. 30 juin 2018 à 10:46, Tomas Kislan<EMAIL_ADDRESS>a
écrit :

What content country do you have set up in your Amazon account?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kodi-connect/kodi-alexa-video/issues/16#issuecomment-401530400,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AmyByNWlPM8T0UY_Qy2LC8eVzuz6db1Dks5uB0j2gaJpZM4U8-gA
.

And it still doesn't work with US?

i will try this evening at home and reply back. now i have only channel in
my kodi and you confirmed that tv channel are not supported by your skill.
do you have any video/tv shows source so i can test videos and tv shows
thank you
Le lun. 2 juil. 2018 à 10:50, Tomas Kislan<EMAIL_ADDRESS>a
écrit :

And it still doesn't work with US?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kodi-connect/kodi-alexa-video/issues/16#issuecomment-401740771,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AmyByHpCjkQqQTozul5BJweVb26MrlQcks5uCezXgaJpZM4U8-gA
.

You need to have content available locally

do not work with locally content

And is the local content consumed by the Kodi video library?

Sry about this I have installed it all but when I ask it to play or find a movie  it says nothing found what am I doing wrong help plz

@wiggysyourdaddy Same issue .. you need to have your content in Kodi's video library


SSH fails intermittently

Description:

Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

https://github.com/ballerina-platform/ballerina-performance-cloud/runs/3194801190?check_suite_focus=true


Call handleOnDock explicitly for devices that doesn't listen for dock …

…events
Signed-off-by: Siva Penke<EMAIL_ADDRESS>

I don't think we can call handleOnDock() unconditionally. What if the device is in developer mode?

Looked at it closer. Unfortunately the polling check I believe is still needed since we had problems with the listener not getting events on some devices. Based on your idea, I have come up with https://github.com/liaxim/GearVRf/tree/alwaysdock which should work both for your and the general case.

@sivapenke What do you think the commit in https://github.com/liaxim/GearVRf/tree/alwaysdock?

Do you think this is more appropriate? Though it adds an extra 'if' statement it makes readable and does extra check of isMonoscopic Mode for SAVR. Rest looks good.
if(!isMonoscopicMode) {
if (null != mDockEventReceiver && localConfigurationManager.isDockListenerRequired()) {
//drop-in
} else {
//savr
handleOnDock();
}
}

@sivapenke Yes. Will make the change.

@sivapenke Should we close this?

@liaxim Yes.


Update build scripts to use .NET CLI

Update build scripts to use .NET CLI, run tests against Linux container.  Version is still v1.3.0, will update to v1.3.1 after rebasing on #40

CI has also been updated to run "RunTestsWithNetCore" after running "RunTests" step


added separate named temporary files for Windows compatibility

Fixed #53

Hi @jaant,
Could you please run locally the test suit and check if this fixes the issue? It looks a bit more elegant to me since it does not have to create a separate temporary directory.
Please let me know if there are more problems / reasons why you decided to create a temporary directory on Windows.

Pull Request Test Coverage Report for Build 151

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

Totals

Change from base Build 150:
0.1%

Covered Lines:
560

Relevant Lines:
629

💛  - Coveralls

yup, works now with my app. also test_spurplus passes (the other ones depend on pwd module, which is unix-only):
(.env) D:\Code\spurplus_test\spurplus\tests>git show --summary
commit baffcdeabc7c3265d4641a94d239cdf59383b5e3 (HEAD -> patch)
Author: Marko Ristin<EMAIL_ADDRESS>Date:   Mon Aug 26 14:42:10 2019 +0200

    added an unit test for separate named temporary file

(.env) D:\Code\spurplus_test\spurplus\tests>ls
__init__.py  __pycache__  common.py  test_live.py  test_sftp_live.py  test_spurplus.py

(.env) D:\Code\spurplus_test\spurplus\tests>python test_spurplus.py
........
----------------------------------------------------------------------
Ran 8 tests in 0.003s

OK

note that you might want to use the _temporary_file_deleted_after_cm_exit() in SshShell.get() as well, no?

Hi @jaant ,

note that you might want to use the _temporary_file_deleted_after_cm_exit() in SshShell.get() as well, no?

Thanks a lot for spotting this! Could you please test if c6e9b94 works on your Windows system?

@jaant please apologize, I made a silly mistake; the new commit is e00fa37 -- could you please test only this latest commit on Windows? It passes on my system.

ah, sure, but i’m traveling this week and next without my computer => will do once i get back home.
On 28 Aug 2019, at 04:34, Marko Ristin<EMAIL_ADDRESS>wrote:
@jaant please apologize, I made a silly mistake; the new commit is e00fa37 — could you please test only this latest commit on Windows? It passes on my system.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@jaant thanks, please give me a ping when you come back home and run the tests.

yup, the latest commit seems to work:
(.env) D:\Code\spurplus_test\spurplus\tests>git show --summary
commit e00fa37353e2d6c2fdbd5706126ff4ead0b6b4d3 (HEAD -> patch)
Author: Marko Ristin<EMAIL_ADDRESS>Date:   Tue Aug 27 22:31:55 2019 +0200

    SshShell.get uses _temporary_file_deleted_after_cm_exit

(.env) D:\Code\spurplus_test\spurplus\tests>ls
__init__.py  __pycache__  common.py  test_live.py  test_sftp_live.py  test_spurplus.py

(.env) D:\Code\spurplus_test\spurplus\tests>python test_spurplus.py
........
----------------------------------------------------------------------
Ran 8 tests in 0.004s

OK

@yaant thanks a lot for the tests!
@radam could you please merge, bump a patch version and release?

Wrong person?
On Tue, Sep 10, 2019, 2:40 PM Marko Ristin<EMAIL_ADDRESS>wrote:

@Yaant https://github.com/Yaant thanks a lot for the tests!
@radam https://github.com/radam could you please merge, bump a patch
version and release?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Parquery/spurplus/pull/54?email_source=notifications&email_token=AAOMGKKQ4GOGJ4OODVEVEJDQI7S3DA5CNFSM4IPPE35KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6MDBFI#issuecomment-530067605,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOMGKP3P6EDLJKVJFJXWBTQI7S3DANCNFSM4IPPE35A
.

Wrong person?

@radam yes, wrong address, sorry!

@jaant Thank you for doing the change. I just released a new version

thanks for the great module!
On Wed, Sep 11, 2019 at 15:04 GMT, radomsak<EMAIL_ADDRESS>wrote:

@jaant Thank you for doing the change. I just released a new version
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Parquery/spurplus/pull/54#issuecomment-530423010


Add option INCLUDE=id

I wanted :id column.

@ysksn The following should dump the id column along with all of the other colummns:
SeedDump.dump(User, exclude: [])

Does this not work for you?

I want :id, :name, and :age,
but :created_at, :phone, and :updated_at.
Thanks

So your total set of columns is
[:id, :name, :age, :phone, :created_at, :updated_at]

Correct?
How about SeedDump.dump(User, exclude: [:phone, :created_at, :updated_at])? Does that work?

SeedDump.dump(User, exclude: [:phone, :created_at, :updated_at])

It works, but I want option which explicitly add column(s) :id, :created_at, :updated_at
because they are excluded by default.
A)
SeedDump.dump(User, exclude: [])    # Difficult to know if :id is included
SeedDump.dump(User, include: [:id]) # OK, :id is included

B)
SeedDump.dump(User, exclude: [:phone, :created_at, :updated_at]) # Difficult to know if :id is included
SeedDump.dump(User, exclude: [:phone], include: [:id]) # OK, :id is included


Store name added to admin tab

Store name is added to the tab (So you can have multiple stores and distinguish them on the tabs)
Admin translated on the tab

Nope, "Administrador" is "The person who administrates" but I'm talking about the "Administrator zone" :)


[Feature]:  multi llm support,  codegen  integrated

** Multi LLM Manage,   Support codegen **
support codegen models for SQL & code writting.
Describe the solution you'd like
Implement two adapters according to the multi-model access architecture.  first，implement model adapter
class CodeGenAdapter(BaseLLMAdaper):
    pass

And then, implement chat adapter
class CodeGenChatAdapter(BaseChatAdpter):

    """Model chat adapter for CodeGen"""

    def match(self, model_path: str):
        return "codegen" in model_path

    def get_generate_stream_func(self):
        # TODO
        pass

Describe alternatives you've considered
Codegen and CodeT5 both have better code performance, including SQL capabilities.  We will support and use these.

done.   #658


Ci fixes

This gets the CI closer to working again. One remaining problem is in the miniconda installer https://github.com/conda/conda/issues/10431

I assume the issue is the SkyWater PDK? If we update the third_party/make-env submodule to the latest version that should work again -> https://github.com/SymbiFlow/make-env/pull/31

https://github.com/google/skywater-pdk/pull/408 should fix the problem?

Thanks @mithro it does, and the CI is now passing with the skywater make-env fix and this PR. Thanks!

Pulled and merged (but not yet mirrored to github).


rebuild : applying split tree range isolation

What this PR does / why we need it?
From the stable version, range isolation in splay tree is applied.
By not applying node deletion yet, it is expected that removed
nodes return correct index.
Any background context you want to provide?
What are the relevant tickets?

Fixes #
Checklist

[ ] Added relevant tests or not required
[ ] Didn't break anything

I will pull it again after organizing well.


Feature request: hide tooltip when content is empty

At this moment when content is empty the tooltip will still display. To hide it we need to make additional job of styling or 'if-ing'. I believe it would be nice to just hide it when content is empty.

Good idea. You can make a PR for that :)


Notifications do not pop up Runelite client

Describe the bug
Idle notification only adds message to notification center on windows 10 and sometimes flashes the taskbar. I have the "Request focus on notification" setting turned on as seen in the image below .
Expected behavior
Runelite client should pop up to the foreground upon notification.
Screenshots

Environment (please complete the following information):

OS: Windows 10
RuneLite version: <IP_ADDRESS>
Launcher version: "unknown"

I don't believe idle notifications appear when you are actively interfacing with the screen, only when actually afk when screen is focused

I don't believe notifications pop up when you are actively interacting with the screen, only when "afk" while the screen is focused

We have issue already open for this. Also, the option is just named "Request focus" nothing guarantees that request will go through it is OS-dependant


Add auto quantization

Type of Change
feature
API changed or not: add auto as a new quant_level and it's the default tuning behavior.
Description
detail description
JIRA ticket: ILITV-2608

How has this PR been tested?
Pre-CI and extension
Dependency Change?
None

/Azurepipeline run Model-Test


Thoughts on Python package naming/renaming?

Currently my org builds a number of plain Python packages in our catkin_tools workspace, using a fairly simple catkin_tools_python plugin which recognizes the <build_type>python</build_type> and does the right thing as far as calling through to the setup.py script. However, obviously for this to work we need to generate the appropriate package.xml files, and this means having to mirror Python repos (or even pypi source tarballs) internally, and patch them in. In order to be compatible with the similar translation that happens when pypi packages are released into Debian, part of this involves prefixing the names with python- or python3-. Currently, this process is very naive and doesn't try to utilize rosdep or any external mapping info— the handful of exception cases are just managed as a manual overrides list:
https://github.com/mikepurvis/catkin_tools_python/blob/85f746576ea501b30df991b5c00ab8ae0fb5364a/catkin_tools_python/filters.py#L15-L21
As we contemplate moving to colcon, being able to build native Python packages without patching in extra metadata is obviously a win. However, as currently set up, colcon's package discovery mechanism does not do the prefixing, for example from a checkout of the requests source:
requests {path: requests, type: python, name: requests, dependencies: defaultdict(<class 'set'>, {'build': set(), 'run': {'certifi', 'urllib3', 'idna', 'chardet'}, 'test': {'PySocks', 'pytest-httpbin', 'pytest-cov', 'pytest', 'pytest-mock', 'pytest-xdist'}}), hooks: [], metadata: {'get_python_setup_options': <function PythonPackageIdentification.identify.<locals>.getter at 0x7fa09ade2700>, 'version': '2.24.0'}}

This is a barrier to having a catkin/ament package be dependent on a Python package in a way that is agnostic to whether that Python package is in the workspace as source, or packaged.
For my immediate needs, I've built a small augmentation plugin which does the right thing:
$ colcon list
python3-requests {path: requests, type: python, name: python3-requests, dependencies: defaultdict(<class 'set'>, {'build': set(), 'run': {'python3-urllib3', 'python3-certifi', 'python3-chardet', 'python3-idna'}, 'test': {'python3-pytest-httpbin', 'python3-pytest-mock', 'python3-pytest', 'python3-pysocks', 'python3-pytest-xdist', 'python3-pytest-cov'}}), hooks: [], metadata: {'get_python_setup_options': <function PythonPackageIdentification.identify.<locals>.getter at 0x7fef671dc700>, 'version': '2.24.0'}}
python3-requests        requests        (python)

However, I'm wondering if this is something which colcon could/should deal with natively somehow, given the importance of being able to interoperate across the python and native package namespaces.

I understand the need to generate package.xml files when using catkin_tools but in the case of colcon that shouldn't be necessary. And the dependencies declared in the setup.py / setup.cfg file shouldn't have any kind of prefix (or even rosdep specific names). So I am wondering why you couldn't build the Python packages just as is (without a package.xml file)? This should at least make dependencies between such Python packages work.

colcon's package discovery mechanism does not do the prefixing

That is correct. But I would argue that colcon shouldn't do that since this is a highly platform specific mangling.

This is a barrier to having a catkin/ament package be dependent on a Python package in a way that is agnostic to whether that Python package is in the workspace as source, or packaged.
I'm wondering if this is something which colcon could/should deal with natively somehow, given the importance of being able to interoperate across the python and native package namespaces.

I understand the problem for your use case. The actual source of the problem is that rosdep keys used in package.xml files (which are not ROS package names) are arbitrary.
Rather than letting colcon prepend arbitrary (potentially incorrect) prefixes to the package name I think the problem should be resolved at the source of the inconsistency: the rosdep keys. If rosdep could  provide the actual Python package name for such a key colcon could use that package name for the dependency (instead of using the rosdep key itself). That kind of mapping could also easily be implemented in colcon-ros where it conceptionally belongs.
Since rosdep keys only provide a mapping per platform (and not per package manager) we currently can't assume that every rosdep key for a Python package has a pip section which would state the actual package name. I am not sure if iterating all platforms and checking for pip information would be sufficient for this approach to work?

Yeah, I definitely don't want to have to generate package.xml files for Python packages any more!
You're there, but just to be explicit, the situation is that I have a pile of packages, where some are catkin, with <exec_depend> dependencies on debian/rosdep names like python3-requests, and others are plain Python packages, which identify themselves and their own dependencies by the pypi name (like requests). So I need to be able to call PackageDescriptor::get_recursive_dependencies with the list of all these packages, and have them actually resolve (which they won't do today).
My hack approach is to augment the python packages so they identify themselves by the rosdep names, and dependency resolution works naturally (and it will interoperate gracefully in a transitional world where some still have package.xml files patched in). If I understand your proposal, it's to reverse this and basically augment the catkin/ament package DependencyDescriptor entities to map those names back to the original names for the purposes of dependency resolution.
Overall, this makes sense to me. From a purely pragmatic point of view, my inclination would be to use the rosdep database if available, but also have at least as an option a fallback which just mindlessly strips the python- and python3- prefix from dependencies, on the grounds that in the content of dependency resolution, this will basically never be wrong behaviour, and is much safer than the other approach of mindlessly prefixing. In particular, it would be nice if rosdep (and in particular the rosdep init step) could avoid becoming a hard dependency of colcon-ros.

If I understand your proposal, it's to reverse this and basically augment the catkin/ament package DependencyDescriptor entities to map those names back to the original names for the purposes of dependency resolution.

Correct.

From a purely pragmatic point of view, my inclination would be to use the rosdep database if available, but also have at least as an option a fallback which just mindlessly strips the python- and python3- prefix from dependencies, on the grounds that in the content of dependency resolution, this will basically never be wrong behaviour, and is much safer than the other approach of mindlessly prefixing.

Sounds reasonable to me. I would foresee case where this simply wouldn't work. E.g. a package manifest depending on the rosdep key python(3)-catkin-pkg-modules. In that case I think the only viable way would be to query rosdep and get the package name used for pip.


Training doesn't work

First of all, thank you for sharing the code 👍🏻I'm trying to get into ML for my project. I've installed all dependancies successfully, downloaded the dataset and started the training script with python train.py.
It seems to work fine. I'm getting output like this:
Train for 100 steps
Epoch 1/500
2019-11-20 05:27:23.706954: I tensorflow/core/profiler/lib/profiler_session.cc:184] Profiler session started.
 99/100 [============================>.] - ETA: 2s - loss: 0.2444
Epoch 00001: saving model to /Users/aeyoa/Documents/Sites/tf2-mpe/outputs/models/11200527_hg_lr0.0001.hdf5
100/100 [==============================] - 251s 3s/step - loss: 0.2422

After 500 epochs I'm getting loss of 0.0038
Epoch 500/500
 99/100 [============================>.] - ETA: 1s - loss: 0.0038
Epoch 00500: saving model to /Users/aeyoa/Documents/Sites/tf2-mpe/outputs/models/11200527_hg_lr0.0001.hdf5
100/100 [==============================] - 208s 2s/step - loss: 0.0038

But when I'm checking the progress in Tensorboard the training seems to be broken: heatmaps are changing randomly. Here are a few examples:
After 245 epochs

After 246 epochs — suddenly white

After 306 epochs — still no results

After the fitting I've checked the model with model.predict and still didn't get any reasonable heatmap.
Have you tried the latest code for training? Does is work? Do you have any idea why this isn't working?
Thank you

I'm seeing very similar broken prediction heatmaps after 70 epochs of training with loss 0.0046

@tucan9389 I think I found some errors in the code. Here: https://github.com/tucan9389/tf2-mobile-pose-estimation/blob/5248196af945ec81ae1935fd7fbb482e0e3ccf72/hourglass_model.py#L106-L113
Based on my understanding of how the Hourglass network works, it shouldn't merge and average the intermediate heatmaps. Instead, it should just return:
return hg_out, intermediate_heatmap_layers

And later, where you create the model, here: https://github.com/tucan9389/tf2-mobile-pose-estimation/blob/5248196af945ec81ae1935fd7fbb482e0e3ccf72/hourglass_model.py#L43-L45
The intermediate heatmaps should be set as outputs of the model, so it should look something like this:
        hg_next_stage, l2s = self.build_network(inputs, trainable=trainable)

        self.model = tf.keras.Model(inputs=inputs, outputs=l2s)

But for this to work, the data loader should also repeat the label set for each image STAGE_NUM times. So that each intermediate output would get its own label.
Or, am I missing something?

So if anyone lands here: this repo is broken. I managed to get it working with the ^^^ above changes plus I had to replace the whole hourglass model. I used this implementation as a reference: https://github.com/yuanyuanli85/Stacked_Hourglass_Network_Keras/blob/eddf0ae15715a88d7859847cfff5f5092b260ae1/src/net/hg_blocks.py . It's implemented in Keras so it's easy to convert to TF2.

Thank you, @grin! I've already solved my problem using different approach. But it's interesting to get the clarification!

@grin
Thank you for answering.
Currently, the master branch is not properly working. I'm trying to solve some trouble by making a tmp branch. The code of the tmp branch worked properly, and the conversion to the coreml (or tflite) model is being carried out now.
I will merge the tmp branch to the master soon.


Improve documentation

Including the README.

README was partially improved in https://github.com/ChristoWolf/usb-event/commit/ad5adc038c6e2d95e5f2434f90d0be84c96be1fd.


Question - Http Processor

Documentation for http processor type mentions Performs an HTTP request using a message batch as the request body, and replaces the original message parts with the body of the response.
I want to retain the original message part contents after the http processor to be able to use it in an output. For the output i need both the initial part and the response from the http processor.  Is there a way to do it?
Thank you.

Hey @mallikvala, yeah there's two ways you can do this. If you're processing JSON documents then you can perform the HTTP request and insert the result into the original payload with the process_map processor:
pipeline:
  processors:
    - process_map:
        processors:
          - http: {} # TODO
        postmap:
          http_result: .

The other way is to create a clone of the message (creating a batch of two identical messages), only apply the http processor to the second message and then use split to break the batch into two individual messages:
pipeline:
  processors:
    - select_parts:
        parts: [0,0]
    - process_map:
        parts: [1]
        processors:
          - http: {} # TODO
    - split: {}

Works like a charm @Jeffail. Closing the question


Tracking Issue for Packages as (optional) namespaces

Summary
RFC: #3243
Documentation: https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#my-feature
rustc tracking issue: rust-lang/rust#122349
crates.io tracking issue: rust-lang/crates.io#8292
Allow a parent package to open its API namespace for other packages to participate
Implementation

[x] Unstable flag (#13591)
[x] Allow one :: in the middle of package names, prevent publish (#13591)
[ ] Test cases

Depending on a package whose lib.name puts it in a namespace
Path or git dependency under a registry package namespace
Build scripts, see https://github.com/caeg-industries/cargo/commit/f4305ca65fde7b5d10ebdc7abde2c9e62eb427ef
Package IDs, parsing and generating both "name-only" and "URL"

[ ] Publish support including index file system layout

Unresolved Issues

For an implicit bin in a namespaced package, should do something (e.g. warn, error)  about cross-platform hazard of :: being in the name?

Future Extensions
No response
About tracking issues
Tracking issues are used to record the overall progress of implementation.
They are also used as hubs connecting to other relevant issues, e.g., bugs or open design questions.
A tracking issue is however not meant for large scale discussion, questions, or bug reports about a feature.
Instead, open a dedicated issue for the specific matter and add the relevant feature gate label.

I have created this simple package sunform::package1 with the open-namespaces  feature enabled.
cargo-features = ["open-namespaces"]

[package]
...

sunform-package1.zip
When I initialize a Git repository in the local package, commit files, and run
cargo publish --allow-dirty

I get
    Updating crates.io index
error: failed to open: C:\Users\hydro\Downloads\Repositories\sunform-package1\target\package\.sunform::package1-0.1.0.crate

Caused by:
  A sintaxe do nome do arquivo, do nome do diretório ou do rótulo do volume está incorreta. (os error 123)

So for Windows these don't work yet?

@hydroperfox publish support is not done yet, see this task from the task list

[ ] Publish support including index file system layout

I've remembered of binaries, where the default binary's name is derived from the package's name, in which case it requires translating :: to - when writing the binary.


Timer.resumeAll() not working.

I have a issue with Timer.pauseAll() and Timer.resumeAll().
Specifically, I create 2 timers that print to the screen 'A' and 'B' every 500 milliseconds. When I touch the screen they both pause and when I touch the next time they both resume. But only timer 'B' is resumed.
This problem can be easily solved using a tag or id, but new users can stumble upon it without knowing.
timer_issue.zip

Added this info to Gotchas
https://docs.coronalabs.com/api/library/timer/pauseAll.html


BugOrBroken exception when replacing a value with a substitution

Given the config below, we want to make sure that app-test.component entry doesn't get merged with the app.component entry. To achieve this we use the reset-hack, resetting the app-test.component to a value of a different type and reassigning to the correct value:
test {
    component {
        type = test
        properties {
           url = test
        }
    }
}

app {
   component {
       type = app
       properties {
          host = prod
       }
   }
}

app-prod = ${app} {
    component.value = app-prod
}

app-test = ${app} {
    component = reset, component = ${test.component}
}

loading the above config results in:
om.typesafe.config.ConfigException$BugOrBroken: SimpleConfigObject.replaceChild did not find SimpleConfigObject({"component":"reset","component":${test.component}}) in SimpleConfigObject({"app":{"component":{"properties":{"host":"prod"},"type":"app"}},"app-prod":${app}{"component":{"value":"app-prod"}},"app-test":${app}{"component":"reset","component":${test.component}}})

The config loads fine if no substitution is performed on the app-test.component assignment after reset.

Thanks for the report! It fails here as you say with this test case:

    @Test
    def issue356(): Unit = {
        val s = """
test {
    component {
        type = test
        properties {
           url = test
        }
    }
}

app {
   component {
       type = app
       properties {
          host = prod
       }
   }
}

app-prod = ${app} {
    component.value = app-prod
}

app-test = ${app} {
    component = reset, component = ${test.component}
}
"""
        ConfigFactory.parseString(s).resolve()
    }

I can make it work by doing this, so you may have a workaround available:
tmp = {
    component = reset, component = ${test.component}
}

app-test = ${app} ${tmp}


Add SDK methods to invite application members

Connects-to: https://github.com/balena-io/balena-ui/pull/3625
Depends-on: https://github.com/balena-io/balena-api/pull/2526
HQ: https://github.com/balena-io/balena/issues/2121
Change-type: minor
Signed-off-by: Amit Solanki<EMAIL_ADDRESS>
Contributor checklist

[x] Includes tests
[x] Includes typings
[x] Includes updated documentation
[x] Includes updated build output

@balena-ci rebase


ICM r0.2.1 patch release review

Release PR or issue to review

https://github.com/camaraproject/IdentityAndConsentManagement/pull/202
Review actions
Assign this issue to yourself and follow the below list.
For any review action, review the file(s) in the issue/PR listed above.
Put comments in the above issue or PR if they concern non-changed files/text.
Put a short summary of the main review result here into the review issue.

[x] API definition files (YAML) (check version in info & servers objects)
[x] test file(s) availability
[x] changelog updated
[x] readme updated (enforce correct release number / API version naming)
[x] API readiness checklist(s)

Release actions
Assign this issue to yourself or another RM team member and follow the below list.
When done, tick the box in this issue.

[x] API release tracker available on wiki
[x] Review comments addressed (by Sub Project)
[x] Release PR approved (on behalf of Release Management)
[x] PR merged (by Sub Project codeowner)
[x] Release created within GitHub (by Sub Project codeowner)
[x] Release Tracker updated (with creation date of the release and the release tag link)

Additional comments

@tanjadegroot would you be able to take that?

now yes :-) I reviewed and am OK with proposed PR that includes the review comments resolution.

Approved the release PR, waiting for merge and release creation.

Release is created. Meta-release page update with the new release tag, but kept the date of the M2 release (as the patch has not impact on the APIs, and the actual M2 release was done in August).

@tanjadegroot would check the updated meta-release information and then close the issue?

The release tag 0.4.1 is not visible in the repo as the latest version. Should that not be the case ?
The Fall24 meta-release page on the wiki is thus also not updated.
Should we add 3 lines for "patch", "patch date" and "API impacts" in the table ?
"API impacts" would have values value: rx.y(.z): yes/no ?

Fall24 page is updated - so this issue can be closed. Closed as completed


Missing curly brace in variable is not catched.

Hello,
Even in strict mode, this kind of error is not catched:
  $halcyon_dir = "$halcyon_base}/halcyon"

Shouldn't it fail at parsing time already ?
Cheers,

Does puppet fails this ? I don't think it should fail, but who knows ...

I don't know. Do you think it would resolve the variable all the same ? puppetresources doesn't that's for sure.

Given that \} isn't a valid escape, I would expect this :
$a = 'foo'
$b = "$a{"

to mean that $b is in fact foo{.

You are right. That's what puppetresources is doing. I don't know why I believe it was not interpolate.
I will check that puppet is doing the same and close the issue if it fits.

Confirm that it works as expected:
puppet apply -e '$a="toto" $b="$a}hello" notify {"$b":}'
Notice: toto}hello


Feature: check for existing items (migrating media to Media Hub)

Feature suggestion (although this may be an edge case):
I'm migrating over several thousand images from another app, and wanted to check if an image exists before importing. The following is working, but if others find it useful an existing helper method could be implemented (with support for other file instances, eg. UploadedFile/Base64).
return collect($pictures)
    ->reject(function ($picture) {
        return Media::where('original_file_hash', md5_file($picture))->exists();
    })
    ->map(function ($picture) use ($collectionName) {
        return MediaHub::storeMediaFromUrl($picture, $collectionName);
    })

If it's something of interest, I can submit a PR.

Heya! The file hash logic was added for this exact reason. I hadn't gotten around to fully implementing it yet, as I had a lot of features in mind and had limited time.
However, I did add this now in release 1.1.0 and it worked like a charm when I tested it. :)


Pass setState as props to child component in "Thinking in React" page

Hi, i hear taht pass setState() as props to child component is not correct, but in "Thinking in React page" you did that !!, why ?? correct my if i am wrong.

Thanks.

Working on it

Why is it not correct, any reason ?

i hear that pass setState() as props to child component is not correct

Not sure where you heard this, but it's wrong. See this doc, which recommends this pattern: https://react.dev/learn/sharing-state-between-components


TLSv1.0 connections die: "Client TLS handshake failed - OpenSSL Error([('SSL routines', '', 'no suitable signature algorithm')])"

Problem Description
When making TLSv1.0 connections via the proxy, they die with the following log message:

Client TLS handshake failed. The client may not trust the proxy's certificate for api-server.app.invalid (OpenSSL Error([('SSL routines', '', 'no suitable signature algorithm')]))

Steps to reproduce the behavior:

Launch the proxy using mitmproxy --set tls_version_client_min=UNBOUNDED --set tls_version_server_min=UNBOUNDED
Make a TLSv1.0 connection through the proxy; for example, https_proxy=http://localhost:8080 curl --tls-max 1.0 --tlsv1.0 -k https://example.com
Connection will die with the log message warn: Client TLS handshake failed. The client may not trust the proxy's certificate for example.com (OpenSSL Error([('SSL routines', '', 'no suitable signature algorithm')]))

System Information
Mitmproxy: 9.0.1
Python:    3.10.7
OpenSSL:   OpenSSL 3.0.7 1 Nov 2022
Platform:  macOS-12.6.6-x86_64-i386-64bit

The solution is to add --set ciphers_client=DEFAULT@SECLEVEL=0 as well. This is a somewhat non-obvious workaround, so I'm documenting that here in the hopes that it'll be useful to someone else who searches for this problem. (It may be worth putting this in the documentation somewhere?)
Note that the DEFAULT@SECLEVEL=0 workaround may apply to cURL too - i.e. for the above example, the connection will only succeed on my machine with https_proxy=http://localhost:8080 curl --ciphers DEFAULT@SECLEVEL=0 --tls-max 1.0 --tlsv1.0 -k https://example.com).
To be more specific, this is due to a breaking change in OpenSSL 3.0: the default security level of 1 no longer permits SHA1 and MD5 signatures, which breaks the default configuration of SSLv3, TLS 1.0, TLS 1.1, and DTLS 1.0, as well as TLS 1.2 with no signature extensions specified. Specifying DEFAULT@SECLEVEL=0 is therefore necessary to support older TLS versions.


Show view menu overflows screen

Describe the bug
The view list is too long and overflows the screen. There's also no way to scroll this menu.

What version of the XSK are you using?
0.9.2
Seems to be reproducible only locally. Deployment on Kyma is fine.

To Reproduce
Steps to reproduce the behavior:

Open XSK
Click on Window
Mouse over Show view
See menu overflowing

Expected behavior
List does not overflow. This can be achieved in a few ways:

Group views into categories and drill down each category
Add a scroll bar to the view
Reduce the spacing between each element in the list

Screenshots

Desktop (please complete the following information):

OS: MacOS 11.6
Browser Both Safari and Chrome experience this issue

Additional context
Reproducible only locally with docker run. Deployment on kyma seems fine

@dpanayotov please transfer this issue to: https://github.com/eclipse/dirigible/issues


Let's Encrypt support

The sites create command now allows automatically generating a Let's Encrypt certificate instead of specifying the name of an existing cert. This is done by specifying the --lets-encrypt flag instead of providing the CERT_NAME argument.
The certs create command also allows creating a Let's Encrypt certificate by providing the --lets-encrypt flag instead of the path to a public and private key.
Command examples are updated to reflect these options in the contract files.

LGTM


Fix javax.measure dependency on local maven repo

Some kind of workaround, maybe for JPMS?
https://github.com/unitsofmeasurement/indriya/issues/377
units of measurement
uomImpl = { group = "tech.units", name = "indriya", version.ref = "uomImpl" }
javaxInject = { group = "javax.inject", name = "javax.inject", version.ref = "javaxInject" }

Fixed with PR #44


Test failure in 3.0 provided RC4 implementation

I've added an extra test into the evp_libctx tests here:
https://github.com/jon-oracle/openssl/commit/80b3acf4a18ad1f6b96073fcdd36e0eb064d0b20
The RC4 test fails at ./test/evp_libctx_test.c:365 which I think may be due to the key schedule not being re-initialised with the original key:
    if (!TEST_true(EVP_EncryptInit_ex(ctx, cipher, NULL, key, iv))
        || !TEST_true(EVP_EncryptUpdate(ctx, out1, &out1_len, in, sizeof(in)))
        || !TEST_true(EVP_EncryptInit_ex(ctx, NULL, NULL, key, iv))
        || !TEST_int_eq(EVP_EncryptUpdate(ctx, out2, &out2_len, in, sizeof(in)),
                        ccm ? 0 : 1)
        || !TEST_true(EVP_EncryptInit_ex(ctx, NULL, NULL, NULL, iv))
        || !TEST_int_eq(EVP_EncryptUpdate(ctx, out3, &out3_len, in, sizeof(in)),
                        ccm || siv ? 0 : 1))
        goto err;

The failure looks like this:
( SRCTOP=. \
  BLDTOP=. \
  PERL="/usr/bin/perl" \
  FIPSKEY="f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813" \
  EXE_EXT= \
  /usr/bin/perl ./test/run_tests.pl test_evp_libctx )
30-test_evp_libctx.t .. 2/?
        # Fetching RC4
        #
        # ERROR: (memory) 'out1 == out3' failed @ test/evp_libctx_test.c:378
        # --- out1
        # +++ out3
        # 0000:-e89e43fd42e41ec4 0fd19cca03d3255f
        # 0000:+d273825bf7b145e6 8694aed5f439ec12
        #       ^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^
        #
        # OPENSSL_TEST_RAND_ORDER=1620954814
        not ok 198 - iteration 144

To reproduce, use the commit above and build with legacy support, then run:
make TESTS=test_evp_libctx test

After testing with 1.1.1 this seems to be behaviour for RC4. I've added a special case around the reinit test:
https://github.com/openssl/openssl/pull/15276


Return focus to previous window on hide

Comments on how to do this can be found here https://github.com/zeit/hyperterm/issues/398

try doing this:
const { Menu } = require('electron')
Menu.sendActionToFirstResponder('hide:')

Done. Seems to work as expected, thanks for this suggestion

I put a little check to detect if should focus on internal hyperterm window or an external one.(32ff15862d1e5947925aa2d138268dcc7f89bb22)

Any way this can be done on Windows?


facebook currently 403s

:(

Looks like this is a scope issue, I tried authenticating with ?scope=public_profile,basic_info,read_stream,manage_notifications,read_friendlists,publish_actions,user_relationships,user_relationship_details,user_likes,user_activities,user_website,user_groups,user_events,user_photos,user_friends,user_about_me,user_status,user_tagged_places and it works like it's supposed to.
Of course that is way too many permissions and I'll have to actually read the docs to find out which ones we need :cry:

oh worth mentioning that I changed the API URLs to v2.2 before testing this.

aha, nice sleuthing! i'm guessing read_stream will be enough. we might get more data with more of the scopes, but that could be the only one we need to get something and avoid the 403.

confirmed. i tried it with just read_stream and user_status, and it worked ok.

fixed by upgrading to facebook's graph api v2.2 (see snarfed/bridgy#339)...but i still need to go through the review process to get the read_stream permission. leaving this open until i've done that

for future reference, here's the in progress review:
https://developers.facebook.com/apps/355626134468388/review-status/items-in-review/
and the one for facebook-atom:
https://developers.facebook.com/apps/545216882163210/review-status/items-in-review/

fixed by snarfed/bridgy#339


ARROW-5856: [Python] Linking 3rdparty cython modules against pyarrow fails since 0.14.0

This is just quick draft for running the cython tests in the wheel builds.

@ursabot crossbow package -g wheel

@ursabot crossbow package -g wheel

closing in favor of https://github.com/apache/arrow/commit/690823ce5e863db89d9617818bfe7186d0544cca


great idea

I like the idea. Maybe a good opportunity to make Age accessible to less technically savvy people.
Typo:
mixed-up placeholder text for private and public key
Suggestion:
dark mode support
<link<EMAIL_ADDRESS>rel="stylesheet">

turn off browser spell check

<body spellcheck="false">

missing html title
<title>Age tool</title>

Hey @kernelschmelze , thank you for the comment.
This is the initial idea and release but my plan is to make it more mobile-friendly (use a framework?), encrypt/decrypt files, and add encrypt/decrypt with a password.
When I find some free time, I will work on that.

hey @kernelschmelze , thanks for the suggestions!
I added those changes so we can close this issue. A new release is generated v1.1.0


Search is slow

Suddenly, search (and file name search) is behaving far more slowly than it used to. VSCode search was instant, now, it is just faster for me to go to to the sidebar which i dont want to do.

https://github.com/Microsoft/vscode/issues/55


problem in converting fp32 to fp16

Hi guys,
I'm trying to convert onnx model fp32 to fp16.
but I faced This messages in the function 'remove_unnecessary_cast_node'
"The downstream node of the second cast node should be graph output" (onnxconverter_common/float16.py:557)
I knew That message caused by None of Downstream node
I am trying to use the FP16 model as much as possible.
Is there any other method I can try?
Would using TensorRT instead of ONNX improve the situation?
thank you

Could you attach the model? I think this error is saying that you have a cast node with no downstream node, which might be unexpected

@LeoZDong How can I attach the model? Can I just send the onnx file?
and also As a temporary solution, I solved it by changing from torch to float16.


Add Truncindex for container, sandbox and image

fix #222
Signed-off-by: yanxuean<EMAIL_ADDRESS>

Really bumpy. It is passed finally.

Will probably review this a bit later. Will review other PRs first.

@yanxuean I'll take a look at this today.

@Random-Liu   Tks

Do we add truncindex  for the filter of Stat function?

refer to #293.   ContainerStats

@yanxuean Could you add a simple integration test for this? Just create a single container pod, and get pod/container/image with truncated index.

OK, I will add a intergration test.

@Random-Liu @crosbymichael @mikebrow   Thank  for review


Update action to remove deprecation warnings

Currently, running this action generates the following warnings:
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: dblock/create-a-github-issue

and
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

There's a recently merged PR on the upstream repo (https://github.com/JasonEtco/create-an-issue/pull/137) that I believe fixes them. Would it be possible to include those commits in this fork?

@facutuesca Yes, care to contribute? Thanks.

I setup renovate, check HEAD, lmk what else is needed and I'll cut a release.

@dblock It works perfectly, thanks!


Support new JavaScript features available through CEP

Is your feature request related to a problem? Please describe.
I had a [].map() call in my jsx script which worked just fine when I execute it via VSCode and it's ExtendScript plugin. But running that script via nexrender says that map is not a function.
From googling it seems to be some kind of new version of ExtendScript through the CEP platform? See here.
Could this be supported by nexrender?

I would need some time to research what CEP is, I've heard about it a couple of times, but not entirely sure what is that, and how it works.
Maybe you have some high-level general explanation, to help me ease into it?
Cheers!

I wrote my first script for Adobe that day, noticed that my .map() broke in nexrender and did a Google Search about what might be going on. I don't think I can be of any help! 😢

Interestingly enough, there is an additional layer of wrapping for the user script, that should improve overall scripting experience within after effects. And I was sure map should be supported there as well. Unfortunately, seems like it isn't.
Also, could you please share some minimal version of your code, so that I could see the code we are dealing with?

You can’t use ES6 features in a .jsx file run by the current extendscript engine.
That’s not a something that can be added on  nexrender. Nexrender is just passing a file reference to After Effect. Until Adobe updates it’s scripting engine to a more modern one that’s not gonna happen.
I suggest you close this issue.


Cleanup localized casks.

shipit!

This depends on https://github.com/Homebrew/brew/pull/906, which is still in the works, so this cannot be shipped as of yet.

Homebrew/brew#906 was merged, so I think this pr can merge.

There is a bug in brew cask audit, which would result in this PR going green even if some of the languages failed. So this is on hold until https://github.com/Homebrew/brew/pull/1216 is merged and tagged.


convert theme to use new context; new hook based consumer

fixes: #532

A few more problems:
Dialog:

Dropdown:

Empty State:

Error:

Text?? probably:

Form stuff:

Considering the Form stuff, I'm gonna give it a pause.  There's something basic happening in order to have this many issues.
Checking a few storybooks...

Ok, storybooks not looking great either. :sadpanda:
TextInput storybook:

Dialog, but maybe Text:

Text story, Heading, P, etc:

@danethurber I did another git pull and got a bunch of stuff (dunno how I missed that).  With that down, clean, bootstrap later, I'm seeing storybook and docs site looking great.  Sorry about the many red herring reports.


client, identity: remove unnecessary guards around delete()

From https://golang.org/pkg/builtin/#delete

If the map is nil or there is no such element, delete is a no-op.

Signed-off-by: Tobias Klauser<EMAIL_ADDRESS>

This change is

test-me-please

Coverage increased (+0.006%) to 44.609% when pulling 838e3b8d8e8f38cdbdb810982d2b8d24baa849fa on pr/tklauser/unnecessary-guard-around-delete into c58d749816d943c4f9426f67738013f4e8778463 on master.


Add clipboard feature to export character data

Add clipboard button to export character data over clipboard
Reorganize "Level" to the player stats columns (to save characters on the cell)

exported data comes as a JSON
// Character Data example
{"actorIndex":403,"displayName":"Shimo","characterName":"Io","characterType":"Pl0400","sigils":[{"firstTraitId":3029266248,"firstTraitLevel":15,"secondTraitId":296412661,"secondTraitLevel":15,"sigilId":3803414757,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":9398,"notificationEnum":2},{"firstTraitId":1280871463,"firstTraitLevel":15,"secondTraitId":2289754288,"secondTraitLevel":15,"sigilId":3666949793,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":2374,"notificationEnum":2},{"firstTraitId":3689011061,"firstTraitLevel":15,"secondTraitId":3696775008,"secondTraitLevel":15,"sigilId":2451411160,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":221254,"notificationEnum":2},{"firstTraitId":2368396142,"firstTraitLevel":15,"secondTraitId":3696775008,"secondTraitLevel":15,"sigilId":3996352483,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":221249,"notificationEnum":2},{"firstTraitId":1552297491,"firstTraitLevel":15,"secondTraitId":3696775008,"secondTraitLevel":15,"sigilId":1144547182,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":221245,"notificationEnum":2},{"firstTraitId":1552297491,"firstTraitLevel":15,"secondTraitId":3696775008,"secondTraitLevel":15,"sigilId":1144547182,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":221246,"notificationEnum":2},{"firstTraitId":1470847760,"firstTraitLevel":15,"secondTraitId":612907763,"secondTraitLevel":15,"sigilId":56249821,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":30383,"notificationEnum":2},{"firstTraitId":1470847760,"firstTraitLevel":15,"secondTraitId":2335962636,"secondTraitLevel":15,"sigilId":56249821,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":245581,"notificationEnum":2},{"firstTraitId":1470847760,"firstTraitLevel":15,"secondTraitId":99806428,"secondTraitLevel":15,"sigilId":56249821,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":242218,"notificationEnum":2},{"firstTraitId":2373493147,"firstTraitLevel":15,"secondTraitId":2404396813,"secondTraitLevel":15,"sigilId":196723080,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":230650,"notificationEnum":2},{"firstTraitId":801700863,"firstTraitLevel":15,"secondTraitId":3536380170,"secondTraitLevel":15,"sigilId":3896853593,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":258976,"notificationEnum":2},{"firstTraitId":2215087280,"firstTraitLevel":15,"secondTraitId":831329001,"secondTraitLevel":15,"sigilId":2442488633,"equippedCharacter":1292525763,"sigilLevel":15,"acquisitionCount":74761,"notificationEnum":2}],"isOnline":false,"weaponInfo":{"weaponId":393858400,"starLevel":6,"plusMarks":99,"awakeningLevel":10,"trait1Id":2373493147,"trait1Level":10,"trait2Id":2215087280,"trait2Level":7,"trait3Id":5462430,"trait3Level":5,"wrightstoneId":1756474947,"weaponLevel":150,"weaponHp":99,"weaponAttack":2285},"overmasteryInfo":{"overmasteries":[{"id":1170626407,"flags":512,"value":20},{"id":1136089117,"flags":512,"value":20},{"id":3297926103,"flags":512,"value":1000},{"id":2622837811,"flags":512,"value":20}]},"playerStats":{"level":100,"totalHp":37685,"totalAttack":20094,"stunPower":13,"criticalRate":100,"totalPower":21525}}

Needs some help to update the languages JSON, on README it says to update ui.json but most languages doesnt have an ui.json file

Thanks @shimonami, this works! Don't worry about adding the actual language data for other languages, as long as things use the translation functions then that's good enough.


--- layout: slide title: "Welcome to our second slide!" --- Your text Use the left arrow to go back!

description

description


Rename KeyType and JsonWebKey to avoid conflicts with predefined types

When upgrading API Extractor we noticed that KeyType and JsonWebKey defined in KV Keys conflict with the same names in dom.lib.ts which causes them to be renamed as KeyType_2 and JsonWebKey_2 in the API view.
To avoid this conflict, we can rename them locally but still export the conflicted name as deprecated alias.
Open questions:

What should we rename these to
What should we do with other names such as KnownKeyTypes? Rename them as well?

Created as a result of https://github.com/Azure/azure-sdk-for-js/pull/17702/files#r709589284

I feel like this is, similar to a new lint check, finding a problem that we have today. I would prefer to never shadow a global type in the runtime environments we support if we can avoid it.
That said, it's not hard for consumers to alias our poor names to something unambiguous.

Before taking this up, I would be curious to do a sweep across all our packages and collect other such references to get an idea of the extent of this problem

Just a note that this and #17721 are very closely related - in JS's case there's no serialization really and the only difference between our JsonWebKey and the built-in is that we use keyOps instead of key_ops. Other languages have more differences but for us it might be as simple as deprecating keyOps (but not removing it until the next major) and starting to use key_ops.


add basic lighting with normal line

The original code regards the up of the faces as normal lines, the render result is not good. We can read the vertices' normal lines from the model file and get the fragments' normal lines during rasterization.


LREM: take '0' as remove all equal values

@yinqiwen i've tried UT on rocksdb and all cases passed. However, it fails under leveldb. Any suggestions ? thanks.

https://github.com/Terark/ardb/blob/3525728e80f7bdb7afc8841f81c7d2ca4e3637b5/src/command/t_list.cpp#L613
https://github.com/Terark/ardb/blob/3525728e80f7bdb7afc8841f81c7d2ca4e3637b5/src/command/t_list.cpp#L621
create 2 iteators again which cause memory leak


release-notes-extractor 0.1.0

New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

@godrei @lpusok When should I expect it to appear as a new version in the bitrise step selector?


[DO NOT MERGE] Add script to reveal bug in ConstantPruningModifier

Run:
python bug_constant_pruning.py

The Bug was caught and fixed as a part of #1751 , verified the bug fix works!


[Template] - Requesting vue

Maybe a quickstart for Vue 3 would be a good idea, since we already have Svelte in those options. Only fair!

I'd be willing to make one if you'd like - lmk

Hi @xocolatl-aficionado! 👋 Thanks for raising this.
Go for it! Would love to see this contribution. Happy to review as well once you're done, just tag me.


⚠ SceneTime has degraded performance

In 25f6c25, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2284 ms

Resolved: SceneTime performance has improved in 5cbc180 after 12 minutes.


support  building for arm architecture.

Hi,
Just wanted to know to what platforms rust fastembed supports.

I am assuming everywhere https://github.com/pykeio/ort builds and https://github.com/microsoft/onnxruntime/releases is available.

Ok, I'm trying to build sample application mentioned in the README file for aarch64 but I'm facing error linking cc failed exit status: 1
I used below command to build for aarch64
cargo build --target=aarch64-unknown-linux-gnu

Can you try https://github.com/Anush008/fastembed-rs/issues/32#issuecomment-1980615706?

Is this the same as https://github.com/Anush008/fastembed-rs/issues/32?
I see you are trying a cross-platform build. May I recommend https://github.com/cross-rs/cross?

Sure I'll checkout
https://github.com/cross-rs/cross

Fastembed downloads a tarball
https://parcel.pyke.io/v2/delivery/orts/packages/msort-binary/1.17.0/orts-msort_static-v1.17.0-x86_64-unknown-linux-gnu.tgz
The shared library in the tarball is for x86_64 architecture and i believe that while linking that same the process is failing.

With cross-rs I'm facing similar issue error: linking with `aarch64-linux-gnu-gcc` failed: exit status: 1

After analysis I have mentioned the exact issue in ring repo
https://github.com/briansmith/ring/issues/2000


Allow passing the tests' outs to checkers

For some tasks a checker needs not only the input and the program's output, but also the output from out/. One such task is Przyciski from this OI.
Extending the current format should be easy, but I suggest switching to sio2's checker format at least partially, that is passing in, userout and out filenames as args.
Additionally, imo toster should automatically handle test packages both with and without outs.

Hm, I'm not sure how I feel about passing in files, Toster will probably soon stop creating temporary output files for the tested program and switch entirely to doing all that in RAM as it's faster, this would require writing it back into a file, which would slow it back down. Adding some option to pass in the output and handling packages with and without outs does sound like a good idea though (even though I probably won't implement it in time to be useful for the current OI).

Hmm, I agree with the performance argument.
I considered tmpfs, but it isn't a good enough solution, as it would require superuser privileges.