[News] I am leaving SM64JS (Until August)

Hello SM64JS players, I have some bad news. I am leaving the SM64JS community. I know this is going to be very hard on some players but don't worry, I will return in August. The reason I am leaving is because I have a major test tomorrow and I need to turn in my chromebook the same day and the reason I will return in August is because I will get a new chromebook for school but until then, You won't see me online on Discord and SM64JS. Thanks, everyone for being my friends and accepting me as one of you and to the Mods, keep updating the game as you have been and when I return, surprise me with a huge update.
Sincerely, Drok27

Cya, normally I would just close the issue but I suppose I'll keep it open for a day or two for others to see

I am back!

Huh, neat.


Log spam for consumers without READ authorization

Affects Version(s): 2.3.5 and earlier

🐞 Bug report
Together with @anderseknert and @mdanielolsson we were playing around with Kafka authorization policies when we stumbled upon this bug.
If consumer is allowed to DESCRIBE, CREATE, WRITE but not READ, KafkaConsumer::poll will throw TopicAuthorizationException.  KafkaMessageContainerListener will just log it and continue polling, which causes a massive amount of logs to be produced.
Our solution is to pause the consumer for some time and then try again (see PR).
Use those repos to easily reproduce:

https://github.com/mdanielolsson/dockerized-kafka
https://github.com/lukasz-kaminski/spring-kafka-auth-tester

This is already fixed on master: https://github.com/spring-projects/spring-kafka/pull/1332

I've forked repo 4 days ago, so I missed this one. Closing the ticket then.

Having spent some time on the broker side of things implementing custom authorization models for Kafka, we experienced two show stoppers in our clients:

Logs getting flooded by unauthorized clients (~200 million log messages from a single one in a few hours!).
Clients unable to ever recover if unauthorized even once.

The fix recently merged seems to fix the first problem (which of course is great), but not the second one. I'll admit that my general Kafka-knowledge is limited so I'm not sure how errors are generally handled client side, but outside of the fairly static ACL model, authorization decisions could be based on any number of inputs and could vary over time without one unauthorized decision being too exceptional in nature, and certainly not fatal.
While the description to this PR also seems to focus on the log flooding, the fix addresses also the second problem. Should we create a new issue/PR for that, or what would you advise us to do @garyrussell ?

@anderseknert I would entertain another PR if you wish to submit one.
I would name the property authExeptionRetryInterval and, if null (default), treat the condition as fatal and, if not null, add the sleep.
It should apply to all authorization exceptions not just TopicAuthorizationException.
The java docs should warn that it must be less than the max.poll.interval.ms consumer property.
We would also need an update to the what's new and kafka.adoc to mention the new property.
Thanks in advance for your contribution!

Sounds like a good way forward, thanks @garyrussell!
@lukasz-kaminski let's sync on who does what on this and then prepare a new issue and PR with the above points in mind. You can close this one again as the log flooding is indeed solved already.

Thanks @anderseknert and @garyrussell. I'm preparing the PR.

Sounds great @lukasz-kaminski 👍 I created https://github.com/spring-projects/spring-kafka/issues/1336 for that feature so we can keep track of it separately from the logging issue (which is solved already).


Quick Insert compatibility

Quick Insert includes a button on the original Tidy 5e sheet table footer:

On click, it opens Quick Insert:

Note that it has awareness of the item section / type:

Make the necessary changes so that buttons can be added to the table footer via the API, and work with the Quick Insert module dev to see if compatibility can be restored.

Existing Tidy 5e support can be found in the Quick Insert module here: https://gitlab.com/fvtt-modules-lab/quick-insert/-/blob/master/src/systems/dnd5e.ts?ref_type=heads#L76

Hoping for a response here: https://gitlab.com/fvtt-modules-lab/quick-insert/-/issues/114

In the meantime, I will get the API ready and provide a world script

API is ready.
Rather than doing the world script yet, I am now working on merge request for the module author.

PR created here: https://gitlab.com/fvtt-modules-lab/quick-insert/-/merge_requests/24

Sunspots is a hero! Closing as completed.


Accept LinkedMultiValueMap instead of Map<String, String> for POST data

Hi,
I am submitting a POST request in order to get the page but the problem is that in the payload, there can be multiple fields with the same key and different value. This currently it is not possible with Map<String, String> type as a data while LinkedMultiValueMap<String, String> can. For example, HttpEntity client in Spring Boot can accept multiple fields using LinkedMultiValueMap link.

OK, I just saw calling connection.data("key", "value") can add multiple values for same key, but it would be still nice to accept MultiValueMap I think. What is your opinion on this?

The Apache Httpclient has a similar solution called HeaderGroup. https://hc.apache.org/httpclient-3.x/apidocs/org/apache/commons/httpclient/HeaderGroup.html Maybe something similar could be implemented?


Update code of conduct path in contributor covenant badge

Currently it navigates to 404

:tada: This issue has been resolved in version 1.4.0-rc.4 :tada:
The release is available on:

npm package (@release dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Able to scroll contents when reading manual pages

Description of the new feature/enhancement
When I open xfce4-terminal on Linux or MobaXterm on windows, type "man ps" and use the mouse wheel to scroll up and down, magic happens, I am navigating down and up inside the manual page.
When I type "man ps" in WSL MS Terminal then use the mouse wheel, I am scrolled up to the contents of the screen that was before typing "man ps"

Proposed technical implementation details (optional)
I have no idea what magic is behind the scenes in Linux and MobaXterm, so I don't know what to technically propose. Try and reproduce for yourself and you'll understand what I describe.
I don't even know if this belongs here or to microsoft WSL. or need fixing in both :)

Sure is!
/dup #3321


Option to sort by submit date instead of lexicographically

Right now we always sort by the value of the grouping column. condor_q instead sorts by submit date. If we do a query we may actually know the cluster submit date; if we don't, a good proxy might be the first event timestamp we see for that cluster.
I think it's debatable whether the default sort should be lexicographic or submit date, but for historical reasons submit date might be the way to go (that's how condor_q does it...).

Note that by construction you can also sort on cluster ID if QDate is not known.

Good point! Depends on whether we also want to actually show the submission date, which might be useful for people who don't set batch ids.

Jotting down some thoughts...
Your standard user doesn't set batch names. For them, submissions are largely identified by order and submission time ("the second batch of jobs I submitted", "the jobs I submitted yesterday morning"). For them, we want to order by submit date/clusterid, because the batch name is not significantly more meaningful... unless they happen to be using DAGMan, in which case grouping by batch name regains its meaning because the jobs in the DAG share a batch name but not a cluster id.
If you do set batch names yourself, you probably want to order them lexicographically, because (if your naming scheme is meaningful), you probably wants batches with similar names to appear near each other (even if it's just run-1, run-2, ...). You'll still want the option to sort by submission date though.
I'm this close to thinking we should set the default grouping and sorting options by some system of rules based on what we discover in the initial query. Maybe just "if any batch names are not the Condor-set default, the default grouping is by batch name and the sorting is lexicographic. If not, the default grouping is by clusterid and we sort by submission date and then clusterid if submission date is not available".

Hi @JoshKarpel -
I'm distrustful of "magic behavior".  We really ought to have a deterministic default for sorting (adhering to the condor_q behavior unless we can clearly articulate why it's wrong as in the case with -cluster) and be able to give an option for overriding this.
Let's go with sorting by earliest jobid in the batch (as that's our understanding of condor_q's behavior).
Brian


Revert back the expected virus scan result body.

The virus scan result body keeps changing from time to time so a very specific string
will fail. Currently the virus scanner is returning just 'true' or 'false'.

If the virus scanner result has changed between "Everything OK : true" and "true" more than once, wouldn't it be safer to use a regexp to test for a successfully-passed scan, rather than relying on an exact match?

Actually, the idea that it might start responding with "Virus found : true" is a good point. Maybe we should test for ["Everything OK : true", "true"].include?(response)


Create additional info blocks

You can make your resume less official by adding more entertaining info, like your hobbies and interest, or what do you like, some interactive content or joke, easter egg, etc.

Hobbies : Reading
Social Media
Programming


Support Mkdir/CreateFile with configured default umask in HDFS API

What changes are proposed in this pull request?
Support creating directories and files with default permissions based on configuration propertykey in HDFS API.
Why are the changes needed?
Alluxio Hdfs api hasn't the corresponding implementation of Mkdir without permission parameter. If it does not carry permission, the umask property in the configuration item shall prevail.
Does this PR introduce any user facing changes?

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.10 please

@Jackson-Wang-7 do we need this in main? Thanks!

alluxio-bot, cherry-pick this to main please


SAMOA-13: Update "How to contribute" guidelines

Apart form the small fixes above, +1

+1

Seems there are some build issues.
@matthieumorel are you able to have a look at them?
Otherwise I will try to fix them later this week.

Thanks, it looks OK.
I opened a PR to your branch to address the comments in this thread.
If you review them, merge them, and push to the PR I will commit it to the master branch.
If you cannot do that, just +1 to the changes on this thread and I will commit them.


Add Angular 9 support

Hello, i have been facing an error while angular 9 upgrade process of my project.
`Compiling @oasisdigital/angular-material-search-select : es2015 as esm2015
Error: Error on worker #5: Error: Failed to compile entry-point @oasisdigital/angular-material-search-select (es2015 as esm2015) due to compilation errors:
node_modules/@oasisdigital/angular-material-search-select/fesm2015/oasisdigital-angular-material-search-select.js:203:18 - error NG1010: Value at position 1 in the NgModule.imports of SearchSelectModule is not a reference: [object Object]
203         imports: [
~
204             CommonModule,
~~~~~~~~~~~~~~~~~~~~~~~~~
...
211             ReactiveFormsModule
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
212         ],
~~~~~~~~~
at ClusterWorker.compile (/Users/umutcelik/code/magistum/magistum-pwa/node_modules/@angular/compiler-cli/ngcc/src/main.js:177:27)
at Worker.<anonymous> (/Users/umutcelik/code/magistum/magistum-pwa/node_modules/@angular/compiler-cli/ngcc/src/execution/cluster/worker.js:44:42)
at Worker.emit (events.js:321:20)
at process.<anonymous> (internal/cluster/worker.js:32:12)
at process.emit (events.js:321:20)
at emit (internal/child_process.js:881:12)
at processTicksAndRejections (internal/process/task_queues.js:85:21)
at ClusterMaster.onWorkerMessage (/Users/umutcelik/code/magistum/magistum-pwa/node_modules/@angular/compiler-cli/ngcc/src/execution/cluster/master.js:158:27)
at /Users/umutcelik/code/magistum/magistum-pwa/node_modules/@angular/compiler-cli/ngcc/src/execution/cluster/master.js:46:95
at ClusterMaster.<anonymous> (/Users/umutcelik/code/magistum/magistum-pwa/node_modules/@angular/compiler-cli/ngcc/src/execution/cluster/master.js:238:57)
at step (/Users/umutcelik/code/magistum/magistum-pwa/node_modules/tslib/tslib.js:136:27)
at Object.next (/Users/umutcelik/code/magistum/magistum-pwa/node_modules/tslib/tslib.js:117:57)
at /Users/umutcelik/code/magistum/magistum-pwa/node_modules/tslib/tslib.js:110:75
at new Promise (<anonymous>)
at Object.__awaiter (/Users/umutcelik/code/magistum/magistum-pwa/node_modules/tslib/tslib.js:106:16)
at EventEmitter.<anonymous> (/Users/umutcelik/code/magistum/magistum-pwa/node_modules/@angular/compiler-cli/ngcc/src/execution/cluster/master.js:232:32)
at EventEmitter.emit (events.js:321:20)`

