0.8.19 has bug

Building Windows application...
SEVERE: error[E0658]: use of unstable library feature 'cstr_count_bytes'
SEVERE : error : could not compile super_native_extensions (lib) due to 1 previous error [C:\cgz\n_desktop\build\windows\x64\plugins\super_native_extensions\super_native_extensions_plugin_cargokit.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(254,5): error MSB8066: “C:\cgz\n_desktop\build\windows\x64\CMakeFiles\12c8607d114ee118ee87392f4b679c73\super_native_extensions.dll.rule;C:\cgz\n_desktop\build\windows\x64\CMakeFiles\075bb8925e1cbcc12769fb923e861807\super_native_extensions_plugin_cargokit.rule”的自定义生成已退出，代码为 -1。 [C:\cgz\n_desktop\build\windows\x64\plugins\super_native_extensions\super_native_extensions_plugin_cargokit.vcxproj]
Error: Build process failed.
[√] Flutter (Channel stable, 3.24.0, on Microsoft Windows [版本 10.0.19045.4651], locale zh-CN)
• Flutter version 3.24.0 on channel stable at C:\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 80c2e84975 (2 weeks ago), 2024-07-30 23:06:49 +0700
• Engine revision b8800d88be
• Dart version 3.5.0
• DevTools version 2.37.2
• Pub download mirror https://pub.flutter-io.cn
• Flutter download mirror https://storage.flutter-io.cn

Can you do rustup update and try again?

Please reopen if the issue exists after running rustup update.

This problem still exists.

Can you please post output of rustc --version and verbose flutter build log?

rustc 1.78.0 (9b00956e5 2024-04-29)

Latest stable version is  1.80.1. It should have been updated after rustup update.

rustc 1.80.1 (3f5fd8dd4 2024-08-06)
Thank you very much. It's okay after upgrading to the latest version


Try to place the slider and NPJ together and adjust the code structure if needed

We have a JoinAgoTable, which can be filled with algorithms like this:
algos={newAbstractJoinAlgo(),
newNPJ(),
newNPJSingle()
};
We can use the table to select algos in join processor by the 'findAlgo'
findAlgo(JOINALGO_NPJ)->join(windowS.data(),windowR.data(),sLen,rLen,4);


Scrolling feature needed in fragments

In some phones, the views cannot be seen properly. Putting things inside a ScrollView would help.

oh, just for the sample...

So can i work on the sample codebase?

i added a scrollview on the sample

can i work on some other issue sir?

you can add others issues if you want

I was actually looking for such a library for pretty long and now since you have come up with it i wanted to contribute in some way. I just wanted to know should i focus on the sample app or on making new shapes?

You Can try to create some useful shapes
Le 23 févr. 2018 17:12, "Rajat Kumar Gupta"<EMAIL_ADDRESS>a
écrit :

I was actually looking for such a library for pretty long and now since
you have come up with it i wanted to contribute in some way. I just wanted
to know should i focus on the sample app or on making new shapes?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/florent37/ShapeOfView/issues/1#issuecomment-368055290,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFfQXJ1LnW_XQiKpH4HT-HfdHvCsxl6qks5tXuNbgaJpZM4SRFf2
.

Okay i will try to do so


win: work around sharepoint scandir bug

It has been reported that for SharePoint connections mapped as a drive,
uv_fs_scandir() returns "." and ".." entries when the expectation is
that they should be filtered out.
After some investigation it looks like the driver returns ".\0" and
"..\0" for those entries, that is, it includes the zero byte in the
filename length.  Rewrite the filter to catch those entries as well.
Fixes: https://github.com/nodejs/node/issues/4002
R=@piscisaureus
Maybe a better fix is:
if (wchar_len > 0 && info->FileName[wchar_len - 1] == L'\0'))
  wchar_len -= 1;

Thoughts?
CI: https://ci.nodejs.org/job/libuv+any-pr+multi/195/

LGTM. I like your proposal of skipping the null byte like that, though I wonder if windows doesn't do any other esoteric things under some planetary alignments.

That was my thinking as well.  I'd like to hear @piscisaureus's opinion.

Looks like this fell through the cracks :-S Maybe we should land this, Ben?

Let's try pinging @piscisaureus one more time.

Ping :-)

Maybe @orangemocha could weigh in? Otherwise, LGTM.

I think the change works as it is, so LGTM. Though I also would prefer the alternative version, of eating up spurious '\0' chars at the end, it's a bit cleaner and it also avoids allocating the extra wchar later on.
If you decide to go that route, consider replacing the if with a while for added robustness, as in:
while (wchar_len > 0 && info->FileName[wchar_len - 1] == L'\0'))
  wchar_len -= 1;

Updated to use a simple while loop, PTAL.  CI: https://ci.nodejs.org/job/libuv-test-commit/33/

LGTM

LGTM


PageLayout doesn't extend the sidebar divider when extending the height.

Description
On GitHub, I've seen some pages using the PageLayout component, but there's a problem: when you can't scroll, the PageLayout doesn't stretch to fill the whole page, which means the divider ends up getting cut off halfway down the page. To fix this, I tried making the PageLayout fill up all the available space from its parent in a few instances, but this didn't solve the issue – the divider still gets cut off.

💬 Related internal conversation
Steps to reproduce
Check my CodeSandbox
Version
Latest
Browser
Safari

I could fix your issue by adding height: '100vh' in line 46 of your code.


fix: Move elevator closure alternate route description outside <s> tag to allow expected pronunciation of punctuation

Asana task: Elevator alert audio not reading periods
Polly (or the SSML spec, really) does not allow for nested <s> (sentence) elements. Internally, Polly converts period-delimited text inside SSML to sentence elements. This means that if you drop a full paragraph of text into a sentence element, it would be interpreted something like <s><s>first sentence</s><s>second sentence</s>...</s> which is not valid SSML. To avoid this, Polly silently ignores periods inside of <s> elements and we end up with a crazy run-on: <s>first sentence second sentence ...</s>.
This means we need to avoid putting the PIO-provided alternate route text inside of an <s> element to have it get read out as expected. This change makes that fix.

[ ] Needs version bump?

(Added Hannah as well in case Paul is busy with realtime_signs work)


Failure to connect to server with auth disabled, still results in error "No API token provided"

Problem: Currently package returns API request error even when server does not require authentication.
Cause: Excessive pre-request validation on token property from package settings.
Desired behavior: Provide a way to allow user to bypass this check when necessary, but still trigger the validation for legitimate cases.
Suggested fix: Allow token property to be empty or omitted from package settings if not required by provider. Either by explicitly setting "token": null, or by adding a separate property like "auth": false.

The error message is triggered by the exception raised on line 472 in the following code snippet.
https://github.com/yaroslavyaroslav/OpenAI-sublime-text/blob/be04e41555be38cadfab9f047f90a0d2aa7debe7/plugins/openai_worker.py#L468-L474

Simply ensuring that the token key exists in the plugin config and has any value of length > 10 seems to be a good workaround for now, though it looks like this should be pretty straightforward to fix.

WORKAROUND PUT HERE

As long as the docs updated this is actually not a bug but a feature to implement. Originally by some forgotten sense I made this check local, but I think it's really worth it to leave it to the remote server to decide whether or not token is required and if it if what format and length it should be. So I believe those checks could be safely deleted.

Fixed in 4.2.7


Fix flatten_indices with empty indices mapping

Fix #5038

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


File changes to add Rackspace CDN to the landing page

Changes the following files to add Rackspace CDN to the landing page.
Text for right pane - modified: src/docs/index.rst
Left nav pane - modified: src/site_source/_includes/docs-sidebar.html
Adds region -modified: src/site_source/regions/index.htmlNote: The file for the logo is located at https://7d69c24efa79ed2429e6-1e1c5ac10380ee72a01b057764b93d72.ssl.cf1.rackcdn.com/icon-cdn.png
I did not find where these are stored in the repo for developer.rackspace.com

Still pulling in too many files! Closing.


not able to add to twine format

Hi, does anyone have luck of adding this module into twine's story format? Tried web and local version, both keeps loading and never finish.

Apparently Twine made another breaking change on how story formats are imported.

https://bitbucket.org/klembot/twinejs/issues/348/twine-21-manual-installation-of-updated
Seems like they are working on fixing this issue. Next release will contain a fix.
Workaround for now is using a legacy version of twine (offline)

thanks,  the latest version Twine got it fixed


New Scraper: Atlanta Board of Zoning Adjustments

Create a new scraper for Atlanta Board of Zoning Adjustments
Website: https://www.atlantaga.gov/government/departments/city-planning/office-of-zoning-development/zoning/board-of-zoning-adjustment-bza
Jurisdiction: City of Atlanta
Classification: Development
The Board of Zoning Adjustment is a quasi-judicial board of five appointed members that hears applications for variances and special exceptions from the zoning ordinance, as well as appeals of administrative decisions. When making a decision concerning an application, the BZA takes into consideration the recommendations of the Office of Zoning and Development staff and the Neighborhood Planning Unit (NPU) as well as testimony given at hearings.

Upcoming meetings: https://www.atlantaga.gov/home/showpublisheddocument/52804/637696324658970000


Readme Improvements

Please provide a summary of the feature
Readme has been one of the more antiquated development portions and could use some improvements. I'd like the enhancements to reinvent readme into a more approachable, minimal, and visually appealing one.
Some features that I would like to see, but are not limited to just the list below.

Removal of setup instructions from Readme and direct users to the contributing.md and wiki
Add some beautification to it! Buttons, photos, hopefully matching Natalie's design color.
Add deploy to cyclic button
Remove tasks

PR directly into the development branch
This has been assigned to @praxeds - Please coordinate with her if you're interested in helping her out.

In the works already 💪🏼

Completed!


Yapf format sometime not working !!!!

Environment data
VS Code version:  1.8.1
Python Extension version: 0.8
Python Version:   3.6.2
OS and version:    windows 10
Actual behavior
yapf format not working occasionlly
Expected  behavior
format the python source code
Steps to reproduce:
Set vscode format when save
and modify python source code when save

Logs
Output from Python output panel

Output from Console window (Help->Developer Tools menu)

workbench.main.js:sourcemap:10 spawn ctags ENOENT: Error: spawn ctags ENOENT
	at exports._errnoException (util.js:1050:11)
	at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
	at onErrorNT (internal/child_process.js:367:16)
	at _combinedTickCallback (internal/process/next_tick.js:80:11)
	at process._tickCallback (internal/process/next_tick.js:104:9)
e.onUnexpectedError @ workbench.main.js:sourcemap:10
workbench.main.js:sourcemap:9 [Extension Host] (node:440) DeprecationWarning: os.tmpDir() is deprecated.


several issues involving action toggle-preview-wrap

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[x] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
This is how I'm testing preview wrapping
fzf --preview="cat {}" --bind "ctrl-w:toggle-preview-wrap"

In versions 0.25.0 and master pressing ctrl-w produces no effect; the action doesn't seem to work at all.
In version 0.23.1 it's working as expected, meaning:

