SSL Not functioning

Hello, I'm trying to see if this solution is viable for us. I'm trying get a basic fake api working over https. But I'm not even getting into the API gateway.
My configuration is as follows (generated from the online config tool):
{
  "version": 2,
  "port": 8001,
  "extra_config": {
    "github_com/devopsfaith/krakend-httpsecure": {
      "allowed_hosts": [
        "api.site.com"
      ],
      "ssl_proxy_headers": {
        "X-Forwarded-Proto": "https"
      },
      "ssl_redirect": true,
      "ssl_certificate": "/opt/gateway/certs/site.crt",
      "ssl_private_key": "/opt/gateway/certs/site.key",
      "sts_include_subdomains": true,
      "ssl_host": "api.site.com"
    }
  },
  "host": [
    "https://api.site.com"
  ],
  "output_encoding": "json",
  "timeout": "3000ms",
  "cache_ttl": "300s",
  "name": "My API Gateway",
  "endpoints": [
    {
      "endpoint": "/testing",
      "method": "GET",
      "output_encoding": "json",
      "concurrent_calls": 1,
      "extra_config": {
        "github.com/devopsfaith/krakend/proxy": {
          "static": {
            "data": {
              "new_field_a": 123,
              "new_field_b": [
                "arr1",
                "arr2"
              ],
              "new_field_c": {
                "obj": "obj1"
              }
            },
            "strategy": "always"
          }
        }
      },
      "backend": [
        {
          "url_pattern": "/",
          "encoding": "json",
          "extra_config": {},
          "sd": "static"
        }
      ]
    }
  ]
}

My Output is as follows when running krakend in debug mode
root@vastyk:/etc/krakend# krakend run -c krakend.json -d
Parsing configuration file: krakend.json
 ERROR: unable to create the gologgin logger: getting the extra config for the krakend-gologging module
 WARNING: building the etcd client: unable to create the etcd client: no config
 DEBUG: creating a new influxdb client
 DEBUG: no config fot the influxdb client. aborting
 ERROR: Unable to load custom config from the extra config
 ERROR: opencensus: no extra config defined for the opencensus module
 INFO: no config for the bloomfilter map[github_com/devopsfaith/krakend-httpsecure:map[allowed_hosts:[] ssl_redirect:true ssl_certificate:/opt/gateway/certs/site.crt ssl_private_key:/opt/gateway/certs/site.key ssl_host:api.site.com]]
 ERROR: registering the BloomFilter: no config for the bloomfilter
[GIN-debug] [WARNING] Running in "debug" mode. Switch to "release" mode in production.
 - using env:   export GIN_MODE=release
 - using code:  gin.SetMode(gin.ReleaseMode)

 DEBUG: Debug enabled
[GIN-debug] GET    /__debug/*param           --> github.com/devopsfaith/krakend-ce/vendor/github.com/devopsfaith/krakend/router/gin.DebugHandler.func1 (4 handlers)
[GIN-debug] POST   /__debug/*param           --> github.com/devopsfaith/krakend-ce/vendor/github.com/devopsfaith/krakend/router/gin.DebugHandler.func1 (4 handlers)
[GIN-debug] PUT    /__debug/*param           --> github.com/devopsfaith/krakend-ce/vendor/github.com/devopsfaith/krakend/router/gin.DebugHandler.func1 (4 handlers)
 ERROR: JOSE: no signer config /testing
[GIN-debug] GET    /testing                  --> github.com/devopsfaith/krakend-ce/vendor/github.com/devopsfaith/krakend/router/gin.CustomErrorEndpointHandler.func1 (4 handlers)
[GIN] 2018/09/11 - 18:41:36 | 200 |  116.989573ms |    <IP_ADDRESS> |  GET     /testing

In Postman if I send a request to https://api.site.com:8001/testing I get "Could not get any response" immediately. In the running display of KrakenD I have no connection attempt.
In Postman if I send a request to http://api.site.com:8001/testing I get the fake api response I expected. and I see the debug print out as you can see in my output snippet above. Why is this not accepting SSL connections? I noticed in my startup there  is one ERROR that is listed as "Unable to load custom config from the extra config". Not sure if that is of use?

Hello @kmccmk9 ,
Before going any further I would like to make sure we have the basics in place. If you have as allowed_hosts the value  "api.site.com" KrakenD will make sure the request is coming from a host identified with exactly this name.
As you said you were using Postman, my first thought is that you are not forging the request header so Postman will be always identifying itself as "localhost" or similar. The easiest test you can do is to do a curl forging the header:
curl -H 'Host:api.site.com'  https://api.site.com:8001/testing

Of course you can remove the allowed hosts from the configuration to do the test as well.
As a final note, if you intend to use krakend as the "api.site.com" the allowed hosts should have your clients hostnames, not krakend's. For instance client.site.com. Unless of course you have everything in the same machine.
Please let me know if this works

Ahhh... I think i misunderstood you the whole time! I was assuming something you didn't say...
The httpsecure package is a wrapper of the unrolled/secure package. It adds additional security when dealing with https requests, such as forcing the user to use https instead of http, prevent XSS, disallow connections from strange hosts, etc... but you need ALWAYS a SSL terminator in front of KrakenD-CE as it will listen ONLY in plain HTTP.
So, if you are connecting directly to KrakenD using HTTPS this won't work! You need the balancer or another piece acting as a terminator. For instance, in AWS we usually place an ELB in front of krakend that deals with the SSL, and then we use the middlware httpsecure for the rest of the https magic.
Good news is that KrakenD will support this by itself in the next release (before EOY) as we will add ListenAndServeTLS
Hope that now I got it right! 😊

Ok that makes much more sense. Sounds like for our particular use case we will wait for an update that includes ListenAndServe rather than running another instance of a HA Proxy, or similar configuration. Thank you for detailing that out. Small suggestion, that on the website and documentation, explain more about "Support SSL" out of the box. I find that very ambiguous as demonstrated from this GitHub issue haha. Thank you again, can't wait for the new release and giving it another evaluation.

Note taken. The whole site needs a revamp, specially the documentation (we need volunteers!)
I will let you know in this issue if the ListenAndServe can be tested before the release.

Hi @kmccmk9,
Version 0.6.1 is released and now you can add listening TLS.
Thanks!


[Pocket] ...

Extension
https://www.raycast.com/vimtor/pocket
Description
Running this code does not work. Error 403 is received upon submitting the last snippet to get the access token.
Steps To Reproduce

OS X Ventura
Terminal
Vivaldi or Safari
Terminal

Current Behaviour
Fails to authorize.
Expected Behaviour
Authorizes and generates access token.

I can confirm the same issue.

It seems the issue comes down to the format of the strings generated. No matter how you try to post it says it is wrong
It will submit but not send any access key back, then submitting it again I get error 403 which seems to indicate one of the original 2 strings is invalid, perhaps because it was already used once.
Maybe I need to wait longer.

I can confirm the same issue.

What Terminal are you using? I also tried bash and had the same result.

I can confirm the same issue.

What Terminal are you using? I also tried bash and had the same result.

Solved, the issue is as suspected, the Mac Terminal. I used a free tier of GCS to run the commands and it works perfectly fine. Not super happy with Ventura or ZSH shell especially. Run it from a different terminal or cloud shell and you will be fine, might even work downloading a mobile terminal app.

Solved, the issue is as suspected, the Mac Terminal. I used a free tier of GCS to run the commands and it works perfectly fine. Not super happy with Ventura or ZSH shell especially. Run these same commands from a different terminal or cloud shell and you will be fine, it might even work downloading a mobile terminal app.
Additionally you might try adjusting settings on Apple Terminal. If I have more time I can try the same later, and put a followup linked to this, but the issue with the extension itself is not really an issue.
@vimtor Closing this ticket, but maybe you have some feedback on what might be causing this issue in the built in Mac Terminal.

I was using iTerm / zsh. I'll try a different one, thanks!
On Sun, Jul 9, 2023, 9:57 AM TransRapid @.***> wrote:

I can confirm the same issue.
What Terminal are you using? I also tried bash and had the same result.
—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/issues/7441#issuecomment-1627772686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7NWB6T4E6AIYKSDSPWA5VTXPLPJLANCNFSM6AAAAAA2DLSZWI
.
You are receiving this because you commented.Message ID:
@.***>

To be honest, I am a bit frustrated with Pocket. It started working, but then it suddenly stopped. They have a custom OAuth flow, so I can't use the built-in Raycast method. I'm a bit out of ideas for the extension :(

To be honest, I am a bit frustrated with Pocket. It started working, but then it suddenly stopped. They have a custom OAuth flow, so I can't use the built-in Raycast method. I'm a bit out of ideas for the extension 😢

Do you mean as far as an inline extension without having to use any external program? The extension works, but something weird happens when using zsh vs bash. It seems that one of the initial inputs is misinterpreted by Pocket API-and then produces a bad output and doesn't even know it's bad until you put in the access key generated from that and it doesn't add up properly.

I had a lot of improvements ready to be shipped, but since this wasn't working they didn't see the light.
https://github.com/raycast/extensions/pull/2991
Any ideas @pernielsentikaer?

To be honest, I am a bit frustrated with Pocket. It started working, but then it suddenly stopped. They have a custom OAuth flow, so I can't use the built-in Raycast method. I'm a bit out of ideas for the extension 😢

Do you mean as far as an inline extension without having to use any external program? The extension works, but something weird happens when using zsh or bash on Ventura specifically. I imagine something more related to some Terminal settings. It seems that one of the initial inputs is misinterpreted by Pocket API-and then produces a bad output and doesn't even know it's bad until you put in the access key generated from that and it doesn't add up properly.
Working extension is super useful btw.

I had a lot of improvements ready to be shipped, but since this wasn't working they didn't see the light.
https://github.com/raycast/extensions/pull/2991
Any ideas @pernielsentikaer?

@TransRapid @vimtor I faced the same the same issue and managed to complete the auth flow by following Pocket documentation:
curl -X POST https://getpocket.com/v3/oauth/request \
-H "Content-Type: application/x-www-form-urlencoded; charset=UTF-8" \
-H "X-Accept: application/x-www-form-urlencoded" \
-d "consumer_key=YOUR_APP_CONSUMER_KEY&redirect_uri=pocketapp1234:authorizationFinished"

curl -X POST https://getpocket.com/v3/oauth/authorize \
-H "Content-Type: application/x-www-form-urlencoded; charset=UTF-8" \
-H "X-Accept: application/x-www-form-urlencoded" \
-d "consumer_key=YOUR_APP_CONSUMER_KEY&code= YOUR_REQUEST_TOKEN"

If this works for you guys the documentation should be changed.


[CHANGE ME] Re-generated  to pick up changes in the API or client library generator.

This PR was generated using Autosynth. :rainbow:
Here's the log from Synthtool:
synthtool > Cloning googleapis.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:90f9d15e9bad675aeecd586725bce48f5667ffe7d5fc4d1e96d51ff34304815b
Status: Image is up to date for googleapis/artman:latest
synthtool > Running generator for google/cloud/oslogin/artman_oslogin_v1beta.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/os-login-v1beta.
.circleci/config.yml
.circleci/npm-install-retry.js
.eslintignore
.eslintrc.yml
.github/CONTRIBUTING.md
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.jsdoc.js
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node11/common.cfg
.kokoro/continuous/node11/test.cfg
.kokoro/continuous/node6/common.cfg
.kokoro/continuous/node6/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/docs.cfg
.kokoro/continuous/node8/lint.cfg
.kokoro/continuous/node8/samples-test.cfg
.kokoro/continuous/node8/system-test.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node11/common.cfg
.kokoro/presubmit/node11/test.cfg
.kokoro/presubmit/node6/common.cfg
.kokoro/presubmit/node6/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/docs.cfg
.kokoro/presubmit/node8/lint.cfg
.kokoro/presubmit/node8/samples-test.cfg
.kokoro/presubmit/node8/system-test.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.nycrc
CODE_OF_CONDUCT.md
codecov.yaml
npm WARN deprecated<EMAIL_ADDRESS>CircularJSON is in maintenance only, flatted is its successor.
npm WARN deprecated<EMAIL_ADDRESS>no longer maintained

><EMAIL_ADDRESS>install /tmpfs/src/git/autosynth/working_repo/node_modules/grpc
> node-pre-gyp install --fallback-to-build --library=static_library

node-pre-gyp WARN Using request for node-pre-gyp https download
[grpc] Success: "/tmpfs/src/git/autosynth/working_repo/node_modules/grpc/src/node/extension_binary/node-v57-linux-x64-glibc/grpc_node.node" is installed via remote

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/protobufjs
> node scripts/postinstall

npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

added 1141 packages from 1274 contributors and audited 4393 packages in 22.77s
found 0 vulnerabilities

><EMAIL_ADDRESS>fix /tmpfs/src/git/autosynth/working_repo
> eslint --fix '**/*.js'

