Fails when running in a deployment job

When used in a deployment job you get this message
"Checkout of repository 'MyRepoAlias' is not supported. Only 'self', 'none', or a repository alias are supported"
If specifying an inline reference it gives this instead
"The checkout step references the repository 'git://DemoProj/MyRepo@refs/head/master' which is not defined by the pipeline"
Any idea when deployment jobs will support this feature?

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

ID: 8a0499c0-bd1c-47a0-b472-54a5d6c159b4
Version Independent ID: ec3feae0-4f23-68ab-6307-75fde645468a
Content: Check out multiple repositories in your pipeline - Azure Pipelines
Content Source: docs/pipelines/repos/multi-repo-checkout.md
Product: devops
Technology: devops-cicd
GitHub Login: @steved0x
Microsoft Alias: sdanie

Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

Also, the product teams aren’t always watching this repository, so to make sure it gets in front of them for triage, please submit your idea here.

Currently investigating, apologies for the inconveniences. As a workaround please turn all of the jobs with checkout steps to be 'regular' jobs by calling them as - job: instead of - deployment:
https://docs.microsoft.com/en-us/azure/devops/pipelines/process/phases?view=azure-devops&tabs=yaml#specifying-jobs

@JaDevGuy - Thanks for reporting this issue. I created a bug and I have a fix that should get in this week, but because of the holidays, it won't start rolling out until January sometime. Sorry for the oversight.

Thanks @jpricket ! I'll standby for the fix and test again once it's ready. Appreciate the quick turnaround!

@JaDevGuy - Thanks for reporting this issue. I created a bug and I have a fix that should get in this week, but because of the holidays, it won't start rolling out until January sometime. Sorry for the oversight.

any idea when in january it's rolling out?

The fix has rolled out to our first ring already. It should be completely rolled out in the next 3 weeks.

The fix has rolled out to our first ring already. It should be completely rolled out in the next 3 weeks.

@jpricket I've recently noticed some unexpected logic failures in some of our repos that all depend on some standard templates, and it's related to the value of the built-in variables['build.sourcebranch'].
In particular, this variable previously returned refs/head/master, but now it returns refs/heads/master (the additional s on heads).
I'm curious if that is related to this fix, or if this was part of another change, and if there is any documentation of this (breaking, for us) change we can refer to in documenting this change?

@JoeBrockhaus I don't think that is related to my changes, but refs/heads/ is the correct prefix. This variable should come directly from the server. The agent just passes it along.
@leantk Do you know of any changes in the build.sourcebranch variable lately?

I'm running into this issue with windows-latest using deployment job
Code looks like
main.yml
resources:
  repositories:
  - repository: adf_publush
    type: git
    name: my_data
    ref: adf_publish

stages:

- template: deploy.yml
  parameters:
    varPath: config/common/variables.yml
    envVarPath: config/dev/variables.yml
    environment: dev

deploy.yml
parameters:
  - name: varPath
    type: string
    displayName: "varPath: The path to the common variable template"
  - name: envVarPath
    type: string
    displayName: "varPath: The path to the common variable template"
  - name: environment
    type: string
    displayName: "environment: The name of the environment to deploy"

stages:
- stage: deploy_${{parameters.environment}}
  displayName: ${{parameters.environment}} Deploy
  variables:
  - template: ${{parameters.varPath}}
  - template: ${{parameters.envVarPath}}
  jobs:
  - template: ./templates/adf_pre_deploy.yml
    parameters:
      environment: ${{parameters.environment}}
      agent_pool: $(agent_pool)

adf_pre_deploy.yml
parameters:
  - name: environment
    type: string
    displayName: "environment: The name of the environment to deploy"
  - name: agent_pool
    type: string
  - name: adfPublishBranch
    type: string
    default: adf_publish

jobs:
- deployment: deploy
  displayName: ${{parameters.environment}} deploy
  environment: ${{parameters.environment}}
  pool:
    vmImage: ${{parameters.agent_pool}}
  strategy:
    runOnce:
      deploy:
        steps:
        - checkout: self
          persistCredentials: true
        - checkout: ${{parameters.adfPublishBranch}}

and I receive


Feature Request: remove "= abap_true" when calling boolean methods

Example:
      IF zcl_m_stock_transfer_order=>is_no_confirmation_necessary( i_reswk = i_line->supplyingPlant
                                                                   i_emwrk = i_line->ReceivingPlant ) = abap_true.

should convert to
      IF zcl_m_stock_transfer_order=>is_no_confirmation_necessary( i_reswk = i_line->supplyingPlant
                                                                   i_emwrk = i_line->ReceivingPlant ).

and maybe replace "= abap_false" in the same context with prefix NOT

Please note that the ABAP Cleaner can only see the code that is open in the current editor. This rule could therefore only work for method definitions that exist in the current editor.

Ah, that reduces the usefulness (substantially, but not to zero)


Get active queue name

Sounds simple enough, a way to get the currently running queue name.

Sorry for the delay in response - been a busy month.  What if there are multiple queues running - does it return an array of active queue names?  Just curious, what's the use case for which you need this?

Use case for me was primarily debugging (I solved my problem in another
was).
But I can also imagine that one would like to check wheter a certain queue
is running if there is a possibility that multiple are running.
So someting like:
If($.ajaxq.IsRunning("myQueue")){
//do something
} else if($.ajaxq.IsRunning("myOtherQueue")){
//do some else
}
Op 14 jan. 2016 17:01 schreef "Dustin Smith"<EMAIL_ADDRESS>
Sorry for the delay in response - been a busy month. What if there are
multiple queues running - does it return an array of active queue names?
Just curious, what's the use case for which you need this?
—
Reply to this email directly or view it on GitHub
https://github.com/Foliotek/AjaxQ/issues/25#issuecomment-171682788.


[RFC] Add Secure Container (SES/LavaMoat)

How do you envision this feature/change to look/work like?
Maybe plasmo build --lavamoat or plasmo build --secured or plasmo build --ses?
We can either:

make a lavamoat optimizer OR a lavamoat stream (if their bundler has a stream interface)
inject the lavamoat dependency checker at the resolver level
inject the lavamoat ses wrapper at the transformer level <- this requires most work unit

What is the purpose of this change/feature? Why?
For extension that requires high degree of dependency integrity and sandboxing, lavamoat helps a lot with supply chain atk:
https://github.com/LavaMoat/lavamoat#readme
(OPTIONAL) Example implementations
https://github.com/LavaMoat/lavamoat#readme
https://www.npmjs.com/package/@lavamoat/lavapack
(OPTIONAL) Contribution

[ ] I would like to contribute to this RFC via a PR

Verify canary release

[X] I verified that the issue exists in plasmo canary release

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

The manual integration with SES is simply to import it and call the lockdown function at the top of the entry file.
That should work. Also SES remote integration is not possible since extension cannot run remote code.

https://github.com/LavaMoat/LavaMoat/blob/main/packages/browserify/examples/01-simple-js/README.md


fix(webpack-dev-server) : enable webpack to rebuild changes in node_modules by default

Issue: https://github.com/storybooks/storybook/issues/5572
What I did
I removed the "ignore: /node_modules/" from watchOptions.
Webpack should be able to rebuild changes inside node_modules by default
Even on very huge projects, I never had any performance issue, so I don't see the point of ignoring the node_modules.
How to test
You have to run storybook, and changea file inside the node_modules folder.
Webpack should be able to see the change, and rebuild accordingly.

Is this testable with Jest or Chromatic screenshots?
I don't think so
Does this need a new example in the kitchen sink apps?
No idea
Does this need an update to the documentation?
No

If your answer is yes to any of these, please make sure to include it in your PR.

How can I make the checks successful ?

@rgranger don't worry about the checks 😖

@shilman Yes, i'm currently working on a very huge project (~40 Mo in dev), and I never had performance issues. The initial building is a bit long, but the watch rebuild is pretty fast.
I need this because i'm working with private dependencies, and thus i'm constantly modifying them in node_modules (while being npm linked or not).

@rgranger you should be able to override this in "full control mode" right? Have you tried that?

@shilman Yes I did override the option in full control mode like this :
export default (baseConfig) => {
  baseConfig.watchOptions = {
    ignored: false,
  }

  return baseConfig
}

But It took me times to figure this out when I upgraded storybook from previous major version.
And it doesn't feel quite right to override this options to set it to false...
@ndelangen Here are the results of webpack build times on my huge project :
Webpack build / rebuild times
Before (watchOptions: ignored node_modules)

build : ~ 25s
rebuild : ~ 1.7s

After (watchOptions: ignored false)

build : ~ 26s
rebuild : ~ 1.8s

Oh, that's probably the reason why WatchMissingNodeModulesPlugin didn't work for me

Seems good to me. Unfortunately, I won't have any time soon to push another commit

related: https://github.com/storybooks/storybook/pull/6566

wait with merge until 5.1 has been released

Related:
https://webpack.js.org/configuration/watch/


[Core feature request] Actions in subdirectories

What feature do you need?
Being able to consume an action that has its action.yml not in repository root, but in some (sub)directory instead.
Do you have an example usage?
See e.g. https://github.com/actions/cache/tree/main/restore - there's an action in restore directory.
Then it's used in the workflow YAML like this:
- uses: actions/cache/restore@v3

Is there a workaround for not having this feature? If yes, please describe it.
Maybe somehow using _customArguments, but it would look pretty nasty and definitely type-unsafe.

In wrapper-generator, it should be modeled as subdirectories after the version. It mimics the layout of the action's repository:
actions/actions/cache/v3/restore/action-types.yml
actions/actions/cache/v3/restore/commit-hash.txt

@Vampire requested support for actions/cache/restore@v3 and actions/cache/save@v3: https://kotlinlang.slack.com/archives/C02UUATR7RC/p1674580678412759
@madhead requested support for superfly/flyctl-actions/setup-flyctl@v1 in https://github.com/krzema12/github-workflows-kt/issues/595

Generate a nested class inside the root class, like: CacheV3.Restore. Remember about a case where an action doesn't have a "parent" action. It will require grouping by parent action inside the wrapper generator to generate a single class for such group.


Update all the structures, body, and responses as per the comments

closed as changes already implemented in api swagger


Fix to Nexus deploy naming

There was an issue with deploying the sonatype-nexus chart using helm template redhat-cop/sonatype-nexus --version 0.0.5 --set=nameOverride=nexus --set=fullnameOverride=nexus | oc apply -f- as the configure job which is run fails because the app label was incorrect.
This PR variablises the app label to enable the configure job to work when the label is not app: sonatype-nexus.
The nexus.name variable doesn't seem to be picked up from the _helpers.tpl file so I've just used the same defaulting. If anyone with more helm knowledge knows any better way to do this I'll be happy to change.

Looks like linting has failed, but the action doesn't seem to suggest where. Has an image dependency changed which has broken the linting pipeline?

@Tompage1994 you need to bump the version, that's why linting is failing (that's normally why it fails anyways!)

@Tompage1994 you need to bump the version, that's why linting is failing (that's normally why it fails anyways!)