ctrl-w is a toggle (wrap/unwrap)
wrapping spaces all lines correctly, including blank lines
unwrapping spaces all lines correctly, including blank lines

In versions 0.24.{0..4} I have seen two different issues depending on the version

ctrl-w wraps once then it doesn't unwrap (no toggle)
wrapping spaces line incorrectly, meaning that an existing blank line is written over by the wrapped line above it

In summary, action toggle-preview-wrap has stopped working and the last good version was 0.23.1.

Thanks for the report. Should be fixed on master!

Fix confirmed on release 0.25.1.   Thank you!


[Don't Merge] Draft PR Fix passwordlessIT

/azp run java - spring - tests

/azp run java - spring - tests

Hi @benbp
we want to run our IT:java - spring - tests on China region, but it failed with following errors
"details": [
        {
          "code": "ProvisioningDisabled",
          "message": "This subscription is restricted from provisioning MySQL servers in this region. Please choose a different region or open a support request with service and subscription limits (quotas) issue type."
        }

Could you help look at this problem?


Fix equality typos in Documentation for Ryy, Rxx and Rzx gates

Summary
This PR resolves #13225 by fixing the typos in the documentation for the Ryy, Rxx and Rzx gates.
Details and comments
If I am not mistaken, these are the proper equalities. I also checked the result with numpy:
>>> import qiskit
>>> import numpy as np
>>> Ryy = qiskit.circuit.library.standard_gates.RYYGate(theta=np.pi)
>>> np.isclose(np.array(Ryy), -1j*np.kron(np.array(Y), np.array(Y)))
array([[ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True]])
>>> Rxx = qiskit.circuit.library.standard_gates.RXXGate(theta=np.pi)
>>> np.isclose(np.array(Rxx), -1j*np.kron(np.array(X), np.array(X)))
array([[ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True]])
>>> Rzx = qiskit.circuit.library.standard_gates.RZXGate(theta=np.pi)
>>> np.isclose(np.array(Rzx), -1j*np.kron(np.array(X), np.array(Z)))
array([[ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True]])

Thanks for the PR @jofrevalles, the fix looks good! The test failures seem unrelated, that shouldn't be triggered by a change in the docstring.
To continue with the PR, could you sign the CLA in the message above? 🙂

Thanks for the PR @jofrevalles, the fix looks good! The test failures seem unrelated, that shouldn't be triggered by a change in the docstring.
To continue with the PR, could you sign the CLA in the message above? 🙂

Done! Thanks @Cryoris

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.2%) to 88.988%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
4
93.23%

Totals

Change from base Build<PHONE_NUMBER>5:
0.2%

Covered Lines:
55314

Relevant Lines:
62159

💛  - Coveralls


azcopy.exe batchfile has Cannot perform sync due to error: cannot list blobs error

azcopy version 10, running from CMD manually is working perfectly, but if run it from batch file has got the following error,
Cannot perform sync due to error: cannot list blobs. Failed with error -> github.com/Azure/azure-storage-blob-go/azblob.newStorageError, /home/vsts/go/pkg/mod/github.com/!azure/azure-storage-blob-go@v0.7.0/azblob/zc_storage_error.go:42
===== RESPONSE ERROR (ServiceCode=AuthenticationFailed) =====
Description=Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.
RequestId:d18e490a-601e-0049-764a-a7dd37000000
Time:2019-11-30T06:53:15.3658143Z, Details:
AuthenticationErrorDetail: Signature size is invalid
Code: AuthenticationFailed
GET https://v6buildarchive.blob.core.windows.net/test?comp=list&include=metadata&restype=container&se=2022-11-29t14%3A35%3A31z&sig=-REDACTED-&sp=rwdlacup&spr=https&srt=sco&ss=bfqt&st=2019-11-30t06%3A35%3A31z&sv=2019-02-02&timeout=901
User-Agent: [AzCopy/10.3.2 Azure-Storage/0.7 (go1.13; Windows_NT)]
X-Ms-Client-Request-Id: [3c78577c-68cd-4d4f-58f7-e3c31e899186]
X-Ms-Version: [2018-03-28]
RESPONSE Status: 403 Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.
Content-Length: [400]
Content-Type: [application/xml]
Date: [Sat, 30 Nov 2019 06:53:14 GMT]
Server: [Microsoft-HTTPAPI/2.0]
X-Ms-Error-Code: [AuthenticationFailed]
X-Ms-Request-Id: [d18e490a-601e-0049-764a-a7dd37000000]
anyone has a idea about it? storage blob contributor permission has been given to azure account as well. run it from AD admin account.

Question : When running the batch file, are you logged in as the same user as when running it directly from CMD?
One thing to try would be running it from a Powershell script. If that works, it would suggest the problem in the batch file is possibly something about string quoting.

@JohnRusk I have used the same Domain admin account when I run the batch file. I tried to run same azcopy command in windows power shell manually,  there is no error at all.
This is only given above error when run in batch file.

@fredsofttech Sounds like it might be this problem, or something like it: https://github.com/Azure/azure-storage-azcopy/issues/177#issuecomment-474349070

@JohnRusk thanks so much for your hint here， John, it's all sorted.


Update IoT Agent North Ports

Is your feature proposal related to a problem? Please describe.
No
Describe the solution you'd like
Create standard for IoT Agent North Ports

Get applications/devices list
Get application/device

Covered by another repository


Update key-vault.md

Corrected typo

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/multitenant-identity/key-vault.md
:white_check_mark:Succeeded
View

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


Fixing incorrect mapping generation when ShallowCopyForSameType, UseDestinationValue and UseDestinationValue are used

This PR Resolves #599 and resolves #600.
Originally the destination transform was applied also when destination value was used creating not-compilable code.
Also when UseDestinationValue and ShallowCopyForSameType were set, then were not called and mapping was not generated

Nice work! Thank you!


Adds a requestFilter parameter to allow changing the outgoing request

The v2 user posts API changed to only accept multipart form data, leaning on the browser's inbuilt support for this.  Node doesn't have such support built in but you can use the form-data module to mimic it.
Problem is you need to set the Content-Type header on the outgoing request at runtime to something appropriate.  This PR adds a requestFilter option to all resource methods, similar to the existing responseFilter one.
Probably should be a minor version bump.

Closed as #105 does the job better while we let the internal stuff settle.


cmatrix: 1.2a -> 2.0

Motivation for this change
fixing packages rryantm can't update
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @AndersonTorres

nix-review and binary work here.


v0.8.0 release

sorry for the long wait :)

Here's the final changelog:
v0.8.0

Added FROST implementation to schnorr_fun.
Update MuSig implementation to latest spec
Make Point serialization and hashing consistent (use 32 byte form)
Add to_xonly_bytes and from_xonly_bytes to Point<EvenY>
Allow Zero points to serialize
Remove requirement of CryptoRng everywhere
Rename from_scalar_mul to even_y_from_scalar_mul to be more explicit
Remove XOnly in favour of Point<EvenY>
Replace .mark system with methods for changing each marker type.
Make From<u32> work for Scalar regardless of secrecy
Merge AddTag and Tagged into one trait Tag
Add NonceRng impls for RefCell and Mutex
Add Ord and PartialOrd implementations for (public) Scalar and Point
Change the from_bytes type commands to not assume secrecy in Scalar and Point.
Update to rust-secp256k1 v0.25.0


image transformation doesn't work

hello,
Image transformation doesn't work if I use Force webp.
Image URL: https://i.imgur.com/xjaMIKB.png
image URL with statically and width setting: https://cdn.statically.io/img/i.imgur.com/xjaMIKB.png?w=200
image URL with statically, width and force webp setting: https://cdn.statically.io/img/i.imgur.com/xjaMIKB.png?w=200&format=webp
This problem I got for more than a week. I thought it was a temporary error.
Thanks.

I'm having this issue as well when using width and WebP together.
This URL (without WebP) changes the width to 700:
https://cdn.statically.io/img/codetheweb.blog/assets/img/posts/javascript-dialogs/cover.png?w=700
This URL (with WebP) does not:
https://cdn.statically.io/img/codetheweb.blog/assets/img/posts/javascript-dialogs/cover.png?w=700&format=webp

Hi there,
Please use f=auto instead. Even though format=webp still works, it will be abandoned in the future. Details on upcoming Docs. Follow @staticallyio for updates.
Cheers,
Frans

Thanks so much, that fixed it for me! ❤

Awesome!


fix : Modify code generation script to include DCS Concerto model types

Closes #943

Modifed code generation script to include DCS Concerto model types
Changes

Modify code generation script to include DCS Concerto model types

Related Issues

Issue #943

@sanketshevkar hey , i had to make little changes to make it work , but i have pushed the commits after running codegen script

@sanketshevkar hey , sorry the build was failing cause i changed the exports , i have fixed them in the recent commit . What do i do next ?


Bug Report

In exchange for that piece of cake, I'll run it on linux and provide a bug report.

If you can also provide a build for the downloads section, we have a deal.


scheduling two tasks at the same time.

Hi, I'm trying to schedule two tasks at the same time on the same date that both require a particular resource whose isNotReservable field is set to true. As far as I understand, these two tasks should not conflict with one another; however, I can't get something like this to work. To be more explicit, the following data results in the scheduler successfully scheduling t2, but failing to schedule t1:
let tasks = [{
    id: "t1",
    duration: 1,
    minLength: 1,
    resources: ["r1"],
    available: later.parse.text("after 1:00am and before 1:01am")
}, {
    id: "t2",
    duration: 1,
    minLength: 1,
    resources: ["r1"],
    available: later.parse.text("after 1:00am and before 1:01am")
}];

let resources = [{
    id: "r1",
    available: later.parse.text("after 1:00am and before 1:01am"),
    isNotReservable: true
}];

let start = new Date("2020-01-01");
let frequency = later.parse.text("on the 1 day of January in 2020");
let s = schedule.create(tasks, resources, frequency, start);

Is this expected behavior? Or, perhaps I'm doing something wrong?

@mgoldenberg Were you able to solve this? I'm running into the same problem in my early testing of this library. I know this project is no longer maintained but I haven't been able to find an alternative. Thanks in advance.

@mgcox2: I tried and tried, but to no avail. If you manage to find a solution, do let me know.

@mgoldenberg: Try adding the priority option to your tasks. The task with the higher priority should be scheduled, the other should not.

@mgoldenberg have you found a solution to this issue?

Unfortunately, I never did. I had started digging through the source, but then I got bored... alas.


chore(types): Add JSDoc To Compiler, Compilation, and Connected Classes Types

What kind of change does this PR introduce?
Chore/Types

Did you add tests for your changes?
N/A

If relevant, link to documentation update:
N/A

Summary
Add Typing support to Compiler, Compilation. In this process I typed connected classes that would error, because of type issues. The TypeGraph gets pretty deep and other files.

Does this PR introduce a breaking change?
No

Other information

No worries I'll sit on it.

