Feature request: Allow sorting Extensions

Could you please allow sorting extensions by the number of installs (for example) ?

From what I remember, this extension by dnbard does what you're after and more.

There's an extension called Extensions Rating by @dnbard which provides this feature.

Or you can use the http://brackets.dnbard.com/ web site

Thanks, good to know :)
IMO, such a feature should be a default core extension, don't you think so?

@redmunds that's great. Thanks for sharing.

Yes, we would like to expose this in the core Brackets UI in the future.  One issue is that the extension "download" counts are currently not that reliable: updating an existing extension is recorded as a new download, and occasionally bots will generate thousands of false downloads.  But if we iron out some of those kinks, this would be a great built-in feature.
I see a few existing backlog items somewhat related to this -- e.g. Extension Ratings and Reviews or better Extension Download Number Reports -- but none that track it directly ...so leaving this issue open for now.

I'd like to do a big batch of extension-related improvements sometime in the next few months. There are a number of improvements on both the registry itself and how we expose it in Brackets that would be nice to have, this included.

@ryanstewart does these changes in registry will be backward compatible to registry format that Brackets are using today?
Also is there a list of what changes it will be?
I'm asking because my extension - https://github.com/dnbard/brackets-extension-rating are dependent on Registry information and I want to integrate with them as soon as possible.


Allow for simple replacement of variables that are part of other variables (like APP_SRC and DIST_DIR)

The ProjectConfig allows to replace values from the extended SeedConfig. That works fine as long as the replaced variable is not as part of another variable. However, when it comes variables that are used as part of the definition of other variables, the replacement does not work as expected. For example:
export class ProjectConfig extends SeedConfig {
    constructor() {
        super();
        this.DIST_DIR = 'output'; // instead of 'dist'
        this.APP_SRC = 'app'; // instead of 'src'
    }
}

The variables for DIST_DIR and APP_SRC have properly been created. However, when running "npm start", you will notice that the CSS File has not been included in index.html and when running "npm run build.prod", the output gets partially created in 'output' and 'dist'. The variables partially use the values from ProjectConfig and also from SeedConfig.
This is because ProjectConfig needs to call SeedConfig's constructor before being able to set the variables. SeedConfig then initializes it's own variables as follows:
...
ASSETS_SRC           = `${this.APP_SRC}/assets`; // = src/assets
...
DEV_DEST             = `${this.DIST_DIR}/dev`; // = dist/dev
PROD_DEST            = `${this.DIST_DIR}/prod`;  // = dist/prod
TMP_DIR              = `${this.DIST_DIR}/tmp`;  // = dist/tmp
APP_DEST             = `${this.DIST_DIR}/${this.ENV}`;  // = dist/prod or dist/dev
...

Those variables use the original values of APP_SRC and DIST_DIR such that the overwrite of ProjectConfig is more or less useless. The workaround is the have ProjectConfig overwrite ALL those variables. Unfortunately, this is not very stable regarding updates of angular2-seed since one needs to watch out for newly introduced variables like the ones above.

IMO it's a valid point though changing it would require a bit of planning to not cause existing projects a bit of pain.
//cc @ludohenin @mgechev @NathanWalker

I totally understand, @d3viant0ne . Just to make the issue report complete, here is what needs to be changed when you want to change DIST_DIR properly:
- DIST_DIR
  |- DEV_DIST
  |- PROD_DIST
  |- TMP_DIR
     |-- SYSTEM_BUILDER_CONFIG
  |- APP_DEST
     |-- CSS_DEST
     |-- JS_DEST
     |-- APP_ROOT

APP_SRC only affects ASSETS_SRC and potentially APP_ASSETS (ProjectConfig currently overrides it which is why it would work).

@d3viant0ne see comment in #579

@Paladinium makes sense. This is part of the roadmap in #579 so we can close this particular issue.

Fine for me, @mgechev .


[WIP] Optimize dashboard performance

Work in progress branch to sort out some performance issues in the dashboard from #180

Changes Unknown when pulling c0bb97288775d4f7617e84fe3904ed198ca9df45 on cuducos-dashboard-performance into ** on master**.

Changes Unknown when pulling 2b1fd46854ae45532a5df2d1e23b5a0c126a5d9a on cuducos-dashboard-performance into ** on master**.

I got it from https://stackoverflow.com/questions/2204058/list-columns-with-indexes-in-postgresql.
But in the pgAdmin it's easy to see.
"select
t.relname as table_name,
i.relname as index_name,
array_to_string(array_agg(a.attname), ', ') as column_names
from
pg_class t,
pg_class i,
pg_index ix,
pg_attribute a
where
t.oid = ix.indrelid
and i.oid = ix.indexrelid
and a.attrelid = t.oid
and a.attnum = ANY(ix.indkey)
and t.relkind = 'r'
and t.relname like 'test%'
group by
t.relname,
i.relname
order by
t.relname,
i.relname;"
Return all table names,indexes, and columns.

Hi @thiagoalencar,
Adapting that command to:
"select
t.relname as table_name,
i.relname as index_name,
array_to_string(array_agg(a.attname), ', ') as column_names
from
pg_class t,
pg_class i,
pg_index ix,
pg_attribute a
where
t.oid = ix.indrelid
and i.oid = ix.indexrelid
and a.attrelid = t.oid
and a.attnum = ANY(ix.indkey)
and t.relkind = 'r'
and t.relname like 'core_reimbursement'
group by
t.relname,
i.relname
order by
t.relname,
i.relname;"

I got:
    table_name     |                      index_name                      |    column_names
--------------------+------------------------------------------------------+---------------------
 core_reimbursement | core_reimbursement_applicant_id_dc5b4033             | applicant_id
 core_reimbursement | core_reimbursement_cnpj_cpf_9035d450                 | cnpj_cpf
 core_reimbursement | core_reimbursement_cnpj_cpf_9035d450_like            | cnpj_cpf
 core_reimbursement | core_reimbursement_congressperson_name_08c45884      | congressperson_name
 core_reimbursement | core_reimbursement_congressperson_name_08c45884_like | congressperson_name
 core_reimbursement | core_reimbursement_document_id_53a6e416_uniq         | document_id
 core_reimbursement | core_reimbursement_issue_date_1ad82d84               | issue_date
 core_reimbursement | core_reimbursement_last_update_3fee6abc              | last_update
 core_reimbursement | core_reimbursement_month_b87da546                    | month
 core_reimbursement | core_reimbursement_pkey                              | id
 core_reimbursement | core_reimbursement_receipt_fetched_f95c4dbf          | receipt_fetched
 core_reimbursement | core_reimbursement_state_9c5760b1                    | state
 core_reimbursement | core_reimbursement_state_9c5760b1_like               | state
 core_reimbursement | core_reimbursement_subquota_id_2d6a31d0              | subquota_id
 core_reimbursement | core_reimbursement_year_92263c07                     | year
(15 rows)

Good, following the query sent in the telegram group,
"year DESC, issue_date DESC, id DESC"
From Django model, create an index with this three columns: 'index_together = ["year", "issue_date","id"]'
Reference:https://docs.djangoproject.com/en/dev/ref/models/options/#index-together
If you wish to get better result and your queries sorts are fixed you can create index on postgresql direct and define the sorting order:
''CREATE INDEX  xxx ON core_reimbursement (year DESC,issue_date DESC, id DESC );"
Reference:https://www.postgresql.org/docs/9.0/static/indexes-ordering.html
https://devcenter.heroku.com/articles/postgresql-indexes

From Django model, create an index with this three columns: index_together = ["year", "issue_date","id"]

Sounds a good idea, gonna bring back the WIP label and test it to see if we have a better result.

If you wish to get better result and your queries sorts are fixed you can create index on postgresql direct and define the sorting order:

That's definitively a no-go for me. This adds a lot of overhead for maintenance, provision and deploy. Also adds a lot of difficulties for the community to replicate our development and production environment.

Wow! @thiagoalencar you nailed it.
Before index_together I timed it 5 times locally, then I applied the migration and ran it 5 times again:

Attempt
Without index_together
With index_together

1
14.119s
0.077s

2
8.867s
0.070s

3
9.395s
0.068s

4
9,969s
0.037s

5
8.392s
0.040s

Ready for code review @jtemporal ; )

Changes Unknown when pulling 153c6b4abd7a727afc20b0ca4196d252044bc69d on cuducos-dashboard-performance into ** on master**.


Expose "jse_shortmsg" publicly

When receiving a complex VError that potentially has many MultiErrors as causes, it would be nice to be able to pick apart this error and display it for human consumption. As it is, the "message" property contains details of the cause, and if it's a MultiError it only prints the first (e.g. "first of 1 error: ..."), and there's no public method of getting just the top-level error message that doesn't refer to these MultiErrors.
For example, given the following VError:
> var component0 = new VError.MultiError([ new VError('problem0'), new VError('problem1'), new VError('problem2') ], 'something bad happened with component0');
> var component1 = new VError.MultiError([ new VError('problem0'), new VError('problem1'), new VError('problem2') ], 'something bad happened with component1');
> var e = new VError(new VError.MultiError([ component0, component1 ]), 'unable to do thing because of problems');
> e.message
'unable to do thing because of problems: first of 2 errors: first of 3 errors: problem0'

I want to computationally explore and print this VError object in my command line utility to be able to iterate over these nested errors with VError.errorFromList and only display jse_shortmsg for readability.
This is possible by reaching into the VError and using the jse_shortmsg directly, but given that it's not documented I'd prefer not to rely on it. If this was a public property, I could rely on it existing and build an iterator over these nested VErrors to provide human friendly feedback. The end goal I have in mind here is having the "cmdln" library do this iteration/printing to display all error messages.

I realise now that this is pretty much what WError is for. WError is intended to hide the underlying errors from the top error, but they still exist as causes that can be explored.
A rough example similar to the one I initially provided:
var VError = require('verror');

var component0 = new VError.WError(new VError.MultiError([
    new VError('problem0'),
    new VError('problem1'),
    new VError('problem2')
]), 'something bad happened with component0');

var component1 = new VError.WError(new VError.MultiError([
    new VError('problem0'),
    new VError('problem1'),
    new VError('problem2')
]), 'something bad happened with component1');

var error0 = new VError.WError(new VError.MultiError([ component0, component1 ]),
    'unable to do thing because of problems');

console.error(error0.message);
VError.errorForEach(error0.cause(), function (suberr0) {
        console.error(' -', suberr0.message);
        VError.errorForEach(suberr0.cause(), function (suberr1) {
                console.error('   -', suberr1.message);
        });
});

Prints:
unable to do thing because of problems
 - something bad happened with component0
   - problem0
   - problem1
   - problem2
 - something bad happened with component1
   - problem0
   - problem1
   - problem2

I'll close this issue out.


Missing parenthesis in code example

The closing parenthesis is missing in the current code example, making copy/paste of the code failing.

