Disabling profiler because SIGPROF handler is already in use.

Originally reported on Google Code with ID 393
I got this warning when I use profile on my virtual machine of “2.6.18-274.12.1.el5xen
#1 SMP Tue Nov 29 14:18:21 EST 2011 x86_64 x86_64 x86_64 GNU/Linux”.
But I didn't install SIGPROF handler myself, I debug my main and sigation()the SIGPROF,
I Got this:
$2 = {__sigaction_handler = {sa_handler = 0x37ec4d62c0 <profil_counter>, sa_sigaction
= 0x37ec4d62c0 <profil_counter>}, sa_mask = {__val = {18446744073709289215,<PHONE_NUMBER>6913,
140737488344672,
<PHONE_NUMBER>,<PHONE_NUMBER>45072,<PHONE_NUMBER>48,<PHONE_NUMBER>7520, 0,<PHONE_NUMBER>,<PHONE_NUMBER>76,
46912498372608, 9830400,<PHONE_NUMBER>,<PHONE_NUMBER>45336, 9924248, 0}}, sa_flags = 335544320,

  sa_restorer = 0x37ec4302d0 <__restore_rt>}

I do not install profil_counter myself, and why I got the warning? any help?

What is the expected output? What do you see instead?

What version of the product are you using? On what operating system?

Please provide any additional information below.

Reported by thunder54007 on 2012-01-11 12:15:53

Apparently I hit into the same issue. Some hours of debugging give me the answer - the reason in -pg flag to compiler. google-perftools sees that SIGPERF handler is not set to DFL or NUL and decides to not to install own handlers. And SIGPERF handlers are installed before main from glibc library if you building with -pg .

Hi Dmitry.
Thanks for update. But I'm not sure what your intention is. Do you want us to fix anything? In this case there doesn't seem to be anything to fix.

My intention was to put a notice for someone who hit this problem in order to save his time for debugging. For sure there is nothing to fix in gperftools. Thank you for your attention!

Hi,
I'm facing the same issue.
" Disabling profiler because signal 27 handler is already in use.
nFrames decoded: 300 (176x144 @ 1255.98 fps)
PROFILE: interrupts/evictions/bytes = 0/0/64"

So you're not supposed to build your program for gcc profiling (i.e. no -p or -pg or stuff like that). Perhaps we should change the message to point this out more clearly.

Perhaps we should change the message to point this out more clearly.

Yes please.  That would have saved me searching for this page.

Agree. Good opportunity for someone to contribute little improvement.


staticWindowsAPIServerConfig is missing some flags

Is this a request for help?:

No
Is this an ISSUE or FEATURE REQUEST? (choose one):
ISSUE
What version of acs-engine?:
Issue can be reproduced with v0.18.1 and head

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes
What happened:
When provisioning a Kubernetes cluster with Windows agents, we noticed some flags are missing from the apiserver config. For example, when "enableEncryptionWithExternalKms": true is added to the apimodel.
What you expected to happen:
Expected to see --experimental-encryption-provider-config flag in kube-apisever.yaml
How to reproduce it (as minimally and precisely as possible):
Provision a Kubernetes cluster with Windows agents and "enableEncryptionWithExternalKms": true in the apimodel.
apimodel:
{
  "apiVersion": "vlabs",
  "properties": {
    "orchestratorProfile": {
      "orchestratorType": "Kubernetes",
      "orchestratorRelease": "1.10",
      "kubernetesConfig": {
        "enableEncryptionWithExternalKms": true
      }
    },
    "masterProfile": {
      "count": 1,
      "dnsPrefix": "blahblahblah",
      "vmSize": "Standard_DS2_v2"
    },
    "agentPoolProfiles": [
      {
        "name": "agentpool1",
        "count": 3,
        "vmSize": "Standard_DS2_v2",
        "availabilityProfile": "AvailabilitySet"
      },
      {
        "name": "windowspool2",
        "count": 2,
        "vmSize": "Standard_DS2_v2",
        "availabilityProfile": "AvailabilitySet",
        "osType": "Windows",
        "osDiskSizeGB": 127
      }
    ],
    "linuxProfile": {
      "adminUsername": "azureuser",
      "ssh": {
        "publicKeys": [
          {
            "keyData": "blah"
          }
        ]
      }
    },
    "windowsProfile": {
      "adminUsername": "azureuser",
      "adminPassword": "blah"
    },
    "servicePrincipalProfile": {
      "clientId": "blah",
      "keyvaultSecretRef": {
        "vaultID": "blah",
        "secretName": "blah",
        "version": "blah"
      },
      "objectId": "blah"
    }
  }
}

Anything else we need to know:
cc @bnookala

will provide a PR for this soon

@PatrickLang FYI this looks like it might result in broken Windows clusters

Just to add more data:
The current behavior here https://github.com/Azure/acs-engine/blob/v0.18.1/pkg/acsengine/defaults-apiserver.go#L76-L77 will result in flags like --cloud-provider and --cloud-config missing from kube-apiserver.yaml when it's a hybrid cluster. But I'm still able to create resources on azure via kubernetes....why is that?


Can't seem to connect Baby Buddy to Home Assistant

Hi there. Firstly, thank you for creating this integration. I'm about 3 weeks away from my first child being born and would like to be able to incorporate some kind of logging system into HA so that my partner and I can keep track of what has/hasn't happened with the baby,
I am totally green when it comes to a lot of this stuff so have been fumbling my way through setting up a working Home Assistant setup that i'm currently happy with but have been really struggling to get Baby Buddy up and running. I first tried installing it using Docker Compose on a Raspberry Pi 3B, which I have some other containers running on without issue. But I couldn't for the life of me figure out Baby Buddy. The database seem to generate well enough but i can't connect to the app via the local IP address.
After several hours reading over the various threads on the repo, and feeling very in over my head, i thought i'd give it a shot setting it up via the Heroku method. why not? I wasn't holding out much hope but i registered an account and hit the "Deploy to Heroku" button on the Baby Buddy repo and lo and behold, it installed and worked without a hitch. I now have an external URL hosted on Heroku that I can access a Baby Buddy instance. hooray!
However, this victory was quickly quashed as I now can't seem to connect it to HA via that URL. Seemed like it should be easy, no? I installed your component and setup my config like so:
sensor:
 - platform: babybuddy
     address: http://*****.herokuapp.com/
     api_key: **************
     ssl: false

Rebooted HA and crossed my fingers - but no cigar. No senors populated into HA after a reboot. I even tried creating a few sleep and tummy time events in Baby Buddy as I thought that perhaps it wouldn't create sensors if the logs were empty, but still nada.
I'm really not sure what I'm doing wrong. Any advice or troubleshooting tips you might have would be super appreciated.

I can try and help. Please post your Home Assistant error log.

thank you let me track it down

ahh actually! i managed to sort it. turns out i needed to remove the "http://" from the URL as it was posting it like this when trying to connect: "http://http://*****.herokuapp.com/"
Now i can see 3 sensors created. one for the baby, and for last_sleep and last_tummy_time. Should have checked the logs from the start! rookie mistake. Thanks for pointing me in the right direction :)
Are you familiar with Heroku at all? i'd much rather have Baby Buddy running locally on a raspberry Pi, but it was such a struggle getting this far that i'm nervous to futz around with anything further now that I have it running.
 I noticed that it's not possible to send info to Baby Buddy yet via this integration... in my reading earlier i saw someone had setup a button system to update Baby Buddy via Node Red/MQTT, may have to look into that next.

Are you familiar with Heroku at all? i'd much rather have Baby Buddy running locally on a raspberry Pi

Not really. I run Baby Buddy via Docker and recommend that.

Yes.

I noticed that it's not possible to send info to Baby Buddy yet via this integration...

Not yet. See issue #6.


PostPolicyV4 classes could be improved

Classes defined in PostPolictyV4.java are not as perfect as they can be.
Enhancements and bugs are listed below:
class PostPolicyV4

the class description is stingy, need to put more words and point to usage example
public methods miss Javadoc
no verification for given URL string is a valid URL
BUG:  getFields() returns the map as is, so one can modify PostPolicyV4 objects after creation
(suggested fix:  make fields immutable)

class PostPolicyV4.PostFieldsV4

BUG: public method getFieldsMap() returns map that allows to alter PostFieldsV4 instances without Builder (no control)
(suggested fix: deprecate method getFieldsMap(), make the map immutable, define getters)

class PostPolicyV4.PostConditionsV4

BUG getConditions() returns mutable Set
BUG getConditions() exposes non public class PostPolicyV4.ConditionV4
(suggested fix: make PostPolicyV4.ConditionV4 public, make conditions immutable)

@frankyn could you look at problems listed in the description and say what you think about them.

Good catches @dmitry-fa, can you move forward if time permits?

@frankyn, sure I'll fix that issue. And then cover that functionality with tests (in a separate PR)

@frankyn I found another issue which I don't know how to address: when I specify the content-length the POST request fails with with "400 Bad Request"
I use ITStorageTest.testSignedPostPolicyV4 to play with post policy and I just updated
PostFieldsV4 fields = PostFieldsV4.newBuilder().setAcl("public-read").build();
to
PostFieldsV4 fields = PostFieldsV4.newBuilder().setAcl("public-read").setContentLength(11).build();
Should it be setContentLengthRange(from, to) instead?

https://cloud.google.com/storage/docs/authentication/signatures#policy-document states:
All valid HTML form fields except Content-Length can use exact matching.

ContentLength range support is in conditional policy: https://github.com/googleapis/java-storage/blob/02dc3b5e5377d8848c889647e72102cd9acc646d/google-cloud-storage/src/main/java/com/google/cloud/storage/PostPolicyV4.java#L270-L273
ContentLength in the Policy is wrong and should be deprecated with a note.

