Support for tablock hint

Hello everyone, we are very happy with the evolution of each version of babelfish. The only problem we still have is the conversion of large bases. It's always a challenge. Would it be possible to implement tablock so as not to give the error in fullconvert and we can use this tool that is already consolity?. If not, we can try to implement it for you, we just need hints where to change in the code. Thank you for your attention. Unfortunately, the tool manufacturer is not interested in the support.

Hello
I am very interested in having a discussion with you on this. Should we have it here, through regular mail or in a meeting? You can reach me at amazon.com as sksql.
Cheers,
/Stefan

Hello Sefan, thank you very much for your feedback. I'll send you a message on your LinkedIn with my contacts so we can talk.
Cheers,
Alexandre

Hi, I've managed to reproduce the error using the following code:
create database [testdb]
USE [testdb]
GO

SET ANSI_NULLS ON
GO

SET QUOTED_IDENTIFIER ON
GO

CREATE TABLE [dbo].[mytable]([r_e_c_n_o_] [int] IDENTITY(1,1) NOT NULL CONSTRAINT [primarykey557] PRIMARY KEY CLUSTERED
(
	[r_e_c_n_o_] ASC
) WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON, OPTIMIZE_FOR_SEQUENTIAL_KEY = OFF) ON [PRIMARY]
) ON [PRIMARY]
GO

-- The following query works properly with TABLOCK
select top 5 * FROM dbo.[mytable] with (TABLOCK)

-- However, the following bulk insert does not work with "WITH (KEEP_NULLS, TABLOCK)".
insert bulk dbo.[mytable] ([r_e_c_n_o_] Int) with (KEEP_NULLS, TABLOCK)

All commands work as expected except for the INSERT BULK query, which returns the following error:
syntax error near 'TABLOCK' at line 22 and character position 63


add a data template for creating a course content map

Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
this is part of #279
What's this PR do?
This PR adds a new json template which renders a map from UUID to relative link for all pages. We need this because for the resource collections we're storing the site name and the UUID for pieces of content. With this map we'll be able to get complete URLs to request the data.json file for each piece of content.
How should this be manually tested?
Check out the rendered course. There should be a json file at /content_map.json that looks like what I described.
Note that you'll need to check out this PR for ocw-hugo-projects for it to work correctly (for the course theme Hugo config) https://github.com/mitodl/ocw-hugo-projects/pull/119

Can you give an example of a rendered content_map.json ?

@pdpinch yes, it will look something like this:
{
  "013ea582-8958-24c8-61df-cd7bf1e6526e": "/resources/mit18_s997f11_exercise_17/data.json",
  "01d3a55d-eacc-1672-2d62-09afbfd9ee44": "/resources/mit18_s997f11_exercise_2/data.json",
  "01e581a8-1cba-3b53-0d29-7eb44635a452": "/resources/lecture-3-using-files/data.json",
  "01f41609-eedf-c2d7-11bd-02593bab20d4": "/resources/18-s997f11_unit4_img2/data.json",
  "0523856e-c9fc-9e71-66c9-d2d9aecf63e3": "/pages/vectorization/complex-numbers/data.json",
  "072d4e9f-65ad-773a-5386-2ee2572818f7": "/resources/18-s997f11_unit3_img4/data.json",
  "08cfb41f-1f5e-b1e9-bb10-ba9eae5b1b45": "/pages/library/projects/data.json"
}

this file is rendered for each course, and will be accessible at $COURSE_BASE_URL/content_map.json. It basically just maps the UUID for every piece of content (which is how we can uniquely identify a piece of content on the studio side) to the URL for that piece of content's data.json file.


Feature/add content type rendering

I think most of this is fairly straight forward, the parts that need some explanation are:

The memoisation - I found the webhooks seem pretty undocumented but I speculate that urls for an attachment are unique and so I memoised the filetype lookups to avoid repeated calls to the same file

Use of python-magic - again, not knowing much about how Trello handles attachments I have found in the past that the simplest case of just checking extension is not reliable. Checking the content-type the server responds with normally works but since it is possible for a server to lie about this (although I'm sure that s3 doesn't) I prefer to test the file itself to be sure

Closing without merging (as that would make the test redundant for anyone else ;-))


How to choose a suitable camera for research?

Hi!  autoware team,
I have been using Autoware developed by you recently. I am so grateful to you for developing Autoware. I am a student now and I am going to use Autoware to do research. Now I have a problem with choosing cameras. Could you recommend me some suitable cameras, including resolution, fps and so on，please？Thank you very much.

Ubuntu version:16.04
ROS version:1.12.14
Autoware branch: master

This really depends to your needs.
Autoware supports several cameras as you might know.

FLIR, PTGrey
AVT
Baumer
IDG
Following your camera needs, (resolution, fps) you'll also need to select a suitable lens for the application you are pursuing (wide, tele, or by fov etc).


Document data model modification process

Describe adding/removing objects/properties

@therebelrobot I added this page https://github.com/sfbrigade/brigadehub/wiki/Modifying-The-Data-Model I need to look up how indexes in MongoDb works so we can have efficient queries.


pkg/operator: fix flag typo

Added the node exporter manifests here for testing purposes and so they won't pollute the Alertmanager PR. Let me know if you think that's ok. The actual fix is in pkg/operator/petset.go.
@fabxc

:+1:


faust: 2.54.9 -> 2.59.5

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

nixpkgs-review revealed no new failures on macos 10.15 & nixos x86_64 before the last push, haven't tried again..

@risicle Thanks for testing!

2.59.5 was a pre-release, 2.59.6 is a proper release.

on nixos x86_64 nixpkgs-review failed in faustStk i am not sure how it is related to your patch. But it seems that it duplicates parts of Faust. https://github.com/NixOS/nixpkgs/blob/8fcd9a317391dffeb4475f2f8810fd2b7c217ba4/pkgs/applications/audio/faustStk/default.nix#L4 also provides faustPhysicalModeling but with a spelling mistake faustPhhysicalModeling.

@olafklingt Thanks for testing!
faustStk is a duplicate package, I have removed it in another PR: https://github.com/NixOS/nixpkgs/pull/215561/commits/5c559bd0656b96ed651814fc2fc24b054a053cdf


Clarify that remote SSH access is disabled by default and expand upon…

… the details required to set the SSH TCP port field to enable remote SSH access

Hi @deddc23efb,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


WIP: Update network-mpi.c

Dan's Version of network-mpi.c with queue
Dan, I'm creating a pull request so I can add in comments/feedback to specific lines of your code.

If this merge represents a feature addition to ROSS, the following items must be completed before the branch will be merged:

[ ] Document the feature on the blog (See the Contributing guide in the gh-pages branch).
Include a link to your blog post in the Pull Request.
[ ] One or more TravisCI tests should be created (and they should pass)
[ ] Through the TravisCI tests, coverage should increase
[ ] Test with CODES to ensure everything continues to work

@yaciud is this still a work in progress? Or, is this branch in some final state that works for you?

I consider it a work in progress, but progress on it has stalled. In quick succession the May update to ROSS happened and the clusters I had access to got wiped or had a malfunction, so my code became outdated and it became difficult to test. I believe I have versions that work with the current version of ROSS, but there are still remnants from the outdated MPI layer in there that I'd like to get rid of before considering the work in its final stage.
Also by versions I mean that I have a 3 different ways of solving the problem written and working. I'd like to optimize 2 of them a little bit more, but they all seem to be improvements over the original network layer. It would probably take less than 2 days to get everything ready if needed.


docs - add info on working group members and meetings

Added:

working group members
meetings

meeting presentations
forthcoming meetings

links to mdip
cute bus as logo

🙏.
(Btw The multiple checks were on purpose...but fine with making consistent 🤙)

(Btw The multiple checks were on purpose...but fine with making consistent 🤙)

Ah, apologies, I guess I misinterpreted. Feel free to put them back!


Feature Request - Add tooltip to nodes

Would be nice to have a tooltip option on the node similar to the line. This would be an additional tooltip, not a replacement for the line tooltip.

@kurkle I would love to contribute this feature, but after many many hours of trying to implement it I am really struggling to get this working. I have tried making a node controller that draws the Nodes as an independent dataset which got me close, but it wasn't fully working and was a hacky implementation. I would love some architectural guidance from you on how you think this should be implemented. Would to happy to go forth with some help and get this in to the repo. Hope you are having a great holiday, and thanks for the great library!

I will be doing some work with this in the upcoming weeks and I think I need a tooltip for nodes too. So stay tuned. I can't give any valid advice from the top of my head, sorry!

In that case I will stay tuned! If you need an extra hand, I am happy to help where I can. Have a good holiday!

Looks like its more like "upcoming years", lol. I hope I find the time some day.

Hi @kurkle!
Thank you for this awesome library!
I really need these tooltips on the nodes, do you think you can find some time to implement this feature?


dts-gen vs tsc -d, --declaration

The two commands seem the same. What are the differences and which one should I use?

dts-gen works differently from TypeScript, Flowtype, Tern, etc. because it simply examines the objects as they appear at runtime, rather than needing the source code that creates the object. This means no matter how the object was written, anything, including native objects, can be given an inferred shape.
This trade-off comes with a price -- you'll see a lot of anys in function parameters and return types. You may also see properties that are not intended for public use. dts-gen is meant to be a starting point for writing a high-quality definition file.

Now I get it, tsc won't generate header for normal js packages. Yet, is there a way to generate type definitions with dts-gen for packages locally installed? Seems that is not supported.

@huashiyiqike  use npm | yarn link


Chemical reactions can occur in disposal tubes mid-flight.

Description
This was originally reported by Raven#1990 on Discord. I have verified it on a local environment updated to master as of around 12:30 AM on 11/09/2021.
How to reproduce:

Get two beakers filled with chemicals that react (iron foam is great for this, but potassium + water works too)
Get in a disposal unit and flush yourself.
While on your journey through disposals, put beaker one into beaker two.
The chemicals will react without harming you, even if they're something that normally would (potassium explosion).

https://user-images.githubusercontent.com/7907891/140946595-3bd26600-c564-488a-9027-c3c380b21844.mp4

not harming you seems like the real bug, otherwise this is kinda fine


Help needed to handle performance issue v3.x