Thank you. we haven't tried this internally with v9 yet either, I'm sure it will be a modest fix.

I am in the process of revamping the source code to use the default file arrangement and features of current Angular CLI 9. So not only will it get Angular 9, but also the build etc. will be simplified and 100% off-the-shelf Angular standard.

I just published version 1.1.1. It upgrades everything to Angular 9, removes a few bespoke build/publish mechanisms in favor of the capability built into Angular CLI.
I didn't have time to test with anything less than the very most current Angular and Angular Material, but if you are moving up to those then you should find it works very well.


read in str files

Currently it's only possible to read in the topology (rtf) and parameter (prm) as two files, the option to read in (str) files, a combined version of rtf and prm file would be very helpful

Codecov Report

Merging #81 (352a22b) into master (2d503aa) will decrease coverage by 0.09%.
The diff coverage is 61.11%.


If mqtt server is down stop loop

If for some reason mqtt server stops (wifi connection still alive), the loop is blocked for x seconds.
Is possible to have a no stop loop?

This is related to #640. You can try that fork and see if it fixes your problem.


Add document upload functionality

https://playmoneystudios.atlassian.net/browse/PLAY-26

Might want to add an example of this functionality to the README

I will create a separate ticket for that


Pitch page login should take user back to pitch page (not sign them up)

Currently - user clicks login on any pitch page and it directs them to the action page after sign up or account creation.
Aspirational - user logs in or creates an account and at the end of the auth flow the modal goes away and they are on the same pitch page and one click away from sign up.

closing, duplicate of #1289


RPED fixes

About the PR

Fixes #19041
part exchanger system now uses 100% less .Owner calls.
the replacing audio now only cuts off if the doafter failed (dorkitos)
it passes around a list of a key value pair which is kinda sloppy but this code is niche and not in a hot loops so perf be damned.
Media

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

Changelog

:cl:

fix: Fixed the RPED sound effect cutting off at the end.

Just the thing I mentioned on discord around tuples i.e. List<(EntityUid Entity, MachinePartComponent Component)> instead of using Item1 / Item2 (if it works for args).


rpmmd: use repo name as ID for DNF, instead of list index

All of the repo definitions added after RHEL-8.3 consistently contain
'name' of each repository. However the rpmmd functions don't use it, but
instead use the index of the repository in the list of repositories.
This makes it a bit harder to determine the appropriate repository in
case of DNF failures such as:
2021/10/01 10:33:53 Failed to load subscriptions. osbuild-composer will
fail to build images if the configured repositories require them: no
matching key and certificate pair
Errors during downloading metadata for repository '3':
  - Status code: 403 for
    https://rpmrepo-storage.bucket.vpce-08d3201af28373567-o10c2v4q.s3.us-east-1.vpce.amazonaws.com/data/rhvpn/el8/sha256-518e7c455fb0bb23e32679c45a45cc269bdd5bd915b560b98bfd699aa4f100e0
(IP: <IP_ADDRESS>)
panic: Could not depsolve: DNF error occured: RepoError: Error occurred
when setting up repo: Failed to download metadata for repo '3': Cannot
download repomd.xml: Cannot download repodata/repomd.xml: All mirrors
were tried

goroutine 1 [running]:
main.main()
        /home/thozza/devel/osbuild-composer/cmd/osbuild-pipeline/main.go:128
+0x1505
exit status 2

Modify osbuild-pipeline to use the 'name' from repository definitions.
In case it is not defined, fall back to use repo-<idx> value to
prevent Name being set to an empty string.
Modify methods in rpmmd to use the repo Name, instead of taking the
index as an argument.
Add 'name' to each repository definition in
tools/test-case-generators/repos.json, which missed it.
Errors from DNF now look like this:
2021/10/01 10:35:58 Failed to load subscriptions. osbuild-composer will
fail to build images if the configured repositories require them: no
matching key and certificate pair
Errors during downloading metadata for repository 'ha':
  - Status code: 403 for
    https://rpmrepo-storage.bucket.vpce-08d3201af28373567-o10c2v4q.s3.us-east-1.vpce.amazonaws.com/data/rhvpn/el8/sha256-518e7c455fb0bb23e32679c45a45cc269bdd5bd915b560b98bfd699aa4f100e0
(IP: <IP_ADDRESS>)
panic: Could not depsolve: DNF error occured: RepoError: Error occurred
when setting up repo: Failed to download metadata for repo 'ha': Cannot
download repomd.xml: Cannot download repodata/repomd.xml: All mirrors
were tried

goroutine 1 [running]:
main.main()
        /home/thozza/devel/osbuild-composer/cmd/osbuild-pipeline/main.go:128
+0x1505
exit status 2

Signed-off-by: Tomas Hozza<EMAIL_ADDRESS>This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] create a file in news/unreleased directory if this change should be mentioned in the release news
[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

Abandoning this PR in favor of #2023.


Improve error handling

@eapache for review
Problem
I noticed a problem with the RecordLoader example in the README when records were missing.  We weren't fulfilling the load for ids if they are missing in the query result.
When we do come across a bug in a loader, we should also make these errors easier to debug by checking for unfulfilled promises in the loader and in the executor to provide a helpful error message.
I also noticed that we aren't actually handling a GraphQL::ExecutionError properly, since it wasn't behaving as if the error was raised in a field's resolve proc.  The error was causing the whole request to fail, without showing other fields, instead of being treated as a field error.
Solution

I added GraphQL::Batch::Loader#fulfilled? to make it easier to check for unfulfilled promises in #perform
GraphQL::Batch::Loader#resolve now raises GraphQL::Batch::BrokenPromiseError if a promise is still pending after #perform
GraphQL::Batch::Executor#wait now raises GraphQL::Batch::BrokenPromiseError if the promise is still pending after all loads have been performed
I added an on_reject handler in the get_finished_value override in the execution strategy

:+1:


Refine definition of KiCad

What the site currently classifies as KiCad is, according to their documentation, three entirely different formats:

Schematic files: Two different files, same format (and extension). Still in use.

.sch: 5,501 samples collected out of ~224,968 results

Board files: Original PCB layout format, now superseded by new S-expression format.

.brd: 2,907 samples collected out of ~107,658 results

Pcbnew layout: Primary PCB format used by KiCad suite, with a syntax based on S-expressions:

.kicad_pcb: 16,808 results
.kicad_mod: 284,942 results
.kicad_wks: 306 results
fp-lib-table: 4,482 results

Technical specifications of each format are covered in a PDF file, downloadable from KiCad's website.
I collected a fresh harvest of both sch and brd files, with the breakdown graphs included in each silo:

.sch breakdown
.brd breakdown

Notes

KiCad formats are data formats, not programming languages. I'm not sure why @pchaigno chose the latter in #2309 / #2187, but these formats are essentially just lists of coordinates, property lists, and object descriptions. XML has more in common with programming languages than these do.

A considerable number of sch search results were Scheme files. I ran an additional search to gauge the extension's usage better, and concluded that it's common enough to include it as a recognised Scheme extension. There were numerous Racket files as well, but I don't know the difference between Scheme and Racket, so I left the latter as-is.

Sample files were sourced from faffing around with vanilla KiCad and gEDA-gaf installations, and me clicking anything that looked like a drawing tool. Needless to say, I couldn't find any samples released under a clear permissive license, so I hacked together my own.

Scheme sample sboyer.sch released to the public domain (source confirmed here).

Obligatory sidenote: The harvester.js snippet I wrote to collect public search results has been moved to a Gist. Everything else has been torched and salted, with local copies eradicated by rm ‑rfP. I've never felt palpable disgust over such sloppy code. NFI what I was thinking or doing when I wrote that bullcrap.If you want to calculate a summary of unique repositories, you can use the standard Unix toolchain:

# Filter list of unique repositories
grep < urls.log -iEoe '^https?://raw\.githubusercontent\.com/([^/]+/){2}' |\
uniq | sed -Ee 's,raw\.(github)usercontent,\1,i' > unique-repos.log

Update
New samples added from recently-discovered repositories which are, thankfully, released under a permissive license:

986-Studio/FelinaePurr: BSD-3-Clause
gatechipdl/GameOfLight: MIT/Expat
akhenakh/nanoAPRS: MIT/Expat
agustinBassi/juegoSimon: BSD-2-Clause

Sources shared with #3744.

Haha, merci! This is only the first half of my research; I'm currently working on the second. =) It's related, but a separate format.

Soon, a monopoly on GitHub's grammars?

I have a special project in mind for the future regarding grammars and GitHub. =) But I have an infuriating habit to announce my intent to start projects before I actually do so, and then later lose interest when something else distracts me later on. So I'm saying nothing until I actually do start working on it.
I believe I've alluded to it to you before, though. ;)

I believe I've alluded to it to you before, though. ;)

This one? :smiley:

In a sense. It won't be a canonicalisation of TextMate, though. I envision it being more of an evolution: building upon every strength the TextMate grammar format has, while addressing the glaring weaknesses that've contributed to its growing fragmentation between editors - no sense of context, zero macro or variable support for grammar authors, and most of all, no support for multiline pattern-matching.
Preserving the existence of a portable, flexible and approach grammar format is the first half of the project. The other... well, language recognition. Remember my first grammar submission to Linguist, where I mistakenly believed that grammars were needed for classification accuracy? Well I was damn wrong, but that misconception started an interesting train of thought.  If authored carefully, a grammar could reliably identify languages based on the frequency of matches, and the scopes assigned to them. The executable would be passed a file, and a list of grammar scopes to test. It'd then spit out a line-delimited list of multipliers describing how confident it is in each match. So for a start, there's your not-a-language right here:
$ syn something.sol --scopes js,tsx...
0.025 TeX
0.25 JS
...

