Can't use $select without including id field

Hi
$select is not working with NeDB as stated in the general query documentation. In order to $select specific fields, you're forced to always include id, otherwise the following error is thrown:
Error: Can't both keep and omit fields except for _id
    at /Users/carlosdelaorden/Dev/rp/real-property-server/node_modules/nedb/lib/cursor.js:80:73
    at Array.forEach (native)
    at Cursor.project (/Users/carlosdelaorden/Dev/rp/real-property-server/node_modules/nedb/lib/cursor.js:79:8)
...

Steps to reproduce
Create the simplest NeDB service you can, add REST transport and try to $select any fields. The sample in the README should be enough.
For example, with this service:
const app = feathers()
  .use(compress())
  .options('*', cors())
  .use(cors())
  .use(bodyParser.json())
  .use(bodyParser.urlencoded({ extended: true }))
  .configure(rest())
  .configure(services)

function services(){
  this.use('/users', service({ Model: userModel(), paginate: false }))
}

function userModel(){
  return new NeDB({
    filename: path.join('data', 'rp.db'),
    autoload: true
  })
}

The following query fails:
localhost:3000/users/?$select=email
But this works fine:
localhost:3000/users/?$select=email&$select=_id
EDIT: calling it via code works as expected! However, _id field is included. The exception is thrown via REST.
userService.find({ query: {
  $select: ['email']
}})
.then(res => console.log(res))

Possible cause and solution
The issue is caused by how NeDB handles projections. From their documentation at https://github.com/louischatriot/nedb#projections:

The syntax is the same as MongoDB: { a: 1, b: 1 } to return only the a and b fields, { a: 0, b: 0 } to omit these two fields. You cannot use both modes at the time, except for _id which is by default always returned and which you can choose to omit.

It clearly states that projections by default include id and you should hide it explicitly by sending { _id: 0 }. Looking at the code of this library, this case is not taken into account in utils/getSelect:
https://github.com/feathersjs/feathers-nedb/blob/master/src/utils.js#L15
I think it's an easy fix, would be to check if the user has included the idField in the select. If it's not, it should be added with 0 so NeDB actually hides it from the result. This means that getSelect helper should also have access to the service config to be able to use whatever idField the user has chosen.
As stated in my edit, calling the service from code doesn't throw but indeed includes the idField. Maybe it's related to the REST query conversion?
If this is not the desired behaviour, then the docs should probably reflect this non-standard query behaviour. Otherwise I'd be happy to contribute the fix if it sounds OK - and add a test for this!  :)

Not including the id field if it hasn't been selected is a good call. After writing tests for it in #73 however, I think your actual problem (since it is only cause via REST) is that there is no way to decide if a URL like localhost:3000/users/?$select=email should be serialized into an object ({ $select: 'email' }) or an array.
You can make that explicit for the query string parser using localhost:3000/users/?$select[]=email.
Fix for excluding the id will be released as v2.7.0


update compile sdk to 33

Update the compileSkdVersion to 33 due to building confilcts of the toolbox-ui-flutter.
The sdk's above 28 are supporting the AndroidX library which is used by the toolbox-ui-flutter.

change was included without merge


Any way to get error message/reason a glfwCreateWindow crashes?

Hello,
I am developing for the Oculus Rift and in certain instances calling glfwCreateWindow() crashes with an exception.
I am wondering how I would go about troubleshooting this as when I don't have the Oculus Rift plugged in everything works fine.
I understand it may not be a problem with GLFW but I need to know why glfwCreateWindow() crashed. Is there a way to get a detailed error message or reason why a crash would occur (and not just a NULL handle)?
Thank you.

Is it okay to close this? It seems unlikely that Oculus would add support for older Intel cards to their SDK.


Why does this keep adding different numbers?

For an example add a 9 to this<PHONE_NUMBER>99999 and it will be converted to 10,000,000,000,000,000, even adding very long numbers get changed, this isn't great for a web app dealing with money...
I can't see any docs revealing why this is happening or how to stop it, tried decimalScale={undefined}, nothig changed...
Please, help

I think it has to deal with parseFloat having issues with big numbers
I'm not sure, but it might have to do with this line https://github.com/cchanxzy/react-currency-input-field/blob/242ec908d6eb34b1767b21b21515d5ea25e3c80e/src/components/CurrencyInput.tsx#L192
The bug I'm experiencing seems to be complained about here too...

https://stackoverflow.com/questions/4288821/how-to-deal-with-big-numbers-in-javascript
Check this out

Figured this plugin once used Number() as seen here https://github.com/cchanxzy/react-currency-input-field/blob/5c42788be926e929b2da2f8ab89bf666a3dd2f4a/src/components/CurrencyInput.tsx#L183 what went wrong?

Testing Number also have same bug... You can use any library capable of Big Numbers?

This library uses parseFloat too but the test I ran on it seems OK, even with wide numbers https://www.npmjs.com/package/react-number-format
Is there something else going on under the hood?

I tried pushing an update yesterday, but figured the library uses Intl.NumberFormat.formattoparts and I can't get that to work with Big Number string.

Hi, thanks for getting in touch.
Yes, I am aware there are some issues when a large digit number is entered. It was never my intention to handle large numbers like<PHONE_NUMBER>99999 so I've never really looked into handling it to be honest.
In the future I might implement a solution, but if you're in a rush I would recommend either:

