alternative git URL via SSH in additional to via HTTPS

Cloning via HTTPS is difficult from a server located within China. However cloning via SSH works fine.
It'd be nice for vim-plug to support SSH based cloning when HTTPS doesn't work.

Type:

[ ] Bug
[ ] Enhancement
[x] Feature Request
[ ] Question

OS:

[ ] All/Other
[x] Linux
[ ] OS X
[ ] Windows

Vim:

[ ] Terminal Vim
[ ] GVim
[x] Neovim

Put this before the first Plug statement.
let g:plug_url_format = 'git@github.com:%s.git'

Plug '...'
...

See https://github.com/junegunn/vim-plug#global-options


Getting stuck on muffin new

I tried it a couple of times every time it outputs undefined and just freezes there.

I know this isn't a mongo issue because I can log into mongo just fine

any ideas what could be the problem? I really wanna try this out

Can you post your node version and the version of muffin? :blush: Really weird. Also: It looks like although your database password is "muffin" (which has 6 chars), there are more than 6 chars in the upper image for the DB password...

sorry wrong screencap. the password is "l33tpass"... which works fine.

Muffin is at version 0.2.0

And in case you wanna check the roles

@fa7ad Can you clone the master branch and try it again (please uninstall muffin-cli before that)? Should work fine now!

yeah the master branch worked, I had to manually create the collections though. and apparently i have to recreate users for each database before attempting to authenticate (according to StackOverflow) :frowning:

Also, I entered all the information (username/password) when muffin new asked for them, but now I have what looks like a blank templated .env file... room for improvement? or bug?

Hehe... Yeah, I think the "creating users for each database" is just how MongoDB works.
Regarding the .env file: Yep, I haven't implemented that the command line utility changes the contents of the file on the fly before writing them to the destination folder. So yes, there's still room for improvement and I'm planning to add it soon. However, a PR is also welcomed :blush:
Closing this for now, since the main issue seems to be resolved.

Just opened https://github.com/muffinjs/cli/issues/52

Should be fixed on master. Will release it with the next minor version!

thanks, I was gonna take a look and maybe PR but, looks like you already fixed it :smile:


Automap

MCR’s enhanced environments will improve visual navigation via greater use of visually distinctive landmarks and environmental narrative—e.g. a map section dressed as HAB that leads to a section dressed as SCI which leads to a section dressed as ENG creates a memorable progression—reducing accidental backtracking and reliance on Automap for navigation.
That said, we will still include some form of Automap as a significant Classic gameplay feature (e.g. it shows BOBs’ locations on Rescue levels).
Or Automap will not replicate the original exactly:

The map must be a translucent overlay (as in AO) so it does not block the in-world view.

Whereas the original fixes map orientation and rotates the player arrow, our player arrow always points upwards and the 2D map rotates around it. This eliminates AO’s motion sickness effect where the translucent 2D map slides sideways or backwards as the Player runs forwards.

We will probably use a simplified 2D render of the level (information), omitting nooks and crannies (visual noise) of the new, more detailed 3D architecture. This could be as simple as a “ball and stick” representation of the level’s room+corridor topology, though will probably be a roughly to-scale representation of the general floorplan with unimportant detail simplified or omitted.

We might forego the original 2D map’s “line-of-sight construction”, where walls do not appear unless and until the Player looks at them. We could recreate this effect, e.g. using 2D raycasting in the 2D geometry, but if showing the entire 2D room upon entry does not significantly change gameplay then that is much simpler to implement.

We might incorporate any map fragments and checkpoint locations that the user has previously read in a Terminal.

On Explore maps, include an objective counter so that users know how many of the required locations they have visited and how many remain to be discovered before they can exit the level.

When navigating stacked rooms, rooms on other floors should have higher translucency/different colors/be omitted to improve legibility of the current floor.

Notes
First step is to gather examples of more recent FPS automaps for comparison, determine what is improved in their designs, and decide which of those improvements are appropriate to MCR’s retromodern gameplay and aesthetic, then update this ticket to a requirements list for implementation.
Features which are not applicable to Arrival (e.g. Explore counter) can be deferred until after Demo.

We can recreate these effects, fog of war is just a bunch of masks, and we can do it quite easily if we get some clipping, our map will most likely be 3d internally and displayed as 2d, rendered from another camera anchored to the player


HL-273: More info list items to review frontpage

Description :sparkles:
Add to the list of handling applications, applications with status more information required.
Show styled last modification date column
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #573 (1451bac) into develop (2f2c602) will decrease coverage by 70.77%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop     #573       +/-   ##
============================================
- Coverage    95.67%   24.90%   -70.78%
============================================
  Files          230       96      -134
  Lines         9040     2349     -6691
  Branches         0      599      +599
============================================
- Hits          8649      585     -8064
- Misses         391     1705     +1314
- Partials         0       59       +59

Impacted Files
Coverage Δ

...nies/migrations/0002_unique_company_business_id.py

backend/benefit/common/utils.py

...kend/benefit/calculator/migrations/0001_initial.py

...ared/oidc/migrations/0002_eauthorizationprofile.py

backend/benefit/calculator/api/v1/views.py

backend/benefit/users/api/v1/views.py

backend/benefit/messages/serializers.py

backend/benefit/common/permissions.py

...s/migrations/0026_alter_applicationbatch_status.py

...s/migrations/0012_create_application_number_seq.py

... and 316 more

Continue to review full report at Codecov.

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


Create GroundPinAI.md

📝 Description
Introducing GroundPin AI – your vigilant guardian in the realm of digital security.Empower your digital ecosystem with the unyielding protection of GroundPin AI
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

@abhigarg48
Hey mate:
Please change the branch from the main to the beta please. Thank you!

Okay, Let me look into it.
On Mon, Oct 23, 2023 at 11:51 AM DonQ @.***> wrote:

@abhigarg48 https://github.com/abhigarg48
Hey mate:
Please change the branch from the main to the beta please. Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/illacloud/illa-builder/pull/2939#issuecomment-1774506119,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWGNFCMDCLPTWARXI4S6K3DYAYEFLAVCNFSM6AAAAAA6D5N6PWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZUGUYDMMJRHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Duplication error if duplication element contains an array (with elements inside it)

Link to reproduction
No response
Environment Info
Payload:beta.96

Describe the Bug
In admin dashboard if I am trying to duplicate an item which its self contains an array ,I am getting the following error on UI:
field is invalid: id
I think this happens because the id s of nested elements are mistaken also copied (BUG) while they should be unique.
Reproduction Steps
Just create an array with an embedded list...
Populate the list and then try to duplicate
Adapters and Plugins
No response

Running in the same error with relationship fields I guess.
Tried to replace these (nested) values with their ids in a before duplication hook, but the error occurs before the hook will be called :(


init commit

the configuration is done in invenio.cfg already. invenio-global-search does only provides the functionality, but does not configure it, maybe not totally yet, but the primary intention is it.


modified the datasource roles

This is POC.

modified the datasource roles

crc nifi tasks updated
composite,sat,pat,attendance,telemetry,progresscard,static,teacher_attendance.


5.1.7 Security Functional Requirements SFR FPT_VIV_EXT.1

The Application Notes for SFR FPT_VIV_EXT.1 ‘VMM Isolation from VMs’ says “FPT_VIV_EXT.1.2 addresses several specific mechanisms that must not be permitted to bypass the VMM and invoke privileged code on the Platform.
At a minimum, the TSF should enforce the following:
a. On the x86 platform, a virtual System Management Interrupt (SMI) cannot invoke platform System Management Mode (SMM)
b. An attempt to update virtual firmware or virtual BIOS cannot cause physical platform firmware or physical platform BIOS to be modified
c. An attempt to update virtual firmware or virtual BIOS cannot cause the VMM to be modified”
If these 3 items are required, they should be included in the actual text of the SFR element FPT_VIV_EXT.1.2 as Application Notes are not requirements and not tested.

These are examples to be used as guidance in evaluating a TSS description that is now required in place of the attestation statement.


pipx fails to install

$ pipx install grip
Fatal error from pip prevented installation. Full pip output in file:
    /home/$USER/.local/state/pipx/log/cmd_2024-08-18_19.29.59_pip_errors.log

pip seemed to fail to build package:
    docopt>=0.4.0

Some possibly relevant errors from pip install:
    error: subprocess-exited-with-error

Error installing grip.

$ cat /home/$USER/.local/state/pipx/log/cmd_2024-08-18_19.29.59_pip_errors.log
PIP STDOUT
----------
Collecting grip
  Downloading grip-4.6.2-py3-none-any.whl.metadata (961 bytes)
Collecting docopt>=0.4.0 (from grip)
  Downloading docopt-0.6.2.tar.gz (25 kB)
  Installing build dependencies: started
  Installing build dependencies: finished with status 'error'

PIP STDERR
----------
  error: subprocess-exited-with-error

  × pip subprocess to install build dependencies did not run successfully.
  │ exit code: -11
  ╰─> [0 lines of output]
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× pip subprocess to install build dependencies did not run successfully.
│ exit code: -11
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

Python 3.12.5
pipx 1.6.0
pip 24.1.1 from /usr/lib/python3/dist-packages/pip (python 3.12)
Rhino Linux rolling release based on
OS: Rhino Linux 2024.2~RC (server) x86_64
Kernel: Linux 6.10.4-061004-generic
Shell: bash 5.2.21

Oh I forgot to add, for anyone who hits this error:
apt install python3-docopt
python3-docopt is already the newest version (0.6.2-6).

why use pipx instead of just pip itself? just curious


[Bug] TTL starts counting from the beginning of a request instead of the end

This bug has only recently come to my attention when I started using shorter TTL values, and using a chatty model (QwQ). But it's very easy to reproduce with a very short TTL (like 10 seconds) and a prompt that will take longer to run than the TTL.
Steps to reproduce:

Set TTL to 10 of a sufficiently large model
Ask the model to tell a story. Make sure it generates a story that takes longer than 10 seconds to generate

Expected outcome:

The model finishes generating the story, and the TTL will then start to count, giving you 10 seconds to ask a followup question

Actual outcome:

llama-swap prints a "!!! Unloading model Qwen2.5-Coder-32B-Instruct-Q4_K_S, TTL of 10 reached." message midway through the generation. Thankfully it does not unload the model while it's still generating.
But it does instantly unload the model after the prompt is done, resulting in reloads of the model if you ask a followup question.

Suggested fix:

Consider the model idle when it finishes processing all requests, and start counting towards the TTL when that happens.
Consider the model busy as soon as a new request comes in. The model is considered busy until it finishes. Only then will the TTL start counting.

Thanks for reporting this. I think I know exactly where the issue is. I’ll take a look

this should be fixed in v0.1.5.


Adding note to deploying nebari section

Reference Issues or PRs

What does this implement/fix?
Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[ ] The content is written with plain language (where relevant).
[ ] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ ] All links describe where they link to (for example, check the Nebari website).
[ ] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

Hi @asmijafar20, I left some comments above; let me know if you need any help:smile:

cc: @kcpevey @trallard

HI @kcpevey, I am not sure what still needs to be addressed from your review, sorry, would you mind including any follow-ups into a separate issue if they still need to be addressed?

@viniciusdc it was on another PR. this one looks good to merge.


vcpkg integrate install stopped working on Linux

Describe the bug
vcpkg integrate install fails
Environment

OS: Ubuntu 22.04 LTS
Compiler: gcc-11, clang-14

To Reproduce
Steps to reproduce the behavior:
Currently fails on Travis-CI:
https://app.travis-ci.com/github/acgetchell/CDT-plusplus/builds/257819339
Worked 5 days ago:
https://app.travis-ci.com/github/acgetchell/CDT-plusplus/builds/257657548
Same travis.yml file here:
https://github.com/acgetchell/CDT-plusplus/blob/develop/.travis.yml
Expected behavior
I expected it to work as it's done hundreds of times before.
Failure logs
vcpkg integrate install
write_contents("/home/travis/.vcpkg/vcpkg.path.txt"): No such file or directory

https://app.travis-ci.com/github/acgetchell/CDT-plusplus/jobs/588548845#L927
Additional context
I suspect this broke with the update of vcpkg-tool to 2022-11-10.
https://github.com/microsoft/vcpkg/commit/8430e98114632544ee3d0d6c4a1b933ab3f9dccb

@BillyONeal Could you help take a look this issue? Thanks in advance.

I'm a bit confused: integrate install is to integrate with Visual Studio and there's little reason to run that on Linux.

Okay. Well, it accidentally worked before, and doesn't now, which seems to be fine.

Okay. Well, it accidentally worked before, and doesn't now, which seems to be fine.

I mean, just crashing like it looks is happening isn't great either. I was replying not to say 'go away' but to better understand what you're trying to do.

Thank you! I understand you're all very busy so if this is an unsupported use case I'm fine with that. Maybe add something like "looks like you're trying to run this on Linux which is unsupported" to short-circuit this error.

Thank you! I understand you're all very busy so if this is an unsupported use case I'm fine with that. Maybe add something like "looks like you're trying to run this on Linux which is unsupported" to short-circuit this error.

I just tried this and it didn't repro for me but I added defenses anyway.


Update cicd docker image to 1.6.2

Current version (1.7.1) has some bugs with the terraform version(1.3.0). So, changing the version to 1.6.2 which has terrraform version 1.2.9.

/gcbrun


Add Transfer funds UI

fixes #66
This PR creates the UI for Transfer page.

@haseebzaki-07  solve the conflicts

hey @gupta-ritik please review this PR

hey @gupta-ritik please review this

@gupta-ritik please review

This Has been merged by gssoc team
here - https://github.com/sanjay-kv/Open-source-Practice/pull/1462
@haseebzaki-07 You'll get the points
@gupta-ritik You can close this PR. Dont give labels to it further


Feature Suggestion

There is no need to open all the suggestions for web surfing anymore. A side-view panel will automatically appear when you hover over a search suggestion while searching.
This will save time and you can better and more accurate search results with this new feature.
All the search suggestions will be tailored just for you. Searching will be more effective now.

Please claim the task on your dashboard so that we can grant you points.


[BUG] Lack of documentation!

Please make documentation consistent about all the features you offer in plugin & how to install & debug on various OS that plugin runs!
Thanks!

#368 is in work


Ap-2258 Turn ingress off for uat and staging

AP-2258 Turn ingress off for uat and staging
Link to story
Turn ingress off for uat and staging because we have it configured in the cloud platform environments repo already
This prevents duplicate ingress controllers.
Also, we use a ModSecurity rule on the CloudPlatform ingress that we configured for each environment.
Checklist
Before you ask people to review this PR:

[x] Tests and rubocop should be passing: bundle exec rake
[x] Github should not be reporting conflicts; you should have recently run git rebase master.
[x] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[x] The PR description should say what you changed and why, with a link to the JIRA story.
[x] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

Codecov Report

Merging #2486 (b63ee60) into master (5a3403a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #2486   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          506       506
  Lines         9869      9869
=========================================
  Hits          9869      9869

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 5a3403a...b63ee60. Read the comment docs.


Inference with GPU

This PR adds:

Functionality to pick up device for inference from config.
Config section titled, inference_args in case we want to extend this functionality to other constraints later.
Updated tests to reflect new changes.

Todo:

[x] Check this works on baskerville.

This all looks good my end, nothing more to add.


Question about RoPE code

I found here exist a difference in rope implementation mostly on permutation. Does this difference not affect the final result ？ I'm not quite sure what I'm thinking. Sincerely ask for your advice  : )
Paper version should be:

version in this repo:

The ordering is different. So it wont affect training from scratch but you cant load a model trained with different ordering.

Thanks for your answer : ) Is there exist some reason that the latter implementation was widely used in code instead former one ?

It's easier to code


add citation, update readthedoc config, add 3.12 in CI

Closes #66

https://support.zenodo.org/help/en-gb/24-github-integration/96-how-does-a-citation-cff-file-affect-metadata-of-my-github-release
For our needs, the CITATION.cff file seems more relevant that the .zenodo.json file

@DocOtak I let you add your email address if you wish so

https://docs.readthedocs.io/en/stable/build-customization.html#install-dependencies-with-poetry

The python 3.12 error seems to come from:

the fact the numpy version that supports 3.12 dropped 3.8
jupyterlab needs to be updated

Looks like xarray also dropped 3.8 support, we should do the same.
Some versions policies: SPEC0, NEP 29, Xarray follows NEP 29 for now but I know they discuss switching to SEPC0. If I recall correctly, SPEC0 is faster to drop support for things.
I personally usually keep my python one point release behind, so when 3.12 came out, I migrated my personal work from 3.10 to 3.11.

Seems ready to merge for me.
I dropped python 3.8 and added python 3.12.
I added a test to verify that the minimal version available in the pyproject file are passing the tests (and I discovered that we need pint>=0.20)

Seems ready to merge for me.
I dropped python 3.8 and added python 3.12.
I added a test to verify that the minimal version available in the pyproject file are passing the tests (and I discovered that we need pint>=0.20)

@DocOtak could you review?

Nothing jumped out at me reading the changes with my eyes, which is the classic "looks good to me" type response I suppose. I'll poke at the actual code tomorrow morning (UTC-7). It might be worth experimenting with switching things as much as possible to ruff in a later PR. Absolutely support you being "first author" on anything related to this if the ordering in that citation.cff file matters.

P.S. I did see your PhD advisor at Ocean Sciences, he was quite busy with folks interested in his poster so I didn't actually get the opportunity to introduce myself.

@DocOtak I moved everything to ruff instead of fixing stuff that we would have removed later. Can you review?

Awesome, I think this is ready to go in. I guess there hasn't been a black run for a while as the style is slightly changing. And I don't know if isort as ever been run on the project.
This should close #70 since flake8 will basically be dropped.


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2023-08-21T00:41:30+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
2da78410ee9651b93768bac2a038335ceedb20c6
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
2da78410ee9651b93768bac2a038335ceedb20c6
link
true
/test unit-tests-arm64

integ-basic-arm64_istio
2da78410ee9651b93768bac2a038335ceedb20c6
link
true
/test integ-basic-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
2da78410ee9651b93768bac2a038335ceedb20c6
link
true
/test unit-tests-arm64

integ-basic-arm64_istio
2da78410ee9651b93768bac2a038335ceedb20c6
link
true
/test integ-basic-arm64

integ-pilot-multicluster_istio
2da78410ee9651b93768bac2a038335ceedb20c6
link
true
/test integ-pilot-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In 318cf95, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 10427 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 8297ea1.


New Update

Bro it's been a very long time around 8+ months and there is no update regarding this application. It needs many fixes like subtitles dynamic scaling so that it can fit within the margins, low audio output from stream, slow connections, Buffering, etc.
Please I am waiting for you to get active again and release a very major update soon.

subtitles dynamic scaling

https://github.com/itszechs/DriveStream/issues/98

low audio output from stream

Please check your video files (or speakers perhaps), there is no low audio issue.

slow connections, Buffering, etc

Please check your connection and size of the file you're trying to stream.


📝 Update includes in ./docs/zh/docs/advanced/wsgi.md

Thanks for the effort! :cake:
This was handled in another PR, so I'll close this one now. :coffee:


Update lt.coffee

Hello, @poshus1984! Thank you for contributing. Can you sign our CLA so that I can merge your pull request?

Done.


[feature request] minlen for xwindow module

I want to make the width of the xwindow module fixed, so that it will always keep the same width despite what's written in it. The maxlen parameter is helpful for not making the module too wide, but it would be great if also a minlen parameter would be implemented, or maybe you can achieve what I'm trying to do in some other way. Is this something you can think about or it's a silly request?
By the way, this bar is awesome. For real.

This should already be possible using token specifiers.
See the wiki for more details

You're totally right. No need for anything new.

This should already be possible using token specifiers.
See the wiki for more details

I'm not sure what I'm doing wrong, I followed the Wiki section mentioned above, but ...
label = %percentage:3%
... does not seem to keep my coreuse module (nor RAM and swap modules) from shifting other modules left when the percentage goes over 9% or 99%
I also tried the following snippets (althought the Wiki describes these as "unreleased"), to no avail as well:
label-minlen = 3 label-alignment = right
Am I doing something incorrectly? Also, is there any timeline for release the "label-minlen" feature?
Thanks!


Referencing ElasticSearch query in a json file

Referencing ElasticSearch query  in filter would make life really easy.  I know that Kibana dashboard could be used but I think, this alternative also foce using ElastAlert  .yaml  rule file.  I think it would be much simpler if  we could have some placeholders in our  ElasticSearch query that ElastAlert replace.
We would reference this .json file in our filter section.
I appreciate your feedback on this.

Yelp/elastalert is no longer maintained. Please use jertel/elastalert2.
https://github.com/jertel/elastalert2


Improve markup in README

[ ] Have all code blocks (even one-liners) in triple back-ticks instead of single ones.
[ ] Add a word "shell" or "bash" in all code blocks.

closed by #345


Webpack is too slow build