Secondly, an important characteristic about Synapse (I may as well just spill my damn guts, now nothing will ever get started 😆 ) would be the simplicity of what it does. It doesn't output CSS, or HTML, or anything but an ordered sequence of offsets in a file tagged with human-readable descriptions ("scopes"), which are up to a higher-level implementation like Atom or GitHub to deal with. Hopefully it'll be pure Unix in spirit: think grep or ack with semantic pattern-matching.
There's the idea, now I've jinxed it. 😢

Gotta finish this off first, though: I've been promising to do it since last September. Nearly there...

Real-time troff rendering with HTML5 canvas technology. 👍  Only taken me 3-4 months, hahahah.

If authored carefully, a grammar could reliably identify languages based on the frequency of matches, and the scopes assigned to them. The executable would be passed a file, and a list of grammar scopes to test. It'd then spit out a line-delimited list of multipliers describing how confident it is in each match.

I think that was the approach taken by Pygments to classify files. The issue with this approach is that it's rather costly, in particular if you have a long list of grammars to interpret. It would be nice to have some actual benchmarks for this approach though.

Nope. See, Pygments uses lexical parsing (IIRC), whereas Synapse grammars use the same nested, regexp-based syntax that TextMate grammars have always used. My grammars are probably the most complex, structured and "semantically authored" grammars out there, and if anybody wants to compete with my belligerently OCD scrutiny, step up to the plate. My Roff grammar is probably the best example of how this can be done well.
 The flexibility of the current format is one thing I refuse to part with, and not everything that receives highlighting is, dare I say, a "language".

Oh, and grammar authors who've taken care to structure their grammars semantically can assign arbitrary "weights" to each pattern that can influence how strongly a successful match impacts language recognition.
function name ( param ) would, for example, hold much greater weight over, say, a floating point literal.

@pchaigno While I'm reminded by #3751, do you feel that Eagle would be better represented as data rather than markup? I suppose there's a grey area where page-description languages overlap with image data, but Eagle seems pretty clearly composed of immutable data...


Bulk Text Notifications 📱

Description
We want to add the infrastructure to be able to send bulk text notifications to members.
Acceptance Criteria
There are two main parts to implementing this:

[ ] #471
[ ] #472

After these issues are implemented, there will be functionality for members to add their phone numbers. Then, when we want to send a mass text notification, we can pass in all the added phone numbers to the new job and then utilize Twilio to send texts to all those numbers.

I would like to take on this project!

@imnotrafa this project is actually divided up into separate issues, so only those issues can be assigned


Recipe:  for loop can be replaced with enhanced for

Before :

List<String> elements = List.of("Hello","World");

for (int i = 0 ; i < elements.size(); i++){
       System.out.println(elements.get(i));
 }

After :

List<String> elements = List.of("Hello","World");

for (String element : elements) {
      System.out.println(element);
 }

See  : https://www.cis.upenn.edu/~matuszek/General/JavaSyntax/enhanced-for-loops.html

This may be easier once data flow analysis is available to track the usages of the index.


[BUG] useUserContextInSSR not working anymore

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I remove the  from the account menu because I want to render the menu on the server side. I have also activated "useUserContextInSSR". When I log out, isLoggedIn is wrong on the client and server side. When I log in with a customer account, isLoggedIn is true on the client side and false on the server side.
And because the values are different on client-side and server-side, you also get a hydration warning. And I know that you can easily fix it with the  tag. But in this case, if you want to render it on the server side, that's not the point.
Expected Behavior
When I enabled the "useUserContextInSSR" option, I expected the UserContext to be available for client-side requests and server-side requests. Currently we got wrong responses with useUser() composable
Steps To Reproduce

Go to the stackblitz reproduction
And click on the top right corner "Open preview in new tab" and click "Connect to project".
Look in the Browser console (Client side) from the new tab. No user is logged in so you get "User is logged in: false". If you look in the Stackblitz console (Server side) there is the same: "User is logged in: false".
Click on the top right corner sign in in the preview tab from Step 2.
Sign in (If you have any account) or Sign up (if you not have a account).
After the registration is complete try to login. After login reload the Tab from Step 2. Now you can see in the browser console (CLient side) "User is logged in: true", but if you look in the stackblitz console (Server side) there is: "User is logged in: false"

Environment
No response
Anything else?
Reproduction link vue-demo-store:
https://stackblitz.com/edit/github-5gcazj?file=components%2Faccount%2FAccountMenu.vue

hey, thanks for reporting it. It will be fixed together with https://github.com/shopware/frontends/pull/1192 I'm working rn. the problem was that cookie with a context token couldn't be read in the SSR. The PR will help to solve that.

@mkucmus Thank you :)

The current draft is here. :raised_hands:


Update WHSSDatlas_v1.01.jsonld

@lzehl Thank you for being patience with me and teaching me along the way.
I hope all is correct now.


Using AngularSplit in a sub module requires use of undocumented forChild()

In using AngularSplitModule, I could import and access it in the app-module level no problem. However, I wanted to also use it in a child module deep in the hierarchy. This is not clear or documented how to do so. After much searching,  I found I needed my child module to access it via:
import {AngularSplitModule} from 'angular-split';