/tmpfs/src/git/autosynth/working_repo/samples/quickstart.js
  27:25  error  "@google-cloud/os-login" is not found  node/no-missing-require

✖ 1 problem (1 error, 0 warnings)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>fix: `eslint --fix '**/*.js'`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>fix script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/kbuilder/.npm/_logs/2018-11-20T12_39_30_004Z-debug.log
synthtool > Cleaned up 2 temporary directories.
synthtool > Wrote metadata to synth.metadata.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #112   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           4      4
=====================================
  Hits            4      4

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 4683430...301f423. Read the comment docs.


Update generic param

Use W for Write
closes https://github.com/rust-bitcoin/rust-bitcoin/issues/3590

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.385%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
19695

Relevant Lines:
23906

💛  - Coveralls

If we're gonna rename the generic why not also rename the variable?

nack, we definitely need to change the variable name to.

var name updated as well


<IP_ADDRESS> - 2024-10-30

Added

Possibility to use static method for constructing an object instead of normal constructors.

simple usage

  [MakeBuilder(typeof(Entity), staticFactoryMethodName: nameof(Entity.CreateEntity))]

It may be useful when your entity has private constructors and you create it by factory methods.

#62

#60


LOGGING-193 Update Log4j 2 Imports

Update the Log4j 2 OSGi imports to list all used packages.

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


svg.namespaceClassnames doesn't work

Hi guys,
I'm using svg-sprite with Gulp (via gulp-svg-sprite plugin) and the svg.namespaceClassname parameter doesn't work. I'm trying to get rid of class prefixes and setting this parameter to false has no any effect on the output — every class is still prefixed.
Here is my config. As simple as possible to eliminate any influence of other parameters:
mode: {
    symbol: {
        svg: {
            xmlDeclaration: false,
            namespaceClassnames: false,
            doctypeDeclaration: false
        }
    }
}

Other options, like xmlDeclaration are working fine.

Some update on this.
If I change the default value of the namespaceClassnames in the sources — https://github.com/jkphl/svg-sprite/blob/master/lib/svg-sprite/config.js#L61 — it works.
So, the future works by itself.
Something wrong on the way. I'll keep digging, but maybe you can advice me what might be wrong with my configuration above?

@side-by-side I'm pretty sure your problem is that the svg configuration options are not mode specific but top-level options instead. Your configuration should rather look like this:
svg: {
    xmlDeclaration: false,
    namespaceClassnames: false,
    doctypeDeclaration: false
},
mode: {
    symbol: {
        /* ... */
    }
}

I always suggest to use the online configurator for creating the base skeleton ... Good luck!

Just came back to post the same answer and you're here already.
Thank you for looking into this! It was definitely the config issue.
But the reason I was confused, that the other options, like xmlDeclaration are working even from under the mode → symbol parent (like in my initial config).


Please add a otk.meta example

I'm struggling a bit to understand the otk.meta directive. Right now it says:
Under the `otk.meta` namespace any data can be stored that other applications
need to use from an omnifest.

Expects a `map` for its value.

otk.meta.osbuild-composer:
  boot_mode: uefi

I would love an example what is done with the data, is it put into the generated manifest.json? is it just part of the .otk file for other tools to inspect?
Related to this to me it is not clear why it's it's own directive. I wonder if we could define the metadata just via the normal otk.define.
So
otk.define:
 meta:
  osbuild-composer:
   export: image

i.e. I think I would love to have some example/explaination how this will be used.

This is going to be superceded by #85.


feat: refresh secret on delete when polling is disabled

Right now the only way to use kubernetes-external-secrets with some providers in certain circumstances,  is to set environment variable DISABLE_POLLING: true, which will lead to no-op on every _poll().
For example, using AWS SSM Parameter Store backend provider with several hundred of ExternalSecret instances provisioned via that provider leads to rate limit errors within AWS API.
See https://github.com/godaddy/kubernetes-external-secrets/pull/211 and https://github.com/godaddy/kubernetes-external-secrets/pull/156 for more background on this scenario.
In the case if DISABLE_POLLING set to true, there is no way to refresh internal Secret instance after it is created by ExternalSecret for the first time, other than completely removing and then re-creating ExternalSecret.
This is not convenient in the case if ExternalSecret instance is controlled by Helm/FluxCD, and might lead to deployment inconsistencies.
This patch allows for refreshing ExternalSecret instance in case if Secret instance was removed. It is achieved by tracking names of present Secret instances across all namespaces, and making the poll in case if  for some ExternalSecret the corresponding Secret instance is missing.
This patch is non-intrusive, and the new behavior is executed only in case if environment variable POLL_INTERNAL_SECRETS is set.

@Flydiverny
I had just updated my patch with recent master code, and it seems to be working fine for my use-case, and all auto-tests do pass.
If we can get this reviewed and merged into 8x series, it would be very appreciated !


Puzzling behavior, larger models take less time to evaluate

Hello,
Thank you very much for this awesome evaluation tool.
I've been using it to evaluate my models and reproduce some results on Open LLM Leaderboard. I'm having a kind of mystery that I'm trying to wrap my head around:

I'm evaluating LLaMA base models. I noticed that for 30B models with hf-causal-experimental takes quarter of the time needed for 13B models with hf-causal.

30B model throws an OOM when running with hf-causal but runs fine with hf-causal-experimental.

Any input would be much appreciated!

Sorry, silly mistake on my side. It turns out my 13B models were float32 whereas my 30B models are float16. I still do not the source of (2) though.


Regex update for video/audio codec from MediaInfo

MediaInfo output has at some point been changed from "Codec" to "Codec ID". This commit will support both existing and newer versions of MediaInfo output as the new regex contains an optional non-capturing group

Hello, I feel that this open source project is no longer ahead. If you want to go forward for this project, you can transfer Pull Requests to multimediainfo. This is a project forkd by MediaInfo.
Technology from Google Translate


changed order_rank_by to order.rows.by with deprecation

Here's the new PR following PR #120. This now includes deprecation warnings and the new argument name is order.rows.by.

This is little bit too complex solution. Moreover, we are planning to rename these parameter all at once, but I think you could do that.
We have similar thing going on in mia currently: https://github.com/microbiome/mia/pull/562
I can give you more info in the following days since I have not decided the names of parameters for miaViz. Then we can have a short chat about the renaming. Does this sound good plan?


#208 fixes module not found

Issue #208
Fixes Error:
Webpack Compilation Error
Module not found: Error: Package path ./dist/config.default is not exported from package /Users/XXX/work/visual_regression_testing/node_modules/cypress-image-diff-js (see exports field in /Users/XXX/work/visual_regression_testing/node_modules/cypress-image-diff-js/package.json)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Module not found: Can't resolve 'uuid'

This happens after installation.
"next": "12.1.6",
"react": "18.1.0",
error - ./node_modules/react-best-gradient-color-picker/lib/components/Portal.js:9:0
Module not found: Can't resolve 'uuid'
Import trace for requested module:
./node_modules/react-best-gradient-color-picker/lib/components/EyeDropper.js
./node_modules/react-best-gradient-color-picker/lib/components/Controls.js
./node_modules/react-best-gradient-color-picker/lib/components/Picker.js
./node_modules/react-best-gradient-color-picker/lib/index.js
./pages/app/content-engine.js
https://nextjs.org/docs/messages/module-not-found

Will add to peer dependencies now, you can also install in your project. Thanks for pointing this out.

Fixed in v2.2.21


Sizing limitations vary by Region

This document does not call out that you cannot create more than a /29 within the WESTUS2 region.  The max listed here is /28 but WESTUS2 region says no more than 10 IPs.  If there are other limitations they should be mentioned here.
Update: Seems it is related to dev/test EA subscription offer, which is also not called out, so might not be region specific. If there are limitations on this resource within dev/test EA offer, it should be mentioned here.

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

ID: 6f5d33e9-bcb3-3358-f859-c2ff3e71b131
Version Independent ID: 5498280e-dd3c-bd46-9877-3506ad926dbd
Content: Azure Public IP address prefix
Content Source: articles/virtual-network/public-ip-address-prefix.md
Service: virtual-network
Sub-service: ip-services
GitHub Login: @asudbring
Microsoft Alias: allensu

Closing this issue.  Language in the documentation states you can create up to a /28 but the default azure limits per subscription is 10 IPs which is smaller than a /28.  So you have to open a ticket to be able to create a /28 in your sub.  I feel like this should be referenced without having to deep dive into multiple documents to find the limitation.


Error when trying to run Pretrained STL weights.