@frankyn
Looking at methods of PostPolicyV4.PostConditionsV4.Builder I came up to the following:
addContentLengthCondition(ConditionV4Type type, int contentLength)
should be deprecated and updated to do nothing
addExpiresCondition(ConditionV4Type type, long expires)
addExpiresCondition(ConditionV4Type type, String expires)
addSuccessActionStatusCondition(ConditionV4Type type, int status)
make sense only with MATCHES type and could be deprecated by methods accepting just one parameter, like addExpiresCondition(long expires)
or ["starts-with", "$success_action_status_condition", "5"] could be interpreted somehow?
addAclCondition(ConditionV4Type type, String acl)
addBucketCondition(ConditionV4Type type, String bucket)
addCacheControlCondition(ConditionV4Type type, String cacheControl)
addContentDispositionCondition(ConditionV4Type type, String contentDisposition)
addContentEncodingCondition(ConditionV4Type type, String contentEncoding)
addContentTypeCondition(ConditionV4Type type, String contentType)
addKeyCondition(ConditionV4Type type, String key)
addSuccessActionRedirectUrlCondition(ConditionV4Type type, String successActionRedirectUrl)
could check that type != CONTENT_LENGTH_RANGE


Add I18N Support

In addition to ZH_CN UI, also support EN UI.

Got!

Implemented by https://github.com/pingcap-incubator/tidb-dashboard/pull/35


feat: add support for env_var_extensions autodetect configuration

Some projects contain env vars that are have non-standard naming conventions and don't follow the common .tfvars and .tfvars.json patterns. In this case autodetect fails to pick up the variable files as part of the ProjectLocator configuration.
This change adds support for a autodetect configuration to allow us to specify non-standard naming conventions. This, alongside the existing env_names configuration field should satifsy most tfvar requirements.
However, we still have the issue of the tfvars-env.json usecase which is used by some customers. I've had to make exceptions for this in this PR because I think adding "configurable" support for this might be a bit more complex. In future we might look to add a env_var_pattern config, which accepts a regex. This means we could catch the prefix/suffix combo scenario.

seeing some wierd stuff testing this with real data, hold off reviewing


reference: fix broken alias for DomainRegexp

relates to https://github.com/distribution/distribution/pull/4031

