a potential bug in container.py

Description
When I was running the lean4 example, I met an error in the function _unmount_files under container.py. The error is "Directory not empty". I tried to print the directory and found there exists .lake. It seems that the mechanism of moving and deleting files belonging to the dst folder in function _unmount_files is somewhere not robust. To solve this issue, I simply modify shutil.rmtree(dst) into shutil.rmtree(dst, ignore_errors=True) in line 144. But I'm not sure whether this solution is safe.
Detailed Steps to Reproduce the Behavior
My code:
from lean_dojo import LeanGitRepo
repo = LeanGitRepo("https://github.com/yangky11/lean4-example", "3f8c5eb303a225cdef609498b8d87262e5ef344b")

from lean_dojo import Theorem, Dojo, ProofFinished
theorem = Theorem(repo, "Lean4Example.lean", "foo")
with Dojo(theorem) as (dojo, init_state):
    print(init_state)
    result = dojo.run_tac(init_state, "rfl")
    assert isinstance(result, ProofFinished)
    print(result)

Logs in Debug Mode
You can also see my print here, i.e. dst="/mntnfs/med_data5/yufei/.cache/tmp/tmp62wukmc1/lean4-example/workspace/lean4-example" and os.listdir(dst) -> ['.lake']
2024-06-05 13:06:47.688 | WARNING  | lean_dojo.interaction.dojo:__init__:152 - Using Lean 4 without a hard timeout may hang indefinitely.
2024-06-05 13:06:47.688 | DEBUG    | lean_dojo.interaction.dojo:__enter__:164 - Initializing Dojo for Theorem(repo=LeanGitRepo(url='https://github.com/yangky11/lean4-example', commit='3f8c5eb303a225cdef609498b8d87262e5ef344b'), file_path=PosixPath('Lean4Example.lean'), full_name='foo')
2024-06-05 13:06:47.692 | DEBUG    | lean_dojo.data_extraction.trace:get_traced_repo_path:87 - The traced repo is available in the cache.
2024-06-05 13:06:55.182 | DEBUG    | lean_dojo.interaction.dojo:_modify_file:356 - Modifying Lean4Example.lean
2024-06-05 13:06:55.186 | DEBUG    | lean_dojo.interaction.dojo:__enter__:195 - Launching the proof using <class 'lean_dojo.container.NativeContainer'>
2024-06-05 13:07:04.303 | DEBUG    | lean_dojo.container:run:189 - lake build Lean4Repl
2024-06-05 13:07:26.145 | DEBUG    | lean_dojo.container:run_interactive:219 - lake env lean --threads=1 --memory=32768 Lean4Example.lean
2024-06-05 13:07:26.789 | DEBUG    | lean_dojo.interaction.dojo:_read_next_line:530 - REPL> {"tacticState": "a : Nat\n⊢ a + 1 = Nat.succ a", "sid": 0, "error": null}
TacticState(pp='a : Nat\n⊢ a + 1 = Nat.succ a', id=0, message=None)
2024-06-05 13:07:26.790 | DEBUG    | lean_dojo.interaction.dojo:_submit_request:491 - {"sid": 0, "cmd": "rfl"}
2024-06-05 13:07:26.792 | DEBUG    | lean_dojo.interaction.dojo:_read_next_line:530 - REPL> {"tacticState": "no goals", "sid": 1, "error": null}
ProofFinished(tactic_state_id=1, message='')
2024-06-05 13:07:26.792 | DEBUG    | lean_dojo.interaction.dojo:_cleanup:289 - Cleaning up.
2024-06-05 13:07:26.792 | DEBUG    | lean_dojo.interaction.dojo:_cleanup_container:299 - Cleaning up the container.
/mntnfs/med_data5/yufei/.cache/tmp/tmp62wukmc1/lean4-example/workspace/lean4-example
['.lake']
2024-06-05 13:07:36.638 | DEBUG    | lean_dojo.interaction.dojo:_cleanup_tmp_dir:319 - Cleaning up the temporary directory.
Traceback (most recent call last):
  File "/mntnfs/med_data5/yufei/lean/LeanDojo/src/lean_dojo/container.py", line 145, in _unmount_files
    shutil.rmtree(dst)
  File "/mntnfs/med_data5/yufei/anaconda3/envs/lean/lib/python3.9/shutil.py", line 734, in rmtree
    _rmtree_safe_fd(fd, path, onerror)
  File "/mntnfs/med_data5/yufei/anaconda3/envs/lean/lib/python3.9/shutil.py", line 667, in _rmtree_safe_fd
    _rmtree_safe_fd(dirfd, fullname, onerror)
  File "/mntnfs/med_data5/yufei/anaconda3/envs/lean/lib/python3.9/shutil.py", line 667, in _rmtree_safe_fd
    _rmtree_safe_fd(dirfd, fullname, onerror)
  File "/mntnfs/med_data5/yufei/anaconda3/envs/lean/lib/python3.9/shutil.py", line 673, in _rmtree_safe_fd
    onerror(os.rmdir, fullname, sys.exc_info())
  File "/mntnfs/med_data5/yufei/anaconda3/envs/lean/lib/python3.9/shutil.py", line 671, in _rmtree_safe_fd
    os.rmdir(entry.name, dir_fd=topfd)
OSError: [Errno 39] Directory not empty: 'lib'

Platform Information

OS: linux
python 3.9.18
LeanDojo Version 1.9.0
elan 3.1.1 (71ddc6633 2024-02-22)
Lake version 5.0.0-6fce8f7 (Lean version 4.7.0)

Sorry for the late response. I'll be able to take a look probably next week.

Hi,
I ran the code but was unable to reproduce the error. Nevertheless, adding ignore_errors=True should be fine.


FEAT - Add quadratic datafit with no access to the target

Description of the feature
Exact feature
Solve the following optimization problem
$$\arg \min_{\beta} \frac{1}{2n} || X \beta ||^2  + \frac{1}{n} \beta^\top X^\top y +  \text{penalty} \enspace,$$
with no access to $y$, but with access to $X^\top y$.
Additional context
Context,
I have been discussing with @shz9 to implement a specific datafit for genomic applications (@shz9 is finishing his PhD on statistical analysis of genomics data). From what I understood, genomics data are sensitive: one does not have access to the target $y$: one only has access to the design matrix $X$, and an estimation of $X^\top y$ (usually estimated from another dataset).
Steps
I guess we have to add the datafit  $$\frac{1}{2n} || X \beta ||^2  + \frac{1}{n} \beta^\top X^\top y \enspace ,$$ and handle the fact there is no $y$ provided.

Thanks @QB3  that sounds doable
How would we pass the current design ? y does not have a default value in Solver.solve, so it's required. Making it a keyword arg may break some stuff.
Thoughts @Badr-MOUFAD @QB3 ?

It is true that the form of the problem is a bit different but it fits our framework as we can write the datafit as $F(y, X\beta)$
As you mentioned @mathurin, while it breaks our conventions, I don’t think it would break the code.
We can create a new datafit and pass in (X, XTy) to Solver.solve
Thinking about it, adding support of this case, we will have in total three cases that don't abide by the conventions: Cox, and LinearSVC.
Perhaps, we should start thinking about a more flexible API that encloses all these cases.

There's a working (withoutintercept) implementation with minimal overhead (we inherit most of the stuff from Quadratic) in #250
I need to get the intercept update step right, but it was not painful at all to implement !

