domain_for_core_training_provider

Proposed changes:

closes #10145

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

I'm not really sure how to add a useful integration test here. What I could test is something like: If I change things in the domain (in some valid way -- i.e. I can't even directly reuse the "prune_domain" thing, I think) and re-train the whole pipeline (including featurizers because without them the core parts won't be affected as much), then does the core part look just like as if I trained without those changes to the domain -- but that whole test would work without this component :D. With this component included, I'm not sure how to approach that 🤔 . Any ideas?

I'm not really sure how to add a useful integration test here. What I could test is something like: If I change things in the domain (in some valid way -- i.e. I can't even directly reuse the "prune_domain" thing, I think) and re-train the whole pipeline (including featurizers because without them the core parts won't be affected as much), then does the core part look just like as if I trained without those changes to the domain -- but that whole test would work without this component :D. With this component included, I'm not sure how to approach that 🤔 . Any ideas?

I would do something similar to test_models_not_retrained_if_only_new_responses in test_model_training.py. i.e. change all the things which are pruned and check that it doesn't cause core to retrain.

I'm not really sure how to add a useful integration test here. What I could test is something like: If I change things in the domain (in some valid way -- i.e. I can't even directly reuse the "prune_domain" thing, I think) and re-train the whole pipeline (including featurizers because without them the core parts won't be affected as much), then does the core part look just like as if I trained without those changes to the domain -- but that whole test would work without this component :D. With this component included, I'm not sure how to approach that 🤔 . Any ideas?

I would do something similar to test_models_not_retrained_if_only_new_responses in test_model_training.py. i.e. change all the things which are pruned and check that it doesn't cause core to retrain.

ok yeah, I can include that as well -- had thought if I only test that the domain items get removed in the expected way, then I don't need to test that fingerprints will be the same and hence retraining won't happen, but it doesn't hurt 👍
What I had described above was another kind of test that checks correctness of the approach -- ie.. whether actually removing that information from the domain is ok and leads to the same result as if we did not remove it and retrain. But I think there's no good way to do that, no? 🤔

What I had described above was another kind of test that checks correctness of the approach -- ie.. whether actually removing that information from the domain is ok and leads to the same result as if we did not remove it and retrain. But I think there's no good way to do that, no? 🤔

Ah yeah I see - I think you're right that would be hard to do. I can't think of any way apart from the approach you mentioned above, and yeah seems a bit overkill/heavyweight.

I think it's still worth having a test in test_model_training that shows we can change these things in the domain without retraining policies.

Hmm, I thought I was doing that in test_train_core_with_original_or_provided_domain_and_compare. I'm basically using the provider to modify the domain instead of removing keys from the domain manually -- and then pass in that modified domain for re-training and check that the policy would be loaded from cache.  Does that make sense or am I skipping something then?
I could add the same changes to the domain manually again, but the other test should already take care of asserting that the respective keys are really removed/replaced, no? 🤔

Hmm, I thought I was doing that in test_train_core_with_original_or_provided_domain_and_compare. I'm basically using the provider to modify the domain instead of removing keys from the domain manually -- and then pass in that modified domain for re-training and check that the policy would be loaded from cache. Does that make sense or am I skipping something then?
I could add the same changes to the domain manually again, but the other test should already take care of asserting that the respective keys are really removed/replaced, no? 🤔

Missed that you'd added that there (was expecting it in a different file) - the test looks good 👍

@joejuzl @ancalita , any idea why e.g. pytest tests/core/test_processor.py -k test_restart_triggers_session_start now fails because because an action_default_fallback is predicted -- and only on Ubuntu?
We now get:
E       assert {'recipient_i...rephrase it?'} == {'recipient_i...there name1!'}
E         Omitting 1 identical items, use -vv to show
E         Differing items:
E         {'text': "sorry, I didn't get that, can you rephrase it?"} != {'text': 'hey there name1!'}
E         Use -v to get the full diff

while the events after in the handle_message call (before running prediction and returning the tracker that causes the error) look fine and end, as expected, with a slots set: {'event': 'slot', 'timestamp':<PHONE_NUMBER>.124695, 'metadata': {'model_id': '0a92dbf884b7409b9b4b37acf3db4fc8'}, 'name': 'name', 'value': 'name1'} .

@joejuzl @ancalita , any idea why e.g. pytest tests/core/test_processor.py -k test_restart_triggers_session_start now fails because because an action_default_fallback is predicted -- and only on Ubuntu?

hmm that's really weird... RegexMessageHandler should return a confidence of 1.0 (unless overwritten).
Is the component definitely running?

@joejuzl @ancalita , any idea why e.g. pytest tests/core/test_processor.py -k test_restart_triggers_session_start now fails because because an action_default_fallback is predicted -- and only on Ubuntu?

hmm that's really weird... RegexMessageHandler should return a confidence of 1.0 (unless overwritten). Is the component definitely running?

The RegexMessageHandler works fine, the event sequence produced by handle message is
(Pdb) tracker.events
deque([ActionExecuted(action: action_session_start, policy: None, confidence: 1.0), <rasa.shared.core.events.SessionStarted object at 0x179a8fac0>, ActionExecuted(action: action_listen, policy: None, confidence: None), <rasa.shared.core.events.UserUttered object at 0x179aadf70>, <rasa.shared.core.events.SlotSet object at 0x179aadd90>, <rasa.shared.core.events.DefinePrevUserUtteredFeaturization object at 0x179aa19d0>, ActionExecuted(action: action_default_fallback, policy: RulePolicy, confidence: 0.3), BotUttered('sorry, I didn't get that, can you rephrase it?', {"elements": null, "quick_replies": null, "buttons": null, "attachment": null, "image": null, "custom": null}, {"utter_action": "utter_default", "model_id": "a2f6bc2d229a4beeb0e7c305fb79b711"},<PHONE_NUMBER>.705086), <rasa.shared.core.events.UserUtteranceReverted object at 0x179a89b20>, ActionExecuted(action: action_listen, policy: followup_action, confidence: 1.0)])

with the UserUttered containing
(Pdb) tracker.events[3].intent
{'name': 'greet', 'confidence': 1.0}

@joejuzl  @ancalita  Found it: When parsing the stories and turn them into event sequences, we automatically insert explicit SlotSet events based on the from_entity mapping. The domain_for_core_training provider removed those mappings :). (References are in the docstr of the provider component, if you want to have a look).
After the fix, the only addition that is left is the "required slots" which was what Tobi asked for initially. Moreover, I couldn't find any references to forms in the story to event logic so, I'll enable merge if this passes now.


stop always reload tabs when switching groups

"Tabs opening" is set to "Full"  but would like an option to not reload tabs every time when switching between groups.  tabs should stay open and untouched when switching between groups.  reloading the tabs every time when switching groups causes unneeded network traffic and interrupts other activities for example paused video on youtube, place is lost; form being filled out is lost.  etc.

Select:

opening mode: discarded (never load the tab until you click on it)
closing mode: hidden (don't close the tab)


KYLIN-4055 cube quey and ad-hoc query return different meta info

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 4973

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 28.074%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

query/src/main/java/org/apache/kylin/query/adhoc/PushDownRunnerJdbcImpl.java
0
2
0.0%

Totals

Change from base Build 4972:
0.0%

Covered Lines:
23448

Relevant Lines:
83523

💛  - Coveralls

Codecov Report

Merging #823 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #823      +/-   ##
============================================
+ Coverage      25.6%   25.61%   +<.01%
- Complexity     6048     6049       +1
============================================
  Files          1398     1398
  Lines         83523    83523
  Branches      11701    11701
============================================
+ Hits          21390    21396       +6
+ Misses        60078    60074       -4
+ Partials       2055     2053       -2

Impacted Files
Coverage Δ
Complexity Δ

...ache/kylin/query/adhoc/PushDownRunnerJdbcImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../apache/kylin/cube/cuboid/TreeCuboidScheduler.java
66.15% <0%> (+2.3%)
0% <0%> (ø)
:arrow_down:

...che/kylin/stream/core/storage/CheckPointStore.java
66.66% <0%> (+3.03%)
22% <0%> (+1%)
: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 42ba57f...c148d6e. Read the comment docs.


Get help button is now a dependency of some troubleshooters

You should let get help button untouched as default because this buttton is now a dependency from new troubleshooters.
Without this button you cannot run the new troubleshooters (because is happen all in the get help app). Another things are all great maybe the best  debloat and optimize tool.

Dont forget to give the successor to Bloatynosy, Winpilot, a shot.


skeleton of config panel

skeleton of configuration panel components

I'm seeing this:

Is that expected?

nice, looks good to me

Why is it showing your changes in "files changed" section after I merged. Is that just how github works or do I need to rebase

Why is it showing your changes in "files changed" section after I merged. Is that just how github works or do I need to rebase

might just be because you merged main into your branch? I’m not really sure, did you rebase main into yours?


feat(DataTable): Add sorting params to header onClick callback

With this change the callback of DataTable header's onClick gives back the params of sorting as an object.
Like:
{
  sortHeaderKey: 'name',
  sortDirection: 'DESC,
}

New

Sorting params added to the callback function.

hi @moczolaszlo it looks like CI is failing because the bundle size threshold needs to be increased https://github.com/IBM/carbon-components-react/blob/master/package.json#L365
on a side note, is there an associated issue for this pull request for additional context?

Hi @emyarod
I updated the bundle size threshold, and I rebased the PR.
We need this parameters, because we want to sort a lot of data with paging.

Hi @moczolaszlo and @asudoh. I'm pretty new to React and Carbon. I'm trying to get both header's key and sort direction for my custom sort function. I have this onClick function:
<TableHeader
    {...getHeaderProps({
        header,
    })}
    isSortable={header.isSortable}
    onClick={(e, sortParams) =>
        console.log(e, sortParams)
    }>

However, it seems that sortParams is undefined. Am I doing this wrong?


meaning of #Frame

Hi,
Thanks for sharing this impressive work.
When I read the paper, I feel puzzled about #Frame in TABLE 3. (Comparison with the state-of-the-art on Kinetics400&600) in the paper https://arxiv.org/pdf/2201.09450.pdf. As for UniFormer-S， the #Frame is 16×1×4, what do 16,1, and 4 represent for？
Best regards！

#Frame = number_of_frames x number_of_crops x number_of_clips.
#Frame = 16x1x4 means we perform multi-clip testing with 4 clips and 1 center crops.
#Frame = 16x3x1 means we perform multi-crop testing with 1 clip and 3 crops (left, center and right).
In previous, CNNs[1] often adopts multi-crop and multi-clip testing at the same time, e.g., 16x3x10.
Recent video Transformers usually adopt multi-clip testing for Kinetics and multi-crop testing for Sth-Sth [2].
More analyses n our paper.
[1] Feichtenhofer C, Fan H, Malik J, et al. Slowfast networks for video recognition[C]//Proceedings of the IEEE/CVF international conference on computer vision. 2019: 6202-6211.
[2] Fan H, Xiong B, Mangalam K, et al. Multiscale vision transformers[J]. arXiv preprint arXiv:2104.11227, 2021.

Some hyperparameters can also be found in config.yaml and run.sh.
Please refer to README.
Multi-clip testing for Kinetics
TEST.NUM_ENSEMBLE_VIEWS 4
TEST.NUM_SPATIAL_CROPS 1

Multi-crop testing for Sth-Sth
TEST.NUM_ENSEMBLE_VIEWS 1
TEST.NUM_SPATIAL_CROPS 3

#Frame = number_of_frames x number_of_crops x number_of_clips. #Frame = 16x1x4 means we perform multi-clip testing with 4 clips and 1 center crops. #Frame = 16x3x1 means we perform multi-crop testing with 1 clip and 3 crops (left, center and right).
In previous, CNNs[1] often adopts multi-crop and multi-clip testing at the same time, e.g., 16x3x10. Recent video Transformers usually adopt multi-clip testing for Kinetics and multi-crop testing for Sth-Sth [2].
More analyses can be found in Figure 7 in our paper.
[1] Feichtenhofer C, Fan H, Malik J, et al. Slowfast networks for video recognition[C]//Proceedings of the IEEE/CVF international conference on computer vision. 2019: 6202-6211. [2] Fan H, Xiong B, Mangalam K, et al. Multiscale vision transformers[J]. arXiv preprint arXiv:2104.11227, 2021.

Thanks for your kind reply.
As stated in [2], "the input to the network are T frames with a temporal stride of τ ; denoted as T × τ". As far as I concerned, this is the sampling method  in readme .
So is there is any relationship between the #Frame and sampling method? If the #Frame = 16x3x10, what is the input to the network for one inference?
Best regards!

In fact, #Frame means the total input frames for testing.
For #Frame = 16x3x10, one inference will input 16 frames, but we will repeat 3x10 times and finally average all the predictions.

Thanks！


Fix beforeConnect docs

https://docs.altv.mp/js/articles/events/before-player-connect.html
"You can't compare passwordHash with anything yet. A function will come in the future to compare this.
"
and remove the player kick
and fix the params

https://github.com/altmp/altv-types/commit/d06d2d623793737971776418c81b4a6cebc488d3


Log in EOA wallet, enter DAO detail page, My Info model always loading even my-info api has return data

Description
Log in EOA wallet, enter DAO detail page, My Info model always loading even my-info api has return data.
Steps To Reproduce

Log in EOA wallet
Enter DAO detail page

Current Behavior

My Info model always loading even my-info api has return data

Expected Behavior

My Info model should display data after my-info api returned data

Environment

Platform: PC
Node: v18.18.0
Browser: Chrome 126.0.6478.56

fixed


Limit the version: "..." match to the project function

⭐ Suggestion
Update the "regular version" regex to limit matching to the def project function.
💻 Use Cases
version: "..." should not be matched and replaced outside the def project function.
This was initially overlooked because dependencies do not declare their versions in this way. But if another function were to declare a version key there might be an issue.

:tada: This issue has been resolved in version 1.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update moto to 0.4.20

Current coverage is 75.28%

Merging #46 into master will increase coverage by +1.15% as of 9e5dd11

@@            master     #46   diff @@
======================================
  Files            5       5
  Stmts          174     174
  Branches        22      22
  Methods          0       0
======================================
+ Hit            129     131     +2
  Partial          3       3
+ Missed          42      40     -2

Review entire Coverage Diff as of 9e5dd11

Powered by Codecov. Updated on successful CI builds.


Regex not working expected

Versions:

VueJs: 2.4.4
Vee-Validate: 2.0.0-rc.18

Description:
I show an alert if input text has url or e-mail but vee-validate not work expected with my regex.
My regex here work successfully;
https://regex101.com/r/cjl2AZ/3
But following example is not working as I expected;
https://jsfiddle.net/m67d8f4x/102/
Steps To Reproduce:

Just remove the g flag. duplicate of #925

@logaretm thanks for the answer but I want if my text contains URL and email show an alert can you help me with this case?

What is the purpose of this alert? if you want to do something else on input besides validation just add @input listener and do whatever you want in the handler. VeeValidate is only concerned with validation.

@logaretm If the input contains URL or email input not valid.
It's my example for this case;
https://jsfiddle.net/m67d8f4x/104/
If I write like this "Hi everyone my name is bla bla" input must be valid.
If I write like this "Hi everyone my name is bla bla visit here www.google.com" input must be not valid.
In my example you can write any text without URL or email you can see input not valid and add any URL like "www.google.com" you didn't see any errors.I have a reverse situation in my example
Sorry for bad English.

Just a create a custom rule:
https://jsfiddle.net/logaretm/m67d8f4x/105/


Link to Unit and Conformance Test Badges

Summary: Add Unit and Conformance Tests badges for all repos. Show badges in README.
Workflows

Name: <LANG> Unit CI
Name: <LANG> Conformance CI

Fields
For example the fields for adding these badges for workflow Node.js CI would be this:

IMG: https://github.com/GoogleCloudPlatform/functions-framework-nodejs/workflows/Node.js CI/badge.svg
LINK: https://github.com/GoogleCloudPlatform/functions-framework-nodejs/actions?query=workflow%3A"Node.js+CI"

Example

(We should make action names be something simple like unit instead of Node.js%20CI

Fixed with #37


useComponentValue broken in prod

When using the useComponentValue hook in production code (vite), I see the following error
TypeError: Cannot read properties of null (reading 'useState')
    at react_production_min.useState

This error only shows up in prod, and can be traced to this hook (toggling the useComponentValue hook on and off creates the error).
The error still appears with matching @types/react and react versions.
@smallbraingames

I'll second this, am also seeing this issue and have currently "fixed" by copy-pasting the hook into our client code

Is this still an issue?


README section on controller tests should explain need for "include Devise::Test::IntegrationHelpers" in rails 5

Issue
https://github.com/plataformatec/devise/issues/4819
Problem
In the Controller tests section of the README, it is stated that Devise::Test::ControllerHelpers needs to be included for devise to work in tests.
However, from Rails 5 controller tests no longer belong to ActionController::TestCase, and now belong to  ActionDispatch::IntegrationTest. Therefore, Devise::Test::IntegrationHelpers needs to be included instead of Devise::Test::ControllerHelpers.
Solution
I added a short passage in the README explaining this difference.

Great idea, done!

Thanks!


I'm doing something wrong for sure, but still gonna ask for help about modcharts

So I've been trying to make player's notes dissapear(dont ask why), but when I open up the song this message appears in console
"source/PlayState.hx:246: Mod chart: false - assets/data/notgonnagivenamelol/modchart.lua", using -debug doesnt show anything
I did everything what I should, any ideas why it shows it?

if the folder name has a space in it, something (probably Lime) dies inside and doesn't return the file properly.
also the modchart has to be named modchart (saying this just in case)

if the folder name has a space in it, something (probably Lime) dies inside and doesn't return the file properly.
also the modchart has to be named modchart (saying this just in case)

It is named modchart, and the folder has space in it, I'm guessing I just have to remove the spaces and use dashes instead

if the folder name has a space in it, something (probably Lime) dies inside and doesn't return the file properly.
also the modchart has to be named modchart (saying this just in case)

It is named modchart, and the folder has space in it, I'm guessing I just have to remove the spaces and use dashes instead

yeah spaces mess with asset loading. idk why

excuse me, do I hear issue with spaces
dies inside

It is named modchart, and the folder has space in it, I'm guessing I just have to remove the spaces and use dashes instead

You can name your song with spaces in-game,
but the files will be loaded with dashes instead
Example:
song name: This Song is Cool
internal name: this-song-is-cool

Yeah I had a name of my song with dashes in name for Dialogue and with spaces for charts, I was too lazy to change the name so I let the folder stay, now its usefull


datePicker on modal popup behind it

Modal with datePicker
I'm place datepicker on a modal. It showed behind of modal.
How to reproduce
Modal is already open. Tap datepicker and popup with dates showed behind of modal.
https://stackblitz.com/edit/clarity-light-theme-v3-next-rrieqj
Versions
App

Angular: 9.0
Clarity: 3.0-next8

This is related to https://github.com/vmware/clarity/issues/4213, same fix should address both of these.

Still present in RC1.
Workaround/hack (saas)
::ng-deep .clr-popover-content.datepicker
  z-index: 9000

This issue is resolved by this PR #4693


insertion of Demo error

Describe the bug
When I tried to import the demo from the community server, I encountered the following error message
`
Demo links
`Command failed: psql -c "\copy rounds(number,start_tick,start_frame,freeze_time_end_tick,freeze_time_end_frame,end_tick,end_frame,end_officially_tick,end_officially_frame,team_a_name,team_b_name,team_a_score,team_b_score,team_a_side,team_b_side,team_a_start_money,team_b_start_money,team_a_equipment_value,team_b_equipment_value,team_a_money_spent,team_b_money_spent,team_a_economy_type,team_b_economy_type,duration,end_reason,winner_name,winner_side,overtime_number,match_checksum) FROM 'C:\Users\22818\AppData\Local\Temp\cs-demo-manager\2023-12-08_23-57-49_8_de_vertigo_G2_vs_Navi_rounds.csv' CSV DELIMITER ','" "postgresql://:@*****:****/csdm"
错误:  null value in column "winner_name" of relation "rounds" violates not-null constraint
����:  失败, 行包含(110, e455f0729ce3191c, 21, 132942, 138279, 134094, 139428, 0, 0, 0, 0, G2, Navi, 12, 9, 2, 3, 26550, 21200, 8800, 1000, 0, 0, semi, eco, 0, 0, null, 0, 0).
����:  COPY rounds, line 21: "21,132942,138279,134094,139428,0,0,0,0,G2,Navi,12,9,2,3,26550,21200,8800,1000,0,0,semi,eco,0,0,,0,0,..."
Steps to reproduce
1.Select folder
2.select demo file
3.analysis demo
4. found error
Expected behavior
success but i can‘t analysis my demo
How often does this bug happen?
Every time
Log file
No response
Environment information
windows 11 x64
Additional context
No response

Could you please upload the demo?
It should be done in the "Demo links" template section.

@TResult97 This demo is corrupted - it stops in the middle of a round, the duration/round timeline is missing, and it doesn't contain the messages that a noncorrupted demo should have at the end.
Your demo:

Regular demo:

The next version will show a better error message to diagnose this issue that is not fixable.


Remove chatbox

not sure if this is the right place, but how do you remove the chatbox entirely from the website?
i have disabled everything from the dynmap configuration.txt ind i dont understand html, so can anybody explain it to me?

To disable the chatbox you will need to disable dynmap's chatbox component. This done by commenting out all lines relating to that component in configuration.txt.


Resolve sporadic failures in example-start running multiple servers

This PR resolves issue https://github.com/cypress-io/github-action/issues/657 "CI: Sporadic failures in example-start running multiple servers".
It modifies the action .github/workflows/example-start.yml:

by disabling update-check through environment variable NO_UPDATE_CHECK = 1 and
by updating serve to the latest version (14.1.2). serve Version 14 requires a minimum of Node.js 14. This is compatible with GHA v5 which removes Node.js 12 support. (The Node.js 14 version is in maintenance status and is currently available as version Node.js 14.21.1.)

Verification
Check that jobs of https://github.com/cypress-io/github-action/actions/workflows/example-start.yml are now regularly succeeding.
Release
The commits are tagged with test: which does not cause a release to be triggered. The PR only affects the CI runs of github-action, not the executable part of github-action.
The commits should be automatically listed in the release notes when a later PR with fix:, feat: or perf: commit tag is merged.

@jaffrepaul
All tests were successful!

Thanks @MikeMcC399! The action version bumps are handled in this PR. Let me try to get that merged today so you can rebase master and this will be even smaller.

Waiting for #644

@jaffrepaul
Thank you for the merge of your #644! Rebasing this PR was painless and as luck would have it all tests have passed!
So it's over to you now.


⚠ 366.ru has degraded performance

In 503446f, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 2472 ms

Resolved: 366.ru performance has improved in 444cc4f after 8 minutes.


Cannot view more than 250 records in sqlite table due to MAX_EXECUTE_RESULTS = 250

Currently there is no way to see more than 250 records in sqlite database tables using Stetho. Because of below setting in Database.java. Can you please explain is there a way for pagination or alternative to see all records in sqlite table using Stetho
/**

The protocol doesn't offer an efficient means of pagination or anything like that so
we'll just cap the result list to some arbitrarily large number that I think folks will
actually need in practice.

Note that when this limit is exceeded, a dummy row will be introduced that indicates
truncation occurred.
*/
private static final int MAX_EXECUTE_RESULTS = 250;

There is no way to see all records in stetho if you have more then 250 rows. There is no pagination in the protocol.
You could possibly sort and then select specific ranges.

Stetho should have a setter to let the dev set the number of rows.
If the dev didn't set it, then assuming 250 is appropriate.

There is no way to see all records in stetho if you have more then 250 rows. set LMIT and OFFSET for more data

@jasta how do you download database files using stetho?

See: ./scripts/dumpapp files


Apple-Website Clone

Project title
Apple-Website Clone
Project Description
Developed a website cloning project where the objective is to replicate the functionality and design of an existing Apple website.
Web stack
HTML,CSS,JavaScript
Screenshot(mandatory)

Additional Context(mandatory)
Are you contributing under any open-source program ?
Yes, KWOC 2023

@Dhammashree23 Welcome to METAVERSE!!!!!
please go through contributing.md before contributing


proxy

Dear ,
how to setup proxy for this program?
is there a chance if i used it alot that may instagram block my ip to login API?

No, the program doesn’t need your help to cover its tracks, just don’t uses it for illegal purposes

Thank you ,
so i the password list over 5000 password, the program will go through all the list right?

I programmed it to try everything, and to retry it if it gets a weird response, just don’t abuse it

it stuck searching for proxies ...

@nuriodaci thanks for letting me know, I’ll look into it as soon as possible.
What OS are you using? What version of Python are you running? How much RAM does your pc have?
How is your internet speed like? Do you use VPN to bypass geofencing? What country do you live in?

@Pure-L0G1C
Windows 10
Python 2.7
4GB Ram
7-8 Mpbs internet
I don't use VPN
Turkey
:)

@nuriodaci can you access the following links? https://sslproxies.org, https://free-proxy-list.net,
https://free-proxy-list.net/anonymous-proxy.html, http://spys.me/proxy.txt


Add schema definition for files in input_data task

In the input_data task there are cases where a file is optional and so if it does not exist in the folder, the flow will continue.
But, it will later fail because the code will reference that table. So this issue should create an empty dataframe for missing optional files and take in column names as parameters.

In decorators.py:
  if not os.path.exists(group + '/' + name):
      # It is not optional
      if not inner_optional_flag:
          raise FileNotFoundError(f"Non-Optional file missing: {name}")
      else:
          logger.info(f"Optional file missing: {name}")
          continue  # Skip processing if file is optional

So we are in the else case when the optional file is missing. This means that the file is not critical to the data but we need some placeholder in order for the flow to properly execute.
Right now we are handling it in the ETL like:
    if os.path.exists(settings.dictionary_dir + '/poison_substance_dictionary.csv'):
        previous_dictionary = pd.read_csv(settings.dictionary_dir + "/poison_substance_dictionary.csv", header=0)
    else:
        previous_dictionary = pd.DataFrame(
            columns=['input_substance', 'output_substance', 'drop', 'resolved'])

There are two approaches I am thinking of:

Passing in the column list when we are doing input_data and it will create it in the else case.
Create an input_dictionary handler (since in the ETL dictionaries are being imported separately from the data)
Default create all dictionaries based off a dictionary schema file and set_context on top of it, if the file does exist.

I like option #1, but I'd argue it would be good to pass the entire schema with dtypes through.
@input_data(parallel=False)
def get_data() -> dict:
    """
    Gets our flow data and stores it inside the context object.
    """
    return {
        settings.dictionary_dir: {
            'labs_dictionary': {
                'optional': True,
                'file': 'multi_column_dictionary_labs.csv',
                'default': {
                  'input': 'string',
                  'output': 'string',
                 'resolved': 'Int64',
                },
            }
        }
    }

Another separate option would be to have the flow provide a default dictionary as a df. We could just pass it around and Raritan would still not need to know what is in default.
default_dictionary = pd.DataFrame(columns=['input', 'output'])

@input_data(parallel=False)
def get_data() -> dict:
    """
    Gets our flow data and stores it inside the context object.
    """
    return {
        settings.dictionary_dir: {
            'labs_dictionary': {
                'optional': True,
                'file': 'multi_column_dictionary_labs.csv',
                'default': default_dictionary,
            }
        }
    }

I think option 1 could make the return function very long. E.g. medication topic has 5 dictionaries.
The second option is more like what we did for the filtering so I like that! I'll try that. Kind of like the 3rd suggestion I had - making a dictionary schema file that defines:
dictionary_name: {
                  'input': 'string',
                  'output': 'string',
                  'drop': 'Int64',
                 'resolved': 'Int64',
}

This is better than the current ETL setup anyways because we aren't currently defining the data types.


Game files not updating automatically

Game files are not updated automatically, even if I change the database to dynamic and it remains cached, it does not receive the necessary files.
}
https://github.com/user-attachments/assets/9271f96e-20a6-440a-9122-afb736d2b072

I fixed this dynamic mode: 9f90d36
PS: to update your game files, you need to use the update.php mode, like this cdn/gamefiles/update.php/86400/. The 86400 is the max time this file will be "cached". If DF update the swf in their servers, when requesting a specific swf, if the cache file is older than 86400 seconds (1 day), it will be requested to DF servers and the file will be saved.
The dynamic mode is when we have some local files (and patched files) and we have low disk space (or dont want to store all gamefiles)

I updated it to update.php/8644/ as you said, and when I enter the game it gives an error saying load failed.
I tried dynamic.php, same thing. (I applied your fix)
I can only enter the game files files by placing them in the folder beforehand.


Add EndpointSlice blog post in pt language

Add EndpointSlice blog post in pt language
Partial fix for #13939

/label tide/merge-commit-squash

/label tide/merge-method-squash

@jhonmike thanks for the suggestion, commited them. Added the merge squash method so Prow will squash the commits when the PR is accepted :)

@jhonmike can you issue also a /lgtm in a comment, so this gets merged?
Thanks!!

/lgtm
Sorry @rikatz, it was automatic


Add option to retrieve solar time

Add option to retrieve solar time according to current position

Closed in https://github.com/stkw0/zcock/commit/d625cfd2d75286d143c7da0ab464d67d632e4380


Exec to a pod results with "Failed to run cmd.exe" after few minutes

Description
When trying to exec to any pod everything goes smoothly, but after almost 3 minutes "Failed to run cmd.exe" message appears making further terminal use unusable.
Impact
User cannot use terminal longer than few minutes.
Environment and steps to reproduce
To better understand a problem, small video file is attached with all required steps.
https://youtu.be/_fMO1Ha9Qoc
Expected behavior
Exec into pod shouldn’t result in unusable terminal after a few minutes.
Additional information
Tested on Headlamp version 0.14.0 using Google Chrome on Mac machine.

@eresgie , Thanks for the detailed report and the helper video. This makes it very clear what the failed UX is. We'll look into this.
Can you in the meanwhile confirm that the container you successfully exec into is a Linux one (i.e. not using cmd.exe)?

@joaquimrocha Thank you, but I'm using Web UI and server version deployed on Kubernetes, so it could be hard for me to test it.


Weaken Precondition on State.CallReturn

Currently, we have the following preconditions on State.CallReturn:
function method CallReturn(vm:State) : (nst:State)
        requires vm.RETURNS? || vm.REVERTS? || vm.INVALID?
        requires this.CALLS?
        requires this.MemSize() >= (outOffset + outSize)
        // Contract address of executing account must still exist.
        requires vm.RETURNS? ==> vm.world.Exists(evm.context.address)

The one in particular I'm looking at here is related to the MemSize() requirement.  This is potentially too string in the case that outSize == 0.  Basically, when a contract call is not returning any information ... then we don't really need this requirement.  It could be weakened to the following:
requires outSize == 0 || this.MemSize() >= (outOffset + outSize)

That would make life a bit easier in proof objects for the case when outSize == 0 (e.g. for a transfer())

The other option is to remove the precondition entirely, and update the code so that it does the expansion.  Hmmmm, I wonder what is supposed to happen?

Ok, the issue is that the Bytecode.Call() is where the memory is expanded.  Therefore, when we are in the state CALLS we have an invariant that MemSize() >= outSize + outOffset.  Potentially we can just add that invariant ?

Ok, finally.  Adding the invariant to Dafny seems basically impossible with the current config.  However, the weakened precondition mentioned first makes sense as it reflects how memory expansion works --- i.e. that no expansion takes place when the length to expand is zero (outSize in this case).


Shall we get rid of 2.0-dev branch?

My understanding is that the 2.0-dev branch may have been used before creating / syncing https://github.com/kata-containers/kata-containers/tree/2.0-dev/tools/packaging.  Right now, its existence may be confusing (at least it's for me).
Shall we get rid of the branch entirely?

It sounds good to me, but I don't know who created the branch originally.
@jcvenegas - do you know the history?

I agree, now this is more valid as we have move to main naming

Branch removed


Suggestion: --initial-transfer alternative based on IV instead of CP

Documentation for --initial-transfer:

Start the bot with a pokemon clean up, keeping only the higher CP of each pokemon. It respects -c as upper limit to release.

It would be nice to have something similar but based on potential/IV instead of CP.

The docs are wrong, it does on both iv and cp. Set config.cp=0 nad config.pokemon_potential to the iv value (0.0 - 1.0)

Correct, initialen transfer takes info account CP and IV. Documentation needs to be changed.

Autocorrect on phone :( initial transfer takes into account ...

Is there an command-line flag equivalent to -c (upper CP limit okay to release) for IV?

Ah, that’s probably --pokemon-potential. I’ll update the docs.

Yep that's right :)
Thanks!

I’ll also assume that -c is a typo for -cp. ;)

PR: #197. Closing this issue.


crictl not taking --auth which is passed via CLI if registry is already configured in config.toml

I have old set of keys for auth in my conf.toml, when I tried to pull image via crictl and passed new token via --auth but seems to be crictl still taking old token which is in my conf.toml

pls reopen if this is still a problem


Migrate some files to ObjC++

Summary:
Converts a handful of .m files to .mm.
Changelog: [Internal]
Reviewed By: christophpurrer
Differential Revision: D51214054

Closing as abandoned internally


Enterprise 3.7.6 => 3.7.7 stomps configuration files

Running on Ubuntu 20.04, just did an apt update, and then an apt upgrade.
The ArangoDB Enterprise 3.7.7 package just replaces the local systemd service files with its own default configuration.
Never could get the hang of Thursdays, but having that happen at 0200 Pacific time allowed me to achieve visibility with customers in Europe.

This is still broken in 3.7.10
And it's wrecking my Saturday.


WIP Ruby2

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2227   +/-   ##
=========================================
  Coverage     86.89%   86.89%
  Complexity     5122     5122
=========================================
  Files           452      452
  Lines         20304    20304
  Branches       2175     2175
=========================================
  Hits          17643    17643
  Misses         1892     1892
  Partials        769      769

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 8f95d3b...d6ced18. Read the comment docs.


Bump http-server version

There is a known bug on http-server which could break the demo server.
It fix issue #724

Thanks @maxdow. Let me test this and merge the PR.


types for exec don't seem to be applied properly

The library supplies Typescript definitions (thanks! 😀), but the definitions don't seem to work for the exec() function. I can see it has been prototyped, but not in a way TS seems to recognise it.
      // no problem, TS knows `promise` is type Promise<YtResponse>
      const promise = ytdl('http://example.com', { writeInfoJson: true });

      // TS error: Property 'exec' does not exist on type '(url: string, flags?: YtFlags | undefined, options?: Options<string> | undefined) => Promise<YtResponse>'.
      const subProcess = ytdl.exec('http://example.com', { writeInfoJson: true });

I'm trying to see if I can make a PR to fix this, but I'm a bit of a Typescript noob. I think? the problem is the default export is a function with properties, and the solution might involve something like this

Hum, weird. I'm not TS expert neither ☹

I think I've got it. I had some help here


doi_common.add_orcid is dropping names

Looks like when I add a new orcid record using the doi_common function add_orcid, the family names and given names are not included in the new record. Here's what I did:
import name_match as nm
nm.initialize_program()
orcid_collection = nm.DB['dis'].orcid
doi='10.1021/jacs.4c03092'

And later on... (ignore the silly error around line 10)

Samba was successfully added to the database with the metadata on the last line, but his names were not included.

fixed!


Weld uses synchronized in ApplicationScopedContextualInstanceStrategy, so carrier thread is pinned

(This is not a Helidon issue, but more of a tracking issue to keep track of Weld's JIRA progress.)
See WELD-2788.
We have a user who has an application-scoped bean that does lots of expensive, IO-intensive work in its constructor. This is not a good thing to do under any circumstances, but Weld could be a little nicer here.
Specifically, Weld currently uses the synchronized keyword to ensure that (as you would expect) one and only one instance of the application-scoped bean is ever created. This is not virtual thread-friendly.
If it used Lock instead, then it would be virtual thread-friendly. WELD-2788 represents a probe to see if they'll be receptive to a PR that does this.
To reiterate, the proper thing is for the construction of a bean NOT to do lots of expensive IO.

Addressed: https://github.com/weld/core/pull/2978


Fix python compute_engine_creds and oauth2_auth_token interop tests.

-- grpc metadata keys need to be lowercase
-- switched python to use GOOGLE_APPLICATION_CREDENTIALS credentials
This also reveals some issues with python auth story:
-- user needs to implement metadata_transformer
-- there seems to be no metadata key checking/downcasing logic in python
It would be good to have issues for these, if we don't have them already.
Partially fixes #3842.

Looks like python's compute engine creds and oauth2_auth_token tests passed indeed.


Transmission plugin ignores addpaused for magnet links

I'm trying to figure out why this code is necessary. The comment doesn't make sense to me. Before I started using Flexget, I used to add magnet links in the paused state all the time. I don't see why magnet links would need an override of the addpaused option. Can anyone clarify?

It is unpaused to ensure the meta data is downloaded, which is required for stuff like main_file_only.

It'd make more sense to show me a warning if I have conflicting options enabled than to silently override my choice whether or not that's the case.

Ideally, it should pause it once it has been magnetized, but I haven't gotten around to it. Feel free to create a PR.

I don't need those additional options, so I just commented out the override. It's working as expected. Thanks for the background info though.

Sure, no problem. It's still something I'd like to add eventually though.

Would it make sense to add a separate option that forces a pause in all cases? Or, as I already suggested, show a warning for the combination of addpaused and main_file_only and others. I'm not a huge fan of commenting out production code. :wink:

I think we just need something here that calls pause if options['post']['paused'] and not downloaded or something on those lines.
https://github.com/Flexget/Flexget/blob/develop/flexget/plugins/plugin_transmission.py#L587-L591

Wait, is this the exact opposite #672? Do we just have some logic inverted somewhere?

I came across that issue before filing this one and yes, it had me confused. I just assumed it was outdated.

@Flexget/transmission anyone want to look into this and #672?

I will have some time in the holidays.

:+1:

Well, I cannot verify #672 because in my test case (there) addpaused is correctly ignored, the magnet-torrent downloaded immediately and never paused by flexget or transmission or whatever.
But now I'm a bit lost: do we want to change it? How it should work?

@tarzasai I am thinking that if addpaused: yes, then the magnets should be paused as soon as they are magnetized. Just my opinion though.

@cvium this makes sense. But it is the more complex to do.

perhaps right after this line we can just call cli.stop_torrent (is it the right method?).

Seems the right place to do it and I don't see any other method for doing it. Worth a try.

guess we forgot to close this one...


Regarding PreCo dataset

The PreCo dataset (https://preschool-lab.github.io/PreCo/) seems to be unavailable as no email was received for many days after filling in the details on the page. Though I know this is not an issue of this repository, I could not place my hands on it anywhere and thought maybe the community can help me here. Can someone point me to any version of the dataset available elsewhere?
Thank you

I believe that I got the replay almost instantly after filling in the info.
So I would suggest contacting them directly (there is a contact email on their page).
Anyway, I'll keep the issue open, hoping someone can help. Good luck!

@GorkaUrbizu thanks for the immediate reply. I tried multiple email ids and also mailed them a while ago and haven't gotten a reply. I am afraid that it has been down recently. If anyone could create a link for the downloadable link with the files I would be heavily grateful as it would help my academic research move forward.


MM-12982 Fix multiple message menus in single post

Summary
Fix multiple message menus in a single post. Similar to the fix on mobile.
Ticket Link
https://mattermost.atlassian.net/browse/MM-12982
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed

Considered add a test here but there wasn't anything existing that was to extend to test this case. Good news is that I'm working with this code as part of my work on interactive dialogs so I'll be adding tests for this there

Cherry-picked to release-5.5


cannot get lib num from blockstream

When I'm trying to connect to GRPC server using method Blocks(ctx, req, opts...) there is an error:
(firehose) cannot get lib num from blockstream, retrying {"retries": 4, "error": "tracking fetch errors in order: tracker block not found"}
sf.yaml config:
  args:
    #    - merger
    #    - mindreader-node
    #    - relayer
  - firehose
  flags:
    common-blockstream-addr: :9002
    firehose-grpc-listen-addr: :9000
    common-chain-id: "56"
    common-network-id: "56"
    mindreader-node-bootstrap-data-url: ./mindreader/genesis.json
    mindreader-node-log-to-zap: false
    mindreader-node-arguments: *skipped*

Hmm sorry for such long delay, have you succeed in solving your problem?

"Older" versions of the firehose used try to sync from a block stream (either the mindreader directly, or the relayer) to "track" what is the current head of the chain and determine if it is ready to serve data.
With your configuration, it is understandable that it couldn't connect (unless you had another process listening on port 9002).
If you want to run the firehose without a live source, you should set an empty string as the common-blockstream-addr.
Since v1.0.0, there isn't any concept of "tracker" anymore, so you shouldn't see this error.
Since this affects an older version, I will go ahead and close this issue, feel free to comment on it to open again if you need more help!


Using Admin Api update configuration successfully,but the json file not changed.when I restart the Ocelot service,the configuration was restored.

Expected Behavior / New Feature
Using Admin Api update configuration successfully,but the json file not changed.when I restart the Ocelot service,the configuration was restored.
        public static IWebHostBuilder CreateWebHostBuilder(string[] args) =>             WebHost.CreateDefaultBuilder(args)                 .UseStartup<Startup>().UseUrls("http://<IP_ADDRESS>:8000")           .ConfigureAppConfiguration((hostingContext, builder) =>           {               builder.AddJsonFile("Ocelot.json", false, true);           });
Ocelot.json file not changed.So I restart the service,the configuration was restored.
Actual Behavior / Motivation for New Feautre
when the configuration updated by admin api successfully,the configuration file should be updated synchronously?
Is there a way to solve this problem?
Steps to Reproduce the Problem

Specifications

Version:
Platform:
Subsystem:

@xiaoqiuGit thanks for your interest in the project.
There are a few similar issues already open. The way I handle the config just isn't very good.
I think what is happening...when you edit the config using the admin api it actually updates ocelot.{environment}.json on the disk. When you then restart ocelot it reads the configuration from ocelot.json because you do AddJsonFile("Ocelot.json") and that will overwrite whatever is in ocelot.{environment}.json.
This is a bug and I will fix it ASAP.

if You manually change the file while Ocelot is running it will not pick up the updates. The reason for this is Ocelot translates the file into an internal configuration. It is a bit messy and I know it needs to be improved :(

This is fixed in version 8.0.3, please check it out!


Add alert when fluentd queue buffers grow too large

Should we not warn at 100MiB as that is what the default Opensearch http.max_content_length is set as? Perhaps even warn at 99MiB

100 MiB seems like a sane default (99MiB could still technically be posted)


Converting to Diffusers ... Error

Converting to Diffusers ...
Traceback (most recent call last):
File "/content/convertodiff.py", line 1115, in
convert(args)
File "/content/convertodiff.py", line 1066, in convert
text_encoder, vae, unet = load_models_from_stable_diffusion_checkpoint(v2_model, args.model_to_load)
File "/content/convertodiff.py", line 835, in load_models_from_stable_diffusion_checkpoint
checkpoint = load_checkpoint_with_text_encoder_conversion(ckpt_path)
File "/content/convertodiff.py", line 817, in load_checkpoint_with_text_encoder_conversion
state_dict = checkpoint["state_dict"]
KeyError: 'state_dict'
rm: cannot remove '/content/stable-diffusion-custom': No such file or directory
Conversion error
Conversion error

same for me for some models. Which types are supported? safetensor, ckpt, pruned?

i was trying to use a .ckpt

Merged models don't convert for now, I'll fix it soon, try converting it to diffusers here https://huggingface.co/spaces/diffusers/convert-sd-ckpt

Merged models don't convert for now, I'll fix it soon, try converting it to diffusers here https://huggingface.co/spaces/diffusers/convert-sd-ckpt

didnt work for me, im trying to use this
https://huggingface.co/darkstorm2150/Protogen_x3.4_Official_Release/blob/main/ProtoGen_X3.4.ckpt

Came here hoping I would find an easier workflow, but for the moment this works.
Use the Automatic1111 Dreambooth extension to unpack the diffusers AND repack them to a CKPT
That CKPT is now trainable here.
(I can't get that Dreambooth to work in Colab, hence the runaround)

I tried with the new HassanBlend safetensor, (round 8gb), failed the same way in Dreambooth

Seems to work now, could train Protogen (but didn't like the model. Dreamlike v2 much better)!
@TheLastBen thanks for the fix 🎁

what error are you getting ?

CKPT_Path:
/content/gdrive/MyDrive/SD Data/Shirayuki_General_V1-fp16.ckpt
Or
CKPT_Link:
text hier einfügen
A CKPT direct link, huggingface CKPT link or a shared CKPT from gdrive.
Converting to Diffusers ...
Traceback (most recent call last):
File "/content/convertodiff.py", line 1115, in
convert(args)
File "/content/convertodiff.py", line 1066, in convert
text_encoder, vae, unet = load_models_from_stable_diffusion_checkpoint(v2_model, args.model_to_load)
File "/content/convertodiff.py", line 847, in load_models_from_stable_diffusion_checkpoint
info = unet.load_state_dict(converted_unet_checkpoint)
File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1667, in load_state_dict
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for UNet2DConditionModel:
Missing key(s) in state_dict: "up_blocks.0.upsamplers.0.conv.weight", "up_blocks.0.upsamplers.0.conv.bias", "up_blocks.1.upsamplers.0.conv.weight", "up_blocks.1.upsamplers.0.conv.bias", "up_blocks.2.upsamplers.0.conv.weight", "up_blocks.2.upsamplers.0.conv.bias".
Unexpected key(s) in state_dict: "up_blocks.0.attentions.2.conv.bias", "up_blocks.0.attentions.2.conv.weight".
rm: cannot remove '/content/stable-diffusion-custom': No such file or directory
Conversion error
Conversion error
Conversion error

Did you vhoose the correct version in "custom_model_version"

Pretty sure I didn't touch the setting, and I only do 1.5

if the model is public, send me the link, I'll test it

Both versions here are failing e.g.:
https://huggingface.co/hesw23168/SD_Shirayuki_Model/tree/main

they hold a different architecture, they are not supported by the diffusers library

Is there a way to convert them? Merging, safetensors, etc?

They probably modified the architecture of the model to avoid people training on them, but you can try converting them from a format to another then testing if they convert to diffusers


fix(disable-row): allow disabling through side effects without mutating rows

What kind of change does this PR introduce? (check one with "x")

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[X] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

@fh1ch Could you please review this MR?


change colour nav bar top and bottom

Hi guys,
We are trying to change the top navbar to a different colour but we can't find the location/path for the item. We've managed to change the colour for all other menus under app/assets/stylesheets/variables path.

Kindly assist

Hi @Ebrahim1010,
If you use either Firefox or Chrome browsers, you can right click on the area on the page and select 'Inspect element'. This will show you the underlying HTML and CSS that is being rendered. For example, in the screenshot below you can see that the navigation bar is picking up its background color from the '.navbar-inverse' class.

If yours also indicates that it is picking up the styling for .navbar-inverse, then the simplest way to update it to use the colors defined in the variables css file would be to change that navbar-inverse to navbar-branding in the view file:
https://github.com/DMPRoadmap/roadmap/blob/a08d7516c452f7c9b68a9014e5f38be61f3d2d08/app/assets/stylesheets/blocks/_navbars.scss#L52


Specifying ActiveRecord descendents custom views

I have a lot of columns in my model.[1] I love Hirb. Now Hirb can't do all of its magic because I have too many columns. But let's face it, I don't want all the columns. I only want certain columns.
I searched a bit in the docs, and tried the following:
Hirb.add_dynamic_view("Wadget", helper: :auto_table){ |o| {fields: [:id, :name, :token, :important_columns_only]} }

Though when I called my Wadget.all, this spec is still not activated as expected. I suspect it's because ActiveRecord::Base is there.
I realize I'm probably expanding Hirb's scope a little by asking this, but this is something I think that's not unreasonable to provide, especially with the prevalence of AR.
I'd like to add this to the wishlist: a way to indicate to Hirb fields I want (and this touches on #96 with fields one doesn't want). Ideally, this could be done in the config file, but I'll leave to the creator the liberty to say what he would accept.
What I would propose is to either make what I tried work (either by having add_dynamic_view prepend the list in order of priority, or having Hirb choose a view by the most specific), or being able to specify this in YML config:
views:
  fields:
    activerecord/base:
      exclude:
        - updated_at
        - created_at
        - secret_column
    wadget:
      only:
        - id
        - name
        - important_stuff

[1]: (It's probably not the ideal structure, but it's an old application with a lot of legacy)

Thanks for reporting your issue! This is one of my 13 active issues. Use that link to check how soon your issue will be answered. Don't forget to check your issue against this project's CONTRIBUTING.md. Cheers.

any chance this gets done? I'd really like this feature.

I hear pull requests are fun ;o)


[Question]  Fine-Tune

Question
I know there are some issues on Fine-Tuning, but I feel there is still some information lacking.
I have a custom dataset that I curated:
 [{
        "image_id": "data/common_final_images/COCO_train2014_000000102655.jpg",
        "Question 1": {
            "question": "q1",
            "answer": "a1"
        },
        "Question 2": {
            "question": "q2",
            "answer": "a2"
        },
        "Question 3": {
            "question": "q3",
            "answer": "a3"
        },
        "Question 4": {
            "question": "q4",
            "answer": "a4"
        }
    },
.......
}]

These questions are not chat i.e the order doesn't matter and they are independent of each other. I want to Fine-Tune LLAVA 7B on it. It seems like I have to call scripts/train_lightning.sh but It has both pre-training and instruction-tuning steps in it.
All I want to do is to take an already trained LLAVA model, just do some more instruction tuning on my custom 180 examples and see the results. Would appreciate the guidance on it. Thanks

HI @ahmadmustafaanis
Thank you for your interest in our work.  You can choose to: (1) keep them the same way as is; (2) split them to single-turn conversations.  Both are correct, but one may be better for your use case, which you may need to ablate.
You can choose to run the instruction tuning after either the pretrained llava checkpoint (stage 1), or instruction-tuned llava checkpoint. Both should work, and again, specific to your use case, one may be better than the other.
You can skip and comment out the pretraining step in train_lightning.sh :)

Thank you. I was a little confused about the data preparation, since the original data which I was looking was in the format of
[{'from':'human'/'gpt', 'value':'value'}]

So should I change my data format as well?
Also a lot of instructions were starting from <image> tag and ending with the same <image> tag as well. I wasn't able to understand it.

Hi @ahmadmustafaanis
For our latest code base, we removed that random placement, as we find that it is not giving additional performance gain as we have in the initial development stage. So, you can just put the  in the front.


Revert "[BEAM-14117] Unvendor bytebuddy dependency (#17317)"

This reverts commit 4b5efc3db490b4cc6a017a917cde561ace03f9b1.
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @cushon
cc: @kennknowles, @lukecwik

LGTM

Run Java_PVR_Flink_Batch PreCommit

This was superseded by a downgrade of bytebuddy version until the fix within bytebuddy is resolved.


[runwhen-local-feedback] simplify contribution feedback

Observation
In order to allow users to share their best troubleshooting commands with others in the community, we need to simplify the contribution process so that users can easily:

fire and forget (e.g. ship us a command, describe its purpose, and let RunWhen do the rest)
request help from the RunWhen team with troubleshooting commands

Possible Suggestions
I'm currently proposing a GitHub Issues or Discussions based approach, which should help constrain the conversation to this specific repo and allows teams to easily collaborate with nothing more than a GitHub account.
Any other details or context

We've opened up github discussions and linked them to the commands rendered in runwhen-local.


Add Package Project Url

This PR added a link to documentation for this package, for quick access from nuget browser.

@AntonyVorontsov What do you think about it?

Yup, sure, thank you


Refactor OS plugin to incorporate OCI OS UploadManager for enhanced  functionality.

Description
https://jira.oci.oraclecorp.com/browse/ODSC-43644
Refactors OS plugin to incorporate OCI OS UploadManager for uploading artifacts. The change enhancing functionality and improving performance.
Tests

For completeness could you please test with a LLM model with >= 7b parameters?


Improve node.js example in docs

This official documentation on dockerizing a Node.js app is so misleading, and should highlight not for production.
This official documentation on dockerizing a Node.js app is so misleading, and should highlight not for production.https://t.co/ieb0qkIxLK— Manoj Mahalingam (@manojlds) February 16, 2016

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

I'm the new beginner and I think I can work on this issue if it is not fixed. But I don't understand what "should highlight not for production" means.

Thanks @moxiegirl. Let me elaborate:

You don't do npm install in the production container.
You don't do just npm install, you have to do npm install --production to make sure only production dependencies are installed.

I'm going to modify the dockerfile according to your advice and add npm install --production into the dockerfile, like this:
FROM   centos:centos6
# Enable Extra Packages for Enterprise Linux (EPEL) for CentOS
RUN     yum install -y epel-release
# Install Node.js and npm
RUN     yum install -y nodejs npm
# Install app dependencies
COPY package.json /src/package.json
RUN cd /src; npm install --production
# Bundle app source
COPY . /src
EXPOSE  8080
CMD ["node", "/src/index.js"]
@manojlds Is it right?

#dibs

I wonder if we should use the official nodejs image in this example, in stead of using a (also very outdated) centos6 image as base. i.e. https://hub.docker.com/_/node/

@thaJeztah Yes, you should! centos6 uses Node v0.1, the current version is >4.x! I took the official Node.js Documentation (which is way better on this one, sorry) and updated the example accordingly, see my pull request #22128, not the issue.
Node.js Documentation: https://nodejs.org/en/docs/guides/nodejs-docker-webapp/

@thaJeztah I'm going to be removing this piece. I'm working on devicemapper stuff at the moment.


Add the module to the Testcontainers Modules registry

Proposal
After the process change in Testcontainers, the module should be submitted on our own. The guidelines are documented in https://github.com/testcontainers/community-module-registry
References

https://github.com/testcontainers/community-module-registry

Done


RFC WiP: injectable message validator for unknown fields.

In order to address https://github.com/envoyproxy/envoy/issues/6818 and https://github.com/envoyproxy/envoy/issues/6651, it's necessary to move away from having control over unknown fields be a static global (see https://github.com/envoyproxy/envoy/blob/4d4748ea243e0ab0538bcb0894a64bb0da617b55/source/common/protobuf/utility.h#L187) and switch to a model where we can:

Inject different validators, e.g. one for static config when ingesting bootstrap resources, another for xDS.
Validation state, so that we can reach back to the server stats for the purpose of tracking unknown fields.

This PR introduces a visitor pattern for the validator and starts to do the threading needed for dependency injection here.
It's possible that we could extend this visitor for deprecated fields in the future to provide finer grained control over these alongside unknown, if there is a use case.
However, there is a lot of churn and tedious boiler plate to make this work. This PR is only partially complete (it won't be build today, doesn't deal with listener or server modules yet), and is already quite large.
Interested in getting some initial feedback on this and possibly other approaches before growing this any further.
@mattklein123 @alyssawilk WDYT? Other ideas or Keep On Truckin?
Signed-off-by: Harvey Tuch<EMAIL_ADDRESS>

@mattklein123 OK, I'm continuing down this path. So far I've converted the //source/... but not tests. It's pretty tedious, so I'm hoping someone speaks up if they do object to this approach before I dive into the test code.

@mattklein123 this is ready for review. I suggest doing a pass as is, without CI passing if there are still failures, as any failures probably are interesting to independently examine as they get fixed.


Reorder suggested CORS Middleware location

Fixes #13660
Thanks @filipbekic01! 🏍

@guardrex  - @scottaddie  is out. Did you verify this with PU?

Unauthenticated preflight requests fail when CORS Middleware comes after Authentication Middleware.
This gets ugly here now because we don't also say that Static File Middleware has to come after CORS Middleware in scenarios where static files require CORS.
We sought to address this in a general way on https://github.com/aspnet/AspNetCore.Docs/issues/10839 because any middleware that may use CORS has to come after the CORS Middleware to work.
Perhaps, that PR (for the CORS topic) didn't go far enough. I did a quick search on the repo, and I don't see a single sample or text reference specifically for CORS and Auth Middleware ordering. I can open an issue to address it further in the CORS topic, but I'm 🏃 for a bit if you want me to work it.
For now, I'll ping for engineering. First tho, I'll try to touch this up for static files.

Ah! ... no need for a new issue.
We can piggyback that little update on the existing 3.0 update for CORES. :smile:

@anurse Do you have a sec to look? It's a quick one.


Add support for downgrading packages

Add the "-d" (downgrade) option, which causes upgraded packages to be
processed instead of newly installed items. Output will be in the
format:
package_name1=old_version1 [package_name2=old_version2 [...]]
This is intended to support downgrading packages to their previous (and
presumably working) version, in the event that an upgrade results in
breakage.

Thank you for making this tool available... it looks like it will be very helpful!
I've always found the downgrading of packages after a problematic upgrade to be an awkward, manual process... fortunately it doesn't occur all that often. But adding support for this scenario feels like a fairly natural addition to this project, so I decided take a stab at it here. :) Hopefully it meets with your approval.

The original PR had a typo in the getopts argument, which erroneously made it believe that the -h option required an argument. This is corrected in the version which was just force-pushed.

Thanks for joining, I appreciate your efforts :)
Correct me if I'm wrong, but there is a big problem with downgrading in apt: there are no "previous" versions for packages in *-upgrades and *-security. As only a new version is released, the previous version is gone.
The only reason why the sample code you added to the readme worked is this:
# apt policy dnsmasq
dnsmasq:
  Installed: (none)
  Candidate: 2.84-1ubuntu2.1
  Version table:
     2.84-1ubuntu2.1 500
        500 http://archive.ubuntu.com/ubuntu hirsute-updates/universe amd64 Packages
        500 http://security.ubuntu.com/ubuntu hirsute-security/universe amd64 Packages
     2.84-1ubuntu2 500
        500 http://archive.ubuntu.com/ubuntu hirsute/universe amd64 Packages

As you can see your "previous" version was the one from hirsute which is always available. Now take a look at what I have in focal right now:
# apt policy dnsmasq | grep -v i386
dnsmasq:
  Installed: (none)
  Candidate: 2.80-1.1ubuntu1.4
  Version table:
     2.80-1.1ubuntu1.4 500
        500 http://de.archive.ubuntu.com/ubuntu focal-updates/universe amd64 Packages
        500 http://security.ubuntu.com/ubuntu focal-security/universe amd64 Packages
     2.80-1.1ubuntu1 500
        500 http://de.archive.ubuntu.com/ubuntu focal/universe amd64 Packages

So if at some point in time I had 2.80-1.1ubuntu1.3 installed and upgraded it to 2.80-1.1ubuntu1.4, there is no way to undo because the version is no longer available. I could download it from launchpad and install manually, but that's another story. Hence there is only one generic way to downgrade: downgrading to the versions from the main repo (not focal-upgrades or focal-security but focal) which is not what most users would call "undo the last upgrade".

That's an unfortunate wrinkle I wasn't aware of. You're absolutely correct... this makes the intended purpose unreliable, at best. :(


Snap pops should not spill jugs

Description

I threw one at the cmo as clown which coincidentally landed on the pile of jugs nearby and all of them spilt onto the floor so they killed me on sight. My heart literally sank when I realised what happened, and apologising was not enough to quell their anger. 0/10.
Screenshots

to fix your trouble check this fix, i see it in another issue,
https://github.com/Jostino/RimWorld-it/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://github.com/Jostino/RimWorld-it/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

well i guess its closer to feature request actually


Terminal Inputs

While using gpt from the 'gogpt' command the user input doesn't know how to interpret arrow keys or hotkeys.

@carrotflowerr as in

up/down for previous/next prompt (history)
Left/right to move the caret

?


Meta: Consider pinning issues

On GitHub, you can pin an issue via its sidebar and it will appear on the issues tab like this https://github.com/microsoft/vscode/issues. This is a good place to highlight issues of general interest (perhaps an important project) or long term goals, like a roadmap, so that potential contributors can see what's important.
I don't have suggestions for specific issues, but hopefully some issues spring to mind.

Thanks for the suggestion! Our current approach is to use issue labels to guide contributors toward issues to work on.


DX-485 Add MCC Codes & dataScore Confidence Score to Transactions EP …

…& Enrich EP

@basiq-ash
Suggest removing this bit: "Find out more about pending
transaction and how to deal with them within your app."
The match type description should be something like this : "Indicates the method used to match a payment transaction with a merchant or business classification code. Description for each enum;
merchantFull: Indicates an exact match between the transaction and a known merchant in the database.
merchantPartial: Suggests a partial match with a merchant, usually based on substring or fuzzy logic.
mcc: Match is done using the Merchant Category Code associated with the transaction.
ml: Match is determined through machine learning algorithms for classification.
The match score description should be "The confidence score for the matchType which is a float value between 0 and 1. The higher the number, the higher the confidence in the match"

Thanks @kambasiq Open #307 to fix.


Refactor algo

Codecov Report

Merging #45 into master will increase coverage by 0.02%.

@@            Coverage Diff            @@
##           master     #45      +/-   ##
=========================================
+ Coverage   99.07%   99.1%   +0.02%
=========================================
  Files           6       7       +1
  Lines         324     334      +10
  Branches       57      59       +2
=========================================
+ Hits          321     331      +10
  Partials        3       3

Impacted Files
Coverage Δ

probscale/validate.py
100% <100%> (ø)
:white_check_mark:

probscale/viz.py
97.56% <100%> (-0.38%)
:x:

probscale/algo.py
100% <100%> (ø)

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 7c68aba...06cce45. Read the comment docs.


[BUG] window not darwn possibly due to size set to 0x0

Checklist before submitting an issue

[x] I have searched through the existing closed and open issues for eww and made sure this is not a duplicate
[x] I have specifically verified that this bug is not a common user error
[x] I am providing as much relevant information as I am able to in this bug report (Minimal config to reproduce the issue for example, if applicable)

Describe the bug
Earlier I reported, I am experiencing the zombie windows as described in #255 but in the meantime I believe this is actually a different issue.
The issue is, that windows are not drawn and when several windows should be open, it varies on each reload which window is actually drawn.
Snippet from eww logs:
Gtk-WARNING **: 17:25:31.006: Negative content width -25 (allocation 1, extents 13x13) while allocating gadget (node button, owner GtkButton)

(eww:4525): Gtk-WARNING **: 17:25:31.006: Negative content width -25 (allocation 1, extents 13x13) while allocating gadget (node button, owner GtkButton)

(eww:4525): Gtk-WARNING **: 17:25:31.006: Negative content width -25 (allocation 1, extents 13x13) while allocating gadget (node button, owner GtkButton)

(eww:36582): Gtk-WARNING **: 18:09:23.543: Negative content width -15 (allocation 5, extents 10x10) while allocating gadget (node box, owner GtkBox)

(eww:36582): Gtk-WARNING **: 18:09:23.543: Negative content height -15 (allocation 5, extents 10x10) while allocating gadget (node box, owner GtkBox)

(eww:36582): Gtk-WARNING **: 18:09:23.543: for_size smaller than min-size (1 < 3) while measuring gadget (node trough, owner GtkScale)

(eww:36582): Gtk-WARNING **: 18:09:25.545: Negative content width -15 (allocation 5, extents 10x10) while allocating gadget (node box, owner GtkBox)

(eww:36582): Gtk-WARNING **: 18:09:25.545: Negative content height -15 (allocation 5, extents 10x10) while allocating gadget (node box, owner GtkBox)

(eww:36582): Gtk-WARNING **: 18:09:25.545: for_size smaller than min-size (1 < 3) while measuring gadget (node trough, owner GtkScale)

(eww:36582): Gtk-WARNING **: 18:09:27.546: Negative content width -15 (allocation 5, extents 10x10) while allocating gadget (node box, owner GtkBox)

(eww:36582): Gtk-WARNING **: 18:09:27.546: Negative content height -15 (allocation 5, extents 10x10) while allocating gadget (node box, owner GtkBox)

(eww:36582): Gtk-WARNING **: 18:09:27.546: for_size smaller than min-size (1 < 3) while measuring gadget (node trough, owner GtkScale)

(eww:36582): Gtk-WARNING **: 18:09:29.547: Negative content width -15 (allocation 5, extents 10x10) while allocating gadget (node box, owner GtkBox)

(eww:36582): Gtk-WARNING **: 18:09:29.548: Negative content height -15 (allocation 5, extents 10x10) while allocating gadget (node box, owner GtkBox)

(eww:36582): Gtk-WARNING **: 18:09:29.548: for_size smaller than min-size (1 < 3) while measuring gadget (node trough, owner GtkScale)

$ eww windows
*tags
*left
notes
uhr
*right

For reference I removed my eww config and placed the example bar and also killed picom. This is when I saw this tiny sqare top center of my screen, and xprop on it revealed:
$ xprop
_NET_WM_DESKTOP(CARDINAL) = 0
_NET_WM_STATE(ATOM) = _NET_WM_STATE_ABOVE, _NET_WM_STATE_STICKY, _NET_WM_STATE_SKIP_TASKBAR, _NET_WM_STATE_SKIP_PAGER
WM_HINTS(WM_HINTS):
		Client accepts input or input focus: True
		Initial state is Normal State.
		window id # of group leader: 0x800001
_GTK_THEME_VARIANT(UTF8_STRING) = "dark"
_NET_WM_STRUT_PARTIAL(CARDINAL) = 0, 0, 57, 0, 0, 0, 0, 0, 0, 2559, 0, 0
_NET_WM_STRUT(CARDINAL) = 0, 0, 57, 0
XdndAware(ATOM) = BITMAP
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x2, 0x0, 0x0, 0x0, 0x0
_NET_WM_WINDOW_TYPE(ATOM) = _NET_WM_WINDOW_TYPE_DOCK
_NET_WM_SYNC_REQUEST_COUNTER(CARDINAL) = 8388613, 8388614
_NET_WM_USER_TIME_WINDOW(WINDOW): window id # 0x800004
WM_CLIENT_LEADER(WINDOW): window id # 0x800001
_NET_WM_PID(CARDINAL) = 36582
WM_LOCALE_NAME(STRING) = "de_DE.utf8"
WM_CLIENT_MACHINE(STRING) = "manjobook"
WM_NORMAL_HINTS(WM_SIZE_HINTS):
		program specified minimum size: 2304 by 43
		program specified base size: 0 by 0
		window gravity: Center
WM_PROTOCOLS(ATOM): protocols  WM_DELETE_WINDOW, WM_TAKE_FOCUS, _NET_WM_PING, _NET_WM_SYNC_REQUEST
WM_CLASS(STRING) = "eww-bar", "eww-bar"
WM_ICON_NAME(STRING) = "Eww - bar"
_NET_WM_ICON_NAME(UTF8_STRING) = "Eww - bar"
WM_NAME(STRING) = "Eww - bar"
_NET_WM_NAME(UTF8_STRING) = "Eww - bar"

Reproducing the issue
I tried to rustup update and cargo clean before doing a fresh build of eww.
As mentioned above I tried my personal config as well as the example config.
Expected behaviour
Windows should just be drawn.
Additional context
I have not checked, if this appears in any other WM then leftwm yet (I am pretty positive though, that it is not leftwm as I reset my git repo tree to a version I new was working cleanly and had the same issue), also have not checked if this issue is know with gtk bug tracker.

:thinking: tried bspwm and i3 and they seem to run ok.
so it has to be somewhat related to my leftwm session, but I don't see how atm.
Any further ideas how I could debug this?

Ok, this is very weird.
When I set the windows to wm-ignore = true the are getting always drawn. but reserve does not work anymore, which kind of makes sense.
I have one window wich has (deflisten workspace-layout 'leftwm-state -w 0 -s "{{workspace.layout}}"') the return values in my WM config can be one of the following: Monocle, MainAndDeck, LeftWiderRightStack, CenterMainBalanced
When the window has size 0x0 and I change to one of the latter two layouts it suddenly gets drawn again.
The xprop output is unchanged by this.
Would it make sense to set the program specified base size in WM_NORMAL_HINTS(WM_SIZE_HINTS)?

So is the issue that the window, when given size :width 0 :height 0 in the config, actually has the configured size of 0px*0px when initially opened, not respecting the fact that there is content to be rendered - and after changing the value the window size adjusts to fit the content?

Windows actually have a configured size:
Here is a snippet of two of the windows:
(defwindow left
           :monitor 0
           :windowtype "dock"
           :stacking "overlay"
           :geometry (geometry :x "21px"
                               :y "13px"
                               :width "295px"
                               ; :height "34px"
                               :height "44px" ; this is a workaround, as gtk is expanding the height to this anyways
                               :anchor "top left")
           :reserve (struts :side "top" :distance "55px")
    (sidestuff_left))

(defwindow tags
           :monitor 0
           :windowtype "dock"
           :geometry (geometry :x "0px"
                               :y "13px"
                               :width "19%"
                               ; :height "34px"
                               :height "44px" ; this is a workaround, as gtk is expanding the height to this anyways
                               :anchor "top center")
           :reserve (struts :side "top" :distance "55px")
           :wm-ignore true
    (tags))

Could this be related to rust-nightly?
It seams odd to me, that rustup says I'm on 1..57.0-nightly (5ecc8ad84 2021-09-19) but starshp prompt detects 1.56.0-nightly for my repo direcotry..

WM_NORMAL_HINTS(WM_SIZE_HINTS):
		program specified minimum size: 320 by 44
		program specified base size: 0 by 0

Is this set somewhere in a depenency (gtk?) or are you setting these 'manually'? As I can't grep for it, probably dependency, right?

Context: LeftWM 0.2.8, Git-Hash: 3c80a4a
@VuiMuich i changed the base size, it doesn't solve the issue.
Also i compared eww with polybar and lemonbar. They both have only a few lines of xprop output.

Polybar and lemonbar use xcb to initialize the window i think.

These are screenshots i took
:wm-ignore false:

:wm-ignore true:

As you can see there's a small dot on the first screenshot. It is the same window which shows "example content".

fixed with https://github.com/leftwm/leftwm/pull/498

@VuiMuich do we still assume this to somehow also be a bug / non-spec-compliant behavior in Eww, or do you reckon this was just weird behavior from leftWM? I've kinda lost track of this issue over the past few weeks, I'm sorry ^^

Well the fix on the leftwm side doesn't hurt, as it basically prohibits resizing for dock type windows.
Together with @ZetaNumbers and @AethanFoot (who then contributed leftwm/leftwm#498) we found that the i3 fix from #258 introduced this behaviour.
Specifically this if in crates/eww/src/app.rs lines 377ff:
    if actual_window_rect.x != gdk_origin.1 || actual_window_rect.y != gdk_origin.2 {
        gdk_window.move_(actual_window_rect.x, actual_window_rect.y);
    }

I am not sure at this point, if leftwm isn't operating on spec in this regard.


Merge to live!

Range.moveTo section
Fix link
Correcting headers
Correcting headers

Docs Build status updates of commit af58175:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in https://SiteHelp and include the URL of publishing report as well as below information: One or more errors occurred..

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.

@AlexJerabek I accidentally merged this to live even though there was a validation error. Can you please revert this, fix whatever is wrong (it could just need to be remerged to master) and let me know? Thanks!

@lindalu-MSFT The build report says it's a backend error. I'll kick off another build of the master branch to confirm.

@AlexJerabek OK I'll keep my eye on it! Thanks for responding so quickly.


Trying to test checkpoint of a model during training process

I tried to merge Lora adapter of a model in training process, but I get this result
Loading Bunny from base model...
Loading checkpoint shards:   0%|          | 0/4 [00:00<?, ?it/s]
Loading checkpoint shards:   0%|          | 0/4 [00:00<?, ?it/s]
Traceback (most recent call last):
File ".../Bunny/script/merge_lora_weights.py", line 26, in
merge_lora(args)
File ".../Bunny/script/merge_lora_weights.py", line 10, in merge_lora
tokenizer, model, image_processor, context_len = load_pretrained_model(model_path, args.model_base, model_name,
File ".../Bunny/bunny/model/builder.py", line 67, in load_pretrained_model
model = BunnyLlamaForCausalLM.from_pretrained(model_base, low_cpu_mem_usage=True,
File ".../.local/lib/python3.10/site-packages/transformers/modeling_utils.py", line 4225, in from_pretrained
) = cls._load_pretrained_model(
File ".../.local/lib/python3.10/site-packages/transformers/modeling_utils.py", line 4728, in _load_pretrained_model
new_error_msgs, offload_index, state_dict_index = _load_state_dict_into_meta_model(
File ".../.local/lib/python3.10/site-packages/transformers/modeling_utils.py", line 993, in _load_state_dict_into_meta_model
set_module_tensor_to_device(model, param_name, param_device, **set_module_kwargs)
File ".../.local/lib/python3.10/site-packages/accelerate/utils/modeling.py", line 286, in set_module_tensor_to_device
raise ValueError(
ValueError: Trying to set a tensor of shape torch.Size([128256, 4096]) in "weight" (which has shape torch.Size([32000, 4096])), this looks incorrect.
$ls run_dir
checkpoint-2400
I only see the checkpoint dir,
$ls run_dir/checkpoint-2400
adapter_config.json        latest           rng_state_1.pth  special_tokens_map.json  trainer_state.json
adapter_model.safetensors  README.md        rng_state_2.pth  tokenizer_config.json    training_args.bin
global_step2400            rng_state_0.pth  scheduler.pt     tokenizer.json           zero_to_fp32.py
How to merge Lora before training finish? It take long time and I want to check if the model is ok (by checking generation) before it is done.
Setup: Llama-3
deepspeed 2 lora finetuning

Ok solved
I can just point at "bunny-lora-llama3-8b-Full-dataset-copy/checkpoint-2400"
model = AutoPeftModelForCausalLM.from_pretrained(
checkpoint_path,
device_map="auto",  # Automatically distributes the model across available GPUs/CPU
trust_remote_code=True,
torch_dtype=torch.float16
)


ioloop.install() no longer necessary for pyzmq >= 17

Resolves #52

Weird that travis-CI isn't reporting status here, but it's green: https://travis-ci.org/github/rdeits/meshcat-python/builds/680375559

Unfortunately, this breaks Drake due to it using python3-zmq (16.0.2) from Bionic:
https://github.com/RobotLocomotion/drake/pull/13155
https://drake-cdash.csail.mit.edu/testDetails.php?test=37559421&build=1287215
I've confirmed that reverting this single commit enables the Drake tests to pass.
Is it possible to revert this commit and keep the vestigial part in there? Or do a small version check?

@EricCousineau-TRI Can you open a PR with the version check guard on the looper install statement?  That way, we'll know for sure that its compatible with Drake instead of asking rdeits to test it for us.

Yup, will do!

Filed: https://github.com/rdeits/meshcat-python/pull/58
Will update Drake PR.


mind mapping

建议xminder重绘一下@MrXsc

It's not an easy question. 🤤
Basically, Mr. He @SSHeRun has made a wonderful work.

ok,I will close this issue.


Iterate slice by reference (not by value)

Iterating a Slice(T) with #each leads to iterating the slice by value. This is usually fine when T is a primitive (e.g. UInt8) or a Reference to a class instance, but when T is a struct it leads to getting a copy. Sometimes this is what we want, but at other times we'd like to avoid the copy and/or need to mutate the struct in-place.
This stems from the nature of Ruby where everything's supposed to be a Reference or a primitive, not a value that is passed by copy (except for primitives).
To solve this, we currently need to get the raw pointer and loop/increment it manually, but the intent ain't immediately explicit, in addition to be ugly Crystal:
slice.size.times do |i|
  ptr = slice.to_unsafe + i
  # ...
end

We could at least abstract a method, for example #each_ref(&) or #each_reference(&). We'd still have to deal with a Pointer(T), but at least the iteration part would be much more readable:
struct Slice(T)
  def each_ref(&)
    size.times { yield to_unsafe + i }
  end
end

slice.each_ref do |ptr|
  # ...
end

I found this post talking about a similar problem with Go: https://jvns.ca/blog/2024/08/06/go-structs-copied-on-assignment/. The story of that blog post is that the author previously hadn't even noticed that structs are copied on assignment before and it came up in an iteration situation.
In Go you can explicitly pass structs by reference with taking a pointer (&thing, return type *Thing). Apparently dereferencing happens implicitly, so this is quite transparent for the receiver.
Anyway, the loop in the following function still caused an implicit copy, so the returned value is not a reference to the array item but a copy from the loop assignment (note: because the function returns a reference to this copy, it cannot be placed on the stack).
func findThing(things []Thing, name string) *Thing {
  for _, thing := range things {
    if thing.Name == name {
      return &thing
    }
  }
  return nil
}

This is simple to resolve by changing the iteration style. Apparently taking the pointer directly from an array index accessor doesn't involve a copy and it directly returns a reference to the array item:
func findThing(things []Thing, name string) *Thing {
  for i := range things {
    if things[i].Name == name {
      return &things[i]
    }
  }
  return nil
}

Following the already existing Pointer#map! I think an associated Pointer#each would fit quite well. And Slice could provide a safe/convenient method on top of that, using the inherent size.
Considering this, the method should perhaps be named Slice#each_pointer?

Though actually I don't see the relation to Pointer#map! - it actually yields T and not Pointer(T) as we want here.
Then Pointer#each would not be relevant either because it should stay consistent with #map.

I think that the core problem isn't just iterating by reference but also accessing a value by reference into the slice.

What do you mean by accessing?
You can do (slice + 7).to_unsafe to access the 7th index element. Or slice.to_unsafe[7] without bounds checking.

I mean the + part to get a pointer to i, not dereference the pointer at index i to get the value. I can do (slice + i).to_unsafe but this is far from explicit, while something like slice.pointer_at(i) would be.

Yeah, I figure pointer_at could be a good addition.


Media ContentTypes Localization enhancements

As per Taxonomies, ContentPickerFields and others contents (see #7352) we should consider to enhance localization of media's Content Types (Image, Documents, etc).
By now if you have a multilanguage site and you want to seriously manage SEO and w3c recommendations you have to do a lot of manual tricks in order to manage alternate textes, title, etc... (You have to upload a new media - even if it is the same of the master content - and then you can manage the media alt, title, etc)
I have some ideas on how we could manage it.
A feature "Media Localization Extensions"

It adds a LocalizationPart to interested Content Types (I have to go deeper in code to understand where it would be better to add it).

Translating a "Media" should clone the media file too? I think yes, it should. What do you think?
In the list of media we should put in evidence the language of that media in order to not mislead the user.

It adds a Setting "Localize media on translating" to all MediaLibraryPickerFields (default is true). When a content - which contains a MediaLibraryPickerField - is translated and the setting is true

all the medias already translated will be replaced with their translation
all the medias not trasnlated will be cloned, so user can write the translation of textes.
The Localization of Content and its medias should stay always in sync

@sebastienros @MatteoPiovanelli-Laser @jersiovic @BenedekFarkas @urbanit What do you think?

A couple comments to add to that.
Regarding cloning the actual files:
I don't know about that. Maybe they are already managed in such a way that there would be no need for that. On top of that, not al media have a corresponding file in storage.
Regarding the fields:
Basically what we did for ContentPickerField, so I don't see any issue there.

I don't think media files should be cloned. But there needs to be a way to change which file is attached if the translation needs a different one.

Otherwise I don't see how localization should be different than another content item for media. Some fields should be synced, content should be cloned, ...

Yeah, I agree, the files themselves should not be cloned (at least initially), so basically the MediaPart.Path property is translation-independent - I think what matters most here is the title (caption), which should also be cloned, but can be changed (translated).
Although the idea that the files could also be independent across translated versions sounds interesting (e.g. they might have a specific flag (even literally) or some other, language/country-specific feature on the image), but I think that brings us to a bigger question: Should it be possible during editing a Media item to change the actual file in the storage in points to? On a related note, I usually associate Media items with Images at first glance, but actually if you're thinking about documents, it totally makes sense to have the localized versions point to different files - in that case, there should be a generic way to edit the item and select/upload another file (or files).

I think handling the actual media, be it a file or whatever, is the actual crux here.
It will be images, videos, documents, oembed content. And ideally there should be a single interface handling all of that, independent of media type and storage.
I don't know in detail what is going on there at the low level, so I will need to look into it before being able to say more.


add widedeep model

I took the dataset for this model from here:
https://github.com/IntelAI/models/blob/master/datasets/large_kaggle_advertising_challenge/README.md
I supply already preprocessed dataset.
I took the model from here:
https://github.com/IntelAI/models/blob/master/benchmarks/recommendation/tensorflow/wide_deep_large_ds/inference/fp32/README.md

@MarcelWilnicki this PR doesn't need a lot of work to get to mergeable state, let's get it done so it's not left behind

@MarcelWilnicki this PR doesn't need a lot of work to get to mergeable state, let's get it done so it's not left behind

I agree, I will deal with it after I am done with mlperf bert model


[OS/2] Size fields should be uint16, not int16

All these fields, as well as underlineThickness in the post table, should be uint16, not int16, since negative sizes are meaningless. (The correct type is already used for the optical point size fields.) Of course, correct typing helps implementations to catch font errors.
int16	ySubscriptXSize
int16	ySubscriptYSize
...
int16	ySuperscriptXSize
int16	ySuperscriptYSize
...
int16	yStrikeoutSize
...
uint16	usLowerOpticalPointSize
uint16	usUpperOpticalPointSize

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

ID: 1ae9457c-0d3d-c137-588b-81d54a086f07
Version Independent ID: 658c6d22-b9d1-1007-9cdb-6898911c9c75
Content: OS/2 - OS/2 and Windows metrics table (OpenType 1.9) - Typography
Content Source: typographydocs/opentype/spec/os2.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: alib

These were signed from the beginning since (for whatever reason) the implementation in OS/2 used signed values. Conceptually unsigned would make sense, but changing now could potentially lead to fonts with large values that overflow existing implementations. The best remedy would be to add a recommendation that values be greater than zero.

See the OT 1.9.1 alpha for draft revisions addressing this issue.

This is fixed in OT 1.9.1. See the OT 1.9.1 beta for draft revisions addressing this issue.


[Bug] Operator don't scale up if replicas in cluster are set to 0

Search before asking

[X] I searched the issues and found no similar issues.

KubeRay Component
ray-operator
What happened + What you expected to happen
I installed a managed cluster in Azure, with two node pool (one for cpus and one for gpus), and I installed kube-operator and autoscale.

When I try follow example:
https://docs.ray.io/en/latest/cluster/kubernetes/examples/rayjob-batch-inference-example.html#kuberay-batch-inference-example
the operator correctly scale up the cluster and execute the job. Once the job finish, the cluster scale down correctly too.
When I try follow example:
https://docs.ray.io/en/latest/cluster/kubernetes/examples/gpu-training-example.html
the job is correctly executed.
anyway, in the second case, if I set in ray-cluster.gpu.yaml
replicas: 0
minReplicas: 0
The operator don't scale up the cluster and the job is not started. The issue looks similar to #321 but I didn't understand if there is a fix.

Reproduction script
Execute the example described in:
https://docs.ray.io/en/latest/cluster/kubernetes/examples/gpu-training-example.html setting in ray-cluster.gpu.yaml
replicas: 0
minReplicas: 0
Anything else
Because I'm using Ray in cloud, my aim is to scale up automatically and free resource when are no longer necessary. How I can do?
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

The YAML file doesn't enable Ray autoscaling. You need to set enableInTreeAutoscaling in your RayCluster YAML. See
https://docs.ray.io/en/latest/cluster/kubernetes/user-guides/configuring-autoscaling.html#enabling-autoscaling for more details.

Thank you. I was expected that writing minReplicas and maxReplicas in configuration files the autoscaling was automaticaly enabled, so I din't add this option. Now it works perfectly.