Forking the project and implementing your own solution
Avoid large numbers
Use a different library (https://www.npmjs.com/package/react-number-format is a good shout)

Sorry my reply isn't particularly helpful.

Hi, glad to hear from you...
The issue has been affecting other parts of my dev since 2 days now(mostly my fault since i don't like proceeding with development when there's a pending bug to fix)
I had a fork pulled  and started implementing the solution using bignumber.js library for all number-based implementations..
I'm done with it though, been testing to be sure i didn't break anything...

Forking the project and implementing your own solution
Avoid large numbers
Use a different library (https://www.npmjs.com/package/react-number-format is a good shout)

Thanks for all suggestions, i'm unable to avoid large numbers since i'm in the finance sector.
After rigorous testing, i figured the react-number-format i mentioned also faced the same issue and going deep into the code, i saw it uses parseFloat even in more lines than yours 😊
I appreciate your work on the library and it was as simple and advanced as i want, kinda have all features i wanted, if you want i can upload the fork to github too maybe it might help you solve a major problem for many people too facing same issue, just that you now have bignumber.js as a dependency
Good luck and thanks again


No method mapboxShape

Somehow compiler doesn't see mapboxShape method

@kondranton can you post your project's podfile? If not, can you check if you are using the MapboxGL subspec?

No, I don't use it, can you tell me how to add it, please?

Use pod 'GEOSwift/MapboxGL' instead of pod 'GEOSwift'

Thanks a lot. That worked.


fix kingston, Ontario

new: https://opendatakingston.cityofkingston.ca/explore/dataset/civic-addresses/information/
broken:
http://results.openaddresses.io/sources/ca/on/city_of_kingston

@ingalls Here's one area that I noticed was light on addresses! The source above still works and the "last updated" timestamp on the page notes July 30, so I think they're processing this on the daily.


make github connector idempotent

fixes #413
Discussion
This tweaks the trackcalls decorator so that stage can use its own has_been_called attribute and guarantee that it only executes once.
I didn't do the same for the huggingface connector bc I'm pretty sure it's already idempotent, but it would be easy if we wanted to.
Testing
Wrote a cute little unit test, since it was easy
Documentation
no docs

📚 Documentation preview 📚: https://garden-ai--417.org.readthedocs.build/en/417/

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

Project coverage is 76.40%. Comparing base (752eee3) to head (1caa1b1).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #417      +/-   ##
==========================================
+ Coverage   76.08%   76.40%   +0.31%
==========================================
  Files          46       46
  Lines        2789     2801      +12
==========================================
+ Hits         2122     2140      +18
+ Misses        667      661       -6

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


Disable uniform scale by default

Fixes #545

Thanks for the PR! I've left a comment in the issues about this :)


Opening a link for a specific tournament will not load the page for that tournament

When attempting to share a link for a tournament, the page will not load when opened from that link. For example, when opening https://fantasyfit.netlify.app/tournament/1 , one will see this screen. This issue is not present when navigating to that link from the app.

Thanks. Confirmed bug. Hypothesis - auto fetch does not occur at this endpoint therefore certain states are not defined when attempting to navigate to this endpoint directly


Project missing license

Please add a license so I know if I can build off of this repository.

Hi @mackenziea524 , thanks for noticing. I never expected this project to actually reach an audience. I added the Apache license in commit 5b7ef6a0ae3c8b633e4e36981b2e54f64c053135. So please feel free to play around with the code.


Loss_l : inf in training ?

Hello, first of all, thanks for your code releasing.
I got the training loss inf, acutally loss_l = inf, i use your original code (only fixed some bug), but i don't know why i got inf.
Parameters: lr:0.004, batchsize:32, base_model:vgg_reducedfc.pth
GPU: 1080ti
Any comments will be appreciated.
Thanks very much!

@transcendentsky  You may look at this pull for details.

I have tried this way, but the problem still exists.

I tracked the code and find the problem comes to prior (prior_box) , the code s_k = self.min_sizes[k]/self.image_size get 0 because these are two integers. I compared it with code in  https://github.com/amdegroot/ssd.pytorch , and There should have a from __future__ import division on the top and code goes right.

@transcendentsky  The current code has no been tested in Python2.


Fix no peer issue

Motivation
User do not specify mainnnet network expecting lodestar to get boot enrs of mainnet but it turns out we did not, hence it gets stuck
Description

If network is known, pull boot enrs to add to discv5 later
Print number of bootEnrs in verbose node just in case we see the issue again
Reproduced and verified in one of our mainnnet nodes

Closes #4657

@tuyennhv Thanks.
Out of interest, without the flag --network mainnet, the Lodestar INFO log says network is mainnet:
Dec-28 06:16:23.220[]                 info: Lodestar network=mainnet, version=v1.2.2/stable/f093b46, commit=f093b468ec3ab0dbbe8e2d2c8175f52ad88aa35f
The INFO log is unchanged with the flag set, but one does get boot enrs.


d^ deletes current character

General information

Basic examination

[x] I have read through the README page
[x] I have the latest version of zsh-vi-mode
[x] I have tested with another terminal program

Problem description
Deletion using d^ will include the character the cursor is under.
Reproduction steps

Insert some text into the shell prompt and move the cursor towards the middle
Enter normal mode and press d^

Expected behavior
d^ should not delete the character the cursor is under.

Hi @musjj
This issue has been settled down, please update to the latest version.
Thanks & Regards

Cool, it's working. It looks like you've also added the d0 chord, thanks for that!


feat: add support for maxItems in run options

This adds support for optional maxItems param to start and call methods on actors and tasks.
This param will be used by pay per result actors to limit max items that will be charged to customer.
Should be merged after https://github.com/apify/apify-worker/pull/734 is released.

Should be merged after https://github.com/apify/apify-worker/pull/734 is released.

I hope it won't mind I merged this @novotnyj, will hold the stable release before that PR is merged.

It's ok, it won't break anything🙂


Constantly getting staging cert

Hi there,
I've run this tutorial multiple times (on the same kubernetes cluster), and I followed the Cleanup Resources Section between each tutorial run. I also executed the following commands between each run:
kubectl get customresourcedefinitions
kubectl delete customresourcedefinition <thelistedresourcedefs>
to eliminate any custom resource definitions.
The first time through the tutorial, I used the staging certs as is default in the tutorial. Then, I changed every "letsencrypt-staging" to "letsencrypt-prod", and I changed
spec:
acme:
server: https://acme-staging-v02.api.letsencrypt.org/directory
to
spec:
acme:
server: https://acme-v02.api.letsencrypt.org/directory
in cluster-issuer.yaml
Is the old staging cert getting cached or stored somewhere that I'm not cleaning up? I feel like I'm missing something, but I can't put my finger on it.
Edit1) My kubernetes version is: 1.9.11
Thanks!

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

ID: ada58160-dc50-b977-d3bc-65f92406e16d
Version Independent ID: 5c596e04-fddb-effe-3b72-575f4df42d4e
Content: Create an HTTPS ingress with Azure Kubernetes Service (AKS) cluster
Content Source: articles/aks/ingress-tls.md
Service: container-service
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

Thanks for the feedback! We are currently investigating and will update you shortly.

Just adding some more information as I get it.
It looks like the cert manager pod had problems (kubectl get pods --all-namespaces):
kube-system   opining-fly-cert-manager-5d584686f-rz9xq                      0/1       CrashLoopBackOff   79         6h

I ran through the tutorial for a 4th time this morning (just so I could try to watch when the cert-manager pod breaks), literally changing no configuration from the last runs, and it worked. The cert-manager pod also shows no errors.
It always disturbs me a little when I can't root cause something because I want to be able to fight the problem in the future, but I'll take it for now.
Any ideas as to what may have occurred?

@jpetitte Sorry for the late reply.
I went through the doc. I configured my environment with staging certs. After the I created a new clusterissuer and certificate for prod and then created an ingress with the prod cluster issuer.
It still showed me the staging cert, because I didn't change the secret name (tls-secret) in this example.
That secret "tls-secret" is created when i created the staging certificate. Since the same name is referenced while creating the prod certificate also, Cert manager didn't create a new one and used the existing certificates.
I have changed the secret name to something else (eg tls-secret-prod) and then recreated the prod certificate.  This time all worked as expected. Generally those secrets are created in default namespace. We can describe them to have a look at the certificates inside those secrets.
you can also describe the certificates to find out more about that certificates status kubectl describe certificate <certificate name>.
If we look at the document side, from the beginning if we pick staging or prod, Document will work fine.
If we configure for one environment and try to migrate to the other environment, Then we will end up with the same certificate.  I think we can add a note to change the secret name when migrating.
I am note sure why cert manager crashed. We need to look at the logs to find that out.
I hope this is the issue occurred to you.  Please let me know.

I had the same issue and the cause was that the tls-secret was not deleted by running the steps described in the "Clean up resources" section. After running kubectl delete secret tls-secret the problem was solved.

@iainfoulds I think we can change the secret name to "tls-secret-staging" so that the users will change that to prod when they are trying out stage and prod on same cluster.
Also consider adding a step in "clean up resources" to delete the secret.
Please investigate further and make appropriate changes.

I'm also getting this problem. I have confirmed that the certificates I have in tls-secret-prod are indeed the real certificates for my domain, and confirmed that my ingress is using the correct hostname and secret. Yet I am still being served the "Kubernetes Ingress Controller Fake Certificate", which I don't know where it comes from as I have no tls-secret-staging or tls-secret secret.

Update: I followed the instructions at https://github.com/jetstack/cert-manager/blob/master/docs/tutorials/quick-start/index.rst and finally managed to get a valid production certificate.

How to solve that?

Hey @iainfoulds
@Marusyk is having a similar issue and I see this item is still open. Did we ever come up with a game plan for this doc/ issue? I can enable people for support if needed as well.

@MicahMcKittrick-MSFT Let's Encrypt would fall under advisory, best-effort through CSS - https://docs.microsoft.com/en-us/azure/aks/support-policies#aks-support-coverage

Thanks @iainfoulds
So for anyone having issues with the prod certificate, we would suggest you reach out to Azure Technical Support. If you do not have the ability to open a technical support ticket, you can email me at<EMAIL_ADDRESS>and provide me with your Azure SubscriptionID and link to this issue. I can then enable your subscription for that support request.

@iainfoulds you should consider @jakaruna-MSFT suggestion again. It would be a simple change and would provide clarity for new users. Problems like this cause hours of frustration and headaches for your team to triage. I think you made the wrong call on this.

Sorry you ran into problems, @tomingoglia. The docs were changed months to ago to name the secret tls-secret-staging and add clean-up steps. Sorry you've missed those steps.
The comment I'd made was that issues with using Let's Encrypt fall under best-effort support if you reach out for additional assistance. You're welcome to reach out to Azure support, and they'll do their best to provide assistance. That's not a doc issue I can resolve though.
CC: @mlearned who leads the AKS docs now.

@iainfoulds I haven't gotten to this step yet...so I'm not having this issue. There are other inconsistencies that I am seeing in the docs vs. my results and that caused me to begin reading through these items. After reading your response, it indicated that you would not be making changes. Thanks for the quick reply! This may help clear things up for people reading through these issues.

@jpetitte Sorry for the late reply.
I went through the doc. I configured my environment with staging certs. After the I created a new clusterissuer and certificate for prod and then created an ingress with the prod cluster issuer.
It still showed me the staging cert, because I didn't change the secret name (tls-secret) in this example.
That secret "tls-secret" is created when i created the staging certificate. Since the same name is referenced while creating the prod certificate also, Cert manager didn't create a new one and used the existing certificates.
I have changed the secret name to something else (eg tls-secret-prod) and then recreated the prod certificate. This time all worked as expected. Generally those secrets are created in default namespace. We can describe them to have a look at the certificates inside those secrets.
you can also describe the certificates to find out more about that certificates status kubectl describe certificate <certificate name>.
If we look at the document side, from the beginning if we pick staging or prod, Document will work fine.
If we configure for one environment and try to migrate to the other environment, Then we will end up with the same certificate. I think we can add a note to change the secret name when migrating.
I am note sure why cert manager crashed. We need to look at the logs to find that out.
I hope this is the issue occurred to you. Please let me know.

worked for me when I created a new secret.


First Step

After testing the git can be used for this project, it's time to make a first step. I need commit and push the basic design of the app.

Finished! Now this project is starting!!!😊


Map 2 trees data

It's in dataprep.r where this will get resolved.
How do you get an ArcGIS Online public layer to be mappable or saved locally as a shapefile?

There was spam that there's a brand new set of Esri R libraries.
I'm working at what is currently line 138 in data_prep.r

for now I faked it by turning the service into a shapefile and downloading that.
tldr; https://r.esri.com/r-bridge-site/docs/layers/read-layers.html


reviews and ratings

created reviews and ratings page

https://www.figma.com/file/LEKdq0JqyuzzOGxv0tZWjh/WalletFriendly?node-id=925%3A5109


Update builders to vector of strings in privacy struct

Motivation
According to mev-share specification builders are specified by their names which you can find on this page
https://github.com/flashbots/dowg/blob/main/builder-registrations.json
Solution
Updated datatype to accept strings
PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

Thank you
Will update in a bit
-------- Original Message --------
On 6/27/24 5:07 PM, Aurélien  wrote:

@.***(https://github.com/igorline) The Privacy struct is used for eth_sendBundle also, see:
https://github.com/alloy-rs/alloy/blob/097e21a607395fc9a496d620c14934a736989e14/crates/rpc-types-mev/src/eth_calls.rs#L180
So I think you should add a generics Privacy setting it to Address for eth_ and String for mev_.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@leruaa eth_sendBundle according to specification does not have privacy field https://github.com/flashbots/mev-share/blob/09d9ea87d4f907aa69ffd914802209ae8ea680a4/specs/bundles/refund-recipient.md#eth_sendbundle
but eth_sendPrivateTransaction does have this field, though again builders are specified using their names according to this https://docs.flashbots.net/flashbots-auction/advanced/rpc-endpoint#eth_sendprivatetransaction
Am I missing something?

@igorline No you are right, I read too fast, sorry!


"Collapse consecutive whitespace" operation does not collapse all possible unicode whitespace

The "Collapse consecutive whitespace" operation does not work when applied to certain whitespace unicode characters.
To Reproduce
Steps to reproduce the behavior:

First, import this openrefine project: Clipboard.openrefine.tar.gz
Then, run the "Collapse consecutive whitespace" operation on the only column

Current Results
The cell is not edited
Expected Behavior
The cell should be edited to "hello world"
Versions

JRE or JDK Version: 11
OpenRefine: 3.6-SNAPSHOT

Datasets
Real world dataset where this appears: https://opendata.paris.fr/explore/dataset/lieux-de-tournage-a-paris/information/?disjunctive.type_tournage&disjunctive.nom_tournage&disjunctive.nom_realisateur&disjunctive.nom_producteur&disjunctive.ardt_lieu
Additional context
Discovered while doing a demo at Dataharvest 2022

Hi, I write to let you know that I am interested in this issue and I would like to work on it

fileformat.info is not where you want to look at for Unicode specifications.
You should concern yourself with only those Unicode chars that fall into the Zs space separator category of the Unicode standards. Code Point Types https://www.unicode.org/versions/Unicode14.0.0/ch02.pdf#G286941 and Chapter 4.5 covers General categories https://www.unicode.org/versions/Unicode14.0.0/ch04.pdf
And you need to understand some of the rules in Unocode Text Segmentation annex that concern SpacingMark and https://www.unicode.org/reports/tr29/tr29-39.html
such as

General_Category = Zs and not LineBreak = Glue

@AtesComp This issue might be best left for those that have an intimate understanding of Unicode especially concerning category Zs.
Or you can inspect some of the libraries that do have some maps.  For instance, search for "space" within
https://github.com/apache/commons-text/blob/master/src/main/java/org/apache/commons/text/translate/EntityArrays.java

I tried to find within Unicode.org an HTML page that shows the Zs category, but gave up after a bit.  I think it's in the data downloads for sure.  Anyways, here's a quick primer on a listing from another site, which I have no idea how complete it is:  https://www.compart.com/en/unicode/category/Zs

For sure file format.io is NOT the place for a proper look at the specification.  I was just noting the awful presentation others do on the subject.  The fact that the Java regex failed in this issue was surprising.
The standard for the latest "Unicode" is documented at https://www.unicode.org/Public/UCD/latest/
But that is not a clear representation of UTF-8 which is by far the most used translation encoding.
I've had quite a bit of experience with Unicode over the last 20 years.  Here is a general overview of the related issues:
https://stackoverflow.com/questions/2241348/what-are-unicode-utf-8-and-utf-16#:~:text=A code unit is the,Unicode of 21 bits maximum
The UTF-8 standard is documented here:
https://www.ietf.org/rfc/rfc3629.txt
I can assuredly and unoccquivically report that the U+00A0 Unicode character is universally misrepresented by UTF-8 compliant processors.  As seen here , the encoding for U+00A0 (000 1010 0000) code point must be converted to a 2 byte representation as it fits in the following part of the conversion table:
U+0080 | U+07FF | 110xxxxx | 10xxxxxx
And is, therefore, 11000010 10100000 or C2 A0
Many implementation naively and erroneously reference the U+00A0 Unicode character as 00 A0...including most Java libraries and specifically the one you mentioned.
I hope that is way more convincing.

I knew I had these references somewhere...
You can use the following site to do some Unicode look ups:
https://util.unicode.org/UnicodeJsps/list-unicodeset.jsp
For the interested Zs category, use in the input box:
[\p{Zs}]
Then, press the Show Set button.  It lists the 17 characters referenced above.
However, there are no transform references.
The following link discusses the various transforms:
https://unicode.org/faq/utf_bom.html
If you really want to do a full blown transform converter, get the ICU4J code:
https://icu.unicode.org/
https://github.com/unicode-org/icu
But why since UTF-8 is the dominant transform.
Here's the RegEx Unicode site:
https://www.regular-expressions.info/unicode.html
(See the "Do You Need To Worry About Different Encodings?" part for some hints)
From GeekforGeeks:
https://www.geeksforgeeks.org/how-to-get-and-set-default-character-encoding-or-charset-in-java/
indicated UTF-8 is the default encoding--wrong. Actually, it depends on your OS settings. Early Java was UTF-16 centric. The RegEx function is clearly working on either UTF-16 or raw Unicode code points. Why don't they document it!
The majority JavaScript as UTF-8 will need to be transcoded to UTF-16 for the Java RegEx. Which means:
...get strUTF8 from JavaScript...
String strUTF16 = new String(strUTF8.getBytes(StandardCharsets.UTF_8), StandardCharsets.UTF_16);
...do your regex work, then
String strUTF8 = new String(strUTF16.getBytes(StandardCharsets.UTF_16), StandardCharsets.UTF_8);
...send strUTF8 back to JavaScript...

Or precompile some regex pattens with CANON_EQ and you might get lucky.
And to cap it all off, you can change the default encoding of the JVM using the confusingly-named property file.encoding.
To be clear, even though I knew most of this from past experience, I've still made mistakes with my current coding.
What a royal mess!

This has convinced me of the bigger mess and security issues around JavaScript:
https://encoding.spec.whatwg.org/
I'm re-examining my UTF-8 / UTF-16 encoding issue.

Microsoft OS encodings are supposedly UTF-16 based, but experience shows OpenRefine ingested Excel spreadsheets often have what looks like a UTF-8 NBSP at the end of some text cells
JavaScript is UTF-16 for all strings
Then, the JavaScript RegEx expressions work only on UTF-16 data

I'm having an issue with the U+00A0 vs C2 A0 NBSP character.  I am definitely getting a \uC2A0 character embedded in some OpenRefine ingestions. They appear as an extra space at the end of text in the UI (especially in the GREL expression results).  I would think the \uC2A0 would either show up as a weird A with a space on the UI or an oriental character, but it doesn't--it's a blank space. I thought this was just a UTF-8 character not getting handled properly.  Now, I think its some fundamental OpenRefine ingestion issue.
On another note...
I consider whitespace as "\p{Cc}\p{Co}\p{Cn}\p{Z}".
Please review these and check me on it.  Reject as needed.

I don't want to pop off again before I have some definite info. I will do some comprehensive tests with a data set and recreate it to see if it is related.  I'll create a new issue if not.  I'll look at the various font setting to see if I can get it to display any different.  It appears the same as in the spreadsheet.  I'm suspicious about MS playing games with a regular space and nbsp to manage "presentation" to the user.  I just don't yet understand how it could be \uC2A0 and not \u00A0 in JavaScript.
I've been tracking the Java UTF issues for some time but didn't read about that.  They are not going to default to the local system setting anymore?  That should at least make it a little more predictable.

Hi, thanks for all the inputs. I will send my pr to the issue tomorrow

So I'm not at all expert in encoding or Unicode / UTF8 / UTF16 so I can't comment on any of the underlying issues being raised here, but I want to draw focus on how this issue is framed and how it works:
The title of this issue is:

"Collapse consecutive whitespace" operation does not collapse all possible unicode whitespace

The "Collapse consecutive whitespace" function actually just applies the GREL:
value.replace(/\s+/,' ')
which, in code is doing:
                    Pattern pattern = (Pattern) o2;
                    return pattern.matcher(str).replaceAll((String) o3);

Where str is the cell value, o2 is /\s+/ and o3 is " "
(see https://github.com/OpenRefine/OpenRefine/blob/7e9c2971a61db1449154b9980e7d9ef9f7c6274a/main/src/com/google/refine/expr/functions/strings/Replace.java#L57)
So in terms of ensuring the "Collapse consecutive whitespace" function deals with the specific use case it would seem to me that by far the simplest approach would be to extend the regular expression used in the GREL replace to deal with additional whitespace characters - for some definition of what "whitespace characters" are included.
For example we could use
value.replace(/[\p{Cc}\p{Co}\p{Cn}\p{Z}]+/," ")
based on the comment from @AtesComp

I consider whitespace as "\p{Cc}\p{Co}\p{Cn}\p{Z}"

To be honest this seems to me reasonably aggressive in its scope so we might want to discuss whether there are situations where this would cause issues.
More conservatively we could go for something narrower like:
value.replace(/\p{Z}+/," ")
which I think is equivalent to what @thadguidry suggests when they say:

as of Unicode v14 there are 17 graphic chars in the Zs category [....] So we should try to add all 17 into our source as a map

Finally I'd note that that this is marked as a "good first issue" currently - which would suggest that it's been assessed as a relatively straightforward task. I think if this is a matter of tweaking the regex used then it's very straightforward and definitely a good first issue. But if the scope is more broadly dealing with whitespace in OR then it's by no means straightforward and should be re-labelled :)

Let's remove the "good first issue" tag simply by virtue of the amount of text to read in this issue :)


Question : why not depend directly on nock ?

Hi, I was considering building an http interface to nock when I discovered hock that looks very close to what I need.
But when I opened package.json I saw you didn't depend on nock.
I was wondering why you chose not to forward configurations requests to actual nock ? Doesn't it suit your needs ?

Hi there! Hm, we haven't even considered this when writing hock. I believe we could have used nock's logic without its monkey patching, but writing our own just made sense at the time, since we needed very little features.
Feel free to reopen if you have any more questions!


Support for Mandrill

Self-explanatory - we need a mandrill.py backend - docs here - http://help.mandrill.com/entries/22092308-What-is-the-format-of-inbound-email-webhooks-

We have one now


Add package & executable parameter to container

This allows the user to use a component container implementation other than component_container, such as component_container_isolated and component_container_mt.

Hi,
Thanks for the documentation line, I'm merging.
Good idea and thank you for the PR.


[CFM-63] add FridgeReport db class

Updates

Changed table name: FridgeCheckin is now FridgeReport
Built out FridgeReport class

wrote logic to add a Fridge_Report to the database

Removed State from range key of Fridge table
Added unit tests

Testing

Run Unite Tests and confirmed all tests passed

python -m pytest tests/unit -v

All of the tests pass for me locally and the code looks good to me! :) Two observations on the test_fridge_report: you imported layer_test and get_ddb_connection- are you testing that here as well? I see it tested in test_fridge. Also, a few of the lines are a little long- not sure if a tweak in formatting is necessary? I like how you formatted the tests overall- they're easy to follow.

All of the tests pass for me locally and the code looks good to me! :) Two observations on the test_fridge_report: you imported layer_test and get_ddb_connection- are you testing that here as well? I see it tested in test_fridge. Also, a few of the lines are a little long- not sure if a tweak in formatting is necessary? I like how you formatted the tests overall- they're easy to follow.

