Feature request: Collapsing parts of SQL code

It's not happening for me a lot but it might happen a lot for other people but currently I'm writing pretty big stored procedure (currently 257 lines but I'm not even in half of writing it). What would help to write and also read it in future is code collapsing feature so I can collapse SELECTs, IFs, WHILEs, etc.
Another useful thing would be to highlight clause's points.
For example:

when I "stand" in SELECT word it would highlight that word and also the semicolon which ends that clause
when I "stand" in IF word it would highlight IF and ELSE/ELSEIF and END IF
etc.

Oh. And maybe some "TODO" list which reads it's "contents" from comments beginning with "#TODO:"

Big fan of the Collapse request.  Used another editor that would do this (can't remember which one, probably TOAD for Oracle).
Some nice features:

the editor would put a + sign at the top of a section of comments.  Pressing it would collapse all of the contiguous comment lines into a single dashed line.  Pressing it again would expand it.
the editor would put a + sign at the top of a section of SQL -- same behavior.  The "section" would typically be anything that could be easily discerned: a set of parentheses; the major sections of the query (SELECT, FROM, WHERE); any sub-query (again, parentheses)... etc.

I can think of some relatively easy alternatives, too:

A "button" to "remove all newline (carriage return) characters in the selected section of code
(and an UNDO for that!)

I though Collapse feature have been implemented in 2017, but I was not able to find this. Please Confirm whether it is implemented.

I'm on 6.0.0, windows, and it's available on Databases -> Editors-> SQL Editors-> Miscellaneous  (Folding Enabled).

yrimal is right, there is limited "folding" (collapse code) available, but it could be much better (more useful).  Today it only allows you to collapse an entire query.  Better if it would allow you to collapse smaller parts like:

a whole sub-query within a larger query
a long CASE statement
the Select section of a query, but not the FROM and WHERE (like, when you know what fields you want and you're working on the joins and filters...)

The best implementation I've seen is to simply let you select any block of text you want, and collapse it (like hiding columns or rows in a spreadsheet)

Hi @yrimal, Thank you. I have found the setting.

For anyone reading this... I found an interim "workaround".  In dBeaver, you can take a section of SQL code, and put it in a variable using the @set command.  Then you can call that variable in your query.  This is sortof like collapsing the code - makes it easier to read.  (It's also a handy trick if you repeat the same logic multiple times in your query -- for example, repeating the same CASE statement multiple times.)
Example:
@set FROMFIELDS = field1, field2, field3, field4, field5, CASE WHEN field6 = value1 THEN value2 ELSE value3 END
/* this can be a very long string of stuff */
SELECT ${FROMFIELDS}
FROM table1
WHERE ... etc.

The ticket duplicates #2156

It would be great to be able to create "sections", from #beginSection ... to #endSection, so you can use those 2 tags to delimit the section and collapse whatever you want in between.

So, is this implemented or not?


LTE-FPV version with usb-modeswitch

remove wfb-ng and more packages
add usb_modeswitch
changed /etc/network/interfaces
changed /packages/majestic/majestic.mk (fix for compile as fpv)
tested at huawei e3372h

When compiling a new config, I ran into a situation where one of the name elements must be among ultimate , lite or fpv and it must be located last. This was about majestic.mk, although I still had to make corrections. But a number of commands for trimming the defconfig name, as I understand it, expect a mandatory part like fpv to be located last.
The very name of the config indicates that the assembly is intended for use in conjunction with a 4G LTE modem and at the same time with FPV, more precisely as a blank for such use. In fact, the majestic streamer and a number of tunnels / vpn remained there. And the means for the operation of the e3372 modem were integrated.


Add Benchmark test

to run kbench and record results periodically
for issue #3540
Signed-off-by: Yang Chiu<EMAIL_ADDRESS>

I'm thinking if we can measure the average CPU usage of each pod.

I'm thinking if we can measure the average CPU usage of each pod.
There are some issues related to the high CPU usage of instance-managers. Maybe we can provide more statistics in the benchmark. longhorn/longhorn#3889

There is a CPU idleness profiling option in kbench, but it's disabled by default because:
CPU_IDLE_PROF environmental variable: the CPU idleness profiling measures the CPU idleness, but it introduces extra overhead and reduces the storage performance. By default, the flag is disabled.
Should we enable it?

I'm thinking if we can measure the average CPU usage of each pod.
There are some issues related to the high CPU usage of instance-managers. Maybe we can provide more statistics in the benchmark. longhorn/longhorn#3889

There is a CPU idleness profiling option in kbench, but it's disabled by default because:
CPU_IDLE_PROF environmental variable: the CPU idleness profiling measures the CPU idleness, but it introduces extra overhead and reduces the storage performance. By default, the flag is disabled.
Should we enable it?

No, we can't.
Except for this way, maybe need a pod measuring CPU usage using kubectl top periodically.

Fixed:
(1) make TEST_SIZE configurable
(2) reuse terraform scripts from regression tests
For CPU usage measurement, tracked by https://github.com/longhorn/longhorn/issues/3903

@yangchiu Two comments needed to resolve, then we can merge this. Thanks!

Tabs are replaced with spaces.
For cleaning up orphaned resources, #3913 has been created to track this.


Avoid race condition in DefaultJsonNameTable.Get by ensuring array index is evaluated before accessing the array

From https://github.com/JamesNK/Newtonsoft.Json/issues/1753
Couldn't see a way to unit test this so put a comment in to make sure it doesn't get reverted

What if we mark _entries with volatile keyword?

I think even better approach is remove _mask and just compute it:
var entries= this._entries;
var mask = entries.length - 1;
use entries and mask.

I added your change with https://github.com/JamesNK/Newtonsoft.Json/pull/1781
I made a small modification to also copy the entries to a local variable.


Unable to create/start local cluster on Windows 7 machine

We have windows 7 with VS 2015 Update 3 and Service Fabric Runtime 6.1.456.9494 installed along with Service Fabric SDK 3.0.456. There are no errors while installing either the runtime or sdk, but still while setting up local cluster it fails with following errors, and this is happening in all the machines we are using.
2>WARNING: Service Fabric Cluster is taking longer than expected to connect.
2>
2>Waiting for Naming Service to be ready. This may take a few minutes...
2>Connect-ServiceFabricCluster : No cluster endpoint is reachable, please check
2>if there is connectivity/firewall/DNS issue.
2>At C:\Program Files\Microsoft SDKs\Service
2>Fabric\Tools\Scripts\ClusterSetupUtilities.psm1:620 char:12
2>+     [void](Connect-ServiceFabricCluster @connParams)
2>+            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2>    + CategoryInfo          : InvalidOperation: (:) [Connect-ServiceFabricClus
2>   ter], FabricException
2>    + FullyQualifiedErrorId : TestClusterConnectionErrorId,Microsoft.ServiceFa
2>   bric.Powershell.ConnectCluster
Faulting application name: Fabric.exe, version: 6.1.456.9494, time stamp: 0x5a66710e
Faulting module name: FabricCommon.dll_unloaded, version: <IP_ADDRESS>, time stamp: 0x5a666cec
Exception code: 0xc0000005
Fault offset: 0x000007febd3e6d31
Faulting process id: 0x2854
Faulting application start time: 0x01d3a64824f31fdc
Faulting application path: C:\SfDevCluster\Data_Node_0\Fabric\Fabric.Code\Fabric.exe
Faulting module path: FabricCommon.dll
Report Id: 62dd8dbf-123b-11e8-bbb0-00505699504f
I don't see anything useful in Trace logs either.
CrashDump.zip
TraceLogs.zip
I tried resintalling microsoft visual c++ 2012 redistributable package (x64) , tried running cleanupfabric , cleanupcluster but still the issue persists.
Also unisntalled and reinstalled service fabric runtime and sdk multiple times, still issue persists.

Please re-open if you are still facing this issue, but closing at this time.


Github image resizing (new feature request)

Unfortunately, github does not support image resizing.
See https://stackoverflow.com/questions/24383700/resize-image-in-the-wiki-of-github-using-markdown
It would be good that this markdown extension be configurable to insert the HTML code instead of the markdown if requested. So the user can specify the image size.
Instead of :
![picture 3](images/01-cmake-open.png)
Having the possibility to get:
<img src="images/01-cmake-open.png" width="48">
For instance in the config:
  "markdown-image.base.codeType": "markdown" | "html",
  "markdown-image.base.imageWidth": 48,

This feature was release. You can update to 1.1.9 to use it.

Thanks a lot, I am going to test it.
On Monday, May 17, 2021, Hancel Lin @.***> wrote:

This feature was release. You can update to 1.1.9 to use it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/imlinhanchao/vsc-markdown-image/issues/39#issuecomment-842175900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVV6TXOQS4BFGR7VFGKYCDTODPLBANCNFSM44U4CPHA
.

--
Envoyé avec Gmail Mobile

Hi~Is that feature match what you want? If it's ok. please close this issue.


Add did:github provider?

Is your feature request related to a problem? Please describe.
I would like to quickly prototype a network of agents representing public organizations issuing and requesting credentials  over DIDComm. Here is an example of a QR out-of-band messaging refering to the DIDs of two public institutions:
{
    "typ": "application/didcomm-plain+json",
    "type": "https://didcomm.org/out-of-band/2.0/invitation",
    "id": "<id used for context as pthid>",
    "from": "did:github:HappyCars",                                          // <-- did:github
    "body": {
        "goal-code": "id.happycars.start-car-rental",
        "goal": "Rent a car from Happy Cars Inc",
        "requires": [
            {
                    "type": "DriversLicense",
                   "issuer": "did:github:TheOfficialTrafficAuthority"               // <-- did:github
            }
        ]
    }
}

I currently only see providers for did:eth , did:web and did:key .

did:eth provider - https://github.com/uport-project/veramo/blob/next/packages/did-provider-ethr/src/ethr-did-provider.ts
did:key provider - https://github.com/uport-project/veramo/blob/next/packages/did-provider-key/src/key-did-provider.ts
did:web provider - https://github.com/uport-project/veramo/blob/next/packages/did-provider-web/src/web-did-provider.ts (implementation missing for addService)

% veramo did create
? Select identifier provider (Use arrow keys)
❯ did:ethr
  did:ethr:rinkeby
  did:ethr:ropsten
  did:ethr:kovan
  did:ethr:goerli
  did:web
  did:key

Describe the solution you'd like

I would like to be able to do veramo did create  and select did:github . Then also do veramo did add-service  to add a DIDCommMessaging endpoint.
After this has happened I would expect the index.jsonld DID-document to be updated @ https://raw.githubusercontent.com/USERNAME/ghdid/master/index.jsonld
did:github spec - https://github.com/decentralized-identity/github-did/blob/master/docs/did-method-spec/index.md)

Describe alternatives you've considered

I have considered using the did:eth, but this has two downsides for my prototyping use-case: 1. It requires funding. 2. The identifiers which are created are not human-readable, like in my example above with the.
I have considered waiting around to see if Veramo could support did:3 running on Ceramic.

Hi @Arxcis
It should be fairly easy to to support such a provider in Veramo.
Veramo is extensible in many ways, so you could write your own provider for various DID methods.
We distinguish 2 aspects when it comes to supporting DID methods.

resolving a DID
creating/updating a DID document.

For did:githubspecifically, resolving should be really straight forward, as it is a simple HTTP fetch; very similar do did:web
Creating/updating would involve a little bit more work as it would require interacting with the github API. Not difficult, but also not just a single line of code.
You can take a look at how @veramo/remote-server serves a DID document for a did:web. It's likely that a did:github provider would use a similar approach to construct the document for upload.

That being said, it should be relatively easy to use the existing did:web support from Veramo to point directly to a github hosted file, and update it manually, although with the current version of Veramo it would be a bit hacky:

Create a proxy DID:

veramo did create
# pick did:web
# use example.com as alias

veramo server

Then, in another terminal, run:

curl -H "host: example.com" localhost:3332/.well-known/did.json | sed "s/did:web:github.com/did:web:raw.githubusercontent.com:GITHUBUSERNAME:ghdid:main/g" > did.json

Of course, use the real github handle instead of GITHUBUSERNAME

upload the resulting did.json to your github.com/GITHUBUSERNAME/ghdid/did.json
resolve your new DID document to check:

veramo did resolve did:web:raw.githubusercontent.com:GITHUBUSERNAME:ghdid:main

Example:
veramo did resolve did:web:raw.githubusercontent.com:mirceanis:ghdid:main

You would do this every time you update your github hosted did:web

Thank you for the guide @mirceanis  🙏
I tried to do what you suggested.

Create a did:web

% veramo did create
(node:8591) Warning: Accessing non-existent property 'abort' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
? Select identifier provider did:web
? Select key management system local
? Enter alias example.com
┌──────────┬─────────────┬─────────────────────┐
│ provider │       alias │                 did │
├──────────┼─────────────┼─────────────────────┤
│  did:web │ example.com │ did:web:example.com │
└──────────┴─────────────┴─────────────────────┘

Start the server:

 % veramo server
(node:8613) Warning: Accessing non-existent property 'abort' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
🚀 Cloud Agent ready at http://localhost:3332
🆔 did:web:localhost
📨 Messaging endpoint http://localhost:3332/messaging

Curl for the did.json

% curl -H "host: example.com" localhost:3332/.well-known/did.json
{}%

I got an empty object as the result from my CURL. Do you spot an mistakes I did along the way?

Right, I forgot to specify that you need to use @veramo/cli@next
Can you try to run the server using npx @veramo/cli@next server and then running the curl command ?
It should be OK to use the same config file and database.

Right, I forgot to specify that you need to use @veramo/cli@next
Can you try to run the server using npx @veramo/cli@next server and then running the curl command ? It should be OK to use the same config file and database.

This solved my issue. Thank you very much for the quick support 🙏 I think using did:web the way you have suggested here is OK for our use case for now. I will re-open this issue, if a decision to make a did:github-provider is made.


Unnecessary target attribute should cause an error

I found this in one of the tests that currently passes:
#[derive(Delegate)]
#[delegate(Shout, target = "bar")]
pub enum Either<A: Shout, B: Shout> {
    Left(A),
    Right(B),
}

The target attribute isn't needed and doesn't point to anything, so it should throw an error, so we avoid confusing code.

I think unnecessary target attributes should be supported (for example, for some code that is generated automatically). For similar reasons as trailing commas.
If unnecessary target attributes cause are known to cause errors for users then additional attribute like allow_unnessesary_target can be added (or ideally when proc macros gain ability to use custom lints, it can be just a deny-by-default lint).


bootstrap-node: fix hax startup of client-only nodes

Currently, hax starts on the nodes which run confd or ios
which is wrong. Client-only nodes should have hax running also.
Solution: fix bootstrap-node to start hax always on the node,
regardless of its configuration.
Signed-off-by: Andriy Tkachuk<EMAIL_ADDRESS>

@andriytk, what about consul agent? does consul agent start on the client node ?

Yes, it does.

oh yes, I forgot, sorry for the noise.
while read node bind_ip; do
    ssh $node "PATH=$PATH $(which mk-consul-env) \
                          --mode client --bind $bind_ip --join $join_ip &&
               sudo systemctl start hare-consul-agent" &
    pids+=($!)
done < <(get_client_nodes)


Delete "Default value: 90 days"

Delete "Default value: 90 days" from Password expiry duration (Maximum password age), as default is not set since 2021.

@rui0122 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Is path mapping for "Resolve Dependencies as TypeScript Modules" really the best solution?

[...]

TypeScript's path mapping is the best solution.

How so? If you have a more elaborated (TM) file layout such as:

components/services/* having modules like @scope/service1
shared/runtime/*... having modules like `@scope/runtime1
models/* ... having modules like @scope/model1
You would need to add them (path mappings) explicitly, because a generic match like the one you propose only works, if all modules are having the same folder depth/layout.

This hinders scalability and it in general a path mapping does not allow to refactor the file layout, without modifying the path mapping.
I think, this is the wrong approach; lerna bootstrap is explicitly abstracting the file layout of your modules away by sym-linking them correctly. Thus enabling "preserveSymlinks": true in each individual module's tsconfig.json is to me the more flexible (and hence better) solution.
However I like your point of view, why you settled with the path mapping option. Care to elaborate?

hey @ChristianUlbrich ,
i think, this may be a relic from the typescript 2.x times. TypeScript 3.x introduced the project references, which may be more appropriate in such scenarios..
how did you manage to solve your issue?!


chore(package): update @types/node to version 8.0.7

Codecov Report

Merging #174 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #174   +/-   ##
=======================================
  Coverage   95.78%   95.78%
=======================================
  Files         299      299
  Lines        6022     6022
  Branches      307      307
=======================================
  Hits         5768     5768
  Misses        254      254

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 3f10f54...43f3b0a. Read the comment docs.


Open / closed competition?

Is it possible to use external data sources (externally-trained embeddings, Wikipedia, parsing/processing) or should we just train on the data available here ?

This is a very important question. Thank you for posting it here.
We have updated the participate tab of the competition website on CodaLab with the following clarification:
We expect that a successful system will rely on existing linguistic resources. We don't know in advance which resource or which combination of resources is best to be used for our task. Your system can use any resources created automatically (e.g. corpora of naturally occurring texts, word embeddings, knowledge graphs) or by aggregating expert knowledge (e.g. WordNet, FrameNet or other lexicographic resources).
However, you are not allowed to use resources derived from data elicitation from human subjects, such as free association norms, feature norms, relatedness judgements, priming effect data etc.


Update from latest + Bug fix on packed repeating

I also code a little script to convert AeonLucid proto to this format (that's why files are named differently here).
I can share if of any interest.

I'm not sure what you did here.
I just checked by updating my local copy to AeonLucid's latest protos and the only thing that changed between v2.4.0 (which is the current version in node-pogo-protos) and v2.4.2 (latest) is a single comment change. To me it looks like the only thing that changed in this PR is that everything is split into lots of individual .proto files.
As for the bug fix, I'm a little confused why this is suddenly necessary when it worked before. Was there a change in protobuf.js recently? Were you getting errors without the fix?

Fair question :)
I was looking for a proto that did not exist in your repo, so I ended up coding a tool to automate converting format from Aeon to yours. At the end, the proto I was looking for did not exist either in Aeon...
I don't know if you update manually or not, with this PR it looks more like the file in Aeon's. That's really up to you.
About the bugfix, it happens only in repeating field inside message within a message, for example in signature. And I guess there isn't to much people working with signature.

Which proto is the one that was missing? There should be no difference between AeonLucid's current protos and the ones in node-pogo-protos.
Updating the protos is actually pretty easy but looks like I've never actually documented it. I'll go ahead and add some documentation.

it's GET_STORE_ITEMS_REQUEST, as I said it's not in AeonLucid repo either but I just thought it did :)

Whoops, I misread that, sorry.
The system we have in place for updating the protos is pretty solid (I'll add documentation in a minute) so if it's fine with you I'd like to close this PR and you could open a second one with only the bugfix.

Sure, make sense.


[Bug] possible regression on ededupe code in release dev3

Search before asking

[X] I searched the issues and found no similar issues.

Component
Transforms/universal/tokenization
What happened + What you expected to happen
In dev1 release
https://github.com/sujee/data-prep-kit/blob/rag-example1/examples/notebooks/rag/rag_1A_dpk_process_ray.ipynb
Step-3: Chunking
output
Files processed : 3
Chunks created : 2,042
Input data dimensions (rows x columns)=  (3, 12)
Output data dimensions (rows x columns)=  (2042, 15)

Step-4 : EDedupe
Output
Input data dimensions (rows x columns)=  (2042, 15)
Output data dimensions (rows x columns)=  (1324, 15)
Input chunks before exact dedupe : 2,042
Output chunks after exact dedupe : 1,324
Duplicate chunks removed :   718

In Dev-3
https://github.com/sujee/data-prep-kit/blob/rag-example1-dev/examples/notebooks/rag/rag_1A_dpk_process_ray.ipynb
Step-3: Chunking
output
Files processed : 3
Chunks created : 1,973
Input data dimensions (rows x columns)=  (3, 12)
Output data dimensions (rows x columns)=  (1973, 15)

Step-4: EDedupe
output
Input data dimensions (rows x columns)=  (1973, 15)
Output data dimensions (rows x columns)=  (2, 16)
Input chunks before exact dedupe : 1,973
Output chunks after exact dedupe : 2
Duplicate chunks removed :   1971

Note: how resulting number of chunks is only 2
This is the problem.
The result is not chunks but the documents!
This in turn breaks vector search and RAG responses.
Reproduction script
dev1 (expected behavior) : https://github.com/sujee/data-prep-kit/blob/rag-example1/examples/notebooks/rag/rag_1A_dpk_process_ray.ipynb
dev3 (incorrect behavior) : https://github.com/sujee/data-prep-kit/blob/rag-example1-dev/examples/notebooks/rag/rag_1A_dpk_process_ray.ipynb
Anything else
No response
OS
Ubuntu
Python
3.11.x
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

@sujee can you reproduce this problem in ededup w/o first running the chunker.  That is, ededup all by itself.  The chunker does seem to produce different results in the two cases above, so is that somehow the real problem?

another example working on simpler PDFs is here : https://github.com/sujee/data-prep-kit-examples/blob/main/dpk-intro/dpk_intro_1_python.ipynb

After reviewing with @blublinsky we found out that

what you observe is because ededupe is using the doc_id column as a first criteria
the doc_chunk transform is keeping the source doc_id, i.e. all chunks belonging to the same source document will have the same doc_id

We have two solutions

In the current notebook, after doc_chunk we could run the doc_id transform pointing it to a different column, i.e. hash_column="chunk_id", then ededupe can be configured with doc_id_column="chunk_id".
We update the doc_chunk transform such that it renames doc_id to source_doc_id. This way the doc_id can still be used as a "row identifier".

If nobody objects, we can go with both solutions. 1) should easily unlock you, 2) will also make sure we don't fall in the same trap in the future.

I still think that the second solution is a better option. From user's point of view Doc_id is a row identifier and I will preffere to keep it this way to avoid confusion in the future

this seems like it could be a problem from any transform that does splitting of rows.  Would another solution be to run the doc_id transform just prior to ededup and make this a general recommendation when using e/fdedup?

For me the bigger issue was that doc-id column was misleading

@dolfim-ibm @blublinsky  thanks for investigating.
Is (1) I can do within my notebook 100% ?  If so, I will try that, while we put in a long term solution (2).
If I can get some sample code for (1) that would be very helpful.  thx

We have two solutions

In the current notebook, after doc_chunk we could run the doc_id transform pointing it to a different column, i.e. hash_column="chunk_id", then ededupe can be configured with doc_id_column="chunk_id".
We update the doc_chunk transform such that it renames doc_id to source_doc_id. This way the doc_id can still be used as a "row identifier".

If nobody objects, we can go with both solutions. 1) should easily unlock you, 2) will also make sure we don't fall in the same trap in the future.

I was able to use method (1) get ededupe work as expected.  Thanks every one! :-)

Great.
Thanks @dolfim-ibm for fixing this so quickly
Can we, please, close this one.


Feat/fix hardware wallet UI defects

Checklist

[x] JIRA - <link>
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
This PR bumps Cardano JS sdk to enable data sign for Ledger hardware wallet, and also includes a few small UI fixes for signData.

Allure Report
allure-report-publisher generated test report!

processReports: ✅ test report for 7667c850

passed
failed
skipped
flaky
total
result

Total
32
0
0
0
32
✅


amplify push produces "Cannot convert undefined or null to object"

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
16.2.0
Amplify CLI Version
5.1.0
What operating system are you using?
macOS Big Sur
Amplify Categories
api
Amplify Commands
push
Describe the bug
amplify push produces "Cannot convert undefined or null to object"
Expected behavior
amplify push update my cloud resources
Reproduction steps

Create some GraphQL schema with amplify add api
Create a custom function with amplify add function

Make this function depend on api, in my case, the function depends on the GraphQLAPIIdOutput attribute from api (yes I updated backend-config.json manually).

Delete an existing model in the GraphQL schema.
amplify push

The exception happens at this line in amplify-category-function. For some reason the function-parameters.json generated for my function doesn't set the permissions property so selectedCategories is undefined, when amplify tries to get values from it a TypeError is thrown.
function-parameters.json is by default hidden in the vscode project so I never paid attention to it, pretty sure I never updated this file.
After change the function parameters file from {} to {"permissions":{}} the command succeeded. This bug can be fixed by replacing this line from
const selectedCategories = currentParameters.permissions;

to
const selectedCategories = currentParameters.permissions || {};

GraphQL schema(s)
See "Reproduction steps"
Log output
See "Reproduction steps"
Additional information
See "Reproduction steps"

Thank you for reporting this, and the detailed explanation of the cause. I'm going to close this as a duplicate of https://github.com/aws-amplify/amplify-cli/issues/7466. There is also a PR to address this at https://github.com/aws-amplify/amplify-cli/pull/7594.


'Get-SFTPFile' does not exists in v3.0

Trying to use it returns:
CommandNotFoundException: The term 'Get-SFTPFile' is not recognized as the name of a cmdlet, function, script file, or operable program.
Note: Worked fine in 2.3

That commqnd was replaced by get-sftpitem. Replaced commands are in the changelig.md
Sent from my iPhone

On Sep 1, 2021, at 4:00 AM, Neven Dinev @.***> wrote:

Trying to use it returns:
CommandNotFoundException: The term 'Get-SFTPFile' is not recognized as the name of a cmdlet, function, script file, or operable program.
Note: Worked fine in 2.3
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Thanks, I will try it

This has impacted a few of our scripts for SCP. Some systems will only have 2.3.0 installed not 3.0.0. which I'm ok with. I can update the scripts to use Get-SCPItem, but how can I prevent the script from running if 3.0.0 not installed on the system?

as far I remember XXX-SCPItem already exists in v2.2
but you always can test module version
(Get-Module Posh-SSH -List).Version

Max nailed it :) that would be the best way to check
Sent from my iPhone

On Jan 11, 2022, at 2:35 AM, Max Kozlov @.***> wrote:

as far I remember XXX-SCPItem already exists in v2.2
but you always can test module version
(Get-Module Posh-SSH -List).Version
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.

Actually didn't realise scpitem was in the older module. That's totally fine then. Thanks for The quick responses.


obviously this project is no longer maintained - alternatives?

It seems that this project is no longer maintained. And with the new google policies regarding OOB OAuth2 it will completely stop working.
Anyone know of any alternatives? Or anyone willing to continue development of gcalcli?

Why is this closed? Was there any development we're not aware of? gcalcli has not been working properly for a while now and people's questions are not being answered. Also there are no updates or transparency on what will happen with the new google policies and updates.

What about the rest of the questions raised regarding gcalcli?


Adds countdown timer

I have made a countdown timer built using HTML, CSS and vanilla JS. Would appreciate if you will accept my PR.

Great Contribution @ayushknath


Remnant .git directory that doesn't let us clone again

For option to install chart from branch there is remnant .git directory that doesn't let us clone the directory again. We should make sure that while we cleanup inside the charts directory we should also remove the .git directory if it exists.

Believe it is fixed with commit 5e9031


Drop ':z' bind option when using Podman and MacOS

What this PR does / why we need it: This MR drops ':z' bind option when using Podman and MacOS as it may not work correctly, see https://github.com/containers/podman/issues/13631. If the kubevirt repo is used to do for example make test without this MR it may fail with:
Error: lsetxattr /Users/januszm/.docker/secrets: operation not supported

Special notes for your reviewer: More about ':z' and ':Z' bind option, see: https://docs.docker.com/storage/bind-mounts/#configure-the-selinux-label
Release note:

NONE

/retest

/lgtm
/approve


Add extra safeguard to keep OLMT and ELM in sync

Might already be moot, but proposing an extra safeguard in L. 102-104 below to grab specific commit of OLMT to ensure this Dockerfile works with simulation_containers/docker/elm/elm_v2-for-ngee/e3sm_source/e3sm_v2.0.0_ngee.tar.gz in case of stray/early commits to OLMT.

Strange - I am getting this error
10-4-10-218:OLMT sserbin$ git reset --hard 'd1c1bb170777ccab59b0b84780da53278167df89'
fatal: Could not parse object 'd1c1bb170777ccab59b0b84780da53278167df89'.

Oh - I see.  The default is to only grab depth 1 so it doesnt see the past commits.  easy fix.  I see in your Docker file you already fixed that with depth 10!  my bad

Yep, exactly. Now that I think about it again, might actually be safer even to remove the depth argument altogether (or create a tag/submodule of olmt that can be checked out), since if there are more than ~5 more commits to the branch this will fail again (though hopefully we avoid this by keeping everything in separate forks).
I'm building a separate container today that will pull in @fmyuan 's fork of E3SM for testing to help make sure everything matches up in the end.

OK - agreed. removing the depth arg, agree, for now at least, probably best to keep single branch but not restrict past commit history

Sorry that I didn’t mention that I am out of town .
Fengming
On Fri, Aug 26, 2022 at 12:55 PM Rich Fiorella @.***>
wrote:

Yep, exactly. Now that I think about it again, might actually be safer
even to remove the depth argument altogether (or create a tag/submodule of
olmt that can be checked out), since if there are more than ~5 more commits
to the branch this will fail again (though hopefully we avoid this by
keeping everything in separate forks).
I'm building a separate container today that will pull in @fmyuan
https://github.com/fmyuan 's fork of E3SM for testing to help make sure
everything matches up in the end.
—
Reply to this email directly, view it on GitHub
https://github.com/FASSt-simulation/simulation_containers/pull/25#issuecomment-1228719730,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAO4ZQEIVRJYE265IVEPL3DV3DZGRANCNFSM57U7E45Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

No worries @fmyuan!!  we have a temporary work around


Avoid using unicode punctuations in schema docs

This is minor but a possible source of inconveniences in the future: see https://github.com/Grafeas/Grafeas/blob/dda9654a580daec8c8ec7e16eed67589e2a80324/v1alpha1/grafeas.json#L1301

Per https://github.com/Grafeas/Grafeas/issues/50 I'm going to migrate this to proto w/ GRPC (I hope to have the initial proto ready for review today). So this shouldn't be an issue going forward.


fix: URL encode explicit routing values

Part of #12232

This change is

I ran i/cloudbuild/build.sh -t asan-pr
I'm not sure what unit tests I should change for this change

GENERATE_GOLDEN_ONLY is good for developing, but we should remove it and rerun generate-libraries-pr before sending the PR.
I would add a new unit test in golden_kitchen_sink_metadata_decorator_test.cc. Look at the ExplicitRouting* tests for inspiration.
We should probably make identical changes to the REST metadata decorator

I did the first two.
Looking into the rest metadata decorator.


Injecting span attributes to child spans at server level

What feature do you want to see added?
I want to inject custom span attributes into the child spans, I can see withSpanAttributes, that it's working very well, but this will only work at the job/pipeline level or stage level. However, I need the custom span attributes on all pipelines running on the server. Can we have a global configuration at the server level to inject/define custom span attributes?
Upstream changes
No response
Are you interested in contributing this feature?
No response

global configuration at the server level to inject/define custom span attributes

Could you rely on resource attributes? If so, you just have to use config property otel.resource.attributes.

As Cyrille said you can use otel.resources.attributes or OTEL_RESOURCE_ATTRIBUTES environment variable passed to your Jenkins process.
https://opentelemetry.io/docs/specs/otel/configuration/sdk-environment-variables/


LoginController redirectTo method behaviour is not consistent

Laravel Version: 5.5.latest
PHP Version: 7.1.10
Database Driver & Version: postgresql

Description:
I want to use the redirectTo method with a default LoginController.
I've found that redirectTo behaviour is not consistent.
LoginController
    public function username()
    {
        return 'login';
    }

    /**
     * Where to redirect users after login.
     *
     * @return string
     */
    protected function redirectTo()
    {
        $user = Auth::user();

        if ($user->hasRole('consultant')) {
            return route('requests.general.index');
        }

        if ($user->hasRole('admin|exploitant')) {
            return route('engineers.index');
        }

       return route('engineers.index');
    }

Problem : When I connect with a consultant account i'm not always redirected to the correct requests.general.index route.
Steps To Reproduce:

Step 1: Connect with an account with the consultant role.
Step 2: I'm redirected to the correct requests.general.index route.
Step 3: Immediately logout.
Step 4: Connect with the same account.
Step 5: I'm now redirected to '/' route ...

I don't think that the problem is tied with the whole role + permissions routine, I've seen this behaviour with a much simpler redirectTo method.
Maybe in step 5 I'm actually correctly redirected but then I'm redirect to '/' ...
If this behaviour is the correct one, can someone explain me how I can achieve a consistent behaviour ?
Thanks.

This repo is for bug tracking. Maybe you have an issue with your roles?

I do know that this is an issue tracker. As I said I can reproduce this behaviour with a much simpler redirectTo :
protected function redirectTo()
    {
        return route('requests.general.index');
    }

Same steps, exact same problem.

The redirectTo() is only used if there is no intended url. When you browse to the root, and require auth, then the root is your intended url.
Ref: https://github.com/laravel/framework/blob/v5.5.14/src/Illuminate/Foundation/Auth/AuthenticatesUsers.php#L104-L105

Ok, I do understand the intent part but why do I keep an intent on logout is the part I don't get.
If I login and logout 100 times I actually want the redirectTo method to be triggered each time. It's the whole point of having this specific logic in the Controller (or I'm totally clueless 😞 )

That could happen, if on logout, you're trying to redirect to a protected page. Then an "intended" would be stored and followed when you log in. Purely a guess though.

Ok @devcircus this is it, this is logical but not very intuitive.
At the end of the day it wasn't a bug at all and I apologise for thinking it was one.
Thanks to everyone, closing ...

I spent a day on a similar issue a while back. Glad you tracked it down.


程序执行前先git pull后再执行

fix #799

Pull Request Test Coverage Report for Build 430

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
13 unchanged lines in 6 files lost coverage.
Overall coverage remained the same at 41.626%

Files with Coverage Reduction
New Missed Lines
%

cobra/pickup.py
2
33.06%

cobra/cve.py
2
81.5%

cobra/init.py
2
29.11%

cobra/export.py
2
60.0%

cobra/log.py
2
44.93%

cobra/utils.py
3
35.77%

Totals

Change from base Build 426:
0.0%

Covered Lines:
1608

Relevant Lines:
3863

💛  - Coveralls

每次都去GitHub拉一下感觉会很慢，有时候国内的网还是不稳


Gather and verify data for Huggingface release

Gather data on a reserved server and verify it  for the Huggingface release

This should be the last issue solved before publishing the results

All known limitations have been written to the huggingface page FAQ.
Added issue https://github.com/groq/mlagility/issues/196

Note: All data has been validated to the best of our knowledge. All remaining validation tasks that could not be completed as part of this first release can he found here


feat: project search term split

Describe your changes
Ticket: https://pins-ds.atlassian.net/browse/APPLICS-478
Following the pattern as other entities, this PR splits search term into words before querying to get more results.
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[x] I have made corresponding changes to the documentation
[x] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

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

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

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

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


Bug 1994857: targetcontroller: check external dependencies (csr-controller-ca configmap) only on SNO

/assign @sttts
@hexfusion  based on comments from Stefan we might want to change the deps check, he says that relying on managed fields is not reliable enough. What was the annotation you initially wanted to check?

Even this is not the solution. We must not use managedField for some definitive answer. It is best effort from the apiserver and information only.

/hold
alternative being discussed in #1209


Show warning on unclean dependencies

If any checkout in lake-packages differs from the pinned commit, we should print a warning, especially as this can lead to cache misses in mathlib.

Yes, exactly!
An interesting follow-up question might be whether dependencies should be local git repos at all (instead of e.g. checkouts of user-wide bare repos, or direct tarball fetches in the case of GitHub) but that is probably low priority.

@Kha Having gone to implement this, I rediscovered that, even on build, we reset the dependency to the pinned commit (if it is on something else). So, they can never differ. Would you still like Lake to print a warning if it does such an update? Or is it good as is?

Ah, then I must have misremembered my original request: "unclean" usually means that the working directory differs from the checked-out commit. We can probably ignore untracked files(?) but any local change to a dependency is concerning and should generate a warning.


Fix Idle Stream Issue

Fix https://linear.app/livepeer/issue/PS-91/switchboard-idlepending-status-inaccurate
Fix https://linear.app/livepeer/issue/VID-456/recording-stuck-in-progress
It fixes the issue that a stream is Idle ("isActive": true,) after finishing streaming for 90s.
Explanation

In https://github.com/livepeer/catalyst-api/pull/842 we started a loop in which every 1 min we collect metrics from Mist and then call setActive(true) on studio.
That can lead to the race condition between this setActive(true) and setActive(false) sent from STREAM_BUFFER trigger when the stream is stopped
The root cause is that at the time of sending setActive(true) from the metrics collector, we're actually not sure that the stream is still active.
In this (and two other related PRs: https://github.com/livepeer/go-api-client/pull/52, https://github.com/livepeer/studio/pull/1957) I've added a new studio endpoint /heartbeat used to refresh the stream lastSeen state without stating if it's active or not

@gioelecerati FYI: There is also a race condition between calling /setactive and /hook/health, but I've never encountered them having different values. They are both triggered with the same Mist trigger, so I think we can leave it as it is right now.

FYI: There is also a race condition between calling /setactive and /hook/health, but I've never encountered them having different values. They are both triggered with the same Mist trigger, so I think we can leave it as it is right now.

Curious, I saw them having different values in my box - maybe that's not happening outside of it. Thanks for helping out, the investigation & the fixes!

Neat!


Feature: Intelligent copy

Hi!
It would be nice if you could add an "intelligent" copy feature. Many other terminals support it.
If you select a text and click "CTRL+C", it will copy the selected text and remove the selection.
If you click "CTRL+C" normally, it will send the interrupt to the shell like normally.

Perfect plugin idea!

Something to add to this,
The addition of Mouse modifier keys like WSL Terminal (the github project).
Example: Hold shift(or whatever you choose) key to tell the terminal not to send mouse events to Vim, and make text selections for coping to the clipboard.


[Prioritized] Normalize line endings

@JoshLove-msft @scbedd

Doing some digging into this. Found this issue which is basically exactly our scenario.
So I think we need to create a custom Utf8JsonWriter and override the WriteIndent call to insider \n instead of Environment.NewLine.
We should also update our we load the recording to ignore crlf when deserializing from the recording file.

Doing some digging into this. Found this issue which is basically exactly our scenario.
So I think we need to create a custom Utf8JsonWriter and override the WriteIndent call to insider \n instead of Environment.NewLine.
We should also update our we load the recording to ignore crlf when deserializing from the recording file.

I think the issue is with the actual content, not the formatted JSON. When matching we renormalize the JSON anyways so it should handle this for us. But the issue happens when someone adds line breaks based on Environment.NewLine from their own test.

When matching we renormalize the JSON anyways so it should handle this for us.

Yeah I'm seeing this now.

But the issue happens when someone adds line breaks based on Environment.NewLine from their own test into the request content

Hmmm. So add a hardcoded replace on \r\n if the request content is a text content type?

when someone adds line breaks based on Environment.NewLine from their own test into the request content

I can see how this might be the proxy's place to do this, but I'm super leery towards it. Bright side is that this is not  a breaking change. Existing recordings would just have crlf content silently replaced when matching.

Agreed that this shouldn't be a breaking change. I think essentially normalizing the line breaks when matching is the way to go.


Kill stress tests & perform cleanup

Currently stress tests are not providing any added value (and some of them are also failing).
-- remove code for running stress tests (deleted code is good code).
-- shutdown the cluster used for running stress tests (significant resource savings).
CC @sreecha

@sreecha   cool, that's the best of course.

#11027 addresses this

This is done. The issue can be closed.


Get open collective sponsors and display them on UseCasesScreen

Behavior

Load 9 sponsors from open collective

Relevant story in local SB

Done.


Fix/remove console errors merge conflict

Pull origin dev and fix conflicts

@darwin808  just added a fixed merging error in the join and start app


Support form input prop name

For all the form related components (radio group, switch) the react version supports the prop name for use in form element. Is there a way to get this working somehow - or does this need to be supported by the lib?

@smblee Until #121 get sorted, you can manually do it yourself. Add a hidden input like so: <input type="hidden" {name} {value} />
Make sure value is bound to both that input and the appropriate component like RadioGroup.


App version code in google play console is different from pubspec.yaml file

Hi, For some security reason i am checking the app version and send it to backend. In my backend i have the version code like 5. But when i am reading the version code using package_info i am getting the version code like 1005, 2005 and 4005. How can i get the exact version code? Any idea?
I uploaded as splite APKs. In playstore console also showing like 1005, 2005 and 4005
Note: i cant use bundle. we don't have access for certificate file.
Can anyone please help me to solve this?

Hi @sathya4code
can you please provide your flutter doctor -v ,
and your pubspec.yaml
or if possible a reproducible minimal code sample.
What version returns running flutter run --release --verbose ?
Thank you

Hi, Thank you.. I solved this issue using, Platform channel code. Got the version code using Google in app update.

Hi @sathya4code
given your last message I feel safe to close this issue,
if you disagree please write in the comments and I will reopen it.
Thank you


set_surface_albedo! is not GPU compatible

Currently, the function set_surface_albedo! moves some arrays back and forth from CPU to GPU. (see https://buildkite.com/clima/climaatmos-gpulongruns/builds/84#018e2fc1-6d22-4e69-a918-7ecb3831f69d/150-231)
	Argument 4 to your kernel function is of type Base.Broadcast.Broadcasted{CUDA.CuArrayStyle{2}, Tuple{Base.OneTo{Int64}, Base.OneTo{Int64}}, typeof(*), Tuple{Base.Broadcast.Extruded{Matrix{Float64}, Tuple{Bool, Bool}, Tuple{Int64, Int64}}, Base.Broadcast.Extruded{CUDA.CuDeviceMatrix{Float32, 1}, Tuple{Bool, Bool}, Tuple{Int64, Int64}}}}, which is not isbits:
    .args is of type Tuple{Base.Broadcast.Extruded{Matrix{Float64}, Tuple{Bool, Bool}, Tuple{Int64, Int64}}, Base.Broadcast.Extruded{CUDA.CuDeviceMatrix{Float32, 1}, Tuple{Bool, Bool}, Tuple{Int64, Int64}}} which is not isbits.
    .1 is of type Base.Broadcast.Extruded{Matrix{Float64}, Tuple{Bool, Bool}, Tuple{Int64, Int64}} which is not isbits.
	      .x is of type Matrix{Float64} which is not isbits.

closed by #2787


[Bug]:   The dead screen is back

Version

[X] I'm using version 1.6.0

Known issues

[X] I've checked Known issues

Existing issues

[X] I've checked Existing issues

What operating system are you using?
Windows
Operating System Version
Windows 10 version 1909
Reproduction steps

When a periodical break end
Reproduces how often: About 90% of the time

Expected Behavior
The screen goes back to normal after the break
Actual Behavior
The screen goes dead. Keyboard and mouse are still handled but the screen doesn't update. I have to navigate through tabs by using [Alt]+[Tab] to make the screen redrawn.
Relevant log output
main.log
Preferences
config.log (I changed the filetype as Github doesn't allow uploading .json file)
Additional information
No response
Code of Conduct

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

Hi @uahnbu . What do you mean by dead screen? Could you be more descriptive?
Could you also please fill the log output form when issue occurs and your Preferences file?

main.log
config.log (I changed the filetype as Github doesn't allow uploading .json file)
To be more descriptive, below are some examples:

I'm on Github commenting then Stretchly shows up for a break. After the break ends, it fades away to show Chrome screen again. However, I cannot scroll or click or move the pointer of the webpage, although the pointer moves to the clicked location after I've applied the fix below.
I'm watching a video on Youtube. After the break, the Youtube video goes black although it's still playing with sound.

How I fix it: Press Alt+Tab

Thanks, please see the Known issues. It links to the comment with possible fix on Chrome side.

Hmm, maybe an update in Chrome enabled the flag again. Now it has worked.

However, in other Web Apps such as VSCode, the problem still exists. How to disable the flags in those apps?

Maybe this will help? https://github.com/microsoft/vscode/issues/114459 they seems to mention the same flag

Also experiencing this issue on Windows 10. Apps which freeze after a fullscreen break reminder in my (unscientific) testing include:

Todoist
Slack
VSCode

That sucks, TBH I am not sure what to do :( This must be something between Chrome - Chromium - Electron but not sure how I could make some small reproducible example to report somewhere.

@ajaxbits fullscreen break reminder - are U sure problem not exist without full screen remainder. Could you turn off fullscrean break remainder and check if problem exist?
Other thing to try - If this is repeating thing then starting streatchly from command line would at least provide some logs. And this could give some clues where to look. Even better would be source code with debug flags. But getting some sort of logs would be a first step.

Totally, @senpl. I'll see if I can get some logs.

I just ran into this, but I know the cause (I think): I am using a folder on a mounted drive, and there is a network connectivity issue ongoing right now.
So, process to replicate:

Open my folder on a mounted NFS drive, open some file inside of it.
Swamp your network :) :)
Close VSCODE.  You may have to force close it, as it will complain about being unable to complete backups or some such.
Try to open VSCODE again.  You get a black screen, with no means to escape.

Hi, I will close this one as noone complained lately, please open new issue with up to date info if this happens again.


Unable to run

Hi, whenever I try to run the script below I get the following error:
failed to wait for gptscript to be ready: context deadline exceeded
package main

import (
	"context"
	"fmt"
	"os"

	"github.com/gptscript-ai/go-gptscript"
)

func listModels(ctx context.Context) ([]string, error) {
	g, err := gptscript.NewGPTScript(gptscript.GlobalOptions{OpenAIAPIKey: os.Getenv("OPENAI_API_KEY")})
	if err != nil {
		panic(fmt.Sprintf("error creating gptscript: %s", err))
	}
	defer g.Close()
	return g.ListModels(ctx)
}

func main() {
	ctx := context.Background()
	fmt.Println(listModels(ctx))
}

@zakpaw Thanks for your report.
Do you have the gptscript CLI on your path? Otherwise, you can set the GPTSCRIPT_BIN environment variable to point at the gptscript CLI.

I do, Its a full path to gptscript installed with devbox (nix)

OK, awesome! Which version of gptscript are you using?

gptscript version v0.5.0
go mod github.com/gptscript-ai/go-gptscript v0.0.0-20240613214812-8111c2b02d71

OK, thanks for that information!
I see the issue: your version of the gptscript CLI is out of date. Can you try using the latest version (v0.8.3)?

Ah I see, 0.5.0 is the latest available in nixpkgs. I'll update and give it a try. Thanks!


Add-InsightsCapability fails when Perfgen.dll is modified

With the original Perfgen.dll Add-InsightsCapability works fine. You add one line of code to PerfGen.c and rebuild PerfGen.dll then  Add-InsightsCapability fails.
PS C:\Windows\System32> Add-InsightsCapability -Name SampleSysInsight -Library C:\KCSCapability.dll -Confirm:$false
Add-InsightsCapability : System Insights failed to register one or more of data sources for capability 'SampleSysInsight'.
At line:1 char:1

Add-InsightsCapability -Name SampleSysInsight -Library C:\KCSCapabili ...

  + CategoryInfo          : InvalidData: (SampleSysInsight:String) [Add-InsightsCapability], Exception
  + FullyQualifiedErrorId : SystemInsightsError,Microsoft.SystemInsights.Management.PowerShell.AddInsightsCapabilityCommand

It is as though Add-InsightsCapability remembers a checksum of the original Perfgen.dll and fails. Just does not make sense. The error messages don't say much.
Another related question unless I start Perfmon Add-InsightsCapability does not succeed. So Add-InsightsCapability has a pre-requisite that Perfmon should be running. This also does not make sense.
Please hel;p
[Enter feedback here]

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

ID: 75a606dc-b52c-5b24-8c0c-ff57f48dd8cd
Version Independent ID: cfb93dbb-6b10-dee0-e6ec-77c0c95e4d5f
Content: Add-InsightsCapability
Content Source: docset/windows/systeminsights/Add-InsightsCapability.md
Product: w10
Technology: powershell-windows
GitHub Login: @AndreaBarr
Microsoft Alias: v-anbarr

Hi Onur,
I just realized all Perfmon does is load Perfgen.dll. mmc.exe loads Perfmon and that loads Perfgen.dll. So my assumption of Perfmon being a requisite is wrong. I can write my own abcd.msc and make it load Perfgen.dll for that matter. So now my question boils down to why Add-InsightsCapability() fails on modified Perfgen.dll. Perfgen.dll is a MSFT provided sample please Google for details.

This is a product question that is out of scope for this documentation. Please submit your issue to https://windowsserver.uservoice.com/forums/301869-powershell


BatchUpdate generate wrong column name

Description
I try to update  PostTag(mapped to"blog_tags" table) in this way
await _dbContext.Set<PostTag>()
                .Where(tag => addTagIds.Contains(tag.Id))
                .UpdateAsync(x => new PostTag
                {
                    UseCount = x.UseCount + 1
                });

Exception
Microsoft.Data.Sqlite.SqliteException : SQLite Error 1: 'no such column: tag.UseCount'.
   at Microsoft.Data.Sqlite.SqliteException.ThrowExceptionForRC(Int32 rc, sqlite3 db)
   at Microsoft.Data.Sqlite.SqliteCommand.PrepareAndEnumerateStatements(Stopwatch timer)+MoveNext()
   at Microsoft.Data.Sqlite.SqliteCommand.ExecuteReader(CommandBehavior behavior)
   at Microsoft.Data.Sqlite.SqliteCommand.ExecuteNonQuery()
   at Z.EntityFramework.Plus.BatchUpdate.Execute[T](IQueryable`1 query, Expression`1 updateFactory) in C:\Users\Jonathan\Desktop\Z\TFS\EFPlus\Z.EntityFramework.Plus\shared\Z.EF.Plus.BatchUpdate.Shared\BatchUpdate.cs:line 407
   at System.Threading.Tasks.Task`1.InnerInvoke()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot)
--- End of stack trace from previous location where exception was thrown ---

It looks like the Where expression's parameter name is used as the table name
Further technical details

EF version: EF Core 2.2.4
EF Plus version: EF.Plus.EFCore 1.8.27
Database Provider: EF Core Sqlite 2.2.4

Hello @ZeekoZhu ,
Could you provide a runnable project/solution? It will help my developer to get started faster with this issue.
You can send private file here<EMAIL_ADDRESS>Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

@JonathanMagnan  Here is simple reproduce: https://github.com/ZeekoZhu/EFCorePlusSQLiteUpdateIssue

https://github.com/zzzprojects/EntityFramework-Plus/blob/master/src/shared/Z.EF.Plus.BatchUpdate.Shared/BatchUpdate.cs#L1434
This line add a table prefix for column ViewCount

Thank you for the project,
I have re-opened the issue.
Best Regards,
Jonathan

Hello @ZeekoZhu ,
We tried to solve it last week but unfortunately, we were not able to find a good solution.
Unlike SQL Server, a JOIN is not used so it's not easily possible for us to take the current column value. Those kinds of expression UseCount = x.UseCount + 1 is only supported for providers that allows us to use a JOIN
I believe you will need to write yourself the SQL queries for this one ;(


Update README.md

updated ETF version compatibility

It may be compatible, but I've yet to backport it and I don't know when or if I will.


feat(docs): Add bob.build

This was merged as 4a506eef8b8323bca40f492ea3b1ec8a6640ff97 as part of #538.


Fix microk8s import images for offline clusters

Summary
Fixes https://github.com/canonical/microk8s/issues/4253
Changes

Retry listing the cluster nodes until success instead of waiting for a node with status == Ready

LGTM +1


Use go-swagger to generate different client/servers

Hi,
I am currently trying to use go-swagger to generate some part of a Server application automatically. The server has been written with the idea of code generation in mind but we would love to reuse entirely our code transforming the code in go templates. It would be actually enough the export the appGenerator type and have a function that returns it (maybe generator.GenerateServer itself?). I would then use appGenerator to execute the templates.
I would be interested in discussing how I could realize what I want to do, if it is possible with the current go-swagger  or not.
Thanks in advance~

you can provide your own templates for go-swagger the client and server generators allow you specify a directory on disk to add custom templates
There are some docs here: http://goswagger.io/generate/templates/
In VIC they do this: https://github.com/vmware/vic/tree/master/lib/apiservers/templates
https://github.com/vmware/vic/blob/master/Makefile#L274-L281
You can override these templates by using the same names
https://github.com/go-swagger/go-swagger/blob/master/generator/templates.go#L61-L73

Wouldn't this generate roughly the same structure of the server? I don't want to change minor details, I want to have code that looks totally different (but only for the server part, models and clients are more than okay) while using code the parsing and validation from go-swagger. This means different number of files and different functionalities. This is why I thought that having something that could return the AppGenerator could be enough, but I am interested in understanding your opinion.

yeah you're right it would generate a similar structure.
Feel free to make the necessary changes for your case to work and submitting a PR.
If I understand correctly you're not happy with how the operations/parameters/responses are structured and want to replace all of it? Do you have an example of how the structure would look?

Yeah, I would love to have something much simpler. I do not have the current project opensource, but I can create a gist to give you an idea as all this part can actually be opensourced. I will then work on the PR so that we can discuss further.

@casualjim The server will look somehow similar to https://github.com/zalando/chimp/blob/master/api/server.go where the routes part will be generate from the swagger def. The handlers, similar to  https://github.com/zalando/chimp/blob/master/api/handler.go will be able to use something like this https://gist.github.com/Raffo/0bcc37995a15bd383944f099d94d1b82 for response generation. I hope that at least the idea is clear: I want to build a very opinionated server, with only few parts auto generated in order to satisfy an use case that pretty much matches the need of our project but to do so I want to re-use the parts of go-swagger.

@casualjim Sorry for the delay, but I didn't have time at all to work on this. I quickly hacked something to give you an understanding of what I meant with this issue here: https://github.com/Raffo/go-swagger/commit/a196c5954def2bb9f1b335f7bb8011c49c5c1f08
Essentially the only thing that was done is:

Making appGenerator public and accessible from the outside
Created a "main" that can be used to generate any template using Go's templates. This main should be evolved in just executing the templates based on the content of appGenerator (now it is only an hack) -> https://github.com/Raffo/go-swagger/blob/a196c5954def2bb9f1b335f7bb8011c49c5c1f08/cmd/swagger/commands/generate/cmd/main.go

Please note that all of this is an hack and that is not definitive or well done. I just wanted to bump the discussion and give a better understanding of what I meant before.

Up, any update?

wouldn't it be easier to add a few a flag for a config file to customize the generation and the folder layout?

Yes, but do you think it would be actually possible? It looks like the code that is generated has an enforced structure... could you give me an hint on how you would change the code?

I divide the code in 4 types of generation based on the type passed to the templates as context.
Then I allow for a template to be used to specify the path with a filter for joining path parts together so that an empty path part has no effect.
The file name also gets a template so that you can customize the name of the generated file
The template name has a convention that when it starts with asset:[key] it will be looked up from the embedded templates in the swagger binary, otherwise it will look for that template on disk based on the path you specified.
That config should be in an ordered list so that the generation order is deterministic, then instead of hardcoding the generation of each known template it will now use that config and iterate over those configurations to generate the files with context.
As a bonus you'd be able to generate other languages as well from the same object model.

I refactored the codegeneration to the model I described in the previous comment.
https://github.com/go-swagger/go-swagger/tree/gen-layout-configfile
it still needs the config file bit but the machinery for it has been refactored so it can be driven by a config file. The next thing that would need to be implemented is using viper or something to read a config.

Nice, let me know if you would love help to get it finished and if so, what kind of help!

I've added some docs on the config file format here: https://github.com/go-swagger/go-swagger/blob/gen-layout-configfile/docs/use/template_layout.md
Please let me know if this works for you, then I'll move it to master and release a 0.6.1

I didn't get how to test it. I tried the following in github.com/go-swagger/cmd/swagger
 go build .
mir:swagger raffo$ swagger generate server --help
Usage:
  swagger [OPTIONS] generate server [server-OPTIONS]

generate all the files for a server application

Help Options:
  -h, --help                 Show this help message

[server command options]
      -f, --spec=            the spec file to use (default: ./swagger.json)
      -a, --api-package=     the package to save the operations (default: operations)
      -m, --model-package=   the package to save the models (default: models)
      -s, --server-package=  the package to save the server specific code (default: restapi)
      -c, --client-package=  the package to save the client specific code (default: client)
      -t, --target=          the base directory for generating the files (default: ./)
      -T, --template-dir=    alternative template override directory
      -A, --name=            the name of the application, defaults to a mangled value of info.title
      -O, --operation=       specify an operation to include, repeat for multiple
          --tags=            the tags to include, if not specified defaults to all
      -P, --principal=       the model to use for the security principal
          --default-scheme=  the default scheme for this API (default: http)
      -M, --model=           specify a model to include, repeat for multiple
          --skip-models      no models will be generated when this flag is specified
          --skip-operations  no operations will be generated when this flag is specified
          --skip-support     no supporting files will be generated when this flag is specified
          --exclude-main     exclude main function, so just generate the library
          --exclude-spec     don't embed the swagger specification
          --with-context     handlers get a context as first arg
          --dump-data        when present dumps the json for the template generator instead of generating files

mir:swagger raffo$ grep -R "config-file" .
./commands/generate/server.go: 	ConfigFile    flags.Filename `long:"config-file" short:"C" description:"configuration file to use for overriding template options"`

Am I doing something wrong?
Thanks for your work!

you can pass it your own set of templates and see if it meets your needs. I'm unlikely to be able to spend time on this for the next 10 days so there is no rush :).

@casualjim with this example config:
layout:
  application:
    - name: configure
      source: asset:serverConfigureapi
      target: "{{ joinFilePath .Target .ServerPackage }}"
      file_name: "{{ .Name }}_client.go"
      skip_exists: true
    - name: main
      source: asset:serverMain
      target: "{{ joinFilePath .Target \"cmd\" (dasherize (pascalize .Name)) }}-server"
      file_name: "main.go"
    - name: embedded_spec
      source: asset:swaggerJsonEmbed
      target: "{{ joinFilePath .Target .ServerPackage }}"
      file_name: "embedded_spec.go"
    - name: server
      source: asset:serverServer
      target: "{{ joinFilePath .Target .ServerPackage }}"
      file_name: "server.go"
    - name: builder
      source: asset:serverBuilder
      target: "{{ joinFilePath .Target .ServerPackage .Package }}"
      file_name: "{{ snakize (pascalize .Name) }}_api.go"
    - name: doc
      source: asset:serverDoc
      target: "{{ joinFilePath .Target .ServerPackage }}"
      file_name: "doc.go"
  models:
    - name: definition
      source: asset:model
      target: "{{ joinFilePath .Target .ModelPackage }}"
      file_name: "{{ (snakize (pascalize .Name)) }}.go"
  operations:
    - name: parameters
      source: asset:serverParameter
      target: "{{ joinFilePath .Target .ServerPackage .APIPackage .Package  }}"
      file_name: "{{ (snakize (pascalize .Name)) }}_parameters.go"
    - name: responses
      source: asset:serverResponses
      target: "{{ joinFilePath .Target .ServerPackage .APIPackage .Package  }}"
      file_name: "{{ (snakize (pascalize .Name)) }}_responses.go"
    - name: handler
      source: asset:serverOperation
      target: "{{ joinFilePath .Target .ServerPackage .APIPackage .Package  }}"
      file_name: "{{ (snakize (pascalize .Name)) }}.go"

it doesn't seeem to work and I don't get any meaningful log. Any idea of what could be wrong? Thanks again :-)

I found that it wasn't actually using the config when you provided one with -C. I fixed it now and i've updated the doc page.


separate the sources into dedicated directory, fix doc generation

for the documentation generation to work, the code needs to work as a package and be importable, and the package needs to have a valid name (kyber-py won't work), so put all of the sources into the src/kyber directory. Upside: clear separation between shipping code and tests. Also, because everything is in single module, it won't pollute python namespace.
also change it to use relative imports so that any future refactoring is easier (and they're recommended anyway)

Thanks for doing this work. The only thing I'm not sure about is calling the package "kyber" but I don't have a great idea about what to do. It's annoying to have to think about the future here as Kyber will be an antiquated name at some point. Maybe mlkem_py or even kyber_py... can't decide.

Also I think I prefer the absolute rather than relative paths to functions but if relative is the standard then I guess I can accept this.

Thanks for doing this work. The only thing I'm not sure about is calling the package "kyber" but I don't have a great idea about what to do. It's annoying to have to think about the future here as Kyber will be an antiquated name at some point. Maybe mlkem_py, ml_kem_py, py_mlkem or even kyber_py... can't decide and I'm not super happy with any of them

I still see plenty of code that refers to AES as Rijndael... :slightly_smiling_face: and kyber definitely rolls of the tongue better than ML-KEM, unlike Rijndael vs AES...
The thing is, that the package name in PyPI is kyber-py, so the principle of least surprise is that the top level module name should have at least kyber in it. But if you'd prefer kyber_py I can change to that... with relative imports that will be fairly easy :wink:

Yeah ok let's do kyber_py for the package name. I don't think that we should "package" this properly until the final spec is out but it doesn't hurt to pick a name now.

Well, I'd prefer it as a package for integrating into tlslite-ng...

Sure I appreciate that. Originally I was very against this being an importable package as I wanted to dissuade people from using it for cryptography but let's see what state the code is in after a little more work.

Thanks for changing the name.
Could we also tweak the examples in the README so the imports are correct for users?
>>> from ml_kem import ML_KEM128
>>> ek, dk = ML_KEM128.keygen()
>>> key, ct = ML_KEM128.encaps(ek)
>>> _key = ML_KEM128.decaps(ct, dk)
>>> assert key == _key

to something like:
>>> from kyber_py.ml_kem import ML_KEM128
>>> ek, dk = ML_KEM128.keygen()
>>> key, ct = ML_KEM128.encaps(ek)
>>> _key = ML_KEM128.decaps(ct, dk)
>>> assert key == _key

@GiacomoPope also docs needed fixing, I've added them to CI now so it will be clear if/when they get broken again :)


WIP: Logging and Image Updates

This PR will collect changes having to do with logging and feed back to the user as well as address compatibility with version 0.4 of servicex.
Issues

#21 - Move to using the 0.4 version of the C++ transformer by default
#10 - Generate progress data
#6 #11 - Add logging information (using python logging module)

Review

The modifications are pretty minimal here.
There has been testing in Jupiter notebooks by hand to make sure this works. It does require extensions to be installed, however.

Codecov Report

Merging #24 into pr_file_downloads will increase coverage by 0.20%.
The diff coverage is 98.43%.

@@                  Coverage Diff                  @@
##           pr_file_downloads      #24      +/-   ##
=====================================================
+ Coverage              97.72%   97.92%   +0.20%
=====================================================
  Files                      1        2       +1
  Lines                    132      193      +61
=====================================================
+ Hits                     129      189      +60
- Misses                     3        4       +1

Impacted Files
Coverage Δ

servicex/utils.py
97.87% <97.87%> (ø)

servicex/servicex.py
97.94% <100.00%> (+0.21%)
:arrow_up:

Continue to review full report at Codecov.

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


Dagger npm package is sometimes imported incorrectly

This has been reported twice in the last day. When importing @dagger.io/dagger the actual code directory is randomly missing sometimes (only package.json, README, etc. are there). It's not clear when or why as running the exact same commands on almost exactly equivalent versions of node+npm give different results.
User Ringo in discord noticed that if he remove the files section from our package.json then the problem disappears for him: https://github.com/ringods/dagger/commit/a13291631014dcfd07c9202059936a6c51a26ef7
So we should highly consider making that change ASAP, but want to double check with @slumbering @crjm if there's any downsides to removing it. It seems like we could maybe try using .npmignore instead? https://docs.npmjs.com/cli/v8/configuring-npm/package-json#files

Pr opened: #3166


Could you add more details on how to link an Azure key vault to a variable group in ADO?

I'm trying to link a key vault to a variable group in ADO, but I'm running into some issues. I'm following the steps outlined in this document: https://docs.microsoft.com/en-us/azure/devops/pipelines/library/variable-groups?view=azure-devops&tabs=yaml#link-secrets-from-an-azure-key-vault
I'm running into an issue with the UI (which I've reported here: https://developercommunity.visualstudio.com/content/problem/1157776/unable-to-authorize-linking-secrets-from-key-vault.html), so I'm trying instead to do it manually in the portal. However, I'm not sure what service principal I should give the permissions to. The documentation specifies this as:
"In the Add access policy blade, choose Select principal and select the service principal for your client account."
I'm not sure what the 'client account' is, and could use more details. I think I'm missing something about how Azure DevOps works. Is there an application/service principal that gets associated with the project? Is that the service principal that needs to be given access? If so how do I determine what that service principal is (I didn't setup the ADO project)?

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

ID: 4827aa99-ae62-bd63-6f2f-a87a4065ed27
Version Independent ID: c9e461ee-7f42-3503-9460-18fa3a807bbb
Content: Variable groups for Azure Pipelines and TFS - Azure Pipelines
Content Source: docs/pipelines/library/variable-groups.md
Product: devops
Technology: devops-cicd-library
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

@jl-g Thank you for the feedback.
This issue was previously reported under https://github.com/MicrosoftDocs/azure-devops-docs/issues/3774.  Please refer there for further updates.


Deprecate OuiLoadingElastic

Deprecate OuiLoadingElastic component in favor of new component, OuiLoadingDashboards

To deprecate this component, all that needs to happen in the next 1.x release is to add a comment that it is being deprecated. Then, it can be completely removed in 2.0, as removing the component is a breaking change. This work can be done in 2 steps/PR's:

[ ] Add the deprecation comment (merged into main, backported to 1.x)
[ ] Remove the component (merged into main)


Migration errors with the located assembly's manifest definition does not match the assembly reference

I'm trying to generate a database from the terminal of VS code using SQL Operations Studio, whenever I try to generate a migration this error occurs.
dotnet ef migrations add init

Unhandled Exception: System.IO.FileLoadException: Could not load file or assembly '/usr/local/share/dotnet/sdk/2.1.401/DotnetTools/dotnet-ef/2.1.2/tools/netcoreapp2.1/any/tools/netcoreapp2.0/any/ef.dll'. The located assembly's manifestdefinition does not match the assembly reference. (Exception from HRESULT: 0x80131040)

This is how my .csproj looks like now
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp2.1</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="2.1.2" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="2.1.2" />
  </ItemGroup>
</Project>

Any workaround for this?

@andrescorrea Can you please post a project/solution that reproduces the issue you are seeing so that we can investigate more.

I fixed the error adding:
<RuntimeIdentifier>osx.10.12-x64</RuntimeIdentifier>

inside the <PropertyGroup> tag group


image: create image component for rich text editor

Related to #210
We can't use Sanity's focal point feature for images inside the rich text editor as they may be infographics, illustrations, or other figures.

@nicholas-haggerty @BirteThornquist (As discussed with Birte yesterday)
Adding photos to Sanity is split into several fields: The image itself, alt text, caption and attribution.

Since the attribution part is structured content, it makes sense to use a separate field. But in existing solution, this example will be written like
Halfdan Brustad, vice president for Dogger Bank at Equinor. (Photo: Ole Jørgen Bratland)
Are we able to create a rule for the attribution part so that the web can make a string along these lines
 (Photo: ) where (Photo: and ) is "owned" by the web? From previous projects, this makes the texts to appear similar and predictable across the web page. Could it be (Video: Ole Jørgen Bratland) Do we need to prefix with "Photo" at all?
The other option would be to add  Photo: Ole Jørgen Bratland, Video: Ole Jørgen Bratland to Sanity (either ways I think the web would "own" the parentheses ( ) )
We should of course clarify this in Sanity by using a good placeholder og description.
What do you think?

Im not quite sure what you are asking. Happy to take a call.

Edited my comment slightly, the editor didn't show the complete example because I used code brackets...
I think the way forward is for you Nick to discuss this with Birte, she knows the details here :)

Having separate fields for Attribute and Caption is good. These should be optional and empty  - no predefined text/styles.

For the image component we will need some option; full column width, 50% column width either left or right alined. I will provide a sketch for this in Figma. We need to agree if these should be separate components (one 100% width and one 50% width), or variables of one image component.

Currently the max width is hard coded to 800px as we wait for the final article layout.
We will finalize this with issue #244 and #235


Update quickstart template to match modern Sphinx.

Coverage remained the same at ?% when pulling c1736d3fdfee4424f88992991bc48961e72c7864 on nyergler--quickstart-tests into c1983829105756102a59edb1584ced5af8245b37 on master.


TOUCHINPUT struct. Related constants and functions.

I don't really know what to do about the failed check. Any hint?  Thank you.

That was https://github.com/rust-lang/rust/issues/31840 which should be fixed by now. I've started a new build which should work.

check still "Waiting for status to be reported"... What dose it need to be happy?

Oh, I have github set to only accept changes to master after they've been tested on a branch, but appveyor/branch doesn't run on other people's branches, only PRs. So don't mind it, the two pr checks passed and that is what matters.

https://github.com/retep998/winapi-rs/commit/2d2c2d27ee9059590d901c0c5bcca08b591fd0ad already landed into master but that extra merge commit you had tacked on made github think this wasn't merged, so closing because this is actually merged.


PIP-0026 Implementation (first version)

Using URI payments/WEB technology to do payments turns out to be a VERY BAD IDEA.
See Trojan Source vunerability ! Basically unicode control characters used to swap text around for text rendering purposes, left to right, right to left and so forth.
I am pretty sure I could construct a "scam" uri payment example... since all webbrowsers were vunerable at the time of publishing the trojan source doc, that was a few days ago.

Re-tracting this pull request for now... :)


Add keyboard option to enable/disable keyboard navigation and ESC to end tour

In the current version 0.12.0 with keyboard: true, closing the tour with ESC seems not to work. Was this  ever implemented?

See PR #656.
It was easy enough to fix in JS, but this project uses coffeescript.


Does businessDiff treat Sat and Sun as if it's Monday?

I want to show some records to the user if businessDiff is <= 2 days
It works fine if current date is Monday and Tuesday, but for Wednesday I got a surprising result.
If I have some records with timestamp Saturday and Sunday and today is Wednesday it shows business diff as 2 days (same as if the records are from Monday).
I was expecting that the difference between Saturday/Sunday and Wednesday to be 3 (i.e. treating Saturday and Sunday as Friday).
Can someone confirm if this is expected behaviour for businessDiff and how I can achieve my expected behaviour with businessDiff? i.e. not to display records from Sat/Sun if today is Wednesday.

Can you share your setup? Specifically when you run updateLocale to set business days and any other possibile holiday. Can you also please share the piece of code you use to calculate the diff? That would be very helpful to diagnose the issue. :)

Thanks for quick response. Really appreciate you time with this package.
These are my dependencies:
"moment": "^2.22.2",
"moment-business-days": "^1.1.3",
I am actually not running updateLocale at all, as I only care about Saturday and Sunday as this stage. I presumed that for Saturday and Sunday I won't need to do this step. Or do I?
So I am only doing something like this to recreate the issue:
import moment from 'moment-business-days';

describe('moment-business-days', () => {
  context('businessDiff', () => {
    it('should return correct diff for Wednesday vs Sunday', () => {
      // Mon Jan 01 2018 18:00:00 GMT+0000 (Greenwich Mean Time)
      const SUNDAY = moment('2018-01-07T18:00:00Z');
      const NEXT_WEDNESDAY = moment('2018-01-10T18:00:00Z');
      const businessDiff = NEXT_WEDNESDAY.businessDiff(SUNDAY);
      expect(businessDiff).to.deep.equal(3);
    });
  });
});

1 failing

moment-business-days businessDiff should return correct diff for Wednesday vs Sunday:
AssertionError: expected 2 to equal 3

expected - actual

-2
+3

Should the difference be 2 or 3 for the above test? I was expecting the diff to be 3, because at least in my case I only want to show records for Monday and Tuesday, but because of the above result it also grabs the weekend records.

So the negative value is expected since the change in https://github.com/kalmecak/moment-business-days/pull/54
The way businessDiff works is that it counts the working days between two dates. So if you have a date on Wednesday and a date on Sunday, you would count 2 because you have two working days between: Tuesday and Monday. Same for Saturday, the working days are between these days are 2.
Instead if you would subtract 3 days from Wednesday, if would end up on Friday. This is a difference between businessDiff on one side and businessAdd and businessSubtract on the other.


How to create unsigned VersionedTransaction ?

I am basically looking for the VersionedTransaction equivalent of solders.transaction.Transaction.new_unsigned()

The VersionedTransaction object has to have signatures, so use NullSigner to create one that has default signatures


Synthetics monitoring failing without logs

From time to time we're experiencing this weird behavior on our synthetic monitoring:

Steps are not even loaded on uptime
Log shows that step failed when executing "waitForLoadState" even though we don't explicitly use this method

Here's a printscreen:

Here is an example of a journey:
import { journey, step } from '@elastic/synthetics'
import '../../environment'
import { SignInPage } from '../../models'

journey({ name: 'sign-in', tags: ['accounts-auth'] }, options => {
  const { page } = options
  const signInPage = new SignInPage(page)

  step('sign in', async () => {
    await this.page.goto(this.getUrl())
    const login = email
    await this.page.fill('data-testid=login-email', login)
    await this.page.fill(
      'data-testid=login-password',
      password
    )
    await this.page.click('data-testid=login-submit')
    await this.page.waitForNavigation()
  })

  step('select product', async () => signInPage.selectMarketingProduct())
})

Here is an example of our heartbeat:
heartbeat.monitors:
    - type: browser
      name: rd-synthetics-pro-accounts-auth
      id: rd-synthetics-pro-accounts-auth
      schedule: "@every 3m"
      filter_journeys:
        tags:
          - "accounts-auth"
      source:
        local:
          path: "${SUITES_DIR}"

    fields:
      env: production

    tags:
      - production

Versions:
We use @elastic/synthetics: 1.0.0-beta.14 and run our synthetics tests in using docker image: docker.elastic.co/beats/heartbeat:7.15.0

Hi @daniadamatti there have been many improvements in Synthetics since 7.x and the beta version you are using.  In fact we don't recommend using browser checks on 7.x.
With Synthetics going generally available with 8.8, are you able to try a newer version?  Looking at your configuration above, I'd suggest looking at project monitors.

Closing this as we're now in GA.


enable push to myget.org with keeping pushing to nuget.org on tag commit

Current coverage is 76.22%

Merging #331 into master will not affect coverage as of 6e1d882

@@            master    #331   diff @@
======================================
  Files           41      41
  Stmts         4076    4076
  Branches       353     353
  Methods
======================================
  Hit           3107    3107
  Partial        148     148
  Missed         821     821

Review entire Coverage Diff as of 6e1d882

Powered by Codecov. Updated on successful CI builds.

LTGM. Thanks @jayjaywg


How to create new block

How can I easily add new block element (new div with class row or column)? For now I can only add elements inside existing block elements, but I can't create a new one. Is that feature already supported or not?

This can be done in the UI when you hover over a block.  It is the "+" icon when you hover over an existing block.


Add User#update

Add User#update method that supports the V1 and V2 API.
I tested V1 and V2 against a real HipChat server.

Coverage remained the same at 100.0% when pulling 599b279b779152fffc4b1039476fe8586fd3cf3e on gabrielmdeal:update-user3 into aeb06bc1c193a55368feac0a61dc80b3b06b6fff on hipchat:master.

Coverage remained the same at 100.0% when pulling 106b437027e9db43c208077b0de894da0fb2b86d on gabrielmdeal:update-user3 into aeb06bc1c193a55368feac0a61dc80b3b06b6fff on hipchat:master.

I prefer the approach on https://github.com/hipchat/hipchat-rb/pull/181, since it's more explicit on what's going on, but this one supports v1 and v2, which is very nice. What do you think @gabrielmdeal  @craig-gomes ?

@rberrelleza I'm happy to add more documentation to this PR or close this in favor of the other PR.  No worries either way.
I should have more patience!  I had given up on a reply on the other PR.

I'll close this since #181 is moving forward.


Use Drak's fork of Doctrine

Drak's fork is by far the most popular on Packagist.
It sounds like a good idea to consolidate community maintenance efforts on a single fork.
In particular the maintainer of a strong fork by Recras has already agreed to join in to the Drak's version.
Perhaps there is no need to maintain an LExpress' own version?

Hi. Feel free to choose the doctrine1 fork of your choice. This is only a suggestion.


Resolve CI build issues

For example: https://github.com/maxkratz/docker_texlive/actions/runs/8879086444

Resvoled via https://github.com/maxkratz/docker_texlive/commit/5d87e8bef919397c230ed0af414cda3e7599989a.


Symbol resolution and intellisense not working

Describe the bug
As of a few weeks ago the Swift extension won't show errors/warnings, resolve symbols or any editor related feature. Test discovery  and dependency resolution work
To Reproduce
Not really sure, it just stopped working
Expected behavior
The aforementioned features should work
Environment

OS: macOS 14.5
Swift version: swift-driver version: <IP_ADDRESS> Apple Swift version 5.10 (swiftlang-<IP_ADDRESS> clang-15<IP_ADDRESS>) Target: arm64-apple-macosx14.0
Visual Studio Code version: 1.91.1
vscode-swift version: 1.10.4

Additional context
The diagnostics command spits out these

Hi @ptoffy, thanks for the report.
The line "pending response got rejected since connection got disposed" in the logs indicates to me that sourcekit-lsp crashed on startup.
I see your swift.sourcekit-lsp.serverArguments is set to ["-enable-bare-slash-regex"]. -enable-bare-slash-regex is a compiler flag and not a sourcekit-lsp flag, which I think is causing it to crash. Could you try removing this argument from swift.sourcekit-lsp.serverArguments and restarting VS Code?
That said, the error reporting here is not great. I've created https://github.com/swiftlang/vscode-swift/issues/979 to track improving this failure message to give a clear report on why sourcekit-lsp failed to launch.

That seems to have done it! Thanks for spotting that @plemarquand


Fix interface cast syntax

Fix to check for Flusher and Hijacker interface conversion using the comma-ok syntax.

Hi @samber could you take a look at this please :)

Damn. Thank you!

-> v1.14.3


WIP - Add a test to reproduce a bug

PR Summary
Adds a test to make reproduce steps for #11858 to prove that the bug in #11851 is reachable.
PR Context
CI is expected to fail for now. You should be able to see the following errors:
Stop-Service : Service 'testrawservice1 (testrawservice1)' cannot be stopped due to the following error: Cannot stop testrawservice2 service on computer '.'.
At line:1 char:1
+ Stop-Service raw1 -Force
+ ~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : CloseError: (System.ServiceProcess.ServiceController:ServiceController) [Stop-Service], ServiceCommandException
+ FullyQualifiedErrorId : CouldNotStopService,Microsoft.PowerShell.Commands.StopServiceCommand

Stop-Service : Collection was modified; enumeration operation may not execute.
At line:1 char:1
+ Stop-Service raw1 -Force
+ ~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : NotSpecified: (:) [Stop-Service], InvalidOperationException
+ FullyQualifiedErrorId : System.InvalidOperationException,Microsoft.PowerShell.Commands.StopServiceCommand

After #11851 the InvalidOperationException will disappear.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

Please prefix the PR with WIP if it is not ready to merge

Closing as I don't plan to complete this at all.


Update HttpEventCollectorSender.java

Related to issue #12

Thanks for doing this work @bmagni-ob, this change is totally reasonable!
Can you please do the following:

Make a new PR against the develop branch
Fill out the Individual Contributor Agreement.

We'd also really appreciate if you added a test for this :smile:

Just did the PR on the develop branch and filled the ICA, accordingly.
As for the test, I will have to download your code to do it, so it will take me some time.
Best regards,


gopages: Link to repository

Instead of including the whole code as static HTML, it would be nice to have a command line flag to configure the repositories base path:
When clicking the Exit func on https://johnstarich.com/go/gopages/pkg/github.com/johnstarich/go/gopages/cmd/ it would open https://github.com/JohnStarich/go/blob/master/gopages/cmd/exiter.go#L12 instead of https://johnstarich.com/go/gopages/src/github.com/johnstarich/go/gopages/cmd/exiter.go#L12.
An option could look like -repo github.com/JohnStarich/go which gets expanded to https://<repo>/blob/master/<source-file-and-line> where <repo> expands to the value given by flag and <source-file-and-line> to, well... The source file and line. In the above example, that would be gopages/cmd/exiter.go#L12.
GitHub and GitLab use the same syntax, so that should be covered.

Sounds great to me 👍
I've started a draft with my progress thus far #4
I figure a Go template for link generation is probably the most flexible approach.

Awesome, looks great so far!

@lukasmalkmus Released in version v0.1.6 🎉
Let me know how it works out! 😄

Works like a charm, thank you!


Protocol Earnings Table on Dashboard Overview

Summary of changes
Provide a detailed description of proposed changes.

Protocol Earnings Table on Dashboard Overview
Update Chart Tooltips

Proposed area of change
Put an x in the boxes that apply.

[ ] apps/bridge-dapp
[ ] apps/hubble-stats
[ ] apps/stats-dapp
[x] apps/tangle-dapp
[ ] apps/testnet-leaderboard
[ ] apps/faucet
[ ] apps/zk-explorer
[ ] libs/webb-ui-components

Reference issue to close (if applicable)
Specify any issues that can be closed from these changes (e.g. Closes #233).

Closes #1983

Screen Recording
If possible provide a screen recording of proposed change.
https://github.com/webb-tools/webb-dapp/assets/69841784/0a96bd8e-1624-4d93-a61b-b8ffded62ce1

Code Checklist
Please be sure to add .stories documentation if any additions are made to libs/webb-ui-components.

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Looks good but colors might be off? Colors of the role on the pie chart, bar graph, and even down to the label color. They should be consistent? I thought I might've seen the label for DKG have a different color than the pie chart so worth double checking we have consistency here.

I just checked, they are the same @drewstone