Good spot, thanks @camfrout !


Wallet jest tests should cover both iOS and Android

Expected Behavior
Jest tests should cover both iOS and Android.
See https://blog.expo.io/testing-universal-react-native-apps-with-jest-and-expo-113b4bf9cc44
And cover snapshots too.
Current Behavior
Jest tests only cover Android.

It's still valid for mobile unit tests.


Typst generator

https://crates.io/crates/typst

crate module "typst"
Implement "Typst generator" (generate function)

pub trait TransformerTrait {
    fn parse(document: &Bytes, images: &HashMap<String, Bytes>) -> anyhow::Result<Document>;
    fn generate(document: &Document) -> anyhow::Result<(Bytes, HashMap<String, Bytes>)>;
}

Units test requered for this new feature  (looks at tests in json module)

you should do refcatoring of pdf generation function and move part of logic inside typst.rs generation function


'Failed to compile program' when attempting to submit job to Azure Quantum Workspace using Azure CLI

Describe the bug
When attempting to submit a job to Azure Quantum Workspace I'm getting a 'Failed to compile program' error. The program does compile file as I can run dotnet build and it builds successfully. Please see screenshots below.

To Reproduce
Steps to reproduce the behavior:

Create an Azure Quantum Workspace
Make sure your local Azure CLI can connect to the Workspace
Unzip the attached project
Attempt to submit it as a job to the workspace.

JobTest.zip
I'm unsure which target-id I should be using but I've tried them all and I'm still getting the same problem.
Expected behavior
When attempting to submit job I receive 'Failed to compile program' even though Visual Studio and dotnet build can compile it successfully.
System information

Visual Studio Quantum Development Kit 0.15.2101.126940
Visual Studio Enterprise 16.8.4
Windows 10 Enterprise 20H2,
.NET 5.0.102

@tomaustin700 Thanks for reporting!
We are looking into giving a more actionable message on this one.
Could it be that your project does not use the IonQ/Honeywell templates?
If so, then that would mean that the build succeed because in general it is a valid program, but it uses language features that are not supported by the target hardware. You will get IntelliSense and build diagnostics for those things if you use the IonQ/Honeywell templates.

Hi @bettinaheim,
My project is pretty simple, it's just the basic hello world example so I don't think it needs the IonQ/Honeywell template however I may be wrong - this is all very new to me. My original message does contain my sample project if you want to take a look at it. My workspace does only have the Microsoft QIO provider so do you think that's the case? Thanks

@tomaustin700 Got it - sorry, I did not read carefully enough. Azure Quantum offers both; running quantum inspired optimization algorithms (such as e.g. simulated annealing) and running quantum algorithms on real quantum hardware using Q#. The target id you are using is for quantum inspired optimization (that is a classical solver running on classical hardware). The Q# program you are trying to submit on the other hand is for running on quantum hardware. You can learn more about both in our MS Learn modules.
This module, e.g., tells you more about how to run Q# programs on quantum hardware.
This module, e.g., tells you more about solving optimization problems using quantum inspired optimization.
Here is the list of MS Learn module that, e.g., also contain content about how to setup your workspace in Azure, and other interesting content.

@tomaustin700 I hope this resolved your issue and will close this. Please don't hesitate to reopen it or open a new one if there is more we can help with.


Question about multiple Scroll Spies

Hello,
thank you very much for this powerful plugin!
Could this plugin be used to have multiple scroll spies in the same Vue project? I tried, but only the last scroll spy is working - it is somehow disabling the scroll spies that have been defined before.
The implementation as a Vue directive suggests to me that in every component one could have a different scroll spy, but I can't get it to work and there is no error message in the console.
Please advise.
Thank you very much!
Lucas

@bennyxguo I am facing the same issue, did you ever find a solution @lutuh

@peter-brennan Hi Peter, I don't think I did. I cannot recall exactly, but the project in question uses only one scroll spy as of now.


Cannot run deepq.experiments.run_atari with gpu?

Hi,
I encountered "segmentation fault" when running
python -m deepq.experiments.run_atari

with tensorflow-gpu==1.8
However this command goes well with pure cpu tensorflow==1.8
Wondering if it is possible to run deepq experiments with gpu?

It's definitely possible, I've done it.

Hi thanks for your quick reply! Sorry, this may be caused by the incorrect installation of tensorflow-gpu. I can do it on another machine. closing.


EarlyStopping can incorrectly detect metric improvement with positive tolerance

Hello!
The logic of class xgboost.callbacks.EarlyStopping determining whether the metric improved seems to be incorrect if parameter abs_tol is graeter than 0. Namely, its private method _update_rounds defines two helper functions, maximize and minimize, which erroneously determine the event of metric improvement.
In case when metric is intended to be maximized, the following code decides whether it has improved (became greater than previous best):
def maximize(new, best):
    return numpy.greater(get_s(new) + self._tol, get_s(best))

This function returns True if new + tol > best, but if tol > 0, this can be true even when new <= best (for example, new == best - tol/2 so new + tol == best + tol/2 > best), but obviously this is not the case of metric improvement.
Similarly, when metric is intended to be minimized, the following code decides whether it has improved (became less than previous best):
def minimize(new, best):
    return numpy.greater(get_s(best) + self._tol, get_s(new))

This function returns True if best + tol > new, but again, in case tol > 0 and (as an example) new == best + tol/2 this condition is true, but new is actually worse (greater) than best.
This means that when new metric value is slightly worse (not more than abs_tol) than previous best, EarlyStopping considers it as improvement and saves this value as new best (which is actually worse). And this can result in boosting slightly worsening the model each iteration, but EarlyStopping will consider it as an improvement, thus skipping the moment when the model was truly best and yielding incorrect best_iteration and so on.
Fortunately, this behavior can be fixed by substituting + with - in these two helper methods, so it becomes:

maximize: improves if new - tol > best (so new > best + tol);
minimize: improves if best - tol > new (so new < best - tol).

Related: https://github.com/dmlc/xgboost/pull/7066  I will add a min delta.

Hi, I have opened a PR: https://github.com/dmlc/xgboost/pull/7137 could you please help review it?

Hi @trivialfis!
Yes, I'm looking into it now.


live: Use ChannelFollows operation to fetch followed channels

see: https://github.com/Tkd-Alex/Twitch-Channel-Points-Miner-v2/blob/0c5f2f3a1fd8889985acc5cbe58af53d06fb7f53/TwitchChannelPointsMiner/constants.py#L168

Implemented in https://github.com/chowder/ttv-cli/commit/52d38eb2d09319cd8fbe80c5e484da2eae99a622


[Bug]: recurrent_states are difference between chunk and fused_recurrent in GSA

Describe the bug
The class ChunkGSAFunction returns ov, hkt, hvt in the end, and the outer funciton chunk_gsa has perfect doc.
    Returns:
        o (torch.Tensor):
            Outputs of shape `[B, H, T, V]` if `head_first=True` else `[B, T, H, V]`.
        final_state (Tuple[torch.Tensor]):
            Final state tuple having tensors of shape `[N, H, K, M]` and `[N, H, M, V]` if `output_final_state=True`.
            `None` otherwise.

But the class FusedRecurrentGSAFunction returns o, (hkt, hvt) when setting if T == 1 and not q.requires_grad: (usually in decoding) and returns ov.to(q.dtype), hkt, hvt otherwise. This will result in calling recurrent_states[0] fail in decoding, but not in training/prefill. Hence I have to write different code to cache my recurrent_states in prefill & decode.
Steps to reproduce the bug

Expected behavior
Convert to o, hkt, hvt
Environment info

Thank you!


37 backend create user data model

Created the background for the Login and Signup endpoints. The User Entity, Repository, and Controller have been created, to make new endpoints in the Controller, see: https://www.bezkoder.com/spring-boot-jpa-crud-rest-api/. These will have associated custom methods in the Repository.

Changes have been made


Plan to migrate the short descriptions

Acceptance Criteria

[x] Plan written that breaks down steps to migrate short descriptions
[x] Plan is discussed and agreed on by PM and content lead

@atopal @wbamberg Can this be closed?

Yeah, acceptance criteria met. We'll have a follow up about licensing


spec: the fact that non-basic interfaces are not alias-able is not documented

What version of Go are you using (go version)?

$ go version
go version go1.18rc1 linux/amd64

What did you do?

type C[T any] interface{~int; M() T}
type C1 = C[bool] // error
type C2 = comparable // error
type C3 = interface {~[]byte | ~string} // error

I didn't find this is documented in tip spec.

CC @griesemer

This is just a bug. I don't see a reason why there should be a restriction here. It's a trivial change but too late for 1.18.


Logger Sampling debug not working dynamically with AWS Chalice serverless framework.

Current Behavior

It's simply not dynamically logging, but either log (>= 0.6) or not (<= 0.6) based on sampling_rate's argument value.
Steps to Reproduce (for bugs)

chalice local
Postman run collection with iteration.

@app.route('/')
def greeting():
    session = requests.Session()
    path = '/status/500'
    logger.debug('', extra={'statuscode': 500})
    response = session.get(f'https://httpbin.org/{path}')
    response.raise_for_status()
    return {'hello': 'world'}

Environment

Powertools version used: 1.22.0
Python: 3.8

Hey @viniciusao - I'd need to set some time aside to setup a chalice app to better understand the middleware functionality before replying.
From Powertools Logger perspective, sampling is calculated at the instance initialisation (e.g., Logger()), as we explain in the docs.
This often happens at the global scope and has two effects:

Sampling decision will only occur at cold start, not on a per invocation
Chalice local or SAM CLI local with warm container support (container reuse) will have the same effect as warm invocation (sampling calculated once)

I'd suggest two possible outcomes: 1/ Create a feature request to calculate at per invocation - e.g., a new flag for inject_lambda_context decorator, or 2/ Create a Logger instance inside the handler (not practical)


advanced usage section of docs doesn't include instructions for memory management

Describe the bug

The docs section simply explains that it has the functionality, it doesn't explain how it works or how to use it
To Reproduce

go to the docs and search memory
Expected behavior

instructions on how to do memory management
Screenshots

Additional context

The example I beleive is correct -- I was searching the docs for how to clear the memory db and rebuild it, and was unable to find the instructions

this is from the new docs, hopefully it is a worthy improvement

To some degree -- ideally, there's instructions about how I , as a developer or user, can CRUD memories. That's what I was looking for when I clicked the section


Removing Nose-Progressive Dependency

I'm going to take this on and try and get the CircleCI tests above to pass!

Alright @mitchellj, I feel ready for this to be reviewed. Happy to talk through and explain any decisions made.


Font offset.y not working

