Initial steps from 2021-12-15 mtg

R to do:

[x] Table with Hector parameter, median/default, standard deviation
[x] Step to generate a run list—a data frame with (e.g.) run number, Q10 value, beta value

Science question:

Where does fossil fuel C end up and how does that change with changing parameter values (i.e. assumptions about earth system such as beta, Q10, etc)
Could also reframe this a bit more broadly: how does model system C move around w/ different parameter values
Can also potentially use comparisons with observational data to say something about likelihood of parameter values (kind of Bayesian-esque)
…and thus likelihood of what the earth system functionality is (i.e. goal is not just to infer about Hector, but also real life)
This would leverage a probabilistic (parametric) Monte Carlo approach — e.g. the PDFs of 5 key variables, if we’d like to draw 100 times from each 100^5 = 10^10! Too many :)
Key off of Keenan et al. recent paper
Keenan paper, equivalent one for respiration, some of the probabilistic work noted above, airborne fraction

Note also Chris's Python analysis — in Slack and somewhere on Github

Short-term, "dead end" but still useful:

Let's just look at 2100 for now — keep output data size much smaller
Visualize the output as well as the inputs

Next I would suggest we'd like to re-architect things to be more general.
Generating the run list
Option 1 is the csv approach:
paramlist <- read.csv("paramlist.csv")

runlist <- tibble(run_number = 1:n)
for each row in the paramlist {
 param_name <- paramlist$param_name[I]
 runlist[param_name] <- rnorm(n, paramlist$param_mu[i], paramlist$param_sigma[I])
}

Option 2 is simpler and doesn't involve a csv file:
runlist <- tibble(
 run_number = 1:n,
 beta = rnorm(n, x, y),
 q10 = rnorm(n, x, y)
)

Running Hector
For each row of the runlist... (via group_by...do, for, apply) we'll call run_hector() with that row.
run_hector <- function(pdata) {
 for( each column in pdata ) {
  # get the value and set is via `setvar`
}
 # run the hector code, get data, etc.
}

@leeyap asks: what will be next steps after this?

Timing information. How long does n=10 take? n=100?
Parameter side, we want to identify the candidate parameters we're going to be using and justify the choice. Then we'll want to do some research and reasonable PDFs for them (form: normal/lognormal/something else; mu and sigma)
Parameter side, we'll want to think about joint distributions. Do we want to define any? If so, which, and will need to implement that in code
Output side, in Chris's Python analysis, he produced cool graphs of sensitivity over time. We're going to want to do something I similar
Produce and refine visualizations and numerical summaries of outputs

I think this is okay to be closed


Can't forward port in remote VS Code

VS Code Version: 1.54.1
OS Version: Ubuntu 20.04.1 LTS

Issue: I am trying to forward a port but it keeps telling me its not available. Even though it is available. Its also not forwarded yet.
Screenshot: https://i.imgur.com/mZPnWFJ.png
Steps to Reproduce:

Login to Github Codespace or Remote Environment with VS Code
Try to forward a port

Error message: Unable to forward localhost:8000. The host may not be available or that remote port may already be forwarded
My terminal throws this error even though it wasn't previously in use: Error: listen EADDRINUSE: address already in use :::8000

Does this issue occur when all extensions are disabled?: Yes/No

I'm not able to repro this issue.

My terminal throws this error even though it wasn't previously in use: Error: listen EADDRINUSE: address already in use :::8000

What terminal are you referring to here? I want to understand where this error is coming from.

I'm not able to repro this issue.

My terminal throws this error even though it wasn't previously in use: Error: listen EADDRINUSE: address already in use :::8000

What terminal are you referring to here? I want to understand where this error is coming from.

I am referring to the terminal in vs code that lets you run Linux commands. I am using  Ubuntu 20.04.1 LTS for my remote development environment.
In order to resolve the EADDRINUSE issue I just manually killed the port so that I can run my node js application. Now my terminal is able to run my app and I got a message Express server listening on port 8000.
However, I still can't forward my port and VS Code show this error: Screenshot: https://i.imgur.com/mZPnWFJ.png
I would like to think that something is wrong with my node js code. However, I didn't have this problem before the VS Code Update of Version: 1.54.1 with the same exact codebase.
This port forwarding issue is not present on applications that automatically forward the port for you. For example a simple react app. You can try setting one up.
https://reactjs.org/docs/create-a-new-react-app.html#create-react-app

I am now VS Code 1.55.
I was able to get logs. You can find the logs in my public microsoft one drive folder (https://1drv.ms/u/s!AgwAH2eNQuq2iM4sXYWx2tK-LjUSTg?e=hoFrEe)
@alexr00, Responding to your previous post on Mar 9th. I dont have a left over process from previous time. I compeltely shut down my remote machine and restarted it.
I am still getting this error: Error message: Unable to forward localhost:8000. The host may not be available or that remote port may already be forwarded

The link for the logs doesn't work for me. Could you copy and paste them into a comment here?

The link for the logs doesn't work for me. Could you copy and paste them into a comment here?

@alexr00 there are many lines of logs and I don't want to avoid pasting them here. I put the logs in a public github repo: https://github.com/jigar288/vs-code-logs. Please see the .rtf text files

`Steps to Reproduce:
Login to Github Codespace or Remote Environment with VS Code
Try to forward a port`

@jigar288 thanks for the logs.
@wachaudh and @cmbrose this appears to be an issue with Codespaces providing an undefined tunnel. Is there a place that this issue can be moved to for you to take over investigation?

This sounds like an issue @klvnraju reported which @wachaudh was investigating last week Unable to forward localhost:8000. The host may not be available or that remote port may already be forwarded. Do we have an issue to track that already? If not, let's create one.


update docs

I made some edits to make things clearer and address some concerns (rather than discuss separately in a google doc).
I think DSZ-1 is better for small models on big GPUS.
cc:/ @charlesfrye

Closing this PR in favor of #48 because I don't think PRs from forks are setup to pass CI


Fix typo in VisitsStorageApi.ts

Hey, I just made a Pull Request!
Changed subsription to subscription.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hello @ErikMinekus thank you for fixing this - I believe we need a changeset here so can you generate one as part of this PR? thank you.


Incompatible dependencies with tensorflow

tjax requires numpy>=1.21 which seems to conflict with tensorflow's requirement of numpy~=1.19.2. I don't think needing both jax and tensorflow together is an uncommon use case. Jax functions can be converted to tensorflow (e.g., jax.experimental.jax2tf) which allows the use of, amongst other things, tensorflow's model saving/loading machinery.

Looks like https://github.com/tensorflow/tensorflow/pull/48935 was essentially merged.  Should be good next Tensorflow version.


Fixes #104 - Corrected the contribution guideline link in README.md

In this patch I corrected the link in README.md.

Thank you!


Fix/4146 text fix for consent view

Description
This PR adds an extra legal string to be used in the consent view.
Link to Jira
EXPOSUREAPP-4146
Also, Figma
Screenshot

Text change appears also on the screen "Testergebnis liegt noch. nicht vor" – "Ihr Einverständnis". But this is not intended, this screen shall not change.


Support resp_json_key for primitive types

Description
@versions(supported_versions=(">=20.9"), raises=False)
@request(get, "/v1/feature-profile/sdwan/system/aaa/schema", resp_json_key="request")
def get_sdwan_system_aaa_parcel_schema(self, params: SchemaTypeQuery) -> dict:
    ...

resp_json_key is not being taken into an account when parsing response to dict

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/issues/347


Refactor of 813

This might be a cleaner way to do what #813 accomplished.  Let's see.  If it doesn't work, just close it.

Codecov Report

Merging #817 (412e1e5) into master (f12f10a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #817   +/-   ##
=======================================
  Coverage   77.94%   77.94%
=======================================
  Files          24       24
  Lines        5853     5853
=======================================
  Hits         4562     4562
  Misses       1291     1291

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 f12f10a...412e1e5. Read the comment docs.

ok this approach is just wrong apparently.


[QUESTION] Rendering table without 'columns' property?

Is there option to render table without 'columns' property? I mean my data is fetched from the server (JSON format) and do not have fixed structure. So my data is quite dynamic and I don't want to define columns in my template. Consider having this JSON:
data = {     name: 'Tanner Linsley',     age: 26,     friend: 'Jason Maurer' }
and all I want is just to pass 'data' into  without 'columns' and get this kind of table:

Thanks in advance.

Currently no, but it would be easy to dynamically create columns on the
fly.  Just loop over your data and store a unique list of object keys for
all of the rows. Then you can use those keys to generate a very basic
column list.
On Sat, Apr 15, 2017 at 11:43 AM Illya Balabanov<EMAIL_ADDRESS>wrote:

Is there option to render table without 'columns' property? I mean my data
is fetched from the server (JSON format) and do not have fixed structure.
So my data is quite dynamic and I don't want to define columns in my
template. Consider having this JSON:
data = { name: 'Tanner Linsley', age: 26, friend: 'Jason Maurer' }
and all I want is just to pass 'data' into without 'columns' and get this
kind of table:
[image: screen shot 2017-04-15 at 19 41 49]
https://cloud.githubusercontent.com/assets/17590024/25065661/9de593ca-2213-11e7-9e90-53060f4408e6.png
Thanks in advance.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/issues/201, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFUmCV7cjFOvx2AROOPLwqCox6qw_FDxks5rwQHPgaJpZM4M-aZU
.

Closing for now. If you need more details on implementing this, please join inquire on the slack channel


Update terraform.py

Fixes #986

I think that by v1.0, we should have terraform.py as a submodule, so upkeep is simpler.

@sehqlr Agreed.

This passed a basic install + healthy test. Are there any specific tests for this one?

@sehqlr Nope! This should be ready to merge after we release 0.6.

When we move this to a submodule, where would the clone go? In the root directory?

@sehqlr Let's make an issue for that and discuss there.

I made an issue: #1120

Failure on AWS was quota-related. On GCE, Traefik health checks are failing, but that doesn't seem related. DO build passed.


Add emacs-lisp formatting

Created as a draft because it doesn't quite work; I've tried a couple of approaches but they either don't update correctly or require a second write.
I'm sure I'm implementing it incorrectly, so this is more an RFC than an actual changeset :)

