Can't build RealmSwift version 10.12.0 with carthage using xcode 13.1

How frequently does the bug occur?
All the time
Description
This is the log file I have got
error.log
Stacktrace & log output
It always output this error

~/D/e/IOSEko ❯❯❯ carthage bootstrap --platform iOS --use-xcframeworks                                                                                release-14-21-0 ✭ ✱ ◼
*** Checking out realm-cocoa at "v10.12.0"
*** xcodebuild output can be found in /var/folders/sm/8xkrqdm136n2_0mlvdzbv_2w0000gn/T/carthage-xcodebuild.JS73t8.log
*** Downloading realm-cocoa binary at "v10.12.0"
***  Skipped installing realm-cocoa binary due to the error:
	"Incompatible Swift version - framework was built with 5.4.2 (swiftlang-1<IP_ADDRESS> clang-1<IP_ADDRESS>) and the local version is 5.5.1 (swiftlang-13<IP_ADDRESS> clang-13<IP_ADDRESS>)."

    Falling back to building from the source
*** Building scheme "Realm iOS static" in Realm.xcworkspace
*** Building scheme "Realm" in Realm.xcworkspace
*** Building scheme "RealmSwift" in Realm.xcworkspace
Build Failed
	Task failed with exit code 65:
	/usr/bin/xcrun xcodebuild -workspace /Users/sainttail/Desktop/eko-project/IOSEko/Carthage/Checkouts/realm-cocoa/Carthage/Realm.xcworkspace -scheme RealmSwift -configuration Release -derivedDataPath /Users/sainttail/Library/Caches/org.carthage.CarthageKit/DerivedData/13.1_13A1030d/realm-cocoa/v10.12.0 -sdk iphoneos ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive VALIDATE_WORKSPACE=NO -archivePath /var/folders/sm/8xkrqdm136n2_0mlvdzbv_2w0000gn/T/realm-cocoa SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO STRIP_INSTALLED_PRODUCT=NO (launched in /Users/sainttail/Desktop/eko-project/IOSEko/Carthage/Checkouts/realm-cocoa)

This usually indicates that project itself failed to compile. Please check the xcodebuild log for more details: /var/folders/sm/8xkrqdm136n2_0mlvdzbv_2w0000gn/T/carthage-xcodebuild.JS73t8.log

### Can you reproduce the bug?

Yes, always

### Reproduction Steps

Build using carthage on xcode 13.1 for Realm 10.12.0

### Version

10.12.0

### What SDK flavour are you using?

Local Database only

### Are you using encryption?

No, not using encryption

### Platform OS and version(s)

iOS

### Build environment

Xcode version: 13.1
Dependency manager and version: Carthage 0.38.0