Which operating system does the issue occur on? macOS
I tried it with multiple fonts, but it doesn't work. Any idea what can be the issue or if I need to change something else in combination with it?
font:
  normal:
    family: Iosevka Nerd Font # Tried multipe fonts
    style: Regular

  bold:
    family: Iosevka Nerd Font # Tried multipe fonts
    style: Bold

  italic:
    family: Iosevka Nerd Font # Tried multipe fonts
    style: Italic

  size: 12.0

  offset:
    x: 0.0
    y: 0.0 # I tried even 20.0, didn't work

  glyph_offset:
    x: 0.0
    y: 0.0

You should get an error when trying to use floating-point values for offsets. A recent change introduced a modification to the offset and glyph_offset and they should now be whole numbers. So try and see if 20 will work instead of 20.0.

It worked! thanks!
Maybe I was not getting any errors because I'm using the .app & not using alacritty from the CLI, I should do this to debug though. Thanks anyway 👍


default_content_type is being ignored and probably should default to application/octet-stream

__init__ accepts default_context_type as a named parameter, but does nothing with it after setting self.default_context_type. Good solution would be to modify async def file_type(file_path) to be a class method instead of static method, so it can access self.default_context_type. This would also allow moving the media_types dictionary to the class instance, making it modifiable (i.e. easily adding custom media types.)
Other 'text/plain' hard coding should be examined as well.

Fixed in 0.3.2


Notification model

Tracks when an email is sent

Belongs_to Hackathon
Belongs_to Subscription
Has_one User, through: Subscription

Handled by Hackathon::Digest::Listings


[Question] How to collect process_connect 、process_close and process_listen events

I've seen the four golden signlas here: https://github.com/cilium/tetragon/tree/main/docs/security-observability-with-ebpf/03_chapter/00_four_golden_signals . But there are only tutorials about how to collect "PROCESS_EXEC", "PROCESS_EXIT", "PROCESS_KPROBE" events.
I want to know how to collect process_connect 、process_close and process_listen events, any help would be appreciated!
Here is my tetragon configmap:
kubectl get cm -n kube-system tetragon-config -o yaml
apiVersion: v1
data:
  enable-k8s-api: "true"
  enable-process-cred: "false"
  enable-process-ns: "false"
  export-allowlist: '{"event_set":["PROCESS_EXEC", "PROCESS_EXIT", "PROCESS_KPROBE",
    "PROCESS_TRACEPOINT"]}'
  export-denylist: |-
    {"health_check":true}
    {"namespace":["", "cilium", "kube-system"]}
  export-file-compress: "false"
  export-file-max-backups: "5"
  export-file-max-size-mb: "10"
  export-filename: /var/run/cilium/tetragon/tetragon.log
  export-rate-limit: "-1"
  metrics-server: :2112
  process-cache-size: "65536"
  procfs: /procRoot
kind: ConfigMap
metadata:
  annotations:
    meta.helm.sh/release-name: tetragon
    meta.helm.sh/release-namespace: kube-system
  creationTimestamp: "2022-06-15T01:45:41Z"
  labels:
    app.kubernetes.io/instance: tetragon
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: tetragon
    helm.sh/chart: tetragon-0.8.0
  name: tetragon-config
  namespace: kube-system
  resourceVersion: "20294151"
  uid: 1749b71a-e54f-4c4f-9eb8-881c73bf4841

@Furisto Thank you for the explanation and example!

Just dropping in to confirm that those are not in the OSS version. The workaround suggested by @Furisto is a good way to approximate these with what we have in OSS.


Action os change

Shifted the syntax for the OS out of the matrix and back into the runs-on for simplicity

Looks fine

PR to wrong location - ignore


get rid of scripts, make CLI

Would be nice if the CLI tools could automatically be generated from all metric classes

done in #46


Improve Docker image and documentation

It would be great if the Docker image could be improved;

Drill documentation referrs to $DRILL_HOME environment variable which is not set in the Docker Image. Additionally in the doucmenation says it can be found in /etc/drill/conf while in the Docker image Drill has been installed into /opt/drill/conf
Add information how to add (mount) custom configuration files into the Docker image or even better, just make sure there is a Docker section in Drill's documentation whereever appropriate

Thanks for pointing this out.  I'm currently preparing a PR to update the JDK version in the Docker image and I'll give the docs a once over in the process.  We use Jira for ticketing so I'll close this in favour of https://issues.apache.org/jira/browse/DRILL-7999.


Unify Palette/Completion with List widget

This PR adds a List widget, which can be used for displaying lists like the palette or the completion. It changes the current implementations to use the List widget.
The general idea is that the List widget has some items that it renders. This is an im::Vector to make it cheap to clone, or replace (such as filtering it from the containing widget). It is given the focus commands (for list movement) by the containing widget. If it receives ListSelect, then it sends an event to the containing widget that an item was selected (and it sends that event if an entry was clicked), letting them handle it (and also decide whether to close the list at all).
With this, it will be easier to make simple lists. I plan on using it to implement a dropdown button element that we can use for the themes in the settings and in another PR change the git-branch selector to use a List.

I should perhaps just call completion.run_focus_command by itself rather than calling it for each list focus command
This had to make so the LapceCommand implemented PartialEq so that we could have a Data::same implementation.

Okay, I think this should be working now. I believe I pinned down the cause of the jump to reference bug I was having, though I'm not 100% sure since I had a hard time ensuring I could reproduce it.

I believe this should also fix #1202, since I used options for accessing the currently selected option and modified the callers.


Invert weights on Interview

Swap type weights and match weight to participant.
Questioner and question should be heavy; Responder and response should be light.

https://wellcome.slack.com/files/davidpmccormick/F3UVCU2JK/screen_shot_2017-01-20_at_14.38.33.png

Woop!


Bump go to 1.22.9

What this PR does / why we need it:
This is to address a CVE, reported in #331.
Special notes for your reviewer:
Release note:

NONE

Thanks @phoracek  for this uplift. would you please update the docker file go version in the following line to 1.22.7?
https://github.com/k8snetworkplumbingwg/ovs-cni/blob/9f9bb719fd2bd9e51aa05408eaf9781bd59d3c0e/hack/docker-builder/Dockerfile#L9

I have the understanding that go mod file change should have minimum go version that it supports. As earlier version of 1.22.7 has the vulnerability, isn't it better to update the go version to 1.22.7 in go mod file and same in docker file ( ovs-cni/hack/docker-builder/Dockerfile ).

@smoshiur1237 the Go version in the docker-builder should be only relevant to the Go used in test runs. But it's a good catch - I should update it too.
The version that matters for our containerized distribution is set here https://github.com/k8snetworkplumbingwg/ovs-cni/blob/main/cmd/Dockerfile#L10, and it is based on the one listed in Go.mod. Binary builds attached to releases are currently built by me manually (no CI), but they should follow the Go.mod directive.

@phoracek  I can understand the go version is taken from go mod file. Is there any issue to use 1.22.7 ?

@smoshiur1237 yes, following semver, if the fix is available in 1.22.7, it should be a part of all the newer z-stream releases.

/lgtm

/lgtm
/approve


Updates the data source of and modernizes the Feature Layer Query sample

@philium Accidentally dismissed your review.


Does not Close the last tab

Is your feature request related to a problem? Please describe.
If only 1 tab is open, the last tab is not closed, (if you manually close the last tab, the browser opens a new blank tab)
Describe the solution you'd like
The last open tab should be closed
Describe alternatives you've considered
Additional context

The last tab is always 'active' ... currently i exclude the active tabs from the cleanupQuery
i'll see what i can do.

would be great to have the last tab close as an option too :)