@sbeesmiley thanks for those notes. I shouldn't be importing those and I'll delete them. I'll try to make lines smaller thank you :)

This is not just specific to this PR, but we've used a lot of incorrect type hints.
Example:
name: str = data.get("Name", None)

The hint should be Optional[str] and not str, since we're setting the default as None. The hint should've been str if we were using data.get("Name", ""), since the method defaults to a string here.
Let's add a low priority ticket for this correction as well.

This is not just specific to this PR, but we've used a lot of incorrect type hints. Example:
name: str = data.get("Name", None)

The hint should be Optional[str] and not str, since we're setting the default as None. The hint would've been str if we were using data.get("Name", ""), since the method defaults to a string here.
Let's add a low priority ticket for this correction as well.

Nice catch


getting fitted values and residuals

Is there a better way to get the fitted values and residuals after fitting a model?   I used the following code from plot_fit
mu,Y=model._model(model.latent_variables.get_z_values())
values=model.link(mu)
it would be nice if you could just do model.fit().residuals or model.fit().fitted or something like that.

Hi Graham - agreed, it would be nice to make this easier rather than having to construct manually. The results objects are also in need of further documentation, so will make a note of that. For the meantime, the results objects do contain some properties of the model - such as model.fit().signal, model.fit().states, model.fit().data - but will look to document this and add some more properties (like residuals).

Hi,
Is there any update on this post ?
I am struggling to get the residuals of a fitted model.
As far as i'm concerned, model.link(mu) does not work. The function link is unknown., i get the following error: object has no attribute 'link'


ch: use copy instead of direct sourmap cache

In case we load the same cached source file twice, ch won't track that and free the buffer twice.
Originally reported by OSS-FUZZ OS#15921043

LGTM, but I have no context what/how SourceMap is used, maybe @jackhorton can take a look?