Ah magic. Is it worth creating a CONTRIBUTING.md in this repo with that kind of info?

pull request welcome ;) yeah .. the releaser plugin doesn't bump the Chart version for you! how annoying ...
LGTM.


Session-based recommendation

Hello,  Recently, I am trying to implement paper Recurrent Collaborative Filtering for Unifying General and Sequential Recommender with pytorch. By accident I found this project, which is very cool.
However, I have some questions about ImplicitSequenceModel. Paper
Session-based Recommendations with Recurrent Neural Networks
didn't refer to any methods like user_representation. Besides, that paper used GRU,  this project used LSTM instead.There are many technologies such as mini_batch, output_sample..etc in that paper. I think the implementation of this model is completely different from that paper.
I am a newcomer in the field of Session-based recommendation systems.
I wondered whether you  can explain it to me?
Thank you.
@maciejkula

Yes, it is different! But the principles are very similar.


LORAWAN Devicetree sx1262 setup on rak4631_nrf52840 board

Configured dio1-gpios on rak4631_nrf52840 devicetree leads to tx timeouts
Hi all, I've been trying to properly use the zephyr rtos with lorawan on my rak4631 development board for the past days but got stucked on the fact that I could send join requests but always got the error message lorawan: MlmeConfirm failed : Tx timeout.
Here are more info about my setup:

Platform: Wiscore Rak4631 based on nrf52840
Running Zephyr sample subsys/lorawan/class_a
Using VSCode and Platformio as IDE
Debugging with a FTDI232H using Openocd and configuration file board/nordic_nrf52_ftx232.cfg
Ubuntu

Impact
sx1262 lora modem does not work as expected;
Possible solution
After some research, found that the polarity defined at devicetree for this board is inverted. The below code on the file board/arm/rak4631_nrf52840 seems to solve the problem:
&spi1 {
	compatible = "nordic,nrf-spi";
	status = "okay";
	sck-pin = <43>;
	mosi-pin = <44>;
	miso-pin = <45>;
	cs-gpios = <&gpio1 10 GPIO_ACTIVE_LOW>;

	lora: sx1262@0 {
		compatible = "semtech,sx1262";
		reg = <0>;
		label = "sx1262";
		reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
		busy-gpios = <&gpio1 14 GPIO_ACTIVE_HIGH>;
		tx-enable-gpios = <&gpio1 7 GPIO_ACTIVE_LOW>;
		rx-enable-gpios = <&gpio1 5 GPIO_ACTIVE_LOW>;
		dio1-gpios = <&gpio1 15 (GPIO_PULL_DOWN | GPIO_ACTIVE_HIGH)>; // dio1-gpios = <&gpio1 15 GPIO_ACTIVE_LOW>;
		dio2-tx-enable;
		dio3-tcxo-voltage = <SX126X_DIO3_TCXO_3V3>;
		tcxo-power-startup-delay-ms = <5>;
		spi-max-frequency = <1000000>;
	};
};

I don't know if that is the correct way to inform/suggest this correction (sorry if not) but I believe it could help someone in the future not to waste some time to find it out.

Hi @nilmonto
Could you show us the inverted polarity line please? Are you talking about mosi/miso pinout?
Thanks

Hi @nilmonto
Could you show us the inverted polarity line please? Are you talking about mosi/miso pinout?
Thanks

Its about the dio1-gpios configuration, as it is on repository (&gpio1 15 GPIO_ACTIVE_LOW) the driver keeps giving tx timeout as mentioned. After some trials, I found that the configuration of (GPIO_PULL_DOWN | GPIO_ACTIVE_HIGH) is the correct one for the same gpio1 15 pin., so maybe its just a matter of gpio configuration on devicetree file.
Looking at SX1262 datasheet, this dio1 pin refers to a gpio, and seems to be used by chipset firmware as a loopback to zephyr loramac driver, not sure as I am new to the framework.. If anyone with this development kit could do some test to verify the sugested change on .dts it would be nice..

According to SX1262 datasheet you are right : dio (irq) is active high (not low as written in initial dts file)
Based on sx127x, GPIO pull down is required:

Will you open a pull request with your modification?


Rolling back earlier changes

An earlier change had broken some links in the QA guidance. Rolling back those changes.

Fixes #6


Support zIndex in Theme

Clear and concise description of the problem
Both windi and tailwind support this in theme, however there's no such option in uno, and I can't find any relevant posts
https://windicss.org/utilities/layout/positioning.html#z-index
https://tailwindcss.com/docs/z-index#customizing-your-theme
Suggested solution
Add support of zIndex to theme
Alternative
No response
Additional context

Validations

[X] Read the Contributing Guidelines.
[X] Read the README.md of using the package.
[X] Already used the Interactive Docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

HI, do you want use zIndex in plugins ? or what ?

@chz I'll give an example here as how I use it before the migration.
In settings I have
zIndex: {
  'home-header-nav': '4999',
  header: '5000',
  'select-dropdown': '10000',
  modal: '20000',
  'tooltip': '1000000'
}

Then in code, I can have something like
<header className="z-header" /> // will be `z-index: 5000;`
<div className="z-modal" /> // will be `z-index: 20000;`

Thus I can manage all z-index in a single place.

You can use shortcuts instead if desired

@Dunqing Yeah this could be a last resort for me... but since this is the feature supported by both Windi and Tailwind, which is pretty common, so It would be really good if it is built-in by default.

@Dunqing Yeah this could be a last resort for me... but since this is the feature supported by both Windi and Tailwind, which is pretty common, so It would be really good if it is built-in by default

I think there is no need to add to theme. You can define via shortcuts and this resolve issue.

@chz No I don't think shortcuts should be the right place to go.

Lots of users are coming from tailwind / windi and they are used to it.

z-index all in one places is a pretty common pattern and easy to manage

If we define tons of z-index in shortcuts, then what's the purpose of a theme?

Image in the shortcuts we can also do

{
  'shadow-shallow': '0 5px 16px 0 rgba(112, 144, 176, .2)',
  'shadow-header': '0 3px 8px 0 rgba(29, 29, 29, .03)'
}

then what's the purpose of the boxShadow field in the theme? we can eliminate it totally right? and the same logic applies to zIndex IMO.

I think there is no need to add to theme.

Uno is already very flexible, although I don't really want to put it in the theme, but we should consider aligning the tailwind/windi theme configuration.
I will send a PR to try to resolve it.


spanner: Expand test coverage for ResultSets

ResultSets coverage now at 99%.

Marking this WIP until DML changes are in.

@pongad Merged DML changes. Turns out there is no impact because the getStats() that DML changed is in a different class than the one in this test.
This PR is ready to go.

We're transitioning ownership of this repo and I don't have write access anymore. @chingor13 @JesseLovelace can you help?

Does this need to go out with the spanner DML feature or can it wait until after the release is completed?

It can wait until after the release. It's unrelated to DML.
On Tue, Oct 2, 2018 at 12:55 PM Jeff Ching<EMAIL_ADDRESS>wrote:

Does this need to go out with the spanner DML feature or can it wait until
after the release is completed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googleapis/google-cloud-java/pull/3717#issuecomment-426408287,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHP1nUPWg6ajB6INsBhcVRFNOqWbDSiks5ug8SkgaJpZM4W0Z2k
.

SGTM


[FLINK-16992][table-common] Add all ability interfaces for table sources and sinks

What is the purpose of the change
This adds all ability interfaces that are either supported already or mentioned
in FLIP-95. This commit makes sure that all interfaces follow a common naming scheme
like "SupportsXXX", "applyXXX()" and good documentation with consistent terminology.
Brief change log
Adds all ability interfaces with good documentation and examples.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? JavaDocs

CC @wuchong @dawidwys

Thanks everyone for the feedback. I hope I could clarify the mentioned topics.

Thanks @dawidwys. I will merge this once the build is green.


tf.argmax docs don't say which argument is deprecated

System information

TensorFlow version: 1.11
Doc Link: https://www.tensorflow.org/api_docs/python/tf/argmax

Describe the documentation issue
The doc says: "SOME ARGUMENTS ARE DEPRECATED. They will be removed in a future version. Instructions for updating: Use the axis argument instead"
I believe both the dimension and output_type args are deprecated? The update instructions only mention the to, but not the from. This may be a problem with the deprecated_args decorator.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?


False Positive TLEs

Expected Behavior
Timings and TLEs are correct
Current Behavior
Timings and TLEs are often wrong (false positive)
Suggested Fix
Call all executions with time. Parse output to get CPU time.

Fixed in 3aa8200


The dependency 'go.opentelemetry.io/otel/metric/number' is broken

Describe the bug
Execute go mod tidy on  dev branch is failed. It seems like that a third-party depencency is broken.
$ go mod tidy
go: finding module for package go.opentelemetry.io/otel/metric/number
go: finding module for package go.opentelemetry.io/otel/metric/sdkapi
github.com/AsynkronIT/protoactor-go/actor imports
	go.opentelemetry.io/otel/exporters/prometheus imports
	go.opentelemetry.io/otel/metric/number: module go.opentelemetry.io/otel/metric@latest found (v0.28.0), but does not contain package go.opentelemetry.io/otel/metric/number
github.com/AsynkronIT/protoactor-go/actor imports
	go.opentelemetry.io/otel/exporters/prometheus imports
	go.opentelemetry.io/otel/metric/sdkapi: module go.opentelemetry.io/otel/metric@latest found (v0.28.0), but does not contain package go.opentelemetry.io/otel/metric/sdkapi

My Envs

go1.17 and go1.18

To Reproduce
$ git clone https://github.com/asynkron/protoactor-go
$ cd ./protoactor-go
$ go mod tidy

fixed


[docs] Changing the Jump Address of SparkLoad in BrokerLoad

Changing the Jump Address of SparkLoad in BrokerLoad
Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

the PR title format for the document class is typo, please change it


Form Builder - There is no option to change the emailSubjectEn/emailSubjectFr value which is used when we get a form response by email

Maybe this could be a reason to revisit the way we want the form response to look like in our email template. We could use the form title instead of emailSubject.

@srtalbot This was flagged as Severe. What would be the implementation we would like to move forward with?
Either we add the possibility to change the emailSubjectEn and emailSubjectFr in the form builder or we change the code that produces the email content so that it uses the form title (En/Fr).
Could also be seen as a temporary solution. In this case we would keep both emailSubjectEn/Fr properties in place in order to use them in the future.

A decision has been taken on Slack. We are going to set both emailSubject properties to form_title Response.
This can be done achieved through he form builder code.


Integrate CodSpeed benchmarks with github actions

Hey @samuelcolvin
Finally, it's ready 🤓
There is just a small step required on your end to make it run.
I send you the details via email.

Codecov Report

Merging #328 (b7c6a23) into main (463ed7a) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #328   +/-   ##
=======================================
  Coverage   97.35%   97.35%
=======================================
  Files          57       57
  Lines        6890     6890
  Branches       46       46
=======================================
  Hits         6708     6708
  Misses        182      182

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 463ed7a...b7c6a23. Read the comment docs.