There is an example to re-indent elisp in https://github.com/radian-software/apheleia/pull/63

There is an example to re-indent elisp in #63

That's where I start, doesn't quite work :)

What are the issues you're facing with your current implementation? Maybe I can help advise.

What are the issues you're facing with your current implementation? Maybe I can help advise.

You're right that I don't want to use save-buffer, but if I don't then the buffer is left changed. If I try and use the method in #63 the callback throws a wrong arguments error

Pushed changes, this now works in emacs but the test fails for the slightest diff
diff --git a/tmp/apheleia-ft-file-EDn7yr.expected b/tmp/apheleia-ft-file-Uvnxkx.actual
index 7aeecb4..c0d102c 100644
--- a/tmp/apheleia-ft-file-EDn7yr.expected
+++ b/tmp/apheleia-ft-file-Uvnxkx.actual
@@ -1,4 +1,4 @@
 (if (and (< 3 5)
-         (= 1 1))
+        (= 1 1))
     (message "true")
   (message "false"))

there's an issue with tabs vs spaces

Yeah, that's just because indent-tabs-mode foolishly defaults to non-nil. We can disable it in file-local variables, no need to pull in anything from Doom. (That function is just for fixing up plist indentation, which is a definite improvement but just not the issue at hand here.) I've pushed a commit fixing the issue.

Related to the comment, this makes the indentation very wacky

Thoughts?

Technically, this should work for any lisp. Should I add tests for common lisp and maybe scheme too?

OK scratch that, I was tired when I did that.
Apart from the merge conflict I'm about to resume, are you happy with this?

@raxod502 what's left to do here? the lint failure doesn't align with how the function lints elisp.
Think we should adjust the lint step to utilize what we define here so it dogfoods itself. If desired, I can add that to this PR

I feel bad for letting this sit so long, let me see if I can fix up any remaining CI failures and merge this in.


More information on: Unable to resolve module node:http