@NgModule({
    imports: [
        AngularSplitModule.forChild(),

This needs to be documented for consistency and to help save time.  I spent hours trying to figure out how to get use of as-split and as-split-area in child modules.

I've just released 3.0.0, check demos using lazy loading:
https://bertrandg.github.io/angular-split/#/examples/lazy-loaded-module
forRoot/forChild methods are still there but not needed anyore:
https://github.com/bertrandg/angular-split/blob/master/projects/angular-split/src/lib/module.ts


Feature/auth

Implemented authentication. You can now register and login, and the tokens are stored as cookies, allowing for seamless use. The way this is handled (not at the time of writing, but hopefully a couple of days later) is by refetching a new access token when the old one expires, leading to a minimal amount of requests to the server, as well as leaving the user clueless as to what is going on (in a good way ;).

To use the authentication methods now, I've made it work as following:

You want to extract the store and dispatch from the AuthCtx by typing const { store, dispatch } = React.useContext(AuthCtx). This is initialized in App.tsx, so you don't have to mess with it anywhere else.
Import Perform from reducers/auth by typing import { Perform } from './reducers/auth (edit path to correspond to your working dir)
Use the predefined Perform.doLogin/Register/SetUser/... functions, and pass in the dispatch as the final argument. By doing this, the mitochondria (the powerhouse of the cell) will handle the heavy lifting, and update the frontend store as it sees fit.
Handle errors that might be thrown by the function above.

I will try to get this merged into master as soon as possible - ie. when I stop shaking like a aspeløv, as it is quite crucial to the rest of the development of the application.


Implement node-webkit url changes

This changes the app name, and the urls, I have also fixed the tests to look for the new urls

Can someone please merge? :)

+1 it is needed to merge or atm the stock npm version is not working anymore

If the filename changes you propose are wrong, then this is a dupe of #156.
Also, @matteocrippa, a temporary fix is to run the stock builder with the downloadUrl option set to http://dl.nwjs.io/.

@trevorah if you look @ #156 it only changes the download url not the filenames. So it will fail

It would fail for the latest alpha, but your changes would break for all other versions as those filenames have not changed. If you can get both to work, then that would be excellent!

@trevorah if you are so critical about this, why don't you patch it? I currently don't have the time to patch this

@trevorah thanks for the hint!

@matteocrippa no problem.
@jeanres I'll be working on a fix for 0.12.0 this evening (in 7 hours time). Let me know if youre free.

Fixed by https://github.com/mllrsohn/node-webkit-builder/commit/b86431a31fe97ba7b85e52308dbe399701aa6002 and https://github.com/mllrsohn/node-webkit-builder/commit/74e54720fa4b146de9ac245d68351d922c0e9de6 which is now live as version 1.0.6.
Thanks @jeanres


[5.3] Provide access to model collection from Eloquent Builder before pagination

Added methods that allow adding a callback to underlying collection before pagination and other collection methods in Eloquent Builder.
Use Case
There have been times where I've needed to append specific attributes to models, but not with $appends which always includes the attribute. For example, on a route used to search for new users to follow, it would be beneficial to return an attribute that displays if the returned users are following the authenticated user. This kind of attribute requires a user to be logged in and has a specific use case so I would rather control when and how it's used as opposed to adding to $appends.
The problem is that I haven't found a way to modify collections before paginating. In order to modify the models of a collection, there would need to be a way to access the collection before pagination occurs.
What I've tried so far
$users = new App\User;

return $users->append(['is_following'])->paginate(); // Attribute is not added to each model.

return $users->paginate()->append(['is_following']); // Method doesn't exist.

Only works when using first() since this method returns a model.
$users = new App\User;

return $users->first()->append(['is_following']); // Works.

Proposal
$users = new App\User;

$users = $users->onCollect(function($collection) {
        return $collection->each(function($user) {
            $user->append(['is_following']);
        });
    });

return $users->paginate(); // Attribute is added to each model.

There may already be a way to do this so please let me know if there is.

$users = new App\User;

$paginated = $users->paginate();

$paginated->getCollection()->each(function ($user) {
    $user->append(['is_following']);
});

return $paginated;

@crynobone That only returns the collection alone and no longer paginated results.

@tlaverdure
$users->getCollection()->each(function ($user) {
    $user->append(['is_following']);
});

return $users;

shouldn't be the same as
return $users->getCollection()->each(function ($user) {
    $user->append(['is_following']);
});

@crynobone gotcha 👍 . Thanks for the assistance! No need for this PR then.


"there are errors in the source file." when using _axiom

The example program (now with acceptable syntax) cannot be run or proven. WW says "there are errors in the source file." But I cannot see any. That message disappears if the _axiom statement is removed.
//At first there may be axioms:
_axiom(∀Integer x: ∀Integer y : (x+y)*(x+y) = x*x + 2*x*y + y*y)

//Before the main block there may be function declarations like this:
function Integer pow(Integer base, Integer exponent) {
    Integer result := base
    Integer tmp := exponent

    if(exponent = 0) {
            return 1
    }

    while( tmp > 1) {
            result := result * base
            tmp := tmp - 1
    }

    return base
}

//After the function declarations follows the main block:
Integer a := 5
Integer b := 3
Integer c := 0

c := pow(a, b)

Saved it under test/bla.ww, the full error message is:
'Launching _test_bla.ww' has encountered a problem.
There are errors in the source file.

It seems to be related with quantifiers. Here is a MWE:
function Boolean foo()
_requires ∀ Integer i : true
{
return true
}

I still cannot reproduce this here ...
Could you please try it out with the current version I just uploaded to the build server? It displays the syntax error messages in the error dialog which appears when launching. Then we'll see which error message is causing the problem.

It's an encoding problem, even though the file is in UTF-8, it's parsed as something else:
There are syntax errors in the source file:
Line 2: no viable alternative at input '�'

@danielbruns Have you already tried it out?

sorry for the delay; it works fine


Added consplosion example; moved example into examples/libcalc

This PR adds a second small example to sbcl-librarian the purpose of which is to break GC.
For example:
bash-3.2$ python example.py
consplode > 10
consbomb still ticking...
consplode > 1000000
consbomb still ticking...
consplode > 121212
consbomb still ticking...
consplode > 123123213
Heap exhausted during garbage collection: 0 bytes available, 16 requested.
Gen  Boxed   Code    Raw  LgBox LgCode  LgRaw  Pin       Alloc     Waste        Trig      WP GCs Mem-age
 1    4929      0   4920      0      0      0    3   322633696     98336    10737418    9849   0  1.0000
 2    6835      1   6822      0      0      0    8   447426176    119168     2000000   13658   0  0.0000
 3       0      0      0      0      0      0    0           0         0     2000000       0   0  0.0000
 4       0      0      0      0      0      0    0           0         0     2000000       0   0  0.0000
 5       0      0      0      0      0      0    0           0         0     2000000       0   0  0.0000
 6     653    361    234     55      0     10    0    42130416    893968     2000000    1313   0  0.0000
 7    4920      0   3027      0      0      0    0   260407296         0     2000000    7947   0  0.0000
           Total bytes allocated    =<PHONE_NUMBER>
           Dynamic-space-size bytes =<PHONE_NUMBER>
GC control variables:
   *GC-INHIBIT* = true
   *GC-PENDING* = true
   *STOP-FOR-GC-PENDING* = false
fatal error encountered in SBCL pid 50314 pthread 0x118cc3e00:
Heap exhausted, game over.

   0: fp=0x7ffee632d720 pc=0x10006209f2 SBCL-LIBRARIAN.TESTS.CONSPLOSION::CONSPLODE
   1: fp=0x7ffee632d7d8 pc=0x1000625661 (LAMBDA (SBCL-LIBRARIAN.TESTS.CONSPLOSION::N SBCL-LIBRARIAN::RESULT) :IN "/Users/cbokeefe/quicklisp/local-projects/sbcl-librarian/examples/consplosion/bindings.lisp")
   2: fp=0x7ffee632d800 pc=0x1000625871 (LAMBDA (SB-ALIEN::ARGS-POINTER SB-ALIEN::RESULT-POINTER FUNCTION) :IN "/Users/cbokeefe/quicklisp/local-projects/sbcl-librarian/examples/consplosion/bindings.lisp")
   3: fp=0x7ffee632d8f0 pc=0x1000068a48 (FLET SB-UNIX::BODY :IN SB-THREAD::RUN)
   4: fp=0x7ffee632d9e0 pc=0x10000691b2 (FLET "WITHOUT-INTERRUPTS-BODY-10" :IN SB-THREAD::RUN)
   5: fp=0x7ffee632db30 pc=0x100006871f (FLET SB-UNIX::BODY :IN SB-THREAD::RUN)
   6: fp=0x7ffee632dc20 pc=0x1000068414 (FLET "WITHOUT-INTERRUPTS-BODY-3" :IN SB-THREAD::RUN)
   7: fp=0x7ffee632dd10 pc=0x10000682c7 SB-THREAD::RUN
   8: fp=0x7ffee632ddc8 pc=0x10000129e4 SB-THREAD::ENTER-FOREIGN-CALLBACK
   9: fp=0x7ffee632de00 pc=0x10a1bfd5c Foreign function call_into_lisp
  10: fp=0x7ffee632de30 pc=0x10a19daf0 Foreign function funcall3
  11: fp=0x7ffee632de80 pc=0x10a1aef0b Foreign function callback_wrapper_trampoline
  12: fp=0x7ffee632de90 pc=0x1debd233c (unknown lisp object)
  13: fp=0x7ffee632dec0 pc=0x1099992b2 Foreign function meth_dealloc
bash-3.2$

Some tasks

[X] add a small "comands interpreter" to the example, the first one runs the current behavior
[X] add a command hash m to test how big of a hash table can I create and trash before killing the image?

To use as the starting point for additional stress-testing tools, to be added as they are thought of.

I'd push back on adding this as an example since I believe the point of an "example" is to illuminate how to use the library. The consexplosion code here doesn't seem to illustrate the use of any features that libcalc doesn't demonstrate already - it just crashes the image, so I don't think it makes sense to have it be a part of `sbcl-librarian itself. Perhaps as part of the test suite instead? Then again, there's nothing concrete being tested for either.

Agreed that it isn't a proper "example" and we could do with getting some organization right in the first place.

I'll close the PR but retain the branch until I think of a better place to put it.


Not possible to Install on 0.8.9

Describe the bug
I can't install the Module on Foundry-0.8.9, because of a not valid manifest file.
To Reproduce
Steps to reproduce the behavior:

Go to 'Add-on Modules'
Click on 'Install Module'
Search for 'loot sheet'
Press Install
See error

Expected behavior
Dunno, should Install
Screenshots

Server (please complete the following information):

OS: Ubuntu
With Docker: felddy/foundryvtt:0.8.9]

The latest version is not compatible with versions prior to 9.
There is an older release in the releases page that still works with it. Alternatively, upgrade to 9.

On Mar 19, 2022, at 9:49 AM, Julez @.***> wrote:

Describe the bug
I can't install the Module on Foundry-0.8.9, because of a not valid manifest file.
To Reproduce
Steps to reproduce the behavior:
Go to 'Add-on Modules'
Click on 'Install Module'
Search for 'loot sheet'
Press Install
See error
Expected behavior
Dunno, should Install
Screenshots
Server (please complete the following information):
OS: Ubuntu
With Docker: felddy/foundryvtt:0.8.9]
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.


Fix issue #217 and #216 - Only dump ivars for subclasses of Hash and String.

Fix issue #217 and #216 - Only dump ivars for subclasses of Hash and String.

I think this is fine.  Thanks for the PR!

Thanks for the quick merge, @tenderlove!


Filter and sort players by game region

For example, sorting people who play EN (either exclusively or along with JP), and if you're going by rank, sort them by their EN rank, not their JP rank (if that's higher). And same for JP. Probably not super important but I think it'd be nice to be able to sort only EN players by their EN accounts, or the same for JP.

Thanks for suggesting!
I don't really know if that's doable and how hard it will be, but I'll see what I can do :)


more test scripts and test coverage

end to end
unit test
test coverage indicator

Frontend test:

register login - @BeLl0o
interest - @XSiling
homepage - @Niko-Yuan

Backend test:

homepage - @Samanvitha-Sateesha

Rec sys test:

@varunpatro1 @seanlindstrom @snpatilucsd


Add a tock2 submodule with a tock-2.0-dev kernel.

I intend to use this kernel in integration tests for the Tock 2.0 crates. When we are ready to remove Tock 1.x support, I will remove the duplicate submodule.

bors r+


Bugfixes/review pr 1687

Some changes related to the usage telemetry:

Passing system data as event context (will simplify later integrations)
Show warning message on server start to clear preventing users

@dvsrepo Please, provided a warning message for the server startup telemetry check


[WFLY-17581] Prolong timeout because of intermittent test failures

https://issues.redhat.com/browse/WFLY-17581

Hello, mchoma. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.

@jamezp could you please have a look? I believe you was tweaking RestEasy testsuite with something similar.

/ok-to-test

/retest

Thanks @mchoma and @jbliznak


in backend, member page has organization tab displaying how many organization profile linked to a specific member

completed at version 0.6.343


API to create a new local-variable identifier that won't conflict with an existing scope

I'd like an API to come up with a new local-variable identifier that won't conflict with an API in any higher/lower scope.
For example:
class Test {
    void test(String base, String fileName) {
        String path = base + "/" + fileName;
        // does something with 'path'
    }
}

Let's say I want to add the following below the declaration of path: File file = new File(path);. I need an API that allows me to generate a new variable name that won't conflict with any lower scope and any method declaration scope.
For the first implementation, this does not need to work outside the scope of a method body.
Here are some examples of cases and what name might be generated:

Case 1

class Test {
    void test(String base, String fileName) {
        String path = base + "/" + fileName;
        // does something with 'path'
    }
}

Would generate a variable named file.

Case 2

class Test {
    void test(String base, String fileName) {
        String file = base + "/" + fileName;
        // does something with file
    }
}

Would generate a variable named file1.

Case 3

class Test {
    void test(String base, String fileName) {
        String path = base + "/" + fileName;
        // Need to inject new file name here
        if (true) {
            String file = System.getenv("user.home") + "/" + path;
        }
    }
}

Would generate a variable named file1.

@JLLeitschuh I think recipes need to be responsible to generate new variable names to produce reliable results.
Primarily, because visitors may visit any AST element, and the root of the cursor is based on the visited element.
So, there might be identifiers in the name scope that aren't visible due to the cursor position.
We have an example of what you are asking for in RenameExceptionInEmptyCatch. link.

You can theoretically look down the tree, and look up the tree can't you? Or are you concerned that up the tree may have been modified already by another visitor?
In that case, take the cursor, and the current AST element as an argument, and if the cursor value != the current AST element, thrown an illegal argument exception

It's not about modification.
Yes, the recipe may guarantee the tree may be traversed up and down, but a utility can't do that.
Example:
public class Example extends Recipe {

    @Override
    public String getDisplayName() {
        return "";
    }

    @Override
    protected TreeVisitor<?, ExecutionContext> getVisitor() {
        return new JavaIsoVisitor<ExecutionContext>() {
            @Override
            public J.ClassDeclaration visitClassDeclaration(J.ClassDeclaration classDecl, ExecutionContext executionContext) {
                J.ClassDeclaration cd = super.visitClassDeclaration(classDecl, executionContext);
                // The cursor root will be `cd`
                cd = new ExampleVisitor().visitClassDeclaration(cd, executionContext);
                return cd;
            }

            @Override
            public J.VariableDeclarations visitVariableDeclarations(J.VariableDeclarations multiVariable, ExecutionContext executionContext) {
                J.VariableDeclarations mv = super.visitVariableDeclarations(multiVariable, executionContext);
                // The cursor root will be `mv`
                mv = new ExampleVisitor().visitVariableDeclarations(mv, executionContext);
                return mv;
            }
        };
    }

    private static class ExampleVisitor extends JavaIsoVisitor<ExecutionContext> {
        @Override
        public J.ClassDeclaration visitClassDeclaration(J.ClassDeclaration classDecl, ExecutionContext executionContext) {
            // Root of the cursor will be the `classDecl`.
            return classDecl;
        }

        @Override
        public J.VariableDeclarations visitVariableDeclarations(J.VariableDeclarations multiVariable, ExecutionContext executionContext) {
            // Root of the cursor will be the `multiVariable`.
            return multiVariable;
        }
    }
}

The Cursor root is based on the AST element that is visited, so we may look down from the root, but not up.

That sounds like a bug in the following lines:
cd = new ExampleVisitor().visitClassDeclaration(cd, executionContext);

mv = new ExampleVisitor().visitVariableDeclarations(mv, executionContext);

They were called without passing the cursor correctly. They should have been called like this:
cd = new ExampleVisitor().visitClassDeclaration(cd, executionContext, getCursor());

mv = new ExampleVisitor().visitVariableDeclarations(mv, executionContext, getCursor());

I understand that you may not have used the API like this always, but I think that you should be able to safely assume that you will have the entire cursor tree, and if you don't blow up with a requisite exception. The tests should catch that you're not passing the correct cursor to the visit.... call quickly.

I'll check in with the team tomorrow and get back to you afterward.


Support filters with complex transform

Isn't this already supported? Or what exactly is meant by complex transforms?

Nope. I still have no idea how to implement it. Just grep filter tests for "complex transform". They are all failing.


Cannot use variables from artifacts:reports:dotenv in other job image: field

Minimal .gitlab-ci.yml illustrating the issue
---
my_docker_image:
  variables:
    ARTIFACT_ENV_FILE: artifact_vars.env
  script:
    - export VAR_NAME="${CI_JOB_NAME_SLUG//-/_}_reference"; VAR_NAME="${VAR_NAME^^}"  # Uppercase
    - export IMAGE_REF="local-test:ci-${CI_PIPELINE_IID}-${CI_JOB_ID}"

    - echo 'FROM alpine' | docker build -t "${IMAGE_REF}" -
    - echo "${VAR_NAME}=${IMAGE_REF}" |tee -a "${ARTIFACT_ENV_FILE}"
  artifacts:
    reports:
      dotenv: ${ARTIFACT_ENV_FILE}

.say_hello:
  script:
    - echo "Hello World -- From MY_DOCKER_IMAGE_REFERENCE='${MY_DOCKER_IMAGE_REFERENCE}'"

say_hello_with_alpine:
  extends: [.say_hello]
  image: alpine

say_hello_with_needy_alpine:
  extends: [.say_hello]
  needs: [my_docker_image]
  image: alpine

say_hello_with_new_image:
  extends: [.say_hello]
  needs: [my_docker_image]
  image: ${MY_DOCKER_IMAGE_REFERENCE}

Expected behavior
Alle jobs (including 'say_hello_with_new_image') show be run. Like on the 'real' gitlab-ci.
Host information
macOS Ventura 13.6
gitlab-ci-local 4.43.0
Containerd binary
Are you using docker or podman, or any other relevant containerization tool.
Docker Desktop v4.22.1. Docker version 24.0.5, build ced0996, containerd v1.6.21.
Additional context
gitlab-ci-local say_hello_with_new_image --needs fails with:

say_hello_with_new_image    starting :latest (test)
say_hello_with_new_image    copied to docker volumes in 477 ms
Error: Command failed with exit code 1: docker image inspect :latest
Error response from daemon: invalid reference format
[]
at makeError (/usr/local/lib/node_modules/gitlab-ci-local/node_modules/execa/lib/error.js:60:11)
at handlePromise (/usr/local/lib/node_modules/gitlab-ci-local/node_modules/execa/index.js:118:26)
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at Job.pullImage (/usr/local/lib/node_modules/gitlab-ci-local/src/job.ts:760:13)
at Job.execScripts (/usr/local/lib/node_modules/gitlab-ci-local/src/job.ts:622:13)
at Job.start (/usr/local/lib/node_modules/gitlab-ci-local/src/job.ts:474:36)
at /usr/local/lib/node_modules/gitlab-ci-local/node_modules/p-map/index.js:57:22

Add to tests/test-cases/artifacts-reports-dotenv/.gitlab-ci.yml:
# Test with image:$dotenv variable
build-image-ref:
  stage: build
  needs: []
  script:
    - echo "BUILD_IMAGE_REF=alpine" > build.env
  artifacts:
    reports: { dotenv: build.env }
use-image-ref:
  stage: deploy
  needs: [build-image-ref]
  image: ${BUILD_IMAGE_REF}
  script:
    - if [ "$BUILD_IMAGE_REF" = "" ]; then exit 1; else cat /etc/os-release; fi

add test to tests/test-cases/artifacts-reports-dotenv/integration.artifacts-reports-dotenv.test.ts:
test("artifacts-reports-dotenv <use-image-ref> --needs", async () => {
    const writeStreams = new WriteStreamsMock();
    await handler({
        cwd: "tests/test-cases/artifacts-reports-dotenv",
        job: ["use-image-ref"],
        needs: true,
        shellIsolation: false,
    }, writeStreams);

    expect(writeStreams.stderrLines.join("\n")).not.toMatch(/FAIL/);
});

Also, dot env variables are not available in other job variables declarations.
(Usefull for trigger job to pass the variable to the whole downstream)


list transformations by suitability sample

Can be pulled into 100.2.1 and 100.3.0 after merging.

@john0005 Please review when you have time. Don't worry if it doesn't make it into the next release. We can add it after


[Feature] Use Directory.Packages.Props for single version principle instead of .nx-dotnetrc.json

Is your feature request related to a problem? Please describe.
Currently, we use a field in .nx-dotnetrc.json for single version principle. There are some benefits to this:

We control the file, its json, its easier to read / update than an xml file.
Since its custom, .NET SDK version doesn't matter.

However, its non-standard. dotnet doesn't know about this file, and we have to track packages when they are added. We have a sync generator thats entire purpose is to keep up with this stuff, and its a really messy area in the codebase.
Describe the solution you'd like
.NET 7 introduces some capabilities for centralized package management. They use Directory.Packages.props for this.
https://devblogs.microsoft.com/dotnet/announcing-dotnet-7/#central-package-management
We already make use of other directory props files for adding tasks, so this makes sense for us.
Additional context
This feature is only supported in .NET 7+, so some thought will have to be given about supporting existing users of .NET 6 which is LTS. See lifecycle chart here: https://learn.microsoft.com/en-us/lifecycle/products/microsoft-net-and-net-core

@tzuge do you have any thoughts on this one? I think it could simplify things quite nicely.

I agree it would be nice to let the dotnet tool stack handle this behaviour. I'm curious about how it will work with the dotnet cli. If nx-dotnet still has to do special handling (e.g. xml manipulation in the props file), then that seems not as beneficial.

At the end of the day, we will still have to do some manipulation in the props file most likely. I think itd still be worth it since it would move the manipulation outside of the .csproj, which is more likely to have some weird custom stuff

Microsoft has actual documentation for Central Package Management that is slightly more informative than the announcement blog above. This page specifies that .NET SDK 6.0.300+ is compatible with the feature, not just .NET 7+.

I'm actually using this already and the way it currently works is CPM takes precedence - the package versions are added to the Directory.Packages.Props, the csproj files have nuget references without the version specified, and nx-dotnet.rc.json also contains nuget references without package versions, e.g.
{
  "nugetPackages": {
    "FastEndpoints": "",
    "Humanizer": ""
  }
}

I'm not familiar with the internals of nx-dotnet yet, but the way I understand it now, there's no need for nx to manipulate the props files since dotnet add package already takes care of it.
So either detecting if CPM is used or using a configuration flag which would enable handing off nuget versioning completely to cpm would be a good solution to this?  (i.e. not using nx-dotnet.rc.json at all in that case)

I can confirm dotnet add {PROJECT.CSPROJ} package {NUGET_PACKAGE} will auto-detect CPM - modifying Directory.Packages.props and the csproj automatically.

Is there a dotnet command that would set up CPM? If so, I'd happily drop managing nuget versions.

Not that I've found, but the steps are pretty concise:

Add Directory.Packages.props - I think it could start life as simply <Project></Project>
Add <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally> to Directory.Build.Props

I assume from there dotnet cli would just align with CPM (dotnet add, dotnet new).
I do wonder tho, this kinda eliminates any need for the nuget generators in nx-dotnet.  no?
This extension  seems to work nicely in our nx repo.

Yeah, this would eliminate @nx-dotnet/core:nuget-reference as well as @nx-dotnet/core:sync, and also the configuration flags for nuget packages entirely.
It would be a pretty large win 😄.
Would you be up for contributing the change? I haven't worked inside of a .NET repo in over a year personally, so I've not spent time learning how some of the newer stuff works.

Boo,  just found that dotnet new is not resulting in a CPM compatible csproj - it adds the version attributes which result in build errors.
I'll research this a bit more tomorrow.

Sounds good - I'm working on some configuration changes right now anyways so it wouldn't be a great idea to start working on the changeset until I merge them to avoid conflicts.

This is the mentioned PR w/ configuration changes: https://github.com/nx-dotnet/nx-dotnet/pull/787

Confirmed, dotnet new is blind to CPM.
I think it's a minor inconvenience to manually remove Version attributes from the generated csproj files.

I believe this is outside the scope of nx-dotnet  - @AgentEnder  agree?

To see this:
dotnet "new" "xunit" "--language" "C#" "--name" "NxTrends.Test" "--output" "libs/test"
The template "xUnit Test Project" was created successfully.

Processing post-creation actions...
Restoring C:\Users\Chris Leigh\Source\repos\nx-trends\libs\test\NxTrends.Test.csproj:
  Determining projects to restore...
C:\Users\Chris Leigh\Source\repos\nx-trends\libs\test\NxTrends.Test.csproj : error NU1008: Projects that use central package version management should not define the version on the PackageReference items but on the PackageVersion items: Micr
osoft.NET.Test.Sdk;xunit;xunit.runner.visualstudio;coverlet.collector.
  Failed to restore C:\Users\Chris Leigh\Source\repos\nx-trends\libs\test\NxTrends.Test.csproj (in 79 ms).
Restore failed.
Post action failed.
Manual instructions: Run 'dotnet restore'

I think the project generators would need to handle moving version specifiers to the Directory.Packages.Props file (warning if there was a mismatch), otherwise the behavior would result in this failing:
nx g app my-api --template webapi --language C#
nx build my-api

It should be fine to migrate all repos to CPM, we just need to take care to not break folks who werent really using the existing behavior.

The presence of the packages file will force folks into CPM. Meaning they
will see the restore errors similar to what’s above.  They could of course
disable CPM it from the build.props.   But then versions have to be
specified on package references.  So it’s all or nothing.
Using single versioning beforehand meant using the nuget generator.  Mixing
was possible.
Not sure if the better route is to have user opt in at init. And have
generators detect if it’s enabled and act accordingly.  This leaves it as
an option but doesn’t force it.   This could look like no migration but a
generator to turn it on after the fact.
Thoughts?
On Wed, Oct 25, 2023 at 10:00 AM Craigory Coppola @.***>
wrote:

I think the project generators would need to handle moving version
specifiers to the Directory.Packages.Props file (warning if there was a
mismatch), otherwise the behavior would result in this failing:
nx g app my-api --template webapi --language C#
nx build my-api
It should be fine to migrate all repos to CPM, we just need to take care
to not break folks who werent really using the existing behavior.
—
Reply to this email directly, view it on GitHub
https://github.com/nx-dotnet/nx-dotnet/issues/562#issuecomment-1779596456,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA5BBGU3TYR67GXKAY4Z27TYBEZSDAVCNFSM6AAAAAAR2TW2U2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZZGU4TMNBVGY
.
You are receiving this because you commented.Message ID:
@.***>


Implement INKEY with negative argument

INKEY with a negative argument tests if the given fabgl virtual key is currently held down. This is done with a keyboard state map maintained using an experimental mos_setkbvector OS call. (https://github.com/tomm/agon-mos/tree/mos_setkbvector)

Corresponding MOS PR: https://github.com/breakintoprogram/agon-mos/pull/73

@breakintoprogram If there's interest in this PR (and the MOS prerequisite), I could modify it a little to use a 32-byte keyboard state map of boolean bitfields, instead of 256 bytes, which is a bit bloated.

Hi @tomm I've ended up with a slightly different method. Whilst I'm closing this, I'll still credit you in the release notes as our ideas are essentially the same.


Icon class needs a box character to work

Hello, in the generated demo project the icon is represented in the following way with a character code box.

If I remove the  the icon disappears. How can I use the icon without the  character?
Like this...

Thanks!

you can do it two ways.

you can use CSS pseudo element on anything, like .myClass:before, .myClass:after, etc. I use this for headings and buttons. In your case you might do something i:before. You might need a non-breaking space though. If doing this myself, I would use a   with the custom character I set (see below 2), like
a
 and then just set my span in css to `p span { font-family: my_icons }. I find that using custom codes and custom font names is the easiest, and I usually append my in-house project codes to the font names to make sure there's no conflict on users' systems.

On fontello website, choose your icons, then go to "customize codes" tab, and change the codes to whatever you want (the top of the box for each icon). The bottom box will automatically update depending on what character you put in the top box.

hope that helps.

Thanks for the answer, but I was hoping I could just use it out of the box without caring about the underlying character code. The same way as Font Awesome does it, you can use their icons right away like this
In the same way as FA, is this possible with the custom generated Fontello fontset?

I'm sure there is, but I never used it that way.
You can customize those names similar to FA though. Maybe try that? I think there's notes in the CSS file comments too about setting up various ways.

If you're still looking how to do this...
All you need to do is include the CSS from fontello with pseudo elements. I actually needed to use this from something today and it worked just fine. It's the same as I mentioned before, you're just adding the pseudo elements to all classes that contain "icon-" by default.

[class^="icon-"]:before, [class*=" icon-"]:before, { font-family: "my_font"; font-style: normal; font-weight: normal; speak: none; display: inline-block; text-decoration: inherit }

.icon-heart:before { content: '\48'; }

I set mine up a little different with custom characters, and so I could use them as:before and :after but there's nothing wrong with the code included when you generate the fonts. Just add the code as a class to whatever you want and adjust css as needed.

Great thanks I will try this later on :)

no worries ;)
btw, the css is included in the downloaded zip in the css folder. It will have the name of your font. So if you named it my_icons the css file is my_icons.css. I don't use all the CSS as I already have resets in my primary CSS.
btw, you may find this useful. I added :before and :after CSS so that when I add my icons to buttons I can choose which side they are on based on class, with proper margins. In order to do that, I just added new code names in the css. You could probably do this other ways, but this works fine for me.
Below you'll see how I do this. Icons I need for buttons have -a and -b variants for before and after. If the margins pose problems you can override where needed, or just have the variants plus the normal icons like .icon-blah, .icon-blah-a, .icon-blah-b as shown below. This way you won't get any funkiness with margins on your  tags.
HTML:

do something
do something else

CSS:
[class^="icon-"]:before, [class*=" icon-"]:before, [class^="icon-"]:after, [class*=" icon-"]:after { font-family: "my_icons"; font-style: normal; font-weight: normal; speak: none; display: inline-block; text-decoration: inherit }
[class^="icon-"]:before, [class*=" icon-"]:before { margin-right: .5em }
[class^="icon-"]:after, [class*=" icon-"]:after { margin-left: .5em }

.icon-heart, .icon-heart-b:before, .icon-heart-a:after { content: '\48'; }

btw, the default css I think has margins on both sides to keep stuff centered in  tags. My use may be a bit different, so in a case like that I might add a 3rd variant with both margins.

Thanks @JohnEcks
Closing - seems to be solved. Anyway, not directly related to fontello itself.


can't connect from walrus-client  "http: server gave HTTP response to HTTPS client"

I can't connect to the walrus server
#./walrus -http <IP_ADDRESS>:9999
Listening on <IP_ADDRESS>:9999...
Server must be already sinchronized - becuase I can't see it more in the top of processes - passed near 24 hours.
.#/walrus-cli -a <IP_ADDRESS>:9999 balance
Could not get balance: Get https://<IP_ADDRESS>:9999/balance?limbo=true: http: server gave HTTP response to HTTPS client
Before this in the terminal for walrus-cli was exported passphrase from siad wallet.
#export WALRUS_SEED= ....