I cant say I have a ton of context either, but it looks reasonable to me?

@dotnet-bot test OSX static_osx_osx_debug please

@jianchun @jackhorton Thanks for the review!


1.19 Mob head on note block doesn't seem to work

When placing a mob head (cow, creeper, etc...) on note block. The note block just puts out it sound and not the mobs sound.
1.19
Config:
Enable or disable this tweak.
enabled: true
Display name, seen in-game
display-name: "Mob Head Noteblocks"
Description, seen in-game
description: "Makes note blocks with mob heads attached to them play the mobs sound."
Category
category: "Blocks"

All vanilla mob heads appear to be working fine (Creeper, Zombie, Skeleton, Wither Skeleton, Dragon). Any custom mob heads are not currently supported, as there are many varieties of these available.


Bug Report - Default profile not handled

Environment
Windows build number: [run "ver" at a command prompt]
Microsoft Windows [Version 10.0.18362.175]
Windows Terminal version (if applicable):
0.2.1831.0

Any other software?

Steps to reproduce

Change "defaultProfile" to anything other than a valid profile

Expected behavior

Defaults to an existing profile or uses a default
Actual behavior

Nothing

Hey wadda ya know, this is actually something that I'm spec'ing up as part of #754 (see spec in #1258).
Thanks for the report though! I'll leave this open to track making sure this gets done.

Hit it by accident fiddling around with themes and such.

I believe this was actually one of the many things that was resolved by #2422. This was one of the warning cases I decided to include in that initial PR.


Missing entities after v2 upgrade

After upgrading to v2 I am missing many useful entities.
Before upgrade

After upgrade:

After reinstall:

Notably, I am missing: Appliance state, Time to end, Cycle phase, Cycle sub phase, Door state, Door lock
I tried re-adding my machine and reinstalling from scratch (directly to v.2.0.7). Only thing that brings them back is to downgrade to v.1.0.19
My machine is Electrolux Serie 800 (EW8F8669Q9).
I know the issue is the same as #93 but for some reason that was closed without a fix.

Maybe I was a bit fast on the trigger. Seems that Appliance state had to be enabled manually. Cycle phase, Cycle sub phase, Door state, Door lock were just renamed. Time to end is still missing though.
Also noticed there is no timer entity for the delayed start. And door lock seems to be inverted.

Hi, I am in the same situation. After upgrading  I am missing the entities. What happened?


Default values in combination with object destructuring are not type checked within function declarations.

TypeScript Version:  2.5.3
Code
// ===== string literal default value =====

// TS does not give an error for the default value 'z'
function defaultUsingStringLiteralWithinObject({someInput = 'z'}: {someInput?: 'x' | 'y'}) {
    console.log(someInput)
}

// but when calling the function this correctly gives TS2345 (Type 'z' is not assignable to type 'x' | 'y')
defaultUsingStringLiteralWithinObject({someInput: 'z'})

// p.s. TS correctly gives TS2322 when invalid default without object destructuring
function defaultUsingStringLiteral(someInput?: 'x' | 'y' = 'z') {
    console.log(someInput)
}

// ===== enum default value =====

enum IOtherInput {
    x = 'x',
    y = 'y'
}

// TS does not give an error for the default value 'z'
function defaultUsingEnumWithinObject({otherInput = 'z'}: {otherInput?: IOtherInput}) {
    console.log(otherInput)
}

// but when calling the function this correctly gives TS2345 (Type 'z' is not assignable to type 'IOtherInput')
defaultUsingEnumWithinObject({otherInput: 'z'})

// p.s. TS correctly gives TS2322 when invalid default without object destructuring
function defaultUsingEnum(otherInput?: IOtherInput = 'z') {
    console.log(otherInput)
}

Expected behavior:
Default values - in combination with object destructuring - should also be type checked in the function declaration.
Actual behavior:
Default values - in combination with object destructuring - are not type checked in the function declaration. Noticed this behaviour for both string literals and enums.

Your example is actually working as intended. In a construct such as
function foo({ someInput = 'z' }: { someInput?: 'x' | 'y' }) {
    // Type of someInput is 'x' | 'y' | 'z'
}

the type of someInput within the function is inferred to be 'x' | 'y' | 'z', i.e. a union of the destructuring source type and the default value type. However, the type of someInput required when you're calling the function is 'x' | 'y' | undefined. Effectively, this allows you to use a default value that isn't in the domain of the values you're allowed to pass to the function.

Thanks for the quick response!
Shouldn't the same inferred behaviour then also apply to the examples without destructuring? e.g.
type TSomeInput = 'x' | 'y'

function defaultUsingStringLiteral(someInput: TSomeInput = 'z') {
    // whatever
}

But this now gives a TS2322 : 'z' is not assignable to type 'TSomeInput'. Doesn't feel consistent.
Personally I don't immediately see a strong use case for "allows you to use a default value that isn't in the domain of the values you're allowed to pass to the function". There's functionally no difference if the user would pass the default value or not pass the variable at all.
Instead this inferred additional-allowed-value could cause unwanted situations where an old 'wrong' default value (that wasn't refactored because of no type warning and missing unit test) is still being used.

A scenario motivating the design is described in #10577. Arguably, in your example the type annotation is perhaps a stronger indication of intent, but I also think it is entirely reasonable for the default value to contribute to the eventual type. Either way, we can't break the scenario in #10577, so any change would have to accommodate both that and your example. Not sure if that is possible.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


The code after "client.JoinOrCreate" does not execute

Why is Debug.Log("====== JoinOrCreate ======"); always not implemented?And no exceptions are caught.
public class ExampleManager : ColyseusManager<ExampleManager>
{

    ColyseusRoom<RoomState> room;

    override
    protected async void Start()
    {
        Debug.Log("====== start ======");

        try
        {
            this.InitializeClient();
            this.room = await this.client.JoinOrCreate<RoomState>("room");
        }
        catch (Exception ex)
        {
            Debug.Log(ex);
        }

        Debug.Log("====== JoinOrCreate ======");
    }
}

version: 0.14.8

[Resolved] Because the client Schema does not correspond to the server Schema.
Just use "schema-codegen" to solve the problem.


Is it possible to only download certain files from a directory in Drive?

I have a directory in Drive that I'm downloading from using the following command:
!gshell download --with-id 1taHKxS66YKJNhdhiGcEdM6nnE5W9zBb1 --recursive.
The files in the folder are numbered. Is it possible to download just the files ranging from 700-799 for example?

You first need to run gshell ls for that I think
On Sun, Oct 20, 2019 at 12:17 Cyril Zakka<EMAIL_ADDRESS>wrote:

I have a directory in Drive that I'm downloading from using the following
command:
!gshell download --with-id 1taHKxS66YKJNhdhiGcEdM6nnE5W9zBb1 --recursive.
The files in the folder are numbered. Is it possible to download just the
files ranging from 700-799 for example?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/wkentaro/gshell/issues/17?email_source=notifications&email_token=ABA4LE2NR4Y2LSUWIBELO4LQPQ46FA5CNFSM4JCT3A5KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HS7SVVQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABA4LE53GFQQFHXKPHUUC2DQPQ46FANCNFSM4JCT3A5A
.

--
Kentaro Wada / 和田 健太郎
https://wkentaro.com http://wkentaro.com


Remove unnecessary folder #1 (OH)

_static/author is unnecessary, please remove and pull request

fixed via https://github.com/gsantella/ambucs-site-2023/pull/21


feat(runc-shim): add a version flag

This PR adds two things

expose the parse function from containerd-shim crate and move the namespace check to the boostrap function
implement git hash for helping the version information on the runc-shim.

Closes #120

OK, I'll review it ASAP

you've hit https://github.com/containerd/rust-extensions/issues/147#issuecomment-1684250819 with the integration tests.

@Mossaka a rebase should fix the integration test failing

Let me try rerunning the CI

Once we get this PR merged, can we get a new release cut?

@mxpv could we merge this PR?


Shouldn't the VERSION property have a value of 2.0 (not 4.0)?

RFC 5545 notes about the VERSION property of the CALENDAR object:

A value of "2.0" corresponds to this memo.

In ical.net version 4.0.4, the following simple serialization of a Calendar gives VERSION=4.0
            var calendar = new Calendar();
            var ctx = new SerializationContext();
            var factory = new SerializerFactory();
            var serializer = factory.Build(calendar.GetType(), ctx) as IStringSerializer;
            var result = serializer.SerializeToString(calendar);

I'd like to propose that this is a bug. Namely, it causes issues on Apple devices (i.e. Apple devices will choke even on an empty calendar if VERSION=4.0).

Yes, you're quite right.

Fixed in 4.0.5:
https://www.nuget.org/packages/Ical.Net/4.0.5

Thanks for the quick fix! I did however notice that the PRODID has now also been updated. Was that intentional? It now serializes as:
PRODID:-//github.com/rianjs/ical.net//NONSGML ical.net 2.0//EN
Presumably you didn't want to change this one as well? :)

Nope. :)

3.7.3.  Product Identifier
Property Name:  PRODID
Purpose:  This property specifies the identifier for the product that
created the iCalendar object.
Value Type:  TEXT
Property Parameters:  IANA and non-standard property parameters can
be specified on this property.
Conformance:  The property MUST be specified once in an iCalendar
object.
Description:  The vendor of the implementation SHOULD assure that
this is a globally unique identifier; using some technique such as
an FPI value, as defined in [ISO.9070.1991].
  This property SHOULD NOT be used to alter the interpretation of an
  iCalendar object beyond the semantics specified in this memo.  For
  example, it is not to be used to further the understanding of non-
  standard properties.

Format Definition:  This property is defined by the following
notation:
   prodid     = "PRODID" pidparam ":" pidvalue CRLF

   pidparam   = *(";" other-param)

   pidvalue   = text
   ;Any text that describes the product and version
   ;and that is generally assured of being unique.

Example:  The following is an example of this property.  It does not
imply that English is the default language.
   PRODID:-//ABC Corporation//NONSGML My Product//EN

I'm not sure if by "nope" you mean that you did intend to change the PRODID value, but I can't see anything in the spec that says you must have a "2.0" in it.
Indeed, it mentions it "should" contain a value that describes the product and version and that is generally assured of being unique.
The "product version" in this case would be 4.0.5, wouldn't it? In my opinion, it would be useful when looking at a serialized Calendar to know which version of ical.net was responsible (potentially useful for issue-reproduction and debugging). This seems to be the motivation behind the PRODID property if I understand correctly.

Ugh, I was too clever with encoding the PRODID version. Originally I had thought that VERSION meant something about the library, similar to PRODID. That was why I had linked them.
Then you pointed out that VERSION should be 2.0, so I changed it without looking at any other lines. Brilliant. :1st_place_medal:
Anyway, my intent was for PRODID to stay at 4.0, and VERSION to go back to 2.0

