No way to add an empty array of links

I'd like to be able to represent an empty array of links - for example, a link rel 'items' that connects a resource to 0..* items.
I'd like to guarantee that there will always be an 'items' property present in the _links map, and that it will contain an empty array if there are no 'items' links.
Halacious errors if I do: rel.link('items', []), but it's fine for rel.link('items', ['./items/1']) and rel.link('items', ['./items/1', './items/2']).
Is there a reason this isn't supported? I imagine it's an oversight rather than by design, as it'd be easy to accidentally wind up with an empty array there.

I think originally empty arrays were intentionally being excluded from the representation to make client parsing easier. This feature led to a regression at some point. I have checked in a fix to master. Let me know if it is working for you.

On May 11, 2015, at 7:03 AM, Paul Brabban<EMAIL_ADDRESS>wrote:
I'd like to be able to represent an empty array of links - for example, a link rel 'items' that connects a resource to 0..* items.
I'd like to guarantee that there will always be an 'items' property present in the _links map, and that it will contain an empty array if there are no 'iitems' links.
Halacious errors if I do: rel.link('items', []).
Is there a reason this isn't supported?
—
Reply to this email directly or view it on GitHub https://github.com/bleupen/halacious/issues/56.

Works a treat!
Thanks very much - very helpful library and blown away by the speed of response.
Haven't done this before; I have a private build of halacious' master HEAD right now - I guess I can swap that out when you next publish to npm? Do you have any idea when this change will be available through public npm?
Thanks again!

Great!
I just pushed a new patch release 3.1.3.

On May 11, 2015, at 9:42 AM, Paul Brabban<EMAIL_ADDRESS>wrote:
Works a treat!
Thanks very much - very helpful library and blown away by the speed of response.
Haven't done this before; I have a private build of halacious' master HEAD right now - I guess I can swap that out when you next publish to npm? Do you have any idea when this change will be available through public npm?
Thanks again!
—
Reply to this email directly or view it on GitHub https://github.com/bleupen/halacious/issues/56#issuecomment-100911794.

Fantastic. Thanks very much.


Map 1287 dps get location data from new locations inside prison api 1

use the new locationsInsidePrisonApi so that endpoint  '/manage-prisoner-whereabouts/select-residential-location' calls locationsInsidePrisonApi.searchGroups  rather than whereaboutsApi.searchGroups

replaced by another branch


Some emojis NOT vertically middle aligned

Such as the fish: https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>
There's a lot white space on the top.

Maybe because the fish is underwater? But yes, I do think we should fix it. Could you give me a list of all of the emojis that need to be fixed? I could make a pull request with all of those.￼

Currently I only found this one.
On Sun, Nov 1, 2020 at 04:51 Abyss Studios<EMAIL_ADDRESS>wrote:

Maybe because the fish is underwater? But yes, I do think we should fix
it. Could you give me a list of all of the emojis that need to be fixed? I
could make a pull request with all of those.￼
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twitter/twemoji/issues/445#issuecomment-719986978,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWRBAZNFWA6OLTLVSXSSVLSNR2GDANCNFSM4TGC6AJA
.

For the fish emoji, this is intentional.

For the fish emoji, this is intentional.


tv shows

when you create the first show, it goes in fine. but when you add a second show, it overwrites the first show. has anyone had this problem? tiz

Not seen this issue before - Either permission issue, or conflict in Streama's file I'd imagine.
Getting any errors? Any logfiles? What OS you got Streama on?
I'd recommend a reinstall?
Start from scratch. Ensure your PC - whatever you have Streama running on is updated, fully healthy and otherwise is good.

ERROR grails.boot.config.tools.ClassPathScanner - The application defines a Groovy source using the default package. Please move all Groovy sources into a package
this message pops-up. don't know if its relevant. did it on a different machine and it did the same thing. can always create the first one, but it won't create a second one. it always overwrites

Can you attach a video of this behavior?
Can you screenshot your database settings for the TV show table? Is the id set to Auto-increment there? Are you using h2 or mysql or another? The error on startup is irrelevant

streama.zip
not using mysql. its a real basic system. did 644 to the upload directory. besides that, everything else was untouched. tia

Oh ok! this happens WITHOUT a tmdb key? then it is definitely narrowed down!

Seems to have been caused because TvShow.findByApiId(null) would return the first item in the DB, causing subsequent save() calls to overwrite the first. Checking for data.apiId != null before calling  findByApiId in TvShowController prevents this.
Fixed in PR shown above.


WIP. Don't merge! Provide support for lean openwhisk setup

Adds k8s support for Lean openwhisk setup that been merged in: apache/incubator-openwhisk#4216
Resolves Issue #445

i know this is a WIP, but if you are changing the imageTag for the controller, you should also update the tag for invoker and utilities image.  All three images must be kept in synch.

i know this is a WIP, but if you are changing the imageTag for the controller, you should also update the tag for invoker and utilities image. All three images must be kept in synch.

Thanks, hope that's the source of the build problem.


⚠ Bofrost.at has degraded performance

In 2e149fa, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 835 ms

Resolved: Bofrost.at performance has improved in d8c0dbf after 15 minutes.


Register Faculdade Metropolitana de Anapolis

Register Faculdade Metropolina de Anapolis.

@igordps First stage of verification passed, official domain is faculdadefama.edu.br as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@igordps Pull request merged. Thank you!