I've set the repo secret to the value from my CodSpeed dashboard, starting fb3c... and re-run CI, but it look like it's returning the same "Error: Input required and not supplied: token", any ideas?

Hmm, maybe it's because the PR has been created from my fork? I had kind of unexpected behavior in those cases.
I think the token is not even gathered by GitHub actions actually:

Correct, secrets are not available in pull requests from forks. Otherwise anyone could create a PR which used the PYPI_TOKEN  and print it (would have to print it reversed to get around github's obfuscation of secrets, but still) which would mean secrets were not secure.
The best solution is to use ${{ secrets.GITHUB_TOKEN }} which is available on all actions and doesn't require the admin to set a new secret.

@art049 this is working when I create the PR, see #332. This is really exciting.
I'm going to close this PR, and merge #332.


function stubs now return a copy of the input dataset

The PR just changes the outputs of unwritten functions (stubs) to be a copy of the input dataset. This will help with recipe development/testing.

merged! thanks!


fix(gatsby-plugin-nginx): remove trailing slash from incoming requests

What's the purpose of this pull request?
Fix an issue with headers not being properly set when a requested page has an exact match, but the request URL contains a trailing slash.
Removing the trailing slash, however, introduced a problem where locations like /graphql/(.*) wouldn't work without the trailing slash. So we convert the regex generated to allow the trailing slash to be omitted.
How it works?

How to test it?

base.store Deploy Preview

References

preview using this version https://sfj-446d5ed--base.preview.vtex.app/
and passing checks https://github.com/vtex-sites/base.store/runs/5145467188


DBZ-8043 Support FLOAT32 type

Spanner now supports the FLOAT32 type. This change creates a mapping between "FLOAT32" / "REAL" to Kafka's FLOAT32_SCHEMA

Hi @arawind, thanks for your first contribution :tada:.  Would you mind also opening a PR in the main https://github.com/debezium/debezium repository with the same issue tag, adding the FLOAT32 data type to the documentation?  Thanks!

Thanks for the quick response @Naros! I've raised https://github.com/debezium/debezium/pull/5668 to update the documentation.

@arawind Applied, thanks!

Hi @jpechane the pull request wasn't quite ready yet :)
@arawind can you submit a follow-up PR to add integration tests for FLOAT32? Thanks!

@nancyxu123 I guess next time we need better use draft functionality. But TBH you're the one who convinced me this is done by asking me for the review :-).

Sorry for the confusion! I'm working on the integration test. I'll create a follow up PR with the test


Describe command not found in DPAPI module

Hey :)
The "describe" subkey didn't work anymore in DPAPI section, whereas it's in cmdhelper file... It's really strange

I was an old version, corrected in 0.6.3 ;)


Add CAN transport flag to cmake / Upgrade splog version

Related to https://github.com/eProsima/Micro-XRCE-DDS-Agent/pull/294

CI: https://github.com/eProsima/Micro-XRCE-DDS/pull/127

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows

With the changes in https://github.com/micro-ROS/micro-ROS-Agent/pull/110 my builds pass on macOS 👍.


Task - comprehensive specimen - map

Map should show show 3 things:

Geographic area shape
specimen dot.
Error radius or error shape (if both available, the error shape should have a preference)

Right.  We are waiting for new map stuff to do this.

Find mechanism or tile layer that allows English (vs, say, Arabic) legending.


TN-1752 protect /api/report/questionnaire_status from unauthorized views

now staff can only view patients for which a like consent is in place.  tests included.

Hello @pbugni! Thanks for submitting the PR.

There are no PEP8 issues in the file portal/views/reporting.py !

There are no PEP8 issues in the file tests/test_questionnaire_bank.py !

There are no PEP8 issues in the file tests/test_reporting.py !


minor sort_values housekeeping

[ ] ~Closes #xxxx~
[ ] ~Tests added / passed~
[x] Passes black dask / flake8 dask

a few mostly-cosmetic follow-ons to sort_values / #7286

I'm going to go ahead and merge this actually, but @ryan-williams if you feel inspired to keep tweaking please go for it!

thanks @jsignell, I'll try to do another PR for that (also will try hand at Series.sort_values if I have time)


Fix enum pattern matching

When using is pattern matching with enums, the enum value is dropped. For example:
_ = System.Data.ConnectionState.Closed is System.Data.ConnectionState.Closed;

Gets converted to:
_ = System.Data.ConnectionState.Closed is global::System.Data.ConnectionState;

This PR checks if the type is an enum and doesn't match the base typeSyntax (because the right-side is missing) and then adds it back.

Thanks a lot for the PR.
I expanded your test case to cover using static class.
The change is in the latest nuget package.


Help Wanted – Troubleshooting Tips, Tricks, and Strategies

As you know, troubleshooting problems with IAM and its many moving parts can be a challenge. What issues have you run into and solved? Can you share your solution with your peers? Please share only problems for which you've found a solution. If you're still struggling with a problem, please contact Customer Support or post your issue on the AWS Forums.
Share the following information with the IAM doc team as a response to this issue, and we'll consider adding it to the troubleshooting section of the AWS IAM documentation.

What were the symptoms of the problem?
What did you do to troubleshoot?
What did you try that didn't help?
What led you to the correct root cause?
What did you do to correct the problem?

On behalf of all of your colleagues who run into the same problems, we thank you for your participation.
Stephenie Swope
Senior Technical Writer
AWS Identity and Access Management
Carla Spoon
Technical Writer
AWS Identity and Access Management and AWS Organizations
https://docs.aws.amazon.com/iam/latest/userguide/

As a consultant brought in to assess the security posture of an AWS account, I wanted to identify who all in an account is admin.  I built the find_admins command into the open-source CloudMapper to do this.  Explanation at https://summitroute.com/blog/2018/06/12/cloudmapper_find_admins/
In order to help companies with their least privileges, I built the open-source CloudTracker.  See https://duo.com/blog/introducing-cloudtracker-an-aws-cloudtrail-log-analyzer
When trying to identify the least privileges for a new service, the common solution is to provide more privileges than are needed, and then to review CloudTrail logs to identify the actual privileges needed.  This is partially because there are naming differences between API names -> IAM privilege names -> CloudTrail names, which makes figuring out the IAM privileges difficult.  For example the IAM service SES is recorded in CloudTrail as email.  Or the IAM privilege s3:listallmybuckets is recorded in CloudTrail as 's3:listbuckets'.  Some API calls are also not recorded in CloudTrail, making this approach difficult. These issues have been recorded as open issues to the CloudTrail team at https://github.com/awsdocs/aws-cloudtrail-user-guide/issues/2 and https://github.com/awsdocs/aws-cloudtrail-user-guide/issues/1

This is a pretty rookie issue but it might help someone anyway.
I had an issue that looked like I had either misconfigured my role's permissions, or I wasn't using the role correctly in my application (C++ SDK.) I was getting "access denied" on an attempted S3 upload. I had an Aws::Auth::AWSCredentials and I could get the key and secret key from that, but there was no obvious way to associate that information with the actual name of the role or query what permissions it had.
I spent a lot of time staring at my C++ code, the source code for the AWS C++ SDK, and my IAM policies. I also tried setting the AWS_ACCESS_KEY and AWS_SECRET_ACCESS_KEY environment variables to the values from my Aws::Auth::AWSCredentials and then running AWS CLI S3 commands, but I wasn't able to learn anything that way either.
It turns out that the right way to figure out what role I was actually getting credentials for was to look in CloudTrail. I quickly realized that I was using the wrong IAM user, because I was able to associate the access key from my AWSCredentials with an actual user name. So it would be helpful if the "Troubleshooting IAM" section of the user guide would mention CloudTrail somewhere, since that seems like a helpful way to see what your code is actually trying to do with IAM.

I described the more confusing aspects of IAM's naming and how it relates to the API names and CloudTrail here: https://summitroute.com/blog/2018/06/28/aws_iam_vs_api_vs_cloudtrail/

@MaxEliaserAWS for your issue, aws sts get-caller-identity is one of the most important API calls that exists.  Not only will it tell you what user/role you are using for your credentials, but it is also one of the few API calls that cannot be denied, so it always works as long as the credentials are valid, no matter what IAM privileges have been denied or allowed.

Thank you, @0xdabbad00. @MaxEliaserAWS, @0xdabbad00 is correct that get-caller-identity would be helpful in your situation. For more information, see the documentation here: https://docs.aws.amazon.com/cli/latest/reference/sts/get-caller-identity.html.
Also, I have logged your request for adding CloudTrail info to IAM troubleshooting. Good idea! Thanks for your feedback.

We're working to close older GitHub issues for the IAM User Guide, and we appreciate the suggestions that were made here previously. We added section to the IAM troubleshooting section to mention Cloud Trail: https://github.com/awsdocs/iam-user-guide/blob/main/doc_source/troubleshoot_general.md#additional-resources.


Python code for cal_quality_assessment.m?

Any equivalent python code for this evaluation matlab file?
Run cal_quality_assessment.m

Unfortunately, the computation accuracy of Python is much lower than MATLAB. So we do not use Python to do that.


Windows: [TP4] Fix regression on second or later container starts

Signed-off-by: John Howard<EMAIL_ADDRESS>It looks like https://github.com/docker/docker/pull/16370/files, and more likely a PR before it, as the code removed in that PR was dead, somehow stopped the daemon saving the first boot optimization once the container has been started for the first time. I need to go back in history further to find the original PR which broke it. The key is that HasBeenStartBefore is no longer being updated in the container config.
The net effect of this is that the daemon always tells HCS to ignore flushes during boot, regardless of whether a container has been started before. This could cause potential data loss on second and subsequent starts of a container.

LGTM

@jhowardmsft I understand this is needed for TP4, but doesn't this also affect linux and thus needs to be cherrypicked for 1.9.1?
LGTM

@Tiborvass - no it doesn't affect Linux. This is a Windows specific optimisation I introduced in docker. Not used on Linux.


Improve CloseColumnsTransformer

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (fa49895) to head (3fa9709).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #390   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           58        58
  Lines         1509      1511    +2
  Branches        59        59
=========================================
+ Hits          1509      1511    +2

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


Initial version of library and bflc vocabulary

Please add the new file(s) to the list here so they can be validated:
https://github.com/ebsco/builde/blob/main/package.json#L3
You can setup your environment to run this check prior to committing/pushing. Eventually we will setup a CI pipeline which runs this. The validator tool doesn't support wildcards so we have to pass each file in by name.


Add UI for re-ordering services

@adamcooke Can you explain a little more as to how you intended this to work? I wouldn't mind submitting a PR for such functionality.

The Service model has a field called position which is used to set the order that services are displayed in the interface. Unfortunately, this attribute isn't exposed anywhere.
I think I was just thinking of adding a simple JQuery UI sortable widget which would let you just drag and drop the order of services on the service list page.
I'd pop a form around the whole table, put a hidden field  - hidden_field_tag('order[]', service.id) - on each table row and then automatically submit the form when the order was changed. A controller action will receive the order param array with the IDs in order and that can be passed to the model...
Finally, I would likely add a class method to Service called reorder which accepts an array of IDs in the which they should be displayed and update all the services with their new positions.
def self.reorder(ids)
  ids.map(&:to_i).each_with_index do |id, index|
    service = Service.find(id)
    service.update(:position => index + 1)
  end