Hi,
I had a question regarding trying to run inference using pretrained weights for fully unsupervised clustering on the STL10 data.
Since STL10 is RGB and I have a greyscale dataset, I tried doing a grayscale to RGB transformation in a simpler dataloader I created.
def load_data(config):
    transform = transforms.Compose([transforms.Lambda(lambda x: x.repeat(3, 1, 1)),transforms.ToTensor(),
                                    transforms.Resize(tuple(np.asarray([config.input_sz,config.input_sz])))])
    train_data_path = os.path.join(config.dataset_root, "Crops/")
    dataloader = torch.utils.data.DataLoader(torchvision.datasets.ImageFolder(root=train_data_path, transform=transform),
        batch_size=int(config.batch_sz),shuffle=False,num_workers=0,drop_last=False)
    return dataloader

However, when I try to run this code, I get the following error.
Unexpected key(s) in state_dict: "head_A.heads.1.0.weight", "head_A.heads.1.0.bias", "head_A.heads.2.0.weight", "head_A.heads.2.0.bias", "head_A.heads.3.0.weight", "head_A.heads.3.0.bias", "head_A.heads.4.0.weight", "head_A.heads.4.0.bias", "head_B.heads.1.0.weight", "head_B.heads.1.0.bias", "head_B.heads.2.0.weight", "head_B.heads.2.0.bias", "head_B.heads.3.0.weight", "head_B.heads.3.0.bias", "head_B.heads.4.0.weight", "head_B.heads.4.0.bias".
size mismatch for trunk.conv1.weight: copying a param with shape torch.Size([64, 2, 3, 3]) from checkpoint, the shape in current model is torch.Size([64, 1, 3, 3]).
My question is why is the parameter of the checkpoint (64,2,3,3)  Since it is 3 channel, shouldn't it be (64,3,3,3)?  Also, what are the unexpected keys showing up in the state_dict.  Thank you!

The input to that network has 2 channels because it takes the sobel filtered image. (In the suppmat you can find input channel sizes in table 2.)
So I think you don't need to convert to RGB, just sobel filter your greyscale images to get 2 channels.
As for the names, it suggests you are loading my model into a different model template you wrote with your own code. Causing the name mismatch. To avoid this I guess you should load into an instance of the actual class I used. For STL this was ClusterNet5gTwoHead.

OK Thank you for the help. I was not accounting for the 2nd sobel channel.


feat(core): React.PureComponent demo/generators

Since data structures are immutable, use Purecomponent released in 15.3.0. Under the hood, it just runs a shallow comparison on props before tagging the component for update. This also updates generators. although we use memoized selectors for view data, i would still consider this as a good foundational feature to leverage in pure class components and generators...

Therefore, we're changing functional components to behave as pure when they're used inside a React.PureComponent but to rerender unconditionally when contained in a React.Component

We anticipate that this behavior will work well in real-world apps: if you use immutable data, your class-based components can extend React.PureComponent and your functional components will be pure too; if you use mutable data, your class-based components will extend React.Component and your functional components will update accordingly.

https://github.com/facebook/react/pull/6914

Coverage remained the same at 98.346% when pulling 38b5ed0c220c15f2aa6db6e9798251a2e45dcb43 on justingreenberg:feature/react-15.3-pure-component into bfe0c201053ffe052ec2dfe010a7dd2c7f054c10 on mxstbr:master.

Coverage remained the same at 98.346% when pulling 37eec3a9e2d3423588d99c824751c257f5117077 on justingreenberg:feature/react-15.3-pure-component into bfe0c201053ffe052ec2dfe010a7dd2c7f054c10 on mxstbr:master.

Coverage remained the same at 98.343% when pulling 493e43bacd7f889c8ee01f2a516d6c5c34466de0 on justingreenberg:feature/react-15.3-pure-component into c11c0ae23098a018ede1ccee85a5bb784ff3decf on mxstbr:master.

@mxstbr thoughts on this?

Coverage remained the same at 98.429% when pulling 17930c4b952dd905b1383d8f9372b450a929d96d on justingreenberg:feature/react-15.3-pure-component into 96e9441838ba2ff372e8f3ef490e7fa38d29974c on mxstbr:master.

@justingreenberg I read over React.PureComponent Looks like logical progresson for RBP.
Since it's (mostly) a drop-in replacement, I'm inclined to say let's do it. Since PureComponent is (still) officially undocumented, this PR should probably reference something in code comments like React change log
I'm short on time just now, so won't have time to test for a few days...
Seems like an excellent candidate for @mxstbr suggested development branch 👍

@gihrig yes, and of course it's really just a demo (in the default app)... the real meat of this PR is the option to create a PureComponent when using internal generators

How does this work together with ImmutableJS? @justingreenberg I'm not sure we really want people to do this since we do use immutable data structures for cheap nextProps.equals(this.props)…

Coverage remained the same at 98.429% when pulling 70050158a7fe46510eb4cc9d08b17f533b113d20 on justingreenberg:feature/react-15.3-pure-component into b8ea957ed93bf6d4fe8124ba340e5b257ecf5809 on mxstbr:master.

@mxstbr i don't see any immutable comparisons in the current codebase?

I had no idea what I was doing back when I wrote the current example. 🙈 We really should be doing immutable shouldComponentUpdate checks everywhere, but I haven't really updated the example much because I've been thinking about writing a new, slightly more complex one from scratch so people can really get a grasp on all the stuff we use.
Stay tuned for updates on that!

maybe i'm misunderstanding, but i think this means that using legacy Components will cause our stateless functional child components (Button, H1, etc) to rerender on every diff...

Yeah yeah alright alright I'm convinced.

also, can you trigger new appveyor build whenever you have a sec 🙇 😃

On it!

Coverage remained the same at 98.447% when pulling e8890d6a126c5f927bb6c14c8a36822302c85ebc on justingreenberg:feature/react-15.3-pure-component into be522977277d7c8171c437c2796ab28309a02f4d on mxstbr:master.

Let's not merge this for now because React.PureComponent isn't documented yet so it might confuse some people. Ref https://github.com/facebook/react/issues/7379

Coverage remained the same at 98.447% when pulling b378b3ac696cca73f0967e64624ae2e4611b3370 on justingreenberg:feature/react-15.3-pure-component into 340cd5fae31536faf43e4722ca55814b853ee732 on mxstbr:master.

@mxstbr sounds good... i've had some extra time this week (in case you didn't notice 😉) so i'm thinking maybe i'll take a stab at writing the docs later

I had no idea what I was doing back when I wrote the current example. 🙈 We really should be doing immutable shouldComponentUpdate checks everywhere

extending from PureComponent will automatically take care of this :)

I haven't really updated the example much because I've been thinking about writing a new, slightly more complex one from scratch so people can really get a grasp on all the stuff we use.

let me know what you are thinking here... i def had some thoughts on this as well

I really want to get some authentication similar to what I did with login-flow. Showcasing that and the general workflow with sagas could really help people out!
What I'm not 100% on yet is what happens once you're logged in, what were your ideas?

Coverage remained the same at 98.447% when pulling 479974fc2ebcdbc89c9f9c53680b60dd669d10f6 on justingreenberg:feature/react-15.3-pure-component into f92cad7161de3ed8b4fff410b61ba319bd8b90eb on mxstbr:master.

nice! yes authentication comes up a lot 👍
i was thinking about the saga utilities, i wrote a quick POC on my feature/saga-utilities branch:
https://github.com/mxstbr/react-boilerplate/compare/master...justingreenberg:feature/saga-utilities
as you know many people have been confused by the current implementation, so i've been thinking about how we can simplify the API a bit... this is one idea, provides a more familiar api for binding sagas directly to component (instead of route definition) ala react-redux.. provides two  methods run and watch... run just maps over sagas, watch maintains references and cancels when component unmounts

That looks awesome.
Why do you use store.runSaga in app.js? Couldn't you
import sagaMiddleware from 'redux-saga';
/* … */
<SagaProvider run={sagaMiddleware.run} />

No need to attach something to the store, right? If that's the case as I think it is, you could even assume that as a default and only let it be overriden by passing in a custom run?

react's PureComponent has issues with hot reloading https://github.com/facebook/react/issues/7388

The issue seems related to react-transform (deprecated for a while) and react native (not a render target for this project)... Have you tried checking out this pr locally?
Either way, we need to remove https://github.com/danmartinez101/babel-preset-react-hmre anyway, and replace it with https://github.com/gaearon/react-hot-loader v3 or vanilla webpack hmr @mxstbr lmk your thoughts
As a side note, "break hmr" usually just means components hot reload but do not maintain local state, which is generally OK for developing redux applications imo

As a side note, "break hmr" usually just means components hot reload but do not maintain local state, which is generally OK for developing redux applications imo

That's absolutely fine.

replace it with https://github.com/gaearon/react-hot-loader v3 or vanilla webpack hmr

Is that stable yet? AFAIK it isn't… (not that we've ever cared about that much, but you know…)

(not that we've ever cared about that much, but you know…)

agreed lol—we have been using webpack v2 alpha (many feature not documented) without any known issues for months right?? living on the edge dude :metal:

As a side note, "break hmr" usually just means components hot reload but do not maintain local state, which is generally OK for developing redux applications imo

That's absolutely fine.

if you're cool with that, my recommendation would be to pull out babel-hmre and deprecated react-transform and just use webpack's vanilla hmr system

Huston, we have a problem...
This build (and others) are failing due to an eslint (and friends) problem described in #864. This problem is resolved by #862. But #862 is stalled due to Coveralls not completing...

@gihrig thank you for the explanation—this was driving me nuts :)

Coverage remained the same at 98.468% when pulling af36cf600badbf878d216967b688975f7dd42052 on justingreenberg:feature/react-15.3-pure-component into 0e76874e8d353da0a34e75bdc3fdbe0073ebc875 on mxstbr:master.

Coverage remained the same at 98.468% when pulling 7e7ba355e5e6e0a68b56f5c00aa2109c1cdb007e on justingreenberg:feature/react-15.3-pure-component into 0e76874e8d353da0a34e75bdc3fdbe0073ebc875 on mxstbr:master.

Coverage remained the same at 98.942% when pulling 98630d5a884454292321057feb2443f573cf5f45 on justingreenberg:feature/react-15.3-pure-component into ee12ac5c0055991b434dbf8959c5fdc369e88cbb on mxstbr:dev.

@mxstbr is netlify building off dev?

Yeah, check the last commit, it should have a deploy!

Coverage remained the same at 98.942% when pulling 82de16d9ca777307e4a48c1b1ea6376a54f24096 on justingreenberg:feature/react-15.3-pure-component into 7bae788cf344da09a2b67545ffc788216513a653 on mxstbr:dev.

Coverage remained the same at 98.942% when pulling 748a2ffbed772fb49b86f4c40998af56880d4b38 on justingreenberg:feature/react-15.3-pure-component into bacb85fb62d68fc7afd2ee00be0d8c25994d2738 on mxstbr:dev.

Coverage remained the same at 98.942% when pulling 99bd6cbd035c4aac7411640e0c4d587284b3d7cb on justingreenberg:feature/react-15.3-pure-component into 347ed436e86babb79849def968110557c021ed3d on mxstbr:dev.

