Make rule edits apply instantly on the whole page after save

There is one issue. When editing a rule, its json representation is updated in real time before the rule is even saved. But some fields like uid and trigger.ref are only updated after saving. Not sure what's the best way to fix this.

@enykeev thoughts?

LGTM as long as it works, though I wonder whether we should have instead got rid of https://github.com/StackStorm/st2web/blob/rule-instant-update/apps/st2-rules/controller.js#L135-L137

@alex-shnayder We do need to update triggers: while it's the property that probably gets changed the least, it's still strange and unexpected behavior, because a trigger is there but its description is not. So maybe capturing a "post-save" event, or fetching the rule again after save would be a way to solve this? @enykeev and @manasdk can probably contribute to the discussion.

@emedvedev I'm not sure I understand you (or you understood me). Most of the changes to a rule are reflected in its JSON definition instantly after you change the value of a field. But there are some properties that cannot be changed directly in the interface, they are generated server-side. These properties only get updated in the JSON definition after you click the Save button.
Let's say, for example, that you have a rule with examples.event trigger. Its JSON definition looks like this:
"trigger": {
  "ref": "examples.event",
  "type": "examples.event"
}

Then you switch the trigger to core.st2.IntervalTimer. At this point JSON becomes inconsistent:
"trigger": {
  "ref": "examples.event",
  "type": "core.st2.IntervalTimer"
}

After you click Save, a request is made and the definition is updated to the consistent state:
"trigger": {
  "ref": "core.st2.IntervalTimer",
  "type": "core.st2.IntervalTimer"
}

Everywhere else on the page changes are only applied after saving.

I meant a slightly different issue — when you update a trigger, its description does not get updated in the rule list, it's just empty. That needs to be fixed, but refs and uids are less important, we can merge without updating them.

Fixed.

Rebased to fix an issue in newer code.

+1.


Automatically cleaning unicode text

Thanks for this awesome tool! I was wondering if we could include some sanity checking/cleanup for badly behaved text (e.g. all those invalid unicode characters). Could be as simple as running ftfy on all text columns. I'd volunteer to integrate this into datacleaner.

Sounds promising. Please submit a PR with the new functionality along with unit tests to demonstrate how it works.

I've implemented a draft of this but realized it may clash with the functionality of converting all text to numerical values. I wonder how to proceed, as I see it there are two options:

Fix the text before applying the encoding: This is what I'm doing right now, so strings like '>=50' and '>=50' get encoded to the same label.
Make encoding optional: This is tricky, there will be some text-based columns where you want to preserve the text to featurize later (e.g. with a sklearn.feature_extraction.text.TfidfVectorizer) rather than convert them to a label with an encoder. The tricky part is how to specify what columns you want to encode or not.

One way to proceed would be to go with 1 and then tackle 2 in a later issue.


Update CHANGELOG-2.0.md

Fixed links for changelogs for versions 1.1.x & 1.2.x.

Q
A

Type
bug

BC Break
no

Summary
Fixed links for changelogs for versions 1.1.x & 1.2.x.

Thanks a bunch @ajant!

I'm glad I could help.


use fzf with commands like fzf vim

and if you do that, you get the fzf way of selecting a file, and when you press enter you open that file with vim. so basically add the file you found as the last arg of the command afterwards.
also check here https://getpicka.com/guide#install-cli
$ pi cd

Edit your selected file using vim

$ pi vim

Concatenate the file named “foo” and your selected file

$ pi cat foo

Count the number of lines matching the pattern “Info” in your selected file

$ grep Info "$(pi)" | wc -l

Isn't it something you can easily do with shell script?
pi() {
  local x
  x="$(fzf)" && "$@" "$x"
}

Though you'll have to adjust the source according to the command as different commands expect different things (cd expects directories not files).
And the last example doesn't even need any scripting:
grep Info "$(fzf)" | wc -l


Save caption state in localStorage

As mentioned in #311, captions aren't stored in between page loads. This is an attempt to refactor localStorage usage to accommodate saving captions on/off state when the user hits the captions button.

Hey, thanks for this. I was just looking through though and thinking it might be best to keep all the settings under a plyr key in the storage and JSON.stringify() and JSON.parse() an object. We'll be adding more to the storage as time goes on and it'd be good to keep them tidy.

That sounds good to me. I was thinking of doing that in this PR anyways but I didn't want to make a big change if that wasn't the direction you were going. I'll refactor this tonight.

I rebased my branch to pull in the latest changes and reworked my implementation to use a JSON store inside localStorage. I chose to make the key captionsEnabled to allow for future language selection or something under a different captions key.

This looks spot on - nice work 👍
I'm just in the middle of upgrading the Vimeo API but will merge this with that and release today or tomorrow.
Thanks!

Awesome! Thanks so much!

Hey, thanks for this - I've manually merged this into some other refactoring work I've been doing. 👍


Questions about ansible nginx and nfs

It is possible to use this box without ansible and nginx?
How can i completely remove nginx and ansible from config?
Also in order to disable NFS i have to do in config.yml following changes:
vagrant_synced_folder_default_type:
type:

Is it right? I just set them empty because i want to use default sync method.

@heihachi88 - I'm not quite sure what you mean by "How can i completely remove nginx and ansible from config?" — by default, neither will be installed if you have Ansible installed on your host computer and you leave the default webserver setting intact (only Apache is installed by default).
For disabling NFS, you should just set those to '' (empty string), and that will use the default synced folder type.