The problem with this request is that there might be "hidden" tabs in a window, which are currently being ignored (addons like Simple Tab Groups and Tree Style Tabs use this for example ... and if we close the last visible tab ... i think the window will get closed, which might cause unexpected datalose. ... That was the inital reason why i never wanted to close the last tab aka a window.

Seems like the lastAccessTime for active tabs (aka. last visible tabs in a window) is automatically keeped ... so my current algo can not easily be modified.
Sorry, but on this basis, i'll have to reject this request.


x64: Shrink Inst from 72 to 48 bytes

This PR pulls out the effective parts of #4496, boxing the tables in JmpTableSeq. The performance is roughly the same, but sequential runs show a very small improvement in memory usage. The parameter to the SmallVec used in BoxVecMachLabel could do with some tuning: 4 performed better than 16, but I haven't experimented further.
Sightglass spidermonkey benchmark
compilation :: nanoseconds :: benchmarks/spidermonkey/benchmark.wasm

  Δ = 21407759.30 ± 19781778.63 (confidence = 99%)

  branch.so is 1.00x to 1.01x faster than main.so!
  main.so is 0.99x to 1.00x faster than branch.so!

  [2845686334<PHONE_NUMBER>.07<PHONE_NUMBER>] branch.so
  [2867347959<PHONE_NUMBER>.37<PHONE_NUMBER>] main.so

compilation :: cycles :: benchmarks/spidermonkey/benchmark.wasm

  Δ = 77065090.93 ± 71216617.01 (confidence = 99%)

  branch.so is 1.00x to 1.01x faster than main.so!
  main.so is 0.99x to 1.00x faster than branch.so!

  [10244830916<PHONE_NUMBER>9.40<PHONE_NUMBER>8] branch.so
  [10322803658<PHONE_NUMBER>0.33<PHONE_NUMBER>2] main.so

Memory usage on the spidermonkey benchmark
Running the spidermonkey 10 times:
for i in $(seq 1 10); do
  /usr/bin/time -f "%M" -- taskset 0x1 ~/wasmtime/target/release/wasmtime compile benchmark.wasm
done

branch
main
trevor/box-jump-table-vectors

105288
105288

105600
105496

105340
105396

105508
105148

105268
105340

105596
105396

105464
105428

105368
105464

105340
105248

105296
105240

average(k)
105406
105344

I can't find where targets_for_term is actually used. It seems strange to me that the combination of default_target plus targets is always identical to targets_for_term. I would think you could save 24 bytes by just removing that field.
That said... I'd be tempted instead to remove default_target and targets, which I think should save 28 bytes, at the cost of needing an indirection to dig default_target out of the heap allocation instead of having it right there in the Inst. But when default_target is used, so is the rest of the targets array, so I think that shouldn't impact performance.
I also don't see that these vectors are modified after they're constructed. Box<[MachLabel]> is 16 bytes versus 24 bytes for Vec<MachLabel>, so I think you can shave a total of 36 bytes off by doing the above and then switching the targets array to a boxed slice.
This could be a terrible plan for reasons I don't know, obviously. :grin:

They originally existed to satisfy queries from regalloc.rs; although, thank you @jameysharp for persisting here, it seems indeed that targets_for_term is entirely dead now, and I believe targets as well. (MachTerminator used to carry a slice which was built from one of these and regalloc.rs used to query successors directly from branch Insts built from the other; I don't remember why they were separate but there was a reason...).
So perhaps they can just be removed entirely!

targets is used along with default_target in inst/emit.rs to emit the target addresses, so I think we still need one copy, just not two; and I'd still advocate for using the single boxed slice for all targets. Unless there's someplace else to get the targets from that's still around at that point? But I assume cutting the size of this case in half already makes it not the largest variant in this enum any more.

I'm happy to try out the single boxed slice approach, thanks for chasing the uses of these vectors down @jameysharp!

Removing the targets_for_tree field got some additional memory back:
104372
104192
104312
104372
104404
104304
104412
104368
104904
104400

Experimenting with a boxed slice and an non-boxed SmallVec both yielded more memory consumed pretty consistently, so for now a boxed SmallVec seems like the winner. I'm happy to experiment a bit more with this, but that'll need to wait until next week.

@cfallin, I made a few more changes since you reviewed: the targets_for_term field is gone, and I moved some aliases around a bit that were re-defined in other backends.


chore(deps-dev): update @readme/variable to v13.6.6

 PR App
:---:
🧰 Changes

[x] Update @readme/variable to v13.6.6

This PR was released!
🚀 Changes included in v6.38.2-next.2

This PR was released!
🚀 Changes included in v6.38.2


Blueprint routing

Bluprint Router
use API Blueprint files as the router.
Should be a Raxx application that routes to actions that are viable Raxx applications themselves

Prex is a client generator from API blueprints. contact after implemented
Drafter is required by Prex, it is a OS install has Arch and macOS instructions. failed to install within vagrant. would be best not to require this.
Snowboard seams to allow api functions from docker, but not exposing the ast. have opened issue to get api elements
api-elements is a standard representation that stretches to swagger and api blueprint

parse to elements using snowboard https://github.com/bukalapak/snowboard/issues/30

Don't need to use snowboard, lexx and yacc seam good enough


Skip duplicate actions

Perhaps based on the logic of https://github.com/marketplace/actions/skip-duplicate-actions

Limited to only push events.


Include/Import local file dependency

In a project, I have multiple kotlin script files, and I have some methods I need to use in most, if not all script.
Is there an existing way to write these method in a single external file, and include it from the other scripts ?
eg :
utils.kts
fun foo(s : String) : String {
  //...
}

myscript.kts
#!/usr/bin/env kscript

//INCLUDE utils.kts

foo(args[0])

Nice idea, and it seems easy to do.
But how do you write those scripts? Since the IDE won't resolve //INCLUDE you'll get tons of red code, don't you?

I've implemented it in a separate branch for now. See https://github.com/holgerbrandl/kscript/blob/include_directive/test/resources/includes/include_variations.kts for an example. Feel welcome to let me know what you think.

That seems perfect for what I want !

I'm still skeptical if it makes sense to include it into the release branch because of missing IDE support. All the included bits will light up in bright red in the master script. How would you code them efficiently without pain?

Was anything ever decided as to how to add IDE support for //INCLUDE files?

In general, the feature has been added, see https://github.com/holgerbrandl/kscript#ease-prototyping-with-include.
Except for IDE support, which would require a separate plugin. However, its fairly easy to configure the IDE accordingly in my experience.


[multistage] make mailboxID JSON serialized

BREAKING CHANGE: note that this change will not support rolling upgrades. If a deployment is running with the V2 engine, queries will fail during the roll.

Minor PR to change the serialization of mailbox identifier to a JSON format. For example:
{"jobId":"-7324288832996730535_2","from":"localhost:52683","to":"localhost:52683"}

Instead of:
-7324288832996730535_2:localhost:52683:localhost:52683

This is the first PR in a series to introduce virtual servers. It makes it easier for me to change the ServerAddress to also include a partition identifier and makes future changes backwards compatible since JSON has an evolvable schema.
NOTE: I chose to encode the server address in :-delimited instead of JSON to make it easier to read and send fewer bytes on the wire, but it does make them less flexible going forward (you can only add things at the end).

Codecov Report

Merging #10154 (99d6119) into master (4d7da0a) will decrease coverage by 10.16%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10154       +/-   ##
=============================================
- Coverage     34.84%   24.68%   -10.16%
+ Complexity      200       44      -156
=============================================
  Files          2006     1995       -11
  Lines        108718   108388      -330
  Branches      16501    16467       -34
=============================================
- Hits          37880    26754    -11126
- Misses        67566    78830    +11264
+ Partials       3272     2804      -468

Flag
Coverage Δ

integration1
24.68% <0.00%> (+0.17%)
:arrow_up:

integration2
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

 Learn more


Add gbd and gdbserver to images

Referenced issues
https://soramitsu.atlassian.net/jira/software/c/projects/DOPS/boards/43?modal=detail&selectedIssue=DOPS-2654&assignee=6163c8b178e5e40070bcd5f7

Description of the Change
Update base image for kagome-dev
Add gdb and gdb server to images
Remove gcc 10, use gcc 11 in CI instead
Apply new pip politics of safe directories and enforced venv

Benefits

Possible Drawbacks

Usage Examples or Tests

Alternate Designs

Need to activate /venv (discussed and tested locally, all works)


How to use network-attachment-definitions in opensearch helm charts.

Moved from: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/3927
@ganilmca commented on Mon Apr 24 2023
Hi Team,
We want to deploy  OpenSearch-cluster with network-attachment-definitions.
Can any one please help us to deploy the cluster with network-attachment-definitions.
What changes are required from our side.
Thanks & Regards,
Anil

@manasvinibs commented on Mon Apr 24 2023
@opensearch-project/triage transfer this to the helm-charts team https://github.com/opensearch-project/helm-charts

@prudhvigodithi Could you please advise on this?
CC: @TheAlgo @DandyDeveloper @peterzhuamazon


Map fetching fails in edge cases

The current approach to finding beatmap folders relies on the belief that the respective difficulty strings are always at least 60% smaller than the name strings. This is not always true.
A refactoring of the folder searching algorithm is necessary.

What it?

there is such a problem.

I will then delete it

@ssCherry Please create an Issue and add all relevant details. I will delete our conversation here once you have done that.

This was largely mitigated in the find_beatmap refactor.


includes all updates + actions (work in progress)

allow resource resolver to handle async operation in the client's context.
bring the branch to latest of the master
added support for icon image in actions (work in progress)
added support for background image / media / media icon image / image

@nesalang Perfect! Does exactly what I was looking for. Really like how your implementation uses the delegate method that was already there so it doesn't add any more delegate methods. Thanks again for the helpfulness and responsiveness, it makes me very excited to continue using this fantastic project :)


Bump cairo-vm

Bump cairo-vm to latest main.
This also required bumping cairo to 2.8.2 which required bumping our rustc version.
This pulls in an upstream fix which makes this hack obsolete.
Issue Number: N/A
Type

[ ] feature
[x] bugfix
[ ] dev (no functional changes, no API changes)
[ ] fmt (formatting, renaming)
[ ] build
[ ] docs
[ ] testing

Breaking changes?

[ ] yes
[x] no

I'll wait for this PR before raising mine.


dat-store as a main component of CLI

Improvements to make syncing multiple dats a first-class property of dat.
This is a proposed UX for integrating dat-store into the CLI:
$ dat run-service --config=myConfigFile.ext
Starting dat service...

Sharing dat on the network

$ dat clone dat://ade32bfg2e... /Users/okdistribute/data/nature-report
$ dat dat://ade32bfg2e... /Users/okdistribute/data/nature-report
Downloading...
Done.

Difference: for create/share/sync and clone, service knows that this dat was created
Difference: the service starts watching for changes and serves the dat history on the network

$ dat stop /Users/okdistribute/data/nature-report
$ dat stop dat://<key>
Stopping syncing...

New feature: if you want the service to stop sharing the dat
If the service isn't running, say it isn't running

$ dat info

New feature: Opens local website for dat-store that has a list of your dats and allows you to view progress/control their versions/etc

The future of dat sync and share
sync and share are confusing right now and do different things. instead, they ought to do the same thing and reduce the scope of the cli.

misc thoughts, updated constantly
nota bene, some points might contradict others
it would be nice if..

the cli consumed stdin in standard unix fashion, as that has proven to be a powerful pattern when used appropriately (and lets the other tools and programs that exist do the heavy lifting)
we had user personas defined publicly, i'm sure CS&S have a lot of wisdom to share based on their meetings with academics and their prowess in the terminal (i.e. who is the cli for? are uxers expected to have git experience? people who have torrent experience?)
the semantics mirrored git when it makes sense (dat init for example)
you never needed to create a shell script to catch the crashing dat daemon process (but that the process instead was self-healing, and maybe launched a child it could reboot when it dies)
you could list and manage the dats that have been created by using the cli
you could tell the daemon about new dats for it to seed and it would do so without needing to restart
if using the cli was composable with other tools, able to be used as part of a toolchain

so dat init already exists! yay
dat init --seed should be the same as dat </path/to/folder>
dat run-service
dat install-service
dat add dat://<key> dat://<key2> dat://<key3>

dat share/sync /path/to/folder /path/to/folder2 /path/to/folder3
dat clone dat://<key> /path/to/folder

Yeet


Update Everything (almost)

closes #123 (since it has merge issues)
Changes
.travis.yml
updated to current + LTS versions
see https://nodejs.org/en/about/releases/
package.json
Updated all dependencies to the latest version. Except istanbul and readdirp. Istanbul was deprecated and the new version is called  NYC. I didn't swap it out as I cannot see it being used anywhere. ReadDirP was updated to the newest 2.x version. It wasn't updated to 3.x because the callback API was deprecated. A separate pull request to update the code to use promises or streams is needed.
src/sloc.coffee
Wrapped a line because the linter was complaining.
Tests
two tests fail, but they seem to fail in master aswell
      2) should support agda
      1) should support rpy
travis fails #127 due to the two errors above

thx!


Reduce privileges during Reborn, not StartProcess

Daemon (child) should be able to run some code with root privileges,
open low ports and logs, for instance.

This change started causing me issues:
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x4 pc=0x4c974e]

goroutine 1 [running]:
github.com/sevlyar/go-daemon.(*Context).child(0xc208032410, 0x0, 0x0)
        /root/.whitesands/src/github.com/sevlyar/go-daemon/daemon_posix.go:250 +0x35e
github.com/sevlyar/go-daemon.(*Context).Reborn(0xc208032410, 0x0, 0x0, 0x0)
        /root/.whitesands/src/github.com/sevlyar/go-daemon/daemon_posix.go:73 +0x7d
main.main()

What should I do to fix it?


Audio/video calls in Messenger does not work in Ubuntu 18.04 / 19.10

Describe the bug
Audio/video calls in Messenger does not work in Ubuntu 18.04 / 19.10
To Reproduce
Steps to reproduce the behavior:

Start application
Start Messenger
Start Video chat
See nothing

Same for me in Mac OS 10.15.4


Odd behavior: Alternation fails instead of trying second alternative