👍

https://www.nuget.org/packages/Ical.Net/4.0.6


TechDebt: remove leftover Python2 compatibility

The cfnresponse module might still be used by Python2 Lambda's invoked by users @a-detiste, so I don't think we can just remove this.
Thank you for the PR though!

Hi.
Yes I m just going through the package list one by one but I don't know everything about everything.
Greetings


Stabilize long runs

Closes #460

[ ] I have read and checked the items on the review checklist.

This was already addressed as part of #460


get a clear browser[Feature]:

Feature request
Hello,
I have asked this question ( https://github.com/microsoft/playwright-java/issues/555),    Can firefox do it?

Playwright currently doesn't support extension automation in Firefox.


Extend SDK layer 2 docs

Motivation and context

Added more explanations on how to use Core API objects

How has this been tested?

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

@nmanovic , the PR is ready for merging.


feat: Adding more informations on My Shares page (table and modal)

The description of the share is now displayed out on the table of My Shares page and by clicking the new button Information, a Modal pops out, on which you can see multiples informations about the share, as shown below.
This Pull Request is related to the Issue #164 . Do not hesitate if you have any other ideas to add on the page, or if you want to modify these changes.
Images :

I'll see that asap, thank you for the review

Awesome work, thank you!
I'm happy to help on this project !

Is the copy link button in the table still necessary? Because you can copy the link in the model.

Yes for sure, but I think the copy link button remains important to make the copy faster, and not disturb people that copied the link like that, I don't really know, but it's not that disturbing

Don't you think we could completely remove the author property in the model? Because its value is hard coded

Yes, I can remove it, indeed the user is in My Shares, so he knows that it's his share.

If the description is too long, the table doesn't look nice in my opinion, maybe you could limit the description length in the table but show the whole description in the modal

I'll work on this, I can limit the height, but when I try working on the width, I can only use px, I've already had this issue using tables before, but I don't know how to solve it, I put a 300px max-width, so it's good on PC but on smartphones, it's terrible.

Btw. I really like the idea of the progress bar for the share size :)

Yes I found that it would fill a bit the interface and the result is good.

I'll work on this, I can limit the height, but when I try working on the width, I can only use px, I've already had this issue using tables before, but I don't know how to solve it, I put a 300px max-width, so it's good on PC but on smartphones, it's terrible.

I've found a way to bypass this error :
display: table;
table-layout: fixed;

But the problem is, by setting the layout to fixed, the table cannot slide as before.
Maybe we could just hide the description on smaller screens, because anyway we'll not be able to see it

Yes for sure, but I think the copy link button remains important to make the copy faster, and not disturb people that copied the link like that, I don't really know, but it's not that disturbing

Yeah good point.

Maybe we could just hide the description on smaller screens, because anyway we'll not be able to see it

Yes I think this would be the best solution as the description can be seen anyways if you open the modal on mobile.

@stonith404

Yes I think this would be the best solution as the description can be seen anyways if you open the modal on mobile.

Do you know how we can do this directly in React ?
Else we can just make a media query

@pierrbt Mantine does support media queries https://mantine.dev/core/media-query/
Something like this:
      <MediaQuery largerThan="lg">
           {/* the description in here */}
      </MediaQuery>

@stonith404
Just added a MediaQuery

@pierrbt LGTM, thanks again!

@stonith404
No problems, I had time to help.
PS : this week I'll not be able to code, I'm not at home


Image Plugin Not Rendering with Django 1.8

I'm using Django 1.8 and have found that the image template at cmsplugin_filer_image/plugins/image/default.html is not rendering.
I know that Django 1.8 has a number of changes to the template system, including one to the behavior of select_template. Instead of returning an object of type django.template.base.Template it returns and object of type django.template.backends.django.Template.
Thus, on line 104, this code:
    def render(self, context, instance, placeholder):
        self.render_template = select_template((
            'cmsplugin_filer_image/plugins/image.html',  # backwards compatibility. deprecated!
            self.TEMPLATE_NAME % instance.style,
            self.TEMPLATE_NAME % 'default')
        )

returns a django.template.backends.django.Template object with 1.8 and a django.template.base.Template with < 1.8.
I've found that adding a line after this:
self.render_template = self.render_template.template

will make the plugin render, but I'm not sure it's the correct solution.

This is caused by https://github.com/divio/django-cms/blob/3.1.2/cms/plugin_rendering.py#L63:
    if not processors:
        processors = []
    if isinstance(template, six.string_types):
        content = render_to_string(template, context_instance=context)
    elif isinstance(template, Template):
        content = template.render(context)
    else:
        content = ''

I guess the best solution would be either to change the code in cmsplugin-filer as you proposed, or to replace the code in django-cms by:
from django.template.backends.django.Template as BackendTemplate
# ...
elif isinstance(template, (Template, BackendTemplate)):
    content = template.render(context)

I'm not comfortable enough with Django 1.8 changes to say which solution would be better.

This has been fixed in 3.1.x branch (https://github.com/divio/django-cms/blob/support/3.1.x/cms/plugin_rendering.py#L64) which is due to release as 3.1.3 shortly

Fixed on CMS 3.1.3


Opt-out of default major/minor/patch labels

I want all my dependencies updates from dependabot to have label "patch". I have added that to my config file.
version: 2
updates:
  - package-ecosystem: "pip"
    directory: "/"
    target-branch: "develop"
    schedule:
      interval: "daily"
    labels:
      - "chore"
      - "patch"

  - package-ecosystem: "github-actions"
    directory: "/"
    target-branch: "develop"
    schedule:
      interval: "daily"
    labels:
      - "chore"
      - "patch"

If a dependency does a major or minor version change then I endup with duplicate labels. I use these labels to set the version of my releases so they must always be patch when coming from dependabot.
Example PR's https://github.com/DontShaveTheYak/cloud-radar/pull/12 https://github.com/DontShaveTheYak/cloud-radar/pull/14

Looks like major/minor/patch logic was discussed here: https://github.com/dependabot/feedback/issues/134
It also looks like we don't have a way to turn it off, which seems... less than ideal. I'm going to quickly re-classify this as a feature request to do so, which would respect your desire to label it as desired.

Potential options include:
Add a special dependabot-ignore: label prefix
version: 2
updates:
  - package-ecosystem: "pip"
    directory: "/"
    schedule:
      interval: "daily"
    labels:
      # these labels get ignored
      - "dependabot-ignore:major"
      - "dependabot-ignore:minor"
      - "dependabot-ignore:patch"
      # these labels get added
      - "chore"
      - "patch"

Pros:

No additional top level config options required

Cons:

Nobody could add a literal "dependabot-ignore:*" label (turns out nobody seems to use this, but it would have to be documented)

Add a new section for default opt-outs
version: 2
updates:
  - package-ecosystem: "pip"
    directory: "/"
    schedule:
      interval: "daily"
    labels:
      - "chore"
      - "patch"
    default-opt-outs:
      - "automatically-adding-semver-labels"

Pros:

Extensible to the other features Dependabot users may want to opt out of
No restrictions on label names

Cons:

It's a giant grab bag of functionality that will only grow over time (it's not aesthetically pleasing)

Any thoughts?

@asciimike I wish I had some valuable insights to give you but I really just want it to work and both proposals are adequate to me.
 I'm just not sure if it will be needed as I don't know the Dependabot roadmap. Dependabot isn't exactly new and you haven't needed a config key for disabling features this far.
The ability to negate a label isn't that bad and I kinda remember when searching issues that someone had wanted to replace an autolabel from Dependabot and this would allow that. I can't really come up with a scenario where you might want minor and patch but not major but I'm sure someone somewhere has that use case. So this one is also kinda future proof in a way.
I actually like the idea of an enable-semver config key with a boolean value. When I was searching for issues before creating this one, lots of people wanted something similar to this and they wanted Dependabot to create the labels for them. If I recall the Dependabot team did not want to create the labels but things might have changed.
Off topic but you might also want to bring back auto merge. I think a auto-merge key would be amazing. Even if all it did was create the correct branch rule in your repo. If not at least document how easy it is to create the branch rule. I have seen lots of issues where people are complaining that auto-merge did not make it to the new github integration but none of the post mentioned setting up the appropriate branch rule to make it work.

@shadycuz thanks for the feedback. Re automerge, #2268 has the most complete set of thoughts. Agreed that documentation is key here, and I'm working on getting some better docs on Dependabot + other GitHub products (mostly Actions), which might help with this.

Two years later, I still struggle with this. Renovate a dependabot alternative supports this feature. This means I could have my PR's automatically merged if the tests pass because the PR was labeled correctly.

My 1c would be to even change the default behavior altogether to not add any label by default: it is a strong (and as we see incorrect) assumption for the meaning of the labels used in a given repo: IMHO it should be opt-in and likely configurable.


Chore: Update actions/setup-python v4 -> v5

@ModeSevenIndustrialSolutions there's merge conflicts that need to be fixed before this can be merged.

Redundant


Support TLS listening natively

I think this is fine to do, regardless of the notice about SSLv2 in the Ergonomadic readme. If I remember correctly, some of the bigger IRCds have disabled using SSLv3 and below anyways.

Also see https://github.com/edmund-huber/ergonomadic/pull/19 for an actual PR on this, and the concerned raised regarding websockets and it. I think something /along these lines/ should work well, but I'll make sure it integrates nicely with the config format.

This is fixed by c3288823afda90d737b33f149c38bc180abd5381. I'll make the HTTPS websocket support a separate issue (and it's marked as a TODO in the code).


Global styles causing side effects in my own styles

https://github.com/robophil/Product-Tour/blob/4d20ea0c6b20f8c2702dd763f408f0d0f1b8c2e3/css/product-tour.css#L6-L24
Can we move this styles to another scope?
I spent more than an hour trying to do this, but I'm very bad with CSS.
The problem is that these styles are breaking my own styles.

Sure, send a PR
I actually am working on an improvement of the entire library, it’s been a while since I touched this.


Go forEachy || ES5y

While using reactive techniques, I forgot all modern browsers support forEach/map/reduce and stuff. Should change for-loops to this hipsterish stuff too.

Also code would be more compact with => from ES7. Not yet sure I want to use Babel, though.

That arrow functions syntax is ES6 and works in the latest Chrome and Safari.

@Nek cool, you are right, thanks. And forEach is ES5, actually :)

But I plan to try Babel at some point anyway, I'm just uncertain if I like it in a result :).

