Technical details

y .9
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

<!--
Make sure to take this Changelog template out of the comment block in order for it to show up.
no cl weh

all chem ratios should be irrational numbers

Its not 1 because bruizine does not require heat and Bica can be made to create razorium. If you are not careful you will make it due to leftover 10%, chemists should care what they are adding for at least some advanced chems.

nullified by chemmaster existing

0.9 lithium is intentional to make the recipe slightly more difficult/complicated to make. Chemmaster or not I've seen people discussing their strategies on how to make it safely which i think is infinitely better than just a normal 3 chem reaction

0.9 lithium is intentional to make the recipe slightly more difficult/complicated to make.  This sort of thing is fine once we get chem rework but right now it's just an ugly uneven reaction that can be fixed simply by discarding excess. I'd also like to point out that, as it is now, we have no way to define custom amounts in either chem dispenser or chemmaster. It can be reimplemented later and yeah I'd like less even chems eventually but with our current system now is not really the time

lithium add 50 remove 5
bicar add 50

lithium add 50 remove 5
bicar add 50

That still doesn't change my statement at all.  If we have forced partial units then we need to be able to have custom transfer amounts. Imo it's unreasonable to think that we need this medicine to be special, just follow the format of literally every other chem until all of them are changed or a better solution is implemented. Not even the other new meds use the decimal unit.

having every chemical be almost exactly the same is boring especially in a subdepartment with already incredibly low gameplay variation. having to dispose of excess products and be a bit extra careful with how you mix stuff is something new for chem and really not that annoying to deal with compared to other chemical recipes. further standardizing gameplay in a department where the entire gameplay loop is clicking buttons is not good imo.
also, the way you are "fixing" this reaction makes bruizine notably easier to make than the other brute meds, which require heat.

having every chemical be almost exactly the same is boring especially in a subdepartment with already incredibly low gameplay variation. having to dispose of excess products and be a bit extra careful with how you mix stuff is something new for chem and really not that annoying to deal with compared to other chemical recipes. further standardizing gameplay in a department where the entire gameplay loop is clicking buttons is not good imo.
also, the way you are "fixing" this reaction makes bruizine notably easier to make than the other brute meds, which require heat.

the whole system is broken my man, fix it entirely or don't make single special chemicals at all. also we need more items that only mix with heat anyway. heat doesnt fuck up chem gameplay and make it more annoying, and it actually does add more variation. all you did was add more (pointless) clicks that end up being entirely useless because of the already broken system. and again, people can ignore the chem entirely so they do

the whole system is broken my man, fix it entirely or don't make single special chemicals at all.

bruizine and the reaction work fine, the reaction system isn't broken. and if it is somehow then fixing it is preferable to removing stuff

also we need more items that only mix with heat anyway. heat doesnt fuck up chem gameplay and make it more annoying, and it actually does add more variation.

you can add new reagents with heat reactions if you wish, this doesn't mean you have to go to other reagents and make them require heat.

all you did was add more (pointless) clicks that end up being entirely useless because of the already broken system

if it's as useless as you claim then it shouldn't make any difference. bruizine is the only chemical that stands out with an odd mix ratio because it's one of the first ones to be added. unless a maintainer disagrees i really don't see an issue with adding slightly more thinking to make more complex reactions instead of just making the steps longer.

chem gameplay also isnt "boring" to people who like that sort of production-type gameplay. I haven't met a chemist who didn't avoid bruizine entirely due to this reason.

ive seen several chemists make bruizine just fine so it seems like a skill issue.

bruizine and the reaction work fine, the entire reaction system isn't broken, and if it is somehow then fixing it is preferable to removing stuff
"broken" as in its overpowered and is going to be overhauled soon, so just wait

you can add new reagents with heat reactions if you wish, this doesn't mean you have to go to other reagents and make them require heat.
I would like to see reactions involving cooling, mixing a supercooled chem with a superheated chem, etc but changing to a decimal number is a sad excuse for a change in gameplay.

if it's as useless as you claim then it shouldn't make any difference. bruizine is the only chemical that stands out with an odd mix ratio because it's one of the first ones to be added. unless a maintainer disagrees i really don't see an issue with adding slightly more thinking to make more complex reactions instead of just making the steps longer.

there is not "slightly more thinking" you just pour shit in a beaker, throw it in a chemmaster, and get rid of the extra (which has no drawback because of the broken system I mentioned before!) all youre doing is making things more complex which is just entirely bypassed with our current system.

ive seen several chemists make bruizine just fine so it seems like a skill issue.
wouldnt call it a skill issue, just use bicar whitch is a catchall and not have the terrible death chem as well as the extra steps for a relatively boring chemical

I apologize if I am coming off as heated, I've had to deal with a lot of bullshit today. I think the argument is a bit silly because it doesn't actually make the gameplay more fun, nor does it make it harder or more complicated. It's an inconvenience that just discourages people from using the new stuff. I think the partial units should just wait until the refactor to be implemented.