Coverage remained the same at 98.942% when pulling a85ef709b7c5d7a22d28cd405b6a1aaa90d7f433 on justingreenberg:feature/react-15.3-pure-component into e83b1b29a39f5746a038bd6f8cd47f795cab8420 on mxstbr:dev.

Coverage remained the same at 98.942% when pulling a37f4be0c54da0c2f448e15b18f1f97159ca4522 on justingreenberg:feature/react-15.3-pure-component into 2cd4751d0526dd55e316561d646b2a8710c18dd0 on mxstbr:dev.

Coverage remained the same at 98.953% when pulling 336250cda3b8fe655b2911ed8dc6e2eb3824d898 on justingreenberg:feature/react-15.3-pure-component into 2852c09b4d0851ce46eea3dc55517f67fd1ab80d on mxstbr:dev.

monitoring https://github.com/facebook/react/pull/7730

Coverage remained the same at 98.953% when pulling 596c6fe07ffe6557c75866d552ce962ac1c311c2 on justingreenberg:feature/react-15.3-pure-component into 18952bfdcc8ee32778934906ece1799b1daf3987 on mxstbr:dev.

https://facebook.github.io/react/docs/react-api.html#react.purecomponent 🎉

Coverage remained the same at 98.958% when pulling c7f06549ba8b829ccf2053a63433de5bbbb36852 on justingreenberg:feature/react-15.3-pure-component into 1d18cc7d1a6fff18ca083a57dd1797bf13cc64ef on mxstbr:dev.

Is there a way to add linting, to enforce the use of PureComponent?

@mxstbr this should be ready as well needs to be rebased but in meantime if you have a few mins to take the generators for a spin and let me know if there are any general changes will try to handle today :)

Is there a way to add linting, to enforce the use of PureComponent?

I don't think we want to do that, as PureComponents can have unexpected effects in terms of rerenders. If people use them, they should make a choice to do so.

@sedubois sorrry meant to respond!

Is there a way to add linting, to enforce the use of PureComponent?

As max pointed out we probably don't want to force shallow comparisons

Coverage remained the same at 99.177% when pulling 765a6d2793d7f7786bd3fae44a00b39428e17465 on justingreenberg:feature/react-15.3-pure-component into 56f3f68f5d1170a084e081a9525842aca63784b6 on mxstbr:dev.

@mxstbr rebased with dev and CI is passing 👍
@gihrig can you guys confirm generator support is working properly?

Generator support is working perfectly. Thank you so much, glad we can finally land this!


English MV and addons

Might need to talk it with Kix and addon devs

After talking with Kix: English MV and the official addons won't be included.
Remaining: Korean version of TRC, GenGibs


[Draft] E2E QuickStart for Multi-stage Query Engine

Overview
Master PR for trying out the new mulit-stage-query-engine. See: https://www.startree.ai/blogs/apache-pinot-native-join-support
Since the feature branch is not merged back to main branch. Please read the instruction below to try it out.
Try the new engine

run git fetch upstream pull/8662/head && git checkout FETCH_HEAD -b multi_stage_tryout to checkout the tryout branch; you can also use gh pr checkout 8662 to checkout the PR
run mvn install -Pbin-dist -pl pinot-distribution -am -DskipTests to build pinot
run regular quickstart

run cd build; bin/pinot-admin.sh StartZookeeper && bin/pinot-admin.sh StartServiceManager -bootstrapConfigPaths conf/pinot-controller.conf conf/pinot-broker.conf conf/pinot-server.conf&
load your tables normally. for example:

bin/pinot-admin.sh AddTable -tableConfigFile examples/batch/baseballStats/baseballStats_offline_table_config.json -schemaFile examples/batch/baseballStats/baseballStats_schema.json -exec
bin/pinot-admin.sh LaunchDataIngestionJob -jobSpecFile examples/batch/baseballStats/ingestionJobSpec.yaml

run the broker query: curl -X POST http://localhost:8099/query/sql -d '{"sql": "SELECT * FROM baseballStats_OFFLINE"}'

Noted that there are several limitations, please see limitation section below.
Limitations
The feature branch is still under development so please check back to see any updates on this limitation section

generic table are not supported; _OFFLINE or _REALTIME suffix must be attached to the query table name
schema name has to match table name

for example the baseballStats example above, one need to edit the table schema to rename schemaName to baseballStats

[will be supported soon] aggregation is not supported, so count(*) or group by cannot run.

Share your feedback
Please share your feedback on Slack channel: https://apache-pinot.slack.com/archives/C013WKLT5T7

Codecov Report

:exclamation: No coverage uploaded for pull request base (multi_stage_query_engine@6ae5f76). Click here to learn what that means.
The diff coverage is n/a.

@@                     Coverage Diff                     @@
##             multi_stage_query_engine    #8662   +/-   ##
===========================================================
  Coverage                            ?   68.57%
  Complexity                          ?     4570
===========================================================
  Files                               ?     1746
  Lines                               ?    89669
  Branches                            ?    13334
===========================================================
  Hits                                ?    61493
  Misses                              ?    23934
  Partials                            ?     4242

Flag
Coverage Δ

integration2
25.03% <0.00%> (?)

unittests1
66.33% <0.00%> (?)

unittests2
15.22% <0.00%> (?)

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

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 6ae5f76...e11d6a9. Read the comment docs.

closing as it is already merged into quickstart https://github.com/apache/pinot/pull/8980


Docs should clarify how Claude Desktop treats resources

The docs state:

Resources are designed to be application-controlled, meaning that the client application can decide how and when they should be used.
For example, one application may require users to explicitly select resources, while another could automatically select them based on heuristics or even at the discretion of the AI model itself.

I don't think it's immediately clear how Claude Desktop treats resources. From playing around with it, it seems like it requires explicit user selection at least for now. I think it would be useful to make this explicit in the documentation so servers authors know what to expect when debugging.

That sounds reasonable. We should make sure it reflects that Claude Desktop is purely an example. There are other MCP clients. Maybe, if you have a chance, would you give it a shot to make that change and send a PR? I'd much appreciate that.

@dsp-ant. My testing so far shows:

Prompts and Resources are only available in the first message of a chat, and are added to the message as attachments.
Resource Templates aren't exposed/understood by the Claude Desktop.

Is that right? I'm referring to https://modelcontextprotocol.io/clients as well to set my expectations :)

Just want to comment here and say that Tool Results should be able to pass URI's back to Model Context for inclusion, maybe via another prompt for permission?

In the spec, Resource URI's are mandatory in Tool Responses.
https://github.com/modelcontextprotocol/specification/blob/bb5fdd282a4d0793822a569f573ebc36804d38f8/schema/schema.ts#L482-L509
The Client would receive that but it's not specified exactly how the Resource would be presented to the LLM.
The Server can suggest an Audience [ User | Assistant ] and a Priority for each Resource. At the moment I'm interpreting "User" as being "Display to User" and "Assistant" as "Include in Context" with none specified as "Both".
I think a common case may be the Server wanting to return a set of Resource Templates to the Client, so that the User or Assistant can get them on demand if wanted in the next turn. That's obviously not prohibited in the spec.
There's a similar case where the User may wish to upload a binary to send to a tool, for the result to go in context (e.g. Image -> OCR Tool -> Markdown).  That's purely a Client implementation question. I'm tempted to tabulate these scenarios/options. There's also what Claude Desktop actually does vs. other Client implementations.
There's a couple of other discussions that seem relevant so I'll paste them here:
https://github.com/orgs/modelcontextprotocol/discussions/54
https://github.com/modelcontextprotocol/specification/discussions/90

Thanks, this is helpful context too.
Now that I've got Resources being returned from my API, I'd ideally have multi-modal instances of my Resource responses.
For example, right now I'm returning Video Files from a search, which are then query able, based upon a separate, analysis pipeline I've built happening in my API.
Ideally I'd be able to pass in a set of embedded images in responses like this:

And show thumbnails inline for each of these detected scenes.
Maybe I'm thinking of a Resource in the wrong way? Or maybe I should include a hierarchy of Resources in my response? IE imply that a set of Resources belong to a single video?

It seems the protocol specifies a single mimetype for each Resource:

Claude Desktop as a client has limited support for any resource types beyond images. I'm doing quite a lot of testing here at the moment so will report back with what I learn... but planning to integrate with a different Client soon.


Fix file-not-found error in pip installed version

Fixes #3
The engines were renamed but the manifest was not updated, so they weren't included in the pip build.
It also marks the new engines as executable, so they'll be runnable on Linux/Mac without permission errors.

This looks great.


JsonPointer does not take into account the JsonPropertyName attribute

Nuget Package
JsonPointer.Net
Package Version
3.0.3
Operating System
Windows
.Net Target (if relevant, please specify the version in the description)
.Net (5 or after)
A clear and concise description of the bug.
When creating a pointer from an expression, it does not take the JsonPropertyName attribute to resolve the path.
What did you expect?
Take JsonPropertyName attribute into account and be case insensitive too.
Please add test code or steps to reproduce the behavior.
JsonPointer.Create(x => x.ApplicationList.Applications);
public class GHub
{
[JsonPropertyName("applications")]
public ApplicationList ApplicationList { get; set; }
}
Is there any other information you'd like to share regarding this bug?
No response
Code of Conduct

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

Thanks for reporting, but I'd say this is an enhancement, not a bug, since this method of creation was never designed to do this.
That said, I'm not sure this can be done since I don't think I have reflection capabilities inside an expression, which means I wouldn't be able to get the attribute.  I'll have a look though to be sure.

and be case insensitive too

I'm not sure what you mean by this.  This method is generating a pointer.  Case (in)sensitivity is generally a reading concern, not a writing concern.

Here a proof of concept:
if (body is MemberExpression me)
{
    string memberName = me.Member.Name;

    if (me.Member is PropertyInfo propertyInfo)
    {
        var jsonPropertyName = propertyInfo.GetCustomAttribute<JsonPropertyNameAttribute>();

        if (jsonPropertyName != null)
        {
            memberName = jsonPropertyName.Name;
        }
    }

    segments.Insert(0, PointerSegment.Create(memberName));
    body = me.Expression;
}

The information can be cached in memory for future usage to ensure the best performance possible.

I'm not sure what you mean by this. This method is generating a pointer. Case (in)sensitivity is generally a reading concern, not a writing concern.

By convention, C# is PascalCase and JSON camelCase, so when using the expression to obtain the JsonPointer, JsonPatch does not seem to work.

You're not wrong, but I think "case insensitive" is the wrong thing to be focusing on.  What you want is options to:

use the JSON name
support name conversion

and JSON camelCase

Not always.  I've seen plenty of times where people use many different casings as keys in JSON.
I'll work up a version of this that provides some alternate naming options.

Not always. I've seen plenty of times where people use many different casings as keys in JSON.