Description
I'm trying to make an HTTP request on an API,
The implementation in use works fine (I've been using it for a while now).

i installed node-fetch ^3.2.0 with yarn add node-fetch,
when i import the file above, I immediately get
Unable to resolve module node:http in my the console cmd,
and when i comment the import the error goes away.
Suggestion
I also get this suggestion below.
If you are sure the module exists, try these steps:

Clear watchman watches: watchman watch-del-all
Delete node_modules and run yarn install
Reset Metro's cache: yarn start --reset-cache
Remove the cache: rm -rf /tmp/metro-*

Screenshots

Version
0.64.3
Output of npx react-native info
ystem:
OS: macOS 11.5.2
CPU: (4) x64 Intel(R) Core(TM) i5-6267U CPU @ 2.90GHz
Memory: 252.47 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 16.13.1 - /usr/local/bin/node
Yarn: 1.22.17 - /usr/local/bin/yarn
npm: 8.1.2 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: Not Found
SDKs:
iOS SDK:
Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
Android SDK:
API Levels: 28, 29, 30, 32
Build Tools: 28.0.3, 29.0.2, 30.0.2, 32.0.0
System Images: android-29 | Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom, android-29 | Google Play Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7935034
Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_292 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.1 => 17.0.1
react-native: 0.64.3 => 0.64.3
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce

npm 8.1.2 | node 16.13.1 | expo~44.0.0
yarn add node-fetch
run the code in the image above

Snack, code example, screenshot, or link to a repository

Unable to resolve module node:http in my the console cmd,

You're not supposed to use Node HTTP as it's not available at runtime on React Native. Here a guide on how to do networking with React Native https://reactnative.dev/docs/next/network#using-fetch


Helm 0.3.2 deployment document is incomplete

Describe the bug
This is a bug with the documents on how to deploy Azure IIoT into an existing Kubernetes cluster using Helm package manager.
To Reproduce
Steps to reproduce the behavior:

Go to the current latest stable Helm deployment documentation page,
Click create AAD App Registrations link on that webpage,
ServicesApp and ClientsApp app registration instructions in Azure Active Directory are omitted with a placeholder text "TBD".

Expected behavior
I would expect to see detailed instructions on how to register these 2 applications (ServicesApp, ClientsApp) as recommended by the Helm 0.3.2 deployment documentation page.
Screenshots
There is no need for a screenshot. You can simply visit the links I provided.
Desktop (please complete the following information):
Irrelevant.
Smartphone (please complete the following information):
Irrelevant.
Additional context
No further context needed.

@denizetkar, please check the documentation in the main branch. The link will be fixed in the upcoming release.

@denizetkar, we have release 0.4.0 version of azure-industrial-iot chart. You can find the doc for creating AAD App Registration of that version here. For the latest version of the doc please refer to main branch.


Package repository/documentation link unification

Some translations are duplicated for the same package fields; it should be possible to use the same approach. Checked packages to use the same forms in templates.

Removed repeated translations for the same fields
Linked template files to the same translation fields
Added repository site link for nuget packages

Manually removing the non-English translations could cause problems.
The next commit (https://github.com/go-gitea/gitea/commit/f089be91da44ee3e85fbe326a204c7c8a5df3d31) they all came back.


#1996 - 'ignore properties' are ignored

Adresses #1966

Pull Request Test Coverage Report for Build 5087

10 of 12 (83.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.008%) to 54.364%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

bundle/src/main/java/com/adobe/acs/commons/oak/impl/EnsureOakIndex.java
9
11
81.82%

Totals

Change from base Build 5064:
0.008%

Covered Lines:
14414

Relevant Lines:
26514

💛  - Coveralls

@wimsymons I augmented the PR with some testcases, an updated CHANGELOG and a warning, in case at both locations the IgnoredProperties are configured.

Awesome work @joerghoh! Thanks

This PR for fixing ignored properties is no longer ignored.  I'll do a confirmation build against the lastest master branch and merge today.  Thanks @joerghoh !


Deleting source code in source code editor causes server error

Describe the bug

In the source code editor, if the code is deleted, and you try to build the program, a server error is produced.

To Reproduce
Steps to reproduce the behavior:

Create a new Nepo program
Open the source code editor
Delete all the current code in the editor
Click on the 'Build' button
See error

Expected behavior

A message that says 'Build failed' along with the correct error message.

Screenshots

Device information

Type: Desktop
OS: MacOS
Browser: Chrome

Additional Context
This issue is present in EV3 leJOS

works with a sensible error message


#0: Add trace_region_size parameter to test_perf_trace_2cqs_bare_metal

Ticket

Link to Github Issue.

Problem description

Provide context for the problem.

What's changed

Describe the approach used to solve the problem.
Summarize the changes made and its impact.

Checklist

[ ] Post commit CI passes
[ ] Model regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

Force merging because we need this for perf pipelines to pass.


[fpga] Fix clock constraints

The clkmgr instance got renamed recently. As a result, the clock constraints couldn't be applied, leaving many clocks unconstrained on FPGA.

That remains a rather fragile approach, but good enough for a first fix. We really should look into making prim_clock_div use a PLL for Xilinx devices to save us debugging similar issues more often.
@vogelpi can you either look into that, or file an issue?

Thanks for your review and approval @imphil . I think using a PLL inside prim_clock_div wouldn't work because there are just very few of them available. It would probably be better to abort synthesis/or just create a CI failure if the clock constraints can't be applied, or if there are unconstrained clocks. I can look into that.

It shouldn't be that bad, e.g. on the Nexys Video we have 10 PLLs around. If we want to better utilize them we need a clocking primitive that generates multiple clocks at the same time with a single PLL/MMCM, but that's much more work and likely not worth it at all.

just to clarify, are you guys saying each divider should become a PLL with
it's own divider inside?
Does this imply that the source clock would have to be the same reference
clock?
I'm in favor of flagging a CI error when constraints cannot be applied
On Tue, Feb 16, 2021 at 9:25 AM Pirmin Vogel<EMAIL_ADDRESS>wrote:

Merged #5254 https://github.com/lowRISC/opentitan/pull/5254 into master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/5254#event-4337399086, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSV66QQAS6KCC3AO7DLS7KS63ANCNFSM4XWQF76A
.

I think having one PLL per divider is overkill and I thought we had discussed this option, maybe in a different forum a while back. But I might be wrong. I am also in favor of throwing an error if the constraints cannot be applied.

@tjaychen and @imphil, I've filed a PR to abort FPGA synthesis upon failing clock constraint commands here: #5278 .


Syntax Error @KTropicalQueueFactory

There is a syntax error in the class KTropicalQueueFactory.

It works for me. Where do you see a syntax error in KTropicalQueueFactory and which version of the JDK are you using?


Next major version of Bluebird

Recently a new major version of bluebird (3.0.5) was released.
But your library has strict requirement of peerDependency
Can you update your package.json and also release next major version?
Seems like there are no breaking changes

Yeah I will do this right now.

Just published this to npm as 3.0.0


WIP: Add feature for sending abandoned project mail

Fixes #3382
@ericholscher I have added a button which is displayed when a project meets abandon policy and a user wants to use the name of the project. I am working on linking the button to send an email when it is clicked.

Can you add a screenshot of how this looks?
Also, I think it's better if you put a property to the project model, something like is_abandoned, so we can uses this on others part of the code.

@stsewd Here is an screenshot:

Also, I will put the is_abandoned property to the project model itself!

@ericholscher @stsewd I have added a feature for sending mail to the owner of abandoned projects when renaming is required. If a user enters a name which is already used and the project is abandoned, a button is displayed on top to send mail.

If the user clicks on the button mail will be sent to the owner and user will be redirected to the import page with a Mail sent message.

@stsewd How to run the tests written in test_project.py locally ? I was writing the tests as said by you but was not able to run them locally.

@bansalnitish You can check https://docs.readthedocs.io/en/latest/tests.html, let me know if something isn't clear.

Hi @stsewd, I have written the tests. Can you please have a look and suggest improvements!
Thanks :smile:

@ericholscher I have made the requested changes. Can you have a look once!
Thanks!

As per feedback in #3382, I'm closing this. The branch will still be available for reference if needed.


How to access if ups uses password access?

Tried the normal user:pass@IP and that didnt seem to work. It keeps coming up with unknown UPS. Not sure if I am doing something wrong...

The @ is used for UPS name. Authentication was not supported. I've added it in git master but it's completely untested (I don't have a node-red setup right now). Please try installing the git version, it should have username and password fields.

Ok stupid question... how do I install using git version?

nm _ i got it working -- thanks for fixing the code :)

In Palette is another version than here?

I have the same problem. How did you get it to work @sgrayban?
Is there any chance that this change gets into the official node-red store?

I've released 1.2.0 to npm, so this should be updated now.

Still shows and installs 1.1.0 in "manage pallet"
https://flows.nodered.org/node/node-red-contrib-nut-ups

I have no idea how the flows gallery works, I haven't really touched Node-RED in years… could you contact the Node-RED maintainers about that?


[WIP] Change disks for removal while saving inventory

Amazon provider now has a storage manager responsible for collecting the inventory of available block storage. Once cloud volumes are attached to an existing VM, a link between the volume and a corresponding vm.hardware.disk is established. However, as soon as the cloud manager refreshes its inventory, it will delete any disk not collected by it - since cloud manager is not collecting cloud volumes it deletes all links between volumes and VM disks.
Contrary to this, the storage manager does not delete disks collected by the cloud manager while saving the inventory of disks.
Some more details are available in this doc. The problem is that the cloud manager "collects" ephemeral storage and stores it as a one or more disks. Since OpenStack is using similar approach, I suspect the same issue is present there as well.
Currently, save_disks_inventory is only called from within save_hardware_inventory, but I am still not certain whether the approach here is correct.
@miq-bot assign @Ladas
@miq-bot add_label providers/storage, storage, providers/amazon

Alternatively I tried parsing instance's block_device_mappings provided by AWS SDK in the cloud manager parser. This allows me to create disks for ephemeral and EBS storage in the cloud manager. The storage manager can then just update existing disks with size and link between volume and disk.
The problem: since block_device_mappings doesn't provide the size, I'd need to check if the VM currently being parsed exists, load its existing disks and get the size from there...
@Ladas do you have any other suggestion about this?
@miq-bot add_label wip

@gberginc I think we will need to solve this in the Amazon code, since this code is shared and it already works for OpenStack

Closing because we'll try to resolve this issue in the Amazon provider completely. Thanks @Ladas for your suggestions.


please check my pull requests, thank you!

:heart:

I'm really sorry! I just wrote a message in your fork.
See https://github.com/sakura-ryoko/afkplus/issues/5


Add in next links

@GarrettJMU, nit: Can you add passHref flag also? Build failing cause of TypeError, mad that props.href can be undefined.

You can run yarn build locally to mimic the Deploy behavior.

Trying to be lazy and fix these - but yes that's prob best lol @ctjlewis

Should be good now @ctjlewis

1s

Also added pre-push hooks in #109, so local environment will block push if deploy would fail.


Make PidFile#pid return nil for non-numeric pid file contents

If we return a non-numeric pid from PidFile#pid MiqProcess.command_line will cause ps(1)
to output a usage message.
This usage message was seen during the upgrade process if the server was started
through the command line.
https://bugzilla.redhat.com/show_bug.cgi?id=1238423

:+1:


Branch view window date format

Fixes #159

currently codecov throwing coverage warning on the method that is called by UI, but the original tp also did not test methods used by UI or any UI components whatsoever.


Increment versions for appcomplianceautomation releases

Increment package versions for appcomplianceautomation releases

API change check
API changes are not detected in this pull request.


Adds Romania (bug 1192374).

Tests will fail because I'm still waiting on a review for the marketplace-constants version I'm updating to. I'll make sure they pass before merging.
r?

We should only need the marketplace-constants bump. Hell, we don't actually need it (fireplace is the only repos that need it, for the mcc detection).

Read https://github.com/mozilla/zamboni/pull/3318#discussion_r38597731 to understand why tests are not passing :)

Yep, I know. In a meeting so I haven't been able to loop back yet.

Why did you merge it with that migration ? As said in my comment, it was not necessary (the region had already been added), and it's even dangerous (because it uses non-frozen code in migrations)

Ugh, I had a changeset to make exclude_new_region lazily import a region, stashed it to make a rebase, and forgot to get rid of it. Will do so now, thanks for catching that.

IMHO you should just remove the migration entirely (or well, make it a no-op). There is no need for it, and just importing exclude_new_region in it breaks things since it imports non-frozen code in a migration.


Add karma to the Telegram version

It'd be great to have karma in the telegram.

Basic functionality added.
May be the issue must be (re)opened in case of users inconvenience.


PacketBuffer: Make discontiguous LwIP pbuf adoptable

Changes
Make the discontiguous LwIP pbuf adoptable to reduce the PacketBuffer: pool EMPTY. errors.
Tests
Tested on ESP32, both the unicast and multicast message can be received and handled well.

This fixes #21816, right?
It looks to me like the CloneData bits described in #20923 (comment) are not an issue because CloneData only uses public APIs that this PR fixes, right?

we might run out of PBUF pool if we clone the udp message. After we increase the event queue, the error PacketBuffer: pool EMPTY happens when the ScheduleLambda is block and the udp endpoint receives a lot of packets. Described in #23180 comment.

we might run out of PBUF pool if we clone the udp message.

Yes, I understand that.  My questions are:

Does this fix https://github.com/project-chip/connectedhomeip/issues/21816?
With these changes, is calling CloneData on a non-contiguous buffer that's been adopted (which some of our internals will end up doing) going to do the right thing?

And perhaps most concretely: was receiving group messages tested with this change?

Does this fix https://github.com/project-chip/connectedhomeip/issues/21816?

Yes

With these changes, is calling CloneData on a non-contiguous buffer that's been adopted (which some of our internals will end up doing) going to do the right thing?

Yes, we will create a pbuf with the reserve size 0 in CloneData and copy the payload. Same as https://github.com/project-chip/connectedhomeip/pull/25679/files#diff-d518c09d6ad62381f954eb9520bb175375404f40576a60fa6b25f37bc1c288c0L423.

was receiving group messages tested with this change?

Yes, I have tested the group messages receiving on ESP32.


Migrate to @expo/vector-icons

Pull request for https://github.com/esbenp/react-native-clean-form/issues/41

Hi,
thank you for the feedback and the pull request. Not everybody is using expo and so the dependency to the expo package does not make anything better.
I guess to have the more modular, we can provide the icons as attribute and then the enduser can decide what kind of icon should be used.
What do you think. Then all can decide what kind of package they will use. Only the example app will have the dependency then. But that is not that important.
Greets and happy eastern :)

Agree.
I think we need to extract arrow-down icon from vector-icons for the Select component. And implement some things like renderLeftIcon and renderRightIcon for the Button component, this will be a breaking changes because there is no props icon.

New properties are breaking of course. But making this flexible is worth it. But pay this with another dependency should not the way so the icon property would be nice.
If I find some time I will add this.


Add lans as a virtual relationship to ems_cluster

Lans need to be exposed via the API for the creation of transformation mappings. By adding this virtual relationship, they are able to be queried via /api/clusters/:id?attributes=lans
cc: @bzwei, @priley86
@miq-bot assign @gtanzillo
@miq-bot add_label transformation

@agrare please review

@miq-bot add_label gaprindashvili/yes
Needed by https://github.com/ManageIQ/manageiq-api/pull/325 for gaprindashvili v2v support

Gaprindashvili backport details:
$ git log -1
commit 96fdbc36275d13b05fd7e7ebeecd20db10cd81be
Author: Adam Grare<EMAIL_ADDRESS>Date:   Mon Feb 19 20:04:56 2018 -0500

    Merge pull request #17019 from jntullo/ems_cluster_lans

    Add lans as a virtual relationship to ems_cluster
    (cherry picked from commit 4e212e4b276591eae36c2c8a2a5e80803a4c0828)


PY3: Properly deal with strings in REST/Main

Fixes #9794
Status
ready
Description
I'm still debugging the rucio initialization problem, but there seems to be no issue.
This PR fixes the unicode / string issue reported in the traceback. The traceback is a python2 string, so we need to use BytesIO instead of StringIO.
Is it backward compatible (if not, which system it affects?)
yes
Related PRs
None
External dependencies / deployment changes
None

test this please

test this please

test this please

Thank you very much for this deep review, Dario.
An alternative to your suggestion, how about we import different classes but with the same name (such that all the changes are made at import level instead of in the middle of the code)? Something like:
if sys.version[0] == '2':
                from io import BytesIO as StringIO
else:
                from io import StringIO

Yes, that would work, too.
However, we can consider this an exercise for the gradual migration of WMCore. What if we have module A.py with  from io import BytesIO as StringIO, and module B.py that imports A.py but require from io import StringIO ? Renaming modules is fine if we have isolated modules, but I fear that in the long run it could cause problems. Moreover, since during the migration we will have to deal with such errors a lot, I would prefer to be 100% sure of the type of a variable reading only the line where it is defined. I fear that if we rename modules, then I will make some mess since I forget to check at the beginning of the file if an imported module has been renamed. I know that the expense is less beautiful code, but in this case I would trade less beauty for more clarity.
This being said, code style is a matter of opinions, and your opinion always has the priority

I did some local tests and it would only be a problem if we had (not recommended!) imports like:
from A import *

then those modules would be at the main scope of the imported modules.
While, if one does imports like
import A
import B

this would be completely fine and StringIO (or the masked BytesIO) would be in the scope of the module A (or module B).
The motivation I had to have this py2/py3 logic in the imports section is that when deprecating py2, it would be mostly a matter of imports, without touching the internals of the code.
However, I do agree with you that - if we have problems due to this masked importing - things will be much harder to debug. I'm going to apply your suggestion soon. Thanks again Dario!

@mapellidario I have updated this PR following your suggestion, could you please review it once again?
In addition to this, I guess we should revisit all the usage of BytesIO and implement the same logic we have in this PR. Would you see any possible exception to that?

Well, I would be a bit more conservative and would not jump into applying the same logic everywhere. The solution that we found for this case depends on the interface of traceback, which maintained str in both py2 and py3 despite its change of meaning. A different library could choose a different approach and the only way to be sure is to check it case by case.
Since dealing with strings is the most difficult and troublesome part of the migration, I would leave every aspect of it to the gradual migration. If it were not for your open PR, I would not have suggested any change to this module right now.

Sounds good. Thanks for your feedback, Dario.
This has been tested on my VM (py2, of course) and it works fine.


Branding and use of logo

Requirement to add the FOREST logo to the repository and website

Edit README.md to display link to image
Add branding/logo_640x320.png

Pull request #133 solved this issue


Rfink/retrofit2 api

fyi @iamthomaspowell

lgtm

looks like it needs a rebase before merging

rebased&squashed


Use the best bitrate video for twitter embeds

In the previous repo there was a PR that changed how the bitrate sorting works (robinuniverse/TwitFix#48), although it wasn't implemented correctly which made it pick the worst quality bitrate video twitter had. This is a simple fix to that.
No testing done as I don't have a dev env, but I don't see it causing any issues

Will test in a bit, thanks for the PR :)


Obsolete except condition

https://github.com/pypa/pipenv/blob/bf7203148e1e0f57ab86b652d76135f17bf66927/pipenv/patched/yaml2/reader.py#L163
It take me problem when i run
python3 -m compileall ...

Yaml2 is the code for Python2 and it should be touched on Python3

this code is required for version python=2.5 and earlier
Are you sure you need support for these versions?
In the setup.py anonced support for version 2.7 and higher
Also version of python older than 2.7 not supported for a long time
I propose to replace all
except <exception>, exc:
to
except <exception> as exc:
everywhere
If required, I can make pull-request
Why do I need it:
python -m compileall helps to additionally check the code for syntax errors
and also slightly speeds up the code
And because compileall finds this obsolete code it breaks


fix flutter create

In fixing the tests, I broke flutter create. This fixes flutter create and keeps the test working. Filed a bug on path for relative(). I think we can keep this workaround indefinitely. It only takes effect for temp dirs on the mac.

Test?

lgtm

There is one here for flutter create: https://github.com/flutter/flutter/blob/master/packages/flutter_tools/test/create_test.dart#L31. Happy to add one for the new _relativePath method, if that's what you meant.

I meant to catch whatever broke here that we didn't catch.


A check (linter?) to flag changes to centrally managed files (e.g. CODEOWNERS)

FYI @bharathkkb

Another option could be a header to all the files we manage centrally and how to update them in this repo. Linter sounds good too but I think we will anyway a way to differential in some way if we decide to implement opt out for centrally managed files.

Another option could be a header to all the files we manage centrally and how to update them in this repo. Linter sounds good too but I think we will anyway a way to differential in some way if we decide to implement opt out for centrally managed files.

Sounds good - a header won't actually stop the changes, but as the files are centrally managed it is very easy to implement in comparison.

Completed in https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/pull/1577


Twilio send_custom_json tries to set from attribute instead of from_

Rasa version: 1.7.0
Python version: 3.7
Operating system (windows, osx, ...): Ubuntu
Issue:
I am just testing the capabilities of the Twilio connector and when sending a custom json_message, it throws an error as it tries to set the from attribute instead of the from_ attribute.
Error (including full traceback):
[2020-02-20 15:18:23 +0100] [11856] [ERROR] Exception occurred while handling uri: 'http://570158816806510.ngrok.io/webhooks/twilio/webhook'
Traceback (most recent call last):
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/sanic/app.py", line 942, in handle_request
    response = await response
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/channels/twilio.py", line 132, in message
    metadata=metadata,
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/channels/channel.py", line 83, in handler
    await app.agent.handle_message(*args, **kwargs)
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/agent.py", line 486, in handle_message
    return await processor.handle_message(message)
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/processor.py", line 108, in handle_message
    await self._predict_and_execute_next_action(message.output_channel, tracker)
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/processor.py", line 509, in _predict_and_execute_next_action
    action, tracker, output_channel, self.nlg, policy, confidence
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/processor.py", line 620, in _run_action
    await self._send_bot_messages(events, tracker, output_channel)
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/processor.py", line 549, in _send_bot_messages
    await output_channel.send_response(tracker.sender_id, e.message())
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/channels/channel.py", line 189, in send_response
    await self.send_custom_json(recipient_id, message.pop("custom"), **message)
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/channels/twilio.py", line 70, in send_custom_json
    await self._send_message(json_message)
  File "/home/nicolas/anaconda3/envs/chatbot_sms/lib/python3.7/site-packages/rasa/core/channels/twilio.py", line 37, in _send_message
    message = self.messages.create(**message_data)
TypeError: create() got an unexpected keyword argument 'from'

Command or request that led to error:
dispatcher.utter_message(json_message={
            "body": "Hello! Here is a cat",
            "media_url": "https://i.ytimg.com/vi/uHKfrz65KSU/maxresdefault.jpg"
        })

The problem is that the send_custom_json has a mistake here. It tries to set json_message.setdefault("from", self.twilio_number) instead of json_message.setdefault("from_", self.twilio_number)
Content of configuration file (config.yml) (if relevant): Clone of Sara-bot
Content of domain file (domain.yml) (if relevant): Clone of Sara-bot

@nbeuchat thanks for tracking down the bug, would you be willing to open a PR with the fix?

Sure, happy to do so! Probably the shortest PR ever ;-)