end

👍I was thinking the same. I'll try and submit this unless you or anyone else beats me to it.

Did this get anywhere?  Looks like it went dead but i think its an important feature so that messing with the database in the background is not required.

+1 on the feature, it looks like the internals already allow for this to work so it would be a nice to have.

+1

Any news about reorder? I need sorting inside "service group". How I can do this?


Collapsible Groups for ListBox items

Describe the problem
Mod content additions will make the player's List Boxes (for Mutators, Traits, etc.) crowded and difficult to traverse.
Describe the solution
Create a system that allows the modder to assign their content to Menu Groups. If RogueLibs detects a Menu Group, it will create a collapsible/expandable menu group. The ones used in AToM were ideal.
Describe the alternatives
No response

RogueLibs v3 will no longer be updated. Reopen the issue if the issue persists in v4+.


⚠ Bofrost.fr has degraded performance

In f43a376, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 959 ms

Resolved: Bofrost.fr performance has improved in fefb8fe after 13 minutes.


Show "updated date" instead of creation date

Show "updated date" instead of creation date
Hi there,
thank you for the amazing theme. It looks beautiful and works well!
I just started with my own website and blog.
Summary
Is there a way to display the "Last updated" date instead of the creation date on the start-page, atom-page and blog-page?
Motivation
I want to add new things to my posts over time. If its just an addition to an existing topic I don't want to create a new blog post.
Right now the change is only visible to a website visitor if he opens the post and sees the "Last updated".

Hey Sam!
Thanks for the kind words~
Currently there's no way to add the last updated date. It wouldn't be too hard to add, though. Do you picture it below the creation date? Or showing one instead of the other?

Hm, good question... I'm just fiddling with it and have replaced it for now.
But making it similar to the view under the blog title on the blog page might be good too? :smile:
Right now I've set my own entry in the config.toml: prefer_updated_date
and use it in the list_posts.html macro:

I'm thinking we could add a variable post_listing_date_shown (or another descriptive name) with three options:

date
update (or updated?)
both

The logic to show either/both should be easy to implement.
Thoughts?

I think that's a good idea. That way, everyone can decide for themselves what they want.

Great!
Would you like to work on this? It needn't be a complete PR, but anything helps!