An incorrect alias snuck into 152af63ec5c569f074e9cf5d0e409d6928e034d8 (https://github.com/distribution/distribution/pull/4031), and DomainRegexp was aliased to the regex for digests (DigestRegexp).
This didn't affect this repository, as it didn't use the aliases and migrated to the new module, but does affect user of the old module that depend on the aliases.

Ouch!

Yes, massive "face-palm" 🤦 😬 when I found it; https://github.com/moby/moby/pull/46672#issuecomment-1770270371

I'm working on some branches to update go versions, and golang.org/x/net to address the recent CVEs; perhaps we should do another release after that.


Document flux functions

Document the model.abc.photon_flux and model.abc.energy_flux functions, showing how to use them (and how to apply parameters to the loaded SpectralModel object)

I'll also add wrappers so that you can enter floats both as parameters and energy limits

Emphasize the fact that it's a single trapezoid integration, and using it to compute fluxes in a large band requires to sum over several smaller bins


Update automation-tutorial-runbook-textual.md

Semantics. The VMs of the script start in parallel not sequentially

@SnehaSudhirG
Can you review the proposed changes?
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

@SnehaSudhirG - Could you review this public PR?

@SnehaSudhirG
Can you review this old PR and determine whether it needs to be closed or merged?
@MicrosoftDocs/public-repo-pr-review-team

@SnehaSudhirG
Can you review this PR and determine whether it needs to be closed or merged?
@MicrosoftDocs/public-repo-pr-review-team

I sent an email to the content owner today.

I sent a follow-up email.

#sign-off


spelling fixes (round 3)

(see commit message for list of changes)

Hi @jsoref, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

fwiw, these files are dirty (on master, even w/o my changes):
lib/lib.core.d.ts
lib/lib.core.es6.d.ts
lib/lib.core.es7.d.ts (this file isn't tracked)
lib/lib.dom.d.ts
lib/lib.d.ts
lib/lib.es6.d.ts
lib/lib.es7.d.ts (this file isn't tracked)
lib/lib.webworker.d.ts
lib/tsc.js
lib/tsserver.js
lib/typescript.d.ts
lib/typescript.js
lib/typescriptServices.d.ts
lib/typescriptServices.js

Hey @jsoref, do you mean that the files in lib have spelling issues? I think @yuit will be doing some work on modularizing src/lib so we should hold off on that to avoid any merge pains for both of you. :smile:

Also thanks, these bite-sized PRs are much easier to work with!

@DanielRosenwasser I mean that when you run the build system, it overwrites files that are part of the repository with versions that aren't equivalent to the version under version control.
Normally, I'd expect the generated content to match the versioned content, or for it not to be versioned.


⚠ Bofrost.es has degraded performance

In 026f042, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1971 ms

Resolved: Bofrost.es performance has improved in 2479b02 after 6 minutes.


Suggestion

How about being able to manifest in some way to high-insanity players for a short time at the cost of power drain.
Also, power seems to regen quite quickly.

I do like that manifesting idea. Probably won't be a thing this update since I don't understand the networked stuff yet, but should be a good idea for the future.
As for the power regen, next update should have more config options to tweak things like that.

Sounds great.
Though, for manifesting, I was thinking maybe through an audio drone?
Would that be hard to implement?
(just, like, pressing 1 to toggle ghost sounds, maybe 2 gives you a second to say something, albeit extremely distorted)

That sounds really cool.
Maybe you could make a ghost visor of some sort that worked similarly?

I'm not sure what you mean, like on the screens of ghosts? Kinda how living players can see the edges of the mask?

Oh, sorry, I meant glasses or something that let you see the ghost models while they're equipped instead of masking the models.

That could be a fun idea if I decide to do a little custom loot or something like that.

Closing as resolved as of v1.1.0.


Doc Slot data returned name

Documentation mistake
In the README.md:

for the slot room-list-item, the data returned is room not rooms

Fixed https://github.com/antoine92190/vue-advanced-chat/commit/c324bcd370d4d9b9c7cc2ed28fb2a0cc1a11d509


Add try/except warning for ligo.skymap

Solves issue #31 . Also tested on a fresh conda installation with Python3.8.13.
The strict dependency of ligo.skymap is also removed.

This is a duplicate issue with #33 , so we will close this PR as the PR from #33 solves this issue.


Uncaught TypeError: this.getDOMNode is not a function

When scrolling I'm getting an Uncaught TypeError: this.getDOMNode is not a function in below function
scrollListener: function () {
         var el = this.getDOMNode();
         var scrollTop = (window.pageYOffset !== undefined) ? window.pageYOffset : (document.documentElement || document.body.parentNode || document.body).scrollTop;
         if (topPosition(el) + el.offsetHeight - scrollTop - window.innerHeight < Number(this.props.threshold)) {
           this.detachScrollListener();
           // call loadMore after detachScrollListener to allow
           // for non-async loadMore functions
           this.props.loadMore(this.pageLoaded += 1);
         }

This package is no longer being maintained, so if you're using a newer version of React, like 15.x you need to be using the fork I'm running at https://github.com/CassetteRocks/react-infinite-scroller.
You're getting this error because the function for getDOMNode has been moved/renamed in the newer ReactDOM API.

Thanks for the reply. I'm started using your fork. It is working fine thank you.
@danbovey


Conclude what data structure to use as inputs and outputs

Usually, PyTorch uses raw tensors as inputs and outputs to the networks. However, since our implementation is highly modular, and we sometimes want to include things like attentions, masks, etc., we use dictionaries of tensors as our message; this way, one module can use all or some components of the output, as well as output all or some.
We should fix the strategy of using a tensor or a dictionary of tensors as Centre input - it's likely to be the dictionaries in the end, but we need to think about a general strategy, and if maybe we want to use something else.

We should use namedtuples as the data structure - they will be faster than using dictionaries and this might be relevant for our highly frequent function calls.


FR / Idea: Support sub-data-fetcher in GraphQL for EntityViews

Motivation
I cases, where an GraphQL response should include a specific EV (which fetches data from the database), but also needs additional fields which cannot be fetched via the database. Areas like 'ldap configuration' loaded from the database, but also the server connection status connectStatus which uses the ldap client to test the connection. Via would love to be able to use sub data fetchers for that purpose.
Example
This is the core view
class LdapConfigurationView {
   String hostname;
   Integer port;
}

This is the extended view, included a pseudo computed field which does nothing (it's v6ue should be replaced via a sub data fetcher)
class LdapConfigurationWithStatusView {
  default Boolean connectionStatus {
    return null;
  }
}

Our graphql queries would look like
    ldapServerConfigurations(
        "Amount of entries per page" first: Int!, "Start element index for the requested page" offset: Int!
    ): LdapConfigurationWithStatusViewConnection!

The implementation would be
// this is where we try to replace the value via a sub fetcher
  @DgsQuery(field = "ldapServerConfigurations")
  public Object getServerConfigurations(DataFetchingEnvironment dfe)
  {
    final var setting =
      graphQLEntityViewSupport.createPaginatedSetting(dfe);

    if (setting.getMaxResults() == 0) {
      return new GraphQLRelayConnection<>(Collections.emptyList());
    }

    return new GraphQLRelayConnection<>(ldapRepository.getServerConfigsPaged(setting));
  }

// this is where we try to replace the value via a sub fetcher
@DgsData(parentType = "ldapServerConfigurations", field = "connectionStatus")
Boolean ldapConnectionStatus(DgsDataFetchingEnvironment dfe) {
  LdapConfigurationWithStatusView server = dfe.getSource();
  return ldapService.getStatus(server);
}

While this should theoretically work, e.g. when we use a DTO instead of the the LdapConfigurationWithStatusView view, DGS will properly replace the value via the data-fetcher.
What we try to improve
If we cannot use data fetchers, we will need to work arround it using

creating a DTO for the view
test the DFE configurations for the existince of the 'connectionStatus' field selection

// this is where we try to replace the value via a sub fetcher
  @DgsQuery(field = "ldapServerConfigurations")
  public Object getServerConfigurations(DataFetchingEnvironment dfe)
  {
     final var setting =
      graphQLEntityViewSupport.createPaginatedSetting(dfe);

    List<LdapConfigurationWithStatusView> serverConfigurations =
      ldapRepository.getServerConfigsPaged(setting);
    if (setting.getMaxResults() == 0) {
      return null;
    }

    // since we cannot change fields on the viewImpl (only getters) we need to convert this into a
    // DTO to be able to set our computed status field before we return the response
    var serverListDTO = serverConfigurations.stream().map(x -> {
      var mapped = new LdapServerConfigurationListResponse();
      modelMapper.map(x, mapped);
      return mapped;
    }).toList();

    if (dfe.getSelectionSet().contains("edges/node/connectionStatus")) {
      // if the field is set, we calculate its value
      // this would definitely be the best case for a sub-data-fetcher but at the moment
      // sub-data-fetcher seem not to work, when the model is not defined in the graphql schema file, or it does
      // not work with views (we strongly think it's the second one).
      serverListDTO.forEach(config -> config.setStatus(ldapService.getConnectionStatus(config)));
    }

    return new GraphQLRelayConnection<>(serverListDTO);
  }

After discussing with @beikov in Zulip he pointed me to an existing implementation

https://github.com/Blazebit/blaze-persistence/blob/7956aa45921fa4bb9868163939d3af1d88560e2d/examples/spring-data-dgs/src/main/java/com/blazebit/persistence/examples/spring/data/dgs/fetcher/CatGraphQLApi.java#L73
https://github.com/Blazebit/blaze-persistence/blob/7956aa45921fa4bb9868163939d3af1d88560e2d/examples/spring-data-dgs/src/main/java/com/blazebit/persistence/examples/spring/data/dgs/view/CatWithOwnerView.java#L32

This aims at the exact same thing and seems to work. Possible reasons we have a difference could be

[ ] error in our implementation
[ ] a specific name is set via GraphQLName which we do not use (maybe makes a difference in the way gql finds the fields) / schema generation issue
[ ] we use a list / relay, the implementation here just returns a single entry

I just tested the example with 5.2.1 and sub data fetchers are invoked properly, so I assume there is an error in your implementation. You can reopen this issue with a reproducer if you find a bug.


Trying to write page with quotes sometimes crashes Visual Studio

If you try and write this out character by character: @page "foo" you sometimes wont get to the end of the line, VS will crash. Reason why this is inconsistent is that Roslyn will attempt to auto-complete your quotes so if our Razor parse gets canceled and then re-queued with "" being in the document thing work; otherwise, if a parse goes through with @page " VS crashes because of this guy.
@alexgav can we add an end-to-end VS test for this 👍.

c25aadf599b4de61879ffc7f40cded23a36bc9e7


Add test by std

如果是文字类 PR，请按照 中文排版指北 进行编写(提交时可删除该提示)。
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe:
Add test by std

The description of the PR:
Other information:

No need for test code.


Console.log no longer working in netlify-cli/2.35.0

Console.log used to work in previous versions. After updating to 2.35.0, console.log statements no longer show.

Hey @michalhudecek, can you give an example of this! Do you mean the console.logs in Netlify Cli codebase or your site generator or Netlify Functions?

Sorry, I mean in Netlify functions. When I run netlify dev  I used to get console.log output from netlify functions when they were called. Now I do not get anything. I tried various functions I have it is the same everywhere.

Here is a screenshot of the console. The function is successfully called but no console.logs show anymore.

Having the same issue here. Console logs from within exports.handler are not showing up in the terminal. I'm using netlify-cli/2.35.0 as well.

If it helps, I am on Windows 10, node.js v12.16.0

Thanks for reporting this. We have a PR in the works for this one. Please hold on

If you can, please try out the changes from the PR (https://github.com/netlify/cli/pull/722) and provide feedback on if it is working as intended.

👋  This appears to have regressed again. I'm on cli version 2.57.0 and can't see console logs called from my handler function.


fix(#571): fix errors in readme and .env.example, issue #571

Pull request type
Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
There are inconsistencies in the README and the .env.example file as described in #571.
Resolves #571 .
What is the new behavior?
It resolves all issues of #571 .
Note
The updated keypair in the documentation is a valid keypair automatically generated from starknet-devnet.

Payment request #4CB3B8 has been processed and payment is complete. See you around on OnlyDust.


Fix claude login

Fix not able to get org after login.
Found that org is now store in cookie instead of local storage.
Add org in setting, in case new changes and user can update

Thank you for fixing. I have only one question: why add back the brandId setting?

Thank you for fixing. I have only one question: why add back the brandId setting?

hi @sunner, in case the claude update again then we can update org in setting

This setting will confuse most users. Moreover, the param is not changed frequently. Could you remove the setting?

This setting will confuse most users. Moreover, the param is not changed frequently. Could you remove the setting?

@sunner done


Please add support for the onion URL of invidious.

Checklist

[x] I'm reporting a new site support request
[x] I've verified that I'm running youtube-dl version 2019.12.25
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights
[x] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Since invidious is a middle man for youtube, everything but the URL is the same as youtube. That URL is "kgg2m7yk5aybusll.onion"
Description

I would like the onion URL for invidious ( kgg2m7yk5aybusll.onion ) to be supported.

As an alternative, you can simply pass the video ID and use tor socks proxy as follows:
youtube-dl --proxy socks5://<IP_ADDRESS>:9050 -v AH_0a5EoVKc
I actually use an alias as follows and run Tor on port 9350 instead of the default port:
alias ytp='~/bin/youtube-dl --proxy socks5://<IP_ADDRESS>:9350 -v'
$ ytp AH_0a5EoVKc
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--proxy', u'socks5://<IP_ADDRESS>:9350', u'-v', u'AH_0a5EoVKc']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.12.25
[debug] Python version 2.7.13 (CPython) - Linux-4.19.0-5-amd64-x86_64-with-MX-18.3-Continuum
[debug] exe versions: ffmpeg 3.3.4-1mx17, ffprobe 3.3.4-1mx17, phantomjs 2.1.1
[debug] Proxy map: {u'http': u'socks5://<IP_ADDRESS>:9350', u'https': u'socks5://<IP_ADDRESS>:9350'}
[youtube] AH_0a5EoVKc: Downloading webpage
[youtube] AH_0a5EoVKc: Downloading video info webpage
[debug] Default format spec: bestvideo+bestaudio/best
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[debug] Invoking downloader on u'https://r2---sn-5hne6nsy.googlevideo.com/videoplayback?expire=1577299232&ei=wFgDXv6-E8uz-gaQ8aagDA&ip=<IP_ADDRESS>&id=o-AHLe9F5JeNASGwS7KFCocXIX-BUn94yAqEeLWR6WCZAf&itag=299&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278%2C298%2C299%2C302%2C303&source=youtube&requiressl=yes&mm=31%2C29&mn=sn-5hne6nsy%2Csn-5hnekn7k&ms=au%2Crdu&mv=m&mvi=1&pl=22&initcwndbps=1101250&mime=video%2Fmp4&gir=yes&clen=10663395&dur=91.099&lmt=1577246884778208&mt=1577277518&fvip=2&keepalive=yes&fexp=23842630%2C23860862&c=WEB&txp=6316222&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=ALgxI2wwRQIgXRJgSTR2qXyyknLabGj2rbhUZ736CPNITNdep_RLas4CIQD9Glqu33wGlNwGsdrJ46en7kr4blq2rduR1w8FRzKkkg%3D%3D&lsparams=mm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AHylml4wRQIgX0a6qiiuZy6gIkbueI3LMEPxZ1oosDDxnN3rNEwM2QoCIQClBOgccBpSgXNAGBKXjfMP1NH3G2PE3DTHqzGd-XiA_g%3D%3D&ratebypass=yes'
[download] Destination: Merry Christmas 🎄-AH_0a5EoVKc.f299.mp4
[download] 100% of 10.17MiB in 00:29
[debug] Invoking downloader on u'https://r2---sn-5hne6nsy.googlevideo.com/videoplayback?expire=1577299232&ei=wFgDXv6-E8uz-gaQ8aagDA&ip=<IP_ADDRESS>&id=o-AHLe9F5JeNASGwS7KFCocXIX-BUn94yAqEeLWR6WCZAf&itag=251&source=youtube&requiressl=yes&mm=31%2C29&mn=sn-5hne6nsy%2Csn-5hnekn7k&ms=au%2Crdu&mv=m&mvi=1&pl=22&initcwndbps=1101250&mime=audio%2Fwebm&gir=yes&clen=1501798&dur=91.601&lmt=1577246873770461&mt=1577277518&fvip=2&keepalive=yes&fexp=23842630%2C23860862&c=WEB&txp=6311222&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=ALgxI2wwRQIgdXqRHd59f3TgRWwq2IOnlifLHIW7jlet4fKZ92lNkdsCIQDlVYLeUJOgWAfqXqM68KP1V-rnEftTsbHmuQjZ41CYdA%3D%3D&lsparams=mm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AHylml4wRQIgX0a6qiiuZy6gIkbueI3LMEPxZ1oosDDxnN3rNEwM2QoCIQClBOgccBpSgXNAGBKXjfMP1NH3G2PE3DTHqzGd-XiA_g%3D%3D&ratebypass=yes'
[download] Destination: Merry Christmas 🎄-AH_0a5EoVKc.f251.webm
[download] 100% of 1.43MiB in 00:05
[ffmpeg] Merging formats into "Merry Christmas 🎄-AH_0a5EoVKc.mkv"
[debug] ffmpeg command line: ffmpeg -y -loglevel 'repeat+info' -i 'file:Merry Christmas 🎄-AH_0a5EoVKc.f299.mp4' -i 'file:Merry Christmas 🎄-AH_0a5EoVKc.f251.webm' -c copy -map '0:v:0' -map '1:a:0' 'file:Merry Christmas 🎄-AH_0a5EoVKc.temp.mkv'
Deleting original file Merry Christmas 🎄-AH_0a5EoVKc.f299.mp4 (pass -k to keep)
Deleting original file Merry Christmas 🎄-AH_0a5EoVKc.f251.webm (pass -k to keep)

hai. I made this hack. It takes links from an invidious playlist page and converts them to youtube links.
Very simple hack indeed. And it works.
# chmod +x and add it to $PATH

# this script simply takes the video links present on an invidio.us playlist
# page and replaces the url to its youtube equivalent (check last line of this file)

# $1 is the positional argument :: the playlist link on invidio.us
# requires lynx >= 2.8.9

lynx -dump -hiddenlinks=listonly $1 | grep "&list=" \
| sed -E 's/([[:space:]][[:digit:]]+\.[[:space:]])|(&list.*)//g' \
| sed 's/invidio.us/youtube.com/g'

then pipe it to youtube-dl like
invidious-collect-yt-links https://invidio.us/playlist?list=IVPLvn5AlvJIfhiDrTtA0qwywjMO05iCCBM \
| youtube-dl -a -


additional packages for tikz

It seems to me that additional packages for tikz are not taken into account.
sample.md contains :
   {.tikz caption="This is an image, created by **TikZ i.e. LaTeX**."   additionalPackages="\usepackage{adjustbox}"}

but in diagram.lua  :
local pkgs = opt['additional-packages'] or ''

variable pkgs remains empty (opt['additional-packages'] is nil). The package is not added, and compilation fails
(missing \trimbox).
If I add \usepackage{adjustbox} directly in local tikz_template variable, everything works fine.
I don't understand lua enough to make a PR... :)

That line should probably be local pkgs = opt['additional-packages'] or opt['additionalPackages'] or ''; the camelCase version was just not handled. One can also use additional-packages="..." for the time being.

You're right, I forgot that I changed this to opt-additional-packages. The docs have not been updated yet.

Got it... !
You sample.md file contains :
    ```{.tikz caption="This is an image, created by **TikZ i.e. LaTeX**."
        additional-packages="\usepackage{adjustbox}"}
    ...

But it should be :
    ```{.tikz caption="This is an image, created by **TikZ i.e. LaTeX**."
        opt-additional-packages="\usepackage{adjustbox}"}
    ...

The real reference files are those named test/input-*.md now. I'll remove sample.md.

Not sure that test/input-*.md  are good for the final user.  The first thing I did when I found your filter was trying to compile a sample file  to check that the filter matched my needs. (It matched). I close the issue. Thank you for your work and you blazingly fast answers


ACCEPT_EULA command line option

Section "Using command-line options during installation" should mention that a silent installation also requires the command line option ACCEPT_EULA=1 . Installation fails without this.

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

ID: 322970ff-d62a-adeb-c8b3-d1141d3a272f
Version Independent ID: 94d82157-a824-73c8-2add-462ee6b033b1
Content: Get Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-get-the-desktop.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @dave-stagg. I've assigned this issue to the doc author for review.

Thanks for your suggestion @dave-stagg - we've added that information to the article.


✨ Implement heartbeats

Codecov Report
Attention: Patch coverage is 25.30120% with 62 lines in your changes missing coverage. Please review.

Project coverage is 30.29%. Comparing base (225d61b) to head (7f0635c).

Files with missing lines
Patch %
Lines

src/heartbeat.rs
42.22%
21 Missing and 5 partials :warning:

src/marktplaats/bot.rs
0.00%
12 Missing :warning:

src/main.rs
0.00%
10 Missing :warning:

src/telegram.rs
0.00%
9 Missing :warning:

src/client.rs
28.57%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
+ Coverage   28.44%   30.29%   +1.85%
==========================================
  Files          22       23       +1
  Lines         879      934      +55
  Branches      879      934      +55
==========================================
+ Hits          250      283      +33
- Misses        573      590      +17
- Partials       56       61       +5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Three services are sometimes too heavy

For scenarios where there are not many services in the cluster, it is too heavy to deploy three services as function switches. It is recommended to provide a solution to simplify the deployment of the number of services.

@Zhaoyu620 How about we deploy server and api in the same instance?

@Zhaoyu620 How about we deploy server and api in the same instance?

that is great

@SSebo That is great

We provide an All-in-One deployment method, please refer to: All-In-One Deploy Guide


Add encode implementations for SocketAddr and IpAddr

Something I encounter quite often is having metrics based on connection information, so I frequently find myself wanting to use either a SocketAddr or IpAddr in a label.

Friendly ping @rnijveld.

@mxinden Sorry about the slow response, life got in the way a little. I definitely agree with not wanting to use this for high cardinality situations. Although of course such cases could happen just as easily with other label types, I do agree that IP addresses might more quickly result in a cardinality explosion when not used correctly. I've added a little warning text to each of the implementations, I hope that resolves your concerns, let me know if there's anything else I can do!

It is effectively never appropriate to use an address as a label, for reasons of cardinality already discussed.

I don't think "never" is correct. One use-case I can come up with is exposing ones listening address as an info metric. See e.g. https://github.com/kubernetes/kube-state-metrics/issues/498 as an example on how far one can push cardinality in Prometheus with info metrics.


Unknown command: 'sync_waliki'

I can't get waliki to run.
Everything goes ok up until the line:
$ python manage.py sync_waliki

at which point I get an error:
Unknown command: 'sync_waliki'
Type 'manage.py help' for usage.

If I run python manage.py runserver, I get an error:
Performing system checks...

System check identified no issues (0 silenced).
June 22, 2015 - 21:16:40
Django version 1.8.2, using settings 'test_waliki.settings'
Starting development server at http://<IP_ADDRESS>:8000/
Quit the server with CONTROL-C.
Traceback (most recent call last):
  File "/home/username/anaconda/lib/python2.7/wsgiref/handlers.py", line 85, in run
    self.result = application(self.environ, self.start_response)
  File "/home/username/anaconda/lib/python2.7/site-packages/django/contrib/staticfiles/handlers.py", line 63, in __call__
    return self.application(environ, start_response)
  File "/home/username/anaconda/lib/python2.7/site-packages/django/core/handlers/wsgi.py", line 170, in __call__
    self.load_middleware()
  File "/home/username/anaconda/lib/python2.7/site-packages/django/core/handlers/base.py", line 50, in load_middleware
    mw_class = import_string(middleware_path)
  File "/home/username/anaconda/lib/python2.7/site-packages/django/utils/module_loading.py", line 24, in import_string
    six.reraise(ImportError, ImportError(msg), sys.exc_info()[2])
  File "/home/username/anaconda/lib/python2.7/site-packages/django/utils/module_loading.py", line 21, in import_string
    module_path, class_name = dotted_path.rsplit('.', 1)
ImportError: waliki doesn't look like a module path
[22/Jun/2015 21:16:46]"GET / HTTP/1.1" 500 59

And the website says:
A server error occurred.  Please contact the administrator.

I get the same errors running Python 2.7 and 3.4.

hey @ostrokach . could your share your complete settings.py file?
do you want just a vanilla waliki site or you are integrating it in a pre-existent django project? it the case is the former, may be you can check (or use) the demo project
https://github.com/mgaitan/waliki/tree/master/waliki_project

I was trying to test waliki out using a vanilla django site. Basically, I just ran
$ django-admin startproject mysite

and followed the instructions in the waliki README. My settings.py file is pasted below, but it's just the default django file with waliki imports. Using the waliki demo project works as expected (after installing django_allauth and django_analytical), so I'm just playing around with it for now. Thanks :).
#settings.py
"""
Django settings for test_waliki project.

Generated by 'django-admin startproject' using Django 1.8.2.

For more information on this file, see
https://docs.djangoproject.com/en/1.8/topics/settings/

For the full list of settings and their values, see
https://docs.djangoproject.com/en/1.8/ref/settings/
"""

# Build paths inside the project like this: os.path.join(BASE_DIR, ...)
import os

BASE_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))

# Quick-start development settings - unsuitable for production
# See https://docs.djangoproject.com/en/1.8/howto/deployment/checklist/

# SECURITY WARNING: keep the secret key used in production secret!
SECRET_KEY = 'y6sn0u=j#41x1p9jcr4^ai7pos4_r1v%$veatqzhv(%!b93zxv'

# SECURITY WARNING: don't run with debug turned on in production!
DEBUG = True

ALLOWED_HOSTS = []

# Application definition

INSTALLED_APPS = (
    'django.contrib.admin',
    'django.contrib.auth',
    'django.contrib.contenttypes',
    'django.contrib.sessions',
    'django.contrib.messages',
    'django.contrib.staticfiles',
)

MIDDLEWARE_CLASSES = (
    'django.contrib.sessions.middleware.SessionMiddleware',
    'django.middleware.common.CommonMiddleware',
    'django.middleware.csrf.CsrfViewMiddleware',
    'django.contrib.auth.middleware.AuthenticationMiddleware',
    'django.contrib.auth.middleware.SessionAuthenticationMiddleware',
    'django.contrib.messages.middleware.MessageMiddleware',
    'django.middleware.clickjacking.XFrameOptionsMiddleware',
    'django.middleware.security.SecurityMiddleware',
    'waliki',
    'waliki.git',
    'waliki.attachments',
    'waliki.pdf',
    'waliki.slides',
    'waliki.togetherjs',
)

# WALIKI_DATA_DIR = 'waliki_data'

# WALIKI_ANONYMOUS_USER_PERMISSIONS = ('view_page', 'add_page', 'change_page',)

ROOT_URLCONF = 'test_waliki.urls'

TEMPLATES = [
    {
        'BACKEND': 'django.template.backends.django.DjangoTemplates',
        'DIRS': [],
        'APP_DIRS': True,
        'OPTIONS': {
            'context_processors': [
                'django.template.context_processors.debug',
                'django.template.context_processors.request',
                'django.contrib.auth.context_processors.auth',
                'django.contrib.messages.context_processors.messages',
            ],
        },
    },
]

WSGI_APPLICATION = 'test_waliki.wsgi.application'

# Database
# https://docs.djangoproject.com/en/1.8/ref/settings/#databases

DATABASES = {
    'default': {
        'ENGINE': 'django.db.backends.sqlite3',
        'NAME': os.path.join(BASE_DIR, 'db.sqlite3'),
    }
}

# Internationalization
# https://docs.djangoproject.com/en/1.8/topics/i18n/

LANGUAGE_CODE = 'en-us'

TIME_ZONE = 'UTC'

USE_I18N = True

USE_L10N = True

USE_TZ = True

# Static files (CSS, JavaScript, Images)
# https://docs.djangoproject.com/en/1.8/howto/static-files/

STATIC_URL = '/static/'

@ostrokach
TEMPLATE_CONTEXT_PROCESSORS = (
    "django.contrib.auth.context_processors.auth",
    "django.core.context_processors.debug",
    "django.core.context_processors.i18n",
    "django.core.context_processors.media",
    "django.core.context_processors.static",
    "django.core.context_processors.tz",
    "django.contrib.messages.context_processors.messages",
    "django.core.context_processors.request",
    "waliki.context_processors.settings"
)

@ostrokach sorry, I closed the issue accidentally with a non-sense message.  I would like to debug it, because having a good starting experience is a must for new users.
Let me know if to define those template context processors helps.

@mgaitan Really sorry for wasting your time, you can close the issue.
The reason I was getting the errors was because I put
'waliki',
'waliki.git',
'waliki.attachments',
'waliki.pdf',
'waliki.slides',
'waliki.togetherjs',

into MIDDLEWARE_CLASSES rather than INSTALLED_APPS (facepalm...).

ok, cool


How to list ALL

It looks like you must specify the folder depth, any way to list all sub directories?

@maka-io Check out this tree-node-cli package that may do just what you need.

If you want list all sub directories, just type a big number for the -l parameter, like tree -l 1024. I think it's not a good idea to list all sub directories by default when 'l' is omitted, because sometimes the list would be so long that the shell term can't print the whole list but a part of it. Or if you have any good idea you can leave a comment here~


Export Data Error from Database (django.db.utils.OperationalError: too many SQL variables)

Hi,
I found already #1941, which contains basically the same issue.
I've ~3.5k images in my database and I'm getting now the same error when trying to export data:
Traceback (most recent call last):
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/backends/sqlite3/base.py", line 423, in execute
    return Database.Cursor.execute(self, query, params)
sqlite3.OperationalError: too many SQL variables

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

Traceback (most recent call last):
  File "/opt/app-root/lib64/python3.9/site-packages/rest_framework/views.py", line 506, in dispatch
    response = handler(request, *args, **kwargs)
  File "/opt/app-root/lib64/python3.9/site-packages/django/utils/decorators.py", line 43, in _wrapper
    return bound_method(*args, **kwargs)
  File "/label-studio/label_studio/data_export/api.py", line 177, in get
    tasks += ExportDataSerializer(
  File "/opt/app-root/lib64/python3.9/site-packages/rest_framework/serializers.py", line 768, in data
    ret = super().data
  File "/opt/app-root/lib64/python3.9/site-packages/rest_framework/serializers.py", line 253, in data
    self._data = self.to_representation(self.instance)
  File "/opt/app-root/lib64/python3.9/site-packages/rest_framework/serializers.py", line 686, in to_representation
    return [
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/models/query.py", line 280, in __iter__
    self._fetch_all()
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/models/query.py", line 1324, in _fetch_all
    self._result_cache = list(self._iterable_class(self))
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/models/query.py", line 51, in __iter__
    results = compiler.execute_sql(chunked_fetch=self.chunked_fetch, chunk_size=self.chunk_size)
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/models/sql/compiler.py", line 1175, in execute_sql
    cursor.execute(sql, params)
  File "/opt/app-root/lib64/python3.9/site-packages/sentry_sdk/integrations/django/__init__.py", line 560, in execute
    return real_execute(self, sql, params)
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/backends/utils.py", line 66, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/backends/utils.py", line 75, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/utils.py", line 90, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "/opt/app-root/lib64/python3.9/site-packages/django/db/backends/sqlite3/base.py", line 423, in execute
    return Database.Cursor.execute(self, query, params)
django.db.utils.OperationalError: too many SQL variables

My versions:
{
  "release": "1.5.0dev",
  "label-studio-os-package": {
    "version": "1.5.0dev",
    "short_version": "1.5",
    "latest_version_from_pypi": "1.6.0",
    "latest_version_upload_time": "2022-09-27T17:09:34",
    "current_version_is_outdated": true
  },

  "label-studio-os-backend": {},

  "label-studio-frontend": {
    "message": "fix: DEV-3061: Image scaling goes lower than 1 causing small images (# ...",
    "commit": "10dd11839b96d07f8978270896dcd76464c06cc4",
    "branch": "master",
    "date": "2022-08-29T18:16:33Z"
  },

  "dm2": {
    "message": "fix: DEV-2519: Choices selected by default",
    "commit": "dc03b7337518a3129677570127b44456420d4c6c",
    "branch": "master",
    "date": "2022/08/10 15:51:21"
  },

  "label-studio-converter": {
    "version": "0.0.44"
  }
}

Now I've a few questions:

From the documentation (https://labelstud.io/guide/storedata.html#Labeling-performance):

The SQLite database works well for projects with tens of thousands of labeling tasks.

Am I doing something wrong, or why does this happen already for 3.5k images?

@makseq's workaround to export data (https://github.com/heartexlabs/label-studio/issues/1941#issuecomment-1121609239) works - why do I not get any error here? It's the same database in the end.
Is there a way to migrate the SQLite database to PostgreSQL?

Hi,

The SQLite database works well for projects with tens of thousands of labeling tasks.

Good point, I've rewritten this paragraph.

Am I doing something wrong, or why does this happen already for 3.5k images?

Most likely you are using data manager filters or another logical things that make SQL queries complex.

why do I not get any error here? It's the same database in the end.

Because the result SQL query becomes much easier.

Is there a way to migrate the SQLite database to PostgreSQL?

Unfortunately not yet. However, you can export data from the project and re-import it on LS with the PostgreSQL instance.

Are there guides how to export & re-import the data?
I found some fuzzy descriptions how to do this for the labels & data itself, but, for example, what about the users?

Good idea about a guide for this..
You need to recreate the same users (with the same emails) manually or via API / command line.

Below is a workaround I employed to address the issue:
I encountered a similar issue on a machine where I lack administrative privileges, preventing me from updating software. The error stemmed from my SQLite version, specifically 3.26.0, which faced constraints related to the SQLITE_MAX_VARIABLE_NUMBER limit.
I addressed this by adjusting the batch size to 500 here, successfully exporting all files.


Provide package config

It would be helpful if luv would ship a pkg-config file, which would allow to check for required versions etc.
See https://github.com/libuv/libuv/blob/master/libuv.pc.in (and https://github.com/libuv/libuv/blob/988091dada1c9cbe4d4b88b1844ac6f2ad99d0ef/CMakeLists.txt#L386) for how libuv does it.

yes that would be super useful for packing on systems like nixos with non standard paths.

ok, let me try

Please look at here https://github.com/luvit/luv/pull/355

run make BUILD_MODULE=0 install  get
Install the project...
-- Install configuration: ""
-- Installing: /usr/local/lib/libluv.a
-- Installing: /usr/local/lib/pkgconfig/libluv.pc
-- Up-to-date: /usr/local/include/luv/luv.h
-- Up-to-date: /usr/local/include/luv/util.h
-- Up-to-date: /usr/local/include/luv/lhandle.h
-- Up-to-date: /usr/local/include/luv/lreq.h


OpenSSL copyright?

There are a number of files in oqsprov/ that have the OpenSSL copyright boilerplate, which gives copyright to the OpenSSL Authors, which I understand is historical, since this was previously a patchset for OpenSSL proper.
I don't think that's something you want at this point, and I can't see anything wrong with simply removing that boilerplate.  What you still have in there that could remind anyone of the built-in OpenSSL providers, it's mostly bread-and-butter lines that are only vaguely similar.

ACK. We'd probably take the (lib)OQS marking route. Looking at that again, though, // SPDX-License-Identifier: Apache-2.0 AND MIT is a "bit terse".... @dstebila Suggestions?

In liboqs for files we created, we did not put individual license information in each file beyond the SPDX-License-Identifier header, but there was a LICENSE.txt at the root which contained that information.  I think that would be okay here.

Resolved with https://github.com/open-quantum-safe/oqs-provider/commit/d6223cb1984caeaba02cf8ff89108dad2cb37046

You missed a spot :wink:
https://github.com/open-quantum-safe/oqs-provider/blob/d6223cb1984caeaba02cf8ff89108dad2cb37046/test/oqs_test_groups.c#L17-L24

You missed a spot

Thanks for the catch. And it was more than one. Lesson: No change too small to not botch it :-( Hopefully #5 gets this right.   Just the fact that others are already playing with external provider is exciting


Distributed training with network defined in protobuf files

Hi!
I've been experimenting with Caffe2.  I could convert original Caffe models that we have ([net.deploy.prototxt, net.deploy.caffemodel] -> [net.init.pb, net.deploy.pb]) and run them (train) in a single GPU/CPU mode in Caffe2. This was easy and smooth. Everything worked as expected. Guides and source code helped a lot! The question I have is if there is an easy way to train these translated networks in a distributed setup. The data_parallel_model.Parallelize_GPU helper accepts cnn.CNNModelHelper that does not seem to support serialization. Concretely, what would be the best way to go from *.pb files (model and definition) to a distributed training?
Thanks!

/cc @akyrola
Aapo, what do you think is the best way for us to do finetuning, especially with an existing model?
My gut feeling is that, one can manually feed the parameters to the workspace (with device scopes) and then skip the init net. Caffe and Caffe2 training mechanism are indeed sufficiently different that training from a predefined proto is a bit tricky.
Another thing I can think of is to have some script that generates a multi-gpu training plan from a single-gpu net.

If you have model building code in python, you should just do everything Caffe2 way, and then you load the parameters from a pretrained model to replace the ones that were populated by the param_init_net. Just call data_parallel_model.FinalizeAfterCheckpoint() so that the parameters are distributed to all hosts. Unfortunately I don't think we have any example of this in the open source tree... I plan to modify resnet50_trainer.py at some point to include checkpointing though.
But if you just work on a protobuffer models, we don't have a solution for you to parallelize the training. It would not be, at least in principle too difficult to take a converted protobuffer, recognize the gradient operators and parameter update operators and duplicate the model for all GPUs. But we don't have plans to support this. Happy to help someone implement this though.

@Yangqing  how i can run distributed model in caffe2. could you give me  help  of some tips? the tutorial do not give a clear solution. thank you so much


fix typo np.range

np.range(num_classes) changed to np.arange(num_classes).
Thanks for your Great Work 💯

thanks for the pr


cleanup kubelet.service file for disarm-kubulete

Why is this PR needed?
During Skuba disarms kubelete, kubelet.service file was left over.
What does this PR do?
This PR cleans /etc/systemd/system/kubelet.service for disarming kubelet.
Anything else a reviewer needs to know?
In the removed nodes from cluster, we should not see kubelet.service file.
Info for QA
This is info for QA so that they can validate this. This is mandatory if this PR fixes a bug.
If this is a new feature, a good description in "What does this PR do" may be enough.
Related info
Info that can be relevant for QA:

link to other PRs that should be merged together
link to packages that should be released together
upstream issues

Status BEFORE applying the patch
After skuba node remove worker-2, we see kubelet.service left from the worker-2.
Status AFTER applying the patch
How can we validate this issue is fixed? Please provide the steps and the prove this issue is fixed.
Docs
Do not need to change any Docs
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

Also, (not a systemd expert), but wouldn't removing files around need a daemon-reload ?

disarm-kubelet seems to mask the kubelet.service purposely.


SyntaxError: Unexpected end of JSON input

The Tidal API seems to return invalid JSON occasionally.
Unfortunately the logs don't contain the release lookup URL if only a single provider failed (error is currently handled in a place where we don't have access to that information), but here are some extracts of log lines which are likely correlated and might give a hint (although a Tidal lookup for that barcode works for me currently):
Jun 23 03:37:21 harmony deno[103430]: harmony.lookup [INFO] Beatport: Search returned no matching results
Jun 23 03:37:23 harmony deno[103430]: harmony.lookup [INFO] iTunes: API returned no results: https://itunes.apple.com/lookup?entity=song&limit=200&upc=5063381083869&country=jp
Jun 23 03:37:23 harmony deno[103430]: harmony.lookup [ERROR] SyntaxError: Unexpected end of JSON input
Jun 23 03:37:23 harmony deno[103430]:     at parse (<anonymous>)
Jun 23 03:37:23 harmony deno[103430]:     at packageData (ext:deno_fetch/22_body.js:370:14)
Jun 23 03:37:23 harmony deno[103430]:     at consumeBody (ext:deno_fetch/22_body.js:247:12)
Jun 23 03:37:23 harmony deno[103430]:     at eventLoopTick (ext:core/01_core.js:168:7)
Jun 23 03:37:23 harmony deno[103430]:     at async TidalProvider.query (file:///home/harmony/harmony/providers/Tidal/mod.ts:99:21)
Jun 23 03:37:23 harmony deno[103430]:     at async TidalReleaseLookup.getRawTracklist (file:///home/harmony/harmony/providers/Tidal/mod.ts:191:66)
Jun 23 03:37:23 harmony deno[103430]:     at async TidalReleaseLookup.convertRawRelease (file:///home/harmony/harmony/providers/Tidal/mod.ts:209:24)
Jun 23 03:37:23 harmony deno[103430]:     at async TidalReleaseLookup.getRelease (file:///home/harmony/harmony/providers/base.ts:290:19)
Jun 23 03:37:23 harmony deno[103430]:     at async Function.allSettled (<anonymous>)
Jun 23 03:37:23 harmony deno[103430]:     at async CombinedReleaseLookup.getProviderReleaseMapping (file:///home/harmony/harmony/lookup.ts:184:26)
Jun 23 03:37:23 harmony deno[103430]: harmony.lookup [INFO] Beatport: Search returned no matching results

If someone wants to work on this and needs more log samples, just let me know.

Looks like the JSON parsing error is hiding the real error...
That part should be fixed in https://github.com/kellnerd/harmony/commit/736d6eb912dc6b2cb7a07cf30619436b1bc6c24d, let's see which error message we get instead.


fix(cargo): Update minimal dependency versions so that 'cargo build -Z minimal-versions' works

The create does not compile with cargo build -Z minimal-versions because a lot of minimal versions are underspecified. I bumped them until I could build successfully.
Ideally we should also add something like cargo update -Z minimal-versions && cargo build to the Travis CI config so that this does not happen again in the future. I'm not sure where to put that exactly?

Thanks so much!
Sounds like a great idea to me to add this to CI eventually (and I think long-term we'd even do this on cargo publish). For the time being though I think it's ok to skip that as it's still a nightly Cargo feature and otherwise this lib isn't changing a whole lot


Change filename to meshca

@pitlv2109: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio
9c611099a38d3640add179aad8d6d9d12231f201
link
/test integ-mixer-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #17273


getting system properties

Is there any way of getting system properties from a database table (like the __updatedAt or __createdAt)?
If making a raw GET it is possible through querystring like so:
myservice.azure-mobile.net/tables/mytable?__systemproperties=__updatedAt

My bad, discovered that this has already been discussed. So I am closing issue now ..


2.1 projects published to IIS will silently fail HTTPS redirects and HSTS commands

Describe the bug
A change was introduced to remove the default HTTPS port of 443 in the middleware. When a default project is published to IIS, it cannot detect the HTTPS port and will not honor the redirect or HSTS settings in the Startup.cs thus making the site potentially insecure and only throw a warning, which does not appear in development since the default project template sets the https port for IIS Express.
This change was announced via https://github.com/aspnet/Announcements/issues/301
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core 2.1
Create new project using VS 2017 and check "Enable HTTPS"
Use Visual Studio Publish to file
Setup IIS site with bindings for port 80 and port 443 with certificate
Go to the http version of the site, no redirect.

Expected behavior
IIS and Dotnet core should be able to redirect automatically to port 443 when IIS is configured with SSL and the Startup.cs enabled redirection.
By default, the IIS Integration should be setting the HTTPS port in an environment variable, or the default new project should include a comment, similar to current comments in the project new template.
If HTTPS redirect and HSTS is enabled in the Startup.cs, an error should occur to prevent the site from launching in IIS saying the HTTPS port could not be found. Similar to when an appsettings.json file is marked required and not found.
Documentation should include these details on the following pages if no change to the behavior is made.
https://docs.microsoft.com/en-us/aspnet/core/migration/20_21?view=aspnetcore-2.1
https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/iis/?view=aspnetcore-2.1
https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/proxy-load-balancer?view=aspnetcore-2.1
Below documentation should change the Note about IIS to a warning, and be more visible.
https://docs.microsoft.com/en-us/aspnet/core/security/enforcing-ssl?view=aspnetcore-2.1&tabs=visual-studio
Nginx and Apache documentation includes server level configuration settings to automatically redirect http to https thus working around this issue.
cc @blowdart per Twitter

Duplicate: https://github.com/aspnet/IISIntegration/issues/485


Do not permit the use of a modified 'StaticR.rel' file

Looks good to merge, performance tested with Simpsons S08E23 "Homer's Enemy".


posix storage driver: cannot open files

DISCLAIMER: this is a ticket I opened as a private person and not as an employee of ownCloud
Describe the bug
When using the posix storage driver with oCIS 6.3.0, I can't open files
Steps to reproduce

start the docker-compose
create a text file
try to open it

Expected behavior
Being able to open and edit the file
Actual behavior
See a HTTP 404 status code
Setup
Following docker-compose:
---
services:
  ocis:
    image: owncloud/ocis-rolling:6.3.0
    entrypoint:
      - /bin/sh
    command: ["-c", "ocis init || true; ocis server"]
    environment:
      OCIS_URL: https://localhost:9200
      OCIS_LOG_LEVEL: info
      PROXY_TLS: "true"
      OCIS_INSECURE: "true"
      STORAGE_USERS_DRIVER: posix
    ports:
      - 9200:9200
    volumes:
      - ./data/ocis-config:/etc/ocis
      - ./data/ocis-data:/var/lib/ocis
    networks:
      - ocis
    logging:
      driver: "local"
    restart: always

@aduffeck @butonic  Did you see that?

The problem seems to be the missing id cache, The posix driver requires a shared cache so that storageprovider and dataprovider can always look up the paths for all ids.
Adding something like
      STORAGE_USERS_ID_CACHE_STORE: "nats-js-kv"
      STORAGE_USERS_ID_CACHE_STORE_NODES: "localhost:9233"

to your docker-compose.yml should fix the problem. And once https://github.com/cs3org/reva/pull/4779 has been merged the driver will refuse to run without a shared cache.


added inCode flag

added inCode flag to inputRule specs to allow inputRules to selectively apply to code blocks.

Just curious since this is my first pull request -- how long should I expect it to take to have it merged?  days, weeks, months?
Thanks and sorry for being so naive to this process.

Merged as 66d47eaa489ff4eab, tidied up in a993c59c111dc5973102f55. Release as part of prosemirror-inputrules 1.4.0


Fix bundled libSDL2_image.so by building it ourselves and manually editing SONAME and RPATH

I have a "quick fix", that makes toPngFile available on linux.
I have not yet automated the process, and don't yet know how to.
I built libSDL2_image.so myself, with support for only a few formats (although still using stb_image as backend).
By using patchelf I have modified the SONAME of and RPATH of both libSDL2.so and libSDL2_image.so so libSDL2_image.so is able to load libSDL2.so from the runtimes/linux-x64/native-directory.
On systems where SDL2 is installed, dotnet will to the best of my knowledge still invoke ld and find the installed library first.
In probably all cases, that library will have support for anything we do with SDL2_image, so we are "safe" except for edge-cases we do not want to handle anyway.

It would be great to at least document how you have build and changed the two .so files. So that we can build the macOS .dylib files with the same configuration and so that we can remember what to do the next time we want to update the SDL2 artefacts.
Perhaps you could start a BUILD.md file or something like that?

I agree.
I’ll fix up my configure-script st. It works on Linux and is extendable for
Mac.
On Fri, 9 Sep 2022 at 16.33, Ken Friis Larsen @.***>
wrote:

It would be great to at least document how you have build and changed the
two .so files. So that we can build the macOS .dylib files with the same
configuration and so that we can remember what to do the next time we want
to update the SDL2 artefacts.
Perhaps you could start a BUILD.md file or something like that?
—
Reply to this email directly, view it on GitHub
https://github.com/kfl/diku-canvas/pull/11#issuecomment-1242052055, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMEEGGGQN3MSZ6MNGH4VUDV5NDEVANCNFSM6AAAAAAQIUWMGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Closing this as we no longer need SDL2_image


Retry GCP requests on server error

Which issue does this PR close?

Part of #2244
Relates to #2176
Rationale for this change

The S3 implementation currently has request retry support, as we look to move away from rusoto we need to ensure we can preserve this functionality. This PR therefore adds the necessary functionality to the GCP implementation, which can then be reused for AWS and Azure once they switch away from using SDKs.
What changes are included in this PR?

Adds an implementation of exponential backoff, lifted wholesale from the implementation I wrote for rskafa.
Are there any user-facing changes?

Technically yes, GCP requests will now be automatically retried. We could change the default to avoid this but I think it is unlikely to cause issue

Codecov Report

Merging #2243 (e8718ff) into master (f41fb1c) will decrease coverage by 0.06%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #2243      +/-   ##
==========================================
- Coverage   82.30%   82.23%   -0.07%
==========================================
  Files         241      243       +2
  Lines       62437    62493      +56
==========================================
+ Hits        51389    51392       +3
- Misses      11048    11101      +53

Impacted Files
Coverage Δ

object_store/src/client/backoff.rs
0.00% <0.00%> (ø)

object_store/src/client/oauth.rs
0.00% <0.00%> (ø)

object_store/src/client/retry.rs
0.00% <0.00%> (ø)

object_store/src/client/token.rs
0.00% <ø> (ø)

object_store/src/gcp.rs
0.00% <0.00%> (ø)

object_store/src/lib.rs
86.75% <ø> (ø)

...row/src/array/builder/string_dictionary_builder.rs
90.64% <0.00%> (-0.72%)
:arrow_down:

parquet_derive/src/parquet_field.rs
65.75% <0.00%> (-0.23%)
:arrow_down:

arrow/src/datatypes/datatype.rs
63.00% <0.00%> (+0.31%)
:arrow_up:

...rquet/src/arrow/record_reader/definition_levels.rs
89.02% <0.00%> (+1.68%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.


HDDS-10541. Replace GSON with Jackson in debug commands

What changes were proposed in this pull request?
This update addresses compatibility concerns with Java 17 by transitioning from GSON to Jackson in the Debug-Shell module. This change simplifies dependencies and ensures smooth operation across different Java versions.
Parent Jira Issue: HDDS-10538
This patch streamlines our Debug-Shell Implementation.
Affected Classes:
- ChunkKeyHandler.java
- ReadReplicas.java
- ContainerChunkInfo.java

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10541
How was this patch tested?
To test out the changes we have to run :- TestOzoneDebugShell Integration test.
The forked CI also ran successfully.

@dombizita @tanvipenumudy please review

Thanks @ArafatKhan2198 for the patch, @tanvipenumudy for the review.
Tested this with Java 17, works well.


net: mqtt: Debug logging of pointers

Cast pointer to void * for %p parameter.
Otherwise lots of warnings in the log like below:
<wrn> cbprintf_package: (unsigned) char * used for %p argument.  It's recommended to cast it to void * because it may cause  misbehavior in certain configurations. String:"%s: (%p): >>  length:0x%08x cur:%p, end:%p" argument:3

I just wonder what has changed as those warning have not been seen earlier. Seems quite pointless as a pointer is a pointer regardless of the cast.

Don't know, but too many of them causing messages drop in logging over RTT.


Is it normal that nvm install 10 for the first time takes more than an hour?

I created a VPS with Ubuntu 14.04 LTS 32bit and after installing nvm via curl I have ran:
nvm install 10 and it's taking a lot longer than usual. These are the logs:

It's neverending logging similar things.
I remember this happening once on my personal computer a long time ago when I had ubuntu installed as well but it has never happened again until today.

It definitely isn’t normal.
It looks like it’s hanging on the compilation step - is there any chance you copied the .nvm dir from a different machine? If so, try nvm cache clear and try the install again?
If not, then I’m really not sure - presumably the same issue would occur with compiling node downloaded directly from the website.

After trying nvm cache clear and nvm install 10 I got the same result ( I think )
Here's are the logs:
https://gist.github.com/johhansantana/3f68b41b82811c291bb406bab0b4c3d9
I tried installing V9.x using nvm install 9 and it worked :l

It's possible that node v10 isn't compatible with your machine - is it 32 bit or 64 bit? What exact architecture are you on?

Also, what gcc version do you have?

It is 32bit.
Not sure what you mean by gcc

gcc --version - it's the C compiler. node no longer produces 32 bit binaries as of v10, so you'd have to be able to compile it from source. (32 bit machines are very very ancient)

So given that it's compiling, and that it's an old slow machine, and that v9 didn't have to compile, i think it might actually be normal for it to take multiple hours to install.

See https://github.com/nodejs/build/pull/1498 - looks like it's not reliable to be able to use node on 32 bit machines on v10+.

It's not an error. 32 bit binaries are not available. So nvm is compiling it from source. Run it again and wait. It works fine. It compiles and you have to restart. Then you have to type.
nvm use 10.0.0 # or the version you use

I know this post is really old, but i hope the author or another user could tell me if this issue was solved, because i have the same problem trying to install newer versions of node in my laptop, it just display that same message and i don't know if i should just leave it downloadiing or what else to do.
If the author is seeing this... did it worked? I mean, it installed node after a while? Thanks in advance.

I know this post is really old, but i hope the author or another user could tell me if this issue was solved, because i have the same problem trying to install newer versions of node in my laptop, it just display that same message and i don't know if i should just leave it downloadiing or what else to do.
If the author is seeing this... did it worked? I mean, it installed node after a while? Thanks in advance.

As the comments stated here I believe it worked (can't remember).

👋 If anyone revisits this: I just downgraded to 15.14.0 via NVM (owing to issues with 16.* on Silicon Macs), and experienced the OP's issue. The install took about 10 minutes, but was, ultimately, fine.

@mtripney ftr, node 16+ is the only one that supports M1 macs natively; everything else uses Rosetta emulation.

Thanks for noting this @mtripney - Took me about 10 minutes as well installing 12.22.1 on a new Silicon Mac. I was about to bail out but then saw your comment and waited :)

@blairbodnar Glad it helped!

@ljharb You're right, of course, but I’ve had no end of problems with v16, such that I eventually had to downgrade. (One critics example being: https://github.com/vercel/next.js/issues/24564)

👋 If anyone revisits this: I just downgraded to 15.14.0 via NVM (owing to issues with 16.* on Silicon Macs), and experienced the OP's issue. The install took about 10 minutes, but was, ultimately, fine.

can confirm this. just set up my M1 with v15 and it took quite a bit (enough for me to reach this thread) but was, ultimately, fine.

 Im still waiting for this endless log tail.

Can confirm, went to 12.14.1 on a M1 MacBook Pro. Took about 10 minutes, even spun up the fans for the first time ever. But it did finish and is seemingly fine.

Another confirmation here. M1 MacBook Air on Monterey, downgrading to Node 14 for a project I'm working on and after 10 or so minutes everything finished.
Don't close it out!

I can also confirm it worked for me on Macbook Air M1 chip Monterrey, after more than 10 minutes, however, it took about 10 GB of space in disk, is this normal?

@ricardocamacho That's a question to ask node itself; nvm doesn't have any interaction beyond automating the node compilation process.

It seems that I previously had nvm installed through Home-brew. I uninstalled it, created a rosetta-specific terminal and installed my node versions from there
Everything worked fine when I did that

Can confirm that this works on an M2 MacBook as well - just takes a little time, but ultimately the compilation succeeds and v14.18.1 was installed.

I tried installed version 16 and for me also, it took 20+ mins but in the end , it was done

took me almost an hour but it finally worked 😅 (Intel macbook pro touch)

it took about 25 min upgrading from node 16.18.1 to 18.18.0 on my M1 Pro, I also confirm everything went successful

Can also confirm it took a long time (10min+) on M2. This was installing version 20

I must say this thread is a gem, I was going to interrupt the installer before reading this
my experience - 14.17.0 took around 10min on M2, works completely fine, just grab some coffee and wait

Using this method to install NVM10 on my pi2 (which is possible it seems.)
Its still compiling after 6 hours!

I'm on 64-bit machine.
16GB ram, core i7 9750H, arch linux, kernel 6.6
when I try to install nvm install lts/iron it looks like it is compiling from source and takes very very long and still hasn't been installed.
I don't think this is normal. any ideas?

I'm facing the same problem. The nvm is supposed to offer the 'easy-to-use' way to install node/versions, however it's not able to install v.18.18.0 node version

@gpataciertocom this issue is about compilation slowness. Please file a new issue and fill out the entire issue template and I'll be happy to help.

@alirezaalavi87 have similar specs as you running ubuntu, trying to get node v20.18.0. Got into this compilation process as well.
I was mid writing an issue then I heard my fans go low. It was done!
Be patient and cheers when you see "Now using node vXX...."!


Entity's UUID is illegal in command data merge entity

the command
data merge entity 68b6b73a-f79c-49ee-a8ea-471911f553c6 {Item:{tag:{display:{Name:'{"translate":"pl.frame.ldp"}'}}}}
"68b6b73a-f79c-49ee-a8ea-471911f553c6" is underlined in red

Caused by the fix of #199.

@all-contributors
Add yulongjiuqiu for bug :bug:

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

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Hotfix/1.2.2

1.2.2 (2021-11-09)
Added
Fixed

ZIP code and city of the offer recipient are in the same line now

Dependencies
Deprecated

https://github.com/qbicsoftware/offer-manager-2-portlet/pull/855


fix: handle input binding response correctly for http protocol app

Description
The HTTP binding api says user's binding  http response body can be used to directly bind input bindings with state stores or output bindings, but it does not seem to be implemented. This pr will fix this.

Issue reference

Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

any update? I've updated the description.

@zcong1993 we have actually removed this functionality from Dapr on purpose a while back. If you see it in the docs, then it should be removed. We would very much appreciate a PR to the docs to remove this part.


Display Name for msdyn_ProjectServiceCore changed

as per the current information on production the display name for "msdyn_ProjectServiceCore" changed from "Project Service Automation Core" to "Project"

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@efrene PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Links

This PR comments out the Notebook gallery since those aren't real resources we want to display to the public and adds a links.md page containing information from the Content Catalog Google Sheet.

@kmpaul @clyne @brian-rose This PR is ready for review. It looks as though I do not have permission to request reviewers.

@kmpaul @clyne @brian-rose This PR is ready for review. It looks as though I do not have permission to request reviewers.

Looks like the home page is corrupted:
https://projectpythia--20.org.readthedocs.build/en/20/index.html

Looks like the home page is corrupted:
https://projectpythia--20.org.readthedocs.build/en/20/index.html

Looks like the home page is corrupted:
https://projectpythia--20.org.readthedocs.build/en/20/index.html

Good catch! I was so focused on checking the appearance of links.html.
I'll fix this now!

Looks like the home page is corrupted:
https://projectpythia--20.org.readthedocs.build/en/20/index.html

Good catch! I was so focused on checking the appearance of links.html.
I'll fix this now!

Thanks for the quick turnaround, All!

Thanks for the quick turnaround, All!


feat(github plugin): adds authentication via github app

⚠ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

[x] I have read through the Contributing Documentation.
[ ] I have added relevant tests.
[ ] I have added relevant documentation.
[x] I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary
This PR adds a new option to use authentication via GitHub App while creating a new GitHub connection. Then it alters several places in github and github_graphql plugins to fetch the appropriate token given an installation id, which it can fetch based on the repository name and owner.
Does this close any open issues?
Closes https://github.com/apache/incubator-devlake/issues/4801
Screenshots
GitHub app configuration section:

In the repository selection the user/org column will represent the orgs/users where the app is installed and the right side are repositories allowed by that installation:

Other Information
I'm doing this change on behalf of Telia Company (https://github.com/telia-company). I'm not sure if they sent in CCLA, but I'm trying to find out.
I haven't been around this codebase for very long (last two weeks) so any recommendations for changes, what tests to add, or hints on what to do next are very much appreciated.

Hi, @mkmarek , Thank you so much for the contribution.
Can you also provide the permissions required for the Github App so I can test it locally?
The current implementation uses if in many places for the MultiAuth to work which is not ideal IMO,
Ideally, the only file that we need to modify would be backend/plugins/github/models/connection.go and other dirs/files like backend/plugins/github/api and backend/plugins/github/impl should be left intact.
Would you mind taking a look at jira implementation? https://github.com/apache/incubator-devlake/blob/main/backend/plugins/jira/models/connection.go#L42 The idea is to delegate authentication via the MultiAuth` struct and isolate the authentication from other places.
To my understanding, the Github App Authentication works pretty much like the feishu plugin, I suggest we do it like this:

Alias the helper.AppKey as  GithubAppKey and implement the GithubAppKey.SetupAuthentication for each request and/or GithubConnection.PrepareApiClient if some kind of global initialization is required. (ps: an if is needed here or maybe we could delegate the logic to the  Authenticator level as well, it is an open subject that we can discuss)
Embed the helper.MultiAuth and GithubAppKey into GithubConn just like how jira did it.
Revert the changes to the api and the impl folders.

If I understand it correctly:

fetch the installationId by owner/repo
authentication app installation using the appId, secretKey and installationId which produce the token
uses the token for the rest requests

We can not utilize the PrepareApiClient because it works on the connection level and the owner/repo information is missing from the signature.
The problem is: Github App Installation Authentication works on the repo level, is that correct?

Yeah pretty much. It's possible to fetch the installation ID based on owner/repo. It's also possible to fetch all installations and then pick the correct one just based on owner. Or if you know if the owner is either org or user there are endpoints for fetching the token directly just based on owner.
I have made an alternative version of the code. I haven't cleaned it up properly yet. But I can push it now. It removes the if-elses in the impl and api parts and moves everything to connection.go using multiauth. What I got left with in the impl and api files is the minimum that is needed to provide for the connection to work.
I took a bit different approach where it initially fetches tokens for all the installations in the PrepareApiClient function and then uses the tokens in SetupAuthentication based on either a installationid or a repository. The installationid and the repository have to be provided from the outside.

@mkmarek Thanks for the clarification. It is much cleaner now.
I'm wondering if we can do better, the current design of the MultiAuth is flawed and fail to support such a scenario.
It would be nice if we could improve it and keep other files intact.
I would need some time to think about it, or we could merge your work first if it is urgent from your side.

One more thing, I noticed that Git App has Authentication on behalf of a user, will it be possible to do it that way? @mkmarek

I don't think it's that urgent from our side. Until this feature gets in, we can build our own docker images to test things out, and when it gets in, we can switch back to your releases. So there's plenty of time to implement this the right way.
About authenticating on behalf of user. I'm not sure that would fit the same use case. It's used more for CLIs or webpages where you can "login with GitHub". The flow is very different there. In our use case Github App replaces machine users we'd need to have.
With auth on behalf of user we'd need to have either both the app and the machine users or only the app and have real users in the organization go into devlake and perform auth there.. and in both of those cases the existing token solution would be probably better.

@mkmarek  So, installation is on the owner/organization level? Can I use it to select multiple repos under the same owner/org?

I see, LGTM.
@Startrekzky @hezyin What do you think

Hi @mkmarek , can you solve the conflicts and check if the commit messages follow the conventional commits?

@mkmarek We all believe that the limitation is acceptable, would you like to rebase your PR and we can get it to merge ASAP

@klesh @Startrekzky I did a rebase, fixed all conflicts, adjusted the commit message, and retested. Now it should be good to go.

Hi, @mkmarek , I'm trying to verify if it is working correctly but am stuck at the very first step 🥲 "Register new Github App", the options are overwhelming, I'm not sure how to set up correctly.
Would you like to write a document to guide us through the whole process? Thank you very much.
I think we can add a section to https://github.com/apache/incubator-devlake-website/blob/main/docs/Configuration/GitHub.md to describe how to setup the Github App and use it for data collection.


Missing PolyBezierSegment

I am porting a WPF app to Avalonia and found that I'm using in the app PolyBezierSegment class which unfortunately isn't available in Avalonia. There is just BezierSegment which isn't much usable because it has only 3 points whereas PolyBezierSegment can contain as many points as I want.
PolyBezierSegment Class

sounds like a useful extension to me. @grokys you were working on geometry improvements earlier. Do we want this feature in 11.x?
@JakubMarecek If we get a 👍, a PR is welcome :-)

Nevermind, I found there are another breaking bugs in Avalonia, so I rather stick with WPF.

I'm reopening this as the idea is still useful.
If you need xplat support and want to stick with WPF, we can also offer XPF. if you are interested and have some budget, reach out to the team.


QUIC transport template option added to IceRpc-Protobuf-Server and IceRpc-Protobuf-Client

Sister PR to #3955
Closes #3956
This PR adds the QUIC transport option to the Protobuf templates .

You should update CI to test templates with the new param.

Whoops forgot, will add now!


Support WAVE Syntax

https://github.com/lann/wasm-wave
Examples: https://github.com/lann/wasm-wave/blob/main/tests/wasmtime.rs
https://app.zenhub.com/workspaces/golem-cloud-65f5b68988aba80f7d690199/issues/gh/golemcloud/golem/347.
can be done as part of this ticket. Most probably this will be a PR that solves these two things together

@vigoo  I think we need to untag this ticket from milestone 1.0
Let's discuss this

Need a better understanding on the scope. I will talk to @jdegoes

Expecting 2 days

https://app.zenhub.com/workspaces/golem-cloud-65f5b68988aba80f7d690199/issues/gh/golemcloud/golem/347.
can be done as part of this ticket

Fixes in #356