Awesome, thank you!


Check that Chandra regions are being handled properly and masks can be created

This won't be possible until I can get DAXA to make region files from Chandra images, but when it is you'll need to alter your configuration file to point at the region files in that test dataset (I'll let you know when they're sorted).
That should mean that XGA loads in the region files, tries to match to them based on the type of source being declared, and will also be able to exclude contaminating sources from photometric analyses/spectral generation.
Testing this should be simple - you can run the get_interloper_regions() method of a declared source (not a BaseSource though - try a GalaxyCluster) and it should return a set of contaminating regions if everything has been loaded properly.

This shouldn't be the immediate priority (that would be the phot-product-generating functions), but once that is done and I've made some region files for some of the data we have, we need to make sure the image masking is working properly. Then we'll turn our attention to being able to create the regions needed for spectral generation (that'll be another task though).


conda: use chroot instead of bubblewrap

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Please add a comment why bubblewrap is not working

I might have made a mistake, and the issue could have been fixed by #233209, which should be in both master and 23.05 now. @Xe do you mind testing whether conda works without this PR? thanks

closing since I didn't get a response


[quotemark] Excuse more backtick edge cases

This edge cases were previously flagged when they should be ignored, as changing them breaks typescript. This commit makes it so they are ignored. It also organizes a little better, using functions instead of multi-layered conditionals (it was getting confusing).
PR checklist

[x] Addresses an existing issue: #4588, fixes #4645
[x] New feature, bugfix, or enhancement

[x] Includes tests

[ ] Documentation update

Overview of change:
I keep finding other edge cases that I previously missed. I think this is all of them now, but we shall see. Related to #4588.
Is there anything you'd like reviewers to focus on?
See if y'all find any other edge cases.
CHANGELOG.md entry:
[bugfix] Excuse more quotemark backtick edge cases

Any update here? @adidahiya? @JoshuaKGoldberg?

Any update here?

Also if you merge master you'll get the new required CI check, "clean-lockfile"


Windows V1.18.0 x64 - Self signed certifictate in certificate chain

Prerequisites

[x ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Brand new install, first time user. Right when I open the program in regular or safe mode, the +Install screen says 'self signed certificate in certificate chain' in the Featured Packages section and when I try to search. Picture included. So I am unable to install packages. I've also tried from the command line using apm install. I've searched multiple threads that have this issue but all of the solutions already match my setup like checking environment variables and so on.

Steps to Reproduce

Open program
Go to +Install page

Expected behavior: Search and install packages
Actual behavior: Error
Reproduces how often: 100%
Versions
You can get this information from copy and pasting the output of atom --version and apm --version from the command line. Also, please include the OS and what version of the OS you're running.
C:\Users<user>>apm -version
apm  1.18.1
npm  3.10.10
node 6.9.5 x64
python
git 2.10.2.windows.1
visual studio 2015
C:\Users<user>>atom --version
Atom    : 1.18.0
Electron: 1.3.15
Chrome  : 52.0.2743.82
Node    : 6.5.0
Windows 7 x64
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Please see the Atom FAQ entry about self-signed certificates.


DataFrame supports intersect and update readme

Which issue does this PR close?

Closes https://github.com/apache/arrow-datafusion/issues/1082
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Make DataFrame users have the intersect feature and Update the README.
PTAL @alamb @houqp @Dandandan

The ticket has been done!

Looks great -- thank you @xudong963


Properly find EGL and pass any flag defined on the EGL pkgconfig

WPEBackend depends on EGL (it includes eglplatform.h and calls to different EGL APIs) but its not declaring that dependency to CMake.
This is usually not a problem: even if CMake is not aware of this, by including the header things will go fine.
But, when using the EGL libraries from Vivante, the compiler needs also to define the following cflags
-DLINUX -DEGL_API_FB -DWL_EGL_PLATFORM

Otherwise the build will fail.
The Vivante EGL libraries properly define this on their egl.pc file.
We need to inspect this info via pkg-config and pass it to the build.

Today I also hit this issue while trying to make a Wayland-only build. Could we please get this reviewed and merged?


Unable to generate log due to unproper log file directory

Summary
Unable to generate log when the file system is read only.
Version of the used components
Which versions of the Industrial Edge components did you use?

IEDK V1.8 (dm-system v1.6)

Steps to reproduce
With debian squashfs where the rootfs (except for /var /etc , etc) is read-only.
The service tries to generate logs under /usr/bin, which is read-only and by convention not a correct place to generate runtime files.
Try to download the log from IED UI.
What is the current bug behavior?
Error. Unable to generate the log file.

What is the expected correct behavior?
Should be able to download with no service error.
Line of code
https://github.com/industrial-edge/device-kit-system-service/blob/main/internal/systeminfo/systeminfo.go#L328
See PR: https://github.com/industrial-edge/device-kit-system-service/pull/10

add another PR for improving the tests in : https://github.com/industrial-edge/device-kit-system-service/pull/12

@novahs Thank you for your contribution. We will investigate the issue.

Dear @novahs
Thank you so much for your invaluable contribution and support to our project. We appreciate your effort and time dedicated to resolving the issue by providing the code changes.
We are pleased to inform you that we have included your development in the latest release, version release/1.7.1-3. We're grateful for your collaboration.
Best regards,


Prefix glitch with Paper-146

I had set my prefixes for "staff ranks" and set their weight and staff track. Prefixes show up in chat.
Now I made player ranks and when I attempted to set prefixes, they do not show up.
LAN server
EssentialsXChat-<IP_ADDRESS>
Vault v1.7.2-b107
LuckPerms v4.4.27
Paper version git-Paper-146 (MC: 1.14.4) (Implementing API version 1.14.4-R0.1-SNAPSHOT)

Run /lp group <your group> info
If it displays the prefix but not in chat, then it's clearly an issue with either Vault (most unlikely) or EssentialsXChat.

I cleared all my LuckPerms, and Vault data, it is working now.
Thank you, though.


add get it on f-droid link

Adds the F-Droid repo link to the README.md.
One of the packagers finally answered, all that is missing is a new tag, since the fastlane changes are not covered by the most recent one. My recomendation would be:

retagging the latest release to the latest commit
I modify the metadata to point to that instead
F-Droid merges the metadata
this patch here

Thank you for sticking with me through this journey :)