Yes, I'll give it a try!
Von: Óscar @.>
An: welpo/tabi @.>
Kopie: sam @.>; Author @.**>
*Datum: *12.06.2024 01:21:30
*Betreff: *Re: [welpo/tabi] Show "updated date" instead of creation date (Issue #326)

Great!
Would you like to work on this? It needn't be a complete PR, but anything helps!
—
Reply to this email directly, view it on GitHub[https://github.com/welpo/tabi/issues/326#issuecomment-2161759517], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AC6OX3V4KIHUVYLQH2MH4M3ZG6A7JAVCNFSM6AAAAABJEWTRFKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRRG42TSNJRG4].
You are receiving this because you authored the thread.
[Verfolgungsbild][https://github.com/notifications/beacon/AC6OX3QCYNSJ64IWMATATCDZG6A7JA5CNFSM6AAAAABJEWTRFKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUA3HKR2.gif]

Awesome! Don't hesitate to reach out with questions or to create a semi-complete PR~

Added in #330.


Let the user configure their default editor

add editor to the config file
Continuation of #270 and #271

Somehow editing info opens pico and editing script opens code now.


fix: Using named capture groups for the regex

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Start Writing page is not scrolling

I request to UI changes in start writing page as it is not scrolling up

#1134 is same


correct template lookup on Windows:

posixpath.join does not handle windows paths very well.

Now all tests pass on py27 & py34 on windows.

hm OK where were you seeing failing tests, py34 + windows only?

Both py27 and py34. The assert fails as the wrong exception is raised.

I do not have POSIX system handy and there is no Travis integration so I could not test on such a system. Would a Travis integration be welcome?

just this one right?
================================== FAILURES ===================================
____________ LookupTest.test_dont_accept_relative_outside_of_root _____________
Traceback (most recent call last):
  File "C:\Users\classic\dev\mako\test\test_lookup.py", line 96, in test_dont_accept_relative_outside_of_root
    runtime._lookup_template, ctx, "../index.html", index.uri
  File "C:\Users\classic\dev\mako\test\__init__.py", line 99, in assert_raises_message
    return callable_(*args, **kwargs)
  File "c:\python27\lib\contextlib.py", line 35, in __exit__
    self.gen.throw(type, value, traceback)
  File "C:\Users\classic\dev\mako\test\__init__.py", line 85, in raises
    "%r !~ %s" % (message, e)
AssertionError: 'Template uri "../index.html" is invalid - it cannot be relative outside of the root path' !~ Cant locate template for uri '
../index.html'
=========================== short test summary info ===========================
FAIL test/test_lookup.py::LookupTest::test_dont_accept_relative_outside_of_root

travis is just a config file right, sure, show me that

Yes that one.
I'll try cook travis conf (and maybe appveyor one while I'm at it).

thank you !


unsupervised graph-based loss function

Is there any unsupervised graph-based loss function available in the package? I am trying to use GraphSAGE in an unsupervised setting, therefore need to use a loss function only based on neighborhood similarity, like the one defined in equation (1) in the GraphSAGE paper.

Hi, please see https://github.com/rusty1s/pytorch_geometric/issues/1420.


Tests fail because sent2vec missing

Currently when creating the virtualenv we run $(VIRTUALENV)/bin/pip3 install git+https://github.com/epfml/sent2vec.git which should not be neccesary as sent2vec is downloaded automatically. When removing this line though tests fail which affects #71 and #76.

The problem seems to originate from the fact that when running tests the virtualenv is not activated. As such we installed the models separate in the Makefile before running tests
test: $(VIRTUALENV)/.models $(VIRTUALENV)/.deep_learning_models.
Thus the fix of the problem is to add the non_pypi package as a requirement by test: $(VIRTUALENV)/.models $(VIRTUALENV)/.deep_learning_models $(VIRTUALENV)/.non_pypi_packages


database rule low security

if one of the users inspect element this app from chrome and change push to set ( in saveMessage function) all the messages will remove and only new record will added .I change database rule to prevent this situation but I can find a efficient way to solve this problem.please help me how to solve this?

The security rules have been updated to make them a bit more secure and explanatory. ptal


Using secrets

This approach lets you use secrets for running in production.  For an environment variable that should be secret, instead of directly entering its value into config, replace it with secret:/path/to/the/secret.  This little init script will scan the container environment for variables that follow that pattern, read the contents of that file, and replace the variable's value with the file's contents.
There will be an analagous pull on isle-dc to demonstrate how to use secrets in the docker-compose file.  I'm trying to work this in such that if integrating with other things like etcd, vault or aws's secrets solution, that this won't get in the way. It's specifically for the env backend of confd and if you don't pattern your variables this way, nothing will change.

I've made an associated pull to isle-dc: https://github.com/Islandora-Devops/isle-dc/pull/123
I'll try to document it in the README as best I can here.

I decided it fit better in isle-dc than here.  I updated the PR to include a secrets chunk in the README.

@nigelgbanks Any chance we can get this in to help with https://github.com/Islandora-Devops/isle-dc/pull/126?

@dannylamb per your request here https://github.com/Islandora-Devops/isle-dc/pull/126#issuecomment-726818269 I merged this PR


Yet another defvao VBO problem

I have the following spec
(defvao :mesh ()
  (:interleave (:divisor 1)
   (offsets :float 3)
   (colors :float 3))
  (:interleave (:divisor 0)
   (position :float 3)
   (normal :float 3)
   (uv :float 3)))

In my shader I have the following attributes:
layout (location = 0) in vec3 offsets;
layout (location = 1) in vec3 colors;
layout (location = 2) in vec3 position;
layout (location = 3) in vec3 normal;
layout (location = 4) in vec3 uv;

This works perfect.
However, it will NOT work if the instance array (divisor 1 layout) is defined second instead of first, and the shader locations changed to match. It does not matter if the layout is :separate or :interleave - it just seems that given 2 layouts, divisor 1 and divisor 0, the divisor 1 MUST come first.
The result is the shader sees the attributes as if they were empty arrays. Changing vertex colors to any attribute of any instance always results in black.
Luckily, this is a problem that can be easily worked around, ensuring divisor 1 vbo's are always defined first, but it took me a couple hours to realize that and will save lots of headaches in the future if this can be looked into. I can provide more information if needed.

Hrm I had a few ideas for how this might be broken, but none of them seem to be the case.  In particular, given your definition above, if I swap the :divisor between groups, it gets parsed/stored into the second group correctly.
Once that's the case, it's pretty much passed through directly and only depends on INDEX being correct... if that were broken, attributes should be considerably more broken.
I would recommend:

Check that they're parsing correctly for you, i.e. evaluate the defvao, inspect the resulting vao-declaration, look at the GROUPS slot and at a particular group.  DIVISOR is a slot of a vertex-attribute-group.  You should be able to change the divisor and see it in the right group.
src/vao/vao.lisp L60 is where the divisor actually gets set.  Put a print here or something and watch the divisor and index and see if they correlate correctly.

(To clarify, this was an issue enabling attributes when multiple groups were present.)

Fixed.


chore: correctly start backend in docker compose

Improvements

introduce ENV variables for db connection
fix path to backend in docker compose
add ENV variables for test environment to .env and docker compose
document ENV variables in README.md

Required steps before merging

[ ] review documentation changes

Required steps before merging (only if code changed)

[ ] check if all test stage in cicd pipeline run without warnings
[ ] check if branch is behind target branch (if yes, merge)
[ ] review code changes

@n3rdc4ptn @xilef45  What's still open here is that the database is not initialized when starting the compose and also the frontend seems to fail. But let's postpone discussion and work on that until after we spoke about the devcontainer.
frontend  | [Error: EACCES: permission denied, unlink '/app/code/.next/server/middleware-manifest.json'] {
frontend  |   errno: -13,
frontend  |   code: 'EACCES',
frontend  |   syscall: 'unlink',
frontend  |   path: '/app/code/.next/server/middleware-manifest.json'
frontend  | }
frontend  | error Command failed with exit code 1.
frontend  | info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
frontend exited with code 1

now database is inited automatically


Allow checking entity permissions based on devices

Description:
Add support for checking entity permissions based on device they belong to.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.


Calcuate string/char array size correctly

Fix for issues brought up in https://github.com/Diaoul/arduino-ESP8266/pull/11

You're right its pointless, I thought it might be necessary to also send the null terminator but after a few quick tests it seems fine without.

I only ran one test with a char *  but it seems that the problem is fixed now. Good work!

I ran some more tests. Everything is working again as it should. Thanks again!

Thanks for the fix :+1:


use DelayTime instead of Decreasing

I use sublime to edit my file. After I press ctr+s, sublime will format my file and save it again.
dogo will get two modify events and trigger 2 build and run. (because my app is web, it will report port is used error)
If dogo delay 0.5 second to build and run, there will be no such issue.
Maybe we don't need decreasing but just increase DelayTime, then we will not ignore any change.

Try increasing the value of Decreasing. Decreasing works only on Windows operating systems.

I only works on linux. but thanks for the reply.


PHP5 strict standards warning message

What steps will reproduce the problem?
1. enable strict standards in PHP5
2. load up minify

The following warning message will be shown:
Strict Standards: Redefining already defined constructor for class Minify
in .../minify.php on line 212

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

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

Please provide any additional information below.
This can be simply avoided by not having a method name like the class, e.g.
simply renaming the method.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 24 Aug 2007 at 1:07

Fixed in 2.x version

Original comment by<EMAIL_ADDRESS>on 27 Nov 2008 at 3:25

Changed state: Verified

SVN version should be E_STRICT compliant except for Cache_Lite_File. Although
we're
running a patched version of this, I'd rather not make radical changes to it
for full
compliance; Cache_Lite_File is well-tested.

Original comment by<EMAIL_ADDRESS>on 29 Feb 2008 at 2:14

Changed state: WontFix
Added labels: Priority-Low
Removed labels: Priority-High

Original comment by<EMAIL_ADDRESS>on 28 Aug 2007 at 5:26

Changed state: Accepted
Added labels: Priority-High
Removed labels: Priority-Medium

Btw, this is with 5.2.3

Original comment by<EMAIL_ADDRESS>on 24 Aug 2007 at 1:09


fix(plugin-seo): Updated the types for the SEO plugin generator functions to better match the doc format

Closes https://github.com/payloadcms/payload/issues/4544
Description
Updates the documentation and the types for the generator functions used in the SEO plugin to better match the actual type used

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] Existing test suite passes locally with my changes

@paulpopus Is this an actual fix or enhancement?
Also, was this addressed? https://github.com/payloadcms/payload/issues/4544#issuecomment-1861278784

@paulpopus Is this an actual fix or enhancement?
Also, was this addressed? #4544 (comment)

It's a fix because the current types are technically very wrong. That issue is more or less solved, not in the nicest way possible, but this is the solution we had when speaking with @jmikrut


move saving route to config.

In order to do this this also makes the sagas get initialized right after the engine is setup (else we start the root sagas and those can touch engine things before its initialized).

@keybase/react-hackers

@keybase/react-hackers bump

@keybase/react-hackers bump (not for this release)

[ ] after the release

@keybase/react-hackers bump. lint is failing cause of a fix in another pr so this can be looked at


Show matchability throughout the entire survey

Right now needing to answer over 30 questions before getting any information back is disheartening (I see this is also addressed in #103). Something that might help is showing your current matchability throughout the entire survey.
This is how it works on OkCupid too. You can choose to answer just a few questions and still see your matchability with different people, or you can answer more questions for a more accurate score.
How is matchability calculated? I can't find it in the code 😅

Fixed in #195


add license title, remove extra newline

I'll assume this will go the same way as https://github.com/souporserious/react-tether/pull/36, souporserious/react-motion-ui-pack#53 and souporserious/react-view-pager#25.


Zarr Tree Reproject

This creates a POC working zarr tree API. It is currently hard coded to 256 pixel tiles, and uses rioxarray to reproject under the hood. There is a working example to get the map representation of ww3 and plot it at level 0 in xpublish/test_get_chunk.ipynb.
One thing that is not clear right now is why fetching the dataset is so slow, it seems to hang because of this failed request, which we dont handle cuz im not sure what should be returned for them. But getting the data works!!!! Enough to start work on a javascript client test :)

Current Downsides to current approach:

aforementioned 404 errors
The metadata is computed for the entire tree no matter if you need it or not, should be lazy computed since groups can be loaded on their own
tile size should be dynamic
the reprojection requires the entire dataset to be loaded because rioxarray can't do this in parallel or lazily right now (as far as I know) maybe dask can be smart about this. In general, we need to be able to only grab data chunks needed for each tiles reprojection. Not sure anything can do this out of the box right now

Custom mapbox-gl-js source loading zarr tiles from xpublish:

And.... The holy grail (without any optimization) TURN ON THE METADATA CACHE IN THE TREE ROUTER BEFORE RUNNING OR ELSE ITS DUMB SLOW TO LOAD TILES.


Please Update to Endwalker

Love this addon. Thank you in advance!

I’m finishing up finals, and then I’m going to update all of my remaining things for EW.

Good luck on your finals!

Should be fixed as of v1.5.5.


[Node.js] Fix the ROI not working issue

Fix the setRegionOfInterest not working issue
Change to use ROISensor.from() to create ROISensor

@halton PTAL, thanks.

ping @halton

LGTM and merge


Can't build on Arch Linux

The script succesfully installs netcore but when trying to restore the packages it returns:
error: Unable to load the service index for source https://dotnet.myget.org/F/dotnet-core/api/v3/index.json.
error:   The type initializer for 'System.Net.Http.CurlHandler' threw an exception.
error:   The type initializer for 'Http' threw an exception.
error:   The type initializer for 'HttpInitializer' threw an exception.
error:   Unable to load DLL 'System.Net.Http.Native': The specified module could not be found.
error:    (Exception from HRESULT: 0x8007007E)

After some research, the error is referring to the file System.Net.Http.Native.so under:
/Tools/dotnetcli/shared/Microsoft.NETCore.App/1.0.1

But the file is cleary there, I really don't know why It isn't recognizing it.

For clarification: are you trying to build this repo, or some other project using MSBuild?


feat: Better pretty table handling

Terminal width, no hidden columns:

Terminal width, no hidden columns, no record batches:

80 char width, hidden columns:

Explian (multi line string):

Going to go ahead and merge so this has some time to settle before monday. Please leave comments on this pr, and I can address in follow up prs.


Rephrase the last paragraph in Local Variables section of cppguide.html to make it more clear

Rephrase the last paragraph in Local Variables section of cppguide.html to make it more clear

FWIW my reading of the guide is that it consciously keeps language informal. This change while clarifying also makes the language more formal. I might have simplified further:
"It may be more efficient to declare such variables outside the loop, as in:"

Good idea, this make it more clear to me, I will update it to "It may be more efficient to declare such variables outside the loop, as in:"


progressive append/update

wip

Wow, you're making quick work of this.  I'll circle back with any issues I find.

thanks.
i'm feeling better about this than the prior code. easier to read and manage. all in one place.

@motdotla Looks like all the environment variable cases I care about are working with my simple JS wrapper. I'm going to work on integrating it into my larger ClojureScript context and make sure it works there as an additional sanity check.
It looks like the parse function still refers to regular dotenv.parse. Is that intentional? I would like to be able to use this in an non-blocking (async/await) context (avoid readFileSync) so it seems like the right way to do that is to load the file using fs.readFile and then call .parse on the content (rather then using .config which does blocking read/sync of the file). Does that sound right?

good catch. i'll update. i've been wanting to replace dotenv.parse with dotenvx's improved version. finally going to happen.

I've successfully tested this in the context of the ClojureScript project using both the synchronous approach (dotenvx.config) and async using dotenvx.parse.
Thank you again for your quick work on this! This will save me some pain of having to either use bash (insecure) or create my own (less feature-full) implementation from scratch.

You're welcome and wonderful to hear!
Releasing now as 1.24.0


Some mind of PutObjectRequest.SetBody

Hi,I saw the class PutObjectReques have a method SetBody in S3,it defined as
inline void SetBody(const std::shared_ptrAws::IOStream& body) { m_bodyStream = body; }
What my confusion is that SetBody must input a IOStream.when I use PutObjectRequestAsync to upload a file,but this need me change a file to a stream, I think it spent too much time. I think if do this work that change a file to a stream in exector must be better. Maybe SDK have some other method do it. Could  you help me?

Hi @ToBeMyOwnHero ,
I am sorry but I am not exactly understanding what your problem is with the putobject request... But, I would suggest using the TransferManager for your s3 operations, it is meant to make the operations simpler, as it decides for you whether it does sync/async or single/multi part.
Here's a quick sample on how to do uploads:
#include <aws/core/Aws.h>
#include <aws/s3/S3Client.h>

#include <aws/transfer/TransferManager.h>
#include <iostream>
#include <fstream>
#include <time.h>
#include <string>

using namespace Aws;
using namespace Aws::Transfer;

Aws::SDKOptions options;
std::mutex semaphoreLock;

int main(){
    options.loggingOptions.logLevel = Aws::Utils::Logging::LogLevel::Debug;
    Aws::InitAPI(options);

    Aws::String testFileName = "yourfileName.txt";
    Aws::Client::ClientConfiguration client_config("default");
    auto m_s3Client = MakeShared<Aws::S3::S3Client>( "S3Client", client_config);

    Aws::Utils::Threading::DefaultExecutor executor;

    TransferManagerConfiguration transferManagerConfig(&executor);
    transferManagerConfig.s3Client = m_s3Client;

    auto transferManager = TransferManager::Create(transferManagerConfig);

    std::shared_ptr<TransferHandle> uploadHandle = transferManager->UploadFile(testFileName, "bucketName", "keyValue", "text/plain", Aws::Map<Aws::String, Aws::String>());
    uploadHandle->WaitUntilFinished();
    bool success = uploadHandle->GetStatus() == TransferStatus::COMPLETED;

    if(success){
        std::cout<< "this worked"<<std::endl;
        std::cout<<"handler size: "<<sizeof(*uploadHandle)<<std::endl;
        std::cout<<"object total size: "<< uploadHandle->GetBytesTotalSize()<<std::endl;
    }else
    {
        std::cout<<"it didnt work Because : " << uploadHandle->GetLastError()<<std::endl;
    }

    Aws::ShutdownAPI(options);
}

Let me know if this helped you or if I misunderstood your question.


Proposed v3.0.0 of node-openid

I recently decided to put an effort into reworking this package, as I see a lot of people are using it and it is frankly outdated. Therefore, I've done the following:

Transformed package into typescript.
Added es6 module support.
Migrated from callbacks to async/await.
Added validation to allow users to prevent this vulnerability.
Added Extension class which can be extended by users to create their own extensions more easily.
And a lot more refactoring.

This update is obviously breaking, and therefore should be considered a major version change per semver.
It requires node v10.0.0 at minimum, which I would consider a fair compromise for the added functionality. Older versions of this package are of course also still available on npm.
PS: If you experience a weird bug when running npm run test, try to simply add a whitespace (" ") anywhere in the file and saving, and it'll get fixed. This seems to be some obscure jest bug.

Please check this out @havard 👌

OpenID 2.0 is an old standard which is essentially deprecated and replaced by OpenID Connect. I see little use in the modernization steps, and have received no requests to do so.
I will close this and keep the library stable. You are of course free to fork it and use it for whatever purpose you see fit. Thank you for your contribution, though.


Added Passengers + distance calculation

Car is not shown on list of routes.

This one I can live with. I created a ticket for setting min and max values of mileage fields.

Passengers are displayed one on top of another.
There is no way to remove a passenger.

Good spot - on the mobile - Seems that adding bootstrap class broke the select 😐
Removed tested and ready to go ;)


