Cody eval TypeScript dataset

Iteration 2/n on adding TypeScript dataset to Cody completion evaluation suite

https://github.com/sourcegraph/sourcegraph/issues/57841

Test plan
This PR adds completion evaluation datasets.

@slimsag, what do you think about the existing (test cases already in main plus added in this PR) TypeScript dataset quality? Does it test real-world scenarios, in your opinion?
As for now, this PR tests mainly function declaration body completions (multiline completions are usually expected in such cases). Do these test cases seem too generic, and thus LLMs should be familiar to LLMs?
There is no need to review each test case individually; I'm seeking high-level feedback.

@taras-yemets In case it helps, here's the list of use-cases I came up with for Go that I am working through (I won't do all of these, just using this list for inspiration):

[ ] Wrap error
[ ] Check if channel is closed
[ ] Check if struct is empty
[ ] Implement an interface
[ ] Use context
[ ] Cast number to float
[ ] Initialize a map
[ ] Initialize a channel
[ ] Initialize a buffered channel
[ ] Non-blocking channel send
[ ] Select channel
[ ] Parallel spawn of goroutines
[ ] Struct to map
[ ] Struct to JSON
[ ] Merge two maps
[ ] Merge two structs
[ ] Clear map
[ ] Get map keys
[ ] Measure time
[ ] Write test function
[ ] Iterate map
[ ] HTTP write json
[ ] HTTP read SON
[ ] HTTP check status code
[ ] HTTP write error
[ ] HTTP server
[ ] HTTP post request
[ ] HTTP get request
[ ] HTTP get request write file
[ ] Exec command and get output
[ ] Exec command error get stdout
[ ] template.New().ParseFiles()
[ ] template.New().Funcs().ParseFiles()

Thanks for your feedback, @slimsag!
The suggestions and use cases you provided are very helpful, and I'll definitely add test cases to cover most of them!

Moved to a separate repository: https://github.com/sourcegraph/coder-eval/pull/3

Moved to a separate repository: https://github.com/sourcegraph/coder-eval/pull/3


fix(curriculum): use #<name> for element id reference

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #49274

Why these 2 tests failed? Can anyone help please?

@midikaif PRs that are marked as Drafts are often ignored by maintainers, unless you explicitly request help/review.

The tests failed, because of a network issue. I have re-run them; hopefully that issue has passed.

@ShaunSHamilton, I have removed the PR from the draft. what I have to do next, please guide me

@midikaif Now, maintainers need to review your changes.
Please do not click the "Update Branch" button, unless told to. Doing so delays this PR from being merged, because it causes the tests to re-run.

Thanks @ShaunSHamilton @Sboonny @Sembauke , you guys are amazing.


Delete operators/snyk-operator directory

The snyk-operator v1 is no longer maintained by Snyk and should be removed
Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Are you familiar with our operator pipeline?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want to use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Your submission should not

[ ] Add more than one operator bundle per PR
[ ] Modify any operator
[ ] Rename an operator
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description of the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@kat1906 The removal operation is not allowed at this moment. We can remove the definition from the git repository but the operator will remain in the openshift catalog. Let me know if this is an acceptable solution for you.

@Allda is there any indication of when it will be removed from the catalog? Will the catalog eventually sync with this repo? Currently, if a user tried to install any of the versions of snyk-monitor in this repo, it would not work so it needs to be removed from the Openshift Catalog too.

@kat1906 No it won't get sync with the ocp catalog. The removal will require a manual removal of the operator for each supported version and the release a new catalog. We can do that manually for this case but I won't get to it this week, maybe next one. If you are ok with that I can mere this PR and do a catalog sync later.

@Allda no problem, all good with me as long as it gets deleted at some point 😃 thanks


Add API breaks tests for QueryClient.

The tests do not verify anything of the functional part, except whether our code
is complied with, using all possible variants of the use of the public APIs.
Add integration tests for testing API breaks for QueryClient

Shoud be initialized withouth arguments
Test fetchQuadTreeIndex method with required params
Test fetchQuadTreeIndex method with required and optional params
Test getPartitionsById method with required params
Test getPartitionsById method with required and optional params

Relates-To: OLPEDGE-1763
Signed-off-by: Drapak Iryna Angelica<EMAIL_ADDRESS>

Codecov Report

Merging #279 into master will not change coverage by %.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #279   +/-   ##
======================================
  Coverage    90.3%   90.3%
======================================
  Files          57      57
  Lines        1622    1622
  Branches      194     194
======================================
  Hits         1464    1464
  Misses         89      89
  Partials       69      69

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 537f037...b5a0bbc. Read the comment docs.


UI Page [Auth Page]

[+] feat(auth.tsx): add support for resetting access token in resetAccessCode function
[+] fix(auth.tsx): fix formatting issue in resetAccessCode function
[+] feat(locales): add support for sub tips in Auth component for multiple languages :

Add sub tips in Arabic locale (ar.ts)
Add sub tips in Bengali locale (bn.ts)
Add sub tips in Chinese locale (cn.ts)
Add sub tips in English locale (en.ts)
Add sub tips in Indonesian locale (id.ts)

Preview :

@Yidadaa Regarding about Indonesian locale for isApp its supported


GC+: Exclude special paths from garbage collection

Exclude:
Data Listing:

_lakefs prefix

Committed DF

External addresses

dummy object


Request an example

Hi! Are there any examples available? Thoughts on adding one or more to an examples folder? I'm attempting to learn how to use this lib, but am not sure where to start. Thanks!
Of note, from the docs page, it seems like most of the structs dummies for use in typestates. I gather that the following structs and their methods make up the primary API: FdCan, Tx, Rx, FdCanControl. Is this right?

There are some examples available in downstream crates: stm32g4xx-hal stm32h7xx-hal. It would be a nice improvement to have examples in this crate also.
Yes indeed, the methods on FdCan are the "core" of the API.

Thanks! This looks like an outstanding lib. Checking those examples out.


Fix backend build script

Signed-off-by: Mohammad Qureshi<EMAIL_ADDRESS>Description
The working directory will be set to notifications in the build manifest so the build.sh script does not need to change directories before assembling.
Also, after testing the build locally it seems that the find command that was present in the script currently was not finding the zip file, updating it to a command that worked.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Close this as we will separate to two build in two subfolder instead of building all in one take.
Thanks.


Update lights docs

There may be small changes regarding docs as there may be some changes in final PR for lights. They will be implemented alongside changes therefore this PR is ready for review.


Add django 1.7 support

File "/local/lib/python2.7/site-packages/treeadmin/admin.py", line 6, in
from django.utils import simplejson
ImportError: cannot import name simplejson

Getting same issue

note: this is fixed in my branch: ls-django-treeadmin.


hover effect on become contributor button

<motion.button whileHover={{ scale: 1.1 }}
className="bg-black text-white px-6 py-3 rounded-full text-lg font-semibold shadow-lg">

Become a Contributor

</motion.button>
        I think adding this code in the HomePage.jsx file solves the issue, the same hover issue is there with the Submit a template button, do you want to add the same classes to it ?

<motion.button whileHover={{ scale: 1.1 }}
className="bg-black text-white px-6 py-3 rounded-full text-lg font-semibold shadow-lg">

Become a Contributor

</motion.button>
I think adding this code in the HomePage.jsx file solves the issue, the same hover issue is there with the Submit a template button, do you want to add the same classes to it ?

You can create a pr.


Adding serde support(?)

Hi! Thank you for the crate!
I was wondering if you could add serde support, maybe through a feature flag.
I am using your crate and I am missing serde support for FlatEx.
I noticed that you use smallvec. It has serde support through a feature flag, so it should be easy.
Thanks beforehand!

Hi, interesting idea. I have not yet used serde before. Just for my understanding, what is the benefit of serializing an instance of FlatEx over serializing the expression-string and parsing it after deserialization again? Just that you do not need to parse again or is there more?

what is the benefit of serializing an instance of FlatEx over serializing the expression-string and parsing it after deserialization again? Just that you do not need to parse again or is there more?

As far as I understand, there is nothing more.
In my case, as a workaround, I did exactly as you said: Together with the FlatEx, I saved the original string and implemented serialization saving that string. Upon deserialization, I would parse again.
It is not my case, but I would assume that, in distributed computing, when you pass serialized information around, re-parsing each time becomes a waste of time. This brings me to the trade-off you might want to think about.
The implementation of serialization of FlatEx might have the following two extremes of time vs space efficiency (I do not know the details, so you might know better if it applies):

Serializing the original string and parsing on deserialization is space-efficient(?), but not time efficient.
Serializing the struct directly might use a bit more space (?), but it is time-efficient.

Thanks for your feedback. I will try it out.

I experimented a little with serde and stumbled across the problem that function pointers cannot be serialized.
the trait bound `fn(T) -> T: Deserialize<'_>` is not satisfied
But the operators such in FlatEx are all function pointers. One probably needs to serialize the function pointers. There is a crate called serde_closure that might be helpful. I need to think about this. If you have a good idea how to implement this let me know or even feel free to create a pull request.

Implementing custom serialization/deserialization for FlatEx without serializing closures can be done easily as long as no custom operators are used but only the default operators.

Thanks for looking into it!!
I just now noticed that you are using function pointers fn(T) -> T for operators. This brings up another question totally different:
Do you want to support closures that move values in the operators?
If yes, then one can not use function pointers anymore, I found this discussion particularly helpful.
To be more concrete, do you want to support the following? (based on exmex v0.8.2)
use exmex::{parse, BinOp, Operator};