I already use it and I have to say ES6 is ages ahead of ES5.
Please take a quick look at this document to get a feel of it
https://github.com/airbnb/javascript/blob/master/README.md
Also babel can be used as a standalone tool. That's neat 'cause setting up a toolchain is the most cumbersome part of todays JS development.

Also there are usages of both Object.keys()-based loops and for-in-based loops, I should chose the one true way.


nginx + php-fpm + wordpress

nginx + php-fpm + wordpress
nginx in enclave:

do not fork workers due to https://github.com/gramineproject/gramine/issues/468 (also generally dont)
use select instead of epoll due to https://github.com/gramineproject/gramine/issues/489
link pcre static due to undetermined dynloader bug

php in enclave:

patch away sockopt SO_LISTENQLEN and TCP_INFO due to https://github.com/gramineproject/gramine/issues/503
do not fork workers
manually force linking libm static, because the ld loader implements glibc versions incorrectly
remove apparmor from AC

wordpress in enclave:

files are signed by php enclave. manipulating them correctly fails.
installer loads fine but needs mysql server to proceed

wordpress should probably not be checked in.

moved to wordpress repo


cmake failure

I normally us scons, now trying cmake.  I'm not familiar with cmake.
Usually use:
python3 ~/scons/bootstrap.py --config=force --debug=stacktrace --with-boost-include=$(pwd)/../include --with-boost-lib=$(pwd)/../lib --with-eigen-include=/usr/include/eigen3  --prefix=$(pwd)/../.local install CCFLAGS='-g -O3 -march=native' --boost-python-lib=boost_python3
Now to convert to cmake, I have
At least these 2 problems:

wrong python include! (2.7, should be 3.5)
wrong eigen include path

I started with:
cmake -DNDARRAY_SWIG=OFF -DPYTHON_EXECUTABLE=/usr/bin/python3 ..
-- Boost version: 1.60.0
-- Found the following Boost libraries:
--   unit_test_framework
-- Configuring done
-- Generating done
-- Build files have been written to: /home/nbecker/ndarray.new.test/build
[nbecker@nbecker2 build]$ make VERBOSE=1
/usr/bin/cmake -H/home/nbecker/ndarray.new.test -B/home/nbecker/ndarray.new.test/build --check-build-system CMakeFiles/Makefile.cmake 0
/usr/bin/cmake -E cmake_progress_start /home/nbecker/ndarray.new.test/build/CMakeFiles /home/nbecker/ndarray.new.test/build/CMakeFiles/progress.marks
make -f CMakeFiles/Makefile2 all
make[1]: Entering directory '/home/nbecker/ndarray.new.test/build'
make -f include/CMakeFiles/preprocess_includes.dir/build.make include/CMakeFiles/preprocess_includes.dir/depend
make[2]: Entering directory '/home/nbecker/ndarray.new.test/build'
cd /home/nbecker/ndarray.new.test/build && /usr/bin/cmake -E cmake_depends "Unix Makefiles" /home/nbecker/ndarray.new.test /home/nbecker/ndarray.new.test/include /home/nbecker/ndarray.new.test/build /home/nbecker/ndarray.new.test/build/include /home/nbecker/ndarray.new.test/build/include/CMakeFiles/preprocess_includes.dir/DependInfo.cmake --color=
make[2]: Leaving directory '/home/nbecker/ndarray.new.test/build'
make -f include/CMakeFiles/preprocess_includes.dir/build.make include/CMakeFiles/preprocess_includes.dir/build
make[2]: Entering directory '/home/nbecker/ndarray.new.test/build'
make[2]: Nothing to be done for 'include/CMakeFiles/preprocess_includes.dir/build'.
make[2]: Leaving directory '/home/nbecker/ndarray.new.test/build'
[ 38%] Built target preprocess_includes
make -f tests/CMakeFiles/ndarray-eigen.dir/build.make tests/CMakeFiles/ndarray-eigen.dir/depend
make[2]: Entering directory '/home/nbecker/ndarray.new.test/build'
cd /home/nbecker/ndarray.new.test/build && /usr/bin/cmake -E cmake_depends "Unix Makefiles" /home/nbecker/ndarray.new.test /home/nbecker/ndarray.new.test/tests /home/nbecker/ndarray.new.test/build /home/nbecker/ndarray.new.test/build/tests /home/nbecker/ndarray.new.test/build/tests/CMakeFiles/ndarray-eigen.dir/DependInfo.cmake --color=
make[2]: Leaving directory '/home/nbecker/ndarray.new.test/build'
make -f tests/CMakeFiles/ndarray-eigen.dir/build.make tests/CMakeFiles/ndarray-eigen.dir/build
make[2]: Entering directory '/home/nbecker/ndarray.new.test/build'
[ 46%] Building CXX object tests/CMakeFiles/ndarray-eigen.dir/ndarray-eigen.cc.o
cd /home/nbecker/ndarray.new.test/build/tests && /usr/lib64/ccache/c++    -I/home/nbecker/ndarray.new.test/include -I/usr/include/python2.7 -I/home/nbecker/.local/lib/python3.5/site-packages/numpy/core/include -I/usr/include/eigen3/include  -std=c++11 -o CMakeFiles/ndarray-eigen.dir/ndarray-eigen.cc.o -c /home/nbecker/ndarray.new.test/tests/ndarray-eigen.cc
In file included from /home/nbecker/ndarray.new.test/tests/ndarray-eigen.cc:11:0:
/home/nbecker/ndarray.new.test/include/ndarray/eigen.h:26:22: fatal error: Eigen/Core: No such file or directory
#include "Eigen/Core"
^
compilation terminated.
tests/CMakeFiles/ndarray-eigen.dir/build.make:62: recipe for target 'tests/CMakeFiles/ndarray-eigen.dir/ndarray-eigen.cc.o' failed
make[2]: *** [tests/CMakeFiles/ndarray-eigen.dir/ndarray-eigen.cc.o] Error 1
make[2]: Leaving directory '/home/nbecker/ndarray.new.test/build'
CMakeFiles/Makefile2:146: recipe for target 'tests/CMakeFiles/ndarray-eigen.dir/all' failed
make[1]: *** [tests/CMakeFiles/ndarray-eigen.dir/all] Error 2
make[1]: Leaving directory '/home/nbecker/ndarray.new.test/build'
Makefile:138: recipe for target 'all' failed
make: *** [all] Error 2
[nbecker@nbecker2 build]$

Can you give info about your OS?

I just tried again and got a different result
cmake -DNDARRAY_SWIG=OFF -DPYTHON_EXECUTABLE=/usr/bin/python3 ..
-- The C compiler identification is GNU 6.3.1
-- The CXX compiler identification is GNU 6.3.1
-- Check for working C compiler: /usr/lib64/ccache/cc
-- Check for working C compiler: /usr/lib64/ccache/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/lib64/ccache/c++
-- Check for working CXX compiler: /usr/lib64/ccache/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found PythonInterp: /usr/bin/python3 (found version "3.5.2")
-- Found PythonLibs: /usr/lib64/libpython3.5m.so (found version "3.5.2")
-- Found NumPy: version "1.12.0" /home/nbecker/.local/lib/python3.5/site-packages/numpy/core/include
-- Boost version: 1.60.0
-- Found the following Boost libraries:
--   unit_test_framework
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.29.1")
-- Found FFTW: /usr/include
-- Trying to find eigen in
-- Found Eigen3: /usr/include/eigen3 (Required is at least version "2.91.0")
-- Configuring done
-- Generating done
-- Build files have been written to: /home/nbecker/ndarray.new.cmake/build

Now build succeeds.
But it looks like only the headers are installed, the library
libboost_numpy.so
is not installed (and I don't think it got built either)


Example JS to export vertices is broken

Hello,
I am attempting to use the provided JS to export the vertices from the tangram example map: http://tangrams.github.io/tangram/ However it does not work
Could you possibly post the updated instructions (it appears like the JS structure on the map side has changed, possibly including the use of VAOs which might prevent this approach from working on newer versions of tangram)
Alternatively do you have a hosted map of a version of tangram that this vbo-export project supports?

You're right, the JS structure had changed, and I only recently got the export working again… I do plan to update the README as well as link to a hosted version with this functionality built-in, but until then if you'd like a preview of the new solution you can view the work in progress here: https://github.com/meetar/manhattan-project/tree/update
I'll update this issue when the new versions are up.

Ok great, I will continue to follow both projects, and if you feel like there is room for some contribution I wouldn't mind helping out with any bugs / workflow / documentation problems your running into (just open up an issue or something that outlines the problem, and mention me and i will take a stab at it). I have interest in this project for building the 3d data into a minecraft world map, as part of a career/recruitment campaign for my company (we want a 'life size' build of san francisco in game)


[opt](scanner profile) Rename some filed name to keep consistent with audit log. #41993

cherry pick from #41993

run buildall

run buildall

run buildall

run buildall

run buildall


Checkout and basket templates

The checkout and basket apps are currently written as a REST api, the primary motivation being that it allows total customisation of how you integrate it into your website (I was thinking react apps and nice popup modals for easy checkouts).
It would also be useful to provide generic views and templates for separate-page checkouts/basket views.

Given that the project is aimed at creating a universal application, in the initial stage it is necessary to provide an alternative for test order. So as of now to launch a project to set up an applications stripe-payments and braintree-payments.

I think I understand what you are saying - at the moment you would require a braintree, paypal or stripe account in order to test the payments and what we need is a dummy backend? I agree - Ive raised an issue to track this specifically: #6

@alexfromvl...reading over your comment again, perhaps I have misunderstood originally and this issue is actually close to what you meant: #36

@JamesRamm Those changes that you made initially helped to solve the issue of the setting without having to register accounts braintree, paypal or stripe.
Given the development of the project, it will be optimal to transfer the payment settings in the basу.py
With the choice of: for the test environment is disabled, for the finished site is enabled (using the token)

On master now there are generic django views for basket & checkout, plus a checkout template in the project_template which tries to explain and help with some of the more tricky details. I think this is enough to sufficiently close this issue and we can narrow down further problems in more specific issues later on.


Food doesn't look good under yellow light

Consider adding a section to the readme expressing the importance on using a white light if possible for pictures

Open PRs for this kind of stuff.


Clients < 10.0 should be disconnected

Because of the hardfork clients < v10.0 are stuck on the wrong chain. They keep downloading the same blocks on and on and waste lots of bandwidth on uptodate nodes.
From getpeerinfo on my node:
   {         "addr" : "<IP_ADDRESS>:58466",         "services" : "00000003",         "lastsend" :<PHONE_NUMBER>,         "lastrecv" :<PHONE_NUMBER>,         "bytessent" :<PHONE_NUMBER>,         "bytesrecv" : 205288091,         "blocksrequested" : 2,         "conntime" :<PHONE_NUMBER>,         "version" : 70002,         "subver" : "/Satoshi:<IP_ADDRESS>/",         "inbound" : true,         "startingheight" : 363666,         "banscore" : 0     },
And this is just one example peer.

Done


Create 'edab' package?

Some functions should be consistent across EDAB products:

[x] "get data" steps - database pulls should aggregate data and return standard output
[x] standard data - species' strata sets, trophic group, fleet/sector, etc, (e.g., data(strata_sets, package = "edab"))
[x] plotting - takes standard data from "get data" and plots consistently across products (with good aesthetics.)
[ ] ...

Applications

SOE
ECSA
Internal/external collaborations (e.g., edab::update_survdat() could set up as a bash to update monthly and push straight to ERDDAP)
Shiny

Caveats

Not all functions for these products are in R
Not all data are public
Ensure balance in investment/reward
data hashing...

@seanhardison1 addressed this with http://github.com/noaa-edab/ecodata


Fix/ui performance

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This PR makes some optimizations in Part.getNotes, Segment.getNotes in order to reduce the amount of DB calls. This gives two improvements:

Decrease the number of reactive dependencies (improves Tracker performance)
Make fewer, but larger calls to DB (increases performance)

What is the current behavior? (You can also link to an open issue here)
A call to getResolvedSegment could take up to 60ms (when tested locally)

What is the new behavior (if this is a feature change)?
No behavioural change, a call to getResolvedSegment reduced to ~19ms.

Other information:
This PR builds upon https://github.com/nrkno/tv-automation-server-core/pull/117

Status

[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

I only benchmarked this client-side, and I have not tested the functionality yet either :)

@nytamin @jstarpl Is this pr still relevant or is it 'replaced' by https://github.com/nrkno/tv-automation-server-core/pull/220 ?

It is no longer relevant, as #220 implements most of the things addressed in this PR.


gcssbom resource: use the resource's "constructor"

Avoid having custom regexp spread across but reuse the "constructor" instead.

/lgtm


Migrate to MAUI

First - thanks for this great example.  It's an interesting take on rendering to XAML and I found it to be very helpful in a bunch of ways.  That said, Xamarin Forms is officially deprecated :( and I'd really like to see this migrated to MAUI.
The good news is that I've already done the work :)  And in doing the migration, also fixed a bunch of issues due to the platform changes (I'm looking at you StackLayout), added support for Tables and TaskLists, and implemented some other enhancements and fixes.
The thing is -- I don't think a pull request is the proper course of action.  Why?:

The Xamarin Forms version live on as-is in case anyone needs or references it
The entire folder structure has changed due to how MAUI apps are organized
The namespaces have all changed to remove references to Xamarin Forms

So I guess this is really a request to create a new project repository.  If there's no desire to manage it, I'm more than happy to do so -- just wanted to give first opportunity to you.  In any case, the readme should be updated to link to the MAUI version wherever it ends up.
Thanks again!

Since opening this issue, I forked the bares43  version because it had some notable changes, and have completely migrated the project to MAUI.  While doing so, I also added a ton of new features and additional improvements (among which I added mark down table and task list support).
I also refactored the theming system so that it uses the native MAUI/Xamarin Styles and DynamicResources method instead of code.  This gives the developer much more control over how things look and completely separates the visual style from the code.
It's dramatically different from the original libraries now, so I'll probably be posting it as a new project at some point, with credit to yours and bares43's great work.  Will follow-up when it's ready.


Simple currency with optional decimals

I want to use the simpleCurrency formatter but with optional decimals.
For example:
For the number 1200.50 I want the output to be $1,200.50 (if decimals exist the min of 2)
For the number 1200.00 I want the output to be $1,200 (without decimals)
Is this possible?

Any solution? I have the same problem 🤯

I am also looking for this feature. If the decimal digits are there they should be shown, but if they are not, no decimal digits should show.

This is related to #429 as both require some formatting string of the form #.#0, but keeping this open as a reminder.


client key shouldn't be required

As per the parse server docs:

Parse Server does not require the use of client-side keys [...] The Application ID is sufficient to secure your app.

source
This library however raises an error when the client key is not specified. How can we work around this restriction?
Thanks in advance.

@jlhonora The ruby does not require you to provide all the keys, you can initiate it with just the application ID or whatever combination you prefer.