Camel-K Doesn't Create a Service or Ingress for an Integration

I'm trying to use Camel-k with the CXF component to create a pass through proxy for a SOAP service running outside my K8S cluster and I am having trouble exposing the port to the outside world or to other pods in the cluster. I was able to generate a service and ingress following the RestWithUndertow example, but not with my integration. My code is in the picture below.

I am running the following command to create the integration:
kamel run --name=cxf-test --trait service.enabled=true --trait container.service-port=8080 -d camel-cxf -d mvn:org.apache.cxf:cxf-rt-transports-http-jetty:3.3.5 cxfTest.java --dev
I have tried with the traits above and the ingress traits to no avail. My integration yaml looks like the following.

It shows a no http service is required message under type "DeploymentAvailable" and that no host or service is defined under "ServiceAvailable." I have verified that the integration works as a proxy by using the command kubectl exec -it <pod_name> -- /bin/bash to get inside the container and manually send a SOAP request to the proxy. I am unable to access the proxy from outside the container however.
I have used a work around in the meantime by manually creating my own service and applying it to the integration.

Kubectl apply -f cxf-service.yaml
This seemingly fixes the problem and I am able to expose and access the port from outside the cluster. With this method I have been able to get the functionality I was looking for, but it is not ideal however, because the service needs to applied every time the integration code is updated.
What would be the "proper" way to ensure that kamel run generates a service and an ignress?

https://github.com/apache/camel-k/pull/2577 provides examples on how to run integration with different routes (no tls, edge, reencrypt, passthrough).

There is a route documentation on how to use kamel run to setup TLS routes. Reopen if you feel appropriate.

@astefanutti can you close this issue ?

@claudio4j I think that one applies to vanilla Kubernetes. I was actually wondering to what extent the work you did with Routes on OpenShift, applies to Ingresses and vanilla Kubernetes.

ok, never mind, it's ingress.


Previous button icon

setting previous button icon is not working, while next button icon is setting in both

Hi Mast,
Thank you for your feedback. The issue is now scheduled to be fixed in the upcoming release. I will keep you posted!
Regards
iMujtaba

Hi Mast,
The bug has been fixed and a new activeStep property has been introduced in the im_stepper package that gives you access to create steppers in a much easier way. The ability to set the initial step and jump to different steps has also been added.
Please check out the new release here and examples here. Once again, thank you for your feedback!
Warm Regards,
iMujtaba


fix(fonts): remove hints from unknown symbols

This bug is pretty serious, so I'd appreciate a quick review.
What is the previous behavior before this PR?
\small x \in y rendered like this:

What is the new behavior after this PR?
\small x \in y renders like this:

Approach:
As Latin letters seem to be working fine, and the only difference I've detected is the enabling of font hints, I turned on a feature of ttfautohint (-f none -S) to not hint characters that it doesn't know about (which seems to include all mathematical symbols). This seems to fix the problem.
Worry:
ttfautohint 1.8.4 apparently apparently fixed a bug in exactly this feature. (This is actually how I discovered the feature.) It's not released on Ubuntu yet, so it's more difficult to use, but we could roll back more of #2866 to install it in the Dockerfile. But if it's working now, perhaps we could wait on that?
Fixes #3219

As I read more of ttfautohint's documentation, I wonder whether we should use the --reference option so that our bold fonts align better with our regular fonts.  Here's an example of the current height discrepancy:
\mathrm{x}\mathbf{x}

Admittedly not that big a deal, but something to consider...

Could you add a regression screenshot test on another PR?


javascript creates sometimes invalid signatures

This is a weird one. For some reason this exception is thrown from javascript whereas it is fine for native/jvm. https://github.com/fiatjaf/scoin/blob/master/shared/src/main/scala/Script.scala#L531
I do not have a trimmed down program to reproduce, but it can be done via these steps with my toy "sig-pow" program:

clone the repo here https://github.com/VzxPLnHqr/sig-pow
switch to the add-scalajs-target branch
./mill -i sigpow.jvm[3.1.3].run (press enter at each prompt to accept the defaults)
terminates without throwing an exception which means that the transaction which was generated/signed was valid (at least according to the acinq/scoin verification code).

Now do it again but for javascript:

./mill -i sigpow.js[3.1.3,1.10.1].runNode (after it builds and runs, just press enter at each prompt to accept the defaults)
exception might be thrown at the end, but might not be either. Sometimes the sigs it generates are valid, apparently, and sometimes not.

Maybe the underlying javascript library (noble/secp256k1) is not enforcing some bitcoin consensus rules on ecdsa signatures (low s value?) that the jvm/native underlying library (libsecp256k1) does?

Thank you for this report.

I think this commit should fix it, but I am not entirely sure. If you can test it for me and report back that will be amazing.

It may have, but then a exception was thrown pertaining to CryptoPlatform.sign It looks like you are currently requiring CryptoPlatform.sign to return a ByteVector64 but this is problematic for ECDSA sigs since they are not typically 64 bytes in length.

OK, I understand it now. The idea is that the signature returned from .sign() is 64 bytes, in compact format. That is the original API. But to use it in a Bitcoin script you must convert it using compact2der.

Thanks. It is still somewhat strange though because my code was calling scoin.Transaction.signInput which lookcs like it does in fact call Crypto.compact2der.

Then I don't know. If you don't fix this yourself I may try it myself in the near future.

I have been working on it. It seems to, oddly, have to do with the number of signatures generated by the javascript library. The bitcoin transactions that my program generates/signs/verifies can vary from requiring, say, 4 signatures to requiring 16 signatures. It is not a op_checkmultisig transaction but instead is a large transaction that has many individual op_checksig calls.
When the number of signatures is less than 15, the javascript program at least seems to generate valid signatures. However, as soon as I am above 16 signatures, the javascript program somehow ends up creating at least one invalid signature and it fails verification.
The bizarre thing though is that this same issue does not happen on the jvm/native, so it definitely has something to do with how the javascript library generates its signatures.

The jvm, using libsecp256k1, generated the following signatures for the following public keys.
0313fd0f57a343c28b830eccab1dea6bb0a7fd44ed26e63e3c184b2e08bdc0db09 --sig--><PHONE_NUMBER>9df0130583b7492f10f82f7816d72e377661a85978ffcab396e3416bb6666495022068f61ee549d43dff548f0626535c661545755d5f7e75f4f32247e8eb4da1ead301
02f7a2d32d03dd521dee02c62c84898932a33812f50c68b62eca880497ccc7747d --sig--><PHONE_NUMBER>bc8d764369929c6aa4c578eac04eb826dab232de297a7aa260780411ab12e70802206e61f021fccebe8c4b7566df62730bdd10c8c77f4d665d5a41fa39976f22d1c701
03ccb895aae25472a03b6989e8958c6f1a64d16df7d8634b129b1b5d2df75e2a41 --sig--><PHONE_NUMBER>aa1a2c1f77e7f5e9f7d9a067ad4c3412d73fab0fcdf72eba10a54d8a7a74121d02203c63fbdbc0d146d39f389eae682ed6bc7c1d9dcd13145c9768712af3d800ce2f01
02ef4b7c8174e5f1e20647d53ac18de0880fd4736976dbbf4d98736eec5bbeed3d --sig--><PHONE_NUMBER>f3908de2448517cc7bb78403116d319335e80f76eeef27b931e74cdb8a0383f702204aee3d152177bd396a24e7850563ddb5ea17273ec946cc3e320c74113358194201
02699710b3cc42db9a5fe2a589565fdc1fd710d7d568cffde6cbb5a1546452129e --sig--> 304402203d4910c64e410c6b1265a9bfafc9c8733bd266ad6ccc174bb807a91a0ca6f12c0220560fd0c53a969b6808152dd3640eecb96a4d4061ab0e221e257a98179fde131001
03cb9686b0cd6e0a2e5e4bbd3204cec54a5eb74887934aa4353556820f8329e52e --sig--><PHONE_NUMBER>49e71b3ecc68bbcd8f3f49cee24844ccf721fe16b4b2edf0a8eddc988bf3de02205d880f23e9217b656401d2372fc1204c858cd6debe7fc3dfe79a830c615a36a801
03c2972024e3dd80134f5169b395b84d7a0f65223a18d967b171475667c5af67c5 --sig--><PHONE_NUMBER>92d7756b2e472c400d4f1bbde78fe3b663f090ee83977218e839e6a0adbf682f02203916eea61936d952cfaf00b87f8d4b80ebe16a8c5cd5043cebb651cb7356c67101
0214aa8f243bcf6111b157a443b30ba99d65591660d5b34f47de2163d93fa6fe9e --sig--><PHONE_NUMBER>c9d745f08b8829632378f2c9ee9f73abc31750ffb6b0a0c246c4408dcb49c5022034f670501f16d950c7c5776eea79a585d5861d52576aac9c01515f7b8eaa1f2201
021398c1fd987e97669469f66491ecd6d6f8de1a020cc7b029b0757031475f71e2 --sig--><PHONE_NUMBER>c11aae78e3a9601bd6ed6d909cbeb1416e8ce1aee275e981b8a8940e37619ae002202b94cbf8dd9da1aef1850ade5c6ae45dd3ada4b2a8db6aa7792c21f2421531a101
02c06abce22db77d3f4bfc20350db60dc182f68a44d7079666056c84dd3c99dbe1 --sig--><PHONE_NUMBER>fe6e36768e95b8cc0cd307afc228480884969311bdd42ce41f96e902a77358a50220535360d8a4b0f957afe2e15f710a90c2ed3369ac4f9bb59adc73e51822e96e7401
0200f9a57d9ad292143121e66fb98a77b1f595f8ecf0c6d3f553067c0d413a97cb --sig--><PHONE_NUMBER>c647ce439be05fe6c504f0e385a5c0a0e529df3d7af93153ee2e345d5a2f0acc0220732ff89f2f2a4054360f3a008f90d0fd4091a749dc72d8592ab15bfd19a8afe301
031c1cbeb2a14b2723a9581d8808dde768bbd576e689189f64f4af96c53e435e7e --sig--><PHONE_NUMBER>96f7c83403009f0695eba9eda6df0002af45721d85d92445cf508fcb8a65fda3022026ff9cc100c43c8cbbc72fc791c0bdd359a6ad421cd80ab47efe64a955db938c01
02d96745b4f023b7a83802b74e640432ed9c5328c796be29d3a63d6d94c1ecf67f --sig--><PHONE_NUMBER>36a3e80c7316af0d8c3f25558e0886ccc0d31ef02273531e9c0916441712e902200fffc14bd54cea6da579b0d8d2d139281897621a3096ae66eec720c31dcda1bc01
03e20845a0b9405cb5736ab62d0b530904c7e994b5171f6c19f37ee14aee9907ea --sig--><PHONE_NUMBER>fc536853140f6065974dbaede4e91857a5df51ad707216691ecb41836ec824360220550a871ffd8c8daa96bf47b6543b5d4f643668c9135682e20c6cd7229e3214a701
02f642103ea91b1bd39fe5190828e43af965151207988adcbb631a540b7c14d698 --sig--> 304402204a56443c3291409641dc94024f37983384f9a451f35e53098c41c747f56874ce022066982d4ba48ca0d53bccb832d83918b6033c5a87f81a80fd0f7243d5c9cbad0a01
027ffaae6043e994b57ce7e0e42b52ba5893fb6595e9eb9cc0995cbdeef678296b --sig--><PHONE_NUMBER>e0709e558fb59383d1fc56ed3b07ce7c93e57d8eb33bfb5e6854e54ceb69736002201ec6073a595e81b3e9f48e7cd422509bf443296a100e001d3bff60f7630cf76901

