Changing scope with an UISwipeGestureRecognizer causes currentPage bug

I've implemented a UISwipeGestureRecognizer on my FSCalendar instance:
swipe = [[UISwipeGestureRecognizer alloc] initWithTarget:self action:@selector(handleSwipe:)];
swipe.delegate = self;
swipe.numberOfTouchesRequired = 1;
[swipe setDirection:UISwipeGestureRecognizerDirectionDown];
 [_calendar addGestureRecognizer:swipe];

-(void)handleSwipe:(id)sender {
    if (_calendar.scope == FSCalendarScopeWeek) {
        [_calendar setScope:FSCalendarScopeMonth animated:YES];
    }
}

This works, but it causes the calendar to go like 10 years in the future.
I've also tried to use:
 [_calendar setCurrentPage:_calendar.currentPage animated:YES];
after setting the scope, but that didn't work.
Do you have any idea what causes this?

Can't just use the scope handle?

I want to be able to just swipe on the calendar and make it go to the Month view. I don't want to use the scope, because of UI choices

Have u implemented -calendar:boundingRectWillChange:animated: method?

Yes I did!

I have a similar problem. When I change from "month view" scope to "week view" scope the calendar changes to previous month automatically. I don't know what is happening.

@WenchaoD do you have any idea?

calendar.scopeGesture.enabled = YES;

Available in development branch;

Merged into master.


Stage 2 slides

Remaining issues:

I can't figure out how to make the formatting work in the mc-radiogroup feedback options in slide 7. The 2nd question has  tags that just get treated as a string. @Carifio24, do you know if there is an easy fix?
We are waiting for Anna to provide a better version of the image on slide 6.
The image layouts need work.  @heywooddogwood, please fix things like images in the same row that have different heights and/or redo the layouts however you see fit.
The equation layouts are not robust with respect to different screen sizes. We forced the orange equation boxes not to run across the whole screen by specifying a column width, but the equations will spill out of the boxes if a user makes the screen sufficiently narrow. I don't know how to make the card lengths more flexible.

Hmm. Github noted some merge conflicts, so I fixed them on the GitHub site for the most part by just accepting the versions coming from “main” instead of PR 94. When I pulled down the merge commit and ran the app on my computer, the spectrum graph is back to the scatter plot instead of the line graph. I think this means I did something wrong in the merge commit? @Carifio24, would you mind taking a look when you get a chance, please? Thanks!

Everything seems to be functioning fine. There are formatting updates to be made, but those will appear in a future branch/PR.

Things to be done:

turn off sliding slide transitions that still occur for several of the slides.
math slideshow container formatting to that from Stage 1 (e.g. header color)
Make text in alerts/colored cards white
Center MathJax equations
Fix width of cards to have margins
Get MC to match format from Stage 1
Additional cleanup as needed


[ADAP-953] [Bug] Materialized Views not correctly dropping

Is this a new bug in dbt-redshift?

[X] I believe this is a new bug in dbt-redshift
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
When running (or building) a materialized_view model a first time creates correctly, running the same view a second time with no changes generates an error:
"cms_mv_test" is not a view

indeed, when looking at the target/compiled code for the second dbt  i can see this:
    drop view if exists "<db>"."<schema>"."cms_mv_test" cascade;
    create materialized view "<db>"."<schema>"."cms_mv_test"

the database is correctly returning an error as it expects this first line to read:
    drop materialized view if exists "<db>"."<schema>"."cms_mv_test";

I do notice the files dbt/include/redshift/macros/relations/view/drop.sql and dbt/include/redshift/macros/relations/materialized_view/drop.sql appear correct (to my untrained eye) so presumably this some mapping error? Perhaps in the dbt-core layer?
i used the following config:
{{
  config(
    materialized = 'materialized_view',
    on_configuration_change = 'apply',
    auto_refresh = True,
    schema = 'dbt_datarunner'
  )
}}

but this behaviour was true for all values of on_configuration_change and auto_refresh, and regardless of if i made a change to the model file or not, or presence of the --full-refresh flag.
Expected Behavior
To cite docs here for convenience:

Materialized views are implemented following this "drop through" life cycle:

If an object does not exist, create a materialized view
If an object exists, other than a materialized view, that object is dropped and replaced with a materialized view
If --full-refresh is supplied, replace the materialized view regardless of changes and the on_configuration_change setting
If there are no configuration changes, refresh the materialized view
At this point there are configuration changes, proceed according to the on_configuration_change setting

https://docs.getdbt.com/docs/build/materializations
is relevance here is step 2, following a second dbt run should complete without error, by running the following sql:
    drop materialized view if exists "<db>"."<schema>"."cms_mv_test";
    create materialized view "<db>"."<schema>"."cms_mv_test"

Steps To Reproduce

create basic MV under path models/demo/cms_mv_test.sql:

{{
  config(
    materialized = 'materialized_view',
    on_configuration_change = 'apply',
    auto_refresh = True,
    schema = 'dbt_datarunner'
  )
}}

SELECT
	id
	,name
	,surname
FROM demo_table
WHERE
    --<some filters>
GROUP BY 1, 2, 3

run following command:

dbt build --select models/demo/cms_mv_test.sql

rerun the command

dbt build --select models/demo/cms_mv_test.sql

Relevant log output
$ dbt build --select models/demo/cms_mv_test.sql --full-refresh
10:29:27  Running with dbt=1.6.6
10:29:27  Registered adapter: redshift=1.6.2
10:29:27  Found 48 models, 52 tests, 44 seeds, 21 sources, 0 exposures, 0 metrics, 549 macros, 0 groups, 0 semantic models
10:29:27
10:29:32  Concurrency: 4 threads (target='<profile>')
10:29:32
10:29:32  1 of 1 START sql materialized_view model <schema>.cms_mv_test ............ [RUN]
10:29:33  1 of 1 ERROR creating sql materialized_view model <schema>.cms_mv_test ... [ERROR in 0.84s]
10:29:34
10:29:34  Finished running 1 materialized_view model in 0 hours 0 minutes and 6.71 seconds (6.71s).
10:29:34
10:29:34  Completed with 1 error and 0 warnings:
10:29:34
10:29:34    Database Error in model cms_mv_test (models/demo/cms_mv_test.sql)
  "cms_mv_test" is not a view
  compiled Code at target/run/redshift/models/demo/cms_mv_test.sql
10:29:34

Environment
- OS: MacOS Monterey 12.6
- Python: 3.11.5
- dbt-core: 1.6.6
- dbt-redshift: 1.6.6

Additional Context
Disclaimer: i am aware of the existing issues https://github.com/dbt-labs/dbt-redshift/issues/621 and https://github.com/dbt-labs/dbt-redshift/issues/565 i think this is technically different hence why i'm raising it. I won't take issue if this is rolled into those as i expect the required code changes might be the same.
Similarly if this is a user error i'm happy to humbly walk away with a learning on how to work this correctly.

ignore me i believe this is a dupe of CT-3229, sorry for any confusion


646 Use graph model with abms

Changes and Information
Please briefly list the changes (main added features, changed items, or corrected bugs) made:

Create graph nodes and edges for ABM

If need be, add additional information and what the reviewer should look out for in particular:

Merge Request - Guideline Checklist
Please check our git workflow. Use the draft feature if the Pull Request is not yet ready to review.
Checks by code author

[ ] Every addressed issue is linked (use the "Closes #ISSUE" keyword below)
[ ] New code adheres to coding guidelines
[ ] No large data files have been added (files should in sum not exceed 100 KB, avoid PDFs, Word docs, etc.)
[ ] Tests are added for new functionality and a local test run was successful (with and without OpenMP)
[ ] Appropriate documentation for new functionality has been added (Doxygen in the code and Markdown files if necessary)
[ ] Proper attention to licenses, especially no new third-party software with conflicting license has been added
[ ] (For ABM development) Checked benchmark results and ran and posted a local test above from before and after development to ensure performance is monitored.

Checks by code reviewer(s)