EventEmitter.removeListener('change', ...): Method has been deprecated. Please instead use remove() on the subscription returned by EventEmitter.addListener.

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
Please provide a clear and concise description of what the bug is. Include screenshots if needed.
Please test using the latest React Native release to make sure your issue has not already been fixed: https://reactnative.dev/docs/upgrading.html
React Native version:
Run react-native info in your terminal and copy the results here.
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Expected Results
Describe what you expected to happen.
Snack, code example, screenshot, or link to a repository:
Please provide a Snack (https://snack.expo.io/), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
You may provide a screenshot of the application if you think it is relevant to your bug report.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

after upgrading "@react-navigation/native" to "5.9.8" the issue was resolved. Please check the issue you are getting from which package it is, and then try to update the package to see if the issue is fixed.

I'm still facing this issue. My @react-navigation/native version is "^6.0.4". I don't know what's causing it?

@yehyaumar Check if you’re using something like AppState or BackHandler somewhere in your app and change it so it uses the new remove-method on the returned subscription-object (https://reactnative.dev/docs/appstate#removeeventlistener).

I was having a similar issue, however, whenever I updated my side effects to use the .remove() method, TypeScript says,

Property 'remove' does not exist on type 'void'.ts(2339)

Any idea of what I can do to make TS happy?

I was having a similar issue, however, whenever I updated my side effects to use the .remove() method, TypeScript says,

Property 'remove' does not exist on type 'void'.ts(2339)

Any idea of what I can do to make TS happy?

Have you updated to the latest types of React Native<EMAIL_ADDRESS>works for me.

The Appearance module in 0.66.3 seems to also be calling the deprecated method.

Any status on a fix/patch? I am having the same issue. Deprecated call coming from both Appearance.js and Dimensions.js. React-native ver is: 0.67.2

Having the same warning in react-native 0.67.3

I was having a similar issue, however, whenever I updated my side effects to use the .remove() method, TypeScript says,

Property 'remove' does not exist on type 'void'.ts(2339)

Any idea of what I can do to make TS happy?

Have you updated to the latest types of React Native<EMAIL_ADDRESS>works for me.

I'm curious to know if you were able to unit test this? It seems like there is no straightforward way to simulate a user changing state from foreground to background.


14Plus  16.5

FKD vulnerability during runtime with yes and no options, selecting yes prevents access to the software

Install misaka 3.2.7, its the best,

选择yes后就卡死了

Same issue, misaka's kfd exploit straight up is not working on 16.5 iPhone 14.


Capture exception when push token is not found

Pull Request Test Coverage Report for Build 280

1 of 6 (16.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 91.568%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

safe_notification_service/safe/tasks.py
0
5
0.0%

Totals

Change from base Build 278:
-0.3%

Covered Lines:
771

Relevant Lines:
842

💛  - Coveralls


ServerFarmId needs to be fully qualified to show as unchanged

Describe the noise
Microsoft.Web/sites
2016-08-01
Azure CLI
~ properties.serverFarmId: "subscriptions/<subId>/resourceGroups/<resource-group-id>/providers/Microsoft.Web/serverfarms/<server-farm-id>" => "<server-farm-id>"

Deploying a web site, unless you supply the fully qualified path of the serverFarmId in the ARM template, what-if shows this as a change per the example above. I would expect nothing to do changed since the template has not changed.

Isaac, does this only work if the serverFarm is in the same sub & rg as the website? TBH, I'm surprised they accept the "shorthand" version of the serverFarmId. Most people use the resourceId() function when wiring things up like this FWIW.

My feeling is that encouraging people to use the resourceId() pattern is a good thing, whereas just using the short name may work fine now, but leaves more for you to learn later. That's not a strong opinion btw.
Otherwise, you are right that the only benefit is cases where you have the same name and a different resource type. As you note, we don't see often.


#6187 Presenter cache for user presenter

This closes #6187 and should improve dashboard performance.

Retest this, please.

@rafatower, @ethervoid  CR this, please.

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

Ping @ethervoid, @rafatower

:+1:

:+1:


Prevent submitters from editing forwarded reports on Control

Details

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/411546
Tests

[ ] Verify that no errors appear in the JS console

Create a Collect policy in NewDot and upgrade it to Corporate
Invite these members submitter, approver1, approver2
Change the config in OldDot to match this

Submit a report in NewDot as the submitter
Make sure you can still edit all the fields
In OldDot, approve & forward the report to as approver1
Make sure as the submitter you can't edit the fields anymore.

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/22363532-e112-4ca9-ad02-5d9d0fdd8270

MacOS: Desktop

wait this needs a checklist right @youssef-lr ?

yes

Can I review this PR? It appears that this PR is related to this issue https://github.com/Expensify/App/issues/36202

@allgandalf is already assigned @hoangzinh

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

Huh this should have been deployed to staging, from this checklist - https://github.com/Expensify/App/issues/47219

FYI I believe this was deployed to prod yesterday, from this checklist - https://github.com/Expensify/App/issues/47219


how to get "FlowImages_gap-2"? I only get FlowImages_gap-1 after run_inference.py

Please set the gap to [1, 2] in flow/run_inference.py in order to get flows for both gaps 1 and 2
https://github.com/Jyxarthur/flowsam/blob/57c5c40c92c73fd3244631409610c4b2ec1d5097/flow/run_inference.py#L10
If gap 1 flows are already obtained, set gap = [2] instead

OIS, thank you!!


website: use details instead of toggle divs

Use html details element instead of divs and javascript
to handle togglable parts of the page. This makes
the website more friendly to users with disabled
javascript, simplifies the html, and allows us to
remove a few lines of code from the javascript.
changes:

use details + summary elements instead of togglable
divs
adjust the relevant javascript code that handled
toggling and automatic opening of togglable divs
based on url hash

Fixes: https://github.com/golang/go/issues/44000

@googlebot I signed it!


How do i use it on windows?

you can download cmdline hsynz_v0.9.1_bin_windows64.zip  from  https://github.com/sisong/hsynz/releases
or used VC build hsynz\builds\vc\hsynz.sln by youself.

I have downloaded it and the hsync_make is working fine but hsync_http always give http error

you can used hsync_demo for test full process.
for hsync_http download error:

error when downloading .hsyni or .hsynz?
http(s) server support HTTP/1.1 muti-ranges?
http(s) server URL is public?   ( I want test it. )

Error on hsynz (both, but i have hsyni locally).
this is the url:
https://launcher.golden-helmet.tk/out.hsynz

now, not support oldPath outNewPath same path.
Because I haven't figured out how to deal with this situation clearly.
The most critical problem: it is not possible to safely and securely delete files in oldDir that are not in newDir.

I run $hsync_http -dl#https://launcher.golden-helmet.tk/out.hsynz -f out.hsynz.tmp  ，got error log:
TcpSocket::open(): SSL requested...
SSL handshake now...
open_ssl: ssl_handshake returned -0x7780

TcpSocket::close
HttpSocket::_FinishRequest

mbedtls SSL has reported an error...

I run $hsync_http -dl#https://launcher.golden-helmet.tk/out.hsynz -f out.hsynz.tmp  ，got error log:
TcpSocket::open(): SSL requested...
SSL handshake now...
open_ssl: ssl_handshake returned -0x7780

TcpSocket::close
HttpSocket::_FinishRequest

mbedtls SSL has reported an error...

It is using cloudflare with cloudflare certificate. I think they are secure.
Maybe mbedtls is broken

I search the error code , got:
MBEDTLS_ERR_SSL_FATAL_ALERT_MESSAGE               -0x7780  /**< A fatal alert message was received from our peer. */

I fixed -0x7780:  add mbedtls_ssl_set_hostname() when init SSL;
$hsync_http -dl#https://launcher.golden-helmet.tk/out.hsynz -f out.hsynz.tmp  download ok;
code at dev branch.
I test download with single range Request, it's ok;  but when download with two ranges Request, it's fail.

I used curl test the URL:
$curl https://launcher.golden-helmet.tk/out.hsynz -i
run ok
$curl https://launcher.golden-helmet.tk/out.hsynz -i -H "Range: bytes=0-1023"
run ok
$curl https://launcher.golden-helmet.tk/out.hsynz -i -H "Range: bytes=0-50,100-150"
400 Bad Request

next version will support this server(only support single-range, not muti-ranges);
cmdline will add "-r-1" for this.


Improve EventListener / Analytics API to support contextual data from external wrapper like retrofit

As suggested by @yschimke in this https://github.com/square/okhttp/issues/270#issuecomment-399324485, I'm opening a new issue, regarding the analytics API to be able to handle call contextual data.
The idea of this issue is to have a way to give and access context per call / request. This is interesting for monitoring typically when the request is parameterized (via path params, etc.). Or to monitor certain kind of payload, e.g. separate metrics for the various grant type of an oauth endpoint.

Here's a verbatim of the comments in #270

https://github.com/square/okhttp/issues/270#issuecomment-381157742

@swankjesse I would like to monitor request by their route as defined in Retrofit. However this appear to be difficult to capture this information in Retrofit, see square/retrofit#2732. For bootstrapping the idea I talked about using Request.tag to pass along this contextual route data, but I don't think this is wise regarding the tag usage documentation.
Do you think OkHttp could introduce another concept in that regard ?

https://github.com/square/okhttp/issues/270#issuecomment-381291303

how are you planning to do the monitoring? For example, is it a timer? One thing you could do is add a synthetic request header that your "start" timer removes. There are probably other ways, too.

https://github.com/square/okhttp/issues/270#issuecomment-381597941

@adriancole Actually I implemented an eventlistener that feeds prometheus collectors (gauge, summary, counters). Summary are the closest thing to timers if compared to micrometer. (But for reasons outside of this scope I'm using summary instead of histograms).
I considered this approach as well in retrofit, but since I cannot modify the request at the right moment / stage in retrofit I cannot capture the path of the retrofit annotation.

interesting things here are that call is delay started.. so for example, lifecycle would be more foreground with a feature like this.. for example, can you add to the context before the call is executed, or can you only add before it is executed? Is the state immutable after a certain point? If a call is created and enqueued, but doesn't actually end in a network request, does that monkey wrench any assumptions?
Also, sometimes folks want property propagation (explicit context), and other times they want implicit context (like MDC hooks). Properties are much easier.
Just things for teasing out the design.

By the way I hacked thing on friday night. While this work I'm not satisfied by the overall design of my hack as it involves to tweak things at several places.
Regarding immutability in my hacky design, the object (RequestMetadata) set in the tag field is mutable until read.
As both retrofit and okhttp are tied in this here's a verbatim of the https://github.com/square/retrofit/issues/2732#issuecomment-399894314 I just posted.

@adriancole I took the time to play a bit with the call adapter factory and ok event listener. I have a solution that can capture annotations and store them in the request tag, these annotations can be read in the EventListener.
=> https://github.com/bric3/retrofit-okhttp-accounting/
What I don't like with this solution is that, this a quite hacky one

The Call wrapper is relying on the fact that calling .request() on the actual call before executing or enqueuing the call will return the request that will be used in this operations, it could be a copy for example, since the object is _almost immutable this is probably unlikely but given that tag can be a mutable object I don't know if that is a lasting approach.

The tag type is Object and has self (the request itself) value by default; so the Call wrapper has to cast the tag to the needed type, and the value has to be set at the request creation time because a Request is immutable. Currently this is done in the CallFactory, where the request is modified with a builder then passed to ok http client. Design wise that's wrong, that's splitting concerns in different places. What if the tag has to be used for something else.

OkHttp Request builder documents the tag as way to cancel the request. I'm not sure how this is used, but using tag to achieve accounting or monitoring feels like a misappropriate use of the tag.

Attaches {@code tag} to the request. It can be used later to cancel the request. If the tag is unspecified or null, the request is canceled by using the request itself as the tag.

I think this would be very welcome improvement in both retrofit and okhttp to have this kind of feature.
I would like a simpler approach without having to rely on a CallAdapterFactory and a custom CalFactory

OkHttp' request could benefit from a field dedicated to contextual information.
Retrofit should have a request factory, or some kind of request customizer, or request context customizer, that could be leveraged with interface methods annotations like @DELETE.

Tag javadocs lie: https://github.com/square/okhttp/issues/4039

@yschimke oh interesting. But still I find the tag api a bit awkward to use. And one concern is that it is not possible to have different concerns in the tag. Once the tag is set you cannot add a different set of contextual data (symply because the actual type set in the tag maybe another type, set by something else).
Retrofit call adapters could be leveraged to add various aspects like authentication, accounting, monitoring, etc.

Idea:
Promote the tag from an Object to a Map<Class<T>, T>.
ZipkinTag zipkinTag = request.tag(ZipkinTag.class);
Call<?> retrofitCall = request.tag(retrofit2.Call.class);
...

We’d be backwards-compatible by storing the current tag under Object.class. We could enhance Retrofit to attach its Call to the requests it creates, and possibly also to make Call expose more stuff.
Thoughts?

I think that's reasonable on the OkHttp side and quite easy to implement.
Regarding retrofit, I would definitely like a way to access more stuff, especially methods annotation, maybe method name.

Yeah, we have that information but haven’t really had anywhere to expose it. Putting it on the Call is pretty straightforward.
Lets proceed to have typed tags in OkHttp. If anyone is picking this up I’d prefer to not allocate a LinkedHashMap until we have multiple tags, since that case should be quite rare. Perhaps something like this:
class Request {
  ...
  private Class<?> onlyTagType; // Object.class, etc.
  private Object onlyTag;
  private Map<Class<?>, Object> tags; // null unless we have multiple tags
}


I am not able set content type-json in header always getting form data

`
var obj = {
"user_team": {
"match_id": "14",
"contest_id": "1",
"captain_id": "67",
"captain_id1": "67",
"player_ids": ["66", "67", "68", "69", "70"]
}}
fetch('http://<IP_ADDRESS>:3000/api/user_team/create', {
    method: 'POST',
    headers: {
      'Content-Type': 'application/json',
      'Accept': 'application/fanmojo;version=V1',
      'TOKEN':'BObHssp-U9OctAccNSSBzA'
    },
    body: JSON.stringify(obj)

})
`

I'm assuming that you're making a request to another domain or port number and that you need to study CORS more. Please start at https://github.com/github/fetch#read-this-first


npm ERR! code ELIFECYCLE...

root@me:~# npm install -g sails
/
<EMAIL_ADDRESS>preinstall /root/.nvm/v0.10.16/lib/node_modules/sails
node ./lib/preinstall_npmcheck.js

module.js:338
   throw err;
         ^
Error: Cannot find module '/root/.nvm/v0.10.16/lib/node_modules/sails/lib/preinstall_npmcheck.js'
   at Function.Module._resolveFilename (module.js:336:15)
   at Function.Module._load (module.js:278:25)
   at Function.Module.runMain (module.js:501:10)
   at startup (node.js:129:16)
   at node.js:814:3
npm ERR! Linux 3.8.0-29-generic
npm ERR! argv "node" "/root/.nvm/v0.10.16/bin/npm" "install" "-g" "sails"
npm ERR! node v0.10.16
npm ERR! npm  v2.7.5
npm ERR! code ELIFECYCLE

This is driving me insane. It's a Ubuntu server and it's running other node apps already. So I can't install sails... Before I got this error though I succesfuly installed it but then when I ran 'sails' I got nothing. Tried to set my path, clean npm cache... Btw I was using npm 1.2 or so and sails require  < 1.4 so I'm running 2.7.4 now

See this issue: https://github.com/balderdashy/sails/issues/2673
tldr: don't run npm install as root.

Yes I saw that issue before posting, should have mentioned it. But when I don't run it as root I get this:
me@M4KE:~$ npm install sails -g
npm ERR! Linux 3.8.0-29-generic
npm ERR! argv "node" "/usr/local/bin/npm" "install" "sails" "-g"
npm ERR! node v0.12.2
npm ERR! npm  v2.7.5
npm ERR! path /usr/local/lib/node_modules/sails
npm ERR! code EACCES
npm ERR! errno -13

npm ERR! Error: EACCES, rmdir '/usr/local/lib/node_modules/sails'
npm ERR!     at Error (native)
npm ERR!  { [Error: EACCES, rmdir '/usr/local/lib/node_modules/sails']
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   path: '/usr/local/lib/node_modules/sails' }
npm ERR!
npm ERR!  Please include the following file with any support request:
npm ERR!     /home/me/npm-debug.log

npm ERR! Error: EACCES, rmdir '/usr/local/lib/node_modules/sails'

Yea, your permissions are jacked up from running npm as root.chown /usr/local/lib to your user

drwxr-xr-x 7 dav  root 4096 Apr  9 08:40 lib
I had already done that but the error stayed the same. It's not my server btw, but I'm supposed to get my app in prod on it. I guess it's some permission issues like you suggested. Before I created user "dav" there was only root on this server...

Im trying to install sails on a plain 15.04 ubuntu with apt version of nodejs and npm (0.10.25 and 1.4.21) and im getting this errors, even if I don't run npm install -g sails as root.


VPC config updates

https://github.ibm.com/alchemy-containers/armada-storage/issues/832

@narkarum , Can you please run the make fmt for this code looks some issue.
also following will be default if not passed via configuration right?
var maxRetryAttempt = 5
var maxRetryGap = 30

We are setting them in code...if they were not set.

Fixed the issues.

@arahamad


🐛 [FlutterFire CLI] CLI doesn't find Firebase CLI

Bug report

macOS Monterey - version 12.3

FlutterFire CLI ask me to install Firebase CLI even if it is already installed and logged in my account.
Steps to reproduce
>>> firebase --version
10.4.0
>>> firebase login
Already logged in as <my email address>
>>> flutterfire configure
i Found 0 Firebase projects.
FirebaseCommandException: An error occured on the Firebase CLI when attempting to run a command.
COMMAND: firebase --version
ERROR: The FlutterFire CLI currently requires the official Firebase CLI to also be installed, see https://firebase.google.com/docs/cli#install_the_firebase_cli for how to install it.

Expected behavior
I would expect flutterfire configure to find Firebase CLI and run.

Additional context
# ~/.zshrc
export PATH="$PATH:$HOME/development/flutter/bin"
export PATH="$HOME/.yarn/bin:$HOME/.config/yarn/global/node_modules/.bin:$PATH"
export PATH="$PATH":"$HOME/.pub-cache/bin"

alias firebase="`npm config get prefix`/bin/firebase"

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
PASTE OUTPUT INSIDE HERE

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
>>> flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.10.3, on macOS 12.3 21E230 darwin-x64, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.65.2)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

• No issues found!

Please report CLI related issues in dedicated repo: https://github.com/invertase/flutterfire_cli/issues
https://firebase.flutter.dev/docs/cli/#reporting-an-issue


fix(wrapc): wrapc cannot print stack trace

Thank you for contributing to errors!
Please add a summary of your change
Does your change fix a particular issue?
Fixes #(issue)
Check list:

[ ] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 80.00000% with 9 lines in your changes missing coverage. Please review.

Project coverage is 95.94%. Comparing base (4183095) to head (c8a8424).

Files
Patch %
Lines

errors.go
80.00%
5 Missing and 4 partials :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   97.59%   95.94%   -1.65%
==========================================
  Files           7        7
  Lines         458      469      +11
==========================================
+ Hits          447      450       +3
- Misses          9       14       +5
- Partials        2        5       +3

Flag
Coverage Δ

unittests
95.94% <80.00%> (-1.65%)
:arrow_down:

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

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


App crash

Describe your bug here.
Hi. Just came back after the current release, but after i downloaded the app,  the game wont open/crash. Making it unplayable. I also saw some of the people had the same problem.
Fix this if you can.
Crash Log or logcat logs (if existing)
Nop

Are you modding a build from source or with Lua?
I am not modding
What is your build target?
Android
Did you edit anything in this build? If so, mention or summarize your changes.
No, i did not.

android tools problem, There is no solution yet


Service binding supporting deployment config

What this does
We are adding deploymentConfig support by extra flag.
Why flag instead of showing both?
Deployment config is typically problematic with binding (extra rollouts are needed and we cannot make sure that binding will be valid). We needed to provide extra documentation etc. to support that - adding it by default will cause confusion and problems.
It will also break our existing guides :/
Notes
This change doesn't introduce any multiservice aritecture.
This is going to be done in separate feature branch done by @rkpattnaik780
I have prepared layer for that refactoring but we need to work on top of the changes that Rama has right now.
Verification
[wtrocki@graphapi app-services-cli (support-bind ✗)]$ rhoas cluster bind --deployment-config
? Select Kafka instance to connect: pete
Namespace not provided. Using wtrocki namespace
? Please select application you want to connect with example
Binding "pete" with "example" app
? Do you want to continue? Yes
Using ServiceBinding Operator to perform binding
✔ Binding pete with example app succeeded

WIP. FYI @rkpattnaik780

@craicoverflow just review needed. Verification not possible - SBO operator seems to be broken

The error messages need formatting at the last line.

Also for the latter error message, not sure if it was expected to fail, I could do a bind after it.

@rkpattnaik780 That is mostly because of logic - it lets you to select kafka that you do not have on the cluster.
I have left comment to solve that problem - once we move to binding we should read Kafka and Registry connections from cluster and specify their names instead of reading them from backend

Blocked by:
https://github.com/redhat-developer/service-binding-operator/issues/1025
My quess is that we going to merge and release it so we can rebase and continue refactoring it for multiservice architecture


Use the 'master' inspection tag vs 'latest'

The 'latest' tag is no longer being used in the quay registry, so update the script to default to 'master' instead.

I think I'm just going to put the latest tag back against the master service-telemetry-operator build job in quay.io like everything else has. Seems more like a mistake when I recreated the job rather than intentional adjustment.

I think I'm just going to put the latest tag back against the master service-telemetry-operator build job in quay.io like everything else has. Seems more like a mistake when I recreated the job rather than intentional adjustment.

This resolved the issue. Closing this issue with no further action required.


[add-resource] New Plugin/Theme

I would like to add my new plugin/theme to the Grav Repository.
Here are the project details: DennisdeBest/grav-plugin-pre-publish

https://github.com/DennisdeBest/grav-plugin-pre-publish

@DennisdeBest, I took a look at your plugin and I'm afraid I do not quite understand its purpose. Just to confirm my understanding:

In onPageInitialized, the page is checked if its rawRoute matches the slug mentioned in the plugin's config value filter:items:'@page': /blog

If page matches /blog,  the plugin registers event onCollectionProcessed

In  onCollectionProcessed, the plugin loops through all pages in the collection

If page is published and its publish_date is in the future, the page is removed from the collection.

Correct?
Isn't that how Grav works by default? If a page's publish_date is set to be in the future, the collection defined in page /blog will not contain that page. Unless variable published is explicitly set to true which overrides publish_date.
See the docs about header variable published:

By default, a page is published unless you explicitly set published: false or via a publish_date being in the future, or unpublish_date in the past. Valid values are true or false.

The only situation in which the plugin will remove a page, is when the user explicitly marks the page as published and adds a publish_date in the future.
If that is the sole purpose of the plugin, I would watch event onAdminSave and correct the settings if the page is created/updated in Admin, instead of each time when the page is initialized.

Thanks for your reply !
Indeed I did not think of it in that way.
I have just tried it out and if published is not set at all (so neither true nor false) it will indeed take in account the published_date to check if the page has to be in the parent's page collection.
This means that for now this is pretty much an overlay of something already in place and is not very usefull. I was thinking of adding cron jobs to publish tweets and facebooks posts automatically when the published date has been reached so I will come back to this when that is in place.
I tried checking the page URL on onPluginsInitialized this however did not work for me and would always return a 404. I will run it again and find some more information.
I think for now this "plugin" issue can be closed and I will create a new one when some more custom features are added.

Enabling plugin
In onPluginsInitialized you cannot check for the page's url, since the page hasn't been initialized yet. You check instead the requested URI.
public function onPluginsInitialized()
{
    ...
    if ($this->isOnRoute()) {
        $this->enable([
            'onPageInitialized' => ['onPageInitialized', 0],
            'onTwigTemplatePaths' => ['onTwigTemplatePaths', 0],
        ]);
    }
}

private function isOnRoute(): bool {
    $uri = $this->grav['uri']->uri();
    $routes = $this->config->get('plugins.' . $this->name . '.routes');

    $enable = $routes && (
        (gettype($routes) === 'string' && $routes === $uri) ||
        (is_array($routes) && in_array($uri, $routes)));

    return $enable;
}

Thanks for the clarification.
I will keep this in mind. This was my first attempt at a plugin for Grav, I will be more careful in the futur and make things more explicit before submitting anything else.
Sorry to have wasted your time.

No need to apologise, my time has not been wasted... I learned something new!


[KARAF-7609] Upgrade to sshd 2.9.2

Unfortunately, I found a regression in SSHD 2.9.2, see https://issues.apache.org/jira/browse/SSHD-1314
I will probably postpone this update.


Corrected stats write/delete performance and memory issues

Corrected stats write/delete performance issues by deleting old stats in batches (to avoid db timeouts) and do not deleting stats on endpoint delete (they will be cleaned eventually as old stats)
Corrected memory issues in API by queuing up to 10000 stats entries to be saved in case of db issues (the incoming stats will overwrite the oldest in queue if limit is reached)

+1


Add Community Shields

Related to #31

Sooo cool!

I'll try to get approved here as well, so we can get an official Awesome badge 😎


Add Exclude per constant settings for Lint/DeprecatedConstants cop

Adds Exclude settings per specific deprecated constant for Lint/DeprecatedConstants.
Usecase example: monorepo in decomposition state.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Would creating a separate .rubocop.yml work for you? There are many cops with extra config which users may want to tweak for specific folders (monorepo or not) and I don't think adding support for Exclude to each of them is the solution (unless it would be something generic).
You make one common .rubocop.yml in the root folder and have more specific ones in subfolders:
# .rubocop.yml
Lint/DeprecatedConstants:
  'Common':
    Alternative: Deprecations
  'Go':
    Alternative: Here

# subfolder/.rubocop.yml
inherit_from:
  - '../.rubocop.yml'

Lint/DeprecatedConstants:
  'NotDeprecatedEverywhere':
    Alternative: Whatever

Does that look good? If you have a group of common deprecations for multiple folders you can also make a config file to inherit from.

There are use cases where separate configs can't work.
E.g. constant Foo could be allowed for file app/services/foo_service.rb, and deprecated for the rest. OTOH, Bar should be allowed for app/services/bar_service.rb, and deprecated for the rest. In this case config won't help. Could possibly sound like a bad app design, so I'd like to see other opinion on this.
Speaking about the generic solution, I'm not sure how can this be done, so open for suggestions. The best suggestion I have is to extract exclude_file? to some base class.

E.g. constant Foo could be allowed for file app/services/foo_service.rb, and deprecated for the rest. ... Bar should be allowed for app/services/bar_service.rb, and deprecated for the rest

In that case I would just use an inline directive to disable the cop for lines that do access it. RuboCop does something similar with its internal cops that encourage one api over the other: https://github.com/rubocop/rubocop/blob/49c6e92964e87435af7fee6d87bc6c4012fc8f04/lib/rubocop/cop/internal_affairs/create_empty_file.rb and https://github.com/rubocop/rubocop/blob/49c6e92964e87435af7fee6d87bc6c4012fc8f04/spec/support/file_helper.rb#L26-L30. That snippet is the only part where the discouraged method is used since it is implemented in terms of that method. The ruby-lsp does something similar where it creates shorthand names for constants that should always be used (except of course in the place where these aliases are made in the first place).

Generic solution was just a thought. There are no other cops that implement specific excludes like propsed here which is why I mentioned it

Generic solution was just a thought. There are no other cops that implement specific excludes like propsed here which is why I mentioned it

Indeed. That's why I don't like this proposal and would suggest using some other way to achieve more or less the same result (e.g. nested configs, todos, etc).

Ok, got it, thanks for your opinions, guys.
Closing then.


Remove azure specific resources if one exists as a container resource

Today we have both redis and azure redis, sql server and azure sql server. We want to remove these azure specific resources and treat them as an implementation detail of the deployment tool.
Related to https://github.com/dotnet/aspire/issues/1757
Original issue
These should probably just be the same resource type that are interpreted based on the publisher being used and how it is configured.

Closing this as a dupe of what we are doing on this issue #1757


 **[Post a bounty on it!](https://www.bountysource.com/issues/316417-get-a-nerd-merit-badge?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

https://twitter.com/whit537/status/583405869624381441

This is neat, but a minimum order of $250 for 50 is out of our range right now. Let's reopen in the Inside Gratipay repo when there's pressure again.


Unblock: Sky Serie

Channel ID (required)
Sky Serie IT
Reason (required)
Other
Notes
No response

Jjj


Ipopov/406

#406 fixed.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: IgorPopov-MS sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


V8 csb examples not running correctly

Describe the bug (required)
V8 codesandbox examples are not running correctly.
Provide an example via Codesandbox! (required)
https://codesandbox.io/s/github/tanstack/react-table/tree/master/examples/basic

These examples are working https://react-table.tanstack.com/docs/examples/sorting

Should all be working now :)


E3002- SNS Tags Property failing validation

cfn-lint version: (cfn-lint --version) 0.24.5
Description of issue.
Incorrectly errors on Tag property for SNS topics -
E3002 Invalid Property Resources/<resourceid>/Properties/Tags
Tags are a supported property for CFN, specification and documentation needs to be updated.
Example:
  MyTopic:
    Type: AWS::SNS::Topic
    Properties:
      TopicName: MySNSTopic
      Tags:
      - Key: project
        Value: demo

Validated this.  Also its not in the CloudFormation spec so we will have to patch it in.

Fixed in master now waiting for the next release.

https://github.com/aws-cloudformation/aws-cloudformation-coverage-roadmap/issues/7


Internal Server Error

When I try to access the puppboard, I see the following error.

Internal Server Error
This error usually occurs because:
We were unable to reach PuppetDB;
The query to be executed was malformed resulting in an incorrectly encoded request.
Please have a look at the log output for further information.

Where can I find the log file?

i think this is duplicated issue #527

In our case, we doubled the java heapspace and increased the timeout from 20-40 seconds. I know it's not a permanent fix, but it has helped.

Closing as OP’s problem seems to be resolved. Please reopen if needed.


Update azuredeploy.json

The choice of basic as the default is in deference to cost and will work in the simplest cases. The T-Shirt sizing provides the appropriate configuration for different types of workload.


Replicated children - Severe if it's actually the case

So in my game there is a character, character carries around it's name on top of it's head
I've noticed that after some time playing, that name became bolder and bolder, extremely bold at one point
I think pixi-display might somehow be replicating the children, is this possible?
(Couldn't verify the issue yet, will update the thread if it's something else or if it's definitely pixi-display)

Modified the game to work when pixi-display isn't imported, replicated the issue without pixi-display, apologies


Add tests for special characters in beakerlib phase names

Do not merge this PR until the beakerlib changes are released.
This PR adds tests for special characters in beakerlib phase names.  It checks the special characters are correctly handled by beakerlib & tmt and propagated into the internal structures (esp. results.yaml, tmt-report-results.yaml).
Related to:

https://github.com/teemtee/tmt/issues/3333
https://github.com/beakerlib/beakerlib/pull/184

Pull Request Checklist

[x] extend the test coverage

/packit test

Hello @psss, this one should be ready for review. I've modified the tests to always check the values of the concrete subresult extracted using the yq.
The problems in the CI (in Fedora-{41,rawhide}) seems unrelated to my changes.

/packit test

Red tests are irrelevant failures, merging.


Cannot add/search data

Hi,
Thanks for allowing us to try OntoBrowser.
I'm trying to configure a test installation but I'm having problems getting to a working state.
This is my environment:

DB version: Oracle Database 11g Express Edition Release <IP_ADDRESS>.0 - 64bit Production
DB name: "XE"
OS: Centos7 64bit
Java: 1.8.0_161
Wildfly: 11.0.0.Final
Maven: 3.5.2
Graphviz: 2.40.1

I installed all the requirements and created a Oracle user with the following commands:
create user ontobrowser identified by my_password; grant CREATE SESSION, ALTER SESSION, CREATE DATABASE LINK, CREATE MATERIALIZED VIEW, CREATE PROCEDURE, CREATE PUBLIC SYNONYM, CREATE ROLE, CREATE SEQUENCE, CREATE SYNONYM, CREATE TABLE, CREATE TRIGGER, CREATE TYPE, CREATE VIEW, UNLIMITED TABLESPACE  to ontobrowser;
Following the readme, I've tried to import the schema:
sqlplus ontobrowser@XE @create_schema_oracle.sql
but got this error:
(CREATE BITMAP INDEX TERM_IS_ROOT_IDX * ERROR at line 1: ORA-00439: feature not enabled: Bit-mapped indexes )
Then, I've inserted the initial data:
$ sqlplus ontobrowser@XE @insert_initial_data_oracle.sql
no errors here.
At this point, I've tried to load the ontology as described by the docs:
$ curl -s -S -O -L http://purl.obolibrary.org/obo/ma.obo
$ curl -s -S -H "Content-Type: application/obo;charset=utf-8" -X PUT --data-binary<EMAIL_ADDRESS>-u SYSTEM "http://ontobrowser.MYDOMAIN.eu:8080/ontobrowser/ontologies/Mouse%20adult%20gross%20anatomy"
But on Wildfly (which has beed started with ./standalone.sh --debug) I can see this on the standard output:
17:13:36,392 INFO  [io.undertow.servlet] (default task-1) com.novartis.pcs.ontology.rest.servlet.OntologiesServlet: Failed to import ontology: curator not found in request
Also, if I try to login into the web interface, I can see the webapp, but if I try to perform a search, I get an error 500 caused by this exception on Wildfly:
08:21:07,475 ERROR [org.jboss.as.ejb3.invocation] (default task-6) WFLYEJB0034: EJB Invocation failed on component OntologySearchServiceImpl for method public abstract java.util.List com.novartis.pcs.ontology.service.search.OntologySearchService.search(java.lang.String,boolean) throws com.novartis.pcs.ontology.service.search.result.InvalidQuerySyntaxException: javax.ejb.EJBException: java.lang.RuntimeException: Failed to perform Lucene seach with pattern: asdsa
Any idea to troubleshoot the problem?
Thanks a lot
Matteo

Hi Matteo,
The first issue is that the curator username is not being retrieved within the application because user authentication has not been setup. The install docs includes instructions to setup an Apache web server to perform user authentication so the curator username can be passed to Wildfly (and subsequently retrieved by the application). Unfortunately, due to some suspected changes with Wildfly many installers have reported issues with Wildfly not passing on the REMOTE_USER to the application from Apache.
My suggestion for evaluating OntoBrowser would be to setup Basic Authentication directly in Wildfly so the application can retrieve the curator username via the getRemoteUser method. The Wildfly documentation should provide all the instructions required to setup Basic Authentication. Good luck.
Regards,
Carlo.


chore: pre compute the action to do for each change of the stack

This will help to do/display the same thing in the dry-run and the
effective run.
Depends-On: #509

This pull request is part of a stack:

chore: add some datastructure (#506)
chore: move more things in Changes/Local Change object (#507)
chore: move retrieval of change IDs and pull inside get_remote_changes() (#508)
fix: all short sha are wrong (#509)
chore: pre compute the action to do for each change of the stack (#510) 👈


autoconf on el6 fails

[lkarsten@el6 libvmod-example]$ ./autogen.sh
++ pkg-config --variable=datarootdir varnishapi
+ dataroot=/usr/share
+ aclocal -I m4 -I /usr/share/aclocal
configure.ac:1: error: Autoconf version 2.64 or higher is required
configure.ac:1: the top level
autom4te: /usr/bin/m4 failed with exit status: 63
aclocal: autom4te failed with exit status: 63

should be done with the latest pull request

to those receiving email updates: my previous comment was wrong and I have deleted it.

@fgsch pointed out in #18 that this smells like requiring the same fix as https://github.com/varnish/Varnish-Cache/commit/ecfc75783bcfc906a0a5be77f3c93e3a0dfe7a4c but as this was applied to varnish.m4, we should not require it in libvmod-example.
So I'd presume all that needs to be done is to set AC_PREREQ(2.59) as in varnish configure.ac


Create release version 2.2.1

Fix viewport scale factor is NaN when row/col pixel spacing is not set

Coverage remained the same at 63.946% when pulling f13f097f89d0887ccf37d5bced9583288f595d01 on release-2.2.1 into c29647b7ee034a9c9c57a00873cce7965817d929 on master.


[WIP] start of pyqt5 compatibility.

This commit contains the start of PyQt5 compatibility, this allows
the pyfda pyqt application to work with PyQt4 or PyQt5.
This merge request is NOT COMPLETE.  Need to finish the pyqt5 compatibility integration.

Ok, waiting with the merge till you tell me.

Could you make another pull request against my branch "pyqt5"?   Let me know if you need anything else.  I have not upgraded / ported pyqt4 to pyqt5 before.  Not sure what the best practices are for supporting both.

Yeah, I created a branch under the same name pyqt5 and merged your branch into it. I need to have a installable version of pyfda by the end of next week, so I thought I'd take over the search & replace slave job ... The input_widgets files are working under pyqt5 now, I'll tackle the rest of the files during the weekend. I'll push the current state now, but I think it would be more effective if I finished the job.
Thanks once more for pointing me in the right direction, using a central compat.py file.
I think the spyder guys have taken a similiar road by creating the qtpy (nice name :-)  module but I didn't want to add another dependency.

I think pyFDA should be compatible with pyqt5 now. Not quite sure though, because my Anaconda Qt5 environment seems to fall back to Qt4 when something is missing. If your system is pyQt5 only, it would be great if you could do a test run. I've uploaded the update to pip; if everything runs fine I'll upload it to the Anaconda cloud as well.


chore(settings): toggle takes lot of time to update

STR:

go to settings
after you activate and try to deactivate this toggle takes a lot of time, around 10s

macOS, m1

tested on linux: not affected

could also not reproduce on my mac air

retested again. on an m2 mac. couldnt reproduce


feat: add coding guideline about topic message handling

Description
Add coding guideline about topic message handling
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The Reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@ohsawa1204 I'd like to ask you to create a page structure before checking the content. I'm concerned that there is no page index on the left-hand side of the web pages.
Please follow my instruction at first. If you find some improvements on my instruction, please fix it.

add directory name of topic-message-handling to docs/contributing/coding-guidelines/ros-nodes/.pages
add new docs/contributing/coding-guidelines/ros-nodes/topic-message-handling/.pages

nav:
  - index.md
  - 01-supp-intra-process-comm.md
  - 02-supp-waitset.md

rename 00-topic-message-handling.md to index.md
fix missing link to 00-topic-message-handling.md with replace it by index.md

I sent a PR (ohsawa1204#1) to your branch to show my instructions. Best,

@takam5f2
Thank you very much for your fix!
Your fix was merged to this PR.

@ohsawa1204 Thank you for checking all of my feedback. Do you have any action on this PR? If no, I'd like ask AWF reviewers to check this PR.
Best,

@takam5f2
No, I don't have any remaining action.


Stylelint doesn't work when there are decorators

Describe the bug
Core issue: https://github.com/stylelint/stylelint/issues/4761
To reproduce
Run the Stylelint script: npm run lint:stylelint
It doesn't warn of possible errors.
Expected behavior
Warn of possible errors.

Fixed with https://github.com/IBM/pwa-lit-template/pull/483


host: Reporting DMI structures

This patch is about adding DMI capabilities to skydive.
It offer users an option to get what are the hardware capabilities of a particular node.
This usually includes the vendor/product/serial numbers, bios versions, DIMM detection and many other items.
A simple laptop usually expose more than 600 items.
There was several options to make the implementation:

ochapman/godmi
it offer nice structures but is really outdate (4yo)
we need accurate DMI information for supporting recent systems

go-smbios
easy to use but doesn't provides any human readable output
it only reports the row values that have to be analyzed against the DMI specification of that particular version
A sample output looks like : &{{7 19 1} [1 128 1 32 0 32 0 64 0 64 0 0 5 4 7] [L1-Cache]}

paulstuart/dmijson
it parses dmidecode but goes through a json structure which isn't really needed here

dselans/dmidecode
this is the one selected for this patch.
it parses dmidecode and offer simple structures to parse
then we recreate a dynamic structure to add into the metadata
Pros:

it uses dmidecode which is up to date regarding the specification, vendors are usually contributing too
Cons:
it parses dmidecode output, which is string based and could break over time

This light patch was enough to report the full DMI tables into the metadata.
Signed-off-by: Erwan Velu<EMAIL_ADDRESS>

add to whitelist

run tests

ok to test

run tests

add to whitelist

Cool, tests passes

Closing is favor of #1307


Compiler error

Package check was not found in the pkg-config search path.
Perhaps you should add the directory containing `check.pc'
to the PKG_CONFIG_PATH environment variable
No package 'check' found
Package check was not found in the pkg-config search path.
Perhaps you should add the directory containing `check.pc'
to the PKG_CONFIG_PATH environment variable
No package 'check' found
In file included from copybw.cpp:35:
common.hpp:29:10: fatal error: check.h: No such file or directory
   29 | #include <check.h>
      |          ^~~~~~~~~
compilation terminated.
make[4]: *** [<builtin>: copybw.o] Error 1
make[3]: *** [Makefile:62: exes] Error 2
make[2]: *** [CMakeFiles/gdrcopy_lib.dir/build.make:70: CMakeFiles/gdrcopy_lib] Error 2
make[1]: *** [CMakeFiles/Makefile2:112: CMakeFiles/gdrcopy_lib.dir/all] Error 2
make: *** [Makefile:146: all] Error 2

you could try execute the command:
On RHEL
$ sudo yum install dkms check check-devel subunit subunit-devel
On Debian
$ sudo apt install check libsubunit0 libsubunit-dev