Whereas the javascript run of the same program generated the following:
0313fd0f57a343c28b830eccab1dea6bb0a7fd44ed26e63e3c184b2e08bdc0db09 --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab10022053b8129f7958f3d9f1b576f3e379cd744bffe852936247d23ebcb94e23f250f001
02f7a2d32d03dd521dee02c62c84898932a33812f50c68b62eca880497ccc7747d --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab100220747ff719087870d87f818acb11becdc13ad6de58e2dd7859b45277787207c3d601
03ccb895aae25472a03b6989e8958c6f1a64d16df7d8634b129b1b5d2df75e2a41 --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab1002201c97004c6463595536700d159b8a6ff5705f209063c88abde9aabcde61a221a101
02ef4b7c8174e5f1e20647d53ac18de0880fd4736976dbbf4d98736eec5bbeed3d --sig--><PHONE_NUMBER>0bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab10021f45ea5071719a2599644fb6b7a976a053aa3142b0603d2cc98152af4f1a8f6f201
02699710b3cc42db9a5fe2a589565fdc1fd710d7d568cffde6cbb5a1546452129e --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab1002202fcef7d2a0819629185dd3ff9dd3baf2baa588d4a642df52cef41ae2679435d401
03cb9686b0cd6e0a2e5e4bbd3204cec54a5eb74887934aa4353556820f8329e52e --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab100220116593cafbfcf6b0445326d706a46037c2fc068b73547bcfccc6a02b8761bbda01
03c2972024e3dd80134f5169b395b84d7a0f65223a18d967b171475667c5af67c5 --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab10022026e344ff032b0675b79ade5f6acaf67a4c1bff5485a2c58276e00982c65c6f2d01
0214aa8f243bcf6111b157a443b30ba99d65591660d5b34f47de2163d93fa6fe9e --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab1002201b5563696941150abe3478ad1f28f64b5ea94c012ade56267dbb1217b7c4b46a01
021398c1fd987e97669469f66491ecd6d6f8de1a020cc7b029b0757031475f71e2 --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab100220607945a3b17b83bd8f4a61fb59dfaf58bf52ca82e6593bf6bfd73ff15381b51001
02c06abce22db77d3f4bfc20350db60dc182f68a44d7079666056c84dd3c99dbe1 --sig--><PHONE_NUMBER>0bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab100220008e2ec562a43957f90eaed8e403aebbbdcbe3911adb5e17f35555936c0ef76ee01
0200f9a57d9ad292143121e66fb98a77b1f595f8ecf0c6d3f553067c0d413a97cb --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab1002201fcae65e13c4c56df044d8d38682409e4e708b849e67254db8ed185ccdfb1c9a01
031c1cbeb2a14b2723a9581d8808dde768bbd576e689189f64f4af96c53e435e7e --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab100220598eae5d566fd8c30fa4a0140b6d635d37984e1f38888d1249f042d345abeba201
02d96745b4f023b7a83802b74e640432ed9c5328c796be29d3a63d6d94c1ecf67f --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab100220735602d3445181d3228700e123a8826be7cad1f064f115b02937a5632f6d3d6e01
03e20845a0b9405cb5736ab62d0b530904c7e994b5171f6c19f37ee14aee9907ea --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab1002202a2c145befd7e7db27244bcdd62cd161881ecc07d7b4c8e697500b5803e3e7f301
02f642103ea91b1bd39fe5190828e43af965151207988adcbb631a540b7c14d698 --sig--><PHONE_NUMBER>0bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab10021f25efe8868215c1c3446ce30a6108516cf0ead3f1884553c067c8ffdbe4062bd01
027ffaae6043e994b57ce7e0e42b52ba5893fb6595e9eb9cc0995cbdeef678296b --sig--><PHONE_NUMBER>bf169e19c6cc1762ddeb0c8fbcd46d9e7b3131b8e277bc1e55aa841c6d81ab10022000a5adb254534551e326eb685080d3ae2f96b90415f8ccf53ba2c5ed2d30c7d001

Notice how the javascript signatures are almost the same. Only a couple of them are different. Whereas (rightly so) the jvm signatures are in fact all quite different from one another.

I wonder if it has to do with how this sha256Sync and hmacSha256Sync are working in javascript. The output above makes me think that maybe the hmac is somehow caching something that we do not want cached.

Interesting.

The monkeypatch code which exposes hmacSha256Sync to javascript is currently:
def hmacSha256Sync(key: Uint8Array, msg: Uint8Array): Uint8Array =
    ByteVector
      .fromValidHex(
        HashJS
          .hmac(HashJS.sha256, ByteVector.view(key).toHex, "hex")
          .update(ByteVector.view(msg).toHex, "hex")
          .digest("hex")
      )
      .toUint8Array

But the docs from https://www.npmjs.com/package/@noble/secp256k1#signmsghash-privatekey seem to indicate it should be like this:
// signSync counterpart could also be used, you need to set utils.hmacSha256Sync to a function with signature key: Uint8Array, ...messages: Uint8Array[]) => Uint8Array. Example with noble-hashes package:

import { hmac } = from '@noble/hashes/hmac';
import { sha256 } from '@noble/hashes/sha256';
secp256k1.utils.hmacSha256Sync = (key: Uint8Array, ...msgs: Uint8Array[]) => {
  const h = hmac.create(sha256, key);
  msgs.forEach(msg => h.update(msg));
  return h.digest();
};

So, I tried to update the monkeypatch code to be:
  def hmacSha256Sync(key: Uint8Array, msgs: Uint8Array): Uint8Array = {
    val h = HashJS.hmac(HashJS.sha256, ByteVector.view(key).toHex, "hex")
    msgs.foreach(msg => h.update(ByteVector(msg).toHex,"hex"))
    ByteVector.fromValidHex(h.digest("hex")).toUint8Array
  }

It still did not work (in the sense that it still generated the same invalid signatures as before). I wonder if somehow it should return the result of a msgs.fold(..) instead of a msgs.foreach? But what to use as the zero value in the fold?

Now it will work. There are tests, even, and they are passing after a little fight against the ScalaJS compiler which ended with this: https://discord.com/channels/632150470000902164/635668814956068864/1011715303056609332

Let me know if it worked for you.

Nice work! I wondered if there was going to be some nuance with scalajs and the hmac function signature needed for the secp256k1 javascript library.  However, we still have this strange "sometimes a leading 0" in javascript issue.
for same key and message
signature from javascript:
03043021f449e71b3ecc68bbcd8f3f49cee24844ccf721fe16b4b2edf0a8eddc988bf3de02205d880f23e9217b656401d2372fc1204c858cd6debe7fc3dfe79a830c615a36a801
signature
signature from jvm:
304402200449e71b3ecc68bbcd8f3f49cee24844ccf721fe16b4b2edf0a8eddc988bf3de02205d880f23e9217b656401d2372fc1204c858cd6debe7fc3dfe79a830c615a36a801

Notice that the only thing different is the leading zero, and all the other data is the same. Guessing that ByteVector is the culprit here in that it is somehow behaving differently on jvm/js when encoding/decoding hex?