walrus servers are expected to use HTTPS by default so that your traffic is encrypted. It's safe to run them as HTTP if they aren't exposed to the internet, though.
To avoid this error, you must specify the http explicitly:
walrus-cli -a http://<IP_ADDRESS>:9999 balance


Is it possible to implement solid lines?

Is it possible to realize solid lines?
Here is an example of a third-party application, and an example of how it should look in an application, are there any options, how can this be implemented? If so, what options and what needs to be done to do this?

Yes.  Of course it is …
Just look here: http://www.plasm.net/gallery/simple_examples/bezierod.html http://www.plasm.net/gallery/simple_examples/bezierod.html
from Chapter 14 of GP4CAD book.
https://onlinelibrary.wiley.com/doi/book/10.1002/0470013885 https://onlinelibrary.wiley.com/doi/book/10.1002/0470013885
PLaSM (Programming Language for Solid Modeling) is the ancestor of react-planner.
https://onlinelibrary.wiley.com/doi/book/10.1002/0470013885 https://onlinelibrary.wiley.com/doi/book/10.1002/0470013885
It is being now ported to two Julia packages:
https://github.com/cvdlab/LinearAlgebraicRepresentation.jl https://github.com/cvdlab/LinearAlgebraicRepresentation.jl
https://github.com/cvdlab/Plasm.jl https://github.com/cvdlab/Plasm.jl
We are working to finish the Boolean operations in Julia.  Plans for next year are:
Julia plasm server-side and Javascript client-side.
Let me know if you want more details …
—ap

On May 31, 2019, at 5:09 PM, alexlol76<EMAIL_ADDRESS>wrote:
Is it possible to realize solid lines?
Here is an example of a third-party application, and an example of how it should look in an application, are there any options, how can this be implemented? If so, what options and what needs to be done to do this?
https://user-images.githubusercontent.com/49237635/58715194-04973c80-83cf-11e9-89b3-d69114fb5e31.jpg
https://user-images.githubusercontent.com/49237635/58715192-03fea600-83cf-11e9-8b70-4d6b74eee4fc.jpg
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/cvdlab/react-planner/issues/174?email_source=notifications&email_token=AAG2COO2QQGH5RGODZDDRUTPYE5RHA5CNFSM4HR3SWD2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GW7L6DQ, or mute the thread https://github.com/notifications/unsubscribe-auth/AAG2CONX7DCN7IYMOVM6D3LPYE5RHANCNFSM4HR3SWDQ.

Hello,
Any news regarding this update?
Thanks!


[REQUEST] User can create a match without create teams

Hello!
Sometimes I want to create a quick scrim match with G5V but I have to create teams first to create a match. I think it is cool to make create a new match form with options a user can manually enter team name or use existing teams on the database. If you can help me now, I need the instruction to modify G5V and G5API (if required) to have this feature. I really appreciate all the help you gave me.
Thank!

Yeah thanks for your response. I'll close this issue.

Hi,
Considering get5 is not supported in CS2, there may be other cvars you would have to use depending on what plugin is being used. You'd need to check the server logs and see what they say.
Thanks

Hey, thanks for ultrafast response! I am using only MatchZy plugin.
 Executing Warmup CFG from MatchZy/warmup.cfg
[MatchZy] [UpdatePlayersMap] CCSPlayerController count: 1 matchModeOnly: False
[MatchZy] [UpdatePlayersMap] CCSPlayerController count: 1, RealPlayersCount: 0
[MatchZy] [LoadMatchFromJSON] Success with matchid: 5!

Seems like the command is being sent to the server but has no effect.. Do you have any suggestion what might be not working?
Really would appreciate. Thanks

As I said, get5_check_auths was a cvar directly associated with get5. I've never used MatchZy so I don't know if it allows 5v5 without teams being predefined or not. You will have to ask on their support channels to see what values are needed to allow for an authless match.
Thanks!

Appreciate for explaining! Thanks


Reenable the BornAgain Wrapper and Calculator

The goal is to get the existing tests to pass

This is parked as the present version of the python API no longer exposes some key functionality that we use.  At a later stage we might want to reach out to the BornAgain team.


The jsx-closing-bracket-location rule for multi-line props

The jsx-closing-bracket-location rule incorrectly warns for the following.
<Page multiLineProp={{
  one: 1,
  two: 2,
}}> // The jsx-closing-bracket-location rule warns on this closing bracket.
  {content}
</Page>

The jsx-closing-bracket-location rule wants the closing bracket for the opening tag to be on the next line, like this.
<Page multiLineProp={{
  one: 1,
  two: 2,
}}
>
  {content}
</Page>

This is the case for both line-aligned and tag-aligned. My opinion is that this should either be valid for both settings, or be exposed as a separate option.
Making the above example valid would result in including additional props before the closing tag also being valid. This may not be desirable for some, so it may be best for it to be a new option.
<Page multiLineProp={{
  one: 1,
  two: 2,
}} otherProp="Hmmm...">
  {content}
</Page>

I'm not sure why that should be valid, to be honest.


Bug in brushing items