Thank @QB3, this would be great!
Just to add a bit more context: In statistical genetics, we have extremely high dimensional data (10s of millions of features) that we'd like to use to predict a trait (e.g. blood cholesterol levels) or disease (e.g. diabetes). Due to privacy, large cohorts that collect the data don't release either X or y, but they release "summary statistics" in the form of $X^\top y$.
So,  we have to resort to approximations of $X$. In previous work, people relied on the form:
$$
\frac{1}{n}||y - X\beta||_2^2 = \frac{1}{n}\Big[y^\top y -2\beta^\top X^\top y + \beta^\top X^\top X\beta\Big]
$$
We usually assume both $y$ and $X$ are standardized. The first term is a constant. The second term can be defined based on the summary statistics. The third term can be approximated using small, publicly available data (under the assumption that covariance $X^\top X$ between features is roughly the same across cohorts.
Given this, two options are possible for the implementation:

Provide external $X$ that has roughly similar characteristics to censored data (what @QB3 suggested).
Provide external $X^\top X$ (which is a sparse, block-diagonal matrix). This is what practitioners in the field usually do. One main benefit of this second approach is that some of the larger cohorts that release $X^top y$ can also release $X^\top  X$ estimated from their data, because it doesn't violate privacy agreements.

Happy to help with the implementation or testing.

For the implementation passing X and Xty see a working draft implementation at #249
For the one with XtX I guess we need to do something else, like directly modifying our GramCD solver to support being passed XtX and Xty as arguments. I'll give this a try too


Generate error when using API class in a library

When using rpc/bin/generate.dart to generate a discovery file or client files, I get this error if my API class is part of a library:
C:\Users\sanmadjack\Desktop\dart_dev\dartalog>pub run rpc:generate discovery -i lib\server\api\src\dartalog_api.dart -p 8080 -a api
in ShutdownIsolate: Unhandled exception:
Uncaught Error: IsolateSpawnException: 'file:///C:/Users/sanmadjack/Desktop/dart_dev/dartalog/lib/server/api/src/dartalog_api.dart': error: line 1 pos 6: url expected
part of api;
^
Stack Trace:
#0      Isolate._spawnUri (dart:isolate-patch/isolate_patch.dart:352)
#1      Isolate.spawnUri (dart:isolate-patch/isolate_patch.dart:308)
#2      _isolateTrampoline.<_isolateTrampoline_async_body>    (http://localhost:60716/generate.dart:242)
#3      Future.Future.microtask. (dart:async/future.dart:144)
#4      _asyncRunCallbackLoop (dart:async/schedule_microtask.dart:41)
#5      _asyncRunCallback (dart:async/schedule_microtask.dart:48)
#6      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:96)
#7      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:143)
#0      _rootHandleUncaughtError. (dart:async/zone.dart:886)
#1      _asyncRunCallbackLoop (dart:async/schedule_microtask.dart:41)
#2      _asyncRunCallback (dart:async/schedule_microtask.dart:48)
#3      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:96)
#4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:143)
If I change the API class to be separate from a library (so it doesn't have to have the "part of" directive at the top), it generates fine.

Hi sanmadjack,
Could you try pointing the -i option at the file with the 'library' statement rather than the one with the 'part of' statement? It should find the @ApiClass in your 'part of' file since it is part of the overall library.
I will work on providing a better error message.

I have added the better error message as of commit 659ccabd6b6e91e06bf27f4f99f192fdd19b356a.

Thanks, that explains it nicely.


Provide command reference as part of our official Asciidoctor-sourced book

Overview
With the first beta of ecctl, we simply pointed to the command reference from our new Asciidoctor-sourced book in https://www.elastic.co/guide/en/ecctl/current/ecctl-command-reference.html. This topic is a paceholder that takes you to the generated Markdown command reference  in https://github.com/elastic/ecctl/blob/master/docs/ecctl.md.
What we would like is an Asciidoctor-sourced command reference that integrates into the ecctl book and that can be indexed for our elastic.co docs search.
Possible Implementation
Ideally, we would generate the Asciidoctor source directly instead of Markdown and commit it as part of our regular PRs.
An interim solution might be to convert the Markdown via pandoc on a recurring basis, e.g. whenever we release a new version, and to script those steps.
Testing
We would build and proof the command reference content as part of our regular docs release activities.
Context
We've previously been asked not to keep parts on our docs on Github if they are for an official Elastic Product; the command reference falls under the ecctl product umbrella and really should live with the main book.

The initial work for this is complete, closing.


Compilers minimum version

What are the requirements for the gcc, clang, MSVC compilers version other than C99 support?

We currently support GCC 9 and Visual Studio 17 2022. Please see our continuous integration tests.
Do you want to contribute more GitHub actions so that we can test a wider range of compilers and systems?

I'm working on a streamvbyte package for сonan and I need to set minimum compilers version in package config. C99 supports by many compilers, but perhaps there are some other limitations

I have added an extensive collection of CI tests and I have updated the README:
Requirements

A C99 compatible compiler (GCC 9 and up, LLVM 10 and up, Visual Studio 2019 and up).
We support macOS, Linux and Windows. It should be easy to extend support to FreeBSD and other POSIX systems.

For high performance, you should have either a 64-bit ARM processor or a 64-bit x64 system with SSE 4.1 support. SSE 4.1 was added to Intel processors in 2007 so it is almost certain that your Intel or AMD processor supports it.

It is possible that older compilers could be supported but I cannot easily support them because I do not have convenient CI support.

Closing.


MudTreeView doesn't work after V7.0 migration

Bug type
Component
Component name
MudTreeView
What happened?
After migration from V6.14.0 to V7.0, the below working code doesn't work.
I already went through the migration guide but can't find the root cause of this problem.
Please correct this problem.
MudBlazor 6.14.0
 <MudTreeView Class="pa-2" Color="Color.Secondary" Items="@TreeItems" Dense="true" Hover="true" T="TreeItemData"
 SelectedValueChanged="@(async x => await SelectNode(x))" Style="height:775px">
     <ItemTemplate>
         <MudTreeViewItem<EMAIL_ADDRESS><EMAIL_ADDRESS>Value="@context"
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>     </ItemTemplate>
 </MudTreeView>

MudBlazor 7.0.0
 <MudTreeView Class="pa-2" Color="Color.Secondary" Items="@TreeItems" Dense="true" Hover="true" T="TreeItemData"
 SelectedValueChanged="@(async x => await SelectNode(x))" Style="height:775px" SelectionMode="SelectionMode.SingleSelection">
    <ItemTemplate>
        <MudTreeViewItem<EMAIL_ADDRESS><EMAIL_ADDRESS>Value="@context"
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>    </ItemTemplate>
 </MudTreeView>

Expected behavior
After the selection of the specific item in treeview, the related SelectNode method should be invoked.
Reproduction link
https://try.mudblazor.com/snippet/GkGeaBvPnBfuxHuM
Reproduction steps
1.Based on the migration guide, I modified the existing working code.
2.After compile, there are three kind of errors happend.

'TreeItemData<__Main.TreeItemData>' does not contain a definition for 'Title' and does not have an accessible extension method 'Title' that allows the first argument of the type 'TreeItemData<__Main.TreeItemData>. Make sure there is a using instruction or assembly reference.
'System.Collections.Generic.HashSet<Try.UserComponents.__Main.TreeItemData>''System.Collections.Generic.IReadOnlyCollection<MudBlazor.TreeItemData<Try.UserComponents.__Main.TreeItemData>>

Some of the return formats in the block cannot be implicitly converted to a surrogate return format, so the lambda expression cannot be converted to a specified surrogate format.
...

Relevant log output
No response
Version (bug)
7.0.0
Version (working)
6.14.0
What browsers are you seeing the problem on?
Chrome
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@henon


why webpack looking in parent dir? can I make at-loader not look into server dir?

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

What is the current behavior?
can not compile
npm run build:dev or build:prod

I have
admin/
client
server
and compiling in client dir, ts 2.4.1
ERROR in [at-loader] ../server/node_modules/hydrate-mongodb/config/configuration.d.ts:80:46
TS2694: Namespace ''*'' has no exported member 'Db'.
ERROR in [at-loader] ../server/node_modules/hydrate-mongodb/persister.d.ts:46:87
TS2694: Namespace ''*'' has no exported member 'Collection'.
ERROR in [at-loader] ../server/node_modules/hydrate-mongodb/query/queryBuilder.d.ts:1:23
TS2688: Cannot find type definition file for 'rx-core'.
ERROR in [at-loader] ../server/node_modules/hydrate-mongodb/query/queryBuilder.d.ts:2:23
TS2688: Cannot find type definition file for 'rx-core-binding'.
ERROR in [at-loader] ../server/node_modules/hydrate-mongodb/query/queryBuilder.d.ts:3:23
TS2688: Cannot find type definition file for 'rx-lite'.
...

started to work, dunno why


Unhelpful error messages

Cargo outputs too low-level error messages. Currently when running "cargo build --verbose", I get the following error:
The system cannot find the path specified. (os error 3)
Neither the missing path, nor information about the current task is displayed, so I have no way of knowing why this is happening. Whenever cargo receives a "Result" from a stdlib operation, it should wrap any errors, adding information about what it was trying to do when the error occurred, instead of just returning them raw.
edit:
Without --verbose the error is even more unhelpful #1516

Cargo actually puts in quite a bit of effort to produce good error messages, but there are cases where the error message could certainly be improved! Put another way, we're already doing this all over the place and the infrastructure is all there, so the only action here is to drill down to where this is happening and add a more specific error message.
What were you doing to trigger this error message?

I eventually tracked down this particular error to a subfolder several levels down which contained a broken symlink.
Related to #1524, cargo shouldn't care about broken symlinks unless they are actually part of the build, and when they are it should give the path of either the symlink or the missing target.

Do you have a way that I can reproduce this? I've created some broken symlinks and haven't been able to get this error message to show up.

I can reproduce it by creating the following folder structure on windows, and running "cargo build":
$ ls -lR
.:
total 2
drwxrwx---+ 1 Diggory None   0 May  5 23:46 broken
-rwxrwx---+ 1 Diggory None 130 May  5 23:46 Cargo.toml
-rwxrwx---+ 1 Diggory None  48 Nov  4 17:23 lib.rs
drwxrwx---+ 1 Diggory None   0 May  5 23:46 test

./broken:
total 0

./test:
total 0
lrwxrwxrwx 1 Diggory None 14 May  5 23:46 link -> ../broken/link

I'm using cygwin to run the "ls -lR" command, but these are native windows symlinks and the error occurs however I invoke "cargo build".

Can you gist an exact set of commands you're running, as well as the version of Cargo that you're using? I'm still unable to reproduce this on Windows.

Sure:
https://gist.github.com/Diggsey/19276f59c07a26d4f813
I removed lib.rs, since it's not required to reproduce the problem, and was in the wrong place anyway.

Can you clarify what version of Cargo you are running? Also can you show me how to create the directory you're working in from scratch?

$ cargo --version
cargo 0.2.0-nightly (efb482d 2015-04-30) (built 2015-05-02)
I created the directory from windows explorer in my user folder, saved cargo.toml from the gist, and then ran the commands in a cygwin terminal at that location. Non-cygwin methods of creating the symlinks also work, the only requirement is that they are native windows symlinks (on windows admin privileges are required to create symlinks).

I've actually fought this error message before in a different context, and I don't think it's possible for cargo to give a better error message.
Given the lack of activity on this issue, I'm going to close it -- please reopen if this is still a problem!


Reusable Table Component

User Story
As a doctor,
I want a table component that displays patients with their first name, last name, and personal number,
so that I can efficiently access patient information and view their health data received from the Moberg monitor.
As an admin,
I want a table component that displays doctors with their first and last name, person number, username, and password,
so that I can effectively manage doctor accounts.

Acceptance Criteria

[x] Criterion 1:  The table component must contain first name, last name, and personal number for patients' list page.
[x] Criterion 2:  The table component must contain first name, last name personal number, username and password for admin page.
[ ] Criterion 3:  The table component must support dynamic data loading from the database for both patients (for doctors) and doctors (for admins).
[ ] Criterion 4: The patient table should allow doctors to click on any entry (Surname or Personal Number) to navigate to the patient’s detailed health data page.
[x] Criterion 5: Admins should have the ability to edit and delete doctor entries, while doctors cannot edit or delete patient data.
[x] Criterion 6: Admins should be navigated to an edit page upon clicking on 'edit' for any doctor entry.
[x] Criterion 7: The table should be designed in a way that allows it to be reused for different datasets (patients for doctors and doctors for admins) while maintaining consistent styling and behavior.

Additional Information

Implement user authentication and authorization checks to ensure that only authorized users can perform actions specific to their roles.

Notes

Dependencies: This user story depends on a well-defined database schema and the necessary API endpoints for loading and managing data.
Related Issues: #29 #24  #32 #, data base set up(CRUD) etc

We use SSL instead of personnumber now


Error message per line

Is there a way to configure error message per line, except prepending each line with another @defaultMessage line?

Yes, just do (example from Lucene having both variants (https://github.com/apache/lucene-solr/blob/master/lucene/tools/forbiddenApis/tests.txt):
junit.framework.TestCase @ All classes should derive from LuceneTestCase

java.util.Random#<init>() @ Use RandomizedRunner's random() instead
java.lang.Math#random() @ Use RandomizedRunner's random().nextDouble() instead

# TODO: fix tests that do this!
#java.lang.System#currentTimeMillis() @ Don't depend on wall clock times
#java.lang.System#nanoTime() @ Don't depend on wall clock times

com.carrotsearch.randomizedtesting.annotations.Seed @ Don't commit hardcoded seeds

@defaultMessage Use LuceneTestCase.collate instead, which can avoid JDK-8071862
java.text.Collator#compare(java.lang.Object,java.lang.Object)
java.text.Collator#compare(java.lang.String,java.lang.String)

It would just be good to add some documentation about the signature files. So I keep this issue open. That's really needed - sorry for not yet adding this.


Add examples to .npmignore

As of<EMAIL_ADDRESS>(https://github.com/gatsbyjs/gatsby/pull/15284), we now parse GraphQL queries in dependencies to enable packages to ship queries!
The examples in this repo however interfere with this and cause this package to break (details in https://github.com/gatsbyjs/gatsby/issues/15344)
We highly recommend not shipping examples to npm and this PR fixes this by adding it to .npmignore
Cheers from the gatsby team!

ty!

I have the same problem after updating gatsby to 2.13.3 version.
I have two custom themes which are submodules in my main git project. One theme is using gatsby-source-datocms plugin. Any solution for this?

I deleted the folders manually, how can I upgrade the module while I'm using the repository's v2 branch as the source of the package?

either yarn add gatsby-plugin-mailchimp or npm install gatsby-plugin-mailchimp
do neither of these work for you?

Does that support Gatsby v2?

yeah... this plugin is built for v2.  (for v1, we have specific instructions here: https://github.com/benjaminhoffman/gatsby-plugin-mailchimp#using-gatsby-v1)

ok thanks


Update README with new logo

https://github.com/idiotWu/smooth-scrollbar/discussions/461#discussioncomment-2371208
Full changelog:

Add logo svg
Reformat top of readme
Add logo to readme
ImgBot optimize images
Add zer0bin to who's using it

Thanks!


Memoize time calculations

The website lags when changing between light/dark mode, and I think the issue is either these calculations, or updating the graphs. Needs more investigating though. In either case, the calculations should still be memoized because they are somewhat heavy if there's a lot of data.

Closing, because this isn't a problem anymore. It was likely fixed by #85. Can be reopened later though, if necessary.


Is this a virus?

IDK considering you sell trojans

ok so i believe it isnt and im using it, but it keeps saying errors in dev tools console like
   GET https://api.bloxflip.com/ 404 (Not Found)


json unmarshal "null" to empty slice variable(etc: []string{} ) doesn't have err. But when marshal the variable, the value will be "null" instead of "[]"

What version of Go are you using (go version)?
go version go1.10.2
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
linux/amd64
What did you do?
ls := []string{}
if err := json.Unmarshal([]byte("null"), &ls); err != nil {
	fmt.Println(err)
}
fmt.Println(ls)
if result, err := json.Marshal(ls); err != nil {
	fmt.Println(err)
} else {
	fmt.Println(string(result))
}

What did you expect to see?
output:
[]
[]
What did you see instead?
output
[]
null

This is a common misconception in the encoding/json package, but it is very clearly documented:

Array and slice values encode as JSON arrays, except that []byte encodes as a base64-encoded string, and a nil slice encodes as the null JSON value.

If you want to marshal a slice as an empty JSON array, you need to make the slice be non-nil. This has been covered at length before: https://stackoverflow.com/questions/33183071/golang-serialize-deserialize-an-empty-array-not-as-null


Feat:Footer Enhancement required in Community page

Describe the feature
The footer on the community page is misaligned, causing a poor user experience. Elements within the footer, such as text and icons, are not properly positioned, leading to an unbalanced appearance that disrupts the overall layout of the page.
Add ScreenShots
This is the present view of the community page

I would like to change the structure similar to the events page:

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@anuragverma108 please assign me this issue and assign me a level2 label

https://swapreadsconnect.netlify.app/comsp , its ok . that was a redundant page , which is now been deleted .


[🐛] Property quality property not working with launchImageLibrary

quality property not working with launchImageLibrary
I added quality in my options
const options = {
quality: 0.2,
storageOptions: {
skipBackup: true,
path: "images",
},
};
then i made call to launchImageLibrary(options, async (response) => {})
but still i am getting the same image size on selecting the image from library , i want the quality of image to be reduced so the filesize is less

I am unable to reproduce in latest version. What version are u using and provide more details, like the original image size and image size in the response.

same here

Same problem here. Tested on Android. If quality is the only params, their is no change on the image, which will be the same size. If I have other params like maxWidth, the quality can be changed.

Going to close the issue.  No information on the platform, picker version, etc.


[Enhancement] Improve the performance of extract conjuncts/disjuncts

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

The function  extractConjuncts/extractDisjunctive will create a new List evrey time, When query has more than 1000+ OR, AND predicate,  This function will call 1000+ recursively , it cost a lot of time.
The time cost of query from 50s to 4s+  after optimize.

[FE PR Coverage Check]
:heart_eyes: pass : 20 / 21 (95.24%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/Utils.java
20
21
95.24%
[86]

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio backport branch-2.0

@Mergifyio backport branch-2.1

@Mergifyio backport branch-2.2

@Mergifyio backport branch-2.3


The when conditional requires boolean state variable.

This is implicit state contract.
If in a story composition c1s1 sets b1v1 which will be used in the conditional, we should check boolean type at the moment of assignment.

Superseded by #723


node/http: pprof endpoint fix

Interesting that the test did not catch this issue...

Interesting that the test did not catch this issue...
EDIT: just realized this is a change to align behavior with stdlib, not a bug fix.

It is in fact a bugfix. Without the slash it is impossible to access /debug/pprof/heap or /debug/pprof/goroutine


add srcRESEARCH feature (Research Paper - PDF)

Description
for creating those awesome research paper you did in Zenodo.
Expected Behavior
GitHub ---fork---> create research paper ---> create .pdf or .epub.
Current Behavior
No chance to do it.
Associated Data Files
No response

Feature accepted.

Removed .epub requirement due that format disallowed 2 columns output and research papers are generally formatted in 2 columns per page.

Just curious, do you need LaTex?

If it isn't relic enough, sure.

ok .. will open a new  source to track it.

Implemented in 6e7a6bf6138b8b1e2bdd29b0230bc2470bb172de


feat: reset handling fix 1.2a

What I did

refactor: Add AtCacheManager so that we can handle all caching operations in one place
refactor: Move cache-related operations from LookupVerbHandler and ProxyLookupVerbHandler into AtCacheManager
test: Added unit tests covering full behaviour of LookupVerbHandler and ProxyLookupVerbHandler including caching
feat: Handle resets of other atSigns by detecting changes to their public encryption keys
test: Added unit tests covering behaviour when public encryption keys changes detected
test: Added unit tests covering behaviour of the CacheRefreshJob

How I did it
The material code changes are in

AtCacheManager
OutboundClient
AtCacheRefreshJob

The most interesting code is in the 34 new unit tests spread across

lookup_verb_test.dart
proxy_lookup_verb_test.dart
cache_refresh_job_test.dart
reset_handling_test.dart
and a huge amount of shared and reusable mock setup boilerplate in test_utils.dart

All of the other code changes are lint removal and other zero-impact refactoring
How to verify it
Tests pass

Thank you all for your time discussing this PR just now @sitaram-kalluri @srieteja @purnimavenkatasubbu @murali-shris


Open local databases automatically

Is there a way to configure DBeaver to open up locally stored sqlite databases, by doing the heavy-lifting of creating a (temporary) database connection (connection type determined by file extension, say), so that locally stored databases could be opened by right click->open with DBeaver? DBeaver is great tool, but our company deals with a lot of standalone Sqlite databases which get passed around, and having users create connections everytime they want to look inside a database may be prohibitively time consuming. If it's not currently possible, is there a theoretical workaround (even if takes quite a bit of programming, say creating a 'driver' executable that creates a new 'connection' in the data-sources xml file, and then opens up DBeaver with certain command line arguments) to accomplish this?
Thanks!

Sound like a good feature.
I think generating datasources.xml + opening new DBeaver instance is too heavy solution.
This can be achieved with command line parameters for standard dbeaver executable.
Will think about that after 4.0 release.

This feature will be added in 4.0.7.
Command line arguments references: https://github.com/serge-rider/dbeaver/wiki/Command-Line

Works great! For others who might be interested, this allows tweaking the registry a little bit to open up .db files in dbeaver automatically:
Add key HKEY_CLASSES_ROOT.db -> sqlite3_db
Add key HKEY_CLASSES_ROOT\sqlite3_db\shell\Open with dbeaver\command -> C:\Program Files\DBeaver\dbeaver.exe -con "driver=sqlite|database=%1"

out of your command line argument, I made the DBeaver.desktop for Ubuntu (and SQLite)
dbeaver.desktop_example.txt
might help some one ....

I came here from google when wanting to do this on windows. Here's a batch that will open a file in DBeaver on double click as a new SQlite connection:
echo %~f1
@echo off
"C:\Users\<username>\AppData\Local\DBeaver\dbeaver.exe" -con "driver=sqlite|database=%~f1|name=%~n1"


[AKS] Allow updating the ssh key value if cluster was created without ssh key

If the cluster was created with az aks create --name $CLUSTER_NAME --resource-group $RG_NAME --no-ssh-key, this PR allows the customer to update ssh public key with az aks update --name $CLUSTER_NAME --resource-group $RG_NAME --ssh-key-value $YOUR_SSH_KEY_VAULE.

AKS

Better add a new live test case (or update existing one) to test the change

added.

Queued a live test pipeline to verify the change


[simdjson ] Update to 0.8.1 fix missing threads

Describe the pull request

What does your PR fix? Fixes #13761

Update simdjson to the latest version 0.8.1 to fix  missing threads,There are problems with the old version. The latest version has been fixed and changed. I can successfully build and use simdjson:x64-windows in the local test.
Note: No feature need to test

The latest version is 0.8.2, so close this pr.


fix deploy --target-imgref to also use for writing ostree ref

When using deploy --target-imgref, we still use the source image reference when writing the ostree ref, so it shows up in things like ostree-ext-cli container image list etc.
We need to drive the target imgref farther down into the pull stack.

This was fixed by https://github.com/ostreedev/ostree-rs-ext/pull/147


fix for calendar not being reset properly

https://github.com/boxture/oms/issues/3419

TypeError: Cannot read properties of undefined (reading 'src')
    at extended.hiddenInputChanged (component_controller.js:229:1)
    at Binding.invokeWithEvent (stimulus.js:383:1)
    at Binding.handleEvent (stimulus.js:340:1)
    at EventListener.handleEvent (stimulus.js:36:1)
    at extended.refreshInputs (component_controller.js:316:1)
    at extended.refreshCalendar (component_controller.js:407:1)
    at extended.selectDay (component_controller.js:275:1)
    at Binding.invokeWithEvent (stimulus.js:383:1)
    at Binding.handleEvent (stimulus.js:340:1)
    at EventListener.handleEvent (stimulus.js:36:1)```


Add first class support for Pydantic

This PR adds support for Pydantic classes for output validation.
Resources

RFC GRAIL-001 for more details
Demo notebook

To Do

[x] Add support for all RAILs types (including choice/case)
[x] Fix naming bug in output RAIL (the model name isn't correctly used)
[x] Add support for instructions
[x] Add support for Field objects
[x] Extend tests to be a full kitchen sink with deeply nested types
[x] Add support for validators (custom and built in)
[x] Update RFC
[ ] Remove support for scripts (pending discussion)
[ ] Rename format to validators
[ ] Increment RAIL specification (or wait?)
[ ] Add support for EmailStr and UrlStr

Closed for #175.


Being able to call StartUsing twice and StopUsing once

When grabbing an interactable object that doesn't require being grabbed for being used (like the guns in scene 13) and touching it with both controllers at the same time, pressing the 'Use' button on one and the the other will cause 'start using' to be called twice, as expected. However, when releasing one and then the other, 'stop using' will only be called once, when the use button is pressed in the second place is released

Is this still an issue? a lot of work has been done on these areas


Added Moody to ban list. Fixed Vadossum

I might have messed up on the banlist. lemme know if I did.

#17


Chore - Refactor SchemaVersionDAO to Kotlin

Context
SchemaVersionDAO is currently the only main classes that is still in Java code. It needs to be refactored into Kotlin code to bring it inline with the rest of the library (excl. utils files).

Definition of Done
SchemaVersionDAO in Kotlin code with mocking in test working seamlessly.

Merged PR #44, closing this issue.


update framework dependency to ~3.1

Allows this module to be installed with SS 3.2. I'm currently running it on 3.2. and seems fine, if you could merge and release that would be great!

Thanks! Merged and released as 1.0.2

Great stuff, thanks James!
On Tue, Feb 2, 2016 at 10:04 AM, James Goodman<EMAIL_ADDRESS>wrote:

Thanks! Merged and released as 1.0.2
—
Reply to this email directly or view it on GitHub
https://github.com/guttmann/silverstripe-security-headers/pull/4#issuecomment-178189192
.

--
Shea Dawson
Web Developer
LiveSource
mobile +64 20 4028 5647
skype squatchnz
email<EMAIL_ADDRESS>*web *www.livesource.co.nz


Fixup OpenAI Embeddings - fix the weighted mean

Re: https://github.com/hwchase17/langchain/issues/3777
Copy pasting from the issue:
While working on https://github.com/hwchase17/langchain/issues/3722 I have noticed that there might be a bug in the current implementation of the OpenAI length safe embeddings in _get_len_safe_embeddings, which before https://github.com/hwchase17/langchain/issues/3722 was actually the default implementation regardless of the length of the context (via https://github.com/hwchase17/langchain/pull/2330).
It appears the weights used are constant and the length of the embedding vector (1536) and NOT the number of tokens in the batch, as in the reference implementation at https://github.com/openai/openai-cookbook/blob/main/examples/Embedding_long_inputs.ipynb

Here's some debug info:

We can also validate this against the reference implementation:

Reference implementation (click to unroll)
This implementation is copy pasted from https://github.com/openai/openai-cookbook/blob/main/examples/Embedding_long_inputs.ipynb
import openai
from itertools import islice
import numpy as np
from tenacity import retry, wait_random_exponential, stop_after_attempt, retry_if_not_exception_type

EMBEDDING_MODEL = 'text-embedding-ada-002'
EMBEDDING_CTX_LENGTH = 8191
EMBEDDING_ENCODING = 'cl100k_base'

# let's make sure to not retry on an invalid request, because that is what we want to demonstrate
@retry(wait=wait_random_exponential(min=1, max=20), stop=stop_after_attempt(6), retry=retry_if_not_exception_type(openai.InvalidRequestError))
def get_embedding(text_or_tokens, model=EMBEDDING_MODEL):
    return openai.Embedding.create(input=text_or_tokens, model=model)["data"][0]["embedding"]

def batched(iterable, n):
    """Batch data into tuples of length n. The last batch may be shorter."""
    # batched('ABCDEFG', 3) --> ABC DEF G
    if n < 1:
        raise ValueError('n must be at least one')
    it = iter(iterable)
    while (batch := tuple(islice(it, n))):
        yield batch

def chunked_tokens(text, encoding_name, chunk_length):
    encoding = tiktoken.get_encoding(encoding_name)
    tokens = encoding.encode(text)
    chunks_iterator = batched(tokens, chunk_length)
    yield from chunks_iterator

def reference_safe_get_embedding(text, model=EMBEDDING_MODEL, max_tokens=EMBEDDING_CTX_LENGTH, encoding_name=EMBEDDING_ENCODING, average=True):
    chunk_embeddings = []
    chunk_lens = []
    for chunk in chunked_tokens(text, encoding_name=encoding_name, chunk_length=max_tokens):
        chunk_embeddings.append(get_embedding(chunk, model=model))
        chunk_lens.append(len(chunk))

    if average:
        chunk_embeddings = np.average(chunk_embeddings, axis=0, weights=chunk_lens)
        chunk_embeddings = chunk_embeddings / np.linalg.norm(chunk_embeddings)  # normalizes length to 1
        chunk_embeddings = chunk_embeddings.tolist()
    return chunk_embeddings

long_text = 'foo bar' * 5000

reference_safe_get_embedding(long_text, average=True)[:10]

# Here's the first 10 floats from the reference embeddings:
[0.004407593824276758,
 0.0017611146161865465,
 -0.019824815970984996,
 -0.02177626039794025,
 -0.012060967454897886,
 0.0017955296329155309,
 -0.015609168983609643,
 -0.012059823076681351,
 -0.016990468527792825,
 -0.004970484452089445]

# and now langchain implementation
from langchain.embeddings.openai import OpenAIEmbeddings
OpenAIEmbeddings().embed_query(long_text)[:10]

[0.003791506184693747,
 0.0025310066579390025,
 -0.019282322699514628,
 -0.021492679249899803,
 -0.012598522213242891,
 0.0022181168611315662,
 -0.015858940621301307,
 -0.011754004130791204,
 -0.016402944319627515,
 -0.004125287485127554]

# clearly they are different ^

The original implementation of the length safe embeddings comes from https://github.com/hwchase17/langchain/pull/991. Would love you input @Hase-U in case there's something I misunderstood here.

Revert the style changes ^

Thank you for your careful correction.
Certainly, token length should be used for that part

Just want to point out for posterity that since https://github.com/hwchase17/langchain/pull/2330 _get_len_safe_embeddings became the default embedding method, which means some embeddings computed for texts longer than context max token length may be corrupted (the weight on the last batch may be too high).


Duplicate items from view.delta when paging

I'm seeing this behavior using a OneDrive for Business account, haven't tested it with OneDrive Personal.
What I observe is that if I change an Item, all of it's parent folders up to root also appear in the response.
A bit of a contrived example, but if I modify 'Nested Document1' and 'Nested Document2' and set the page size to 1 with the following hierarchy:
root
  => Top Level Folder
      => Nested Document1
      => Nested Document2

When I make the request GET /drive/root/view.delta?$top=1
The first page actually contains 'Top Level Folder' and it's parent 'root'
['Top Level Folder', 'root']
Then requesting the<EMAIL_ADDRESS>the second page contains 'Nested Document1' and it's parent up to 'root'
['Nested Document1', 'Top Level Folder', 'root']
Again when requesting the<EMAIL_ADDRESS>to get the last page it contains 'Nested Document2' and it's parent up
['Nested Document2', 'Top Level Folder', 'root']
Is this the expected behavior?

+1

Yes, this is by design. With view.delta, we always return all of the items in the hierarchy above the modified items to ensure that you have all of the items required to place that item into the correct place in the hierarchy. If you ask for a page size of 1, we return one "changed item" and then all of the parent items of that item. There is definitely overhead in doing it this way, but it helps to ensure the integrity of the data set you generate from the results.


An Academic query for the source code

Hi,sir!
I'm sorry to disturb you in your busy schedule.Your paper "A Perspective of Q-value Estimation on Offline-to-Online Reinforcement Learning"  gives me great inspiration. But I have encountered some troubles when trying to implement the algorithm. If possible, can you make the source code public?

Thanks for you interest! We are polishing related code and will release the whole training code in 1-2 weeks.

Thanks for your reply!I will be waitting for your source code!


Configurable client timeouts

PlanB should expose a flag to configure client timeouts on the server: ReadTimeout, ReadHeaderTimeout, WriteTimeout, and IdleTimeout (some require go 1.8).

closed by #21


Some keys like "[" (left bracket) not working in terminal (German keyboard layout)

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.63.2
OS Version: Windows 10 Enterprise (build 19042.1415)

Steps to Reproduce:

Open the terminal in VSCode with a German Keyboard Layout
Try typing keys like "[" (which is <AltGr> + 8 on a German keyboard). It is not working at all.

I noticed a few other issues:

The closing bracket "]" also doesn't work 100% correctly. The character appears immediately, but pressing e.g. <ENTER> immediately after isn't recognized anymore, i.e., it is necessary to press <ENTER> twice to be recognized.
The ß key doesn't work either. It gets translated into the characters ^\.
The \ also gets translated to the character ^\.
The € sign doesn't work at all.

At least the "[" used to work fine until the update to 1.63.2. Currently the terminal is not usable for coding, because typing "[" is required much too often to have it missing.
Note: The keys work correctly in the regular editor window.

/duplicate of https://github.com/microsoft/vscode/issues/133597
Can you pls confirm this is fixed for you in Insider's?

@meganrogge Thanks for the feedback, and sorry for the delay :wink:
Even though the linked issue describes a quite different manifestation of the problem (I actually don't have to press <ENTER> multiple times after a quote, and not having the [ key available at all is much more limiting as a coder) it seems to have the same root cause: I can confirm that in the Insider's version everything seems to work fine for me.
Is there a way to track when the fix get included into an official release?

yes, it will be released the first week in February.


Corrected PR #996 according to comments in #995.

Changed S3CopyToTable.s3_load_path back to a method according to the
suggestion: https://github.com/spotify/luigi/issues/995#issuecomment-123896142

Modified the accompanying test case accordingly.

Please review @carlineng

Looks good, thank you @kianho

And there's no issues with breaking backward compatibility here guys? Or it just unbreaks what it previously broke?

Correct, this simply reverts the change made in spotify/luigi#996, which broke the redshift S3CopyToTable task.

@Tarrasch, correct, it unbreaks #996

s3_load_path is still decorated as @abc.abstractproperty -- should it be changed to @abc.abstractmethod?  I was using it as a property so this commit broke some of my code (was previously using 1.3.0 and upgraded to master.)

@joeshaw I noticed that too re: @abc.abstractproperty. I'm thinking of making a couple more changes to the redshift module and will be addressing that point.

It appears that @kianho 's PR intended to change s3_load_path back to a method (as noted by the used of self.s3_load_path()). However, L72 shows that the decorator change never occurred.
No one (including myself) had picked up on the inconsistent use of an @abc.abstractproperty being called as a method (self.s3_load_path()).
I will update this in my existing pending PR #1393

good spot @dlstadther, apologies for the oversight.


Update a few libraries

Pull Request Description
What issue does this change request address?
#1972
What are the significant changes in functionality due to this change request?
Updates scikit-learn, statsmodels and ray
Removes using some deprecated python features.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[x] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Mingw Test on 2a3658a : invalidated by @joshua-cogliati-inl
failed in fetch

Hm, some of the regolds look like notable changes to me. Does anyone know how "analytic" or not those gold files were?

Hm, some of the regolds look like notable changes to me. Does anyone know how "analytic" or not those gold files were?

The clustering/datamining ones probably are not very analytic.

Tests affected:

tests/framework/PostProcessors/DataMiningPostProcessor/DimensionalityReduction/SpectralEmbedding
tests/framework/PostProcessors/LimitSurface/testLimitSurfaceIntegralPPWithBoundingError
tests/framework/PostProcessors/TemporalDataMiningPostProcessor/Clustering/MiniBatchKMeans

Comments on differences:
The tests/framework/PostProcessors/DataMiningPostProcessor/DimensionalityReduction/SpectralEmbedding is mostly due to a few points being fairly different.
The biggest difference is the 10th number:

4.9
3.1
1.5
0.1
0
10
0.0283713757382
-0.016950871197

versus:

4.9
3.1
1.5
0.1
0
10
0.0288232499833
-0.015954529555

The tests/framework/PostProcessors/LimitSurface/testLimitSurfaceIntegralPPWithBoundingError differences are solely due to the EventProbability_err being different.  I am wondering if this is being calculated by subtracting two numbers that very close.
Original Windows:
0.01365
Original Other:
0.01371875
New Windows:
0.0138
New Linux:
0.01388125
New Mac:
0.013725
The tests/framework/PostProcessors/TemporalDataMiningPostProcessor/Clustering/MiniBatchKMeans look quantitatively similar:

@wangcj05  or @PaulTalbot-INL  any more questions?

Job Test Ubuntu 18 PIP on f28fa11 : invalidated by @joshua-cogliati-inl
FAILED: Failed tests/framework/InternalParallelTests/ROMscikit

Job Test Ubuntu 18-2 Python 3 on f28fa11 : invalidated by @joshua-cogliati-inl
FAILED: Failed tests/framework/ensembleModelTests/testEnsembleModelNonLinearParallel

Job Test Ubuntu 20-2 Optional on 2a3658a : invalidated by @joshua-cogliati-inl
FAILED: Diff tests/framework/PostProcessors/LimitSurface/testLimitSurfacePostProcessor

Checklist is good. Test are green. PR can be merged.

@joshua-cogliati-inl Could you send out an email to user group since there are libraries update, and the users need to update their libraries also.


[HB-5072][HB-4707] 4.0.0 API Changes & Remove Reward Value from API

Description
This implements the current API changes for Helium 4.0.0. Additionally, the current Helium 3.X rewards do not provide value and generally contain either 0 or 1 in iOS or some string on Android. Given this information, there is nothing a publisher can do or action upon.

Mostly looks good. But this will greatly benefit from an actual RC for future changes.

https://chartboost.atlassian.net/browse/HB-5078 ticket that is blocking this


Undefined variable: response

what is the variable response passed to "new DownstreamResponse" under Downstream Messages example. I am getting an error with this.

Hi,
Thank's for this remarks, there are indeed a mistake in this example, I will remove the following line in the example :
$downstreamResponse = new DownstreamResponse($response, $tokens);
You can remove it now in your code.
Best regards

Thanks really appreciate your help and an awesome package it is.


Add option to reset min and max in bookmarklet

I believe that starting the stats bookmarklet can introduce a spike in the green ms/Frame display, so the "max" often jumps to 30 or higher, even if the framerate is normally a stable 60FPS.
Therefore I typically have to reopen the stats display multiple times to get a fresh min/max:

I have to start the bookmarklet each time I want to take a fresh measurement.
I then press the stats display to get to the green ms/F display.
Only now I find out if it's actually usable by looking at the existing min/max.
if there was already a spike I have to restart the bookmarklet again.

This is a very bad workflow and I'd like to see this improved.

I assume that #55 does this, although I didn't try it yet. But the chances of it being merged are slim?

Try dopa-stats(another implementation)。Right click to reset min/max。

Right click to reset min/max。

I really like this solution.

Try dopa-stats(another implementation)。

Unfortunately, there's no bookmarklet yet.
I also don't really want to trust a repository with 0 stars (or dependencies with a similar situation) that's written mostly in a language that I don't understand, for injecting code into debug builds of our proprietary products.
Therefore, I'd still prefer if there was an implementation as part of this repository (which is owned by a reputable developer and has gotten a lot of reviews).


Add missing leaveUrl parameter to LRAClient#updateCompensator

Signed-off-by: xstefank<EMAIL_ADDRESS>

@xstefank My point was that there are two join methods so you need to have two updateCompensator methods as well.

@mmusgrov should I then update this PR to add that method?

I will add it to this PR. If the outcome of the discussion will be to remove URLs methods I will update this PR to just add a new updateCompensator method

superseded by https://github.com/eclipse/microprofile-lra/pull/55


Redis Configuration from  Environment Variable

I think it would be extremely nice if ActionCable's Redis Connection could be configured by just setting an environment variable - i.e.: 'REDIS_URL=redis://somehost:6739' - in the same way ActiveRecord does with 'DATABASE_URL', whenever it doesn't find the 'config/database.yml' file...
Right now, if I dismiss/forget creating the 'config/redis/cable.yml' file, ActionCable raises an error.

just create the file and set the cable.yml to be the environment variable?

We're going to ship Action Cable with Rails directly, so this file will be present. I don't think this is a good fit because you may well have many Redis instances.

Fair enough :)
@kayakyakr I ended up setting the file like this.
Actually thats similar to how I got it a couple months ago... but I forgot that yesterday.


Add an analyzer to catch inadvertent ephemeral delegates

There are several Win32 APIs that take callbacks, e.g. WNDPROCs or DLGPROCs. These are currently projected as delegates, and if you do the "natural thing" and assign static methods to fields of these *PROC types, the runtime will create a temporary delegate, which gets garbage collected from under you, and you will start getting ExecutionEngineExceptions at random.
These are hard to debug (they don't hint at what went wrong), so they constitute a dangerous sharp edge.
cswin32 should either have a build-time warning, or an analyzer, that validates that you are not falling into the trap of doing the natural-yet-unsafe way of assigning these callbacks.
An alternative is to change the projection of structs taking callbacks to always use function pointers instead of delegates.
See https://github.com/microsoft/CsWin32/issues/623#issuecomment-2230824850

Thanks for filing this. This has come up from two others in the last week besides you, yet before that I hardly heard about it at all. Funny how perfect storms come up like that.
I am curious about your failure. In studying code with others, it seems that modern C# compilers actually store a delegate over a static method in a static field just as an optimization, but with the nice side effect of avoiding the very problem you're seeing. Can you confirm by looking at the compiled IL whether a field for the delegate isn't implicitly created?
Here is a sample of the delegate being cached.

looking through the IL dump, I see that it just creates an instance of WNDPROC which is a delegate, no field is created for the delegate.
WNDCLASSEXW param = new WNDCLASSEXW
{
	cbSize = (uint)Marshal.SizeOf<WNDCLASSEXW>(),
	hInstance = PInvoke.GetModuleHandle((PCWSTR)null),
	lpszClassName = ptr,
	lpfnWndProc = WndProc
};

IL_004c: ldftn valuetype Windows.Win32.Foundation.LRESULT [redacted]::WndProc(valuetype Windows.Win32.Foundation.HWND, uint32, valuetype Windows.Win32.Foundation.WPARAM, valuetype Windows.Win32.Foundation.LPARAM)
IL_0052: newobj instance void Windows.Win32.UI.WindowsAndMessaging.WNDPROC::.ctor(object, native int)
IL_0057: stfld class Windows.Win32.UI.WindowsAndMessaging.WNDPROC Windows.Win32.UI.WindowsAndMessaging.WNDCLASSEXW::lpfnWndProc

We could potentially trigger the analyzer when the ephemeral delegate is used to initialize an argument or struct field that is annotated with the [Retained] attribute (which is described at https://github.com/microsoft/win32metadata/issues/1726). We'd need to work with the win32metadata repo to get the appropriate places attributed.


VS Code Azure Functions extension for Python still has a Dotnet dependency

Azure Functions extension 0.17.1 (Windows) fails with a dotnet dependency error:

@gbowerman what version of the func cli do you have? If you're on the latest and create a new project - it shouldn't need dotnet.

@EricJizbaMSFT
Azure Functions Core Tools (2.7.1158 Commit hash: f2d2a2816e038165826c7409c6d10c0527e8955b)
Function Runtime Version: 2.0.12438.0

However, I think this was being called from the VS Tools Azure Functions extension 0.17.1


Products Urls are not regenerated

I am using Magento 2.2.2
I have around 400 products in my website and these products are imported not created manually. It seems that all categories url working fine but products urls are not working.
I run below command:
 php bin/magento ok:urlrewrites:regenerate
but still product urls are not regenerated and it returning urls like below:
catalog/product/view/id/2/s/product-url/category/19/
I also tried running other command as well:
php bin/magento ok:urlrewrites:regenerate --save-old-urls
Note I have Use Categories Path for Product URLs set to Yes.
But no luck. So, what is the real problem for this?

Hello @ChintanKaneriya
From version 1.4.0 the default command (without params) regenerate only products URL rewrites (direct links to products). To regenerate a URL rewrites of products in some specific category (or in all categories), please, run commands:

in all categories:
php bin/magento ok:urlrewrites:regenerate --entity-type=category

in specific category with ID=19 (category ID from your comment):
php bin/magento ok:urlrewrites:regenerate --entity-type=category --category-id=19

Best regards,
Oleg Koval


Twitter demo code breaks in Python 2.7

  File "build/bdist.macosx-10.10-x86_64/egg/nltk/twitter/twitter_demo.py", line 200
    print("{} ".format(tweet['created_at']), end='')
                                                ^
SyntaxError: invalid syntax

@fievelk or @ewan-klein – would you please take a look at this?

Hopefully 52a894eb4cc8cb410eef30f8ff3e21e70801e54c will fix this.

Thanks @ewan-klein


Next.JS Error in withClerkMiddleware.js

[x] Review the documentation: https://docs.clerk.dev/
[x] Search for existing issues: https://github.com/clerkinc/javascript/issues
[x] Go through package changelog files.
[x] Provide the Frontend API key from your application dashboard.

Package + Version

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[x] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore
[ ] other:

Version:
4.10.2

Browser/OS
If applicable e.g. Chrome latest
Description
Hello, I'm running a turborepo based on https://github.com/clerkinc/t3-turbo-and-clerk/, a T3 Stack with Clerk instead on Next-Auth. Everything works fine except for one error that gets triggered apparently randomly in the middleware.
I'm using Next.JS version 13.1.6 with the same middleware.ts of the example repo and I get the following error:
@badget/web:dev: error -<EMAIL_ADDRESS>(11:0) @ decorateResponseWithObservabilityHeaders
@badget/web:dev: error - Cannot convert argument to a ByteString because the character atindex 126 has a value of 8217 which is greater than 255.
@badget/web:dev: null

If I understand correctly it seems that the middleware is trying to set some additional headers in the response but failing to do so, because of conversion. As I said my middleware is just using the withClerkMiddleware without any custom logic. How can I fix it or debug it further to provide more info?
Thank you in advance and keep up the great work!

Hello @matteobad , thank you for reporting it. Let me take a look and get back to you.

This seems to be the same case as https://github.com/vercel/next.js/issues/44780#issuecomment-1380030158
Could you provide the request body and headers that cause this issue to investigate it further? Until then, i will try to pinpoint it if there is an invalid character in our codebase.

Right now the error is present on every page. I'm using tRPC to call my backend but this doesn't seem to be the problem because I get the same error if I try to create a new Page with nothing in it like this one:
export default function TestPage {
  return (
    <>
      <Head>
        <title>Create T3 App</title>
        <meta name="description" content="Generated by create-t3-app" />
        <link rel="icon" href="/favicon.ico" />
      </Head>
      Budget
    </>
  );
};

And this is the content of my middleware.ts:
import { withClerkMiddleware } from "@clerk/nextjs/server";
import { NextResponse } from "next/server";
import type { NextRequest } from 'next/server'

export default withClerkMiddleware((req: NextRequest) => {
  return NextResponse.next();
});

// Stop Middleware running on static files
export const config = { matcher:  '/(.*?trpc.*?|(?!static|.*\\..*|_next|favicon.ico).*)',};

We are setting some headers when it's unclear if the user is authenticated to access the server. The issue is related to the encoding of the header value which can only contain ISO-8859-1 (latin1) characters.
Either the generated error messages or the values used from the JWT or request contain a non-ISO-8859-1 character.
Thank you for the follow up. Let me try a few more things and add a guard to avoid invalid characters and i will come back to you.

Hi @dimkl, I just saw your PR which should solve my problem (I'll try later). I also tried to debug the middleware to give more information on the above error:

The error is related to the requestState.message that has values like: 'JWT is expired. Expiry date: Wed Feb 22 2023 15:25:56 GMT+0100 (Ora standard dell’Europa centrale), Current date: Wed Feb 22 2023 15:26:35 GMT+0100 (Ora standard dell’Europa centrale). (reason=token-expired, token-carrier=cookie)'
The error seems to be at index 116 where the: ’ char is at. Maybe its because of the IT locale? Since that piece of string is in Italian with that apos

Fixed is deployed in the latest release!


HDDS-6708. [FSO] Make normalize keyPath on key rename request

What changes were proposed in this pull request?
Path normalization missed in o3fs#renameKeyWithFSO, which will result in extra "/" in OM operations especially checkACLs logic.
Fix: Convert #pathToKey() on ozone client and perform #validateAndNormalizeKey() at OM RenameRequest.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6708
How was this patch tested?
Added UT.

Thank you @mukul1987 for the offline discussions about the fix. Please let me know your feedback.

@mukul1987
ofs:// RootedOzoneFileSystem is already removing the beginning slash at the client side and sending it to the OM. Thats why there is no extra slash issue not occurred with ofs://. Say, if user given "/vol-1/buck-1/key-1" then the below logic will construct the path to "vol-1/buck-1/key-1".
BasicRootedOzoneFileSystem.java#L399


Allow has method with numeric indexes

Fix #2182

Codecov Report

Merging #2187 (e362198) into master (09fcda8) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #2187      +/-   ##
============================================
+ Coverage     86.93%   87.01%   +0.08%
- Complexity      663      666       +3
============================================
  Files            33       33
  Lines          1554     1556       +2
============================================
+ Hits           1351     1354       +3
+ Misses          203      202       -1

Impacted Files
Coverage Δ
Complexity Δ

...enssegers/Mongodb/Helpers/QueriesRelationships.php
93.93% <100.00%> (+0.71%)
31.00 <0.00> (+3.00)

src/Jenssegers/Mongodb/Relations/BelongsToMany.php
86.00% <0.00%> (-0.14%)
42.00% <0.00%> (ø%)

src/Jenssegers/Mongodb/Query/Builder.php
89.06% <0.00%> (+0.14%)
162.00% <0.00%> (ø%)

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 09fcda8...e362198. Read the comment docs.

@stephandesouza taking look at this PR it can create some regression like this one https://github.com/jenssegers/laravel-mongodb/issues/2203

No problem, we're here to find the best solution for everyone!

What solution we could create? Any thoughts?
Thanks!

Codecov Report

Merging #2187 (428eacd) into master (4056a73) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #2187      +/-   ##
============================================
+ Coverage     86.91%   87.02%   +0.11%
- Complexity      665      669       +4
============================================
  Files            33       33
  Lines          1559     1565       +6
============================================
+ Hits           1355     1362       +7
+ Misses          204      203       -1

Impacted Files
Coverage Δ
Complexity Δ

src/Helpers/QueriesRelationships.php
94.28% <100.00%> (+1.06%)
32.00 <0.00> (+4.00)

src/Relations/BelongsToMany.php
86.00% <0.00%> (-0.14%)
42.00% <0.00%> (ø%)

src/Query/Builder.php
89.33% <0.00%> (+0.15%)
162.00% <0.00%> (ø%)

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 4056a73...428eacd. Read the comment docs.

Sorry, the test was wrong after suggested review.. This is a problem with the "reverse" foreign key used on this method, and also happens with primaryKey as int
Situation:
Author "id:1" has two books Book "id:A", and book "id:B".
Author "id:'2'" has two books Book "id:C"
Today:
getConstrainedRelatedId() returns

["1", "2"]

Fixes:
getConstrainedRelatedId() returns

[1, 2]

And yes, even a "string integer" needs to be converted to int,

Hello,
I'll take a look tomorrow with a blank project to understand better this issue.
Thanks!


License file

The repo currently has no license file. Isn't it based on https://github.com/Premiurly/polkassembly/blob/master/LICENCE? Could you add this?

Hey @Noc2 Thanks for pointing this out. We have uploaded the License.
https://github.com/polkassembly/polkassembly/pull/628

Thanks!


Docs: Sandbox is re-mounting components on single property changes

Happens with the modal and the form-stepper

Closing as the form stepper issue is duplicated in story #1647


[Bug]: The imports field in package.json is ignored

Version
28.1.2
Steps to reproduce

Clone my repo at https://github.com/tksst/jest-bug-esm-import.git
npm ci
npm test
You should see the error come up

Expected behavior
Test Succeed.
The imports field in package.json is used.
Actual behavior
Test fails.
The imports field in package.json is ignored.
 FAIL  ./index.test.js
  ● Test suite failed to run

    Cannot find module '#ext' from 'index.js'

      at Resolver._throwModNotFoundError (node_modules/jest-resolve/build/resolver.js:491:11)
          at async Promise.all (index 0)

Additional context
My project uses chalk that uses imports field, and jest on my project fails on that error.
package.json:
	"imports": {
		"#ansi-styles": "./source/vendor/ansi-styles/index.js",
		"#supports-color": {
			"node": "./source/vendor/supports-color/index.js",
			"default": "./source/vendor/supports-color/browser.js"
		}
	},

Environment
System:
    OS: Linux 5.15 Ubuntu 22.04 LTS 22.04 LTS (Jammy Jellyfish)
    CPU: (16) x64 AMD Ryzen 7 2700X Eight-Core Processor
  Binaries:
    Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
    Yarn: 1.22.15 - ~/.nvm/versions/node/v16.15.1/bin/yarn
    npm: 8.13.2 - ~/.nvm/versions/node/v16.15.1/bin/npm
  npmPackages:
    jest: 28.1.2 => 28.1.2

Duplicate of https://github.com/facebook/jest/issues/12270


Recover frictionless-py release process, tests, etc

Overview
Currently, the build fails https://github.com/frictionlessdata/frictionless-py meaning that we can't fix or modify it based on the Open Data Editor needs

@romicolman The related PR #1652 has been reviewed and only a small change is required. As soon as it gets merged we can close this one.

@romicolman
I'm going to finish it today

DONE
frictionless-py has been recovered and released


Add url previews

FYI: Moving #88 to "In Progress" in https://github.com/orgs/StampyAI/projects/1/views/3 now that we have the Kanban board...
The volunteer team is growing, so let's try to keep PRs (or commits) well linked to issues, so we don't have to spend half of the meetings cleaning up task boards and to generally have better time in future git archeology when investigating bugs and trying to find out "why did we need this in the first place" 😅

Thank you for finding the issue for me.  I try to login every day to make sure all PRs have the appropriate tags, are linked to existing issues if they exist, and if not are in the project.  Thank you for taking off some of my workload 😄


Included In Semantics

The following signature is provided for 'Included In' in CQL R1 STU 2:
For intervals:   included in precision (left Interval, right Interval) Boolean
For lists:  included in(left List, right list) Boolean
Per the above signatures, a point should not be able to be used as the first parameter to the 'Included In' operator. Per the spec, it appears that the 'In' operator should be used to determine if a point is in an interval or list. However, CQL-to-ELM is supporting the following:
month from .authorDatetime included in Interval[1, 4]
.code included in "value set"
Is the specification correct in its restriction for the point to list/interval comparison, or does something need to be updated in CQL-to-ELM to restrict 'included in' to lists and intervals?

@cmoesel  @brynrhodes
Hi Chris. I understand what you are saying, but then it becomes difficult to prevent 'included in' from being misused, except through some manual review process. For authors, this is a very frustrating situation, because they are being told through the CQL validation that they have valid CQL, but then being told not to do it after some manual review process has completed. This situation recently happened during some measure work. It would be great if there was a hard and fast rule that could be implemented. I hate to make things conditional, but maybe, as a possible solution, for point to interval/list membership tests, 'included in' could be restricted to points that are datetime typed??? That would still allow for the case where someone wants to use 'included in' as 'during'. Thoughts?

My preference would be to avoid special cases.  They're hard to remember.  I think a new signature should be added to included in:
included in precision (left Interval<T>, right Interval<T>) Boolean
included in precision (left T, right Interval<T>) Boolean

and probably the same approach for lists.
To me, it makes sense outside of dates too.  For example, this rings true for me as well:
5 included in Interval[2, 7]

The downside here is that there is more than one way to express the same thing.  Does anyone feel strongly about that?

The change of signatures would be fine with me. The use of 'in' -vs- 'included in' would then become more of an artistic restriction as opposed to a syntactic issue. I also imagine that the use of 'included in' could be controlled through a style guide at that point - e.g., by CMS measure developers.

Yeah.  I was thinking about that.  A next step might be to deprecate one of them -- but that would be a backwards incompatible change, so we'd want to be very careful about that.
But before we get too carried away, I'd still like to hear what @brynrhodes thinks of the whole matter.

Yes, the translator initially disallowed this case, but we opened it up because of this paragraph in section <IP_ADDRESS>:

In other words, the timing phrases in general compare two quantities, either of which may be an date/time interval or date/time point value, and the boundary access modifiers can be added to a given timing phrase to access the boundary of an interval.

And the result is that something like what @cmoesel is pointing out:
5 included in Interval[2, 7]

works as expected. So I agree that this is a place where the spec is missing the overload in the reference.

I've added comment 1561 to address this.


ci: Bump kubevirtci an k8s to 1.23 (#221)

Manual cherry-pick of #221
The last kubevirtci does not work well with the functests at ovs-cni.
This change copy the ssh to node mechanism from kubernetes-nmstate that
is not to work fine, it also bump k8s to 1.23.
Signed-off-by: Quique Llorente<EMAIL_ADDRESS>

/release-note-none


Merge OpenAI Triton till Dec 13rd

It is essential to keep up to date with OpenAI Triton, to get the latest features, and reduce the difficulty to upstream our changes to OpenAI Triton.
From previous upstream merges:
Commits to reland:
UTs to fix: https://github.com/intel/intel-xpu-backend-for-triton/issues/2678, https://github.com/intel/intel-xpu-backend-for-triton/issues/2878
This ticket is continuation of:

#2682
#2577
#2483
#2383

Moved up from Nov 29 to Dec 5 Triton version from OpenAI. [#2899, #2911, #2930, #2944 ]


cmake : add quantize exe

you for got to add the quantize executable to cmake :)
its a bit hacky (adding sources via ../), but i left a TODO that indicates what to do :)

@Green-Sky good catch! is it mergeable as is? if not could you complete it so that i can merge this?

It is mergable as-is. I can you/I can reorganize source into eg. examples/subfolders/ in another pr :)

@Green-Sky sounds good! thanks!


Consider renaming the default branch to main

The git ecosystem is moving away from master to main for the name of the default branch. Would you consider renaming the default branch? GitHub makes this pretty easy I think: https://github.com/github/renaming#renaming-existing-branches
https://sfconservancy.org/news/2020/jun/23/gitbranchname/

Yeah we can do that 👍

Hi @nrempel and @djc ,
The default branch has been renamed to main

Great, thanks! (I guess this can be closed now.)

🎉


access_token never changes

I'm working to implement silent renew at the moment and discovered that the access_token that I'm getting from my IdP never changes.  I logged in this morning and noticed that the access_token I received was the same as the one I had three days ago.  Thinking it might be due to a retained browser storage item, I switched browsers (from Chrome to FF), cleared any residual oidc.******************** local storage objects in FF, then logged in with my IdP.  The access_token I received in FF was the same as the one I got in Chrome this morning which was the same as the one I had three days ago in Chrome.
As far as code, here is my client setting object:
{
    "authority": "https://my-auth-server/identity",
    "client_id": "my-client-id",
    "post_logout_redirect_uri": "https://localhost:9000/ui/loggedout",
    "redirect_uri": "https://localhost:9000/ui/authorize",
    "response_type": "id_token token",
    "scope": "openid profile"
}

I'm simply calling userManager.signinCallback() when the IdP redirects to my authorization callback endpoint.  And I'm retrieving the access_token by calling userManager.getUser().
My expectation (especially with the silent renew work I'm doing which, as I understand it, returns a new access_token) is that each authenticated session would have a different access_token.
What am I doing wrong?

The access token in your code does not change, or the access token in storage does not change?

The access_token in the redirect URL from the IdP login screen to my client's authorization callback endpoint never changes.

I'm seeing this with another IdP too which makes me think it's something to do with how I have oidc-client-js set up.
I updated my client settings to point to a second, completely separate IdP and I'm seeing the same access_token for the IdP across browser sessions and, after logging out and logging back in.  To be clear, this access_token is different than the one provisioned by the first IdP, but every session for this second IdP has the same access_token as other sessions for the second IdP.

I just ran the VanillaJs sample in the repo and used the "code flow w/ IdentityServer" version and got these two access tokens:
  "access_token": "eyJhbGciOiJSUzI1NiIsImtpZCI6IjVDQ0FBMDNFRERFMjZENTMxMDRDQzM1RDBENEIyOTlDIiwidHlwIjoiYXQrand0In0.eyJuYmYiOjE2MTIyMDU0NDUsImV4cCI6MTYxMjIwOTA0NSwiaXNzIjoiaHR0cHM6Ly9kZW1vLmlkZW50aXR5c2VydmVyLmlvIiwiYXVkIjoiYXBpIiwiY2xpZW50X2lkIjoiaW50ZXJhY3RpdmUucHVibGljIiwic3ViIjoiMTEiLCJhdXRoX3RpbWUiOjE2MTIyMDU0NDQsImlkcCI6ImxvY2FsIiwianRpIjoiNDI3RkI3MjBGNDVFQ0FBNkZGQUZCNkIzRThEMUZBNzYiLCJzaWQiOiJEMzY4Q0ZFRDIyNUM3RkY1MEM1MkY4RTIwNzlGQjI1NyIsImlhdCI6MTYxMjIwNTQ0NSwic2NvcGUiOlsib3BlbmlkIiwicHJvZmlsZSIsImFwaSJdLCJhbXIiOlsicHdkIl19.puyBPtKIJfP5kb-x7TeDTUJ5UU5-qEQpZOLua-A5I4aG0Bt3AYWHaBlAiuER6j-yoSD_CAERQBj6zjDichAwwVaUGdPANJUAtPQQNwQggKwKbr-35kNlEAc2LkKf0cEpBLOiAxYXO0QDOtIGH4sQuV2CupLHx9hye0XMqdsLtOwKkbi5OfU44nsvxI2WEmGUTgU3yQss8oZ8E6sO-_AcU1MVe57bi4bUTBEP6Nshwbqd_XhgvJrKjWUQfDDDU5FoJTfneHRur7PoWBAY7MeIjfOy2QBbkY54Q3WcpTvN7MrQ3ioaPduYgtsJvMDmizPsJ0skqCXYnaPX7IYYduAK9A",

  "access_token": "eyJhbGciOiJSUzI1NiIsImtpZCI6IjVDQ0FBMDNFRERFMjZENTMxMDRDQzM1RDBENEIyOTlDIiwidHlwIjoiYXQrand0In0.eyJuYmYiOjE2MTIyMDU0NTYsImV4cCI6MTYxMjIwOTA1NiwiaXNzIjoiaHR0cHM6Ly9kZW1vLmlkZW50aXR5c2VydmVyLmlvIiwiYXVkIjoiYXBpIiwiY2xpZW50X2lkIjoiaW50ZXJhY3RpdmUucHVibGljIiwic3ViIjoiMTEiLCJhdXRoX3RpbWUiOjE2MTIyMDU0NDQsImlkcCI6ImxvY2FsIiwianRpIjoiRDlCQTFCNEFERjQ1NTVDQkNBNEVBOUI1MkEzQkQwOTMiLCJzaWQiOiJEMzY4Q0ZFRDIyNUM3RkY1MEM1MkY4RTIwNzlGQjI1NyIsImlhdCI6MTYxMjIwNTQ1Niwic2NvcGUiOlsib3BlbmlkIiwicHJvZmlsZSIsImFwaSJdLCJhbXIiOlsicHdkIl19.e-zbzDCkJJZGzBqAr8Ep8xditW84cqfJMhAJyOVvmqpNsNzRYo5XanNtN8cjeDkNj5HYahXUuJvJQudHOq6uEgFFaKhwTSP5pilz1QO7guBbgwqhv-4InPMHU8QHmqnjV5JtP-G2O7AhuCV88Z2h-COeoW24zV6KOQkaGSG7qdNW0jXQisDmB3NMbTn3Kl_OlnsMAVNVwberC3H1S1r4ZK9qR2BK_rVI6EVr4Pu0eGOsHPWZd0XEcce3oLkIXfWRTt2YaNauX_LEqNsny_c_26S-uISwvDLNhHRjCD8qzEC6jCvTk-_HDo8jb6KUSxvi9IdE6xGEqLWTY5G6lm9U-A",

So I'm pretty sure we're getting the updated/new access tokens on silent renew.

As for then why you're not seeing it, I'm not sure. The design of the UserManager is that there should be only one instance on the page, and your call to getUser only returns the current data from storage. After the silent renew, you will need to get the updated user into your app layer code.


utility meter gas sensor

I'm sorry that this is off topic, I'm trying to configure the gas meter, can you tell me if I configured the utility meter correctly?

Could you provide some details about your gas meter? How many pulses does the device send per m³?

https://www.samgas.com.ua/lichylnyk-gazu-membrannyy-samgaz-g25-rs2001-2
https://www.samgas.com.ua/sites/default/files/_samgaz_re_red_0_6_3_10-12-2018_14.pdf
I've trouble to translate the PDF document. Could you read the document and tell me the page which is about the low frequency pulses?

On the last reel after the comma is fixed permanent magnet for working with a low-frequency pulse generator. The
design of the counting device has a plastic case in which the counting mechanism itself is assembled with the display
display drums (Fig. 10):

I install yaml, everything is ok, but as soon as I insert the reader into the gas meter, esp becomes offline.

when the sensor is inserted into the gas meter, esp does not want to install yaml

http://joxi.ru/l2ZYZB5Iz9wGlm

after unplugging and turning on the power, esp starts counting from the beginning, does not remember the last value

I will have to build the setup on my own this evening to get an idea what's going on here.

globals:
  - id: total_pulses
    type: int
    restore_value: true
    initial_value: '7470502'

esphome:
  name: gas
  friendly_name: gaz

esp8266:
  board: esp01_1m
  restore_from_flash: true

preferences:
  flash_write_interval: 10min

api:
  encryption:
    key: !secret api_encryption_key

ota:
  password: !secret ota_password

wifi:
  ssid: !secret wifi_ssid
  password: !secret wifi_password
  ap:
    ssid: !secret ap_wifi_ssid
    password: !secret ap_wifi_password

captive_portal:

binary_sensor:
  - platform: gpio
    id: reed
    pin:
      number: GPIO5
    name: "Reed contact closed"
    filters:
      - delayed_on: 20ms
    on_press:
      then:
        - lambda: id(total_pulses) += 1;
        - output.turn_off: led
    on_release:
      then:
        - output.turn_on: led

sensor:
  - platform: template
    name: "Gas Cubic meters used"
    device_class: gas
    unit_of_measurement: "m³"
    state_class: "total_increasing"
    icon: "mdi:fire"
    accuracy_decimals: 2
    lambda: |-
      return id(total_pulses) * 0.001;

output:
  - platform: gpio
    pin: GPIO2
    id: 'led'

Please attach the red wire to GPIO5 / D1 and try again. :-)

I've tested the setup. It's working and the total pulses are stored every 10 minutes to the flash/rtc:
preferences:
  flash_write_interval: 10min

Does the LED of the wemos blink once on every round of the gas meter?

with d1 does not respond

Could you try to describe the issue in detail? You could write the summary in your mother language if you like.

when it was connected to d4, everything worked, but when I connected the sensor to the esp meter, it did not start. When I connected it to d1, the sensor did not work, did not react to the magnet.

You have to use a pull-up.

and how should it be connected according to the scheme? I have resistors.

I will paint the schematics later the day!

I've enabled the internet pull-up of the ESP. Please try this updated YAML:
globals:
  - id: total_pulses
    type: int
    restore_value: true
    initial_value: '7470502'

esphome:
  name: gas
  friendly_name: gaz

esp8266:
  board: d1_mini
  restore_from_flash: true

preferences:
  flash_write_interval: 10min

api:
  encryption:
    key: !secret api_encryption_key

ota:
  password: !secret ota_password

wifi:
  ssid: !secret wifi_ssid
  password: !secret wifi_password
  ap:
    ssid: !secret ap_wifi_ssid
    password: !secret ap_wifi_password

captive_portal:

binary_sensor:
  - platform: gpio
    id: reed
    pin:
      number: GPIO5
      mode: INPUT_PULLUP
    name: "Reed contact closed"
    filters:
      - delayed_on: 20ms
    on_press:
      then:
        - lambda: id(total_pulses) += 1;
        - output.turn_off: led
    on_release:
      then:
        - output.turn_on: led

sensor:
  - platform: template
    name: "Gas Cubic meters used"
    device_class: gas
    unit_of_measurement: "m³"
    state_class: "total_increasing"
    icon: "mdi:fire"
    accuracy_decimals: 2
    lambda: |-
      return id(total_pulses) * 0.001;

output:
  - platform: gpio
    pin: GPIO2
    id: 'led'

work perfectly

http://joxi.ru/gmvVao0sqYL5bm when I change the value to the current one and flash esp, nothing changes in HA, in the sensor:
sensor.gas_gas_cubic_meters_used

I assume this multiplicator is wrong:
    lambda: |-
      return id(total_pulses) * 0.001;

Could you check how many times the reed contact is closed while the last digit of the gas meter is spinning? I assume it's closed once per round. So the multiplicator should be:
    lambda: |-
      return id(total_pulses) * 0.01;

http://joxi.ru/Q2KqbERuLD4NKr then when i changed the value to 0.01

initial values can be changed in HA utility meter. But you need to add it to yaml.

Did you see the YAML above how to set the initial value of total_pulses using a temporary added button entity?

http://joxi.ru/82Q4bdkuwejP0m work))))

Cool! Please compare the value of the gas meter with the value of the ESPHome node in a few days to verify the multiplication is correct and the ESPHome increments the counter just as fast as the gas meter.

and the utility meter does not need to be added for statistics?

No. It's useless in my opinion and shouldn't be added to the YAML.

The Gas Cubic meters used sensor should be sufficient.

for some reason, some impulses are present when there is no gas consumption

How long is the cable between the ESP and the reed contact?

2 meter

The enabled internal resistor isn't probably enough and you could increase the debounce time.

delayed_on: 20ms, how much to change the value?

Please try 150ms. If you miss pulses decrease the value step by step.


Azure Resource Management Core Readme Issue

Section Link

Suggestion:
We should update 'GetIfExists()' to 'GetIfExistsAsync()'  consistent with the code

Section Link

Suggestion:
1. Add Link: https://opensource.microsoft.com/codeofconduct/
2. Add Link: https://opensource.microsoft.com/codeofconduct/faq/
@scottaddie,@AlexGhiondea and @archerzz for notification.

https://github.com/Azure/azure-sdk-for-net/pull/25986


onCancel and onApply events to be provided

I really like the library, but cannot find any events for handling when cancel and apply buttons are clicked. Please provide those ones!

https://wakirin.github.io/Litepicker/docs/events#buttonapply
https://wakirin.github.io/Litepicker/docs/events#buttoncancel

https://wakirin.github.io/Litepicker/docs/events#buttonapply
https://wakirin.github.io/Litepicker/docs/events#buttoncancel


Add alert which warns that the wallet is running in the background

Add alert just after open wallet:
alert - https://zpl.io/VkzL89M
settings - add "Allow run wallet in background" - https://zpl.io/a7vX0n8

checked


Show tip after $state.go

When i have to show next tip on other state,  it shows prev focused area.
After new state is loaded and I go to next step on that state it works well.
On project we're using AngularJs + ui-router.
How it's possible to wait while state is ready, then focus on needed element ?
Part of code:
 if(step == 3){
    $scope.focusElementId = null;
    $state.go('some.state', { param : param });

    $timeout(function(){
      $scope.focusElementId = 'some_id';
      $scope.mainCaption = "lorem ipsum lalalalal"
    }, 1000);

  }

Hi @elpofigisto, nice to meet.
Did you try looking at ui-router's documentation about identifying when transition animation done? maybe their $stateChangeSuccess event handler does the job?
Thanks

Any update @elpofigisto ?

Closing due to lack of responsiveness


Unify saving behaviour of "unknown" and "no_unit"

Adresses #3394.

I think it might be a good idea to replace unit not in ("no_unit", "unknown") here with not unit.is_udunit().
That does seem to have the same effect, but I'm not totally sure if it's a correct equivalent meaning, according to what we want and what cf-units means by it.
I'll report back after looking into the implementation within cf-units...

replace unit not in ("no_unit", "unknown") with not unit.is_udunits()

After some reading up, I conclude that I think this is a good idea.
The cf_units docs for the 'category' property explain that it currently comprises types "udunints, no-units and unknown" : https://scitools.org.uk/cf-units/docs/latest/unit.html#cf_units.Unit.category
From reading the code,
unit.isudunits()
  === unit.category = _CATEGORY_UDUNIT
  === unit.ut_unit is not _ud.NULL_UNIT

It also has specific category methods  is_unknown() and is_no_unit()
So, I think that makes sense.
However I must admit that the case ins undermined by some confusion about what is really public API.  E.G. cf_units does not publish the category values, _CATEGORY_UDUNITS etc, despite the Unit.category being pubic property. etc.
I'm guessing that is just some kind of slip.  Likewise, why don't they publish '_UNKNOWN_UNIT_STRING' and the like ...

Thanks for humouring me !
The more I think, the more this makes sense to me : The objects' ".units" are cf_unit objects, but not all cf_units are valid udunits : so, they only get saved if they are.


[CLI] description in spending proposal does not support spaces

coinstr> spend 6a4a2d7c0cbb7faee9ba0f41260041875a063940048a53f69c77a710cf158973 mohjSavDdQYHRYXcS3uS6ttaHP8amyvX78 1200 "sending back to faucet"
error: unexpected argument 'back' found

However, using no double quotes or spaces seems to work fine:
coinstr> spend 6a4a2d7c0cbb7faee9ba0f41260041875a063940048a53f69c77a710cf158973 mohjSavDdQYHRYXcS3uS6ttaHP8amyvX78 1200 description
Spending proposal 6a4f229faddac9ae110515be1b08f73b407b53d8bbbae6abc11cc1eebba2bee7 sent

It should be fixed now


Use controls without media, for streamed media?

Is it possible to somehow create the controls and indicate the duration of a medium, without specifying a media file?
This would be useful for applications that stream media on demand. It would allow the user to seek to an arbitrary position and a callback would send the requested position to the server, which would in turn seek to that position in the local media file and stream from there on.

Hi,
There is no need to specify a music file to the plugin. This plugin is only displaying a notification with 3 buttons (and some informations), he's not playing a media file.
On Android, the "MediaStyle" notifications allow up to 5 buttons into one notification. But there is no available "seek or progress bar" available for notifications. That's why I think that it would be difficult to have one.
I think that you have to handle this "seek operations" inside of your app (and not in a notification). If a user wants to seek to an arbitrary position, he would have to go back to the app (by clicking on the notification, for instance) and seek using a html/js music player inside of the app.
Did I answer your question?


Delete shim workloads tasks in pod.

This commit supports restarting containers and pods using CRI.
Delete task request now removes tasks from pod's workloadTasks map,
and added DeleteTask function to shimPod interface so new tasks can
use the same ID (ie, so a task can be restarted in a running pod).
Signed-off-by: Hamza El-Saawy<EMAIL_ADDRESS>

Thanks for splitting these up!


Changing or moving the _protected folder

My hosting company installed the software for me from Softaculous. I'm circling back and going through the install steps. Step #3 talks about renaming or moving the _protected folder.
When I changed the _protected folder to _spring, the site does not load in the browser and returns a HTTP Error 550. Also, if i leave the folder name as _protected but moved it out of my public_html folder, again the site does not load in the browser and returns a HTTP Error 550.
    a. What other folders or files do I need to change, when I change the "_protected" folder to "_spring" or simply move the _protected folder out of my public_html folder, so that my website works properly without error?

@CooperAshley2 Why don't you do a complete install from the distribution file ?
You'll need to move your _protected folder AND set the new name in your configuration file.
This last step (set the new location of the _protected folder is done by the installation wizard.
I'd suggest you to follow those step.
You have some link bellow for more information.
We are not related to Softaculous so we don't know how do they work.
You can also look inside the _constant.php for the location of the _protected folder.
https://ph7cms.com/doc/en/install
https://ph7cms.com/doc/
https://github.com/pH7Software/pH7-Social-Dating-CMS/wiki
Doing a clean installation is the first part in building a website (and if you can't do it then maybe you should pair up with someone who has the knowledge). Installation is only the first part !
Good luck

Thank you for the quick response. Whether the files are uploaded via Softaculous or the distribution file my issue is the same regardless. It's been a headache following the install instructions and making changes to a file then having to search through thousands of webpages, forums and Qs & As to learn that you also have to edit additional files for your changes to work. The links to the install and doc, those are great reads for general information but both are useless when you have very specific problems and issues. Thank goodness for Github and you guys who offer your time to help us out. I do appreciate it tremendously.
Now, I can make the changes to the config file, I just need to know what files to change and where it's   located.  Is this it: /_protected/app/configs/* ?

Hi @CooperAshley2
Great question! You will have to edit the _constants.php file located in the root folder of your website.
On line 50, change
define('PH7_PATH_PROTECTED', '/SERVER_PATH/_protected');

to your new path.
Let me know, if this works

@CooperAshley2
You'll need to move your _protected folder AND set the new name in your configuration file.

What's the path to the config file, please?

Hi @CooperAshley2
Great question! You will have to edit the _constants.php file located in the root folder of your website.
On line 50, change
define('PH7_PATH_PROTECTED', '/YOUR_CURRENT_SERVER_PATH/_protected/');

to your new path.
Let me know, if this works

This did not work unfortunately. I changed the names (if folder is left in root) and paths (when i moved folder outside of the root) and site will not load. I had to leave the _protected folder in the root for now.

Hi @CooperAshley2
Great question! You will have to edit the _constants.php file located in the root folder of your website.
On line 50, change
define('PH7_PATH_PROTECTED', '/YOUR_CURRENT_SERVER_PATH/_protected/');

to your new path.
Let me know, if this works

This did not work unfortunately. I changed the names (if folder is left in root) and paths (when i moved folder outside of the root) and site will not load. I had to leave the _protected folder in the root for now.

What PHP error did you get? Or in your log?
If you specify the FULL path to your new protected folder (with trailing slash at the end, it should work)

Please also make sure you don't have any UNIX permission problems with the new location of the PROTECTED folder. You might ask your web host company to check for you.
In my case, it works well (I tried to replicate your exact issue in order to help you better).

Anyway, having _protected folder located in your root folder is totally fine too.
There are many other security precaution in there such as .htaccess all denied in that folder, so don't worry about it :)

Finally @CooperAshley2, Would you be able to Star the GitHub repo? :)


Document reasoning for why CustomMultiChildLayout size can't depend on children and how to do it

Description
The title is taken from https://github.com/flutter/flutter/issues/18426 because this PR does exactly what the issue is title (and fixes a typo).
I will most likely write an article about how to use MultiChildRenderObject as part of the Flutter Clock challenge, but I suppose that it would be against the guidelines to link it in the docs anyway - so I hope that mentioning relevant classes, even though they lack documentation, is sufficient for now. I think that I remember learning about it as quite easy by example when I used it for the first time, which is why I also mentioned Stack, even though there might be easier examples because Stack is fairly complex.
Related Issues
Resolves #18426.
Tests
No tests are needed for documentation only.
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[x] No, no existing tests failed, so this is not a breaking change.

cc @xster @Hixie

@Piinks Thank you for taking a look! I added the explanation to the documentation now, I missed that before, so thanks for pointing it out.
I paraphrased what @hixie and @xster commented in #18426 for the first part btw.

@fkorotkov - this Pull request didn't get any status updates from Cirrus for some reason.  See for example:
query LabeledPullRequestsWithReviews {
  repository(owner: "flutter", name: "flutter") {
    pullRequest(number: 47797) {
      id
      number
      mergeable
      commits(last: 100) {
        nodes {
          commit {
            abbreviatedOid
            oid
            committedDate
            pushedDate
            status {
              state
              contexts {
                state
                targetUrl
                description
                context
              }
            }
          }
        }
      }
    }
  }
}

In the GitHub graphQL explorer.  Other PRs are getting these statuses.  Any idea what happened here?

@dnfield I think it's because of the use of Checks API since #43097. You need to query for a CheckSuite but these objects are not shown in the explorer according to documentation. But with Accept: application/vnd.github.antiope-preview+json header you'll be able to get Commit.checkSuites and something like this should work:
query LabeledPullRequestsWithReviews {
  repository(owner: "flutter", name: "flutter") {
    pullRequest(number: 47797) {
      id
      number
      mergeable
      commits(last: 100) {
        nodes {
          commit {
            checkSuites {
              id
              status
              app {
                name
              }
            }
          }
        }
      }
    }
  }
}

@fkorotkov - ahh I thought I was still seeing statuses on others that are also recent, but I was wrong.  Thanks for the tip!

@creativecreatorormaybenot This change was reverted after it was accidentally merged in yesterday. It looks like there was an analyzer error caused by trailing white space. Would you like to open a new pull request to re-land this change?

@Piinks I am sorry for that - see #48451.

No worries at all! It was not your fault. :) I'll review the new PR so we can get it back in.


SAK-49580 Not all global variables are exported on the xml - S2U-11

https://sakaiproject.atlassian.net/browse/SAK-49580

Thanks as always!