Make heading links red

Closes issue #627 and PBI 63292
Make links within headings red instead of black.

Figure: Headings now red even though they are links

Hey @taineriley1
Can you make sure the underline is also in red. Also, IMO the hovering effect should be the opposite of the normal links... so in the headings, everything does dark grey (font and underline)
Cheers,
Tiago

Hi @tiagov8,
See the latest commits and screenshot.

Figure: Underline is red and link is dark grey on hover


Let two further tests tolerate col info in panics

Needed by https://github.com/rust-lang/rust/pull/42938
I've now ripgrepped for "panicked at" and found no further
test that hardcodes the "filename:line$" format.

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

@bors r+

:pushpin: Commit 02c97af has been approved by Mark-Simulacrum

:hourglass: Testing commit 02c97aff86c295a1c1f9687f84302be5fb54e0d9 with merge eb6cf012a6cc23c9c89c4009564de9fccc38b9cb...

:sunny: Test successful - status-appveyor, status-travis
Approved by: Mark-Simulacrum
Pushing eb6cf012a6cc23c9c89c4009564de9fccc38b9cb to master...


How will the Joint Office of Homelessness update the API's?

Hey Zak, you came up with a process that would allow the city to push 2017 PIT data to the project. Could you document that/link to the documentation here?

We're still concerned about having the city push the data automatically, in case it doesn't line up with the current cards. Because of this, we're going to ask them to provide the data and update the APIs manually.
I'm going to email the city and ask for the data as a CSV. We'll update the current API manually, and leave it to the next folks to build a service that will allow integration (you've probably already though of this, but this would be a great project for the next iteration of Civic @meganmckissack)


Giving the user a list of providers:

This is more of a life improvement more than anything else. Instead of exiting out of the program after incorrect input of  "calls:" , there could be a print("Please select from the following list of options") that keeps the loop running and allows for another entry. So, line 40 in Main.py could have an option to allow the user another try.
This is not a big deal if they have read the code/docs but could help with ease-of-use.
Great job so far.

Will be added in today's commit

Done!


Update kerberos docs with sample deployment

This PR has updated doc changes for quick start using arm template to setup Azure Bastion with Kerberos.

@sivagudivadaz, if this PR is ready for review and merge, please select the "Ready for review" button to move the PR out of Draft stage. Thanks.

@cherylmc
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#reassign:isamorris

@isamorris - Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#sign-off

@ShannonLeavitt yes merge

Merging based on prior indication of signed CLA.


Do we have any ideia how many tokens is used to run the full benchmark in a model?

I would like to run this on Gemini 1.0 Pro and Claude 3 so we have their scores but do we have any ideia of the token usage of this benchmark so we can calculate cost in commercial models?

The token usage is depending on the length you want to test. For example, if you want to test sequence length 128K with our default setting, you will have total tokens 131072 (length) x  500 (samples per task) x 13 (tasks) = 852M tokens. You can decrease the number of samples per task to save your budget.


[APIJSON] Impromve tests to it doesn't rely on SDK content

Remove the dependency of APIJSON tests on SDK to improve stability and
speed.
rdar://89448415

Test improvment.

Resolves rdar://89448415

@swift-ci Please smoke test

@swift-ci Please test

Linux bot failed because of jenkins issue and smoke test passed. Merge.


papermill

I am looking to upgrade to the 2.x of airflow where I used it with papermill.
I am no longer seeing papermill as an option, I do see it mentioned here: https://github.com/conda-forge/airflow-feedstock/pull/66
but if I do:
conda install -c conda-forge airflow-with-papermill

it never resolves. (which makes sense as I don't see it listed on here https://github.com/conda-forge/airflow-feedstock)
Is there a way to get papermill added to the conda packaging of airflow 2.x?

One more note.  It will likely be safer to create a new conda environment with conda create -n <name> -c conda-forge apache-airflow-providers-papermill than to try to upgrade an existing environment.  Better still, we recommend adding conda-forge as a default channel because mixing packages from different channels can have unexpected consequences:
conda config --add channels conda-forge
conda config --set channel_priority strict
conda create -n <name> apache-airflow-providers-papermill


Add ReadvRaw()

ReadvRaw() uses the readv syscall to read an iovector.
Signed-off-by: Joshua Brindle<EMAIL_ADDRESS>

CLA should be there now.


Rest framework conversion

Reopen after PR close cause by renaming

@TeerapatTrepopsakulsin You haven't change url reverse inside test pls change it and modify test case accordingly. Also there are join handler that haven't convert to REST yet. I've plan to do it myself tmr but I change my mind to go to health activity. Since you have time tomorrow pls convert that to rest too, by make class ActivityDetail handle POST request. Also don't forget to merge main and resolve conflict.

Change the URLs of the view to the new ones that use the REST framework.

Request detail

Index: GET - Show list of activities, POST - Create activity
Detail: GET - Show activity detail, POST - Join activity, PUT - Edit activity

Revise the tests to use new URLs.

Does these backend stuffs work with edit page? I got error 404.

  Does these backend stuffs work with edit page? I got error 404.

I have this problem too.

Another thing is, before the next review request please, merge the code from the main. I can't let you merge without reviewing it.


Add plot_maneuver

💡 Feature request
Currently sample() accepts a min and max anomaly. However, it is not always clear how to set these values. For example, in the Going to Jupiter example we have code like:
plotter.plot_trajectory(
    ic3.sample(min_anomaly=10 * u.deg, max_anomaly=180 * u.deg),
    label="Inner cruise 3",
    color="C3",
)

It is not clear why one should set 10 and 180 here.
In our particular case, we know the start and end time of the maneuver.
Another alternative could be to provide a plot_maneuver function that automatically takes care of plotting the whole maneuver but just that, ideally plotting also any intermediate impulses. This way, one could do:
man_flyby = Maneuver.lambert(ic1_end, ss_efly)
plotter.plot_maneuver(man_flyby, label="Inner cruise 1", color="C1")

@jorgepiloto Do you know how this could be implemented?

Fixed in #1359


Consolidate Netty channel flushes to mitigate syscall overhead

Motivation
This is similar to https://github.com/apache/bookkeeper/pull/3383.
When we are writing/reading a lot of small entries, one of the main bottleneck in Pulsar client/broker becomes the CPU usage. A big part of it is caused by the syscall overhead when writing to the socket.
The reason is that we have many independent (and small) operations happening on the connection and each time we call writeAndFlush() on each of them, causing many write() calls on the socket.
Netty has a mechanism to consolidate the flushes on the channel and it improves the handling of many small entries.

[x] doc-not-needed

Do we also need to enable it on Pulsar Proxy?

Do we also need to enable it on Pulsar Proxy?

Good point. Yes, we should do it there as well.

@merlimat should we make this configurable ?


Using composers autoloader

Coverage remained the same at 100.0% when pulling 976a1da628b9687eaec226319022773a14cdc4ef on feature/php-cs-autoloader into 657c16c5aaac472cbb411e34128ede0138959600 on master.

Coverage remained the same at 100.0% when pulling 976a1da628b9687eaec226319022773a14cdc4ef on feature/php-cs-autoloader into 657c16c5aaac472cbb411e34128ede0138959600 on master.


Ability to utilize secret value to populate Dex clientSecret

Is there a way to utilize a secret to populate the value for clientSecret within the dex config options?  There doesn't seem to be a way to include environment variables on a per component basis?

Hi @Kampe Thanks for raising issue with us.
Can you please provide more information on what exactly you are trying to achieve and steps to reproduce.

Sure!
Currently, this is seemingly the amount of config we can use to set up our client  for DEX:
      dex:
        config: |
          connectors:
            - type: github
              id: github
              name: GitHub
              config:
                redirectURI: https://localhost:8080/api/dex/callback
                clientID:<PHONE_NUMBER>7sd67f6
                clientSecret: $dex.github.clientSecret
                orgs:
                - name: org-name
             ```

However, I can't use my own secret to populate these values, both clientID and clientSecret within the DEX configuration block.  This would be wildly useful to everyone to have available.  Ideally they are NOT attached to the $dex.github object and can be their very own secrets as I do not want to touch argocd internal secrets to get to my own personal env vars.

Hi @Kampe , You can also use your own kubernetes secrets.
clientSecret: $dex.github.clientSecret # Alternatively $<some_K8S_secret>:dex.github.clientSecret

Is this what you are looking for ?

@iam-veeramalla please can you elaborate on this because the documentation still seems to be missing information.
I have created a secret in the same namespace as argocd.
apiVersion: v1
kind: Secret
metadata:
  name: argocd-github-secret
  namespace: argocd
data:
  clientSecret: 1ce6dfe5134cd281831b2ab648e9caf5e3d36027
type: Opaque

Then what should the config be?
data:
  application.instanceLabelKey: argocd.argoproj.io/instance
  dex.config: |
    connectors:
      - type: github
        id: github
        name: GitHub
        config:
          clientID:<PHONE_NUMBER>478c81c342
          clientSecret: $argocd-github-secret.clientSecret # or $argocd-github-secret:clientSecret

@brandon-piner-aruba I have come across the same problem. For me, it did not work when I put the value in custom secret.
However with some tinkering :) I got it to run when I put the value dex.github.clientSecret into secret called argocd-secret.
All other attempts of using secret created by me with the name sso , failed with the same message of level=warning msg="config referenced '$sso:dex.github.clientSecret', but the key does not exist in secret"
Could you maybe give some hint @iam-veeramalla on using custom secrets instead ?

We use bank-vaults w/transit so it's easiest for us to use env vars. I found that if you put $ENV_VAR_NAME in a value, you'll see an error, but the value is passed through to the config file and dex supports env vars in its config.
We set the env: value of the dex pod (helm) to have encrypted values, bankvaults decrypts them, and dex plucks them out of the env.
dex-server time="2023-09-25T20:01:50Z" level=warning msg="config referenced '$GOOGLE_OIDC_CLIENT_SECRET', but key does not exist in secret"

@joshperry Can you show how you did it?

What @RafPe showed worked for me.
Here is the docs that should help more:
https://argo-cd.readthedocs.io/en/stable/operator-manual/user-management/#sensitive-data-and-sso-client-secrets
Basically a couple of things to note:

If you are making a secret make sure to include a label in the metadata for app.kubernetes.io/part-of: argocd
If your secret is not named "argocd-secret" then to reference it in the configmap your syntax needs to be $<k8s_secret_name>:<a_key_in_that_k8s_secret>

For example with the secret:
apiVersion: v1
kind: Secret
metadata:
  name: argocd-oidc-client
  namespace: argocd
  labels:
    app.kubernetes.io/part-of: argocd
stringData:
  clientSecret: 'whatevertheactualsecretis'

A helm chart with values similar to the following will use the secret:
# ... other values ...
configs:
  cm:
    dex.config: |
      connectors:
        - type: oidc
          id: oidc
          name: OIDC
          config:
            issuer: https://issuer_address.com/
            clientID: 'whatever_the_client_id_is'
            clientSecret: $argocd-oidc-client:clientSecret
# ... other values ...

Note, in the example the secret is using stringData, but if you use the normal data field remember to base64 encode the value.

@joshperry Can you show how you did it?

Basically we just patched the argocd dex pod (using kustomize) to set our encrypted transit secret values in environment vars, then put the $ENV_VAR_NAME in the argocd dex configmap. The argo code that injects secrets, like @saulrobe posted, will complain that there's no value with that name. It still passes the $ENV_VAR_NAME intact into the dex container's config file which does support env var replacement, so at start dex will replace env var strings in its config from the container env.
If you're unfamiliar with bank-vaults transit interface, it's much easier to encrypt pod environment vars in a gitops repo because of its init-container hijacking method which is not available for resources that don't have an execution component, like k8s Secrets, or ConfigMaps.


PBFT Rebroadcast Soft Votes

Purpose

If have seen enough (2t+1) soft votes in the current round, PBFT should rebroadcast them in second finish state
fix https://github.com/Taraxa-project/taraxa-node/issues/1488
How does the solution address the problem

Changes made

prnet cleaned-up successfully


[Event Request] Table 121 "Purch Rcpt Line" - OnAfterCopyFromPurchRcptLine

Hello. In table 121 Purch Rcpt Line, in the procedure CopyFromPurchRcptLine, I would like to use the event publisher OnAfterCopyFromPurchRcptLine to add my own code. But I need the parameter TempPurchLine. Like this :
local procedure CopyFromPurchRcptLine(var PurchLine: Record "Purchase Line"; PurchOrderLine: Record "Purchase Line"; TempPurchLine: Record "Purchase Line"; NextLineNo: Integer)
begin
    ...

    OnAfterCopyFromPurchRcptLine(PurchLine, Rec, TempPurchLine);
end;

With this event integration :
[IntegrationEvent(false, false)]
local procedure OnAfterCopyFromPurchRcptLine(var PurchaseLine: Record "Purchase Line"; PurchRcptLine: Record "Purch. Rcpt. Line"; var TempPurchLine: Record "Purchase Line")
begin
end;

Do you think it could be possible ?
Thank you for your work.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


FromPyObject for #[pyclass] with T: Clone

I had an idea how to support extracting for pyclasses automatically for T: Clone.
It also has really nice error messages if the user attempts to extract a non-clone #[pyclass] (see the ui test).
Closes #706

Nice :+1: , thank you.
But please wait for the review for a while, since I want to check we can simplify the implementation or not.

Sure thing, take as long as you like!
I tried myself to do some simplifications but couldn't get much further than what's above.
It is possible to add some blanket impl ExtractPolicy for &T and &mut T, for example. But it's not possible to also have a blanket impl for T because it conflicts with the other two.

I did spot a slight simplification!
Also, I realised that if I work the FromPyObject implementation for Vec<T> using this new FromPyObjectImpl trait, we can get the buffer::Element optimization without specialization. See https://github.com/davidhewitt/pyo3/pull/1
It isn't perfect and it's really un-ergonomic to have to implement FromPyObjectImpl for everything to get Vec<T> to work.
So we might want to think a bit longer about this idea. I think if we can find a nice API for it then we can use it to remove specialization from impl FromPyObject for Vec<T>.


feat: condKL_compProd_meas

Add the lemma condKL_compProd_meas, about the kl divergence conditional to a compProd.
This PR depends on #16 and #17.

Merge master?

We can merge, but I don't have permission to do it

Sorry my comment was not clear: I was asking you to merge master into that branch, now that the dependencies are merged, in order to reduce the diff and check for conflicts.

Is it possible to use #35 to get a much shorter proof here?

Is it possible to use #35 to get a much shorter proof here?

Not with the same level of generality, because in the fDiv case we require μ to be a finite measure, while in kl we only require it to be S finite.


Spider Scans Extension Not Workinv

Source information
Spider Scans 1.4.30
Source language
English
Steps to reproduce
Spider scans extension not working in app but works fine in webview

Expected behavior
Should show titles and covers
Actual behavior
Showing no results found
Mihon/Tachiyomi version
"0.17.1"
Android version
"Android 11"
Other details
Should work fine in the app but only working in webview
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

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

Duplicate of #6641


type "blob" does not exist with postgresql

I have largely used this library with sqlite and everything has worked fine. Today, I went to try and use this library with postgresql and encountered this error when attempting to migrate the db:
2023/12/03 20:42:37 /Users/mya/go/pkg/mod/go.pitz.tech/gorm/encryption@v0.0.0-20231003002037-6719424dc92f/encryption.go:159 ERROR: type "blob" does not exist (SQLSTATE 42704)

There's probably an easy fix to this (namely a datatype change) but I need to look into it and also look at adding some tests for databases that gorm supports so this can be more "drop in".

Started on some of this... definitely a problem for MySQL and SQLServer too... need to find out how to handle this accordingly...


fix: input_port_types and other helper functions on pydantic schema

Fixes several schema issue. Closes #986

LGTM, thank you! 👍
(I can't approve because I made the PR)


Remove protobuf

Description

Related Issue

Type of Change

[ ] 📚 Examples / docs / tutorials / dependencies update
[ ] 🔧 Bug fix (non-breaking change which fixes an issue)
[ ] 🥂 Improvement (non-breaking change which improves an existing feature)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🔐 Security fix

Checklist

[ ] I've read the CODE_OF_CONDUCT.md document.
[ ] I've read the CONTRIBUTING.md guide.
[ ] I've updated the code style using make codestyle.
[ ] I've written tests for all new methods and classes that I created.
[ ] I've written the docstring in Google format for all the methods and classes that I used.

https://github.com/Giskard-AI/giskard/actions/runs/7049398784


LSTM training

The training example needs to be updated to match the new script we are using. (https://github.com/keras-team/keras/blob/master/examples/lstm_text_generation.py)
I'll push some changes to this by the end of the day

Thank you @cvalenzuela. Tagging @nikitahuggins so she knows to follow too!

@cvalenzuela should I remove this from the June 15th beta and we'll tackle later if/when performance improves with the latest tfjs LSTM models?

Yes, we have reverted to the previous implementation of the LSTM. See https://github.com/ml5js/ml5-library/pull/150.  I'm currently working on creating a tutorial with the script we already have

closing this because we ended up reverting this class.


Publish a Windows image for each supported Java version

What feature do you want to see added?
It would be nice to publish Windows images for each supported Java version, as since #1773 only a jdk17 image is published.
Related:

https://github.com/jenkinsci/jep/pull/400

Upstream changes
No response

As tests are currently run sequentially, I've kept only the JDK17 and the JDK11 in #1770 to not increase build time too much.
Note: the container images publication on trusted.ci.jenkins.io skips tests so no publication time increase.

As far as I know, this has been completed by

#1820


Formatting issues

Bold and italic doesn't apply on non selected elements. If bold or italic selected and sentence is typed then, respective formatting doesn't apply.

May I work on this issue ?

@hkaur008  pls assign this issue

@rishabhrathore055 Thanks for looking at this issue . actually we are thinking to remove draft.js dependency that's why on this issue the work is not carried forward . By the time you can look on other issues.


Refactor in-place strips shebangs, only if given prefixed path argument

When given a path argument with some kind of prefix:

./foo/bar
../foo/bar
/foo/bar

HLint with --refactor will strip shebangs:
% head tools/promote-restylers
#!/usr/bin/env stack
{- stack script
  --resolver lts-16.10
-}
-- vim: ft=haskell
module Main
    ( main
    )
where

% stack exec -- hlint --refactor --refactor-options=-i ./tools/promote-restylers
% head tools/promote-restylers

{- stack script
  --resolver lts-16.10
-}
-- vim: ft=haskell
module Main
    ( main
    )
where

But, if given a path without any such prefix:

foo/bar

It works fine:
% head tools/promote-restylers
#!/usr/bin/env stack
{- stack script
  --resolver lts-16.10
-}
-- vim: ft=haskell
module Main
    ( main
    )
where

% stack exec -- hlint --refactor --refactor-options=-i tools/promote-restylers
% head tools/promote-restylers
#!/usr/bin/env stack
{- stack script
  --resolver lts-16.10
-}
-- vim: ft=haskell
module Main
    ( main
    )
where

% stack exec -- hlint --version
HLint v3.1.6, (C) Neil Mitchell 2006-2020

By the way, Brittany currently has the same bug, and it feels too weirdly specific to not be the same underlying cause.

Running hlint --refactor with -i calls:
refactor ..\hlint\Sample.hs -i --refact-file empty.txt

Assuming that empty.txt has the contents [].
I tried in a bunch of configurations and it didn't strip the first line away. Could you try that more direct command line, and then if necessary we can raise a bug at https://github.com/mpickering/apply-refact/.
I've also made it so --verbose prints out the refactor command line, since before it was a bit hard to figure out.
CC @zliu41

Definitely makes sense this would be a bug in rafactor.
% echo '[]' > empty.txt
% cat > example <<'EOM'
\`heredoc> #!/usr/bin/env stack
\`heredoc> {- stack --resolver lts-16.10 script -}
\`heredoc> module Main (main) where
\`heredoc> main :: IO ()
\`heredoc> main = putStrLn "hi"
\`heredoc> EOM
% chmod +x example
% ./example
hi
% stack exec refactor -- --version
v<IP_ADDRESS>%

It indeed reproduces with ./example:
% stack exec refactor -- ./example --refact-file empty.txt
Applying 0 hints

{- stack --resolver lts-16.10 script -}
module Main (main) where
main :: IO ()
main = putStrLn "hi"

And works fine with example:
% stack exec refactor -- example --refact-file empty.txt
Applying 0 hints
#!/usr/bin/env stack
{- stack --resolver lts-16.10 script -}
module Main (main) where
main :: IO ()
main = putStrLn "hi"

Note that it's not a "first line" problem; it's specifically a shebang. Everything works fine in files without shebangs, even when invoked as ./example:
% tail -n +2 example | sponge example
% cat example
{- stack --resolver lts-16.10 script -}
module Main (main) where
main :: IO ()
main = putStrLn "hi"
% stack exec refactor -- ./example --refact-file empty.txt
Applying 0 hints
{- stack --resolver lts-16.10 script -}
module Main (main) where
main :: IO ()
main = putStrLn "hi"
%

% { echo "#\!something"; cat example; } | sponge example
% cat example
#!something
{- stack --resolver lts-16.10 script -}
module Main (main) where
main :: IO ()
main = putStrLn "hi"
% stack exec refactor -- ./example --refact-file empty.txt
Applying 0 hints

{- stack --resolver lts-16.10 script -}
module Main (main) where
main :: IO ()
main = putStrLn "hi"

FWIW, since this is so related to the actual syntax used to name the file argument, it's possible it's Linux-only and the windows-style ..\ paths don't trigger it either. :shrug:
^ Mostly adding this here to keep it in context. I'll report it over on apply-refact soon (unless someone else does first). Feel free to close this Issue if you think that makes sense.

I'm ambivalent whether it gets moved to apply-refact and left open here with a link to it there, or closed here - whatever people prefer. But definitely should end up with apply-refact so they can fix it :)

https://github.com/mpickering/apply-refact/issues/88. I'm also ambivalent on if we close this Issue or not.

I tried in a bunch of configurations and it didn't strip the first line away.

Since <IP_ADDRESS>, apply-refact no longer attempts to parse the target file if there's no hint. It just prints the target file as it is. So this problem does not occur if there's no hint. I think @pbrisbin was using an older version.
This is a ghc-exactprint bug, and I raised https://github.com/alanz/ghc-exactprint/issues/92.

This was fixed (a while ago) in ghc-exact-print <IP_ADDRESS>. Closing.


Build error when implements DataBindingComponent

In Github BrowserSample,
I have error with class FragmentDataBindingComponent when implements DataBindingComponent like that
Error:(9, 8) error: FragmentDataBindingComponent is not abstract and does not override abstract method getImageViewBindingInterface() in DataBindingComponent

Can't seem to reproduce. Closing for now.

@JoseAlcerreca @zulu-pham the BindingAdapters have to be provided by the class that implements DataBindingComponent. See https://stackoverflow.com/a/38216344/401025


Add LICENSE

Issue-id
PR Checklist

[ ] I have read the CONTRIBUTING.MD instructions
[ ] I have changed the resourceDefinition.json
[ ] I have generated the resource model (there's a models_generated.go file in my PR)
[ ] I have updated the README.md#resource-status
[ ] I have checked to ensure there aren't other open Pull Requests for the same update/change?

Description

Does this introduce a breaking change

[ ] YES
[ ] NO

Testing

Done


Stage 4 - Track down why best fit galaxy does not always appear on best fit line.

@Carifio24 pointed out that we should check for round off errors.


Initial Virtual Card email should encourage receipt upload like subsequent one

When a VC is charged, the initial email only tells people to look at the expense, and if you click through, you land in a normal view of the expense in processing state. After #5783 we started allowing uploading receipts in this state because there's no reason not to (even if the charge fails it doesn't hurt anything to have the receipt). So I think the initial email should be the same as the subsequent email, encouraging people to upload the receipt and landing them in the expense in an edit state, like the subsequent email does currently.
Initial email:

Receipt request email:

Related to #5951 - the system currently still sends the second email even if the receipt was already uploaded. This would need to be fixed for the UX to work well in this flow.

@alanna : Here I am thinking we change the initial emails View Expense button to Submit Receipt which when clicked will open the expense in the edit state. Does this the solution you have in mind? Is there any wording changes we would need here as well? 🤔

Correct @SudharakaP - I think we can pretty much make the initial email (first screenshot above) the same as the current receipt request email (second screenshot above). We can keep the current receipt request email the same, as long as it doesn't get sent if the person has already uploaded the receipt (as per #5951 ).

Correct @SudharakaP - I think we can pretty much make the initial email (first screenshot above) the same as the current receipt request email (second screenshot above). We can keep the current receipt request email the same, as long as it doesn't get sent if the person has already uploaded the receipt (as per #5951 ).

Thanks I've made sure the working of the two emails match each other; https://github.com/opencollective/opencollective-api/pull/8086/files


Calendar shows dates in the past instead of current date

Once you run the app and you go to search for flight, the calendar instead of opening in the current date, it shows May of 2020 for departure and February 2020 for return date.
Departure date calendar

Return date calendar

The normal behavior would be the calendar to show the current date.

Hi! I'm ready to help with it


Fix invalid_initial() test

When it tries to corrupt buf[], there is a 1/256 chance of test
failure when buf[written - 1] = 0. Try to have always different value
by inverting bits.

Nice! Though this should be done for the invalid_packet() test as well.

Nice! Though this should be done for the invalid_packet() test as well.

Sure. Fixed it as well.


Add hungarian language

Add hungarian translation

Thank you for providing your locale file.
I can't thank you enough.


Bindings for Python

We want to also expose the awesomeness of this library to Python so it because easy to use from within the Python ecosystem as well.

We can start by binding small library parts to Python like version, matchspec, etc.
It would be nice to use Python async for the async parts (like downloading).
We can use pyo3 to generate the bindings.

We have an initial version of python bindings implemented with #279 . More will be added.


Update service adapters to work with latest versions of services

Need to do a sweep of all the services and make sure things work with the latest versions.

This is done for the upcoming release 1.5.


chore: improve demo page

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Server Error: System catalog system_catalog does not exist.

Hi. I work at Harvard FAS Research Computing. We support a group that has used OmniSci on our HPC systems for a few years. We deploy OmniSci using a Singularity container that is built based on your Docker containers. We are behind and currently running version 5.5.5. I am trying to update to the HeavyAI latest version 7.0.0, but I am running into some errors.
I built a Singularity container based on the latest Docker container on HeavyAI DockerHub with:
singularity pull docker://heavyai/heavyai-ee-cuda:latest

I run the Singularity container with the following command:
singularity run --nv -B /etc/nsswitch.conf -B /etc/sssd/ -B /var/lib/sss -B /etc/slurm -B /slurm -B /var/run/munge -B /usr/bin/sbatch -B /usr/bin/srun -B /usr/bin/sacct -B /usr/bin/scontrol -B /usr/lib64/slurm/ -B /scratch/paulasan/55125905/var/lib/heavyai:/var/lib/heavyai --pwd /opt/heavyai /n/singularity_images/OOD/omnisci/heavyai-ee-cuda_latest.sif

The connections seem to be working and the Rebrand migration looks completed, but I suspect that the system_catalog error is causing startheavy to exit. This is the output that I get (I tried to point out the printout from FASRC script):
startheavy 3539282 running
WARN: config file does not exist, ignoring: --config /var/lib/heavyai/heavy.conf
Backend TCP:  localhost:7738
Backend HTTP: localhost:9625
Frontend Web: localhost:8732
Calcite TCP:  localhost:9462
- heavydb 3539291 started
- heavy_web_server 3539292 started
⇨ http server started on [::]:8732
Discovered OmniSci  server listening on port 8732!        # printout from FASRC script
TIMING - Wait ended at: Wed May 24 08:00:33 EDT 2023.     # printout from FASRC script
Rebrand migration: Added symlink from "/var/lib/heavyai/storage/mapd_catalogs" to "catalogs"
Rebrand migration: Added symlink from "/var/lib/heavyai/storage/mapd_data" to "data"
Rebrand migration: Added symlink from "/var/lib/heavyai/storage/mapd_export" to "export"
Rebrand migration: Added symlink from "/var/lib/heavyai/storage/mapd_import" to "import"
Rebrand migration: Added symlink from "/var/lib/heavyai/storage/mapd_log" to "log"
Rebrand migration completed
Server Error: System catalog system_catalog does not exist.
Generating connection YAML file...       # printout from FASRC script
Navigate to: http://localhost:8732
Failed to write to log, write /var/lib/heavyai/storage/log/heavy_web_server.holygpu7c26101.rc.fas.harvard.edu.paulasan.log.ALL.20230524-120032.3539292: file already closed
startheavy 3539282 exited
Cleaning up...

In our older version 5.5.5 the file omnisci_system_catalog is generated, but not in version 7.0.0, that's why I suspect the missing system_catalog is causing a problem.
Could you please provide some assistance on how to solve this issue? Let me know if you need any further information that I may have missed.
Thank you!

Hi Paula,
The issue could be generated because to move from 5.5 to 7.0, you should do an intermediate upgrade step to 6.0 before migrating to 7.0 as stated in the docs.
https://docs.heavy.ai/installation-and-configuration/installation/upgrading-omnisci
The steps to migrate to 6.0 using a docker container follow this link
https://docs.heavy.ai/installation-and-configuration/installation/upgrading-omnisci/upgrading-omnisci-1
Then you can migrate to 7.0. The 7.0 removes render groups, so a backup should be taken before the upgrade.
If you need further assistance, feel free to ask.
I can say that the migration from 5.x to 6.0 is creating a symlink and and is going to rename the system catalog file this way
lrwxrwxrwx  1 mapd mapd      14 gen 25 18:35 omnisci_system_catalog -> system_catalog
-rw-r--r--  1 mapd mapd   57344 apr  4 19:00 system_catalog
but I'm not sure that doing this is enough to have a reliable system.
Have you taken a backup, before the upgrade?
regards,
Candido

Hi Candido,
Thank you for getting back to me quickly. I pulled the 6.0.0 container but I am getting the same error.
To clarify, I don't have a database. We provide the Immerse interface and our users load their databases afterwards. This tutorial shows the interface that I am working on.
I don't have the omnisci_system_catalog stored anywhere. In version 5.5.5, this file is somehow generated when I run the Singularity container. These are the files generated after running the container:
[jharvard@holygpu2c0709 55175673]$ pwd
/scratch/jharvard/55175673
[jharvard@holygpu2c0709 55175673]$ cd omnisci-storage/data/mapd_catalogs/
[jharvard@holygpu2c0709 mapd_catalogs]$ ls -l
total 128
-rw-r--r-- 1 jharvard jharvard_lab 77824 May 24 15:31 omnisci
-rw-r--r-- 1 jharvard jharvard_lab 53248 May 24 15:31 omnisci_system_catalog

However, in version 6.0.0 (and 7.0.0) the system_catalog file is not generated. And since there is no existing omnisci_system_catalog, a symlink cannot be created either?
[root@holygpu7c26101 ~]# cd /scratch/jharvard/55174492/var/lib/heavyai/storage/
[root@holygpu7c26101 storage]# ls -l
total 8
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 catalogs
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 data
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 export
-rw-r--r-- 1 jharvard jharvard_lab    7 May 24 15:17 heavydb_pid.lck
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 import
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 lockfiles
drwxr-xr-x 2 jharvard jharvard_lab 4096 May 24 15:17 log
lrwxrwxrwx 1 jharvard jharvard_lab    8 May 24 15:17 mapd_catalogs -> catalogs
lrwxrwxrwx 1 jharvard jharvard_lab    4 May 24 15:17 mapd_data -> data
lrwxrwxrwx 1 jharvard jharvard_lab    6 May 24 15:17 mapd_export -> export
lrwxrwxrwx 1 jharvard jharvard_lab    6 May 24 15:17 mapd_import -> import
lrwxrwxrwx 1 jharvard jharvard_lab    3 May 24 15:17 mapd_log -> log
[root@holygpu7c26101 storage]# ls -l catalogs/
total 0

Hi Paula,
When you run the 6.0 against a storage of omnisci, files and directories are renamed and links are generated, and the catalogs and datafiles are processed as an upgrade step.
So the question is... Are trying to upgrade from a 5.x or you just want to create an environment with a new version for your users?
Inviato da Outlook per Androidhttps://aka.ms/AAb9ysg

From: Paula C. Sanematsu @.>
Sent: Wednesday, May 24, 2023 9:42:42 PM
To: heavyai/heavydb @.>
Cc: Candido Dessanti @.>; Comment @.>
Subject: Re: [heavyai/heavydb] Server Error: System catalog system_catalog does not exist. (Issue #783)
Hi Candido,
Thank you for getting back to me quickly. I pulled the 6.0.0 container but I am getting the same error.
To clarify, I don't have a database. We provide the Immerse interface and our users load their databases afterwards. This tutorialhttps://www.youtube.com/watch?v=4kraLwo3suI&t=2s shows the interface that I am working on.
I don't have the omnisci_system_catalog stored anywhere. In version 5.5.5, this file is somehow generated when I run the Singularity container. These are the files generated after running the container:
@.*** 55175673]$ pwd
/scratch/jharvard/55175673
@.*** 55175673]$ cd omnisci-storage/data/mapd_catalogs/
@.*** mapd_catalogs]$ ls -l
total 128
-rw-r--r-- 1 jharvard jharvard_lab 77824 May 24 15:31 omnisci
-rw-r--r-- 1 jharvard jharvard_lab 53248 May 24 15:31 omnisci_system_catalog
However, in version 6.0.0 (and 7.0.0) the system_catalog file is not generated. And since there is no existing omnisci_system_catalog, a symlink cannot be created either?
@.*** ~]# cd /scratch/jharvard/55174492/var/lib/heavyai/storage/
@.*** storage]# ls -l
total 8
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 catalogs
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 data
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 export
-rw-r--r-- 1 jharvard jharvard_lab    7 May 24 15:17 heavydb_pid.lck
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 import
drwxr-xr-x 2 jharvard jharvard_lab    6 May 24 15:17 lockfiles
drwxr-xr-x 2 jharvard jharvard_lab 4096 May 24 15:17 log
lrwxrwxrwx 1 jharvard jharvard_lab    8 May 24 15:17 mapd_catalogs -> catalogs
lrwxrwxrwx 1 jharvard jharvard_lab    4 May 24 15:17 mapd_data -> data
lrwxrwxrwx 1 jharvard jharvard_lab    6 May 24 15:17 mapd_export -> export
lrwxrwxrwx 1 jharvard jharvard_lab    6 May 24 15:17 mapd_import -> import
lrwxrwxrwx 1 jharvard jharvard_lab    3 May 24 15:17 mapd_log -> log
@.*** storage]# ls -l catalogs/
total 0
—
Reply to this email directly, view it on GitHubhttps://github.com/heavyai/heavydb/issues/783#issuecomment-1561831861, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHLFBF7A6FEGJHSKPWRONADXHZQDFANCNFSM6AAAAAAYNNXM5Y.
You are receiving this because you commented.Message ID: @.***>

I only want to create an environment with a new version for our users.
When I sent my first inquiry, I was creating the directories following directories manually:
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/data
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/catalogs
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/import
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/export
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/log
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/lockfiles

where $OMNIROOT is a local temporary storage.
After you asked the question about a new environment, I realized that creating those directories manually could pose a problem. So, I removed those mkdir statements. That solved my problem and it seems to work! Well, at least I can get to the Immerse interface, which is all I need. Now I can ask our users to test it.
Thanks a lot for your guidance and time!

Yes, it's not needed creating the subdirectories manually, but just the root /varlib/heavyai (or whatever), and then specify it in the data parameter of config file.
If empty, at the server startup, all the necessary structures will be created amd initialized, but if the directories exist, it will try an upgrade/rebrand.
When I saw these messages I thought that you where trying an upgrade from omnisci 5.5
I'm happy that everything is OK now, and I hope you are appreciating out suite.
Regards,
Candido
Inviato da Outlook per Androidhttps://aka.ms/AAb9ysg

From: Paula C. Sanematsu @.>
Sent: Thursday, May 25, 2023 2:05:20 AM
To: heavyai/heavydb @.>
Cc: Candido Dessanti @.>; Comment @.>
Subject: Re: [heavyai/heavydb] Server Error: System catalog system_catalog does not exist. (Issue #783)
I only want to create an environment with a new version for our users.
When I sent my first inquiry, I was creating the directories following directories manually:
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/data
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/catalogs
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/import
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/export
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/log
mkdir -p ${OMNIROOT}/var/lib/heavyai/storage/lockfiles
where $OMNIROOT is a local temporary storage.
After you asked the question about a new environment, I realized that creating those directories manually could pose a problem. So, I removed those mkdir statements. That solved my problem and it seems to work! Well, at least I can get to the Immerse interface, which is all I need. Now I can ask our users to test it.
Thanks a lot for your guidance and time!
—
Reply to this email directly, view it on GitHubhttps://github.com/heavyai/heavydb/issues/783#issuecomment-1562078566, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHLFBF5A42I6DZBPLRDVES3XH2O4BANCNFSM6AAAAAAYNNXM5Y.
You are receiving this because you commented.Message ID: @.***>

Thank you. Now that I understand what was happening with the existing directories and the migration, it make a lot more sense. Thank you for your help. I reached out to our users for testing, but I think we resolved this issue. I will close it.
Again, thank you very much for your guidance.


BUG: Profile section logo not shown

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Description
In Profile section, logos are not shown

Please, @arghadipmanna101 assign me this issue with the label.
Record

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

Please assign this issue to me under GSSOC .


set rgb as default column color  & change calendar format as time

i have field in database which contain rgb color,
can i set RGB as default colums color format in jexcel v3 ?
and second
in my record db format time, cant show value in column calendar
how to show time picker only & show my record in time in column, in jcalendar ?
thanks in advance

You need a custom column for the time only.


Computgraph xc

Tested RuntimeGeneratedFunction.
Might be useful for back end.

Codecov Report

Merging #98 (478d29b) into computgraph_kun (7dd8ee5) will increase coverage by 0.44%.
The diff coverage is n/a.

:exclamation: Current head 478d29b differs from pull request most recent head 8c67b59. Consider uploading reports for the commit 8c67b59 to get more accurate results

@@                 Coverage Diff                 @@
##           computgraph_kun      #98      +/-   ##
===================================================
+ Coverage            53.66%   54.11%   +0.44%
===================================================
  Files                   43       43
  Lines                 2808     2785      -23
===================================================
  Hits                  1507     1507
+ Misses                1301     1278      -23

Impacted Files
Coverage Δ

src/frontend/parquet.jl
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Zeta zero on boundary

I thought we justified our approach. @daviddritschel What do you think?

(if it needs changing, this could still be done in a later PR in any case)

I prefer having a final good solution now. We can add an option if necessary that forces zero on the boundary.


adding torch_cuda*

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #850 (089a631) into master (0d03049) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #850      +/-   ##
============================================
- Coverage     70.38%   70.32%   -0.06%
+ Complexity     5068     5065       -3
============================================
  Files           501      501
  Lines         22357    22359       +2
  Branches       2321     2321
============================================
- Hits          15735    15724      -11
- Misses         5395     5414      +19
+ Partials       1227     1221       -6

Impacted Files
Coverage Δ
Complexity Δ

...ine/src/main/java/ai/djl/pytorch/jni/LibUtils.java
43.55% <0.00%> (-0.40%)
18.00 <0.00> (ø)

api/src/main/java/ai/djl/repository/Artifact.java
89.65% <0.00%> (-3.45%)
36.00% <0.00%> (-2.00%)

...gine/src/main/java/ai/djl/dlr/engine/DlrModel.java
53.12% <0.00%> (-3.13%)
4.00% <0.00%> (ø%)

api/src/main/java/ai/djl/inference/Predictor.java
87.50% <0.00%> (-2.89%)
23.00% <0.00%> (-1.00%)

api/src/main/java/ai/djl/util/Platform.java
46.51% <0.00%> (-2.33%)
8.00% <0.00%> (ø%)

...g/src/main/java/ai/djl/serving/util/Connector.java
55.40% <0.00%> (-1.36%)
21.00% <0.00%> (ø%)

...ngine/src/main/java/ai/djl/mxnet/jna/LibUtils.java
44.73% <0.00%> (-1.32%)
15.00% <0.00%> (ø%)

...ne/src/main/java/ai/djl/mxnet/jna/StringArray.java
92.85% <0.00%> (ø)
9.00% <0.00%> (-1.00%)

...rc/main/java/ai/djl/paddlepaddle/jni/LibUtils.java
51.14% <0.00%> (ø)
14.00% <0.00%> (ø%)

...ain/java/ai/djl/modality/cv/util/NDImageUtils.java
64.47% <0.00%> (+1.31%)
21.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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

download links needs to be fixed as well

done.


samples, subsys, tests: Use ARRAY_SIZE() whenever possible

The ARRAY_SIZE() utility macro will actually test the parameter types,
and ensure that it is only called with arrays, and not arrays decayed
to pointers.
Changes were performed with a simple Coccinelle script.
Signed-off-by: Leandro Pereira<EMAIL_ADDRESS>

Codecov Report

Merging #8397 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #8397   +/-   ##
=======================================
  Coverage   64.61%   64.61%
=======================================
  Files         423      423
  Lines       40293    40293
  Branches     6801     6801
=======================================
  Hits        26034    26034
  Misses      11126    11126
  Partials     3133     3133

Impacted Files
Coverage Δ

tests/net/lib/http_header_fields/src/main.c
95.55% <100%> (ø)
:arrow_up:

...pool_concept/src/test_mpool_merge_fail_diff_size.c
100% <100%> (ø)
: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 2e0af08...ecdbdeb. Read the comment docs.


fix inline CSS not working

thanks akuma

Don't thank me, I requested a review on it and not my CSP code. Only got it from the issue linked to my PR :')

Coverage remained the same at 33.139% when pulling 8405c48d86ee8f6e99c5c2bbc6e7452e89b0ee59 on Kiloutre:patch-45 into 53ba669ba6021787003e997552e417ac4c8cd134 on NyaaPantsu:dev.

Changes Unknown when pulling bda85c7d34a856bf28dacd87d7a032dc6dc35338 on Kiloutre:patch-45 into ** on NyaaPantsu:dev**.


[Feature]: Implement PR Title Checker workflow for Conventional Commit Prefixes

Is your feature request related to a problem? Please describe.
Description:
The project would benefit from a PR title checker that ensures conformity to conventional commit prefixes. Currently, the following prefixes are considered valid:

build:
chore:
docs:
feat:
fix:
perf:
refactor:
revert:
style:
test:

Problem:
Enforcing consistent and descriptive PR titles following conventional commit prefixes helps maintain clarity and organization in the project's version history.
Describe the solution you'd like
Implement a PR title checker that validates each PR title against the specified conventional commit prefixes. This will ensure that PR titles adhere to the established guidelines.
To-Do:

[ ] Develop the PR title checker script to verify titles against the defined prefixes.
[ ] Integrate the checker into the project's CI/CD pipeline.
[ ] Define error messages or warnings for non-compliant PR titles.

Checklist

[X] I have starred the repository
[X] I have checked the existing issues
[X] I am willing to work on this issue
[X] I am an SWOC 2024 contributor

How do you propose we add this, @harmeetsingh11?


[graphql/rpc] limit num fragment defs

Description
Bound the number of fragment definitions
Test Plan
Manual, coming up
If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Similar question here as in the PR that introduced a variable limit -- why are we limiting the number of fragment definitions?

Similar question here as in the PR that introduced a variable limit -- why are we limiting the number of fragment definitions?

Similar response, plus fragment defs technically count as nodes we have to analyze.


separate app config to a different repo

and use makefile to complete the orchestration based on local environment

configuration repo moved to https://github.com/digihunchinc/orthanc-config


Use entrypoints to install petablint as a command

Added missing requirement "colorama"
Update README.md accordingly
Move /bin/petablint.py into main module and add Python entrypoint

Now it's possible to call petablint like any other unix command (e.g., grep, cat, etc.). Python automatically takes care of this when pip installing.

Codecov Report

Merging #85 into master will decrease coverage by 5.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #85      +/-   ##
==========================================
- Coverage   59.93%   54.47%   -5.46%
==========================================
  Files           5        6       +1
  Lines         609      670      +61
  Branches      129      140      +11
==========================================
  Hits          365      365
- Misses        220      281      +61
  Partials       24       24

Impacted Files
Coverage Δ

petab/petablint.py
0% <ø> (ø)

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 7915af4...dc3fca3. Read the comment docs.

@yannikschaelte good call! I didn't realize that you had the develop/master dichotomy. Perhaps it might be helpful for new developers to make the develop branch the default one.
I've merged the changes from develop into this branch then pointed the merge to the develop branch. Looks like one of the changes (adding the colorama requirement) had even already been fixed!


containenv should be language-agnostic

virtualenv is for python projects.
Our alternative should, initially, inherently support python (3), go, & node.js. Other languages can be added later.

First steps:
research what each of these languages uses to identify project dependencies (like the requirements file in python) so we know what we'll need to consult for things to install when creating a new containerenv.

python -- requirements.txt
go -- see https://golang.org/doc/code.html && http://jbeckwith.com/2015/05/29/dependency-management-go/
node.js -- package.json; see https://devcenter.heroku.com/articles/node-best-practices

Nice work @jputrino.


[AutoPR cognitiveservices/data-plane/TranslatorText] typo: cognitiveservices/data-plane/TranslatorText

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5484

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/4455


DateTime columns get from database have Kind property Unspecified.

I setup a simple unit  test that adds a row to the database and the date that is added to the database has Kind property set when inserting it as Utc by default. As its added using DateTime.UtcNow.
When i get the same object back from Database using dapper / dapper fastcrud the same field has the correct datetime but the Kind property is Unspecified - when it should be Utc.
I realize that this could be something that maybe need to be put in dapper. Is there any overrides we can do to fastcrud library that can set the kind property to the datetimes retrieved?

Not specific to any of the two libraries. It is a known annoyance. The SQL field has no space to store the time zone info. People recommend switching to datetimeoffset.

@moxplod how did you eventually solve this problem?

I ended up not implementing anything at the Data access layer for this, As that would affect all the dates while performing the mapping to DTOs. Was'nt sure if the performance would be good when we do that. In our scenario - we had to have the kind property populated in a few situations. So its handled in a higher layer when we need to convert to User's timezone - it checks if the Kind property of the datetime passed in is unspecified - then sets it to Utc.

When you attempt to convert to user's time zone, wouldn't an Unspecified kind behave the same as UTC?

I cannot remember exactly - but some method was blowing up with an
exception that the DateTime has to have kind set as Utc.
On Thu, Oct 6, 2016 at 5:25 PM, MoonStorm<EMAIL_ADDRESS>wrote:

When you attempt to convert to user's time zone, wouldn't an Unspecified
kind behave the same as UTC?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MoonStorm/Dapper.FastCRUD/issues/39#issuecomment-251997088,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEV-m8k-Ys80c_1ph-mBXhlJOu-0ptI4ks5qxRLngaJpZM4JUc62
.


proposal: make runtime.reflectcall followable by pprof

Context: http://stackoverflow.com/q/31419307/532430
Proposal: allow pprof to follow reflective function invocations via runtime.reflectcall so that code "after the jump" can be profiled.
I think the idea is simple enough but if anything was unclear just ask and I'll try to clarify.

To be honest, I'm not sure this has to be a proposal.  I think the current behaviour is a bug.  I don't know what is involved in fixing it, but I can't imagine that anybody would object to a good fix.

Agree, this needn't be a proposal.

This is a bug in pprof alone. The full information is in the profile, but the elision of runtime frames is getting confused by the reflect call and eliding everything below it.


Fix URL Formatting Issues

Current Behavior:
The edit_webhook, delete_webhook, and get_webhook_by_id functions use the following URL format:
"{}v0/webhooks/{}/?api-key={}"

This causes these function calls to fail.
Problem:
The current URL format includes an unnecessary slash before the webhook_id  query parameter, leading to incorrect URLs and failed function calls.
Expected Behavior:
The functions should use the correct URL format:
"{}v0/webhooks/{}?api-key={}"

With this format, the functions should successfully call the webhooks API without errors.

Thanks for catching this! It has been closed with #27


Add security tests for lack of SSL

Starting with a check for '#8 Add support for SSL'

The latest version already has a number of security tests, so going to rename this issue to reflect the SSL request

expired to do


Electron Vue Project - Error: Failed to compile

description
I built a test project with Vue-CLI 3 and electron-builder plugin running Typescript and Babel. When I tried to connect to Trilogy (sql.js), I get the error below.
source
import Trilogy from 'trilogy'

import store from '@/store'
import { storiesSeed, productsSeed } from '@/database/testDBSeed'

const db = new (Trilogy as any)(store.getters.dbPath, {
	client: 'sql.js'
})

export async function initDB() {
	const didLoadStories = store.getters.didLoadStories
	if (!didLoadStories) {
		await loadStories()
	}

	const didLoadProducts = store.getters.didLoadProducts
	if (!didLoadProducts) {
		await loadProducts()
	}
}

error
 ERROR  Failed to compile with 12 errors                                                                                21:59:28

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'closeSync' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'dirname' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'dirname' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'dirname' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'dirname' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'mkdirSync' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'mkdirSync' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'openSync' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'readFileSync' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'resolve' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'statSync' from non EcmaScript module (only default export is available)

 error  in ./node_modules/trilogy/dist/trilogy.mjs

Can't import the named export 'writeFileSync' from non EcmaScript module (only default export is available)

expected

What should happen in this situation?
Proceed with database ops.

actual

What happened that's considered unexpected?

The app crashed as soon as i saved file above with trilogy code.
I added '.mjs' to vue.config.js to no avail. I'm really excited about this library but this is an unfortunate development. Has this happened before?
module.exports = { 	configureWebpack: { 		resolve: { 			extensions: ['*', '.mjs', '.js', '.vue', '.json', '.ts'] 		} 	} }

I found the solution in another library on GitHub: https://github.com/apollographql/apollo-link-state/issues/302
I edited vue.config.js in project root directory and now it's working:
module.exports = {
	configureWebpack: {
		resolve: {
			// .mjs needed for Trilogy
			extensions: ['*', '.mjs', '.js', '.vue', '.json', '.ts']
		},
		module: {
			rules: [
				{
					test: /\.mjs$/,
					include: /node_modules/,
					type: 'javascript/auto'
				}
			]
		}
	}
}

I hope this helps someone.


Repo sync working branch

  Caution: Large pull request
This public repo applies limits on the number of files that can be merged in a single pull request (PR):

Warning Warning message and ability to unblock PR using the Merge large pull request* label - 30
Block Block message and PR is blocked - 100

This pull request will change 44 files, which exceeds the warning limit of 44.
Before this pull request can be merged, you must review the list of files it contains and their changes to confirm they are correct and intentional. This includes files you did not change and/or do not own. If there are changes you aren't familiar with, contact the individuals who submitted the commits included in this pull request and review the changes with them.
The individual who merges this pull request is responsible for ensuring the changes included in it are correct, and for resolving any issues that might result in merging it.
To merge this pull request
If you have determined that the changes in this pull request are correct, add the Merge large pull request* label to it, wait for the max/pr-file-count check to pass, and then click Merge. To add a label, click the gear icon next to Labels and then select Merge large pull request*.

To reject this pull request
If you can't confirm the changes in this pull request are correct, click Close pull request at the bottom of this page to close it without merging. Work with your team to revert any changes that aren't correct.
* You must have write or triage access to this repo to add labels.


DALL•E 3 support

The big guys strongly want to support the DALL•E 3 interface Well, DALL•E 3 feels too good to use on the web

@hehua2018 DONE! Please chk latest version!


Configuration

I've read the README, but it doesn't matter, configuration is MANDATORY for any application, plugin, whatever, even more when that application is implementing multiple independent features (In this case, topbar icons and title bar).
Look at these apps for good configs:
https://extensions.gnome.org/extension/1011/dynamic-panel-transparency/
https://extensions.gnome.org/extension/708/panel-osd/
https://extensions.gnome.org/extension/977/recent-items/
I'd love to send a merge request, but don't have time leider.
This is a convienent extension you made, improve it by allowing configurations.

There is nothing actionable in this issue. Without mention of what configuration you are missing, there is not much to discuss. Writing MANDATORY in capital doesn't change anything.
Configuration come with a high price. Higher code complexity to handle various configuration, maintenance of the configuration screen, migration of configuration data from a version to the next, translations (yes, there are a TON of people outside the UK/US). Apparently, you have no time to manage this.


Improve performance of MarkSeries using shouldComponentUpdate

Prompt
Improve rerendering performance of MarkSeries component by using shouldComponentUpdate
Problem
The MarkSeries component receives several props of type function (onNearestX, onNearestXY, etc.). If these functions are called, the MarkSeries component will automatically rerender all datapoints by mapping this._renderCircle on them, regardless if the change is necessary.
Proposal
Only rerender when props affecting the view are changed.
Before and After
Before

After

Is this optimization useful in other series?
And can you provide the benchmarks you used to validate it? (I know you did some testing, that should be in this PR for future reference.)
I'm generally a bit wary of shouldComponentUpdate optimizations that are not super simple, they often seem to backfire on you...and React DOM diffing often is all the optimization you need given that rendering is usually the most expensive part.

I would strongly caution against any work that reproduces every single one of the props that a series might take in a single place. This effectively means that anything a new prop is introduced it will need to be duplicated into this one specific series.
I agree with @ericsoco's sentiment that this could be moved into abstract series so that components are not updated when their listeners are called. Given that we name listeners in a standard way (onNearestX onMouseOver onBlanketyBlank) it could probably be inferred which props need to be checked from a simple heuristic regex. Adding this to AbstractSeries would certainly be a big win and would probably lighten up the load of some of the other more complex series, such as contour etc.
Also plus 1 for some bench marks, proving you're right is good.


coil-pin

something form the group
regarding the coil, wait for version 0007, then you get a dedicated pin for the coil ...
thank you!

thank you for this functions, it is now in the Aetherone!

Implemented


/usr/bin/qemu-lite-system-x86_64 meet Wrong!!!

Description of problem
I follow steps as :
https://github.com/kata-containers/documentation/blob/master/install/docker/centos-docker-install.md
Expected result
when I run command "docker run busybox uname -a".
Results as follows:

Actual result
when I don't use kata setting ,just use docker ,i can run command successful

docker info :

I have solved this problem ,because i use librbd.so file is tooooo old. ummm ，so sorry

@xiaoding945 - no problem! - thanks for reporting, and letting us know what you found. It is good to have the history recorded here on github so others can search if they hit a similar problem!


Feature needed on main branch - Branch 808 add pgrouting type

Pirmasens Fair will be happening on 12 Nov. All agents for Pirmasens will be deployed on a digital ocean machine concurrently on the main branch.
I am creating an IsochroneAgent which is outlined below

Uses SPARQL to query for points of interest's locations
Automates the generation isochrones from the locations
Calculate based on different transport mode (Walk, Cycle, Drive)
Calculate population coverage within isochrone
Input edge table to select flood_cost_table for calculations
Planned to be used for two use cases -  Pirmasens 15 minute smart city and King's Lynn Flood Router.

Pre-requisite:
IsochroneAgent will be needing pgrouting features from branch 808.  Currently branch 808 features satisfy the development of IsochroneAgent, no additional features required at the moment.
Timeline:
Planned 4 weeks time to develop IsochroneAgent.
The suggestion is to merge 808 features to main in 3-4 weeks, so that there will be sufficient time to troubleshoot and test in case anything (stack-manager or any other agents) breaks on main branch.
Do let me know your comments on this and if there is already a timeline planned out on merging 808 branch into main. Thank you.

Small correction, just for the record: We aim to deploy everything on a physical server in PS.


Support "and", "or", "not" on where/filterBy query

Describe the bug
Support "and", "or", "not" on where/filterBy query
Expected behavior
based on hasura

definitely going to support this, but this feature won't make it into the 0.1.0 release. Theres a good chance it'll be added in the next minor release


pull request for e2e test at 01/28/2017 07:45:45

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


making some changes in the documentation styling

Improving the Documentation, like highlighting the main diseases that the website can detect ->  #121

Was the issue assigned to you?

I made a issue, thinking the Documentation could be improved

I made a issue, thinking the Documentation could be improved

but before creating a PR you should have wait for us to assign you the issue, thats the part of code of conduct for any open source project.

I made a issue, thinking the Documentation could be improved

but before creating a PR you should have wait for us to assign you the issue, thats the part of code of conduct for any open source project.

Ya sure. Thanks for letting me know

Thanks @utkarsh0702 I will surely try to make some significant changes


devicelab win-2 bot is down

bot has been missing for 6+ hours.

applied salt states, and bot is back up.


feat: add function and unit tests

:tada: This PR is included in version 1.7.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add a SyntaxVisualizer panel

This implements a syntax visualizer view, which is styled very similarly to the panel that ships in the roslyn-sdk for VS. There are 2 parts to this view:
The top tree view shows the entire syntax tree of the current file, including the type of the node and the span of the node. It also uses the same type icons that vscode uses in other contexts to indicate whether the node element is a class or struct. Selecting a node in the tree highlights the span in the current document that corresponds to that node, and clicking in the document reveals the node in the tree view.
The bottom view is another tree view that contains detailed information about the currently-selected node in the top view. It automatically updates any time the top selection is changed, and contains the node type, the syntaxkind, the results of calling GetTypeInfo/GetConversion, GetSymbolInfo, and GetDeclaredSymbol, and all public properties of the node. The omnisharp-roslyn side of this PR is here: https://github.com/OmniSharp/omnisharp-roslyn/pull/2078.

I still need to add a setting to turn this panel on/off. My current thought is off-by-default.

I still need to add a setting to turn this panel on/off. My current thought is off-by-default.

I still need to add a setting to turn this panel on/off. My current thought is off-by-default.

I agree with your intuition. =)

@filipw @JoeRobich would it be possible to get reviews on this and the corresponding omnisharp-roslyn side PR?


Decide whether and how to add default syscalls to the SeccompProfile CRD

Part of the idea of #117 was to perhaps ensure that a standard set of default syscalls were present in a profile so that a minimum viable pod could be launched in kubernetes on e.g. runc. This issue is to evaluate whether this is worthwhile and discuss how to implement it.
The first consideration is that it's possible that the syscalls used by runc could change over time, and in fact they haven't been fully documented (https://github.com/opencontainers/runc/issues/2097) so we would need to maintain our own list. Moreover, a container runtime other than runc might be used, and container managers on top of runc like docker/cri-o might also call other syscalls. Also, the list might be different depending on the architecture.
Another consideration is the pause container, which seems to use an additional set of syscalls on top of what is required for runc. Should those syscalls be included in the defaults?
Finally, the actions to take for a syscall aren't a blanket "allow" or "deny", there is also e.g. "log", "trace", "kill", etc. If a user creates a complex profile with different actions for different syscalls, should the default syscalls use the "allow" action or the "log" action? What happens if the user adds one of the default syscalls to their "log" rule, but the controller is automatically adding it to the "allow" rule?
One idea could be to simply maintain an example SeccompProfile that contains the minimum required syscalls instead of building it into the CRD/controller. Thoughts?

I agree this is a though topic. Maybe we could start with documentation around that topic and explain to the user:

Which syscalls are going to be expected by nearly every workload, because they're required by the OCI runtime (like runc)
Which syscalls are additionally required, for example to run kubectl exec -it …

We could scope the documentation on an x86_64 environment and describe that this will differ on different architectures.
Adding the minimal example as well as an e2e test around it seems like a good idea, too. We can show to the user how to use that minimal profile as a starting base for profile development. And that brings me to the next question: Would it be possible to stack/combine two profiles now that we have a CRD? We could define a new field baseProfile for a CRD and afterwards just merge them together and write it on disk, right?

Would it be possible to stack/combine two profiles now that we have a CRD? We could define a new field baseProfile for a CRD and afterwards just merge them together and write it on disk, right?

I think that could work...so it would look something like this:
---
kind: SeccompProfile
metadata:
  name: base
spec:
  defaultAction: SCMP_ACT_ERRNO
  syscalls:
  - action: SCMP_ACT_ALLOW
    names:
    - capget
    - ...
---
kind: SeccompProfile
metadata:
  name: hello-world
spec:
  baseProfile: base
  targetWorkload: hello
  syscalls:
  - action: SCMP_ACT_ALLOW
    names:
    - arch_prctl
    - ...
  - action: SCMP_ACT_LOG
    names:
    - pause
    - ...

/var/lib/kubelet/seccomp/operator/default/hello/hello-world.json:
{
  "defaultAction": "SCMP_ACT_ERRNO",
  "syscalls": [
    {
      "action": "SCMP_ACT_ALLOW",
      "names": ["capget", "arch_prctl", ...]
    },
    {
      "action": "SCMP_ACT_LOG",
      "names": ["pause", ...]
    }
  ]
}

Still, what should happen if there are syscalls with different actions in the base versus user-defined profile? e.g.
base:
  syscalls:
  - action: SCMP_ACT_ALLOW
    names:
    - capget

user-defined:
  syscalls:
  - action: SCMP_ACT_ERRNO
    names:
    - capget

merged:
...
  "syscalls": [
    {
      "action": "SCMP_ACT_ALLOW",
      "names": ["capget",...]
    },
    {
      "action": "SCMP_ACT_ERRNO",
      "names": ["capget", ...]
    }

That would be an invalid profile, I think? But I guess no different than if the user had created just a single invalid profile...

Would it be possible to stack/combine two profiles now that we have a CRD? We could define a new field baseProfile for a CRD and afterwards just merge them together and write it on disk, right?

I think that could work...so it would look something like this:
---
kind: SeccompProfile
metadata:
  name: base
spec:
  defaultAction: SCMP_ACT_ERRNO
  syscalls:
  - action: SCMP_ACT_ALLOW
    names:
    - capget
    - ...
---
kind: SeccompProfile
metadata:
  name: hello-world
spec:
  baseProfile: base

What about baseProfileName to have a more strict reference?

…
merged:
...
  "syscalls": [
    {
      "action": "SCMP_ACT_ALLOW",
      "names": ["capget",...]
    },
    {
      "action": "SCMP_ACT_ERRNO",
      "names": ["capget", ...]
    }

That would be an invalid profile, I think? But I guess no different than if the user had created just a single invalid profile...

Yes, in the first iteration we could leave out those validation steps from my point of view. But it would be a good addition to check for syntactically, but logically wrong profiles later on. :+1:
Thinking about some other pitfalls here but I do not see anything right now…


Traceback error

Doing:

from kik import KikApi
kik = KikApi(BOT_USERNAME, BOT_API_KEY)
config = kik.get_configuration()
config.webhook

Errror:
http://puu.sh/o929u/140f3d4b29.png

@Jarmahent Thanks for the report. Looks like a python3 bytes/str issue, PR incoming to fix.

This should be fixed by #6 in 1.0.4


Add WPGraphQL 0.13.0 support

There were no breaking changes for this plugin, so this plugin supports 0.10.3 to <0.14.0 now.

published in<EMAIL_ADDRESS>


[FEATURE REQUEST] OSE SRD Compatibility

I was wondering if there was a way to make it so that players using OSE Infravision could have the vision deactivated if they are in eyeshot of a light source?

https://oldschoolessentials.necroticgnome.com/srd/index.php/Hazards_and_Challenges#Infravision
So, in this instance, the dwarf should not be able to see the red areas because the area they are in is lit.

This kind of thing may be possible with V10's Vision Modes. But PV is not going to implement system-specific vision modes. It should be the system's job to do so.


Documentation for Blade

Turbolinks is built / tested with Blade, but Blade lacks any kind of documentation.

Blade provides a convenient Sprockets API, much like sprockets-rails does. It also sets up a QUnit test runner.
Apologies for the lack of documentation. I plan to write some and welcome help. Closing because this isn't really a Turbolinks issue. Feel free to open one with Blade.

Thanks @javan. Was reluctant to open over there since it's a personal repo, and there's not much activity from anyone besides yourself :). I figured that now was a decent time to ask, though, since it's a dependency of Turbolinks, which is (I think?) included in Rails by default.


Showing simple/basic statistics in admin page

Added new function in each datastore(users, messages, conversations) that retrieves the size of each.
Called each function and assigned the returning integer to its corresponding size.
Displayed sizes in the front end
Created tests for each datastore function created

Looks pretty much perfect to me, my only serious question would be why an Integer type was returned as opposed to an int type for your three count functions.


[DOCUMENTATION] Enhance current documentation

Is your request related to a problem? Please describe.
The current documentation does not always have syntax highlighting, examples and does not make use of all the features we could make use of.
Describe the solution you'd like
Enhance the documentation (this does not currently include writing more documentation) by doing the following things:

[ ] Add description, status, tags and more to current pages
[x] #105

Describe alternatives you've considered
None, there needs to be something done.
Additional context
None.

https://github.com/I-Language-Development/I-language-rust/commit/3a049f9e379c4c81fcf508cf29351ff7eb1de396 also contributes to this, even though it isn't the documentation itself.


jenkins: switch runtime tests from 4.9 to net-next on master

As discussed in community meeting: this is required for #17058.

This PR does not need to run any CI and can be marked as ready-to-merge once all reviews are in.

test-runtime-4.9


how about the fps?

@BoPang1996
how about the inference time?

1.5fps on V100

@BoPang1996
Get it, thanks.

Is the limiting factor the 3D NN architecture? I was hoping it would be able to run a bit faster to work in real time. Either way great work, really enjoyed reading the paper!


[Bug]: iFrame - On setting onClick event to navigate to a different page, user is unable to return to the original page where the iFrame is present.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
https://www.loom.com/share/c99ecf443f2944e1aae6082ce067a355
Steps To Reproduce

Drop an iFrame widget and add in the srcDoc details
Set the onClick event to navigate to a different page (same window) onSrcDocChanged
Change the content of the SrcDoc
User gets navigated to the set page (page 1 in this case) but is unable to come back to page 4 (original page)

Public Sample App
No response
Version
Cloud

Stats

Stat
Values

Reach
16

Effort (months)
0.5

@somangshu this is a critical issue in some sort, because the user can't go back to the older page anymore and there's no workaround other than exporting and updating the DSL manually. If you also the think the same, we can bump the priority.

This is the default behaviour on the iFrame widget, As soon as the widget load, and the src is set, the onSrcDocChanged is triggered, we need to restrict this particular event, on page load, when the iframe widget is loaded, so not trigger the event. cc @techbhavin we have to first identify is this is possible. Adding it to the current sprint for tracking

@somangshu Yes, we can do as you mentioned things.


How can I externalise dependencies like React so that it doesn't need to be bundled per component

Is this possible? I am experimenting with importing multiple components for a client's custom DLS, but each time I import I am packaging fonts and React core. Can I build with external vendors and supply these through a window global for example?
Update
One strategy could be to asynchronously load JS dependencies and attach them to the window global scope yourself, making them "preloaded" for other components that use them. But I'm wondering if this would be considered bad practice or if it could cause unknown side-effects due to being asynchronous JS within constructor / init.
For example, using https://github.com/ded/script.js
It would also require changing webpack config to declare these libraries as externals, so that they are still bundled correctly. Would be great to get your thoughts on this approach.

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

ID: dfca439f-2902-4e1c-47b2-e96c1fe6d78f
Version Independent ID: a0406aa3-5035-a4db-3706-ee3b5dd908a5
Content: FAQs | MicrosoftDocs - Power Apps
Content Source: powerapps-docs/developer/component-framework/faq.md
Service: powerapps
GitHub Login: @ghurlman
Microsoft Alias: grhurl

Thanks for your feedback. We are closing this issue because it has remained idle for over 2 years. If you're still experiencing the issue described here, please reopen or create a new issue.

Thanks for your comment, @lukesiedle. This issue has been assigned to the doc author for review.


Check empowerment module performance

Empowerment more than doubles sim runtime.
With empowerment:
import fpsim as fp
empwr = fp.Empowerment()  # The module that will equip women with empowerment attributes
empwr_anz = fp.empowerment_recorder()
s1 = fp.Sim(location='kenya', empowerment_module=empwr, analyzers=empwr_anz)
s2.run() # 10.8 seconds

Without empowerment:
s1 = fp.Sim(location='kenya')
s2.run() # 4.2 seconds

The empowerment analyzer is the time sink -- keeps track of too many things by default
import fpsim as fp
import sciris as sc

sc.tic()
s1 = fp.Sim(location='kenya')
s1.run() # 4.0 seconds
sc.toc()

sc.tic()
s2 = fp.Sim(location='kenya', empowerment_module=fp.Empowerment(), analyzers=fp.empowerment_recorder())
s2.run() # 10.8 seconds
sc.toc()

sc.tic()
s3 = fp.Sim(location='kenya', empowerment_module=fp.Empowerment())
s3.run() # 6.7 seconds
sc.toc()


Portable toplevel exit

Experimentation with the #382 PR opened to get the CI to test my changes on multiple versions.

Pushed the changes to #382 so this can be safely closed.


Run tests against Zarr 3

There's an alpha release available now that can be installed using pip install --pre
https://pypi.org/project/zarr/3.0.0a0/

I had a quick look at this and it seems like quite a bit of stuff is broken. I hit a wall at finding a way to specify a name for a new array in a Group. This seems pretty basic and I don't have time to chase down the details, so I'll try again when the next alpha comes out.

I'll try again when the next alpha comes out

There is no next alpha according to https://github.com/zarr-developers/zarr-python/issues/1777.

I just had another go with the tip of the v3 branch, and still hitting walls with array creation. The support for creating v2 arrays seems to be pretty thin, and it's not clear at all to me how we're supposed to go about it. I don't really know where to start tbh.

 The Zarr v3 core spec does not cover strings.
They are likely to be a future extension:

The set of data types specified in v3 is less than in v2. Additional data types will be defined via extensions.

Thanks Tom!
Yeesh, the lack of strings is scary. Looks like we'll be on v2 for a long time then.

fwiw I would love to chart a path to getting more dtypes into zarr v3 (since i happen to be working on the v3 fill value normalization right now). as you noted, there's a dtype extension mechanism built into the spec but we haven't exercised it yet. Could you share or link to a description of how you are using string arrays, either here or in a discussion over in zarr specs? That might help kick-start things.

Hi @d-v-b, thanks for commenting! We'd actually like to encode variable-length strings, which use an object dtype and a numcodecs.vlen.VLenUTF8 codec.
I've added a comment explaining our use case to the discussion at https://github.com/zarr-developers/zarr-specs/issues/83

I filed the following issues to improve Zarr Python v3 API compatibility here:

[ ] https://github.com/zarr-developers/zarr-python/issues/2027
[ ] https://github.com/zarr-developers/zarr-python/issues/2028

As an experiment I tried creating a VLenUTF8Codec which uses numcodecs.vlen.VLenUTF8. I can successfully run a test that writes a VCF Zarr file and then validates it:
pytest 'tests/test_vcf_examples.py::test_by_validating[sample.vcf.gz]'

Amazing! :tada:

I updated https://github.com/tomwhite/bio2zarr/tree/zarr-v3 to use the code from https://github.com/zarr-developers/zarr-python/pull/2036 and the test still passes.


Fix bucket subresources query

Fix for https://github.com/basho/riak_cs/pull/1057

Without this, erlcloud_s3 tries to create a request as bucketname.s3.amazonaws.com?l&o=&c=... and fails. This is because s3_xml_request only takes attributes as a list of strings.

+1

Real plus one :see_no_evil:


Findings and Problem counts are inconsistent in VS Code

:tada: This issue has been resolved in version 0.48.4 :tada:
The release is available on:

v0.48.4
GitHub release

Your semantic-release bot :package::rocket:


[FLINK-27500] Validation errors should not block reconciliation

This PR solves the issue detailed in https://issues.apache.org/jira/browse/FLINK-27500 by reconciling using the previous (valid) spec if validation fails for some reason.
This allows rollback/jm-recovery operations to proceed even if an invalid CR was submitted in thea meantime

cc @Aitozi @morhidi


load the correct module depending on env

I made an identical change in a subsequent PR, so looks good to me!
If this gets merged I can rebase and update my own PR.

Identical change included in subsequent PR has been merged.


Downport 731

OK, I guess this part is ready for now (#15 )

downport
docs update
linter conf change

702
allow defines (<740 without macros is a pain)
allow non top definition (please ! this is much more readable and logical)
remove constraint on FS naming (please ! FSs are usually iterators and already have those brackets, adding prefixes to them is imho unnesessary)

at least no macros please

ok, will have a look how to remove them

similar to https://github.com/larshp/abapGit/pull/3185 ?

yeah ... readability is a must ... guess it should be possible to make a universal utility for this ... =)))) maybe ...

done


can we scaffold a module inside wasmd using ignite cli?

faced lot of errors when ran ignite scaffold command inside wasmd

Please check with the ignite team: https://github.com/ignite/cli/issues/1837#issuecomment-1024966299


ln: Implement -L -P to make tests/ln/hard-to-sym.sh work

ah? bizarre, i will have a look

Thanks :)


Add support for linux_{ppc64le,aarch64}

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

This will unblock: https://github.com/conda-forge/openldap-feedstock/pull/5


Debugging using PDB instructions do not work

When following the instructions in http://docs.qgis.org/2.2/en/docs/pyqgis_developer_cookbook/ide_debugging.html#debugging-using-pdb I get the following error:
Got exception:  global name 'pyqtRemoveInputHook' is not defined

closing as stale,  If this is still relevant, please reopen
Cheers


style: #12 remove frozen_string_literal strings

ref #12

[x]  # frozen_string_literal true strings were removed
[x] cop Style/FrozenStringLiteralComment was disableed in commit 7aa4750ced7f42a44049f5e872dbc10a376c4892

@lain0 could we restrict this frozen_string_literal to use? To fail build if someone added this option again


No synced nodes available

After installing and setting up Firefly v0.3.1. (Windows 10 Pro) I get the error message "No synced nodes available".
"Manual node selection" does not work either.

Same as here, you need to update to v0.4 https://github.com/iotaledger/firefly/issues/882#issuecomment-819290833

Ok, ich habe es auch gerade gesehen. ThanX.

Ok, I just saw it too. ThanX.


No such file or directory

Description
It's working on a simulator, but when changing to a device, there's a build error.
Checklist

[X] I have read the README before submitting this report.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
can run on a device
Actual behavior
build error :
no such file or directory: '/Users/user/Library/Developer/Xcode/DerivedData/SwiftUIIntrospect-dedgxvczfmevyagrtjzlvhjnlbge/Build/Products/Debug/PackageFrameworks/SwiftUIIntrospect-Dynamic.framework/Versions/A/SwiftUIIntrospect-Dynamic'
Steps to reproduce
clone repo
open SwiftUIIntrospect.xcworkspace with Xcode
build / run Showcase on a device
error occurs
Version information
SwiftUIIntrospect 1.1.1
Destination operating system
iPhone X / iPhone SE 2nd / My Mac
Xcode version information
15.0.1
Swift Compiler version information
swift-driver version: 1.87.1 Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.108 clang-15<IP_ADDRESS>)
Target: arm64-apple-macosx14.0

same issue

See https://github.com/siteline/swiftui-introspect/issues/318


Set up development pipelines for all headless platforms

Create development publishing pipelines for testing new UI features on their way to production instances.

We have an odi-publishing pipeline now. Though at this point I think we can start thinking of it as a series of repeatable steps rather than one static entity. Assigning this to @aaronhans  since  digital.ca.gov should be the only one left. And @chachasikes for visibility.
(Also - Maybe we should change the title so the meaning is more scannable during board grooming sessions?)

@zakiya Happy for better issue name suggestions.
digital.ca.gov is done. (I checked it off above.)

I created a ticket to optimize the pipeline for digital.ca.gov. The production instance of that site is still hosted on github pages, will move that to S3 as I setup all the staging/test pipelines from the pantheon WP instances
https://github.com/cagov/digital.ca.gov/issues/40

Great work on this @zakiya!
Recommend closing this ticket now


ex-soap - problems with installation

Hey jorge07,
I need to install ext-soap in order to install one of my dependencies. How do I do it with your image? Can you describe this in few words?
Thanks in advance,
Oskar Barcz

Hi Oscar
You'll need to add php7-soap in the [Dockerfile] (https://github.com/jorge07/symfony-5-es-cqrs-boilerplate/blob/symfony-5/etc/artifact/Dockerfile#L7) apk add step in the dev step and the prod one. This means that there's probably room for improvement in the Dockerfile to be a bit more DRY, will give it a thought.

Okey, but I'm kinda lost there - on regular server I'd use
apt install php74-soap,
and what exactly I'm supposed to do here?

It's an alpine OS and uses apk as package manger. You can find base image and OS version here and search for the official packages you need here. Codecast repository is included in base image so you can install packages from there by simply adding @cast at the end of the package name.
Example: RUN apk add php7-soap@cast

Now it's clear, thanks a lot for your work - I didn't expected such fast response!

Added this PR as reference #204


Bump version to 3.3.2

Bump version to 3.3.2

Coverage remained the same at 99.686% when pulling 0a123cc35d4332617c10e6b33bd6141cfb8e72e3 on bump-version into ea4d606ac527213cfb1bde517e592b4d4863639c on master.


fix #6 replace __ctx with _ctx$

My fix for #6. The compiler was outputting __ctx instead of using the CTX_VAR which causes an undefined error.

Thanks!


Convert de translations from solidus 1.2 -> 1.3

Recent changes made to admin translations in solidus moved many of the keys. This was done to better use the ActiveModel translation conventions.
I wrote a script that scans through the locale files in solidus_i18n looking for missing keys when compared to en.yml in core. Since these translations are just moved, the script attempts to make the same moves in this locale as were made for english.
Reviews would be appreciated to find any blatant mistranslations.

Thank you Luuk and Thomas.


x/tools/godoc,gddo: do not swallow code after output comments if it is not in concluding lines

What did you do?
Pushed example code to github:
func ExampleScanner() {
	frame := byframe.Encode([]byte("test"))
	s := byframe.NewScanner(bytes.NewReader(frame))

	for s.Scan() {
		fmt.Println(string(s.Frame()))
		// Output: test
	}
}

Here's the link for the project: https://github.com/ysmood/byframe/blob/7f2ce7731dfc716eb39bb99f4a93e2f43ce11fac/byframe_test.go#L32
What did you expect to see?
On the godoc website we should see
frame := byframe.Encode([]byte("test"))
s := byframe.NewScanner(bytes.NewReader(frame))

for s.Scan() {
    fmt.Println(string(s.Frame()))
}

What did you see instead?
The last } was missing:
frame := byframe.Encode([]byte("test"))
s := byframe.NewScanner(bytes.NewReader(frame))

for s.Scan() {
    fmt.Println(string(s.Frame()))

Here's the link for the godoc: https://godoc.org/github.com/ysmood/byframe#example-Scanner

I believe this is because your Output: comments are before the closing brace. As you can see here - https://golang.org/pkg/testing/#hdr-Examples, output comments are always concluding line comments.
Can you move the output comment below the closing brace and check ?

Thank you for your quick response, If I move it out, it will work properly.
But I still think this is a bug or something need to be optimized. If my code format is wrong the go test should fail or the godoc website should report the misuse, not silently swallow the close-brace.

Should be doable with a little bit of work. The offending code is here -
https://github.com/golang/tools/blob/2e34cfcb95cb3d24b197d58fe6d25046b8f25c86/godoc/godoc.go#L588-L591
Instead of just blindly wiping anything after the index, we need to start from bottom and stop after the first non-comment line.
Note that this is affects both godoc and godoc.org. The two have different codebases.

this works on pkg.go.dev.


add decodeJsonAsAssoc config param to allow get data as an array

@cosenary @vinkla please, take a look at this small PR


feat: Dropping support for MySQL 5.7

This PR removes logic used to handle MySQL version 5.7. Self-managed customers using RDS MUST upgrade to MySQL 8 before applying this version of the module. The upgrade is not required in cases where RDS is not used, but strongly recommended.
See: https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/USER_UpgradeDBInstance.MySQL.html

This PR is included in version 4.8.0 :tada: