Exposing the log flag and integrating with better logging plugin

This PR does a few things...

Make sure all files use internal logger class, instead of loading winston directly.
Adjusts internal logger class to support the --color flag.  This will be useful in our VSTS build logs where color is not supported.
Disables showLevel for the winston transport.  This means no more info: ... before every log entry.
Using a new webpack logger that support log levels: none (custom), minimal, compact, expanded, and verbose.  See simple-progress-webpack-plugin.  Currently pointing to my fork until @dominique-mueller has the opportunity to release.

@blackbaud-johnly tagging you as this would require entries for the --log and --color flags.

Codecov Report

Merging #357 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #357      +/-   ##
==========================================
+ Coverage   99.22%   99.22%   +<.01%
==========================================
  Files          71       71
  Lines        1685     1686       +1
  Branches      264      265       +1
==========================================
+ Hits         1672     1673       +1
  Misses         13       13

Flag
Coverage Δ

#builder
100% <100%> (ø)
:arrow_up:

#runtime
94.11% <ø> (ø)
:arrow_up:

#srcapp
100% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

config/webpack/serve.webpack.config.js
100% <ø> (ø)
:arrow_up:

config/webpack/common.webpack.config.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@Blackbaud-BobbyEarl Just one comment. I like that all logging goes through a single source now!

@Blackbaud-SteveBrush I believe this is ready for another look.  Since you first approved, I've solidifed on --logColor, --logLevel, and --logFormat as the available options.  I've also migrated the logging code to @blackbaud/skyux-logger.


Add to render_controlls subroutine

I created the "render_controlls" subroutine to improve readability.
Please review!

Please Re-review!
(Please note a3727b6 commit)

Nice catch in print_window!


Project Feedback!

👍 Congrats on completing the third project!
I have provided a detailed Project 3 Feedback Guide here which covers the most common issues with this submitted project. Read through the feedback guide point-by-point to determine how you could improve your submission.
Let us know if you have any other thoughts or questions about this assignment. Hopefully by now you feel pretty comfortable with all the major pieces to basic Android apps (Views, Controllers, Models, Authentication, API Communication, Preferences, ActionBar, et al) and see how they all fit together.

It looks like the following features are not reflected on your GIF walkthrough:

User Story 3: User is displayed the relative timestamp for each tweet "8m", "7m"

In order for us to count these towards your submission, please record another GIF that captures these features. Once you do, please push your updates and submit your assignment again through the course portal within 48 hours from the posted deadline so that we can regrade it.
/cc @codepathreview


Remove link to nonexistent DEVELOPING.md

Just a quick edit to remove this broken link. This file must have been removed at some point.

Thanks for catching this! I removed the file after overhauling and refactoring a lot. I should definitely write up a new one though.


Revert "Added support for table prefixes for MySql"

Reverts spatie/laravel-migrate-fresh#42

Reverting because the tests fail locally with 1) Spatie\MigrateFresh\Test\MigrateFreshTest::it_can_create_a_fresh_database Illuminate\Database\QueryException: SQLSTATE[42S02]: Base table or view not found: 1051 Unknown table 'laravel_migrate_fresh.test_migrations' (SQL: drop table test_migrations)

I want this prefix dropper ;)


revert: revert the palette version to 37.2.2

The type of this PR is: Revert

Description
Since upgrading the palette version to 37.2.3  was merged, two tests have been failing. To unblock other works, this PR reverts the change:  https://github.com/artsy/force/pull/13318

Eh, I don't see how this would cause a test failure. There's literally only the addition of a new text style.

Ah, it's not that it's something in here. Probably the portal change.

Actual bug with DeepZoom that I'm fixing. Now that Portals rely on state, there's a slightly strange set up in this component where there's a ref inside of the modal and an effect outside of that that is looking for it. So it doesn't initialize.

PR incoming


Add ChatGPT-based PR Reviewer and Summarizer

@harjotgill whoa this looks cool in theory.. can you paste some examples of what the output looks like? I'm hesitant to add new GH actions to a repo without any explanation or examples of how they work.
Thanks! 🙏

@harjotgill whoa this looks cool in theory.. can you paste some examples of what the output looks like? I'm hesitant to add new GH actions to a repo without any explanation or examples of how they work.
Thanks! 🙏

example - https://github.com/fluxninja/aperture/pull/1563

Just renamed ChatGPT -> OpenAI. ChatGPT as a offering has a different data usage policy than OpenAI API (which this workflow uses).


Make "deployer" optional

Link to issue: https://github.com/skyscrapers/terraform-website-s3-cloudfront-route53/issues/43

Made the requested changes @iuriaranda, thanks for the feedback!

Thanks @arjitj2 ! I'll cut a new release with your changes. It will be version 5.3.0