I agree with that, the JSON file that I'm trying to parse uses camelCase and snakeCase in the same file... But with System.Text.Json, you can use the option to be case insensitive and it works pretty well with my implementation as I only need to specify the JsonPropertyName attribute on snakeCase properties.

Please see v3.1.0 and updated docs.


OSX Setup failed.

Hello!
I followed the guide on OSX, but after command ”npm install” there are lots of errors.

“bower install” is ok.

“npm start” is also has errors.
It seems cannot find ‘pokemon-go-mitm’ module, but i can find it in my filesystem.

Should i install some dependent libraries first?
Thanks for the awesome work!

What version of protoc do you have installed? And did you try npm install pokemon-go-mitm?

It’s protobuf-3.0.0-beta-4.
And i tried install pokemon-go-mitm:

Maybe try removing the node_modules folder and trying again. Anyone else have any ideas?

Finally succeeded!
Just close the proxy!
Thank you very much!

 Have fun!


how about this API SoftModalMessageBox

Private Declare Function SoftModalMessageBox Lib "user32" (pMsgBoxParams As Any) As Long
Public Enum eMsgParams
cbSize
hWndOwner
hInstance
lpszText
lpszCaption
dwStyle
lpszIcon
dwContextHelpId
lpfnMsgBoxCallback 'call back
dwLanguageId
pwndOwner
wLanguageId
pidButton
ppszButtonText
cButtons
DefButton
CancelID
Timeout
unk2_2 = 21
unk2_3
unk2_4
End Enum
Public Enum WindowsVersion
Earlier
Later
End Enum
Public WinVersion As WindowsVersion
Public Function WindowsVersion() As Long
' Windows 10 100
' Windows 8.1 63
' Windows 8.0 62
' Windows 7 61
' Windows Vista 60
' Windows XP 51
' Windows 2000 50
Dim SystemSet As Object
Dim System As Object
Dim S As String
Dim i As Long
Set SystemSet = GetObject("winmgmts:").InstancesOf("Win32_OperatingSystem")
For Each System In SystemSet
S = System.Version
Next
i = InStr(S, ".")
S = Left$(S, i - 1) & Mid$(S, i + 1)
i = InStr(S, ".")
If i Then S = Left$(S, i - 1)
WindowsVersion = Val(S)
End Function
Public Function MsgBoxEx(arrText$, strTitle$, strPrompt$, Optional ByVal DefButton As Long = 0, Optional ByVal Timeout As Long = -1, Optional ByVal CancelID As Boolean = True)
Dim md_msg() As Long
Dim lBtn() As Long
Dim sTxt() As String
Dim i&, U&
If WindowsVersion() < 61 Then
WinVersion = Earlier
ReDim md_msg(17) As Long
Else
WinVersion = Later
ReDim md_msg(24) As Long
End If
mySplit arrText, sTxt, "|" 'better than vba.split
U = UBound(sTxt)
ReDim lBtn(U) As Long
For i = 0 To U '
lBtn(i) = U - i + 2 '5 4 3 2, and 2 is the last
Next i
md_msg(eMsgParams.cbSize) = 40 '
md_msg(eMsgParams.lpszText) = StrPtr(strPrompt)
md_msg(eMsgParams.lpszCaption) = StrPtr(strTitle)
md_msg(eMsgParams.dwStyle) = vbYesNo 'odd, should set this
md_msg(eMsgParams.DefButton + WinVersion) = DefButton - 1
md_msg(eMsgParams.CancelID + WinVersion) = CancelID '
md_msg(eMsgParams.cButtons + WinVersion) = U + 1 'buttons count
md_msg(eMsgParams.pidButton + WinVersion) = VarPtr(lBtn(0)) 'ButtonID
md_msg(eMsgParams.ppszButtonText + WinVersion) = VarPtr(sTxt(0)) 'Button text
md_msg(eMsgParams.Timeout + WinVersion) = Timeout '
MsgBoxEx = SoftModalMessageBox(md_msg(0))
End Function

Hi,
I tried to run your code but I have VBA 64 bits and it does not work. I am sure with some conditional compilation can be made to run on 64 bits as well. However, this repo is intended to run on both Mac and Windows and so a native approach made the most sense. Thanks for your suggestion!


404 after deploying

Hi!
I've just tried deploying this to 1.7.0 and it seems to have worked and deployed properly, with the last line of the logs reading "Starting connect web server on http://<IP_ADDRESS>:9000", but when I go to the app's URL, I get a 404. Any idea why?

I think it's a broader problem related to Deis; see https://github.com/deis/deis/issues/3827.


Escape disallowed tags

A reopening of #169 after fixing build and tests and rebasing onto fresh master.

Is escaping child tags, even those that are otherwise acceptable, the behavior most developers would want? Or would they prefer to escape just the forbidden tags? Maybe an option?

Is escaping child tags, even those that are otherwise acceptable, the behavior most developers would want? Or would they prefer to escape just the forbidden tags? Maybe an option?

I would expect it to be all or nothing, that is, either escape the tag and its entire subtree or don't escape at all. Escaping just the top tag would render an unexpected DOM tree (and perhaps one that makes no sense).
Does the package modify the DOM tree like that in other cases?

Well, if we filter out a tag entirely, we don't remove all of its child
tags. To do so would greatly limit the tool's utility in our original use
case, which is copying and pasting from webpages, Word, etc. into a rich
text editor without bringing in unwanted crud, while keeping as much of the
allowable stuff as possible.
So if we start escaping the unwanted tag name rather than removing the tag,
it doesn't make sense to suddenly treat the contents differently? I think?
At the very least I think there could be different preferences on what
should be done. But, escaping the tag is not a use case we have here to
start with, so maybe it makes more sense to those who have it.
On Fri, Nov 1, 2019 at 10:52 AM yehonatanz<EMAIL_ADDRESS>wrote:

Is escaping child tags, even those that are otherwise acceptable, the
behavior most developers would want? Or would they prefer to escape just
the forbidden tags? Maybe an option?
I would expect it to be all or nothing, that is, either escape the tag and
its entire subtree or don't escape at all. Escaping just the top tag would
render an unexpected DOM tree (and perhaps one that makes no sense).
Does the package modify the DOM tree like that in other cases?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/sanitize-html/pull/312?email_source=notifications&email_token=AAAH27OCE2H3XJSDQX2J7JLQRQ7CZA5CNFSM4JHETOZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC3ESTQ#issuecomment-548817230,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27OJY3R37QMGR443C3TQRQ7CZANCNFSM4JHETOZQ
.

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his

@boutell Modified the API as you suggested, the code is even nicer now!
Instead of a boolean, you can now specify a mode via disallowedTagsMode which can be either discard (default), escape (the behaviour you suggested, escaping just this tag) or recursiveEscape (the behaviour I suggested).
Added the needed documentation, and once you give an OK, I'll open the appropriate PR in DefinitelyTyped to document the new option and enum.

@boutell Any news?


wiki

This PR contains documentation for the la-metro-councilmatic wiki. I do not plan to merge the wiki repo into master (though I will need to merge the images), but rather copy it over to the repo's wiki. PR's are just nice places to request feedback.

I would like to make our original Google doc obsolete (i.e., delete it entirely, or focus it to just the after-hours policy). I need to decide which other parts of that Google doc should be moved to this Wiki. Maybe:

something about "useful commands"
something that outlines the after-hours policy (though maybe we should point to an internal document, rather than making that info available open source)

https://docs.google.com/document/d/1X34EU06gErdZ1psg8niSqDMk0nH-M1vsP-RNPLNUsF4
@hancush would you have time to give this a first pass? It's still missing lots of details, but I'd love to talk about it from a high-level perspective, before going any further.

@hancush - another batch of edits!
also, for info about Friday support nights, I think we should keep that as an internal document. It seems odd to outline our protocols for client relations in a publicly accessible wiki. what do you think?

an overview of the (template) file structure would be helpful—i got twisted up between event.html and events.html and all of the partials.


Documented default value for -Confirm incorrect for Move-ADDirectoryServerOperationMasterRole

Move-ADDirectoryServerOperationMasterRole behaves as if the "-Confirm" switch defaults to True. Without specifying that switch, I am prompted to confirm the move of each FSMO role type.
However, the documentation incorrectly states that the default value for this switch is False.  This should be updated.
Additional improvements:

Change the description from "Prompts you for confirmation before running the cmdlet." to "Prompts you for confirmation of each chosen FSMO role before executing a move. Override this behavior by specifying -Confirm:$false. (useful for scripts)"

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

ID: 001745f8-db7f-53ce-444a-e0a45b590ef9
Version Independent ID: 7520d272-a7f0-438d-76a6-710d9cb7af82
Content: Move-ADDirectoryServerOperationMasterRole (addsadministration)
Content Source: docset/windows/addsadministration/Move-ADDirectoryServerOperationMasterRole.md
Product: w10
Technology: powershell-windows
GitHub Login: @andreabarr
Microsoft Alias: v-anbarr

@e0i . Assign user to look into this issue

@fshields Hello,
Thank you for trying to make MicrosoftDocs a better place.
By default switch parameters are always false. unless explicitly specified - Switch Parameters. Here you are not prompted because of the -Confirm parameter but because the impact of the cmdlet is High. You could read more about cmdlet confirm impact and preference - $ConfirmPreference, ConfirmImpact
Let me know if you have any questions.


Docs: update web.yml

The sample web.yml used for deploying points to a prerelease version of oranda; it should probably be using the latest instead. I also updated oranda's own web.yml to use the latest release.

Rolled back the Oranda upgrade in the workflow; it tripped the link checker.

Updated to use the dynamic latest URL instead of 0.2.0.


Setting up the geolocation in  LCS

Why is APAC not on the list of geographies? Thanks in advance.
[Enter feedback here]

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

ID: 2d2fd431-c11f-d562-e81e-c41ee7987720
Version Independent ID: 2d2fd431-c11f-d562-e81e-c41ee7987720
Content: Available geographies for Dynamics 365 finance and operations apps - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/deployment/deployment-options-geo.md
Service: dynamics-365-cross-app
GitHub Login: @ShaileshNikam-MSFT
Microsoft Alias: shailesn

Hi @jlabustro,
Thanks for your feedback.
The reason APAC is not in the table is because they're not in a data resident region.
The geographies in the list provide data residency for Dynamics 365 Finance and Operations.
Thanks for the query though.


Add row group size option to ParquetWriter.create

The default row group size for Parquet is 128MB but for the purpose of querying, official Parquet documentation recommends large row groups https://parquet.apache.org/documentation/latest/.
Though for the purpose of performing potentially multiple transformations and ones that require merging them in specific order, it is much better to have much smaller row group size to reduce memory footprint.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8015/
Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8017/
Test PASSed.

alluxio-bot, merge this please


Create Navbar

Implementations: closes #23

Created app Navbar with component styling.