Thanks for following up on this. I've retagged the latest commit as 1.10.4 which should hopefully unblock the process over at F-Droid.
I unfortunately can't accept the commit as is until the app is live on F-Droid, but I'm happy to update it myself once it's ready.


[Info arch] Page layout main

fixes https://github.com/mapbox/dr-ui/issues/332
This PR will be home to the PageLayout redesign "feature branch." The redesign will encompass a number of smaller changes that will each be merged into this branch before merging into the main info arch branch.
How to test

QA checklist

[ ] No errors logged to console.
[ ] Accessibility score in Chrome DevTools Audit/Lighthouse is 100% with Lighthouse version: #.#.#.
[ ] Component is accessible at mobile-size viewport.
[ ] Component is accessible at tablet-size viewport.
[ ] Component is accessible at laptop-size viewport.
[ ] Component is accessible at big-monitor-size viewport.
[ ] Create a PR in a site repo, copy the component, and test it. Push to staging and let the reviewer know they can also test the component there.

Open the test cases app locally on:

[ ] Chrome, no errors logged to console.
[ ] Firefox, no errors logged to console.
[ ] Safari, no errors logged to console.
[ ] Edge, no errors logged to console.
[ ] IE11, no errors logged to console.
[ ] Mobile Safari, no errors logged to console.
[ ] Android Chrome, no errors logged to console.

I don't think this intermediary branch is needed anymore, so I rebased https://github.com/mapbox/dr-ui/pull/344 to info-arch-main.


StaticFileMiddleware now passes over to the next middleware in the pi…

…peline when invalid characters are encountered. Fixes issue #84

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

Hmm, I'm guessing on Linux the < is valid which causes my unit test to fail. Should I replace the < character with Path.GetInvalidPathCharacters()[0]?
What do you recommend @davidfowl?

/cc @Tratcher PTAL

Let me know when you want me to squash my commits.

Just minor comments, then squash it.

@Tratcher Squashed. AppVeyor continually queued for some reason, Travis failed with the message

Failed to fetch http://security.ubuntu.com/ubuntu/dists/trusty-security/main/i18n/Translation-en  Hash Sum mismatch

Sorry, I was thinking about this some more and I think we've applied the fix to the wrong layer. StaticFilesMiddleware is abstracted over different kinds of file systems via FileProviders. The implementations we ship are for Physical files and Embedded files. The list of invalid characters is different between these two implementations, and probably for others as well. Checking for invalid characters is the responsibility of those implementations, not this middleware. Hence https://github.com/aspnet/StaticFiles/issues/84.
In this instance the only call to a Path API in this repo is inside of the IContentTypeProvider implementation FileExtensionContentTypeProvider. Since that's where the problem is isolated to, that's where we should apply the fix. Otherwise we risk blocking legitimate scenarios. I think we need an implementation of GetExtension that's agnostic across file providers and doesn't choke on weird characters. It just needs to split on the last dot.

Sure, well done catching that. Where should we go from here? I'd like to take a stab at that change if possible.

Go ahead and try it. You can re-use this PR or create a new one.

@Tratcher Wondering what to do with the unit test for this change as the only invalid path character on Unix based systems is the string terminator \x00. So I think if I try to add the following unit test, the Travis build will fail:
[Fact]
public void InvalidCharactersAreIgnored()
{
    var provider = new FileExtensionContentTypeProvider();
    string contentType;
    Assert.True(provider.TryGetContentType($"{System.IO.Path.GetInvalidPathChars()}.txt", out contentType));
}

Because the string passed in to TryGetContentType will appear empty (i.e. \x00.txt).

$"file{System.IO.Path.GetInvalidPathChars()}.txt"? C# allows nulls in strings, it doesn't terminate them.

Ah, that's odd. I've had an issue years ago where a null terminator in the middle of an XML file caused the contents to be improperly parsed, that and the inspector doesn't display characters after the null terminator. You learn something new every day :)
I'll close this pull request and start a new one.


Stop logging messages

Hi, what is the easiest way to stop logging unnecessary data like messages? As we only use the project for sending messages and saving the incoming ones takes unnecessary data space.

I also need info about this.

Do you know how to prevent messages from being deleted from the bank when I read them?


CLI: create the right tailwind.config after selecting typescript

First of all, i love your UI its awesome, thanks for that.
If I installed the UI with the CLI, it would be great to create the tailwind.config.js based on the decision if the project uses JavaScript or Typescript. So in my opinion it the tailwind config file should (if typescript is selected) be created as tailwind.config.ts (instead of tailwind.config.js)

This seems doable from a technical perspective, so I suspect there's a philosophical reason to avoid it.
You'd have to add new typescript templates for the tailwind config in packages/cli/src/utils/templates.ts, ideally following the format of appending _JS to the existing non-typescript versions, and adding new templates with the current name.
You'd then have to tie in the new templates based on the config.tsx flag to this logic:
https://github.com/shadcn-ui/ui/blob/1f16cf47288c445ff54b41740fd89d35473ac46c/packages/cli/src/commands/init.ts#L232-L238

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Unable to upgrade to Ghost 5 from 4.48

Issue Summary
Unable to upgrade from 4.48
Steps to Reproduce
Just ghost update and... not functionnal
Ghost Version
4.48
Node.js Version
v14.19.1
How did you install Ghost?
Docker
Database type
MySQL 8
Browser & OS version
Alpine Linux, v3.14.6 (container)
Relevant log / error output
the changelogs for full details:
* Ghost - https://github.com/tryghost/ghost/compare/v5.0.0...v5.0.1
* Admin - https://github.com/tryghost/admin/compare/v5.0.0...v5.0.1

🪄 Love open source? We're hiring [Node.js Engineers](https://careers.ghost.org/product-engineer-node-js) to work on Ghost full-time

✔ Fetched release notes
✔ Downloading and updating Ghost to v5.0.1
✖ Updating to a major version
An error occurred.
Message: 'SELECT value FROM settings WHERE `key` = "active_theme"; - SQLITE_ERROR: no such table: settings'

Code of Conduct

[X] I agree to be friendly and polite to people in this repository

It's a bug from your tool, it upgrade all v4 versions and now it can't detect my database in settings.... It watch about SQLite... I'm in Mysql8 and I have active_theme key

That's YOUR docker image I use

The Docker image is Docker's official Ghost image, and is maintained by the community.


Revert "regression tpcds sf100 plan shape regression cases…