Hi @sainttail you need to upgrade to our latest version v10.18.0(https://github.com/realm/realm-cocoa/releases/tag/v10.18.0), so Carthage work with your current installation of Xcode (13.1).

@dianaafanador3
Thanks for your response.
I'm asking this because I can successfully build this by using xcode 13.1 clone realm-cocoa and manually build RealmSwift inside xcode
Is there a suggestion for me to make this work for carthage because I need to stick with this version of realm since our 3rd library depend on this particular version. They only provide Realm.xcframework.
However, because our app also use RealmSwift, thus I need to manually build this myself.

Hey @sainttail
You can use another dependency manager for your third party library and RealmSwift (SPM or Cocoapods), I don't know if this is not possible in your case for any reason?

I will try this tomorrow the 3rd that we use is manaully download from the site.

Hey @sainttail You can use another dependency manager for your third party library and RealmSwift (SPM or Cocoapods), I don't know if this is not possible in your case for any reason?

Closing this as there is no issue here


BUG: don't raise on empty usecols

Title is self-explanatory.

yep this fine. ping on green.

[Current coverage][cc-pull] is 84.23%

Merging [#13402][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master     #13402   diff @@
==========================================
  Files           138        138
  Lines         50805      50805
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42795      42796     +1
+ Misses         8010       8009     -1
  Partials          0          0

Powered by Codecov. Last updated by [d405bf2...8eed8d1][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/d405bf26a03b3ecdaa30b62c4b1dd5cb22930329...8eed8d13bfc93883bd5b6e259e460693b419d1b4
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/13402?src=pr

@jreback: Travis is giving the green light.  Ready to merge if there are no concerns.


Added registry tests

Fixed alias symbol bug (https://github.com/prayerslayer/js.spec/issues/19)

Coverage increased (+0.2%) to 89.922% when pulling 0fb1387c1b18f408a7a090eaba126c7cd473c705 on mattbishop:registry-tests into 8515fd61173fc6ae143b005d4bfab9730c73b2d6 on prayerslayer:master.

Coverage increased (+0.7%) to 90.421% when pulling d8531bea03d847db2ede1cffc74e8df89e9a26bf on mattbishop:registry-tests into 8515fd61173fc6ae143b005d4bfab9730c73b2d6 on prayerslayer:master.

Yes I agree. How about we sort out a spec (!) for the registry in a separate issue and then I can adjust this PR to match.

That sounds like a good idea. Since I don't know how to do this properly I made a brain dump in https://github.com/prayerslayer/js.spec/issues/21, feel free to comment!

Closing after discussion in #21


Removing profile picture

Subject: Trying to remove picture
Description:
I wanted to remove my profile picture but I can't seem to do it.
Results:
I should have been able to remove my profile picture.
Backend version: develop-8c664ae7
Frontend version: bug-search-page-zoom-3a31410c
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/profile/edit/about
User: exaustralian (335)

Not planned right now


Polygon layer: How to get attribute popup even if Fill setting is disabled?

I have noticed that with the Polygon layer, e.g. the demos at:
https://deck.gl/#/documentation/deckgl-api-reference/layers/polygon-layer
and at
https://kepler.gl/demo/movement_pittsburgh
If I turn off the Fill color setting, then the attribute popup is only visible when one mouse hovers over the shape's stroke. If the stroke width is set to a small value, this can be quite difficult to hit and get to see the attribute popup (or user might not even know they can get the attribute popup).
I tried enabling Fill color, and experimenting with the opacity, setting close to zero, which enables the attribute popup display again. But it also makes the lines close to transparent, so that's not going to work.
Are there any secret settings or plans for the hit-testing algorithm that could improve this?

you can redefine fill color accessor with low alpha like this:
getFillColor: d => return [128, 0, 0, 0];;

so stroke color won't be impacted.

Nice workaround! thanks


Review script for paper "2022ApJ...926...56K" made with predictor "DataLinkerCondenseDetailsWithTemplate"

Created by Paper Data Linking Bot
Please review the download script and instrument details for the paper.
bibcode: 2022ApJ...926...56K
predictor: DataLinkerCondenseDetailsWithTemplate

The instruments in the paper are accuratley identified as HMI and AIA.
HMI data script fetches data successfully with warning:
SunpyUserWarning: VSO-C500 :soap:Server : ERROR:  relation "vso.aia_lev1" does not exist
AIA unsuccessful retrieval of data.

The time range is broad, but this is not a problem. The paper references data from instruments HMI and AIA at specific dates and times. These dates and times span from 2010 through 2015.


Can't navigate to new Scene

Version
Tell us which versions you are using:
"react-native-router-flux": "^4.0.0-beta.27",
"react-native": "^0.52.2",
Expected behaviour
Be able to navigate between two scenes with Actions.Loading2()
Actual behaviour
If I add a reducer can see if the reducer is called, but nothing happens
Steps to reproduce
Here is my whole app:

import React, { Component }  from 'react';

import {
  Scene,
  Router,
  Stack,
  Reducer,

} from 'react-native-router-flux';

import {Text} from 'react-native'

import {
    Actions,

  } from 'react-native-router-flux';
class Loading extends Component{
    render(){
      return (
        <Text
          onPress={Actions.Loading2}>LOADING</Text>
      );
    }
  }

  class Loading2 extends Component{
    render(){
      return (
        <Text
          onPress={Actions.Loading2}>LOADING2</Text>
      );
    }
  }

const reducerCreate = params => {
  const defaultReducer = new Reducer(params);
  return (state, action) => {
    console.warn('ACTION:', action);
    return defaultReducer(state, action);
  };
};

 class App extends Component{

  render()
  {
    return (
      <Router
      createReducer={reducerCreate}>

      <Stack
          key="root">
          <Scene key="Loading" component={Loading} title="Loading" />
          <Scene key="Loading2" component={Loading2} title="Loading2" />

      </Stack>

    </Router>
    )
  }
}

export default App

I would expect clicking on the text to switch the scene, but nothing ever happens. I have tried every iteration of Actions.Loading, Actions["Loading"], etc that I can think of and nothing seems to work
I have also switched out the Stack with a Scene, with no effect.
I feel like I must be missing something super basic, but I cannot figure this out.

rm -rf node_modules
rm yarn.lock
yarn cache clean
yarn install

If you're on Android : cd android && ./gradlew clean
Then run on your device


make an S3 fetcher

fetch.s3, should be similar to fetch.sciencebase in that we can authenticate for the user and check timestamps to download IFF necessary

attempt was made here: https://github.com/aappling-usgs/water-use-15/pull/1/files#diff-f324d75fc4ef6de518ee85ca776b9fc3R4

:+1: yes, that's what inspired me to create this issue. thanks for linking. that code may already be complete for the actual fetching, in which case all the thought work still to do will be for the fetchTimestamp function


How to set up code: in errorFormat

Hi
in config/api.php, there is a setting key "errorFormat", default is
'errorFormat' => [         'message' => ':message',         'errors' => ':errors',         'code' => ':code',         'status_code' => ':status_code',         'debug' => ':debug',     ],
after i tried to use dingo's response->error() and other methods , looks like only I can setup the "status_code" and "message", is there any way/method to set the code in error response? like
{
"code" => 1001,
"message" => "error message",
"status_code" => 403
}
thanks?

:code is a standard replacement so it should work out of the box, assuming your exception has a code associated with it. If it does not have a code then it won't be included.
The $response->error() method (and related methods) don't implement the adding of the code. So you'd need to throw the exception yourself.
throw new HttpException(404, 'User not found.', null, [], 5);

That should include a "code": 5 in the error response.

thanks


allow non-zero exit cpupower frequency-info -g

Works - Proof

Shows that the latest image was built


Update on npm scripts, and lint components

Hey! Thanks for your work :)
Here is some little fixes added to the build/lint process, and a few linting.

Thanks a lot for the work. Great, I didn't know that npm now resolves the paths automatically.


Encryption symbol misplaced

Hello, i believe encryption symbol is misplaced.
PragmataPro font.

Could you please provide some more info?
What platform? What version of vim,a minimal vimrc to test against.

Not Able to reproduce on osx with iterm or terminal.app

Tested against all the built in themes

What terminal?

Iterm2

Iterm2

And have you tried a different font/terminal?

I think this is a font or terminal issue. There is nothing vim-airline can do against. So closing.


About example.py

Hello, when I run example.py, an error come up:
Using TensorFlow backend.
Traceback (most recent call last):
File "/home/xinan/PycharmProjects/mtcnn-master/example.py", line 27, in
from mtcnn import MTCNN
File "/home/xinan/PycharmProjects/mtcnn-master/mtcnn/init.py", line 26, in
from mtcnn.mtcnn import MTCNN
File "/home/xinan/PycharmProjects/mtcnn-master/mtcnn/mtcnn.py", line 37, in
from mtcnn.network.factory import NetworkFactory
File "/home/xinan/PycharmProjects/mtcnn-master/mtcnn/network/factory.py", line 26, in
from keras.layers import Input, Dense, Conv2D, MaxPooling2D, PReLU, Flatten, Softmax
File "/home/xinan/.conda/envs/face-mtcnn/lib/python3.4/site-packages/keras/init.py", line 3, in
from . import utils
File "/home/xinan/.conda/envs/face-mtcnn/lib/python3.4/site-packages/keras/utils/init.py", line 6, in
from . import conv_utils
File "/home/xinan/.conda/envs/face-mtcnn/lib/python3.4/site-packages/keras/utils/conv_utils.py", line 9, in
from .. import backend as K
File "/home/xinan/.conda/envs/face-mtcnn/lib/python3.4/site-packages/keras/backend/init.py", line 1, in
from .load_backend import epsilon
File "/home/xinan/.conda/envs/face-mtcnn/lib/python3.4/site-packages/keras/backend/load_backend.py", line 90, in
from .tensorflow_backend import *
File "/home/xinan/.conda/envs/face-mtcnn/lib/python3.4/site-packages/keras/backend/tensorflow_backend.py", line 13, in
from tensorflow.python.keras.utils import tf_utils
ImportError: cannot import name 'tf_utils'
Process finished with exit code 1
And my enviorment is:
Keras (2.3.1)
mtcnn (0.1.0)
numpy (1.16.5)
opencv-python (<IP_ADDRESS>)
pip (9.0.1)
tensorflow-gpu (1.8.0)
I want to ask help, thanks a lot!

Seems to be a compatibility problem between your keras and tensorflow versions.
Check it here: https://stackoverflow.com/questions/57985406/cannot-import-name-tf-utils-when-using-importing-keras

Seems to be a compatibility problem between your keras and tensorflow versions.
Check it here: https://stackoverflow.com/questions/57985406/cannot-import-name-tf-utils-when-using-importing-keras

Thanks a lot! When I downgrade keras to 2.2.0, it works well!!!


ID already in use

I guess my comment was missed at https://github.com/kodadot/nft-gallery/pull/9711#issuecomment-1995125485
tested again and was able to reproduce
opened 2 different accounts in different anonymous browser tabs and minted 5 nfts each at the same time:
error at: 0:34
edit: also messes nft serial numbering
https://github.com/kodadot/nft-gallery/assets/36627808/2a1c1f53-2e5d-4143-bd16-65392b1f6a82

looks like queue is not used or?

looks like queue is not used or?

not sure, but if ppl massmint drops at a similar time we will see errors like this happening

https://github.com/kodadot/private-workers/issues/97


How to inter-blockchain communicate in dawn 3.0?

we find that the struct "generated transaction" for inter-blockchain communication in dawn 2.0 was deleted in the newest version. thus the contract "proxy" failed in the new test net.
the new variable "region" in transaction_header is used to distinguish blockchain, how can we use it to inter-blockchain communicate?

dawn 3.0 is outdated by now and region is not available anymore. Please open a new issue if you have a similar question with version 1.0. Thanks!


Router logs overhaul

Checklist

[x] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated.
[x] Changelog entry has been written

This PR is part of the logging system overhaul. I don't think it's complete, just wanted to start discussion on where I'm headed with this.
Here are the important points on this PR:

logs are going: router -> fluentd -> rabbitmq -> fluentd -> elasticsearch

no more logstash, we do all the parsing in fluentd

end-to-end latency is currently max 10s (see the flush_interval option in the two fluent.conf files
it uses a topic exchange in rabbitmq which makes it easy to attach another queue and copy log messages off elsewhere.

current hierarchy is lagoon.<cluster>.logs.<log_type>
e.g. you can attach a queue to the lagoon exchange with a routing key like lagoon.*.logs.router, and get all the router logs from all clusters.

build the images using:
docker build -t lagoon/router-logs-collector services/router-logs-collector
docker build -t lagoon/logs-tee services/logs-tee
docker build -t lagoon/logs-dispatcher services/logs-dispatcher

Questions:

does the topic queue hierarchy look okay, and extensible?
how should we go about merging this work?
any other comments?

Changelog Entry
Overhaul the router logs system.
Closing issues
n/a

This is pretty outdated. I'll create a new PR.


Remove show() from convergence and static

This removes bokeh show() from the convergence and static analysis.
Automatically showing the plots is not expected when running tests or
scripts. With this modification, the figure is returned and the user can
plot that if wanted.
As an example:
static_analysis = rotor.run_static()
show(static_analysis)

Codecov Report

Merging #136 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #136   +/-   ##
=======================================
  Coverage   63.45%   63.45%
=======================================
  Files           9        9
  Lines        1943     1943
=======================================
  Hits         1233     1233
  Misses        710      710

Impacted Files
Coverage Δ

ross/rotor_assembly.py
75.54% <100%> (ø)
:arrow_up:

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 0bece02...41582d7. Read the comment docs.


feat: keycloak upgrade

Checklist

[ ] Architecture Design Records have been added as adr/*.md and appended to list in adr/_index.md, if applicable.
[ ] The values-schema.yaml file and test/** fixtures have been updated to reflect code changes, if applicable.
[ ] The OpenApi Schema from redkubes/otomi-api project is compatible with definitions from values-schema.yaml file, if applicable.
[ ] Helm releases are meeting otomi's baseline security policies, if applicable.
[ ] Helm chart and helmfile changes are tested against upgrade scenario, if applicable.

There is some comment about this branch. See: #1283

Where are the groups? I created 2 teams but they dont show up in keycloak, now I cannot assign users to a group.

At the moment Teams/groups are not automatically added in keycloak

Perhaps the Otomi-tasks needs to be update to use compatible client.

From the screenshot I see you are checking the master realm in keycloak instead of otomi. Please change the realm in the top left dropdown box and check again.


How Istio works in Dev Spaces

Does Istio work with Dec Spaces now? Our company is using Istio for quite a long time, and we're about to implement Dev Spaces for development. But I got stuck on how to implement Isito. I didn't find any useful info online about this.

Hi @Zachery2008, thanks for bringing this issue to us!  Unfortunately Dev Spaces currently does not integrate with Istio. However I will bring this issue to the team and we will continue to investigate.  In the long term it's possible we may support this.

Please support Istio.  We're looking to leverage it for its security aspects in the Health care space.  So its vital to us.

Support for an ITSIO sidecar shouldn't be too hard to add. https://rancher.com/ has a fairly mature flow for adding ITSIO sidecars to running pods. I am more on the ops side but it seems like this workflow could lend itself to something like Dev Spaces.
I'd like to add that it is highly important for those of us in a highly regulated environments to be using the fewest amount of security products which accomplish our goal. Decreasing complexity is a cornerstone of properly deployed access controls. ITSIO is already in place in our environments and we are looking closely at azure adoption. Being able to incorporate our current production and staging ITSIO rules to our development space would significantly increase our adoption rate on the azure services side.
Love what you're doing with Dev Spaces to enable developers, keep it up.

Not so easy IMHO :
https://github.com/MicrosoftDocs/azure-docs/issues/33166

Not so easy IMHO :
MicrosoftDocs/azure-docs#33166

Really sheds some light on it. I still want the feature though.

I've been doing alot of homework on how to handle a dev cluster vs a QA or Prod cluster configuration and how to handle deployments.  The one thing here that I think allows me to work around this limitation is the use of the K8s admission controller.  So by not having Istio in dev is fine for us.  But in the QA & Prod environments we're looking to utilize the admissions controller to add configuration that may not have been there in the dev environment to account for those environmental differences.
Microsoft, you should probably give some guidance on this aspect for those that need Istio.


Blacklisting channels for commands.

I saw some people mention in chat of removing the ability to run anywhere else but command channels, I think this pull request solves that, idk I'm kinda a noob, so if you see anything wrong just point it out/fix it.

^^

discord snowflakes are strings smh

twitter snowflakes

@DIBSTERYT fix this

https://github.com/DanBot-Hosting/DanBotHostingStats/blob/master/bot/discord/events/message.js#L107


Update style-resources-loader to 1.2.1

What problem does this feature solve?
The creator of style-resources-loader added support for .CSS files (for PostCSS variables/mixins/custom media/etc. importing):
https://github.com/yenshih/style-resources-loader/issues/7
Please update the version in Nuxt so that we can reference PostCSS .css files in the styleResources block of build configuration as outlined here:
https://nuxtjs.org/api/configuration-build/
What does the proposed changes look like?
package.json
"style-resources-loader": "^1.2.1",

This feature request is available on Nuxt community (#c7576)

@nathanchase Happy to see a PR :wink:

@manniL Done!
https://github.com/nuxt/nuxt.js/pull/3738
My first pull request, actually. Hope I did it right!

You rock @nathanchase !


chore(dep) Upgrade to latest Ergo v0.22.0-alpha.3

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>Changes

Upgrade to latest Ergo 0.22.0-alpha.3
Use Logic Manager when adding model files

Coverage remained the same at 92.683% when pulling 0e40efc8f795bdd8530abd85bf9ef22a56c4aa1c on js-alpha.3 into b76b79d6611df1ae8f6e24da4f01c3e9cd9ce6f9 on master.


Also test against PHP 7.2

Context
PHP 7.2 has been released.
What has been done

Updated the travis.yml file

How to test

Let travis do the job

Hey @tiagobrito thanks for the contribution!
It's not really necessary to test the lower reqs for 7.2, if we find the lowest compatible version of each library using 7.2 we are gonna limit it too much and not gain a lot from it! I do believe people using 7.2 has updated libraries.
We can introduce the lowest 7.2 during the next minor/major!

@arubacao You should rebase this with master, the php cs fixer is already working there.
Thanks for your contribution.

@renatomefi @tiagobrito done :)

Thanks so much @arubacao


OS X 10.8: Missing third party modules - iso8601 and sqlalchemy

OS X 10.8.2 does not include the python modules iso8601 and sqlalchemy by default. Consider adding checks for third party modules.
$ ./glacier.py vault list
Traceback (most recent call last):
  File "./glacier.py", line 36, in <module>
    import iso8601
ImportError: No module named iso8601

Mac users can run the following commands to fix these dependencies:
sudo easy_install iso8601
sudo easy_install sqlalchemy

@amedee solution fixed it for our ubuntu box


Parallelize multiple OpenAI requests to combat discrepancies with completions (even with temperature=0)

Large language model outputs are not deterministic, even when setting the temperature to 0. Therefore, the model may output different SQL queries given the same natural language queries. Some of these SQL queries may be incorrect (either not what the user is asking for, or straight up triggers an Exception).
We have retries (see function text_to_sql_with_retry in api/app/api/utils.py) to regenerate SQL queries until the language model produces a query that doesn't trigger an exception, but this results in higher latencies (not by a ton, but enough to be noticeable). One thing we could do is parallelize K requests to OpenAI for generating SQL, then have K results to iterate through right off the bat.

This would be game changing! thanks for working on this @alex-sun-kuo
Would we also check the K completions for exceptions and retry them? What's your thinking around that?
I think the current exception -> retry is powerful, especially with combatting column name hallucinations etc.

@asaprahul i would parallelize the first K attempts as a batch. so for example, run 3 completion requests in parallel. iterate through them, trying to execute them. the first generated query in this initial batch that executes without failure, return SQL query and results. if all K fail, take the first query that failed and its error message, and proceed with the retry logic on that one (not on all K)

solid. excited to try this out!

https://github.com/caesarHQ/textSQL/pull/41 done


Fisher 4.x

Fisher >=4.0 uses install now.

I assume it is backwards compatible?

@franciscolourenco add still works, but shows a deprecation notice. Or do you mean something else?

I meant – does install already work in fish v3?

@franciscolourenco No, it's fisher install now.

So the new instructions are incompatible with fisher 3? Will fisher update automatically?

So the new instructions are incompatible with fisher 3?

Yes.

Will fisher update automatically?

No, existing users will upgrade to 4.x by running fisher self-update.
Related: Fisher 4 Release Notes

@franciscolourenco Do you need anything else here?

No was just concerned that users with fisher v3 won't be able to install.
Thank you for the pull request! ❤

Since Fisher will print an error, I think users will probably come knocking at my door first (if they can't figure out how to upgrade). We'll see. Thanks!


Add Pytorch MNIST Example

Adds a working pytorch MNIST example, following the new spec.
I0425 17:19:27.115946<PHONE_NUMBER>10080 submission_runner.py:220] Final mnist_pytorch score: 9.956104

Since pytorch uses classes to store state, whereas jax is more functional, the key change to the spec.py needed is to extend ParameterTree type to include torch.nn.Module. We will need to resolve a better naming for this type, as it is no longer a Tree.
The parameter names should also change in some of the spec functions. For example, in update_params, the name current_params is no longer accurate... since this is a pytorch module, we can call its functions such as:

current_params(input) - run forward pass
current_params.eval() - switch to eval mode

One proposal (unimplemented) to fix this is to rename ParameterTree -> Model, and current_params -> current_model.
A few TODOs to resolve with JAX (cc: @znado )

[x] match model architecture in the JAX implementation
[ ] when tensor/model loading to device occurs
[x] create shared workload class between JAX/PyTorch for common properties
[x] resolve RNG between JAX/PyTorch
[x] standardize JAX/Pytorch on a single eval_model function

OK great, I think this PR is ready to merge then, and we will have some follow-on work to standardize:

device loading
naming convention
rng safety

@mikerabbat : Yeah, might be something worth talking about. In our spec.py, the ModelState and Paramsare almost always passed together, the only exception beingdata_selection`:
https://github.com/mlcommons/algorithmic-efficiency/blob/e4406e574f4372a5025a9cc4e853d675fc4334de/spec.py#L274-L281
So we might have an opportunity for simplification. However, a clear separation between the "parameter" container, and "other auxiliary info" might be more natural to JAX users (?) than a tuple.

Good point @hanlint about model_state and current_params almost always being passed together. If the motivation for passing current_params to data_selection, along with workload, is to allow the data selection function to potentially call model_fn on data to evaluate the loss or something else, then we probably ought to pass model_state there too. I'll create a separate issue for this.


nixos/tests/atd: wait for atd to start

Motivation for this change
The test failed non-deterministically on Hydra when an at command was issued before the atd daemon was running.
ZHF #45960, please backport.
Things done

[x] NixOS test passes locally for x86_64-linux and i686-linux

@GrahamcOfBorg test atd

Success on x86_64-linux (full log)
Attempted: tests.atd
Partial log (click to expand)
machine: exit status 1
syncing
machine: running command: sync
machine: exit status 0
test script finished in 108.13s
cleaning up
killing machine (pid 597)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/g17v5ylnmgamn70pwaxmajfhyr04w17b-vm-test-run-atd

Success on aarch64-linux (full log)
Attempted: tests.atd
Partial log (click to expand)
machine: exit status 1
syncing
machine: running command: sync
machine: exit status 0
test script finished in 109.47s
cleaning up
killing machine (pid 631)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/lz1z98vg50wfdk8xl3jlj8f1fmravbqf-vm-test-run-atd

Backported in 711ff4b6877560b92d1fa503594a3c463526c763


Update SDK version to 1.19.1

What does this PR do?
This change updates SDK version to 1.19.1 after its release, and also updates plugin version for the sample apps to 1.9.0.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #171 (46c55bf) into develop (1168e62) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #171   +/-   ##
==========================================
  Coverage      86.63%   86.63%
  Complexity       177      177
==========================================
  Files             21       21
  Lines            576      576
  Branches          72       72
==========================================
  Hits             499      499
  Misses            49       49
  Partials          28       28


add SNLI model

there are serval changes in this pull request,
1, clean up some code
2, modify some modules, including lstm, linear
for example, the forward function of encoder.lstm was changed.
before:
    def forward(self, x):
        x, _ = self.lstm(x)
        return x

now:
    def forward(self, x, h0=None, c0=None):
        if h0 is not None and c0 is not None:
            x, (ht, ct) = self.lstm(x, (h0, c0))
        else:
            x, (ht, ct) = self.lstm(x)
        if self.get_hidden:
            return x, (ht, ct)
        else:
            return x

after modifying this, the former usage is also compatible.

fix some mislead comments in embed loader and rename some variables

add SNLI model

modify the index_field function in Vocabulary
before:

    def index_field(self, field_name, vocab):
        for ins in self:
            ins.index_field(field_name, vocab)
        return self

now:
    def index_field(self, field_name, vocab):
        if isinstance(field_name, str) and isinstance(vocab, Vocabulary):
            field_list = [field_name]
            vocab_list = [vocab]
        else:
            classes = (list, tuple)
            assert isinstance(field_name, classes) and isinstance(vocab, classes) and len(field_name) == len(vocab)
            field_list = field_name
            vocab_list = vocab
         for name, vocabs in zip(field_list, vocab_list):
            for ins in self:
                ins.index_field(name, vocabs)
        return self

if we need to index serval fields, we can pass two list to this function instead of calling this function serval times. meanwhile, the former usage that passing both a string and a Vocabulary is also compatible.

Codecov Report

Merging #99 into master will decrease coverage by 2.85%.
The diff coverage is 20.32%.

@@            Coverage Diff            @@
##           master     #99      +/-   ##
=========================================
- Coverage   78.56%   75.7%   -2.86%
=========================================
  Files          72      73       +1
  Lines        3467    3627     +160
=========================================
+ Hits         2724    2746      +22
- Misses        743     881     +138

Impacted Files
Coverage Δ

fastNLP/models/snli.py
0% <0%> (ø)

fastNLP/loader/dataset_loader.py
74.03% <10.52%> (-10.97%)
:arrow_down:

fastNLP/modules/encoder/linear.py
100% <100%> (ø)
:arrow_up:

fastNLP/modules/decoder/MLP.py
21.87% <25%> (ø)
:arrow_up:

fastNLP/core/metrics.py
73.86% <27.27%> (-3.11%)
:arrow_down:

fastNLP/core/vocabulary.py
84.09% <50%> (-0.8%)
:arrow_down:

fastNLP/core/dataset.py
87.14% <60%> (-4.8%)
:arrow_down:

fastNLP/core/tester.py
82.85% <60%> (-1.76%)
:arrow_down:

fastNLP/core/trainer.py
76.02% <62.5%> (-1.12%)
:arrow_down:

fastNLP/loader/embed_loader.py
79.48% <62.5%> (ø)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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

Many thanks 😃
For LSTM, the change is reasonable.
For index method, "pass two list to this function instead of calling this function serval times". Is this an expected behavior? @choosewhatulike

For index method, "pass two list to this function instead of calling this function serval times". Is this an expected behavior? @choosewhatulike

it's good to add new features. and it support old APIs

Many thanks 😃
For LSTM, the change is reasonable.
For index method, "pass two list to this function instead of calling this function serval times". Is this an expected behavior? @choosewhatulike
Everything goes fine. Ready to merge.

if we need to index several fields at the same time, we can call the index function once, which may reduce the lines of code and seems more clear.


Heat time doesn't works

I have an issue with the heat time... if i set it to 10 it doesn't print almost anything, if I set to 240 I'm getting the same results like I set 100...
I'm using this tiny printer from adafruit (https://www.adafruit.com/products/2751) so I don't know if I need to set different parameters...the model used for other raspberryPi project is Zijiang ZJ-58...hope you can help me solving this!
Thank you

Since that's a different model, it's quite possible that the initialisation codes required to set up heat time are different. It would be worth checking that, by comparing the code tables, but I can't tell from the Adafruit site what the actual make & model is...


Fix asset compression target to support asset bundle attributes other than basePath, baseUrl, js, and css.

When I put a config in the asset compression target using

    'targets' => [
        'all' => [
            'class' => 'yii\web\AssetBundle',
            'basePath' => '@webroot/assets',
            'baseUrl' => '@web/assets',
            'js' => 'js/all-{hash}.js',
            'css' => 'css/all-{hash}.css',
            'jsOptions' => ['some', 'options'],
        ],
    ],

or create a new custom AssetBundle with custom attribute for production.
class \my\custom\AssetBundle {
    public $foo;
}

all of the attributes except baseUrl, basePath, js, and css got disappear.
Wondering whether this is intended.

Resolved by commit c9fc60fe741aad2b9b67122df6dbd6426df511ba


Expose Http.Sys's Http/2 Reset error code API

Http.Sys always uses the Http/2 error code Cancel when you abort a request.
B) gRPC wants to report NO_ERROR.
A new Http.Sys API is being added to allow changing the error code. (What is it?).

@JamesNK


There are some wrong in README.md

wrong: mongoose_types.insert_many([{'name': 'Meercat', 'not_into', 'Snakes'},{'name': 'Yellow mongoose': 'eats': 'Termites'}])
desc: the argument is not a correct python dict
wrong: list(coll.find({'weight': {'$gt': 1}))
coll.delete_one({'name': 'Meercat'})
desc: the variable coll is not existed, should be mongoose_types

Woops. Thank you. Will fix tomorrow.

Fixed. Thanks again.


[FEATURE] Add multiple improvements

- Feature 1: Custom headers
I want this feature to enable custom headers. Please see the Cloudflare blog post here. Links for testing and example is given below.
- Feature 2: Early hints and asset preloading
I want this feature to speed page loading by preloading assets. More information is here or here.
- Feature 3: SEO
I want this feature to improve SEO by adding relevant meta tags in the code. More information is here.
Useful reference
https://blog.cloudflare.com/custom-headers-for-pages
https://securityheaders.com for testing (example: https://securityheaders.com/?q=https%3A%2F%2Fhitrobotics.com%2F)
https://blog.cloudflare.com/early-hints
https://mathieuhendey.com/2020/10/bundling-and-preloading-hugo-assets
https://mathieuhendey.com/2020/10/seo-for-hugo-blogs/

For Feature 1: https://gohugo.io/news/0.67.0-relnotes/

For Feature 2, I have created an issue to track its progress #323
For Feature 3, DoIt has already optimized for SEO. See
https://github.com/HEIGE-PCloud/DoIt/blob/main/layouts/partials/head/seo.html
https://github.com/HEIGE-PCloud/DoIt/blob/main/layouts/partials/head/meta.html

Thank you both for your replies and links!


BatchMessage

In one of the initial designs of the protocol, we conceived the idea of BatchMessage, mostly to improve protocol efficiency, but then after a while we just ignored it as it didn't seem particularly interesting as optimization.
@StephanEwen recently suggested that a BatchMessage could in fact make sense, only in the response stream (from service endpoint to runtime), not for optimization purposes, but to provide a way to either commit all entries or no entries. This can be an interesting property for SDKs that wants to provide a different programming model, e.g. actor based models where there are no "await" points, where the side effect don't need any wrapping within a ctx.sideEffect or similar.
An interesting aspect of this message is that this would be an optional contract of the protocol, as only SDKs that wants to push this message to the runtime needs to implement it.

This task is not really actionable right now, will close this.


[Bug][Devlake] Can't edit scope config (TypeError: Cannot read properties of null (reading 'map'))

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I can't edit a scope config, I get an error

Must be a frontend thing, can't seem to find anything in the error logs.
What do you expect to happen
Window for changing scope config should open.
How to reproduce
--> Projects
--> Detail of your project (gear icon)
--> Edit Data Scope and Scope Config
--> Button 'Edit Scope Config'
--> Select your scope config and click button 'Associate Scope Config'
--> Select your scope config and click the edit icon (pencil)
Error is displayed :
TypeError: Cannot read properties of null (reading 'map')

Anything else
Every time.
Version
v0.18-beta8
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Hi @KlemenDanfoss, thank you for your report.
Could you please provide a screenshot of the browser console?

@mintsweet , sadly nothing more than this

@KlemenDanfoss Thank you, can I ask what plugin it is?

@KlemenDanfoss Thank you, can I ask what plugin it is?

Azure devops

@KlemenDanfoss Thank you, I will try to reproduce it today.

Seeing the same error with gitlab.

Seeing the same issue with GitHub after upgrade to 0.18.0, wasn't doing it prior; Jira show's my scope config from prior but it doesn't appear in the list to edit (maybe unrelated).

I don't see the problem with v0.19.0-beta2

I replied earlier that I couldn't see the problem on 0.19.0-beat2 but I do see it.  I have to revert back to version 0.17 because I need access to that screen.

Hi @mintsweet, do you need more information to resolve this issue?  Is there any workaround we can use in the mean time?
I tried 0.19.0-beta3 and it still has the problem.
Thanks.

Hi @MarcViola, can you provide the last api screenshot about this error?
You can find it in brower devtool network. Thank you.

OK, @MarcViola, thank you for your information,  I'll try to reproduce it today.

Hi, @MarcViola it's a pity that I tried to reproduce it again today, but it still failed. If possible, please provide a screen recording to help me troubleshoot it. Thank you

@mintsweet Unfortunately I'm not allowed to record the screen.

Hi, @MarcViola Can you take a screenshot of the response of the offending scopeConfig? like:

@KlemenDanfoss @MarcViola Fixed and released in beta6(https://github.com/apache/incubator-devlake/releases/tag/v0.19.0-beta6)


80 chars per line formatting can be broken by formatting tags

I've found that the 80 char per line limit can be broken by inline formatting.  Is there a recommended way to get this to behave itself or is this a bug?

  This text isn't counted when calculating where to break a string for 80 character line lengths.

I've added a fix for this to my PR, #83

Long words, such as urls, can also go over 80 char.  Any preferences on how to handle these?

Updated PR #83 to handle wrapping long words.


Set the URL to the active sidebar item

Currently, the IntersectionObserver sets the URL. The URL is then set to the section that’s becoming visible, not to the active item (which is the first item in the sidebar, where the section visible).

@hanspagel  Please assign it to me, I'll pick this up.

Done @Mohib834 , thanks for wanting to tackle this ✨

I am able to pinpoint the issue but couldn't figure out the full solution yet.
basically, it is happening because the intersection observer is getting triggered as soon as the next section enters the view triggering the active state of the next element on the sidebar and removing it from the current one.
I am able to adjust the threshold depending on the height of each section. So this is somewhat fixed.
onMounted(async () => {
  // Setting different threshold depending on the height of element.
  await nextTick()
  const _threshold =
    (intersectionObserverRef.value?.offsetHeight ?? 0) >= window.innerHeight
      ? 0.3
      : 0.8

  useIntersectionObserver(
    intersectionObserverRef,
    ([{ isIntersecting }]) => {
      if (!props.id) {
        return
      }

      setSidebarIdVisibility(props.id, isIntersecting)

      if (isIntersecting) {
        emit('intersecting')
        const newUrl = `${window.location.origin}${window.location.pathname}#${props.id}`
        window.history.replaceState({}, '', newUrl)
      }
    },
    {
      rootMargin: `0px 0px -20% 0px`,
      threshold: _threshold,
    },
  )
})

But things breaks when there are two visible sections with very small height comes into the view, when this happens the intersection observer first trigger for the first section and then for the other modifiying the state of active sidebar element and also modifying the url.
Would love to have some help on this please @hanspagel @marclave

Good find! I’m about to do a bigger refactoring and introduce a useNavigation composable and I think this would solve the issue, too. Would you mind if I take from here?

Good find! I’m about to do a bigger refactoring and introduce a useNavigation composable and I think this would solve the issue, too. Would you mind if I take it from here?

Sure, Thanks.


Provide ARM64 release binaries

It would be great to provide ARM64 release binaries (for both Linux and macOS). I noted the current build.py script doesn't account for processor architecture, but a manual build of an ARM64 Darwin/macOS binary from a AMD64-based Linux host using Go 1.17 did not produce any errors. The resulting binary seemed to work as expected, but I did not perform any extensive testing.

Kubescape images are built with arm support

For linux, the multi-architecture (like aarch64, s390x, i586, armv7l, ppc64le etc.) is now already available with package managers:

https://github.com/kubescape/kubescape/issues/1143
https://github.com/kubescape/kubescape/issues/1142

I will work for a solution on the arm64 for macOS later.


[iframe] Video always autoplay. How to disable autoplay?

I swear that I have completed these tasks before submitting:

[ x ] I have read the README
[ x ] I have checked that the issue has not been reported yet

Bug Report
Environment

React Native

System:
OS: macOS 11.2.1
CPU: (8) x64 Intel(R) Core(TM) i5-1038NG7 CPU @ 2.00GHz
Memory: 127.68 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
npmPackages:
react: 16.13.1 => 16.13.1
react-native: 0.63.3 => 0.63.3

Libraries

react-native-render-html: 6.0.0-alpha.17
@native-html/iframe-plugin: ^2.1.0
react-native-webview: ^11.2.3

Description
When render iframe The Vdo is auto play. How to disable autoplay video?
This is my code

const IframeRenderer = (props) => {
    const iframeProps = useHtmlIframeProps(props);
    // Do customize the props here; wrap with your own container...
    return iframeProps ? <HTMLIframe {...iframeProps} /> : null;
};

IframeRenderer.model = iframeModel;

const renderers = {
    iframe: IframeRenderer
}

const html = ''
return <RenderHTML
                            renderers={renderers}
                            WebView={WebView}
                            renderersProps={{
                                iframe: {
                                    scalesPageToFit: true,
                                    autoplay: false,
                                    webViewProps: {
                                        autoplay: false,
                                        mediaPlaybackRequiresUserAction: true
                                    }
                                }
                            }}
                            source={{html: html}}
                            contentWidth={width}/>

@luckybelle The issue you are reporting is not inherent to the Iframe plugin, but rather to using a WebView component with a src referring to a video file instead of HTML file.
You can reproduce the issue very easily with react-native-webview alone:
<WebView
  mediaPlaybackRequiresUserAction
  source={{
    uri:
      "https://storage.googleapis.com/demo-coj/exhibition/17bc59130550baf741080ddaf18e8154.mp4",
  }}
/>;

I suggest either:

Report a bug in react-native-webview issue tracker
Create a custom iframe renderer which conditionally uses react-native-video library to render when src is a media URL. Below is a quickstarter sample:

import * as React from 'react';
import WebViewIframeRenderer, {iframeModel} from '@native-html/iframe-plugin';

const IframeRenderer = function (props) {
  const src = props.tnode.attributes.src || '';
  if (src.match(/\.mp4$/)) {
    return /* An element based on react-native-video */;
  }
  // Defaults to the WebView-based iframe renderer
  return <WebViewIframeRenderer {...props} />;
};

IframeRenderer.model = iframeModel;

And of course, you'll nee to add it to renderers.iframe prop.


Bug report, Segmentation fault

In slate/tester.cpp "strcpy(mem, "trombone\n");" cause segmentation fault since char* men isn't initialized. Please fix it.

The reason may be different from what I thought before. It seems that *pointer = 0xffffffffffffffff after code "*pointer = shmat(buf_id, 0, 0);" in slateMalloc function. Do you know the reason?

It is because the key of shmget  equals 0, which is read from din.

I find the accurate reason. The type of variable size in SlateClient::do_malloc() is int, if you change it to size_t, the bug will be solved.


Empty Persistent Loot Window Issue

At times there's an empty persistent loot window, unknown cause.
Seems to happen more frequently when using the Mouse-wheel-loot method that a lot of farmers / boosters have adopted. Or maybe I just loot more items and have seen it more that way.
Only ever saw it when in a group.

Seen the same thing, I believe it is related to looting while in combat.

happens mostly in elv ui


Added support for allow_nulls option for boolean column.

The motivation behind this pull-request is that the default behaviour of the gem with boolean columns is not very logical. Entities which have attribute deleted == false are currently considered deleted (because false is not nil).
When using boolean column type it is now possible to pass allow_nulls options which is true by default. When set to false we check for false values in deleted column instead of nils when checking if the entities are deleted or not.
Thus it is possible to define deleted column as:
t.boolean :deleted, :boolean, null: false, default: false

And then in the ActiveRecord model:
acts_as_paranoid column: 'deleted', column_type: 'boolean', allow_nulls: false

The default behaviour is preserved and is backwards-compatible.
This new case is fully covered with tests.

+1

@Nu-hin Could you rebase? <3

Rebased.

@Nu-hin Thank you!!


Qcfe/instance customization

Description
This PR fixes ingresses name generation for environments with non-default mode.

/deploy-staging

/merge


Version 5.5 bug

Hi,
I changed my rxjs version to 5.4.1 because 5.5.0 doesn't work.
When I tried to run the app this problems are display to me:

And bug any route too

My systemjs.config.js
`(function(global) {
// map tells the System loader where to look for things
var map = {
'@angular':   'node_modules/@angular',
'moment':     'node_modules/moment',
'rxjs':       'node_modules/rxjs'
};
// packages tells the System loader how to load when no filename and/or no extension
var packages = {
'moment':     { main: 'moment.js', defaultExtension: 'js' }
};
var ngPackageNames = [
'common',
'compiler',
'core',
'forms',
'http',
'platform-browser',
'platform-browser-dynamic',
'router',
'router-deprecated',
'testing',
'upgrade'
];
// Individual files (~300 requests):
function packIndex(pkgName) {
packages['@angular/'+pkgName] = { main: 'index.js', defaultExtension: 'js' };
}
// Bundled (~40 requests):
function packUmd(pkgName) {
packages['@angular/'+pkgName] = { main: 'bundles/' + pkgName + '.umd.js', defaultExtension: 'js' };
}
// Most environments should use UMD; some (Karma) need the individual index files
var setPackageConfig = System.packageWithIndex ? packIndex : packUmd;
// Add package entries for angular packages
ngPackageNames.forEach(setPackageConfig);
var config = {
defaultJSExtensions: true,
map: map,
packages: packages
}
// filterSystemConfig - index.html's chance to modify config before we register it.
if (global.filterSystemConfig) { global.filterSystemConfig(config); }
System.config(config);
})(this);
`
Do you know what can I do to fix this?

I am also running into problems with 5.5.0. It seems it no longer patches Observable with the loaded operators. Is there a way to patch them or are you required to use the "::" operator?

Check closed issues. Already solved and there will be fix in new version.

Thank you. I am currently using a workaround
// Setup what we need of the rxjs system.
import { Observable } from 'rxjs/Observable';
import { mergeMap } from 'rxjs/operator/mergeMap';
import { take } from 'rxjs/operator/take';
import { map } from 'rxjs/operator/map';
import { mapTo } from 'rxjs/operator/mapTo';
import { takeUntil } from 'rxjs/operator/takeUntil';
import { race } from 'rxjs/operator/race';
import { filter } from 'rxjs/operator/filter';
import { of } from 'rxjs/observable/of';
import { race as Srace } from 'rxjs/observable/of';

Object.assign(Observable.prototype, { mergeMap, take, map, mapTo, mergeMap, flatMap: mergeMap, takeUntil, race, filter });
Object.assign(Observable, { race: Srace, of });

When will the release be up?

BTW: The import using @reactivex/rxjs vs rxjs is going to be a problem until redux-observable uses the new import. Or is there a standard way to alias modules that way?

closing as v6 changed most of import ergonomics.


How to update bundler itself to version in gemlock

This is for our CI system.  They have a bundler version preinstalled that is slightly older than the one we used to create our Gemfile.lock.  I'd like to add a command to update the bundler version to the one after "Bundled With" at the bottom of the Gemlock.  I scanned the docs and web and didn't find anything.  Is this easily achievable?

You can use the postit  gem or the BUNDLE_ENABLE_TRAMPOLINE environment variable (https://github.com/bundler/bundler/blob/master/lib/bundler/postit_trampoline.rb#L15)

Thanks @segiddins !


Adplugin docs update

I have added sections on creating custom AD device and plugin classes. This is based on my experience writing a new plugin class and reading through the docs/ code. There may be things I have wrong please check that it makes sense prior to merging.

Travis failures are unrelated caproto issues, so I think we are good.

Thanks for the comments @danielballan I will look at them tomorrow. I am still learning a lot of the doc syntax, so thanks for the pointers.

I am happy to rip that last commit off....

Thanks @awalter-bnl and @tacaswell for the updates. Looks good to me!

Before we merge, are failing TravisCI builds considered fine? I'm seeing E       ValueError: invalid literal for int() with base 10: '/data/2018/09/04/'...

Yeah, that's the caproto issue that is showing up in the ophyd tests. I aim to fix it, but first I need to get the caproto CI straightened out.

Is this good to go, @tacaswell. The last comment ("thrashy thrashing...") prompts me to check that the thrashing is complete. :- D

Thanks @danielballan for the clarification. Merging.

gah, I meant to tag after this was merged.  Oh well.

Oh... Re-tag maybe?

It is a document update, not sure it warrants a re-tag just for this.

Oh... Re-tag maybe?

Since this is kind of orthogonal to the content of the release, just a general improvement, I don't think we need to worry about making a new tag. We always publish the master branch docs.

OK, sounds good.


Discovery Theater Brazil Missing

Source
https://mi.tv/br
Channels
The Discovery Theater channel disappear from EPG
Name in previous versions of xml: DiscoveryTheaterHDLatinAmerica.us
link in mi.tv website: https://mi.tv/br/canais/discovery-theater-hd
Notes
No response

Fixed in https://github.com/iptv-org/epg/commit/42153d6b457db70faef5bd943f157a495839486c


Cannot infer value type

Latest Vscode, Latest Master
---@generic V: string
---@param list V[]
---@return {[V]: true}
util.list_to_map = function(list)
  local map = {}
  for k, value in pairs(list) do
    map[value] = true
  end
  return map
end

bug.log

This is caused by the current generic implementation defect, and the generic implementation needs to be refactored in the future.


Provide fallback for docs version if cookies disabled

[description]

provide fallback if cookies disabled
check results array length to detect if no results

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/c90eb877ac5ff0584e6f2254b1d1de5f8df2f858/


(BOLT-982) Create plan executor

CLA signed by all contributors.

Looks good. Going to squash and merge when it passes CI.


fix: ghcr publish bug

Rationale
close #1393
Detailed Changes
Test Plan

I'm not sure if it can run successfully. @chunshao90

https://github.com/apache/incubator-horaedb/pull/1396


fix(versions): update Activiti/activiti-cloud-dependencies versions

UpdateBot pushed maven dependency: org.activiti.cloud.acc:activiti-cloud-acceptance-tests-dependencies to: 7.0.8

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.acc:activiti-cloud-acceptance-tests-dependencies 7.0.8

/test this
/approve

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-x-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies 7.0.51

/test this
/approve

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.query:activiti-cloud-query-dependencies 7.0.52

/test this
/approve

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.query:activiti-cloud-query-dependencies 7.0.53

/test this
/approve

/test this
/approve

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies 7.0.52

/test this
/approve

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies 7.0.53


Remove redundant API method

While attempting to document the code, found an API call that was unnecessary.

listNonPythonKernels is not required
We list non-python kernels when fetching all kernels (including Python) in the listKernels method

I think we used this code path in the old code and is now redundant.
As Peng mentioned, writing the documentation helps identify issues & I think its already proved beneficial here (I think).

Codecov Report

Merging #8747 (f3f9685) into main (d712c97) will decrease coverage by 0%.
The diff coverage is 57%.

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

@@          Coverage Diff          @@
##            main   #8747   +/-   ##
=====================================
- Coverage     71%     71%   -1%
=====================================
  Files        381     381
  Lines      24535   24533    -2
  Branches    3777    3777
=====================================
- Hits       17654   17626   -28
- Misses      5361    5380   +19
- Partials    1520    1527    +7

Impacted Files
Coverage Δ

...t/datascience/kernel-launcher/localKernelFinder.ts
81% <ø> (-1%)
:arrow_down:

src/client/datascience/kernel-launcher/types.ts
100% <ø> (ø)

.../datascience/notebook/notebookControllerManager.ts
79% <57%> (ø)

...ience/variablesView/variableViewMessageListener.ts
77% <0%> (-23%)
:arrow_down:

...ent/common/application/webviewViews/webviewView.ts
67% <0%> (-11%)
:arrow_down:

src/client/datascience/dataScienceSurveyBanner.ts
69% <0%> (-7%)
:arrow_down:

.../client/datascience/debugLocationTrackerFactory.ts
92% <0%> (-4%)
:arrow_down:

...lient/datascience/variablesView/notebookWatcher.ts
90% <0%> (-3%)
:arrow_down:

src/client/common/application/webviews/webview.ts
73% <0%> (-2%)
:arrow_down:

...atascience/interactive-window/interactiveWindow.ts
67% <0%> (-2%)
:arrow_down:

... and 2 more


[Event Request] Codeunit 22 Item Jnl.-Post Line - OnUpdateOrigAppliedFromEntryInRepeat

Hi, would it be possible to get an event OnUpdateOrigAppliedFromEntryInRepeat in codeunit 22, procedure UpdateOrigAppliedFromEntry, like this :
    local procedure UpdateOrigAppliedFromEntry(OldItemLedgEntryNo: Integer)
    var
        ItemApplEntry: Record "Item Application Entry";
        ItemLedgEntry: Record "Item Ledger Entry";
    begin
        ItemApplEntry.SetCurrentKey("Outbound Item Entry No.", "Item Ledger Entry No.");
        ItemApplEntry.SetRange("Outbound Item Entry No.", OldItemLedgEntryNo);
        ItemApplEntry.SetFilter("Item Ledger Entry No.", '<>%1', OldItemLedgEntryNo);
        if ItemApplEntry.FindSet then
            repeat
                if ItemLedgEntry.Get(ItemApplEntry."Inbound Item Entry No.") and
                   not ItemLedgEntry."Applied Entry to Adjust"
                then begin
                    ItemLedgEntry."Applied Entry to Adjust" := true;
                    ItemLedgEntry.Modify();
                end;
                OnUpdateOrigAppliedFromEntryInRepeat(OldItemLedgEntryNo,ItemApplEntry);
            until ItemApplEntry.Next = 0;
    end;

    [IntegrationEvent(false, false)]
    local procedure OnSplitItemJnlLineOnBeforePostItemJnlLine( OldItemLedgEntryNo : Record "Item Ledger Entry" ; ItemApplEntry : Record "Item Application Entry")
    begin
    end;

?
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


grid-row mixin should accept gutter value

The grid-row mixin should accept a gutter value for nested behavior, as it can not be assumed that the default is used.

We'll implement this for the float grid as well as the flex grid.

Thanks!
On Thu, Dec 3, 2015 at 6:23 PM Geoff Kimball<EMAIL_ADDRESS>wrote:

We'll implement this for the float grid as well as the flex grid.
—
Reply to this email directly or view it on GitHub
https://github.com/zurb/foundation-sites/issues/7115#issuecomment-161821720
.


Can't import from CDN

Thanks for this library. I like it better than the more popular ones.
It would be nice if I could import it inside a <script type=module> tag without a bundler. Right now, when I try
import Between from 'https://rawgit.com/sasha240100/between.js/master/build/between.js'

it fails because there's no default module, and when I try something like
import * as Between from 'https://rawgit.com/sasha240100/between.js/master/build/between.js'

I get t is undefined. Can you release a build that's been updated to work with this syntax?

jsDelivr can pack modules for you, so
import Between from 'https://cdn.jsdelivr.net/npm/between.js/+esm'

works.


2 general questions

Hello!
In my quest to research how effective emacs can be as a c++ ide, I have come across this repo. I am working in an environment where clang is not available, so my first question is whether there is any problem or limitation with using GCC/G++ instead of clang?
My next question is about the architecture of cmake-ide: how does this setup compare or overlap with the functionality outlined in these many videos about using emacs for c++:
https://www.youtube.com/watch?v=HTUE03LnaXA
The packages used in those videos are: http://barisyuksel.com/cppmode/package_versions.png
There are so many packages out there and I'm not an emacs expert, but I do know that many different packages tend to compete with each other for functionality. Does the cmake-ide solution bundle together the functionality mentioned in these other packages/videos?
Many thanks!

You can compile with gcc, but some packages rely on clang and there's nothing I can do about that (flycheck, auto-complete-clang) Try and see, I guess.
As for other options, I only wrote my own because I thought the others were worse. If you ask my opinion, I'm obviously going to tell you to use cmake-ide ;) The only way you can make a decision is to try them all, I'm afraid.


New release?

Any chance of a new release? It would be nice to have #45 in a published release :wink:

It's running! https://github.com/coursier/sbt-shading/actions/runs/8389207333

Nice, thanks!


Refracting Of Code And Deployement Of Website

Changes

readme.html put under website-src/src by the name of index.html
css of readme.html refactored to styles.css
deployed to vercel link

My general thanks to the creator of this index as it has helped numerous times
Final Changes

Finished the website which is easy to update which uses strapdown js
Updated all the information

https://github.com/Igglybuff/awesome-piracy/pull/524#issuecomment-848969254


[Bug] The tracing module of dubbo-samples cannot run with dubbo3.3.1

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache Dubbo Component
Java Samples (apache/dubbo-samples)
Dubbo Version
dubbo3.3.1 jdk 17 springboot 3.0.7
Steps to reproduce this issue
zipkin结合运行不起来
What you expected to happen
zipkin结合运行不起来
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[X] Yes I am willing to submit a pull request on my own!

Code of Conduct

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

Please resubmit in English only.

Do you have detailed error information and cases?


Upgrade CMakeLists.txt to use C++14

Failed with, looks like need to update NDK
stderr: g++: error: unrecognized command line option ‘-std=c++14’
    When running <c++ preprocess_and_compile>.
    When building rule //lib/fb:fbjni#compile-pic-assert.cpp.odfda776a,default.
 FAILED

Any thought on using https://github.com/react-native-community/docker-android. It's used by react-native, and has buck and NDK 17c.
I can work on a PR for this.

Why are you not using https://cmake.org/cmake/help/v3.1/prop_tgt/CXX_STANDARD.html ?

I have create the pr migrate to docker: https://github.com/facebook/yoga/pull/884.

Hello,
I submitted an alternative to this which uses CXX_STANDARD. It also only requires c++11 rather than c++14.
https://github.com/facebook/yoga/pull/887
(I wasn't aware of this PR before submitting my own, but now that both exist, it seemed good to mention the other one.)

Created this PR to solve Travis CI build failure which is fixed by other commit here
Closing this.


Toggle light/dark doesn't work with web server

Describe the bug
I have checked #16 and my baseUrl is fine :)
The toggle works when I run the blog with hugo server.
It doesn't when I generate the static files with hugo and run the site with an HTTP server (nginx or http-server node module).
I could reproduce with the example site.
My website is deployed here: https://colin.cafe/#
The source code is here: https://github.com/ColinMaudry/colin.cafe
hugo version
hugo v0.111.3-5d4eb5154e1fed125ca8e9b5a0315c4180dab192+extended linux/amd64 BuildDate=2023-03-12T11:40:50Z VendorInfo=snap:0.111.3

Steps to reproduce
cd colin.cafe
hugo
cd public
http-server

Expected behavior
When clicking on the sun icon the theme should switch to dark.
Device information (please complete the following information):

OS: Ubuntu 22.10
Browser

Firefox 111.0.1
Vivaldi 5.7.2921.65

Thanks for working on it ☕

Hi Colin. Thanks for supporting my work.
When I clone your repo locally, the problem vanishes (see the screen recording below).
But when I use, GitHub codespace to develop/test, the problem appears.
This is strange.
https://user-images.githubusercontent.com/122173059/228923610-6939c902-b867-48d4-a93b-54cb227ca9dd.mp4

Where is https://colin.cafe/ hosted?

Hi @ColinMaudry . It's a small mistake, but took a lot of time to debug. 😄
The problem is that [params] is missing in your config.toml.
This is your config.toml:
https://github.com/ColinMaudry/colin.cafe/blob/1bc8a7983040a3cc09c8ec2c0970135bb4daf429/config.toml#L17-L27
This is the config.toml from the theme's exampleSite:
https://github.com/hugo-sid/hugo-blog-awesome/blob/ce9f89bda93816cc284561c9e4a5e30d389b830a/exampleSite/config.toml#L34-L43

On a side note, I was wondering why in the above comment, the lines referenced from the theme's config.toml are displayed in a nice code snippet where as lines referenced from your repo are not.
I navigated to GitHub docs, where it is mentioned that:
"This type of permanent link will render as a code snippet only in the repository it originated in. In other repositories, the permalink code snippet will render as a URL."

I had the same issue. For me it was solved when I fixed the domain from:
baseURL = <final domain I planned to use>
to
baseURL = <the domain I am currently using for testing>

It did, thanks!


Add Enrich EF API

Fixes https://github.com/dotnet/aspire/issues/1403
Microsoft Reviewers: Open in CodeFlow

Add a playground sample or modify one with the new apai

@davidfowl example added in README and in the Postgres playground.

Did we update one of the playground samples?

@davidfowl yes


Snowflake Add-In Querying Problems

Hello,
I am having issues with the Snowflake Excel Add-In and was wondering if anyone has any idea what the problem is?
When querying Snowflake using the Excel Add-In I am having issues retrieving data. When I execute the SQL for a table/view it returns no data, no matter how long I wait. However if I then click the Connection icon twice, the data is then retrieved into Excel. If I then create a new sheet in the workbook and click Query, then the same table of data is retrieved automatically again. If I then change the SQL to execute against a different table/view, again no data is rertieved. If I click Connect the original set of data is retrieved and not the new table.

@MartinChisnall1964 Has this issue been fully resolved?

Yep all good now. Thanks for your help.


Upgrade Jörmungandr integration to 0.8.3 / 0.8.4 (when available)

Issue Number

N/A
Overview

[x] I have bumped nix revision to 0.8.3
[x] I have re-generated arbirtary binary blocks using 0.8.3
[x] I have re-generated the genesis file using<EMAIL_ADDRESS>(no change here)

Comments

Just to see if it builds. We expect 0.8.4 soon.

bors try


graphic novel transcriptions

a short term goal is to have searchable graphic novels, even if they are manually transcribed into a script that is just manually inserted/interleaved into the epub/html of each chapter.
a longer term goal would be to enable transcription by showing the chapter and letting users annotate the speech bubbles (with OCR being a low priority, since manual review/editing would probably be necessary either way)

Manually transcribed graphic novels that are bundled as an epub with a table of contents will be handled just fine by the book processor and search. We might need to introduce additional styles to the standard set in order to allow a more script-like appearance (one column dedicated to the character names and one for their text).

this may also assist interior art transcript and translation


[Papercut][Sw-14787] fix voucher select in cancel orders

In the cancelled order the voucher selection now shows % if percental and a symbol if absolute. The voucher Description is also added to tell which one is which.

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
https://issues.shopware.com/#/issues/SW-14787

How to test?
Reload the snippet database. Create a percental and an absolute voucher. Open the cancelled order window. Select an Element where no voucher is already send. Open the Customer feedback and select a voucher. (Must be tested for eng and ger because of the snippets)

Hey @TobiasWilming,
thanks for your contribution. I checked your PR and having a tip for you. You're using the default syntax of Ext.String.format. Due to the fact that our Javascript files in the backend are parsed by Smarty and Smarty's delimiters are { & }. We created an override fowhich enables you to use [0] instead of {0} for variable placeholders, so you don't need the {literal} tags in your snippets.
Take a look at the override:
https://github.com/shopware/shopware/blob/5.2/engine/Library/ExtJs/overrides/Ext.Template.js
Here's a working example:
https://github.com/shopware/shopware/blob/5.2/themes/Backend/ExtJs/backend/customer/view/detail/window.js#L202
Your branch is out-of date with the base branch. Would you be so kind and rebase it please?


fixes 'Cannot set property '_renderItem' of undefined' when adding a nested resource

Fixes #2722

Spec is failing, can you take a look?

I'm looking into, it looks like it fails when using Capybara >=2.10.0. I haven't used Capybara much so it's taking me a little while trying to figure out what in that update is causing the test to no longer pass

I think the test that is failing is a bum test
From what I can tell, the block for have_selector was not getting run prior to Capybara v2.1.0 which was released about a week ago (2016-10-05) since that functionality wasn't implemented until that version (https://github.com/jnicklas/capybara/commit/b5655e9b9bde69aa9fba19902334b76c99f62148#diff-5b48908a3d2bd3374aaef7b6394f1912R39)
# https://github.com/sferik/rails_admin/blob/master/spec/integration/basic/edit/rails_admin_basic_edit_spec.rb#L76
describe 'edit with has-and-belongs-to-many association' do
    before do
      @teams = FactoryGirl.create_list(:team, 3)
      @fan = FactoryGirl.create :fan, teams: [@teams[0]]
      visit edit_path(model_name: 'fan', id: @fan.id)
    end

    it 'shows associated objects' do
      is_expected.to have_selector '#fan_team_ids' do |select|
        # This block doesn't seem to run prior to Capybara v2.1.0
        expect(select[0]).to have_selector 'option[selected="selected"]'
        expect(select[1]).not_to have_selector 'option[selected="selected"]'
        expect(select[2]).not_to have_selector 'option[selected="selected"]'
      end
    end
end

@tmesquita thank you! :+1:

Awesome, thanks!


Use custom tracker servers

Hi, I wanted to know if there is a way to use custom tracker server.
I did not manage to find info in Docs.
Thanks in advance.

When adding a torrent, you can specify an array of trackers using announce
https://github.com/feross/webtorrent/blob/master/docs/api.md#clientaddtorrentid-opts-function-ontorrent-torrent-

As of 2016_06_10 I do not know anything about WebTorrent. I'm just trying it out myself as an ordinary user, but it certainly failed my tests. Namely, I like to share VirtualBox virtual machine images as bit-torrents, because then people can get an environment that is already set up, where the Linux dependency hell is, not solved, but mitigated to the point that symptoms do not immediately show and people can download those huge blobs by pausing the download, use their internet connection for work during the day and continue downloading my giga-blobs during the night.
Long story shrt-er: a giga-blob that fits to be the test case here:
http://technology.softf1.com/mmmv_raspbian_t1/mmmv_gigablast_container_t1/
The torrent is seeded from my home page hosting account. With Transmission BitTorrent client it works just fine, but if I use the instant.io, then I get
Downloading torrent from
http://technology.softf1.com/mmmv_raspbian_t1/mmmv_gigablast_container_t1/2015_12_07_mmmv_gigablast_container_t1.ova.torrent
Error downloading torrent: Failed to fetch

I understand that I am expected to copy-paste a magnet link to the form text field, but there are no options for torrent files, so I thought that it should then distinguish a link to a .torrent file from a Magnet link. The Magnet link from Transmission did not work either. Usually the command that I use for generating torrent files on Linux is something along the lines of (please replace seeding URL and file name with Yours):
nice -n20 mktorrent -a udp://tracker.openbittorrent.com:80/announce,udp://explodie.org:6969/announce,udp://11.rarbg.com/announce,udp://tracker.ccc.de:80,http://announce.torrentsmd.com:6969/announce -w http://technology.softf1.com/raudrohi/2013/09_04_v_25/bittorrent_seeds/raudrohi_v_25.tar.bz2 ./raudrohi_v_25.tar.bz2

The mktorrent is usually available from Linux distribution package collection. The web browser that I use is openSUSE Linux standard Google Chrome fork. So, the WebTorrent is a nice project, but if I were to start a project like that, then I would be very scared of the lack of control of the various Browsers. In stead of trying to integrate everything to the browser, I would try to make browser plugin that makes it easy to work with some custom BitTorrent client that can actually be developed, changed, design-decision-and-release-wise controlled by the project team. May be You can re-rail the WebTorrent project by dumping the WebRTC backend and focusing on 2 new parts of the project:
part_1]  some fine, multiplatform, BitTorrent client library, in C++, where You determine its API.
part_2] the myriads of clients to that client, including command line client, various browser plugins,  a stand alone GUI, etc.
That way Your investment will not get lost, if the technology changes, in stead of Web browsers people want to run the bit-torrent client at home appliance (which You may sell as a pre-configured-multimedia-home-Network-Storage-device-on-home-LAN), give it download commands, magnet links, etc. from their cellphone, while at work, and may be, due to the nasty corporate firewalls and the Chinese Wall, etc. stream some downloaded videos from that home NAT to their cellphone.
By separating the hard-to-do and expensive and technical-quality-critical BitTorrent client library, which later can also connect to other networks, Freenet, if it is still around at all, etc., from the GUI/UI part, You can keep up with the fancy trends and mitigate risks, while having more resources freed up to invest to the expensive-to-develop C++ library.
I'ts just my view. There are many views in this world.
Thank You for reading. :-D
P.S. If You want a ~20GiB test case, then You may take a look at
http://technology.softf1.com/various_virtual_appliances/2016/

Hey @martinvahi,
Super detailed content :smile: it may belong as a separate issue though.
It is possible I'm misunderstanding you, so please correct me if I'm wrong. It doesn't sound like this is directly related to @Lev09's question.
It sounds like your comment is a blend between usability issues on instant.io and questions about the underlying tech powering the webtorrent module. Perhaps split off the instant.io bits to a new issue on https://github.com/feross/instant.io/issues/new and move the rest here https://github.com/feross/webtorrent/issues/new ?

I do not want to be the one, who shovels work from my yard to the yard of others, but may be it is better, if You do copy-paste-cut-and-cite part, because it is Your idea to split my comment. I do not mind it being split, but I'm nobody here, so it's for the others to figure out, where something fits.
May be one way to split my previous comment is to just cite it in 2 separate places by saying something in the lines of: comment Foo at Bar, lines x-to-y?
Thank You for the compliment, of course. :-)

@martinvahi For instant.io or any website where you plan to download your .torrent files you must have CORS-enabled in your webserver.
On the other topic, the idea is to get wrtc module to work natively and hopefully others implement it in others language.

@DiegoRBaquero  Thank You for Your answer, but web browser settings are in a role of global settings that affect all sites that the web browser visits. One site requires X, another site requires not-X and may be a third site depends on X or not-X for security. Sticking to the defaults, even if the defaults are not always the smartest, allows at least some assumptions to hold.
Currently I have not studied the CORS part thoroughly, but may be some coross-browser plugin frame-work allows to create some browser plugins that will have more access than plain web pages. I haven't worked on those, but quick googling gave me
http://kangoextensions.com/
It's not free software, but it does have the following advertisement:
"Kango is free for open source non-profitable projects."
There were a few others, but I suspect that the same pattern will emerge as it was with JavaScript GUI libraries: at first I had to write my own, because there were only a few closed source alternatives, and then, after about 3 years, other open source libraries emerged and nobody needed the closed source ones. Therefore, if You design Your architecture to be sufficiently modular, You can dump the non-free cross-browser-plugin framework later, without re-writing the core part of Your project.
So, thank You for your advice,
thank You for Your answer, but
I do not think that changing web browser settings is the solution that solves the issue.


[Feature Request]: allow to use http scheme for self hosted oauth providers

Description
It would simplify the development process if we could pick the http scheme for self hosted oauth providers like keycloak.

+1


Removed setup.py based install

No review needed.

coverage: 33.944%. first build when pulling 6557714dcde2105712d7c46533543e22f2801e4e on andy/remove-setup-based-install into 6d5565f6e71291c283d578ae0bf797a20daa758a on master.


Persist Tokens Across Updates

Is there some way we can put the token files gm_tokens.json and microsoft_tokens.json into persistent storage so it doesn't have to generate new ones whenever the container is rebuilt?

As-is, if a user's TOTP key changes or their MFA method changes then the user would need to update their addon config and/or GM account following a container rebuild.
If the token data can persist, they are very long-lived tokens and aren't reliant on any MFA to refresh themselves once issued.
Not a huge problem, just a 'would be nice'.

Will do. That makes sense to have in general.

for sure 👍👍👍

Added!
Optional for both OnstarJS and my GMAuth class
https://github.com/metheos/OnStarJS/commit/21881f1b151ad4f79be857ec0b5b5c8f4c1085cb
    onStar = OnStar.create({
      deviceId: DEVICEID,
      vin: VIN,
      username: ONSTAR_USERNAME,
      password: ONSTAR_PASSWORD,
      onStarPin: ONSTAR_PIN,
      onStarTOTP: ONSTAR_TOTPKEY,
      checkRequestStatus: false,
      tokenLocation: "./tokenStorage",
    });

I set it up with your recommendation of /ssl/vehicle1 and it's working fine. This should also open up the possibility of a utility to create the Microsoft token set with a separate tool for our frozen neighbors to the north 😇


[Question] Is there a standard way to add user function to filter the picture?

There aren't.
Only modifying of the source code.


The installed module is not the latest code

Hi,
First of all, great library! But i have an issue that with npm install meta-auth i get the version 1.1.0 which seems to be the latest release. But when i try to use it i receive an error:
uncaught TypeError: res.forEach is not a function
    at Object.$.get [as success] ((index):34)
    at i (jquery-2.2.4.min.js:2)
    at Object.fireWith [as resolveWith] (jquery-2.2.4.min.js:2)
    at z (jquery-2.2.4.min.js:4)
    at XMLHttpRequest.<anonymous> (jquery-2.2.4.min.js:4)

Checking the node_modules the index.js is not the same as https://github.com/I-Gave/meta-auth/blob/master/index.js so it seems there is a release missing. If i replace the index.js for the latest code it is working fine.
Thanks,
Agustín.

Thanks for catching this.


How to describe text/csv response?

I have a case, where text/csv response is the most compact way to return data.
Could not find in documentation how to describe that, hence tried my best:
Description(b => b.Produces(200, contentType: "text/csv"));

It kind of works, but has one glitch - it produces OpenAPI spec like this:
 "200": {
            "description": "Success",
            "content": {
              "text/csv": null
            }
          },

to my understanding, it should be not "text/csv": null, but "text/csv": {} - not sure if that is related to .net 9 bug or not.
As a result, swagger UI is failing to render responses part.
I've also tried:
Description(b => b.Produces(200, contentType: "text/csv", responseType: typeof(void)));

but that resulted into same issue.

@dj-nitehawk - that worked, thank you!


Can not using the ollama in docker container. ERROR: [LLM] http error

It is a great plugin and I love it. But I found an error here.
[LLM] http error: error sending request for url (http://localhost:11434/api/generate): connection closed before message completed

Following the config in the readme.
{
    "huggingface/llm.nvim",
    opts = {
      -- cf Setup
    },
    config = function()
      local llm = require("llm")
      llm.setup({
        api_token = nil, -- cf Install paragraph
        -- for ollama backend
        backend = "ollama", -- backend ID, "huggingface" | "" | "openai" | "tgi"
        model = "starcoder2:7b",
        url = "http://localhost:11434/api/generate",
        tokens_to_clear = { "<|endoftext|>" }, -- tokens to remove from the model's output
        -- parameters that are added to the request body, values are arbitrary, you can set any field:value pair here it will be passed as is to the backend
        request_body = {
          parameters = {
            max_new_tokens = 60,
            temperature = 0.2,
            top_p = 0.95,
          },
        },
        -- set this if the model supports fill in the middle
        fim = {
          enabled = true,
          prefix = "<fim_prefix>",
          middle = "<fim_middle>",
          suffix = "<fim_suffix>",
        },
        debounce_ms = 150,
        accept_keymap = "<C-y>",
        dismiss_keymap = "<C-n>",
        tls_skip_verify_insecure = false,
        -- llm-ls configuration, cf llm-ls section
        lsp = {
          bin_path = nil,
          host = nil,
          port = nil,
          version = "0.5.2",
        },
        tokenizer = {
          repository = "bigcode/starcoder2-7b",

        }, -- cf Tokenizer paragraph
        -- tokenizer = nil, -- cf Tokenizer paragraph
        context_window = 4096, -- max number of tokens for the context window
        enable_suggestions_on_startup = true,
        enable_suggestions_on_files = "*", -- pattern matching syntax to enable suggestions on specific files, either a string or a list of strings
      })
    end,
  }

The MOST  wirred thing is that I can curl the answer to the same model & api url, and my vscode continue plugin can communicate this is ollama which is running on a docker container but this plugin cannot!
Thank you for your time and reply!

This maybe caused by llm.nvim plugin. So you can close it.


Using variables defined in EGL/GLES cmake modules

Also cleaning up header paths for EGL/GLES

Sorry, I was out of office for the week. I'll share the resnet benchmark results on Monday. And also check the modification for the generalized EGL/GLES changes that you've added by running it on the target. Thanks for sharing!


Implement partials, render function parameter

Summary

Implement the mustache partial feature.
Add a render parameter to render functions.

Result

Allows scenarios like this:

instantsearch.widgets.hits({
      ...
      templates: {
        item: 'Template using {{> partial }}',
        partial: 'Partial template {{variable }}'
      }
  });

Allows scenarios like this:

instantsearch.widgets.hits({
     ...
      templates: {
        item: (function($item, $render) {
          var template = generic_template;
          if ($item.type == "special") {
            template = special_template
          }
          return $render(template);
      })
  });

Or this:
instantsearch.widgets.hits({
     ...
      templates: {
        item: (function($data, $render) {
          return $render('My template with a {{> partial }}', data.partial_data);
       }),
       partial: partial_template,
      }
  });

Is there a reason you prefer this towards "function" templates?

Perhaps this PR is complicated by the fact that it actually introduces 2 changes. I decided to do it that way because both changes were to the same section of code, so if I made separate PRs they would likely conflict with each other.
My motivation for adding support for partial templates is fairly obvious, since it's the reason behind why they were invented in the first place: it allows the template creator to modularize and reduce duplication within a template (for example, if a template displays a name in several places, it can use a "name" partial template to render it in each case).
My main motivation for adding a "render" parameter to function templates was that I was looking for some way for the hits widget to be able to use a different template for each row based on the data in the row (which is a feature others have asked for in the past). The cleanest way I could come up with was to create a function template that was able to choose the template based on the data and then render it--but to do the rendering, it needed all the original context, much of which wasn't available to the function, only to its caller. So I created a render parameter that allowed the render to happen with all the original context.

That's an interesting addition to the current templating system.

The cleanest way I could come up with was to create a function template that was able to choose the template based on the data and then render it--but to do the rendering, it needed all the original context, much of which wasn't available to the function, only to its caller.

This might mean that we didn't modularize our templates sections enough, which is likely. Could you explain what wasn't possible with a function template exactly here?
Eventually, we want to move away from Hogan and rely on a more powerful renderer though. This change would have been gracefully accepted earlier but we don't think it makes sense to add this feature if we want to get rid of Hogan all together soon.
Thanks a lot for your contribution though – feel free to open an issue first next time to make sure that a change is appropriate to land in the codebase.

This might mean that we didn't modularize our templates sections enough, which is likely. Could you explain what wasn't possible with a function template exactly here?

What I was trying to do is have two (or more) completely different templates for rendering search hits and choose one based on the value of one of the fields of the item being rendered. I was leaning towards using Hogan templates because they seemed less verbose for fairly complex output, but I guess I'd be open to using a function template instead if that would do what I want. I was also hoping to avoid combining all the templates into a single large function template with an if statement or other logic to choose the relevant part to render.

Thanks a lot for your contribution though – feel free to open an issue first next time to make sure that a change is appropriate to land in the codebase.

No problem. I thought since I had figured out how to do it, it made sense to submit it as a PR. So you're not interested in either part of the PR? The Mustache partial implementation could be pretty useful until you actually do move away from Hogan, and the render parameter could be adapted to provide something like partials for function templates even after you do move away from Hogan (by changing it so that you pass a template name instead of the full content of the template to the render function).

Thanks for the efforts @glassfordm, we really appreciate it.
As explained in https://github.com/algolia/instantsearch.js/pull/4133#issuecomment-534290126, since what you're trying to achieve can be done with JavaScript functions and that we don't plan to invest more in Hogan, we'll close this PR.


Added Color, Date and Range parameters

Color: Expects an RGB hex string with optional # prefix
Date: Has to be datetime or np.datetime64 type

The Date parameter could support bounds like Number but I'm not sure how important that is.

Coverage decreased (-0.6%) to 68.633% when pulling ec6d8acd55637b0854fefd44f23910ecf6a3065d on color_and_date into d96e0ccb3aead97a10309f751ab5fd1a3eaa264f on master.

@jbednar Added bounds validation for Date along with a Range parameter which also has bounds validation.

Coverage decreased (-1.9%) to 67.34% when pulling d83535393bdd50555b2607c399472853c364dfd4 on color_and_date into d96e0ccb3aead97a10309f751ab5fd1a3eaa264f on master.

Coverage decreased (-1.9%) to 67.34% when pulling d83535393bdd50555b2607c399472853c364dfd4 on color_and_date into d96e0ccb3aead97a10309f751ab5fd1a3eaa264f on master.

Coverage decreased (-1.9%) to 67.34% when pulling d83535393bdd50555b2607c399472853c364dfd4 on color_and_date into d96e0ccb3aead97a10309f751ab5fd1a3eaa264f on master.

Coverage decreased (-1.9%) to 67.34% when pulling d83535393bdd50555b2607c399472853c364dfd4 on color_and_date into d96e0ccb3aead97a10309f751ab5fd1a3eaa264f on master.

Coverage decreased (-1.9%) to 67.34% when pulling d83535393bdd50555b2607c399472853c364dfd4 on color_and_date into d96e0ccb3aead97a10309f751ab5fd1a3eaa264f on master.

Coverage decreased (-1.8%) to 67.428% when pulling 341c64e09fbac2abd9465839afa1a9c2391c9a27 on color_and_date into d96e0ccb3aead97a10309f751ab5fd1a3eaa264f on master.


Why do you mark ItemX readonly?

The filed ItemX isn't readonly in official .NET

I was just wondering that myself after checking out the nuget package and getting compile errors.


Expr in fixes

Assorted expression evaluation tests and a few fixes.

Ack!  no!  That was not intentional... Thanks for noticing that.  Update pushed.

Thanks, I also just noticed there are two 5's and 6's...

Okay, got serious.  Ran:
cat test*.py | grep 'def ' | sort | uniq -c
and found another test ( test_metacat_query_q ) that had a duplicated name...
This time I think we really have them all. :-)

I thought I had deleted one of the copies of test_metacat_query_q in my last pull request because it seems that it wasn't just the name that was duplicated but the whole test.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/160 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 00d3a60:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


OpenType ordinals

There is an OpenType feature specifically for ordinals, called ordn.
We should use this instead of superscript.

Typst documentation on adding OpenType features:
https://typst.app/docs/reference/text/text/#parameters-features

Font "Source Serif Pro" has this feature enabled.


Improving K-9 Logging

Expected behavior
Users should easily be able to report issues, with enough detail to allow us to debug them.
Actual behavior
We get issues opened with little to no information. The recommended way to diagnose problems involves (for all recent Android devices) connecting to adb. This essentially restricts bug reports to Android developers only.
Problems arising
In a normal app which is funded by a development team, you'd use something like ACRA which then pumps data to a back-end server.  K-9 doesn't have this option. It's an open source project, there's no hosting or funding available for that kind of solution.
Often the bugs we see are not crashes per see, which are fairly straightforward to solve, but instead Doze / data usage related.
Possible solution 1 - Save logcat output
The reason we can't recommend aLogCat is because it no longer has access to our logs. Therefore we must read the logs ourselves. Obviously if we crash just reading them is insufficient. So we also have to write them to a file.
One option is to implement a rolling logging solution. This would:

Run logcat on the device in our process. It turns out this is fairly straightforward, if rather ugly:

http://www.journal.deviantdev.com/android-log-logcat-to-file-while-runtime/

Periodically roll the log file. We don't want to take up too much space. This could be a configurable option - users with complex problems could increase the amount stored.
Allow the user to save logs to external storage

Our issue form would then say:
Upload the the latest log file from K-9's directory on your phone.

We need to carefully test this to make sure that the background logging itself doesn't cause any issues.
Possible solution 2 - Stop using android.util.Log
The alternative is more drastic. We implement a new logging architecture. This would give us full control - we could flush to disk periodically (and when getting an uncaught exception) rather than all the time.

I'm for option 2 using Timber.
In debug builds we could output to Android's system log. In release builds we write log messages to a file. I'd like us to display a warning message in the UI when file logging is enabled. That should reduce the chances of users leaving file logging enabled accidentally.

I would suggest to use crashlytics. It is something like ACRA but you do not need to host your own backend und its completly free.

I think this can be closed, since Timber is included now in the lib and the main app.

Nope, we still have work to do. We need to start writing Timber logs to a file.

I would like to suggest Lumberjack project that provides special Timber.Tree for file logging based on logback and slf4j. We have possibility to flexibly configure amount of files, their size and name pattern (based on date or number).
I can help with integrating this in project.

@fobo66: Pull requests for this are very welcome. But please note that we won't make the feature available to users before the requirement to display a persistent notification is implemented. Another option might be to automatically disable logging to a file after a certain amount of time.

Yes, please enable a freakin' logfile.