Hi folks,
I am using Asp.net Core 2.0, Angular 5 (Without Angular Project Template) and Webpack 3.8.1
here is my Webpack code as below:
var path = require('path');
var webpack = require('webpack');

var HtmlWebpackPlugin = require('html-webpack-plugin');
var CopyWebpackPlugin = require('copy-webpack-plugin');
var CleanWebpackPlugin = require('clean-webpack-plugin');

const ROOT = path.resolve(__dirname, '..');

console.log('@@@@@@@@@@@@@@@ USING DEVELOPMENT  @@@@@@@@@@@@@@@');

module.exports = {
    devtool: 'source-map',
    performance: {
        hints: false
    },
    entry: {
        'polyfills': './AngularSrc/polyfills.ts',
        'vendor': './AngularSrc/vendor.ts',
        'app': './AngularSrc/main.ts'
    },

    output: {
        path: ROOT + '/wwwroot/',
        filename: 'dist/[name].bundle.js',
        chunkFilename: 'dist/[id].chunk.js',
        publicPath: '/'
    },

    resolve: {
        extensions: ['.ts', '.js', '.json'],
    },

    devServer: {
        historyApiFallback: true,
        contentBase: path.join(ROOT, '/wwwroot/'),
        watchOptions: {
            aggregateTimeout: 300,
            poll: 1000
        }
    },

    module: {
        rules: [
            {
                test: /\.ts$/,
                use: [
                    'awesome-typescript-loader',
                    'angular-router-loader',
                    'angular2-template-loader',
                    'source-map-loader',
                    'tslint-loader'
                ]
            },
            {

                test: /\.(png|jpg|gif|woff|woff2|ttf|svg|eot)$/,
                use: 'file-loader?name=assets/[name]-[hash:6].[ext]'
            },
            {
                test: /favicon.ico$/,
                loader: 'file-loader?name=/[name].[ext]'
            },
            {
                test: /\.css$/,
                use: [
                    'to-string-loader',
                    'style-loader',
                    'css-loader'
                ]
            },
            {
                test: /\.scss$/,
                include: path.join(ROOT, 'AngularSrc/assets/theme'),
                use: [
                    'style-loader',
                    'css-loader',
                    'sass-loader'
                ]
            },
            {
                test: /\.scss$/,
                exclude: path.join(ROOT, 'AngularSrc/assets/theme'),
                use: [
                    'raw-loader',
                    'sass-loader'
                ]
            },
            {
                test: /\.html$/,
                use: 'raw-loader'
            },
        ],
        exprContextCritical: false
    },
    plugins: [
        function () {
            this.plugin('watch-run',
                function (watching, callback) {
                    console.log('\x1b[33m%s\x1b[0m', `Begin compile at ${(new Date()).toTimeString()}`);
                    callback();
                });
        },

        new webpack.optimize.ModuleConcatenationPlugin(),

        new webpack.optimize.CommonsChunkPlugin({ name: ['vendor', 'polyfills'] }),

        new CleanWebpackPlugin(
            [
                './wwwroot/dist',
                './wwwroot/assets',
            ],
            { root: ROOT }
        ),

        new HtmlWebpackPlugin({
            filename: 'index.html',
            inject: 'body',
            template: 'AngularSrc/index.html'
        }),

        new CopyWebpackPlugin([
            { from: './AngularSrc/assets/layout/images/*.*', to: 'assets/layout/images/', flatten: true },
            { from: './AngularSrc/assets/icons/*.*', to: 'assets/', flatten: true },
            { from: './AngularSrc/assets/js/*.*', to: 'assets/', flatten: true },
        ])
    ]
}

If I Run - development or Watch - development, It will take 30 to 45 sec. Why too long build? Is It possible to build faster?
I am waiting for your response.
Thanks in Advance!

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


Build docker image for OTel Kubernetes operator

What does this pull request do?
This PR introduces the build of a docker image (based on chainguard) that can be used with the opentelemetry kubernetes operator.
This also introduces a change of behavior when loading the instrumentors to cover this use case.
When running in the k8s operator loading of an instrumentor may fail because the environment in which python extensions are built does not match the one from the running container.
There are at least two cases:

different python version
different kind of wheels, e.g. manylinux vs musllinux

To avoid the distro loading to fail catch ImportError here, that is the kind of exception we see when loading shared objects or cython extensions fails.
Related issues
Refs #82

GH secrets have been created now


Fix failure in az aro update when missing azure-credentials object

Which issue this PR addresses:
Fixes ARO-2898

Fixes
What this PR does / why we need it:

Test plan for issue:
Unit tests updated for new functionality.
Manual tests performed on locally running RP.

Create a new cluster and get admin kubeconfig
Save existing secret oc get secret -n kube-system azure-credentials -o yaml > azure-credentials-old.yaml
Delete secret from cluster oc delete secret -n kube-system azure-credentials
Run az aro update on cluster
Get newly created secret oc get secret -n kube-system azure-credentials -o yaml > azure-credentials-new.yaml
Compare secret contents diff --color azure-credentials-old.yaml azure-credentials-new.yaml

Is there any documentation that needs to be updated for this PR?
None

@SudoBrendan It took a lot more doing than I thought it would, but I managed to get it to use Apply() rather than Create()/Update(). There's an outstanding bug with fake.Clientset (kubernetes/client-go#1184) that makes Apply not work properly but I was able to use the same Reactor that's in the bug's discussion.


[ADD] <CSS-Tricks: A Website About Making Websites>

Site URL
https://css-tricks.com/
Description of the above provided site
CSS-Tricks is a leading web development resource offering tutorials and articles focused on CSS, as well as other front-end topics like HTML, JavaScript, and design. Known for its practical solutions and community-driven forum, it's a go-to site for developers of all levels.
Category
Languages
Sub Category
css
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[ ] I am willing to work on this issue (optional)

Good Morning,
I am willing to work on this issue.


Chore/PrivateRouteHome-C2D-16

Problem
C2D-16 on Jira.
Solution
Added a private route to /home
Change Summary:
PrivateRoute home, Route to '/' with new component 'Landingview'
and added some sass changes because the hover jitter was annoying
Dev-Ops
If you added an ENV variable, please check off that you've updated the following

[ ] Key & Sample value to .env & sample.env
[ ] GCP Secret Manager (Both development and production)
[ ] Github Secrets (Added a development and production variable)
[ ] Github Workflows .github/workflows/dev.yml & .github/workflows/prod.yml
[ ] webpack-config.js

Images/Important Notes (optional):

@krishnakpandian Just to confirm, we are doing squash and merge right?

@krishnakpandian Just to confirm, we are doing squash and merge right?

Yeah it's better if we squash and merge, I just switched it over


Mapview with custom styleJson offline

Hello peeps! I have a problem when trying to use a MapView while being offline. I don't understand what I am doing wrong.
I use version 10.0.0-rc.1. I use a MapView with a custom styleJson prop to display some images looking like the following, where CUSTOM_URL is pointing to our own server to fetch these image tiles:
layers: [
      {
        id: 'plan',
        source: 'custom',
        type: 'raster',
      },
    ],
    sources: {
      custom: {
        tileSize: 256,
        tiles: [
         `CUSTOM_URL/tile/{z}/{x}/{y}`
        ],
        type: 'raster',
      },
    },
    version: 1,
  };

Now I want to only make these tiles available for offline usage so the user can still see them and zoom in on them while being offline. I create a pack upfront using  MapboxGL.offlineManager.createPack with options:
{
 bounds: [
      [180, 85.051129],
      [-180, -85.051129],
    ],
    maxZoom: 3,
    minZoom: 0,
    name: id,
    styleURL: customURL2
}

where customURL2 is pointing to an endpoint on our own server again which returns the exact style json as used in the styleJson of the MapView.
When create the pack,  turn off network and show my MapView it gives me errors like this on Android:
Failed to load tile: Unable to resolve host "SOME_URL": No address associated with hostname
and on iOS Mapbox error MapLoad error Failed to load tile. Note that the online flow using this approach shows our tiles correctly and even if I opened the MapView with these tiles before, turn off network and open it again it shows the tiles correctly, so I assume it has to with how I create the pack or something.
I hope someone is able to help me with this!

@AlessandroAries thanks much for the bug report.
Have you tried to use the same styleURL, both for offline a MapVIew?
I'm 99.9% that it's an upstream behaviour so not much we (rnmapbox community) can do about that


Gravity

Is there any way, I can display this toast on the centre of the screen? not by giving hardcoded value like
setTranslationY(100);

Why would you want this? Loadtoast does not block anything like a spinner dialog does so it would be a really bad UX choice...

Closed due to no response.


langchain.graph RDFGraph does not read OWL

System Info
langchain verion: 0.0.237
python version: 3.11.4
Who can help?
@hwchase17
When loading an OWL graph in the following code, an exception occurs that says: "Exception has occurred: KeyError 'op'". It appears that RFGraph class is having trouble with the owl standard specifically.

graph = RdfGraph(
    source_file="example.ttl",
    serialization="ttl",
    standard="owl"
)

I found this issue when using RDFGraph within the GraphSparqlQAChain class
Thank you very much for your help.
Information

[X] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[X] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Steps to reproduce:

Use the following code

from langchain.chains import GraphSparqlQAChain
from langchain.graphs import RdfGraph

graph = RdfGraph(
    source_file="example.ttl",
    serialization="ttl",
    standard="owl"
)

graph.load_schema()