let v = 2;

let ops = [Operator {
    repr: "*v",
    bin_op: Some(BinOp {
        apply: move |a: i32, b: i32| a * b * v,
        prio: 1,
    }),
    unary_op: None,
}];
let to_be_parsed = "1 *v 1";
let expr = parse::<i32>(to_be_parsed, &ops)?;
assert_eq!(expr.eval(&[1])?, 2);

In any way, when people want to store functions or closures, what I see is something like Vc<Rc<dyn Fn(T) -> T>>. This would accept closures that move values. But this is not a solution (I think), because serializing a reference Rc does not save the function(?). That should be one of the reasons serde_closure exists.
Changing all function pointers for serde_closure::structs::Fn feels a bit excessive (and I do not the performance drawbacks it would have).
By the way, can one reconstruct the original expression from FlatEx? If so, serializing the original expression and parsing upon deserialization does not feel like a bad idea anymore (and you could add this unparse method on the way :) )

Thanks for your feedback. Interesting question. I like the simplicity of function pointers. If someone can make a strong case for closures that capture stuff from their surroundings, I will re-consider them.
Regarding serialization, I currently think in the direction of unparsing and re-parsing. FlatEx currently does not keep track of the used operator and variable representations in the string that was parsed. It only keeps function pointers and variable indices. I could use arbitrary representations. On the other hand, knowing the representation would be helpful for debugging.

If someone can make a strong case for closures that capture stuff from their surroundings, I will re-consider them.

Sure, sounds fair :)

I currently think in the direction of unparsing and re-parsing.

Just to be sure, how is the re-parsing done? I am thinking of the operators needed to parse the expression, as they are not in the expression, how is one going to parse it?
If it is not possible, one might go for the following:

Implement Serialization for default operators.
Through trait bounds, implement serialization for FlatEx
Then, for user-supplied operators, they will have to implement serialization to serialize FlatEx... which I think is fair

Yes. For re-parsing, one needs the operators.

One could simply store a reference to the input string and reparse it on deserialization with default operators and default literals.
Not sure, how much value this would add.
One could also serialize default operators by their string-representations. On the other hand, parsing a not too long string just takes something like 10 μs. Not sure how beneficial this could be.

All operators, custom or not, have the same type. Not sure how one could implement different serializations.

Okay! I think we came to the conclusion that, to add serde support, one would need to change all function pointers for serde_closure::structs::Fn.
With this, one could serialize the parsed expression together with operators (i.e. FlatEx), and simply restore up deserialization (no need to re-parse).

Do you have a use-case were re-parsing is too slow? Or can you think of one?

I do not think it is a "common" case, but this is the closest to something useful I can think of. In short, "distributed recursive parsing", or "distributed tree construction".
Say you have a really big expression to parse. Then, you would like to do the following:

Find quickly a binary operator with high priority (while not parsing the whole input)
Send each sub-inputs to two workers to parse (you send only the substring(+operators))
Repeat until workers receive an expression without binary operators
The last workers parse the expression and send it back to their parent (here, FlatEx would be serialized)
The parent puts together the expressions from their childs (two deserializations and one serialization)
The root gives back the whole expression parsed

Interesting use case.
For the moment I have implemented just unparse and used it to implement Display, since I need this for debugging while developing new features anyway.

Nice! Thank you!


Bugfix/apache licence

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
if you change the schema you must patch the test databases as well, see Updating the schema
the PR must not fail unit testing

Description
Using one or more sentences, describe in detail the proposed changes.
Apache licence header updated to ensure consistency
Use case
Describe the problem. Please provide an example representing the motivation behind the need for having these changes in place.
All files should have the same licence header, one file was missing it. Also updated the notice for better Ensembl casing and added spaces around the apache url for increased clarity
Benefits
If applicable, describe the advantages the changes will have.
all files have a licence and it is the same across all files
Possible Drawbacks
If applicable, describe any possible undesirable consequence of the changes.
NA
Testing
Have you added/modified unit tests to test the changes?
NA
If so, do the tests pass/fail?
NA
Have you run the entire test suite and no regression was detected?
NA

Pull Request Test Coverage Report for Build 55

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 26.841%

Totals

Change from base Build 48:
0.0%

Covered Lines:
288

Relevant Lines:
1073

💛  - Coveralls


zsh completions

I just installed the zsh completions for mlem and I get a very long error message:
File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/globals.py", line 37, in get_current_context
    return t.cast("Context", _local.stack[-1])