Closing out the question as answered (also, Vagrant now manages Ansible inside the guest VM on its own if you don't have it installed on your computer).


Added k8s tested and incompatible versions

Fixes https://github.com/kubeflow/website/issues/1272

This change is

Preview: https://deploy-preview-1282--competent-brattain-de2d6d.netlify.com//docs/started/k8s/overview/#minimum-system-requirements
/assign @jlewi

/lgtm
/approve
Thanks @sarahmaddox


sketch.min.js isn't up to date

the warning in the Chrome console when I used sketch.min.js (#76)

I'm fairly certain the updating of sketch.min.js from commit 392c76cf332565529e2515bfaea8562976378c0b has resolved this issue.


use the ssh-config-file with the actual ssh cmd

Q
A

Issue Type
Question

Deployer Version
master

Local Machine OS
---

Remote Machine OS
---

Description
One can provide the server config with a configFile like this:
$production->configFile('path/to/file');
In the first place I assumed this file would then be used with the actual ssh command.
But as it turns out is just used to gather some information out of it.
So is there a reason why this is not done?
See here:
https://github.com/deployphp/deployer/blob/master/src/Server/Remote/NativeSsh.php#L51
In my local dev environment I added -i path/to/file in the above mentioned class and it worked out well.

Config file is used only for getting config about servers.

Thanks @elfet for your fast response.
OK, that's what I figured. But is there something preventing us from adding the possibility to use the ssh-config file in the command?
If not I would be happy to create a pull request to add that feature.

You want to not to use config file?

There actually a research #980 from me, i'm going to improve whole ssh system at deployer.

I want to use the native ssh implementaion and use the config file as a parameter.
Right now the config file is only used to fetch information.
I want to use it for the actual connection.
Thus adding the possibility to use all the features we have available in the native ssh command like ProxyCommand etc.
Like this:
    /**
     * {@inheritdoc}
     */
    public function run($command)
    {
        $serverConfig = $this->getConfiguration();
        $sshOptions = [
            '-A',
            '-q',
            '-i __PATH_TO_SSH_CONFIG_FILE__',
            '-o UserKnownHostsFile=/dev/null',
            '-o StrictHostKeyChecking=no'
        ];

And I like the approach of using native ssh :)

@mwr i will implement this. All $sshOptions should be configurable.

@elfet looking forward to that. Will be a great addition.

#980

@elfet please close the ticket as it's done.


fix default portrait animations

Fade Up In/Fade In Out/Fade Down Out is outdated. Change them to the new ones.

this fix is very sigma

I was wondering why did I get those errors all the time...

Hey @endaye thanks for the PR! I've looked into it and this is not quite the correct fix.
It would be correct to change the defaults to "Fade In Up", "Fade Out Down" and "Fade Cross".
However the main issue is that "Fade Up In" was actually also intended to work (mainly to allow for some flexibility when specifying animations). This seems to have been a mistake on my end and is relatively easily fixable.


Speed up "labels_to_flows"?

First of all, thank you for the great software, applying your pre-trained neural network to my images works pretty well already. However, I wanted to test if it gets even better when I re-train myself - unfortunately I can't wait 2,000 hours for pre-processing, see below. I have three Tesla V100 graphics cards with 32GB each, 40 CPUs and 377GB RAM available. The image size is ~5,000 x 5,000 pixels, but I also have a dataset with ~15,000 x 15,000. Am I doing something wrong, or is there another way to speed this up?
(cellpose) saskra@a:~/PycharmProjects/cellpose$ python -m cellpose --train --dir /home/saskra/PycharmProjects/b/ --pretrained_model cyto --train_size --use_gpu --diameter 300. --img_filter "_img"
** TORCH CUDA version installed and working. **
>>>> using GPU
>>>> pretrained model /home/saskra/.cellpose/models/cytotorch_0 is being used
>>>> during training rescaling images to fixed diameter of 30.0 pixels
Running test snippet to check if MKL-DNN working
see https://pytorch.org/docs/stable/backends.html?highlight=mkl
** MKL version working - CPU version is sped up. **
NOTE: test data not provided OR labels incorrect OR not same number of channels as train data
NOTE: computing flows for labels (could be done before to save time)
  1%|█| 5/714 [15:37:52<1991:42:15, 10113.03s/it]

The image size is too large. How many pixels across for a single cell? Downsize to ~30 pixel ROIs please, at least for training.

Thanks, that helped.


3 charts

Description
implement CBarChart
implement CPieChart
implement CCoinLabel
implement CStatusCard
Checklist

[x] Tested changes locally.
[x] Ensured code follows project standards.
[ ] Updated documentation.
[ ] Added necessary tests.

Additional Notes
Charts are not displayed properly on large screens.
for #3

Chart in dashboard takes too much space that the Aside menu wouldn't open properly.

Rename custom.. components to Custom

Resolve conflicts with master


Fixed compatibility issue with matplotlib 3.1.x.

Fixes #310.

Should add that this solution is compatible with matplotlib versions from 2.0.0.

If there are no further comments, I will merge during the weekend.


Move to Typescript

This moves the codebase to typescript.
Would want to add a note in the readme to run before pushing new changes.
npm run build

@tianhaoz95 Can you verify this by the changes by uploading the package?

I have added pull request to the triggering whitelist of GitHub actions to run end to end test for this change and all the future ones to come. If the end to end test is happy I'd say it's good to go.

Also, it's interesting that the integration tests are not triggered by pushing code to your fork, but you can check it in the pull request status.

@tianhaoz95 Made the changes you asked the build still seems to be failing.

Looks like a issue with the token.
Can you verify if its not a issue on your end?

Interesting. That seems to be a problem on my end where the forked repositories don't have access to the token used to run the integration test. For now, I think your change looks good. I just opened #18 to keep track of this issue.

@all-contributors please add @preetjdp for code


deleteGroups admin API implementation

Added deleteGroups API implementation

@Nevon what changes are need? Can't seem to find out change request here...

There were no more changes required. I just hadn't re-reviewed the PR since last time. Looks good to me now!


chore: 🤖 upgrade markuplint

to fix the workaround I introduced here, because it is fixed with this. 🎉
This PR is to make sure it is fixed without degradation, I'll leave it up to you if you merge even alpha version. 👍

Leave this PR unmerged.
I will merge this PR when that modification is officially released. 👍


Remove the should_fail/should_pass separation in E2E tests

Our E2E test suite separates tests into two directories: should_fail and should_pass. IIRC we were using these directories to determine what to expect, but we now do that with test.tomls and it doesn't really matter which directory a test sits in.
My suggestion is that we should not organize tests in this way. If you are working on a feature, all tests, failing or not, should be in the same directory.
So now it looks like:
test/src/e2e_vm_tests/test_programs/should_pass/passing_test_for_feature_a
test/src/e2e_vm_tests/test_programs/should_fail/failing_test_for_feature_a

And my suggestion is:
test/src/e2e_vm_tests/test_programs/feature_a/passing_test
test/src/e2e_vm_tests/test_programs/feature_a/failing_test

I considered this when introducing the test.toml testing, but it would've been too big a diff to do it then.  But I agree, we really don't need to organise based on the test type, but rather what it is actually testing.


Low memory not called

It doesn't trigger "Low Memory" on Nexus 7 2013 with Android 5.0.1 or Nexus 5 Android 5.1.0.
Shouldn't it trigger it? If not maybe it would be good to mark it in README.

Hey,
Yeah actually the indicator is for the while system. On some devices this will never be triggered, but it should still produce the desired effects on your app once the free RAM counter goes down far enough.
You're right though, this should he documented.
Cheers.


General question

is it possible to start the outdoor unit without an indoor unit? can you help?

this com protocol is between the ESP and the indoor unit. the indoor unit later on requests the outdoor unit to start.
If you would like to do that you have to crack the S1 S2 com protocol which is the protocol between the indoor and outdoor units.
Unfortunately I can not do that on my side.


[VL] Adapt setup scripts for the changes in upstream velox

What changes were proposed in this pull request?
Adapt https://github.com/facebookincubator/velox/pull/9198
How was this patch tested?
Exists CI.

cc @PHILO-HE, thanks.

Incorporated in https://github.com/apache/incubator-gluten/commit/ca49ab9ce13e2c96250446ef6f3deb0539eef801. Thanks @liujiayi771!


Documentation can be improved for newcomers

I have begun to play with Hive and it is awesome, but sometimes frustrating because documentation is lacking some information. This might be specially difficult for newcomers (including me).
For example:

How do I even init Hive? I would modify this page to inform that you need to import path_provider on Android and iOS, but not web. I would also add a link to the corresponding sample. It is hard to guess without looking at the sample.

So now I know how to open a box. Where should I open it? Should I use FutureBuilder on the Scaffold or may I use FutureBuilder in an individual component? Does it matter?

 You are calling it before Hive.init(dir.path), I didn't know it was possible and there is nothing in the documentation.

Then we learn how to read to a box. Awesome. But how do I listen for the changes? I want to observe them, is this possible? Is it compatible with stream? Can I put together with RxDart and debounce them? Will it work only with FutureBuilder or anything else? I later saw the watch changes page, but you could link them.

Regarding delete, I would probably add a comma in the you can either override it using for example put() or delete it. It felt weird/confusing reading the first time. You could also add another line, like box.delete('key'); // value was already deleted, returns nothing immediately. Those snippets could be more complete, exactly like you did here.

I think that's it.

Thank you very much for taking the time to give me feedback.
I agree with every point and start improving the docs. I think it is really lacking "best practices"...

Another thing that made me go into "???" is the samples. I automatically went here which directed me here which directed me here. Couldn't you keep the samples in the same repository? Or at least provide a nice README that avoids two clicks?

Another question, when does snapshot.error != null happen? Sometimes? Never (as long as I configured Hive correctly)? I couldn't find anything. Worth adding to the documentation.

snapshot.error != null happens for example if you try to write to a directory where you don't have write permission. It is just good practice to handle a possible error in the FutureBuilder .
hive_flutter does not have a documentation yet.
you should add a description for typeId in the sample.
What do you mean by that?

you should add a description for typeId in the sample.

You use value 35 randomly. On documentation, I found it could be any value between 0 and 255. You could add a comment saying "this value is random and could be any value between 0 and 255" so we don't have to look in the documentation.

Ah, yes :+1:

If I correctly setup Hive, snapshot.error != null will never happen, right? I am wrapping my full app with hive, so I want to be sure it will never fail catastrophically randomly.

If I correctly setup Hive, snapshot.error != null will never happen, right? I am wrapping my full app with hive, so I want to be sure it will never fail catastrophically randomly.

It won't happen because of Hive. But if you init Hive with a folder where you don't have write access it will fail.

@bernaferrari I'm currently working on a new version on the docs and I'd love to get your input or even a pull request. See #165.


fix

can you plx fix the parsing error on android tv on your next update when installing this app thank you

can you plx fix the parsing error on android tv on your next update when installing this app thank you

Send me a screenshot.

can you plx fix the parsing error on android tv on your next update when installing this app thank you

Send me a screenshot.

In preparation for my exam I am busy with my studies. A new update will be available on June 16-30. And I will try to fix this problem.


Add other necessary changes for dark mode

just add some feature that theme depend's on client system theme

Add other necessary changes for dark mode.

1

Thanks for the the pull request.
Unfortunately the dark mode with those changes look totally unreadable:

When you want to test it, click on "I'm Feeling Lucky" and see if it still looks ok!

Thanks, that looks already better!

Three problems remain:

the favicon icon is not readable
the github image at the bottom right of the page is not readable (this is an svg, so it should be possible to adapt it easily without having a second asset
the color of the links is a shade of blue that is to dark, there is not enough contrast to read it clearly

Thanks!
I also added the dark mode for the favicon, now I think it looks ok!

Thank you. This was my first pr and it was exciting to be able to contribute to a repository. Of course, the UI of the site can be improved


New chapter failure

When I go to make a new chapter, or go to write for a story, the AHHHGGGHH!!! 500 YEARS DUNGEON! You broke the site! error pops up.

fixed


Revert "Revert "[HEVCe] Fix temporal layers reset issue""

Previously, #2601 got reverted due to incorrect option inherting condition for Scale

The condition to inherit option for Layer[i].Scale from init par to reset par:  Layer structure is empty (all zero)


what's the exact url?

I try both of these
https://www.tiktok.com/@sooshimango
https://www.tiktok.com/@sooshimango/video/7228919985777757442?lang=en
import asyncio
from aiotiktok import Client

tiktok = Client()

async def main():
    data = await tiktok.get_data(url="https://www.tiktok.com/@sooshimango/video/7228919985777757442?lang=en")
    print(data)

asyncio.run(main())

Traceback (most recent call last):
File "test.py", line 10, in
asyncio.run(main())
File "/Users/johndpope/opt/miniconda3/envs/spleeter/lib/python3.7/asyncio/runners.py", line 43, in run
return loop.run_until_complete(main)
File "/Users/johndpope/opt/miniconda3/envs/spleeter/lib/python3.7/asyncio/base_events.py", line 587, in run_until_complete
return future.result()
File "test.py", line 7, in main
data = await tiktok.get_data(url="https://www.tiktok.com/@sooshimango/video/7228919985777757442?lang=en")
File "/Users/johndpope/Documents/gitWorkspace/aiotiktok/aiotiktok/client.py", line 82, in get_data
video_id = await self.get_video_id(url)
File "/Users/johndpope/Documents/gitWorkspace/aiotiktok/aiotiktok/client.py", line 34, in get_video_id
if original_url == self.tiktok_url or "video" not in original_url:
UnboundLocalError: local variable 'original_url' referenced before assignment

sorry, that's my problem, already i fix it.
u can install new version 2.5.3 from PyPi

thanks so much - this is working great.


Pressing 'a' in normal mode inserts (multiple-cursors-apply)

After highlighting multiple words, and entering normal mode( by pressing 'v' ) pressing  'a' inserts (multiple-cursors-apply) twice and the plugin doesn't work.
My settings in .vimrc
let g:multi_cursor_next_key='<C-a>'
let g:multi_cursor_prev_key='<C-s>'
let g:multi_cursor_skip_key='<C-d>'
let g:multi_cursor_quit_key='<Esc>'

I use neocomplete & have added this
" Called once right before you start selecting multiple cursors
function! Multiple_cursors_before()
  if exists(':NeoCompleteLock')==2
    exe 'NeoCompleteLock'
  endif
endfunction

" Called once only when the multiple selection is canceled (default <Esc>)
function! Multiple_cursors_after()
  if exists(':NeoCompleteUnlock')==2
    exe 'NeoCompleteUnlock'
  endif
endfunction

in my .vimrc
The error :

Make sure you're using latest version of vim (or at least 7.4)
Check if you have other mappings for a in Normal mode
9 сент. 2015 г. 1:58 AM пользователь "Arindam Banerjee" <
<EMAIL_ADDRESS>написал:

After highlighting multiple words, and entering normal mode( by pressing
'v' ) pressing 'a' inserts (multiple-cursors-apply) twice and the plugin
doesn't work.
My settings in .vimrc
let g:multi_cursor_next_key=''
let g:multi_cursor_prev_key=''
let g:multi_cursor_skip_key=''
let g:multi_cursor_quit_key=''
I use neocomplete & have added this
" Called once right before you start selecting multiple cursors
function! Multiple_cursors_before()
if exists(':NeoCompleteLock')==2
exe 'NeoCompleteLock'
endif
endfunction
" Called once only when the multiple selection is canceled (default )
function! Multiple_cursors_after()
if exists(':NeoCompleteUnlock')==2
exe 'NeoCompleteUnlock'
endif
endfunction
in my .vimrc
The error :
[image: screenshot from 2015-09-09 04-21-31]
https://cloud.githubusercontent.com/assets/7141867/9749471/c2e8ab3e-56aa-11e5-9926-e058d21072e0.png
—
Reply to this email directly or view it on GitHub
https://github.com/terryma/vim-multiple-cursors/issues/160.

Vim version - 7.4 patches 1-854
normal-mode a is not mapped to anything.
In fact the same problem happens when pressing i or I or A - anything that takes to insert mode from normal mode.

i and a should work. I and A are known to be not working, this is expected.
This must be something about your configuration and a mixture of plugins. Try turning off some plugins and check again. Keep disabling plugins until it starts working.

 Seems like it is an issue with some setting in my .vimrc.
After some commenting & restarting vim the error lies in these lines
 if has("autocmd")
     au InsertEnter *
                  \ if v:insertmode == 'i' |
                  \   silent execute "!gnome-terminal-cursor-shape.sh ibeam" |
                  \ elseif v:insertmode == 'r' |
                  \   silent execute "!gnome-terminal-cursor-shape.sh underline" |
                  \ endif
      au InsertLeave * silent execute "!gnome-terminal-cursor-shape.sh block"
      au VimEnter * silent execute "!gnome-terminal-cursor-shape.sh block"
      au VimLeave * silent execute "!gnome-terminal-cursor-shape.sh underline"
  endif

Contents of gnome-terminal-cursor-shape.sh
#!/bin/sh
DEFAULTPROF=`dconf read /org/gnome/terminal/legacy/profiles:/default`
DEFAULTPROF=`echo "$DEFAULTPROF" | sed -e "s/^'/:/" -e "s/'$//"`
dconf write /org/gnome/terminal/legacy/profiles:/$DEFAULTPROF/cursor-shape "'$1'"

This file just changes cursor in gnome-terminal according to modes. Taken from vim.wikia.com(http://vim.wikia.com/wiki/Change_cursor_shape_in_different_modes).
This is quite important for my workflow -
Any ideas how to rectify this ?

Interestingly, the gnome-terminal is VTE compatible -
thus the following code
let &t_SI = "\<Esc>[6 q"
let &t_SR = "\<Esc>[4 q"
let &t_EI = "\<Esc>[2 q"

plays nicely with the multiple-cursors.
I have replaced this with the previous code & everything works.
Still, any idea what was happening in the previous code ?

same here. GNOME Terminal 3.18.3

@aj07mm Are you also using the dconf settings to change the cursor ? Try the VTE settings to change the cursor, it works with gnome-terminal.


Provisioning State and Power State of VM

Hi ,
Can we get list of possible provisioning states and power states while creating VM?
Thanks,
Vikram

Hi @VikramAIS
Reserved provisioning states are "Succeeded", "Failed" and "Canceled". Then this is a random string that can change over time and decided by the service team ("MountingDrive", "SelectingHosts", etc.). If it's random string, it's considered "provisioning in progress".
I don't know about "power states", I will ask the question.

Would be nice to have estimated information about % provisioning (deployment) as a number from 0 to 100. Where 100 = Succeeded.

@derekbekoe @yugangw-msft do you know if it's available (% provisionning)?

This page has the power state list

do you know if it's available (% provisionning)?

I don't think so.


Added a sample for face detection on video from webcam

I think it is useful sample, even though OpenCV's C API is being deprecated.

Thanks :+1:


ISSUE-1109 Provide execute latency into in processor and sink for time-series manner

add 'executeTime' into "misc" for processors and sinks

it will not be available in sources

This closes #1109

The change is tiny. Most of changes are in test.

@arunmahadevan

We don't show that thing in overviewMetrics and "process time" is represented as "latency" so if we want to include it as well, we should rename latency to back to "process time" as well so that we can have both and no one represent the latency. UI change is needed as well.

execute latency is out of whilelist in metrics consumer. I was requested to reduce metrics as much as possible so I excluded all of unused things for that time. I'm adding it in test cluster, but it means we are required to fix it from Ambari side as well if we want to get it by default.
Could you file an issue and assign as well?

We don't show that thing in overviewMetrics and "process time" is represented as "latency" so if we want to include it as well, we should rename latency to back to "process time" as well so that we can have both and no one represent the latency. UI change is needed as well.

In storm its named "Process latency" and "Execute latency". I think we can follow this? Yes we need to rename in UI as well.

I found execute latency is out of whilelist in metrics consumer. I was requested to reduce metrics as much as possible so I excluded all of unused things for that time. I'm adding it in test cluster, but it means we are required to fix it from Ambari side as well if we want to get it by default.
Could you file an issue and assign as well?

https://github.com/hortonworks/streamline/issues/1112

@arunmahadevan
We need to restart topologies in test cluster to take effect. Oddly I can't open the Streamline (it shows blank page) in test server. Really appreciate if you could handle it.
And I think there's nothing to do in SAM side for handling whitelist. Please file an issue to Ambari side.

@HeartSaVioR , raised Ambari issue - https://issues.apache.org/jira/browse/AMBARI-22696
You meant to restart topology to see executeTime or for something else ? I have restarted the topology.

@arunmahadevan Execute time should be shown in test cluster after restarting topology, cause I modified configuration in test cluster manually.

@arunmahadevan @shahsank3t
Updated PR to add latency in overall metrics.

topology: latency / prevLatency
source: completeLatency / prevCompleteLatency
processor/sink: executeTime / prevExecuteTime, processTime / prevProcessTime

Note that I just used *Time instead of *Latency since @shahsank3t would like to handle the rename from UI side.

Thanks @HeartSaVioR , merged to master. Need to check the status of the Ambari patch to whitelist metrics.


Adding some new magic to effects array

Great repo! Following with more later.
Added effects:

Banishment
Greater Invisibility
Hex
Modify Memory
Summon Aberration
Telekinesis

Removed concentration from condition array.

@amandafilizola this is now live in v1.1.0.

Excited to use it and also the new concentration feature! Thanks!


Converting hyper response directly into iron response

Hey guys,
This isn't really a bug, more of some advice on how to optimise a proxy handler.  What I mean by a proxy is that a client will request a URL handled by Iron, and then an Iron handler will proxy this request onto a backend service.
At the moment I'm using hyper to act as a HTTP client, converting the URL to the proper form and passing the response from hyper back to the client.  This code works, but it adds about half a second delay.  I have a suspicion this is because I am reading to a string, which is too slow.
Is there a way to pass a hyper response directly into the iron response so I can stream it out rather than allocate a string?
I've tried:

Implementing WriteBody for hyper::response but I get the the impl does not reference any types defined in this crate
Creating a Box with the hyper response casted response.body = Some(Box::new(Box::new(*(&res)))); but I get the std::io::Read does not have a constant size known at compile-time

Here's my current (trivial) implementation:
fn main() {
    let mut router = Router::new();
    router.any("/*", handler);
    Iron::new(router).http("<IP_ADDRESS>:3000").unwrap();
}

fn handler(req: &mut Request) -> IronResult<Response> {

    let proxy_url = build_proxy_url("http://example.com/", req.url.clone());

    println!("Received request for URL: {}", proxy_url);

    let client = hyper::Client::new();

    let mut res = client.request(req.method.clone(), &*proxy_url)
                        .headers(req.headers.clone())
                        .body(&mut req.body)
                        .send()
                        .unwrap();

    let mut response = Response::new();

    let mut response_body = String::new();

    res.read_to_string(&mut response_body).unwrap();

    response.status = Some(res.status.clone());
    response.headers = res.headers.clone();
    response.body = Some(Box::new(response_body));

    Ok(response)
}

fn build_proxy_url(base_url: &str, req_url: Url) -> String {

    let mut response = String::new();

    response.push_str(base_url);
    response.push_str(&*req_url.path.join("/"));

    match req_url.query {
        Some(ref query) => {
            response.push_str("?");
            response.push_str(query);
        },
        None => ()
    };

    return response;
}

Just an update: some of the slowness happens due to DNS lookups in hyper, converting the request to an IP address resolves this.  The Client::new() call is probably the culprit, but not sure how to persist a client instance across multiple requests.


[objc] Add a new SourceWriter to replace the TextWriter

Allow enabling/disabling the writer (needed for other features like
protocols);

Deal with tabs in a single place - so output looks better;

Build and tests passed.


"Poaiana Galdei" is listed in the drop-down menu

Description:
"Poaiana Galdei" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Poaiana Galdei" in the search bar.

Expected results
The location is listed in the drop-down menu as "Poaiana Galdei, Alba (Galda de Jos)".


Code Quality: magic args removed descriptions of commands

We will need to write descriptions to make them findable by find, to help people out without having to refer to the readme. Simply the args will not do.

[ ] list_items
[ ] list_equips
[ ] list_AI
[ ] list_Body
[ ] give_item
[ ] give_equip
[ ] give_lunar
[ ] create_pickup
[ ] remove_item
[ ] remove_equip
[ ] give_money
[ ] give_exp
[ ] family_event
[ ] next_boss
[ ] next_stage
[x] seed
[ ] fixed_time
[ ] time_scale
[ ] stage_clear_count
[ ] list_player
[ ] god
[ ] kill_all
[ ] true_kill
[ ] no_enemies
[ ] spawn_as
[ ] spawn_ai
[ ] spawn_body
[ ] respawn
[ ] change_team
[ ] add_portal

Incorporated 1e248fc3c1bcf6ee8d78306635a0136073c757dc


Docker and Docker Compose Files

Closes #28

Needs a way to get FLASK_SECRET_KEY from the environment before this can be merged

Just tested and it finally imports the flask secret key from the .env file. Should be good to review @bot190


add Afifah school

@Rat-mouse Please provide us with:

the school official website URL, if different from the domain you are submitting,
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

If the school does not officially disclose the email domain, then should I close this pull request?


Registry entry mismatch with server (same versions)

Hi, when I try to join my server with 1.0.4, I get a registry entry mismatch screen. I've confirmed that the library, attribute mod and the main mod are the same version on client and server.

https://mclo.gs/Suub7a6

The server has the spawner module enabled and you have it disabled on your client.


I'm missing something super obvious trying to use anchore/syft:latest to scan an image I've just built.

I'm in "learning mode" and very new at this.
The following command works just fine for what I need:
 syft --scope all-layers "local/hello_world:container" | sed 's/ *$//' > hello-world-image/container/sbom.txt 2>> 'test-reports/run_sbom.txt'

The sed is in there to trim the trailing whitespace before pre-commit rejects the commit.
This outputs
 ✔ Loaded image                                                    local/hello_world:container
 ✔ Parsed image        sha256:096ae06aa0c42259a53beabc1b781dc161236d6dc007fd3cfc7682e86c0e0505
 ✔ Cataloged packages  [100 packages]

I end up with a file containing all the dependencies in a container I've just built. That file will be put into source control. This is all batched up in a simple script and is providing what we need.
BUT ... it requires the dev to install yet another tool (and that can be a lot of tools by the time you've got all the things working ... and then, keeping everyone up to date ... more work). So, transition to using the official container (as documented).
So as the image is built locally, I started with what seems an obvious replacement
docker run anchore/syft:latest --scope all-layers "local/hello_world:container" | sed 's/ *$//' > hello-world-image/container/sbom.txt 2>> 'test-reports/run_sbom2.txt'

But that outputs
1 error occurred:
	* failed to construct source from user input "local/hello_world:container": unable to load image: unable to use OciRegistry source: failed to get image descriptor from registry: GET https://index.docker.io/v2/local/hello_world/manifests/container: UNAUTHORIZED: authentication required; [map[Action:pull Class: Name:local/hello_world Type:repository]]

I'm sort of lost as to how to fix this. I think this is a "correct" error. Docker in docker related? So somehow I need to run the docker run command with the local image being accessible in some way.
This is still me learning. Part of this is to NOT just throw the image into AWS ECR and be done with it. By having the SBOM created as part of the dev workflow/pipeline, we can verify/accept anything changing within the newly built container. And we'll be pulling all the SBOMs together and so across all our images, if there's a CVE we need to deal with, which ones are affected.
Some of this MAY already. But the learning first and using second is where we are.
I did SOME research and added -v /var/run/docker.sock:/var/run/docker.sock.
docker run -v /var/run/docker.sock:/var/run/docker.sock anchore/syft:latest --scope all-layers "docker.io/local/hello_world:container"

This worked! But is it right/safe/anyone still here after NOT running away scared by the newbie mistake?

Hey @rquadling, it sounds like you figured out what to do, and for the right reasons: when running something within a docker image, you need to provide a way for it to access the host's docker instance, which is what you are doing by exposing the docker.sock. The error you mentioned contains this bit, which is of importance:
failed to get image descriptor from registry: GET https://index.docker.io/v2/local/hello_world/manifests/container: UNAUTHORIZED: authentication required

... you see is because when running as a container with no reference to the docker.sock or other method of indicating how the docker daemon can connect, syft is unable to use the docker daemon and attempts a direct registry connection (note the URL). But of course, that isn't something you have access to in the docker main registry and that isn't what you want anyway because what you want to scan is a locally built image only available on the host.
That's all to say: you did the right thing. 🙌

I have tried moving the locally working setup to BitBucket (so we can ensure the SBOM is valid)... and I've got the same issue.
But this time I think this is a protection by Atlassian to stop inappropriate access to the Docker Daemon. A very detailed blog about this : https://staaldraad.github.io/post/2019-07-11-bypass-docker-plugin-with-containerd
With that, it looks like using BitBucket Pipelines, to run the official Syft container, and getting it to scan an image just made in the pipeline is not possible. Installing Syft in the pipeline is the obvious solution and was what I started with locally.
Another option is to build Syft into a "toolbox" for the pipeline, but that's not something we should be doing (containers being a single thread/service/feature is what I've been doing).
If there's nothing else to try (and I've looked and the blog is pretty clear that the mounting of the docker daemon is a no-no), then I'm done for the issue.
Thanks for the help.

@rquadling it's possible you could configure Docker to use remote access, instead of the docker.sock: https://docs.docker.com/config/daemon/remote-access/, beyond that: you could run a local registry (maybe just with the registry:2 image) and configure syft to directly connect to that registry. But unfortunately, I don't have a lot of specifics to BitBucket or your specific setup to be able to suggest any other solutions.

@rquadling Another option might be saving off the container as a tar, and making that available to the syft image:
docker save alpine:latest > foo.tar
docker run --rm -v "$PWD:/data" anchore/syft:latest docker-archive:/data/foo.tar

This works for me, and no docker socket manipulation needed, just mounting the path to the tar as a volume into the syft container that gets run.
I think this has tradeoffs with regards to surfacing layer information, and using disk space on the worker, but if you just want syft to scan all the files that are present in the final built image, I think it will work.

👋 @rquadling I think I found the issue that was causing you some pain regarding how the sockets are setup
I filed a PR with stereoscope so the tools should now just work out of the box and find and connect to the correct socket when the system is darwin and the old default socket path is not available:
https://github.com/anchore/stereoscope/pull/217


#1167 Fixed ThaliMobile network type to use global network type if presented

This change is

PR is added to the queue for testing as 3. task. (90583c6)

Hi @artemjackson, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

PR is added to the queue for testing as 4. task. (9c44d97)

PR is added to the queue for testing as 5. task. (be52ee4)

Test (Success) 86877966 build is completed (90583c6)
See https://github.com/ThaliTester/TestResults/tree/8687796690583c6_Fixed_ThaliMobile_network_type_to_use_global_network_type_if_presented_artemjackson/ for the logs

Test (Fail) 86877966 build is completed (9c44d97)
See https://github.com/ThaliTester/TestResults/tree/868779669c44d97_Fixed_ThaliMobile_network_type_to_use_global_network_type_if_presented_artemjackson/ for the logs

Test (Fail) 86877966 build is completed (be52ee4)
See https://github.com/ThaliTester/TestResults/tree/86877966be52ee4_Fixed_ThaliMobile_network_type_to_use_global_network_type_if_presented_artemjackson/ for the logs

PR is added to the queue for testing as 6. task. (6098730)

Reviewed 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

thali/NextGeneration/thaliMobile.js, line 76 at r2 (raw file):

      return function() {
        var args = arguments;
        return wifiMethod.apply(null, args)

We can apply just arguments without args variable.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

thali/NextGeneration/thaliMobile.js, line 76 at r2 (raw file):
Previously, andrew-aladev (Andrew Aladjev) wrote…
> We can apply just arguments without args variable.

yes, it's just for clarity

Comments from Reviewable

PR is added to the queue for testing as 7. task. (99efd68)

Merged as a hotfix

Test (Fail) 86877966 build is completed (6098730)
See https://github.com/ThaliTester/TestResults/tree/868779666098730_Fixed_ThaliMobile_network_type_to_use_global_network_type_if_presented_artemjackson/ for the logs

Test<PHONE_NUMBER>583c6(90583c6) has failed
See https://github.com/ThaliTester/TestResults/tree/8687796690583c6_Fixed_ThaliMobile_network_type_to_use_global_network_type_if_presented_artemjackson/ for the fail logs

Test (Fail) 86877966 build is completed (99efd68)
See https://github.com/ThaliTester/TestResults/tree/8687796699efd68__1167_Fixed_ThaliMobile_network_type_to_use_global_network_type_if_presented_artemjackson/ for the logs


stages/ostree.preptree: add recommends argument

recommends allow creating a commit with rpm's Recommends or not. It's true by default as per https://github.com/coreos/rpm-ostree/pull/1513/files
Fix https://github.com/osbuild/osbuild/issues/1267 - another PR in osbuild-composer will be needed

or the option doesn't apply since we download our rpms with the rpm stage shrug

Exactly, see also
https://github.com/osbuild/osbuild/issues/1134

this won't work - fix would be to enhance deps resolution higher level


Error running docker on OSX with ZSH and Homebrew Docker

npm ERR! path /src/packages/perspective/package.json
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall open
npm ERR! enoent ENOENT: no such file or directory, open '/src/packages/perspective/package.json'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

npm ERR!  Exit status 254

versions:
> docker --version
Docker version 18.05.0-ce, build f150324

> zsh --version
zsh 5.3 (x86_64-apple-darwin17.0)

I believe this is due to the note mentioned here.  Installing Docker via the installer on the CE website should configure this correctly, though I am not a docker expert.
As there is nothing really to fix on the Perspective side, closing for now - feel free to re-open with further details if this has been closed in error.


Need to check this works on an up-to-date Visual Studio installation …

…before merging

I wanted to check it on Visual Studio Mac, and was blocked by #4


Error while publishing a NPM package to Github

I'm using your action to publish a NPM package with Yarn, but I get this error. I don't know if I'm missing something 😅.
So I hope you can help me. This is the error:
Run simenandre/publish-with-yarn@v1
  with:
    npm-auth-token: ***
  env:
    NPM_CONFIG_USERCONFIG: /home/runner/work/_temp/.npmrc
    NODE_AUTH_TOKEN: XXXXX-XXXXX-XXXXX-XXXXX
Run npm version --allow-same-version --no-commit-hooks --no-git-tag-version from-git
  npm version --allow-same-version --no-commit-hooks --no-git-tag-version from-git
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    NPM_CONFIG_USERCONFIG: /home/runner/work/_temp/.npmrc
    NODE_AUTH_TOKEN: XXXXX-XXXXX-XXXXX-XXXXX

npm ERR! code 1[2](https://github.com/devbyray/future-ui/actions/runs/6961876462/job/18944403710#step:6:2)8
npm ERR! An unknown git error occurred
npm ERR! command /usr/bin/git --no-replace-objects describe --tags --abbrev=0 --match=*.*.*
npm ERR! fatal: No names found, cannot describe anything.

npm ERR! A complete log of this run can be found in: /home/runner/.npm/_logs/202[3](https://github.com/devbyray/future-ui/actions/runs/6961876462/job/18944403710#step:6:3)-11-22T19_28_16_69[4](https://github.com/devbyray/future-ui/actions/runs/6961876462/job/18944403710#step:6:4)Z-debug-0.log
Error: Process completed with exit code 12[8](https://github.com/devbyray/future-ui/actions/runs/6961876462/job/18944403710#step:6:9).

This is how my yarn-publish-github-package.yml file looks like:
name: Node.js Package on Github

on: workflow_dispatch

jobs:
  build:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3

      - name: Use Node LTS ✨
        uses: actions/setup-node@v3
        with:
          node-version: lts/*
          registry-url: https://npm.pkg.github.com
          cache: yarn

      - name: Install dependencies 📦
        run: yarn install

      - name: Build 🔨
        run: yarn build

      - uses: simenandre/publish-with-yarn@v1
        with:
          npm-auth-token: ${{ secrets.GITHUB_TOKEN }}

Great question!
This seems to be an authentication issue, because I don't think the authentication you do with registry-url will publish-with-yarn.


Various fixes to C# warnings

Fix reference to NotificationSceneInstantiated in the documentation.

Follow up to https://github.com/godotengine/godot/pull/64410.

Suppress REFL045 in a false positive scenario.
Suppress CA1710 caused by Array not having the Collection suffix.
Suppress CS0108 caused by shadowing members in derived classes.

Fixes https://github.com/godotengine/godot/issues/15763

Suppress CA1716 caused by parameter names matching C# keywords.
Use SymbolEqualityComparer in ScriptPathAttributeGenerator to fix RS1024.
Suppress dependencies of SourceGenerators package to fix NU5128.
Suppress nullability warnings in source generators because we already have a Debug.Assert (but it's missing the nullability annotations in netstandard2.0).
~Implement disposable pattern in GodotSynchronizationContext (and make it sealed)~.

Extracted to https://github.com/godotengine/godot/pull/72053.

TODO

[ ] Fix CA1001: Type owns disposable field but is not disposable.

[ ] Array and Dictionary: This was already discussed a bit in https://github.com/godotengine/godot/pull/64089#discussion_r940589373 but I think it's not as straightforward as the rest of the fixes and it should probably be handled in a separate PR.
[ ] Variant: It already implements IDisposable so I don't know why it complains.

[ ] Fix RS2008: Enable analyzer release tracking.

Not sure how to fix this one. The quick fix in VSCode does nothing.

Fix RS2008: Enable analyzer release tracking.
* Not sure how to fix this one. The quick fix in VSCode does nothing.

I think this is about https://github.com/dotnet/roslyn-analyzers/blob/main/src/Microsoft.CodeAnalysis.Analyzers/ReleaseTrackingAnalyzers.Help.md

Fix CA1001: Type owns disposable field but is not disposable.
* [ ]  `Array` and `Dictionary`: This was already discussed a bit in

This won't be a simple change. It will require some design decisions.

Fix RS2008: Enable analyzer release tracking.
* Not sure how to fix this one. The quick fix in VSCode does nothing.

I think this is about https://github.com/dotnet/roslyn-analyzers/blob/main/src/Microsoft.CodeAnalysis.Analyzers/ReleaseTrackingAnalyzers.Help.md

Yes but since the quick fix doesn't seem to do anything in VSCode, should I create the markdown files manually? Suppress the warning?

Fix CA1001: Type owns disposable field but is not disposable.
* [ ]  `Array` and `Dictionary`: This was already discussed a bit in

This won't be a simple change. It will require some design decisions.

I don't intend to solve this warning in this PR but I wanted to mention it since it's somewhat related.

Rebased and extracted breaking changes to https://github.com/godotengine/godot/pull/72053.

Can we close this one? The only remaining point is the several instances of CA1001, and this PR does not fix them. We should track that particular issue in its own thread instead.

Yes, this has been superseded by the multiple PRs that are now linked in the PR description.


'dateFnsLocalizer' is not exported from 'react-big-calendar'.

Do you want to request a feature or report a bug?
BUG
What's the current behavior?
Attempted import error: 'dateFnsLocalizer' is not exported from 'react-big-calendar'.
"react-big-calendar": "^0.23.0",
"react": "^16.9.0",
What's the expected behavior?
Export dateFnsLocalizer

Change dateFnsLocalizer to DateLocalizer, its in the breaking changes

@makis-spy It was not published yet.


Add table footer support

Fixes #113

Coverage remained the same at 100.0% when pulling c86918618b01b964142c8630626e7b266c7a1759 on table_footer_support into 4b7bf4cecc64e11e8c2b3b80d4ae1ed331e819b5 on master.


Fix flaky WireframeUtils test

What does this PR do?
This change fixes flaky M return false W checkIsValidWireframe(){ covered by another with solid background } test.
Test was failing because there was a possibility to generate wireframe without background color, so it has isOpaque == false.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1208 (a995ec2) into feature/sdkv2 (932cecb) will increase coverage by 0.00%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           feature/sdkv2    #1208   +/-   ##
==============================================
  Coverage          82.59%   82.59%
==============================================
  Files                351      351
  Lines              12480    12475    -5
  Branches            2058     2056    -2
==============================================
- Hits               10307    10303    -4
- Misses              1533     1535    +2
+ Partials             640      637    -3

Impacted Files
Coverage Δ

...src/main/kotlin/com/datadog/android/DatadogSite.kt
83.33% <0.00%> (-3.33%)
:arrow_down:

...ain/java/com/datadog/opentracing/PendingTrace.java
57.76% <0.00%> (-1.72%)
:arrow_down:

...in/com/datadog/android/log/internal/LogsFeature.kt
84.16% <0.00%> (-0.99%)
:arrow_down:

.../android/rum/internal/monitor/DatadogRumMonitor.kt
90.81% <0.00%> (-0.54%)
:arrow_down:

...rc/main/java/com/datadog/opentracing/DDTracer.java
56.07% <0.00%> (-0.42%)
:arrow_down:

.../android/rum/internal/domain/scope/RumViewScope.kt
95.18% <0.00%> (-0.18%)
:arrow_down:

...lin/com/datadog/android/rum/internal/RumFeature.kt
92.03% <0.00%> (ø)

...android/log/internal/logger/TelemetryLogHandler.kt
85.71% <0.00%> (ø)

...android/v2/core/internal/net/DataOkHttpUploader.kt
94.37% <0.00%> (ø)

...ndroid/telemetry/internal/TelemetryEventHandler.kt
72.80% <0.00%> (ø)

... and 4 more


How to Create, Register and Use middleware with modules?

I have created one search module with my laravel project Now i need to allow only authenticated user to visit URLs on this search module
If i use Main project midlleware with this Search module then it working fine
But how can i define and use midlleware with that Search Module
I haven't found much document on this thing so can any one plz guide me

You can register middleware using $this->middleware() function inside your service provider.

I'll close this as there hasn't been a reply in a while, feel free to comment if needed.

You can register middleware using $this->middleware() function inside your service provider.

Can you make an example? For me isn't working


Any plans to support showing cc version?

As the title suggests. I'm currently developing a project that requires some c source files compilation using cc. And I wish to add the version of cc in the build information.

Hi @sshwy
shadow-rs support  extended hook function.
If you want add cc  result in you build project. I suggest use new_hook function.
The detail example ,please see

build.rs
main.rs

Thank you for your advice! That'll work for my project.

I feel happy for you. If you have any questions, please consult again.


[Java][Client] 2.7.2 regression. MultiTopicsConsumer  org.apache.pulsar.client.api.PulsarClientException$NotSupportedException: not support shrink topic partitions

Describe the bug
After upgrading java client to 2.7.2 version we started to notice following logs:
2021-05-26 18:00:39.236 ERROR 11646 --- [r-client-io-1-1] o.a.p.c.impl.MultiTopicsConsumerImpl     : [persistent://public/batch/embedded_ROUTING1_bdcccf7eeaec7f5e0e7d243451433ac2ae8506d4] not support shrink topic partitions. old: 1, new: 0
2021-05-26 18:00:39.237  WARN 11646 --- [r-client-io-1-1] o.a.p.c.impl.MultiTopicsConsumerImpl     : [MultiTopicsConsumer-f03d6] Failed to subscribe increased topics partitions: org.apache.pulsar.client.api.PulsarClientException$NotSupportedException: not support shrink topic partitions

This happens couple of minutes of not pushing new messages to listened topic.
Another consequence of that are exceptions when trying to ack message that was in progress coming from that consumer:
org.apache.pulsar.client.api.PulsarClientException$InvalidMessageException: null
	at org.apache.pulsar.client.impl.ConsumerBase.acknowledgeAsync(ConsumerBase.java:373)

Expected behavior
There should be no errors.
Additional context
After upgrading server and java client to 2.7.2 version from 2.7.1 version, but it's problem in java client.

I guess that issue can be somehow related to that change: https://github.com/apache/pulsar/pull/9660

It seems that the PR https://github.com/apache/pulsar/pull/10708 fixed the error log problem. I make some tests by the latest master and error logs not support shrink topic partitions. old: 1, new: 0 are not present.
But I didn't reproduce the ack problem. Could you provide the steps that how to reproduce the ack problem?

Hi @gaoran10 I haven't tested current master and it's quite hard to reproduce the issue.

This should have been fixed by #10708

@merlimat this is a regression on branch-2.7.
#10708 is only for master branch (2.8.0...)
we should cherry pick to branch-2.7, correct ?

Hi @merlimat, @eolivelli
Can #10708 be cherry picked to new 2.7 release?

it is already marked to be cherry picked.

Ahh. Ok. Thanks :)

https://github.com/apache/pulsar/pull/10708 has fix it and cherry-pick to 2.7, release 2.7.3, so close it.


Added fireship tailwind tricks link  issue-701

Related Issue #701
Closes #701
Description
I had made a pr before with indentation and conflicts and changes made the code messy ,so I closed the prev. pr and created new one. Hope now things are ok.
Screenshots

Checklist

[ x] My code adheres to the established style guidelines of the project.
[ ] I have included comments in areas that may be difficult to understand.
[x ] My changes have not introduced any new warnings.
[ x] I have conducted a self-review of my code.

@JasonDsouza212 do add the Gssoc tag along with the level in this pr ,it would be kind of you : |)


[Documentation] style prop details disappeared from View page

As can be seen in the screenshot, the details for the style prop are missing from the View docs page. This is a regression between RN 0.43 and 0.44:

Thanks for reporting this. We noticed this but have yet to work on a fix.

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.

@hramos This issue still repros. Should we reopen it?

Yep, definitely want to fix this.

Any update on this?

No updates. As far as I am aware, this is still up for grabs.

@RyanMitchellWilson if you need to see more details about style, you can look at the RN 0.43 docs as a workaround: http://facebook.github.io/react-native/releases/0.43/docs/view.html#style

Still a problem, can this please be put back into the current documentation and kept up to date?

We've made some changes to the docs recently. We are no longer using react-docgen, so you can go ahead and add the missing props directly to the markdown source for each of these components. The docs are now at https://github.com/facebook/react-native-website.

The solution in the latest docs looks good to me. The style prop on the View page has a link to the View Style Props doc.

Great to hear, that's one of the things I had to manually re-create when react-docgen was removed.

Doesn't seem like the problem was fixed, can this be reopened? Still not seeing anything for style in the docs for View

@RyanMitchellWilson I think this was fixed in the  latest version of the docs . You can see that style is a link.

@rigdern that doesn't really help me and others who are using previous versions. Plus when you go to https://facebook.github.io/react-native/ you are not directed to that version of the website, you are placed on the latest version, which currently is 0.51. This issue was meant to fix all the wrong documentation to help the people using those versions of react-native, not just the people using the very latest.

@RyanMitchellWilson that's a good point.
@hramos should the fix be back ported to previous versions of the React Native docs?

You can still access past versions of the site by clicking on the version number at the top. Example:

Version 0.5 style props: https://facebook.github.io/react-native/docs/0.5/view-style-props.html
Version 0.49 style props: https://facebook.github.io/react-native/docs/0.49/view-style-props.html
Style props from master: https://facebook.github.io/react-native/docs/next/view-style-props.html

This repository no longer contains the docs. Please do open an issue at https://github.com/facebook/react-native-website if you'd like to suggest changes to the way View style props are documented.

Ya I know you can get to previous versions that is fine, it is not intuitive to get to the master version is what I was saying.
I guess I'll go open a new issue in react-native-website even though I've had this one open for 7 months.


dashboards: make reusable from other clusters

The datasource needs to be set to VictoriaMetrics instead of a hardcoded, cluster specific UID to make these dashboards reusable.
Tested and working in Mamba World.

Merged, as this is a hotfix-type mod for downstream use and attested to work as intended on that particular downstream world.


Unable to find libclang.so

When using cross to cross compiling crate zstd in PR https://github.com/ryankurte/cargo-binstall/pull/148, the CI that use-cross: true all failed with libclang.so not found.
Edit:
The PR enabled feature bindgen of zstd and bindgen uses libclang.so to parse C headers.

https://github.com/ryankurte/cargo-binstall/runs/6665668295?check_suite_focus=true#step:7:569

works kinda on the newest images, either pull the newest unreleased cross version to override whatever actions-rs does, or set
CARGO_TARGET_${TARGET}_IMAGE=ghcr.io/cross-rs/armv7-unknown-linux-gnueabihf:edge

seems like there's some more issues though, we're missing linux-libc-dev-<target>-cross on the linux images

We are using cross via actions-rs/cargo by setting use-cross: true.
Does changing env work for actions-rs/cargo?

Yes, it works for everything using cross.
You can also install cross before invoking use-cross and if actions-rs finds the cross binary it won't/shouldn't override it
You can adjust the image in the Cross.toml too ofc :)

@Emilgardis The build still failed:
  /usr/arm-linux-gnueabihf/include/limits.h:123:16: fatal error: 'limits.h' file not found

As mentioned, missing linux-libc-dev-<target>-cross @NobodyXu
We should include that, pr pending :)

As mentioned, missing linux-libc-dev--cross @NobodyXu

Ok, sorry I missed that part.

hmm, seems like that's not it

@Emilgardis We need it to be statically linked so that it can run on everywhere, so we probably cannot use external version of libzstd that is dynamically linked.

of course, the issue is not with cross now however, you need to install zstd and clang-3.9 in the image that cross uses so that it/cargo/bindgen can find the relevant files.


Add test commandline to the output of failed tests in jck reporting

Updates xUnit xslt script to print out test path (url) in the junit output
Also prints command line to reproduce the failed test - this will allow users to not have to download the output zip and look into .jtr file to find the command line to execute.
Related to backlog/issues/187

Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Tested here: Test_grinder/job/Grinder/3834/testReport/junit/api_signaturetest_sigtest/html_java_security_jgss/javasoft_sqe_tests_api_signaturetest_SignatureTestBase/


App not building after upgrading to latest flutter 2.9 in master

After upgrading from Channel master, 2.6.0-12.0.pre.826 to Channel master, 2.9.0-1.0.pre.384,
facing this issue, once run and trying to install the app
Launching lib/main.dart on Android SDK built for x86 in debug mode...
Running Gradle task 'assembleDebug'...
../../../flutter/.pub-cache/hosted/pub.dartlang.org/platform-3.0.2/lib/src/interface/local_platform.dart:46:19: Error: Member not found: 'packageRoot'.
io.Platform.packageRoot; // ignore: deprecated_member_use
^^^^^^^^^^^
FAILURE: Build failed with an exception.

Where:
Script '/Users/xxx/Development/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 1102

What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.

Process 'command '/Users/xxx/Development/flutter/bin/flutter'' finished with non-zero exit value 1

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 23s
Exception: Gradle task assembleDebug failed with exit code 1

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 2.9.0-1.0.pre.384, on macOS 12.0.1 21A559 darwin-x64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
[✓] Chrome - develop for the web
[!] Android Studio
✗ Unable to find bundled Java version.
[!] Android Studio
✗ Unable to find bundled Java version.
[✓] Android Studio (version 2020.3)
[✓] IntelliJ IDEA Community Edition (version 2021.2.3)
[✓] Connected device (2 available)
[✓] HTTP Host Availability
! Doctor found issues in 2 categories.

This is an issue with the platform package, which was fixed in its 3.1.0 version and likely is transitive for you through some other package, which didn't update the version yet. Can you attach an output of flutter pub deps, so we can see what this package is?
In the meantime you can fix it by overriding the version in your pubspec.yaml like so
dependency_overrides:
  platform: ^3.1.0

As noted this is an issue with needing to update one or more of your dependencies.
Closing as this is not a Flutter bug.

@stuartmorgan one of the transitive deps that is causing this for me is the current version of path_provider, whose bugs I believe are supposed to be filled in this repo. Or should a new issue be opened for that?

@maks If path_provider were pinning a broken version of platform this would be the place to report that, but that's not the case. No change to path_provider is needed here.
Re-reading the original report, I'm not sure why @nt4f04uNd mentioned waiting for packages to update and suggested using overrides (a method of last resort) when there's no evidence in the report that either is necessary. Unless some dependency is pinning 3.0.2 (extremely unlikely), all that's needed is to run a package update on the app.

Re-reading the original report, I'm not sure why @nt4f04uNd mentioned waiting for packages to update and suggested using overrides

I'm sorry, I forgot about this command, and that 3.0.2 can be upgraded to 3.1.0 with it

@stuartmorgan my apologies, I saw a reference to path_provider having platform: 3.0.0 (pinned to 3.0.0) in a comment in a SO thread and didn't notice that the commenter had got the dependency version spec string wrong and that its actually platform: ^3.0.0.
I've left a comment on that SO answer to try to prevent future readers making the same mistake.


[Feature] Android support for MediaStore.Downloads

Not critical, but it would be nice to be able to read media from MediaStore.Downloads too. Thanks 😉

any update on this feature? i want to be able to get images from downloads folder

Please provide the version of the package that you're using. And:

photo_manager: ^1.2.1
I am using AVD emulator with API 30, Android 11.1
Images from here are not showing up

These are the directories it was able to detect which have images

Unable to detect images from downloads even on the example project and also my own project

Just created a new simulator and the Download folder exist and displayed.

So AFAICT this is a device/entity specific issue. Closing.

The same as in @momoDragon's case I've been able to get images on simulator from Recent and Pictures, but not from the Downloads.
Android 11.0 API 30
photo_manager 1.2.2
Code I used to get the albums:
PhotoManager.getAssetPathList(     onlyAll: true,     type: RequestType.image )
@AlexV525 Can you give us a link to the app code you've used for testing this issue?

@AlexV525 Can you give us a link to the app code you used for testing this issue?

I'm using the example to test.

So, yeah, I've tested the example on the same simulator as described above and got only Recent and Pictures, but not Downloads. Though I have pictures in that directory as well.

I think this is because the MediaStore.Downloads table (mentioned by @krokyze) is not in the selections yet, as it follows from the plugin's code:
val selection =             "${MediaStore.Images.Media.BUCKET_ID} IS NOT NULL $typeSelection $dateSelection $sizeWhere) GROUP BY (${MediaStore.Images.Media.BUCKET_ID}"
According to this link it looks like it has to be there for Android 10+ to get files from the Downloads:

These are just my 2 cents for this issue. Can not do more because I'm not an Android dev. Hope it will help...

So now this issue become an environmental issue. How did you add your files into the Downloads folder? I simply drag all images to the emulator then let it import them.

Also can you try with the latest version (1.2.7)?

Yes, the same as you, I just dragged my images onto it. And yes, I've tried with 1.2.7.

Yes, the same as you, I just dragged my images onto it. And yes, I've tried with 1.2.7.

If you're using drag, you can check whether those images are showing in the photo app. Once they appear, the download folder should be appear too. If they don't, try a reboot first.

If you're using drag, you can check whether those images are showing in the photo app. Once they appeared, the download folder should appeared too. If they don't, try a reboot first.

I've tried what you suggested.
Images sent to the simulator are instantly available in the Downloads directory through the Files app. But Google Photos app seems to not refetch data from Downloads until reboot. So finally after reboot it started working as expected.
Thank you for assistance. I think I need to conduct more testing on real devices. The solution with 'rebooting' doesn't look promising for the end-user experience. But it may work in a different way on real devices.

Thank you for assistance. I think I need to conduct more testing on real devices. The solution with 'rebooting' doesn't look promising for the end-user experience. But it may work in a different way on real devices.

That is somehow related to MediaStore scanning, it's not triggered instantly as users might expected, and can only be triggered randomly, which is why you can see there is a menu named PhotoScanner in the Photos app.

Ok, thank you for the explanation! But anyway do you think if selecting from the MediaStore.Downloads table in addition to the MediaStore.Images can have a chance to solve this issue?

But anyway do you think if selecting from the MediaStore.Downloads table in addition to the MediaStore.Images can have a chance to solve this issue?

The root cause is the scanning, as I mentioned before. So I don't think that targeting the Downloads would help. Maybe this time it happens with the Downloads, but next time with other albums/folders.


Portfolio development

Personalize portfolio in 3 commits by:

Updating index.html with personal details

Adding picture of myself

Changing the addRandomGreeting function to addRandomFact

Lgtm. I also tried to suggest an indenting edit but but feel free to take it or leave it.


collector failed to handle concurrent job using hadoop

What version of pinpoint are you using?
v1.7.3
Describe your problem
We have pinpoint 1.7.3 agent deployed for java application, and pinpoint collector, web connected with hadoop 2.7.6 (three nodes cluster), but we got error from the collector as below,
2018-09-12 19:51:26 [WARN ](o.j.n.c.SimpleChannelHandler       :76 ) EXCEPTION, please implement com.navercorp.pinpoint.rpc.server.PinpointServerAcceptor$PinpointServerChannelHandler.exceptionCaught() for proper handling.
java.util.concurrent.RejectedExecutionException: Task com.navercorp.pinpoint.collector.receiver.tcp.AgentBaseDataReceiver$3@4a9b01d8 rejected from java.util.concurrent.ThreadPoolExecutor@5df91120[Running, pool size = 8, active threads = 8, queued tasks = 1024, completed tasks = 0]
at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2063)
at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:830)
at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1379)
at com.navercorp.pinpoint.collector.receiver.tcp.AgentBaseDataReceiver.requestResponse(AgentBaseDataReceiver.java:172)
at com.navercorp.pinpoint.collector.receiver.tcp.AgentBaseDataReceiver.access$100(AgentBaseDataReceiver.java:55)
at com.navercorp.pinpoint.collector.receiver.tcp.AgentBaseDataReceiver$1.handleRequest(AgentBaseDataReceiver.java:137)
at com.navercorp.pinpoint.rpc.server.DefaultPinpointServer.handleRequest(DefaultPinpointServer.java:374)
at com.navercorp.pinpoint.rpc.server.DefaultPinpointServer.messageReceived(DefaultPinpointServer.java:320)
at com.navercorp.pinpoint.rpc.server.PinpointServerAcceptor$PinpointServerChannelHandler.messageReceived(PinpointServerAcceptor.java:384)
at org.jboss.netty.channel.SimpleChannelHandler.handleUpstream(SimpleChannelHandler.java:88)
at org.jboss.netty.channel.DefaultChannelPipeline.sendUpstream(DefaultChannelPipeline.java:564)
at org.jboss.netty.channel.DefaultChannelPipeline$DefaultChannelHandlerContext.sendUpstream(DefaultChannelPipeline.java:791)
at org.jboss.netty.channel.Channels.fireMessageReceived(Channels.java:296)
at org.jboss.netty.handler.codec.frame.FrameDecoder.unfoldAndFireMessageReceived(FrameDecoder.java:462)
at org.jboss.netty.handler.codec.frame.FrameDecoder.callDecode(FrameDecoder.java:443)
at org.jboss.netty.handler.codec.frame.FrameDecoder.messageReceived(FrameDecoder.java:303)
at org.jboss.netty.channel.SimpleChannelUpstreamHandler.handleUpstream(SimpleChannelUpstreamHandler.java:70)
at org.jboss.netty.channel.DefaultChannelPipeline.sendUpstream(DefaultChannelPipeline.java:564)
at org.jboss.netty.channel.DefaultChannelPipeline.sendUpstream(DefaultChannelPipeline.java:559)
at org.jboss.netty.channel.Channels.fireMessageReceived(Channels.java:268)
at org.jboss.netty.channel.Channels.fireMessageReceived(Channels.java:255)
at org.jboss.netty.channel.socket.nio.NioWorker.read(NioWorker.java:88)
at org.jboss.netty.channel.socket.nio.AbstractNioWorker.process(AbstractNioWorker.java:108)
at org.jboss.netty.channel.socket.nio.AbstractNioSelector.run(AbstractNioSelector.java:337)
at org.jboss.netty.channel.socket.nio.AbstractNioWorker.run(AbstractNioWorker.java:89)
at org.jboss.netty.channel.socket.nio.NioWorker.run(NioWorker.java:178)
at org.jboss.netty.util.ThreadRenamingRunnable.run(ThreadRenamingRunnable.java:108)
at org.jboss.netty.util.internal.DeadLockProofWorker$1.run(DeadLockProofWorker.java:42)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
I also checked the log in agent, nothing help.
2018-09-12 20:32:08 INFO  Sending AgentInfo TAgentInfo(hostname:VM_66_91_centos, ip:<IP_ADDRESS>, ..........................
jvmInfo:TJvmInfo(version:0, vmVersion:1.8.0_181, gcType:SERIAL))
2018-09-12 20:32:12 WARN  request timed out while waiting for response.
What have you done?
Steps you have taken to trigger the problem, for example:

Launched '...' with agent
Tried tracing '...' (see agent log below)
Collector outputs trace log (see collector log below)
No trace shows up in the web UI (see web log below)
all aboves were done and logs genated as above.

Screenshots
If applicable, add screenshots to help explain your problem.
don't need screenshot
Logs
If applicable, please attach agent/collector/web DEBUG log that includes the part where you have encountered the problem. In case of agents, including the start-up log may be of great help.
Additional context
Add any other context about the problem here, such as affected library for agents, how your collector/web/hbase is set up if applicable.
it is very typically java aplication using tomcat or jar, also we have multiple applications using pinpoint to monitor, so I believe the problem is not with the applications but pinpoint collector itself.
BTW: we have testing environment has the same pinpoint version installed, the only difference is using single node Hbase, but it works very well.
thanks a lot for your help.

hi @pikuzi
The problem can be adjusted by modifying the number of threads and the size of the queue.
(in pinpoint-collector.properties)
However, a completed tasks value of 0 means that the logic may be stopped at some point.
I recommand to record the collector's thread dump in that situation.
and try to find where the operation is not completed
thanks.


Error loading libritts model

Describe the bug
I'm trying to load the LibrittsPretrainedLibrittsFinetuned, but am running into a few errors.  Here is what I have so far:
import yaml
from espnet2.tasks.mlm import MLMTask
import torch
from argparse import Namespace
from huggingface_hub import hf_hub_download

repo_id = "richardbaihe/a3t-libritts"
chkpnt_file = hf_hub_download(repo_id=repo_id, filename="LibrittsPretrainedLibrittsFinetuned/train.loss.ave_5best.pth")
chkpnt = torch.load(chkpnt_file)
config = yaml.safe_load(open(hf_hub_download(repo_id=repo_id, filename="LibrittsPretrainedLibrittsFinetuned/config.yaml")))
args = Namespace(**config)
args.use_scaled_pos_enc=False  # <- This is missing from the config, set to False to match vctk config?
model = MLMTask.build_model(args)
model.load_state_dict(chkpnt)

I needed to download a feats_stats.npz file, which I found here.  I think this should be the right one?
With this I'm still getting the following error:
AttributeError: 'Namespace' object has no attribute 'encoder'

Is there an updated config that I can use to load the model, or am I doing something wrong?  Thanks for your help in advance!

Hi @lematt1991 , this is a TTS model finetuned with A3T checkpoints. so, the task is notMLMTask , but from espnet2.tasks.tts import TTSTask. Hope this help solve your problem.

@richardbaihe is the libritts pre-trained A3T model publicly available?  Or only the one finetuned on the TTS task?  Thanks!

OK, thanks for clarifying


InputAttributes.Autocomplete does not accept strings

(Raised a similar issue in Bridge at https://github.com/bridgedotnet/Bridge/issues/2970)
Currently the AutoComplete property on input attributes only accepts an enum with a value of On or Off, however it's valid to have arbitrary strings and also a set of defined ones (like new-password and address-line1, etc.):
https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#autofill-detail-tokens
Should this be Union<string, AutoComplete>?

Makes sense. I see that Bridge have already made the change on their end!


Resize UI has incorrect calculation when switching from MB to GB

When I switch from MB to GB as the unit of measure on the resize panel the calculation is incorrect. Ex. 1024MB should become 1GB but it does not.

Fixed.


Drop terraform remote, use ansible s3 tasks

See individual commits for details

lgtm, comments pending

lgtm, pending the readme note.


Spotify -> cant play this right now -> seems related 2 winget

Brief description of your issue
installing Spotify trough winget gives me can’t pay this right now on all songs,
I’ve not had this issue before when I had internet.
Steps to reproduce
run winget install spotify (i did use
function wi ($target) {
    Winget install $target
}

try and play song.
Restart pc.
Try and play song
Expected behavior
song plays
Actual behavior
I get the error
cant play this right now
Environment

Windows: Windows.Desktop v10.0.19043.928
Package: Microsoft.DesktopAppInstaller v1.12.11692.0

spotify ps5

I can't replicate this issue, I am able to play all available songs. Try reinstalling Spotify.

The issue was with drivers of the soundcard.


Does each sub-mixture instructed both with zero-shot and few-shot?

We have four sub-mixture: muffin, t0-sf, cot, niv2. And a set of instruction settings: zsopt, zsnoopt, fsopt, fsnoopt. Are scaling flan use the all the combinations?
If so, does few-shot setting leak data information for zero-shot setting on a same dataset?
If not, can you give us the exact instruction settings on each sub-mixture?

@sxthunder Yes, we use all combinations in our final model. I'll update the README soon to clarify this.
As mentioned in #19 there is data leakage between the few-shot exemplars and and other training examples over the course of training. I would note though that we trained all our models this way, and it doesn't seem to have had a negative impact as far as we can tell. It's possible that splitting off a few-shot exemplar set of data might improve results, but it would also reduce the few-shot exemplar diversity. Curious to hear the results if anyone does this experiment.

@shayne-longpre Hi，Thanks for your reply. I have read your latest paper: The Flan Collection: Designing Data and Methods for Effective Instruction Tuning , which explain many training details of Flan-2022. Now we know four key elements of instruction tuning: COT, Input Inversion, Mixture Balance and Few Shot templates.
After reading a series of instruction tuning papers(Flan, T0...), can we conclude the possible steps to reproduce chatgpt:

it is necessary to finetune a naive pre-training model by dataset-instruct-tuning(using open source dataset with instructions, like Flan and T0). To preliminary learning how to follow a instruction, and improves its zero-shot ability on unseen task.
using some hand-write instruction dataset(like SFT step of instruct-gpt), because instructions in step 1 has some difference in open-ended. Like people will directly ask chatgpt "how to learning deep learning" without the instruction "ask the following question"
finally using RLFH to align the model output to human.....