Hello @BenMKT Your work looks great!
Highlighs
✔Navbar and Navlinks created
✔Styles added
✔Linters pass.
However I do have one comment.

In your design the Navbar has a complete border while in the original design

The navbar only has a bottom line division. Maybe you can change the border to be only a bottom border to match the original design. What do you think?


pub use EmptyServiceProvider

this allows a user to actually follow the instructions under:

Get type of instance of ServiceProvider

cause it requires the explicit use of EmptyServiceProvider as first type param in ServiceProvider

This isn't needed to call ServiceProvider::new(), I'm using it fine without it. What's your use case for needing it?
For reference I use it like so
fn some_function() {
  let service_provider = ServiceProvider::new()
    .add_transient(...)
    // ...more dependencies registered

  ...
}

Your example is not trying to create  a type alias, please try to do what the quoted part of the readme does

I misunderstood what documentation you were referring too, but it makes sense. In general, I would discourage fully qualifying the type of the ServiceProvider and just let the compiler infer it with type parameters as needed.

yeah right now its not just discoraged its not possible though documented :D


Fix "lxc file push  //"

Currently LXD appears to assume that the target filename will always match the source and that the path provided as the target must be a directory (and therefore creates it).
This should be changed to only use the source filename if the specified target is a directory. If it's not, the target should be assumed to be a file in an existing directory. If the directory in question doesn't exist, an error should be returned.

basically, if  is not a directory, return an error right?
In containerFilePut()
replace os.mkDirAll() with os.Stat(), and directory don't exist return an error.
Other remain same?

Note quite.

If target is an existing directory, write the file using the source filename
If target isn't a directory but dirname(target) exists and is a directory, then the user wants us to write with a different filename
If target doesn't exist and dirname(target) doesn't exist either, then return an error.

Thanks, you can assign this to me.

Thanks. I would assign it to you if github would let me... Apparently issues can only be assigned to project members...

The fix has been merged. Thanks!


Provide TSV export

4 columns:

subject
predicate
object
named graph (e.g redundant, non-redundant)

With CURIEs
Why:

many tools speak TSV (pandas, sqlite, cut, grep)
loading the ttl files using the OWLAPI causes inference that the OPs are actually APs. I believe querying by robot will route through the API causing injection of these unwanted triples
having the two files combined into one is useful

Discussing this on the translator reasoning call.. kgx format may be good

In the near future I would like to stop producing the non-redundant graph, because it is not adequately non-redundant. Instead I will provide SPARQL CONSTRUCT (and possibly Souffle) post-processing instructions. So for the time being it would be cleaner to produce two files. Sound okay?


should convert() take 2 inputs (data_in & code_map), or just 1 single pre-merged data-map?

2 input -- recipe (code_dict: {code_in, code_out, weight_out}) + ingredients (data_in: {code_in, value_in}
1 input -- prepackaged meal kit: code_in, value_in, code_out, weight_out
output -- code_out, value_out = sum_by_code_out(value_in, weight_out * value_in)

1 input approach is simpler --

removes need to confirm coverage of code_dict on data_in
removes need for column existence checks across code_dict & data_in
plays nicer in pipelines.


fix(Intersection): remove bad export

:tada: This PR is included in version 4.0.3-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


Allow location of startup scripts to be overridden

Hi Alex,
The Linux guest environment is rewritten, and is located in our development GitHub branch:
https://github.com/GoogleCloudPlatform/compute-image-packages/tree/development
The logic for running scripts has completely changed, so your pull request may no longer be applicable. Please feel free to update or close this pull request accordingly.


EDU-1006 use plyr in abc notation and ear training

Closes https://educandu.atlassian.net/browse/EDU-1006

Such a good cleanup and simplification, especially for a "one-size-fits-all" media player approach.


Make novnc the default console in RPC13 and later

For more info see
https://review.openstack.org/#/c/228197/

Since we have this already in Kilo implemented I like to enable this ASAP because SPICE is not usable.

RPC Docs Bug : https://github.com/rpcdocs/rpcdocs/issues/457

what should this target?
12.0.1 11.1.6 and 12.1.0?

For the moment, please create a 13.0.0 milestone and put it there.

Since this is targeted for r13, I'm going to update the title of the issue because it's a bit ... confusing.

Shouldn't the milestone be set on 13.2.0 ?

@evrardjp See the waffle board, I'm not on board with moving this to 13.2.

@BjoernT Selecting this for 13.0 would be the contrary of the use cases: it would be a new feature that comes "non free."

@evrardjp  That is wrong, it is already implemented and in use for some customers since Kilo
Spice is horribly when using any GUI type OS. Postponing longer the availability of novnc is hurting customers. . Defacto it is a non free feature for us

The change to novnc is being tracked in https://github.com/rcbops/u-suk-dev/issues/131 and so this issue should no longer be required.


Refactor build.sh in openebs/maya

There are multiple components (or go binaries) being built from https://github.com/openebs/maya. Each of these component have their own build scripts located under:
#<maya-repo>/buildscripts/<component-name>/build.sh
#Example:
#<maya-repo>/buildscripts/mayactl/build.sh
#<maya-repo>/buildscripts/exporter/build.sh
#<maya-repo>/buildscripts/apiserver/build.sh

Much of the build.sh code is similar except for binary names and the main-go file.
It will be good to have a refactored single build.sh file with the binary specific values pass as parameters to the script. A common build.sh file can be located under : <maya-repo>/buildscripts/build.sh

I would like to take this one up :D

sure @salonirk11 go ahead. Let me know if you need any help.

How do I test the refactored build.sh? I am unable to find commands in the installation guide which call the three different scripts.

@salonirk11  there is GNUMakefile targets which triggers individual buils.sh. You can run
make all to run all of them in single shot or run single component using
make apiserver to build Maya-Apiserver
make exporter to build Maya volume exporter etc..

Please review PR #643 for https://github.com/openebs/maya


Error while downloading images in absence of content script

Describe the bug
The following error occurs when trying to download a photo from Behance provider:
popup.html:1 Access to XMLHttpRequest at 'https://mir-s3-cdn-cf.behance.net/project_modules/max_1200/180a6536683721.5725932eb3e24.jpg' from origin 'chrome-extension://obckdhkhgdbigeihagclikjmlioanfbb' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
See the screenshot below.
This error is shown by some other providers also and at the same time images from some providers like Flickr, Thingiverse, Museums Victoria, etc does not give any error.
The error only occurs in chrome and opera browser while the download feature works as expected on firefox. The error goes when we set a contents_scripts  field in the manifest which targets at least all https web pages. I have used this as a temporary solution with an empty content script but the problem with this approach is that the extension is shown to have access to web pages and therefore a better solution is needed instead of the current hack.
To Reproduce
Steps to reproduce the behavior:

Build the extension
Remove the content_scripts field from chrome and opera manifest.json.
Search for any image from Behance provider.
Press the download button.

Expected behavior
The image should be downloaded without any error.
Screenshots

Desktop (please complete the following informa.tion):

OS: Windows (OS Build 17763.615)
Browser: Chrome [Version 76.0.3809.100 (64-bit)], Opera [Version : 62.0.3331.116]
Version [e.g. 1.0.0]

Additional context
The extension uses download.js(npm) for downloading images.
The code which handles the downloading of image and attribution can be found in infoPopupModule.js

@makkoncept I tried reproducing this issue in Chrome but couldn't. Can you check if you are still getting this error?

@codingonHP, I am also not getting the error. This is strange. I think we should not remove the contents.js file right now because this may be temporary. Waiting for more insight into this.

@codingonHP I am getting the error again. Actually, I didn't reload the extension after removing the content_scripts field from chrome manifest.json. We need to reload the extension in chrome://extensions/ after we make the change in the manifest file.

created a PR for it :)
https://developer.chrome.com/extensions/xhr


Automatic Retry on Archive Extraction Failure

We are frequently encountering intermittent unzipping errors when using uv with AWS CodeArtifact, resulting in failed installations during uv pip install operations. These errors appear to stem from network issues, as re-running the installation command usually resolves the problem.
Example Error:
error: Failed to prepare distributions
  Caused by: Failed to download `pyarrow==18.0.0`
  Caused by: Failed to extract archive
  Caused by: unexpected BufError

For instance, NPM's handling of similar issues:
npm warn tarball tarball data for<EMAIL_ADDRESS>(sha512-yCcfVdiBFngVz9/keHin9EnsrQtQtEu3nRykNy9RVp+FiPFFbPJ3Sg6Qg4+TkmH0vMP5qsTKgXSsk80HRwvdgQ==) seems to be corrupted. Trying again.

It would be highly beneficial if uv could automatically retry the extraction process upon failure, similar to how NPM handles tarball data errors.

We do generally retry these. What version of uv are you on?

We do generally retry these. What version of uv are you on?


Added support for BigQuery relation renaming

resolves #2520
Description
Implements functionality to rename BigQuery relations using dbt.
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

Hey @azhard, thanks for contributing! This looks great, I have some minor feedback on ordering and some extra tests but this seems good.
Also, could you please add an integration test of some sort? The 054_adapter_methods test folder is probably appropriate. I'd hate to have this regress.