IndexError: list index out of range

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/xxxxxxxxx/.venv/bin/mlem", line 8, in <module>
    sys.exit(app())
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/main.py", line 328, in __call__
    raise e
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 778, in main
    return _main(
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 206, in _main
    self._main_shell_completion(extra, prog_name, complete_var)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 765, in _main_shell_completion
    _typer_main_shell_completion(
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 651, in _typer_main_shell_completion
    rv = shell_complete(self, ctx_args, prog_name, complete_var, instruction)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/completion.py", line 156, in shell_complete
    click.echo(comp.complete())
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/_completion_click8.py", line 101, in complete
    completions = self.get_completions(args, incomplete)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/shell_completion.py", line 273, in get_completions
    return obj.shell_complete(ctx, incomplete)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1742, in shell_complete
    results = [
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1743, in <listcomp>
    CompletionItem(name, help=command.get_short_help_str())
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1307, in get_short_help_str
    elif self.help:
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/mlem/cli/main.py", line 190, in help
    cmd_name = self._get_cmd_name_for_docs_link()
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/mlem/cli/main.py", line 85, in _get_cmd_name_for_docs_link
    ctx = click.get_current_context()
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/globals.py", line 40, in get_current_context
    raise RuntimeError("There is no active click context.") from e
RuntimeError: There is no active click context.
ppTraceback (most recent call last):
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/globals.py", line 37, in get_current_context
    return t.cast("Context", _local.stack[-1])
IndexError: list index out of range

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/xxxxxxxxx/.venv/bin/mlem", line 8, in <module>
    sys.exit(app())
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/main.py", line 328, in __call__
    raise e
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 778, in main
    return _main(
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 206, in _main
    self._main_shell_completion(extra, prog_name, complete_var)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 765, in _main_shell_completion
    _typer_main_shell_completion(
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 651, in _typer_main_shell_completion
    rv = shell_complete(self, ctx_args, prog_name, complete_var, instruction)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/completion.py", line 156, in shell_complete
    click.echo(comp.complete())
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/_completion_click8.py", line 101, in complete
    completions = self.get_completions(args, incomplete)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/shell_completion.py", line 273, in get_completions
    return obj.shell_complete(ctx, incomplete)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1742, in shell_complete
    results = [
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1743, in <listcomp>
    CompletionItem(name, help=command.get_short_help_str())
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1307, in get_short_help_str
    elif self.help:
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/mlem/cli/main.py", line 190, in help
    cmd_name = self._get_cmd_name_for_docs_link()
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/mlem/cli/main.py", line 85, in _get_cmd_name_for_docs_link
    ctx = click.get_current_context()
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/globals.py", line 40, in get_current_context
    raise RuntimeError("There is no active click context.") from e
RuntimeError: There is no active click context.
Traceback (most recent call last):
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/globals.py", line 37, in get_current_context
    return t.cast("Context", _local.stack[-1])
IndexError: list index out of range

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/xxxxxxxxx/.venv/bin/mlem", line 8, in <module>
    sys.exit(app())
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/main.py", line 328, in __call__
    raise e
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 778, in main
    return _main(
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 206, in _main
    self._main_shell_completion(extra, prog_name, complete_var)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 765, in _main_shell_completion
    _typer_main_shell_completion(
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/core.py", line 651, in _typer_main_shell_completion
    rv = shell_complete(self, ctx_args, prog_name, complete_var, instruction)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/completion.py", line 156, in shell_complete
    click.echo(comp.complete())
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/typer/_completion_click8.py", line 101, in complete
    completions = self.get_completions(args, incomplete)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/shell_completion.py", line 273, in get_completions
    return obj.shell_complete(ctx, incomplete)
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1742, in shell_complete
    results = [
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1743, in <listcomp>
    CompletionItem(name, help=command.get_short_help_str())
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/core.py", line 1307, in get_short_help_str
    elif self.help:
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/mlem/cli/main.py", line 190, in help
    cmd_name = self._get_cmd_name_for_docs_link()
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/mlem/cli/main.py", line 85, in _get_cmd_name_for_docs_link
    ctx = click.get_current_context()
  File "/home/xxxxxxxxx/.venv/lib/python3.8/site-packages/click/globals.py", line 40, in get_current_context
    raise RuntimeError("There is no active click context.") from e
RuntimeError: There is no active click context.

However, I get no completion. I'm no expert on zsh-extensions and will just de-activate it for now, but I feel that it's still something that might need attention at some point.

Can you share how do you install it (ei exact commands)?
To be honest, we never tested auto-completion provided by typer :(
I just tried it myself and I dont get this error - it says it successfully installed (tried both zsh and bash)
However it just does not work 🤣 But maybe there is something wrong with my env

OK I got it to crash like you described inside docker env - so there is def something wrong with mine. Will take a look

@aguschin can you TAL? I think the error is due to trying to get click context for getting command name for docs link - it seems when run in completion mode there is no active click context

@mike0sv , yes it initially didn't work at all, just said that it created a ~./zfunc/ folder and put it in there. I thought that this folder might be used somewhere in the oh-my-zsh config, which I don't use. So I just plainly sourced it at the end of my ~/.zshrc. So, similar observations as you had.


Building image from Dockerfile behind proxy: fail on gpg --recv-keys

Hi
When trying to build the postgres image from Dockerfile, running behind http proxy and despite proper docker configuration, i will fail on receiving gpg keys with the following output:
+ gpg --batch --keyserver ha.pool.sks-keyservers.net --recv-keys B97B0AFCAA1A47F044F244A07FCC7D46ACCC4CF8
gpg: keybox '/tmp/tmp.JJpMHuFUjB/pubring.kbx' created
gpg: keyserver receive failed: Connection timed out

I am able to handle this by adding --keyserver-options http-proxy=$http_proxy on each required line from de Dockerfile (i can even manage its automation for mantaining a local modifyed Dockerfile). I am just wondering: if this change or some variant can be harmlessly added to the official Dockerfiles, it will reduce the ammount of "docker behind proxy" googling when triying to base an image on a slightly modified version of the official image (like those Volume-less versions discussed on other issues).
There are some related issues like #288 #262 etc. and #443, collapsing on somthing like: "you should pull the built image". I think that there are valid use cases for building slightly modified images, and reducing needed modifications leads to less bugs.
Javier Clavijo.

It doesn't discuss this exact issue (of downloading via proxy), but it's very similar, so I'd recommend you check out https://github.com/docker-library/faq#openpgp--gnupg-keys-and-verification (one of the solutions there could very likely work for your use case too).

It's in fact very similar. i'll try it --next week-- and post some notes for future reference before closing the issue. Just for clarification: https://github.com/tianon/rawdns should do the trick, hijacking DNS and redirecting the GET request trough $http_proxy, no pgp-happy-eyeballs needed. Am i right?
I thought that running the build itself from inside a container with the docker socket bind-mounted, and redirecting the full http traffic for that container should be a feasible (perhaps not better nor easyer) solution.

No, rawdns itself will not solve the problem because you actually need something to proxy the traffic through your http_proxy (rawdns will just allow you to munge the DNS replies however you please, which would allow you to transparently redirect the keyserver requests to something that does respect your http_proxy environment variable, thus making your build process independent of your http proxy).

Closing since this seems resolved


the ranking of completion result

Hello, very nice work.
I have a question about the completion ranking when reading TabNine website. Following figs show that the result with the highest probability is not always presented at the top 1 position.

is there a strategy behind the tabnine to decide the trade-off between probability and completion length?
thanks a lot.

Was going to report this as well:

Would be nice to instead of having it alphabetically, having it by probability.


Remove setting the WebClient.Proxy to null

It should be fine in most of the cases to just use the system proxy settings.
If anyone has issues with this approach, we can still make the WebClient.Proxy setting configurable somehow.
This is the pull request for the issue #57.

Sorry for the late merge & inconvenience. Thanks :+1:


MWPW-135675 Fix marquee video

Update marquee videos to work with new inline fragments ( #1242 )
Resolves: MWPW-135675
Homepage test url:

Before: https://homepage-brick--homepage--adobecom.hlx.live/homepage/index-loggedout
After: https://homepage-brick--homepage--adobecom.hlx.live/homepage/index-loggedout?milolibs=marqinlinefrag

Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/tat28144/regional-edits/marqueevideo?martech=off
After: https://marqinlinefrag--milo--adobecom.hlx.page/drafts/tat28144/regional-edits/marqueevideo?martech=off

https://github.com/adobecom/milo/pull/1258 touching some of the same code as @seanchoi-dev's PR too. May want to meet in the middle?

#1258 touching some of the same code as @seanchoi-dev's PR too. May want to meet in the middle?

@seanchoi-dev 's update is an improvement to lighthouse scores, but not a blocker for homepage migration like this PR.  I'd like to merge this PR and then have Sean adjust his code as needed.  But we need this one merged today.


v1.9.1: cannot create container due to missing cgroup.procs file

We have this internal tool for deploying our containers, the main script looks like this:
https://gist.github.com/yoanisgil/6277a17e1766b533b70d
We have been using it for some months now with Docker 1.8.x. Recently we decided to upgrade to Docker 1.9. However I hit this bug from time to time and on what it seems to be a very random manner:
2015-11-20 10:08:07,180 - roll.single.deploy - INFO - Starting container 2b0e5d7e34402e145d73385b2711e11efa3ef945c561d90ff732f0bec27fea04
None None
Traceback (most recent call last):
  File "./single-roll", line 44, in <module>
    deploy.run()
  File "/home/ygil/dev/docker-roll/roll/single/deploy.py", line 111, in run
    self.client.start(new_container.get('Id'))
  File "/home/ygil/.envs/docker-roll/lib/python2.7/site-packages/docker/utils/decorators.py", line 21, in wrapped
    return f(self, resource_id, *args, **kwargs)
  File "/home/ygil/.envs/docker-roll/lib/python2.7/site-packages/docker/api/container.py", line 344, in start
    self._raise_for_status(res)
  File "/home/ygil/.envs/docker-roll/lib/python2.7/site-packages/docker/client.py", line 142, in _raise_for_status
    raise errors.APIError(e, response, explanation=explanation)
docker.errors.APIError: 500 Server Error: Internal Server Error ("Cannot start container 2b0e5d7e34402e145d73385b2711e11efa3ef945c561d90ff732f0bec27fea04: [8] System error: open /sys/fs/cgroup/devices/system.slice/docker-2b0e5d7e34402e145d73385b2711e11efa3ef945c561d90ff732f0bec27fea04.scope/cgroup.procs: no such file or directory")

We run the script like this:
SLEEP=0 DOCKER_HOST='tcp://aws038:2375' ./single-roll docker-roll-config.yml crakmedia/crakrevenue-affiliate:5.10.1

Here is some context information (providing this since our deployment script uses docker-py):
Pip
pip list
coverage (3.7.1)
docker-py (1.5.0)
docker-roll (0.0.5)
nose (1.3.7)
pip (7.1.2)
pyaml (15.6.3)
python-consul (0.4.2)
PyYAML (3.11)
requests (2.8.1)
setuptools (18.0.1)
six (1.10.0)
websocket-client (0.32.0)
wheel (0.24.0)

$ docker version
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 13:25:01 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 13:25:01 UTC 2015
 OS/Arch:      linux/amd64

OS Information
OS: CentOS 7
Kernel version: uname -a
Linux aws074.crakmedia.lan 3.10.0-229.4.2.el7.x86_64 #1 SMP Wed May 13 10:06:09 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

Docker was installed from CentOS's official repo:
$yum info docker-engine
Installed Packages
Name        : docker-engine
Arch        : x86_64
Version     : 1.9.1
Release     : 1.el7.centos
Size        : 36 M
Repo        : installed
From repo   : docker-main-repo
Summary     : The open-source application container engine
URL         : https://dockerproject.org
License     : ASL 2.0
Description : Docker is an open source project to build, ship and run any application as a
            : lightweight container.
            :
            : Docker containers are both hardware-agnostic and platform-agnostic. This means
            : they can run anywhere, from your laptop to the largest EC2 compute instance and
            : everything in between - and they don't require you to use a particular
            : language, framework or packaging system. That makes them great building blocks
            : for deploying and scaling web apps, databases, and backend services without
            : depending on a particular stack or provider.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

I must say I cannot consistently reproduce this. In the beginning I thought it was related to either Swarm or the fact that it takes two API calls to run a container using docker-py (see here https://github.com/docker/docker-py/issues/857). However after further testing, I hit the same bug when not using Swarm.

This is a dup of #17653
Please see this comment for a workaround, this is a systemd issue.
https://github.com/docker/docker/issues/17653#issuecomment-155609224

@crosbymichael thanks for the quick reply. Will look into mentioned ticket.


[SPARK-26696][SQL] Makes Dataset encoder public

What changes were proposed in this pull request?
Implements the solution proposed in SPARK-26696, a minor refactoring that allows frameworks to perform advanced type-preserving dataset transformations without carrying Encoder implicits from user code.
How was this patch tested?
New test added to DatasetSuite.
@gatorsmile per your request

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/7798/
Test PASSed.

Test build #102172 has started for PR 23620 at commit d2d5ed2.

Code change & test removed per discussion.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/7799/
Test PASSed.

Test build #102173 has started for PR 23620 at commit 1ce8809.

@ssimeonov, can you update PR description? How did you test?

@HyukjinKwon PR description updated. Per the lengthy discussion, no test is needed (initial test was removed).

No test is okay. I am asking how you tested this PR simply. Manually tested?

Also, please clarify what case this PR supports in PR description.  I think that's what @gatorsmile initially meant at https://github.com/apache/spark/pull/23620#discussion_r252752815.
Is it something to support
def foo[A](ds: Dataset[A]): Dataset[A] =
  ds.toDF().as[A](ds.encoder)

foo(spark.range(1)).show()

instead of
def foo[A: Encoder](ds: Dataset[A]): Dataset[A] =
  ds.toDF().as[A](implicitly[Encoder[A]])

import spark.implicits._
foo(spark.range(1)).show()

? please be diligent about elaborating it in the PR description. I don't particularly support this usecase since it's only few lines diff and I was wondering what downside it brings by carrying Encoder implicits.

@HyukjinKwon the patch was tested with an automated test that was removed per request. The PR description was explicit about this.
Re: motivating the change, I think this is the purpose of the linked JIRA issue as opposed to the PR description but I have nevertheless added the note you suggest to the description.

I cannot see how it was tested before, this is rebased. Did the test work without this patch before? That's what I can guess from the comments.
Can you clarify this in PR description to deduplicate efforts like I made just now?

I think this is the purpose of the linked JIRA issue as opposed to the PR description

Please be explicit about JIRA and PR. Otherwise people read it and make the reproducer by themselves to test and verify it. For instance, https://github.com/apache/spark/pull/23620#discussion_r255358804 would have not been asked if you clarified this.

Test build #102173 has finished for PR 23620 at commit 1ce8809.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test build #102172 has finished for PR 23620 at commit d2d5ed2.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102173/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102172/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/7807/
Test PASSed.

Test build #102189 has finished for PR 23620 at commit 1ce8809.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102189/
Test PASSed.

No @HyukjinKwon, the test did not work before as it accessed a private property that this PR made public.
I am at a loss what about what more needs to happen for this one-line change...

@ssimeonov . It's partially true.

the test did not work before as it accessed a private property that this PR made public.

@HyukjinKwon . The following was @ssimeonov 's test case.
test("as tuple using instance encoder") {
    val data = Seq(("a", 1), ("b", 2), ("c", 3))
    val ds = data.toDS()
    checkDataset(
      ds.toDF().as[(String, Int)](ds.encoder),
      data: _*)
  }

The problem of @ssimeonov 's test case was that the following works . So, @gatorsmile and I had a doubt about that.
test("as tuple using instance encoder") {
    val data = Seq(("a", 1), ("b", 2), ("c", 3))
    val ds = data.toDS()
    checkDataset(
      ds.toDF().as[(String, Int)],
      data: _*)
  }

What @ssimeonov is saying is that ds.encoder will not be parsed.
In addition, @ssimeonov also suggested another example in the comment. But, we prefer to keep this PR simple.

Thanks. So, am I being correct the only usecase by this fix is:
def foo[A](ds: Dataset[A]): Dataset[A] =
  ds.toDF().as[A](ds.encoder)

vs
def foo[A: Encoder](ds: Dataset[A]): Dataset[A] =
  ds.toDF().as[A](implicitly[Encoder[A]])

? If so, I don't feel strongly about this PR. Let me leave it to you guys then.

I am at a loss what about what more needs to happen for this one-line change...

It's one-liner change but it is important since it exposes something as an API at Dataset. Here is quite a core API entry point. I myself already have few one-liner PRs that had to provide a lot of details.
Adding or exposing new APIs in those is being discouraged IMHO in general. Let's be sure and clear about what API you're adding and why it's needed in particular for some core APIs. I know I am being a bit hysterical about it but my impression was that the purpose of this PR wasn't clear.

Anyway, I am okay as is. I personally don't see why the usecase is important but the change is small, and we properly marked it as unstable.

thanks, merging to master!


Vlang is not working properly

Hi @kerolloz,
Thank you.
I am aware of that, I am trying to fix that for a few months now. This is an API problem that I need to look into more. :)

Since I have problems with running V inside a isolate on a Judge0 API side, I have decided to discontinue V for now.


python3Packages.zigpy: 0.36.1 -> 0.36.2

Motivation for this change
Update to latest upstream release 0.36.2
Change log: https://github.com/zigpy/zigpy/releases/tag/0.36.2
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

5 packages failed to build:

home-assistant
python38Packages.zigpy-xbee
python38Packages.zigpy-znp
python39Packages.zigpy-xbee
python39Packages.zigpy-znp

12 packages built:

python38Packages.bellows
python38Packages.zha-quirks
python38Packages.zigpy
python38Packages.zigpy-cc
python38Packages.zigpy-deconz
python38Packages.zigpy-zigate
python39Packages.bellows
python39Packages.zha-quirks
python39Packages.zigpy
python39Packages.zigpy-cc
python39Packages.zigpy-deconz
python39Packages.zigpy-zigate

Part of #135832


delayConnection causes hang when used with request.abort()

When using delay connection, request.abort() does not terminate the request correctly, so the test hangs until heap is full. Using request.end() instead of abort fixes the issue. Sample case:
		it('should return an error if the downstream doesn\'t return within the timeout', (done) => {
			let httpRequest = new HttpRequest(10, 1);
			nock('http://localhost:1234')
				.get('/test/abc')
				.socketDelay(100)
				.reply(200, successRsp);

			let request = httpRequest.httpGetRequest('http', 'localhost', '1234', '/test/abc');

			request
				.then(rsp => {
					throw new Error('httpGetRequest returned success unexpectedly.', rsp);
				})
				.catch(err => {
					expect(err.message).to.equal('request timed out http://localhost:1234/test/abc');
					done();
				});
		});

In code being tested:
request.setTimeout(timeout, () => {
				request.abort();
				reject(new error('request timed out ', url));
			});

Suspect this is related to #882


fix: CI tests

Description
Getting the test workflow to run successfully.
Motivation and Context
Reviewing the test workflow we can see that the validation is always executed in the root directory. Therefore we always get the message Terraform initialized in an empty directory!, link of a case.
For this I used the option to modify the working directory of the workflow steps.
Furthermore, to be able to carry out the tests, I had to add the ordering logic by size again. I'm sorry, but I'm not clear on the reason for the exclusions of the regex filter. This github action is executed at the root path of the checkout, therefore there will be no "versions.tf" file, but yes the "tests/versions.tf" or similar, always with the subpath included in the value.
For example in ./tests the github action gets this array ["tests/versions.tf","tests/terraform-aws-alb/wrappers/versions.tf","tests/terraform-aws-alb/versions.tf","tests/terraform-aws-alb/examples/complete-nlb/versions.tf","tests/terraform-aws-alb/examples/complete-alb/versions.tf","tests/0.13/versions.tf","tests/0.12/versions.tf"] in the versionConstraintSearch function.
How Has This Been Tested?
I have been testing this fork to see the behavior
Screenshots (if appropriate):

I think the error about the path ./tests/terraform-aws-alb/wrappers testing is correct, the main.tf has a reference to the path ../ in the source property. This causes the terraform version 0.13.1 to be established in wrappers, but in turn the source references have 1.0.0, causing the conflict. This case will also occur in this PR.

On the other hand this error is also an correct case, aws provider version 4.27 requires at least terraform version 0.13 or later.

I think the error about the path ./tests/terraform-aws-alb/wrappers testing is correct, the main.tf has a reference to the path ../ in the source property. This causes pthe terraform version 0.13.1 to be established in wrappers, but in turn the source references have 1.0.0, causing the conflict. This case will also occur in https://github.com/terraform-aws-modules/terraform-aws-alb/pull/264.

Correct - this is the source of the issue we are trying to mitigate. I don't want to evaluate the wrappers directory at all when validating with min version check. The only thing that will end up evaluating that directory will be the "max" workflow because the wrapper pre-commit hook will always be set to this value currently
so to share a bit more about our workflows to get a better view:

We want to find all of the "root" directories where a set of Terraform configuration files live. This could be a module root directory, an example root directory, or a project root directory. We find ALL of these root directories by searching through *.tf files to find the required_version (since you can only have one of these, but you may have none in which our pre-commit hook will end up complaining if you do not have it - so all roads lead back to this as signfiying a root directory). We want to find these various root directories to a) be able to parallelize checks and b) we want to evaluate them on their own (note - this does not do full validation of all dependency version requirements currently)
Once we have our directories, we then have two separate workflows. One set where we look at each directory and determine the minimum supported terraform version, and one set where we use the maximum supported version and evaluate the entire project](https://github.com/terraform-aws-modules/terraform-aws-alb/blob/858ae988b3efe72f7d69335ca065c09cbb4c76d7/.github/workflows/pre-commit.yml#L59).

It is in this 2nd step where we evaluate the minimum version where we want to ensure we are not evaluating anything under the wrappers sub-directory. You can also see the relationship between this action and https://github.com/clowdhaus/terraform-composite-actions/tree/main/directories - we have to ensure the root directory search logic is the same or else we will have a mis-match in evaluation
So now I think we are close, I believe both of the actions search logic is nearly identical, now its just getting this action to return the versions as intended

:tada: This PR is included in version 1.2.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


xmlsec1: new package at 1.2.29

Description of package
XML security library. Dependency for oath-toolkit
Checklist

[x] Latest upstream version.
[x] I agree to maintain this package (required).
[x] I agree to notify KISS if I can no longer maintain this package (required).

:+1:


Union type of records deserialize incorrectly

Discussed in https://github.com/MarshalX/atproto/discussions/119

Originally posted by Scrxtchy July 27, 2023
I've had this problem for a while and had it swept under the rug for a while. I still cannot think of a solution to this.
When retrieving a post though feed.getposts() if the post has an image, but the post has an embed of another record, the images in the maid record as returned with the type of app.bsky.embed.images#image, which is in contrast to when the Post does not have a record embed, it includes the app.bsky.embed.images#view in the embeds, giving back the impageproxy urls for the media.
I am confused as to why this has happened, as when looking at the network requests on the website, the app.bsky.embed.recordWithMedia#view record type contains both a app.bsky.embed.record#viewRecord to contain the record of the quoted post, as well as the app.bsky.embed.images#view images containing the urls that I have been seeking out

fixed in 5ae9acfe702fc021d7cbe40770b4cc4b0237361e


Exporting & Loading Trained Model?

Is there a method to save a trained (GaussianNb)model and the load it?
I'm just learning how to use Rust, I just managed to implement a Gaussian Naive Bayes classifier model, is there any way to use the "predict" method without having to train the whole model again?
I know that in libraries like Sklearn you can export them models and then load them in .pkl formats, is there a similar implementation in linfa?
Thank you so much!

The models should implement the Serde trait, so you can serialize them using something like ciborium

I'm having a little problem hehe, I'm trying to serialize the model (I'm using the example of linfa_svm) but I don't know if I'm using the correct syntax since I get the error in the line where I use cbor:
the trait bound `MultiClassModel<ndarray::ArrayBase<ndarray::data_repr::OwnedRepr<f64>, ndarray::dimension::dim::Dim<[usize; 2]>>, usize>: serde::ser::Serialize` is not satisfied
the following other types implement trait `serde::ser::Serialize`:

This is the code I'm using  (linfa_svm/examples/winequality_multi.rs) :
let model = train
        .one_vs_all()?
        .into_iter()
        .map(|(l, x)| (l, params.fit(&x).unwrap()))
        .collect::<MultiClassModel<_, _>>();
 let pred = model.predict(&valid);

//Trying to serialize model
 let save_model = cbor!(model).unwrap();

Could you give me a more detailed example? I would appreciate it too much!

Seems like MultiClassModel and linfa-bayes have no support for Serde. Weird. We'll need to add that.

Oh! I will try with normal SVM then, thank you!

One last question,
I already managed to serialize the SVM model without Multiclass to CBOR, using ciborium, I also managed to de-serialize it in another file and convert it to Value, the last step would be to convert it from Value to SVM, any idea how to do this?
Code for creating and exporting the model :
    let model = Svm::<_, bool>::params().pos_neg_weights(50000., 5000.).gaussian_kernel(80.0).fit(&train)?;

    //Serializing the trained model with ciborium
    let value_model : Value = cbor!(model).unwrap();
    let mut vec_model : Vec<u8> = Vec::new();
    let _cebor_writer = ciborium::ser::into_writer(&value_model, &mut vec_model);

    //Esporting it to a .cbor file
    let path: &Path = Path::new("./model.cbor");
    fs::write(path, vec_model).unwrap();

Attempt to use the trained model in other .rs program :
    let mut file = File::open("./model.cbor").unwrap();
    let mut data: Vec<u8> = Vec::new();
    file.read_to_end(&mut data).unwrap();

    let model_value : Value = ciborium::de::from_reader::<Value, _>(&data[..]).unwrap();
    let model: Svm<_, bool> = model_value.deserialized().unwrap(); // Error

But I keep getting error when trying to converting form ciborium::Value to SVM, the rust-analyzer suggests : consider specifying the generic argument: ::<Svm<_, bool>>, I guess I have to pass the SVM serve-deserializer but I don't know how to do that.
I know this has nothing to do with linfa, but I really think that exporting and importing the models can be very useful.
Thank you!

My bad, it turns out the example SVM model uses Svm<f64, bool> not Svm<_, bool>. I only changed the line to :
let model: Svm<f65, bool> = model_value.deserialized().unwrap();
And it works super cool!

One last question, I already managed to serialize the SVM model without Multiclass to CBOR, using ciborium, I also managed to de-serialize it in another file and convert it to Value, the last step would be to convert it from Value to SVM, any idea how to do this?
Code for creating and exporting the model :
    let model = Svm::<_, bool>::params().pos_neg_weights(50000., 5000.).gaussian_kernel(80.0).fit(&train)?;

    //Serializing the trained model with ciborium
    let value_model : Value = cbor!(model).unwrap();
    let mut vec_model : Vec<u8> = Vec::new();
    let _cebor_writer = ciborium::ser::into_writer(&value_model, &mut vec_model);

    //Esporting it to a .cbor file
    let path: &Path = Path::new("./model.cbor");
    fs::write(path, vec_model).unwrap();

Attempt to use the trained model in other .rs program :
    let mut file = File::open("./model.cbor").unwrap();
    let mut data: Vec<u8> = Vec::new();
    file.read_to_end(&mut data).unwrap();

    let model_value : Value = ciborium::de::from_reader::<Value, _>(&data[..]).unwrap();
    let model: Svm<_, bool> = model_value.deserialized().unwrap(); // Error

But I keep getting error when trying to converting form ciborium::Value to SVM, the rust-analyzer suggests : consider specifying the generic argument: ::<Svm<_, bool>>, I guess I have to pass the SVM serve-deserializer but I don't know how to do that.
I know this has nothing to do with linfa, but I really think that exporting and importing the models can be very useful.
Thank you!

I use cbor!(model), it show errors,how to resolve?
the trait bound MultiClassModel<ndarray::ArrayBase<OwnedRepr<f64>, Dim<[usize; 2]>>, Option<&str>>: serde::ser::Serialize is not satisfied
the following other types implement trait serde::ser::Serialize:
bool
char
isize
i8
i16
i32
i64
i128
and 196 othersrustcClick for full compiler diagnostic
lib.rs(222, 42): Actual error occurred here
lib.rs(222, 9): required by a bound introduced by this call
ser.rs(435, 35): required by a bound in value::ser::<impl ciborium::Value>::serialized

The models should implement the Serde trait, so you can serialize them using something like ciborium

Wonder what was the motivation of not simply exposing params and intercept of linear model and letting user decide how to serialize or deserialize it? Requiring to use serde for such a simple operation seems like an unnecessary overcomplicated design for to me.


BLE notification time inconsistent

I have an ESP32S3 connecting via BLE to multiple MbientLab MetaMotionS devices. The BLE client on the ESP32 subscribes to a characteristic on each device, which is supposed to send a notification update at 100Hz. This works fine when using Bluedroid, but unfortunately Bluedroid causes other problems down the line and we want to switch to NimBLE.
However, for some reason, when using NimBLE, most of the notifications from all devices except the last one are dropped. Each time a new device is connected, the device that was previously the most recent connection suddenly develops this problem, in addition to all previous ones. Only about 10% of the notifications go through, which is far too few for our use case. We found that changing the connection interval with setConnectionParameters made more of the notifications go through, but still not most, even when trying the max value.
Since the issue doesn't occur with Bluedroid, and nothing else is changed, the problem seems like it must be with NimBLE.
Does anybody know why this problem occurs or how to fix it?
Note: we use NimBLE through the esp-nimble-cpp library, but this seems unlikely to be the issue since it just provides an interface for BLE connection and callbacks just immediately call our code.

Oddly enough, setting the connection parameters will reverse the issue: the first connected device will now be the fast one, and the others, including the last one, will be slow. This seems to be the case even if I set the connection interval to a very large value, instead of very small, and even happened sometimes when I set invalid connection parameters and got an 0x0212 HCI error. So maybe it’s not the actual parameter values that are affecting anything, but some side effect?


s3 open fails while s3.upload_fileobj works ok

Hi,
Any idea why
with open(s3_file_path, 'wb', transport_params=transport_params) as fout: returns
FileNotFoundError: [Errno 2] No such file or directory: <s3_file_path>,
while s3.upload_fileobj(bucket, s3_file_path) works with no errors?
(all directories in the path exist in the bucket)
Thanks!
(1.8.3)
Udi

Please post the full stack trace.

found the issue - it was a missing s3:// prefix.
thanks.


Is this project still being maintained？

Do you want to request a feature or report a bug?
I maintain a typescript version of draftjs，Is there anyone interested in participating
What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. You can use this jsfiddle to get started: https://jsfiddle.net/gmertk/e61z7nfa/.
What is the expected behavior?
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?

https://github.com/gland2015/gland/tree/master/packages/draft-ts

It's still very heavily used at Facebook, so definitely still being maintained. I'm not sure who is in charge of the open-source release at the moment though.

@gland2015 I definitely would be interested in being part of a community-driven fork of Draft.js that's under frequent development. But your repo is not going to get much attention unfortunately due to it being in Chinese. GitHub is still overwhelmingly English-based projects

@jkhaui  thanks，It's easy to change to English，The problem is that I may not have enough time and ability right now。。。

It's still being maintained in the sense it's what facebook uses, so fixes / features we implement keep being released (look at the commit history). There's no dedicated team working on OSS triage full-time, and Facebook web in general is hard at work on the new beta site, so that's why the repo in general is quiet.

Thanks @mrkev, @Daniel15  for your insight . I was wondering if Facebook still plans to look at mobile support?

AFAIK mobile has never been quite fully supported. See: https://github.com/facebook/draft-js#browser-support


removed version constraint of cython

Pull Request Test Coverage Report for Build 408

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 55.511%

Files with Coverage Reduction
New Missed Lines
%

goodman_pipeline/core/tests/test_check_version.py
1
80.95%

goodman_pipeline/core/check_version.py
4
75.0%

Totals

Change from base Build 407:
0.0%

Covered Lines:
2196

Relevant Lines:
3610

💛  - Coveralls

outdated


Beef can implement const Deref?

In the case of Beef::borrowed can be implemented const Deref. Will it be done?

impl const Trait is a long way from being stable, so this is probably a bit premature :)

Yes, I understand. But unstable rust is my love. I will wait

We already have const_fn feature which requires nightly, so if we can put this behind a feature flag (same or a new one) I'd be happy to accept a PR.

Check out my PR #49.
I believe it was possible to avoid code duplication.

Any sort of access to underlying data is fine. I don't mind if it's incredibly convoluted, inconvenient and I have to wrap it in
std::str::from_utf8_unchecked(std::slice::from_raw_parts(cow.get_raw_ptr(), length))

I just need a const way to access the data.
Currently can't even transmute the Cow into a pointer to access it in const context.


kong.ServiceResponse.GetRawBody() Get incomplete information in go

i use kong 2.2.1,when i  run kong.ServiceResponse.GetRawBody() in go, i just get incomplete information, for example :
i visite a url,return html content Content-Type: text/html; charset=UTF-8,  Look at the picture below

but i found kong.ServiceResponse.GetRawBody() return just the first line of words,Look at the picture below,why? :

I know why,because grep can only show one line

I know why,because grep can only show one line


How to check if an Arg has been setted

Hi there!
I'm trying to implement a CLI that responds to mypgm add --path="../HelloWorld.md".
This command can also take as parameter --group like mypgm add --group="../mygroup/".
I don't understand how works args, and I'm continuing to check how to launch my program using args in command :-/
Currently, I check if path has been setted using if path != nil {... but it seems it does not work

Ok so it seems that, if I want to check if an Arg has been setted, I have to check if *pointerToArg != "" {... (which is ugly :-().

Also, is it possible to explicity set the argument?
For example: mypgm add --groupe="HelloWorldUsers" and mypgm add --path="./" ?
Thanks!

Ok, finally I updated my code to use flags instead of args and it works well! \o/

It would be cooler if the value was nil if it wasn't set instead of a pointer to the default value imo.


{Compute} az vm create: Fix warning log for public ip

Related command

az vm create
Description

Issue - Close: #26516
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute

Since this change is customer-facing, may I ask if the title should be start with [Compute] instead of {Compute}?


File Transfer Keyword Args

[ ] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

This pull request is stale because it has been in review for at least three days. It will be closed in two more days.

This pull request is stale because it has been in review for at least three days. It will be closed in two more days.


Small fix for hyperlink

Missing (
Signed-off-by: andrew.s.moore<EMAIL_ADDRESS>

:) I guess I still don't do DCO via webpage correctly

Ah yeah, I think the DCO bot gets confused sometimes if you email address isn't public. It expects the no-reply email address from github.

#26 it is :)
I am never trying to edit with the browser view again


Implement matrix_0 and scprod_0

Implement line 14 and 15 from the pseudocode.
Blocks #22

Implemented, but do not push until tested. Calculation of W not done yet, need to figure out what issue it belongs to.


Support server side redirects in static mode

Currently Nuxt static mode redirects client side what is not recommanded in a SEO, better would be 301 redirects on a server.
Describe the solution you'd like
Many static site hostings support redirects, but they have to be configured somehow. AWS S3 can be configured by object meta data. Firebase can be configured by a json.
We need at least some sort of hook what Nuxt will call for each redirect, so custom script can generate hosting configuration.
Describe alternatives you've considered
Fetch redirects from CMS out of Nuxt, but this will not cover redirects initiated only in Nuxt.

We are approaching the Nuxt 2 EOL date (June 30, 2024) - see this article for more information. I'm closing this issue as it's marked as a Nuxt 2 related enhancement and it's not a critical issue.
That doesn't mean it might not be relevant for Nuxt 3. If it is, please feel free to open a new issue (or just comment, and I can reopen it). 🙏
Thank you for your understanding and for your contribution to Nuxt! 🎉


Feature request: Specify the default sidecar image to use with the operator

Right now when creating a humio cluster CR and do not supply a sidecar image, it uses the default hardcoded image value in the operator. Similar to the Strimzi operator, we would like to see default images being able to set in the helm chart values of the operator rather than specifying with each humio cluster

Fixed by https://github.com/humio/humio-operator/pull/849


How to process/retrieve the result of a task and how to pass a non serializable context to the task?

Hello,
Thank you for your work. :)
I am wondering the following things:

how could someone retrieve the result of a task?
how could someone run a callback or call another service with the result of a task?
how could someone use clients or database pool from within a run function?

(Feel free to tell me if you would like me to split this into multiple issues.)
E.g. I would like to store the result inside the database or transmit the result to another service (in-memory) without doing an API call to localhost if I am running the workers within an API.
And I don't want to build the client/db pool for every task from environment variables.
async fn run(&self, queue: &mut dyn AsyncQueueable) -> Result<(), FangError>

run signature is Result<(), FangError> thus I could not make my own AsyncWorker<AQueue> or AsyncWorkerPool<AQueue> that would handle the result of a task.
And since a task should be serializable, I cannot provide to it clients (database or otherwise) that I would not want serializable but that I would want to access in the run function.
I am thinking that the only way would be to attempt to access a global reference to a singleton or something alike, e.g.:
async fn run(&self, _queue: &mut dyn AsyncQueueable) -> Result<(), FangError> {
    // do stuff
    let result = ...;
    let pool: Pool<Postgres> = get_global_postgres_pool().expect("Posgres pool is not initialized");
    pool.acquire().await?.execute("<INSERT result into table>").await?;
    Ok(())
}

Though I am not sure that the compiler would allow me to do so.

Did I miss something?
Or do you currently use a workaround to achieve this behavior?
Or would that kind of feature be out of scope of fang and you don't need this?

Maybe it is part or could be part of the following discussion https://github.com/ayrat555/fang/discussions/101 ?
I understand that this could/would considerably complicate the implementation, but no harm in asking 😇.

Seems doable to have a global pool, e.g. https://stackoverflow.com/a/63153898
Still interested in your take on this ;)

Hello,
Have you figured out how to retrieve the result of a task?
Thanks

More specifically I would want the task to output a stream of data

It would be nice if we could pass a MPSC sender instance through the task's run method. It would allow data streaming between the consumer and the senders (workers).
Let me know if there's already a way to do that, I can't find a way to pass a sender instance to the tasks.

There is no current way to pass non-serializable object to a task.
One work-around you can do is to make the Task fields to be database pool, or connetion url which are both serializable and the work will store in the database the stuff you want. After that, do another task that will fetch from database every cetain amount of time.
The main problem that this
async fn run<T>(&self, queue: &mut dyn AsyncQueueable) -> Result<T, FangError>

can not be implemented yet, I think (I am not sure because Rust has been changing since i have implemented it).
The reason is that async traits do not allow generics, at least at the moment  we have done the implementation.

I have been reasearching about this and I think since Rust 1.75 this change could be possible.
As soon as Sqlite PR and Mysql closes #141 , I will open a experimental PR exploring this.

Seems doable to have a global pool, e.g. https://stackoverflow.com/a/63153898
Still interested in your take on this ;)

not a good choice in global.


Add tags for 1.0.0-beta8 version

Hi!
When I'm trying to install 1.0.0-beta8 version via npm, I actually receive 1.0.0-beta6 version which don't support react-router: 2.0.0.
@Scarysize Could you please update tags?
Thanks!

Hm, are you installing it with a specific version or just the latest? I´m getting beta8 if I do npm install redux-router

I install 1.0.0-beta8 version.
I'm using react-router: "2.0.1"
Actually in node_modules folder in redux-router package.json  I see 1.0.0-beta8 version, but if compare reduxReactRouter.js in github repo and in node_modules folder I have completely different versions
Github https://github.com/acdlite/redux-router/blob/master/src/reduxReactRouter.js
'use strict';

exports.__esModule = true;
exports['default'] = reduxReactRouter;

function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { 'default': obj }; }

var _redux = require('redux');

var _reactRouter = require('react-router');

var _historyMiddleware = require('./historyMiddleware');

var _historyMiddleware2 = _interopRequireDefault(_historyMiddleware);

var _constants = require('./constants');

function reduxReactRouter(_ref) {
  var routes = _ref.routes;
  var createHistory = _ref.createHistory;
  var parseQueryString = _ref.parseQueryString;
  var stringifyQuery = _ref.stringifyQuery;
  var routerStateSelector = _ref.routerStateSelector;

  return function (createStore) {
    return function (reducer, initialState) {

      var baseCreateHistory = undefined;
      if (typeof createHistory === 'function') {
        baseCreateHistory = createHistory;
      } else if (createHistory) {
        baseCreateHistory = function () {
          return createHistory;
        };
      }

      var history = _reactRouter.useRoutes(baseCreateHistory)({
        routes: routes,
        parseQueryString: parseQueryString,
        stringifyQuery: stringifyQuery
      });

      ['pushState', 'push', 'replaceState', 'replace', 'setState', 'go', 'goBack', 'goForward', 'listen', 'createLocation', 'match'].forEach(function (funcName) {
        if (!history.hasOwnProperty(funcName) && typeof history[funcName] === 'function') {
          throw new Error('History API does not support function: ' + funcName);
        }
      });

      var store = _redux.applyMiddleware(_historyMiddleware2['default'](history))(createStore)(reducer, initialState);

      store.history = history;
      store[_constants.ROUTER_STATE_SELECTOR] = routerStateSelector;

      return store;
    };
  };
}

module.exports = exports['default'];

This is what throw errors in console:
      var history = _reactRouter.useRoutes(baseCreateHistory)({
        routes: routes,
        parseQueryString: parseQueryString,
        stringifyQuery: stringifyQuery
      });

reactRouter.useRoutes is deprecated.
if I install last github commit version, everything works fine.

The github isn´t always up-to-date with the published npm package. The react-router v2 change was merged a while ago into the repo, but I just published it yesterday to npm with the beta tag. You can install it via npm install redux-router@beta.


schedulerArguments field missing from JobDescription.equals method

Also missing in clone constructor, toString and hashCode methods.

Doh!
Will fix


bookmark work in calendar

Fixes #2039
Sample :-  https://shubham3211.github.io/calendar-fix/index.html

@agbilotia1998 the forked repo was not updated , that is why empty calendar is appearing.


Refresh JPBParallaxTableViewController

How would I go about refreshing the tableview? I want to refresh it instead of zooming into the image when the user gets to the top.

[[self tableView] reloadData will do it. If you mean something like a pull-to-refresh, it doesn't exist as part of this library.


allow kubectl drain without the --force flag

kubectl drain by default is blocked when there is orphan pod on draining the node.
By adding the field controller: true to the ownerReferences of instance manager pods, we allow the user to run kubectl drain without the --force flag
Now users can run something like:
kubectl drain phan-cluster-v25-worker-1 --delete-local-data=true --force=false --grace-period=-1 --ignore-daemonsets=true --timeout=120s

longhorn/longhorn#1286

Also, we need to be mindful about --timeout=120s. Not sure if PodDistruptionBudget can be affected.

Also, we need to be mindful about --timeout=120s. Not sure if PodDistruptionBudget can be affected.

How are you thinking that this could affect PodDistruptionBudget of engines?

Also, we need to be mindful about --timeout=120s. Not sure if PodDistruptionBudget can be affected.

How are you thinking that this could affect PodDistruptionBudget of engines?

I've checked the reference, it should be fine. I was worried about kubectl will take some actions (e.g. remove PodDistruptionBudget) after the timeout. But it looks like the drain will simply fail after the timeout.

Thank you!


Remove sep from fastparquet calls

[x] Closes #8201
[x] Passes black dask / flake8 dask / isort dask

Looks like this is failing a scheduler import test on windows, though I'm not sure what part of this change would cause that.


Use Ubuntu rather than Debian as the default and recommended image

This is a follow-up of https://github.com/microsoft/vscode-dev-containers/issues/533, since the discussion there moved from Python to an OS choice.
This implies not using the official language images as the base image (such as library/node), since most of them do not get precompiled with Ubuntu (and why would they, since containers, in general, are not faced for human interaction = development).
If you are interested in this topic, please make sure you read https://github.com/microsoft/vscode-dev-containers/issues/533 at least until where this new Issue was created.

Replying to https://github.com/microsoft/vscode-dev-containers/issues/533#issuecomment-685294559:

Node includes yarn for similar reasons. Most of these originate from the buildpack-deps base image which itself is not intended as a deployment image.

It's true that those images are not intended for deployment, but I say that those images are also not intended for development. They're intended for build cycles, continuous integration, wherever a human is not needed.

There's also a huge value if you are doing container deployment in using the same base for development that you do for deployment.

I can be very wrong, but I can't think of a single reason why doing this would be beneficial. In the end, the developers would still have to develop the Dockerfile for the deployment version and test it. Working inside of "the same" image does not exempt any work. And it's not possible at all to have the same image, because the development image would need to have docker installed and the deployment image not. Again, this for me makes no sense.

The idea is to start from a known, solid base, then add rather than reinvent from the ground up. No image can fit all needs, which is why we're also trying to get things in a reusable form for custom solutions.

Interesting. In my opinion, a known and solid base would be ubuntu. 😀
I imagine a day where we can have any kind of combination between language stacks on images, and have them prebuilt automatically and programmatically, just by setting what scripts from the library to run.
Something like:
mcr.microsoft.com/vscode/devcontainers/base:ubuntu
mcr.microsoft.com/vscode/devcontainers/base:ubuntu-node
mcr.microsoft.com/vscode/devcontainers/base:ubuntu-python
mcr.microsoft.com/vscode/devcontainers/base:ubuntu-node-python
mcr.microsoft.com/vscode/devcontainers/base:ubuntu-docker-java-node

Always relying on a solid base image for general purpose, which can be either ubuntu or debian, but not the language's official images.

Of curiosity, if you assume for a moment that we would not want to use PPAs that are not official - what advantage does Ubuntu provide over Debian for you?

Well, this is a hard discussion. My main argument is: why not Ubuntu?
Ubuntu is far more popular than Debian, even for development. Developers/humans, in general, prefer to work with an OS they know already.
The fundamental difference between Ubuntu and Debian is that Ubuntu focuses on being practical when Debian focuses on being totally free and open source. One of the things that makes this very clear is the PPA.

There's always a Debian vs Ubuntu vs CentOS vs Fedora argument which is why I'd like to get to some specifics on what Ubuntu provides that Debian does not. We've opted to follow the Debian/Ubuntu path for a wide variety of reasons and the topic here does not seem to be about that (though it also comes up). In terms of what comparable recent releases of Ubnutu and Debian provide, Ubuntu adds:

PPAs
????

Really, the question is not about Ubuntu vs Debian, but instead whether there is sufficient benefits to moving away from heavily used Docker base images intended for build and run. The downside of moving away from Docker base images each image is  no longer in the same OSS ecosystem as millions of other Docker users as things around the image change. "Going it alone" always comes with risks.  We'll be relying on the comparatively modest number of contributors here instead.  (When an Ubuntu variation exists, happy to include it.)
In terms of the build - a concrete example of a benefit for Docker is using the well known builder pattern:
FROM mcr.microsoft.com/vscode/devcontainers/typescript-node:12 AS development

# Build steps go here
FROM development as builder
WORKDIR /app
COPY src/ *.json ./
RUN yarn install \
    && yarn compile \
    #  Just install prod dependencies
    && yarn install --prod

# Actual production environment setup goes here
FROM node:12-slim AS production
WORKDIR /app
COPY --from=builder /app/out/ ./out/
COPY --from=builder /app/node_modules/ ./node_modules/
COPY --from=builder /app/package.json .
EXPOSE 3000
ENTRYPOINT [ "/bin/bash", "-c" ]
CMD [ "npm start" ]

If your dev stage matches your prod stage's base, you will have fewer unexpected issues.

I doubt a Node developer would prefer to use the node base image (and thus having to rebuild the entire container if he just quick need to switch the version for testing) rather than another one with nvm installed.

In terms of what comparable recent releases of Ubnutu and Debian provide, Ubuntu adds:

PPAs
????

More packages available to install is a quite reasonable advantage, don't you think? Not only PPA, but Ubuntu's repositories often offers more packages as well.
Compared to Debian, it also offers more recent versions of the packages.
Another compelling reason is: VSCode Remote Containers and Codespaces are pushing hard the concept of dotfiles for personalization. Since Ubuntu is the most popular OS, the chances are that the user's dotfiles were made for running in Ubuntu. Switching to Ubuntu would enable an easier transition for the dotfiles, or with fewer tweaks at least.

As an update here, we're talking with the Oryx team about getting an Ubuntu flavor of their image so we can base codespaces-linux (aka "kitchen sink") on it going forward.
I'll be adding the distro into the README on each definition so we are able to give people visibility to which ones are on Ubuntu, Debian, Alpine, or perhaps "other" at some point. C++ now has Ubuntu flavors as well. Other definitions will be tied to their base images for the reasons described above for the time being.
/cc: @bamurtaugh @burkeholland @2percentsilk

Other definitions will be tied to their base images for the reasons described above for the time being.

Makes sense and I also agree with it, contradicting my previous opinion.
I just thought of another reason why Ubuntu can be a better choice:

For most of the applications, one of the main targets is Ubuntu Linux, mainly because of its popularity. I believe we can say that there are more chances to find packaged targeting Ubuntu (or at least with precompiled binaries distributed) rather than Debian.
Also, Ubuntu can run any package made for Debian, but the vice-versa is not true. This means that indeed there are more packages available when using Ubuntu.

Yes!


Iv 1209 tss

A general comment about using RSC in all these scripts: should we be using sudo to before each rsc call? (would apply to all the scripts in this repo)

Sudo shouldn't be needed, scripts run as the same user as the secrets file.

We should rename rll/setup_software_repo.sh to rll/install_security_updates.sh


Bug on initialization of the app using generated apk

Describe the bug
When doing the test directly from the android studio it works fine, the admob is initializing very good.
But when we are doing the build from the ionic capacitor cmd and use the generated apk , that is where the white screen happening upon the initialization of the app.
Within the initialization below using ionic 5 with admob latest version
export class AppComponent {
constructor(
private platform: Platform,
private splashScreen: SplashScreen,
private statusBar: StatusBar,
private dataService: DataService
) {
// Initialize AdMob for your Application
AdMob.initialize();
this.initializeApp();
}

To Reproduce
The bug is occuring upon the initialization of the app
Expected behavior
it should proceed to mainscreen
Screenshots
error response from the android studio
2020-11-10 12:42:59.711 25142-25142/? E/Capacitor/Console: File: http://localhost/vendor-es2015.js - Line 44076 - Msg: ERROR TypeError: Cannot read property 'initialize' of undefined
2020-11-10 12:42:59.724 25142-25142/? W/Capacitor/Console: File: http://localhost/vendor-es2015.js - Line 109555 - Msg: Native: tried calling StatusBar.styleDefault, but the StatusBar plugin is not installed.
2020-11-10 12:42:59.725 25142-25142/? W/Capacitor/Console: File: http://localhost/vendor-es2015.js - Line 109561 - Msg: Install the StatusBar plugin: 'ionic cordova plugin add cordova-plugin-statusbar'
2020-11-10 12:42:59.726 25142-25142/? W/Capacitor/Console: File: http://localhost/vendor-es2015.js - Line 109555 - Msg: Native: tried calling SplashScreen.hide, but the SplashScreen plugin is not installed.
2020-11-10 12:42:59.727 25142-25142/? W/Capacitor/Console: File: http://localhost/vendor-es2015.js - Line 109561 - Msg: Install the SplashScreen plugin: 'ionic cordova plugin add cordova-plugin-splashscreen'

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [All android]
Version [all]

Additional context
Add any other context about the problem here.

That is not a problem with the plugin. No native plugins are ti be found in your app.
It is probably an cap sync error.

Cannot read property 'initialize' of undefined

This is code issue. Please check these code. Thanks.


Validate that data exists when read by read operator.

This will involve checking that the desired data is returned. and not an empty cubelist.
This should raise an error if it does.
It needs to be checked if iris does this already when loading.

I think we are already doing this.


nixos-render-docs: Parsing error when nesting blocks without ::::

Describe the bug
When a block opened with ::: contains another block using the same, both blocks are closed by the first ::: encountered.  The second ::: is rendered as text.
Steps To Reproduce

Use the following in documentation (in my case, a nixdoc comment under lib/) :

:::{.example}
## Showcase a parse error in `nixos-render-docs`
Consider having a note in here:

:::{.note}
Note not not
:::
:::

Render the documentation (nix-build doc)
Open the result in a browser, navigate to the testcase.

(there's presumably a way to run nixos-render-docs outside of nixpkgs' documentation as well)
Expected behavior
Nesting blocks (without using ::::) should either cause a parse error, or only the innermost block should be closed by :::.
The latter seems preferable, as the :::: syntax is:

apparently undocumented: I only learnt of it by checking the source code before opening a bug ;
significantly less discoverable: while a user may likely intuit that they can nest blocks, needing a different syntax for it came as a surprise (and it remained unclear whether deeper nesting would require :::::, :::::: and so on) ;
not composable: users must modify blocks when moving part of a document into or out of a block.

Additional context
Issue encountered when writing documentation for lib.packagesFromDirectoryRecursive: #359898
Notify maintainers
@NixOS/documentation-team

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Do you have some error log as a reference?
I am wondering which tool creates this bug.
Nrd i guess

Yes, this is clearly nrd. My hunch is that fixing this will require staring at the code quite a bit (even if the fix will be a two-liner), so the most effective workaround would simply be documenting the intended (nesting with more repetitions) behavior for now. It's not really wrong that it will do the wrong thing if held wrong, so throwing an error is merely nice to have.


Automatically install license

If a license file was specified before calling rocm-create-package, install it to the correct location. If one wasn't specified, scan for a LICENSE(.*) file, then if one isn't found add it as the license file and install to the correct location.

If one wasn't specified, scan for a LICENSE(.*) file, then if one isn't found add it as the license file and install to the correct location.

I don't think this is a good idea. If there's some sort of edge case in which your scanner picks up the wrong file, it will probably take a while for anyone to notice. This is the sort of thing that developers should be explicit about.

All libraries that are packaged now require a license file. I am not sure if there are edge cases. Perhaps limiting the scan to license.txt or license.md may be safer.

I am not sure if there are edge cases.

I was thinking it was recursive, but ${CMAKE_SOURCE_DIR} would be the source of most unexpected behaviour. Given that there's now a check that exactly one file is found, and Liam has reminded me that this isn't going to recurse through CMAKE_BINARY_DIR when that's a subdirectory of CMAKE_SOURCE_DIR, I think we're fine.


Want to Write Backend Of Contact Page #980

Description
Career Queries Data Getting Stored in Database And Made emailjs Service as well to Send Acknowledgment Mail
Fixes #980
Replace issue_no with the issue number which is fixed in this PR
Screenshots

Checklist

[ X] Tests have been added or updated to cover the changes
[X ] Documentation has been updated to reflect the changes
[X ] Code follows the established coding style guidelines
[ X] All tests are passing

Hey @thestarsahil to make Email Js Work you need to add your Credentials here

https://www.youtube.com/watch?v=qxWDVRyc95E refer to this Video For setting up Credentials if required

And Please Assign it Level 2.


Add token as a field in the UI

Token can be set in the format templates but to simplify this it should be possible to set this as a field and have the ispindle format use this by default.

added to dev


Azure: fix error on resolving image version with multiple locations.

The original code hasn't use the latest runbook to resolve image. So the location is not set to single value.

Thanks for this fix @squirrelsc

@LiliDeng, please help merging it.


Run base image with RUST_BACKTRACE on

Useful to have on by default since this is still experimental stuff and people are going to open issue reports that do not provide enough info.

@yeastplume ptal


Add specification for article purchase functionality

The specification should contain the
the business concepts
screens, and its states
requirements related to the functionality
How the screens are connected, how the related navigation works
UI design
Design of the implementation

Done: https://github.com/Balintus00/kotlin-clothing-webshop/wiki/Checkout-and-item-ordering


where is the source code of modified caffe used by the perception module.

The readme.md (https://github.com/ApolloAuto/apollo/tree/master/modules/perception)
says a modified caffe is used, but no link to the source code of the modified version is available.

@csukuangfj I don't think this part is open-sourced.

@xiaoshuliu
thank you for the comment.
Many open source projects are used inside Apollo including caffe. Are there any reasons to make Apollo open sourced but not the modified caffe ?

@csukuangfj I'm sorry I don't know the reason...

Current the modified caffe is not open sourced. It is provided as a .so library.

@lianglia-apollo Hi Liangliang, I have two questions now regarding this issue.

Is there any plan to open source this part right now?
The .so library is compiled using gcc 4.8. Is it possible to request .so file complied using gcc 5+?

Thanks for your answer!
Yuesong

It's retired in master HEAD.


LoadBalancer option for common service

I have a setup on Google Kubernetes Engine where I deploy a cluster in each of three regions. To index data, I need access from outside the cluster, but all I need is an internal IP.  I am using annotations that GKE provides for this but it requires being able to set spec.Type: LoadBalancer for the service.
apiVersion: v1
kind: Service
metadata:
  annotations:
    cloud.google.com/neg: '{"ingress":true}'
    networking.gke.io/internal-load-balancer-allow-global-access: "true"
    networking.gke.io/load-balancer-type: Internal
spec:
  type: LoadBalancer
  ...

is there a way to set the type for the commonService

There is no way to currently do this, but it is something that we could look into.
My first thought is that the common service is used by the operator to communicate with the Solr Cloud within the Kubernetes cluster, so if we instead make it a loadBalancer service, would the same internal DNS entry still exist?


HUE Installation Error: [ipython[0.10]] Error 1, hue/build/.devtools] Error 2

I'm trying to build HUE on my laptop and am getting the following error (for make apps):
Note: I have ipython already installed on my computer.

I also had this problem. And I installed 3.8-dev by pyenv

@chozee how did you solve it?

I solved it, use "PREFIX=/data/soft/test/ make install" not "make apps"


Fixed configration changes are not reflected in function initialize().

Yes, current code sets only $this->anchor_class in constructor. Why? Is it a bug?
If so, it is better to move the code in constructor to initialize() function.


Disks play universally in a world, not based on location

that's definitely odd... they should behave identically to vanilla music discs, unless i missed something

Are your audio files mono? I think if they're stereo, Minecraft will play them with unlimited attenuation, so they need to be converted to mono to properly work.

i don't actually include any audio (except in the 1.12.2 version, i include 1.13 music there); the mod uses the vanilla sounds

Seeing as they're designed to be played directly to a player, maybe those files are stereo... of course, there's not much you can do there, seeing as you can't redistribute Mojang's assets.


message_logger: 0.2.0-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository message_logger to 0.2.0-1:

upstream repository: https://github.com/ANYbotics/message_logger.git
release repository: https://github.com/mikekaram/message_logger-release.git
distro file: melodic/distribution.yaml
bloom version: 0.10.0
previous version for package: null

@clalancette thanks for the input. Well, I'm not the author but I know that it uses the ROS logging mechanism or std::cout depending on whether ROS is installed. Maybe I should rename this also to anybotics_message_logger.
Thanks,
Mike


Could not find gem jekyll-remote_csv

Hi, @chrismytton
I am trying to install the gem but It appears to not be uploaded to RubyGems. Do you know if there is any problem? Am I doing something wrong? I am trying to install the gem into my Jekyll project.
Could not find gem 'jekyll-remote_csv (>= 0) ruby' in any of the gem sources listed in your Gemfile or available on this machine.
Thanks!

Hi @miguelcarvajal! I've just released version 0.1.0 to Rubygems, so you should be able to install it now. I hadn't released it before because it's a bit of a work in progress, so if you find any rough edges or confusing parts please do open a ticket!

Thanks!


Fix: README - Couchdb.Connector.Writer.create/2 is undefined. Use Couchdb.Conn…

…ector.Writer.create_generate/2 instead.

Coverage remained the same at 100.0% when pulling 09d0c59686ca952b0c4f4300c941f16dda235009 on Morsicus:fix_readme into 42ee102e71c72d4c29fc2814c93e7fc98b7c8efb on locolupo:master.

Thanks!