… (#19913)"
This reverts commit a6674bb7b1a291097eb9f7027e58a6735d5b2bed.
Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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...

run buildall


Adding sharp package for Image Optimization

We need to decide if we want to add shap package for image optimization.
ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
Warning: For production Image Optimization with Next.js, the optional 'sharp' package is strongly recommended. Run 'yarn add sharp', and Next.js will use it automatically for Image Optimization.
Read more: https://nextjs.org/docs/messages/sharp-missing-in-production

If we stay on Vercel I think that no work needs to be done. According to https://nextjs.org/docs/messages/sharp-missing-in-production "Note: This is not necessary for Vercel deployments, since sharp is installed automatically for you.". We can close this issue.


Thumbnail component page missing from docs menu.

This page:
http://foundation.zurb.com/emails/docs/thumbnail.html
is not included in the documentation component side menu.
It's a useful component, and I stumbled upon it accidentally while browsing the CSS file.

Right now the thumbnail component was a concept for emails, though it is not fully implemented across all email clients. The code is there but there are issues across the board. We'd like to add it officially, but can't until it tests out.
Is this something you can jump in on? How did it work for you?

In Apple Mail the border is seen, in GMail it's not.

Also the hover effect is lost once CSS is inlined. I don't think it's a thumbnail issue, because the effect is lost with buttons as well.

Yeah the hover part is an open issue on https://github.com/jonkemp/inline-css - still looking to resolve that separately
The thumbnail border showing on all email clients is key.

Closing in favor of https://github.com/zurb/foundation-emails/issues/220


Replace == with = in documentation

I found the documentation confusing because the extra =. anyhow = "=1.1.0" does seem to be valid in Cargo.toml but I don't think it is ever used because I don't see how it is different to anyhow = "1.1.0".
I replaced the test with one using ^ as I've seen that used in Cargo.tomls before so it's probably more worth testing than =.
Hope this is helpful. Thanks for creating a great tool.

Oh thanks, lemme take a look

I found the documentation confusing because the extra =. anyhow = "=1.1.0" does seem to be valid in Cargo.toml but I don't think it is ever used because I don't see how it is different to anyhow = "1.1.0".

The extra = is for an exact version. See the doc here: https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#comparison-requirements
1.2.0 can match 1.3.0 but =1.2.0 can only match 1.2.0.
I think it would be nice to add a bullet point before the one with the exact version as this is the most common use case.

I replaced the test with one using ^ as I've seen that used in Cargo.tomls before so it's probably more worth testing than =.

When there is no comparison, the caret is implicit. So it's not very useful but the exact version is useful. (Explained here https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-cratesio )

I made some improvements, lemme know if its fine for you.

I can add more informations about the comparison but i'm not sure how do we call ">" and "<". Like maximal or minimal version or something ? @cecton

The best README are complete enough but without too much information. You can add the link to the doc of cargo if you want but I wouldn't mention the other comparisons, not even ^. They're not very useful and knowing that you can put anything after the first = is already enough.

So, let's merge
Thanks both for the PR and the review


fix: panic index out of range for invalid series keys [Port to 2.7]

This is a cherry-pick of 6af0be9 (introduced in https://github.com/influxdata/influxdb/pull/24565) for a port the 2.7 branch.
Original pull request comment:
Panics can be avoided by checking the validity of the return from ReadSeriesKeyLen, at the moment the size check resulting from this function is entirely discarded. The contained call to binary.Uvarint contains an implicit error, depending on the returned value:

If an error occurred, the value is 0 and the number of bytes n is <= 0

In every occurrence of ReadSeriesKeyMeasurement, the ReadSeriesKeyLen function is called prior to it, determined by glancing over the output of rg "ReadSeriesKeyMeasurement" -B 5 across the full codebase.
This PR refactors the flow slightly to include ReadSeriesKeyLen returned value checks and the appropriate action to avoid panicking when trying to access memory out of bounds, it has a knock-on effect such that the ParseSeriesKey function can return nil at another point which indicates an invalid result and the appropriate action should be taken.
Closes: https://github.com/influxdata/influxdb/issues/24598

[ ] I've read the contributing section of the project README.
[ ] Signed CLA (if not already signed).

I also encountered this issue in version 1.8. Can you tell me the cause of the entire panic? Is it due to corrupted TSI index data？

@ningque123456 - This was a bug in Influxdb.

ok , Is there any other solution besides upgrading (which might not be feasible for me in the short term)? For example, deleting all _series files and restarting InfluxDB?

That sometimes helps
On Tue, May 21, 2024, 18:12 ninngQue @.***> wrote:

ok , Is there any other solution besides upgrading (which might not be
feasible for me in the short term)? For example, deleting all _series files
and restarting InfluxDB?
—
Reply to this email directly, view it on GitHub
https://github.com/influxdata/influxdb/pull/24599#issuecomment-2123695682,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARIQHJCQUIRHVAC6FQCVUV3ZDPWIXAVCNFSM6AAAAABCGWTYZ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRTGY4TKNRYGI
.
You are receiving this because you commented.Message ID:
@.***>

ok ,thanks


Recaptcha not showing dependent on url encoding.

Issue description

Hey all not sure is this the correct place, but couldn't find anywhere else to report client recaptcha problems.
When a url contains certain characters e.g % recaptcha fails to show.
Environment

OS name and version:
PHP version:
Web server name and version:
google/recaptcha version:
Browser name and version:

Reproducing the issue

URL (optional): https://recaptcha-demo.appspot.com/recaptcha-v2-invisible.php%a

User steps

Visit page https://recaptcha-demo.appspot.com/recaptcha-v2-invisible.php and notice recaptcha on bottom right.

Visit page https://recaptcha-demo.appspot.com/recaptcha-v2-invisible.php%a and notice no recaptcha.

Also happens on this page. https://www.globetrotter.ch/de/3286/Reisevorträge-�Unterwegs-mit-�-|-Globetrotter.htm

I had the same problem and i've found a solution for me - using encodeURIComponent (not url encoding!) for generating my urls and everything works good

I had the same problem and i've found a solution for me - using encodeURIComponent (not url encoding!) for generating my urls and everything works good


Create struct direct

Drop constructor

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

Project coverage is 39.64%. Comparing base (49a963e) to head (1236c45).

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
- Coverage   39.78%   39.64%   -0.14%
==========================================
  Files          20       20
  Lines        1478     1478
==========================================
- Hits          588      586       -2
- Misses        810      812       +2
  Partials       80       80

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


599 - Extend Bookings

Add Extension tables to DB - extend the GET /premises/{id}/bookings endpoint to return the current extensions in the body.
Add POST /premises/{id}/bookings/{booking id}/extensions endpoint to create an extension:

Extension must be to a date later than the current departure date

fedb60362ab615557fc4e3f030486520d6a3d461 seems to be an empty commit BTW 😃


index.html.in

The german index.html.in doesn't contain @CUPS_VERSION@ for the version string. It still says CUPS 2.0.2.

For what version? I don't see that in the master or 2.1 branches in Git...

https://github.com/apple/cups/blob/master/doc/de/index.html.in
line 26

OK, got it and a fix for the master branch is now merged.


Won't Compile with calloop feature; Compiler reports Loop Handles with Distinct Types

I anticipate needing multiple event sources in my application so I tried using calloop and with a WaylandSource and a Timer source for testing.
However when I merged the example code from wayland-client documentation and the sample code from calloop, with what appear to be minimal required changes so they share the same data type ("Game") it does not compile due to a strange error and rustc seems to produce an incorrect error report (it mentions a definition on a line and when I go to that line something else entirely is there):

error[E0308]: mismatched types
--> src/main.rs:37:44
|
37 |     let wl_event_src_token = wl_src.insert(evl_handle)
|                                     ------ ^^^^^^^^^^ expected LoopHandle<'_, Game>, found a different LoopHandle<'_, Game>
|                                     |
|                                     arguments to this method are incorrect
|
= note: LoopHandle<'_, Game> and LoopHandle<'_, Game> have similar names, but are actually distinct types
note: LoopHandle<'_, Game> is defined in crate calloop
--> /home/xxxx/.cargo/registry/src/index.crates.io-6f17d22bba15001f/calloop-0.11.0/src/loop_logic.rs:68:1
|
68 | pub struct LoopHandle<'l, Data> {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
note: LoopHandle<'_, Game> is defined in crate calloop
--> /home/xxxx/.cargo/registry/src/index.crates.io-6f17d22bba15001f/calloop-0.10.6/src/loop_logic.rs:44:1
|
44 | pub struct LoopHandle<'l, Data> {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
= note: perhaps two different versions of crate calloop are being used?
note: method defined here
--> /home/xxxx/.cargo/registry/src/index.crates.io-6f17d22bba15001f/wayland-client-0.30.2/src/calloop.rs:54:12
|
54 |     pub fn insert(self, handle: LoopHandle) -> Result<RegistrationToken, InsertError>
|            ^^^^^^

To reproduce simply start a new cargo project, add the dependencies to Cargo.toml:
calloop = "*"
wayland-client = { version = "*", features = [ "calloop" ] }
wayland-protocols = "*"

And then the src/main.rs:
use calloop::{EventLoop, timer::{Timer, TimeoutAction}};
use wayland_client::{protocol::wl_registry::{WlRegistry, Event},
                     Connection, WaylandSource, Dispatch, EventQueue,
                     QueueHandle};
struct Game {
    _dummy: u32,
}

impl Dispatch<WlRegistry, Game> for Game {
    fn event(_: &mut Self, _: &WlRegistry, event: Event,
             _: &Game, _: &Connection, _: &QueueHandle<Game>,) {
        if let Event::Global { name, interface, version } = event {
            println!("Wayland event: {} {} v{}", name, interface, version);
        }
    }
}

fn main() {
    static mut game: Game = Game { _dummy: 0u32 };

    let mut event_loop = EventLoop::<Game>::try_new()
                .expect("Failed to create event loop");
    let evl_handle = event_loop.handle();

    let cxn = Connection::connect_to_env().unwrap();
    let display = cxn.display();

    let mut wl_q: EventQueue<Game> = cxn.new_event_queue();
    let wl_qh = wl_q.handle();

    let _wl_registry = display.get_registry(&wl_qh, game);

    let wl_src = WaylandSource::<Game>::new(wl_q)
                .expect("Failed to create wayland source for event loop");

    let wl_event_src_token = wl_src.insert(evl_handle)
                .expect("Failed adding wayland event queue to event loop");

    let timer_period = std::time::Duration::from_secs(1);
    let timer_src = Timer::from_duration(timer_period);

    evl_handle.insert_source(timer_src,
                    |event, _meta, shared_data| {
                        event_loop.get_signal().stop();
                        TimeoutAction::ToDuration(timer_period)
                    })
                .expect("Failed to insert timer in event loop");

    event_loop.run(
                    std::time::Duration::from_millis(16),
                    &mut game,
                    |&mut g| {
                        wl_q.flush().unwrap();
                        match wl_q.dispatch_pending(&mut g) {
                            Err(err) => {
                                println!("Wayland event dispatch error: {:?}", err);
                            },
                            Ok(_) => {},
                        }
                    }
                ).expect("Error during event handling");
}

While I'm not entirely sure I correctly merged the example code I can't see why the rust compiler thinks these types are distinct and this seems to be what the documentation for both projects suggested doing.
Even if I'm missing something important and this issue is readily closed, perhaps the (fixed?) multi-source calloop sample above is a good candidate for wayland-rs/wayland-client/examples/, a test case, or a note in the documentation mentioning what's gone wrong if folks see this confusing error.

Aha, looks like it's indeed a multiple-version issue:

calloop = ""
wayland-client = { version = "", features = [ "calloop" ] }

cargo tree

shows calloop v0.10.6 was selected for wayland-client while a more recent version was selected for the main project. Despite this the compiler referred to the same file as if it were the source of the both definitions. The fix is to specify the same version as wayland-client uses
(and there's a ton wrong with the sample code that then shows up so my apologies for suggesting poor code)


Adobe logo not showing up as expected when retrieved from the federal project

Description
Ensure centralized images always show up - also on non-centralized navigations. Authors should be able to freely link to centralized assets, such as the adobe logo which is used across a lot of consumers.
Related Issue
Resolves: MWPW-143239
Testing instructions
Images should always show up, in all of those scenarios.

Consumer based navigation + consumer based SVGs used
Consumer based navigation + centralized SVG
Centralised navigation + centralized SVG

a SVG/image can be a promo element within a menu, a logo in the brand block or in the logo block of the navigation.
Screenshots (if appropriate):

Test URLs
Homepage where the issue was raised initially:

Before: https://main--homepage--adobecom.hlx.page/homepage/index-loggedout?martech=off
After: https://main--homepage--adobecom.hlx.page/homepage/index-loggedout?martech=off&milolibs=fix-federal-svg-bug--milo--mokimo

Acrobat:

Before: https://www.stage.adobe.com/acrobat/online/sign-pdf.html?martech=off
After: https://www.stage.adobe.com/acrobat/online/sign-pdf.html?martech=off&milolibs=fix-federal-svg-bug--milo--mokimo

BACOM:

Before: https://business.stage.adobe.com/fr/customer-success-stories.html?martech=off
After: https://business.stage.adobe.com/fr/customer-success-stories.html?martech=off&milolibs=fix-federal-svg-bug--milo--mokimo

CC:

Before: https://main--cc--adobecom.hlx.live/?martech=off
After: https://main--cc--adobecom.hlx.live/?martech=off&milolibs=fix-federal-svg-bug--milo--mokimo

Blog:

Before: https://main--blog--adobecom.hlx.page/
After: https://main--blog--adobecom.hlx.page/?milolibs=fix-federal-svg-bug--milo--mokimo

Milo:

Before: https://main--milo--adobecom.hlx.page/ch_de/drafts/ramuntea/gnav-refactor?martech=off
After: https://fix-federal-svg-bug--milo--mokimo.hlx.page/ch_de/drafts/ramuntea/gnav-refactor?martech=off

@sigadamvenkata @3ch023 @Blainegunn Could we get this merged next? GWP wants to deliver centralized navigations by march 6th, and this is required for the release.


remove table prefix

Without a customizable table prefix, it will be easier to handle schema migrations.
(breaking change: datamodel)

to be addressed later


Crash on iOS when navigating too quickly with a11y means

For example, If you change from one screen to another and then quickly try to focus on something on the new screen, the app crashes (no useful exception is given).

This is fixed in the engine. No code changes are necessary on the framework side. The fix should be automatic next time the engine is updated.


Layout issues on front end

Bug report
Preconditions
1. framework Version. Master

Steps to reproduce:
1-  on product view page - https://prnt.sc/yxj9kh
2-  search product in mobile view, images are not coming properly - https://prnt.sc/yxkzdm while actual image is -https://prnt.sc/yxl8rh

Starting with these two,

[ ]  On product view page -> https://prnt.sc/yxj9kh
[ ]  Search product in mobile view, images are not coming properly -> https://prnt.sc/yxkzdm while the actual image is -> https://prnt.sc/yxl8rh

product search option is not working properly in mobile view
open shop in mobile view
search any random keyword
it will show all products while in web it will show Whoops!
No Results Found

@devansh-webkul please fix wishlist or remove icon alignment in cart https://prnt.sc/z6jut5

@vaishaliwebkul,
This seems to be working fine.


CB-11621 Add<EMAIL_ADDRESS>release blog post

@sgrebnov, @TimBarham, could you please take a look

Couple of comments. Other than that looks good to me.

LGTM


Update error directions to point to Reporting Bugs section

Homebrew Cask should error out and point to the documentation online, instead of blindly suggesting a string of commands. This also lets us be reactive to changes and potentially catch errors and change directions on the fly, as needed.

Absolutely in support of this. That string of commands hasn’t been serving us as it used to for a while, now.

I am interested in working on this. Is there a particular location in the source where the string of commands is located, a format for the new messages and a place to point to? I saw the docs repo but it seems a bit odd to point people to a repo of markdown files

I think if you search the source for UPDATE_CMD you will find this message.

@narendasan https://github.com/Homebrew/brew/blob/master/Library/Homebrew/cask/lib/hbc/utils.rb#L7
https://github.com/Homebrew/brew/blob/master/Library/Homebrew/cask/lib/hbc/utils.rb#L146
Should be changed to say something like: Please follow latest commands at https://github.com/caskroom/homebrew-cask/blob/master/doc/reporting_bugs/pre_bug_report.md
Maybe one of the other maintainers has a shorter subset of commands they want to put there?
But personally, I'd be OK if we just pointed people to https://github.com/caskroom/homebrew-cask#reporting-bugs - you'd need an internet connection to use HBC, anyway.

So I was looking around and found these references as well:
https://github.com/Homebrew/brew/blob/master/Library/Homebrew/cask/lib/hbc/artifact/base.rb#L52
https://github.com/Homebrew/brew/blob/master/Library/Homebrew/cask/lib/hbc/artifact/uninstall_base.rb#L90
Should these lines be changed as well (probably to something other than the bug reporting docs)?

Also I ran brew cask-tests and the version test fails. Is this expected or am I missing something


[plot] Function to convert data to colors with a colormap

This PR adds a function applyColormapToData which convert a numpy array to an array of RGBA colors by using a colormap. Data can be of any dimension.

LGTM


Will it conflict installing by soucecode ,with installing by pip

I have install tensorflow by pip.
And Because I have to add some code in tensorflow ,so I need to build tensorflow by sourcecode.
And when run bazel build -c opt --config=cuda //tensorflow/cc:tutorials_example_trainer
it will report
configure: error: zlib not installed
Target //tensorflow/cc:tutorials_example_trainer failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 2.743s, Critical Path: 2.24s
But in fact ,I have install zlib.
INFO: Elapsed time: 2.743s, Critical Path: 2.24s
[@sjs_88_31 tensorflow]# yum install zlib
Loaded plugins: fastestmirror, langpacks
Loading mirror speeds from cached hostfile

epel: mirrors.neusoft.edu.cn
_Package zlib-1.2.7-13.el7.x86_64 already installed and latest version
_Nothing to do
So how can I fixed it?
My Os is cenos7.1.

And how can i add souce files if just only using pip install?

Please run with --verbose_failures and post the results.
I am not sure whether Centos 7.1 is supported by bazel. You can also file a bazel bug to get more help on bazel issues.

Closing for now due to insufficient information.  @liumilan: I am happy to reopen if there are more details.


Request : Text Box - Padding Option via Settings

Option to add a padding for the Text Box (similar to font-size/width-height adjustment). It looks good and minimalistic as it is now, just thought that adding a padding for the Text Box would make it nicer to look at.
If you can't add it as an option, could you maybe make it a default style with about 5 ~ 10px padding for the Text Box?
Thanks and Good Job for making this add-on!

Sorry for not getting to this sooner.
A padding option has been added in the new version (1.2.0) released today. The default value is 5px.

Looking good now. Thanks!


hand detection

Hi, thanks for the great work! May I ask which model you used to obtain hands_boxes in the ControlNeXt-SVD-v2-Training/meta_info_example/meta_info/1.json file?

Have you solved this problem? I see you closed the issue.


Websocket based on headers

Do you want to request a feature or report a bug?
Bug
What did you do?
https://www.haproxy.com/blog/websockets-load-balancing-with-haproxy/
want to connect to websocket using http headers, similar to what is provided by haproxy :
frontend public
  bind *:80
  ## routing based on websocket protocol header
  acl hdr_connection_upgrade hdr(Connection)  -i upgrade
  acl hdr_upgrade_websocket  hdr(Upgrade)     -i websocket

  use_backend bk_ws if hdr_connection_upgrade hdr_upgrade_websocket
  default_backend bk_web

What did you expect to see?
using socket.io
The client is able to connect directly without traefik
client --> (ws-http) --> server
But fails
client --> traefik(wss-https) --> (ws-http) -->  server
What did you see instead?
the client starts to poll and then fails with 404
Output of traefik version: (What version of Traefik are you using?)
1.3.5
What is your environment & configuration (arguments, toml, provider, platform, ...)?
using docker swarm on ubuntu 17
[entryPoints]
		[entryPoints.http]
		address = ":80"
		[entryPoints.http.redirect]
			entryPoint = "https"
   	[entryPoints.https]
   		address = ":443"
     	[entryPoints.https.tls]

[acme]
entryPoint = "https"
onDemand = true
#...
[file]
watch = true

  [backends.b_socket]
    [backends.b_socket.servers.server1]
    url = "http://m_chat:80"
    weight = 10

  [frontends.f_socket1]
   entrypoints = ["http","https"]
   priority = 8
	 passHostHeader = true
   backend = "b_socket"
   	[frontends.f_socket1.routes.test_1]
   	rule = "Headers:Connection, Upgrade"

  [frontends.f_socket2]
   entrypoints = ["http","https"]
   priority = 8
	 passHostHeader = true
   backend = "b_socket"
   	[frontends.f_socket2.routes.test_1]
   	rule = "Headers:Upgrade, websocket"

  [frontends.f_socket0]
   entrypoints = ["http","https"]
   priority = 8
	 passHostHeader = true
   backend = "b_socket"
   	[frontends.f_socket0.routes.test_1]
   	rule = "Path:/socket.io"

# few other rules based on path

If applicable, please paste the log output in debug mode (--debug switch)
time="2017-08-10T08:37:38Z" level=debug msg="Round trip: http://m_chat:80, code: 404, duration: 1.775782ms tls:version: 303, tls:resume:false, tls:csuite:c02f, tls:server:xxx.com"
time="2017-08-10T08:37:40Z" level=debug msg="Round trip: http://m_chat:80, code: 404, duration: 2.754365ms tls:version: 303, tls:resume:true, tls:csuite:c02f, tls:server:xxx.com"

Any possibility / workarounds for this ..

Can you try with the latest version of traefik (1.3.7) ?
If it doesn't work with 1.3.7, can you try with a traefik without https ? client --> traefik(ws-http) --> (ws-http) --> server
Thx

I'll close the issue due to inactivity but feel free to re-open if needed.

I want to know if traefik support socket.io that upgrades from http to websocket.


refactor: avoid duplication between AbstractResult subclasses (#1050) V2

What this PR changes/adds
Same as #1050 that was reverted to permit to #1088 to be merged quickly.
Why it does that

Further notes

Linked Issue(s)
Closes #1050
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[ ] added relevant details to the changelog? (skip with label no-changelog)
[ ] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

@jimmarino done, ready for merge


Converted toc.yml from existing toc.md

Hi @ManikaDhiman - As requested the TOC has been converted into YAML. Please review.

#sign-off

@ManikaDhiman: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@v-hearya: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Add Photo Supreme for PostgreSQL v3

Add the multi-user / multi-machine version of Photo Supreme v3
with the PostgreSQL v9 backend.

Made all changes requested

Could you please squash your commits?

Thank you for the contribution. It needed some fixes, so they were made in e1a6ec353a8087e63177e4edd8cab018f9308529. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


feature/combined-chips

Codecov Report

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

@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage        ?   96.49%
=======================================
  Files           ?        7
  Lines           ?      114
  Branches        ?       24
=======================================
  Hits            ?      110
  Misses          ?        4
  Partials        ?        0

Continue to review full report at Codecov.

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


Added Simulation DDMS API

Added Simulation DDMS postman collection API.

@dfcarpenter - can you please review and approve the PR?


Add SUSHI testing for BMCs from OpenCUBE

Reassign to appropriate HPE folks after the dev summit

@dev-zero

Just out of curiosity, what is SUSHI testing? Is there somewhere that explains what it is?


Remove dead code in crypto/pkcs7

aww, c'mon, you don't want to leave this for future incentives? :)


Support Cucumber Expressions, ref #1002

Summary
Support for Cucumber Expressions.

[x] Basic support
[x] Deprecate Transform (but try to make it work with current API)
[x] Add API for defining custom parameter types
[x] Print snippets as Cucumber Expressions
[x] Print alternative expressions above snippets

Because the Cucumber Expressions library can generate multiple expressions for an undefined step,
snippets contain all of them, with all but one commented out:
Given("I have {float} {cucumis} in my belly") do |float, cucumis|
# Given("I have {float} {veg} in my belly") do |float, veg|
  pending # Write code here that turns the phrase above into concrete actions
end

Checklist:

[x] I've added tests for my code
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I think this is pretty much done, apart from making a new release of Cucumber Expressions. I'm really excited about putting this in the next release.
Any reservations/objections/questions?

I've released cucumber-expressions 4.0.3. This can be merged to master now. Any objections or questions?

Hi @aslakhellesoy,
Thanks for your contribution and welcome to the committers team!
You can now push directly to this repo and all other repos under the cucumber
organization.
Please continue to use branches and pull requests. When someone on the core team
approves a pull request, you're welcome to merge it yourself.
Cheers,
Aslak Hellesøy
Creator of Cucumber

Hi @aslakhellesoy,
Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾
In return for we ask you to:

✅ Please continue to use branches and pull requests. When someone on the core team approves a pull request (yours or someone else's), you're welcome to merge it yourself.
💚 Commit to setting a good example by following and upholding our code of conduct in your interactions with other collaborators and users.
💬 Join the community Slack channel to meet the rest of the team and make yourself at home.
ℹ Don't feel obliged to help, just do what you can if you have the time and the energy.
🙋 Ask if you need anything. We're looking for feedback about how to make the project more welcoming, so please tell us!

On behalf of the Cucumber core team,
Aslak Hellesøy
Creator of Cucumber

Wow thanks @aslakhellesoy!

@aslakhellesoy So what's left on aruba's side is add the cucumber expression steps as well correct?


insert columns are not quoted

when trying to insert eg:
$db->insert('table',
array(
'accountID' =>  27,
'order' => 123
)
);
its not executing becouse order is not quoted. Works fine in update etc..
Please fix.

I have the same thing with the latest version 1.1.2.
Using insert command, the output generated by Medoo is:-
(key, value, userid) VALUES ('TEST', 'This is a test', '0')
As you can see the column names are not is quotation marks. From my object the input was:-
["key" => "TEST", "value" => "This is a test", "userid" => "0"]
Worked fine with passed 1.0.2.
Thanks

i have the issue after an update to 1.1.2.


RFC: Restrict number of consumed messages

Idea: Have a receiver that only consumes 'max_messages' and shuts down automatically after that.
The corresponding test fails on every second test run with ConnectionClosed Exception. The former test_consumer_timeout always hangs always in the register call and must be stopped with Ctrl-C (I've tried several rewrites of this test and it always hangs). It looks like that there another test that messes up the state somehow.

Seems to be quite old and not used...


first feedback

Yay, looking good - i'll give it a whirl tonight.
my 5ct from first looks:

love the offsets tab!
jogging and feed-radios don't keep their setting and i get an infinite "you have to select them" loop
is there a "home" button somewhere?
connect/port settings can be moved from front-panel to settings panel - "connect on load" does the job for me - bring a dialog to front only when there's no port/no grbl
what screen size is it designed for? scaling the app only works vertically.
does it remember window position (+size)
the jogging is very sensitive (with a mouse) - it's hard to move just 1 unit. will test repeat rates @ real hardware
what's the idea behind "special positions"? tool changing position etc

Closing this.
I investigated remembering location etc and it seems a lot of effort. Might revisit once other enhancements are completed.
Thank you for your inputs.


UI localization - hu

Added hungarian to the list of languages - resource file can be pulled from transifex

Hello,
Thanks for your PR.
The file plugins/hypervisors/kvm/pom.xml don't need to be modify to add the Hungarian language (mistake?)
It's would be better to add this language on master branch, and cherry-pick after on 4.5 / 4.4 branches.
File tools/transifex/sync-transifex-ui.sh must be modify to add the ISO code of Hungary
In line:
LIST_LANG="ar ca de_DE es fr_FR it_IT ja ko_KR nb_NO pt_BR ru_RU zh_CN"
Can you update your pull request? after I can commit on the repo, and make the sync between Transifex/Cloudstack UI.
(I would more better if you can finish the Hungarian translation on Transifex before, for 4.6 and 4.4 :-))
<EMAIL_ADDRESS>

Hmm, the repo cleanup should have been merged https://github.com/apache/cloudstack/pull/221


add health check for webhook server

https://github.com/kubernetes-sigs/controller-runtime/pull/1588 is added in sigs.k8s.io/controller-runtime v0.10.0 but we still use v0.9.0 https://github.com/bestchains/fabric-operator/blob/4a741481a4ffadce29b5ffbdb9d2ad8857df1f95/go.mod#L33


Add test app and tests

Codecov Report

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

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   89.55%
=========================================
  Files             ?        2
  Lines             ?       67
  Branches          ?       14
=========================================
  Hits              ?       60
  Misses            ?        3
  Partials          ?        4

Continue to review full report at Codecov.

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


Adding ability to disable disqus on a single post

With this you can add disableDisqus = true to a page to disable loading Disqus, even if it is enabled site wide. This is useful for pages that are not blog entries, but informational, like about pages. See #26 and #27, but it was removed for some reason in a later commit

https://github.com/cloudflare/wrangler/pull/126#issuecomment-507328059

Would love to have this feature!

In the meantime, you can override the about page by creating a section folder inside the layouts folder, and then creating an about.html file inside section:
{{ define "main" }}
{{ partial "profile.html" . }}

{{ .Content }}

{{ end }}

Thanks for your PR! I have add this ability in the new commit https://github.com/nodejh/hugo-theme-mini/commit/7a04c022ea788b60f3aa3039ec38d834750479f1


Add Push Api

Hi
Added the ability to submit licenses via api to it.

Hello! Thanks for your contribution, but I don't think that this is needed now. It's a great feature, but most of people just won't use it. Also, your API isn't sending such variables: referral_count, private_key, peer_endpoint, peer_public_key, interface_addresses, they are quite important.

Implemented in 13978c3e569be3606135658e72c9e80372eb975e


Problem adding ipcRenderer to browser part of generated electron app

Summary
After adding an ipcRenderer require to a base Electron app, I am getting build errors and am not sure how to proceed to fix it through lwc-services
Steps To Reproduce:

npx create-lwc-app@latest ipctest
Do you want to use the simple setup? No
Package name for npm ipctest
Description My amazing LWC app
Author Richard Cook
Select a package manager npm
Select the type of app you want to create Electron app
Select a bundler Webpack
Use TypeScript or JavaScript JavaScript
Do you want a basic Express API server? (y/N) n
...
cd ipctest
npm run build:development

electron scripts/main.js
This produces a working Electron app.

Add the following line after line 1 in my/app.js
const { ipcRenderer } = require('electron');

npm run build:development

Expected result
I expected a successful build that would let me build out ipcRenderer code.
Actual result
I get the following output
🆘  Module not found: Error: Can't resolve 'fs' in '/Users/rrc/gitcode/ipctest/node_modules/electron'
Module not found: Error: Can't resolve 'path' in '/Users/rrc/gitcode/ipctest/node_modules/electron'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "path": require.resolve("path-browserify") }'
- install 'path-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "path": false }
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build:development: lwc-services build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:development script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/rrc/.npm/_logs/2021-08-08T03_20_08_337Z-debug.log
Additional information
I tried adding fallback settings to lwcWebpackConfig, I got a build to happen but errors appear in the Electron browser window.
macOS Catalina, npm 6.4.13

Unfortunately I can't really help here. Electron seems to want to use fs, which is a node server module, that is not available in-browser.

I have it figured out.
I think I was originally looking at older electron tutorials about how to do ipc communication, I got it working when I set up a preload.js and practices around that.
I've made an example where when you click a button a message is sent from the render window to the main window, then back from the main window to the render window, within the context of lwc-oss.
Code is at https://github.com/rrcook/letest2
I think this is important in an electron app, please have a look and consider if you can incorporate this, or your own improvements, in to the electron app part of create-lwc-app.


[Journeys] HTTPGet method returns incorrect data from the server

Reproducible: always.
Preconditions

Set HTTP method as GET.

Steps to reproduce

Set the value in the URL field as https://car-project.azurewebsites.net/api/journeys/recent/56
Click on the 'Send' button.

Actual result
Method HTTPGet returns data from the server by userId.
Expected result
Method HTTPGet shall return data from the server by journeyId.
User story and test case links
User story #4 #14
Labels to be added
"Bug" ("bug"), Priority ("pri: ")

Bug fixed.