Hi. I am just exploring Ohm in the online editor.
When using this grammar:
Demo {

  // does not work as expected
  Query = EmptyQuery | NotEmptyQuery

  // works as expected
  // Query = NotEmptyQuery | EmptyQuery

  EmptyQuery = ""
  NotEmptyQuery = any+
}

I am confused to as why it makes a difference in which order I put the alternations.
With Query = NotEmptyQuery | EmptyQuery:

the empty string will match the EmptyQuery
any non-empty string will match the NotEmptyQuery

However, with  Query = EmptyQuery | NotEmptyQuery :

the empty string will match as EmptyQuery
any non-empty string will fail as EmptyQuery and Expected end of input.

Why?! The reference says about the Alternation

Matches the expression expr1, and if that does not succeed, matches the expression expr2.

Should the parser not try the second alternative, i.e. NotEmptyQuery?
(When I alter the rule to EmptyQuery = "" end, it works as expected).
Is this a bug? Or am I misunderstanding something here?

Sorry for not responding earlier!
This is a common confusion, especially for people are used to the behaviour of regular expressions. In PEG parsers like Ohm, the | operator is an ordered choice that commits to the first successful match, even if it's partial.
In your example, EmptyQuery will always succeed and consume no input. That's why NotEmptyQuery is never tried in the "does not work as expected" case. The fact that this choice eventually leads to an overall failed parse does not factor into it.
There are pros and cons to both approaches. This behaviour of PEGs has the advantage that it maps in an very direct way to a hand-written, top-down parser.


Question: Lidl Lights model number