print(graph.get_schema)```

Use the following graph

@prefix : <http://example.org/pump-ontology#> .
@prefix owl: <http://www.w3.org/2002/07/owl#> .
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix xml: <http://www.w3.org/XML/1998/namespace> .
@prefix xsd: <http://www.w3.org/2001/XMLSchema#> .
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .
@base <http://example.org/test-ontology> .

<http://example.org/test-ontology> rdf:type owl:Ontology .

:hasTopping rdf:type owl:ObjectProperty .
:Object rdf:type owl:Class .
:Pizza rdf:type owl:Class ;
       rdfs:subClassOf :Object .
:Topping rdf:type owl:Class ;
         rdfs:subClassOf :Object ,
                         [ rdf:type owl:Restriction ;
                           owl:onProperty :hasTopping ;
                           owl:someValuesFrom :Topping
                         ] .

Expected behavior
For the graph to load and for graph.get_schema to produce the classes (Pizza and Topping) and object property (hasTopping) in the class.

Fixed and merged. Thanks all!


MonacoConverter: Set ActiveParameter in SignatureHelp Info

Hi, sorry for this cold PR. I can make an official issue to link back to this PR if that's the standard. For now, I can describe the PR in more detail as the fix is relatively simple and nothing protocol breaking. Seems like from the contribution guidelines, I may need to bump the minor version?
Context
Was able to successfully plug in lua-language-server via websocket [I had to fork and make some changes to accept monaco-editor and not monaco-core-editor for syntax highlighting] and trying to flesh out some features. I noted that one of the gaps was ActiveParameter in the SignatureHelp was not switching to the next one when I hit the "," trigger character like in VSCode.
I confirmed that the payload from the server is correct: [note that the payload is not sending activeSignature || activeParameter at the top SignatureHelpResponse. Also excuse my signature... you can see how early in the experimental phase I am in for my pet project lol]
{"id":52,"jsonrpc":"2.0","result":{"signatures":[{"activeParameter":1,"documentation":{"kind":"markdown","value":"helloFelix is a quick test. Will this be picked up by Monaco editor?"},"label":"function helloFelix(activation: string|table, queue_name: string, tote_id: string)","parameters":[{"label":[20,44]},{"label":[46,64]},{"label":[66,81]}]}]}}

Solution
Turns out it was simply not setting activeParameter in the converter. 3 LOC.

Well spotted
Indeed, activeParameter in a new parameter in LSP 3.16.0
However, it seems there is other things that need to be updated to fully work with the new LSP version.
I'll make a PR including all the required changes soon

Here it is: https://github.com/TypeFox/monaco-languageclient/pull/253
Thanks you for spotting it :+1:

The other MR has been merged and will be published soon
If everything is ok for you, you can close this one
Thank you again for your help!

@CGNonofr  No problem! I am actively developing for my own needs too so it's no biggie : ) Thanks so much for addressing this so quickly and making more fixes. Closing this PR.


::clearnpcs command is broken

If you try to do ::clearnpcs it does remove the npcs but it gives the following error:
java.lang.IllegalArgumentException: Entity (Npc{id=1, name=Man}) belongs in (Region{coordinates=RegionCoordinates{x=381, y=382}}) but does not exist.
	at org.apollo.game.model.area.Region.removeEntity(org/apollo/game/model/area/Region.java:286)
	at org.apollo.game.model.World.unregister(org/apollo/game/model/World.java:315)
	at java.lang.reflect.Method.invoke(java/lang/reflect/Method.java:498)
	at RUBY.block in (root)(C:\Users\Jess3\Downloads\apollo-master\apollo-master\.\data\plugins\cmd\npc\spawn.rb:66)
	at RUBY.each(uri:classloader:/jruby/java/java_ext/java.lang.rb:12)
	at RUBY.block in (root)(C:\Users\Jess3\Downloads\apollo-master\apollo-master\.\data\plugins\cmd\npc\spawn.rb:66)
	at org.jruby.RubyProc.call(org/jruby/RubyProc.java:308)
	at RUBY.execute(C:\Users\Jess3\Downloads\apollo-master\apollo-master\.\data\plugins\bootstrap.rb:52)
	at org.apollo.game.command.CommandListener.executePrivileged(org/apollo/game/command/CommandListener.java:51)
	at org.apollo.game.command.CommandDispatcher.dispatch(org/apollo/game/command/CommandDispatcher.java:39)
	at org.apollo.game.message.handler.CommandMessageHandler.handle(org/apollo/game/message/handler/CommandMessageHandler.java:34)
	at org.apollo.game.message.handler.CommandMessageHandler.handle(org/apollo/game/message/handler/CommandMessageHandler.java:14)
	at org.apollo.game.message.handler.MessageHandlerChain.notify(org/apollo/game/message/handler/MessageHandlerChain.java:57)
	at org.apollo.game.message.handler.MessageHandlerChainSet.notify(org/apollo/game/message/handler/MessageHandlerChainSet.java:45)
	at org.apollo.game.session.GameSession.handlePendingMessages(org/apollo/game/session/GameSession.java:96)
	at org.apollo.game.service.GameService.pulse(org/apollo/game/service/GameService.java:162)
	at org.apollo.game.GamePulseHandler.run(org/apollo/game/GamePulseHandler.java:37)
	at java.util.concurrent.Executors$RunnableAdapter.call(java/util/concurrent/Executors.java:511)
	at java.util.concurrent.FutureTask.runAndReset(java/util/concurrent/FutureTask.java:308)
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$301(java/util/concurrent/ScheduledThreadPoolExecutor.java:180)
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(java/util/concurrent/ScheduledThreadPoolExecutor.java:294)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(java/util/concurrent/ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(java/util/concurrent/ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(java/lang/Thread.java:745)

This is potentially very bad... cc @ryleykimmel

https://github.com/apollo-rsps/apollo/blob/master/game/src/main/org/apollo/game/model/area/Region.java#L283
That set contains the npc being removed, so the npc has been registered correctly in the region, however if you do
local.contains(entity)
it returns false, so i'm guessing this issue is related to equals() / hashcode()

@ryleykimmel @Major- possibly have this fixed locally. Wouldn't look to much into it 'til I can confirm.

That sounds likely @thispixel; not sure if simply removing it from the region before the repository is adequate... seems like quite a fragile fix

@Major- not really a fragile fix, why should any Mob be set inactive (index = -1) before pre-unregister tasks have completed?
Also letting MobRepository#remove return a boolean is stupid, there really isn't a way to recover from it or do something else. Throwing an exception is likely the better option as something is clearly wrong if the Mob index does not match the expected position in the repository.

It's fragile because reordering two seemingly-independent lines of code will break the behaviour.

@Major- close this please

@ryleykimmel if u do the command with this fix you can't walk and it says
WARNING: Too many messages in queue for game session, dropping...
Feb 05, 2016 5:37:56 PM org.apollo.game.session.GameSession messageReceived
WARNING: Too many messages in queue for game session, dropping...

Curious to why we queue inbound messages anyway, why not handle them instantly? @Major- @garyttierney

Passing the decoded result (i.e. XMessage) into the game thread requries locking; decoding them all at once means we only have to acquire the lock once, instead of every time we get a message.

There should be a better way to do that, dropping legitimate messages after 10 is strange

Also looked into this, we can move that from the game thread and process them in the GameSession alone instantly.

10 is left over from the original release, so I think we can definitely bump this up (20 or 25 should be fine).


Create canary that skips lock file

Today, I was unable to use a fresh install of @aws-amplify/backend-cli because of an issue with a dependency:
➜  gen2-monorepo-demo git:(main) npx amplify --version
node:internal/modules/cjs/loader:1077
  const err = new Error(message);
              ^

Error: Cannot find module 'aws-amplify'
Require stack:
-<EMAIL_ADDRESS>    at Module._resolveFilename (node:internal/modules/cjs/loader:1077:15)
    at Module._load (node:internal/modules/cjs/loader:922:27)
    at Module.require (node:internal/modules/cjs/loader:1143:19)
    at require (node:internal/modules/cjs/helpers:121:18)
    at Object.<anonymous> (/Users/xss/repo/gen2-monorepo-demo/node_modules/@aws-amplify/codegen-ui-react/dist/lib/utils-file-functions/hooks/useNavigateAction.js:39:23)
    at Module._compile (node:internal/modules/cjs/loader:1256:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1310:10)
    at Module.load (node:internal/modules/cjs/loader:1119:32)
    at Module._load (node:internal/modules/cjs/loader:960:12)
    at Module.require (node:internal/modules/cjs/loader:1143:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
<EMAIL_ADDRESS>  ]
}

We should implement a canary that skips the lock file to ensure that fresh installs of the CLI are usable. It should run regularly so that we can catch this issue quickly and resolve it, since it means a primary production use case is not available.

added workflow https://github.com/aws-amplify/amplify-backend/commits/main/.github/workflows/canary_checks.yml


Graphql error on landing

on landing beta, might come from lastEvent query?

yes, it is the same error with #4172

yes, it is the same error with #4172

Merged


Make primary attributes at least recommended

Related Issue:
Fixes #664
Description of changes:
This PR makes two changes:

Updates the metaschema to enforce that attributes labeled with group: primary may not be labeled with requirement: optional
Updates existing violations of that new rule so that all existing primary / optional attributes are made primary / recommended

I agree with this one too - I think there was an issue to this effect some time ago.


please convert fast hunyuan 6 step model to gguf version

please convert fast hunyuan 6 step model to gguf version..
https://huggingface.co/FastVideo/FastHunyuan
thanks

yes ping me wen done


Adding augmentation from albumentation

Hello
@yhenon
I wanted to add more augmentation and was considering adding augmentations from albumentation library.But can this be added without any severe modifications because i was looking at this function
class Resizer(object):
    """Convert ndarrays in sample to Tensors."""

    def __call__(self, sample, min_side=608, max_side=1024):
        image, annots = sample['img'], sample['annot']

        rows, cols, cns = image.shape

        smallest_side = min(rows, cols)

        # rescale the image so the smallest side is min_side
        scale = min_side / smallest_side

        # check if the largest side is now greater than max_side, which can happen
        # when images have a large aspect ratio
        largest_side = max(rows, cols)

        if largest_side * scale > max_side:
            scale = max_side / largest_side

        # resize the image with the computed scale
        image = skimage.transform.resize(image, (int(round(rows*scale)), int(round((cols*scale)))))
        rows, cols, cns = image.shape

        pad_w = 32 - rows%32
        pad_h = 32 - cols%32

        new_image = np.zeros((rows + pad_w, cols + pad_h, cns)).astype(np.float32)
        new_image[:rows, :cols, :] = image.astype(np.float32)

        annots[:, :4] *= scale

        return {'img': torch.from_numpy(new_image), 'annot': torch.from_numpy(annots), 'scale': scale}

so here the annotations are being rescaled.So would this be an issue in using albumentation out of the box?
Thanks in advance.

it depends which transforms you use. Transforms that modify the image shape (such as zoom, rotate,...) need to have the bounding boxes modified alongside. Transforms that do not modify the image shape (like brightness, gaussian noise) can be safely applied with no changes to the bboxes :)


html label cannot render the online image

I tried to render
digraph {
    //splines = ortho;
    //rankdir="LR";
    node [shape = box, color=blue];
    edge [dir = none];

I0000 [label=<

清
袁
1122

>]
}

in http://viz-js.com/  . It doesn't work unless I remove the 'img' element. But in offline graphviz, it works. Anything I missing? or this is a bug?

While you can use the image attribute with Viz.js (even HTTP URLs), viz-js.com doesn't support this, since it currently has no way of loading images to determine their dimensions.  tags should work, but I haven't tested them.
When using images with Viz.js you'll need to specify the dimensions of the image:
Viz("digraph { ... }", {
  images: [
    { href: "http://example.com/test.png", width: "400px", height: "300px" }
  ]
});

I just tried. It still doesn't work in http://viz-js.com even I add dimensions to my img element. I really hope that it can work properly with

Sorry, viz-js.com doesn't support that. And I believe Graphviz doesn't support width and height attribute on  elements, but I could be wrong.
Images are only supported using the images option when using Viz.js as a library.
It would be nice to be able to use images in the web UI, so I've added an issue for that: #126


Raise error if advisory info not found

Right now we're using these library funcs in prepare_release.py in aos-cd-jobs
https://github.com/openshift-eng/aos-cd-jobs/blob/ca376b5909e85a544db3d6810196e24b29dc8d91/pyartcd/pyartcd/pipelines/prepare_release.py#L640
Instead of having the caller figure out when something is wrong,
raise error here.

/lgtm


Incorrect button inputs on GC using Xbox One S controller

I am using an Xbox One S controller (on gamecube using universal fw 1.0) and some of the button inputs are mapped incorrectly even when using the Default Gamepad Only preset.
Here is a list of buttons on the Xbox One S controller and how they are registering in a controller test application:
Xbox LB = GC Y
Xbox Menu and Share = GC Z
Xbox X = nothing
Xbox B = GC X
Xbox Y = GC B
Neither Bumper register as Z as they should, and there is no start button. (You'd be surprised how many games require you to press start to begin playing.) Both sticks register correctly, the DPAD registers correctly, and the triggers register correctly.
Output config mode is set to Gamepad, though behavior is the same in GamepadAlt mode.

Is that specific to v1.0, is v0.19.1 ok?
What is FW version of Xbox ctrl (you can check in the xbox accessories win 10 app)

It's currently on 3.1.1221.0
I'll update it and see if that changes anything.

I have not tried 0.19.1 blueretro FW, this is my first time using it.

Updating the controller firmware fixed the issue. Thanks!

Awesome, thanks for reporting original FW version, that's good information!
Btw for gamecube you probably want to use the gamecube merge trigger preset, to emulate the GC triggers end button.

Good call! I'll give that a shot. Thanks for the late night support!


re-add view.matrix.org

we seem to have lost view.matrix.org as a viewing option from matrix.to

https://github.com/matrix-org/matrix.to/issues/136


ui - edges are now editable

Same as: https://github.com/arangodb/arangodb/pull/12039

Thanks for spotting vadim. There it needs do be disabled. Will fix


Do not parse nil as a boolean for the accounts vat location valid

The parse blows up when parsing a VatLocationValid field of type nil. This will avoid trying to parse in such a case and allow that tri-state field to exist.
cc/ @bhelx
tests:

with VAT location validation turned on and an account that is not in the EU
The following should not blow up and return False:

var account = Accounts.get('<account_code>');
Console.WriteLine(account.VatLocationValid);

with an account in the EU (that has been validated)
the above snippet should reflect the validation of that account.

:+1:


gap in vertex numbering because of issue in splitVertices

This example reveals that when pr[1:2,1:2] is processed in splitVertices(), we have an extra skip in the vertex numbering. Ultimately, this results in numVertices being too large and graphID_2_declID having an extra element (and perhaps other effects on maps -- haven't looked carefully).
code = nimbleCode({
    w[1:2] ~ dmnorm(zeros[1:2],pr[1:2,1:2])
})
model = nimbleModel(code)

This occurs because in
uniqueCurrentVertexIDs <- unique(currentVertexIDblock)

currentVertexIDblock is a 2x2 matrix of NAs and unique returns a 1x2 matrix of NAs instead of a single value. This seems like it would occur any time we have a matrix or array of more than 1-dimension (I guess unless some splitting is occurring).
I think we simply want
uniqueCurrentVertexIDs <- unique(c(currentVertexIDblock))

but would be good for @perrydv to take a second look.

Nice catch @paciorek.  I can't see a scenario where the extra trailing element of graphID_2_declID would be used, and if used it has value 0 which is a valid indicator of "no decID exists", so it should be skipped by any using code anyway.  However I agree it is extraneous, and it is complicated to think through every implication in more complicated settings and determine if it is universally harmless.  You fix makes sense.  I am about to file a PR with that fix and see if it passes testing.


stream list: Prevent creating streams with the same name

Display toast message when user tries to create a stream with the
same as an already existing one.
Fix #3447.

I am not sure whether this is the best approach. Instead of showing a toast message, we could display an error above the "Create" button. This is the approach followed in the realm screen when we are trying to connect with an invalid organisation URL (like "a.zulipchat.com").

I force pushed some changes because:

I found it hard to spot the toast message when the keyboard is up. I think a red error message is more suitable in this case.

The warning should also appear when we try to edit a stream, not only when we create a stream.

Thanks @ungps for sending this!
A few comments:

I agree that a red error message sounds better than a toast.
For a PR like this where UI is an important part, please post screenshots in the PR thread; they help a lot for discussion/review.
I think adding a separate property to the state isn't the preferred React-idiomatic way to do this.

One UI consequence is that this version gives an error only when you go hit the submit button -- potentially after typing a bunch of description. Better would be when you blur the name field, or maybe even while you're typing (so in handleNameChange).
This post looks to have a pretty good description of what I think is a more preferred way to handle this: https://medium.freecodecamp.org/how-to-use-reacts-controlled-inputs-for-instant-form-field-validation-b1c7b033527e

Does this behave right when editing an existing stream? I think it may not. Be sure to test and think about that case too (which also uses this code, as its name suggests.)

Thank you @gnprice for review. I pushed a new version.

In both create / edit stream page, the error message is not displayed and the button is disabled when the "Name" field is empty.
In edit stream page, the error message is not displayed and the button is disabled when name field wasn't modified.  If description, access (private / public) or the case of the stream name was modified, the button is enabled (the error message still hidden)

Previews:
https://drive.google.com/file/d/13gJJwZvAV-5FLo6PETmw5vruVa-KDzXZ/view?usp=sharing
https://drive.google.com/file/d/16ayxOeDiLQ7Y1H2IUMhUYRmDJvNyuddU/view?usp=sharing

Thanks again @ungps !

In edit stream page, the error message is not displayed and the button is disabled when name field wasn't modified. If description, access (private / public) or the case of the stream name was modified, the button is enabled (the error message still hidden)

This sounds like a basically independent change, right? Let's make this a separate commit, either before or after.
Then that change can also be explained in its own commit message. Everything we change in a commit should be covered by something described in the commit message -- you could satisfy that by adding a mention of this change to the one commit message, but it's easier to understand if it's separated out.

Nit: Let's use a name like canSubmit rather than canUpdateStream. The latter sounds like it might be referring to the contrast between "Update" and "Create", which after all appears in the user text on the very same button.

Thanks for the videos -- but would you also post an actual screenshot, right here in the thread (so the image shows up right here on the GitHub PR page)? That makes it a lot easier to see and discuss. (In fact, one of the two videos doesn't even load for me.)
In particular I'd like to see a screenshot of the error message -- that's a new bit of design introduced by this PR, so we'll want it to be part of the discussion.

Thank you too, @gnprice ! I pushed new changes to

split the commit into two commits
Changed canUpdateStream to canSubmit because is more suitable.
I am pretty crowded with homework assignments at university and I couldn't keep up with #3450; but I hope I fixed merge issues correctly.

Below are a couple of screenshots (with caption):
Creating new stream - name field not filled (no error message, button disabled)

Creating new stream - stream name already in-use (display error message, disable button)

Editing stream - no field changed (no error message, button disabled)

Editing stream - name changed in upper case (no error message, button enabled)

Editing stream - stream name ("announce") already in use (error displayed, button disabled)
In both night mode / day mode:

Thanks @ungps for those screenshots!
One thing they show is that this layout is confusing:

The error is much too far from the field it's about; it's actually closer to the "Description" field below, so it looks like it's about that.
For some pictures of good design for this situation, search for "error" in the Material guidelines:
https://material.io/design/components/text-fields.html#anatomy
Our text fields aren't Material Design in general (although they'd probably be better if they were); but that may help illustrate something to move toward.

Thanks again, @gnprice!
Indeed, the message was way too far from the field. I hope this is better now. I also changed the colour of underline in case of error.


issue with test run

Hi,
I got this error message when I tried to run the test code:
snakemake -d .test/ecoli --cores 1 --use-conda

[Tue Jan 30 16:31:01 2024]
Error in rule create_db_intervals:
    jobid: 21
    input: results/GCA_003018455.1/data/genome/GCA_003018455.1.fna, results/GCA_003018455.1/data/genome/GCA_003018455.1.fna.fai, results/GCA_003018455.1/data/genome/GCA_003018455.1.dict, results/GCA_003018455.1/intervals/master_interval_list.list
    output: results/GCA_003018455.1/intervals/db_intervals/intervals.txt, results/GCA_003018455.1/intervals/db_intervals
    log: logs/GCA_003018455.1/db_intervals/log.txt (check log file(s) for error details)
    conda-env: /fs/scratch/PAS1554/snpArcher/.test/ecoli/.snakemake/conda/ec2d2883921c842412450a0289e25d36_
    shell:

        gatk SplitIntervals -L results/GCA_003018455.1/intervals/master_interval_list.list         -O results/GCA_003018455.1/intervals/db_intervals -R results/GCA_003018455.1/data/genome/GCA_003018455.1.fna -scatter 1         -mode INTERVAL_SUBDIVISION         --interval-merging-rule OVERLAPPING_ONLY &> logs/GCA_003018455.1/db_intervals/log.txt
        ls -l results/GCA_003018455.1/intervals/db_intervals/*scattered.interval_list > results/GCA_003018455.1/intervals/db_intervals/intervals.txt

        (one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!)

Removing output files of failed job create_db_intervals since they might be corrupted:
results/GCA_003018455.1/intervals/db_intervals
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: .snakemake/log/2024-01-30T163042.968256.snakemake.log

Thanks

Hi @vitorpavinato, sorry for the trouble running the test dataset. Could you provide your Snakemake version, as well as the output from the log file specified in the your post?

Hi @cademirch,
Sure, here is the snakemake version: 7.32.4
Where I can find the log file?
I am using a SLURM cluster and I ran on scratch.
I try to find the file with
find / -n 2024-01-30T163042.968256.snakemake.log 2>/dev/null

But I didn't return anything.
Thanks for the prompt response.

The log files should be in the .test directory within the snparcher
directory.
On Tue, Jan 30, 2024 at 17:13 Vitor Pavinato @.***>
wrote:

Hi @cademirch https://github.com/cademirch,
Sure, here is the snakemake version: 7.32.4
Where I can find the log file?
I am using a SLURM cluster and I ran on scratch.
I try to find the file with
find / -n 2024-01-30T163042.968256.snakemake.log 2>/dev/null
But I didn't return anything.
Thanks for the prompt response.
—
Reply to this email directly, view it on GitHub
https://github.com/harvardinformatics/snpArcher/issues/154#issuecomment-1918185330,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKVQJ4UDFWU3N6NVBRBZ66LYRGLFBAVCNFSM6AAAAABCSDQJBGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJYGE4DKMZTGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great,
Here is what I got from the log file:
Building DAG of jobs...
Your conda installation is not configured to use strict channel priorities. This is however crucial for having robust and correct environments (for details, see https://conda-forge.org/docs/user/tipsandtricks.html). Please consider to configure strict priorities by executing 'conda config --set channel_priority strict'.
Using shell: /usr/bin/bash
Provided cores: 1 (use --cores to define parallelism)
Rules claiming more threads will be scaled down.

[Tue Jan 30 16:30:52 2024]
rule fastp:
    input: data/local_fastq/my_sample1_1.fastq.gz, data/local_fastq/my_sample1_2.fastq.gz
    output: results/GCA_000008865.2/filtered_fastqs/SAMN12676327/SRR10058855_1.fastq.gz, results/GCA_000008865.2/filtered_fastqs/SAMN12676327/SRR10058855_2.fastq.gz, results/GCA_000008865.2/summary_stats/SAMN12676327/SRR10058855.fastp.out
    log: logs/GCA_000008865.2/fastp/SAMN12676327/SRR10058855.txt
    jobid: 11
    benchmark: benchmarks/GCA_000008865.2/fastp/SAMN12676327_SRR10058855.txt
    reason: Missing output files: results/GCA_000008865.2/filtered_fastqs/SAMN12676327/SRR10058855_2.fastq.gz, results/GCA_000008865.2/summary_stats/SAMN12676327/SRR10058855.fastp.out, results/GCA_000008865.2/filtered_fastqs/SAMN12676327/SRR10058855_1.fastq.gz
    wildcards: refGenome=GCA_000008865.2, sample=SAMN12676327, run=SRR10058855
    resources: tmpdir=/tmp, mem_mb=4000, mem_mib=3815

Activating conda environment: .snakemake/conda/f32d3b737d797443a34140e7912c58cd_
[Tue Jan 30 16:30:53 2024]
Finished job 11.
1 of 26 steps (4%) done
Select jobs to execute...

[Tue Jan 30 16:30:53 2024]
rule collect_fastp_stats:
    input: results/GCA_000008865.2/summary_stats/SAMN12676327/SRR10058855.fastp.out
    output: results/GCA_000008865.2/summary_stats/SAMN12676327_fastp.out
    jobid: 12
    reason: Missing output files: results/GCA_000008865.2/summary_stats/SAMN12676327_fastp.out; Input files updated by another job: results/GCA_000008865.2/summary_stats/SAMN12676327/SRR10058855.fastp.out
    wildcards: refGenome=GCA_000008865.2, sample=SAMN12676327
    resources: tmpdir=/tmp

[Tue Jan 30 16:30:53 2024]
Finished job 12.
2 of 26 steps (8%) done
Select jobs to execute...

[Tue Jan 30 16:30:53 2024]
checkpoint create_db_intervals:
    input: results/GCA_003018455.1/data/genome/GCA_003018455.1.fna, results/GCA_003018455.1/data/genome/GCA_003018455.1.fna.fai, results/GCA_003018455.1/data/genome/GCA_003018455.1.dict, results/GCA_003018455.1/intervals/master_interval_list.list
    output: results/GCA_003018455.1/intervals/db_intervals/intervals.txt, results/GCA_003018455.1/intervals/db_intervals
    log: logs/GCA_003018455.1/db_intervals/log.txt
    jobid: 21
    benchmark: benchmarks/GCA_003018455.1/db_intervals/benchmark.txt
    reason: Missing output files: results/GCA_003018455.1/intervals/db_intervals/intervals.txt
    wildcards: refGenome=GCA_003018455.1
    resources: tmpdir=/tmp, mem_mb=5000, mem_mib=4769
DAG of jobs will be updated after completion.

Activating conda environment: .snakemake/conda/ec2d2883921c842412450a0289e25d36_
[Tue Jan 30 16:31:01 2024]
Error in rule create_db_intervals:
    jobid: 21
    input: results/GCA_003018455.1/data/genome/GCA_003018455.1.fna, results/GCA_003018455.1/data/genome/GCA_003018455.1.fna.fai, results/GCA_003018455.1/data/genome/GCA_003018455.1.dict, results/GCA_003018455.1/intervals/master_interval_list.list
    output: results/GCA_003018455.1/intervals/db_intervals/intervals.txt, results/GCA_003018455.1/intervals/db_intervals
    log: logs/GCA_003018455.1/db_intervals/log.txt (check log file(s) for error details)
    conda-env: /fs/scratch/PAS1554/snpArcher/.test/ecoli/.snakemake/conda/ec2d2883921c842412450a0289e25d36_
    shell:

        gatk SplitIntervals -L results/GCA_003018455.1/intervals/master_interval_list.list         -O results/GCA_003018455.1/intervals/db_intervals -R results/GCA_003018455.1/data/genome/GCA_003018455.1.fna -scatter 1         -mode INTERVAL_SUBDIVISION         --interval-merging-rule OVERLAPPING_ONLY &> logs/GCA_003018455.1/db_intervals/log.txt
        ls -l results/GCA_003018455.1/intervals/db_intervals/*scattered.interval_list > results/GCA_003018455.1/intervals/db_intervals/intervals.txt

        (one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!)

Removing output files of failed job create_db_intervals since they might be corrupted:
results/GCA_003018455.1/intervals/db_intervals
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: .snakemake/log/2024-01-30T163042.968256.snakemake.log

Thanks. Can you also paste in the split intervals log? Should be in .test/ecoli/logs

is this one found at .test/ecoli/logs/GCA_003018455.1/db_intervals ?
16:31:01.301 INFO  NativeLibraryLoader - Loading libgkl_compression.so from jar:file:/fs/scratch/PAS1554/snpArcher/.test/ecoli/.snakemake/conda/ec2d2883921c842412450a0289e25d36_/share/gatk4-<IP_ADDRESS>-0/gatk-package-<IP_ADDRESS>-local.jar!/com/intel/gkl/native/libgkl_compression.so
Jan 30, 2024 4:31:01 PM shaded.cloud_nio.com.google.auth.oauth2.ComputeEngineCredentials runningOnComputeEngine
INFO: Failed to detect whether we are running on Google Compute Engine.
16:31:01.474 INFO  SplitIntervals - ------------------------------------------------------------
16:31:01.474 INFO  SplitIntervals - The Genome Analysis Toolkit (GATK) v<IP_ADDRESS>
16:31:01.474 INFO  SplitIntervals - For support and documentation go to https://software.broadinstitute.org/gatk/
16:31:01.474 INFO  SplitIntervals - Executing as<EMAIL_ADDRESS>on Linux v3.10.0-1160.102.1.el7.x86_64 amd64
16:31:01.474 INFO  SplitIntervals - Java runtime: OpenJDK 64-Bit Server VM v1.8.0_382-b05
16:31:01.475 INFO  SplitIntervals - Start Date/Time: January 30, 2024 4:31:01 PM EST
16:31:01.475 INFO  SplitIntervals - ------------------------------------------------------------
16:31:01.475 INFO  SplitIntervals - ------------------------------------------------------------
16:31:01.475 INFO  SplitIntervals - HTSJDK Version: 2.22.0
16:31:01.475 INFO  SplitIntervals - Picard Version: 2.22.8
16:31:01.475 INFO  SplitIntervals - HTSJDK Defaults.COMPRESSION_LEVEL : 2
16:31:01.475 INFO  SplitIntervals - HTSJDK Defaults.USE_ASYNC_IO_READ_FOR_SAMTOOLS : false
16:31:01.475 INFO  SplitIntervals - HTSJDK Defaults.USE_ASYNC_IO_WRITE_FOR_SAMTOOLS : true
16:31:01.475 INFO  SplitIntervals - HTSJDK Defaults.USE_ASYNC_IO_WRITE_FOR_TRIBBLE : false
16:31:01.476 INFO  SplitIntervals - Deflater: IntelDeflater
16:31:01.476 INFO  SplitIntervals - Inflater: IntelInflater
16:31:01.476 INFO  SplitIntervals - GCS max retries/reopens: 20
16:31:01.476 INFO  SplitIntervals - Requester pays: disabled
16:31:01.476 INFO  SplitIntervals - Initializing engine
16:31:01.785 INFO  SplitIntervals - Shutting down engine
[January 30, 2024 4:31:01 PM EST] org.broadinstitute.hellbender.tools.walkers.SplitIntervals done. Elapsed time: 0.01 minutes.
Runtime.totalMemory()=1164443648
***********************************************************************

A USER ERROR has occurred: Badly formed genome unclippedLoc: Query interval "CP027599.1 : 1 - 5942969" is not valid for this input.

***********************************************************************
Set the system property GATK_STACKTRACE_ON_USER_EXCEPTION (--java-options '-DGATK_STACKTRACE_ON_USER_EXCEPTION=true') to print the stack trace.
Using GATK jar /fs/scratch/PAS1554/snpArcher/.test/ecoli/.snakemake/conda/ec2d2883921c842412450a0289e25d36_/share/gatk4-<IP_ADDRESS>-0/gatk-package-<IP_ADDRESS>-local.jar
Running:
    java -Dsamjdk.use_async_io_read_samtools=false -Dsamjdk.use_async_io_write_samtools=true -Dsamjdk.use_async_io_write_tribble=false -Dsamjdk.compression_level=2 -jar /fs/scratch/PAS1554/snpArcher/.test/ecoli/.snakemake/conda/ec2d2883921c842412450a0289e25d36_/share/gatk4-<IP_ADDRESS>-0/gatk-package-<IP_ADDRESS>-local.jar SplitIntervals -L results/GCA_003018455.1/intervals/master_interval_list.list -O results/GCA_003018455.1/intervals/db_intervals -R results/GCA_003018455.1/data/genome/GCA_003018455.1.fna -scatter 1 -mode INTERVAL_SUBDIVISION --interval-merging-rule OVERLAPPING_ONLY

Thanks for posting that @vitorpavinato. Unfortunately, I cannot seem to recreate this. Could you let me know your python3 version? I suspect this may be the issue. For reference, I ran the same test on SLURM using python3==3.11.4.

Yes, sure. The conda environment I set has Python 3.12.1. I also should mention that I used conda instead of mamba in here:
mamba create -c conda-forge -c bioconda -n snparcher snakemake
mamba activate snparcher

Could you try with Python 3.11.X? I believe this is an issue with Snakemake and fstrings in 3.12.

Thanks for the update @vitorpavinato. We've pinned the python and snakemake version in the docs now.


ESP8266 support

Added support for using a ESP8266 to connect to a Kaifa MA309M (Salzburg AG). Successfully tested using GPIO1 + GPIO3 for TX/RX. I had to disable UART logging by setting baud_rate: 0, as otherwise this GPIOs are used for that.

Thanks!


[HOT FIX] fix stats functions in the recent patch

What changes were proposed in this pull request?
Builds failed due to the recent merge. This is because PR#18309 needed update after this patch was merged.
How was this patch tested?
N/A

Test build #78573 has started for PR 18415 at commit 6c3b7d4.

cc @srowen @cloud-fan

@wzhfy so this needs to go into master/2.2?

Test build #78573 has finished for PR 18415 at commit 6c3b7d4.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

@srowen Only master, tasks of removing sql conf were only merged in master.

Test build #78577 has started for PR 18415 at commit 6c3b7d4.

Test build #78577 has finished for PR 18415 at commit 6c3b7d4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

LGTM

merged to master


Add some sort of analytics to the page.

Add some sort of analytics to the page so we can see if it gets used.

I've enabled Cloudflare web analytics.


can PyAV be executed on android arm64 with ffmpeg compiled with --enable-static ?

Hi
Overview
I need to compile for Android (arm and arm64). Could it be possibile?
Expected behavior
Use PyAV for getting video metadata information without run 'ffmpeg' as external subprocess
Actual behavior
I tested on linux and macosx and everything works fine
Traceback:
No error
Investigation
I wonder: is it possible to import PyAV in my python scripts and use ffmpeg's functions without run it as subprocess, on Android devices?
Research
I have done the following:

[x] Checked the PyAV documentation
[x] Searched on Google
[x] Searched on Stack Overflow
[x] Looked through old GitHub issues
[x] Asked on PyAV Gitter   I asked but no reply
[x] ... and waited 72 hours for a response.

Additional context

Absolutely no idea, sorry.


Allow user-supplied handler to return a Promise

This implements handling of a thenable in handleRequest.  Allows user to return a promise, e.g.:
this.get('/api/song/12', function(request) {
  return new Promise(function(resolve) {
    resolve([ 200, {}, '{"id": 12}' ]);
  });
});

Closes #57

🚀

@trek possible to do a minor release with this?  I'd like to pull it into ember-cli-mirage


error.code is inaccessible due to 'fileprivate' protection level

When a request failed, I used to be able to check the error code, like this:
failure: { error in
    if error.code == 401 {
        //...
    }
}

Now with Swift 3 I can't do that anymore, I get that error: "'code' is inaccessible due to 'fileprivate' protection level".

That was fast. I'm already using Carthage, though, and still no luck. :/

see OAuthSwiftError
it could be an .tokenExpired or .requestError for code 401 (with last code, not released yet you have attribute underlyingError)
you can do a if
failure: { error in
    if case .tokenExpired == error {
        //...
    }
}

or a switch
failure: { error in
    switch error {
        case .tokenExpired (let underlyingError):
        //...
       case .requestError (let underlyingError):
        //...
       default:
    }
}

then you can cast to NSError or there is in swift _code

Ah, awesome, thanks!

@phimage So I was doing what you described in your last answer here, but now I updated to the latest version and I get "Enum case 'tokenExpired' not found in type 'Error'". Any ideas?
PS: You should consider adding a change log. It would be a huge help figuring out this sort of stuff.

please refer to release for change log


Add ORFEO ToolBox to the list of processing tools

Thank you for your suggestion! Unfortunately I don't think this tool fits the scope of the list of processing software.
I think the list of processing software should mainly include tools that are more or less specific to Sentinel data or specilialized in processing it, i.e. athmospheric processors, datacube cloud solutions, etc.
Including a general tool such as ORFEO ToolBox would beg the question - Why not include, GDAL, QGIS, GRASS, ArcGIS, etc.
Generalized tools are a better fit for other awesome-lists such as:

https://github.com/sacridini/Awesome-Geospatial
https://github.com/RoboDonut/awesome-spatial
https://github.com/sshuair/awesome-gis


Save selected reference table on Reference tab

It would be great to save the last selected reference table on Reference panel. You don't have to switch between tables every time when you start to work with the Reference panel.

Verified.


Remote OPNsense / Zabbix Proxy

Hello all,
I have a remote OPNsense firewall that I would like to manage with Zabbix. On the remote side I have a Zabbix proxy vm setup and talking back to my main Zabbix server, over a VPN. Do I install the Zabbix agent on the OPNsense firewall and point it to the Zabbix proxy vm, instead of pointing it directly to the main Zabbix server?
Thanks,
Steve

When you set the proxy to monitor the agent, then yes :)


Allow repeat opcode to be set programmatically

Feature description
Currently the repeat opcode in miden assembly requires the developer to specify the number of loop iterations:
repeat.5
    # 5 loop iterations
end

However, it would be extremely useful to allow the number of loop iterations to be set programmatically. For example:
push.5
repeat
     # 5 loop iterations
end

Why is this feature needed?
I know that there is a while opcode where a similar functionality could be achieved, however, allowing the repeat opcode to be set programmatically would be useful.

Interesting! I think what will need to happen is that the assembler will need to transform this into a while loop under the hood. Not sure yet how easy it would be.

The assembler could track the operand stack state in abstract (like we do in the compiler), and transform a runtime-bounded repeat like this to a while.true predicated on the current iteration count. This is more or less how something like for i in items.len() { .. } gets translated by the compiler anyway. Doable in the assembler, but not sure if the benefits outweigh the costs in terms of implementation complexity.
Somewhat related, but at some point I'd like to sit down and analyze whether it is feasible to represent something like goto-with-labels in a MAST-like structure (i.e. MAST, but maybe with some new/modified primitives). With that we could allow a much richer set of control primitives to be represented. I'm convinced it has to be possible, but until then, we unfortunately have to find clever ways to represent the control flow we want using the current set of primitives.

Somewhat related, but at some point I'd like to sit down and analyze whether it is feasible to represent something like goto-with-labels in a MAST-like structure (i.e. MAST, but maybe with some new/modified primitives). With that we could allow a much richer set of control primitives to be represented. I'm convinced it has to be possible, but until then, we unfortunately have to find clever ways to represent the control flow we want using the current set of primitives.

#519 is something similar to this. I am not sure this is possible in MAST - but I also didn't think Dyn blocks were possible originally :)


adguardhome: fix eval error when source does not exist and source for aarch64-darwin

Description of changes
Currently this derivation produces an error when evaluated on a system for which a source is not available. fe7acb625cb1a4c3b7a33018d486456b6699e4be fixes this and c4d00d46615a838d760dcc09c42389642841712f adds the source for aarch64-darwin.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 170509 run on aarch64-darwin 1

1 package built:

adguardhome


Warning: "Avoid app logic that relies on enumerating keys on component instance" during snapshot tests

Test Utils console logs warning message during snapshot testing.
Steps to Reproduce
    it('should render correctly', () => {
      const wrapper = mount(Foo);
      expect(wrapper).toMatchSnapshot();
    });

Actual Behavior
Test passes/fails as expected, but test Utils console logs warning message during snapshot testing.
console.warn node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:40
      [Vue warn]: Avoid app logic that relies on enumerating keys on a component instance. The keys will be empty in production mode to avoid performance overhead.

Expected Behavior
Test should pass/fail as appropriate without warning message.

Shouldn't it be expect(wrapper.html()).toMatchSnapshot(); instead of the whole wrapper object?
(especially in a test described as 'it should render correctly')
Then you are snapshotting just the html and that warning does not appear.

wrapper is a complicated class with a ton of un-serializable things, such as the circular dependencies Vue creates in it's vdom tree.
I'd recommend doing what is suggested above. You want to match against the html, right? Either way, this is not a VTU specific issue; Vue is telling you not to enumerate keys on the instance.

Yes. This was simply an oversight by me. Thank you and I apologize.

No need to apologize!! Good luck 👍


useQuickActionRouting Only Triggers on Initial App Start, Not in Background State

Hi!
I'm having an issue with the useQuickActionRouting hook. When the app is fully closed, the hook correctly intercepts actions. However, if the app is only in the background (not completely killed), the action isn’t picked up.

I have the same issue.
Without having more information about your environment, it's difficult to help.
But in my case, this problem appears since Expo52 and RN0.76.
I think the package is not compatible with Expo52 yet.
@EvanBacon any plan to update the package to make it compatible?
Thank you.

I am also using SDK52 and RN 0.76. I don’t know if this happened since I only used it since SDK52.

From: Tom Corvus @.>
Sent: Thursday, November 21, 2024 11:18:17 AM
To: EvanBacon/expo-quick-actions @.>
Cc: Joost @.>; Author @.>
Subject: Re: [EvanBacon/expo-quick-actions] useQuickActionRouting Only Triggers on Initial App Start, Not in Background State (Issue #31)
I have the same issue.
Without having more information about your environment, it's difficult to help.
But in my case, this problem appears since Expo52 and RN0.76.
I think the package is not compatible with Expo52 yet.
@EvanBaconhttps://github.com/EvanBacon any plan to update the package to make it compatible?
Thank you.
—
Reply to this email directly, view it on GitHubhttps://github.com/EvanBacon/expo-quick-actions/issues/31#issuecomment-2490703896, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APRJ7NQQPQRADEMSVGUN6U32BWXOTAVCNFSM6AAAAABRYTZTCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJQG4YDGOBZGY.
You are receiving this because you authored the thread.Message ID: @.***>

I confirm that listener works on Expo51/RN0.75 when the app is in background or closed so the author have to make it compatible with Expo52.

Paging Dr. @EvanBacon :)

I encountered the same issue after upgrading to Expo 52
After digging into it, I've been able to make it work and submitted an upgrade on https://github.com/EvanBacon/expo-quick-actions/pull/32
In the meantime I'm using the following patch expo-quick-actions-npm-2.0.0-48c4cc5a85.patch

Closing this as I can confirm this is fixed in the latest release.


CI: Push detached head to master

#!/bin/bash -eo pipefail
git config user.email "${GITHUB_EMAIL}"
git config user.name "CircleCI Deploy"
git add templateflow/conf/templateflow-skel.zip
git commit -m "rel(${CIRCLE_TAG}): Update S3 skeleton file [skip ci]"
git push origin master

[detached HEAD ce41662] rel(0.4.1rc2): Update S3 skeleton file [skip ci]
 1 file changed, 0 insertions(+), 0 deletions(-)
 rewrite templateflow/conf/templateflow-skel.zip (73%)
Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.

Total 0 (delta 0), reused 0 (delta 0)
To github.com:templateflow/python-client.git
   a0adcb7..0c0deb9  master -> master

Didn't see this. Although most of the releases will be done off of master, I don't think this would fully improve the situation when releasing from a different branch. I'm testing that right now

I think our two approaches were equivalent.

It is still merging the full branch into master when releasing from the branch

Right. Did yours not?

Didn't check, assumed it wouldn't. Let me try again :(

In both cases, you're committing on top of the latest release, and pushing the new commit to master. It will try a fast-forward merge. If what you want is to check out master and not merge from the branch:
git fetch origin
git checkout origin/master
git add ...
git commit ...
git push origin HEAD:master


Freezing a mock does not override an injected instance

Current Behaviour
When injecting two concrete instances, the second instance overrides the first.
When injecting a concrete instance, followed by a mocked instance, the second instance does not override the first.
This makes the idea of performing some tests which use a mock and some tests which use an implementation (using a test class constructor to share a fixture between tests) fairly awkward.
Expected Behaviour
When injecting two concrete instances, the second instance overrides the first.
When injecting a concrete instance, followed by a mocked instance, the second instance overrides the first.
Example of Expected Behaviour
using System.Collections.Generic;
using AutoFixture;
using AutoFixture.AutoMoq;
using Moq;
using Xunit;

namespace SomeNamespace
{
	public class AutoFixtureTests
	{
		[Fact]
		// Passes
		public void Should_OverridePreviouslyInjectedString()
		{
			const string test1 = "test1";
			const string test2 = "test2";

			var fixture = new Fixture();

			fixture.Inject(test1);
			fixture.Inject(test2);

			Assert.Equal(fixture.Create<string>(), test2);
		}

		[Fact]
		// Fails
		public void Should_OverridePreviouslyInjectedInstance()
		{
			var fixture = new Fixture().Customize(new AutoMoqCustomization());

			var sut1 = new List<int>();

			fixture.Inject<IList<int>>(sut1);
			fixture.Freeze<Mock<IList<int>>>();

			var sut2 = fixture.Create<IList<int>>();

			Assert.NotSame(sut1, sut2);
		}
	}
}

Hi @charles-salmon,
Thank you for taking the time to report this.
The behaviour you're observing is by design. Allow me to explain. 🙂
If you look at the implementation of the Freeze<T> method:
var value = fixture.Create<T>();
fixture.Inject(value);
return value;

You'll see that all it's doing is creating a specimen of T and injecting it into the fixture. Indeed, the Freeze method came about as a handy shortcut to this very operation. As @ploeh wrote in his post introducing the Freeze method back in 2010:

It turned out that we used this coding idiom so much that we decided to encapsulate it in a convenience method. After some debate we arrived at the name Freeze, because we essentially freeze a single anonymous variable in the fixture, bypassing the default algorithm for creating new instances.

Given that, the fact that freezing the same type T as an injected instance yields the same instance is expected.
Now, to address your point:

This makes the idea of performing some tests which use a mock and some tests which use an implementation (using a test class constructor to share a fixture between tests) fairly awkward.

Given that a fixture represents the context in which a test runs, you certainly can have multiple tests share the same fixture; however, this comes at the cost of mixing multiple—and possibly conflicting—concerns.
The xUnit Patterns book sums it up nicely:

The biggest issue with a [Shared Fixture](url
http://xunitpatterns.com/Shared Fixture.html) is that it can lead to "collisions" between tests possibly resulting in Erratic Tests, since tests may depend on the outcomes of other tests. Another issue is that a fixture designed to serve many tests is bound to be much more complicated than the Minimal Fixture needed for a single test.

This "collision" is exactly what happens when a test expects an object of type T to be a specific instance, while another test expects T to be a fake object.
Since I can't think of a test scenario where it makes sense for the same type T to both be a mock and a concrete instance (although I'm happy to be proven wrong), I suggest you have these tests use different fixtures, each setup to serve their particular scenario.


fix: remove @typescript-eslint/require-await

What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[x] Bug fix
[ ] New feature
[ ] Other, please explain:
What changes did you make? (Give an overview)
Removed @typescript-eslint/require-await from the rules & unit test.
Which issue (if any) does this pull request address?
Fixes #217
Is there anything you'd like reviewers to focus on?
Thanks so much for the in-depth discussion of this change.

Oops, sorry, left a tab open, forgot I had already hit submit.


Modify chart to support master branch

The chart currently is using images of Harbor 1.5 which contains three databases, this commit reduce them to only one to support master branch codes.

Pull Request Test Coverage Report for Build 7578

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

Totals

Change from base Build 7569:
0.0%

Covered Lines:
11085

Relevant Lines:
17154

💛  - Coveralls


User configurable UDP ports

Please add user configurable source and destination port in SoulissApp
From https://github.com/souliss/souliss/issues/97

the user is now able to choose UDP port (default is 230)


Search Results: Update pagination so that it works like angular-ui example

Mostly this has to do with making ellipses work as they do in this directive:
http://angular-ui.github.io/bootstrap/#/pagination
From @yawitz audit.

I got my local Samplestack branch working with the angular-ui pagination component. The version I have working is basically the last one shown in the angular-ui examples, the one with the ellipses.
http://angular-ui.github.io/bootstrap/#/pagination
This example is most similar to the pagination shown in the @yawitz wireframes.
However, there are substantial differences in how this version of the pagination works and how the @yawitzpagination is spec'd. A few differences:
@yawitz: Persistent first/last buttons are inside the prev/next buttons
UI Bootstrap: Persistent first/last buttons are outside the prev/next buttons
@yawitz: Button sets are adjusted so that size of pager does not vary for long result sets
UI Bootstrap: Size of pager can vary for long result sets depending on where the user is in the set.
@yawitz: First/last buttons persist but are never duplicated
UI Bootstrap: When first/last buttons are persisted (via the "boundary-links" setting), those buttons can sometimes be functionally duplicated (e.g., "First" outside, "1" inside)
Realizing the design in the wireframes will require either rewriting some of the angular-ui pagination component or rolling our own (starting with the version I had working already).  So that's where we are with that.

I leave it to you guys with the hope that you'll find/choose the path of least resistance! :)

I'm OK to ship with the default angular-ui pagination component, but would like to leave this open/deferred to revisit post 1.0, or at least convert it to a new task for later (keeping the wireframe spec as-is).
Background: The wireframe spec is a distillation of a number of pagination components I found in my research + some UX judgement calls. I'd like to revisit the details when we have a little more time.

Thanks for the input, @laurelnaiad and @yawitz. I think going with the angular-ui pagination as-is for now is a good idea given the schedule and bug list. I do like the concepts you've surfaced in your pagination diagrams, @yawitz.
I will tidy up the work and move to a PR.

@yawitz may need to open new issue/rFE to revisit in future release.

Can you clarify the process here: if I want to keep the spec as-is, but note the difference between implementation and spec, is it appropriate to classify and log the spec details as an RFE, vs. leave this issue open and deferred to 8.0-2 or later? (I'm not sure forcing the spec to fit the incomplete implementation is the right thing to do; seems a little backwards.)

If we're leaving the spec as is and not implementing it, I think it's a Bug that we can leave untargeted or on 8.02 or something. If we're changing the spec, then a marker to revisit in the future is an RFE.... basically RFE's say "let's consider changing the requirements" and bugs say "The implementation doesn't match the requirements."

Defer for this release (1.1) - high effort / low priority. Leave open for a discussion next release to see if any other middle ground / configuration can be done to get implementation closer to spec, based on what is out of the box. Then update wireframe spec to reflect revised agreement. Not worth investing significant effort as a part of Samplestack to design a custom widget (but may be relevant for future/other projects to build reusable components in this design).


Show who can see archived submissions

Fixes #3388.
This PR adds an indication as to what user roles can see an archived submission based off of existing settings.
Screenshots
With default Hypha settings

After updating settings to allow Staff and Staff Admin to see archives

After updating settings to only allow Staff Admin (not Staff) to see archives

The po and pot files I tend to update in separate PRs for practical reasons.
If you update po and pot files in more than one PR you tend to get merge conflicts. No problem keeping them here, good to get them updated.

@theskumar this is great feedback, thanks a ton!
I was originally working to make it more sentence-like (ie. "role, role and role") but wondered how that would impact translations if different linguistics were to come into play. If that might be an issue I can refactor my last commit. Definitely makes sense to have it more modular though.

@frjo Noted with the translation files - would it make more sense to revert them to what's on main and follow up with the altered files? I have translations committed to #3636 that I can revert too and then do all of them at once after it's all merged.

Reverting this back to a draft as I add features from #3389.

@frjo @theskumar I believe this is ready to go. The only piece that is still up for discussion would be the behavior of the 403 Forbidden page when a user without archive access tries to view and archived submission. The options I see are:

Redirect the user to the submission dashboard
Create a new "Access Denied" page/view that would allow all instances of raise PermissionDenied to be a little more descriptive and clean
Leave the existing functionality as is

Let me know your thoughts!

@frjo @theskumar I believe this is ready to go. The only piece that is still up for discussion would be the behavior of the 403 Forbidden page when a user without archive access tries to view and archived submission. The options I see are:

Redirect the user to the submission dashboard
Create a new "Access Denied" page/view that would allow all instances of raise PermissionDenied to be a little more descriptive and clean
Leave the existing functionality as is

Let me know your thoughts!

Explicit is better than implicit. I would suggest going with -- Create a new "Access Denied" page/view that would allow all instances of raise PermissionDenied to be a little more descriptive and clean.

@theskumar All sounds good! The functionality for the 403 page is pretty much mirrored from the 404. It can be customized in Wagtail admin and is triggered when any django.core.exceptions.PermissionDenied error is raised. By default, it looks like the following:

With the Wagtail admin prompts:

This was also tested in the archive context (user with staff role attempted to view an archived submission) and worked well.


add an error code for the logger post if the body does not exist

Most likely, the body handler is not wired in for the logger post handler. Add this error to the lightapi.net with root cause and resolutions.

 add an error code for the logger post if the body does not exist light-4j master