Let's see how this merge goes 😬

blocked by https://github.com/webpack-contrib/uglifyjs-webpack-plugin/pull/317

Still strange these errors are showing up. I'd fix but I'm not sure what is causing them or where to look.

Seems like Jest snapshot mismatch.

I'll give it a shot now.
On Tue, Jun 26, 2018, 9:34 AM Mohsen Azimi<EMAIL_ADDRESS>wrote:

Seems like Jest snapshot mismatch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webpack/webpack/pull/7299#issuecomment-400379990, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADQBMJpdW5ZDp2xgZ-XUWgJA0c5baYfkks5uAmKQgaJpZM4T-Uwk
.

WOOOHOOOO Travis is passing!!! 🎉

I'll try and pick these errors up tonight.

Thanks for cleaning up the remaining bits.
On Fri, Jun 29, 2018, 10:16 AM Tobias Koppers<EMAIL_ADDRESS>wrote:

@sokra approved this pull request.
Let's merge it. There is probably stuff missing, but afaik nothing is
wrong.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webpack/webpack/pull/7299#pullrequestreview-133308718,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADQBMGcTAAFpldHKc_YpONtdnxs8QSdGks5uBmDzgaJpZM4T-Uwk
.

Wait on Travis pass and merge

Great job everyone! this was a big one!

Thanks for getting the final fixes, I've been swamped. This is so exciting to see ready. Now plugins can really be typed well.


[filter] clear filter onXClick

Summary
Summary the request and address the questions as appropriate.

What is the change?
clear filter onXClick
Why should it go in Clarity?
Does this change impact existing behaviors? If so how?
Currently clicking on X, the filter field will disappear and the filter text/criteria will remain
If this change introduces a new behavior, is this behavior accessible?

Use case
Currently, the user has to manually click into the input field and delete the characters that are supposed to be filtered
Examples
Are there any examples in other projects that demonstrate this feature or enhancement?

Workarounds
Is there a workaround for this feature at the moment, even if it is less than ideal?
partial workaround: clear all filters button

Any effort towards improving the UX for datagrid filtering should be put towards #1632.
This is very specific and, while applying to your specific case, would cause problems for other apps as it overloads the closing of the existing filter panels.
This overloading would make filtering unusable for users who rely on keyboard navigation.


unix,stream: don't clear writable flag on shutdown

12be29f correctly updated uv_shutdown() to clear the writable
flag on Unix platforms. However, the change is causing test
failures in Node.js. This commit reverts the change so that the
1.39.0 release is unblocked.

Node+libuv integration CI: https://ci.nodejs.org/view/libuv/job/libuv-in-node/159/
libuv CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/2013/

None of the CI failures appear to be related, and macOS passed.

I agree with Ben here. I'd revert the whole commit and investigate what's going on.

Closing this in favor of a full revert in https://github.com/libuv/libuv/pull/2968.


Frigate labels from double take do not show in lovelace card

So double take has the option to send recognised sublabels back to Frigate. This shows in frigate, but doesn't seem to show in the lovelace card.
Not sure why, seems to pull the same event log?

frigate calls them sublabels, it would be a great feature to have

Quick clarification: Do you just want to see them on events, or be able to search by them, or both?

In the frigate ui you can do both... if you want to see when "x" got home it is nice to put in their name and the doorbell cam to find the answer quickly.

Will be in the next build (requires new builds of Frigate itself, the integration, and the card).


Icons in header documentation

Documentation specifies 'not recommended' for icon only in header then displays an icon only header in the examples.
https://clarity.design/documentation/header
Recommendation:

Header examples:

Hello. I would like to be assigned to this issue. I will need further clarification though: is it enough to pull out of the webpage the icon-only example?

Hi Andrei,
You should remove both the icon-only image example, and the corresponding section of the sample code below the header images.
Thanks for helping with this!

Should it also be taken out of the old documentation?

Thanks for so quickly submitting the change.
As you also updated the dev-app page, now Set 3 of the visual tests is failing.
Please check if the failing tests are relevant now and change them if needed or update the reference images.
Write back if you need any help with this.

Since I removed the example in the middle with the logo icons this is why the test is failing. I would like to know what is the best way to edit these images so that the tests pass?

Hi @andreiconstantinbytex - it looks like your changes affect the reference images for our visual regression tests in set 3.
✘ navigation header-types default [chrome]
✘ navigation header-types header-dropdown [chrome]

You can update the reference images by running the command:
npm run fix:visual:light set3 && npm run fix:visual:dark set3

NOTE: you will need to have Docker installed and running on your system in order to run the gemini regression tests. You can read more here or let me know if you have a specific question.


Update README.md

Adding myself to contributors
:warning: This is a Pull Request Template :warning:

Checkout all the things you've done by placing an x within the square braces below.

Proposed Changes

[x] I've read the CODE OF CONDUCT and abide to it.
[x] I've read the CONTRIBUTING.md
[x] I've forked the repository.
[x] I've created a branch and made my changes in it.
[x] I understand opening a PULL REQUEST doesn't mean it will be merged for sure.

Sorry! Already made the requested changes :)


Is this project dead?

Dang this was already asked 2 years ago 🙈 Well here is something fresh:
https://github.com/sass/libsass-python
cheers!

Yep, I have been using that one ;-)


Creates empty divs at the bottom of my page

When I use this directive in my code, it creates empty divs at the bottom of my page for every upload element/button/anchor I create above, and these elements sit 2174 pixels below my original element/button/anchor.  The divs have an absolute positioning with transparent background and opacity of 0.  Any clue what is going on here?

I know it's very late but I am here.
Any sample code? http://jsfiddle.net/ can be of some help.


[Bug]: jest.useFakeTimers ignores fakeTimersConfig with create-react-app

Version
29.5.0
Steps to reproduce

Run npx create-react-app myapp && cd myapp
Add this to dependencies in package.json:"jest": "^29.5.0",

Run npm update --save
Add this code to src/App.test.js:test('jest.useFakeTimers', () => {
  let now = new Date('2023-06-04T22:00-04:00');
  jest.useFakeTimers({now: now});
  expect(new Date()).toEqual(now);
});

Run npm test

Expected behavior
I expect the test to pass.
Actual behavior
The test fails. The error log says:
    expect(received).toEqual(expected) // deep equality

    - Expected  - 1
    + Received  + 1

    - 2023-06-05T02:00:00.000Z
    + 2023-06-05T03:21:12.376Z

      11 |   let now = new Date('2023-06-04T22:00-04:00');
      12 |   jest.useFakeTimers({now: now});
    > 13 |   expect(new Date()).toEqual(now);

2023-06-05T03:21:12.376Z is the time at which I ran the test.
When I open the test in a debugger and step into jest.useFakeTimers, I see this code:
const useFakeTimers = (type = 'modern') => {
  if (type === 'legacy') {
    this._fakeTimersImplementation = this._environment.fakeTimers;
  } else {
    this._fakeTimersImplementation = this._environment.fakeTimersModern;
  }

  this._fakeTimersImplementation.useFakeTimers();

  return jestObject;
};

The behavior of this code is different from what is described in the docs. The code seems to take a string instead of a fakeTimersConfig argument.
Additional context
No response
Environment
System:
    OS: macOS 13.3.1
    CPU: (8) arm64 Apple M2
  Binaries:
    Node: 20.2.0 - /opt/homebrew/bin/node
    npm: 9.6.6 - /opt/homebrew/bin/npm
  npmPackages:
    jest: ^29.5.0 => 29.5.0

It seems like somehow I'm getting Jest 27 (which is the default when you initialize a project with create-react-app). The useFakeTimers('modern') call changed between Jest 27 and Jest 28. But I don't understand how this is happening.
I see Jest 29.5.0 in node_modules:
$ rg version node_modules/jest/package.json
4:  "version": "29.5.0",

I see implementations of useFakeTimers that take fakeTimersConfig in jest-runtime/build/index.js, @jest/fake-timers/build/modernFakeTimers.js, and @jest/fake-timers/build/index.d.ts.
However, I see an implementation of useFakeTimers with the type = 'modern' arg in react-scripts/node_modules/jest-runtime/build/index.js. So it looks like react-scripts is somehow keeping me on an older version of Jest?

I fixed this by running npm run eject and then upgrading Jest.


Evaluate eigenvalues in MinEigenOptimizer more efficiently, if possible

Summary
The evaluate_at_bitstr function used in the MinimumEigenOptimizer used a statevector simulation to compute eigenvalues of an operator. This was an intermediate solution, up until 0.7. is released. Now the CircuitSampler and the opflow can be used to evaluate this expression elegantly and more efficiently.
Also deprecates the LegacyBaseOperators as input.

Since the application modules today generate legacy operators, i.e. the converters, ising apps etc, until such time all these are changed too, to output opflow natively, I would have thought this use should preserved and not deprecated. We should make the change so everything is consistent and using opflow at the same time and not in bits. As it is the tests are now emitting deprecated warnings and presumably any other code using optimization would too.

I removed the warning in 285b3c0 👍


Hotfix viewport

Coverage remained the same at 58.577% when pulling 5a260cffac397503d8a9c8f75af7a4c18a9294c5 on hotfix-viewport into e03bb1a5b5f00703077ca7b9591fa0b0231c2010 on master.


[WIP] adds an error emit if the wifi chip hangs

I'm still having problems with WiFi on the Tessel, as per https://forums.tessel.io/t/tessel-stops-running-after-15-minutes/859/60.
Are you still planning on merging this PR?

@mihar we didn't get around to test this PR and now most of the community's development effort is towards Tessel 2. Would you be interested in testing out this patch and let us know if it fixes your issue?

@johnnyman727 I would be interested yes, since right now my Tessel is useless for what I wanted to use it

Fantastic - just make sure to check out this branch and follow the updating procedure to load the code on your Tessel.


Re: Add centerMode option (and add current() to public API)

Hi, @meandmax.
I fixed #11 #16.
Delete setCurrentSelector function and "js_current" & "js_clone" selectors (because it was not necessary).
Please let me know if you have advice or questions.
Thanks.

@meandmax
Didn't fix following bugs, sorry. I fix bug "center slide isn't centered correctly" as soon.
https://github.com/meandmax/lory/pull/16#issuecomment-94311486

For positioning maybe something like this:
(slides[index].offsetLeft - (frameWidth / 2) + (slides[index].clientWidth / 2)) * -1

instead of:
slides[nextIndex].offsetLeft * -1

cheers Max

@meandmax
Thanks for advice, and change code at https://github.com/meandmax/lory/blob/master/src/lory.js#L315.
before:
var nextOffset = limitOffset(slides[nextIndex].offsetLeft * -1);

after:
var nextOffset = limitOffset(slides[nextIndex].offsetLeft * -1);

if (options.centerMode) {
    nextOffset = limitOffset((slides[index].offsetLeft - (frameWidth / 2) + (slides[index].clientWidth / 2)) * -1);
}

result:

It looks buggy. I continue to fix bug.