Hello, im setting up Modsecurity for my WebApp in a Kubernetes cluster using the docker image owasp/modsecurity-crs:3.3.2-nginx.
Im not done handling false positives actually, i work on rules for that and im facing some performance issues on some endpoints.
I have an Api working with json payloads, some endpoints can process singles complexe objects and some endpoints can process list of complex objects.
In some case the json payload may have more than 8k-10k args.
I've made a rules to check the numbers of args :
SecRule &ARGS "@ge 2000" "id:110100,phase:2,pass,t:none,log,auditlog,msg:'Le flux contient beaucoup de parametres (2000+)',severity:'CRITICAL'"
The fact is that a request that take 200ms when the engine is off, can take up to 10s when the engine is on.
I have tried to use the directive SecArgumentsLimit but the request cannot be processed because the payload seems to be broken
Is there a way to tell the engine to stop processing rules when an amount a time has allready been used ? it's not perfect because the whole crs rules may not have been checked. I could accept that a request normally processed in 200ms can take up to 600ms/800ms or even 1s in the worst case, but not 10s like actually.
What would be the best (or the less worst) solution to resolve that case and keeping a certain security level with acceptable performance ?
Thx for your help
My configuration is
SecRuleEngine On
SecRequestBodyAccess On
SecRequestBodyLimit 13107200
SecRequestBodyNoFilesLimit 131072 #not working with v3.x
SecRequestBodyLimitAction ProcessPartial
#SecArgumentsLimit 2000
SecPcreMatchLimit 1000
SecPcreMatchLimitRecursion 1000
SecResponseBodyAccess On
SecResponseBodyMimeType text/plain text/html text/xml application/json
SecResponseBodyLimit 131072
SecResponseBodyLimitAction ProcessPartial
SecTmpDir /tmp/
SecDataDir /tmp/
SecArgumentSeparator &
SecCookieFormat 0
SecUnicodeMapFile unicode.mapping 20127
SecStatusEngine On
Logging
SecAuditEngine RelevantOnly
SecAuditLogType HTTPS
SecAuditLogFormat JSON
SecAuditLog elk_url
SecAuditLogRelevantStatus "^(?:5(?!02)|4(?!04))"
SecAuditLogParts ABCHIJKZ

Hello @Fillonj ,
There is an inherent challenge with a very large number of arguments run against a substantial set of rules that test them. There doesn't need to be a major issue with any particular rule -- the mere volume of this M x N use case can degrade performance. This is exactly the sort of situation that motivated the creation of SecArgumentsLimit.
In your case, with a very large number of json args being legitimate, that particular configuration item is, of course, less helpful.
There is no control within ModSecurity to stop running rules after a given amount of dwell time.
Your strategy of reducing the rule execution when these very-numerous-args situations are encountered is probably reasonable.
You could explore implementing a rule that tests for this situation and then does something like remove lower-priority rules in its actions. For example, using ctl:ruleRemoveById.
Some additional things to keep in mind:

if you are running that sort of rule in phase:2, you'll want to make sure that it runs before the phase:2 rules that you wish to remove or modify
some rules impacting performance may run against numerous targets. In such cases you might want to leave the rule still running but only against the non-problem targets (e.g. REQUEST_HEADERS) and use ctl:ruleRemoveTargetById instead to accomplish this.
some rules might examine ARGS and/or ARGS_NAMES. If you use the previous item to remove targets from a rule, be aware that the rule in question will also no longer examine query args. If you care, you could probably address this use case by first having a global SecRuleUpdateTargetById to explicitly add ARGS_GET and/or ARGS_GET_NAMES and then remove ARGS and/or ARGS_NAMES with the ctl: action.

Just some initial thoughts. I hope they are of some assistance.

Hi @Fillonj ,
Something like that set of rules is another option. Just be aware that in ModSecurity v3, ctl:requestBodyAccess=Off currently behaves in a non-intuitive manner. Specifically, it will turn off processing of all phase:2 rules. One consequence of this is that phase:2 rules that examine ARGS will stop running against both arguments populated from the body and query args.


Issue311

This PR is to fix the issue #311
Thi issue can be fix by setting the PlotOption plotlyfig called TriangulatePatch to true when we run fig2plotly. Next line please

fig2plotly(gcf, 'offline', false, 'TriangulatePatch', true);

Still having the same output, even with that call:
load wind
xmin = min(x(:));
xmax = max(x(:));
ymin = min(y(:));
ymax = max(y(:));
zmin = min(z(:));
xrange = linspace(xmin,xmax,8);
yrange = linspace(ymin,ymax,8);
zrange = 3:4:15;
[cx,cy,cz] = meshgrid(xrange,yrange,zrange);
coneplot(x,y,z,u,v,w,cx,cy,cz,5);

fig2plotly(gcf, 'offline', false, 'TriangulatePatch', true);

Still having the same output, even with that call:
load wind
xmin = min(x(:));
xmax = max(x(:));
ymin = min(y(:));
ymax = max(y(:));
zmin = min(z(:));
xrange = linspace(xmin,xmax,8);
yrange = linspace(ymin,ymax,8);
zrange = 3:4:15;
[cx,cy,cz] = meshgrid(xrange,yrange,zrange);
coneplot(x,y,z,u,v,w,cx,cy,cz,5);

fig2plotly(gcf, 'offline', false, 'TriangulatePatch', true);

Dear @xarico10 are you sure that you update the repo with the changes?
Please make sure plotlyfig.m file have the following lines from line #196


Data issue in Paper Table 7 .

Thank you for the inspiring paper in FSCIL!
When reading your paper, it seems that the experimental result of session 4 on CUB200 (Table 7 in the appendix, SoftNet, c = 93% ) is a little misleading. Is it a typo? Or it's the actual results?

Sorry for that point. It seems to be a typo: 67.27 to 62.27.


Bad request replied upon request of a voluntary exit message