如果把只适用于四腿的护甲，或只适用于无腿的护甲，快速用shift+左键穿戴时（我是手速非常快的人），有一定几率会出现2个一模一样物品，而且还是真的物品，不是虚的物品。
If the armor is only applicable to four legged armor, or only applicable to legless armor, when you quickly use shift+left button to wear it (I am a person with very fast hand speed), there is a certain probability that two identical items will appear, which are real items, not virtual items.
Video:
https://github.com/user-attachments/assets/da817abe-b158-4145-a4c5-bbc5b4f90dbf
https://github.com/user-attachments/assets/18a223b3-7e00-4280-879b-314e43ae0eb2

该Bug在生存模式适用
This bug is applicable in survival mode

Should be fixed in f747653603a43bcdc035b9fa5ca882550af547e9


Remove ContinueWith for ReenterAfter

https://medium.com/@joni2nja/why-you-should-not-use-continuewith-in-your-async-code-c9eaf6087e64
Not sure if this really matters in the end, but let's play nice

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


Add github actions

We want to add github actions so that we can run linting and testing on each commit

[x] Deployment
[ ] Linting
[ ] Testing

Deployment action added
https://github.com/ajitsinghkaler/devto-clone/commit/49d782eca08f3086bbc184582aaca781ba28ba98


Display issue for new comments

On mobile devices, we give second and third level replies less margin-left compared to larger screens:

However, when entering the comment, these rules don't appear to be respected. The comment box should be aligned with where the comment will show up.

There is also a secondary issue when replying to 3rd level comments. A reply to a 3rd level comment will show as a 3rd level comment as well. Therefore, that reply box should be shown in the same place as where the comment will end up. However, the reply box is currently indented to level 4.

Assigning to @sathomas for fix

Note that we've already had a ticket for this in the past, so it looks like we might have regressed here: https://github.com/oaeproject/3akai-ux/pull/3210#issuecomment-27379187

I think this matches the requirements. To @nicolaasmatthijs to check.

There are still some functional issues with this:

The input box for 4th level replies is still indented, whilst the actual 4th level reply comment is not. 4th level replies (and beyond) should all be shown at level 3, including the input box. Not that this is working as expected on mobile devices, but not on larger devices.

The new comment box is now cut off on the right side

Finished review. Back to @sathomas for follow-up.

In the comment picture above (https://cloud.githubusercontent.com/assets/109850/5741000/2dbda6e8-9bbd-11e4-85d6-2188633395b0.png) the top padding above Your Comment seems excessive, making it look asymetric to me, not sure if that's intended.

That was a lot of CSS pain just because Bootstrap 3 added one overflow: hidden. Back to @nicolaasmatthijs to check

There is a remaining issue with this. When the x time ago indication is long enough and the actual comment is pushed down, the comment shows below the profile picture, whilst it should be showing to the right of it.

Back to @sathomas for follow-up

tweaked the margin setting; back to @nicolaasmatthijs


[BrokenLinksH2] Fix path in link

Docs Build status updates of commit eff4607:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav-app/LocalFunctionality/Switzerland/swiss-value-added-tax.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Library only supports one row

The examples do not work properly. Color of the display can be set, but text does not appear.
When setting up the display as one row, the text is being displayed:
    lcd.begin(16, 1);
But then only one row is usable and the contrast of the displayed text is very low.

I ran into this issue, and I'm using the Grove shield, it was switched to 3.3v instead of 5v (there's a small switch on the side beside the reset button).

Yes thank you bigkevmcd, this was it. It actually is mentioned on the wiki page of the library but that notice slipped my attention.


Impala DVT connection does not allow use-ssl parameter that is allowed in impyla under the hood.

The use-ssl parameter is not included in the DVT connection parameters, however, the Impala connection I am working with today is kerberized and requires SSL flag to be true. I performed the following to get this to work. Plan to eventually submit a pull request once I get some time, but for now, here is the solution:

Missing python library: sudo pip3 install kerberos
Used port 21050 instead of 21000 (21000 works for CLI impala shell)
Update DVT source code to accomodate new parameter use_ssl

Add following lines of code after Line 35, 42,and 48 respectively:
professional-services-data-validator/third_party/ibis/ibis_impala/api.py
after line 35: use_ssl=False
after line 42: use_ssl = (use_ssl, False)[use_ssl is None]
after line 48: use_ssl=use_ssl,

Add the following line of code after Line 131 in:
professional-services-data-validator/data_validation/cli_tools.py
after line 131: ["use_ssl", "Use SSL when connecting to HiveServer2 (default is False)"],

cd to professional-services-data-validator folder
Then run command to build new code: sudo python3 -m pip install .

Now this connection should work:
data-validation connections add --connection-name <Conneciton_Name> Impala --host  --auth-mechanism GSSAPI --kerberos-service-name impala --port 21050 --use-ssl True

closed by PR #790


How to activate lsp source in another filetype

Hey @hrsh7th , I'm starting to use luapad again, and I noticed that completion source is disabled . The filetype is lua.luapad, I tried adding the file type to nvim_lsp setup function:
  require'nvim_lsp'.sumneko_lua.setup({
    on_attach = on_attach,
    filetypes = {'lua', 'lua.luapad'}})

But It didn't fix the issue, so I guess it has something to do with nvim_compe, vsnip source and path sources works fine though.
Thanks

nvim-compe will detects LSP Client via vim.lsp.buf_get_clients.
So I think the case does not activated language server on the luapad filetype.

Thanks


optional labels for kibana ingress

I am running multiple IngressControllers and I need to add labels to my Ingress objects to select which controller to use.
Config example for values.yml:
ingress:
  enabled: true
  annotations:
    ingress.kubernetes.io/protocol: https
  labels:
    my-ingress: internal

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

Hi @paltryeffort, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in your Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

I think you need to add a default for that to the values.yaml file as well?

Hi @paltryeffort,
Thanks for your PR.
Can you sign the CLA: https://www.elastic.co/contributor-agreement

jenkins test this please


[5.2] Add seeAuthentication test methods

The seeAuthentication method is useful to test login forms, the dontSeeAuthentication is useful to test logout forms and you can use seeIsAuthenticatedAs to test masquerade functionalities (for example check that an administrator was logged in as a particular user).

Example ^_^
    public function testBasicAuthenticationExample()
    {
        $user = new User();

        Auth::guard('web')->login($user);

        $this->visit('/')
             ->see('Laravel 5')
             ->seeIsAuthenticated()
             ->seeIsAuthenticatedAs($user);
    }

@GrahamCampbell I am not so sure if seeIsAuthenticated should receive a parameter. Because:
seeIsAuthenticated("web") doesn't sound right? seeIsAuthenticatedWith("web") sounds better.
But then seeIsAuthenticatedWithAs is worse?
I also didn't want to add too many methods.

Pretty cool! I'm wondering if the see verb is required here.
Perhaps $this->isAuthenticated(); for example. Authentication is only really visible if you have the UX for it, after all. :)

@daylerees you are right and I thought about the same, but isAuthenticated looks more like a method that will return a boolean, and not an assertion.
Some alternatives could be:
assertAuthentication (+2000 results in Google)
assertAuthenticated (+250 results)
or assertIsAuthenticated (less than 10 results)
expectAuthentication could work as well.
If only we knew an expert naming functions and methods...
(ping @taylorotwell)

Perhaps beAuthenticated.

@daylerees again, that sounds more like a requirement than an assertion.

shouldBeAuthenticated?

@GrahamCampbell +1 That sounds fine.

I'm not totally sold on the `seeIsAuthenticated* terminology. Not bad. Thinking on it.

@taylorotwell  expect* or should* seems to be the best prefix proposals so far.
expectAuthentication or shouldBeAuthenticated unless you come up with a better suggestion :)

@sileence  Yay merged!

Awesome! @taylorotwell are we going to keep seeIsAuthenticated?


user settings: Fix user avatar not updated in settings page.

On uploading new avatar, user avatar in settings page doesn't
get updated. Cause we have set src attribute of wrong html
element instead of image element in user settings page.
Fixes #8680

Nice, merged, thanks @YJDave!
@rheaparekh FYI -- this probably was a regression introduced in #8225; I guess we both missed this when testing.

@timabbott Ahh I see where I went wrong, I'll be careful.


Please give us more visual control over the Explorer (root folder presentation, font size, indent of the first item, and line height).

There have been several feature requests here for adjustments to the Explorer. Most of them seem to have been closed and dismissed (aside from workbench.tree.indent, which has provided an immense improvement).
I switched to VS Code from PyCharm, and though there are a few features I miss, I've generally been very happy with VS Code.
But I opened PyCharm the other day and immediately noticed how much more comfortable I was with the Explorer window. I worked with it for a couple days (after reassigning keyboard shortcuts to be more similar to VS Code), and though I still prefer VS Code, the PyCharm explorer was much more comfortable. I'm honestly not completely sure why this is. But there are several things I noticed that seemed to be important:

The root directory is selectable and can be interacted with as if it were any other folder in the project.
The font size is smaller.
The line height is greater than it is in the editor. (I think this might be important, as it makes the window appear to be less cluttered.)
There's a little bit less horizontal space between the expansion arrow icon and the folder icon. (I think this might be less impactful than I think it is. I read another issue about the alignment of the arrows, and I might be unconsciously attributing importance to this because I read that issue.)

Again, I'm not sure why the PyCharm explorer is so much easier to work with than the explorer in VS Code. I think that issues 1-3 would address this. (4 might be important, too. I don't know.)
I understand why you might think this is unimportant. When you're working on anything, it's easy to get used to the patterns you're familiar with. Every user has a different subjective experience of the interface and the layout, and like the designers, we all get used to what we're accustomed to working with, and we might not recognize there's anything that could use improvement until we experience something that's different.
Anyway, I think the user experience could be vastly improved by giving us a little more control over the visual presentation of the Explorer.

The root directory is selectable and can be interacted with as if it were any other folder in the project.

We support this in workspaces, see:

2 and 3 are fair feature requests.


CLI is sometimes sending a function upload retry even if the request succeeded

This is only happening sometimes and noticed during investigating an error thrown by API during upload
The request fails the first time, it is then retried and returns a 200 OK. A 3rd request comes in which fails because the function was already updated. All 3 requests are milliseconds apart. Deploy finishes correctly.
Same behavior here.
All deploys are CLI deploys. There are probably more examples to be found going through this list.
Expected behavior: request is not retried if it is successful

Thanks @ingride, we would need to dive into the code here:
https://github.com/netlify/js-client/blob/4cb1f240bac527305ff98032a801accf22dea5bf/src/deploy/upload-files.js#L97
to make sure the retry mechanism is working as expected.

Note that all api.* methods (including the ones used during files upload) also have some generic retry logic here.
We are currently using node-fetch under the hood. I am wondering if switching to got would make sense from that angle, since they have some pretty advanced retry logic out-of-the-box.

