Support for meta tag robots field

Hi,
thanks for this great package!
I have made changes that allow the robots meta tag to be handled from filament as well.
I also modified the tests by predicting the new field.
I hope it can be useful.
Thanks

Works great, thank you very much! Just released in 1.4.0.


Run d.ReadAndDetect as goroutine, stop it when I quit the detecting

@brucegao77 if there is still an issue, feel free to open an issue with clear steps to reproduce and I can take a look at it.

Thanks for your project, it's very helpful!
I'm a new gopher and I had a same issue with #4.
But in my case, two of your solutions didn’t work.
I finally sovled the problem by using os.Exit(0) as HandleFunc().
Yes, when I finished record, I use cmd.Run() to start it again...


Maybe alert users of reduced WebAuthn support in Firefox.

Firefox has yet to support mac Touch ID and PassKeys. It may be worth providing a notice somewhere speaking to that fact.
When you try to configure a key or log in, you'll get this unhelpful dialogue:

Is this resolved? I have problem with Firefox on Linux, my Yubikey is not more working with the two factor plugin on my Wordpress install. (Not that other website like Github works well in Firefox with the same key, so it's really plugin related).
I was about to give a try to this plugin but then saw this issue. What's going on with WebAuth and Firefox?

This should be resolved in v2.5 of the plugin via #874


Add Helm commands cheat sheet

FIXED in https://github.com/eficode-academy/helm-katas/commit/36cb713e9bb3b1c6493baf10d67eac7d6c668123


Change ValueError to UserWarning when level is > dwt_max_level

For consistency with Matlab and with old (~3 years prior) releases of PyWavelets, do not raise an error if the user sets level in wavedec* to a value greater than that returned by dwt_max_level.
At the moment, I have left a UserWarning if no coefficients will be unaffected by the signal boundaries to help guide users toward a reasonable number of levels in typical applications.
closes #396

Codecov Report

Merging #403 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #403   +/-   ##
=======================================
  Coverage   84.52%   84.52%
=======================================
  Files          22       22
  Lines        3600     3600
  Branches      627      627
=======================================
  Hits         3043     3043
+ Misses        490      489    -1
- Partials       67       68    +1

Impacted Files
Coverage Δ

pywt/_multilevel.py
92.75% <100%> (ø)
:arrow_up:

pywt/_wavelet_packets.py
77.57% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 233c5a6...abcc4a2. Read the comment docs.

I don't see an issue with that. Merged. Thanks @grlee77


Implement auto-completion script on stdout or custom destination

Is your feature request related to a problem? Please describe.
When packaging (as I am for Arch Linux) CLI-tools it's often appreciated by users to have completions automatically installed. This is not doable in the clean way with jfrog-cli, as there is no way to produce a completion script in a location different than the one in the users home directory (that I know of).
Describe the solution you'd like to see
The jfrog completion commands would take a flag that will output the completion script either on stdout or in a file with a settable location. This would allow me to install completion alongside the program in my package without touching the users home directory.
For example:
$ jfrog completion -o some_location bash
$ cat some_location
#!/bin/bash
_jfrog() {
    local cur opts base
    COMPREPLY=()
    cur="${COMP_WORDS[COMP_CWORD]}"
    opts=$( ${COMP_WORDS[@]:0:$COMP_CWORD} --generate-bash-completion )
    COMPREPLY=( $(compgen -W "${opts}" -- ${cur}) )
}

complete -F _jfrog -o default jfrog
complete -F _jfrog -o default jf

Or:
$ jfrog completion --stdout bash
#!/bin/bash
_jfrog() {
    local cur opts base
    COMPREPLY=()
    cur="${COMP_WORDS[COMP_CWORD]}"
    opts=$( ${COMP_WORDS[@]:0:$COMP_CWORD} --generate-bash-completion )
    COMPREPLY=( $(compgen -W "${opts}" -- ${cur}) )
}

complete -F _jfrog -o default jfrog
complete -F _jfrog -o default jf

Thanks!

Thanks for sharing this requirement @carlsmedstad.
Can you share with us the scenario in which you wouldn't want the competition script to be stored under the user home directory. You mentioned "when packing CLI tools". Can you elaborate more about that?

@carlsmedstad,
I think https://github.com/jfrog/jfrog-cli/pull/1426 should allow this by changing the default behavior of jfrog completion ... commands - instead of installing the completion scripts in ~/.jfrog/, it prints the completion script to the screen.

Thanks for the quick response!

Can you share with us the scenario in which you wouldn't want the competition script to be stored under the user home directory? You mentioned "when packing CLI tools". Can you elaborate more about that?

I'm maintaining a package for jfrog-cli in the AUR (Arch Linux User Repository), the definition of which can be found here. I would like to install completions as part of my package but it's bad form to touch the users home directory during installation. All installation steps should happen in an isolated directory.
So I would like to be able to add generation of the completion scripts to my package without touching the home directory of the person installing the package. The package will install the completion scripts globally to /usr/share/bash-completion/ and the corresponding directories for the other shells.

I think https://github.com/jfrog/jfrog-cli/pull/1426 should allow this by changing the default behavior of jfrog completion ... commands - instead of installing the completion scripts in ~/.jfrog/, it will print the completion script to the screen.

@yahavi Yes, this is precisely the change I'm looking for :+1:

Thanks for this information @carlsmedstad!
So it looks like @yahavi's #1426 is what's needed.

As of the latest version, my package now installs completions so I'm closing this issue.
Thank you @yahavi & @eyalbe4! Much appreciated.


Feature/logback auto refresh

Coverage increased (+20.9%) to 59.631% when pulling c8344ba82f6defc9f3e0fb8be1ba3f6a222eff94 on jenkinsci:feature/logback-auto-refresh into df7b805fa3be34d600866f351ff9f9be00b8f50a on maximecharron:master.


[FEAT]: Document Upload Api should return details

What would you like to see?
When uploading documents through the API, the response should return the details of the uploaded document. I would expect to see the same details that are returned from a Get request to /documents (expect for a single document).
It would also be useful if the document item in the response included the full path to the document, as it would be used in a call to update-embeddings.

https://github.com/Mintplex-Labs/anything-llm/issues/570


Release 1.8.0

Last minor release of Koalas:

Add a warning message about PySpark version, etc.
Remove pandas bounds (and other dependency bounds)

The release will be in early next week

Done


Multi Language Support

Can you add support to other languages ? Because I want to create a skill in german language
Can I translate the plugin with poedit ? Or what do I have to do in order to get a German skill

If you want to take an early look, the language branch is https://github.com/alleyinteractive/voicewp/tree/86-lang and contains the .pot file for translation
It'll probably get merged in tomorrow since it needs to go through a code review process first.
Doc updates to come.

My blog is multiside blog. And additinal it is translated with the transposh plugin. Is it possible in future to support such a constelation?
This means to make a skill available in the same blog for several languages?
For me was not clear how it realy will run because Developer accounts allow only to pass the test if i publish minimum once a week.
So my first try was not successful...


Physics Engine and Minecraft Engine differ in interpreting y axis

The y axis components in position packets in MC are of the Eye height, but because the physics engine just sends the y position, it is sending the foot position. This causes an illegal stance if you're successfully connected, or prevents you from successfully connecting if you don't send a correct packet to begin with.
The reason that the 1.7 bot was unable to navigate was because of this check on the server:
https://github.com/Bukkit/mc-dev/blob/master/net/minecraft/server/PlayerConnection.java#L94-L99
Because the stance was not correct the y axis didn't match what the server told us, so the server was never marking us as 'ready'
(Still at loading screen state is assumed when CheckPosition is false)
Currently navigation is working in the 1.7 feature branch on my repo, which is currently PR'd to @roblabla (https://github.com/roblabla/mineflayer/pull/2) so that it can be part of the PR to this repo. (https://github.com/andrewrk/mineflayer/pull/173)
The only reason I put this issue here is to bring attention to this issue and open discussion about how we should fix it.
Lines of concern regarding the current fix are here: https://github.com/nevercast/mineflayer/blob/feature-1.7/lib/plugins/physics.js#L328-L339

This is now "fixed" by the fact that the server sends the foot position in 1.8. (see d5d362c9f0599ccff00cfbf822334362bb544036 )
So is that issue fixed ?

Well, the client is still flying in the air. I'll close this when the physics engine starts working again :P

Ok :)