the following error occurred after this last update :(
Error: Missing resource instance key

  on .terraform/modules/dev-app/site-main/main.tf line 84, in resource "aws_iam_policy_attachment" "site-deployer-attach-user-policy":
  84:   policy_arn = aws_iam_policy.site_deployer_policy.arn

Because aws_iam_policy.site_deployer_policy has "count" set, its attributes
must be accessed on specific instances.

For example, to correlate with indices of a referring resource, use:
    aws_iam_policy.site_deployer_policy[count.index]

opsie, I have missed that in my review. I've released a new version of the module (5.3.1) with the fix. Sorry about that.


WIP: feat: add color picker

Minor design inconsistencies

Right-hand side alignment
Border color?

Deployed at: https://alva-issues-763.surge.sh

Deployed at: https://alva-site-issues-763.surge.sh


file missing

Hi, when I run the code, there is no 'invert_nn_caffe' or 'invert_nn' function in the patch. Could you please give some suggestions? Thanks a lot

Those functions are called within 'experiment_run()'

It's fixed now. rename the function calls into "invert_nn_dw"


It would be nice to configure max exclusion items, so get that from ENV

This test is icky -- my first attempt was to actually set the ENV, but that gets used on class load. Advisement appreciated.

First of all, I don't think we should introduce environment variables as a way to configure RuboCop at this point. Up until now we've only used the YAML configuration files and command line options. Take a look at #1531, which adds a command line option --exclude-limit. This solution was not merged. It has a flaw in that it generates Exclude lists in place of Max etc, rather than only for Enabled: false, something that the current master gets right. Other than that I think it's mostly valid as a solution to the problem you're solving here, so use parts of it as you see fit.

Closing in favor of #2004.


fix(launch): correct entrypoint path from disabled git repo subir

Fixes

Fixes WB-13821

Description
Recreates the rel-path (to cwd()) when git_disabled instead of root.
Example repo structure, where train.py inits a run with wandb.Settings(disable_git=True) and calls log_code:
git_repo
   ├── src
   |   ├── engine
   │   │   ├── train.py

Prod:
cd src && python engine/train.py --> results in job with entrypoint: src/engine/train.py
This branch:
cd src && python engine/train.py --> results in job with entrypoint: engine/train.py

🤖 Generated by Copilot at 4137065
This pull request improves the consistency and accuracy of the codePath and program_relpath settings in the system information and the wandb settings, by using a helper function that respects the disable_git setting and the git repository root. This affects the files system_info.py and wandb_settings.py.
Testing
How was this PR tested?

🤖 Generated by Copilot at 4137065

Sing, O Muse, of the skillful coder who refined
The system's data, and the codePath value aligned
With the disable_git setting, using a helper function wise
As Hephaestus, who forged the thunderbolts for Zeus in the skies

I think this is simpler


Joystick

The joystick isn't appearing when clicking on "Connect" button.
LG-G4 (LG-H812)

The "Connect" button only link the overlay to the app (means the joystick param changes made will affect the connected app)
Use "Toggle Floating View" in the menu (main page)


UI Gui/Library documentation

please create a UI Documentation it is very beautiful man I want to use it

there is kinda one on the wiki, but it's prob a little outdated tho
https://github.com/7GrandDadPGN/VapeV4ForRoblox/wiki/Documentation

Schuduled for the rewrite.

I begged for this 2 years ago bro 😭

💀

I begged for this 2 years ago bro 😭

well now I actually made a ui library that doesn't take a phd to understand so /shrug

Hoping to accomplish this soon.

no its really not LOL, the rewrite is way better & easier to understand.

how tf do you use this um so confused

no its really not LOL, the rewrite is way better & easier to understand.

when is the rewrite?

we all whould love to know

even i use vape guilbrary, for my 'Break in 2' script lol

no its really not LOL, the rewrite is way better & easier to understand.

when is the rewrite?

when an exploit gets stable & I finish the promotional material.

7GrandDad, you should add the 'Object' like 'Slider.Object' to your docs, so, other devs that uses your GuiLibrary, knows about it

Yummy

noway

after 1 year 🎉


Small addition to allow it to variably change its sample count.

When doing the sampling part in the rendering code, it used to use a simple for loop with a fixed amount of loops (decided by sampleCount).
In my version, I changed it to a while loop and I recorded the amount of time it takes to do the sampling. When the sampling takes longer than x amount of milliseconds, it stops sampling. Currently at a target FPS of 60, it may only take 1ms to do the sampling. However, that is still enough to get anywhere between 40 and 170 samples per pixel on my computer (1080ti and i9-7940x). To prevent black frames, it must do at least one sample per pixel. In order to prevent it from taking up too much power, I added a limit of 256 samples per pixel (although my computer does not reach it).
With 4 samples per pixel:

With variable samples per pixel:

Thanks for the PR! It's certainly much smoother looking. Unfortunately, this drops frames on my box.  We might just make the number of samples configurable. :man_shrugging:

After trying some more, I just cannot get it to work. The draw call is async and nothing in the sampling loop acts as some sort of synchronisation. I do not know any reliable way to do that.
The only thing that I can think off, is to keep increasing the sample count until you start seeing frames being dropped.


fix(docs): Documentation review suggestions for Readme

Signed-off-by: Uma Praseeda<EMAIL_ADDRESS>

Thank you!


Fix the code to reflect the description "occurred today"

@microsoft-github-policy-service agree [company=""]

@microsoft-github-policy-service agree

@MashaMSFT
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

The underlying article has been refreshed, with this change included. This PR will close automatically.


[C4GT] - Starter pack for location master

Project Details
Most projects need a "master database" of Indian locations - Villages, Districts, States, PIN codes etc. Almost each project reinvents it, and they also need to invest in finding the data from a trusted source, eg: Post office or LGD data. I'm proposing creating a solution using registries that is able to provide all such master APIs.
Learning Path

https://github.com/orgs/Sunbird-RC/discussions/676
Sunbird RC docs
Essential reading Registries | Registries vs databases

Complexity : Medium
Skills Required

Angular
Typescript
Postman / API

Name of Mentors

Vikram K
Pawan A

Project size
~8 weeks
Project Set Up
Features to be implemented (Acceptance criteria)

In India at least there are multiple hieararchies that are possible (LGD defines two) and wikipedia has another (https://en.wikipedia.org/wiki/Administrative_divisions_of_India), the post office data may have another
Allow defining a hierarchy (eg: State > District > Subdistrict > Town / City / Village). The solution should provide for various different hierarchies that are commonly used.
Write script to take data from a well known source, and populate data in the defined hierarchies. Should factor in including well known IDs like ISO codes, state codes (eg: Maharashtra is 27 in GST DB)
Should be possible to run queries like - get list of states; get districts and cities for selected State; for given PIN code get state, district and cities

Hey I would like to contribute to this project. Could you please guide me? Looking forward to submitting a proposal for this.

Hi @coolbung
I would like to contribute to this project. I would appreciate it if you could provide guidance on how I can get started with the project. Should I directly submit a proposal or is there any specific process to follow?

Hey @coolbung I would really like to contribute to this project . Will submit the proposal for the same .


Scrollbars are always hidden

The visualization views never show scrollbars. While this may be okay if you have a touchpad, it is not okay for people with mice since they often don't support vertical scrolling.
We should use overflow: auto instead of overflow: hidden.

horizontal?

Prefer this

Over this

I don't think the issue is overflow:hidden, otherwise you can't even scroll.

Even better: only show the scrollbars like we do in the main gallery but also do this in the bookmark gallery.

Not an issue anymore


Rye wrongly assumes a project is managed by Rye if there is a pyproject.toml

Steps to Reproduce
Create an empty project with a pyproject.toml file:
mkdir test
cd test
touch pyproject.toml

Then rye show outputs:
project: <unnamed>
path: /home/louis/Downloads/test
venv: /home/louis/Downloads/test/.venv
target python<EMAIL_ADDRESS>venv python<EMAIL_ADDRESS>virtual: false
configured sources:
  default (index: https://pypi.org/simple/)

This indicated that Rye thinks that the project is managed by Rye, but it is not the case (it could be manage by anything else such as Poetry).
Additionally, Rye does not seem to honor the managed flag in the pyproject.toml configuration:
[tool.rye]
managed = false

Adding these lines results in the same output from rye show.
Expected Result
Rye shouldn't assumes a project is managed by itself solely based on the presence of a pyproject.toml file since other Python project managing tools such a Poetry may use it for this same purpose.
This results in the impossibility to manage a Python project with Poetry using a Rye "system" Python toolchain (global-python  = true).
Actual Result
Poetry fails to find Python toolchains fetched by Rye.
Version Info
rye 0.37.0
Stacktrace
No response

What is the more specific harm here? It would make it easier to plan what to change if it was more specific about what's wrong. IMO it's only good that rye can interact to some extent with projects it doesn't manage - for example rye run works on unmanaged projects today. And rye show should not modify the project it is inspecting (that would be a bug).

What is the more specific harm here? It would make it easier to understand what to change if it was more specific about what's wrong. IMO it's only good that rye can interact to some extent with projects it doesn't manage - for example rye run works on unmanaged projects today. And rye show should not modify the project it is inspecting (that would be a bug).

Harm is this: This results in the impossibility to manage a Python project with Poetry using a Rye "system" Python toolchain.
I encountered this problem too. I want to use the global Python package managed by Rye to work with projects that are not managed by Rye. If a project is managed by Poetry, Rye incorrectly assumes it is Rye project and won't let me call python -m poetry.

Inspired by https://github.com/astral-sh/rye/discussions/998#discussioncomment-10385800 I did this as a workaround:
In ~/.zshrc or ~/.bashrc:
export<EMAIL_ADDRESS># replace with your Rye global environment path

pyga() {
    # Save current PATH in a backup variable
    export OLD_PATH="$PATH"

    # Add Rye-managed Python to the beginning of PATH
    export PATH="$RYE_PYG:$PATH"

    echo "Global Python activated. Run 'pygd' to restore the previous PATH."
}

# Function to deactivate and restore original PATH
pygd() {
    if [ -n "$OLD_PATH" ]; then
        export PATH="$OLD_PATH"
        unset OLD_PATH
        echo "Restored the original PATH."
    else
        echo "No previous PATH to restore."
    fi
}

Running pyga will then override the PATH to use the global Python environment managed by Rye.


How to filter the merged transcripts

Hi, there! I have used the stringtie2 to the genome-based transcripts assembly. I used the hisat2 to do the alignment of the RNA-seq data , and then the picard the remove the PCR errors. Finally, used the stringtie2 to assembl the transcripts. I finally used the IGV to check some transcripts. There are some transcripts that are merged from two nearby genes, as showed in the following picture. Is there some parameters that can be used to filter them? or some scripts? Need help!

Thanks so much!
Yizhong Huang

This is usually caused by read alignments spanning/bridging the two genes when they are very close to each other, and there is currently no easy solution for that - if the "evidence" in the read alignments data points to that.  What organism is this? It's also useful to look at the read alignments track in IGV, check if there are a lot of reads spanning that intergenic space etc.
Those genes seem to be very close to each other (hard to tell without seeing the annotation track), it's not clear if the "fusion" happens due to the terminal exons overlapping (TSS of one gene too close to TES of the other, or post-TES polymerase run-through?), or due to spurious (spliced) read alignments creating false "junctions" linking the two genes.
A script could be devised to split such "chimeric" transcripts but that would be a band-aid solution covering for a possibly deeper issue --  it would be interesting to look closer at WHY that really happens when it does -- there could be situations where such "fusion transcripts" across neighboring genes might be "real" and not just alignment artifacts (e.g. in case of genes sharing a transcriptional unit, i.e.  polycistronic transcription which has been shown to be possible in eukaryotes as well, not just in bacterial operons).

Thanks for your quick reply. Agree with you. I have checked the IGV with the RNA-seq mapped data ,as showed below. As there were so many mapped read, I just showed parts of alignments. What's your suggestion about it ?
Thanks again for your kind help!

Hi, I had the same issue; I tried to decrease the maximum intron length in the alignment which solved the problem ~ so far!

Hi, I had the same issue; I tried to decrease the maximum intron length in the alignment which solved the problem ~ so far!

Sounds great! How to set the parameter to do it  and have you solve this problems? I used the exons number (below 7) in the UTR region to filter the transcripts. I also check it in the IGV， almost all the fusion transcripts can be identified.

Hi, I had the same issue; I tried to decrease the maximum intron length in the alignment which solved the problem ~ so far!

Sounds great! How to set the parameter to do it and have you solve this problems? I used the exons number (below 7) in the UTR region to filter the transcripts. I also check it in the IGV， almost all the fusion transcripts can be identified.

I solved the problem by reducing the maximum intron length in the alignment step, not the assembly. Check your aligner documentation and change this parameter.

Thanks so much. I used the hisat2. To align the illumina paired data. And I check  the --max-intronlen  for it . But how to set this parameters, I confused .
Thanks so much.

2022年3月11日 02:09，AmrSaadeldin @.***> 写道：
maximum intron length


Updating Italy outcomes to most recent version

Trello card: https://trello.com/c/4IPj3ndU

@sistephens @leenagupte could you please add a link to the Trello card? Thank you

Hi @sistephens, can we close this PR?  Not sure why is still open. There is a Trello card linked to it in the Done column.


New meeting time for Telecom User Group on 3rd Monday - APAC Time Zones

Telecom User Group meeting time on 3rd Mondays - APAC Time Zones
The CNCF Telecom User Group meeting is currently held on the 1st and 3rd Mondays of the month at 8 AM PT.
The group would like to reschedule the meetings on the 3rd Mondays of the month to be compatible with APAC Time Zones, starting on August 19th. The APAC Telecom User Group meeting will recur on a monthly basis on the 3rd Monday of the month. (The meetings on the 1st Mondays of the month will continue to be held at 8 AM PT.)
The proposed meeting time for Monday, August 19, 2019 is 7 - 8 PM Beijing Time (11 AM - 12 PM GMT).
Does this time work for folks in the Telecom User Group?
Please vote in the comments.

Would you consider scheduling it one hour later so it will be somewhat doable for US Pacific time zone? The proposed time is 4AM which will practically exclude west coast participants. 5AM will make it a bit more tolerable.

Time works for me in general and would work on Monday, August 19. Could just imagine that it is too short term for a newly scheduled meeting already on next Monday.

Good point. We'll plan to cancel the meeting on the 19th so the new date has more time to percolate.

7-8pm beijing time is more friendly for APAC paticipants. Thanks for the considerations !

Yes, works fine


oarepo runtime Q1 2023 release

Tasks

[x] pinned main
[x] tests
[x] release
[x] readme documentation
[x] dev documentation

Required for

[x] NR
[x] MOSBRI

@mesemus can we close this issue?


is it possible to apply many to one only to the selected text?

Good extension!
However, when i run many to one, it would apply to all the space in my doucument. is it possible to apply many to one only to the selected text?
Thanks

Currently the feature is not available.


FunctionProps.layers doesn't actually attach a layer to the function, as opposed to function.addLayers

The layer isn't being attached to the function with a construct defined like this:
const migratorFunction = new Function(stack, 'MigratorFunction', {
	handler: 'src/handlers/custom-resources/migrator/handler.migrate',
	timeout: 60,
	layers: [
		new lambda.LayerVersion(stack, 'MigrationsLayer', {
			code: lambda.Code.fromAsset('layers/migrations'),
		}),
	],
});

However, it works like this:
const migratorFunction = new Function(stack, 'MigratorFunction', {
	handler: 'src/handlers/custom-resources/migrator/handler.migrate',
	timeout: 60,
});

migratorFunction.addLayers(
	new lambda.LayerVersion(stack, 'MigrationsLayer', {
		code: lambda.Code.fromAsset('layers/migrations'),
	}),
);

Layers are not added in dev mode because there's not a good way to support them locally

Layers are not added in dev mode because there's not a good way to support them locally

Assuming the layer resources exist within the repo, is it feasible to create a local env variable that points to the layer?
I created a utility function within my repo to point to the layer resource when the IS_LOCAL env variable is set to true.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


reported internal ip address / docker

Hi!
Piaware reports back the internal IP address and exposes that as a link on the users flightaware page.
https://flightaware.com/adsb/stats/user/ shows the internal IP and port of the local webinterface
under "site information": "Web Interface: View live data (requires local network connection)"
which show something like: http://<IP_ADDRESS>:8080/
Especially with using docker this internal address might not even be accessible.
It would be nice if the behaviour could be set / overwritten.
Like behaviour:
default export any internal address)
manual (configure protocol (http/https), host/ip, port as whatever you like)
auto-docker (get the hosts ip/port if this is even possible

Here's the existing logic: https://github.com/flightaware/piaware/blob/master/package/fa_sysinfo.tcl#L292
What's the equivalent for detecting a docker environment and extracting the right address?

I just went over this, and there's three things you can do, all of them "not recommended" or "not reliable".

Set the docker hostname to the real hostname instead of a container ID, and do a DNS lookup on that. We do this. Not reliable because it assumes you control the docker.
Tunnel it into docker using an environment variable. Not reliable for the same reason.
Exposing the docker management API inside the container, which basically gives you root on the host. Not recommended.

As said, an option to define the behaviour and one to define a manual value for the URI could be a fix.
I agree IPv6 would  most likely be the most handy thing, though a question would be even if you
use v6 in containers if you would expose it there or concentrated at an other point.

Not doing address translation is kind of a religion for v6 promoters.
Anyway, yeh, just making it some parameter or setting you pass to the program to override the ip address is the way to go. That way it doesn't matter if it's docker or some other NAT/PAT system that's hiding the address from you.


Support separated URI params

I implemented following parameters to build URI.

host
port
username
password
database

This pull-requests also support uri option as before, so this PR has perfect backward compatibility.
Benefit
Validation for each options
We can validate each options more detail.
Additionally, if user uses Embulk and something GUI interface together, this option should be more useful.
Possible to adjust connection settings
We can adjust timeout, number of connection, connection lifetime and other options if needed.
This is a part of code to create MongoClient in my PR.
new MongoClient(addresses, Arrays.asList(credential));

This code can be rewriten to as follows.
MongoClientOptions options = MongoClientOptions.builder()
                    .connectTimeout(600)
                    .connectionsPerHost(10)
                    .maxConnectionLifeTime(1000)
                    .build();
new MongoClient(addresses, Arrays.asList(credential), options);

@sakama This separate parameter way only support to connect 1 MongoDB server.
Is this what you intention?
mongoexport accept same way options, but it accepts multiple hosts and replicaset.
Ref: https://docs.mongodb.com/manual/reference/program/mongoexport/#options

@hakobera

This separate parameter way only support to connect 1 MongoDB server.
Is this what you intention?

Make sense. So how about following implementation?
I read JavaDoc of Mongo Driver for Java, it seems possible.
# config.yml
type: mongodb
seeds: # user can use one of seeds or uri fields
- {host: example.com, port: 27017}
- {host: example2.com, port: 27017}
database: my_db
collection: my_collection
username: user
password: pass

// createClientFromParams()
...
List<ServerAddress> addresses = new ArrayList<>();
addresses.add(
    new ServerAddress("example.com", 27017),
    new ServerAddress("example2.com", 27017)
);

if (task.getUsername().isPresent()) {
    MongoCredential credential = MongoCredential.createCredential(
            task.getUsername().get(),
            task.getDatabase().get(),
            task.getPassword().get().toCharArray()
    );
    return new MongoClient(addresses, Arrays.asList(credential));
}
else {
    return new MongoClient(addresses);
}

I implemented my idea I wrote above.
User can use one of seeds or uri fields.
Use seeds
# config.yml
type: mongodb
seeds:
  - {host: example.com, port: 27017}
  - {host: example2.com, port: 27017}
database: my_db
collection: my_collection
username: user
password: pass

Use uri
# config.yml
type: mongodb
uri: mongodb://myuser:mypassword@localhost:27017/my_database
collection: my_collection
username: user
password: pass

@sakama Sorry for late response, almost LGTM for me, but I want to change some points.
For separate config style
Property naming seeds is difficult to remember, just hosts is better.
In my opinion, property nameing is aligned Mongoid which is ORM developed by mongodb.
https://docs.mongodb.com/ruby-driver/master/tutorials/mongoid-installation/#configuration
Align this configuration, multiple hosts configuration style is like:
# config.yml
type: mongodb
hosts:
  - example.com:27017
  - example2.com:27017
database: my_db
collection: my_collection
username: user
password: pass

Do you have strong opinion to use seeds and divide host and port in configuration?
For URI style
In your comment, duplicate username and password.
And your implementation, if uri is defined, username and password properties are not used. So this is typo in a comment or wrong implementation?
# config.yml
type: mongodb
uri: mongodb://myuser:mypassword@localhost:27017/my_database
collection: my_collection
username: user
password: pass

@hakobera
Thanks for your review.
For separate config style

Do you have strong opinion to use seeds and divide host and port in configuration?

I have no special preference for option name seeds. So I will change option name to hosts.
As for port number, I prefer - {host: example.com, port: 27017} style.
Because it's easy to validate port number when people uses with other system(UI, script, workflow engine).
For URI style

this is typo in a comment or wrong implementation?

Sorry this is a typo.
As you thinking, both of username and password are not used when uri option is used.
So one of following definition is correct.
# config.yml
type: mongodb
uri: mongodb://myuser:mypassword@localhost:27017/my_database
collection: my_collection

# config.yml
type: mongodb
hosts:
...
database: my_database
collection: my_collection
username: user
password: pass

@sakama

As for port number, I prefer - {host: example.com, port: 27017} style.
Because it's easy to validate port number when people uses with other system(UI, script, workflow engine).

OK, I have no strong opinion of that, so please go ahead to use {host: example.com, port: 27017} style.

@hakobera OK. I changed username to user.
Travis CI failed due to Failed: error connecting to db server: no reachable servers.
It seems mongoimport command failed to connect to Mongo servers.
But embulk successfully connect to them.
I think Mongo Server didn't woke up yet when mongoimport command was executed.
The command "./gradlew package" exited with 0.
3.53s$ mongoimport --db $MONGO_DATABASE --collection $MONGO_COLLECTION --type json --drop src/test/resources/my_collection.jsonl
2016-08-22T02:21:42.748+0000	[........................] my_database.my_collection	0B/1.38KB (0.0%)
2016-08-22T02:21:43.263+0000	[........................] my_database.my_collection	0B/1.38KB (0.0%)
2016-08-22T02:21:43.263+0000	Failed: error connecting to db server: no reachable servers
2016-08-22T02:21:43.263+0000	imported 0 documents

@sakama https://github.com/hakobera/embulk-input-mongodb/pull/27 fix the issue.
It is already merged into master.
So merge master or rebase.

@hakobera Thank you for fixing.
I rebased and CI passed successfully!

@sakama I released embulk-input-mongodb v0.5.0 which include this improvement.

👍


fix) run cmd output differs from events cmd output #82

approved

Codecov Report

Merging #94 into master will increase coverage by 3.96%.
The diff coverage is 80.9%.

@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
+ Coverage    71.7%   75.67%   +3.96%
==========================================
  Files          50       52       +2
  Lines        1428     1443      +15
==========================================
+ Hits         1024     1092      +68
+ Misses        370      317      -53
  Partials       34       34

Impacted Files
Coverage Δ

pkg/node/core/core.go
100% <ø> (ø)
:white_check_mark:

util/containerprovider/docker/stdOutWriter.go
31.25% <ø> (ø)
:white_check_mark:

pkg/node/tcp/getEventStreamHandler.go
11.11% <ø> (ø)
:white_check_mark:

util/containerprovider/docker/stdOutLogger.go
53.57% <ø> (ø)
:white_check_mark:

util/containerprovider/docker/stdErrLogger.go
53.57% <ø> (ø)
:white_check_mark:

pkg/node/core/containerCaller.go
100% <100%> (ø)
:white_check_mark:

pkg/node/core/getEventStream.go
100% <100%> (ø)
:white_check_mark:

pkg/node/core/opCaller.go
100% <100%> (ø)
:white_check_mark:

util/pubsub/eventRepo.go
100% <100%> (ø)

util/pubsub/utils.go
50% <50%> (ø)

... and 4 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 db176c9...b85e57e. Read the comment docs.


feat: support Yarn PnP

What's the problem this PR addresses?

Gridsome doesn't work with Yarn PnP.
Gridsome is passing loaders to webpack using their name instead of absolute path
Gridsome has some undeclared dependencies

Fixes https://github.com/gridsome/gridsome/issues/551 (cc @Js-Brecht)
Closes https://github.com/yarnpkg/berry/issues/1266 (cc @borisdiakur)
How did you fix it?

Use require.resolve for webpack loaders (https://github.com/yarnpkg/berry/tree/eff8ffd607242d80adbeaa05ebf5453a68dd299b/packages/yarnpkg-doctor#no-unqualified-webpack-config)
Add undeclared dependencies (https://github.com/yarnpkg/berry/tree/eff8ffd607242d80adbeaa05ebf5453a68dd299b/packages/yarnpkg-doctor#no-unlisted-dependencies)
Resolve user defined plugins relative to the context (gridsome.config.js)
Add https://github.com/arcanis/pnp-webpack-plugin (the only PnP specific thing in this PR)
Added an E2E test to make sure all dependencies are declared and required correctly

I noticed the e2e test passed successfully. Great job @merceyz :) Do you think this PR is ready to be merged?

Do you think this PR is ready to be merged?

Yup, it's ready

Any news on this great feature ?

@merceyz Do you have some time to resolve the conflict and update this PR? I would like to rebase-merge these commits to preserve the individual changelogs for each package. Sorry for taking this long to get back to you...


Fix the bar sliding up on an invalid captcha

change all positions to fixed
This fixes the bar sliding up on an invalid captcha
Stuff like this: https://puu.sh/vhY6v.png

Doesn't this break mobile?

nope, tested it on mobile. In our case position:fixed vs position:absolute shouldn't even be a difference as the webapp doesn't scroll


Try to generate more meaningful documentation from the generated CRUD routes

Created a new ticket here: https://github.com/nestjsx/crud/issues/618
Waiting for response

Response was received, bump 🚀

This was solved in https://github.com/chocolat-chaud-io/stator/pull/144


Generate module command adds route path to wrong line

🐞 Bug report
Command (mark with an x)

- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [x] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Description
ng generate module ranking --route ranking --module app.module command is working very well. But if there is a data property to routes in app-routing.module. The command add route path to wrong line.
🔬 Minimal Reproduction
ng new example-app --routing
ng generate module ranking --route ranking --module app.module
Add data property to ranking route, then my app-routing.module.ts like below
import { NgModule } from '@angular/core';
import { Routes, RouterModule } from '@angular/router';

const routes: Routes = [
  {
    path: 'ranking',
    loadChildren: () => import('./ranking/ranking.module').then(m => m.RankingModule),
    data: { path: 'test', name: 'Test' },
  },
];

@NgModule({
  imports: [RouterModule.forRoot(routes)],
  exports: [RouterModule],
})
export class AppRoutingModule {}

After, run the following command
ng generate module home --route home --module app.module
It adds my home route path to the wrong line like below:
const routes: Routes = [
  {
    path: 'ranking',
    loadChildren: () => import('./ranking/ranking.module').then(m => m.RankingModule),
    data: { path: 'test', name: 'Test' },
    { path: 'home', loadChildren: () => import('./home/home.module').then(m => m.HomeModule) },
  },
];

Can you please provide the output of ng version?

Can you please provide the output of ng version?

Sure,
Angular CLI: 8.1.2
Node: 10.16.0
OS: darwin x64
Angular: 8.1.2
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Package                           Version
@angular-devkit/architect         0.801.2
@angular-devkit/build-angular     0.801.2
@angular-devkit/build-optimizer   0.801.2
@angular-devkit/build-webpack     0.801.2
@angular-devkit/core              8.1.2
@angular-devkit/schematics        8.1.2
@ngtools/webpack                  8.1.2
@schematics/angular               8.1.2
@schematics/update                0.801.2
rxjs                              6.4.0
typescript                        3.4.5
webpack                           4.35.2


Analyse log messages about missing TLDs (JSPUI leftovers?)

Describe the bug
Several log messages about missing TLDs are generated when running DSpace 7.3 (pre-release) backend:
24-Jun-2022 09:44:40.808 INFO [main] org.apache.catalina.startup.HostConfig.deployDirectory Deploying web application directory [/usr/local/tomcat/webapps/rest]
24-Jun-2022 09:44:50.321 INFO [main] org.apache.jasper.servlet.TldScanner.scanJars At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
24-Jun-2022 09:44:50.564 INFO [main] org.apache.jasper.servlet.TldScanner.scanJars At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
24-Jun-2022 09:45:06.150 INFO [main] org.apache.catalina.startup.HostConfig.deployDirectory Deployment of web application directory [/usr/local/tomcat/webapps/rest] has finished in [25,339] ms
24-Jun-2022 09:45:06.151 INFO [main] org.apache.catalina.startup.HostConfig.deployDirectory Deploying web application directory [/usr/local/tomcat/webapps/server]
24-Jun-2022 09:45:15.329 INFO [main] org.apache.jasper.servlet.TldScanner.scanJars At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
24-Jun-2022 09:45:43.289 INFO [main] org.apache.jasper.servlet.TldScanner.scanJars At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
24-Jun-2022 09:45:43.320 INFO [main] org.apache.catalina.startup.HostConfig.deployDirectory Deployment of web application directory [/usr/local/tomcat/webapps/server] has finished in [37,168] ms

This could be an indicator that some JSP leftovers are still present in the DS7 backend code base. Further analysis is required.
To Reproduce
Steps to reproduce the behavior:

Build DS 7.3 backend
Start Tomcat
Check DS application log file (or stdout)

Expected behavior
No messages about missing TLDs in DS application log file.

I might be mistaken, but I think this message comes from Tomcat & there's nothing DSpace can do to affect it: https://stackoverflow.com/questions/40204124/how-to-fix-jars-that-were-scanned-but-no-tlds-were-found-in-them-in-tomcat-9
Basically, Tomcat always scans JARs for TLDs, but if you don't want that behavior in Tomcat 9 (or later) you can configure
tomcat.util.scan.StandardJarScanFilter.jarsToSkip=*.jar

I guess we could consider documenting this setting as part of the Tomcat setup for DSpace.  But, I don't believe there's anything we can change in DSpace to remove the messages from dspace.log

On Mon, Jun 27, 2022 at 12:49:51PM -0700, Tim Donohue wrote:

I might be mistaken, but I think this message comes from Tomcat & there's nothing DSpace can do to affect it: https://stackoverflow.com/questions/40204124/how-to-fix-jars-that-were-scanned-but-no-tlds-were-found-in-them-in-tomcat-9
Basically, Tomcat always scans JARs for TLDs, but if you don't want that behavior in Tomcat 9 (or later) you can configure
tomcat.util.scan.StandardJarScanFilter.jarsToSkip=*.jar

I guess we could consider documenting this setting as part of the Tomcat setup for DSpace.  But, I don't believe there's anything we can change in DSpace to remove the messages from dspace.log

It's annoying, but it's only a little clutter in the logs and a small
delay in startup.
The referenced SO article links to the Tomcat documentation for the
JAR scanner.  If you only want to disable scanning for a specific
webapp. rather than globally, which is probably wise, there's a way to
do it in an individual .
  Implementation is
container-specific.
--
Mark H. Wood
Lead Technology Analyst
University Library
Indiana University - Purdue University Indianapolis
755 W. Michigan Street
Indianapolis, IN 46202
317-274-0749
www.ulib.iupui.edu


Make font family / font weight / font colour configurable

Is your feature request related to a problem? Please describe.
Currently the font is displayed using a serif font in yellow color
Describe the solution you'd like
Allow the option to choose a custom font, may it be a local one or a webfont.
Describe alternatives you've considered
Consider the font config in prompts?
Additional context
Related: #172

That would be really helpful, as the default font does not handle language specific chars (like german "Umlaut").

Well you can already do that... It says how to change the font in the README

No response


[BUG] Atlasmap is not working with nested collection

Description :
Looks like this issue has been raised before. I am working with mapping the field to nested array field. Adding the details below.
Details

Map the source field on the root level to specific index on target level where I have tried using combination of copyTo and ItemAt shown the issue link mentioned below but the solution doesn't seem to be working.
Also there is one more situation, where I have to map 1 source field to multiple target field.
For example: In below example , I have to map field_1 to targetField_1 and targetField_2.
https://github.com/atlasmap/atlasmap/issues/3680

Source :
{
     field_1 : string
     field_2: string
}

Target :
{
  "targetFieldArrayLevel1" : [
    {
      "targetFieldArrayLevel2":[
        {
          "targetField1": "string" // (source field : field_1)
        },
        {
          "targetField2": "string" // (source field : field_2)
        }
      ]
    }

  ]
}

Atlasmap version : 2.5.2
If this is not an issue, Can you provide me details on how can I achieve the results?

Hello. I have debugged this issue in my local. Is there a chance for me to pick this up? @mattrpav

Yes, please provide a PR


Bump to latest version of harp

The current version of harp locked in grunt-harp doesn't work with node v4.0.0. This PR fixed that!

Thanks! I will bump on npm later tonight.

When you do a release could you let me know please? Thanks!

oh, just saw your response. Awesome :)

@shovon mind bumping a new version?

@jansepar Done. You can grab it at @0.18.0-0.0.0.


VS Code insider expending on new virutal desktop macOS

VSCode Version: 1.29.0-insider - 2018-10-30T06:06:04.104Z
OS Version: macOS Mojave 10.14

Steps to Reproduce:
Since last update, can't open VS Code in virtual desktop when I click on expending button.
Does this issue occur when all extensions are disabled?: Yes

@AlexTeixeira we fixed this meanwhile, please update to latest.


⚠ GitLab Server has degraded performance

In 1d20645, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 4541 ms

Resolved: GitLab Server performance has improved in 7c26ab3.

Resolved: GitLab Server performance has improved in d607ee8.


MSVC Compiler Warning

I am getting compiler warnings on Visual Studio version 15.6. It works fine on Visual Studio version 15.9 but we have some people using 15.6. The warning message appears in both latest from master and the latest release.
Warning:

c:\program files (x86)\microsoft visual studio\2017\community\vc\tools\msvc\14.13.26128\include\xutility(2830): error C2220: warning treated as error - no 'object' file generated (compiling source file spirv_glsl.cpp)
3>c:\program files (x86)\microsoft visual studio\2017\community\vc\tools\msvc\14.13.26128\include\xutility(2830): warning C4996: 'std::move_backward::_Unchecked_iterators::_Deprecate': Call to 'std::move_backward' with parameters that may be unsafe - this call relies on the caller to check that the passed values are correct. To disable this warning, use -D_SCL_SECURE_NO_WARNINGS. See documentation on how to use Visual C++ 'Checked Iterators' (compiling source file spirv_glsl.cpp)
3>c:\program files (x86)\microsoft visual studio\2017\community\vc\tools\msvc\14.13.26128\include\xutility(2830): note: see declaration of 'std::move_backward::_Unchecked_iterators::_Deprecate' (compiling source file spirv_glsl.cpp)
3>spirv_cross_containers.hpp(431): note: see reference to function template instantiation '_BidIt2 std::move_backward<T*,T*>(_BidIt1,_BidIt1,_BidIt2)' being compiled
3>

Doesn't really fix it for people not using CMAKE

I tried fixing it with pragmas, but it does not work apparently. I only see MSVC 2015 output through AppVeyor and iterating on these things take forever. Please either add /wd4996 yourself to your custom build system, /D_SCL_SECURE_NO_WARNINGS or propose a PR with the fix.


Make bin/setup not complain about packagecloud

cc @kahlouie

LGTM :+1:

in case someone else deploys next, this was not deployed, but was merged for local setup ease.


REQUEST: New macro @CurrentIterationPath ?

Every sprint we create a product backlog item, where we apply a One-Click action that adds several default Tasks to this backlog item. However, the iteration path is not the same as the parent backlogitem. We would like the option to add a related child-item and change the value of the iteration path to the current sprint. (Cannot set it unfortunately in the work item template, used to create these new items)
Maybe add this field tot the 'Add new linked work item'-Action or as a usable macro in the 'Change field' action.

I second that, this would be really helpfull. Now I still have to update the iteration for each task.


New functionality - Automatic NoStatus parameter

I think it’s possible to avoid the use of noStatus and let the plugin choose if a device is available with a status or not.
It can be difficult for wired device, but this question as to be checked

Added in v1.8.0

Added in v1.8.0


bugfix: fix ipc-shutdown emission + update example

really fix #42: emit 'ipc_shutdown' instead of 'ipc-shutdown'
also update command-on-exit example

Seems to work :+1:


APIM does not honour the context template provided

Description:
Create an apictl project as attached
oauth2.zip
It has following met information in api.yaml file
id:
  providerName: admin
  apiName: Petstore
  version: 1.0.0
type: HTTP
context: /petstore/v1
contextTemplate: /petstore/v1

The import the project to the APIM. Then publish the api from lifecycle tab without any other change. Then API will be expose with the context /petstore/v1, which is correct
Then do some modification to API and re publish it. Then API will be exposed with /petstore/v1/1.0.0.
Seems like APIM is appending the version to the context irrespective of the context template.
Steps to reproduce:
Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

@Rajith90 shouldn't we specify where the version is in the context template?
And, if the version of the API is v1, any particular reason the version is mentioned as 1.0.0?
IMO, the correct one should be:
id:
  providerName: admin
  apiName: Petstore
  version: v1
type: HTTP
context: /petstore/v1
contextTemplate: /petstore/{version}

Can we check the above one?

Issue here is in microgateway we do not append version to the context. The open API can contain the version attribute , but thats not appended to context to which api getting extended.  That is why when creating the apictl project we do not include  the version in the context template. So apim should take a look at the context template and if it does not contains version then it should not modify the context

This is possible by making defaultVersion: true. When creating the project for microgw APIs, we need to use that.


Add a VCR module who intercepts, record and replay API requests

Add a VCR module inspired by https://github.com/linkedin/sepia.
When imported, this module will be activated by the presence of the VCR_MODE environment variable:

record: external call to API made with fetch will be executed and the the response saved to the filesystem
playback: responses to API calls will be read from the filesystem and an error will be thrown if the answer is missing

This module save the API request responses to the file system and identifies them by using a hash of the input parameters (url + verb + body).
Unfortunately I couldn't commit the tapes to LFS because it does not works with forks, there is 2 solutions:

merge this PR and then open a new one with the tapes (@coyotte508 you just need to runVCR_MODE=record pnpm test and commit the tapes)
give me temporarily write access to the repo ("With great power comes great responsibility" would say uncle ben and so I)

(Previous obsolete PR https://github.com/huggingface/huggingface.js/pull/105)
fix https://github.com/huggingface/huggingface.js/issues/91 and https://github.com/huggingface/huggingface.js/issues/67

Contributors would be unable to git push w/ LFS if they add new inference endpoints.
I don't think we test the exact content of the binary responses, so maybe when a blob/buffer is returned we can just store an empty array?
Then we don't need to use LFS for the tapes.

I don't think we test the exact content of the binary responses, so maybe when a blob/buffer is returned we can just store an empty array in the tape?

I will update the tape accordingly :+1:

Hm it seems that tapes are now stored with Git LFS (I don't understand!)
Also the lint file was reverted somehow? (using test:ci which was removed since).
Thinking about it, if we want to run the tests on browser where we can't use node:fs to read the tapes, maybe we can store all the tapes in a single json file? Then import it. It should work both in a browser & node context. (might need to change tsconfig.json to allow importing json directly).

I think I'm almost good with this one, I just have a problem with one of the test because I get an Error overloaded with pnpm test -- -t "questionAnswer"
I will try again later (I'm already using an auth token)


'terminus site info' would nicer not to require --nocache after site creation

[bwood@mbp ~]$ grep -i version ~/code/drupal/bwood/cli/php/terminus.php
define( 'TERMINUS_VERSION', '0.3.2-beta');

[bwood@mbp ~]$ terminus sites create --product=21e1fada-199c-492b-97bd-0b36b53a9da0 --name=ob7-t032b-01 --label=ob7-t032b-01 --org=430c1947-354f-459a-8755-f38293aef105
Creating new Drupal 7 installation ...
Working .............................
Success: Pow! You created a new site!

I need the site uuid...
[bwood@mbp ~]$ terminus site info --site=ob7-t032b-01
Error: Exception thrown - No site exists with this name

What...? Oh maybe I need:
[bwood@mbp ~]$ terminus site info --site=ob7-t032b-01 --nocache
+----------------+---------------------------------------------------------------------------------------------+
|
+----------------+---------------------------------------------------------------------------------------------+
| holder_id      | a5814357-ecde-46aa-af6e-82336848c660                                                        |
| name           | ob7-t032b-01                                                                                |
| created        |<PHONE_NUMBER>                                                                                |
| framework      | drupal                                                                                |
| holder_type    | user                                                                                |
| service_level  | free                                                                                |
| purchase_date  |<PHONE_NUMBER>                                                                                |
| upstream       | git://github.com/pantheon-systems/drops-7.git, 21e1fada-199c-492b-97bd-0b36b53a9da0, master |
| owner          | a5814357-ecde-46aa-af6e-82336848c660                                                        |
| last_code_push | 2015-01-22T00:50:25,                                                                        |
| preferred_zone | chios                                                                                |
| id             | 673f2411-8107-4f70-8b11-2edcabefe60c                                                        |
+----------------+---------------------------------------------------------------------------------------------+

Some people might not think to try --nocache.  If it's got to be required, it would help to output a message about this in the 'sites create' output.

--nocache actually seems to clear the cache.  After I use it once I don't need it again.  I would expect --nocache` to simply bypass the cache for this one command.
Maybe I missed it, but I don't see a terminus "cache clear" command.  That would be helpful.

The site create function should be clearing the cache. I'll fix that. There is a clear cache function terminus cli clear-cache

Ah, I didn't notice the cli subcommands.
Another thing that might help would be something like terminus help --all which would spit out all commands and subcommands. If there was another switch like --raw (or something) which disabled the system pager (help shows up in less for me) I could do terminus help --all --raw | grep cache and find the command I need.

Closing this one since the issues raised have been either handled or raised elsewhere.


added changes related to crypto-server

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
spritteego, @Yibarr, @ayouroukou, @RuChiang, @CONVORDIV, @dustinltran, @lbinh2510, @naheller, @niclaz, @SinaSafari, @BramSikkens, @wwjjqq, @0xcryptodad, @Tonypote33, @saparaloda01, @yuyaxiong, @SorcererGray, @yaodianmi, @KarenWang7, @bscottdavis, @yusufunlu, @pbk0, @liu-shilong, @judasnow, @rixwoodling, @cmyk, @arejula27, @davidhpearce, @shukwong, @allala0, @vitye, @navono, @shern91, @emann-u-el, @shuzhang1989, @seeseesky, @onlyice, @S-Voss, @astaswastika, @owlinstack, @shancci, @HemersonCarlos, @DainisGorbunovs, @asfman, @betterlife2022, @Pradisha, @tuan9x, @Dwyte, @vicken, @Objective-Joey


Publish-BCContainerApp not working headless because of dependencies

PLEASE DO NOT INCLUDE ANY PASSWORDS OR TOKENS IN YOUR ISSUE!!!
Describe the issue
I'm feeling really stupid, but I haven't found documentation about this yet.  I'm hoping someone can help.  When I publish a PTE on a fresh Docker image on my test machine, I'm getting a request asking if I want to install Extension & Dependencies.  This won't work with a pipeline that I'm trying to set up to automatically keep my testing machine up to date with the latest images.
Scripts used to create container and cause the issue
 publish-bccontainerapp -containername "myDemo" -appfile '.\NameOfMyApp.1033.app' -credential $mycreds -skipVerification -install -upgrade -sync

Full output of scripts
Publishing C:\ProgramData\BcContainerHelper\Extensions\myDemo\<a_tenant_guid>\NameOfMyApp.1033.app Synchronizing NameOfMyApp on tenant default
Installing NameOfMyApp on tenant default                                                                                MicrosoftDynamicsNavServer$BC/default
Dependencies found without -Force specified. \r\n Do you want to install Extension & Dependencies?
[Y] Yes  [N] No  [?] Help (default is "Y"):

Screenshots

Additional context

does it happen all the time?
Answer: it only happens when I am installing on a fresh instance, obviously because the dependencies are already installed
did it use to work?
Answer: not on a fresh instance

I'm 100% sure that it's me not understanding something, but I haven't found documentation or any setting that allows me to do this, and the -Force setting is not propagated through the function, so it tells me to stuff myself when I try to specify -Force.
How do I get around this?  Thanks in advance.

Curiousity...  Has this been replicated by anyone else?  I ask because 22.5 is coming out and it would be nice to not have to manually update my testing machine.

Sorry for this - I never managed to make a repro.
Any chance you can supply me with the App File to repro this?
Also, what is the dependency from the app?
Thanks

I hope you meant app.json?  It's attached (with minor modifications.)
modified app.json.txt

And what version of Business Central is your container?
Can you dump the full log of container creation and the error you get from Publish-BcContainerApp?

It'll take me a bit to get that.  Just letting you know I've received this and will update when I get there.  Thanks.

Hey there.  I've extracted the script that I use and done it step by step to ensure that I capture the output.  Here is it, in .TXT form, and I've taken out my client's name (replaced with %%NAMEOFMYCLIENT%%) for privacy reasons.
Step by step.txt
The BC Container Helper version is: 5.0.5

Container Creation Log.txt
The container version is:
22.5.59966.60395 (as taken from the Docker log.)
The full container creation log is also attached.
Again, to be clear, if I hit "yes" here, things progress fine.  But that doesn't help, because I'm trying to use this with a pipeline.  It says that -force wasn't specified, which I'd love to be able to do, but your function doesn't pass that through...
Anyway, thank you very much for looking into this.
-J

And also, further to my last.  It only does this the FIRST TIME I install my app on a fresh machine.  Every time now it'll work just fine until I update again.  And while that's a workaround, I want to be able to update my test machine off-hours in an automated way.

Incidentally, my workaround has now stopped working entirely.  Do you have an idea when the "ships in a future version" might become the "current version"?  👍
Thank you for this amazing tool!

If you download the prerelease version of BcContainerHelper - it should work.

I'm running 5.0.6-preview1070 and no dice, the -Force parameter still comes up with "a parameter cannot be found that matches..."  Is that the wrong version?  It seems to be the latest...  Thanks.
PS it DOES seem like the PR isn't done yet?  https://github.com/microsoft/navcontainerhelper/pull/3184
Or am I misreading this?  (I'm an Azure DevOps guy, not a GitHub guy.)

My bad - you are right - it wasn't approved - will ask for approval right away

Preview should be released now - containing this fix.

Just checked, this works beautifully in preview-1071.  Thank you!  Don't know what your workflow is, but this issue can be closed.  Thanks again!


ENH: extend new_type support

Summary by Sourcery
Enhance sequence loading functions to support new_type with annotation databases and add tests for genbank format detection and sequence loading with annotations.
Enhancements:

Extend support for new_type in sequence loading functions by adding annotation database handling.

Tests:

Add tests for genbank format detection and sequence loading with annotations.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

25 of 26 (96.15%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 91.935%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/cogent3/init.py
20
21
95.24%

Totals

Change from base Build<PHONE_NUMBER>0:
0.001%

Covered Lines:
32410

Relevant Lines:
35253

💛  - Coveralls


Rename timestamp to time for consistency

time is shorter. While the query results return time, write endpoint expects timestamp for points.

Yep, we even talked about this internally yesterday. We'll be changing to time, but we may keep timestamp in there as an undocumented one. Have to check with a few early partners that this might cause an issue with.

@pauldix -- this is a much simpler code change if we don't have to support timestamp because we can just use straightforward Go marshalling. Go doesn't allow a field to have two possible keys in the JSON.
Please check with the partners when you get a chance.

upvote

@pauldix is it ok to only have time and not support both time and timestamp?  Just checking if we got an answer on it so we can update this ticket.

checking in if this will happen

Yes, it's going to happen.

cool, waiting for this to drop.


FileNotFoundError: [Errno 2] No such file or directory: 'sam_pt\sam_vit_h_4b8939.pth'

way am i getting this error any buddy knows the cause full error
"  File "gradio_app.py", line 351, in
fire.Fire(run_demo)
File "C:\Users\elio\anaconda3\envs\venv_wonder3d\lib\site-packages\fire\core.py", line 141, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "C:\Users\elio\anaconda3\envs\venv_wonder3d\lib\site-packages\fire\core.py", line 475, in _Fire
component, remaining_args = _CallAndUpdateTrace(
File "C:\Users\elio\anaconda3\envs\venv_wonder3d\lib\site-packages\fire\core.py", line 691, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "gradio_app.py", line 268, in run_demo
predictor = sam_init()
File "gradio_app.py", line 67, in sam_init
sam = sam_model_registrymodel_type.to(device=f"cuda:{_GPU_ID}")
File "C:\Users\elio\anaconda3\envs\venv_wonder3d\lib\site-packages\segment_anything\build_sam.py", line 15, in build_sam_vit_h
return _build_sam(
File "C:\Users\elio\anaconda3\envs\venv_wonder3d\lib\site-packages\segment_anything\build_sam.py", line 104, in _build_sam
with open(checkpoint, "rb") as f:
FileNotFoundError: [Errno 2] No such file or directory: 'sam_pt\sam_vit_h_4b8939.pth'"

Hello, after I downloaded the segmentation model according to the end of the error and placed it in the project directory, the error no longer prompts, this is the download address of the segmentation model: https://huggingface.co/spaces/abhishek/StableSAM/blob/main/sam_vit_h_4b8939.pth


Step 2 under "To enable and view the trace log" is wrong

Just try it...

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

ID: 073ab477-ac4c-9951-9d50-9d979176cff1
Version Independent ID: 384cd6f7-2249-3f33-9b9e-1c6abeae69cb
Content: AD FS Troubleshooting - Auditing Events and Logging
Content Source: WindowsServerDocs/identity/ad-fs/troubleshooting/ad-fs-tshoot-logging.md
Product: windows-server
Technology: identity-adfs
GitHub Login: @billmath
Microsoft Alias: billmath

#assign:IngridAtMicrosoft

#assign:xelu86

Hey @ThomasRapidis,
Thanks again for contacting Microsoft. We value your feedback.
A documentation change has been made to better reflect the steps for the mentioned section and will be live after the next business day. We will now be closing this thread.
Thank you and have a most pleasant day. #please-close


Add tonic example

https://tonicdev.uservoice.com/knowledgebase/articles/765846-how-do-i-customize-the-example-for-my-npm-package

The example is now showing up, but it has a weird issue where the last example is failing.  The name regex seems to be swallowing the \


Cleanup old decommissioned agents

Assuming that if an agent is decommissioned (not stuck decommissioning) for a long time, it should be safe to clean it up the same way as dead/missing agents.
cc - @ssalinas

I don't know that this is actually safe. I think we also need the condition that we have not received offers from it in that amount of time as well. Currently, if an agent is still active with the mesos master, but decommissioned for us, we are continually receiving offers. So, if we were to delete it from our agents list -> get a new offer, we will actually end up adding it back as active. There is a maintenance mode in mesos master (old issue https://github.com/HubSpot/Singularity/issues/948) that we could integrate with to stop offers coming in when decomissioned

🚢


Dont find the index.tlp

Hi i follow your instructions and delete an index.ctp and replace for an index.tlp but dont recognize, what i have to do?
Do you have a project example in github maybe?

Make sure it is .tpl and not .tlp

@estvmachine I think you can close this issue now. Also the recommended extension is .twig now. That should also make such typos less likely ;)

+1 .twig


feat: 🎸 launch a retry for a lot of errors

also increase the resources. Also, fix the admin app

Let's hold this commit until https://github.com/huggingface/huggingface_hub/issues/1503 and https://github.com/huggingface/datasets-server/issues/1346 are fixed

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Closing


Links to Remix Source Repo and Remix Roadmap error

Visit https://remix.run/dashboard/support
Click on Visit Remix Repo

Same happens for Visit Remix Roadmap

Not applicable anymore


Unable to dump

when I used the juicer dump, I received the error report as below, which only happened when dumping chr1 matrix at resolution of 10K with KR normalization. It works well with other chromosomes, resolutions or normalizations in the same hic file.
The parameters I used for juicer dump:  observed KR  1 1 BP 10000.
The parameters I used for juicer pre: -q 10  hg38
Besides I also tried specifying the chr and resolution: q 10 -c 1 -r 10000 hg38
But I cannot still deal with this problem.
Unable to dump
java.io.IOException: Normalization missing for: 1_1_BP_10000
at juicebox.data.DatasetReaderV2.readNormalizedBlock(DatasetReaderV2.java:731)
at juicebox.data.MatrixZoomData.dump(MatrixZoomData.java:633)
at juicebox.tools.clt.old.Dump.dumpMatrix(Dump.java:265)
at juicebox.tools.clt.old.Dump.run(Dump.java:493)
at juicebox.tools.HiCTools.main(HiCTools.java:97)

This isn't a bug and should instead be posted at aidenlab.org/forum.html
On Tue, Nov 12, 2019 at 9:36 PM ShawnKwokgxx<EMAIL_ADDRESS>wrote:

when I used the juicer dump, I received the error report as below, which
only happened when dumping chr1 matrix at resolution of 10K with KR
normalization. It works well with other chromosomes, resolutions or
normalizations in the same hic file.
The parameters I used for juicer dump: observed KR 1 1 BP 10000.
The parameters I used for juicer pre: -q 10 hg38
Besides I also tried specifying the chr and resolution: q 10 -c 1 -r 10000
hg38
But I cannot still deal with this problem.
Unable to dump
java.io.IOException: Normalization missing for: 1_1_BP_10000
at
juicebox.data.DatasetReaderV2.readNormalizedBlock(DatasetReaderV2.java:731)
at juicebox.data.MatrixZoomData.dump(MatrixZoomData.java:633)
at juicebox.tools.clt.old.Dump.dumpMatrix(Dump.java:265)
at juicebox.tools.clt.old.Dump.run(Dump.java:493)
at juicebox.tools.HiCTools.main(HiCTools.java:97)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/142?email_source=notifications&email_token=AAK2EW5WPPINDB35CH5OXZLQTNR2HA5CNFSM4JMOMB22YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HY332JQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EW4GROLZPCFF7M5TI7DQTNR2HANCNFSM4JMOMB2Q
.

--
Neva Cherniavsky Durand, Ph.D.
Assistant Professor, Aiden Lab
www.aidenlab.org


Update fs2-io to 2.5.6

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2", artifactId = "fs2-io" } ]

labels: test-library-update, semver-patch

Superseded by #71.


Progress Bar Animation Triggered in Next.js Link with Nested Button Despite preventDefault and stopPropagation

I've encountered an issue using Next.js where nesting a button inside a Link component and attempting to stop the event propagation and default behavior does not prevent the progress bar animation from being triggered. This occurs even when the URL does not change as a result of the button click.
I have set up an example to demonstrate this issue: CodeSandbox Example

// Next.js Link wrapping a custom component with a button
<Link href="/test">
  <Box />
</Link>

// Box component with a button that should prevent link navigation
"use client";

export default function Box() {
  return (

some product box

      <button
        onClick={(e) => {
          e.preventDefault();
          e.stopPropagation();
          alert("added to cart");
        }}
      >
        Add product to cart
      </button>

  );
}

For now I use data-disable-nprogress on those Links.

Upgrade to 2.3.10 and use data-prevent-nprogress={true} on the item to be disabled :
<button
   data-prevent-nprogress={true}
   onClick={(e) => {
      e.preventDefault();
      e.stopPropagation();
      alert("added to cart");
   }}
>
   Add product to cart
</button>

This should fix the problem.

Thank you, it works great now.


Support the auth type: v3applicationcredential

Request
Our openstack playbooks to create/delete a VM use, to access the platform, the auth mode password. While this approach is not bad, it forces us to pass the auth parameters to every task where we access the platform and also to use the snowdrop PSI Team password.
Such a user and password should be mainly used by the Operator in charge of the management of our RHOS PSI instance instead of being used too to create/delete VMs using Ansible playbook.
This is why it should be better to use as auth_type : v3applicationcredential as such a mode supports to use an application id/secret associated to a role, can be rotated, etc. See documentation for more information:

Note: The v3token mode could also be an interesting alternative but it is not well documented, suffer from many issues such as: Service Catalog is empty, ansifact_facts not supported, etc if you dont use exactly version x.y.z of Ansible + openstackdsk

If e decide to use it as mod, then several tasks will be required as:

[ ] Review and document how to generate the Id/Secret, rotate it and what lifecycle should be
[ ] Select and define the role to be used by the application credential
[ ] Refactor the playbooks to use it as auth_mod
[ ] Log on one time instead of having to do it for every task. See: discussion around module_default and group

This is how I implemented the v3applicationcredential authentication.
    - name: List Fedora images
      openstack.cloud.image_info:
        auth_type: "v3applicationcredential"
        auth:
          auth_url: "https://rhos-d.infra.prod.upshift.rdu2.redhat.com:13000"
          application_credential_id: "loremipsumdolorsitametconsecteturadipiscingelit"
          application_credential_secret: "loremipsumdolorsitametconsecteturadipiscingelitseddoeiusmodtemporincididuntutlaboreetdoloremagnaaliqua"
        properties:
          os_distro: "fedora"
      register: image_info_result

@cmoulliard

If you want to define a variable with the authentication information so it is reused you can also do this.

What about this where we dont have to pass or override some auth parameters ?
- hosts: localhost
  module_defaults:
    group/openstack.cloud.openstack:
      auth_type: "v3applicationcredential"
      auth:
        auth_url: "https://rhos-d.infra.prod.upshift.rdu2.redhat.com:13000/v3"
        application_credential_id: "9c920....e8"
        application_credential_secret: "Kfi4....hRQ1JYQ"

  tasks:
    - name: List images
      openstack.cloud.image_info:
        filters:
          os_distro: "fedora"
      register: image_info_result

@jacobdotcosta

That can only be achieved either by using environment variables or using a named cloud.
From the Ansible docs:
Dictionary containing auth information as needed by the cloud’s auth plugin strategy.
For the default password plugin, this would contain auth_url, username, password,
project_name and any information about domains (for example, user_domain_name or
project_domain_name) if the cloud supports them. For other plugins, this param will
need to contain whatever parameters that auth plugin requires. This parameter is not
needed if a named cloud is provided or OpenStack OS_* environment variables are present.

IIUC, named cloud requires having a local clouds.yaml file with the authentication information. Something like:
clouds:
 cloud_name:
    auth:
      project_name: "XXXXXXXXXXXXXXXXXX"
      username: "XXXXXXXXXXXXXXXXXX"
      password: "XXXXXXXXXXXXXXXXXX"
      user_domain_name: "XXXXXXXXXXXXXXXXXX"
      project_domain_name: "XXXXXXXXXXXXXXXXXX"
      auth_url: "XXXXXXXXXXXXXXXXXX"
    region_name: "XXXXXXXXXXXXXXXXXX"
    interface: "XXXXXXXXXXXXXXXXXX"
    identity_api_version: 3

That can only be achieved either by using environment variables or using a named cloud.

I dont follow you here. Where are you looking to use clouds.yaml fil ?

I was checking for options where we don't need to pass or override the auth parameters on the playbook and roles, and those are the only 2 options I see.

I was checking for options where we don't need to pass or override the auth parameters on the playbook and roles, and those are the only 2 options I see.

I vote to use as no override is needed, can be declared one time = init step, etc
module_defaults:
    group/openstack.cloud.openstack:
...


Updated plotting for readability

Altered plotting for trace, pairwise, and histograms so that axis and tick labels are always readable. Also added special-case parameter names used in BART (kappa, gamma_1, gamma_2, alpha, beta) are changed to their proper formatting.

Thanks Michael, but the parameter reformatting are specific to BART/transit rather than something general as to go into MC3.
The resizing of fonts and panels does belong here I think. So that I would take into MC3 (although from experience, all sizing issues really depends on the specific matplotlib version, as the default fonts have changed a lot over the past years).


Fix Securitydefinitions having wrong name

This prevents the name field being used as overall name of the definition itself.
fixes #616

This PR is now ready for review.

This PR is now ready for review again.

Good work @Postremus . Merging now.


Needs punc: COMMAND should be an executable, a chained or a quoted command will not work.

File: engine/reference/commandline/exec.md
I think this line needs some punctuation:
COMMAND should be an executable, a chained or a quoted command will not work.

Thanks for sharing this issue. This particular content is maintained in an upstream repo. If the issue is still applicable, please help create an issue in the repo at https://github.com/docker/cli/tree/master/docs/reference/commandline


Change node: problem with JSONata and $lookup when used with a chain (~>) operator

What are the steps to reproduce?
Send messages in quick succession (such as with a split node) to a change node configured to do a $lookup in JSONata using the chaining operator. (see example flow)
What happens?
The change node emits this error:
"Invalid JSONata expression: Argument 2 of function "lookup" does not match function signature"

And from that point any message sent to it won't work.
What do you expect to happen?
I expect this to work. It broke in Node-RED 0.19.0, worked prior to this.
Please tell us about your environment:

[ ] Node-RED version: 0.20.5
[ ] node.js version: 8.15
[ ] npm version: 6
[ ] Platform/OS: MacOs
[ ] Browser: Safari

[{"id":"d10dc7f.f9241b8","type":"change","z":"26437d6.f72e102","name":"lookup with ~>","rules":[{"t":"set","p":"payload","pt":"msg","to":"{\t   '1': 'goat',\t   '2': 'cheese'\t} ~> $lookup($string(payload))","tot":"jsonata"}],"action":"","property":"","from":"","to":"","reg":false,"x":600,"y":380,"wires":[["3a429d39.22976a"]]},{"id":"3a429d39.22976a","type":"debug","z":"26437d6.f72e102","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"false","x":930,"y":380,"wires":[]},{"id":"ef556b69.b978f","type":"split","z":"26437d6.f72e102","name":"","splt":"\\n","spltType":"str","arraySplt":1,"arraySpltType":"len","stream":false,"addname":"","x":390,"y":380,"wires":[["d10dc7f.f9241b8"]]},{"id":"514ba59c.28eff4","type":"inject","z":"26437d6.f72e102","name":"","topic":"","payload":"[1,2]","payloadType":"json","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":230,"y":500,"wires":[["ef556b69.b978f"]]},{"id":"f48614b5.fef1c8","type":"inject","z":"26437d6.f72e102","name":"","topic":"","payload":"1","payloadType":"num","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":230,"y":180,"wires":[["d10dc7f.f9241b8"]]},{"id":"2214ce8.31041b2","type":"inject","z":"26437d6.f72e102","name":"","topic":"","payload":"[1]","payloadType":"json","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":230,"y":340,"wires":[["ef556b69.b978f"]]},{"id":"684840a.05628c","type":"comment","z":"26437d6.f72e102","name":"Always works","info":"","x":230,"y":140,"wires":[]},{"id":"6fe1d59.c37c2ac","type":"comment","z":"26437d6.f72e102","name":"Breaks the change node forever","info":"","x":290,"y":460,"wires":[]},{"id":"c9969e9d.8c62","type":"change","z":"26437d6.f72e102","name":"lookup without ~>","rules":[{"t":"set","p":"payload","pt":"msg","to":"$lookup({\t   '1': 'goat',\t   '2': 'cheese'\t}, $string(payload))\t","tot":"jsonata"}],"action":"","property":"","from":"","to":"","reg":false,"x":610,"y":600,"wires":[["3a429d39.22976a"]]},{"id":"5f560f46.11214","type":"comment","z":"26437d6.f72e102","name":"Fails","info":"","x":570,"y":340,"wires":[]},{"id":"1e8c4e7e.1f1d12","type":"comment","z":"26437d6.f72e102","name":"Works","info":"","x":570,"y":560,"wires":[]},{"id":"5cf5af58.3d0468","type":"inject","z":"26437d6.f72e102","name":"","topic":"","payload":"[1,2]","payloadType":"json","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":230,"y":600,"wires":[["9ce71d82.5c2e1"]]},{"id":"9ce71d82.5c2e1","type":"split","z":"26437d6.f72e102","name":"","splt":"\\n","spltType":"str","arraySplt":1,"arraySpltType":"len","stream":false,"addname":"","x":410,"y":600,"wires":[["c9969e9d.8c62"]]}]

This is an issue with parallel execution of a jsonata expression that involves ~.
The following code reproduces it in a vanilla JSONata test case:
const jsonata = require("jsonata");

const expr = jsonata("{'1':'goat','2': 'cheese'} ~> $lookup($string(payload))");

expr.evaluate({"payload":1}, {}, function(err,result) {
    console.log("1",err);
    console.log("1",result);
});
expr.evaluate({"payload":2}, {}, function(err,result) {
    console.log("2",err);
    console.log("2",result);
});

As a callback is provided to the evaluate function it is put into async evaluation mode. This allows the second call to evalute to be made before the first has completed.
 But we certainly benefit from being able to do so for other types of expression as we don't have to queue work up.
Think we need some input from @andrew-coleman here.
Is this an issue with jsonata or is it entirely our fault for calling evaluate in parallel?
One workaround would be to inspect the expression for ~> (assuming that is the magic piece causing the issue) and put the node in to a more blocking evaluation mode. That would limit the performance hit to just those sorts of expression.

Would it be any use if you wrapped the invocation of evaluate in a promise?  E.g.:
var jsonataPromise = function(expr, data, bindings) {
    return new Promise(function(resolve, reject) {
        expr.evaluate(data, bindings, function(error, response) {
            if(error) reject(error);
            resolve(response);
        });
    });
};

Then jsonataPromise(expr, {"payload":1}) would return a promise that you could either chain, or use Promise.all() if you want to run them in parallel.
Not sure if that helps

We try to avoid promises on the main message path due to the performance overhead - learnt that the hard way.
We can queue up the evaluations if needed (via promises or otherwise), but I wanted to confirm this error was a natural result of how we were using jsonata and not a bug in jsonata itself.

Sorry, I should have paid more attention to the original bug report.  This is clearly a JSONata bug.  Interestingly, this test case works against jsonata v1.3 - I did something to regress it since then.  Please feel free to raise an issue... https://github.com/jsonata-js/jsonata/issues/new

Version 1.6.5 of JSONata was released with the fix !


Table function documentation is broken

see https://duckdb.org/docs/api/c/table_functions, the function formatting doesn't work

Thanks! I'm travelling at the moment so I will take a look later. In the meantime, I recommend using the older version: https://duckdb.org/docs/archive/1.0/api/c/table_functions.

This has been fixed since, closing as resolved.


fix: remove wl akmod

Pulling in Broadcom fix from Bluefin. See https://github.com/ublue-os/bluefin/pull/2070 and https://github.com/ublue-os/bluefin/issues/1783.
Related Discourse thread: https://universal-blue.discourse.group/t/more-broadcom-wifi-madness-possible-ujust-akmods-script-typo/4836/5

this broke things for certain people, fun @NiHaiden


Quiz App

Describe the project you want to add with tech stack
To add Quiz App made using React.js .
Expected behavior
User has to choose the correct option from the given options.
Screenshots (optional)

@MohitGupta121 @TusharKesarwani @pranjay-poddar
Please assign it to me.


Program:Win32/Wacapew.C!ml detected

Alert " Program:Win32/Wacapew.C!ml " showed when I was downloading the new release. Is it still safe to download?

yes


Create data binding system

The data is too untamed and needs to be bound.
Data binding systems can be used for UI, first and foremost. So a UI element receives some kind of value binding, and then it reads/writes values to that binding.

Made some half-functioning binding system. I'll leave making it fully functioning to the future me.


Bump to 1.7.0

@PaulFurtado @pranav @chaoranxie

Could you tag in git this release ?

https://github.com/hubspotdevops/puppet-nexus/releases/tag/1.7.0


Model Description

Adds a Model Description drop down menu based on which model is selected in Select Model, that is configurable in config.yml.
This is branched from a previous pull request that includes a short description on the page, but it still needs more work. The functionality for that feature is there, but there is no example for it in config.yml, so it will be easy to build upon without releasing it into the main branch.

Delete the commented out code

Delete the commented out code

I deleted the commented out code.


Accessory cog icon does not display on Chrome iOS / iPad OS

Describe The Bug
On Chrome in Win10 in the Accessory page: Mouse-over an Accessory icon fades in a cog icon which can be clicked to show a very useful accessory service values page.
On Chrome in iPad OS in the Accessory page: the cog icon never displays, you have to guess where the right place is to tap and open the accessory service values page.
It would be good to have the cog icon displayed all the time in Chrome on iOS / iPadOS so the user knows where to tap. Especially as the common pointer method is the Fat-Finger
Logs
No response
Config
No response
Homebridge UI Version
homebridge-config-ui-x v4.53.0
Homebridge Version
v1.7.0
Node.js Version
v20.10.0
Operating System
Ubuntu / Debian (or a variant)
Environment Info

[ ] Using Docker?
[ ] Using Hyper-V?
[ ] Using hb-service?

Raspberry Pi Model
Raspberry Pi 4 B

@jsiegenthaler
The fact that the cog does not display is not a mistake but normal operation - does not display on devices with a narrow screen. This is how it is currently programmed.
I suggested that a long press on the accessory should open the modal and in it: device control, below the settings, and even below the button that gives the ability to remove from the cache.

Need to rework the accessory tile. In some ways for consistency, I would like to scrap the long tap, and a single tap brings up any accessory control (with a button in the modal for more info) and if the accessory does not have control within the UI, then immediately show the accessory info.
Or maybe somehow mirroring what HomeKit does now with common accessories like lights and switches where a part of the tile is for control, and another part is for opening accessory info. Although I do not like the 'unknown'-ness of homekit accessory tiles in that it takes a bit of trial and error to know what click will do what.


v6 Slideshow plugin

The slideshow plugin doesn't seem to be inheriting any CSS or JS in the material and foundation templates.

Now in RC1 for foundation (via Slick).  MDL does not support a slideshow option and Slick does not work with MDL in my testing.  So, I will continue to evaluate it for MDL.


LICENSE or COPYING file

It would be super helpful to include a LICENSE or COPYING file in your repo.
https://wiki.creativecommons.org/wiki/CC0_FAQ#May_I_apply_CC0_to_computer_software.3F_If_so.2C_is_there_a_recommended_implementation.3F
Recommends using:
https://creativecommons.org/publicdomain/zero/1.0/legalcode.txt

I hate files :) The CC0 blurb is currently in the code and README. Is there an additional reason to have it in COPYING/LICENSE apart from GitHub parsing it? Automated tools that extract licenses from packages maybe? What's your specific concern?

It's largely to make it clear that you are licensing the project under CC0. Without the LICENSE text (which the majority of human and automated reviewers rely on), it's difficult to allow use of this project in a fast manor without further investigation.

+1 for this, especially considering pkg.go.dev won't show package documentation unless it has an OSI approved license (which unfortunately CC0 is not as explained in the OSI FAQ https://opensource.org/faq#cc-zero).
Still though it's a big help for automated tools to have a LICENSE file.


Pipe: use file mode in forced-log extractor when pipe is stopped to avoid OOM

When pipe is stopped and data is inserted continuously, forced-log extractor will keep queueing tablet insertion events, which may lead to OOM. So we may switch to file mode when pipe is stopped and the queue is reaching capacity.

We will use hard-link based wal pin strategy to avoid issues caused by wal not deleted. Thanks for working for this.


conda-forge installable package

Are there any plans on making a conda-installable package at conda-forge for this project? I think other projects using it will benefit from a conda-forge package.

Related to this - in yesterday's F@H alchemy call @jchodera mentioned the potential need for folks like Perses to be able to grab the data from this repo for their benchmarks in a simple programmatic manner. Having a conda-forge package may not be the full solution here, but having something packaged up in a manner like https://www.mdanalysis.org/MDAnalysisData/ might be useful.

I think this is probably stuck on requiring #18 first


Always on top / Keep this window floating not working

Ray <IP_ADDRESS>
PHP 7.4
Windows 10
The application window no longer stays on top of other windows. Also, clicking on the task bar icon does not bring the window to the front. You can see it flash on top of another app for a split second but it does not stay on top

I just tested this on my Windows machine and it seems to work just fine. Could you please try removing the app from your system, and reinstalling it?
If it still doesn't work, make sure no other apps are interfering with the float-on-top behaviour, and create a new issue with reproduction steps.

Just wanted to provide an update from my end. A reboot didn't fix it but there were some Windows updates that I went ahead and installed, rebooted, and now everything is fine again. I haven't seen the issue happen since I originally posted so not sure what was going on. I have seen other apps act funny like this and after a Windows update things return to normal so...

Thanks for the update and glad to hear you got it working!


[VS Code] An internal WebSocket error occurred.

Error:
An internal WebSocket error occurred. Please see the innerException, if present, for more details. TrackingId:70afc140-c30a-4197-acb1-cb1079f657f5, Address:sb://vsls-prod-ins-asse-private-relay.servicebus.windows.net/b0b1e71b5a7ef9c0252d1a738422747605b2--982734aa-88aa-4b24-9f05-743e93e4d584, Timestamp:6/27/2018 5:06:36 AM
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.335

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.24.1

platformName
Windows

platformVersion
10.0.16299

Hi @toffysoft , Thank you for reporting this. Would you give us some more information ? Is it happened before you start sharing session? Also, it would be great if you attach your logs , so we can investigate further.

@Priya91 to investigate further.

@toffysoft Can you please share your logs..

@toffysoft You can get your logs by running the "Live Share: Export Logs" command in VS Code.  If you could attach them to this issue or email them to<EMAIL_ADDRESS>that would help is understand what might be going on.


fix teardown_hook对response的修改体现在测试报告上 #430

Coverage increased (+0.06%) to 79.769% when pulling 7b9ce278e7c3345e46f52c5379f8a7eb58a9939c on sirfengyu:master into 0d29ad5fd05863e3975e0690a54ff497e2538161 on HttpRunner:master.

Coverage increased (+0.06%) to 79.769% when pulling 7b9ce278e7c3345e46f52c5379f8a7eb58a9939c on sirfengyu:master into 0d29ad5fd05863e3975e0690a54ff497e2538161 on HttpRunner:master.

Coverage increased (+0.06%) to 79.769% when pulling 7b9ce278e7c3345e46f52c5379f8a7eb58a9939c on sirfengyu:master into 0d29ad5fd05863e3975e0690a54ff497e2538161 on HttpRunner:master.


Add Irfan Ansari to Contributors list

Hi  @irfan-ansari-me, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Use "request" instead of "since" for payload

Using "since" will get RequestTImeOut error:
Request Timeout({"error":{"code":"RequestTimeout","message":"Request timeout waiting for the device. You can contact support at https://aka.ms/iotcentral-support. Please include the following information.

So, the body will work for sending commands like the following example:
{
    "request":  "<the value should be sent>"
}

@KuiMing : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

Hi @KuiMing - thanks for proposing this change. I've just checked and agree with your change. Many thanks for pointing this out.
Regards, Dominic
#sign-off


EIP tagging ability needed for CFE

The EIPs created by the BIG-IP module need the ability to be tagged. This would allow a user to meet the Cloud Failover prereqs for AWS which state proper NIC tagging and EIP tagging. Currently a workaround is required to use data objects to retrieve EIP IDs based on public IP outputs. User requires ability to tag EIP during BIG-IP module deployment.
Similar to use case in https://github.com/F5Networks/terraform-aws-bigip-module/issues/22
workaround
Specifically, the secondary IP of the NIC which is associated with the EIP is the value of the tag. The following two tags are required on each EIP.
# Public VIP info
data "aws_eip" "bigip_vip" {
  public_ip = module.bigip.public_addresses["external_secondary_public"][0]
}
data "aws_eip" "bigip2_vip" {
  public_ip = module.bigip2.public_addresses["external_secondary_public"][0]
}

# tagging
resource "aws_ec2_tag" "bigip2_vip_label" {
  resource_id = data.aws_eip.bigip2_vip.id
  key         = "f5_cloud_failover_label"
  value       = var.f5_cloud_failover_label
}
resource "aws_ec2_tag" "bigip2_vip_ips" {
  resource_id = data.aws_eip.bigip2_vip.id
  key         = "f5_cloud_failover_vips"
  value       = "${element(flatten(module.bigip.private_addresses["public_private"]["private_ips"][0]), 1)},${element(flatten(module.bigip2.private_addresses["public_private"]["private_ips"][0]), 1)}"
}

This is an ugly workaround but just showing the tags needed. The failover vips requires the IPs of the secondary NICs as tag values for CFE to function. My workaround is above. If we had ability to apply EIP tags, that would help.

HI @JeffGiroux , we are already made changes to tag external EIP, based on external_nic_failover_tag
https://github.com/F5Networks/terraform-aws-bigip-module/blob/main/main.tf#L63-L68

Per CFE requirements, the NIC and EIP have different tags. The way the fix has been implemented will mean adding all NIC and EIP tags into the new tag variable. That is fine for me…but will those additional NIC mapping tags caused a problem with the EIP? And will the additional EIP VIP up addres tag cause a conflict with the NIC?


convert_image_dtype overflows with low precision floating point dtypes

While working on improving performance of convert_image_dtype in #6795, I found several cases where convert_image_dtype is silently failing for low precision floating point dtypes torch.float16 and torch.bfloat16:
import torch
from torchvision.transforms import functional as F

# torch.{float16, bfloat16} to any integer dtype
image = torch.tensor(1.0, dtype=torch.float16)
print(image, F.convert_image_dtype(image, torch.uint8), F.convert_image_dtype(image, torch.int8))

# torch.{int32, int64} to torch.float16
image = torch.tensor(2**31 - 1, dtype=torch.int32)
print(image, F.convert_image_dtype(image, torch.float16))

tensor(1., dtype=torch.float16) tensor(0, dtype=torch.uint8) tensor(-128, dtype=torch.int8)
tensor(2147483647, dtype=torch.int32) tensor(nan, dtype=torch.float16)

Converting an valid (b)float16 image in the value range [0.0, 1.0] to any integer dtype overflows the computation. This stems from the fact that eps is fixed:
https://github.com/pytorch/vision/blob/7a62a545ce76f43ccc5cfe0009131f7db14ae7b5/torchvision/transforms/functional_tensor.py#L90-L93
This value is simply to large for (b)float16:
>>> image = torch.tensor(1.0, dtype=torch.float16)
>>> image.mul(255 + 1.0 - 1e-3)  # float16 -> uint8
tensor(256., dtype=torch.float16)
>>> image.to(torch.float32).mul(255 + 1.0 - 1e-3)  # float32 -> uint8
tensor(255.9990)
>>> image.mul(255 + 1.0 - 7e-2)  # float16 -> uint8 with adjusted eps
tensor(255.8750, dtype=torch.float16)

The whole point of eps is to be as small as possible to have an even value distribution. See https://github.com/pytorch/vision/pull/2078#issuecomment-613524965 for details.
We could simply make eps dependent on the input dtype in a function similar to
https://github.com/pytorch/vision/blob/7a62a545ce76f43ccc5cfe0009131f7db14ae7b5/torchvision/transforms/functional_tensor.py#L47

Converting a int{32, 64} image to float16 should not be possible since it can't hold the maximum values:
>>> torch.finfo(torch.float16).max
65504.0
>>> torch.iinfo(torch.int16).max  # ok
32767
>>> torch.iinfo(torch.int32).max  # not ok
2147483647
>>> torch.iinfo(torch.int64).max  # not ok
9223372036854775807
>>> torch.finfo(torch.bfloat16).max  # bfloat does not have this issue
3.3895313892515355e+38

We are already raising an error for unsafe float to int conversions
https://github.com/pytorch/vision/blob/7a62a545ce76f43ccc5cfe0009131f7db14ae7b5/torchvision/transforms/functional_tensor.py#L78-L83
so we could simply do the same here.

cc @vfdev-5 @datumbox

I agree with the proposal for guarding against types that are likely to overflow. I suspect that float16 is also very likely to be problematic with many of our F kernels. Perhaps it's worth adding an error for that as well.

Related: https://github.com/pytorch/pytorch/issues/35666 https://github.com/pytorch/pytorch/issues/41527 https://github.com/pytorch/pytorch/issues/66707