I'll try to get a consistent reproduction. Was planning to do some stress testing on the CLI deploy as a part of https://github.com/netlify/cli/issues/745 and https://github.com/netlify/cli/issues/484 so it might be useful here too.

FYI, wrote a utility to seed a site with files and functions:
https://github.com/erezrokah/netlify-build-reproductions/blob/03310ae888957df6e6c971d1852c7b3c02be6991/seed.js#L93
Usage (all arguments are optional except --siteDir):
node seed.js --siteDir=public --fileCount=1000 --minSize=10 --maxSize=20 --depth=5 --functionsDir=functions --functionCount=10 --functionSize=1


addLocationNodeWithConfirmedLocation adds node to current location not the target location.

let coordinate = CLLocationCoordinate2D(latitude: 33.697950, longitude: 72.970916)
let location = CLLocation(coordinate: coordinate, altitude: 19.39)
let image = UIImage(named: "pin")!
let annotationNode = LocationAnnotationNode(location: location, image: image)
//annotationNode.scaleRelativeToDistance = true
self.sceneLocationView.addLocationNodeWithConfirmedLocation(locationNode: annotationNode)
it sets the "annotationNode" to current place on scene not the destination place of given latitude longitude.

@yasirIqbal11 - I have some questions for you:

Are you using this as a library in your app or just playing with the example app?
Have you tried using a different image so you can distinguish it from other pins?
Are you still having this issue?

Hello @intere
I'm also facing same issue, what @yasirIqbal11 said
please advice as i'm using this as Library in my project and using different views(labels) for different nodes
Also i wanted to mention that Annotation nodes are moving when we move device.
AnnotationNodes should be stick to the latitude, longitude, altitude.
func loadPlaceNotes() {
    var annotationNodes: [LocationAnnotationNode] = []
    // PlaceNoteManager.shared.getPlaceNotes() will fetch my data(lat, long, alt, text etc)
    for aPlace in PlaceNoteManager.shared.getPlaceNotes() {
        let coordinate = CLLocationCoordinate2D(latitude: (aPlace.location?.latitude)!, longitude: (aPlace.location?.longitude)!)
        let location = CLLocation(coordinate: coordinate, altitude: (aPlace.location?.altitude)!)

        let aLabel = NoteLabel(frame: CGRect(x: 0, y: 0, width: 200, height: 60))
        aLabel.text = aPlace.note?.text
        let aNode = LocationAnnotationNode(location: location, view: aLabel)

        annotationNodes.append(aNode)
    }

    sceneLocationView.addLocationNodesWithConfirmedLocation(locationNodes: annotationNodes)

}
Please advice asap

@yasirIqbal11 - I have some questions for you:

Are you using this as a library in your app or just playing with the example app?
Have you tried using a different image so you can distinguish it from other pins?
Are you still having this issue?

Please see my question and advice

closing due to inactivity.

@yasirIqbal11. please make sure that "Privacy - Location When In Use Usage Description" is set in your info.plist


Redirect loop on php built-in server

I try to run your bedrock solution with the built-in php server on development env.
My .env file:
DB_NAME=test
DB_USER=root
DB_PASSWORD=
DB_HOST=localhost

WP_ENV=development
WP_HOME=http://localhost:8000/
WP_SITEURL=http://localhost:8000/wp/
AUTH_KEY='xxxx'
SECURE_AUTH_KEY='xxxx'
LOGGED_IN_KEY='xxxx'
SECURE_AUTH_SALT='xxxx'
LOGGED_IN_SALT='xxxx'
NONCE_SALT='xxxx'

And the server command:
php -S localhost:8000 web/index.php

If I go on http://localhost:8000/wp, I'm having a redirect loop on install.php.
If I try http://localhost:8000, I'm simply redirected on http://localhost (apache) for no reason.
Can you reproduce the issue?
Tip: Have a foreman / built-in dev server solution could be a nice addition. :+1:

Ok found why. Just add the -t option to specify root directory:
php -S localhost:8000 -t web

Are you interested for a PR managing built-in server and foreman?
Thanks.

The built-in server is awesome for quick stuff but we try and recommend dev/prod parity by using the same web server across environments (see https://github.com/roots/bedrock-ansible).
I'd like to get the Wiki going with some topics so that would be a good place to document this.
And thanks for posting the solution :+1:

I'm actually just getting a white screen.
Running from php -S <IP_ADDRESS>:8000 -t web it just returns white screen. Any tips?

@joshuairl Seems to be another error.
Have you any log? You better ask your question here to get support I think: https://discourse.roots.io/c/bedrock :+1:


Update metadata_table argument of create_ome_zarr_multiplex.py

Ref:

https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/399

We should remove the Union from metadata_table.
What about splitting into two arguments as shown below? (@jluethi)
  @validate_arguments
  def create_ome_zarr_multiplex(
      *,
      input_paths: Sequence[str],
      output_path: str,
      metadata: Dict[str, Any],
      image_extension: str = "tif",
      image_glob_patterns: Optional[list[str]] = None,
      allowed_channels: Dict[str, list[OmeroChannel]],
      num_levels: int = 5,
      coarsening_xy: int = 2,
      default_metadata_table: bool = True,
      metadata_table_files: Optional[dict[str, str]] = None
  ) -> Dict[str, Any]:

Then we would implement a check that the two new arguments are mutually exclusive (either you set default_metadata_table=True or you provide a non-None metadata_table_files is not None, but not both at the same time).

Let's simplify it a bit further and drop the default_metadata_table & mutually exclusive part.
Just: metadata_table_files: Optional[dict[str, str]] = None
If metadata_table_files is None => default behavior.
If a user provides them => load the specific metadata tables instead of parsing the metadata.

Agreed.
Should we also rename metadata_table (in create_ome_zarr) to metadata_table_file (with the same logic)?


Feature/update codecgen

Updating the github.com/ugorji/go/codec dependency to the latest and regenerates the keys.[generated].go file with the updated codecgen executable.

@andrewmeissner Could you format your commit titles to something like
glide: update github.com/ugorji/go/codec
client: regenerate

And does client build after your patch?

@gyuho I updated the commit messages.  I ran a go build -o client *.go inside the client dir and it seemed to build just fine.  If I need to build it another way let me know and I'll do that as well.

@andrewmeissner It doesn't build in our CIs. Did you run ./scripts/updatedep.sh in your first commit?

@gyuho I was not aware (hahaha).  Just ran it and committed the files it modified.  :)

Thanks! We will merge this when CI passes.

@ugorji We had to manually patch the generated code to pass our linting tests. Please see last two commits if interested. Thanks!

@gyuho thanks much for the heads up. I will look into why the redundant stuff was there, and the yhnn2=0 one also.


Update sbt to 1.10.5 in main

About this PR
📦 Updates org.scala-sbt:sbt from 1.10.2 to 1.10.5
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (1.10.2).
You might want to review and update them manually.
package-lock.json
src/test/scala/io/iohk/atala/prism/node/NodeServiceSpec.scala

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-sbt", artifactId = "sbt" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:early-semver, old-version-remains, commit-count:1

Superseded by #876.


Breakline not turning false working for texts with different font styles

Category
Bug
Version
Please specify what version of the library you are using: [  3.10.0 ]
Expected / Desired Behavior / Question
When texts with multiple styling are added, the text shouldn't break to the next line by default
Observed Behavior
When texts with same styling are added, the text works fine but when multiple styling are added the texts breaks to a new line.
Steps to Reproduce
Here is a Jsfiddle to recreate the bug. As you see, I have also specified breakLine: false.

Is it the default behaviour that texts with "\n" inside the text strings sets breakLine: true when the mutiline  text format is used?


[URGENT] Update Facebook SDK to version 4.37.0

Hi,
I'm unable to login into my App using Facebook API because this library's SDK is out of date. In the output console the following message appears: "Your Facebook SDK is out of date. We recommend upgrading to the latest Version 4.37.0, to ensure your app's performance is not affected and to take advantage of our newest features!".
Please, could you update this library to the latest SDK? Thank you in advance.

Not encountering the same issue but Facebook contacted me and said we're in violation of Platform Policy 8.2, otherwise the app will be subject to enforcement:

Please make sure your app is using the most recent version of our SDK for Login. You can find more information on our SDK for Login and other Login-related products here: https://developers.facebook.com/docs/facebook-login.

I was about to build manually from source but realized it's only on 4.36.0.

Not encountering the same issue but Facebook contacted me and said we're in violation of Platform Policy 8.2, otherwise the app will be subject to enforcement:

Please make sure your app is using the most recent version of our SDK for Login. You can find more information on our SDK for Login and other Login-related products here: https://developers.facebook.com/docs/facebook-login.

I was about to build manually from source but realized it's only on 4.36.0.

Which version are you using currently?

@peppe130 currently using the latest version from NuGet, 4.34.0, which is using the recently deprecated API v2.7
https://www.nuget.org/packages/Xamarin.Facebook.Android/
https://developers.facebook.com/docs/graph-api/changelog/version2.7

I don't believe 4.36 vs 4.37 is the issue here.  According to facebook's docs, API 3.0 will only start to be required as of Jan 8, 2019.
The error suggests updating to the latest version of the SDK which is 4.37 and we will work to release ASAP, however I don't believe in either of these cases that it alone will fix your issues.
Any more information you can get out of the policy violation details from your app account would be helpful here.

Thanks @Redth. I personally didn't encounter the login issue @peppe130 posted, our logins still work just fine. However, our app is in danger of being marked for removal due to the SDK version violation.
Not sure how common these checks happen but the email I received from Facebook doesn't appear to be automated and I was able to request for an extension from the case manager by sending them links to similar SDK update requests in this repo.

@marcobaldo not sure what version you're using, but according to Facebook's docs, 4.35 is using Graph API 3.1 which means our current stable version should be using 3.1 as well.  One thing I've noticed in my own apps is having to go into the Facebook developer console and change the version my particular application was using there.  Perhaps that's something to investigate?

@peppe130 @marcobaldo We have pushed out an update for Android to 3.7 and iOS will be updated next week.
Regards

@peppe130 @marcobaldo We have pushed out an update for Android to 3.7 and iOS will be updated next week.
Regards

Thank you so much man.

Again, this issue should not be closed!!


Grouping of fixes for 7.3.2

Replaces #19297, #19299 and #19301

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.

Also see the follow up #19311.


Update New-SPSiteMaster.md

Filled in placeholder text.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/New-SPSiteMaster.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


README.md Update

Deleted already done to-do and also added the tower floors crash faq
Feel free to close this pr if its unneeded

Merged thanks 🙂


feat: bundle into moduleJS and split chunk

why I did

commonJS deprecate in Vite

what I did

bundle file into moduleJS
split chunk and lazy load page

result

faster build from 5 sec to 2 sec

lazy load chunk

must be squash and merge