I think that issue is done ? (well of course the enums are not up to date, so there may be problem with some blocks but I don't think it concerns this issue)

Yup, the enums not being up to date isn't part of this issue. Closing !


Currying functions for performance gain

I went through all the files in src/*, and curried functions that were repeatedly used within a while or for loop outside of an if statement. I've tried to have some form of literate programming by maintaining the naming convention and appending _partial. I did not add comments, since it seemed somewhat straightforward what was going on. I would be more than happy to add comments if you will tell me what you would like me to explain. This should address #148 .

Ok, so I feel there may have been some misunderstanding somewhere!
The idea was to add these methods everywhere for users to call themselves:
GO.intersects(x) = Base.Fix1(GO.intersects, x)

As anshul demonstrated. This gives us a "curried" Fix1 objects to use in map and similar functions for the convenience and nice syntax. So we could do map(intersects(mygeom), othergeoms) to get a vector of Bool, rather than always having to do map(g -> intersects(g, mygeom), othergeoms). Does that make sense?
Currying anonymous functions explicitly like you have here can sometimes give a performance gain where it can help the compiler (for pretty arcane reasons), but I'm not sure it will be measureable in this case.

I think I understand what you are saying.
Just to be sure though. You want to use Base.Fix1 and Base.Fix2 to clean up and make the code more readable/semantic.
This leaves us with two options; build on top of what I already have, or remove it. Which would you prefer?

Hmm... not really!
Essentially I'm talking about a different PR to what you have! It would just define methods like this:
GO.intersects(x) = Base.Fix1(GO.intersects, x)
GO.disjoint(x) = Base.Fix1(GO.disjoint, x)
# etc etc

For all of the predicates in the geom_relations folder. It would also add tests for all of these in the related test files.

I didn’t get what you are saying from what they were saying. Also, I completely missed the part where you said draft PR. So, with your okay, I plan to do the following

Close this PR
Clone again or revert current repository
Open a draft PR
Start working on what I believe you want

Sound good?

First maybe write out in your own words what you think we want! Just to be sure there is no communication issue this time

Interesting. Thank you for that response. That gives me clarity. I feel this issue is not a good fit for me. You'll need to get someone else to work on this for you. Thank you for the opportunity. Best of luck to you.

No worries


fix DS3231_A1_Day and DS3231_A2_Day issue

Hi adafruit,
I modified DateTime::dayOfTheWeek and RTC_DS3231::adjust to reflect day-of-week register as specified on DS3231 and DS1307 (see discussion here). Unit tested on uno 328p (test file).

A possible limitation is that PCF8523 works with 0 to 6 for day-of-week.
Cheers,
Robson

As I stated in a comment to issue #193, this would be a breaking change, and breaking changes should be avoided on a published library many people rely on.
Would you mind testing the change I proposed in that issue?

Hi @edgar-bonet.
I like you solution better, thanks. I just tested your fork here and all tests passed.
One more thing, as mentioned before RTC_DS1307 uses 1-7 as well. That means that RTC_DS1307::adjust may need to use dowToDS3231 as well. I don't have that IC to test here though.
Let me know when you create the PR for your changes.
Cheers.

I just tested your fork here and all tests passed.

Cool. :-)

RTC_DS1307::adjust may need to use dowToDS3231 as well

I am not convinced it is worth changing the code for the DS1307, as that RTC doesn't have alarms, and computing the day of the week can take a non-trivial amount of CPU cycles. One use case I can imagine where it would be useful to write the day-of-week register is if RTClib is used to set an RTC which is then read by another library that does rely on that register. But it sounds like a convoluted scenario to me.

Let me know when you create the PR for your changes.

OK done: PR #195. I added one comment to explain why we are updating this register on this particular model of RTC:
// The RTC must know the day of the week for the weekly alarms to work.

Great. Cheers.


feat: separate commands by usage

This draft PR is based on a few principles of interaction, and loosely based off of nvm or update tools like it:

dev-cli use would only use local versions, warning if they need to be installed
dev-cli update will update to the latest version on the given channel, installing if needed
dev-cli install will install and link a new version (could be used instead of use, but it could fetch from the internet)

:tada: This PR is included in version 1.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v1.5.0
v1.5.0

Your semantic-release bot :package::rocket:


ripgrep: fix shell completions when cross compiling

Description of changes
https://github.com/NixOS/nixpkgs/pull/270521 fixed cross compilation but the result was missing shell completions. this patch fixes shell completions too.
$ nix build '.#ripgrep'
$ tree result
result
├── bin
│   └── rg
└── share
    ├── bash-completion
    │   └── completions
    │       └── rg.bash
    ├── fish
    │   └── vendor_completions.d
    │       └── rg.fish
    ├── man
    │   └── man1
    │       └── rg.1.gz
    └── zsh
        └── site-functions
            └── _rg

11 directories, 5 files

$ nix build '.#pkgsCross.aarch64-multiplatform.ripgrep'
$ tree result
result
├── bin
│   └── rg
└── share
    ├── bash-completion
    │   └── completions
    │       └── rg.bash
    ├── fish
    │   └── vendor_completions.d
    │       └── rg.fish
    ├── man
    │   └── man1
    │       └── rg.1.gz
    └── zsh
        └── site-functions
            └── _rg

11 directories, 5 files

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

combed through the output of nix log ./result this time to make sure there weren't any non-fatal errors.

The change in here looks good, however I spotted another issue: the installCheckPhase will never be run on cross builds (that's hard-wired in mkDerivation as I learned now and I'll open an issue for that).
I'm not keen on skipping a basic functionality test just because of some shortcomings in the current stdenv. A possible workaround which forces the face to be executed is
diff --git a/pkgs/tools/text/ripgrep/default.nix b/pkgs/tools/text/ripgrep/default.nix
index 8aa91640fd1f..b16b350757e4 100644
--- a/pkgs/tools/text/ripgrep/default.nix
+++ b/pkgs/tools/text/ripgrep/default.nix
@@ -42,7 +42,10 @@ in rustPlatform.buildRustPackage rec {
       --zsh <(${rg} --generate complete-zsh)
   '';

-  doInstallCheck = canRunRg;
+  postInstall = ''
+    export doInstallCheck=${toString canRunRg}
+  '';
+
   installCheckPhase = ''
     file="$(mktemp)"
     echo "abc\nbcd\ncde" > "$file"

Opinions?
cc @zowoq @globin @SuperSandro2000

Thanks!
And sorry for the long wait!


DrawMeshInstanced Bug in Unity SDK

Unity 2020~2021 version -  Leapmotion SDK  Editor Error fix
Capsule hand Graphic.DrawMeshInstanced()
Summary
Change the Unity function "DrawMeshInstanced()" to "DrawMesh()". Because There is constantly occured error log in editor  when run Leap motion sample "Capsule hand".
Error log

"DrawMeshInstanced does not support the shader 'Standard' because it does not read any instanced properties. Try switching to DrawMeshInstancedProcedural if the shader is doing procedural instancing"

According to Unity API document, GPU instancing function can be used by Graphic.DrawMesh() function when check "Enable GPU Instancing" checkBox. So there is no disadvantage about performance. After change function, there is no error log and work properly.
https://docs.unity.cn/2021.1/Documentation/Manual/GPUInstancing.html
Contributor Tasks

[x] Add any release testing considerations to the MR for the next release.
[x] Check any relevant CHANGELOG files have been updated.
[x] Ensure documentation requirements are met e.g., public API is commented.
[x] Consider any licensing/other legal implications for this MR e.g. notices required by any new libraries.

Reviewer Tasks
Add any instructions or tasks for the reviewer such as specific test considerations before this can be merged.
Use emojis in review threads to communicate intent and help contributors.

[ ] Code reviewed.
[ ] Non-code assets e.g. Unity assets/scenes reviewed.
[ ] Documentation has been reviewed. Includes checking documentation requirements are met and not missing e.g., public API is commented.
[ ] Checked and agree with release testing considerations added to MR for the next release.

Thanks for making a contribution - will review soon.

Hi beargrllys, can you add some more information about what you are doing when you see these errors please? Also which version of Unity are you using?

I used Unity 2021.3.11f1

An update - we need to look into other potential causes of this issue.

Hey, just as a note, Unity seems to have a known issue for this type of problem here
It seems to not actually be an issue, and instead just shows a false error that is clear-able.
@beargrllys did you find the error stopped you from seeing the capsule hands render, or was it just that there was an error in the console? :)

Closing this as we've had no response.


spelling error in readme.md

Fixing a spelling mistake in README.md

Hi @scuba323 ,
thanks for fixing it :)
Best regards,


beszel-agent: init at 0.7.4; beszel: 0.6.2 -> 0.7.4

This updates beszel to 0.7.4 and splits the agent in a separate package.
The beszel binary (the "hub", you can think of it as prometheus) bundles a web ui and weights in at some 37MB. While that's not much disk space in the grand scheme of things, it seems unnecessary to have that and the agent (think, prometheus node exporter) in the same package.
Since the beszel package was first merged only a few hours ago (see https://github.com/NixOS/nixpkgs/pull/345444) and didn't make into the stable branch yet, I don't think we should worry about the removal of the beszel-agent binary from that package (but IDK if there are official policies about that).
Tagging the most recent contributors from https://github.com/NixOS/nixpkgs/pull/345444:  @Aleksanaa @Bot-wxt1221
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

You don't fit CONTRIBUTION.md. Please do following things then I will review it:

format with nixfmt
remove with lib
split maintaiers commit into another committer

Webui is too dirty.

Thanks @Bot-wxt1221, and apologies: it's been a while for me and CONTRIBUTION.md must have been updated in the meantime (at least, I don't remember it saying anything about formatting).
I'll review your comments in a while, after re-reading CONTRIBUTION.md :)

@giorgiga The only diffrence between them is subPackages. We should define something like generic.ix.

@giorgiga

I suggest you to modify based on my code.
Actually only subPackages should be modified. We can define generic.nix in beszel.passthru and call it in beszel-agent.

@Bot-wxt1221  about your https://github.com/NixOS/nixpkgs/pull/354978#issuecomment-2466711499

You don't fit CONTRIBUTION.md.

I re-read CONTRIBUTION.md but I didn't find what I am infringing specifically. Can you elaborate?
(if it's that the addition of a new maintainer should be in its own commit, I am already in maintainers.nix).

Please do following things then I will review it:
format with nixfmt

Apologies, that wasn't a requirement last time I contributed.
Done using nixfmt-rfc-style.

remove with lib

I don't mind doint that and will do in the next iteration).
For future reference/my own self improvement, could you explain what's wrong with it?

split maintaiers commit into another committer

(see above: I am already a maintainer)

I suggest you to modify based on my code.
Actually only subPackages should be modified. We can define generic.nix in beszel.passthru and call it in beszel-agent.

Subpackages IMHO make sense for stuff that is usually installed as a bundle (eg. coreutils), but I don't see any benefit in this case. Also, I doubt most nix/nixos users even know about subpackages... (search.nixos.org does not show them in any way)
Can you elaborate the benefit of having subpackages?
If it's to share version and hashes in order to make updating (marginally) less cumbersome, could we add a third beszel-common.nix file somewhere instead? (does the nixpkgs directory layout allow for such thing?)
In general... I don't think it's bad to have a modicum of replication for the sake of simplicity.

(just realized I had committed the files for 0.7.3 - fixed)
@Bot-wxt1221 can you elaborate on the reasons behind the changes you requested?

(actually, I won't post a new PR)


AspectJ Gradle plugin

@kriegaex / Team, I am trying to introduce AspectJ to my gradle project, but couldn't do it so far with different plugins available for gradle. May I know if there is a recommended AspectJ plugin that needs to be used for the gradle project?
I am using gradle 8.4 on Java 17
Thanks
Ravi

This project is, as the name implies, a Maven plugin. And GitHub issues are not for support requests but for bugs or feature requests for that plugin. Please behave. You violated netiquette in multiple ways at once:

You abused a bug tracker for a support question.
You asked about a totally different tool. Gradle is not Maven.
you cross-posted your Stack Overflow question here.
Even on Stack Overflow, you hijacked a Maven question instead of searching for Gradle-related questions first and then maybe ask a new one.

Please learn some manners. Like this, you are not going to make friends.


add dev channel

@posriniv
Change to this file looks good to me. But I did not see your change to https://github.com/IBM/ibm-iam-operator/tree/master/deploy/olm-catalog/ibm-iam-operator


Split parameterBuilder from rewriteEngine

Ref #2405.

Pull Request Test Coverage Report for Build 8128

57 of 74 (77.03%)  changed or added relevant lines in 10 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.002%) to 57.754%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/SQLBuilder.java
4
5
80.0%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/wrapper/PreparedStatementExecutorWrapper.java
0
1
0.0%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/wrapper/StatementExecutorWrapper.java
0
1
0.0%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/ParameterBuilder.java
35
37
94.59%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/engine/ShardingSQLRewriteEngine.java
5
10
50.0%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/engine/MasterSlaveSQLRewriteEngine.java
0
7
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/SQLBuilder.java
1
74.29%

Totals

Change from base Build 8127:
0.002%

Covered Lines:
7854

Relevant Lines:
13599

💛  - Coveralls


Returning Version instead of Author & vise versa in Alerts

Under alertInfos() in alert.go, I found it return version, author. However, all alert services are calling the function author, version := t.alertInfos(), so we are getting incorrect values in all alert messages (Slack, Telegram, etc).
With Build Version Enabled:

Without Build Version Enabled:

Verified using latest Semaphore Develop Build

Fixed by #1934


Shrink size of react-mdl

Hey there,
First of all: Thanks for your work 👍
I just peeked into my node_modules folder and i found, that react-mdl takes up about ~45MB of disk space which is around 10% of my total node_modules folder. (the reason i looked)
Is it really necessary to have ALL css files in /extra installed by react-mdl or could /extra/css be put into the .npmignore?

Users could use custom css for custom styles so yes unfortunately this is required :/
With v2, this process will be a lot easier and users will be able to customize the styles using settings file instead.
Be patient ^^ ;)


Immer isn't updating draft value when it is reassigned directly

🐛 Bug Report
Immer does not update a value in the draft when it is set directly.
Link to repro
https://codesandbox.io/s/immer-playground-zp1um?file=/src/index.js
To Reproduce
Steps to reproduce the behavior:

Create a object x = { name: 'José' }
Assign to another variable the produce of x.
Try to replace all the draft:
y = produce(x, draft => { draft = { name: 'Carla' }  })

Observed behavior
The variable y is returning the object: { name: 'José' }
Expected behavior
The result of the produce should reproduce the changes in the draft. As it is reassigned to another object, then the expected return should be: { name: 'Carla' }.
Environment
We only accept bug reports against the latest Immer version.

Immer version: 6.0.3
[x] I filed this report against the latest version of Immer
[ ] Occurs with setUseProxies(true)
[ ] Occurs with setUseProxies(false) (ES5 only)

That is working as expected, as your construction re-binds the local javascript variable draft to something else. It is similar to how let i = 3; i = 7; doesn't suddenly make 3 a 7 but only rebinds the local variable i.
Please read https://immerjs.github.io/immer/docs/return for how to replace the current state with new state.

@mweststrate i got it. Apologizes for this. Thanks a lot.

https://immerjs.github.io/immer/docs/pitfalls#dont-reassign-the-recipe-argument


[mtl] hangs on background/foregrounding in [CAMetalLayer nextDrawable]

Short info header:

GFX version: ll - rev: "bd7f058efe78d7626a1cc6fbc0c1d3702fb6d2e7"
OS: macos (probly ios too)
GPU: Radeon Pro 455 2048 MB

While an OSX app is backgrounded, attempting to call into -[CAMetalLayer nextDrawable] causes a hang (~1sec presumably a timeout).
The incorrect synchronization in the quad example masks the issue a bit. The hang is more obvious in this gfx-hal-tutorial example.
command_queues[0].submit(submission, Some(in_flight_fence)); // hangs sometimes

You would think that disabling drawing after the Focused(false) event is sent would fix the issue, but, sometimes the backgrounding event races with command buffer submission. After the hang occurs, then the Focused(false) event gets received - sometimes.
Apple really really wants you to draw only when they tell you to draw. There are a few options for being notified that OSX is expecting a new frame.

CVDisplayLink/CADisplayLink api - cube example from moltenvk

this has the downside of running in another thread on OSX

NSViewDelegate::draw/NSView::drawRect / UIViewDelegate::draw / UIView::drawRect

glfw example
metalkit tutorial

NSView::updateLayer - glfw

I hacked up a CVDisplayLink impl to test and verified that the render thread was no longer hanging.
I believe the correct place for a fix should be in winit, but I don't see how it would work the current API.
Posting this here in case anybody has some opinions or ideas.

Thank you for such a detailed issue! I believe it's still our responsibility to avoid requesting the next drawable if we know that the metal layer is hidden. We should be able to check for it at the surface level, perhaps returning SurfaceLost error.

FYI I'm currently investigating the same issue in my experimental multi-window branch in sokol_app.h. When one of the windows is becoming obscured, the main thread stops for somewhere between half a second and a full second in [MTKView currentRenderPassDescriptor] (which presumably is calling [CAMetalLayer nextDrawable], which causes the freeze).
I guess the problem also exists with a single window, but it's not noticeable when rendering stops on an obscured window.
One strange thing I'm seeing in Instruments is, that the pause seems to be related to "vsync requests" stopping:

Those "vsync requests" happen in 16.6 ms intervals, until a window becomes obscured, than the vsync requests seems to "dry out" for a while, until they start happening at regular intervals again after usually around a second. Since no new drawables are presented, new drawables get stuck waiting to be presented, causing the whole thread to stop in [nextDrawable].
I haven't found a solution so far, it doesn't seem to help to skip drawing when a window is obscured (via NSWindow.occlusionState), it seems it's already too late when this triggers to "occluded" :/

Some new info before I file this problem under "known issue" and move on:
The short freeze even happens in the official Xcode "Game" example (which uses MTKView, but MTKView isn't the problem, but [CAMetalLayer nextDrawable].
To reproduce: add the following lines around the [MTKView currentRenderPassDescriptor] (which calls nextDrawable under the hood):

...then run the sample, and occlude its window with another window. Each time the window is fully occluded, the logging output will stop with the "before..."
TBH I'm not sure now if this isn't a bug in macOS that should be fixed by Apple (I'm on the 11.4 Beta (20F5046g)).

I’ve been banging my head against a wall the past few weeks trying on and off to solve a similar issue (nextDrawable hanging for one second after entering or exiting full screen), and I finally found the solution: stop your own render loop while the window is in transition. Instead, let the OS tell you when to draw using needsDisplayOnBoundsChange so that the window remains updated during e.g. custom full screen animations.
In my window delegate I have something like the following:
- (void)windowWillEnterFullScreen:(NSNotification *)notification
{
	[view stopDisplayLink];
}

- (void)windowDidEnterFullScreen:(NSNotification *)notification
{
	[view startDisplayLink];
}

- (void)windowWillExitFullScreen:(NSNotification *)notification
{
	[view stopDisplayLink];
}

- (void)windowDidExitFullScreen:(NSNotification *)notification
{
	[view startDisplayLink];
}

which corresponds to the following in my view:
- (void)startDisplayLink
{
	CVDisplayLinkStart(displayLink);
}

- (void)stopDisplayLink
{
	CVDisplayLinkStop(displayLink);
}

FWIW I haven’t been able to reproduce memory leaks / hangs after re-focusing the window following a period of occlusion.

I ran into the same issue and we solved it in a similar way to @lunacookies:
// in our NSWindowDelegate
- (void)windowDidChangeOcclusionState:(NSNotification*)note {
	if (_displayLink) {
		if (self.occlusionState & NSApplicationOcclusionStateVisible) {
			CVDisplayLinkStart(_displayLink);
		} else {
			CVDisplayLinkStop(_displayLink);
		}
	}
}


Remove fixed position default

As suggested by @andrewzey in https://github.com/benwiley4000/react-responsive-audio-player/issues/6, the default position: fixed; bottom: 0 style applied to the .audio_player element makes the component less modular/portable. Those styles are now removed, and instead one may either apply those or similar styles via CSS, or by passing a React style object as a prop to the component.
e.g.
<AudioPlayer
  playlist={playlist}
  style={{ position: 'fixed', bottom: 0 }} />

Thanks!


UIBULKED-14 Show columns on the preview of records matched

Purpose

https://issues.folio.org/browse/UIBULKED-14
Approach

TODOS and Open Questions

Learning

Pre-Merge Checklist
Before merging this PR, please go through the following list and take appropriate actions.

[x] I've added appropriate record to the CHANGELOG.md
Does this PR meet or exceed the expected quality standards?

[x] Code coverage on new code is 80% or greater
[x] Duplications on new code is 3% or less
[x] There are no major code smells or security issues

Does this introduce breaking changes?

[x] If any API-related changes - okapi interfaces and permissions are reviewed/changed correspondingly
[x] There are no breaking changes in this PR.

If there are breaking changes, please STOP and consider the following:

What other modules will these changes impact?
Do JIRAs exist to update the impacted modules?

[ ] If not, please create them
[ ] Do they contain the appropriate level of detail?  Which endpoints/schemas changed, etc.
[ ] Do they have all they appropriate links to blocked/related issues?

Are the JIRAs under active development?

[ ] If not, contact the project's PO and make sure they're aware of the urgency.

Do PRs exist for these changes?

[ ] If so, have they been approved?

Ideally all of the PRs involved in breaking changes would be merged in the same day to avoid breaking the folio-testing environment.  Communication is paramount if that is to be achieved, especially as the number of intermodule and inter-team dependencies increase.
While it's helpful for reviewers to help identify potential problems, ensuring that it's safe to merge is ultimately the responsibility of the PR assignee.

Test errors found. See https://jenkins-aws.indexdata.com/job/folio-org/job/ui-bulk-edit/job/PR-31/1/


Display actual values in preferences

Hi,
Don't you think it's better to display the actual values without forcing the user "click" on the item?
(i only updated the English version of strings.xml. But I think even with the original text, It is fine.)
redge

I thought about it when I originally implemented the app. I think summary info is more useful than the actual value. The only exception is IMEI number which user needs to see to enter on the server side.

Just that the summary does not explain much. (because there is not much to explain ...)
Original title :     Server port
Original summary:   Tracking server TCP port
My new title :    "Server TCP port"
new summary : the port...
(I mean it quite obvious that it is the tracking server)
Original title: Server address
Summary: Domain name or IP address
The summary does not bring much. Only that you can also enter an IP.
So that's why the new title is :  Server name or IP address
So by moving the little additional info in the summary to the title, we can also display the value.
But I let you decide. It's your app :-)

My main issue is that I would have to get all translations again. It's not easy and I don't see that much value in this change to be worth the effort.

OK. that a good point...

We can keep the pull request open in case in future I would have to get some new strings translated and then merge it.


Readme needed

Needed readme file for each folder describing the algorithms inside and information about files added.

I added readme for sorting-algorithms folder.


Cherry Grove Dimension no trees

I'm trying to create a single biome dimension of Cherry Trees using the Cherry grove biome modifier with the Single controller. I have a bunch of efficiency dimlets and the peaceful dimlet, as well as all the default modifiers so there are no structures, effects, or dimlet huts. Skies and time are normal. However, every single time I generate a world, the biome is the Cherry Grove, but there are no trees. At least, not cherry trees. Industrialcraft rubber trees and random small jungle trees do spawn, but cherry trees are completely absent.
I'm running the newest Dw20 1.10 pack, with nothing except the ExtremeReactors config modified back to default fuel usage.
I recently had to load up a backup from a corrupted save, resulting in several dimensions existing but not being able to access any of them. I deleted these dimensions, but my minimap data for them still exists. Every time I create a new dimension, I'm using an old ID and when I enter, it reloads my Journeymap data, but over-writes the map as I explore. I'm not sure if this is causing any issues, but I don't really want to keep re-creating new dimensions and wasting ~3 million RF per until I start reaching new IDs.

What kind of terrain does the dimension have?

Everything is default, I'm sure. I put in a Normal Terrain dimlet. I tried some other biome controllers, but most of them tell me I have "too many biomes for XXX controller type".

Hmm, sounds like something which is handled in the biome itself and I don't have control over that. Perhaps the cherry biome only generates trees in the overworld or something

Ahhh, perhaps. Shame, but I guess it can't be helped. Thanks so much, honestly didn't expect a response this soon (it's 4 am here), but I'll just try something else. Thanks again!

I've done a bunch more testing, just generating random dimensions.
It seems like none of the Biomes O' Plenty biomes will spawn trees, except for Rubber Trees from IC2, and trees from Pam's Harvestcraft (With the fruits and things on them).

Doesn't seem to happen with vanilla biomes though.

I found this issue which seems to be similar:
https://github.com/Glitchfiend/BiomesOPlenty/issues/818
I'm using a newer Forge version (2254). Maybe it just hasn't been fixed yet.

Ahhh I figured it out, really silly issue.
Seems that Dw20 1.10 modpack has BoP generation disabled by default, meaning that the biomes won't actually be generated, regardless of whether you enable it after creating your world or not.
Mods that add custom dimensions aren't able to use the BoP generation and create said biomes properly. The grass changes color and the blocks generate, but apparently not trees.
The solution is to create a new world after enabling BoP generation.
Sorry about that, but thanks for trying to help anyway.


First version of a generic retry decorator

This a generic decorator that can handle the retry logic.

@arnevogt Feel free to use this decorator in your branch as well.
Also feel free to adopt it to the retry logic (I feel I just handle max tries, not max retries here).

close this because I integrated the changes in #59


Issue859 phor from vos

Main changes:

compute_diagnostic_vos() now also computes a horizontal-projection VOS

Issues:
Fixes, in devel, issue #859

Hello @Didou09! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tofu/data/_class8_vos_broadband.py:

Line 286:13: E265 block comment should start with '# '
Line 286:80: E501 line too long (84 > 79 characters)
Line 287:13: E265 block comment should start with '# '
Line 288:80: E501 line too long (90 > 79 characters)
Line 605:1: E303 too many blank lines (3)
Line 611:1: E302 expected 2 blank lines, found 3


Please change file release names.

Please just change the file names of your releases. It makes it so much harder to find a mod file released as "villagertools-1.20.1-1.0.3.jar" when I'm searching CurseForge for a mod that's actually called "Villager Artifacts".

lol the same. I was wondering where is the mod called villager tools


fix: also check exceptions/stack trace when filtering log messages

Unfortunately the log filter from my PR here does not work because the string "Config does not exist: file:///etc/datahub/plugins/frontend/auth/user.props" is not part of the (formatted) log message, instead this is part of the exceptions/stack trace.
As a solution, we could use the string "Exception starting propertyUserStore /etc/datahub/plugins/frontend/auth/user.props" for the filter, but then all exceptions related to the user.props file would be excluded, therefore I have enhanced the LogMessageFilter to also check the exceptions resp. the stack trace, then the filter from my previous PR is working (and this time I have verified that it really works 😇).
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[x] Links to related issues (if applicable)
[x] Tests for the changes have been added/updated (if applicable)
[x] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[x] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Summary by CodeRabbit

Bug Fixes

Improved log message filtering to handle throwable proxies, ensuring more accurate and reliable log management.

Assigning null directly to throwableString before the if caused throwableString to no longer be effectively final (although null is definitely better than an empty string) -> My last commit should fix this.

Aiming to merge these PRs today once I get CI passing.


Add --prettier flag for Prettier-formatted changelogs

This PR adds a new --prettier flag. When enabled, this flag signals that the tool should expect the changelog to be formatted using Prettier. By default, this flag is set to false, ensuring backward compatibility with all existing CHANGELOG.md files.
The primary motivation for this change is to overcome a current limitation of auto-changelog. At present, our tool does not support the changelog format provided by Keep a Changelog (version ^1.1.0).
Fixes: #60

I'm not sure. The update command would need to be paired with an external formater everytime it's executed. But it could be another flag (--ignore-format?) available only for validate.

I'm not sure. The update command would need to be paired with an external formater everytime it's executed. But we could add another flag (--ignore-formatting?) available only for validate.

We run Prettier regardless, hence why it's currently ignored.

Not always at the same time as updating the changelog though.
There's also the validate command; it's sensitive to formatting changes so we'd need to run prettier programmatically during validation to avoid breaking that.

If we ignore formating, enforcing that the sections are always in the same order may be more complex.
Yes @Gudahtt, this is why I passed the formatter to both validate and update commands.

Ok, makes sense!

If we are not willing to add a formatter, I suppose I could workaround like this:
Replace update with:
auto-changelog update && prettier

And replace validate with:
auto-changelog validate --fix && prettier

and then check if the CHANGELOG.md file changed.


Update c/storage to 9b10041d7b2ef767ce9c42b5862b6c51eeb82214

As the title says. Need this for CVE fixes.

@rhatdan PTAL. We need this out today. If you make a tag now I can repush, but it needs to happen quickly.

@TomSweeneyRedHat @haircommander @giuseppe PTAL

/lgtm
/hold

@mheon I'll let you watch it and take off the hold when you are ready (because it's past his French bedtime)

SGTM, thanks!

/hold cancel


No sound on HA with Eufy indoor cam 2k pan & tilt

Hi Alex
I have a Eufy indoor cam 2k pan & tilt connected to HA and the video is great but i cant seem to get audio from it.
Have I forgotten something or is it not possible with that specific camera?

Switch to this addon and check your camera supported codecs. Use transcoding if it needed.
https://github.com/AlexxIT/go2rtc


pytorch -> ONNX

The pytorch dependency is huge, and only has limited value.
Let's try to replace it by ONNX, which is hopefully smaller. Alternatively, we should move the extractors that depend on pytorch to a separate package.

Quick test by @mremreozan


Provide Do Re Mi Fa So La Ti Do note names

From @jm_fayard (twitter)
Could you add a settings to display the notes as Do Re Mi Fa So La Ti Do?
Yes C=DO, C#=DO#, Cb=DOb
For B, we actually call it SI (not Ti) in France/Italy/Spain

As per my other bug #22 , we could have a parameter that calls a function based on the current displayed value of the note. Like, if you are in Italian, you call translate() that in turn calls a function toItalian() that receives "C #" and then decides how to do the translation. For Italian, for example, it would just be a matter of getting the first letter and doing a lookup: "C" -> "Do" and then adding whatever follows the space " #", so you get "Do #". For German, the translator could be a bit smarter and make "C #" into whatever they call it. By using a function, we can do more than a lookup if it is needed. And we only need this doe when rendering to d3, not in the internal logic.
So it would be a matter of having a "language" parameter that goes into each object, and changing the renderers to use the results of such a function.

I'm very sorry, but I've decided that internationalization of GD is too big a task for me to do properly, so I'm going to keep it as an English only resource. Please feel free to fork other language versions.

I'm very sorry, but I've decided that internationalization of GD is too big a task for me to do properly, so I'm going to keep it as an English only resource. Please feel free to fork other language versions.

I think I will analyze internationalization requirements and send a PR. I am expecting some spare time to work on this next weeks!


Initial commit

Adding jupysql into the Jupyter guide.
Additional example for plotting on top of DuckDB.

@Alex-Monahan ready for review!

ping

ping

Howdy! It's part of my plan for today! I had bucketed today as part of the weekend since it is a US holiday.

@Alex-Monahan looks great! Pretty neat feature, will definitely use it and provide feedback!
I think this is ready to go.

Do we wait for a release to see it in the docs?

Do we wait for a release to see it in the docs?

we should only have to wait for the deploy, but it looks like the deploy is broken somehow - i'm looking into it

Should be up now! The deploy failure was GitHub having issues


New package: ktsu.BuildMonitor version <IP_ADDRESS>

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-pkgs/issues/160180

Automatic Validation ended with:

Executable C:\Program Files\BuildMonitor\ktsu.BuildMonitor.exe returned exit code: -532462766

Automated error analysis suggests -532462766 may mean EXCEPTION_COMPLUS (Unhandled Exception)
(Automated response - build 886.)

@stephengillie any idea how I could find out what this exception is? When installing via the local manifest I am able to successfully install and run the application with no exceptions.

When installing via the local manifest I am able to successfully install and run the application with no exceptions.

It would be better testing local manifests with Tools/SandboxTest.ps1 in Windows Sandbox. Each developers may have their own development environment, and may have different packages pre-installed. Those pre-installed packages may affect the installation process of you local manifests.
Windows Sandbox is a pure, brand new VM instance each time you launch it. The start point will always be the same on all developers. Tools/SandboxTest.ps1 performs same actions on every Sandbox instance, so you might be able to find some important key.

@stephengillie any idea how I could find out what this exception is? When installing via the local manifest I am able to successfully install and run the application with no exceptions.

This same happens with a pair of other packages in the repo (Ryujinx.Ryujinx and Ryujinx.Ryujinx.Ava) on a regular basis. I've been manually swapping labels to allow them to pass, and would be happy to do the same here.

duplicate

I spent today trying to get this working, but it seems as this is an OpenGL application using the Silk.NET library (same as Ryujinx) I had no luck getting it to actually run in the sandbox, perhaps why its failing in the automation.
I am adding the vcredist installation to my nsis installer and I'll submit another release.
Thankyou both.

I have remade the release including the vc redist package in the installer and updated the hash in the manifest.
As before, this installs and runs for me locally, as I have not been able to get opengl via Silk.NET to run in the sandbox yet.

Possibly related: https://github.com/microsoft/winget-pkgs/issues/12906

Possibly related: Support OpenGL for package validation. #12906

Yeah, it seems like the same issue I was facing with validating it locally in my own sandbox, which makes sense.

Manual Validation ended with:

Application: ktsu.BuildMonitor.exe CoreCLR Version: 8.0.624.26715 .NET Version: 8.0.6 Description: The process was terminated due to an unhandled exception. Exception Info: Silk.NET.GLFW.GlfwException: ApiUnavailable: WGL: The driver does not appear to support OpenGL    at Silk.NET.GLFW.Glfw.<>c.<.cctor>b__143_0(ErrorCode errorCode, String description)    at Silk.NET.GLFW.Glfw.CreateWindow(Int32 width, Int32 height, String title, Monitor* monitor, WindowHandle* share)    at Silk.NET.Windowing.Glfw.GlfwWindow.CoreInitialize(WindowOptions opts)    at Silk.NET.Windowing.Internals.WindowImplementationBase.CoreInitialize(ViewOptions opts)    at Silk.NET.Windowing.Internals.ViewImplementationBase.Initialize()    at Silk.NET.Windowing.WindowExtensions.Run(IView view)    at ktsu.io.ImGuiApp.ImGuiApp.Start(String windowTitle, ImGuiAppWindowState initialWindowState, Action onStart, Action`1 onTick, Action onMenu, Action onWindowResized)    at ktsu.BuildMonitor.BuildMonitor.Main() in D:\a\1\s\BuildMonitor\BuildMonitor.cs:line 34
Faulting application name: ktsu.BuildMonitor.exe, version: <IP_ADDRESS>, time stamp: 0x664796e4
Faulting module name: KERNELBASE.dll, version: 10.0.19041.1202, time stamp: 0xc9db1934 Exception code: 0xe0434352 Fault offset: 0x0000000000034f99
Faulting process id: 0x273c
Faulting application start time: 0x01dacbcd624488ae
Faulting application path: C:\Program Files\BuildMonitor\ktsu.BuildMonitor.exe
Faulting module path: C:\Windows\System32\KERNELBASE.dll Report Id: afc39ea1-facc-4841-b29e-0b04e724f848
Faulting package full name:
Faulting package-relative application ID:

(Automated response - build 886.)

@stephengillie I'm not really sure how I should proceed with this as it seems like there isn't good enough support in windows sandbox to perform the validation. Is this the same situation all OpenGL applications are in? Do you know of any that pass validation I could compare?

Hi @matt-edmondson,
I was moving too fast and didn't stop to read the error.

https://github.com/microsoft/winget-pkgs/issues/12906
https://github.com/microsoft/winget-pkgs/issues/107639

Verify:


Update README.md

Same as mikeio, added download button.

My thought was that if you can 'quantify' how much people are using/downloading your package, you can better allocate resources based on demand.

My thought was that if you can 'quantify' how much people are using/downloading your package, you can better allocate resources based on demand, as in if we have 60k downloads a month on Mikeio and 1k on modelskill, maybe we should focus on fixing the bugs of mikeio first and modelskill later, or the other way around.

Good thought, although one should take these stats with a grain of salt. I think there's definitely other valid reasons to prioritize resources aside from installs, but useful to be aware of I agree.


fix: Throw SyntaxError instead of FetchError in case of invalid JSON

Closes #498.

also see #549, but it should be done in a separate PR.


Too many dynamic script compilations within, max: [75/5m]

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v7 (for Elasticsearch 7.x)
when i use v7.0.18 to update some record,this problem produced:
[script] Too many dynamic script compilations within, max: [75/5m]; please use indexed, or scripts with parameters instead; this limit can be changed by the [script.max_compilations_rate] setting
can you tell me how to solve this  problem?

This is typically due to a problematic script design. You can read about all of this in e.g. https://alexmarquardt.com/2020/10/21/elasticsearch-too-many-script-compilations/.


[Roadmap] 1.2.0 Roadmap

@dmlc/xgboost-committer Please add your items here by editing this post. Let's ensure that

Each item has to be associated with a ticket
Major design/refactoring are associated with a RFC before committing the code
Blocking issue must be marked as blocking
Breaking change must be marked as breaking

For other contributors who have no permission to edit the post, please comment here about what you think should be in 1.2.0.

[x] GPU port of Accelerated Failure Time objective for survival analysis (#5714)
[x] Optimized JSON IO (#5772).
[x] Callback function for device DMatrix (#5629).
[x] Device DMatrix for dask interface (#5583, #5901).
[x] Integrate dask interface with Python asyncio. (#5639, #5862)
[x] Upgrade rabit. (#5680)

Blocking pull requests

[ ] https://github.com/dmlc/xgboost/pull/5934
[ ] https://github.com/dmlc/xgboost/pull/5929
[ ] #5171

Regarding distributed learning, Databricks has this big warning on their XGBoost docs: a cluster running Spark has to be re-started if an XGBoost job fails.
This is bad user experience, as a shared cluster might be running a number of other things and it would have to be shut down when a job fails, killing everything else.
The related issue #4826 hasn't had an update in a long time, so not sure if that's still the case with 1.1.0.

@ldesreumaux We would like to remove the experimental label after we make JSON serialization robust. #5772 adds a whole suite of tests to ensure that floating-point values are preserved by JSON serialization.

I'll also bring up re-visiting the default parameters issue, #4986 and the linked issue there, 1.2.0 might be a good point to at least change the min_child_weight.

I think we will have to do it in next release.  Which is not a bad thing, I'm targeting cat data support so we might unify a lots of internal code and will be able to support a unified set of parameters.

Is categorical data support planned for 1.2.0 @trivialfis ?

@dmlc/xgboost-committer It's been more than 2 months since we release 1.1.0, and we made 130 commits since then. Let's think about releasing 1.2.0 soon. Consider which pull requests are blocking.

I think https://github.com/dmlc/xgboost/pull/5929 is definitely something BLOCKING
https://github.com/dmlc/xgboost/pull/5171 is something good to have as the first version supporting Spark 3.0 with a GPU capability

[ ] New callback functions (#5612)
[ ] Expose rabit context for dask interface. (#5765, #5408)

Removed from the roadmap.

Added blocking PRs

I can add gpu_hist unit test today, @CodingCat @hcho3

@wbo4958 I marked gpu_hist as "blocking". We'll make sure it's included in 1.2.0.

@thvasilo Categorical data support won't be included in 1.2.0. cc @trivialfis

@hcho3 Can branch out?

Sure, let me do that this Friday.

Now that all blocking PRs have been merged, let me prepare a new release branch.

1.2.0 is now released.


RsaKeyGenerator class Replace("\r\n", "")  bug in linux

linux ‘\r\n’ is ‘\n‘

[``](

[ ] 1. url

)

With your close and reopen, I also closed and reopened vs2019

I'm sorry;
Misoperation;
Just now the network is unstable!
------------------ 原始邮件 ------------------
发件人: "Zhiqiang<EMAIL_ADDRESS>发送时间: 2019年9月20日(星期五) 下午2:31
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS>"State<EMAIL_ADDRESS>主题: Re: [stulzq/RSAUtil] RsaKeyGenerator class Replace("\r\n", "")  bugin linux  (#13)
With your close and reopen, I also closed and reopened vs2019
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Remove the "file://" storage type

What this PR does / why we need it:
1: Remove the "file://" storage type so that the local path is not confusing any more and only means pvc
2: Refine the message so that if the pvc path does not exist it will report "Local path %s does not exist." rather than the misleading message of storage type.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #621
Special notes for your reviewer:
The python changes will take effect next time when we release kfserving sdk and storage-initializer image.

This change is

Now the "file://" type cannot be applied and the message is like:
Error from server: error when creating "sklearn.yaml": admission webhook "inferenceservice.kfserving-webhook-server.validator" denied the request: storageUri, must be one of: [gs://, s3://, pvc://] or match https://{}.blob.core.windows.net/{}/{} or be an absolute or relative local path. StorageUri [file://test] is not supported.

/retest

/retest

/retest

/retest

/retest
The testing is really flaky...

/retest

/retest

/retest

Seems we need to take a look the CI tests, that's not stable ...

For current situation, if we still keep it there will be some ugly changes since now the storage are all from the storage-initializer, and each model have different naming rule to define the path(model_dir, model_base_dir, model_path). It's broken for a long time and no one use it, we need consider whether to keep it, or if yes how to make it valuable.

/assign @ellis-bigelow @yuzisun
Now fixed the local development mode for model, please take a look. I did not get the chance to do the e2e test yet but I think it should work...

I'm not sure I understand this go portion of this PR. When I said local development story, I was thinking about running the model server locally, nothing about the controller.

@pugangxa what's the conclusion here?

@pugangxa what's the conclusion here?

Seems no one uses this "file://" type, if it's not important, I think we can just close this PR.

Close for now and later we can reopen if anyone encounter this issue for "file://" storage.


Add algo/localplan

Adds algo/local_plan
Local plan computes a local plan of action for each pair of nodes in the (got, want) graph pair.
Depends on other PRs in flight, don't review yet.

This has some already added files listed as add, can you rebase?

Fixed -- thanks!

Ok, this should be ready now.

/hold cancel


removed unneeded trim from Prometheus charts

The dash at the end left-trims the next line which result in something like annotations:volume.beta.kubernetes.io/storage-class: ....

I signed it!


Machine Learning in Health: Predicting Pregnancy Complications | PyData Global 2021

Machine Learning in Health: Predicting Pregnancy Complications | PyData Global 2021
https://www.youtube.com/watch?v=h3pcKTTDmuY
00:20 - Begin of presentation
01:02 - Personal introduction
01:47 - Content
02:40 - PreFree is ...
03:08 - Medical background - Preeclampsia
03:45 - Preeclampsia, Unspecific Symptoms - Devastating Complications
05:20 - Current Care Path
06:31 - User Research Confirms ..
07:14 - PreFree Solution for Patient-Centered Care
08:15 - PreFree Application
09:48 - Physicain Frontend
10:41 - Mobile Application
11:00 - Data
13:22 - Modelling - goal
13:38 - Modelling - workflow
15:29 - Modelling - feature selection
16:50 - Modelling - handling missing values
17:25 - Modelling - feature engineering
19:43 - Modelling - model selection
21:12 - Modelling - final step: hyperparameter tuning
21:22 - Modelling - results
21:39 - Modelling - attempt to quantify generalisation variance
23:30 - Modelling - upcoming
24:07 - Modelling - deployment!
24:22 - Data Protection
25:50 - We are interested in ...

Thank you very much for preparing the timestamps @ktzhao ❤
May I ask someone responsible here to publish these timestamps on youtube?
I would highly appreciate that 👍


nixos/writefreely: init

Description of changes

This PR adds a new NixOS module for running WriteFreely (pkgs.writefreely).
Example usage (with the default SQLite database):
{
  services.writefreely = {
    enable = true;

    admin.name = "nixos";
    # You can provide an initial password, otherwise the default will be set to "nixos".
    # admin.initialPasswordFile = "/opt/my-initial-admin-pass.txt";
  };
}

Example usage (with the MySQL database):
{
  services.writefreely = {
    enable = true;

    admin.name = "nixos";
    # You can provide an initial password, otherwise the default will be set to "nixos".
    # admin.initialPasswordFile = "/opt/my-initial-admin-pass.txt";

    database = {
      type = "mysql";

      # We can spin up MySQL locally.
      createLocally = true;

      # A password is required for WriteFreely to work.
      passwordFile = "/opt/my-db-pass.txt";
    };
  };
}

Example usage (with nginx):
{
  services.writefreely = {
    enable = true;

    admin.name = "nixos";
    # You can provide an initial password, otherwise the default will be set to "nixos".
    # admin.initialPasswordFile = "/opt/my-initial-admin-pass.txt";

    nginx.enable = true;
  };
}

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module
[x] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thanks for your PR. Can you please update to latest master.

Hey @ck3d, thanks for letting me know. I'll see if I can find some time to rebase these changes 👍

Alright, PR should be updated @ck3d :+1:


Missing credits, legality and the proper use of brands/logos.

Godot version
Both 3.x and master branch
System information
All
Issue description
Continuing this thread https://github.com/godotengine/godot/pull/42359 about console placeholders I have concerns about the proper use of logos in export templates.

Android
Android robot is licensed under Creative Commons 3.0 Attribution License and I don't see any mention of it in neither the source code nor the editor Third-party Licenses. Also "Unless expressly authorized by Google through written agreement, the Android wordmark and custom typeface may not be used."
Mac OSX
Since 2016 it's macOS, no X anymore. The logo resembles the current Mac logo (not macOS), but it's not proper.
iOS
Logo is different since 2017.
HTML5
"This HTML5 logo is licensed under Creative Commons Attribution 3.0 — all are free to use and reimagine as they see fit."
Microsoft
Windows logo
Linux mascot
"Permission to use and/or modify this image is granted provided you acknowledge me<EMAIL_ADDRESS>and The GIMP if someone asks".
Steps to fix it:

add missing credits
get permission where necessary
add missing trademark informations
replace some logos with allowed ones
consult it with your legal staff (Godot exists as a member project of the Software Freedom Conservancy)

Steps to reproduce
Go to Project>Export...>Add...
Minimal reproduction project
No response

See also https://github.com/godotengine/godot-proposals/issues/2124.

See also https://github.com/godotengine/godot-design repository for this information. I haven't verified if the proper important is stored there yet.

See also godotengine/godot-design repository for this information. I haven't verified if the proper details is stored there yet.

The godot-design repository does not include the platform icons (or any editor icons for that matter). Instead, platform icons are stored in each platform's folder so third-party platforms can supply their own icon.


django.jQuery

What is the last line of the JS?  I don't understand.  Should one pass an instance of jQuery?

Have a look at the Django documentation . When using the Django admin, it uses it's own namespace django.jQuery


Connection attempt hangs with no traffic observed

Hi 👋,
In 1.0beta6 I'm not able to connect to any servers from the tracker list.  With Wireshark running, I don't see any traffic sent to the server:

As a sanity check I built the app from the 1.0beta5 tag and all was well.  Possible regression?
Thanks for all your work on this project!

I'll take a look I'm doing a bunch more work in that code as I was seeing some issues with passing login info through bindings.

Let me know if beta7 fixes this for you at all. :
I've only seen this with servers that require server agreement before connection finishes (something I'll need to add support for eventually). Does Mobius have this feature?

I'm still seeing the same connection hang behavior in beta7.
Mobius supports both the 1.2.3 agreement-less login flow and the 1.5+ flow that expects a reply to the agreement, which I'd maybe be suspicious of, except that I'm not even seeing the TCP handshake on 5500 reach the server when the client hangs.  So it seems like the client isn't reaching a point where it tries to establish a TCP connection.
Some data points that I don't understand:

I can consistently connect to my local dev server on <IP_ADDRESS>:5500 from 1.0beta6 or 7
I can never connect to any server that isn't localhost, either by specifying the IP or clicking on the tracker listings
1.0beta5 consistently works, which seemingly rules out environmental oddities on my end (I think)

I think I know what changed. I believe I enabled fast TCP connections in b6. I think. I'll check in the morning and revert if so. That's what I get for messing around. 😅

I'm seeing the same behavior. I'm happy to capture packets if that helps.

Thanks! I'll be posting a new build maybe tonight that hopefully fixes the issue. I wish Apple had better documentation on its socket library NWConnection. It's extremely sparse at the moment.

Just an update. When logging in, the progress bar never completes, but I am able to chat and see other's chats. I can't see the user list and have no ability to see files or news.

Just published a new release. Let me know if you're still seeing this. I was seeing some weirdness connecting to the public server System 7 Today and fixed it.

The new build looks good to me. Thanks!! 🎉

New build works great on my 1.9 server, works great on Mobius servers, but I still can't complete the login process on my 1.2.3 server. Feel free to test it out here theabode.ath.cx:5500

I'm currently not handling the flow where a server requires people agree to the agreement before continuing. Need to think on how to handle this a bit. Thanks!

Id treat it as auto agree to skip it by with the caveat that will
eventually be working post beta world.
On Tue, Jan 9, 2024, 6:05 PM Dustin Mierau @.***> wrote:

I'm currently not handling the flow where a server requires people agree
to the agreement before continuing. Need to think on how to handle this a
bit. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/mierau/hotline/issues/4#issuecomment-1883936887, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABX3DRHEX4SXESJBIL2574DYNXELJAVCNFSM6AAAAABBNSG7ZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBTHEZTMOBYG4
.
You are receiving this because you commented.Message ID:
@.***>

Going to mark this issue as closed for now. Thanks everyone. Created a new issue for agreement accepting. Auto-accepting may be the first version, I agree. I think I specifically didn't implement that because if someone is requiring agreement acceptance, there is probably a reason for it. :)


Type of standard function "MakeHeader" should be changed from protected to global, Codeunit 7322 "Create Inventory Pick/Movement"

old:
    local procedure MakeHeader()
    var
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeMakeHeader(WhseActivHeader, AutoCreation, IsHandled);
        if IsHandled then
            exit;

        if AutoCreation and not LineCreated then begin
            WhseActivHeader."No." := '';
            WhseActivHeader.Insert(true);
            UpdateWhseActivHeader(WhseRequest);
            NextLineNo := 10000;
            if not SuppressCommit then
                Commit();
        end;
    end;

new:
    procedure MakeHeader()
    var
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeMakeHeader(WhseActivHeader, AutoCreation, IsHandled);
        if IsHandled then
            exit;

        if AutoCreation and not LineCreated then begin
            WhseActivHeader."No." := '';
            WhseActivHeader.Insert(true);
            UpdateWhseActivHeader(WhseRequest);
            NextLineNo := 10000;
            if not SuppressCommit then
                Commit();
        end;
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10221.


[besu] updated network-proxy-none.yaml

Commit to be reviewed
fix(besu): updated network.yaml

Primary Changes
---------------
1. Updated the genesis file path in platforms/hyperledger-besu/configuration/samples/network-proxy-none.yaml

Reviewed by
@suvajit-sarkar

As we are mounting the bevel repo at /home/bevel as mentioned here, if we mention the genesis file present at /home/user/bevel path, the ansible playbook fails stating the following path could not be created (using the bevel-build container)


Add cookbook template

Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
Add the cookbook template as a demo app.
Describe alternatives you've considered
N/A
Additional context
N/A

duplicated of at_demos#36


POST media API issue

POST requests to upload physical PDF or other document files through the python API are incorrectly identifying the mime type of uploads.  Will modify to make this in line with PUT (put_media) function which appears to work correctly.

Should be addressed in latest commit to properly send binary file on uploads.


Allow enabling DEBUG logs for databricks-sql-connector

resolves #7860

Description

This change allows us to turn on DEBUG level logs for databricks-sql-connector when invoked from dbt.
Regarding documentation updates: I'm going to open a PR for dbt-databricks that includes more fine-grained control over the logs from databricks-sql-connector. The doc update for this PR will be included at that time.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

@aranke would either you or @peterallenwebb (or both!) be willing to take a peek at @jerco's comment here?
His key question is:

Is there any way to put this logic inside the dbt-databricks adapter instead?

We should be good to merge this, just checking if you have signed the CLA?

Yes @aranke I've signed the CLA :)
@dbeatty10 I responded to JerCo's comment in that thread. If this makes more sense in dbt-databricks we can move it there. The existing solution is meant to follow the pattern set for the snowflake connector.

@cla-bot check

@aranke CLA check is passing now. I'll let you either merge this (or not) depending on the outcome of this discussion: https://github.com/dbt-labs/dbt-core/issues/7859#issuecomment-1600056641

This won't actually work since it only enables the legacy logger that's being deprecated, see #8643 for inclusion in standard dbt logging

I'm going to close this in favor of https://github.com/dbt-labs/dbt-core/pull/8643 (which has been merged).
We can re-visit this (including re-opening this PR) if needed.

I'm going to close this in favor of https://github.com/dbt-labs/dbt-core/pull/8643 (which has been merged).
We can re-visit this (including re-opening this PR) if needed.


Ubuntu 18.04 upgrade for CI

Ubuntu 16.04 is not supported anymore.
Ubuntu 18.04 is necessary to resolve #22076 because we require a more up-to-date rpmbuild

It would be good to check whether other distros have the same or newer glibc as Bionic. I suspect Debian 9 has older glibc.
Older glibc is forward-compatible with newer, but the reverse isn't true -- Programs compiled with newer glibc won't run on older glibc.
I'll try to check the versions on the different distros some time today. Should work from just booting up a couple of LiveCD images in VirtualBox and running ldd --version on each of them...

We can also just check the Debian repositories: https://tracker.debian.org/pkg/glibc
Their old-stable (indeed Debian 9, Stretch) is on glibc 2.24
Ubuntu Bionic has glibc 2.27
so you are right about this possible implication. But also to think about: if targeting Debian as supported platform, and you want backwards-compatibility for their older stable releases, then building on Ubuntu might not be the best platform, because it reaches End-of-Life earlier than its Debian counterparts.

There is this PPA with many backports, including RPM, for Xenial: https://launchpad.net/~savoury1/+archive/ubuntu/backports
Maybe we could use that PPA, or make our own (to be more safe)
it is a bit problematic that official Xenial support is now dropped, so there are no more official Xenial repositories hosted by Ubuntu/Canonical themselves.

I will close this PR in favor of #22433 because of the glibc complications mentioned by @DeeDeeG


az image builder create

Describe the bug
az image builder create does not contain the vmProfile config properties:
"vmProfile":
{
"vmSize": "",
"osDiskSizeGB": ,
"vnetConfig": {
"subnetId": "/subscriptions//resourceGroups//providers/Microsoft.Network/virtualNetworks//subnets/"
}
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/image-builder-json?toc=/azure/virtual-machines/windows/toc.json&bc=/azure/virtual-machines/windows/breadcrumb/toc.json
To Reproduce
These are required to support key customer functionality, and customers have asked for this to be made available.
Please can these configuration properties be added to match AZ PowerShell. thanks!

Hi @qwordy , could you please help with this issue ? Thanks

It is not supported yet. Added to S174. You can use --image-template as a workaround.


Adding user name validation correction

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!

@googlebot I fixed it.


titleize for label produces capitalized words in multi-word symbol or string

As I've stated in the name I get capitalized words in multi-word symbol or string in a form.
Example: :test_case produces Test Case in the view.
I'd like it to be Test case
In the form_builder.rb the label method uses the titleize method on the content which gives the result above.
Is there a way to work around this?
I'm thinking of something like:
def convert(symbol)
    symbol = symbol.to_s
    if symbol.include?("_")
        symbol.sub!("_", " ").capitalize
    else
        symbol.capitalize
    end
end

It would work also for strings.
regards,
seba

@sebastjan-hribar Thanks for this suggestion. A quick research revealed that it's a common pattern to capitalize and not to titleize form labels (Eg. "Confirm password" and not "Confirm Password").
I'd say to change the behavior here (see https://github.com/lotus/utils/pull/85). @lotus/core-team Thoughts?

@jodosha I had a look at the capitalize in lotus/utils#85 as you suggested and that would be great.
regards, seba


Update DEST_DIR_SUFFIX in case of pr

Recently on crc side[0] we have enabled the regex to capture bundle info and as part of this regex we are capturing the custom bundle which is random generated number like
crc_okd_vfkit_4.16.7_amd64_2342465234654.crcbundle but for bundles generated by the pr have suffix .pr<pr_number> which means again adjust the regex. We can still identify the bundle with _<pr_number> and this way we don't need to update the regex.

https://github.com/crc-org/crc/pull/4343

/cherry-pick release-4.18

/cherry-pick release-4.17

/cherry-pick release-4.16

/retest


compaction_level0_phase1: change default streaming-kmerge without validation

After this PR is merged, deployed, and guaranteed to not be rolled back,
we can remove the field from pageserver.tomls.
refs https://github.com/neondatabase/neon/issues/8184

We actually don't need this intermediate step.


Typescript Types for REST API Response in Client code

Question
Is there a npm package that provides the necessary type definitions for the client side?
To be more precise:
When requesting  the api at /api/navigation/render/main-navigation in some client - what typescript type does the returned data have?
Example
Some client trying to fetch navigation data from a strapi backend via REST:
export async function getNavigation()
	// : ???
{
  const response = await fetch( "/api/navigation/render/main-navigation", {} );
  if( ! response?.ok ) {
    throw new Error( `Error when fetching data from '${url}': returned HTTP status '${response.status}'` );
  }
  const data = await response.json();
  if( !data ) {
    throw new Error( `Error when fetching data from '${url}': no data!` );
  }
  return data // as ???
}

Hello @EsGeh ,
there is no such type yet but good shout! I'm putting this request to backlog.

Added in v2.3.1


tokenReceivedCallback is not called when using MSAL AngularJS with Azure B2C

I'm submitting a

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[X] Other... Please describe:

Browser:

[X] Chrome version 75.0.3770.100
[ ] Firefox version XX
[ ] IE version XX
[ ] Edge version XX
[ ] Safari version XX

Library version

Library version: msal.js 1.0.2

Current behavior

After successful login, tokenReceivedCallback is not called...
Expected behavior
tokenReceivedCallback should be called.
Minimal reproduction of the problem with instructions
msalProvider.init({
    clientID: "Client ID here",
    authority: "https://login.microsoftonline.com/tfp/myb2caad.onmicrosoft.com/B2C_1_SignUpSignIn",
    validateAuthority: false,
    //consentScopes: ["https://myb2caad.onmicrosoft.com/test/user_impersonation"],
    tokenReceivedCallback: function (errorDesc, token, error, tokenType) {
        debugger;
        if (token) {
            console.log('Token type: ' + tokenType + ' Token: ' + token)
        }
        else {
            console.log(errorDesc + error)
        }
    }
},
    $httpProvider // Optionally, pass http provider to inject request interceptor to attach tokens
);

Doing:
msalService.loginPopup();
and then:
$rootScope.$on("msal:loginSuccess", function () {
    console.log("loginSuccess");

    var token = msalService.userInfo.idToken;
});

I can see that userInfo.isAuthenticated is true and there's an id token present.
However, tokenReceivedCallback is never called. Can you spot the problem or give some hints where to look for it?
Note that this is a B2C AAD. Maybe I'm forgetting to set something elsewhere.

@leniel can you try doing a .then off of login popup and see if that triggers, login popup is a promise but redirect requires the callback to be registered so on page reload we can call it.

The problem was happening because the app was messing with hashPrefix. I had to do:
app.config(['$locationProvider', function($locationProvider) {
    $locationProvider.html5Mode(false).hashPrefix('');
}]);

The hashPrefix was interfering with the callback.
It's mentioned here: https://www.npmjs.com/package/@azure/msal-angularjs


DR-967 - Activate domain

Changes:

Add Activate functionality.
Refactor Add domain call to use the same as Activate.
Add Activate Test.

Nice! Do the tests pass?

Yes :D

Can i merge this? @andresmoschini  @nreal

I think so, @nreal could you review at least @mvillaseco's tests and verify if he is requesting API as you expect?


Always change language where necessary if path changes

A fix for this issue:
https://github.com/kalinchernev/using-i18n/issues/1

It's no worries, for the example here it's good. In actual project some
checks whether its necessary to make the change of language will be easy to
make
On Fri, 13 Dec 2019, 12:58 Dean Clatworthy<EMAIL_ADDRESS>wrote:

@deanc commented on this pull request.
In src/layouts/index.js
https://github.com/kalinchernev/using-i18n/pull/3#discussion_r357613450:

import withI18next from "../i18n/withI18next";
-const Layout = ({ children }) => {
+const Layout = ({ children, location, pageContext: { locale }}) => {
+

const { i18n } = useTranslation()

useEffect(() => {
i18n.changeLanguage(locale)
}, [location, i18n, locale])

Sure, but think of it this way. It's used inside the hook, so it is a
dependency.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/kalinchernev/using-i18n/pull/3?email_source=notifications&email_token=AAOVTFCRFRTIH22EUBDTNHTQYN2IBA5CNFSM4J2DC4IKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCPDURAY#discussion_r357613450,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOVTFCCCR2JNGTT6DCZJN3QYN2IBANCNFSM4J2DC4IA
.


CVE-2017-3156 backport

This is a backport of the patch from here: e66ce235 to 2.6.x. Although I'm opening the PR, I do understand that 2.6.x is EOL, and therefore this might not be merged or ultimately released. If anyone does feedback, it will be very gratefully received.

I think the last 2.6.x security release was a once-off and so I don't think there is any point in merging anything to the branch.


IFS search to ignore certain directories

When searching the IFS, it tends to take an absolutely long time when searching large trees.
I think we need a way to define a set of directories to be ignored when searching in the IFS.
--exclude-dir=dir

This is a good idea, but is it worth adding more configuration to define a list of paths to ignore?
How should the user tell the extension what extensions to ignore?

Decided it will be part of the extension config and not tied to a connection.


README: unified middleware format

Describe the pull request
unified middleware format
Checklist

[x] I agree to follow the Code of Conduct by submitting this pull request.
[x] I have read and acknowledge the Contributing guide.
[ ] I have added test cases to cover the new code.

Thanks for the PR!
However the current formatting style is on purpose. Links to code and links to pages are different.

Actually, just double checked first four links are not to code. I'll change them to non-inline-code style, which seems more appropriate.


add merge on parent node.

Very usefull for complex structured component

Thanks for the pull requests @sebadiaz . Did you sign the CLA http://www.rethinkdb.com/community/cla/ yet?
Pinging @chipotle for review

it is now the case

The more I look at this, the more this looks like it should be a Cookbook recipe rather than a new example in merge; a lot of what's going on here is as much the product of map/concatMap as merging.
Could someone give me an example of a use case for this for the cookbook? I can see what's going on in Sebastian's example, but I'm not clear on when you'd want to do that with real data.

I'm not sure what a good example use case is.
@sebadiaz did you need this query in a specific context / for a specific application?

Given the age of this pull request and that no more information was provided, I'm closing it. We can re-open (or open a new PR) if necessary.


Convert Records to XML

It would be awesome, if it possible to get the records as xml

This can be implemented. The serialisation would be very simple, with keys in the records converted into XML tags and the values as the tag contents.
For example the following record:
{
    "key1": "value",
    "key2": "value 2
]

would be serialised as
<record>
  <key1>value</key1>
  <key2>value</key2>
</record>

Would this be acceptable?


why do you use ZeroMQ

Hi, I am reading your project and I love this project
I have a question, why do you choose ZeroMQ in your project instead of others like RabbitMQ or Kafka, are there any special feature but others dont have it. Can you share me about it. I am wondering when I should choose ZeroMQ instead of others like RabbitMQ or Kafka

Hi there!
Those three are pretty different and you wouldn't exchange those without some serious consideration.
RabbitMQ is a general-purpose message broker that supports many different protocols. You can use the different Exchanges to achieve different routing of messages.
 That means that the server keeps track of all the messages and if it should keep it, or delete it because no one is gonna read them anymore.
Kafka is a message bus built for vast data streams with very high throughput and replay capability. It is very fast, handles a lot of data, and scales pretty well.
It uses what is called a 'dumb broker / smart consumer' model, which means that the server just keeps all messages for a given time. It relies on the consumers to decide whether they needed those messages or not, dealing them to all consumers alike.
ZeroMQ is a simple, yet very powerful framework you can use to write your own message-broker implementations with. You'd use it when none of the other implementations fits your needs. Or you would use it to do simple process-to-process communication between two servers, for instance, because it has such a neat and simple data abstraction for socket-to-socket communication.
The reason why I chose ZeroMQ is because this repository is a simple example I use for teaching at school :)
Sincerely, Gerald

test
From: Hoang Ngo @.>
Sent: Freitag, 27. Jänner 2023 05:15
To: UnterrainerInformatik/htl-zeromq @.>
Cc: Subscribed @.***>
Subject: [UnterrainerInformatik/htl-zeromq] why do you use ZeroMQ (Issue #2)
Hi, I am reading your project and I love this project
I have a question, why do you choose ZeroMQ in your project instead of others like RabbitMQ or Kafka, are there any special feature but others dont have it. Can you share me about it. I am wondering when I should choose ZeroMQ instead of others like RabbitMQ or Kafka
—
Reply to this email directly, view it on GitHub https://github.com/UnterrainerInformatik/htl-zeromq/issues/2 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABY7V67BEL25AIFCKUPMHE3WUNDVNANCNFSM6AAAAAAUIIFWTA .
You are receiving this because you are subscribed to this thread.  https://github.com/notifications/beacon/ABY7V6Z3PK3HBYPEC5N4KM3WUNDVNA5CNFSM6AAAAAAUIIFWTCWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHFZ3USXY.gif Message ID: @.*** @.***> >


Refactor test schemes

Probably should let https://github.com/istio/istio/pull/22656 merge first to give this some more coverage

merged into https://github.com/istio/istio/pull/22707


Any plan for v2 examples?

Any plan for v2 examples?

Good question -- I think we're going to use the forum's wiki category: https://caddy.community/c/wiki/


Missing manual installation for Bootstrapping - Disconnectec scenario

The following needs to be done, otherwise Registration will fail
https://docs.microsoft.com/en-us/powershell/gallery/how-to/getting-support/bootstrapping-nuget#manually-bootstrapping-the-nuget-provider-on-a-machine-that-is-not-connected-to-the-internet

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

ID: 7d14a302-9b02-8eb4-1b49-b52580a67472
Version Independent ID: 10c55050-5b52-a48d-bfb2-eba0a4494900
Content: Install PowerShell for Azure Stack
Content Source: azure-stack/operator/azure-stack-powershell-install.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@taeheenator , Thank you for your feedback, We will investigate and get back to you.

@taeheenator Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@taeheenator Thanks for pointing this out. I've added the step and it should show up with our next build. Thanks!

#please-close


I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

Thanks for the fix. Is now a good time to do an audit to check if all post effects are correctly freeing any allocated resources?

Thanks for the fix. Is now a good time to do an audit to check if all post effects are correctly freeing any allocated resources?

It looks like everything is fine with other post effects


[pattgen,dv] Improve pattgen coverage for V2 milestone

This includes the following updates:
1] Break up the mid-range bins into more bins for the counters
2] Added the missing covergroup
3] Match the covergroup names with the testplan
Signed-off-by: Madhuri Patel<EMAIL_ADDRESS>

Looks like CI failed:

* `xmvlog: *E,ECGISM (../src/lowrisc_dv_pattgen_cov_*/pattgen_cov_if.sv,*|45): Semicolon (';') symbol is not specified.` has 1 failures:
    * Test default has 1 failures.
        * default\
          Line 729, in log /azp/agent/_work/1/s/scratch/HEAD/pattgen-sim-xcelium/default/build.log

                xmvlog: *E,ECGISM (../src/lowrisc_dv_pattgen_cov_0.1/pattgen_cov_if.sv,351|45): Semicolon (';') symbol is not specified.
                    cp_prediv_cnt_ch0: coverpoint {prediv_q0}}{
                                                              |
                xmvlog: *E,ECGEGN (../src/lowrisc_dv_pattgen_cov_0.1/pattgen_cov_if.sv,351|46): The "endgroup" keyword in covergroup declaration is not specified.
                      bins minim_0 = {PredivMinValue};


How do i get a LayoutConstraint 's constant use Masonry, or how do i compare two MASConstraint 's size

How do i get a LayoutConstraint 's constant use Masonry, or how do i compare two MASConstraint 's size?

@renlongyu there's not really any built in support for this. A Masonry constraint can result in multiple NSLayoutConstraint's and they're not easily comparable due to the way they install and sorts.


Please make sure you are using the server version of the FTB modpack error.

I am completely new to synology/docker and I have no idea what it is trying to tell me when it says to use the server version of the ftb modpack, the modpack I am trying to use is Stacia Expert
https://www.curseforge.com/minecraft/modpacks/stacia-expert/files/3393840
I don't know what to do for this error because it has gotten as far as knowing the modpack it is meant to be using, and that is the extent of my knowledge for this error. I would like to know how to fix this as I am just trying to run a modded server on my synology for my brother and I to play, but I cannot get past this.
Sorry if I do not seem like I know very much about this as I am completely new to everything remotely related to docker.
I have already browsed other issues of this same error but I have no idea how to import a script whilst trying to setup the running of the server for the first time, so it doesn't help me.

I don't think I have tried out that modpack myself, but a couple of things to double check:

Make sure you grabbed the "server" file located in the "Additional Files" section

If you used the server file, then try using the option described here

I tried to use the server file in the additional files but I just worked around it with instead of grabbing the link for the modpack, I just use a regular forge server and add the necessary mods, thanks though.

I'm having this issue as well, with this mod- https://www.curseforge.com/minecraft/modpacks/medieval-minecraft-modpack/files/3406762

I have the same issue, should this issue be reopened?


Documented version specific upgrade issues in K3s upgrade documentation

Closes #3723
Reference: https://rancher.com/docs/k3s/latest/en/upgrades/

Hi @katran001 - can you find an appropriate resource on the k3s team to help review this docs issue for Jen?


Warming up index is failing

I tried to use the latest version of summa to index a large IPFS dataset (bafyb4iemblftubydyhfo6xhw56zrhudy2xexqb25f7awrahe3qfplse5g4).
I am using:

docker:  izihawa/summa-server:0.12.0
python client: aiosumma==2.30.1

The indexing works and I can perform searches.
But when trying to run summa-cli <IP_ADDRESS>:8082 warmup-index "xxxxx" --is-full and after waiting 10-20 hours, nothing happens.  I have received a number of reports about connectivity issues through iroh, library which is responsible for IPFS in Summa. I'm into this issue right now and seems here is a subject to change. For now, workaround is the following:

do usual ipfs get <your cid> through kubo
mv <your cid> data/bin/<any_index_name> where data/bin is a directory for data you have set up in summa.yaml
summa-cli <ip> attach-index <any_index_name> '{"file": {}}'

Index will be available under name any_index_name. Sorry for inconvenience, I will left the issue opened until figure out a reliable solution.

Thanks.
 Failing retries were just hanging doing nothing.

We've decided to detach Iroh functionality, it seemed to be too immature. This way I've described above is the only one for now.
I'm reconsidering ways of how to integrate Summa with Iroh because it remains one of the main use case, but need some time to deploy more reliable approaches.

No problem. Thanks for your work.


just a question

Is there a version of iceberg for Hyper as well as Terminal....poked around but haven't turned one up...

@rcrooks I created Iceberg for Hyper and it's now available. Please enjoy it!
https://github.com/cocopon/hyper-iceberg

thanks!