[11] pry(main)> client = Parse.create(application_id: 'foobar')
=> #<Parse::Client:0x007f9009f53de0
@api_key=nil,
@application_id="foobar",
@backoff_factor=2,
@host="https://api.parse.com",
@interval=0.5,
...
[13] pry(main)> client.query('_Installation').get()
I, [2016-06-30T17:39:42.196518 #84545]  INFO -- : Parse query for /installations {"where"=>"{}"}
I, [2016-06-30T17:39:42.197089 #84545]  INFO -- : get https://api.parse.com/1/installations?where={}
I, [2016-06-30T17:39:42.690031 #84545]  INFO -- Status: 401
Parse::ParseProtocolError: : unauthorized
from /Users/rhymes/.rvm/gems/ruby-2.3.1/bundler/gems/parse-ruby-client-18c7f9952564/lib/faraday/extended_parse_json.rb:30:in `process_response'

As you can see I only get an error when I actually call the server. Do you have a traceback to provide?

@rhymes hmmm, can't seem to replicate it now, so I'm closing this.
In any case, the docs are a bit misleading in this part:
:api_key        => "<your_api_key>", # required

Anyways, thanks a lot for taking a look.

@jlhonora I've updated the docs, thanks for the catch!
https://github.com/adelevie/parse-ruby-client/commit/9b79ec7dfb1726815636c9fa1b3429c57bf068fa


HLS: autoPlay doesn't work when there's jQuery instance on the page

I've been happily using 0.51 version of Clappr to play my .m3u8 streams and tried updating to a newer release. After trying 10 of them I determined that I can't get any version of the player from .65 on to play in Chrome (latest version 39). I suspect it's related to the above change, but I do not know:
HTML5Video.canPlay = function(resource) {
if (Browser.isSafari || Browser.isMobile || Browser.isWin8App || Browser.isLegacyIE) {
return true;
} else if (Browser.isChrome || Browser.isFirefox || Browser.isIE) {
return (!!resource.match(/(.*).(mp4|webm)/));
}
};
I tried forcing the response to 'true' (as it was before) or add "m3u8" (my stream type) to the matcher, all I get are the pulsing dots and no video.
Any ideas?

hey @konashark can you provide your source url?

Also, if you're compiling from scratch, I would suggest you to try playing your stream on  http://cdn.clappr.io. If it works, your workspace is dirty of old dependencies (we changed a lot of them during this time range).

I haven't been compiling anything - I was just taking the clappr.js and clappr.map files from the 'dist' folder of the releases. Am I missing some critical step that I should know?
I can't give you my url directly as it is inside a corporate network, but happily, I see that if I plug the url into the cdn.clappr.io page under the same browser, it plays there! I will need to continue experimenting, but so far it has been 100% reproducible, drop in the files from .64 and it plays, drop in the files from .65 or beyond and it doesn't. (And by the way - a big Thanks to you guys for taking on such an incredibly useful project!)

sure @konashark you can send me directly on<EMAIL_ADDRESS>

hey @konashark I think we introduced a bug when setting width/height as percentage. Version 0.0.83 fix this, I could play the video using the parameters you provided me in private. Here is the full code of my jeff.html :)

Make sure that you're loading version 0.0.83 of the player (by typing Clappr.version on your console). Tell me if it works for you and if so, please consider close this issue, ok?

Hey @konashark can you please try to play this url and tell me if works? http://flv.io/jeff.html

@konashark latest version of clappr doesn't support width/height as percentage anymore. I suggest you to use window.innerWidth and window.innerHeight instead of 100%

hi @konashark any update on this? I couldn't reproduce it by adding jquery-2.1.1 in the same page that the player is embedded on.

Hi!
I temporarily had to move on, but in response to your question I set up an
Apache server on my wife's MacMini and tried the same test code (Mac
Yosemite + Chrome 39) there to see if it behaved any differently.
Exactly the same results as on my work laptop: m3u8 plays automatically if
no jQuery. With jQuery, it won't auto start, but clicking the play icon
starts it playing. Works fine under Safari. MP4 auto plays with or without
jQuery.
If I don't use a web server, then it won't play m3u8 at all... presumably
some cross domain issue or something.
-Jeff
On Thu, Jan 15, 2015 at 9:43 AM, Flávio Ribeiro<EMAIL_ADDRESS>wrote:

hi @konashark https://github.com/konashark any update on this? I
couldn't reproduce it by adding jquery-2.1.1 in the same page that the
player is embedded on.
—
Reply to this email directly or view it on GitHub
https://github.com/globocom/clappr/issues/323#issuecomment-70127852.

Hey,
Yes, if you don't use a web server, no flash components will work. So, let's check if I understood. The scenario is: auto play doesn't work for latest clappr trying to play m3u8 source on chrome 39 with jquery embedded on the same page. Am I right?

Yes. All tests on a Mac.
On Sat, Jan 17, 2015 at 11:29 AM, Flávio Ribeiro<EMAIL_ADDRESS>wrote:

Hey,
Yes, if you don't use a web server, no flash components will work. So,
let's check if I understood. The scenario is: auto play doesn't work for
latest clappr trying to play m3u8 source on chrome 39 with jquery embedded
on the same page. Am I right?
—
Reply to this email directly or view it on GitHub
https://github.com/globocom/clappr/issues/323#issuecomment-70380656.

great @konashark, I could reproduce here. I'll try to fix right now.

it should be fixed now. Please note that we do not support percentage on width/height anymore, so I suggest you to use window.innerWidth and window.innerHeight instead of 100%. We're going to bump a new version in a few hours. Thank you for all the support and patience!

Horray!
I grabbed the latest version and now I see the .m3u8 autoplay in Chrome

jQuery on the Mac.
I'm glad you found it... thanks for your help, and I'm glad I'm not crazy.

-Jeff

Flávio Ribeiro<EMAIL_ADDRESS>January 21, 2015 at 8:21 AM
it should be fixed now. Please note that we do not support percentage
on width/height anymore, so I suggest you to use |window.innerWidth|
and |window.innerHeight| instead of 100%. We're going to bump a new
version in a few hours. Thank you for all the support and patience!
—
Reply to this email directly or view it on GitHub
https://github.com/globocom/clappr/issues/323#issuecomment-70867741.


As a Metric user I want corrector queries to use DB indexes

Currently query executed by get_faulty_raw_documents() is slow because it does not use indexes:
https://github.com/nordic-institute/X-Road-Metrics/blob/develop/corrector_module/opmon_corrector/database_manager.py#L110-L118
It can be solved for new installations by adding new index to raw_messages in xroad-metrics-init-mongo:
https://github.com/nordic-institute/X-Road-Metrics/blob/develop/collector_module/opmon_mongodb_maintenance/create_indexes.py#L32
[('corrected', ASC), ('xRequestId', ASC)]

Or by running indexing command manually:
db.raw_messages.createIndex({"corrected": 1, "xRequestId": 1})

Additional notes: If would be great to mention the need to create new indexes in release notes.

Hello,
Thank you for the report. We will look into it.


Fix ShardingFilter import

Stack from ghstack:

#187
-> #186

@nivekt has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


ISPN-14594 SADD plus internal impl for set

https://issues.redhat.com/browse/ISPN-14594
initial dev for set commands

you need to add some test for distributed mode as well, to execute tests from owner and not owner

there's any example I can use as reference?

DistributedMultimapListCacheTest @rigazilla

DistributedMultimapListCacheTest @rigazilla

added DistributedSetTest.java


odd text block, line height formatting on profile bios

example:

@csherron What's the issue here? Is it still a problem?

line breaks - look at his legend on browser and compare it to mobile

I'm going to say this is post-launch instead. We can't really do much about this right now without writing our own HTML renderer.


GetInfoExtensionResponse parsing issue

tried to create
use 1.6.0 SDK version
get request on get extensions
and parse the 101 extension with jakson over GetInfoExtensionResponse.class
started the get that issues couple of days ago on UAT environment
And have the next error stack trace:
Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Request processing failed; nested exception is java.lang.IllegalArgumentException: Unrecognized field "empty" (class com.ringcentral.definitions.GetExtensionInfoResponse), not marked as ignorable (24 known properties: "contact", "departments", "regionalSettings", "partnerId", "permissions", "site", "extensionNumber", "setupWizardState", "customFields", "extensionNumbers", "uri", "account", "profileImage", "type", "id", "roles", "statusInfo", "references", "name", "serviceFeatures", "status", "callQueueInfo", "hidden", "upWizardState"])
api              | 2021-06-01T12:42:21.433594156Z  at [Source: UNKNOWN; line: -1, column: -1] (through reference chain: com.ringcentral.definitions.GetExtensionInfoResponse["empty"])] with root cause
api              | 2021-06-01T12:42:21.433601128Z com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "empty" (class com.ringcentral.definitions.GetExtensionInfoResponse), not marked as ignorable (24 known properties: "contact", "departments", "regionalSettings", "partnerId", "permissions", "site", "extensionNumber", "setupWizardState", "customFields", "extensionNumbers", "uri", "account", "profileImage", "type", "id", "roles", "statusInfo", "references", "name", "serviceFeatures", "status", "callQueueInfo", "hidden", "upWizardState"])

@salnikovp0  I recommend you to try the latest SDK version 1.7.3
And it is appreciated if you could post some code snippets to reproduce the issue.
And can you reproduce the issue in sandbox or production environment?

I have access only to UAT environment.
here is a code example in java
var response = executeRequest("GET", "account/~/extension", "");
var map = mapper.readValue(response.body(), Map.class);
var usersNotActivated = (List<Map<String, String>>) map.get("records");
var user = usersNotActivated.stream()
.filter(a -> a.getOrDefault("extensionNumber", "").equals("101"))
.findFirst();
if (user.isPresent()) {
return mapper.convertValue(user, GetExtensionInfoResponse.class);
} else {
return null;
}
it fall in the parsing GetExtensionInfoResponse.
again this code worked long time(6 month)

That's not the way that we are expecting you to use the SDK.
Please check https://github.com/ringcentral/ringcentral-java/blob/master/samples.md#listextensions
You could just replace the whole method with
GetExtensionListResponse result = rc.restapi().account().extension().list();
return result.records.filter(....).findFirst();

The exception is thrown by return mapper.convertValue(user, GetExtensionInfoResponse.class);
What is mapper?  We don't support it if it is your customer code.
And it should be GetExtensionListInfoResponse instead of GetExtensionInfoResponse  according to https://github.com/ringcentral/ringcentral-java/blob/master/src/main/java/com/ringcentral/definitions/GetExtensionListResponse.java#L13


Refs 231: add /settings route

so that we can use the settings navigation

this adds a second route, which we will use in the navigation.  We could look at removing the first route, but i wanted to do this incrementally and not break all tests

⚠ This task currently requires qe-approval, but this PR does not fully resolve the task.  Please contact QE to determine appropriate testing required.

https://issues.redhat.com/browse/HMS-363


Fixes An Image get orphaned from its imageSet if a previous image build failed.

Description
When updating an image from a failed image build the new update image get orphaned from it's ImageSet.
Fixes # THEEDGE-1479
Type of change
What is it?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I run go fmt ./... to check that my code is properly formatted
[x] I run go vet ./... to check that my code is free of common Go style mistakes

Codecov Report

Merging #501 (9833413) into main (1339724) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #501   +/-   ##
=======================================
  Coverage   31.02%   31.02%
=======================================
  Files          33       33
  Lines        3226     3226
=======================================
  Hits         1001     1001
  Misses       2131     2131
  Partials       94       94

Impacted Files
Coverage Δ

pkg/services/images.go
11.61% <0.00%> (ø)

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 1339724...9833413. Read the comment docs.


[PE-D][Tester D] Excessively long claim descriptions may lead to misleading cutoff in text

In screenshot below, a long description (although unlikely) may lead to incorrect reading of data. This is different from the previous issue as fixing the text overflow for Policy enumeration may differ from that of claims. Alternatively, the overflow may be fixed with ellipsis truncation to indicate a longer text.

Further width:

Labels: type.FeatureFlaw severity.VeryLow
original: jloh02/ped#9

same UI bug

not a bug


nixos/podgrab: add user/group/dataDirectory options

Description of changes
I want to use it in conjunction with Audiobookshelf, having control over the user/group makes it easier.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] 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/)
24.05 Release Notes (or backporting 23.05 and 23.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3691

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3773


ChildView throws exceptions outside MojoShell

We're missing a null check.
Fixes #2949

@apwilson

lgtm


support token authentication

According to this post: https://github.blog/2020-12-15-token-authentication-requirements-for-git-operations/ login/password authentication will stop working by August.
This extension probably needs some work to migrate to the new authentication system

Hi @TomVergote , Thanks for your comment and the article. The article you posted might not affect the extension since you have the possibility to generate a personal access token in github and use it instead of the password (https://docs.github.com/en/github/authenticating-to-github/creating-a-personal-access-token).  The implementation for the authentication in Java would remain the same.
However, I have investigated the usage of SSH Keys for authentication and it wouldn't be very large effort to implement this functionality. I will take into account implementing authentication using ssh keys in the next release of the extension.

As of now SSH Keys are not planned to be added as an authentication option because all the Git providers out there support Personal Access Tokens. When that will change, we'll add the SSH keys as well.


README instructions fail for downloaded copy

The README instructions fail if the source code is downloaded rather than cloned because the downloaded copy does not contain a git repository.

I've updated the README in the develop branch.  Thanks for reporting this.


Define a BadPixelMap Class

Define a new class in data.py that can handle bad pixel maps, in a similar way as other data types are handled.

@kjl0025 I gave it a first stab in branch 'badpixel': data.py. Please review and modify as convenient?

Closed with PR #83


Add capitalism

About the PR
Ho boy.
This is one step further towards a fully functioning economy system in SS14.
There are, however, some problems with this PR (which is why was a draft), namely:

Money sprites seem to not update when spawned (spawning as an assistant makes credits display as 1)
Even when they do update, shit goes wrong sometimes and you get 130 credits displaying as a stack of 10 instead of 100 (didn't RTFM hard enough? illegal rsi?)
Mime satchel doesn't contain money, because it's a generic satchel. Fixable with a fake mime satchel.

Screenshots

Changelog

:cl: router

add: Added capitalism. Everyone spawns with some amount of credits now.

what the fuck is this merge

this needs to be split into two PRs, no reason they should be in the same

update: everything now appears as a 1 credit bill when spawned, so that could be merged (but then someone has to deal with StackVisualizer code later, probably me)
as for borders, they tend to disappear out of PVS when outside of its range, and it causes clients to rubberband

I mentioned it on discord but I don't think the cash should be added to people at roundstart:

Waste of entity count for fluff, especially considering it takes up a pocket slot.
Economy still TBD.

Better for mappers to just add it where needed (e.g. casino).

Then again, a 50 player server creates at least 500 entities (7 in backpacks + PDA + jumpsuit + boots), more depending on roles, credits would be at least useful in RP scenarios (gambling, barter, etc).
Is it REALLY this big of a concern?

In SS13 para you never spawned with money as an entity, it was part of a bank account on your ID card or with a round start account and PIN number.
That way if you didn't want to use it you didn't have to. Otherwise it's a stack of ways to fly about in space.

Otherwise it's a stack of ways to fly about in space.

 I'll comb through the code to see how I could avoid that

i think having jobs spawn with these for now is a little silly, i'm fine with the entities being in but they should probably just be in the vault and as salvage rewards or something for a little fluff

you coming back to this @BasedUser ?

you coming back to this @BasedUser ?

They are Russian, I'd give them a longer grace period.

sure am


How to send Messages into a topic

Hi everybody,
the messaging example works fine so far but there is no example how to send a message to a topic?
Someone has an example or idea howto?
I dont know how to send the http post request and which authkey i have to take.
Please help.

@mole84
https://firebase.google.com/docs/cloud-messaging/android/topic-messaging
scroll down to "Topic HTTP POST request"
It looks like the C++ and Unity docs have managed to get mangled, I've poked our tech writers to fix it.
Cheers,
Stewart

Hi stewartmiles,
thank you, but what means "Bearer" and how to get the Key after "Bearer"?
"Authorization: Bearer ya29.ElqKBGN2Ri_Uz...HnS_uNreA"
Best regards
Mole

https://security.stackexchange.com/questions/108662/why-is-bearer-required-before-the-token-in-authorization-header-in-a-http-re
The token is accessible by:

Opening the Firebase Console
Click the gear next to Project Overview
Click Project Settings
Click the Cloud Messaging tab
Find the Server Key field, that's the authorization key you need to use.

I recommend testing with the curl command line first (following the example).  If you're sending messages from your C# project, I hope that's from your backend.  If you embed your server key in your mobile application other developers could unpack your app, steal your auth token and send messages to your users.

thx, i got it!
It works now, i put a php script to my server and send the pushNotification by curl!