@meandmax
I fixed "center slide isn't centered correctly" bug c9b9f4dc. I guess it worked.

@meandmax
I removed current function, because I felt it unnecessary and not casually add function to public API.

Thanks a lot @kubosho, it`s getting better! Still the slide is not centered correctly and I was also wondering why it is necessary to use centerMode in combination with infinite.
I really think I have to rewrite a couple of things to get this feature up and running without infinite while keeping the current size of the slider.
cheers Max

@meandmax
I fixed "slide is not centered correctly" to 2e64c964b17a:

jsfiddle: https://jsfiddle.net/Lm8L8r51/11/embedded/result/

I really think I have to rewrite a couple of things to get this feature up and running without infinite while keeping the current size of the slider.

Which of the following is closest in meaning?

delete code to https://github.com/kubosho/lory/blob/feat/center-mode/src/lory.js#L198-L200
http://codepen.io/kubosho_/pen/yNLNEe the look like be rewritten


[Benchmark] Added MMGenBench benchmark

[Benchmark] Added MMGenBench benchmark

hey @kennymckormick do you think it's ready to merge?

hey @kennymckormick do you think it's ready to merge?

Please help fix the lint first

hey @kennymckormick do you think it's ready to merge?

Please help fix the lint first

It's the prompt passed to MLLM, how to fix it?

hey @kennymckormick do you think it's ready to merge?

Please help fix the lint first

It's done

Hi, @lerogo ,
I didn't find any documents  / descriptions about the MMGenBench you contributed on the Internet. Please add some introductions to the  benchmark somewhere. If the benchmark is just for your personal use, you can also keep it in your own fork (no need to create a PR).

Hi, @lerogo , I didn't find any documents / descriptions about the MMGenBench you contributed on the Internet. Please add some introductions to the benchmark somewhere. If the benchmark is just for your personal use, you can also keep it in your own fork (no need to create a PR).

We are sorry, we are working as quickly as possible to perfect our code and documentation. Below is the link to our paper: https://arxiv.org/abs/2411.14062

Hi, @lerogo , I didn't find any documents / descriptions about the MMGenBench you contributed on the Internet. Please add some introductions to the benchmark somewhere. If the benchmark is just for your personal use, you can also keep it in your own fork (no need to create a PR).

We are sorry, we are working as quickly as possible to perfect our code and documentation. Below is the link to our paper: https://arxiv.org/abs/2411.14062

Cool, will try to run the evaluation and then merge it.

Thanks! I've uploaded the code. Is it possible to add a link to our benchmark inside the VLMEvalKit homepage?
https://mmgenbench.alsoai.com/

Thanks! I've uploaded the code. Is it possible to add a link to our benchmark inside the VLMEvalKit homepage?
https://mmgenbench.alsoai.com/

Added a new line in the news section.

dd


how can i use this options??

hello
i have one question
how can i use this options?
Options : 'plotOptions.series.data.draggableX' AND 'plotOptions.series.data.draggableY'
Description : 'If dragging is enabled on the series, set this to false to prevent dragging on a single point.'
can i take sample code about this options?
thank you

Hi @rldbs925
You can find sample code from the official demos, for example: http://jsfiddle.net/highcharts/AyUbx/


Change RHDH Operator API group to rhdh.redhat.com

Due to the retirement of Janus-IDP, we need to change the API group for RHDH Operator. Since this is a new operator, we propose to change the API group from janus-idp.io to rhdh.redhat.com.
Several other operators in redhat-developer use the same name convention:

RHOAS Operator: rhoas.redhat.com
Observability Operator: observability.redhat.com
Application Services Metering operator: applicationservices.redhat.com

Note that there are operators (GitOps, Service Binding, Web Terminal) that keep their existing API group due to upstream or historical reasons.

Closing this now that https://github.com/janus-idp/operator/pull/201 is merged (and ported to the main branch as well)
/close


Update efs-csi-driver.md

When doing the lab for the efs provisioner the page doesn't match up with the youtube video.  Following the directions I got an error because the storage namespace didnt exist.  I created the namespace manually and the rest of the lab was fine.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

fixed in #967


[Breaking change]: .NET 9 enables CET support by default

Description
In .NET 9 Preview 6, the apphost and singlefilehost started to be marked as Intel CET compatible (they are compiled with /CETCOMPAT option). It was done to enhance security of .NET applications. That imposes a limitation on shared libraries that .NET applications can load and interop with. They are not allowed to set thread context to a location with instruction pointer that is not present on the shadow stack or in a table of allowed continuation addresses for exception handling.
Version
Other (please put exact version in description textbox)
Previous behavior
Shared libraries loaded into .NET process were able to set thread context using SetThreadContext, RtlRestoreContext/NtContinue or using their exception handlers to any location in the process address space.
New behavior
Shared libraries loaded into .NET process are only allowed to set thread context using SetThreadContext, RtlRestoreContext/NtContinue or using their exception handlers to locations that are present on the shadow stack or in a table of allowed continuation addresses for exception handling (generated by /EHCONT compiler option or the SetProcessDynamicEHContinuationTargets API). Trying to change a thread context to any other location results in termination of the process.
Type of breaking change

[X] Binary incompatible: Existing binaries might encounter a breaking change in behavior, such as failure to load or execute, and if so, require recompilation.
[ ] Source incompatible: When recompiled using the new SDK or component or to target the new runtime, existing source code might require source changes to compile successfully.
[ ] Behavioral change: Existing binaries might behave differently at run time.

Reason for change
Enabling CET for .NET applications enhance the security of the .NET applications by adding hardware-enforced stack protection that offers robust protection against ROP exploits (return oriented programming).
Recommended action
Workarounds:

Adding <CETCompat>false</CETCompat> to the .csproj of the application opts out of the CET
Use the Windows Security app, group policy etc. to opt out of the hardware supported stack enforcement for the specific .NET application as described in https://learn.microsoft.com/en-us/defender-endpoint/enable-exploit-protection for details.

Feature area
Interop
Affected APIs
No response

In the description, we can include a link to https://techcommunity.microsoft.com/t5/windows-os-platform-blog/developer-guidance-for-hardware-enforced-stack-protection/ba-p/2163340 that provides additional technical details.
The primary recommended actions should be ones that do not compromise security: fix the native library to be CET compatible, or to request the fix from the native library maintainer; or avoid dependency on the offending native library.


Annournce new traceroute BQ table

This change is

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 15 at r1 (raw file):

Everyone wants traceroute data in BigQuery has quick easy way to reconstruct the path of hops for the same test for years. But the schema designed many years ago put the hops of the same test in different rows, which made this task particularly hard.

To address this need from many of our partners and researchers, M-Lab is delightful to announce that BigQuery tables of traceroute data with new schema is available to public now. The new schema has one test per row, and all hops of this test inside the same row.

Suggested changes:
To make our traceroute data in BigQuery more useful, researchers have sought an easy way to reconstruct the path of hops{:target="_blank"} for the same test. This task was particularly hard because the schema, which was designed many years ago, put the hops of the same test in different rows.
To address this need from many of our partners and researchers, M-Lab is delighted to announce that the traceroute BigQuery table in the aggregate dataset is now available to the public. The new traceroute schema has one test per row, and all hops for a single test are inside the same row.

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 15 at r1 (raw file):
Previously, critzo wrote…
Suggested changes:
To make our traceroute data in BigQuery more useful, researchers have sought an easy way to reconstruct the path of hops{:target="_blank"} for the same test. This task was particularly hard because the schema, which was designed many years ago, put the hops of the same test in different rows.
To address this need from many of our partners and researchers, M-Lab is delighted to announce that the traceroute BigQuery table in the aggregate dataset is now available to the public. The new traceroute schema has one test per row, and all hops for a single test are inside the same row.

the formatting in the above is not showing correctly:
To make our traceroute data in BigQuery more useful, researchers have sought an easy [way to reconstruct the path of hops][1]{:target="_blank"} for the same test. This task was particularly hard because the schema, which was designed many years ago, put the hops of the same test in different rows.

To address this need from many of our partners and researchers, M-Lab is delighted to announce that the `traceroute` BigQuery table in the `aggregate` dataset is now available to the public. The new `traceroute` schema has one test per row, and all hops for a single test are inside the same row.<!--more-->

[1]: https://nemelor.wordpress.com/2018/08/28/access-to-mlab-traceroute-data-from-google-bigquery/

The above makes some grammar suggestions, adds a link to previous external researchers' work attempting to order traceroute hops, and adds the  tag which signals where the front page summary of the post should be cut off.

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 17 at r1 (raw file):

To address this need from many of our partners and researchers, M-Lab is delightful to announce that BigQuery tables of traceroute data with new schema is available to public now. The new schema has one test per row, and all hops of this test inside the same row.

Here is the new schema, which also adds ASn annotation for traceroute source, destination and hops:

Suggest capitalizing N in ASN, and add a comma after "destination"
Here is the new schema, which also adds ASN annotation for traceroute source, destination, and hops:

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 25 at r1 (raw file):

![schema_3]({{ site.baseurl }}/images/blog/2019-08-01-traceroute/schema3.png)

The new BigQuery table preserves all information in legacy tables, with better Geolocation annotation coverage using time-based Maxmind databases. The Geolocation annotation rate for traceroute source and destination are 100%. For traceroute tests since 2017, the Geolocation annotation rate for hops almost doubled from ~20% in legacy BigQuery table, to >40% in current BigQuery table. The ASN annotation rate is about 97%.

When you say "legacy tables" which tables do you mean? base_tables.traceroute, base_tables.traceroute_legacy or another? I would like to provide the full table name with the dataset and possibly link to it in BigQuery.

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 25 at r1 (raw file):
Suggested text changes in this paragraph:

The new BigQuery table preserves all information that was in the previous table, measurement-lab.base_tables.traceroute, with better geo-location annotation coverage using time-based Maxmind databases. The geo-location annotation rate for traceroute source and destination are 100%. For traceroute tests since 2017, the geo-location annotation rate for hops almost doubled from ~20% in the BigQuery table, to >40% in current BigQuery table. The ASN annotation rate is about 97%.

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 15 at r1 (raw file):

To make our traceroute data in BigQuery more useful, researchers have sought an easy way to reconstruct the path of hops{:target="_blank"} for the same test. This task was particularly hard because the schema, which was designed many years ago, put the hops of the same test in different rows.
To address this need from many of our partners and researchers, M-Lab is delighted to announce that the traceroute BigQuery table in the aggregate dataset is now available to the public. The new traceroute schema has one test per row, and all hops for a single test are inside the same row.

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 27 at r1 (raw file):
This heading should have two # signs since the top heading is added automatically. Also suggested using 'Sample Queries' as the heading :

Sample Queries

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 29 at r1 (raw file):
Suggested edit

Here are some sample queries to access the traceroute table with the new schema:

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 49 at r1 (raw file):

GROUP BY ts
ORDER BY ts DESC

Suggested formatting. You can enclose sql queries in a block like below so they are formatted as sql. Also adjusted indentation to make it more readable. Heading is also changed to three levels in (###)
### Count how many traceroute tests per day given a time range
SELECT
  ts,
  COUNT(*) AS num
FROM (
  SELECT
    DATE(TestTime) as ts
  FROM `measurement-lab.aggregate.traceroute`
  WHERE
  DATE(TestTime) BETWEEN DATE("2016-01-01") AND DATE("2018-06-30")
)
GROUP BY ts
ORDER BY ts DESC

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 74 at r1 (raw file):

GROUP BY ts
ORDER BY ts DESC

Suggested formatting changes, similar to previous:
### Count rate of source and destination IP being annotated with geo-location information
SELECT
  ts,
  COUNTIF(s_long IS NOT NULL AND s_lant IS NOT NULL) / COUNT(*) AS s_geo_success_rate,
  COUNTIF(d_long IS NOT NULL AND d_lant IS NOT NULL) / COUNT(*) AS d_geo_success_rate
FROM (
  SELECT
    DATE(TestTime) as ts,
    Source.Geo.longitude AS s_long,
    Source.Geo.latitude AS s_lant,
    Destination.Geo.longitude AS d_long,
    Destination.Geo.latitude AS d_lant
  FROM `measurement-lab.aggregate.traceroute`
  WHERE
    DATE(TestTime) BETWEEN DATE("2016-11-01") AND DATE("2016-11-30")
)
GROUP BY ts
ORDER BY ts DESC

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 102 at r1 (raw file):

GROUP BY ts
ORDER BY ts DESC

Suggested formatting changes, similar to prev.
### Count how many hops were in “New York” per day.
SELECT
  ts,
  COUNTIF(hop_ip IS NOT NULL) AS total_hops,
  COUNTIF(hop_city = 'New York' ) AS num_ny_hops\
FROM (
  SELECT
    DATE(TestTime) as ts,
    Parseinfo.TaskFileName,
    hops.Source.IP AS hop_ip,
    hops.Source.City AS hop_city,
    hops.Source.CountryCode AS hop_country
  FROM
    `measurement-lab.aggregate.traceroute` AS traceroute,
    UNNEST(traceroute.Hop) as hops
  WHERE
    DATE(TestTime) BETWEEN DATE("2017-08-01") AND DATE("2017-08-31")
)
GROUP BY ts
ORDER BY ts DESC

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 129 at r1 (raw file):

GROUP BY ts
ORDER BY ts DESC

Suggested changes, similar to previous:
### Count how many hops were in a specific ASN
SELECT
  ts,
  COUNTIF(hop_ip IS NOT NULL) AS total_hops,
  COUNTIF(hop_asn = 15169 ) AS num_asn_hops
FROM (
  SELECT
    DATE(TestTime) as ts,
    hops.Source.IP AS hop_ip,
    hops.Source.ASN AS hop_asn
  FROM
    `measurement-lab.aggregate.traceroute` AS traceroute,
    UNNEST(traceroute.Hop) as hops
  WHERE
  DATE(TestTime)  BETWEEN DATE("2017-08-01") AND DATE("2017-08-31")
)
GROUP BY ts
ORDER BY ts DESC

_posts/blog/2019-08-01-traceroute-bq-newdata-available.md, line 156 at r1 (raw file):

GROUP BY ts
ORDER BY ts DESC

Suggested changes, similar to previous:
### Count the annotation rate of ASN for hops
SELECT
  ts,
  COUNTIF(hop_ip IS NOT NULL) AS total_hops,
  COUNTIF(hop_asn <> 0 )/COUNTIF(hop_ip IS NOT NULL) AS rate_asn_hops
FROM (
  SELECT
    DATE(TestTime) as ts,
    hops.Source.IP AS hop_ip,
    hops.Source.ASN AS hop_asn
  FROM
    `measurement-lab.aggregate.traceroute` AS traceroute,
    UNNEST(traceroute.Hop) as hops
  WHERE
    DATE(TestTime)  BETWEEN DATE("2017-08-01") AND DATE("2017-08-31")
)
GROUP BY ts
ORDER BY ts DESC

a discussion (no related file):
Hi @yachang -- I've made some suggested content and formatting changes here. With these, this post
If you agree with them I can push a commit and merge.


有没有对应支持、可用的富文本编辑器？

Your questions.

Environment.

Mobile model: iphoneX
Mobile system: iOS 8
Dev platform: darwin/windows/linux
Node: 8.x.x
Npm: 5.x.x

What do you think may be caused by the analysis of the problem.

Reappearance of the problem (It's best to have a screenshot).

没有，可以通过 webview 和 app 交互引入一个比较好的库


Documentation:  Make sure readme file is ready for release 0.1.0

Please select a category the issue is focused on?
No response
Let us know where something needs a refresh or put your idea here!
Sense check and make sure format is ok. Does anything else need to be added?

Hi Edoardo,
Please kindly find below for my review comments.

the link [{admiral} References and Documentation (https://pharmaverse.github.io/admiral/devel). doesn't work, page is 404 not found.
I am wondering if a Description or Purpose part is nice to have at the beginning of this page?
under the scope section, should we add currently available domains under it?

Hi @jiegaoo thanks for your review.  The next step would be for you to make a branch and make these updates!

Could you please find the
Love the idea, let's do it.
same.

Let me know if you have any questions or need any help.

@jiegaoo - thanks for your updates. as discussed, upon reflection the "Expectations" section in the readme is just repeating material exposed in the "Purpose" section, so I think it can be removed. Please could you also updated the "Purpose" section to say "To provide a complementary (to admiral) toolbox that enables users to develop ophthalmology disease area datasets and endpoints.
You can do the updates in your existing issue branch (no need to create a new one) as long as you bring it up to speed with what is in devel by running git merge devel in the console after having pulled devel. Please also re-build the site once done.


fix: Fix isssue on callbacks execution order #635

Create a unique thread to execute callbacks and fix issue on callbacks execution order #635

@pwespi Could we get a version with this fix? Thank you

@pwespi I really need this new version too please!

Thank you for the contribution!
Note the the API to set the custom handler is only available from API level 26. I added the version check.

@all-contributors please add @lhd-biosency  for code

This has been released with version 3.1.3.


add advanced rag example for astra

plug in llamaparse + recursive retrieval with astradb

@jerryjliu This is fantastic and worked ALMOST end to end for me. The only change i'd suggest is:
from llama_index.vector_stores.astra import AstraDBVectorStore

# define two storage classes representing two collections (to compare advanced approach vs. baseline)

astra_db_store_advanced = AstraDBVectorStore(
    token=ASTRA_TOKEN,
    api_endpoint=ASTRA_API_ENDPOINT,
    collection_name="astra_v_table_llamaparse_advanced",
    embedding_dimension=1536
)
astra_db_store_base = AstraDBVectorStore(
    token=ASTRA_TOKEN,
    api_endpoint=ASTRA_API_ENDPOINT,
    collection_name="astra_v_table_llamaparse_base",
    embedding_dimension=1536
)

I'd remove the namespace portion from the calls in this cell, as i did here - it'll fall back to the default namespace - only because otherwise they have to ensure this is created in the first place. This is somewhat of a legacy component of our system, and as of now we're for the most part assuming everything is under the default namespace, and hence not providing that parameter.

good point! sounds good


When we try to open session credentials window is prompting.

When we try to open session credentials windows is prompting and it is not accepting the credentials.
Azure DevOps Version: On Prem Azure DevOps Server 2019 Update 1.1.

Hello, it seems there might be an issue with your environment. Are you using a VPN?

Hello, it seems there might be an issue with your environment. Are you using a VPN?

Yes we are using vpn. Application is hosted in secure vlan.

Turn off the VPN and test it. I have tested it in all our environments, and it has worked fine. If it doesn't work, we can schedule a web session.

We tried with office network as well. It is the same.


Bose QC35 not entering DFU mode

Hello!
I tried to downgrade Bose QC35 but it seems to me that it can't go into DFU mode. Can you give an advice? Thanks
C:\Users\user\Downloads>bose-dfu-v1.1.0-win-x86_64.exe info
USB serial: 072546Z62705849AE
HW serial: 072546Z62705849AE
Device model: Wolfcastle
Current firmware: Main <IP_ADDRESS>8

C:\Users\user\Downloads>bose-dfu-v1.1.0-win-x86_64.exe enter-dfu -f
[INFO  bose_dfu] Note that device may take a few seconds to change mode

C:\Users\user\Downloads>bose-dfu-v1.1.0-win-x86_64.exe info -f
[WARN  bose_dfu] Device has not been tested with bose-dfu; by proceeding, you risk damaging it
[WARN  bose_dfu] Cannot determine device's mode; command may damage devices not in normal mode
USB serial: 072546Z62705849AE
Error: USB transaction error while requesting info field

Caused by:
    hidapi error: A device attached to the system is not functioning.

C:\Users\user\Downloads>

It looks to me like it did enter DFU mode. info only works for devices in non-DFU mode, which is why it failed when you forced it to run against the DFU-mode device with -f. list is the command that'll show what devices are connected and what mode each one is in. The current release of bose-dfu will report the QC35 II's DFU mode as "unknown mode", but you should still be able to run download if you use -f.
Be aware that the QC35 II seems to have an additional "ext" firmware that bose-dfu isn't able to flash yet, which means it might not work as expected once you downgrade it. See #6 for details. Based on others' reports, it shouldn't brick your device or make it unusable though, so feel free to try and report back. If you run into issues, you can always use bose-dfu to flash 2.5.5 and restore your device to its current state.

You are right. The device was in DFU mode.
I have two QC35 devices, not QC35 II. Anyway, the firmware looks similar, both have the "ext" component.
I made downgrade from v3.0.3 to v2.5.5, and then from v2.5.5 to v1.0.6. When downgrading to 1.0.6 the headset lost the pairing list but I did pair again and everything is fine. I don't know about the "ext" component of the firmware (which is almost 10 times bigger than the DFU file), it seems to me that is not needed. Tested with two melodies, sounds good so far. Thank you!
v2.5.5 -> v1.0.6
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe info
USB serial: 072545Z71115050AE
HW serial: 072545Z71115050AE
Device model: Wolfcastle
Current firmware: Main <IP_ADDRESS>8
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe enter-dfu -f
[INFO  bose_dfu] Note that device may take a few seconds to change mode
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe list
05a7:400c 072545Z71115050AE Bose QuietComfort 35 [UNTESTED device in unknown mode]
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe download Wolfcastle_1.0.6_stack_plus_app.dfu -f
[WARN  bose_dfu] Device has not been tested with bose-dfu; by proceeding, you risk damaging it
[WARN  bose_dfu] Cannot determine device's mode; command may damage devices not in DFU mode
[INFO  bose_dfu] Update verified to be for selected device
[INFO  bose_dfu] Beginning firmware download; it may take several minutes; do not unplug device
[INFO  bose_dfu::protocol] Waiting 44.33s, as requested by device, for firmware to manifest
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe leave-dfu -f
[WARN  bose_dfu] Device has not been tested with bose-dfu; by proceeding, you risk damaging it
[WARN  bose_dfu] Cannot determine device's mode; command may damage devices not in DFU mode
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe info
USB serial: 072545Z71115050AE
HW serial: 072545Z71115050AE
Device model: Wolfcastle
Current firmware: Main <IP_ADDRESS>5
C:\Kit\BOSE_DFU>
and back to v2.5.5 (for regaining noise cancellation setting in Bose Connect app)
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe info
USB serial: 072545Z71115050AE
HW serial: 072545Z71115050AE
Device model: Wolfcastle
Current firmware: Main <IP_ADDRESS>5
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe enter-dfu -f
[INFO  bose_dfu] Note that device may take a few seconds to change mode
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe list
05a7:400c 072545Z71115050AE Bose QuietComfort 35 [UNTESTED device in unknown mode]
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe download Wolfcastle_2.5.5_stack_plus_app.dfu -f
[WARN  bose_dfu] Device has not been tested with bose-dfu; by proceeding, you risk damaging it
[WARN  bose_dfu] Cannot determine device's mode; command may damage devices not in DFU mode
[INFO  bose_dfu] Update verified to be for selected device
[INFO  bose_dfu] Beginning firmware download; it may take several minutes; do not unplug device
[INFO  bose_dfu::protocol] Waiting 47.157s, as requested by device, for firmware to manifest
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe leave-dfu -f
[WARN  bose_dfu] Device has not been tested with bose-dfu; by proceeding, you risk damaging it
[WARN  bose_dfu] Cannot determine device's mode; command may damage devices not in DFU mode
C:\Kit\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe info
USB serial: 072545Z71115050AE
HW serial: 072545Z71115050AE
Device model: Wolfcastle
Current firmware: Main <IP_ADDRESS>8

Great, thanks for the report! Would you mind also sharing the output of list in non-DFU mode? With that information, I can mark the QC35 as partially supported in the next release just like the QC35 II.
Closing this issue since you were able to downgrade.

Here is the needed info:
`D:\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe info
USB serial: 072546Z62705849AE
HW serial: 072546Z62705849AE
Device model: Wolfcastle
Current firmware: Main <IP_ADDRESS>8
D:\BOSE_DFU>bose-dfu-v1.1.0-win-x86_64.exe list
05a7:40fe 072546Z62705849AE Bose QuietComfort 35 [compatible device in normal mode]
D:\BOSE_DFU>`


[WFLY-18204] Upgrade WildFly Core to 21.1.0.Beta2

Jira issue:
https://issues.redhat.com/browse/WFLY-18204

Tag: https://github.com/yersan/wildfly-core/releases/tag/21.1.0.Beta2
Diff: https://github.com/yersan/wildfly-core/compare/21.1.0.Beta1...21.1.0.Beta2

        Component Upgrade

[WFCORE-6305] -         Upgrade to Jandex 3.1.1

[WFCORE-6414] -         Upgrade bouncycastle from 1.73 to 1.75 (resolves CVE-2023-33201)

[WFCORE-6417] -         Upgrade to Jandex 3.1.2

[WFCORE-6418] -         Upgrade Elytron Web to 4.0.0.Final

        Enhancement

[WFCORE-6154] -         Log reasons for implicit module dependencies

[WFCORE-6369] -         Eliminate unnecessary collection copying in AbstractAddStepHandler constructors

Linux with security manager - JDK 11 failure is a legit error likely due to the BC upgrade, looking at it now.
Note: BC upgrade Jira had the wrong fix version, I've just corrected it and now it is shown in the details section above.


Yatin: Watch ^Best[\W_]+Office[\W_]+365[\W_]+email[\W_]+Signature[\W_]+software$

Yatin requests the watch of the watch_keyword ^Best[\W_]+Office[\W_]+365[\W_]+email[\W_]+Signature[\W_]+software$. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

^Best[\W_]+Office[\W_]+365[\W_]+email[\W_]+Signature[\W_]+software$ has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Rejected by tripleee in Charcoal HQ. Reason: 'rejected on author's request in chat'


✨ Remove cache when catalog is deleted

Description
We add a finaliser to ClusterCatalog which enables us to remove catalog cache from filesystem on catalog deletion.
Fixes #948
Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

I like the direction of this. Maybe not something to do in this PR, but once we have a separate catalog controller, I think it would make sense to have that controller be the thing that also populates operator-controller's cache.
Right now, the operator-controller catalog cache is populated on-demand when a ClusterExtension is reconciled. The outcome is that some ClusterExtension reconciles take quite a bit longer because they have to pause to pull and extract catalog content from catalogd.
If we moved pull/extract logic to a separate ClusterCatalog controller, then operator-controller could do this work async. At that point the only ClusterExtensions that would still have to wait are the ones that are reconciled just after a ClusterCatalog has new data available.

Trying to understand why tests are failing. https://github.com/operator-framework/catalogd/issues/389 and the fact that pull interval is set to 1s in e2e makes it hard to debug.


Update tonew stable version

Updating operating-hours to 0.0.15
=> Automatic reset implemented

From where do you know that this version is "stable"? Normally we require stable to have at least some installations or time passed by, so that you can be sure to not have a broken version.
In special cases on very important bugfixes we allow faster stable update ... But this seems not the ase here, or?

OK, i have checked it.
also it is possible to set it to latest.

OK, so tomorrow the user can install beta. wright?

i closed this pR because i will check it with testers in Beta and maby do some changes


JWT token sessions

As a backend developer, I want to be able to verify the JWT tokens in API requests through middleware, so that the requests are passed correctly if the JWT is correct.

[x] #104
[ ] #105

In the future, we could consider having a shorter expiry time, with an authentication middleware refreshing the JWT token each time it is used.


tests: increase coverage

Also added some missing constraints from s.date

Codecov Report

Merging #67 (5a930ab) into main (f4526fe) will increase coverage by 1.68%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #67      +/-   ##
==========================================
+ Coverage   97.36%   99.04%   +1.68%
==========================================
  Files          47       47
  Lines        1251     1255       +4
  Branches      261      261
==========================================
+ Hits         1218     1243      +25
+ Misses         28       11      -17
+ Partials        5        1       -4

Impacted Files
Coverage Δ

src/validators/DateValidator.ts
93.54% <100.00%> (+0.95%)
:arrow_up:

src/validators/UnionValidator.ts
100.00% <0.00%> (+19.14%)
:arrow_up:

src/constraints/DateConstraints.ts
100.00% <0.00%> (+21.62%)
:arrow_up:

src/validators/DefaultValidator.ts
100.00% <0.00%> (+22.22%)
: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 f4526fe...5a930ab. Read the comment docs.


update regex-replace to upgrade Jackson to 2.10.0

The lower version has security vulnerabilities.

 update regex-replace to upgrade Jackson to 2.10.0 master

 update regex-replace to upgrade Jackson to 2.10.0 1.6.x


Switching homebrew to use meterup fork

Since this repo seems pretty inactive, I created a PR on homebrew to update the github-release formula to use the @meterup @kevinburkemeter fork (https://github.com/meterup/github-release), starting with release 0.7.5.  This includes a fix for #97 among other things.
See https://github.com/Homebrew/homebrew-core/pull/53891

Hi @iainbethune,
First of all: pretty inactive is an understatement. It's just under poor maintainership. I haven't prioritized working on this project since I started working for $DAY_JOB years ago.
So I'm happy that @kevinburke (AKA @kevinburkemeter I reckon) has agreed to take over maintainership.
You may have noticed that github.com/aktau/github-release now redirect to github.com/github-release/github-release.
Thanks Kevin for the offer of taking over maintainership, you'll serve the users much better than I did.

Thanks Nicolas! I believe this is resolved now so going to close.


Fix fragile explicit cert date tests.

The tests used localtime to format "today's" date, but then extracted a GMT date from the cert.  The comparison breaks when run late in the evening west of UTC, or early in the AM hours east of UTC.
Also took care of case when test runs at stroke of midnight, by accepting either the "today" before the cert creation, or the "today" after, should they be different.
Fixes fragile tests in #21716

I changed $today{...} > 0 to defined $today{...}, so that if by some chance the test fails, there won't also be a warning from Perl about comparison with undefined values.  Please re-conform approvals.

@nhorman @levitte Second approval (after cosmetic improvement)?

Approval confirmed

Merged to the master branch. Thank you.


No Fortnite client token

Bro I tried it 100 times... after I start Fortnite there is a /account/api/oauth/exchange request but no more token, if I capture traffic in the loading screen. I recorded it for you it's only 1 minute https://www.youtube.com/watch?v=nHBqp-IusyA what am I doing wrong?

@monkeydioude Can you link to this PHP api?

@NickSuperb I think I found the API https://github.com/Tustin/fortnite-php

The issue is that your Fortnite game won't verify Fiddler's certificates since it's using libcurl with verifypeer=true by default.
You can override this by adding the command line option -HTTP=WinInet in the Launcher (click the gear icon), and make your computer trust the Fiddler root certificate.

@jeffreyhorner Apparently they have fixed this loophole, did you find another possibility to bypass the SSL pinning?

No I haven't scoped out this issue in awhile.


Implement trees with with random depth

In the original paper, the trees have random depth, but this current implementation uses only fixed tree depth.

Fixed with https://github.com/christophM/rulefit/pull/17


[Feature Request]: Quality check for extracted articles (javascript code in articles)

Problem statement
A few examples from the data provided in #54 that are worth taking a look at.
Examples that include javascript code in them:
 },
  "https://www.thegatewaypundit.com/2023/06/fcking-grifters-meghan-markle-faked-podcast-interviews-her/": {
    "publishing_date": "2023-06-19 23:45:04+00:00",
    "title": "\"F*cking Grifters!\" - Meghan Markle Faked Podcast Interviews - Her Voice was Dubbed In Later - The Princess-Victim Was Paid $20 Million for the Gig | The Gateway Pundit | by Jim Hoft | 2",
    "body": "Not a bad gig if you can get it.\nLast week Prince Harry and Meghan Markle were fired from their podcast gig with Spotify. The royal fraudsters had their staff sit for the interview and Meghan’s voice was dubbed in later.\n\nThey were paid a cool $20 million for the gig. The released 12 podcasts.\n\nSpotify fired them last week.\n\nSpotify exec Bill Simmons called the royal pair “f-cking grifters.”\n\nThe New York Post reported:\n\nFOX News reported on the scandal.\n\n!function(r,u,m,b,l,e){r._Rumble=b,r[b]||(r[b]=function(){(r[b]._=r[b]._||[]).push(arguments);if(r[b]._.length==1){l=u.createElement(m),e=u.getElementsByTagName(m)[0],l.async=1,l.src=\"https://rumble.com/embedJS/u5o49d\"+(arguments[1].video?'.'+arguments[1].video:'')+\"/?url=\"+encodeURIComponent(location.href)+\"&args=\"+encodeURIComponent(JSON.stringify([].slice.apply(arguments))),e.parentNode.insertBefore(l,e)}})}(window, document, \"script\", \"Rumble\");\n\nRumble(\"play\", {\"video\":\"v2smpy4\",\"div\":\"rumble_v2smpy4\"});",
    "authors": [
      "Jim Hoft"
    ],
    "topics": []
 },

"https://www.thenation.com/article/archive/nation-readers-summer-books/": {
    "publishing_date": "2010-07-26 19:33:38+00:00",
    "title": "Nation Readers' Summer Books",
    "body": "Two weeks ago we did an informal poll of Nation writers and editors asking them what they were reading this summer. The survey produced some interesting results but not nearly as interesting as the reading suggestions tendered by our readers. \n\n\n\t\t\t\t\t\tfreestar.config.enabled_slots.push({ \n\t\t\t\t\t\t\tplacementName: \"thenation_right_rail\", \n\t\t\t\t\t\t\tslotId: \"thenation_right_rail_80975\",\n\t\t\t\t\t\t\ttargeting:{\n\t\t\t\t\t\t\t\ttn_author: ['the-n'],\n\t\t\t\t\t\t\t\ttn_articleid: [80975],\n\t\t\t\t\t\t\t\ttn_ptype: 'article',\n\t\t\t\t\t\t\t\ttn_keyword: [false],\n\t\t\t\t\t\t\t\ttn_subject: ['books-and-'],\n\t\t\t\t\t\t\t\ttn_pos: 'rectangle_1',\n\t\t\t\t\t\t\t\ttn_loc:'atf'\n\t\t\t\t\t\t\t}\t\n\t\t\t\t\t\t});\n\t\t\t\t\t  \n\nReading lists from coast to coast poured in to our website, our Facebook page, our Twitter feed and our own Nation social network....
},

"https://www.thenation.com/article/archive/vanderbilt-memorial/": {
    "publishing_date": "1869-11-18 23:55:09+00:00",
    "title": "The Vanderbilt Memorial",
    "body": "“The Commodore’s acts have touched the public,\nmore or less nearly, in a spot which is tender.”\n\n“The Commodore’s acts have touched the public, more or less nearly, in a spot which is tender.” \n\n\n\t\t\t\t\t\tfreestar.config.enabled_slots.push({ \n\t\t\t\t\t\t\tplacementName: \"thenation_right_rail\", \n\t\t\t\t\t\t\tslotId: \"thenation_right_rail_77069\",\n\t\t\t\t\t\t\ttargeting:{\n\t\t\t\t\t\t\t\ttn_author: ['the-e'],\n\t\t\t\t\t\t\t\ttn_articleid: [77069],\n\t\t\t\t\t\t\t\ttn_ptype: 'article',\n\t\t\t\t\t\t\t\ttn_keyword: [false],\n\t\t\t\t\t\t\t\ttn_subject: ['history'],\n\t\t\t\t\t\t\t\ttn_pos: 'rectangle_1',\n\t\t\t\t\t\t\t\ttn_loc:'atf'\n\t\t\t\t\t\t\t}\t\n\t\t\t\t\t\t});\n\t\t\t\t\t  \n\nAll of Mr. Vanderbilt’s acts have been in a sense private acts; which consideration has not, however, exempted all of them from public criticism–some of it, certainly, not gracious...
},

"https://www.thenation.com/article/archive/probable-effect-impeachment-trial/": {
    "publishing_date": "1868-05-01 01:48:32+00:00",
    "title": "The Probable Effect of the Impeachment Trial",
    "body": "Americans exercise their good sense by not caring much about the impeachment of President Andrew Johnson.\n\nCSU Archives/Everett Collection.Andrew Johnson \n\n\n\t\t\t\t\t\tfreestar.config.enabled_slots.push({ \n\t\t\t\t\t\t\tplacementName: \"thenation_right_rail\", \n\t\t\t\t\t\t\tslotId: \"thenation_right_rail_76564\",\n\t\t\t\t\t\t\ttargeting:{\n\t\t\t\t\t\t\t\ttn_author: ['the-e'],\n\t\t\t\t\t\t\t\ttn_articleid: [76564],\n\t\t\t\t\t\t\t\ttn_ptype: 'article',\n\t\t\t\t\t\t\t\ttn_keyword: [false],\n\t\t\t\t\t\t\t\ttn_subject: ['history'],\n\t\t\t\t\t\t\t\ttn_pos: 'rectangle_1',\n\t\t\t\t\t\t\t\ttn_loc:'atf'\n\t\t\t\t\t\t\t}\t\n\t\t\t\t\t\t});\n\t\t\t\t\t  \n\nAmericans exercise their good sense by not caring much about the impeachment of President Andrew Johnson.\n\n….Owing to a combination of circumstances, the trial has produced absolutely no excitement in the public mind. All attempts to make it “sensational” have failed utterly...
},

And links to other articles that are of low quality:
"https://www.thenation.com/article/world/qa-india-israel-azad-essa/",
"https://www.thenation.com/article/archive/nation-readers-summer-books/",
"https://www.thenation.com/article/archive/probable-effect-impeachment-trial/",

I found this pattern mostly in The Nation, it would be good to look there first. Although other publishers might also have it (the first example is from Gateway Pundit).
Solution
Maybe better parsing rules can be applied for these examples or some urls from the sitemap can be removed.
Additional Context
No response

Thanks for posting this!
This seems to be an interesting case here. I tried to reproduce the extraction with the following lines.
from fundus import PublisherCollection
from fundus.scraping.html import HTMLSource
from fundus.scraping.scraper import Scraper
from fundus.scraping.pipeline import Pipeline

url = "https://www.thenation.com/article/archive/nation-readers-summer-books/"

publisher = PublisherCollection.us.TheNation
source = HTMLSource([url], publisher=publisher.publisher_name)
scraper = Scraper(source, parser=publisher.parser)
pipeline = Pipeline(scraper)

but the article came back fully extracted without the script. Maybe that has something to do with spam protection and only occurs with a high enough request frequency.
Nonetheless, having some kind of quality control / sanity check to prevent text like this is a great idea. Any ideas here?


Unrecognized SSL message, plaintext connection?

The jar is working file for 2 hours i am getting the info  from Gitlab, but suddenly it will throw error
Unrecognized SSL message, plaintext connection?
I am using HTTPS for connection, please provide your inputs how to resolev this error.

@ricky001
I would need a lot more information, a stack trace would be especially helpful.
My initial inclination would be that this is somewhere outside the of gitlab4j, but without additional information it is very difficult to come to any conclusions.

@ricky001
Can this be closed?

Hello @gmessner  yes we can close it


Vagrantfile: Remove LXC provider; move Docker provider to default

Along with #12395, this also means the default dev environment is now based on Ubuntu 18.04 (bionic).
Testing Plan: Still working on testing the documented instructions on all platforms.

Great!  As a follow-up idea for this, it'd be cool if we added an option for booting xenial (e.g. a ~/.zulip-vagrant-config option); obviously not part of this PR; I'm mostly thinking about this as a step towards having a better documented development/testing procedure for adding a new platform locally.

I successfully tested these instructions on Ubuntu 18.04, Ubuntu 16.04, and Debian 9, after adding a note about docker.io being absent in Debian 9 and using Docker CE there instead.

Hmm, putting docker before virtualbox makes an initial vagrant up fail on Windows with VirtualBox installed:
$ vagrant up
Bringing machine 'default' up with 'docker' pdovider...
The executable 'docker' Vagrant is trying to run was not
found in the %PATH% variable. This is an error. Please verify
this software is installed and on the path.

even though vagrant up --provider=virtualbox would work. This seems contrary to the documentation.

PR sent to hashicorp/vagrant#10879.

Arg, that's annoying.  I guess we may end up needing to (1) document that error message and (2) potentially tell our macOS/Windows users a minimum Vagrant version that's not released yet.

This is really awesome, I've just tested and it works on NixOS (previously a Arch machine, where in both cases, the vagrant-lxc environment absolutely did not work) which is my primary development machine (and until now, has been unsuitable for Zulip development).

Excessive amounts of detail
NixOS is strange in that everything is in a different place from other distros and all managed by a declarative configuration file.
For example, instead of installing a header globally, you need to enter a nix-shell (defined in the same language as the global system configuration, the Nix language) with the package that includes the header defined in it, doing this will include the header in the search paths for any compilers. The advantage of Nix is that you end up with all the dependencies of your projects (headers, compilers, tools, etc) defined in one place that anyone with the Nix package manager can use in a single command (on any distribution, not just NixOS).
It's off-topic for this PR, but if the project was interested in including a shell.nix in the project (and documenting that installing Nix and running nix-shell is enough to get the non-inside-the-VM dependencies installed) then this is the shell.nix that worked for me when I tested this:
{ pkgs ? import <nixpkgs> {} }:

(pkgs.buildFHSUserEnv rec {
  name = "zulip";
  targetPkgs = pkgs: (with pkgs; [
    pkgs.vagrant
    pkgs.docker
    pkgs.git
    pkgs.openssh
  ]);
  runScript = "bash";
}).env

@davidtwco I have a NixOS system too. With that said, I’d just like to point out the inherent nihilistic absurdity of using Nix to run Vagrant to run Docker to run Ubuntu to run Virtualenv to run Python to run Zulip.
Some day, computer scientists will learn how to build a packaging system that creates fewer problems than it solves.
:fire::dog::coffee::fire:

I added a horrendous monkey patch for hashicorp/vagrant#10879 and verified that vagrant up works again on Windows with VirtualBox. So this may be mergeable now.

Merged, thanks @andersk!
@davidtwco I'd love a PR adding a NixOS instructions section for Zulip Vagrant setup including adding that shell.nix in tools/setup/ to make it convenient; you're far from the first person to want to run a Zulip development on a NixOS system, so I think it would definitely add value.


How can I access okd web interface using an external/static ip?

Just saw this in vagrant doc, config.vm.network "public_network", ip: "<IP_ADDRESS>"
Will try and see what happens.

Didn't work. Tried config.vm.network "public_network", ip: "" and changing the set-hostname to publicIP as well. Any help would be appreciated.


Is it possible to scale down a clustered samba server instance?

Hi, I have deployed a Samba server with clustering feature enabled. According to the documentation, I can change the number of Samba servers by configuring scaling.minClusterSize in smbshare CRD, and this will directly impact the number of replicas in the Samba Statefulset.

minClusterSize: The minimum number of Samba server "nodes" to host the
share. The operator is permitted to run more servers in some conditions.

Based on the documentation and actual testing, it appears that when modifying a running Samba instance, I can only increase the number of replicas by raising scaling.minClusterSize(scale out), but cannot decrease the number of replicas by lowering the parameter(scale down).
I would like to understand why this parameter is designed to represent the "minimum quantity" rather than being designed to be "exactly equal to the current replicas quantity in the Statefulset." What is the difficulty in reducing the cluster quantity? On the other hand, I also whant to know the recommended approach for decreasing the number of replicas manually.(Should I do anything rather than just modify the replica number if I want to do a scale down?)
Thanks!

You may remember that you had to enable this experimental feature, it's experimental precisely because of items like this. To try and summarize the main issue - kubernetes pods are dynamic and easy to create and destroy, however Samba's CTDB is more oriented towards managing actual physical nodes. The CTDB configuration is not very flexible and expects to retain references to nodes that have been removed in the config file by we try to generate the config file based on what pods are present. Therefore it was simply not implemmented (yet) due to the complexity.
The minimum quantity question is related but not fully overlapping. There was a desire to not force the operator to prematurely shrink a working cluster. Additionally, we have the ability to host multiple shares on one "instance" (one smbd or a cluster of smbds). It permits the system to host a share with a minimum of 2 pods on a cluster with 3 (or more) actual pods for instance.

Thanks for the reply. It seems that scaling down involve considerations related to CTDB configuration. To the best of my knowledge, CTDB uses config file(/etc/ctdb/nodes) to maintain the relationship between IPs and PNNs (node number). When removing a node, one should follow the procedure to remove nodes in CTDB to keep the correct line numbers. Specifically, the config file still stores the information about removed nodes(by commenting out the lines). I agree that ensuring the consistency of CTDB configurations while operators scale up or down pods is a complex issue.

That's exactly the issue. I don't think it's insurmountable but what I think would be better would be to work with the Samba team to support alternative means of configuration that work better with k8s or other orchestration frameworks. This is a long term thing IMO. Please continue to try out the samba-operator but be aware that the CTDB mode will have more sharp edges - bug reports and assistance there are very welcome - but I've been pulled in other directions for now and don't have time for large projects improving the operator at the moment.

Okay. Still thanks for the advice. I think I can try investigating this issue and see if I can do something.

Hi, recently I have more time to study this issue. To scale down the replica of a clustered samba instance, I think there are several steps to do:

Modify updateStatefulSetSize() function in samba-operator to deal with scaling down case.
Once replica number of StatefulSet is modified, samba-operator should inform the remaining pods that there is a deletion.
The remaining pods update CTDB configuration and reload. (follow node remove steps in ctdb docs)

For implementation, My initial idea is using preStop hook in k8s to deal with this. By adding the hook, one can perform CTDB configuration when a pod is terminated by k8s api server. The modification should also include sambacc, which provides a method to update state file and node file in the shared storage, then reload ctdb. For example, the modified Samba server pod in StatefulSet would be like:
apiVersion: v1
kind: Pod
metadata:
  name: samba-server-0
spec:
  containers:
  - name: ctdb-delete-node #or put it in current existing containers
    image: quay.io/samba.org/samba-server:v0.4
    lifecycle:
      preStop:
        exec:
          command: ["samba-contaner", "ctdb-delete-node"]

So this is my current rough idea, not sure if there’s anything I thought wrong or didn’t take into consideration. If everything is fine, I can try to implement this feature and send a PR. Thanks for any advice!
Ref:
https://kubernetes.io/docs/concepts/containers/container-lifecycle-hooks/#container-hooks
https://kubernetes.io/docs/tasks/configure-pod-container/attach-handler-lifecycle-event/
https://ctdb.samba.org/manpages/ctdb.1.html#:~:text=also "ctdb getcapabilities"-,reloadnodes,-This command is

@phlogistonjohn So this is my current plan for scaling down a clustered Samba server, the situation is when a user wants to reduce the replica number by editing minClusterSize in smbshare. The detailed steps is as follows:

updateStatefulSetSize() updates modified size to the value of replica in statefulset.
K8s automatically deletes the pod with the greatest name index in statefulset.
When the pod is terminating, preStop hook is triggered and execute commands.

    lifecycle:
      preStop:
        exec:
          command:
          - /bin/sh
          - -c
          - sleep 1 && samba-container ctdb-delete-node --hostname=$HOSTNAME --take-node-number-from-hostname=after-last-dash

samba-container ctdb-delete-node edits the state of deleted node in ctdb-nodes.json from ready to deleted. (I delay this operation for 1 sec considering ctdb reloadnodes must be executed after the node is actually disconnected. I'm not sure this delay is necessary.)
When ctdb.manage_nodes detects the state change of the deleted node, it comments out the ip in /var/lib/ctdb/shared/nodes and do ctdb reload.
After manage_nodes updates the change, the delete node is removed from ctdb cluster

[root@samba-service-smbserver-0 /]# cat /var/lib/ctdb/shared/ctdb-nodes.json
{"nodes": [{"identity": "samba-service-smbserver-0", "node": "<IP_ADDRESS>", "pnn": 0, "state": "ready"}, {"identity": "samba-service-smbserver-1", "node": "<IP_ADDRESS>", "pnn": 1, "state": "deleted"}]}

[root@samba-service-smbserver-0 /]# cat /var/lib/ctdb/shared/nodes
<IP_ADDRESS>
#<IP_ADDRESS>

[root@samba-service-smbserver-0 /]# ctdb status
Number of nodes:2 (including 1 deleted nodes)
pnn:0 <IP_ADDRESS>      OK (THIS NODE)
Generation:887371738
Size:1
hash:0 lmaster:0
Recovery mode:NORMAL (0)
Leader:0

I have implemented and tested it in my local environment. I want to ask some advices to see if there are any mistakes and maybe send a PR to samba-operator and sambacc. Thanks.

the ctdb manpage has a section procedure to remove nodes which involves shutting down nodes to be removed (corresponding to deleting pods), modifying the nodes file, and calling ctdb reloadnodes.
It would be great to add this mechanism to the operator code for reconciling the cluster in reaction to a change in the cluster size setting.
But a potential problem is: what happens to clients connected to the node to be removed?
This isn't even covered in the samba/ctdb documentation. Data might get corrupted.
The most graceful  way I know for  moving a client to a different cluster node is to move the public address it is connected to to a different node using ctdb moveip.
We might need to bring some changes into upstream Samba like a command/API for triggering poplock/lease breaks to all clients connected to a given smbd/IP.


[TIR] StmtFunctor ReGenerateDef

In this PR, I introduce a StmtFunctor ReGenerateDef for deep copy all definition nodes in PrimFunc (including Var, Buffer, and IterVar). This functor can create a new PrimFunc with the same behavior as the old one but contains different Nodes.
This Functor may help TIR fusion or inline multiple PrimFuncs
cc @tqchen @jinhongyii

cc @junrushao1994 @vinx13 @jinhongyii @MasterJH5574 please help to review

cc @vinx13

Please take another look @tqchen @jinhongyii @junrushao1994 @MasterJH5574

Some additional ideas for possible regressions
Symbolic shape case that needs remap
Symbolic vars needs to be mapped to the same one
def fn(a: T.handle, b: T.handle, n: T.int32)
   m = T.var("int32")
   A =  match_buffer((n, m), a)
   B = match_bufer((n, m*2), b)

Some additional ideas for possible regressions
Symbolic shape case that needs remap
Symbolic vars needs to be mapped to the same one
def fn(a: T.handle, b: T.handle, n: T.int32)
   m = T.var("int32")
   A =  match_buffer((n, m), a)
   B = match_bufer((n, m*2), b)

Thanks for this interesting case!
As a further step, there might be usecases where buffers are declared as divisible by power-of-2, for example:
def fn(a: T.handle):
    m = T.var("int32")
    A = match_buffer(a, (m * 8, ), "float32")

@tqchen do you think this is a case we want to support?

@junrushao1994 what you are describing is not a case we support right now, but can be something we want to support in the future and indeed an useful case

Any updates?

@junrushao1994 @tqchen please take another look at it.
also cc @spectrometerHBH @wrongtest @jinhongyii @MasterJH5574


[PORT] add recipient in BotFrameworkHttpClient.PostActivityAsync if value is null

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3781

Fixes: #3684
Description
If Activity.Recipient is null, this can cause skills to throw errors as the Recipient field is expected. This PR sets Recipient to an empty ChannelAccount if the value is null.
Context
https://github.com/microsoft/botframework-sdk/issues/5785
Testing
Added positive and negative tests for BotFrameworkHttpClient.PostActivityAsync() changes
Changed projects

integration

Dupe of https://github.com/microsoft/botbuilder-js/issues/1997
Fixed by https://github.com/microsoft/botbuilder-js/pull/2067


Optimal sparse decompression

Core

Make sparsity column-major for column-wise Jacobians (forward mode) and Hessians
Make sparsity row-major for row-wise Jacobians (reverse mode)
Implement decompress! for SparseMatrixCSC (when dir = :col) and its transpose (when dir = :row)
Give up on symmetric coloring and decompressing for Hessians at the moment

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.64%. Comparing base (09a7da0) to head (3b92804).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #223      +/-   ##
==========================================
- Coverage   95.64%   94.64%   -1.01%
==========================================
  Files          74       15      -59
  Lines        3839     1775    -2064
==========================================
- Hits         3672     1680    -1992
+ Misses        167       95      -72

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


gateway state

There is a bit of a caveat when adding a new inverter via the settings file which was the intended way of doing things. In essence we would need to know if the inverter has been opened before (so we can start a perpetual task) or if the inverter is new and an ordinary open device task should be run.
We could solve this by having a flag (or counter) for how many times an inverter has been opened. I'm actually think this could be a good idea to have also in the configurator UI. Maybe even a time when the device was added and last connected to would be good to have.
Come to think of an UI would need some way of getting the status of the inverter... much as we do via the REST api... maybe send the commands this way also... eg. just build a new communication channel via the graphql subscriptions... this way we would not need to build something special for the status... the rest api is available via the qraphq api...

or we can simply e.g. serialize the blackboard to some state and have a state subKey...

Ok the latest iteration would be to have a state subKey, a request subKey for sending commands to the gateway compare to e.g. POST and a response subKey. The settings will be more or less a "read only ting" for the gateway to store settings.
State would contain everything e.g. crypto info, wifi ssids, devices (and their state).

Saving several device configurations and loading them should be possible as of 7f8c9b5f727ceefb65775fa11adc31860620c1a4.
Short:
A modbus configuration is saved for every unique (by MAC/SN) successful connection. The configurations are saved in the state (saved in RAM) and settings (saved in Backend).
If the gateway reboots, the state is wiped, but the gateway fetches the settings from backend and tries to open every single saved configuration and the state updates accordingly.

Maybe even a time when the device was added and last connected to would be good to have.

I like this, as it would help us keep the track of configurations and help decide what to remove and what to keep. The user should also be able to reset the settings, but that's for another issue.
Closing this since most of the issues mentioned here are implemented, apart from the UI stuff...


[update] add tests => test/acceptance, test/unit

CR ready: just getting started

[x] basic Jest (unit) tests
[x] build tweaks

TODO: in other PR

deep Postmate unit test interface
remove gulp dependency

why the huge diff

@jonathanong Package lock