Crypto.signatureToDER ultimately returns a ByteVector which it gets from ByteVector.fromValidHex(hex) and we can see here that val hex = s"30{$length}..." (point being that hex itself obviously starts with 30 as it is supposed to, yet somehow when toHex is called on the resulting signature (a ByteVector), the returned hexstring has a 0 prepended to it (sometimes, not all the time, and only seems to be in javascript, not jvm or native).
https://github.com/fiatjaf/scoin/blob/master/shared/src/main/scala/Crypto.scala#L404

Yes, I think this is an issue with ByteVector. Just confirmed on javascript the following:
raw hex: 3043021f449e71b3ecc68bbcd8f3f49cee24844ccf721fe16b4b2edf0a8eddc988bf3de02205d880f23e9217b656401d2372fc1204c858cd6debe7fc3dfe79a830c615a36a8
ByteVector.fromValidHex(hex).toHex: 03043021f449e71b3ecc68bbcd8f3f49cee24844ccf721fe16b4b2edf0a8eddc988bf3de02205d880f23e9217b656401d2372fc1204c858cd6debe7fc3dfe79a830c615a36a8

The above output was made via a simple cats-effect IOApp:
    val weirdJSissueDemo = for {
         hex <- IO("3043021f449e71b3ecc68bbcd8f3f49cee24844ccf721fe16b4b2edf0a8eddc988bf3de02205d880f23e9217b656401d2372fc1204c858cd6debe7fc3dfe79a830c615a36a8")
         _ <- IO.println(s"raw hex: $hex")
         bv <- IO(ByteVector.fromValidHex(hex))
         _ <- IO.println(s"ByteVector.fromValidHex(hex).toHex: ${bv.toHex}")
    } yield ()

Not sure if this would work, but for signature-related code in scoin on javascript, it looks like the js library expects input as a hex string. So when we have a ByteVector and need to get it to the js library, it looks like right now we are sending such hextring over by calling _.toHex on the ByteVector. I will see what happens if it is changed to _.toHex.dropWhile(_ == '0'). Maybe that will fix it?

I do not know much about DER, but I wonder if the issue is that in Crypto.signatureToDER the value val hex = s"30${length}02${rLen}${rHex}02${sLen}${sHex}" seems to, sometimes, be of a size which is not even. So I think that is actually the problem because it is fed into ByteVector.fromValidHex (but it is not actually valid hex).

OK, now this is an error elsewhere, on the DER encoding specifically, right? Let me take a look.

Nice, my code for signatureToDER is what is broken, very likely, it was ported from @noble/secp256k1 to Scala by my own hands and never tested.

OK, now this is an error elsewhere, on the DER encoding specifically, right? Let me take a look.
Yes, if you put something like:

def signatureToDER(...) {
...
require(hex.size % 2 == 0, "INVALID HEXSTRING of odd length")
ByteVector.fromValidHex(hex)
}

and then feed it the following, it will fail the require and throw an exception.
      val r = BigInt("1939826268000007359143909510553084398066457921122334221326722093796335285214")
      val s = BigInt("42305490613398507257462290570133199576497244294403847577481978090143620937384")
      val sig = Crypto.signatureToDER(r,s)

but if you feed it:
      val r = BigInt("27720179373560178646241075848627520324989088086870667600053818420475967500588")
      val s = BigInt("38926848564652280097082943871331288670328626516021751159001707213882054808336")
      val sig = Crypto.signatureToDER(r,s)

it will pass.

Nice, my code for signatureToDER is what is broken, very likely, it was ported from @noble/secp256k1 to Scala by my own hands and never tested.

Ok, glad we got to the bottom of it then!

For reference: https://bips.xyz/66
It looks like the DER encoding has to start with 30 always, so is libsecp256k1 wrong here? This makes no sense.

We didn't get to the bottom of anything!

It looks like the DER encoding has to start with 30 always, so is libsecp256k1 wrong here? This makes no sense.

Agreed. I am so confused!

We didn't get to the bottom of anything, my code seems to be correct!

But val hex = ... sometimes is of odd length, so it cannot be correct, right?

for same key and message
signature from javascript:
03043021f449e71b3ecc68bbcd8f3f49cee24844ccf721fe16b4b2edf0a8eddc988bf3de02205d880f23e9217b656401d2372fc1204c858cd6debe7fc3dfe79a830c615a36a801
signature
signature from jvm:
304402200449e71b3ecc68bbcd8f3f49cee24844ccf721fe16b4b2edf0a8eddc988bf3de02205d880f23e9217b656401d2372fc1204c858cd6debe7fc3dfe79a830c615a36a801

Notice that the only thing different is the leading zero, and all the other data is the same. Guessing that ByteVector is the culprit here in that it is somehow behaving differently on jvm/js when encoding/decoding hex?

Look at this weird thing. You are saying there is an extra zero, but there is not, the length is the same.

I took a long time to understand what you were saying.

Just tested it....success! Nice work! Now I think we can safely say that we got to the bottom of it.

Awesome, thank you!
That is great to hear because I had a dream in which I had forgotten to replace the toPaddedHex function name in one place and everything was breaking because of that but I couldn't find because I only had access to the GitHub interface and not my editor.


Support grouping files by URL domain

Add the ability to group generated files into folders based on the URL domain of the annotated article, if enabled via the plugin settings.
I find that some file structure helps it to feel manageable with 200+ annotated articles. Even if most folders only contain one article file. It's disabled by default because it'll not work with existing synced files.
I'm not sure how many additional uses cases there are to justify adding a fully customizable path based on a template.

This needs to be rebased if https://github.com/weichenw/obsidian-hypothesis-plugin/pull/39 merges, but should apply similarly.


@Html.Display() and @Html.Editor() should also check for template for declared type

@Html.Display() and @Html.Editor() sometimes do not check for templates matching the declared type (aka ModelType) of a property.  this is confusing and likely leads to duplicate templates that must be kept in sync.
my suggestion is to always check the ModelType just before Collection (or Object in the single-valued case) if ModelType is an interface and the model is non-null. this fix is simple but may be controversial if named interfaces should (for some definition of "should") be checked before one of the instance's base types.
for example, with a model containing
  public IEnumerable<string> OwnedAddresses { get; set; }

the property may be initially null, set to a List<string> instance in a subset of action methods, and stored as a string[] array in a ModelStateDictionary entry. checked templates for each of those types include the following:
1.
EditorTemplates/IEnumerable`1
EditorTemplates/Collection

EditorTemplates/List`1
EditorTemplates/Collection

EditorTemplates/String[]
EditorTemplates/Array
EditorTemplates/Collection

note only Collection is common and templates for that may be too general. with my suggestion EditorTemplates/IEnumerable1` would also appear in all three lists.

root cause is TemplateRenderer.GetViewNames() walks the RealModelType base types back to object but mostly ignores implemented interfaces. since no instance has an interface base type, the declared type is also normally ignored.

Closing because we are not planning to implement this.


Eradicate and conditional field initialization

Test case:
import android.support.annotation.*;
public class CondFieldInit {
    Object o1;
    @Nullable Object o2 = null;
    public CondFieldInit() {
        if (o2 != null) {
            o1 = new Object();
        }
    }
    public static void main(String[] args) {
        CondFieldInit x = new CondFieldInit();
        x.o1.toString();
    }
}

Eradicate finds no issues for the above code, but running the code fails with an NPE.  I'm not sure if this is a bug or whether my expectations are off, in terms of what kind of guarantees Eradicate is intended to provide around field initialization.

Here's another similar test:
import android.support.annotation.*;
public class CondFieldInit2 {
    @Nullable Object f1;
    static Object condInit(boolean b) {
        CondFieldInit2 x = new CondFieldInit2();
        if (b) {
            x.f1 = new Object();
        }
        return x.f1;
    }
    public static void main(String[] args) {
        Object y = condInit(false);
        y.toString();
    }
}

Not sure if this is the same problem or not.  If it's different, let me know and I can open a separate issue.

Thanks for the reports.
In both cases, referencing the object before the conditional triggers the expected report (Object tmp = o1; and Object tmp = x.f1;), so perhaps the same mechanism can address both.
I'll do some experiments and see how that affects the rest of the system.

Ok, great, thanks for looking into it!

DEUS LUZ FORÇA AXE

Hi @msridhar, there is a new treatment of conditionals in https://github.com/facebook/infer/commit/319e3ed271e81618bdd809823c2dbb9e065e2794.
This affects the treatment of conditionals in general, when a value is accessed for the first time in one branch of the conditional, and is not accessed in the other branch. The tests we have run look positive so far.

That sounds great!  Did you leave this issue open deliberately?  If the tests above are behaving as expected now, I'm fine with closing.  It may take a bit time before we are able to run this new version across our code base.

Great, let us know if you notice anything. Closing for now.


duplicated data when login with email on uppercase

When you are login with an already registered email but in uppercase, you can login normaly but page loads an empty config account.-

Thanks, you are right, it will be corrected in the next version!


Check enough token for all UDC deposits are available

Resolves https://github.com/raiden-network/raiden/issues/6169
For mainnet testing we're not minting tokens but use the ones in the existing accounts.
In the case of not enough token available for all nodes to be funded, we ran into the issue above.
This check should fix that issue. It is however not applicable to the "normal" scenario player, as the minting would be done in the next step and it will fail in every case.

Looks good, but we can't merge this into master until we have a proper mint switch.


WIP: add mempool.nixbitcoin.org

Live at https://mempool.nixbitcoin.org

https://mempool.nixbitcoin.org is now running beautifully fast with Fulcrum.

Superseded by https://github.com/fort-nix/nixbitcoin.org/pull/42


Update index.md

Codecov Report

Merging #137 (a3f63f8) into master (6ebb2fa) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a3f63f8 differs from pull request most recent head 613e219. Consider uploading reports for the commit 613e219 to get more accurate results

@@            Coverage Diff            @@
##             master     #137   +/-   ##
=========================================
  Coverage     39.67%   39.67%
  Complexity      511      511
=========================================
  Files           125      125
  Lines          2982     2982
  Branches        370      370
=========================================
  Hits           1183     1183
  Misses         1631     1631
  Partials        168      168

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


Update the content related to dashboard deployment

What type of PR is this?

[ ] bug
[ ] feature
[ ] enhancement

What problem(s) does this PR solve?
Issue(s) number:
Description:
How do you solve it?
Special notes for your reviewer, ex. impact of this fix, design document, etc:

[ ] pls sync with the en doc


Autorun kafka-cmd setup

Currently if the user restarts kafka then they will likely run out of disk space since information about the retention isn't preserved. We should make sure this is always set.

This is resolved by nature of the new global limits that are set for retention. Confirmed that this is no longer an issue.


Permissions to create the File Share

What are the minimum permissions needed to create the File Share that will be used as a clouddrive?  I have a user who is Contributor on a storage account in a suitable region, the "Resource group" is available and populated, yet the "Storage account" and "File share" buttons are greyed out.  Are permissions on the Resource Group itself obligatory?  Could this be explained in the docs?  Thanks

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

ID: 2ccf23ec-8d85-84fb-2c46-61a414d466ef
Version Independent ID: 52b6949e-8ea5-c0c7-d0ab-7bf0ba9ed076
Content: Azure Cloud Shell overview
Content Source: articles/cloud-shell/overview.md
Service: azure
Product: unspecified
GitHub Login: @jluk
Microsoft Alias: juluk

@JoNevi Thanks for your feedback! We will investigate and update as appropriate.

We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


traceur-compiler returns annotation parameters as arrays

See comment here: https://github.com/google/traceur-compiler/blob/28be0752241477c7d47214c6e9955e965a3243a5/src/codegeneration/AnnotationsTransformer.js#L95
Which is consumed in DI module
https://github.com/aurelia/dependency-injection/blob/68f8745ca0a642115cd16cb99e327f6516291d75/src/container.js#L33
A simple change to line 33 fixes this:
keys[i] = parameters[i].is || parameters[i][0];

Yes. They aren't following their own spec at this point. However, I'd be willing to make this change if....you would sign our CLA and send me a pull request. Would you do that?


Code-service requires a docker list

Using code-service in docker, we tend to need a variety of environments, like I need GPU, then I need the cuda version of the code-service, of course, there are many other situations, here is not to add a The version of code-service.
At the same time, I have a small problem. Whenever the code-service is updated, how can I update it in docker? I am now building the dockerfile from scratch.

Attached to the project I have done with cuda, https://github.com/JavanTang/code-server
Docker pull registry.cn-hangzhou.aliyuncs.com/test_tzf/test:0.1.7

@JavanTang, @nhooyr explicitly said that they don't need to support every use case - means that you should provide your own stack for your own needs.

This issue is too unclear to resolve. I'm not sure what you're asking. If you're still having a problem, please open a new issue with a more clear description.


fix: running on windows

Added a possibility to run binary on windows and instructions how to install gobackup as a windows service with nssm.

Ok, it seems that I missed a couple of things and more code needs to be fixed to make it work on windows.


Network: IPAddress

getIP method shall return an IPAddress instead of a String. This allows the application to better decide what to do with the information.

@nhjschulz Could you please fix this issue in Network , now that you implemented IPAddress?
https://github.com/BlueAndi/DroidControlShip/blob/17e7f4adf03700e9bc9b47afbc158b3c47bf8eb3/lib/HALInterfaces/src/INetwork.h#L111

Fixed with PR #155


ADD autocomplete project names #completion

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
there are lots of projects in ddev list with long names and common prefixes on my pc
Describe your solution
I type ddev command and type a letter and hit Tab
I get a list of project names that match
Describe alternatives
using the mouse -- oh the humanity!
Additional context
not to be confused with ddev completion, which works well

I don't think this is easy to do, but a contribution is welcome. It sure would be nice!

I've raised https://github.com/ddev/ddev/pull/5732 to address this. Not sure where to put tests and there's an open question there for what (if anything) to do with the --select flag. Details in PR description.