I was looking at the recent PR (#598) about a Lidl smartbulb and I see that powercalc will look at the model name between parentheses. The problem here is that the 'same' Lidl bulb can have numerous vendor and model names.
For context - around December 2020 Lidl in Ireland started selling their Smart home kit. I bought a pile of different bulbs (GU10, ES27), plugs and LED strips over several shopping trips. In January 2022 they started selling them again and I again bought more to swap out the remaining non-smart bulbs.
The GU10s have a variety of names and models even though they are the 'same' bulb from Lidl's point of view. It is the bulb which can do the warm-cool mode or the 'millions of colours' mode. I believe it is this model.
According to Phoscon these are the various names and manufacturers in my stock:
December 2020 Batch

Vendor = LIDL Livarno Lux        Model = HG06106A
Vendor  = _TZ3000_kdpxju99 Model = TS0505A
Vendor  = _TZ3000_quqaeew6 Model = TS0505B
Vendor  = Heiman Model = TS0505A

January 2022 Batch

Vendor = _TZ3000_quqaeew6        Model = TS0505B

I also checked this information in deConz and it matches.
Remember these are all the 'same' Lidl GU10 smartbulbs.
Lidl are obvious just sourcing from lots of different manufacturers but they are going to make life very complicated for auto discovery.  Is there some option for specifying aliases in the model.json (or dir name)?

There is a possibility for aliases in here: https://github.com/bramstroker/homeassistant-powercalc/blob/master/custom_components/powercalc/const.py#L94.
We can just add an alias there when we are 100% sure it is the same model, with same power characteristics.


fix(typing): Fix typing in websocket listener class

Description
Fixes #3763
It seems that declaring the actual method is the only way to comply with mypy in such cases.
There is a ruff rule that forbids empty methods in abstract classes, had to disable that for the relevant module.
Closes

Is there anything preventing this PR from being merged?

@all-contributors add @olzhasar for bugs and docs


Destructuring produces error but property access type checks

Here's a minimal example showing what I suspect is a bug in Flow.
Tested with Flow v0.32
/* @flow */

type Foo = {a: number};
type Bar = {b: string, params: Object};

function isBar(x: Foo | Bar): string {
  if (x.params !== undefined && x.params !== null) {
    // x.params definitely exists
    const { params } = x; // produces an error
    const y = x.params; // type checks
    return 'bar';
  }
  return 'foo';
}

I just tried to reproduce this and I got an error on both the destructuring and member access lines.
It turns out that this behavior is not a bug, but rather a consequence of width subtyping. It isn't enough to check that x.params is non-null and non-void to discriminate between Foo and Bar.
Due to width subtyping, an object { a: 0, params: "whatever" } is a valid instance of Foo. Because of this, Flow does not refine x to only Bar in the conditional block body.
You can use an exact object type for Foo to both forbid width subtyping and make the type test do what you want, type Foo = {| a: number |}.

Weird, I definitely only get errors in the destructuring.
I can understand why it's expected not to work though, thanks for the explanation!
P.S.: I don't think the website has been updated to reflect the new Exact types addition; I could be wrong though.


tsv-sample named field support

This PR is a follow-on to the named field PR series start with PR #284. It adds named field support to tsv-uniq. It works the same way as the support added to tsv-select, see PR #284 for more info.
Named field support is backward compatible with numeric fields. As with the other tools, this support is not documented yet, even in the help text. Documentation will be added prior to performing release.
This PR also adds some additional header field variation matches to the tsv-select tests. Mainly tests for field names requiring escapes. This is well tested at the unit test level, but its also useful to have some tests at the executable level.
With completion of this PR, all the tools taking fields as command line arguments have basic support for named fields. Documentation needs to be done (include help text), and there's a certain amount of cleanup to do in command line argument processing code for several tools.
This is a step towards enhancement request #25.

Codecov Report

Merging #292 into master will decrease coverage by 0.01%.
The diff coverage is 98.18%.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
- Coverage   99.33%   99.32%   -0.02%
==========================================
  Files          18       18
  Lines        6657     6697      +40
==========================================
+ Hits         6613     6652      +39
- Misses         44       45       +1

Impacted Files
Coverage Δ

tsv-split/src/tsv_utils/tsv-split.d
98.48% <ø> (ø)

tsv-sample/src/tsv_utils/tsv-sample.d
99.36% <98.18%> (-0.08%)
:arrow_down:


Telegram Web Calls don't Function (Stuck at Exchanging Encryption Keys)

Describe the bug
If you attempt to call someone or someone calls you in Telegram Web, the process will be stuck at "Exchanging Decryption Keys" after you/the user answer(s) the call.
To Reproduce
Steps to reproduce the behavior:

Go to web.telegram.org
Select a user
Click on telephone icon to call

Expected behavior
A ringing tone and call starts if the user answers it.
Screenshots
Desktop (please complete the following information):

OS: Windows 7 x64
Supermium: Version 122.0.6261.85 (Official Build) (64-bit)

Additional context
Note that you need to have a Telegram account and have someone with Telegram account too in order to perform this.

Have you confirmed that the standalone Telegram client is able to exchange the keys? You have ruled out that this is not a network issue, server side or local?

Have you tried both variants of the Telegram web client (type A and type G, iirc)?

I have the same issue. I see "Exchanging Encryption Keys" for a while and then my call is ended on the web. Desktop version works without issues. I use latest versions of everything on macos.


Devel

Now using Leaflet and Mapzen Search geocoding as well as boundaries generated by city of Chicago neighborhood GeoJSON. Also using bower and grunt for dependencies. TurfJS used to determine whether in neighborhood or not.

Updated to pull the neighborhood from Mapzen Search reverse geocoding if it's outside of Chicago. Should be good to go


IP parsing with X_FORWARDED_FOR

Hi, @pyepye!
I experienced an error using your library: there was no ip present in the geoip database.
Digging deeper, I found that the problem is in that line:
ip = x_forwarded_for.split(',')[-1].strip()

I have the following servers configuration with X-Forwarded-For headers configured:
Balancer NGINX -> Backend NGINX -> Django
The header that reaches Django looks like this: 93.81.xxx.xxx, 172.31.xxx.xxx. The line above will select the last one which is local ip address. That causes an exception at GeoIP
AddressNotFoundError
The address 172.31.xxx.xxx is not in the database

As far as I know, it is a common behaviour for proxies to create a chain of IPs that starts with client IP and is followed by proxies.
So, i'd suggest the following change for this line:
ip = x_forwarded_for.split(',')[0].strip()

Would like to know your thoughts, thanks.

Hey @unqx
Sorry for the late reply, I've not had much spare time recently.
Yep that does look like a bug, the first IP is the most likely to be the client's address.
I've just published version 0.7.1 to Pypi which now uses your suggested line / the first IP.
Hopefully that was the only issue with the GeoIP stuff, it doesn't have any tests wrapping it so I've been a little worried it wouldn't work as expected.
Thanks
Matt


Allows strategy used in Brave to avoid allocating on lookup

The Brave tracing library has a modified version of this type, which we
use for tracking in-flight (pending) spans.
https://github.com/openzipkin/brave/blob/master/brave/src/main/java/brave/internal/recorder/PendingSpans.java
I forgot to raise the feature back until now, which completely dodges
classloader issues and the need to use thread locals. If your map key
overrides equals in a way that it allows equality with a weak ref of
itself, you don't need any ceremony for lookup-based operations. This
can be a perfectly valid tradeoff for internal code, especially vs
holding up a classloader.

Sorry, this slipped past me somehow. I will need to run a few benchmarks to see what happens with the cast but all in all, I see the use case. Is this still something you would find useful?

I think what would happen is we could trade our copy for a shaded impl here, reducing the unit tests we need etc. All depends on this library staying Java 6 floor though.. if that changed we'd need to inline it again :P https://github.com/openzipkin/brave/blob/master/brave/src/main/java/brave/internal/collect/WeakConcurrentMap.java

I just looked at this and I don't really understand how supplying a different instance in these methods can make sense. The WeakKey implementation checks for the instance type of the latent key and without using the original latent key, this would stop to work, wouldn't it?

here's our comment on weakkey. ack it is weird :p
 static final class WeakKey<T> extends WeakReference<T> {
    final int hashCode;

    WeakKey(T key, ReferenceQueue<? super T> queue) {
      super(key, queue);
      this.hashCode = key.hashCode(); // cache as hashCode is used for all future operations
    }

    @Override public int hashCode() {
      return hashCode;
    }

    @Override public String toString() {
      T value = get();
      return value != null ? value.toString() : "ClearedReference()";
    }

    /**
     * While a lookup key will invoke equals against this, the visa versa is not true. This method
     * is only used inside the target map to resolve hash code collisions.
     */
    @Override public boolean equals(Object o) { // resolves hash code collisions
      if (o == this) return true;
      assert o instanceof WeakReference : "Bug: unexpected input to equals";
      return equal(get(), ((WeakReference) o).get());
    }
  }
``

Ah, I see. That's quite hacky since it relies on the entry always calling the equals method with itself as an argument. I'll see if I can make that a bit cleaner and if I manage, I'll happily merge the cleaned-up version.

Alright, cleaned this up. Could you test if this actually works out for you?


Support ircv3 features

Would be nice if Convos could support some/all of the features listed on https://ircv3.net/irc/. I'm not jumping on this issue though, since libera.chat (my main network) does not support too many features: https://ircv3.net/support/networks#networks-1 There are some test networks though, that I might use to see what to implement.
Probably need to make some new tasks to cover the different features:

[x] Capability Negotiation: CAP, CAP 302, cap-notify
[x] WebIRC: WEBIRC - #546
[ ] Echo Message: echo-message
[ ] Message IDs: message-ids, msgid
[ ] Monitor: monitor
[ ] Labeled Responses: labeled-response
[ ] Away Notifications: away-notify - #360
[ ] Message Tags
[ ] Userhosts in NAMES: userhost-in-names
[ ] Setname: setname
[ ] Client-Only Tags: react, reply, typing
[ ] Multiple Prefixes: multi-prefix
[x] SASL Authentication: SASL v3.1, SASL v3.2, SNI #356, STS
[ ] Invite Notify: invite-notify
[ ] Account Tracking: account-notify, account-tag, extended-join
[ ] Changing Usernames and Hostnames: chghost
[ ] Server Time: server-time
[ ] Batches: batch

With IRCv3 being adopted more and more, it would be nice if Convos added more IRCv3 support.
Right now it is now even listed on the IRCv3 clients table, see: https://github.com/ircv3/ircv3.github.io/issues/481


Flask-Security-Fork is merged in Flask-Security

Hello! I wanted to circle back on this. Can you rebase with the latest and re-submit this?

Unfortunately, this appears to break the build.

I don't see how is it connected with Flask-Security:
File "/home/travis/build/Netflix/security_monkey/security_monkey/__init__.py", line 246, in <module>
    from security_monkey.sso.views import mod as sso_bp
  File "/home/travis/build/Netflix/security_monkey/security_monkey/sso/views.py", line 458, in <module>
    api.add_resource(AzureAD, '/auth/aad', endpoint='aad')
  File "/home/travis/virtualenv/python2.7.14/lib/python2.7/site-packages/Flask_RESTful-0.3.3-py2.7.egg/flask_restful/__init__.py", line 387, in add_resource
    self._register_view(self.app, resource, *urls, **kwargs)
  File "/home/travis/virtualenv/python2.7.14/lib/python2.7/site-packages/Flask_RESTful-0.3.3-py2.7.egg/flask_restful/__init__.py", line 417, in _register_view
    if endpoint in app.view_functions.keys():
AttributeError: 'Blueprint' object has no attribute 'view_functions'

Could it be related to the upgraded Flask version?

@mikegrima it looks like it has been fixed in Flask-RESTful.

Coverage remained the same at 71.699% when pulling c3aae7f4322c7a91e31c81ea697e7b8aec329313 on jirikuncar:patch-1 into ffd639648ebdc42317a9f2015bb2b5f3818367c1 on Netflix:develop.


CDH 5.10 - Hue 3.11 Spark Notebook Issue

Hi- I'm running CDH 5.10 and enabled Hue Notebooks using safety valve configuration. I also installed Livy 0.3.0 and provided appropriate safety valve config in Cloudera Manager.
It seems like the spark code in the notebook snippet is running on the cluster through Livy, but the results don't get displayed, I instead get a message that "No logs available at this moment". I was able to confirm that the code is running in the cluster ok by curling the previous session statements.
See below and please advise if anyone has run into this problem and resolved it through configuration.

[hdfs@ip-10-179-XXX-XXX logs]$ tail -1 2017_03_03.request.log
10.179.XXX.XXX - - [03/Mar/2017:16:12:31 +0000] "GET /sessions/16/statements/0 HTTP/1.1" 200 -
[hdfs@ip-10-179-XXX-XXX logs]$ curl 10.179.XXX.XXX:8998/sessions/16/statements/0
{"id":0,"state":"available","output":{"status":"ok","execution_count":0,"data":{"text/plain":"res0: Array[Int] = Array(2, 3, 4, 5, 6)"}
Below is the service wide hue_safety_valve config in CM:
[desktop]
allowed_hosts=*
app_blacklist=
[notebook]
show_notebooks=true
[spark]
'# URL of the REST Spark Job Server.
server_url=http://10.179.XXX.XXX:8998/
'# List of available types of snippets
languages='[{"name": "Scala", "type": "scala"},{"name": "Python", "type": "python"},{"name": "Impala SQL", "type": "impala"},{"name": "Hive SQL", "type": "hive"},{"name": "Text", "type": "text"}]'
'# Uncomment to use the YARN mode
livy_server_session_kind=yarn

Hi,
Did you solve this problem ?I ran into exactly the same issue with CDH 5.9.3
Regards,

After some debugging, result are displayed if I had a breakpoint :
      $.post("/notebook/api/check_status", {   <-- HERE
        notebook: ko.mapping.toJSON(notebook.getContext()),
        snippet: ko.mapping.toJSON(self.getContext())
      }, function (data) {}

Then I remove the breakpoint manually.
If i do the previous test  the variable self.status == "available" and result are displayed. I don't see any loading snipper in Hue.
If i don't do that the variable self.status == "waiting" and nothing  happen
Any help would be appreciated
Regards


Would be better to support port and host both

https://github.com/theia-ide/yangster/blob/cd8d38e3024fb41dd77804a1c59089432e27491c/theia-yang-
extension/src/backend/backend-extension.ts#L50
socket.connect(socketPort)
Here in the current implementation, only port is allowed.
I think to support both port and host will be much useful.
Required Socket API
NodeJs Socket.connect API

What is the usecase you envision? The language server needs to see the same file system than the backend.

Aha, I didn't know the designed purpose before.
OK, I can understand that.
Please close this issue then.

ok


onedrive (SharePoint): file integrity check fails because SharePoint modifies specific files

There have been multiple reports that SharePoint modifies files uploaded (mostly Office files), so this issue is opened to track the problem. Currently there seems to be no way to disable this behaviour, but a possible workaround is to set the length of the affected files to -1 and their hash to an empty string so they can be ignored.

Update: There is an issue on OneDrive's side about this: https://github.com/OneDrive/onedrive-api-docs/issues/935. Will keep tracking.

It doesn't appear that microsoft views this as a problem.  I seem to be able to sync (without any verification, of course) using 1.48.0 if I pass --ignore-checksum --ignore-size.  Is there a better way to get past this?

There is a workaround that worked for me described in issue #2036

I find some thing to do about this issue.
This is a bug of SharePoint.
After uploading, some files with certain extensions will be modified, 11-12 bytes longer than the original file.
I try to upload the same file with a extension .xxx, this bug is not triggered.
So maybe we can add a mode flag, uploading files with two steps, upload to a strange name and rename it to the correct name.

A mode that would only cache for downloading but not uploading would probably work as well

I find out what SharePoint Backend is doing with one of my JPEG images.
This JPEG seems to have a bad header information.
The density in JPEG header is 96 x 96.
The density in EXIF(which is JPEG APP1 Part) is 72 x 72.
SharePoint modified the density in JPEG header to 72 x 72, and add 12bytes to EXIF part.
All bytes in EXIF after the EXIF size-bytes is not modified, only 12 bytes appended.
I don't know why, maybe a bug of Microsoft?
It seems there is a filter program in the process of upload API, which is correcting the information of some files.
But if you upload the file with a different extension and rename it to JPEG after uploading, this filter program does not work.
I think this is a bug of SharePoint/OneDrive Business.
It should not write back the corrected information to the uploaded file.

Is vfs cache enabled?

I checked this without rclone.
I just call the Microsoft Graph File API to upload large file directly.

I post this bug in Microsoft Graph Feedback. Can anyone help to vote it ?
https://feedbackportal.microsoft.com/feedback/idea/34e3c738-291c-ee11-a81c-000d3a7a48db

Done

The problem is that Sharepoint modifies some of the uploaded files by itself as soon as they are uploaded. Adding properties. Changing DPI etc. so they have different size and of course checksum.
The solution is to set ParserDisable property of the destination Sharepoint list to True. Not in powershell, not in admin panel, not in Graph API.
You can do it via Sharepoint REST api available at https://contoso.sharepoint.com/sites/abcd/_api/
or use wrapper like the "CLI for Microsoft 365" tool.

@hjastr thank you for that. Do you think that is something we should put in the docs, or is that something rclone should do itself?

In my opinion there should an option when creating new Sharepoint remote in rclone. And check of this attribute in sync and copy command with warning if not set. That would give users best experience.
And imagine all of that time humanity will save on googling "Failed to copy: corrupted on transfer: QuickXorHash crypted hash differ."

@hjastr thank you for that. Do you think that is something we should put in the docs, or is that something rclone should do itself?

I find a way to solve this problem.
update with a safe filename and then rename it.
this problem only happens in a small set of file extensions.
for example,
abc.jpg may be modified but abc.jpg.xxx never modified and no file will be modified after renaming.


Add new Type Ia Super-Chandra model

Type Ia Super-Chandra : This one comes from the study Margot and I did on Super-Chandra models. Mostly based on SNe 2012dn and 2009dc, but influenced by the lightcurve of the best fit model. It was used for all of our bolometric corrections. Reference Fitz Axen and Nugent, (2023) ApJ, vol. 953, p. 13.
Fluxed, time-evolving spectral series.
Photometry V

From https://c3.lbl.gov/nugent/nugent_templates.html

closed with e8a17e4


Move samples & label validation to the distributor.

Move samples & label validation to the distributor - most validation is stateless, so not point in doing it 3 times with all the rpc overhead.  Should move a small amount of load of the ingester.
In doing this we also separate out label validation from sample validation, hopefully reducing the amount of times we have to validate a set of strings.
One downside is that the distributors don't know the CreationGracePeriod, so this flag is duplicated.  These other flags have been deprecated and moved, and will need adding to the distributor during upgrade:

-ingester.reject-old-samples -> -validation.reject-old-samples
-ingester.reject-old-samples.max-age -> -validation.reject-old-samples.max-age
-ingester.validation.max-length-label-name -> -validation.max-length-label-name
-ingester.validation.max-length-label-value -> -validation.max-length-label-value
-ingester.max-label-names-per-series -> -validation.max-label-names-per-series

TODO:

[x] Improve and make consistent all the errors & metrics. (Byran did it!)
[x] Consider not rejecting entire batch if one series / metric fails validation. (Done - this is preserving the ingester behaviour)

Thanks!  Should all be addressed now.

I think, when upgrading a running system to include this change, you should ensure the distributor is fully upgraded before starting the ingester rollout.

Upgrade instructions should mention -validation.create-grace-period


feat(range): provide an example to display the value

Description
This PR provides a new example in the input range documentation to show how to display the value as a label for better usability.
Regarding a11y, here is how I defined the HTML structure and attributes:

Avoided a double  and chose a
Tested with VoiceOver, the value was already announced so I chose to aria-hidden="true" it to avoid a double announce for the same value.
Finally used what's provided by a11y expert described in https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/pull/1714#issuecomment-1377055559

[x] Check the rendering compared to the ODS design guidelines
[x] Auto-check HTML markup for a11y (will need a real a11y review after that)
[ ] Enhance documentation (split HTML code and JS?)

Types of change

New feature (non-breaking change which adds functionality)

Live previews

https://deploy-preview-1714--boosted.netlify.app/docs/5.3/forms/range/#usability

Checklist

[x] I have read the contributing guidelines
[x] My code follows the code style of the project (using npm run lint)
[x] My change introduces changes to the documentation
[x] My change introduces changes to the migration guide
[x] I have updated the documentation accordingly
(NA) I have added tests to cover my changes
[x] All new and existing tests passed

after some checkings, i think that we should use an output tag, as follow:

  Example range
  <output id="customRange4ValueText" for="customRange4" class="fw-bold">

<input type="range" class="form-range" min="0" max="100" step="1" id="customRange4" onchange="updateLabelValue()">

it's seems to work well with screen readers

after some checkings, i think that we should use an output tag, as follow:

  Example range
  <output id="customRange4ValueText" for="customRange4" class="fw-bold">

<input type="range" class="form-range" min="0" max="100" step="1" id="customRange4" onchange="updateLabelValue()">

it's seems to work well with screen readers

It confirms what we talked about with @louismaximepiton in private. Thanks for the feedback @Aniort. I'll make some changes to go with it!

@Aniort, this PR's ready for final a11y review based on https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/pull/1714#issuecomment-1377055559. Should be quick to double-check 😄

to avoid a double vocalization with both screen readers Jaws & NVDA please add ariahidden to output tag:
xx
and it'll be ok !


Policy for restricting clusterroles for CSR's

…tests, chainsaw tests and artifacthub-pkg.yaml
Related Issue(s)

Description

Checklist

[x] I have read the policy contribution guidelines.
[x] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
[x] I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

@chipzoller Please review this PR and let me know if any updates are needed.

Hi @chipzoller I have updated the files. Please check.


Don't lock overrides if we're updating them

There was already a function for this, keep, it was just forgotten to be
called.
Closes #2766

r? @wycats
(rust_highfive has picked a reviewer for you, use r? to override)

r? @brson

@bors r+

:pushpin: Commit feccad0 has been approved by brson

:hourglass: Testing commit feccad0 with merge 26cb07b...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


PR to make some path problems

(Put an X inside the [ ] to denote check mark [X].)

[x] If creating a new file :

[x] added links to it in the README files ?
[x] included tests with it ?
[x] added description (overview of algorithm, time and space compleixty, and possible edge case) in docstrings ?

[x] if done some changes :

[x] wrote short description in the PR explaining what the changes do ?
[x] Fixes #277

[ ] other

@goswami-rahul I have updated it.


Tables with OIDs > 2^32-1 cannot be registered

Table registration fails for some tables that have OIDs greater than<PHONE_NUMBER> (the largest positive 4-byte signed integer, 2^31-1).
According to https://www.postgresql.org/docs/11/datatype-oid.html

The oid type is currently implemented as an unsigned four-byte integer.

But the table_id column of user_tables, where the OIDs are stored, is an integer (4 bytes, signed).
This is affecting customers now. We probably haven't seen it until now because you need to create lots of tables for Postgres to get to use so large OIDs, but there are cases with many sync tables where this is happening (because each synchronization replaces the table, so uses up a new OID).
We need to modify the metadata schema to fix this. Since Postgres doesn't have unsigned integers, I think the best solution would be to change the type of table_id to the type oid so that we don't increase the size unnecessarily.
@alrocar This comes from RT, but I'd to have some backender to look at this, because I'm not sure what implictations could have this change.

I'm reopening this: After changing the table_id column to oid type it now accepts any valid OID value, but ActiveRecord still doesn't (it validates the column as a signed integer)
Carto::UserTable.columns.find{|c| c.name=='table_id'}.type
=> :integer

It seems the problem does not come from validation: I've been able to save a UserTable with a table_id of<PHONE_NUMBER> (tampering with the service); I'm checking it.

OK, this is what's going on (it's the cast type of the column):
c = Carto::UserTable.columns.find{|c| c.name=='table_id'}
c.type_cast_for_database(2164557046) # RangeError:<PHONE_NUMBER> is out of range for ...
class OidType < ActiveRecord::Type::Integer
  def max_value
    (1 << 32) - 1
  end
  def min_value
    0
  end
end
c.instance_variable_set '@cast_type',  OidType.new
c.type_cast_for_database(2164557046) # =><PHONE_NUMBER>

Now, we just need to found a less hacky way of replacing the type cast of the column...


Added script for smooth movement of a gameobject

Updated comments on the PlayerMovement script

personally i'd prefer not to put new projectsettings/ and packages/ in.. i can merge if you remove those.
or what do you think? @EmpireWorld  @Sacristan

Sure, No problem. I'll remove them and update the repository.
On Sat, Jul 18, 2020 at 6:12 PM mika<EMAIL_ADDRESS>wrote:

personally i'd prefer not to put new projectsettings/ and packages/ in.. i
can merge if you remove those.
or what do you think? @EmpireWorld https://github.com/EmpireWorld
@Sacristan https://github.com/Sacristan
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/UnityCommunity/UnityLibrary/pull/44#issuecomment-660477571,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJN4EPZDEJNVZU3C5ARMJZDR4GKC3ANCNFSM4O5QL76A
.


pyramid_basemodel is incompatible with zope.sqlalchemy 1.2

https://github.com/zopefoundation/zope.sqlalchemy/blob/master/CHANGES.rst
That's due to
To make things clearer we renamed the ZopeTransactionExtension class to ZopeTransactionEvents. Existing code using the 'register' version stays compatible.

pr #14 seems to be fixing this issue

@thruflo thanks for merging #14 with a fix. Any chance on releasing this fix to pypi?
Also, would you like any help plugging CI and possibly dependabot to automatically run tests against newer dependency versions?

I'm conscious I'm not active with this any more -- would you be interested in taking over maintaining? There's https://github.com/thruflo/pyramid_basemodel and also https://github.com/thruflo/pyramid_simpleauth

I would be interested, but just in the pyramid_basemodel, not the pyramid_simpleauth. Or not yet, since I already maintain an auth extension for pyramid (pyramid_fullauth).
There's a chance I'll use pyramid_basemodel at work, which will give me an opportunity to direct a bit of the effort to the pyramid_basemodel itself.

OK, great.  I'm happy to transfer repo and I can add you on PyPI for the package?
On simpleauth, I'd also be happy to flag up in the readme that fullauth may be a better option if you're actively maintaining it?

Hmmm.... You can flag up, with fullauth, link to https://trypyramid.com/extending-pyramid.html and call for a maintainer (in case someone would want to use simpleauth anyway).
My pypi user is https://pypi.org/user/fizyk/

Thanks for taking this on :)


I want to apply another theme

Thank you for your convenient projects.
I'm wondering how to apply another jekyll theme. (I want lanyon.)
But, in my opinion, it's not compatiable.
Is there any solutions?

it is all just html and MD... see my use of the system http://churchcrm.io/www2/

@DawoudIO  Hi George it says file not found.

http://churchcrm.io

Src code via https://github.com/ChurchCRM/ChurchCRM.github.io

Can you explain how you managed to change the theme?

Just different layouts
https://github.com/ChurchCRM/ChurchCRM.github.io/tree/master/_layouts

That means you only copy the layout files from any theme you wish you apply, and that's it?

@DawoudIO thx Mr George so cutting and pasting those layouts above should auto change the theme?


Advance throttling --Request count does not throttle

Description:
Note : This is working some users but for me it did not. Below are user credentials
Should check on PizzaShack-damith :1.0.0 api
u.n - damith
p.w - damith@123
you can check this with hosted distributed setup. Tested with two throttle policies created

damithReqCountPolicy
dnwickThrottle1

Steps to reproduce:
Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Reduced the priority because the issue is not always reproducible.

Hi damith, I noticed that the request count used in. the policies is 5 &. 3 respectively. I assume that the reason you did not experience a throttle out as soon as you reached limit could be that the request count is too low,
since we have something like a periodic timer which checks if the api is throttle out, by the time you reach the count the timer has not yet market it as throttled out. You can try testing this scenario again with high values such as 15 or 20,

What is the timewindow used here?
5 req per 1 min and 3 req per 1 min?

yes its 5 req per min and 3 req per min

Hmm, the request should be throttled out at least after some requests even though it is greater than 3 or 5..
@dnwick did we create a revision and deploy the API after adding these policy changes?
Seems like the API already deleted in the distributed setup now..

Yes did several times. Anyways others could throttle out. If this is not reproducible lets park it for now


Order ahead

whhhaaaaaaaa?!?!?!?!?

Some eateries support ordering ahead. This information is given to us in the JSON dump from CIT. https://now.dining.cornell.edu/api/1.0/dining/eateries.json


Added click listener to the logo

fixes: #66
Description:
Initially there was not closing functionality in the logo.
Added Click Listener to close the sidebar on clicking the logo.

@ghozt777 Great work!
PR merged


send read side ID in read side request

Is your feature request related to a problem? Please describe.
n/a
Describe the solution you'd like
It would be nice to have access to the read side ID inside the read side request proto. for context, a current team is considering reusing the same image/container to serve many read sides with separate offsets. while this might not be a great design for all use cases, IMO makes sense to have access to the read side ID.
@Tochemey we had originally given access via headers to some other read side details (sequence number, tag, etc), so perhaps we could move these into the request proto.
Describe alternatives you've considered
send as a header
Additional context
n/a

The read side ID is the identifier of an actor within the cluster. We can circle back on this. I just want to let all of us know that the ID is like the actor persistent ID.

The read side ID is the identifier of an actor within the cluster. We can circle back on this. I just want to let all of us know that the ID is like the actor persistent ID.

@Tochemey I was referring to the projection_name


tensorflow.examples.tutorials.mnist not currently downloadable

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
from tensorflow.examples.tutorials.mnist import input_data
mnist = input_data.read_data_sets("/tmp/data/", one_hot=True)

raises http.client.RemoteDisconnected: Remote end closed connection without response, if you don't already have the data cached. I believe this is because the underlying data SOURCE_URL is currently down.

...It's back up again now. Looks like it was just a temporary disruption.

I encountered the same problem just now.  how do I fix it?

Using the same code:
from tensorflow.examples.tutorials.mnist import input_data
mnist = input_data.read_data_sets("/tmp/data/", one_hot=True)
raises a timeout error: TimeoutError: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
What would cause this?

The issue is that the website where MNIST is canonically hosted sometimes goes down. I think an ideal solution would be to find or set up a mirror, then fall back to it? Or just find a more stable source.

I also have the 10060 problem and it always happened. This seems like not the host problem. Do you know what caused this?

Reopening because others seem to have this problem.

I would suggest that if you encounter this problem, check http://downforeveryoneorjustme.com/http://yann.lecun.com/exdb/mnist/ just to make sure.

I cannot reach this website

Confirmed. I'm also having MNIST issues. Is there any alternative URL we can use for this which will still be compatible with existing MNIST tutorials. I'm relying on these as baselines for experiments, I'd rather keep the data and TF format the same.

Still down seems few days... Other related issue #8116

Same project here, solved by downloading file from a different source. I think we should put in place a mirror somewhere.

Also cannot access.

Closing as duplicate of https://github.com/tensorflow/tensorflow/issues/8126

@auserdude If you retrieve the .gz files from any other source (wayback machine?) and drop them in your data_dir then TF will just use them.  It does not validate a checksum AFAIK.


Evidence of exception driven control flow in bouncy castle TLS connections

See write up below by one of our engineers:
Using Yourkit, I can demonstrate an excessive amount of exceptions are created surrounding the normal function of TLS connections between Niagara Workbench and the Niagara Daemon.
See the following top offenders in Workbench:

See the following top offenders in Niagara Daemon:

This the output of a functioning TLS connection(s) between Niagara Workbench and Niagara Daemon with 5 instances of the Platform Administration View open to the local Niagara Daemon.
The counts associated with:
java.security.spec.InvalidParameterSpecException
java.lang.IllegalStateException
org.bouncycastle.crypto.InvalidCipherTextException
javax.crypto.AEADBadTagException

Will increase indefinitely as long as the connections remain open.
Exceptions are expensive
Control flow using exceptions is an anti-pattern
We should see if the exception generation can be reduced as much as possible to encourage performance of TLS connections.
See a possible solution to submit to BC team to sanity test the first character of a string as a valid IPv4 / IPv6 character before falling back to the exception driven design:
D:\bc-java>git diff
diff --git a/core/src/main/java/org/bouncycastle/util/IPAddress.java b/core/src/main/java/org/bouncycastle/util/IPAddress.java
index 8f57263f4..9877ff295 100644
--- a/core/src/main/java/org/bouncycastle/util/IPAddress.java
+++ b/core/src/main/java/org/bouncycastle/util/IPAddress.java
@@ -45,6 +45,10 @@ public static boolean isValidIPv4(
         {
             return false;
         }
+        if (Character.digit((int)address.charAt(0), 10) < 0)
+        {
+            return false;
+        }
         int octets = 0;
@@ -127,6 +131,11 @@ public static boolean isValidIPv6(
         {
             return false;
         }
+        char firstChar = address.charAt(0);
+        if (Character.digit(firstChar != ':' && (int)firstChar, 16) < 0)
+        {
+            return false;
+        }
         int octets = 0;

Further evidence of exceptions in 1.70 BC provider with TLS connection to the Niagara Daemon:

Thanks, I've now overhauled the IPAddress class to remove all Integer.parseInt calls. Expect a new beta release to test in the next few days.

Thx Peter!

New beta is up at https://downloads.bouncycastle.org/betas/ .

@bsmith-tridium-com Were you able to test the beta and compare to your earlier results?


Instructions

The readme or instructions for 3.0 beta is not staying caught up with all the latest changes that's happening every 2.5 minutes,  prior to this moment here I have been testing the heck out of this thing for @daveismyname  but without the instructions Being caught up as changes are made how can I get it going properly and continue testing? People people slow down if you make some change in your pull request also update the readme at least put something in there for crying out loud you getting out of hand. Just a quick note about the change would be nice but put it in the readme.md real quick.

@daveismyname in case you didn't see it in the other issue, is the way of using the database helper completely gone now, is there a whole new way to have to set up and use databasing? Will eloquent still work with SMVC?

@daveismyname  why is the code from database helper in the framework twice now.  And are you seriously going to keep that new database stuff in there when already if people did not want to use the database helper they could use eloquent right now, you even did a video on that.

In the current branch you can still use the Smvc\Helpers\Database. But it's recommended to move away from it as I think it will be removed. (And I hope so).
About the documentation, I agree we should update it, but I did wrote some instructions inside of the closed pull requests.
You should take a look there:
https://github.com/simple-mvc-framework/framework/pull/241
https://github.com/simple-mvc-framework/framework/pull/243
Why not provide multiple options, and stay open for the developer too choose the one he wants....

there's been a lot of activity today I'm at work so cannot fully test it but I DO PLAN TO test these, I've said in the readme the instructions will be updated once these changes are finalised to make it easier for you once have all the information.

@tomvlk  the database helper will be replaced with what?

@jimgwhit  With our own DBAL written by @tomvlk ?

@daveismyname I know I have no say so, but I beg you to please slow down that service stuff should have nothing to do with databases at all. Look at how laravel uses the query builder, services has nothing to do with that. Services are supposed to be specialized classes. I  already gave a good example of how laravel uses service classes did people just not to understand that?
I think @tomvlk means well, but we would be better off just using eloquent and the query builder if @tomvlk wants to kill off the database helper which I like using. The key to the framework was the easy to use database helper by Killing that Dave you will killing the framework.

What was wrong with Dave's DBAL?

@jimgwhit No one kill your beloved Database, it is just somewhat enhanced, from what I seen.
You can use, as always, your favorite coding style, you just have to call the Database in a different way. And while this thing is done in \Smvc\Core\Model, you can practically ignore the changes.

In previous comment you said replaced.

I think it will be replaced before 3.0 release, or at least that would be good to prevent it from not working anymore after some changes.
And just because 3.0 is a major new version with breaking changes..

@jimgwhit While right now you use:
$database = new \Smvc\Helpers\Database::get();

In future you have to write:
$database = \Smvc\Core\Database\EngineFactory::getEngine();

That's all! From there you use your style coding as always.

Where does this line go
$database = \Smvc\Core\Database\EngineFactory::getEngine();

@jimgwhit Right what \Helpers\Database::get() used to do...

@jimgwhit please try to read and understand others comments before you get upset about what is coming. You are creating alot of noice over what is not a big deal as lucky has pointed out a number of times.


version for deploy target 12.x?

Hi,
is there an older version that will play nicely with RN 71.x?
Thank you for your help!
U.

fixed in latest release.


how to visualize

Thank you very much for your work, I would like to ask how to visualize

Thanks for your attention to our work. You can try Open3D to visualize segmented point clouds.


How to implement multiple action buttons?

This is the desired notification:

Knock's landing page has something like this:

How can we achieve this?

Thanks!


Add posh-dotnet (PowerShell tab completion for the dotnet CLI)

CLI/posh-dotnet - posh-dotnet
PowerShell tab completion for the dotnet CLI.
This module enhances the tab completion of the dotnet cli 2.0 (which is disabled by default) and also provides support for the dotnet cli 1.0 (which had not tab completion at all)

It seems to me that CLI section is not a good place for it. In that section we have libraries to build CLI-based programs. posh-dotnet is a tool (or extension to PowerShell) to help us running commands, I'd rather put it in the Tools section


Where is https://timefold.ai/model/job-shop-scheduling/

Run Model from the Planning Models leads to https://github.com/TimefoldAI/timefold-quickstarts/tree/stable/use-cases/food-packaging not JSSP/Job Shop Scheduling Problem
Where to find the example?

Food packaging is a variation on job shop scheduling  . We currently don't have a dedicated quickstart.

I'm closing this issue, as we have an answer.
Next time, please ask a question or start a discussion, both of which are options on the "New issue dialog".


[REVIEW]

GitHub Profile Link
https://github.com/octonawish-akcodes

Looks great! Some suggestions from the live stream https://www.youtube.com/watch?v=UYiuVStI85M

suggest trying to organise the badges differently - try columns
maybe try the banner at the top

Thanks @eddiejaoude I will work on that.


tested Settings component to verify text is rendering'

Description
Tested the Settings component to ensure the text in an h2 was rendering
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[x] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[x] Test A
[ ] Test B

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

Needs to be reviewed.


Error: Fixable rules must set the meta.fixable property to "code" or "whitespace"

Oops! Something went wrong! :(
ESLint: 8.19.0
Error: Fixable rules must set the meta.fixable property to "code" or "whitespace".
Occurred while linting /Users/Projects/test/src/app/main.ts:1
Rule: "no-comments/disallowComments"
at Object.report (/Users/Projects/test/node_modules/eslint/lib/linter/linter.js:1088:35)
at processComment (/Users/Projects/test/node_modules/eslint-plugin-no-comments/index.js:9:21)
at Array.forEach ()
at Program (/Users/steve/Projects/test/node_modules/eslint-plugin-no-comments/index.js:22:22)
at ruleErrorHandler (/Users/Projects/test/node_modules/eslint/lib/linter/linter.js:1114:28)
at /Users/Projects/test/node_modules/eslint/lib/linter/safe-emitter.js:45:58
at Array.forEach ()
at Object.emit (/Users/Projects/test/node_modules/eslint/lib/linter/safe-emitter.js:45:38)
at NodeEventGenerator.applySelector (/Users/Projects/test/node_modules/eslint/lib/linter/node-event-generator.js:297:26)
at NodeEventGenerator.applySelectors (/Users/Projects/test/node_modules/eslint/lib/linter/node-event-generator.js:326:22)

I think the eslint update could cause it. I will fix the code and docs this week. Thanks for reporting 👍

So I have come back to this issue and I cannot reproduce it however, I added fixable property anyway. Let me know if the issue still occurs


Open options in the context menu for EXEs and MSIs don't work

The Open options for an EXE or MSI file don't do anything
When I double click to or use the context menu to open an Executable file or a Microsoft Installer package, it does nothing
To Reproduce
Steps to reproduce the behaviour:

Go to 'Any EXE or MSI file'
Double Click any "MSI" or "EXE" file OR Click on 'Any Open Option in the Context Menu (Open, Run as Another User, Run as Administrator)'
See error: (nothing happens)

Expected behaviour
I expected the file to Open, Run as Another User or Run as Administrator, depending on what I chose from the Context Menu
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS Version: Windows 10 Home Insider Preview Build 21332.rs_prerelease.210306-1859
App version: v<IP_ADDRESS>

Additional context
The Problem Persists After uninstalling and reinstalling the app
Log file
debug.txt

Known issue with the newer insider builds. This is probably for microsoft to fix.
In order to confirm the issue, could you post a screenshot of the right click context menu of Files? Thanks!

Edit: I'm using a Windows 10 Pro Preview, and it Doesn't work for any file.

Thanks for this report. I can confirm this is fixed on the development branch and a fix will be included with the next public release.


Sharing xcscheme for tvOS.

Not sure why this scheme isn't shared; noticed it when tvOS framework was missing with Carthage. Hope this helps.

:+1: thanks, good find!

No problem 🙂