On a web3signer functioning correctly (i.e signing correctly in combination with Teku on a testnet beacon chain) with web3signer version: web3signer/v20.11.0+9-g6e71e7b/linux-x86_64/-na-openjdk64bitservervm-java-11 (docker), when a Voluntary Exit Message (VEM) is requested, web3signer respond with a « Bad Request ».
Log on web3signer:
signer_1         | 2020-12-28 12:12:50.621+00:00 | vert.x-eventloop-thread-2 | ERROR | LogErrorHandler | Failed request: http://signer:9000/api/v1/eth2/sign/0x8400ead5497554f557a7efd17e3515ac817369a6ef388352057ef2fdcb9b75d0a23731eccfd510bfd8d85fd73300d105
According to https://consensys.github.io/web3signer/web3signer-eth2.html#operation/ETH2_SIGN the payload shown below is correct (i.e: all required fields are populated)
POST /api/v1/eth2/sign/0xa5bb1e8378db17c9a0bbe27a6ddf5be4bfcbc6514ef4d5bf94b739791093f7a1dfc86400b414c014f9a8e94697d138b2 HTTP/1.1
Host: signer:9000
User-Agent: Go-http-client/1.1
Content-Length: 176
Accept: */*
Accept-Encoding: application/json
Content-Type: application/json

{"fork_info":{"fork":{"current_version":"0x00000001","epoch":"0","previous_version":"0x00000001"}},"type":"VOLUNTARY_EXIT","voluntary_exit":{"epoch":"0","validator_index":"1"}}

HTTP/1.1 400 Bad Request
content-length: 11

Bad Request

@Magicking fork_info.genesis_validators_root seems to be missing in your supplied payload. The current OpenApi spec at https://consensys.github.io/web3signer/web3signer-eth2.html#operation/ETH2_SIGN does show it so it is possible that an earlier version that you has used might be missing it.
{
  "type": "VOLUNTARY_EXIT",
  "fork_info": {
    "fork": {
      "previous_version": "string",
      "current_version": "string",
      "epoch": "string"
    },
    "genesis_validators_root": "string"
  },
  "signingRoot": "string",
  "voluntary_exit": {
    "epoch": "string",
    "validator_index": "string"
  }
}

@Magicking fork_info.genesis_validators_root seems to be missing in your supplied payload. The current OpenApi spec at https://consensys.github.io/web3signer/web3signer-eth2.html#operation/ETH2_SIGN does show it so it is possible that an earlier version that you has used might be missing it.
{
  "type": "VOLUNTARY_EXIT",
  "fork_info": {
    "fork": {
      "previous_version": "string",
      "current_version": "string",
      "epoch": "string"
    },
    "genesis_validators_root": "string"
  },
  "signingRoot": "string",
  "voluntary_exit": {
    "epoch": "string",
    "validator_index": "string"
  }
}

Not sure to follow, what might be the issue as the Genesis Validators Root is specified as optional in the spec you link and GVR is sent by Teku according to this discussion -> https://consensys.slack.com/archives/GU2QYGEF4/p1607478266205100?thread_ts=1607468953.204600&cid=GU2QYGEF4

Not sure to follow, what might be the issue as the Genesis Validators Root is specified as optional in the spec you link and GVR is sent by Teku according to this discussion -> https://consensys.slack.com/archives/GU2QYGEF4/p1607478266205100?thread_ts=1607468953.204600&cid=GU2QYGEF4


Convert search to command palette

Hi Dries. This resolves #834
Fyi, search on mobile is broken on the live version.
I'd ditch the articles/threads snippets and I'd leave only the titles. What do you think about it?

Amazing! that's incredible work @geowrgetudor 👏
I'll try to review this as soon as I'm able.

This lgtm. Just one question about commented out code.

@geowrgetudor thank you so much for this one! :)

We should exclude "/" trigger while the editor is focused. Didn't cross my mind when I was implementing it, but I just stumbled upon this while typing an URL path into a reply form. I'll look into it.


fix: correct default value that was wrong

Fixing the default value that was wrong in the documentation for the sSortDescending field.

Thanks for this! However, per the Contributing document PRs need to go to DataTablesSrc. I can make the change manually, or if you want to send a PR there, please do.

Thank you for the warning. I tried to search the repository with the source code but there was no one found.
I will open the pull request there.


Upgrade commons-text

Upgrade of commons-text to 1.10.0

I wonder if we need this since Quarkus will be managing this dependency from 2.14.0 and hopefully in 2.13.3:
https://github.com/quarkusio/quarkus/pull/28645

PR is not needed anymore, Fixed in quarkus in https://github.com/quarkusio/quarkus/pull/28645

Replaced by https://github.com/apache/camel-quarkus/pull/4198


Adding primitives

I wanted to add primitives in the domain, for example consider the primitive:
unsafeCast : {a : *} -> {b : *} -> a -> b
With reduction rule
unsafeCast {a} {b} x ~> x (if x is not neutral)
I thought it would be simple to add is a new kind of Head of neutrals, but I'm hitting on the problem of when to actually reduce them. For example if you have the closure:
\v -> Neutral "unsafeCast" [a, b, v]
Then applying this closure will not actually recompute and reduce the primitive.
One way I thought of is to "guard" closure application such that after a closure application it is checked if a primitive can now be reduced. This means having to go through the spine of the neutral as well, because you might have something like:
\v -> Neutral "unsafeCast" [Neutral "unsafeCast" [a, b, v]]
Does this approach seem reasonable to you, or do you have a better idea maybe?
My goal is implement the elements of "The Gentle Art of Levitation". I was hoping that adding Desc and the constructors and other constructs as primitives would allow me to keep my syntax simpler. Basically I want axioms that also can reduce if their spine takes a certain form.

Here are a some examples for handling primops. In summary, we have for a primop:

A new symbol (nullary constructor) in the core syntax.
A new neutral head, for when the evaluation of a primop gets stuck.
An evaluation rule in eval, which consists of as many VLam-s as needed to apply the primitive to enough arguments, so that the computation rule can be tested. We return a primop-headed neutral if the evaluation rule is stuck.

There are more ways to make this work, I usually use the above scheme because it allows me to partially apply a primop without restriction, and generally treat is as a postulated constant with extra computation rules.

Thanks, that's clever. I will try to implement something similar.

You could also implement fst and snd for dependent pairs like this, couldn't you? Although that does make the head more complicated than having fst and snd as eliminations in the spine. What do you think?

Yes, and what's better is that we don't have to change spines at all. We simply have Fst :: Tm, and eval Fst = VLam $ \v -> vFst v or something like this, where vFst builds the usual projection spine.

Aah that's true, but would a fst/snd headed neutral have downsides compared to a more complicated spine?

Yes, projections in spines are clearly much better than as heads. In older versions of Coq, they actually had projections as symbols, and then they switched to primitive (spine) projections, which were clearly more convenient and efficient.
One point of neutrals is that we want as easy as possible access to the expression which blocks computation. If projection is a constant, it's not actually the head which blocks computation, but its argument, and if we have something like a neutral fst t1 t2 t3, then the blocking t1 expression is buried under all outer applications.
For eta-conversion checking it's also clear that we want projections in spines. It works the same way as eta conversion for functions. Type theoretically, functions and sigmas are often grouped together as "negative" types, and I think I've even seen function application being called a "projection".

While on the topic of eta-conversion. I saw Agda has eta-conversion for the unit type (for records in general). If I add a primitive unit type then can I just conclude (if I follow your system in elaboration-zoo) that whenever I unify () ~ x or x ~ () that it should be accepted? Since both sides are typechecked before unifying I can assume that x : UnitType and so x must be ().

() ~ x and x ~ () works, but this is weaker than unit eta in Agda, where we also have x ~ y for variables x : () and y : ().
For non-empty record types, we get the full eta this way, only the unit type is where we don't get it. Agda has type-directed conversion checking, so it implements unit eta by matching on the type.
I personally don't think that type-directed conversion is worth it just for unit eta; it adds more complexity and has some performance cost.

Ah I see, thanks! Luckily the weak unit-eta is enough for my purposes.

I have another question, related to the weak eta law for unit. If we add an equality type:
Eq : (t : U) -> (x y : t) -> *
Refl : (t : U) -> (x : t) -> Eq t x x

And the weak eta:
Refl ~ x

Is this consistent? Will it allow us to define equality elimination (because the weak eta law for unit is enough to get induction for unit)?

Is this consistent?

Yes. It's called uniqueness of identity proofs (UIP). But it's inconsistent with univalence.

Will it allow us to define equality elimination (because the weak eta law for unit is enough to get induction for unit)?

No. With only UIP and no elimination, it's consistent to assume Eq A B ~ Unit.

If we have Void = {A : *} -> A and Refl ~ x and Unit = Refl {*} {Void}, would that be consistent with transport. My feeling is yes.

Sure, you're not doing anything with Eq, just defining Void and Unit.

Great, so in that scenario you get unit with (weak) eta for free.


table storage implementation

Hey, I want to suggest to add a table storage implementation. I would happy to help implement it.

hi @wellDan28 ! This would be really appreciated! How can I help you to get up speed with Azurite development? I could start a Wiki which describes the repo structure, design etc.

hey @arafato,
More documentation is always great, I can't see anything specific that I need right now,
I'll ask here if I would have any questions
I'll start to implement the following actions:

Create Table
Delete Table
Insert Entity
Query Entities

I think that's enough for a first "table implementation PR" , other actions implementation should be in a separate PRs

@wellDan28 this is super cool! Will this be in-memory only or do you also plan to persist to disk? I don't have a specific opinion about that, Queue Emulator is not persisting to disk at all at the moment for example while Blob Storage does (files of course and also metadata).

I'm not sure yet, I'll update when I'll have more insights.

@wellDan28 do you need any help with the Table Implementation? I can help you with that

@wellDan28 @pmanlukas : basework is finished, feel free to grab an item :-)


Upgrade Paraview to version 5.0.

Upgrade available only for versions of OS X >= 10.7. For OS X 10.6,
leave existing version.

Thank you for the contribution. It was merged directly as 6af1289e2104e27d3a4ccf661aba4fb956c180e6 to keep commit history cleaner. Your contribution is still credited to you.


[WIP] bots: Allow users to configure incoming webhook bots

Going forwards, Zulip will need to let it's users configure their bots' BotConfigData (for example see the discussion under issue #10907). For embedded bots and outgoing webhook bots, the keys that can be configured are predetermined and users can already modify them from settings > your bots > active bots. Now for Incoming webhook type bots there are no predetermined keys as we do not strongly associate a bot with an integration (e.g. multiple integrations might use the same incoming webhook bot - e.g. A GitHub + GitLab notification bot) - so letting users set these keys-value pairs freely seems like a flexible approach.
This PR introduces 2 commits:

Changes the backend to send config data for incoming webhook type bots in the initial state as well as for update events. Patching the config data is already possible for all kinds of bots and this did not require any changes (it was just not used up until now).

[WIP] Modify the UI to allow users to modify their bot config data - add new keys or edit existing ones. Pending work for this commit:

remove key option
polish UI - the table currently looks ugly.
polish code a bit - just some minor refactoring
move the CSS (but to which file?)

@timabbott and @eeshangarg I'd appreciate a code review as well as any general opinions you guys have about this.
Getting this completed and merged would make it possible to begin work on developing a framework for how to allow our incoming webhook type bots to contact an API (as a callback), as @eeshangarg suggested.

remove key option
polish UI - the table currently looks ugly.
polish code a bit - just some minor refactoring
move the CSS (but to which file?)

I'll be adding these soon (most likely tomorrow).

Just to make sure I understand here, is the model that we're allowing custom configuration fields?  I think structurally, we actually want something very similar to the embedded bot model, where the user picks an integration (e.g. "GitHub") and then we offer the BotConfigData form fields appropriate for that integration.
We want to move the UI model to the user picking an integration for incoming webhooks (e.g. "GitHub") anyway, because that lets us prepopulate the integration logo.  But since that's sorta a prerequisite to being able to do something good with bot config data here, maybe we should pause this PR once the first commit is merged, and work on making it possible to do that before picking this up again?  I think we could reasonably structure it like this without too much work:

Clean up and merge the first commit from this PR.
At the database layer, have a BotConfigData field available to all integrations called integration_id that will be managed as a dropdown in the UI (and corresponds to the machine-readable integration name, i.e. the first argument of WebhookIntegration in zerver/lib/integrations.py`).  Users select the integration they want via a dropdown after picking "Incoming webhook" as the bot type, that has a "Custom" option (to get the legacy behavior).  (Actually, realistically, we can offer this with all integration types, since the logo behavior described here we'll want regardless).  This is also the piece we'd need for continuing the BotConfigData UI work in this PR.
Once we've done the above, we can auto-populate the "avatar" feature with the integration's logo from the data zerver/lib/integrations.py.  This is a big UI improvement we've wanted for a long time.  To support this, we'll likely need to send some of the zerver/lib/integrations.py data in page_params so the browser has access to the logo URLs.  API-wise, we'll want to think about how to manage setting the avatars; I'm not sure what the right approach is; we can discuss once we've completed the above.

Does that plan make sense?

Yup, it makes sense, and it sounds like a good plan. My only concern is that this is a pre-requisite for the Phabricator integration (issue #9221) since we need to store the API key there. So that would be blocked until this can be done.

Clean up and merge the first commit from this PR.
@timabbott what would you like to see cleaned up here?

It's OK to have another project be blocked while we implement this piece correctly; hopefully it won't take too long.
For clean up, I just meant fixing the comments about duplicated code and unnecessary database queries I made above.

For clean up, I just meant fixing the comments about duplicated code and unnecessary database queries I made above.

Just finished this. Are there any other changes you'd like to see in the first commit?


A spinner that's Offstage still consumes resources

If you put a spinner widget, or some other long-running animation, in an OffStage widget, it'll consume all your battery.

closing in favour of https://github.com/flutter/flutter/issues/5469


Remove message about optional pre-release features

"Configuration Manager doesn't enable this optional feature by default. You must enable this feature before using it. For more information, see Enable optional features from updates."
This is no longer needed.  In current branch, this is a "Release" feature (as of several releases back, not sure which one it started, but I checked 1902 and it is "Release" there).

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

ID: 0a429ba7-b6ca-4def-d32f-1face0cf2864
Version Independent ID: a23d121d-cc09-61e9-5b70-f6e15cf1574a
Content: Introduction to certificate profiles - Configuration Manager
Content Source: sccm/protect/deploy-use/introduction-to-certificate-profiles.md
Product: configuration-manager
Technology: configmgr-protect
GitHub Login: @mestew
Microsoft Alias: mstewart

Via internal thread, confirmed that this feature is optional, not to be confused with pre-release.


Update index.rst

Remove "for students" from item (3), to be more clear about the target audience (instructors).
Fix broken Jupyter logo.
Small language changes.

Thanks again!!!!


Remove explicite log procerros env var

Description
Copy of https://github.com/microsoft/ApplicationInsights-Python/pull/311
We are now leveraging OT env vars for our configuration instead of custom ones. Confirmed that OT sdk uses OTEL_BLRP_SCHEDULE_DELAY as specified here in the usage docs.
Fixes https://github.com/microsoft/ApplicationInsights-Python/issues/265
PR that added these env vars to OT sdk: https://github.com/open-telemetry/opentelemetry-python/pull/3237
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Constantly gettng 'SendRequest: Non-success status received :: Status Code: ServiceUnavailable'

Hi, first I want to thank you for the library, I had been looking for something similar for a while.
I am however facing the same issue over and over, I would say 95% of the times, I get the following response from the API:
Message=SendRequest: Non-success status received :: Status Code: ServiceUnavailable; Reason: ; Content: <!DOCTYPE html PUBLIC '-//W3C//DTD XHTML 1.0 Transitional//EN' 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd'><meta content='text/html; charset=utf-8' http-equiv='content-type'/><title>Bing</title>

<img alt='Bing' src='data:image/gif;base64,R0lGODlheAAiAMQAAL/a9ECR3n+16e/2/BB11jCI28/j95/I7mCj5K/R8d/t+XCt5iB+2f+mFI++7FCa4f/u0/+sI/+xMZvF7v/YmP/04v/dpwBs0////wAAAAAAAAAAAAAAAAAAAAAAAAAAACH5BAAAAAAALAAAAAB4ACIAAAX/ICCMAmaeaKqekNQ0kbXOdG3feI4JV3/ptMhrCAEaj8gcz5dECYeNYnNKNS571dYrQql6v7XrT6UAmAG5itQ2OAMUScPZkFI4HoEeYBIIOBwTXmInCQgFPogMDw5wSHZ5iD4BAnQ2BguHkRcFCwmQiGgCAAUFaFViAwIEmqwXCJUom3mmJwCfrZIHM7a4vRdoBwYjsFNiqz4ECA5nBwsMkQsDsT60Aw+RDMpmzZmSjScIkQWiAAkC3ay0YDusDLq7twXSJqAmA+gB6ii8yLDh3itGIUKwboWYHgjm1TiASB49aiY+EUhwA8CxTSbEEKzxrwfFgigObsTB0MeDh3ow/xioh2OljwQDLha4MeBZDwYgQyYysgARnHpXZur4hAAAonc2HCAiVlAM0hwKEJWo1zEAkCt+WNowCjEnOx/fdHSzWu+TVR1YuXrEobRrTlRHxmIoK+mqpAGITt6weYGAVxNqL3zUgdfHgrkQzdrtYbWnj6crOl4o8RfDxbM62qaky3jxBav3EEVbYa1h5YxHdRi4jPIXBsVo62IIjWzBGwwKCl288EBhZXSQZ6zu1xoNbCWyM+7GRSD4X9o9HoRNkQoRAVicP3vGbM8Bula9T6MonfcArdwIdhcglp37Daw0ypFI4Mk609P8fN0Mnn0kDvhAHHARAb6Jp5IzzCEwWIAKffRBxwINOkfDAQ0eZkQCAxlIGgAHkLDMdBoiIVmBIZYIUkkpmagiSBi6teKLVTjWA4gwwsgXgZawVuOOhDRE4gkGoHMfjy+i2JcABaaym4RErpjAcgEsIIAh2OjT5I5K+sIAkld2aU8CEPLFwCQA/OglkQMMoIABbCqQpokhAAA7'/>

Bing services aren't available right now

We're working to restore all services as soon as possible.
We know you want to get back to searching. Please check back soon.

Ref A: A1C0134338234D4AA480524F216CB616 Ref B: PAR02EDGE0112 Ref C: 2018-09-27T17:12:16Z

I am getting this on UWP, and Android, with every possible configuration (recognition mode, streaming on/off, etc.). Every once in a while I get a succesful response from the API, but then I get this response again and I have no idea why. Any insight would be very much apprciated.

Same as #14 let's see if Nate comes up with some ideas, I've no clue I'll get back to debugging next week....

On Android today is working 9 out of 10, I swear I didn't change the code, so it must be something else maybe in the OS?
I changed the code for iOS too but I can't deploy it at the moment so I'll try it as soon as I can.
Thanks!

Nate,
I am getting the 'ServiceUnavailable', but only on the initial request after deploy
All subsequent request are fine.
I'm already using the Android Http Request Handler and Native TLS 1.2
Any thoughts?
Bean

The only hint I can give to reproduce the bug is: -Start your app under wifi -Stream an audio to the service - it works! -Switch to cellular network -Stream again - failure most of the times!

From: BeanStalka<EMAIL_ADDRESS>Sent: Monday, October 1, 2018 7:45:23 PM
To: NateRickard/Xamarin.Cognitive.BingSpeech
Cc: Paolo Ferrazza; Comment
Subject: Re: [NateRickard/Xamarin.Cognitive.BingSpeech] Constantly gettng 'SendRequest: Non-success status received :: Status Code: ServiceUnavailable' (#15)
Nate,
I am getting the 'ServiceUnavailable', but only on the initial request after deploy
All subsequent request are fine.
I'm already using the Android Http Request Handler and Native TLS 1.2
Any thoughts?
Bean
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FNateRickard%2FXamarin.Cognitive.BingSpeech%2Fissues%2F15%23issuecomment-425998631&data=02|01||2c1a4c50eee34d96466c08d627c5ab2b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636740127257209437&sdata=LNu8rTFtVaZflTpIfKXbscm2HHOtKhWLHfRiVpNxzDw%3D&reserved=0, or mute the threadhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOxBAFF2u6rPW6dF3QihZG2gW9dKlqyIks5uglSzgaJpZM4W9G7b&data=02|01||2c1a4c50eee34d96466c08d627c5ab2b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636740127257209437&sdata=h2VfSo7V31ueG0ACaxWSK8ySth6HPKcBO9z%2Bwr4sye8%3D&reserved=0.

As a side note iOS is fixed.

I don't know about @BeanStalka and @bzglm but I tested this on many devices in the last days and it's working, maybe it was a problem on MS side?

closing this since it seems to have been resolved


Coheed and Cambria: Some albums are semi-duplicated.

https://droptune.co/artists/5818-coheed-and-cambria
Specifically:

https://droptune.co/albums/112696-coheed-and-cambria-the-color-before-the-sun

https://droptune.co/albums/112697-coheed-and-cambria-the-color-before-the-sun

https://droptune.co/albums/112717-coheed-and-cambria-the-afterman-ascension-deluxe

https://droptune.co/albums/112714-coheed-and-cambria-the-afterman-ascension-deluxe-edition

One listing includes Apple Music, the other doesn't.

@pnomolos Ah yes. This was an issue with viewing capitalization. Rolled a fix out, but it wasn't retroactive.


ListGenerator

Which service(blob, file, queue) does this issue concern?
https://github.com/Azure/azure-storage-python/blob/2709a0bb5b1647803fc25ee1292c6fc43782ed89/azure-storage-common/azure/storage/common/models.py#L53
What problem was encountered?
The ListGenerator does not stop/reset cleanly.
Expectation. After the first call to list(data_gen) then either subsequent calls it is either reset and  the same data is returned or if you consider it exhausted then return [] or None
Have you found a mitigation/solution?
This is a guess. I do not know enough about the way this is being used. Otherwise, write a ListGerator for the purpose.
Try following the
__iter__ and __next__ pattern where the __iter__ sets/resets and __next__ does the work.
Note: for table service, please post the issue here instead: https://github.com/Azure/azure-cosmosdb-python.

Hi @douglasmacdonald, please take a look at the sample to understand how table_service.query_entities should be used.
In short, when you write list(table_service.query_entities(table_name)), all the entities are returned as a list, meaning that the iterator follows the marker until there is no more items or the max number of results is reached. The ListGenerator returned by query_entities at this point should not be reused.
Please let me know if this is clear now. Thanks!

Thank you. I understand.

"The ListGenerator returned by query_entities at this point should not be reused."

But this was not the point I was making.
If you are happy with this behaviour that is fine.  I was only making the point that, generally, in python generators do get called again. A user using list(...) on a generator where the underlying data has not changed might expect the same thing to happen when they call list on it again. If not, it should be either clearly returned empty [] or StopIteration or new data.  Even if you do not want it used again, Returning a blob of data form the end is confusing. If you don't want it to be used again then a StopIteration seems like the best answer for you.
The code sample I gave you shows that a class is not necessary to get the behaviour I'd expect.
Thank you for the Python interface.


vue3 + element-plus  warn Failed to resolve component: happy-scrolly

[Vue warn]: Failed to resolve component: happy-scrolly If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.

抱歉，目前还未测试是否兼容 vue3...


.form-text follow-up

Follow up on https://github.com/twbs/bootstrap/pull/30565

We use <small> in our docs, which might not always be correct, the font size is not smaller per se.
We still use text-muted in the horizontal form here: https://twbs-bootstrap.netlify.com/docs/4.3/forms/overview/#form-text
We should share properties between .form-text, .valid-feedback and .invalid-feedback

I think we better remove display: block from .form-text and replace the <small>s with <div>s and use a <span> in the horizontal form.

Hm, agree. I can take this to.

Go ahead, thanks man!


feed lifecycle event should have a fully resolved trigger name

In looking through some code for the providers, I noticed that when a user creates an event feed, the request carries a trigger name that is not fully resolved.
For example, when creating an alarm feed,
wsk trigger create t --feed whisk.system/alarms/once ...
The feed provider receives the following request:
{
"authKey":"...",
"lifecycleEvent":"CREATE",
"triggerName":"/_/t"
}
The trigger name isn't fully resolved as it uses the implicit/placeholder _. This means the feed handler has to resolve the namespace. It can do this by doing a lookup using the provided key against the openwhisk controller:
GET https://apikey@apihost/api/v1/namespaces
In the providers I looked at, the namespace is being resolved another way: by using the __OW_NAMESPACE activation context value. This is OK as well but if we just treat the feed creation as an arbitrary API call, this won't work. So either the client has to resolve the trigger name fully, or the provider has to do it (or both?).
I am contemplating a change to the feed specification to make the trigger name fully qualified (ie no underscore) and so clients must do this and providers can reject requests that don't conform.
Current specification https://github.com/apache/incubator-openwhisk/blob/master/docs/feeds.md#implementing-feed-actions
Related to https://github.com/apache/incubator-openwhisk-cli/issues/416

I neglected a "fix the docs" only approach: we can update the docs to say the trigger name may use the implicit namespace, and that it's the provider action's responsibility to resolve the namespace fully... which the current openwhisk providers are doing anyway.


Made Parent available to children within PresenterBase

Makes for a happy family :)

Okay.
By the way, what kind of case is it necessary for?

Thanks, I was holding some model data within the parent PresenterBase and just wanted an easier way to access it from the child.


Ingest plain email interactions

Description of change
This changes the email ingestion task to use plain email processor instead of calendar.
Checklist

[x] Has this branch been rebased on top of the current main branch?

Explanation
The branch should not be stale or have conflicts at the time reviews are requested.

[x] Is the CircleCI build passing?

General points

Other things to check

Make sure fixtures/test_data.yaml is maintained when updating models
Consider the admin site when making changes to models
Use select-/prefetch-related field lists in views and search apps, and update them when fields are added
Make sure the README is updated e.g. when adding new environment variables

See docs/CONTRIBUTING.md for more guidelines.

This may be addressed by another PR.


What is the idea?

Hello,
I always wondered what the idea behind math anti-spam was.
Because the last time I checked computers were better at maths than humans.
Regards
Henry

The idea is to provide a simple barrier to unsophisticated spam bots whilst sparing the user the pain of very difficult or at least irritating CAPTCHAs.
Its also much better for accessibility as screen readers are more easily able to read out a question to a user and the answer is much more simple to find out.
Whilst this solution appears trivial to bypass, anechdotally, I can report that adding such trivial questions to forms that were being targeted reduces that spam to 0 and we haven't had spam instances after using it even after 2 years of a previously abused form.


Cannot use ::class with dynamic class name

Describe the bug
We keep getting an error thrown from this package:
Cannot use ::class with dynamic class name
It is coming from here:
/nystudio107/craft-code-editor/src/autocompletes/SectionShorthandFieldsAutocomplete.php:167
I think it is this line in the current version:
https://github.com/nystudio107/craft-code-editor/blob/d2d9c8706fc1a6f40253fc78762101ce9b5ac386/src/autocompletes/SectionShorthandFieldsAutocomplete.php#L174
Expected behaviour
I think something like this would solve it:
https://github.com/nystudio107/craft-twigfield/commit/bff508e2e239fd55b8a17ea56fc60db5e655f418
Versions

Plugin version:
"nystudio107/craft-seomatic": "3.4.68",
Craft version:
"craftcms/cms": "3.8.5",
PHP - 7.4.33

I'm sorry I somehow missed this issue, thanks for re-filing it under SEOmatic to get my attention.

Addressed via https://github.com/nystudio107/craft-code-editor/commit/c8c04db13fcabbe31e3f5d04b828b04c37afe577

Fixed in version 1.0.18 -> https://github.com/nystudio107/craft-code-editor/releases/tag/1.0.18
You can update via:
composer update nystudio107/craft-code-editor


Improved the sunrise and sunset cards UI

Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

Issue Resolved: #28
Description: Improved the UI of sunrise and sunset cards

@Bhavneet1492 made the changes, do drop a review when free


open-graph image very large 1500x1500

I use a source png which is roughly 200x200 but I the generated open-graph.png image is 1500x1500 and it doesn't look ok when sharing. What am I missing?

Nevermind fb wants large images... My problem is that is doesn't seem to fit a square image properly for sharing, so I need to adjust the aspect ratio or the original


DEV: Using shortcuts like CMD + C on message text inserts the text c in the composer.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open any chat
Select a text inside one message
Copy the text using CMD + C

Expected Result:
Text is copied to clipboard
Actual Result:
text c is inserted in composer field
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: happening in dev version
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/231807764-dbe3216a-ac5a-4d2a-8fae-d9e4c6ed49bd.mov
Expensify/Expensify Issue URL:
Issue reported by: @getusha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1681329759993079
View all open jobs on GitHub

Triggered auto assignment to @joekaufmanexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Looks like this was a regression that we already fixed in this PR yesterday. Since this was a regression, and it's already been fixed, I'm thinking we just need to pay the reporting bonus here to @getusha
Curious if you agree @PauloGasparSv @rushatgabhane

Yes @joekaufmanexpensify this issue was fixed by this P.R., but in the slack thread we asked for a new issue to be created for:
"Pressing the paste shortcut on Web/Desktop (cmd + v, ctrl + v) while a chat is open should switch the focus to the composer."
That's a new feature and [we didn't include it that P.R. since it's purpose was to focus the composer inbox whenever the user typed something and that feature requires different changes.
@joekaufmanexpensify @kavimuru should we keep this issue open and re-use it to implement that?

@getusha I see! I thought there was no GH for that yet! Do you mind sharing the GH link in the slack thread?
I think we can close this if there is another issue then @joekaufmanexpensify :D

Great! To summarize:

This issue was a a regression associated with this issue.
We fixed this already in this PR.
There's no C+ payment here (since no C+ review).
There's no contributor payment, since this was fixed as a result of a regression introduced by the contributor's other PR.
We need to pay @getusha a $250 reporting bonus for reporting the bug.

Adding external label so we can create an upwork job

Current assignee @joekaufmanexpensify is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @mollfpr (External)

Triggered auto assignment to @jasperhuangg (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@jasperhuangg @mollfpr unassigning both of you, I just added the external label to create an upwork job so we can pay a regression reporting bonus. We already fixed this regression.

@getusha offer sent for $250!

@getusha $250 sent and contract ended!

Closed upwork job

Regression is fixed, and regression reporting bonus paid. Closing as this is all set. Thanks everyone!


Can't save in psd

Hello. I did a little script that save's every layer as an png file, but I would like to make it able to save in PSD.
I tried using PhotoshopSaveOptions instead of PNGSaveOptions but it just return some errors.
This is my working PNG code.
import photoshop as ps
import os

def main():
    app = ps.Application()
    doc = app.activeDocument()
    i = 0
    for y in doc.artLayers:
        j = 0
        for x in doc.artLayers:
            if i == j:
                x.visible = True
            else:
                x.visible = False
            j = j + 1
        i = i+1
        options = ps.PNGSaveOptions()
        if not os.path.exists(doc.path + '/' + y.name):
            os.mkdir(doc.path + '/' + y.name)
        imagepath = doc.path + '/' + y.name + '/' + y.name + '.png'
        doc.saveAs(imagepath, options, True)
    app.doJavaScript(f'alert("Task done!")')
    print(doc.activeLayer)

if __name__ == '__main__':
    main()`

If I use ps.PhotoshopSaveOptions() instead of ps.PNGSaveOptions() it will give this error.

Traceback (most recent call last):
File "C:/Users/bruno.freitas/Desktop/PhotoshopLayer/Main.py", line 26, in
main()
File "C:/Users/bruno.freitas/Desktop/PhotoshopLayer/Main.py", line 17, in main
options = ps.PhotoshopSaveOptions()
AttributeError: module 'photoshop' has no attribute 'PhotoshopSaveOptions'
Process finished with exit code 1

And if I use ps.save_options.PhotoshopSaveOptions() instead of ps.PNGSaveOptions() it will give this error.

Traceback (most recent call last):
File "C:/Users/bruno.freitas/Desktop/PhotoshopLayer/Main.py", line 26, in
main()
File "C:/Users/bruno.freitas/Desktop/PhotoshopLayer/Main.py", line 21, in main
doc.saveAs(imagepath, options, True)
File "C:\Users\bruno.freitas\AppData\Local\Programs\Python\Python38-32\lib\site-packages\comtypes\client\lazybind.py", line 182, in caller
return self._comobj._invoke(descr.memid, descr.invkind, 0, *args)
File "C:\Users\bruno.freitas\AppData\Local\Programs\Python\Python38-32\lib\site-packages\comtypes\automation.py", line 728, in _invoke
self.__com_Invoke(memid, riid_null, lcid, invkind,
_ctypes.COMError: (-2147220261, None, (None, None, None, 0, None))
Process finished with exit code 1

I'm new to Python and not a pro programmer, so there's a great possibility that I'm just doing something very wrong. Can someone help me

Hi @brunosly,
Thank you for your report, I double-check the code missing the PhotoshopSaveOptions in the session,
I will fix this soon and add an example to show you how to save active documents psd.

@brunosly I updated the code to fix this issue and in below have two examples may help you.

https://github.com/loonghao/photoshop_python_api/blob/master/examples/export_layers_as_png.py
https://github.com/loonghao/photoshop_python_api/blob/master/examples/save_to_psd.py

Let me know if you have any other questions.
Cheers,
Hal

Thanks! It works perfectly now!
Also great work on this API, it's amazing.

@all-contributors please add brunosly for Bug reports.


DHCP scope not set correctly if custom is used

@DarrylvanderPeijl your change was lost somewhere :). Can you take a look? DHCP scope is not set correctly. I lost variables in DSC config.

Fixed in latest commit


Phi-3.5 medium model and system message not being observed

Please provide us with the following information:
Phi-3.5 medium model and system message not being observed

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

System prompt support was introduced to Phi-3-mini in the June 2024 update; however, no update was made for Phi-3-medium.  We suggest switching to Phi-3.5-MoE as an upgrade of Phi-3-medium as the MoE model is a much better and cost effective model.


Fix unused type parameters ts error

Closes #335

@zemse I think you forgot to commit changed files. Running yarn test:fix locally and committing new files should do the trick.
Maybe we can enable noUnusedParameters in our tsconfig so we can reproduce this problem in the future?

Maybe we can enable noUnusedParameters in our tsconfig so we can reproduce this problem in the future?

I tried setting in the global tsconfig, but it gives couple more errors. So lets do in another PR.


Remove session from cache before clearing it.

Hi,
We noticed that after logout we were able to make successful API calls with a saved session cookie. After some testing and reading debug logs we found following lines in the log to be of suspect.
src/util.c(2413):  oidc_util_hdr_err_out_add: Set-Cookie: =; Path=/; Expires=Thu, 01 Jan 1970 00:00:00 GMT; Secure; HttpOnly; SameSite=None
src/cache/common.c(640):  oidc_cache_set: enter:  (section=s, len=0, encrypt=0, ttl(s)=-1613130005, type=shm)
src/cache/common.c(668):  oidc_cache_set: successfully stored 0 bytes in shm cache backend for key
The session cookie was cleared from the client but deleting the cache entry seemed to fail. This PR is a suggestion to remedy the situation. After this change we can see following log lines and access with saved session cookie fails.
src/cache/common.c(643):  oidc_cache_set: enter: 4330eb78-0176-4608-907d-ab7df74d2491 (section=s, len=0, encrypt=0, ttl(s)=-1613472125, type=shm)
src/cache/common.c(671):  oidc_cache_set: successfully stored 0 bytes in shm cache backend for key 4330eb78-0176-4608-907d-ab7df74d2491

thanks, that's a good catch; if you can add something to the ChangeLog and add yourself to the AUTHORS file, I'll merge

for the record: this is now included in a release, 2.4.7: https://github.com/zmartzone/mod_auth_openidc/releases/tag/v2.4.7


Inquiry About ECE Implementation in Your Code

I’m reading your paper and reviewing the code, but I’m having trouble finding the implementation of the ECE. Could you kindly help me locate it?
Thanks for your work!

especially the implement of chronologically arranged collection


Remove jq from packages

This change removes jq from the list of needed packages as it's nowhere used inside the module and produces another unnecessary dependency on the EPEL repository.

LGTM


Removing the java warning as its pretty pointless

also removed game servers cuz... yeah

I may of forgot to remove lavalink


Current version should not be taken from the package.json file which …

…will be bundled by the Angular build:

On 8 Jun 2021, at 08:16, FABIANO Serge @.***> wrote:
…will be bundled by the Angular build:

The package.json is not bundled at all in the final build: the version string is just replaced in the bundled files by the version is in the package.json file.
Do:
npm run build:prod
cd dist
ls -R | grep package -> nothing is returned
The current way of centralising a version is the documented angular way of doing it and is compliant with any tool automatic software releasing.
--
Jérôme Benoit aka fraggle
Piment Noir - https://piment-noir.org
OpenPGP Key ID : 27B535D3
Key fingerprint : B799 BBF6 8EC8 911B B8D7 CDBC C3B1 92C6 27B5 35D3

The package.json is not bundled at all in the final build: the version is just replaced in the bundled files by the version string in the package.json file.

Why it is appearing in the screenshot above then?

The package.json is not bundled at all in the final build: the version is just replaced in the bundled files by the version string in the package.json file.
Why it is appearing in the screenshot above then?

Dunno, report the bug to the software developer.
--
Jérôme Benoit aka fraggle
Piment Noir - https://piment-noir.org
OpenPGP Key ID : 27B535D3
Key fingerprint : B799 BBF6 8EC8 911B B8D7 CDBC C3B1 92C6 27B5 35D3

Dunno, report the bug to the software developer.

I find it normal that an import of a dependency in the code is bundled by Angular.
To not bundle it, it should be in the 'asset' directory and loaded via HTTP.
Here it's part of the code.

Dunno, report the bug to the software developer.
I find it normal that an import of a dependency in the code is bundled by Angular.
To not bundle it, it should be in the 'asset' directory and loaded via HTTP.
Here it's part of the code.

The import is on a field of a JSON file, TS replace the field by the value at transpilation. The tool you are using is buggy.
--
Jérôme Benoit aka fraggle
Piment Noir - https://piment-noir.org
OpenPGP Key ID : 27B535D3
Key fingerprint : B799 BBF6 8EC8 911B B8D7 CDBC C3B1 92C6 27B5 35D3

The import is on a field of a JSON file, TS replace the field by the value at transpilation. The tool you are using is buggy.

The import is on a field of a JSON file, TS replace the field by the value at transpilation. The tool you are using is buggy.

I don't think so.

The import is on a field of a JSON file, TS replace the field by the value at transpilation. The tool you are using is buggy.
I don't think so.

That's how TS transpiler deal with JSON field, it’s in the documentation and that’s why importing the version from the package.json is the documented angular way of centralising a version number.
Do in dist:
grep -R mochatest *
Output nothing, the file is not imported in any ways.
--
Jérôme Benoit aka fraggle
Piment Noir - https://piment-noir.org
OpenPGP Key ID : 27B535D3
Key fingerprint : B799 BBF6 8EC8 911B B8D7 CDBC C3B1 92C6 27B5 35D3

grep -R mochatest * Output nothing, the file is not imported in any ways.

Ok, it's more relevant than the 'ls' command.
I think that both statements are true, we have a package.json bundled but only with the version property.
So I'm fine with it.
I close the PR.

Well, I would have kept it with one small change: it offers an helper to properly get the version, the helper just have to be changed to return the version in package.json file.


Rust: support empty declarations

Things like:
pub trait Empty {}
pub struct Foo;
pub trait Bar {
    fn foo();
}
pub enum MyEnum {
}

aren't being captured, because the *.inner queries are requiring at least one node inside the parent node.

Seems to work well. Thanks a lot!


Fix for #99

Fix for #99

I think my patch https://github.com/EragonJ/Kaku/commit/f9fb50bc89cd2e8ba03daa22e509be378886fabb should fix this. And also, why you update width in toolbar (!?) that will break the UI definitely.


Minimum monthly pricing

Hi,
The https://azure.microsoft.com/en-us/pricing/details/functions/ page currently says the Premium plan pricing reflects a 50% discount as it is still in Preview. Since the Premium plan is now GA, can we assume we'll be paying double that price?
The price currently listed is "vCPU: ~$163.520000/month vCPU-s" and "Memory: ~$11.68/month GB-s" for Canada Central. For an EP1 instance with 1 core and 3.5 GB of memory this would mean a minimum price of 1163.520000$ for the CPU and 3.511.68$ for the memory so 204.40$ (possibly 408.80$ if the listed price were still reflecting a 50% discount).
At first sight this seems very expensive, are there plans to support smaller SKUs? I'm not sure what's the benefit of this plan versus an App service plan with auto scaling enabled?
Thanks!

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

ID: 3847e16b-92ae-0202-c7e1-ca9a8474e65f
Version Independent ID: a25b6b5b-05ab-b3b2-616c-5d06eadb928d
Content: Azure Functions Premium plan
Content Source: articles/azure-functions/functions-premium-plan.md
Service: azure-functions
GitHub Login: @jeffhollan
Microsoft Alias: jehollan

@FrankGagnon Thank you for your feedback! We will review and update as appropriate.

Thanks again for your feedback, @FrankGagnon! I'll be reaching out to the Functions team internally to get the required updates made on the pricing calculator. Regarding your questions around the comparison to the App Service plan, it might be better to open a new issue on Azure Functions Host repo so that product team members, as well as the rest of the Azure Functions community, can weigh in on it because this channel is more for driving improvements towards MS Docs.

Hi @mike-urnun-msft,
Thanks for your answer. Maybe an example of the cost breakdown for a given scenario would be helpful in the page. You do it for the Function's consumption plan and it makes it really easy to understand!
Once I have a better idea of the pricing breakdown I can make a more appropriate issue on the Functions host repository.
Thanks

A pricing example breakdown for Premium like the one that exists for Consumption is a must!

Thanks, everyone! I've reached out to the Functions team internally, and we are working on updating the pricing page.
As the next step, we are now proceeding to assign this issue to the content author to further review your feedback around pricing breakdown.

Any update on this? The pricing is really confusing - do we have to pay a base price for an EP1 instance and also pay per-second billing according to here? And what of the Preview discount?
Furthermore, the pricing calculator doesn't include the Premium tier at all.
As is is, it's impossible to determine how much Premium tier Function App usage actually costs :confused:

Apologies updating a bit of this. Since this is pricing page related I'm going to recommend we close after this comment and the pricing page should have all details:

The price is still the preview price even though Premium is GA.  This is mostly because we need to give a bit of notice to customers before price changes so didn't change immediately.
The price effective Jan 1 is live on the pricing page https://azure.microsoft.com/en-us/pricing/details/functions/ - you may notice it actually represents a price lower than 2x the current 50% discount
You only pay per instances you use, no execution cost.  You need at least 1 minimum instance per plan running as long as the plan exists, and additional instances are only added as needed if event rate requires scaling.  You can have many hundreds of concurrent executions on a single instnace (depending on your workload) - but can also control the max scale out as needed to.
We are working on updates to the pricing calculator that will make this easier. I'm pushing for them to happen in December but may be early next year.

#please-close


how to write a bilinear interplolation layer?

I want to implement a bilinear interpolation layer just as in caffe which do filter-wise upsampling. The upsample2d now just do repeat. Here is my code, does anybody know what's wrong:
from theano.tensor.nnet.abstract_conv import bilinear_upsampling
`class Deconv2D(Layer):
    def __init__(self, ratio, **kwargs):
        self.ratio = ratio
        super(Deconv2D, self).__init__(**kwargs)
    def build(self, input_shape):
        super(Deconv2D,self).build(input_shape)
    def call(self, x, mask=None):
        return bilinear_upsampling(x, ratio=self.ratio)
    def get_output_shape_for(self, input_shape):
        return (input_shape[0], input_shape[1],
                input_shape[2] * self.ratio, input_shape[3] * self.ratio)`

here are my error message:
raise AsTensorError("Cannot convert %s to TensorType" % str_x, type(x))
theano.tensor.var.AsTensorError: ('Cannot convert (None, None, Elemwise{mul,no_inplace}.0, Elemwise{mul,no_inplace}.0) to TensorType', <type 'tuple'>)

What happened with this?
I'm also very interested in having this capability in Keras.

@9thDimension Just update to the lasted version of Theano, the above code works!


Add multiple machines from one project example

There was still a "real world" example missing from the multiple machine use case.
Also, wouldn't this use case make the target table irrelevant?
The
[dependencies]
cmake = "3.26.4"
python = "3.10"

[target.osx-64.dependencies]
python = "3.11"

example also kind of goes against the philosophy of adding additional requirements to the requirements set (and not replacing them).
IMO the example above should be not solvable since python=3.10, python=3.11 together produce invalid requirements.

Alright, then let's leave target be for now. What I personally don't like is that this now introduces two ways of basically doing the same thing.


Update blast's milestones

Closing as this is blatant commit farming - 6 commits that are clearly extra fluff to boost the amount of work done.

Closing as this is blatant commit farming - 6 commits that are clearly extra fluff to boost the amount of work done.

Firstly, thank you for your attention. The issue with the multiple commits is due to formatting errors caused by online editing. I apologize for any inconvenience this may have caused.
Additionally, I would like to inquire if there are any open development requirements that we could assist with.


chore: remove versionrc.json

Fixes #

Codecov Report

Merging #283 (ebdfcf1) into main (02f6041) will decrease coverage by 1.77%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##             main     #283      +/-   ##
==========================================
- Coverage   94.33%   92.56%   -1.78%
==========================================
  Files           6        6
  Lines         424      444      +20
  Branches      126      132       +6
==========================================
+ Hits          400      411      +11
- Misses         21       30       +9
  Partials        3        3

Impacted Files
Coverage Δ

src/transformer/schema-transformer.ts
84.67% <57.14%> (-5.71%)
:arrow_down:

Continue to review full report at Codecov.

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


SSH channel poll error (Socket error: disconnected) in netopeer2-server log

Hello,
I am observing  call home connecting and disconnecting in netopeer2-server log .

Jan  5 02:15:42 mavdu netopeer2-server[7350]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  5 05:16:08 mavdu netopeer2-server[17267]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 01:46:36 mavdu netopeer2-server[4032]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 02:40:00 mavdu netopeer2-server[13137]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 02:47:22 mavdu netopeer2-server[13137]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 05:18:28 mavdu netopeer2-server[18618]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 05:18:29 mavdu netopeer2-server[18618]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 07:33:13 mavdu netopeer2-server[17230]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 07:39:02 mavdu netopeer2-server[6580]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 07:41:36 mavdu netopeer2-server[8292]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 12:42:43 mavdu netopeer2-server[9788]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
Jan  8 15:52:18 mavdu netopeer2-server[8066]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...

Before call home disconnect , I  always observe error " SSH channel poll error (Socket error: disconnected)."
 Jan  2 13:40:52 mavdu netopeer2-server[4170]: Session 10: SSH channel poll error (Socket error: disconnected).
Jan  2 13:40:52 mavdu netopeer2-server[4170]: Session 10: thread 3 event session terminated.
Jan  2 13:40:52 mavdu netopeer2-server[4170]: (sr_event_notif_send:3712) Error by processing of the request.
Jan  2 13:40:52 mavdu netopeer2-server[4170]: np2srv_sr_event_notif_send failed (sysrepo: Validation of the changes failed).
Jan  2 13:40:52 mavdu netopeer2-server[4170]: Call Home client "test_ssh_ch_client" session terminated, reconnecting...

could you please check,

Hi,
could you please provide the client output, also what client are you using? Is this all that the server prints? If so, then everything seems correct, the client suddenly disconnects so the server reconnects.
Regards,
Michal

Hi Michal,
I've attached netconf server log at that time.
I think abnormal closed client makes netopeer2 server shut down. but I can't find root cause.
=============> full
[1970/01/01 00:02:19.457616, 1] ssh_socket_exception_callback:  Socket exception callback: 1 (0)
[1970/01/01 00:02:19.457816, 1] ssh_socket_exception_callback:  Socket error: disconnected
ERROR: Session 3: SSH channel poll error (Socket error: disconnected).
VERBOSE: Session 3: thread 1 event session terminated.
DEBUG: Sending unsubscribe request.
ERROR: Session 4: SSH channel poll error (Socket error: disconnected).
VERBOSE: Session 4: thread 0 event session terminated.
DEBUG: Sending event-notification request.
DEBUG: unsubscribe request sent, waiting for response.
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
VERBOSE: Call Home client "test_ssh_ch_client" session terminated, reconnecting...
VERBOSE: nc_sock_connect(<IP_ADDRESS>, 4334, 5, -1)
VERBOSE: Trying to connect via IPv4.
VERBOSE: Successfully connected to <IP_ADDRESS>:4334 over IPv4.
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
udhcpc: sending discover
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
VERBOSE: Received an SSH message "request-service" of subtype "ssh-userauth".
VERBOSE: Received an SSH message "request-auth" of subtype "none".
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
VERBOSE: Received an SSH message "request-auth" of subtype "password".
VERBOSE: User "rcncuser" authenticated.
VERBOSE: Received an SSH message "request-channel-open" of subtype "session".
VERBOSE: Received an SSH message "request-channel" of subtype "subsystem".
VERBOSE: Call Home client "test_ssh_ch_client" session 5 established.
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
DEBUG: unsubscribe response received, processing.
DEBUG: Sending unsubscribe request.
DEBUG: unsubscribe request sent, waiting for response.
DEBUG: unsubscribe response received, processing.
DEBUG: Sending event-notification request.
DEBUG: event-notification request sent, waiting for response.
[cpri_ori/sysrepo_thread.cpp#,2532]
values_cnt=4
xpath: /ietf-netconf-notifications:netconf-session-end/username
xpath: /ietf-netconf-notifications:netconf-session-end/session-id
xpath: /ietf-netconf-notifications:netconf-session-end/source-host
xpath: /ietf-netconf-notifications:netconf-session-end/termination-reason
name=rcncuser, id=3, ip=<IP_ADDRESS>, reason=other
DEBUG: event-notification response received, processing.
VERBOSE: Generated new event (netconf-session-end).
DEBUG: Sending session-stop request.
DEBUG: session-stop request sent, waiting for response.
DEBUG: session-stop response received, processing.
DEBUG: event-notification request sent, waiting for response.
[cpri_ori/sysrepo_thread.cpp#,2532]
values_cnt=4
xpath: /ietf-netconf-notifications:netconf-session-end/username
xpath: /ietf-netconf-notifications:netconf-session-end/session-id
xpath: /ietf-netconf-notifications:netconf-session-end/source-host
xpath: /ietf-netconf-notifications:netconf-session-end/termination-reason
name=rcncuser, id=4, ip=<IP_ADDRESS>, reason=other
DEBUG: event-notification response received, processing.
VERBOSE: Generated new event (netconf-session-end).
=============> summary
ERROR: Session 3: SSH channel poll error (Socket error: disconnected).
xpath: /ietf-netconf-notifications:netconf-session-end/termination-reason
name=rcncuser, id=3, ip=<IP_ADDRESS>, reason=other
======> code
worker_thread()
nc_ps_poll()
ret = nc_ps_poll_session_io(cur_session, NC_SESSION_LOCK_TIMEOUT, ts_cur.tv_sec, msg);  //libnetconf2/session_server.c
r = nc_session_io_lock(session, io_timeout, func);
case NC_TI_LIBSSH:
r = ssh_channel_poll_timeout(session->ti.libssh.channel, 0, 0);
rc = ssh_handle_packets_termination(channel->session, timeout,ssh_channel_read_termination, &ctx);
else if (r == SSH_ERROR) {
sprintf(msg, "SSH channel poll error (%s)", ssh_get_error(session->ti.libssh.session));
//ERROR: Session 5: SSH channel poll error (Socket error: disconnected).
session->status = NC_STATUS_INVALID;
session->term_reason = NC_SESSION_TERM_OTHER;
After that, netopeer2-server is killed by SSH channel poll error.
Could you check this situation?
Regards
Boyong

Hi Boyong,
I still do not fully understand. The channel poll error is not such a big problem, the client simply did not close the session properly. But after that netopeer2 continues in the correct way meaning it tries to reconnect the Call Home session. And what do you mean by "makes netopeer2 shut down"? Is the whole process terminated? I do not see any output indicating that but if it is, then please provide the full output up to that point (netopeer2-server -v2 is fine, no need for debug output yet).
Regards,
Michal

190205_np2_terminate_alllog.log
I've attached log you can investigate np2 process is killed suddenly. Please check log.
If you need some debug option, I can reproduce this on our environment.
Regards,
Boyong

Hi Boyong,
firstly, you are using an older version of netopeer2-server, so please update all the libraries, not that long ago there were some enhancements to Call Home in libnetconf2 done, so maybe you do not have those and they can help. Other than that, I do not know why would the server shut down because I cannot find any reason for it in the log. Then again, you may be using such an old version that the reason for its termination was fixed.
Regards,
Michal

Hi,
I did not observe issue again.
it seems  some network issue that caused socket disconnection.


API: BatchListUserRevisions

Proto defs for #1109

@jtoohill Should I mark this is as blocked on #1110 ?

@jtoohill Should I mark this is as blocked on #1110 ?

I don't think so - that change has been merged.

Updated protos. PTAL


CustomInventoryBuilder should take Text.Translatable as name parameter

CustomInventoryBuilder uses
CustomInventoryBuilder name(org.spongepowered.api.text.translation.Translatable name);

shouldn't this be
CustomInventoryBuilder name(org.spongepowered.api.text.Text.Translatable name);

@zml2008 suggested creating this issue when I asked for help on the matter.

I'm aware that there are a few places in Inventory API where Translatable has mistakenly been used instead of Translation, particularly in the Inventory interface itself. I will change these to Translation all at the same time.

Item names can't be clickable etc.

Inventory names can't be clickable etc.

As shown with DisplayNameData and the Flardians plugin during the State of Sponge 3, Minecraft very well will render formatted text.

Yes but the information is replicated to the client as a String. As I said above, I'm changing it to Translation because Translatable was used by mistake.

@Mumfrey add this to the minor issues list?


Public recipe list

add cancancan authorization
Public recipe list
General shopping list view

Hi @fithamlak ,
Your milestone is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

Correct GitFlow ✔
Good implementations ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


API cleanup

[x] Return JSON on errors
[x] Return 201 on create, 404 on not found (ended up skipping 404 on on not found for now)
[x] already appears to have been fine, though perhaps I forgot what the problem was More informative errors for CRUD operations

[x] RowNotFound for get_XYZ
[x] Already exists for create_XYZ

[x] IntantiatedEntity and InstantiatedDatabase return integers for entity_type and db_type --- should they be serialized as more human-readable strings?

Resolved by #57


Perf: BN254 Miller loop

Actually we don't need to compute a full AddMixedStep here. We just need the line that goes through qProj and Q2 but not the new qProj as it won't be used after that.
This PR introduces the private function addMixedStelLineOnly that only computes the line and not the resulting point, and makes AddMixedStep and DoubleStep private for all curves.
benchmark                           old ns/op     new ns/op     delta
BenchmarkMillerLoop-2               210243        209380        -0.41%
BenchmarkMultiMiller/2_pairs-2      353040        352036        -0.28%
BenchmarkMultiMiller/3_pairs-2      495552        493883        -0.34%
BenchmarkMultiMiller/4_pairs-2      637746        635996        -0.27%
BenchmarkMultiMiller/5_pairs-2      779299        777806        -0.19%
BenchmarkMultiMiller/6_pairs-2      921408        919538        -0.20%
BenchmarkMultiMiller/7_pairs-2      1063256       1061098       -0.20%
BenchmarkMultiMiller/8_pairs-2      1206537       1202637       -0.32%
BenchmarkMultiMiller/9_pairs-2      1346727       1342862       -0.29%
BenchmarkMultiMiller/10_pairs-2     1488339       1485146       -0.21%

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: gbotrel:white_check_mark: ThomasPiellard:white_check_mark: yelhousni:white_check_mark: ivokub:x: AlexandreBellingYou have signed the CLA already but the status is still pending? Let us recheck it.


How to read serial data

I tried multiple snippets found in the documentation and here in the issues, but no luck.

It seems its reading the right data at a certain point but then its not working.
My guess is that it's either the buffer size or the new line char '\n', but I am not sure how to tackle that.

There's a bug in setting the configuration. See: https://github.com/jpnurmi/libserialport.dart/issues/24#issuecomment-812918216

@jdekarske you are right 😅 thanks a lot!
This is very very helpful since I couldn't  change the default baud rate for micropython(115200) without side effects. 🥳


Feedback for “SQL schema declaration”

Currently 2023-06-24, the docs say for the sqlite tab InferModel is imported from "drizzle-orm/sqlite-core"
import { InferModel, text, integer, sqliteTable } from 'drizzle-orm/sqlite-core';

const users = sqliteTable('users', {
  id: integer('id').primaryKey(),
  fullName: text('full_name'),
  phone: text('phone'),
})
...

InferModel is not exported from "drizzle-orm/sqlite-core" instead its exported only from "drizzle-orm"
import { integer, sqliteTable, text } from "drizzle-orm/sqlite-core";
import { InferModel } from "drizzle-orm";
...

You can now infer models without importing anything with the $inferSelect and $inferInsert methods on the tables.
https://orm.drizzle.team/docs/goodies#type-api


Create documentation for the TA Application

This PR adds documentation to accompany the new TA Application feature, outlining the current implementation and future concerns.

Wow, looks like GitHub does not allow neat comments to be added to files not currently in a PR, so I will manually start writing here:

[ ] Add attributions, as well as docstring descriptions, to the model files:
https://github.com/unc-csxl/csxl.unc.edu/blob/6dc26524cbfe8138c100b3e99fd356bad3bf9cc9/frontend/src/app/ta-application/application.model.ts#L1-L2
https://github.com/unc-csxl/csxl.unc.edu/blob/6dc26524cbfe8138c100b3e99fd356bad3bf9cc9/frontend/src/app/ta-application/rx-applications.ts#L1-L2
See the other files such as organization models for an example of his comment attribution!
https://github.com/unc-csxl/csxl.unc.edu/blob/6dc26524cbfe8138c100b3e99fd356bad3bf9cc9/frontend/src/app/ta-application/application.model.ts#L26
Especially, clarify what this is.

[] Home component renaming and attributions
https://github.com/unc-csxl/csxl.unc.edu/blob/6dc26524cbfe8138c100b3e99fd356bad3bf9cc9/frontend/src/app/ta-application/home/application-home.component.ts#L5-L9
This component should not be named home - not sure if I mentioned it in the previous CR, but it would be good to rename this to confirmation dialog to prevent confusion.
Also, add the attribution!

Document the TA Application Component
https://github.com/unc-csxl/csxl.unc.edu/blob/6dc26524cbfe8138c100b3e99fd356bad3bf9cc9/frontend/src/app/ta-application/uta-application/uta-application.component.ts#L28-L43
Since we were on a time crunch, I let this slide - but please add attributions to this file and also docstrings for the fields! In the future, we will need to rewrite this file to lessen the number of fields.
https://github.com/unc-csxl/csxl.unc.edu/blob/6dc26524cbfe8138c100b3e99fd356bad3bf9cc9/frontend/src/app/ta-application/uta-application/uta-application.component.ts#L227-L229
Specifically, add documentation for this function above.
https://github.com/unc-csxl/csxl.unc.edu/blob/6dc26524cbfe8138c100b3e99fd356bad3bf9cc9/frontend/src/app/ta-application/uta-application/uta-application.component.ts#L313-L315
Document this, especially what Omit<> is and why it is being used here. This has not been used in our codebase before.
Document the rest of the functions in this file!

[ ] UTA Notice:
https://github.com/unc-csxl/csxl.unc.edu/blob/6dc26524cbfe8138c100b3e99fd356bad3bf9cc9/frontend/src/app/ta-application/uta-notice/uta-notice.component.ts#L17-L20
What is this?

[ ] The Service
https://github.com/unc-csxl/csxl.unc.edu/blob/6dc26524cbfe8138c100b3e99fd356bad3bf9cc9/frontend/src/app/ta-application/ta-application.service.ts#L42-L46
I remember commenting about this in the CR, what are these being used for? We can just inject the academics service instead of using this, and I recommend that this change is made.
Also, add documentation for all functions here.

[] Consider Renaming ta-application to application

To keep with the consistency of the backend (which uses application for this feature), and allow for future expansion, it might be a good idea to rename the ta-application folder in the frontend to either application or applications.
@bwgoulet, @alshayefaziz, @KrisJordan

@bwgoulet / @alshayefaziz how is the progress here?

@alshayefaziz / @bwgoulet we are closing this PR.


Does scollector support Event Tracing for Windows (ETW) on Windows?

There isn't anything build into scollector for ETW. We use an OLE package to access some WMI counters, but even then it is specific counters that are hard coded.
You can always use external collectors to capture whatever metrics you want. There is a sample PowerShell collector that could be converted to gather data from ETW.


When to use enable_fsdp_float8_all_gather?

I find enable_fsdp_float8_all_gather=True is not obviously faster on 8 nodes (64 GPUs).
When could we see the benefit by setting enable_fsdp_float8_all_gather=True?

Answered here https://github.com/pytorch/torchtitan/issues/651


net/http: stop ExampleServer_Shutdown from hanging on error

Running the example code when not having permissions
to bind to port 80 will cause the program to hang after
printing the error message.

I signed it!


Fetching vulnerabilities via api for ubuntu packages by purl with version does not work

Hello everyone,
I tried to fetch vulnerabilities via api with the purl (Package URL). The SBOM from which I am extracting the packages, the purl has following format: pkg:deb/ubuntu/PACKAGE_NAME@VERSION.
What I've tested:

the api works with purls like this for e.g. maven packages. Test:
curl -d '{"package": {"purl"<EMAIL_ADDRESS>https://api.osv.dev/v1/query
Ubuntu purls work if i know the arch and the distribution. Test:
curl -d '{"package": {"purl": "pkg:deb/ubuntu/ruby-sinatra?arch=src?distro=jammy"}}' https://api.osv.dev/v1/query
It works if I split the purl in package-name, version, namespace(==ecosystem). Test:
curl -d '{"package": {"name": "atftp"}, "version": "0.7.git20120829-3.1~<IP_ADDRESS>", ecosystem: "ubuntu"}' https://api.osv.dev/v1/query
But giving the purl only for the same package and version like in 3., returns nothing. Test:
curl -d '{"package": {"purl"<EMAIL_ADDRESS>https://api.osv.dev/v1/query
Additionally i found out that this works for debian packages coming directly from Debian. Test:
curl -d '{"package": {"purl"<EMAIL_ADDRESS>https://api.osv.dev/v1/query

Right now I use (3.) as a workaround, but i would be nice if it could work like (1.).Thank you very much for your help in advance.Greetings,
André

@hogo6002 can you help to answer this?

If you could help this would be great. Meanwhile it tried to understand how the osv.dev service works. My plan was to run a api-server in a Docker/Podman container and test patches on it and connect it to osv.dev database in the internet. Setting up with Podman works:
`
podman build --file gcp/api/Dockerfile -t osv-api .
podman run -p 8000:8000 osv-api:latest
`
But because of missing/wrong ndb credentials I cannot connect to osv.dev database. Did I do a mistake or is such a test setup not possible at all because the access to the database itself is private to google's osv.dev service?

Hi @Andre-85, thanks for reporting this issue!

But because of missing/wrong ndb credentials I cannot connect to osv.dev database. Did I do a mistake or is such a test setup not possible at all because the access to the database itself is private to google's osv.dev service?

You didn't make any mistakes, and this is by design. Currently, only we can set up the API locally because it requires the ndb credentials.
The issue is due to the ecosystem being falsely parsed from the PURL in OSV. We currently get the ecosystem name from purl.type, which works for project ecosystems such as npm and Maven. However, for Linux distributions, several ecosystems may use the same package management system. For example, both Debian and Ubuntu use Debian package management, and the purl.type here would be deb, so we treated the Ubuntu query as a Debian version query. This is why the version query works with Debian but not with Ubuntu (we perform an ecosystem check for version queries).
PURL spec: scheme:type/namespace/name@version?qualifiers#subpath
To solve this, we will use purl.namespace to get the ecosystem for Linux distributions.
Thanks again for reporting this with a detailed explanation of the problem. A fix will be coming soon!

That's great news! I can't wait for testing it!
Thank you very much 👍

The fix has been deployed to prod, and PURL queries for Ubuntu are now working as expected.

Yeah, it works! Thank you so much for your great work! :)


Read gzip CIFF file

Detect if the input CIFF file is gzip encoded and decode the entire file to byte buffer. Otherwise use the existing mmap buffer implementation for uncompressed CIFF files.
Also looking for any comments on if this could be improved in any way. Thanks.

@mpetri - any comments?

The limitations of this patch are that mmap is not used for gzip files.  This means that (using CC-News as an example) an uncompressed ciff file will index CC-News with 256GB RAM (with the help of mmap), while a gzip compressed ciff file will require >256GB RAM due to not using mmap and being dumb by decoding everything to a byte buffer (/usr/bin/time reported around 305GB RAM).  Smaller gzip CIFF files should work ok though of course.
While it does work, the gzip version can use up too much resources. I've been looking at ways to fix that so that mmap is used in all cases, and since we make multiple passes over the ciff file, we can build a "gzip index" on the first pass to decode sections of the gzip file on the fly. It turns out to be more involved that I thought it would be. It may take me sometime to get a prototype working though and flate2 doesn't support all of the zlib operations we need to make the index of entry points. Also not clear (to me) how this would work with parallel iterators either.
So there are a number of considerations here: accept the patch with its limitations (and wait for possible improvements); reject it because of the limitations as there is low benefit; possibly wait for an improved way of processing gzip ciff files with mmap support.

Coming back to this...
I think we merge and revisit later, it's unlikely anyone will be reading >RAM sized CIFF files I guess?

Agree that we revisit later. I'm in favor of not merging now. And, if it is revisited then it will probably be useful and have some benefit to some usecase, and at that time the details can get another look then?

OK, I wont merge, will convert to draft and leave it as-is for now. Thanks!

Some of my intentions that were missing from the previous comment. It is not clear to me that the changes in their current form, are a net improvement.
Therefore, this issue could be closed and be available for reference if this gets revisited in the future.


use Duration for timeouts

This PR replaces u128 millisecond based timeouts with std::time::Duration and adds new connect method connect_timeout(..) where timeout for establishing connection can be specified.

I've submitted a new PR to your branch, which aims to reuse the global timeout settings when calling TCP connect.
Is that okay with you?

Merged your changes.

Appreciate your help & excellent work


Adding s390x cross build support

Adding cross build binary support for s390x.

/check-cla

@pohly @msau42 Could you please have a look ?

@pohly @msau42 Could you please have a look ?
Corrected syntax  for ARCH detection, observed that current if block is not creating binaries for mentioned architecture.
Updated if condition and verified it manually .

/ok-to-test

/assign @pohly

The change itself looks good, I'm just wondering whether should continue to add special build rules like this for different target architectures. Should we just add them for now when people ask for it, or should we stop and try to find a more generic approach, in particular one which then also covers building and publishing of images?
As I don't have the answers for those problems, I myself lean towards simply merging this.

@pohly Thanks for your comments. Yes we are also planning to work on generic images, however for now, adding above would be helpful.

Closing in favor of #76


[CircularProgress] Improve the animation

I still have to address an issue with iOS:

@oliviertassinari I think that's Safari in general, not just iOS

@nathanmarks Yes indeed. As far as I have investigated, that sounds like an issue with calc in the strokeDasharray.


Remove CONDITION from event object completely

Closes #3172
Removes the CONDITION column from "kubectl get events" output and from the Event object
See #3172 for rational

I am good with this.  @lavalamp ?

LGTM-- just a few nits

Nits addressed. Commits squashed.

E2E looks good:
2015/01/14 14:05:33 Passed tests: basic.sh[1/1] certs.sh[1/1] goe2e.sh[1/1] guestbook.sh[1/1] liveness.sh[1/1] monitoring.sh[1/1] pd.sh[1/1] private.sh[1/1] services.sh[1/1] update.sh[1/1]
2015/01/14 14:05:33 Flaky tests:
2015/01/14 14:05:33 Failed tests:
2015/01/14 14:05:33 Success!

LGTM, will merge when travis finishes.


Windows Installer shows warning that “The verified publisher is unknown” - enhancement.

For both windows7 and windows10, there is warning message that "The verified publisher: unknown".
Apparently the user will consider that the software is not published officially.

The problem was fixed in version 0.4.0 of installer. Is this problem relevant to version 0.4.0 or previous?

This issue has been found on v0.33 via https://github.com/ARMmbed/mbed-cli/tree/windows-installer.
Just got the new link of v0.4 and verified ok now.