I'll just leave with a bit of design philosophy on why I made bruizine the way it is.
1: 3 new chemicals, making all three the exact same is boring so I made it a point to give them a few different characteristics to reward doctors and chemists who pay that little bit of extra attention.
2: The dumping of excess chems will be more significant once the chem dispenser 1984 happens, which iirc just has a few more requirements before it's ready to go, I don't remember the issue number but chemists won't have infinite elements anymore.
3: The new brute chems are twice as effective as bicaridine at their respective damage type. Bicaridine is simple to make, but doctors and chemists who work with the slightly more complex systems will be rewarded with faster healing speed.
4: Odd ratios like 0.9 allow razorium to be created accidentally, through artifacts, liquid anomalies, or chemical spills. I wanted razorium/bruizine to have this available (since it's rare for chemicals to be heated naturally) without making bruizine be bland and boring to make

I'll just leave with a bit of design philosophy on why I made bruizine the way it is.
1: 3 new chemicals, making all three the exact same is boring so I made it a point to give them a few different characteristics to reward doctors and chemists who pay that little bit of extra attention.

I'm not saying it's a bad idea, it's just not worth it right now to have the ratios different from everything else when there is going to be a lot of changes in an undetermined amount of time.

2: The dumping of excess chems will be more significant once the chem dispenser 1984 happens, which iirc just has a few more requirements before it's ready to go, I don't remember the issue number but chemists won't have infinite elements anymore.

This is exactly part of the rework I am talking about.

3: The new brute chems are twice as effective as bicaridine at their respective damage type. Bicaridine is simple to make, but doctors and chemists who work with the slightly more complex systems will be rewarded with faster healing speed.

Which is cool but at some point (everything I've seen so far) they just mass produce bicar.

4: Odd ratios (with no heat) like 0.9 allow razorium to be created accidentally through artifacts, liquid anomalies, or chemical spills, without compromising too much recipe complexity. I wanted razorium/bruizine to have this available (since it's rare for chemicals to be heated naturally) without making bruizine be bland and boring to make

I understand, but making it more boring to make is probably not the solution. It's not even a different step, just more clicks which you already mentioned that you wanted to avoid.

3: The new brute chems are twice as effective as bicaridine at their respective damage type. Bicaridine is simple to make, but
doctors and chemists who work with the slightly more complex systems will be rewarded with faster healing speed.

Twice is understatement. 5u of Bica treats 6.(6) puncture/slash/bruise damage. Pun/Lac/Bru heal from 30 to 40. They are very much worth it if you can pay attention while making them and treating patients. You can reduce treatment from few minutes of lying in bed to instantly being sent to nukie front line in best cases and like half minute treatment otherwise tops.
I love those meds though cryo pods are bit sad now as mass brute damage was go to use for them by me usually, you do not always get a clusterfuck of damage types patients for it and caustic can easily be treated with Sig...

3: The new brute chems are twice as effective as bicaridine at their respective damage type. Bicaridine is simple to make, but
doctors and chemists who work with the slightly more complex systems will be rewarded with faster healing speed.

Twice is understatement. 5u of Bica treats 6.(6) puncture/slash/bruise damage. Pun/Lac/Bru heal from 30 to 40. They are very much worth it if you can pay attention while making them and treating patients. You can reduce treatment from few minutes of lying in bed to instantly being sent to nukie front line in best cases and like half minute treatment otherwise tops.
I love those meds though cryo pods are bit sad now as mass brute damage was go to use for them by me usually, you do not always get a clusterfuck of damage types patients for it and caustic can easily be treated with Sig...

36.75 for a single specific brute (3.5 heal and 10.5 OD)
vs
30 for all simultaneously (2 heal and 15 OD)

up to 30, it will do less if you have a single type

up to 30, it will do less if you have a single type

yeah. Thats what I meant.
also the single type thing is where you should use specialized if you have a LOT of it but I feel like not having multiple is rarely the case in combat.
and finally that is also why it restricts you to using bicar. if you use multiple specialized you die worse

the specialty meds are supposed to be more difficult to make because of how efficient they are

its also more difficult to use, difficulty to make doesnt really matter at scale since at the end of the day you have a 200u jug of whatever medicine

the specialty meds are supposed to be more difficult to make because of how efficient they are

It's still not any more difficult to make.   Using it to heal a stubbed toe would be a waste.

its also more difficult to use, difficulty to make doesnt really matter at scale since at the end of the day you have a 200u jug of whatever medicine

That's what it is right now because of infinite chems, hence "let's wait until the overhaul before messing with partial units"

I've been playing a lot of chemistry recently and I gotta say I love the odd bruizine ratio, it's one of the only chems I do have to think about making since not only is it an odd ratio, but it also doesn't produce the same amount of chems as you put in so you can't just make 90/100u and dump it into a jug a few times like with all the other chems.

i think that this weird little quirky chemical is nifty. Like, it's not the most complicated thing to make and the odd ratio doesn't do all that much, but it's more than a lot of other stuff so that's not half bad. It's probably worth keeping until there's some other worthwhile balancing factor.

i think that this weird little quirky chemical is nifty. Like, it's not the most complicated thing to make and the odd ratio doesn't do all that much, but it's more than a lot of other stuff so that's not half bad. It's probably worth keeping until there's some other worthwhile balancing factor.

I suppose I just wish either more chemicals did it, or not have it all. I don't like the weird in between.

more quirky chemicals coming soon godo

more quirky chemicals coming soon godo

Please do. Probably reference this PR as well; it kinda includes reasons as to why it should be included. Definitely look at bicar too, because I still am yet to find any of the new ones used extensively.


python: testresources: Move pbr to propagatedBuildInputs

Motivation for this change
Since testresources declares Requires-Dist: pbr (>=1.8) and imports pbr.version, any user of testresources also needs to have pbr.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"

python38Packages.zeep fails to build, but that’s not a regression

[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @dotlambda

Can you please change your commit message to pythonPackages.testresources: ... ?
That's a more widely used convention.
Otherwise LGTM.

Can you please change your commit message to pythonPackages.testresources: ... ?

Done.


Writing to cache/store after generation

Thank you for the great software!
Do I understand correctly, that at this time it is not possible to write vtile generated by tileserver on demand to cache/store. So that if the same request from the client comes in after some time, same tile would not have to be generated again.
Looks like currently cache/store is only filled by running tilequeue script and generating specified tiles (and it is hard to know which tiles would be requested, very low percentage of high zoom level tiles are ever requested so there is no point to generate them all and it takes a lot of time). Tiles not found in cache/store and generated on demand by tileserver are thrown away?
Thank you

I took out the store code in #85 because Mapzen is trying to optimize its tile response time (by doing more work in tilequeue and relying less on tileserver).
We probably should have kept it in there and configured it to be off on our servers. I'll look at doing that next week, but for now if you want to take a look at that pull request to see what was taken out you can see how we implemented storing tiles in tileserver.

Like @iandees mentioned, We're planning on adding some kind of caching back in ourselves. Sounds like we might want to think about growing an abstraction for this, or maybe use the same store interface that exists but grow a redis implementation for it.

@tomass In the meantime, I suggest using the tagged production release instead: https://github.com/tilezen/tileserver/releases/tag/v1.4.0.

Thank you a lot for information and suggestions! It was really helpful!


Do not migrate active topic management

Purpose of Changes and their Description
This PR forgoes migrating which topics are active, in favor of an approach where we manually re-activate topics after the upgrade.
Link(s) to Ticket(s) or Issue(s) resolved by this PR
#574
Are these changes tested and documented?
Re-uses existing tests.

Closed in favor of:
https://github.com/allora-network/allora-chain/pull/574


Can't load maven website!

I'm trying to download package throught site you provided but this Maven site is not load anymore... it's always show like pic mentioned ☝ ... can't load.

https://mvnrepository.com/artifact/com.ramotion.foldingcell/folding-cell/1.2.3


Add prometheus support to gitlab-ce

The integrated prometheus in Gitlab does currently not work with an RBAC enabled cluster and is also not considered in the charts service template.
This fixes #1191

Note: I actually signed the CLA and am still not sure why it shows up as not signed.

Can gitlab-ce use an external prometheus? There already is a prometheus chart and it might be easier to maintain then if it was reused?

Yes I agree with @kfox1111 that this should make a conditional dependency on the Prometheus chart and be able to point to an existing chart.
Please update with those changes.

Linking this to the discussion underway in #1138

Closing this because it increasingly looks like we're going to deprecate our GitLab charts and point users to the "official" ones. I encourage you to submit this work there.


[api-docs] Update message-components-bug-report.md

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/8ff5f77bc5c0a8a64d875d267a241dca4521ab26

This commit will not effect our lib's code. No changes required.


[Hotfix][E2E][Clickhouse] Fix the bug of clickhouse e2e case

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

title：[Bug][E2E][Clickhouse] Fix clickhouse e2e testcase

title：[Bug][E2E][Clickhouse] Fix clickhouse e2e testcase

Done.


clearing env vars

it should be possible to have an env var deleted when in a directory, for example, you might have JAVA_HOME set globally but within a project you might want it completely unset (not set to an empty string). toml doesn't have null, so I think "false" is the best way to define this:
[env]
JAVA_HOME = false

nobody has asked me about this yet, but I'm sure someone is going to need it

lol I love that this ended up being #404 :)


cannot fold namespaces in c++ files

VSCode Version: 1.8.1
OS Version: Linux

Steps to Reproduce:

Write following code

#include <iostream>

namespace someName{
    void someMethod1(){
        int n = 6;
        return n;
    }

    void someMethod2(){
        int n = 6;
        return n;
    }
}

int main(){}

Try to fold code b/w namespace, no [+] icon is shown in gutter.

duplicate of #3422


Remove deprecated crypto

Per issue #305 and #308, we shouldn't be listing the crypto library in our package.json anymore.
This is included by default in nodejs now, and so simply deleting it from the package.json works -- even with webpack.

Thank you!!


bytes/buffer: add Buffer.Init([]byte)

currently bytes.NewBuffer can be used to initialize Buffer with a custom buf, but in performance-sensitive scenarios, I hope to reuse Buffer to avoid heap allocation and memcpy, so I added the Init method so that I can replace a Buffer's buf.

@cwbhhjl fyi, you need to create a proposal issue and it to be accepted before sending a PR.
This wont be accepted.
You should read: https://github.com/golang/proposal#the-proposal-process

@cwbhhjl fyi once you closed a PR, the bot will close the matching one on gerrit, but afaik gerrit doesn't support reopening.
You need to make a new PR.

@Jorropo
Thanks, I created an issue, if the proposal can be accepted, I will create a new pr.

https://github.com/golang/go/issues/53443


I have some questions about pysc2 baseline agent of Deepmind.

I have some questions about pysc2 baseline agent of Deepmind.
https://deepmind.com/blog/deepmind-and-blizzard-open-starcraft-ii-ai-research-environment/
I'm trying to implement baseline agent illustrated in the pysc2 paper.
My GitHub: https://github.com/chris-chris/pysc2-examples
Question1. How can I implement 'the policy in an auto-regressive manner, utilizing the chain rule'?

I got the idea of masking unavailable actions from policy network result.
But I don't get the idea of 'the policy in an auto-regressive manner, utilizing the chain rule'
If I choose the first action, then how can I get the second parameters and the third action parameters in an auto-regressive manner?

Question2. How can I define 'action space' for A3C for pysc2? (Similar one with Q1)
I'm kind of familiar with Discrete action space.

I got the concept of calculating the spatial action policy using 1x1 convolution layer for dimension reduction.
But what the action space would look like?
It should not be like tf.int32.
Maybe it can be MultiDiscrete action space.
[move_screen(categorical int), [[0](categorical int), (x1(int), y1(int))]]
If it is a MultiDiscrete Action Space, then how can I define policy loss function?

If you know some reference codes on Github, please let me know.
or some relevant papers and articles would be welcome.

While my team and I are still figuring this out, you may want to ensure that you fully understand how autoregressive policy outputs work.
Discrete Sequential Prediction of Continuous Actions for Deep RL was a really good paper on understanding how these types of policies are implemented. While I am not too familiar with Multi Discrete action spaces in tensorflow, it's important to take note of the fact that implementations of this type of policy can easily grow infeasible in the number of parameters.
It would be great to get insight from an engineer on the Deepmind team about this issue (seems like lots of people are having it) and hopefully, we can keep this issue thread open for more discussion as we continue to work on this topic

I am also working on implementing the baselines agent from the release-paper. I think for the mini-games it's enough to set all other arguments as 0 expect the screen and minimap and the paper is clear how to handle those.

I applied ACKTR(A3C) on my Github.
https://github.com/chris-chris/pysc2-examples
It is learning but, I think it needs more improvements.
Trained multiple Pi
for base_action(524) sub1_action(2) sub2_action(3~10) sub3_action(500) x1, y1, x2, y2 (64)
and multiplied loss functions.
Your feedback is welcomed and appreciated.

@chris-chris I think I know the answer to your Q1. I will refer to the equation numbers in the paper. You simply substitute the pi_theta(a|s) expressions in (1) with the identity in (2). Because all of these are inside a log function, the resulting expression will be a sum, not a product.
Therefore computing gradient of a sum is much easier that of a product. For the policy gradient part, you start with a grad(log(pi_theta(a|s))) and end up with grad(log(pi_theta(a^0|s)) + grad(log(pi_theta(a^1|s)) + ...
using simple calculus rules.
If I've got this wrong someone please correct me.
I have two questions myself.
I am interested in how the authors of the paper handled the spatial parameters in their baseline agents. In the paper, it is specifically stated that there is only a single spatial output block, which consists of 1x1 convolution. How then do you select the second point, which is necessary for performing the select_rect(p1, p2) action? (the logical solution seems to me as using simply 2 spatial output matrices, but then what about the reference in the paper?)
I would also like to ask you to elaborate on this sentence from your paper:
'We embed all feature layers containing categorical values into a continuous space, which is
equivalent to using a one-hot encoding in the channel dimension followed by a 1 × 1 convolution.'
I kind of understand this but it doesn't seem as equivalent to me. I agree that in the first layer, the categorical inputs get encoded into some real values anyway but they can be encoded by several neurons in a very different manner, depending on the weights. So how am I to understand this concept?
I'd be very grateful for your answers.

@chris-chris @avolny They only state that there is a single output channel 1x1 convolution for a spatial output. In my interpretation, there may be multiple spatial and non-spatial outputs.
For example, one way to implement it could be to use one output layer (fully connected or convolutional, depending on the argument type) for each of the argument types listed in https://github.com/deepmind/pysc2/blob/master/pysc2/lib/actions.py#L200, as well as for the function identifier. During sampling, one would then first sample the function id from the predicted function identifier distribution, and then sample from the appropriate argument type distribution, depending on the argument types used in the arguments of this function type.
So you would have a single channel 1x1 conv output for each of the types 'screen', screen2', 'minimap', and the non-spatial arguments are similar. When computing the log probs and entropies, you can mask out the arguments which are not used for a specific action sample.
Refering to the paper, 4.2:

In most of our experiments we found it sufficient
to model sub-actions independently, however, we also explored with auto-regressive policies where
the ordering was chosen such that we first decide for the function identifier, then all categorical
arguments and finally, if relevant, pixel coordinates.

The above would correspond to "modelling sub-actions independently".
Also, have a look at this issue for the input feature layers: https://github.com/deepmind/pysc2/issues/116.

@chris-chris @simonmeister
In origin paper, it implemented spatial policy (64x64?) in network architecture. How do we implement this kind of output layer? just flatten as 64x64 fully connected linear layer?

@JIElite in the FullyConv agent, the spatial policy should be taken from the convolutional stream, with a 1-channel convolution as output and no further fully-connected layers.


'Set Selenium Timeout' does not pass timeout variable to selenium

It appears the 'Set Selenium Timeout' keyword does not actually set the timeout.  The code as currently written
    def set_selenium_timeout(self, seconds):
        # . . .
        old_timeout = self.get_selenium_timeout()
        self._timeout_in_secs = robot.utils.timestr_to_secs(seconds)
        return old_timeout

    def set_selenium_implicit_wait(self, seconds):
# . . .

and the code as I suspect it should be
    def set_selenium_timeout(self, seconds):
        # . . .
        old_timeout = self.get_selenium_timeout()
        self._timeout_in_secs = robot.utils.timestr_to_secs(seconds)
        for browser in self._cache.browsers:
            browser.set_script_timeout(self._timeout_in_secs)
        return old_timeout

    def set_selenium_implicit_wait(self, seconds):
# . . .

Also it appears this feature is untested within the rf-s2l acceptance tests.

Ed,  I believe this is one we have discussed already.  Thus I'm assuming this is an old ticket we forgot to close.


Can't access localhost on Windows while running docker on WSL 2

Have you checked that your issue isn't already filed?

[X] I read through FAQ and searched through the past issues, none of which addressed my issue.
[X] Yes, I have checked that this issue isn't already filed.

Bug description
Hi! I've done everything according to the install.md file (meaning, I haven't changed anything in the docker-related files, for example), but still have a problem with the localhost page in my web browser on the Windows side. The docker container itself runs correctly on WSL 2.
How to reproduce the bug

Go to the INSTALL.md file and do everything according to the "Recommended Approach" subsection.
Clone the repository in a WSL directory.
Move forward by entering your directory and then run two docker commands described in the "Local setup using Docker (Recommended)" subsection.
Open your web browser on Windows and try entering the localhost page http://localhost:8080. In my case, there's only the "This page isn’t working" inscription.

Error messages and logs
The error:

The docker ps command output:

The docker logs <container_id> command output:
/usr/local/bundle/gems/citeproc-ruby-1.1.14/lib/citeproc/ruby.rb:1: warning: observer was loaded from the standard library, but will no longer be part of the default gems since Ruby 3.4.0. Add observer to your Gemfile or gemspec. Also contact author of citeproc-ruby-1.1.14 to add observer into its gemspec.
  Logging at level: debug
    Jekyll Version: 4.3.3
Configuration file: /srv/jekyll/_config.yml
  Logging at level: debug
    Jekyll Version: 4.3.3
         Requiring: /srv/jekyll/_plugins/cache-bust.rb
         Requiring: /srv/jekyll/_plugins/details.rb
         Requiring: /srv/jekyll/_plugins/download-3rd-party.rb
         Requiring: /srv/jekyll/_plugins/external-posts.rb
         Requiring: /srv/jekyll/_plugins/file-exists.rb
         Requiring: /srv/jekyll/_plugins/google-scholar-citations.rb
         Requiring: /srv/jekyll/_plugins/hide-custom-bibtex.rb
         Requiring: /srv/jekyll/_plugins/remove-accents.rb
         Requiring: jekyll-archives
         Requiring: jekyll-email-protect
         Requiring: jekyll-feed
         Requiring: jekyll-get-json
         Requiring: jekyll-imagemagick
         Requiring: jekyll-jupyter-notebook
         Requiring: jekyll-link-attributes
         Requiring: jekyll-minifier
         Requiring: jekyll-paginate-v2
         Requiring: jekyll-regex-replace
         Requiring: jekyll/scholar
         Requiring: jekyll-sitemap
         Requiring: jekyll-tabs
         Requiring: jekyll-toc
         Requiring: jekyll-twitter-plugin
         Requiring: jemoji
            Source: /srv/jekyll
       Destination: /srv/jekyll/_site
 Incremental build: disabled. Enable with --incremental
      Generating...

        Generating: Jekyll::Scholar::DetailsGenerator finished in 2.36e-05 seconds.
Fetching external posts from medium.com:
...fetching https://medium.com/@al-folio/displaying-external-posts-on-your-al-folio-blog-b60a1d241a0a?source=rss-17feae71c3c4------2
Fetching external posts from Google Blog:
...fetching https://blog.google/technology/ai/google-gemini-update-flash-ai-assistant-io-2024/
/usr/local/lib/ruby/3.3.0/net/protocol.rb:43:in `ssl_socket_connect': Net::OpenTimeout (Net::OpenTimeout)
        from /usr/local/lib/ruby/3.3.0/net/http.rb:1674:in `connect'
        from /usr/local/lib/ruby/3.3.0/net/http.rb:1580:in `do_start'
        from /usr/local/lib/ruby/3.3.0/net/http.rb:1569:in `start'
        from /usr/local/lib/ruby/3.3.0/net/http.rb:2297:in `request'
        from /usr/local/bundle/gems/httparty-0.22.0/lib/httparty/request.rb:156:in `perform'
        from /usr/local/bundle/gems/httparty-0.22.0/lib/httparty.rb:611:in `perform_request'
        from /usr/local/bundle/gems/httparty-0.22.0/lib/httparty.rb:521:in `get'
        from /usr/local/bundle/gems/httparty-0.22.0/lib/httparty.rb:644:in `get'
        from /srv/jekyll/_plugins/external-posts.rb:80:in `fetch_content_from_url'
        from /srv/jekyll/_plugins/external-posts.rb:62:in `block in fetch_from_urls'
        from /srv/jekyll/_plugins/external-posts.rb:60:in `each'
        from /srv/jekyll/_plugins/external-posts.rb:60:in `fetch_from_urls'
        from /srv/jekyll/_plugins/external-posts.rb:19:in `block in generate'
        from /srv/jekyll/_plugins/external-posts.rb:14:in `each'
        from /srv/jekyll/_plugins/external-posts.rb:14:in `generate'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/site.rb:193:in `block in generate'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/site.rb:191:in `each'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/site.rb:191:in `generate'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/site.rb:79:in `process'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/command.rb:28:in `process_site'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/commands/build.rb:65:in `build'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/commands/build.rb:36:in `process'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/command.rb:91:in `block in process_with_graceful_fail'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/command.rb:91:in `each'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/command.rb:91:in `process_with_graceful_fail'
        from /usr/local/bundle/gems/jekyll-4.3.3/lib/jekyll/commands/serve.rb:86:in `block (2 levels) in init_with_program'
        from /usr/local/bundle/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `block in execute'
        from /usr/local/bundle/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `each'
        from /usr/local/bundle/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `execute'
        from /usr/local/bundle/gems/mercenary-0.4.0/lib/mercenary/program.rb:44:in `go'
        from /usr/local/bundle/gems/mercenary-0.4.0/lib/mercenary.rb:21:in `program'
        from /usr/local/bundle/gems/jekyll-4.3.3/exe/jekyll:15:in `<top (required)>'
        from /usr/local/bundle/bin/jekyll:25:in `load'
        from /usr/local/bundle/bin/jekyll:25:in `<main>'

What operating system are you using?
Windows
Where are you seeing the problem on?
Running locally with Docker
More info
I use Microsoft Windows 10 Pro (10.0.19045 N/A Build 19045) as my main OS and Ubuntu 22.04.4 LTS as the Linux distribution for WSL 2.

FYI you should paste your log between the 3 ticks (`). I believe your error is the same as in #2280.

@george-gca thank you for the reply! I've tried to describe everything related to my problem thoroughly in this comment. In short, I've managed to solve the issue but still haven't found a solution that allows me not to disable the following rows in the _config.yml file:
external_sources:
  # - name: medium.com
  #   rss_url: https://medium.com/@al-folio/feed

I'm afraid I can't help much further. When I used WSL 2, I used a local installation (not docker). My current setup is on Ubuntu.

Are you using docker desktop? If so this might help.

@george-gca thanks for the recommendation! I'll look into it and get back with feedback. No, I've been using Docker Engine + Docker Compose.

Are you using docker desktop? If so this might help.

FYI I was using this technique to launch X applications through VcXsrv long before Windows supported X natively and kept using it as Windows doesn't support the secondary clipboard and has issue with non-US keyboard mappings.
This technique stopped working recently - the IP from resolv.conf pings but it is no longer possible to connect to any port. I haven't found a solution for this yet. One workaround I tried was to add the external IP, i.e. the one Windows uses on the local network, but the connection often breaks killing applications at random times.
I use this only casually on some relative's PC as I personally use Linux only, therefore I can't tell when this broke exactly, but likely in the last 3-6 months (Windows 11, keeping updated but no preview/early versions).

I actually fixed it, rather than taking the ip address form resolv.conf. get the default route:
$wslrt = wsl bash -c "ip -j route list default" | ConvertFrom-Json
$wslserv = $wslrt.gateway
C:\Windows\System32\wsl.exe ~ -- /usr/bin/env "DISPLAY=${wslserv}:0" /usr/bin/rxvt-unicode

The full script is slightly longer, it also gets WSL IP and adds it to allowed xhosts, and it's written as a oneliner...

Can you provide more detailed information? This might help other users (and also users from other projects that might fall here from search).

So the technique described in https://github.com/Microsoft/WSL/issues/1032 (I'm not sure if the comment anchor was on purpose, I was referring to the ticket itself) - using the DNS resolver's IP address to communicate with the host, i.e. this command from Linux:
remoteaddr=$(awk '/nameserver/ {print $2; exit}' /etc/resolv.conf)

In powershell it can be captured with:
$remoteaddr = wsl bash -c "awk '/nameserver/ {print \`$2; exit}' /etc/resolv.conf"

This worked for a while, but not anymore. WSL appears to have changed a bit the networking configuration and while this IP pings, it is no longer possible to use it to access other services.
Using the default gateway, OTOH, appears to work now (documented here although I specify the default route and prefer using json - you may have to install jq in your distro or use the command from the original documentation).
remoteaddr=$(ip -j route list default | jq -r '.[].gateway')

Or from Powershell (no need for jq):
$remoteaddr = (wsl bash -c "ip -j route list default" | ConvertFrom-Json).gateway

Alternatively, another way appears to be getting it directly from the HyperV Interface created for WSL - I can't remember where I found about this but the name has apparently changed from vEthernet (WSL) to vEthernet (WSL (Hyper-V firewall)), so I suspect this may related to the change that broke the original technique.
This should work regardless of the interface name, as long as there is only one marching the wildcard:
$remoteaddr = (Get-NetIPInterface "vEthernet (WSL*)" -AddressFamily IPv4 |Get-NetIPAddress).IPAddress


Unable to install hellosign-embedded from definitely typed

Prerequisites
Please first check README. There are breaking changes from 0.x to 1.0

[x] Did you check the FAQ?
[x] Did you search open and closed issues to see if your issue has already been reported?

Description
I'm able to install all of my typings from Definitely Typed except for one which is hellosign-embedded.
typings install dt~hellosign-embedded --global --save

and
typings install hellosign-embedded --global --source dt --save

both result in:
typings install hellosign-embedded --global --source dt --save
typings ERR! message Unable to find "hellosign-embedded" ("dt") in the registry.
typings ERR! message We could use your help adding these typings to the registry: https://github.com/typings/registry
typings ERR! caused by https://api.typings.org/entries/dt/hellosign-embedded/versions/latest responded with 404, expected it to equal 200

typings ERR! cwd C:\dev\src\Web
typings ERR! system Windows_NT 10.0.14393
typings ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\brian\\AppData\\Roaming\\npm\\node_modules\\typings\\dist\\bin.js" "install" "hellosign-embedded" "--global" "--source" "dt" "--save"
typings ERR! node -v v6.3.1
typings ERR! typings -v 1.3.2

typings ERR! If you need help, you may report this error at:
typings ERR!   <https://github.com/typings/typings/issues>

The other typings I'm able to install just fine are:

bootstrap
jquery
jquery-validation-unobtrusive
jquery.payment
jquery.validation
stripe

Every other typing I've tried to install from DT works as well, it's just this one that's not.

Currently working for me. Let me know if you still have issues 👍

It's working for me now as well. Is there a period of time you need to wait for a new typing in DT to become available through typings?

Usually 5 minutes, but I mentioned in an earlier issue from someone else that it had stalled sometime. I neglected to check when it had stalled though, I'm guessing sometime in the last couple of days.

Reference https://github.com/typings/typings/issues/663#issuecomment-239374222


wip: switch from static to dynamic epochs

Closes #2181

Should be ready for review

This looks good, nice work!


sortable list example

Hi, very good documentation and impressive demo page. good work.
Only problem is the "sortable list" demo, which feels very slow and also has a bug, where any element cannot be dragged to be the last one.
I would like dragging & sorting to feel much much faster, and the way it behaves now is not smooth enough for production-level projects

Hi @yairEO, thank you for the comment.
I couldn't reproduce that bug in my environments (Google Chrome, Firefox, MS Edge, Safari, and others).
All items can be dragged to the bottom of the list.
Could you tell me your environment (name and version of your web browser)?

windows 10, latest Chrome:

Thank you for the information and the GIF.
It seems that only your environment can't drag that.
See:

That works fine in my all environments for test, and that problem has never even once be reported before.
Maybe your Chrome extension, plugin or something avoids that.

I'm on a fresh windows install and also tried it in incognito. still same bug.
Chrome: Version 78.0.3904.108 (Official Build) (64-bit)
on Firefox it works well.
Are you testing on Windows 10 as well?

I tested that on Windows 10 and Google Chrome and recorded it to GIF because you are using those.
And also, I of course tested on Windows 7 and Windows 8.1, that works fine on those.
That is strange. That has never even once be reported before. You can read the source code, it is very short.
I can do nothing because that can't be reproduced, sorry.
Anyway, if you want a library for sorting, you had better use another library because this is not a library for sorting.

No reply came, then this abandoned issue is closed.


Refactor

This is the draft of factoring out a common module so people can see what it looks like and what they think. I started with refactoring common things out of BERT and LLM. The changes were already piling up, so vision folders can happen separately.
Note: It would probably be a good idea to make the repo pip installable so that we don't have to append to the path in order to have a common module.
Verification Todos:

[x] run through the steps in LLM readme (convert c4, verify c4, start training run)
[x] run through the steps in BERT readme (convert c4, verify c4, run the four test commands in the readme)
[x] run LLM tests (pytest)

https://github.com/mosaicml/examples/pull/58 will integrate the triton kernel.
We should move that to the common folder as well.

I'm generally in favor of this kind of refactor, but I think the open question about the overall structure (raised by @abhi-mosaic above) is important. Also, I really don't want us to end up re-building some evolution of YAHP.
My thinking is that we should not shy away from specific projects having lots of custom code when appropriate. This will give us a good chance to showcase examples that rely heavily on common utilities (e.g., BERT and GPT), as well as how to layer in customizations.
Using LegalGPT as an example, I think the decision of where to place that would depend on how much custom code it involves. If the only difference when using LegalGPT is changing the remote address of the data source, for example, then it would probably just make sense to be packaged into the GPT folder (because, in that case, it would very easily re-use the same main.py). However, if LegalGPT requires its own evaluators, metrics, etc. to the point that it would be cleaner to create a separate script for it, then I feel like it would deserve its own folder.

@vchiley Yes, shared, complicated stuff like that should definitely be in common imo
@abhi-mosaic Yes, if it is not pip installable you would have to clone/fork/copy-paste in order to use MosaicGPT and MosaicBERT. My not totally thought through suggestion would be to follow the normal python repo structure, so have a single mosaicml-examples package. And have separate groups for each subfolder (e.g. pip install mosaicml-examples[llm])
@abhi-mosaic / @alextrott16  I also agree it is unclear what to do about the folder structure. The current top level folders indicate model familys (llm), specific architectures (bert, resnet, deeplab), and datasets (cifar). It is difficult to balance the goals of organization and clean code with the goal of each example being self contained/easily copy pasteable. If we make the whole repo pip installable, it would allow legalgpt to from examples.llm.mosaic_gpt import MosaicGPT. I'm not sure I love this structure, but I think this is ok?. The legalgpt example is basically an application of mosaicgpt.
@alextrott16 Agree on not shying away from project having lots of specific code. I think there are basically two categories of stuff that I would like to see in common. 1) obvious shared code (e.g. build_logger). If you can easily use something from another example/common, do it/move it to common. 2) Shared things that are complex (e.g. flash attention/triton stuff). Having complex, error prone code in multiple places is something I would like to avoid. It is also confusing for a user if it is hard to immediately tell if there are differences between (e.g.) the convert_c4 script in bert and the one in llm, and if those differences are salient. If a project needs something custom, it should just do it, and not try to shoehorn it into something more general.

@dblalock I think there has already been drift away from precommit passing...so I only ran pre commit on the files that I modified

@abhi-mosaic @alextrott16 how do you feel about merging this as is, as step one of refactoring? It is going to pretty painful to keep this PR up to date with other changes getting merged in.

@dakinggg if tests are happy, then I'm fine with merging

I'm ok with merging this too. Will keep an eye as I will need to update #61 quite a bit, and I would really like to get that merged today/tomorrow.


support python mlx.array creation from list of mlx.array's

Proposed changes
Adds support for python mlx.array create from list of mlx.array's. Trying to tackle issue 287.
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have updated the necessary documentation (if needed)

Thanks for adding this. I was thinking we should use concatenate and have this be part of the compute graph rather than force an eval.
Concretely when you do mx.array([a1, ...]), it's basically using concatenate under the hood. Does that make sense?

@awni Thanks. I see, yeah that makes sense. I am gonna renovate the code a bit.

Hi @awni ! May I bug you to review again? Thanks! And... Happy New Year 🥳

Hi @Alex-X-W ,
There's two things I think we need to fix with this:

The main point to using stack is to avoid needing to eval the input arrays. I see that you eval them then get the raw pointer based on the type and copy it. That should change to be much simpler. Just put the arrays in a vector, the stack will handle type promotion.
If you are creating an array from a big list (or list of lists) of python scalars it will be really inefficient to convert these to arrays and do a stack. I would change the code so that when there is no nested array it follows a different path and puts the values in vector of the appropriate type and then uses the array constructor.

Thank you @awni for the pointers! They all make sense to me.
Please help take another look, thanks!

PS checked cpu/gpu tests pass + formatting passes


Option to retain directory on pane split

iTerm has a nice little feature where new windows/tabs/splits will use the same directory as their parent.
Personally I like new tabs & windows to start in ~, but I like split panes to start in the same directory as their parent.
Loving Hyper so far!

There's a plugin for that.
https://www.npmjs.com/package/hypercwd
See this list:
https://github.com/bnb/awesome-hyper

omg I went through that list but missed that one. Sorry!


Using Git and GitHub on the Windows platform

Overview
TL;DR
Create course for how to git and github from the Windows platform.
Questions
I'm sure I am going to have questions, but the biggest one I have currently is:

If you were moving from macOS to Windows, what would you want to know about the git experience?
If you were coming from a different version control system, what would you want to know about?

Next Steps

[ ] Flesh out the rest of the outline
[ ] More content about TFS integration
[ ] Identify tell me why content that needs to be reused and potentially re-chunked

Review
No pressing need for 👀  yet. Will call for review in a later comment.

@brianamarie / @crichID / @saraford
So, the content is in a general "rough draft" state. I need some 👀  to review the course and provide some 💭 s on what content I need to include, I am including a checklist / review sheet that should be easy to use when looking at the content.
Additionally, @saraford if there is additional content you need added to the course, please let me know. I am still planning on having this available in a beta form by the time you have your meeting next week.
# Welcome to GitHub for Windows Users
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Get Started
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Install Git
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Configure Git
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# CRLF aka Line Endings
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Store Your Password
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Editors
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# vi Basics
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Configure Other Editors
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Visual Source Integration
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Install the GitHub Extension
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Start a Project
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Make a Commit
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

# Sync?
- [ ] Needs Show Me How
- [ ] Needs Tell Me Why

## Content review

General feedback:

How do we get to this page? Will we add to the navbar/timeline, or for this class will it be URL specific?
Continue buttons aren't working properly :(
I went through this once on Mac looking for formatting things, and once on Windows going through the actual steps. Notes about roadbumps/places we could add clarity are included. 😄
This is SO RAD and it's very impressive that you've hammered out so much content so quickly. :heart: I dream of a future where we can automagically detect the operating system of a user and integrate this content into our existing offerings for all project types. :sparkles: Thank you.

Welcome to GitHub for Windows Users

[ ] Needs Show Me How
[ ] Needs Tell Me Why
Maybe not start both sentences with "although"

Get Started

[ ] Needs Show Me How
[ ] Needs Tell Me Why
Do we need this page to be separate from "Install Git"? What if we had this header and the content for Install Git on this same page?

Install Git

[ ] Needs Show Me How
[ ] Needs Tell Me Why

Configure Git

[x] Needs Show Me How
[ ] Needs Tell Me Why
<EMAIL_ADDRESS>
CRLF aka Line Endings

[x] Needs Show Me How
[x] Needs Tell Me Why
Title including "aka" might lose some with English as a second language

Store Your Password

[X] Needs Show Me How
[ ] Needs Tell Me Why note: There appears to be info here, but the includes may not be set up correctly
This may have happened automagically last time I pushed, because I remember doing something vaguely like that when prompted, and nothing prompted me this time.

Editors

[ ] Needs Show Me How
[ ] Needs Tell Me Why
Same feedback as the "Get Started" page - do we need this extra page? Does it add value?

vi Basics

[ ] Needs Show Me How note: It looks like there's an image supposed to be here, but the includes isn't working properly
[ ] Needs Tell Me Why note: the current tell me why is for configs
I don't know if it's supposed to be "VI" or "vi" or if it doesn't matter, but it's displayed both ways within this page. /shrug

Configure Other Editors

[ ] Needs Show Me How note: It looks like there's an image supposed to be here, but the includes isn't working properly
[ ] Needs Tell Me Why note: the current tell me why is for configs
This looks awesome
This didn't work for me, I had to add atom to my PATH which was a pain and I imagine it's different for different versions of Windows. Is there windows documentation we can point to that we can trust?

Visual Source Integration

[ ] Needs Show Me How
[ ] Needs Tell Me Why
This URL gets me "not found": /on-demand/windows/visual-source

Install the GitHub Extension

[ ] Needs Show Me How note: It looks like there's an image supposed to be here, but the includes isn't working properly
[ ] Needs Tell Me Why
Spelling in first sentence - isn't'tuncommon/isn't uncommon, also that sentence may be a run on sentence, sort of like this sentence.
The instructions there are slick and look great

Start a Project

[ ] Needs Show Me How note: It looks like there's an image supposed to be here, but the includes isn't working properly
[ ] Needs Tell Me Why
:sparkles: :heart:
Sometimes CLICK is in all caps
Do we want to include instructions for how to import GitHub repositories?

Alternate instructions for starting locally:

Open the GitHub integration window: View > Other Windows > GitHub
You may be able to push to GitHub, or you may be told there is no source control. Depending on what you see here, you may need to click Project > Add to Source control
In the GitHub window, push the green button to publish your code "Get Started"
Under "Publish to GitHub", click "Publish"
Make sure the name and description matches your project, and select if you'd like the repository to be private.

Make a Commit

[x] Needs Show Me How
[ ] Needs Tell Me Why
This is awesome!

I think we could get ride of the first section, and only have "Committing all of the changes" and "Committing some of the changes". Also, I think we should consider adding a step about going to "Project" and "changes". I didn't see that for a while.
For committing all of the changes:

In the Team Explorer pane under "Project", click "Changes".
Enter a commit message and make sure the branch listed is the correct branch.

For committing some of the changes:

In the Team Explorer pane under "Project", click "Changes".
Right click the file that you'd like to be committed and select "stage".
Enter a commit message and make sure the branch listed is the correct branch. The commit will only include the changes in the files that are under "Staged Changes".

Sync?

[ ] Needs Show Me How note: It looks like there's an image supposed to be here, but the includes isn't working properly
[x] Needs Tell Me Why
I think you intend for there to be a few bulleted lists, but it isn't coming across
Is there a reason there's a question mark in the title?

Things to add based on convo with Windows peeps:

Example of branching, show on whiteboard, GH desktop, then within VS
Why git
setup walkthrough
git shell demos (show that it exists, and you don't have to login)
merge conflicts
demo beyondcompare / merge conflict resolution tools

@brianamarie I know this is kinda rushed, but can you check this Monday morning and if everything seems decent enough to publish as a hard beta click the Big Green and merge services-web so we can let @saraford point to it during her talk tomorrow?

cc: @crichID I think @brianamarie might be ✈ Monday morning ... so she might not be able to review this prior to tomorrow's class. Your 👀 would be super appreciated.

@beardofedu 👍 from me. I made some commits, most of them have to do with the navigation and the 'continue' buttons, and one has to do with rewording. I think that we can still fine tune the instructions for adding a commit in VS, but it might be best to see what feedback @saraford gets from the current state with real life users.
So many ✨, @beardofedu!!!

Merging this. Course is in a beta format and only available from the drop-down menu.


Support for .NET 6.0 Preview 1 for Client

Hello! I tried upgrading my Blazor WASM Client to .NET 6.0 Preview 1, and GitHub actions failed with the following error message:

Error: Platform 'dotnet' version '6.0' is unsupported.

When will .NET 6.0 Preview 1 be supported for Client? I read about some performance gains and would love to try it out.
Thank you!

Thanks @Zhiyuan-Amos for reporting this, we've added it into Oryx backlog. Will work on supporting it.

@qianz2 I noticed new releases on the oryx branch. Any idea when will they be available over here? :)

Yes, we had a new release, pinging @anthonychu here for questions of the schedule.

@qianz2 Can you please start an internal thread with @miwebst, if you haven't already, with the new version that we should be upgrading to? Thanks.

Hello @qianz2 , any updates? Thanks! 😃

We should have an update to the GitHub Action coming out next week that should include this.

This should be working now. Please give it a try and reopen this issue if you run into any issues.


Update gitignore for iOS

Thanks, I took the liberty to remove the .DS_Store and .idea/ since they are related to the developer workspace and should be put in their global gitignore configuration.


Guidance on improving WebSocketSubject multiplex method

RxJS version: 5.4.2
Code to reproduce: No bug to reproduce, just a question.
Expected behavior:
Actual behavior:
Additional information:
On Line 122 of WebSocketSubject it says   "// TODO: factor this out to be a proper Operator/Subscriber implementation and eliminate closures"
Can one of the maintainers please explain a little more clearly what's the suggested improvement and the benefits it would bring? I'd love to contribute, but am a little confused what we're looking for.
I've noticed @benlesh authored https://github.com/benlesh/RxSocketSubject/blob/master/test/RxSocketSubject/multiplex.js awhile back for RxJS 4, are we looking to take a similar approach here as well?
Thanks

Honestly, at this point we can probably remove that TODO. I initially added that comment because there was a huge performance difference in Chrome/V8 between code that used a lot of closures and code that did not. Since then the gap there has tightened enough to where I'm less worried about it.

Thanks for responding @benlesh. If someone were to further build out this WebSocketSubject  multiplex functionality with more features like the ability for the consumer to send or configurable auto-retry (Likely not belonging in the core rxjs repo, but I’m open to anything), can you opine what your ideal approach would be?
Maybe I’d create an operator that returns a multicasted subject? (Multicasted so that multiple downstream subscribers don’t trigger repeated subscription msgs as the current one does)


Issue in landscape mode

I have a following issue when my app goes to landscape mode. Do you have any suggestions?

I would like alert to be a fixed width instead of going almost entire width of the screen.
Thanks

Hi @vitormesquita Im sorry for the late reply.
It could, as long as the margin applies to the entire malert view.


Resume after pause

I have issue with resume sound after pausing it.

I want to make play pause resume sound.

:beetle: The code:

//SoundManager util that i made.

import Sound from "react-native-sound";

const sounds = {
  "crow_sound": {
    sound: "people-talking-2.mp3",
    path: "https://www.pacdv.com/sounds/people_sound_effects"
  },
  "sally_sound": {
    sound: require("./../../assets/sounds/sallysound.mp3")
  },
};
class SoundManager {

  playUserReview(name) {
    try {
      var sound = sounds[name].sound;
      var path = sounds[name].path;
      if (path) {
        this.userReview = new Sound(sound, path, error => {
          if (error) {
            return;
          }

          this.userReview.play(success => {
            if (!success) {
              this.userReview.reset();
            }
          });
        });
      } else {
        this.userReview = new Sound(sound, error => {
          if (error) {
            return;
          }

          this.userReview.play(success => {
            if (!success) {
              this.userReview.reset();
            }
          });
        });
      }
    } catch (error) {
      console.log(error);
    }
  }

  pauseUsersReviews() {
    try {
          this.userReview.pause(success => {
            if (!success) {
              this.userReview.reset();
            }
          });
    } catch (error) {
      console.log(error);
    }
  }

  resumeUserReview() {
    this.userReview.play((success) => {
      success ? console.warn("success") : console.warn("NotSuccess")
    });
  }
}

module.exports = new SoundManager();

-----------------------

//and this is the function that i used onpress play/pause button.

userNameForPlayingRelatedSound = (username) => {
        if(this.state.soundStatus == "notPlayed") {
            SoundManager.playUserReview(username);
            this.setState({soundStatus: "played"});
            console.warn("played");
        }
        else if(this.state.soundStatus == "played") {
            SoundManager.pauseUsersReviews();
            this.setState({soundStatus: "paused"})
            console.warn("paused");
        }
        else if(this.state.soundStatus == "paused") {
            SoundManager.resumeUserReview();
            this.setState({soundStatus: "played"})
            console.warn("resume");
        }

    }

:bulb: Possible solution

Is your issue with...

[ ] iOS
[x] Android
[ ] Windows

Are you using...

[x] React Native CLI (e.g. react-native run-android)
[ ] Expo
[ ] Other: (please specify)

Which versions are you using?

React Native Sound: last version
React Native: ^0.59.10
iOS:
Android: 9

Does the problem occur on...

[ ] Simulator
[x] Device

If your problem is happening on a device, which device?

Device: Pocophone F1

Hi! Could you fill out the issue template, please?

Hi! Could you fill out the issue template, please?

Done, sorry for that.

Can you explain what is happening? If you remove the try blocks are any errors thrown?

Can you explain what is happening? If you remove the try blocks are any errors thrown?

when i click to play sound for first time it played will.
then i click to pause it paused but when i click again to resume sound, the sound didn't resume it just no sound played.
there is no any error thrown.

What is the solution?

I used a workaround for this issue.
When pausing get the current time and set it to state like
// pause sound             whoosh.getCurrentTime((seconds) =>              this.setcuruntPlayTime(seconds)             );             whoosh.pause();
setcuruntPlayTime(time) {         this.setState({ curuntPlayTime:time  }, () => { }) }
then seek the audio object when resuming, like..
async resume (){         if(!_.isEmpty(whoosh)){ // if whoosh empty, app get crashed           whoosh.setCurrentTime(this.state.curuntPlayTime);           whoosh.play();         }     }

Still not working @asiriPiyajanaka  I used whoosh.setCurrentTime(this.state.curuntPlayTime); whoosh.play() then after success callback of play came.

any solution here? thanks

What worked for me is,
SoundPlayer.playUrl(url);
SoundPlayer.seek(time);


Deprecated imperative apply of Flutter's Gradle plugins

I am trying to migrate away from imperative applies of gadle plugins, as it is now depreacted. However I don't know gradle or android environment so much, I am not sure how to proceed.
So far what I've tried, after migrated to declarative plugins:

adding the plugin declaration inside settings.gradle, in the plugins block :

plugins {
    id "dev.flutter.flutter-plugin-loader" version "1.0.0"
    id "com.android.application" version "7.3.1" apply false
    id "org.jetbrains.kotlin.android" version "1.8.22" apply false
+    id 'com.google.gms.google-services' version '4.4.0' apply false
}

adding the plugin and the classpath in app/build.gradle:

plugins {
   // ...
+    id 'com.google.gms.google-services'
}

// ...

dependencies {
   // ...
+    classpath 'com.google.gms:google-services:4.4.0'
}

However this is failing,
> Could not find method classpath() for arguments [com.google.gms:google-services:4.4.0] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

What would be the correct way of migrating away from imperative applies ?
Thanks a lot :slightly_smiling_face:

Thanks for the link, it helps indeed!
Maybe it's worth updating the documentation here: https://www.customer.io/docs/sdk/flutter/getting-started/#install, so that new customer find the solution directly?


VM configured for FULL backup on Sunstone cant be changed to INCREMENTAL backup

Description
Once configured the backups for a given VM, and specified to FULL backup, this configuration cannot be reverted either via Sunstone or CLI (onevm updateconf VM_ID). It allways get back to first configuration (on this case, the Full backup).
To Reproduce
Configure FULL backup for a given VM via Suntone and after apply configuration, try to reconfigure it and select this time the INCREMENTAL backups.
Expected behavior
The VM backup type should have changed to the new one.
Details

Affected Component: [Sunstone, Storage]
Hypervisor: [KVM]
Version: [6.6.3]

Hello team
May we follow-up on this?
Thanks in advance

Yes it should be reverted. Note that some VMs cannot use incremental, for example VMs with disk snapshots.
Maybe we should add a warning related to this

I will left this issue to log the error, i.e. VM cannot do INCREMENTAL

I will left this issue to log the error, i.e. VM cannot do INCREMENTAL

Thanks Mr. Montero
I will create new VM and will configure it first for FULL backup, and will try to revert it to INCREMENTAL backup. Will let you know the results, to conclude if previous werent able to revert backup type due to disk snapshot.

@rsmontero The config was successfully reverted.
On this specific VM 170 we dont know whats happening, you cant change the backup type or the checkbox for save the volatile disks from backup configuration, as it will reverted after update the config.
Show VM info:
onevm show 170

This is the VM info:
VIRTUAL MACHINE 170 INFORMATION
ID                  : 170
NAME                : WMS UAT DB
USER                : client1
GROUP               : client1
STATE               : ACTIVE
LCM_STATE           : RUNNING
LOCK                : None
RESCHED             : No
HOST                : DEV-N2-kvm
CLUSTER ID          : 100
CLUSTER             : KVM
START TIME          : 11/10 17:04:33
END TIME            : -
DEPLOY ID           : 4a86031e-29ae-4284-b3ca-2ba939f9783a

VIRTUAL MACHINE MONITORING
CPU                 : 211.32
MEMORY              : 64.1G
NETTX               : 11.9G
NETRX               : 14.9G
DISKRDBYTES         :<PHONE_NUMBER>616
DISKRDIOPS          : 38485267
DISKWRBYTES         :<PHONE_NUMBER>248
DISKWRIOPS          : 156115311
ID                  : 170
TIMESTAMP           :<PHONE_NUMBER>

PERMISSIONS
OWNER               : um-
GROUP               : ---
OTHER               : ---

VM DISKS
 ID DATASTORE  TARGET IMAGE                               SIZE      TYPE SAVE
  2 images     vdb    VM-TEST-DATAWM S                    855.6G/10 file   NO
  3 images     vda    VM-TEST-SYSTEM                      66.1G/120 file   NO
  1 -          hda    CONTEXT                             1M/-      -

VM NICS
 ID NETWORK              BRIDGE       IP              MAC               PCI_ID
  0 client network  1        onebr6       X.Y.Z.12   02:00:ac:10:64:0c

SECURITY

NIC_ID NETWORK                   SECURITY_GROUPS
     0 snr network  1             0

SECURITY GROUP   TYPE     PROTOCOL NETWORK                       RANGE
  ID NAME                          VNET START             SIZE
   0 default     OUTBOUND ALL
   0 default     INBOUND  ALL

VIRTUAL MACHINE HISTORY
SEQ UID  REQ   HOST         ACTION       DS           START        TIME     PROLOG
  0 12   3392  DEV-N2-k nic-attach    0  11/10 17:04:34   0d 00h01m   0h00m02s
  1 12   1840  DEV-N2-k disk-attac    0  11/10 17:05:45   0d 00h09m   0h00m00s
  2 12   1792  DEV-N2-k nic-detach    0  11/10 17:14:47   0d 01h48m   0h00m00s
  3 12   176   DEV-N2-k nic-attach    0  11/10 19:02:49   0d 00h01m   0h00m00s
  4 12   1536  DEV-N2-k poweroff      0  11/10 19:04:46   0d 22h36m   0h00m00s
  5 12   8640  DEV-N2-k disk-detac    0  11/11 17:40:47   1d 02h16m   0h00m00s
  6 12   880   DEV-N2-k disk-attac    0  11/12 19:56:57   0d 00h00m   0h00m00s
  7 -    -     DEV-N2-k poweroff-h    0  11/12 19:57:24   0d 00h05m   0h00m00s
  8 12   8704  DEV-N2-k poweroff-h    0  11/12 20:02:44   0d 01h02m   0h00m00s
  9 12   3760  DEV-N2-k poweroff-h    0  11/12 21:05:32   0d 00h11m   0h00m00s
 10 -    -     DEV-N2-k monitor       0  11/12 21:17:24   0d 20h37m   0h00m00s
 11 12   2320  DEV-N2-k backup        0  11/13 17:54:50   0d 09h04m   0h00m00s
 12 12   1920  DEV-N2-k backup        0  11/14 02:59:09   0d 00h04m   0h00m00s
 13 0    6640  DEV-N2-k backup        0  11/14 03:03:59   3d 09h56m   0h00m00s
 14 -    -     DEV-N2-k none          0  11/17 13:00:03   3d 20h29m   0h00m00s

SCHEDULED ACTIONS
   ID ACTION  ARGS   SCHEDULED     REPEAT   END STATUS
    0 backup   100 11/22 06:00 Weekly 3,6  None Next in 20.51 hours

BACKUP CONFIGURATION
BACKUP_VOLATILE="YES"
FS_FREEZE="AGENT"
INCREMENTAL_BACKUP_ID="-1"
KEEP_LAST="7"
LAST_INCREMENT_ID="-1"
MODE="FULL"

VM BACKUPS
IMAGE IDS: 77,97

USER TEMPLATE
HOT_RESIZE=[
  CPU_HOT_ADD_ENABLED="NO",
  MEMORY_HOT_ADD_ENABLED="NO" ]
LOGO="images/logos/windows8.png"
MEMORY_UNIT_COST="MB"

VIRTUAL MACHINE TEMPLATE
AUTOMATIC_DS_REQUIREMENTS="(\"CLUSTERS/ID\" @> 0 | \"CLUSTERS/ID\" @> 100)"
AUTOMATIC_NIC_REQUIREMENTS="(\"CLUSTERS/ID\" @> 0 | \"CLUSTERS/ID\" @> 100)"
AUTOMATIC_REQUIREMENTS="(CLUSTER_ID = 0 | CLUSTER_ID = 100) & !(PUBLIC_CLOUD = YES) & !(PIN_POLICY = PINNED)"
CONTEXT=[
  DISK_ID="1",
  ETH0_DNS="<IP_ADDRESS>",
  ETH0_EXTERNAL="",
  ETH0_GATEWAY="X.Y.Z.111",
  ETH0_IP="X.Y.Z.12",
  ETH0_IP6="",
  ETH0_IP6_GATEWAY="",
  ETH0_IP6_METHOD="",
  ETH0_IP6_METRIC="",
  ETH0_IP6_PREFIX_LENGTH="",
  ETH0_IP6_ULA="",
  ETH0_MAC="02:00:ac:10:64:0c",
  ETH0_MASK="<IP_ADDRESS>",
  ETH0_METHOD="",
  ETH0_METRIC="",
  ETH0_MTU="",
  ETH0_NETWORK="X.Y.Z.0",
  ETH0_SEARCH_DOMAIN="",
  ETH0_VLAN_ID="2718",
  ETH0_VROUTER_IP="",
  ETH0_VROUTER_IP6="",
  ETH0_VROUTER_MANAGEMENT="",
  NETWORK="YES",
  SSH_PUBLIC_KEY="",
  TARGET="hda" ]
CPU="2"
GRAPHICS=[
  LISTEN="<IP_ADDRESS>",
  PORT="6070",
  TYPE="VNC" ]
MEMORY="65536"
MEMORY_RESIZE_MODE="BALLOONING"
OS=[
  UUID="4a86031e-29ae-4284-b3ca-2ba939f9783a" ]
TEMPLATE_ID="39"
TM_MAD_SYSTEM="shared"
VCPU="8"
VMID="170"

Do you see anything strange here Sir? The VM was created from a qcow2 image imported to the OpenNebula. It has a data disk and the OS disk. It was first configured for FULL backup and now cant be reverted to INCREMENTAL backup.

Just in case the keyword is MODE="INCREMENT" not INCREMENTAL...

Yes, sorry for that. I have tested Updating the config from the cli,
specifying that key word too (INCREMENT). The VM config just return to FULL
and the VM doesnt have any snapshot on its disks.
El mié, 22 nov 2023 9:53 a. m., Ruben S. Montero @.***>
escribió:

Just in case the keyword is MODE="INCREMENT" not INCREMENTAL...
—
Reply to this email directly, view it on GitHub
https://github.com/OpenNebula/one/issues/6400#issuecomment-1822921443,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANRJPNYF2NDOO7D2M7JLNH3YFYGXHAVCNFSM6AAAAAA7LXRGXCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRSHEZDCNBUGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Also all disks needs to be qcow2

For the reference:
Check on disks for increment is done here:

https://github.com/OpenNebula/one/blob/master/src/vm/VirtualMachine.cc#L3210 (check for snapshots and volatile)
https://github.com/OpenNebula/one/blob/master/src/vm/VirtualMachineDisk.cc#L1547 (check all disks are qcow2)

Also all disks needs to be qcow2

Yes Sir, all the VM disks are qcow2 disks and none of them have snapshot disks. The problem was resolved recreating the VM from the begining.
It would be nice the warning for trying to configure the INCREMENT backup on a VM that doesnt have qcow2 disks or have snapshots disks :).
Thanks in advance Mr. Montero.

Hi @rsmontero , we find a problem in our cluster, We didn't have the FORMAT property in the disk template.
That's why we can't use the incremental backups in old vms imported from the 5.12 version.
Do you have any suggestion?
Can I suggest to use the DRIVER property instead of FORMAT.
https://github.com/OpenNebula/one/blob/57efabf2d59051788be3aec67bcd8bce36d95566/src/vm/VirtualMachineDisk.cc#L1553-L1577
This is our VM template (part of):
...
CPU = "1"
DISK = [
  ALLOW_ORPHANS = "NO",
  CLONE = "YES",
  CLONE_TARGET = "SYSTEM",
  CLUSTER_ID = "0,100",
  DATASTORE = "images",
  DATASTORE_ID = "1",
  DEV_PREFIX = "vd",
  DISK_ID = "0",
  DISK_SNAPSHOT_TOTAL_SIZE = "0",
  DISK_TYPE = "FILE",
  DRIVER = "qcow2",
  IMAGE = "windows2019-Sep2020",
  IMAGE_ID = "2",
  IMAGE_STATE = "2",
  LN_TARGET = "NONE",
  ORIGINAL_SIZE = "51200",
  READONLY = "NO",
  SAVE = "NO",
  SIZE = "102400",
  SOURCE = "/var/lib/one//datastores/1/bb52083cfd52218c3957ce830b1aaf43",
  TARGET = "vda",
  TM_MAD = "shared",
  TYPE = "FILE" ]
FEATURES = [
...


Enable parallel jobs on the master node

The jobs tied to the master node do not require running serially.
They are not long running or resource intensive, but there are a bunch of them and allowing them to run in parallel will allow the farm to be more responsive.
Especially wrt status page updates (https://github.com/ros-infrastructure/ros_buildfarm/pull/327) and the rosdistro cache (right now the 3 have to run in series)
This also helps stave off the approaching starvation on this machine. The rosdistro caches durations are over 30 seconds and sometime approach 1 minute, and running every 5 minutes means they are using almost half the time available. Running in parallel, there is not a risk of them approaching full dutycycle since they can run in parallel.
I've manually updated the buildfarm and all jobs appear to be operating normally.

-1 to run these jobs in parallel. By that they increase the load on the master and even if run in parallel they will still not any faster.
Instead we should investigate why the rosdistro cache jobs suddenly take so much longer and fix that: http://build.ros.org/job/kinetic_rosdistro-cache/buildTimeTrend

Many jobs have failed over night due to the new concurrency. Please revert all changes you have made to the build farm and restore the original configuration.

The caches were taking longer due to running in parallel. The status pages had similar behavior.

This chart shows the original single track, 4 parallel, then 2 parallel.
I've rolled the master back to one executor, but the load was really not noticeable.(see below) At the left you can see the reconfigure finishing. The cache and status pages running in parallel every 5 and 15 minutes respectively do not show up significantly.


Update handling of JSXText to trim and filter empty text

The recent support for the // @jsx Foo pragma exposed that Flow wasn't properly desugaring JSX.  JSXText children are preprocessed and filtered before being passed to React.createElement or the custom function. Using babel as reference, this is what should happen

Each line of the JSXText is trimmed of whitespace
Empty lines are removed
Tabs are replaces with spaces
The lines are joined together with spaces.

Furthermore, empty JSXText children are thrown away and are not passed to the function. So
<Foo> {true} </Foo>

is correctly parsed as having 3 children (" ", true, and " "), but should desugar to React.createElement(Foo, null, true);

FYI, this would help with our js pretty printer for jsx nodes :)

@vjeux - would it? This isn't a change to the parser.

Oh looks like it's standard for parsers to leave the white space in JSXText nodes (my bad i didn't look at the changes). cc @vjeux i think we will just need to copy this logic out unless we can share it with flow somehow.


OpenBox/AutoStart Config: Syntax Error

Today (or maybe starting a few days ago, I haven't accessed N+ in about a week) I tried to open the interface and just got a black screen.
In Docker logs I am seeing:
2024-11-06 15:37:47 /config/.config/openbox/autostart: 1: Syntax error: "(" unexpected

try pulling the latest image again. It gets rebuilt every night from the nicotine plus official release so if something broke on their side, we would see it on this docker release. Mine is working fine.

Sounds like this was fixed. Please re-open if it is not and I'll try to get around to it at some point


a trivial bug

Thanks for your great work, I am using your code to train a multilingual model, but you seem to have forgotten the initialization of self.use_mel_spec_posterior in TextAudioSpeakerLoader

Hi, good catch! Please check the latest patch and report back, thanks!


elisp-packages/manual-packages.nix: use self-scoped callPackage

Description of changes
Hey @AndersonTorres! I was just about to submit a PR for lsp-bridge and I noticed that #219698 breaks the manual emacs packages. It looks like we should be using the self-scoped callPackage for them instead.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 219982 run on x86_64-linux 1

1 package marked as broken and skipped:

emacsPackages.perl-completion

43 packages built:

emacsPackages.agda-input
emacsPackages.agda2-mode
emacsPackages.color-theme-solarized
emacsPackages.control-lock
emacsPackages.ebuild-mode
emacsPackages.ghc-mod (emacsPackages.elisp-ffi)
emacsPackages.ement
emacsPackages.ess-R-object-popup
emacsPackages.evil-markdown
emacsPackages.evil-textobj-tree-sitter
emacsPackages.evil-tree-edit
emacsPackages.font-lock-plus
emacsPackages.git-undo
emacsPackages.haskell-unicode-input-method
emacsPackages.header-file-mode
emacsPackages.helm-tree-sitter
emacsPackages.helm-words
emacsPackages.idris2-mode
emacsPackages.isearch-plus
emacsPackages.isearch-prop
emacsPackages.jam-mode
emacsPackages.llvm-mode
emacsPackages.matrix-client
emacsPackages.ott-mode
emacsPackages.pod-mode
emacsPackages.power-mode
emacsPackages.prisma-mode
emacsPackages.pygn-mode
emacsPackages.rect-mark
emacsPackages.sunrise-commander
emacsPackages.sv-kalender
emacsPackages.symex
emacsPackages.tree-edit
emacsPackages.tree-sitter
emacsPackages.tree-sitter-ess-r
emacsPackages.tree-sitter-indent
emacsPackages.tree-sitter-ispell
emacsPackages.tree-sitter-langs
emacsPackages.tsc
emacsPackages.urweb-mode
emacsPackages.voicemacs
emacsPackages.yes-no
emacsPackages.youtube-dl

Waiting ofborg.


migrate to go module

Might relate to https://github.com/golang/go/issues/27455 and https://github.com/golang/go/issues/30185

I think merge https://github.com/lyft/flytestdlib/pull/50 and release a new pflags will get this fixed.

@@EngHabu Could you please take a look at this as well? Thanks.

I compiled pflags and verified locally.

https://github.com/golang/go/issues/30515

https://github.com/golang/go/issues/33008

Don't understand why it adds all those h1 lines to go.sum. :(


More smart autocomplete like vim

VSCode Version: 1.23.1
OS Version: macOS High Sierra 10.13.4

Vim Records:

VSCode Records:

When I input ship_name and use down or up key to toggle Suggestions, it can fill what I selected in vim, but I need to press enter  in VSCode. How to set config for that ?

I don't really understand what you are doing in VIM. Can you please provide detailed steps that explain the desired behaviour?

Thanks for your reply.
In vim , if I input ship_name, there are two suggestions in the list ,at that time ,if I want to use ship_name_list, I just enter DOWN  to select the ship_name_list,then it can autofill ship_name_list and do not need to enter TAB or ENTER .
But in VSCode, if I want to input what the VSCode suggestioned I need to press ENTER or TAB .
Sorry , my English is not well .Can you understand what I mean?

In vim , if I input ship_name, there are two suggestions in the list ,at that time

Is there a special way in which you trigger suggestions or does it always fill-in the currently selected item?

I installed plugin named YouCompleteMe in Vim .

/duplicate of #32402

Well done. Thanks for your help.


[BUG] Images in full screen are cropped vertically if taller than device height

Describe the bug
Images taller than the device aspect ratio are cropped vertically.
To Reproduce
Steps to reproduce the behavior:

Have a device with a 4:3 aspect ratio
Open any image taller than that
Be unable to see the whole image

Screenshots
Here you can see the text toward the bottom of this image:

Here you cannot:

Device
Microsoft Surface Duo 2, Android 11, any version of Fritter up to and including 3.0.0-beta1.

This is a duplicate of #390, but is fixed in v3.0.0, and is available in the latest CI build 🙂

Whoops! Sorry, I tried searching existing issues and came up short.


DNS unit test to cover #1096 behavior.

Not entirely satisfying, since we don't have a mock c-ares
implementation, so we still perform a real DNS query, but I've validated
this has the intended behavior (if I comment out the fix it fails).
Fixes #1101.

I'm going to call this a fix for #1101. I think the ideal long term thing to do is add a mockable interface for c-ares, but I don't think this is low hanging fruit.


[GSSOC'23] Main.py is not working

Prerequisites

[X] I am an Open Source Contributor
[x] I checked to make sure that this issue has not already been filed

Description
Error
The main.py has so many errors in it like a basic machine learning model can't predict float values but here y_train and y_test are float which is one of the major errors and the second one which I've seen so far is the problem when fitting the data in linear regression. The third error is in the library itself which is in line 7 plot_confusion_matrix is no more available. The fourth error is in the path or train and test dataset, the way in which they are assigned is not valid.
Solution
I've fixed most of them only 1 remaining
Screenshots

I can also fix this very easily!
So please assign me this issue I'll make all these errors vanish!
Code of Conduct

[X] I follow Contributing Guidelines of this project.

the errors are still there @Rakesh9100 .
1st error is of path

2nd  error is of datatype error

I can see 2nd one is not an error but a warning
Still assigning you @Shiv-Expert2503

@Rakesh9100 I'm done with the code now it's working properly fine!
But I also want to implement Random Forest classifier to increase accuracy. Please allow it also!!

@Shiv-Expert2503 Are you working on this issue or should I close this? If yes pull the latest changes and then work on.

Closing this for having no response.


PBL-37366 PBL-37367 Support for libs and package.json

This PR adds support for libraries.
New features:

Import/export package.json projects
Build projects which have dependencies
Specify project dependencies and search npm on a new page
Autocomplete support for library functions, appkeys and resources

Code changes:

Logic for auto-generated headers moved to ycm-proxy
Refactoring so that gist/git/archive loading share more functionality
Added a generic table editor KVtable, used by the existing appkey interface and the new dependency interface
Added a very large number of tests

Change highlights:

The env var flag NPM_MANIFEST_SUPPORT is used gate off certain functionality
api/project.py gets a new API endpoint save_project_dependencies()
api/ycm.py now sends appkeys/resource info directly to YCM on startup (instead of generating headers)
The database migration adds the Dependency model and also a keywords field for projects
In dependency.py The Dependency model essentially just links name->version, and validates versions to disallow local paths.
In models/project.py The Project model gets methods for getting/setting dependencies and keywords
The Project model also gets methods for getting/setting app_version_label as a semver
KVTable.js is the new table editor (build in good-ol reliable jQuery)
dependencies.js is the dependency UI page with dependencies.html

A large chunk of its code (setup_npm_search_form) is dedicating to wrangling textext to behave how nicely for searching NPM
It uses ModuleCache to keep the version numbers its found in memory in order to cut down on requests to node-modules.come
The code for the search box options form still exists but the form is hidden. (I personally feel like it's worth keeping around to experiment with a little more)

live_settings_form.js has been heavily refactored to support some new callbacks.
settings.js now uses the KV table in place of its own appkey editor, and the appkey editor can switch between old/new style
ycm.js now has functions for sending the appkeys/resource IDs/dependencies to YCM directly.
In utils/project.py, find_project_root_and_manifest() is reworked so that it can check the contents of a package.json file to see whether it has a pebble object
sdk.py is updated to be able to create package.jsons and gets a new function load_manifest_dict() which is soley responsible for reading manifest files.
archive.py uses the new find_project_root_and_manifest() functionality to find the manifest file, and a chunk of its logic is replaced with `load_manifest_dict()
build.py adds logic for building projects which have dependencies
gist.py now also uses load_manifest_dict() to make sure that project loading logic is shared across ClodPebble
git.py uses find_project_root_and_manifest() so that it finds projects in the same way as do_import_archive()
Added version.py, a small library which converts between semvers and Pebble versions.
views/project.py now sends the version validation regex to the client, so it doesn't have to be copy/pasted across CloudPebble. (we could use this kind of thing for a few common regexes)
In fakes.py, FakeS3 regains the ability to read/write from the filesystem

Woo!


Prepare Crust for end to end tests via Routing

Once we're done with the 0.32.0 changes, we will test Crust for  regressions.

Integtrate Crust into Routing
notify QA team to execute the end-to-end tests

Note that end to end tests are on hold until the droplet deployer tool is working


"Clinceni", "Olteni" and "Ordoreanu" are listed in the drop-down menu

Description:
"Clinceni", "Olteni" and "Ordoreanu" are 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 "Clinceni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Clinceni, ILFOV (Clinceni)" and "Comuna Clinceni, ILFOV".

Step 3
Type "Olteni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Olteni, ILFOV (Clinceni)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Ordoreanu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Ordoreanu, ILFOV (Clinceni)".


Update heroku buildpack multi link

This is now maintained by heroku and David Dollars version is deprecated and will be switched off on 1st Jan 2017

This is good.  Heroku now supports multiple buildpacks out of the box.  I think this could be updated even more.


revise how backers avatars are shown

Avatars from individuals making large contributions are lost in the ui. They are smaller than companies that gave much less.  We need to find a way of showing this better. We are only incentivizing corporate sponsors like this.
cc / @cuiki @xdamman
Thank you for taking the time to report an issue 🙏
The easier it is for us to reproduce it, the faster we can solve it.
So please try to be as complete as possible when filing your issue.

URL:
Logged in as: (just provide your username or the url of your profile)
Error message if any:
Expected result:
Browser:
Bonus point if you can add a screenshot :-)
Thank you and have a wonderful day/evening!
(and sorry for inconvenience! We'll do our best to solve this ASAP)

closing for now. let's revise on a future design version of the collective's page


Moved: Vivaldi.Vivaldi version 3.8.2259.42

This pull request is automatically generated by 🥟 Dumplings
Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run

@wingetbot run