[ ] Corresponding issue(s) is/are linked and addressed
[ ] Code is clean of development artifacts (no deactivated or commented code lines, no debugging printouts, etc.)
[ ] Appropriate unit tests have been added, CI passes, code coverage and performance is acceptable (did not decrease)
[ ] No large data files added in the whole history of commits(files should in sum not exceed 100 KB, avoid PDFs, Word docs, etc.)
[ ] On merge, add 2-5 lines with the changes (main added features, changed items, or corrected bugs) to the merge-commit-message. This can be taken from the briefly-list-the-changes above (best case) or the separate commit messages (worst case).

closes #646

Performance Menasures:
The following are the mean results of 10 executions of the abm benchmark for a single thread (OMP_NUM_THREADS=1)
Main:
`
Benchmark                                                    Time             CPU
abm_benchmark/abm_benchmark_50k        3178,6 ms      3177,1 ms
abm_benchmark/abm_benchmark_100k      7128,8 ms      7125,8 ms
abm_benchmark/abm_benchmark_200k      14771 ms       14766,3 ms
`
Graph_ABM:
`
Benchmark                                                    Time             CPU
abm_benchmark/abm_benchmark_50k        3537,5 ms      3535,5 ms
abm_benchmark/abm_benchmark_100k      7611 ms         7608 ms
abm_benchmark/abm_benchmark_200k      15674,8 ms    15669,6 ms
`
So 5-10% performance loss.


Link Zsh completion for Docker CLI

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Version is not included because this does not involve a new version. It only installs completions to proper location.
If this looks good then I may also update docker-edge.

This is already provided by the docker competition formulas.
docker-completion
docker-compose-completion
docker-machine-completion

Thanks for the PR nonetheless.

Sorry, I didn't know that. But why do we maintain another three formula when Docker.app comes with the completion already? The approach feels like more work to do… I guess there might have been some discussion about it?

The formulas have a better way of installing completions and work with all Docker Casks.
  def install
    bash_completion.install "components/cli/contrib/completion/bash/docker"
    fish_completion.install "components/cli/contrib/completion/fish/docker.fish"
    zsh_completion.install "components/cli/contrib/completion/zsh/_docker"
  end

Just curious, is there any Docker Casks besides docker and docker-edge?


Crashing in The Witcher 1 (exception_access_violation)

The game crashes whenever I load a save, try to create a new game, loading into new areas and sometimes just randomly crashes.
I have had a long history of crashes with this game, recently reinstalled windows not only because of this but I hoped it would be a nice side effect. First playthrough went smoothly, I wanted to play it again with mods and dxvk for increased performance but this makes it unbearable.
In case you are wondering, I used this reddit post here install mods properly. I skipped some mods so I wouldn't have to crash because of anything memory related but of course that didn't want to work.
Software information
The Witcher: Enhanced Edition (V1.5A).
Maxed out settings.
System information

GPU: RX 7900XT
Driver: 23.9.3
Wine version: None (Windows 11 22H2 x64)
DXVK version: 2.3

Log files
Game Log:
errlog.txt
DXVK D3D9 Log:
witcher_d3d9.log

Can you list which of the mods you are using.

I am using FCR, Complete Sword Overhaul, Dagger Overhaul, AI Upscaled Textures (AiO AI Textures with RotWW and The Witcher Overhaul Project by lexo1000), The Witcher Overhaul Project (Only sound and blood files), Celin Wraith Fix, Fair Kalkstein Mod, Plentiful Herbs, Han Gives Han NOT Berbercane, Reverend Beard Texture Fix, Faster Movement For Geralt.

I have also patched the game to be Large Address Aware using FPS Counter which I forgot to mention.

Please make an apitrace.

Please make an apitrace.

Trying to launch the game with the apitrace d3d9.dll crashes the game when I start it. I used the 32 bit files.

I tried doing everything I can to get the apitrace to work, are there any known issues with it or do I just suck?

Sometimes it just doesn't work with some games.
I will take a look later when i have the time.

Yeah I saw that it might not work in some games and that's what I thought was happening to begin with but to make sure I reinstalled the game and tried without mods and a lot of different stuff, nothing worked so I guess it is the case after all.
I think you and K0bin have handled a very similar issue before about a year back: https://github.com/doitsujin/dxvk/issues/1763#issuecomment-1152942503
Maybe Renderdoc would be useful?

Maybe Renderdoc would be useful?

No, it wouldn't. Renderdoc captures aren't portable so they can only be replayed on a similar GPU and driver as the one you have. On top of that they only show one frame which is probably not the frame the game crashes in anyway.

So basically dxvk just won't work with The Witcher 1 modded?

Maybe I'll look into it at some point but I simply don't have time to mess with Witcher 1 mods at the moment.

Putting aside that I wouldn't really put my horses on a guide that seems like recommending v-sync as a shorthand for fps limiting.. does the thing work just fine with just native d3d9?
Because once you start to mod everything and the kitchen sink it may just simply be that the game has not enough address space.
(and maybe dxvk-remix could still save you there, but this is besides the point)

@vessbon Using just the mods you mentioned i haven't had it crash upon a new game yet. But it does happen when i use the fps counter tool at least on Linux.
Does it stop crashing if you remove that?

Apitraces of the above crash when using fps counter. One with wined3d where it doesn't crash and one with dxvk where it does.
https://drive.proton.me/urls/1G9ATM1Y50#TOZZq67EFwz3

UPDATE: I can reproduce the issue with a GCC build of DXVK. The problem is that running the game with GDB makes it work fine and Visual Studio doesn't have symbols for the GCC build.
So I have no idea how to further investigate this.

I don't know if this is relevant, but the game, Witcher 1, crashes with dxvk when loading saves or changing zones if running any software that injects any kind of hook, e.g. the game crashes almost 100% of the time with OBS Studio, also I think it crashes with RTSS.
If you close OBS, RTSS etc. but keep dxvk - the crashes stop.
If you do vice versa, remove dxvk, but keep OBS, RTSS, etc, - the crashes also stop.
Go figure.
Also you don't need to record or stream with OBS, just leave it open with game capture set to the witcher.exe
I do run same/similar mods, all tagged [ MANDATORY ] from this reddit guide here.
Also I had same exact issues with Oblivion dxvk+OBS, crashes were less common tho, but on The Witcher 1 it's unplayable.


Add link to source article in alias question

Caveat: It's very likely that this code is incorrect, but I think it may be in the ballpark of what needs to be done for the intended purpose :)

Oh wow, the code was actually correct? I was kinda winging it, tbh :sweat_smile:


Fine-mapping

This PR introduces an outline of the code needed for fine-mapping. I have updated it to be divided into each method:

Fine-mapping data extraction

Extracts LD matrix and summary statistics for +/- 500kb region
Filters both datasets for only matching/intersecting SNPs

DENTIST outlier detection