Thanks for the feedback @beckjake. For the integration test, can you give me some more guidance on what exactly I should be testing / how. So for example am I testing that the rename functionality works across all adapters and if that's correct, how do I "confirm" that a rename took place?
Or alternatively, am I testing the BigQuery specific rename (eg. copy and delete happen, doesn't work on views, etc.) If that's the case, I see similar testing done in test_bigquery_adapter.py file where I can utilize the mock connection so that to me is a little more clear.

So for example am I testing that the rename functionality works across all adapters and if that's correct, how do I "confirm" that a rename took place?

You can just test bigquery for this, since that's all you've added! Here's a test I imagined:

a macro (rename_named_relation or whatever) that builds a from_relation/to_relation based on some text parameters and then calls {{ rename_relation(...) }} with those relations
a seed (call it my_seed) with anything at all
a source that refers to a table named something different (renamed_seed or whatever)
a model that does select * from {{ source('my_source', 'renamed_seed') }}
a test that runs dbt seed, dbt run-operation rename_named_relation --args '{from_name: 'my_seed', to_name: 'renamed_seed'}, dbt run. None of those should error.

Does that seem reasonable?

Basically, I don't care how relations are renamed, but in my ideal world if bigquery offered "alter table rename ..." and we switched to using that, we'd probably want the test to still work.

Yeah that makes a ton of sense to me, I'll look into adding that

Oh, I didn't understand what was going wrong with the postgres tests before, but now I realize. I think you'll need to put this new model/schema.yml into a new folder (maybe bigquery-models?), and override the models property on your new test class:
    @property
    def models(self):
        return 'bigquery-models'

@beckjake updated and 🤞 hopefully it should work now. I also updated Makefile (found a missing comma) and the BigQuery fields in test.env.sample as I realized that those values aren't used anywhere. The bigquery_profile function  only looks for the BIGQUERY_SERVICE_ACCOUNT_JSON and BIGQUERY_TEST_ALT_DATABASE inside certain tests.
Let me know if you'd rather I didn't make these changes or if there's a reason they're written like that and I can revert them.

That all sounds great to me, thank you. I've kicked off the tests and we'll get this merged in for 0.18.0 once the tests pass. I don't really ever use the Makefile, so thank you for fixing that especially!


Question regarding Button "Regeneration"

Hi All,
one question regarding the button regeneration. Is it just for triggering a manual regeneration or
should it also show if an automatic regeneration is ongoing (set true by the device when regenerating)?
I wanted to use it for measuring the energy consumption (3w standard standby consumption + 20w druring regeneration for my iSoft+).
But in the last two weeks the value was not changed and I start to believe that its just for manually triggering the process (which would not make sense?)?
I am using version 1.0.15 of the adapter.
Thx and brgds,
Claus

Hi Claus1985,
I think the button regeneration is only totriggering manual regeneration.
Best regards
Brandy

@arteck is it possible to change the value into one that can be set/changed?
thx and Brgs

Thx for the info. Is it possible to implement a status which shows if the regeneration is running at the moment?

the question is .. can i see this in the api

As far as I could find out it should be possible. In the following screenshot there are two values for regeneration.
One (52) with a parameter stop and one (53) without a parameter. So i guess that 53 should be the status.

I took a closer look at data record #791.
It seems that the first hex digit after the colon is a flag that gives information about an running regeneration. 0xC0=regeneration not active, 0xC1 or 0xC3 regeneration running.
0xC1/0xC3 could provide information about which softener cartridge is currently being regenerated.


Convert Basic TypeDefs.

Judging from #95, TypeDefs (presumably basic ones)  seems to be a blocker.

Superseded by https://github.com/scalameta/paradise/pull/100


[oc-chef-pedant] Fix search tests incorrectly using with_search_polling

with_search_polling works by retrying when an exception is raised. An
empty response from search will not raise an exception, rather the
assertions on the results should also be inside the with_search_polling
block.

cc @chef/lob

:+1:


Pod on declarator with initialization

Hi All,
The attached tests are failing for me with the most recent rakudo.
My reading of S26 would be that it should be valid to have declarator pod trailing an attribute even if there is also an initialization of the attribute.
thanks
Brian

IIRC, the #= form doesn't require any enclosing brackets.

Right...they're optional -- block-trailing.t has tests with enclosing brackets, and why-trailing.t has tests without enclosing brackets.

Thanks @bduggan. I've merged it, converted the tests to use subtests, and fudged the new, failing tests for rakudo.


timeout added to http requests

In our environment, http requests to consul get sometimes stuck because of missing timeout. I duplicated original pull request by @Chimney42 to @cablehead python-consul repo.

+1

Thank you for the pr

Thank you:-) Do you plan to release new version on pypi? @poppyred

@josefflorian Yes, but I still have some problems that need to be fixed, it takes a little time。


Content width missing in 0.12.0-beta.5 version

Hi,
"Content width" missing in the settings of version 0.12.0-beta.5. It was very useful to have the graph in full screen mode.
In addition, the time and date of the last result is also missing from the dashboard
Has it been removed or will it be restored?
Thank you

In addition, the time and date of the last result is also missing from the dashboard
Has it been removed or will it be restored?

So this was done on purpose for the time being. There's an issue with Filament v3 where the method I was using to display that timestamp would cause charts to disappear: #756
Once it's fixed I'll bring back that timestamp.

@alexjustesen
Do you think you can parameterize the font size in the header and in the results table with an environment variable?
Thanks

At the moment there doesn't seem to be a native way to do that in Filament. I'll keep an eye out in case it's added but for now I'm going to say probably not.
https://filamentphp.com/docs/3.x/panels/themes/

@alexjustesen
Just one more question. What is the page where the sidebar is created? I would like to add the sidebarCollapsibleOnDesktop flag. Why did you remove it, do you plan to restore it? By collapsing the sidebar I could recover space in my 4:3 screen
Thank you

Oops, forgot about that...
Here are the docs for it, PR welcome and the file is located at app/Providers/Filament/AdminPanelProvider.php.
https://filamentphp.com/docs/3.x/panels/navigation/#collapsible-sidebar-on-desktop

@alexjustesen
It doesn't matter, if you restore it in the next release I can wait. I thought you deleted it...

They moved from config based features to methods based so it was their upgrade process that didn't pick it up and I don't use it so didn't notice

Thanks, I'll wait for the next release

@alexjustesen
Hi Alex I'll leave you a reminder for the next release.
In the dashboard, the arrows indicating the ping trend appear to be backwards. See attach Thanks

So that one is correct, lower ping is better

Hi, No, in my opinion the string must be green but the arrow points up. The upward arrow indicates a better result.
That's how it was in previous versions...
Thanks

I think I chatted about this before somewhere let me see if I can dig that up.

@corasaniti https://github.com/alexjustesen/speedtest-tracker/releases/tag/v0.12.0-beta.6 collapsible sidebar rolling out


[BUG] - Getting ambiguous name error with v1.9.0

Description
Getting the following error (and many other similar errors) when upgrading to v1.9.0:
error[E0659]: `style` is ambiguous
 --> ~/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tuirealm-1.9.0/src/utils/parser.rs:6:17
  |
6 | use crate::tui::style::Color;
  |                 ^^^^^ ambiguous name
  |
  = note: ambiguous because of multiple glob imports of a name in the same module
note: `style` could refer to the module imported here
 --> ~/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tuirealm-1.9.0/src/tui.rs:6:9
  |
6 | pub use ratatui::*;
  |         ^^^^^^^^^^
  = help: consider adding an explicit import of `style` to disambiguate
note: `style` could also refer to the module imported here
 --> ~/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tuirealm-1.9.0/src/tui.rs:8:9
  |
8 | pub use tui::*;
  |         ^^^^^^
  = help: consider adding an explicit import of `style` to disambiguate

Steps to reproduce
Just upgrade tui-realm to v1.9.0
Expected behaviour
Should not get these ambiguous name errors.
Environment

OS: MacOS ventura
Architecture [Arm]
Rust version: 1.71
tui-realm version: 1.9.0

Additional information
Add any other context about the problem here.

Check your Cargo.toml
the features have changed:
You now have to specify both the tui library tui or ratatui and the backend termion or crossterm.
tuirealm = { version = "^1.9.0", default-features = false, features = [ "ratatui", "derive", "crossterm" ]}

I thought the docs say "ratatui" is the default and I think your toml file says that too. :)
Anyhow, I just tried using tuirealm = { version = "^1.9.0", default-features = false, features = [ "ratatui", "derive", "crossterm" ]}. I still got the same errors. Is there anything else that I'd need to change?
Could you try creating a hello world project with tuirealm 1.9? :)

mhm, weird, maybe it's a cache issue. Try to clean your .cargo/registry