Runs outlier detection using a DENTIST calculation provided by the slalom package (https://github.com/mkanai/slalom/)

SuSiE fine-mapping

Runs SuSiE-inf fine-mapping-inf python package  (https://github.com/FinucaneLab/fine-mapping-inf/)

Fine-mapping Data Extraction
The fine-mapping data extraction uses a fm_data_extraction.py script in /otg/datasets which contains four functions:

get_FM_study_locus() – gets summary statistics and the lead SNP ID for a locus.
get_gnomad_ld_matrix() – uses the lead SNP ID to get the LD matrix within +/- 500kb
get_matching_snps() – Finds intersect of SNPs in summary statistics and LD matrix
allele_flip_check() - Checks alleles match between LD matrix and summary statistics (added in case we have conflicting input data in the future)

I have each of these functions running individually with some UKBB summary statistics taken from the GCP here (the functions work, but I expect they need re-writing as I had conflicting pyspark and hail version errors throughout creating these functions both locally and on a dataproc cluster, for some reason only specific pyspark functions would give a version error).
Also, when submitting a job for this step in the ETL I’ve been getting an error: Error locating target 'otg.fm_data_extraction.FMDataExtractionStep' - so I haven’t been able to test it further yet. I spent a couple of hours trying to address this, as I assume something small is missing in my step setup, but haven’t found the issue.
DENTIST
Dentist outlier detection has 2 functions:

calculate_dentist() – detects which SNPs are outliers and creates a binary column denoting which is or isn’t an outlier. This function takes the filtered summary statistics from the fine-mapping data extraction as input. It also has the option to switch to using CARMA outlier detection, this is unfinished and maybe not worth actually including, but in theory just the outlier detection of CARMA, aside from its fine-mapping, could be applied here.
calc_nlog10p_dentist_s() – this function is nested inside calculate_dentist() provides the final calculation for dentist before it is used for outlier detection.

I haven't tested DENTIST in the ETL but I have a test run of my function here. This function needs the summary statistics with an R2 column measuring LD for all variants with the lead SNP of the locus.
Also just to note, the DENTIST thresholds I used are the default in the SLALOM package (df.R2 > r2_threshold) & (df.nlog10p_dentist_s > nlog10p_dentist_s_threshold) however this rarely detects outliers and suggests we should test other thresholds. For example, the SLALOM paper also tests using just df.R2 > df.t_dentist_s - which in some examples I've run does successfully detect outliers (although usually still a low number).
SuSiE

Currently /otg/method/susie.py is just a placeholder of code copied from the finemapping-inf package. This still needs testing and conversion into pyspark if it is the method we choose to use.
I have also ran these SuSiE functions that I copied into the ETL on an example locus here.

The code for running dentist and susie are completely untested in the ETL. However, I have run the functions individually (https://github.com/hlnicholls/slalom-susie/). I also have some expected output results of SuSiE here for reference

@DSuveges Given that Hannah has left, do we plan on finalising this PR? Is this currently assigned to anyone? And as you reviewed it previously, do you think it'll require only cosmetic changes or a big overhaul?

As this work is now contiuned on different branches, This PR is closed.


feat: merkle proving and verification

#24
#23
#22  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite


fix(versions): update Activiti/activiti-cloud-example-chart versions

UpdateBot pushed helm dependency: activiti-cloud-modeling to: 7.1.56

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-modeling 7.1.56

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-x-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Improve readability of parameters in the recently changed SchemaBuilder interface

I'm learning Orchard CMS and using most recent versions of OrchardCore Preview. I tried to create and index migration logic with ISchemaBuilder.CreateMapIndexTable and ISchemaBuilder.AlterTable like it was done in the learning courses done by Lombiq. However, I noticed that list of parameters in ISchemaBuilder.CreateMapIndexTable was changed during refactoring #249.
It took a while for me to figure out how to correctly use new method together with AlterIndexTable. And most problematic thing was "collection" parameter that doesn't seem clear what is it for and how it works. I believe usual Orchard CMS developer would be happy to see some clear interface comments and parameter names.
Let's figure out the best way to improve that. Here are some ideas that I have:

Collection parameter is actually just a part of a table and key name. So, let's rename "collection" to: "namePrefix", "typePrefix", "prefix" (not good as in case of foreign key it will be in the middle FK_collection_name).
Collection parameter is actually optional and can be empty. But I pushed to send String.Empty anyway. So, let's define a default value for it (empty string or whatever).
Let's add some comments clarifying behavior. Unfortunately, I can't suggest good comments as I'm not fully aware of the ideas behind.

Thanks for the feedback. This is a good start to get the thoughts going. If you want to provide some PRs for the missing documentations it doesn't need to be discussed.


Native library support for Pub

This issue was originally filed by<EMAIL_ADDRESS>
Pub should define a layout convention for extensions with native library components (https://github.com/sam-mccall/dart-sqlite is a simple example).
Pub should be able to build the native components as part of installation, on systems with the appropriate toolchain available.
Pub should also be able to share the resulting platform-specific binary package, so that other users don't have to compile it.

This issue has been moved to dart-lang/pub#39.


Touch up

Codecov Report

Merging #18 into master will increase coverage by 0.46%.
The diff coverage is 94.73%.

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage      89%   89.47%   +0.46%
==========================================
  Files           9        9
  Lines         473      475       +2
==========================================
+ Hits          421      425       +4
+ Misses         52       50       -2

Flag
Coverage Δ

#KituraWebSocket
89.47% <94.73%> (+0.46%)
:arrow_up:

Impacted Files
Coverage Δ

Sources/KituraWebSocket/WebSocket.swift
100% <ø> (ø)
:arrow_up:

Sources/KituraWebSocket/WSServerRequest.swift
65% <ø> (ø)
:arrow_up:

Sources/KituraWebSocket/WSFrameParser.swift
98.16% <100%> (ø)
:arrow_up:

Sources/KituraWebSocket/WebSocketConnection.swift
91.44% <93.75%> (+1.17%)
: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 5e68697...d194abf. Read the comment docs.

@youming-lin Latest commit addresses review comments


[KBM] remap shortcuts is too small and doesn't auto expand

ℹ Computer information

PowerToys version: 0.33.1
PowerToy Utility: Keyboard Manager
Running PowerToys as Admin: yes
Windows build number: 19042.804

📝 Provide detailed reproduction steps (if any)

open Keyboard Manager
remap plenty of keyboard shortcut
resize the Powertoys window

✔ Expected result

list of remapped shortcuts is bigger
list expands with the Windows on resize

❌ Actual result

list is a fixed box that doesn't make use of the space a 4K screen provides

📷 Screenshots

We've updated our UI and this issue is fixed! :)


Rule enforcing typeAnnotation on a complex query is correct

Copying conversation from https://github.com/typescript-eslint/typescript-eslint/issues/4065#issuecomment-955397808 & https://github.com/typescript-eslint/typescript-eslint/pull/6444#discussion_r1102035077: typescript-eslint doesn't have a way to infer the types of nodes when the ESQuery selector is complex:
return {
  'ClassDeclaration, ClassExpression[id]'(
    node
    // ^? TSESTree.Node
    // Expected: something like...
    // TSESTree.ClassDeclaration | TSESTree.ClassExpression & { id: TSESTree.Identifier }
  ) {

Even with these two not(-yet?)-implemented helpers (neither of which may ever be useful for public consumers):

https://github.com/typescript-eslint/typescript-eslint/issues/4065
https://github.com/typescript-eslint/typescript-eslint/issues/6224

...there's no way for custom rules written in TypeScript to enforce that the type of node matches its AST shape. Which means devs can write blatantly wrong code like:
return {
  'ClassDeclaration, ClassExpression[id]'(
    node: TSESTree.Identifier

Is eslint-plugin-eslint-plugin an appropriate place to write a lint rule that makes sure the type annotation on a node matches the node itself?
This rule might be tricky to write given that https://github.com/microsoft/TypeScript/issues/9879 blocks having a TypeScript API to check type assignability. The rule could use basic string comparisons in the meantime.

I'm open to accepting rules targeted at TypeScript rules/syntax, including this one.
I previously added general compatibility with TypeScript rules in v4.
Any new rule targeted at TypeScript rules should just no-op when running against a JavaScript rule.
We can even use meta.docs.category: "TypeScript" to have a separate group/list of our TypeScript-focused rules (related: #341).


always starts in a maximized window

2.12.0
Whenever I start the app, it starts in a full-screen window, even if I quit the app in a non-full screen window.
It also does not remember which tabs I had open

Hi @ccurvey ,
OmniDB 2.12.0 now remembers the names and contents of the tabs that were open. As implementation changed, it won't remember the tabs that you have opened on previous releases.
Please try opening some tabs in 2.12, renaming them, execute them (they need to contain a valid SQL to be remembered) then restart OmniDB to see if they are there.
As for the full-screen window: previously it started on a windowed 800x600 window, and most people simply had to maximize it. Now we decided to always open full-screen. OmniDB still does not remember window size or position of when it was closed, we plan to implement that feature for next releases.
Regards,
William

Hi,
Closing this one, as the window will always be open maximized.
Regards,
William


can not uncheck all delivery locations

To set something to the default delivery locations, you should just be able to uncheck all the selected locations.
If you go in a choose one locations and click "Update Holding Location"
then uncheck that same location and click "Update Holding Location"
The location stays selected if it was the only location

Go into the rails console and run this to set it to empty.
hl = Locations::HoldingLocation.find_by(code: 'scsbcul')
hl.delivery_locations = []

I think this will be obviated by https://github.com/pulibrary/bibdata/issues/1193


Concurrent suites execution

Hello,
do you have a concurrent suites execution on the project's roadmap? It can be really beneficial for headless browsers like Zombie.js, which can be spawned and executed in parallel. At the moment Mocha runs suites one by one, so there is no gain from having multiple headless browser instances.

I would like this ability in theory as well but after thinking on it a bit it would require a bunch of complexity to keep meaningful reporting output. You can use something like gulp-spawn-mocha to run a bunch of separate files in parallel but then you wind up with separate reports (and only if you use a file based reporter, real time reporters give the same mess)
Did you have any ideas for addressing that @emdin ?

@scott2449 hmm I think I miss something. What I had in mind is suites run concurrently ("thread" per suite) and there will be the same final report (5 passed/2 failed). So we'll have he same reports but with undefined order. Since suites identified by their names it shouldn't be a big problem.

@edmin I was saying that I agreed with you but that it doesn't work at the moment. There are currently some concurrency issues with mocha.  Also most reporters are currently real time, like spec, and just interleave all the results.

All right, now I understand. I somehow made it work with node.js using domains, but it is not production ready of course.
But do you think this feature will be useful for Mocha users or may be my use case (Zombie.js tests) is too specific?

This issue has been raised many times, implemented, then removed, because it was too complex and didn't end up buying people much time.

I just started using this with acceptance tests: https://github.com/danielstjules/mocha.parallel

@emdin if you are still working on that let me know thanks, I found domains as being the only way to solve the problem of handling errors thrown from async tests in any given suite


Use async/await instead of control flow in protractor

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
feature - remove control flow from protractor tests

What is the current behavior? (You can also link to an open issue here)
Protractor tests use control flow

What is the new behavior (if this is a feature change)?
Now used async/await functions

Other information:
https://github.com/AngularClass/angular-starter/issues/1766

LGTM


1

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

Pre-requisites

[X] My application to the GitHub Technology Partner Program is approved.

Tasks

[X] MUST follow our general contributing guidelines for voice and markup format.
[X] MUST emphasize how the third-party product works with GitHub.
[X] MUST be written in Markdown format, using one of the templates provided.
[X] MUST include the name and URL of the GitHub technology partner responsible for maintenance of the documentation being contributed. This should be added via the contributor.name and contributor.URL properties in the template's YAML frontmatter.
[X] MUST be proposed via a pull request to this repo following the GitHub Flow.
[X] MUST be located in the root of the content folder. Your filename MUST match the GitHub technology partner name, and use the .md file extension.

Pull Request

[X] MUST reference this issue, e.g. via closes [this issue number].
[X] MUST pass the automated CI checks.
[X] MUST include links to supporting material demonstrating the functionality being documented (this can be a link to a public GitHub repo, or a video / screencast walkthrough).

John Lee
On Fri, Feb 9, 2024, 4:31 AM docs-bot @.***> wrote:

This issue may have been opened accidentally. I'm going to close it now,
but feel free to open a new issue with a more descriptive title or ask any
questions in discussions https://github.com/github/docs/discussions!
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/issues/31545#issuecomment-1935672246, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A663NQSB6X7XYMB2GUFNUQTYSX3JNAVCNFSM6AAAAABDBHCYJKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZVGY3TEMRUGY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Add support form specs

Context
The support form is missing end-to-end specs
Changes proposed in this pull request
Added basic spec covering the end-to-end creation and submission of a request using the support form
Guidance to review
Only specs added, but the journey can be followed at /get-support

 Iterate contact form to increase self-service

THE missing integration test! haha


useDrag doesn't work in mobile

codesandbox
https://codesandbox.io/s/action-sheet-forked-p7prx?fontsize=14&hidenavigation=1&theme=dark
Select mobile mode in browser.
Can't drag smooth, it's strange.

in codesandbox, it can toggle preview mode.
But use dev-tools，it can't work.  I will continue to observe it

see this link in your mobile phone (like safari)
https://p7prx.csb.app/

After some investigation, set touch-action: none will help.
The device will trigger onpointercancel event and then transfer to onDragEnd.
If you want custom control drag drop, use touch-action: none for the DOM.

After some investigation like reading the docs maybe 😏?
There are two things that feel wrong with your sandbox.

Not sure what useState does?
Setting the top and left attributes are not the best way to move DOM elements as you don't benefit from hardware acceleration like when you're using transforms.
onPointerUp and onPointerCancel will mess up with {...bind()} as explained in the docs: https://use-gesture.netlify.app/docs/hooks/#native-react-event-handlers
There's nothing wrong in using initialPosition the way you do but you might want to have a look at the initial option.

I thought it was I change left and top that cause the element can't drag. So I change the postion by dom reference.
As you can see, I annotated the set function from useState.

Yes.But my real production is complex, I will try to use transform.

Thanks, I should read the documentation more carefully  -)


Flyway 8.5.2 includes shell files co-mingled with versioned SQL migrations.

Which version and edition of Flyway are you using?
I am using Flyway 6.5.7. When I run flyway info, I get:
env FLYWAY_EDITION=enterprise /cygdrive/c/flyway-6.5.7/flyway -color=never info
Flyway Enterprise Edition (10 schemas) 6.5.7 by Redgate licensed to  until 2023-12-18
Database: jdbc:postgresql://<IP_ADDRESS>:15432/pricing (PostgreSQL 10.14)
Schema version: 57.0

+------------+---------+------------------------------------------------+----------+---------------------+------------+----------+
| Category   | Version | Description                                    | Type     | Installed On        | State      | Undoable |
+------------+---------+------------------------------------------------+----------+---------------------+------------+----------+
|            | 1       | << Flyway Baseline >>                          | BASELINE | 2018-12-26 04:03:40 | Baseline   | No       |
| Repeatable |         | count tuples the hard way                      | SQL      | 2019-01-04 05:43:49 | Success    |          |
| Repeatable |         | get basic counts                               | SQL      | 2019-01-04 05:59:59 | Success    |          |
| Repeatable |         | get schema migration                           | SQL      | 2019-01-04 06:06:50 | Success    |          |
| Repeatable |         | get table size and tuple counts via statistics | SQL      | 2019-01-04 06:06:50 | Superseded |          |

If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Using the latest causes the issue. I am trying to upgrade from Flyway 6.5.7 to 8.5.2.
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-Line
Which database are you using? (Type & version)
AWS Aurora PostgreSQL
Which operating system are you using?
Cygwin and Linux
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
flyway info.
What did you expect to see?
env FLYWAY_EDITION=enterprise /cygdrive/c/flyway-8.5.2/flyway -color=never info
Flyway Enterprise Edition (10 schemas) 8.5.2 by Redgate licensed to  until 2023-12-18
Database: jdbc:postgresql://<IP_ADDRESS>:15432/pricing (PostgreSQL 10.14)
Schema version: 57.0

+------------+---------+------------------------------------------------+----------+---------------------+------------+----------+
| Category   | Version | Description                                    | Type     | Installed On        | State      | Undoable |
+------------+---------+------------------------------------------------+----------+---------------------+------------+----------+
|            | 1       | << Flyway Baseline >>                          | BASELINE | 2018-12-26 04:03:40 | Baseline   | No       |
| Repeatable |         | count tuples the hard way                      | SQL      | 2019-01-04 05:43:49 | Success    |          |
| Repeatable |         | get basic counts                               | SQL      | 2019-01-04 05:59:59 | Success    |          |
| Repeatable |         | get schema migration                           | SQL      | 2019-01-04 06:06:50 | Success    |          |
| Repeatable |         | get table size and tuple counts via statistics | SQL      | 2019-01-04 06:06:50 | Superseded |          |

What did you see instead?
env FLYWAY_EDITION=enterprise /cygdrive/c/flyway-8.5.2/flyway -color=never info
----------------------------------------
Flyway Teams Edition (10 schemas) 8.5.2 by Redgate licensed to  until 2023-12-18
----------------------------------------
Flyway is up to date
Flyway Teams Edition 8.5.2 by Redgate
See what's new here: https://flywaydb.org/documentation/learnmore/releaseNotes#8.5.2

Database: jdbc:postgresql://<IP_ADDRESS>:15432/pricing (PostgreSQL 10.14)
ERROR: Found more than one migration with version 22.0
Offenders:
-> \migrations\pricing\V22_0__abc-1234-inputs.sh (SCRIPT)
-> \migrations\pricing\V22_0__abc-1234-reports.sh (SCRIPT)

I have tried using -sqlMigrationSuffixes=.sql even though .sql is the default. I have version SQL migrations, e.g. V files ending with .sql co-mingled with Bash shell scripts and Flyway 8.5.2 thinks they are versioned SQL migration files:
Here's what's in my migrations directory that works with Flyway 6.5.7:
$ ls -al migrations/pricing/V22* | awk '{print $10}'
migrations/pricing/V22_0__abc-1234.sql*
migrations/pricing/V22_0__abc-1234-inputs.sh*
migrations/pricing/V22_0__abc-1234-reports.sh*

@tjs145,
That is cool! I have them for the same reason cited in the doc:

Triggering execution of a 3rd party application as part of the migrations (such as a batch upload tool)

I use shell helpers to load (for me that's \COPY) data in bulk for the migration and I use them for data reporting (again, \COPY with SELECT statements) post job exec, e.g. after the migration, my customer wants a "data report" on the changes.
I wrote my own bash scripts around Flyway b/c something like this feature didn't exist and I have many migrations that have inputs (and many more with report outputs).
Sticking the migration naming convention for order of execution should work? I need the -inputs.sh script run first, then the SQL migration, then the -reports.sh script.
Does Flyway Command-line exec() them as subshells?
Could you point me in the right direction in the source where this is implemented? (I'm interested to know more)


[Issue]: Adjustments to EC Code & no project errors for upload validation

Why is this issue important?
To add clarity to upload validation messages so users are clear as to what they refer to
Current State
Blank EC Code
When EC Code is missing from a file, two errors are generating:

No Project Records
References row 12

Expected State
Blank EC Code
We only need the the error that says "Value is required for detailed_expenditure_category" in row 2, column B
Remove the error for row 2, column A - this is just help text for the sheet and there is no value that can be submitted
No Project Records
"Row" column should list 13
Implementation Plan
The following functions need to be added...
Relevant Code Snippets
No response

This is looking good to me in staging!

Here are the updates for the error text: https://github.com/usdigitalresponse/cpf-reporter/pull/307

@vshia The no projects message is looking good now!
However, when I try to test an upload with no EC Code selected in the Cover sheet, I get stuck on the Validation in Progress page and never get to validation results. ex upload in staging: https://staging.cpf.usdr.dev/uploads/146

https://github.com/usdigitalresponse/cpf-reporter/pull/322 should address this now

this is looking good in staging now, thank you @vshia !


Update Contents internal links

Contents section links anchors were not correct and linking to the correct readme section, this is just a quick correction.

Hey @EdgarBarrantes
We recently updated to sync with upstream as we were 230+ commits behind and foundry had changed their project structure this has resulted in a lot of conflicts on this PR. I suggest closing the PR and reopening with latest changes from main. Apologies for the inconvenience!


Add completion support for query arguments

Options like --suggestions=TYPE:QUERY_URL would be good.  This would need embedded shell snippets to have the correct output--like https://example.com/suggestions?q=$it&n=$SURFRAW_example_results.  Maybe there could be mappings (MappingOption) for these URLs?
It's a bit annoying to do, but I intend to use this feature for my OpenSearch elvis generator.
This needs changes in my completions code for Surfraw.

For completions from APIs, I'd like to limit the completions rate.  Should the elvis itself handle this?
Since I'm using bash completions, I'll need some way to check the time since last request.
Possible solutions:

use an environment variable to control it, say __mkelvis_complete_prevtime; or
use a file.

(1) avoids IO (other than the network request), so it's likely to be quicker.  However, this limits its usability outside of individual terminal sessions.
(2) avoids the disadvantages of (1) but it needs rw access to a file.  It's also vulnerable to race conditions (how do I get locks portably?  See https://unix.stackexchange.com/a/22047).  What should it be called and where should it be stored?  in $XDG_CACHE_HOME/surfraw/.previous-completion-time?  XDG_CACHE_HOME should default to $HOME/.cache.

It should support common suggestions formats:

jQuery (as used in nzsl) (JSON)
OpenSearch suggestions (JSON)
OpenSearch suggestions (XML)

Are there any more?

For the interface, maybe suggestions_url and suggestions_url_type can be attributes (and kwargs in constructor) of Elvis?
I'm thinking suggestions_url_type is either a string or an enum.  If a string: jquery (or wherever it comes from), opensearch-json, and opensearch-xml.  I'm liking the simpler API here.

jQuery suggestions use a term query parameter to base their output on.  When building autocomplete widgets, source contains a URL which ?term=TERMS will be appended to.
Regardless, I think all suggestions URL formats should have the URLs parsed (with urlparse(...)) so that format-agnostic mapping code can be used.  Then, only in the suggestions-response parsing, it will branch.

Also, it would be good to allow the user to configure whether they want to complete from the network--say, SURFRAW_complete_network, a boolean value that defaults to true.

More generally, the elvis could handle multiple types of query completions, a few of which would be from the network.
Some ideas:

packages (uses apt-cache or whatever packaging tool is on the system?)

Alternatively, allow the completion interface (e.g., Bash, Emacs) to handle it themselves and just provide the suggestions URLs.
As for providing the URLs, maybe with a -get-opensearch-{xml,json},-suggestions-url.  How would jquery completions be provided?  Do they even provide URLs?


dbeaver/pro#2674 [TE] Shared credentials button is not visible on small screens

Scrolled composite added

https://github.com/dbeaver/dbeaver/assets/32031248/ac1d5910-e9be-4640-b755-63ed1acb9927

Why these "Save credentials" checkbox and "Manage" button are so far from Authentication method combo?

Perhaps because of the "Save credentials" checkbox. I wouldn't say I like it either, but fixing it wouldn't be that trivial.


Add operations relevant for push

When implementing a client to be able to push RTMP towards twitch and Youtube I found that some methods were marked as "Not implemented" or missing, so I've added them.
Two of them change the API a little:

Stream.Write becomes public
Stream.CreateStream and ClientConn.CreateStream get a new parameter to set the chunkSize

Setting the chunk size is of course not really needed, but is nice both from an efficiency point of view and making network captures easier to debug.
It would be nice if these extensions could be included in this repo, since I think that they are generally useful.

Thanks for the review. Sorry that I didn't use the constant I defined myself.
And thanks for this great RTMP library!


Remove welcome banner displayed on StochPy startup

Remove the welcome banner displayed when StochPy is imported
Fulfills FR II.2.3

Manually closed because I merged the pull request before realizing it wasn't tied in to auto close this.


data is not iterable

When passing a nonstring type to renderQRCode such as renderQRCode(123), the util fails with a cryptic message. I suggest either make a strict error + types for when input is not a string or trying to stringify input using JSON.stringify

ref: https://github.com/unjs/listhen/pull/97

It does not allow numbers as the argument. And I think the type is already strict:
https://github.com/unjs/uqr/blob/01ba7fd43de88633ebce052f63599f7bbf8baa70/src/types.ts#L1-L6

Fair enough i think the issue with listhen upsteam is that cli implicitly does that and i didn't notice the problem but maybe making uqr more runtime safe? (i could quickly get the issue but users might have hard time to find)


Subject: Using gst/gst-kvs-plugin on EC2 Linux to extract frames from server

Hi,
I'm encountering issues while attempting to run the gst/gst-kvs-plugin examples on EC2 Linux. My goal is to see whether gstreamer can extract frames from a stream video on server. I got the following error when testing the example, and eventually I will add a filesrc location command.
1. Linux version for vtenc_h264_hw:
When running the following command:
gst-launch-1.0 autovideosrc ! vtenc_h264_hw max-keyframe-interval=30 bitrate=500 ! kvsplugin stream-name=ScaryTestStream channel-name="ScaryTestChannel" log-level=3

I receive the following warning:
WARNING: erroneous pipeline: no element "vtenc_h264_hw
It seems like there might be compatibility issues with certain elements on my ec2 Linux. With that, I assume the stream is followed by the example stream "Foo" from this link.
I appreciate any guidance on resolving these issues.

Hi,
The vtenc_h264_hw element is only available on MacOS, as it's a hardware encoder. Check out the gstreamer documentation, for more information on that. You may want to check out x264enc.
I noticed the link is pointing to Kinesis Data Streams, whereas this repository contains examples for Kinesis Video Streams. If you want to try uploading a video to Kinesis Video Streams, I recommend checking out CPP producer and using the kvssink plugin or the kvs_gstreamer_sample to send video to Kinesis Video Streams. You can also check out the workshop to explore KVS.


Question: Can i use this library for my project to accomplish my task?

Is this a bug report, a feature request, or a question?
Question.
Hello react-native-snap-carousel community! I am new to Javascript/react-native so i have a question. Is it possible to make UI same as on my example with this library, most importantly can i customise somehow Pagination Component so it looks same as mine ? I am using Redux for getting data from server  and storing it in redux store then i want to populate carousel with the data from my redux store . I am getting 20 objects which has data i want to show in cards/views in crousel. Thanks in advance!
Example:

Yes it look like the perfect component to do that be sure to adjust the width of your item to take the full size and no problem

Thanks i already figured it out. Finished this task a week ago.


feat: port Data.Fintype.Array

The dashboard says

port, but completely rewrite in terms of the new array

However, the new Array T isn't finite like array n T was, so the rewrite amounts to just deleting everything in the file.

bors merge


[Bug]  IndexError: too many indices for tensor of dimension 2

Describe the bug
There is a new error in weekly test:

Details:
https://github.com/InternLM/InternEvo/actions/runs/9036801596/job/24836640280
Environment
python 3.10
torch 2.1
Other information
No response

https://github.com/InternLM/InternEvo/pull/217，已修复


Add test coverage for ABI generator

The ABI generator implementation currently lacks test coverage which makes it difficult to refactor.
It'd be good to add test coverage similar to the code generator.

I know it's not the same, but you might find cargo-near integration tests useful. They basically test the whole ABI generation flow e2e, including SDK's ABI generator.

Great, thanks @itegulov !

I'm working on this.


Support setting env variables for all scripts at one time?

So a common use case is that developers want to to always set a certain env variables for all scripts. For instance, the handy NODE_PATH variable that let's developers get rid of relative requires in their scripts.
Currently, if we want to set this, we have to do it in each of the individual npm scripts like so:
"scripts": {
   "test": "cross-env NODE_PATH=./es6 mocha"
   "something": "cross-env NODE_PATH=./es6 node blah.js"
   ...
}

But it would be super handy if we could just do in one place like so:
"cross-env": {
   "NODE_PATH": "./es6"
},
"scripts": {
   "test": "mocha"
   "something": "node blah.js"
   ...
}

I've looked into npm and I actually don't see any great way to support this, but there might be. It would be awesome IMO in any case though. What are your thoughts Kent? Both on feasibility and desirability?

Hey there @scottmas! Long time no see :D
Thanks for the suggestion, but I'm not going to add this to the library. It's against the philosophy of this package of being small and simple. Adding this would add some magic ✨ that I'm not interested in adding.
I realize that it would be much nicer to just add this, but I'm going to go for simplicity and explicitness in the module. Thanks!

Oh, that's cool. I thought that might be the case. I actually just came across your p-s module and I think I might bring the discussion there - I'm digging the clever npm start ... solution


Power off doesn't always reset the session

Sometime files such as session.json are locked/in-use and won't delete before the refresh timeout occurs.

I can't reproduce this outside of a dev env and even then rarely.


vue-router 3.0.5 router onComplete no longer executed

In a vuex action I have a simple block of code in which I am simply calling Vue.router.push() in a method of vuex
Relevant code:
import Vue from 'vue'

login ({ commit }, data) {
  Vue.router.push({
    name: 'dashboard'
  }, () => {
    store.dispatch('app/loaded')
  })
}

What is expected:
The onComplete callback of the router.push call should be executed.
What is actually happening:
The onComplete callback never gets executed after upgrading to 3.0.5 from 3.0.4
Additional info:
I rolled back to 3.0.4 and this issue went away. This is code that I have not touched in 6 months and has had many vue-router upgrades without issue.

OK Just trying to help, I will test with the next version. In the meantime I will lock my package.json @3.0.4.


[pt] translation for comprehensions

Related to #355
This lesson is not in the issue's list of pages to translate, but it doesn't have a translation.
Feel free to give me your feedback @pragmaticivan and anyone who wants to revise this translation.

Thank you @ruan-brandao!  I'll give @pragmaticivan a chance to comment and then we can merge 👍

Ops, still reviewing.

BTW, I noticed that the lessons for Strings and Custom Mix Tasks need translation too. Should I translate them in this PR or open one PR per lesson?

@ruan-brandao individual PRs are usually easier for reviewers since they can be tackled in small chunks.

@doomspork ready!

Thanks, @ruan-brandao and @pragmaticivan!


fix: use webgl 1 pattern to always choose best precision available

This PR (delete as applicable)

Fixes a bug

Describe the changes below:
The bug fixed by the PR:
#6233 MultiPipeline shaders cause crash on older Android devices

In WebGL 1, "highp float" support is optional in fragment shaders
Should have minimal impact on performance (done once per the program run, during the shader compilation) and provides more portability to Phaser on older devices

👍


ytcc: 1.8.4 -> 2.0.1

Motivation for this change
saw this in nixpkgs-update logs
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@jonringer Could you please apply these changes as well:
--- i/pkgs/tools/networking/ytcc/default.nix
+++ w/pkgs/tools/networking/ytcc/default.nix
@@ -15,21 +15,21 @@ python3Packages.buildPythonApplication rec {

   propagatedBuildInputs = with python3Packages; [ feedparser lxml sqlalchemy youtube-dl ];

-  checkInputs = with python3Packages; [ nose pytest ];
+  checkInputs = with python3Packages; [ nose pytestCheckHook ];

   # Disable tests that touch network or shell out to commands
-  checkPhase = ''
-    pytest . -k "not get_channels \
-                 and not play_video \
-                 and not download_videos \
-                 and not update_all \
-                 and not add_channel_duplicate"
-  '';
+  disabledTests = [
+    "get_channels"
+    "play_video"
+    "download_videos"
+    "update_all"
+    "add_channel_duplicate"
+  ];

   meta = {
     description = "Command Line tool to keep track of your favourite YouTube channels without signing up for a Google account";
     homepage = "https://github.com/woefe/ytcc";
-    license = lib.licenses.gpl3;
+    license = lib.licenses.gpl3Plus;
     maintainers = with lib.maintainers; [ marius851000 ];
   };
 }

@jonringer Could you please apply these changes as well:
--- i/pkgs/tools/networking/ytcc/default.nix
+++ w/pkgs/tools/networking/ytcc/default.nix
@@ -15,21 +15,21 @@ python3Packages.buildPythonApplication rec {

   propagatedBuildInputs = with python3Packages; [ feedparser lxml sqlalchemy youtube-dl ];

-  checkInputs = with python3Packages; [ nose pytest ];
+  checkInputs = with python3Packages; [ nose pytestCheckHook ];

   # Disable tests that touch network or shell out to commands
-  checkPhase = ''
-    pytest . -k "not get_channels \
-                 and not play_video \
-                 and not download_videos \
-                 and not update_all \
-                 and not add_channel_duplicate"
-  '';
+  disabledTests = [
+    "get_channels"
+    "play_video"
+    "download_videos"
+    "update_all"
+    "add_channel_duplicate"
+  ];

   meta = {
     description = "Command Line tool to keep track of your favourite YouTube channels without signing up for a Google account";
     homepage = "https://github.com/woefe/ytcc";
-    license = lib.licenses.gpl3;
+    license = lib.licenses.gpl3Plus;
     maintainers = with lib.maintainers; [ marius851000 ];
   };
 }

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109088 run on x86_64-linux 1

1 package built:

ytcc

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109088 run on x86_64-linux 1

1 package built:

ytcc

@sikmir done :)

@sikmir done :)

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109088 run on x86_64-darwin 1

1 package built:

ytcc

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109088 run on x86_64-darwin 1

1 package built:

ytcc


pull request for e2e test at 01/31/2017 19:56:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add "Total Cost of Ownership" (TCO) and "Leaving money on the table"

Adds triggers:
Total Cost of Ownership" (TCO)
Leav(e|ing) money on the table

This is great, thanks for the PR!
Published in v1.2.1


YahooFantasyNFL Doesn’t Recognize League ID

I reinstalled the YahooFantasyNFL app for this season. After I successfully sign in and enter my League ID and add the app, I get the message, “ERROR! Please check your league ID.” I’m successfully connected to Yahoo! according to the app and my League ID is correct.
cc: @jweier

Hey - The app needs to be updated to reflect the 2024 season. I pinged the TidByt team about seeing if we could work together to improve the app and unfortunately I haven’t received a response. I’m a little hesitant to continue development on the app because the updating/debugging process is very challenging without being able to work with them or see the errors.
I guess more to come…hopefully…
-Jay

I think the new GAME_KEY should be 449, though it sounds like there are bigger dev challenges with updating the app.


Cascading updates on DragSource causing performance issues

Describe the bug
https://github.com/react-dnd/react-dnd/blob/5fefffc8c6c1ceced3aa0356fd869354ad74734a/packages/react-dnd/src/decorateHandler.tsx#L100-L105
In componentDidMount on DragDropContainer, this.handleChange() will be called with no initial state (this.state === null), this causes handleChange() to call setState() which causes a cascading update and another React reconciliation process.
To Reproduce
Steps to reproduce the behavior:

CodeSandbox has the template for the simplest example provided in this repo: https://codesandbox.io/s/github/react-dnd/react-dnd/tree/gh-pages/examples_js/01-dustbin/copy-or-move. Open the result in a new window and open DevTools to profile the load times. I'm using Chrome DevTools in this case to look at the React Timings output.
Profile the load time. You should see this in the React Timing from React warning about cascading updates.

Expected behavior
I've skimmed the code and the lifecycles a bit and I think it can be improved fairly easily? I get that componentDidMount calls handleChange to set state values derived from the context consumer on DragDropContext, but it looks like this.state is only being used for memoizing the derived values and simultaneously calling it to trigger a new render if any of the dependant values change? Triggering a new render probably shouldn't be done on componentDidMount since the evaluation of handleUpdate will always trigger another render with the value of this.state being null from the start? Maybe we can set a default value somehow derived from the context to set to the state initially to avoid rerendering at the start.
I made this issue mainly to discuss whether or not I'm understanding the code wrong and it can't be done, or is it a simple fix (or we all agree this performance bottleneck needs to be solved). Either way let me know if I made a mistake in understanding it.
Additional context
FYI cascading updates seem to happen on DropTarget as well as the update cycle of these components as well. I'm omitting those issues from this particular one right now.
For an example of a real world scenario impacted by this bug, my case is on mounting a new DragSource for an image loaded dynamically, and this causes the callback to the image being loaded (onload or React state updates) taking more than 50ms to process.
Thanks for all the work you've done in this <3

There's a PR for it (#1336), shouldn't be marked as wontfix, but the stale bot doesn't know it 😄


Config options to freeze lock files when installing dependencies

Right now, we don't freeze lock files (if present) whenever we install dependencies at any level (root package or test projects). We've found that freezing these files generally causes reliability issues at the test project level if Rugged dies (or is killed) while it's running, since injecting the packaged version of the root project necessarily updates the lock file.
Users may want the extra safety of freezing their lock files at the test packages levels, though. Plus, we probably should be freezing the root project's lock file by default.
So, I propose that we introduce two new config options, called freezeRootProjectLockFile and freezeTestProjectLockFiles. Both of these would be boolean values. The former would default to true, and the latter would default to false.
If true, when installing dependencies, the lock file should be frozen. That's done by adding --frozen-lockfile to the Yarn command, and by using npm ci instead of npm install for npm.

Before work on this issue is started #44 should probably be merged.


Update example dataset

The example dataset will need to be updated following changes in the BEP031 for microscopy.
The data structure and filenames have not changed since the 20201008 release.
As of 20210504, the changes in metadata are:

[x] BitDepth, Channels and Species are no longer in json metadata
[x] FovWidth and FovHeight are now FieldOfView with a 2-number array
[x] Species is in participants.tsv and must follow the Binomial NCBI taxonomy, here rattus norvegicus.
[x] PixelSize is a 2-number array

Since the changes are minor for now, we should wait for the BEP to finalize before updating the dataset.
I will update this issue if further changes are required.

Other changes TODO as of 20210713:
Name of the Repo:

[ ] Update Name to “data_axondeepseg_sem”

dataset_description.json:

[ ] Update Name to “data_axondeepseg_sem”
[ ] Update BIDS version to “1.6.0 - BEP031 v0.0.4”

samples.tsv:

[ ] Add column “sample_type” with value “tissue” for all samples
[ ] Remove all “LongName” fields

samples.json:

[ ] Remove “Unique” from sample_id description
[ ] Add “sample_type” description: "Type of sample from ENCODE Biosample Type (https://www.encodeproject.org/profiles/biosample_type)"

participants.tsv:

[ ] Add column “species” with value “rattus norvegicus” for all participants
[ ] Remove all “LongName” fields

participants.json:

[ ] Add “species” description: “Binomial species name from the NCBI Taxonomy(https://www.ncbi.nlm.nih.gov/Taxonomy/Browser/wwwtax.cgi)”

** In derivatives/labels:**

[ ] Add a dataset_description.json: {"Name": "data_axondeepseg_sem labels", "BIDSVersion": “1.6.0 - BEP031 v0.0.4”, "PipelineDescription": {"Name": "Axon and myelin manual segmentation labels"}}

README.md:

[ ] AxonDeepSeg SEM dataset
[ ] Add dataset/species description
[ ] Ipdate version of the BEP (create an official v0.0.4 in BEP first, add the version and date in the readme)

New release:

[ ] Release a new version of dataset, update links in the BEP

Closing this issue as the dataset was updated in PR #4 and a new release was created (https://github.com/axondeepseg/data_axondeepseg_sem/releases/tag/r20210715).
The repo was transferred from ivadomed to axondeepseg organization and its name updated from data_example_microscopy_sem to data_axondeepseg_sem.
The link to the release was also updated in the BEP.


JACK backend must run at 44.1khz

I'm using the JACK Audio Connection Kit backend for librespot, as I find it extremely capable and stable. However, using the JACK backend for librespot, I have to run the entire server at 44100hz or the audio is completely off pitch. This is clearly due to a lack of sample rate conversion. After checking a few JACK playback programs, it seems the common solution is to use libsamplerate (Secret Rabbit Code) to do the rate conversion to the sample rate the JACK server advertises - ie, sample rate conversion is not built into JACK clients by default.
I found basically untested libsamplerate bindings for Rust, but as before stated the less audio code the better so this may not be the best solution.

Interesting. There are rust sample rate conversions in both the Rodio crate (linear) and the sample crate (step, linear & sinc, though I couldn't get sinc to sound OK).
I expect using either of those would be work pretty well. I have an example of using cpal and the sample crate locally, as a librespot backend... it's a bit buggy.
I'll point it out once I'm home and perhaps have a closer look at the Jack backend.

So after a bit of looking through both, if Rodio could output to a JACK server, that would definitely be best. It uses cpal for audio output, which on Linux, goes thru ALSA and not JACK. When did you try sinc interpolation on the sample crate? It seems there is a resolved bug where sinc(0) = 0 (should be 1) which would have definitely caused issues.
It's looking like the better options would either be to add JACK support to cpal or add the sample crate to this project. Given the stated desire to minimize audio interface code in this project, the former might be better.

Here's my branch using the sample crate. CPAL doesn't perform any re-sampling itself so a Jack backend based on the jack may well be suitable. However I'm not sure what CPAL's stance is on having multiple backend options for a single platform.
I'd be tempted to say that the JACK backend for librespot is broken because it doesn't re-sample, and that a PR using sample would be reasonable. Looks like it's pretty similar to my branch in structure.
Perhaps the jack crate ought to be able to perform re-sampling?

If the route you're interested in taking is to add resampling to CPAL, that should be taken downstream as Rodio uses CPAL as well. Not sure where it would fit but adding a CPAL and Rodio backend feels wrong. I agree that the JACK backend is broken since it doesn't do resampling.
There seems to be a desire to move audio transport code out of librespot and to other separately maintained crates. My personal preference would be to add a JACK backend to Rodio and remove JACK from librespot entirely.

Sorry I wasn't very clear there. The options I see are these (in order of tidyness):

Add a Jack backend to CPAL, and use via Rodio in librespot (I agree that this is the best option, for librespot and the Rust audio community).
Add re-sampling to the Jack backend in librespot (probably the easiest option).
Add re-sampling to the jack crate.
Add re-sampling to a different crate which depends on jack, and use that in librespot.

My branch is experimenting with explicit re-sampling in librespot to allow us to use CPAL directly without Rodio. Hopefully it never needs to be used and any issues we have with Rodio can be resolved upstream.

I'm currently working on updating Rodio to 0.13 which will bring us to latest cpal too. From there it might not be too fiddly to use cpal for Jack

Ok I included the jack host into the rodio backend using your PR branch @willstott101 and it seems to work :) I can use jack as a rodio output device.
When the PR gets merged I can open a new one to add the feature

#548


changed to new openvpn repos

Changed the OpenVPN repositories to the new ones,
It will use OpenVPN stable repo and is now installing version 2.4.3
this solves issue #280
tested the script everything worked well!

Looks good, thanks!


RewardTrainer warns about max_length

System Info

Most recent trl (i.e. main as of a few hours ago)
transformers 4.44.2
torch 2.4.1+cu118
Python: 3.10.12
OS: docker nvidia/cuda:11.8.0-cudnn8-devel-ubuntu22.04

Information

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

Tasks

[X] An officially supported task in the examples folder
[ ] My own task or dataset (give details below)

Reproduction
When running the short RewardTrainer example with Qwen2.5-0.5B-Instruct, it first complains that max_length is not set. Setting it to 8192 runs, but with constant warnings:
/usr/local/lib/python3.10/dist-packages/transformers/tokenization_utils_base.py:2888: UserWarning: `max_length` is ignored when `padding`=`True` and there is no truncation strategy. To pad to max length, use `padding='max_length'`.

However, I can see in the source that samples above max_length are dropped silently, making the warning rather misleading as well as unnecessary.
Expected behavior
Have warnings reflect reality, perhaps by modifying truncation in the tokenization to be always off.

Hi, thanks for reporting, please share the system info

Hi, thanks for reporting, please share the system info

Anything in particular you think affects this problem?

Just share all of it, as explained in the issue template and in bug report guidelines

Done.
On further investigation, I think it may be coming from self.tokenizer.pad in RewardDataCollatorWithPadding
 File "/usr/local/lib/python3.10/dist-packages/transformers/trainer.py", line 1938, in train                                                                                │
│     return inner_training_loop(                                                                                │
│   File "/usr/local/lib/python3.10/dist-packages/transformers/trainer.py", line 2236, in _inner_training_loop                                                                                │
│     for step, inputs in enumerate(epoch_iterator):                                                                                │
│   File "/usr/local/lib/python3.10/dist-packages/accelerate/data_loader.py", line 561, in __iter__                                                                                │
│     next_batch = next(dataloader_iter)                                                                                │
│   File "/usr/local/lib/python3.10/dist-packages/torch/utils/data/dataloader.py", line 630, in __next__                                                                                │
│     data = self._next_data()                                                                                │
│   File "/usr/local/lib/python3.10/dist-packages/torch/utils/data/dataloader.py", line 673, in _next_data                                                                                │
│     data = self._dataset_fetcher.fetch(index)  # may raise StopIteration                                                                                │
│   File "/usr/local/lib/python3.10/dist-packages/torch/utils/data/_utils/fetch.py", line 55, in fetch                                                                                │
│     return self.collate_fn(data)                                                                                │
│   File "/app/trl/trainer/utils.py", line 380, in __call__                                                                                │
│     batch_chosen = self.tokenizer.pad(                                                                                │
│   File "/usr/local/lib/python3.10/dist-packages/transformers/tokenization_utils_base.py", line 3520, in pad                                                                                │
│     padding_strategy, _, max_length, _ = self._get_padding_truncation_strategies(                                                                                │
│   File "/usr/local/lib/python3.10/dist-packages/transformers/tokenization_utils_base.py", line 2888, in _get_padding_truncation_strategies
│ /usr/local/lib/python3.10/dist-packages/transformers/tokenization_utils_base.py:2888: UserWarning: `max_length` is ignored when `padding`=`True` and there is no truncation strategy. To pad to max length, use `padding='max_length'`.                                                                                ```


Feature/UC-152 uve + iteration. Iteration 4

Defines new endpoint for the rule engine integration.

We have added verfiy v2 definition verifies a digital health certificate for authenticity and validates a digital health certificate against a list of rules which vary per country.

Please document the error messages somewhere (if they are not from the rules) and lets do a cross check they are understandable.

@mwicke we are considering removing the endpoint /api/uve/v1/countries/validate. From our point of view, it seems like it is not really adding value to the API at this moment. But we could be missing something? What is your take on this?

@Carlos-Augusto as discussed, we can remove v1/countries/validate in the future, since the v2/verify endpoint makes this one obsolete


[WIP] wsk property command supports setting and deleting multiple properties

Process all auth properties that are specified
Include ok message for each property that is set
Validate new namespace in context of current property values, including those set via same command
Fixes #889

Add tests? WskBasicTests (see line 100).

@dubeejw - Ready for review

@rabbah - Added test set multiple property values with single command that validates setting auth, apihost, and apiversion all in one command

great - where? it's not pushed to this pr?

hmm.. file update was lost during attempt to patch that file to pick up changes from another pr.  Re-merged and squashed commits.

IPG 316

@dubeejw - Test case updated.  Any additional comments?  Thanks!


[FEATURE REQUEST] Enhancement of Footer Section across the web pages

Description
Is your feature request related to a problem? Please describe.
The Footer section across web pages has a few styling inconsistency.
Describe the solution you'd like
So I would like to suggest a footer code in a single file. This file will serve as a source for the footer across all web pages.
Describe alternatives you've considered
Additional Information
Context
Provide any additional context or information that helps explain the feature request.
Potential Implementation (optional)
If you have any ideas or suggestions on how the feature could be implemented, please share them here.
Screenshots or Mockups (optional)
Home page footer:

Features Page footer:

Priority (optional)
Medium.
Additional Notes (optional)
I would like to Contibute to this under SWOC 24.

Kindly assign this issue to me @akshitagupta15june

hey @akshitagupta15june please assign me this task under iwoc'24.

Kindly review and assign this issue to me under SWOC S4. @akshitagupta15june

hey @akshitagupta15june kindly review the issue and suggest some feed back about how you except the issue can be approached.

Hi!! I am Anshdeep singh, contributor in GSSoC'24. I would like to address this issue and make the respective changes.