I did cargo cache -a and recompiled, still got the same errors. :-(

I get the same problem with this sister crate

I confirm this is an issue related to registry cache. It happens because cargo saves sources in your cargo path. Unfortunately the only way is to delete the source folder contained in the error


Convincing of bad state possible when network loads from disk

The way we persist longest_tips means we could be convinced of a bad fork in catastrophic scenario. We really need to persist the whole ledger-builder-controller transition_logic_state.

@bkase I don't remember what you had in mind here, but I think current designs for persistence+long-fork are good enough here?


Data partition vulnerable to rollbacks

(originally reported by @lorenz in T691)
We currently use dm-crypt on top of dm-integrity to provide authenticated encryption. This however leaves us vulnerable to rollbacks of individual sectors or the whole disk.
There is an extension to dm-verity called dm-x which implements a modifiable merkle tree (https://github.com/anrinch/dmx) and lets you store to root of the tree in the TPM or comparable hardware. This would make it much more robust against rollback attacks. The issue is that technically one needs to store a new hash for every commit to disk (sync call, file closed, buffer full) which would quickly destroy the nonvolatile memory of the TPM (usually some form of flash). A potential solution is the use of a hybrid index, but these will present issues when TPM2_Shutdown is not called since they are infrequently committed to NVRAM.

Still quite a bit away: we can do this either via dmx/minintegrity as above, or via bcachefs. Neither of these are mainline and/or battletested.
Let's try this later, as this space is still evolving.


Setup/Teardown takes too long

Describe the bug
The Setup and Teardown step seems to be taking a much longer time lately.
To Reproduce
Example of the time taken:
2020-04-08 08:11:47,723 - DEBUG - run - Setting up...
2020-04-08 08:13:15,338 - DEBUG - container - Checking if image "ternd:test-spdx" is available on disk...

It looks like it takes about 2 minutes to complete which is quite long for removing files and setting up directories.

@nishakm, the function call yaml.safe_load (to load cache) is taking a lot of time here. I tried to load cache using a new thread and it worked fine but then load_full_image started taking much time :(

@nishakm, the function call yaml.safe_load (to load cache) is taking a lot of time here. I tried to load cache using a new thread and it worked fine but then load_full_image started taking much time :(

This makes sense because of the serialization of data to and from the file. Implementing #50 should solve this issue.

I've decoupled solving #50 to this problem. It turns out, replacing yaml with json for the cache provides better performance and it is reasonably straightforward to fix.


card_faces() should return a CardsObject

Is this a suggestion for functionality?

Oh yeah sorry! Was going to write a description/user case but totally forgot!
In my case I first get a card with .Named then I take that object and pass it a long to some other functions that does their magic. But in the case of cards with multiple faces they don't have names and such. So before I do anything with it I have to check if it have faces, if it have faces I do things with the individual faces.
I would like to be able to use the same interface I have built for .Named for faces. Right now I am creating a dummy object that mimics the interface of a CardsObject to pass the json data to, which of course works. But seeing that a face is a card, it would be nice if it behaved similarily to one.

Hmm. I see what you're saying. I think that would involve creating an even more generic class (something like CardFace) that then gets extended by CardsObject since there's a lot of property overlap. Then when we just return CardFace when we ask for a card face we return an array of those objects or the specific object if the index is specified.
I can see all of that being helpful. I'll try and find some time to add that but I can't guarantee anything.


Can't set vm_name property within virtualbox-ovf builder

@themalkolm , should the following config work?
Setting up vm_name property within nested build. Build fails saying it can't resolve variables - build_type, timestamp, build_name and so on.
"vm_name": "pv-{{ build_type }}-{{ timestamp }}",

{
        "type": "vagrant",

        "box_provider": "virtualbox",
        "box_file": ".ovf",
        "box_name": "{{user `input_box_name` }}",

        "builder": {
            "type": "virtualbox-ovf",

            "vm_name": "pv-{{ build_type }}-{{ timestamp }}",

            "output_directory": "{{user `output_directory`}}",
            "http_directory": "{{user `http_directory`}}",

            "headless": "true",
            "boot_wait": "30s",

            ....
        }
    }

Native virtualbox-iso builder allows vm_name setup, it works well.

https://www.packer.io/docs/builders/virtualbox-iso.html#vm_name

I believe I didn’t bother to implement proper template support there. We can enable some specific variables if you want though.


Bug in EMA calculation?

Currently the EMA params are calculated as:
https://github.com/openai/guided-diffusion/blob/27c20a8fab9cb472df5d6bdd6c8d11c8f430b924/guided_diffusion/nn.py#L65
I think it should actually be swapped to:
targ.detach().mul_(1 - rate).add_(src, alpha=rate)

I looks like this is currently just taking an average of < 1 step when rate=0.9999
Source:
https://fastai.github.io/timmdocs/training_modelEMA

Ah, actually ignore me... this is not a bug, I was just mixed up between what was source params and what was target.


Support basic meta-mesh backup agents api clients

The Testnet rewards program is on hold until we can build a backup agent api client that will backup all account balances, transactions, counters and smeshing rewards. We need to make sure that the API supports such backup workflows and prioritize implementing it in API 0.2.
@lrettig - we need to focus on this and come up with methods design to support full meta-mesh backups.

What we would like to enable is an API client that can create a basic backup of the current global state at a specific layer with the data and indexes:

All accounts with non-zero balance. (k: account id, value: counter, balance}, enum of all accounts.
The rewards assigned to each account. L: account id, value: [rewards]

Note that this info is meta-mesh - e.g. not on the mesh itself. The data is all output of the STF of that layer. See the data design epic for more info on this topic.

Seems pretty straightforward. Do we already have endpoints that can do this in #16 or do we need new ones?

We might, we just need to develop a workflow for backing the meta-mesh of a network for a specific layer a recoverable / searchable way and make sure we have specified the api methods needed to support it.

Yup. That part probably doesn't belong in the "api" repository, though -
we'll probably want a separate utility to do this. (Or maybe it could be
part of spacecraft?)


Add more license files

Added more extension handlers to license_header.py. Made it verbose so we keep track of what we're whitelisting/skipping.
Added license headers to files with newly added extension headers.

scala test fails

LGTM, thanks

PR builds passed, got approval, will merge now.


I wonder how can sfu-minimal work

I download webrtc-2.1.18， and follow the Instructions of README.md like this:
I  open jsfiddle.net in one of my windows pc, ip is <IP_ADDRESS>, call it A pc
Run main.go in sfu-minimal;
In A pc chrome browser, Click Publish a Broadcast, a video view screen appears, wait about 3 minuts, Browser base64 Session Description appears: eyJ0eXBlIjoib2Zm................cclxuIn0=
cmd.exe localhost:8080/sdp -d "eyJ0eXBlIjoib2Zm................cclxuIn0="
sfu-minimal.exe response: eyJ0eXBlIjoiYW5...................0ZXNcclxuIn0
copy eyJ0eXBlIjoiYW5...................0ZXNcclxuIn0 to Golang base64 Session Description, int A pc, Then press Start Session.
Then, Logs appears:
checking
connected
In B pc( whose ip is <IP_ADDRESS>) chrome browser, Click Join a Broadcast,  a video view screen appears, wait about 1 minuts,Browser base64 Session Description appears: eyJ0eXBlIjo...............kZGU2ZVxyXG4ifQ==
cmd.exe localhost:8080/sdp -d "eyJ0eXBlIjo...............kZGU2ZVxyXG4ifQ=="
sfu-minimal.exe response: eyJ0eXBlIjoiYW5zd2Vy........................pZGF0ZXNcclxuIn0=
copy eyJ0eXBlIjoiYW5zd2Vy........................pZGF0ZXNcclxuIn0= to Golang base64 Session Description in pc B .Then press Start Session.
Then, Logs appears:
checking
disconnected
I try about 3 times, the same result. I wonder how can it work?

Hey @mabaohong
Sorry I missed this! If you are trying to use Pion and still have issues I am available to help in Slack
thanks


Cannot select word in chrome browser url bar by direction key

I found that I cannot select word when I type word in chrome browser url bar.
Reproduce steps:

key some chinese word into the chrome url bar
press down key to open the candidates
press down to select the candidate

At the step 3, chrome would select the next url recommendation rather than choosing the next candidate for me.
My chrome version: Version 74.0.3729.169 (Official Build) (64-bit)
McBopomofo version: Version 0.9.10 (805)

I can reproduce this bug with McBopomofo. From the visible changes, it seems that when an arrow key is pressed, the current buffer gets committed, and therefore McBopomofo exits the candidate-selection state.
Interestingly, Chrome also has issues with built-in input methods (I tried both Apple's Zhuyin and Japanese IME): when I pressed the down key, Chrome still select the URL recommendation. Apple's built-in input methods, though, did not exit the candidate-selection state. Therefore it is still possible to choose the right candidate. I'm not sure how Chrome behaved before this though, but it seems the current version of Chrome is over-eager with an input method's buffer is still active (i.e. not empty).
Also interestingly, OpenVanilla behaves like Apple's built-in input methods, not McBopomofo. I haven't looked at the code for a long time, but I have to guess it has to do with OV's handling with forced commit events. Perhaps OV does not exit candidate-selection state if the commit event is not from the user.
Based on all this, I encourage anyone who runs into this to file a bug with Chrome, since Apple's own built-in input methods are also affected, and the symptom is what I just described above: Chrome's URL bar is over-eager when an input method still has an active, non-empty composing (edit) buffer.

This bug also exists in Microsoft Edge Chromium. McBopomofo is not the only one which gets troubled with this bug, but also Yahoo KeyKey.

@lukhnos Which version of macOS are you using? I tested the macOS Mojave 10.14.5 built-in Japanese IME together with Google Chrome Dev 77.0.3824.6 and I found that this bug is not triggered.

Thx @lukhnos .
After I installed the newest version directly from this repo, the problem is fixed.


Fixes 404 error by explicitly setting hashMode = true

The default nuxt.options.router.options.hashMode = false.
Therefore, the previous coalescing assignment does nothing. It doesn't disable default opts as the disableDefaultOptions intends (I think?).
This explicitly sets the value to true, in order to overcome the 404 on build.

Hopefully this makes it in!


[Task]: Create Category token for Segmented control component

Discrepancies in Font & Icon color in Segmented control in main repo.
Moving to Needs Design. CC @vasanthappsmith
https://github.com/appsmithorg/appsmith/assets/104895602/28e2364e-ba3c-4b14-ab70-93036cc161e2


Unwanted Default Query Files Loading By Default

Type: Bug
On ADS launch, the software loads query files that were open on the previous exit.  I believe there was an issue with my previous upgrade.  I now can't close a set of queries that were "unsaved" (I created a new query and did not explicitly save it) when the system upgraded.  This set of files (SQLQuery_29, SQLQuery_30...etc) always open by default.  I've tried to saving them, even using "Save As", but I'm unable to prevent this set of files from loading by default every time.
Azure Data Studio version: azuredatastudio 1.46.1 (ba29842b81dec01177415e53948ca2168e69c3f8, 2023-10-02T18:14:22.887Z)
OS version: Windows_NT x64 10.0.22621
Restricted Mode: No
Preview Features: Enabled
Modes:

System Info

Item
Value

CPUs
12th Gen Intel(R) Core(TM) i7-12700H (20 x 2688)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
63.68GB (42.85GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Closing as response provided above to address user concern.


I want to add code of fibonacci series

I want to add code of Fibonacci series using recursion. Please assign it to me @codemistic

Please assign this to me I will surely fix this issue.


Formatting y tick

When formatting the y tick the sizing of the chart is not correct.  At the moment the tick label is cut off.  Notice how the dollar sign is cut off.

This might be related with https://github.com/masayuki0812/c3/issues/982
We have exactly same issue. Manually setting the padding doesn't work because the font size changes between breakpoints.

Sorry for this issue, it's a bug. I think this was fixed by this commit https://github.com/masayuki0812/c3/commit/4140ceb2fca1aae1e3da95d9984639d0980d29df and it's been released as v0.4.10-rc3. Can you try this version?
Btw, 0.4.10 is almost ready to release, so I'll let you know when it's released.

This is still an issue for me.  Would overriding the fonts in css cause an issue?

I don't think so. I tried larger font size, but it seems working.
Btw, I tested with 0.4.10-rc5, so can you try again with this version? And can you give me a fiddle for your issue?

Tested with rc5 and same issue.  Here's the fiddle.  http://jsfiddle.net/qqqptmdx/2/

Thanks for the detail. As I investigated, .c3 svg in c3.css seems the cause. If I removed this selector, it works as expected. I think .c3 svg is not needed because you're specifying #chart svg, or please use only .c3 svg. Anyway, it seems css matter.

OK.  I can live with making that change in my css.


Razor syntax highlighting broken in VSCode when using nullable operator

Describe the bug:
In VSCode, Razor syntax highlighting is broken when using nullable operator "?"
Version used:
VSCode 1.69.2
.Net 7.0.100-preview.6.22352.1
To reproduce:
Open the following in VSCode
@Param1
@Param2

@code {
    [Parameter]
    public string? Param1 { get; set; }

    [Parameter]
    public string? Param2 { get; set; }

    private string Foo()
    {
        return "bar";
    }
}

Expected behavior:
Razor syntax highlighting should work as expected when using nullable operator "?" (see screenshot 1)
Actual behavior:
Razor syntax highlighting is broken when using nullable operator "?" (see screenshot 2)
Additional context:
Screenshot 1 - Syntax highlighting working when not using nullable operator

Screenshot 2  - Syntax highlighting not working when using nullable operator

Additional report: https://github.com/dotnet/razor/issues/6144

Additional report: https://github.com/dotnet/razor/issues/7076

Additional report: https://github.com/dotnet/razor/issues/8184


Superfluous notebook?

There are currently two notebooks that are similar. 'app.ipynb' seems to be the most current. We should provide only one, called 'seg2map.ipynb'

I noticed this today too. Good idea, let's get rid of that extra notebook

app.ipynb is gone


Topic filtering by acceptance status

created capability to filter topics based on topic acceptance: all/accepted/rejected

Otherwise works well, but two small issues:
-selecting "All configurations" does not show anything
-selecting for example "Kesä 2019", and applying filter causes unhandled rejection "TypeError: Cannot read properties of undefined (reading 'email')"

Otherwise works well, but two small issues: -selecting "All configurations" does not show anything -selecting for example "Kesä 2019", and applying filter causes unhandled rejection "TypeError: Cannot read properties of undefined (reading 'email')"

Improved logic to show "all" configurations.
For Kesä 2019 the case seems to be that there is absolutely none sent emails of type "topicAccepted", which results filter to break. Will improve logic and components for displaying topics or no topics at all.