How can I use tf.data.experimental.make_csv_dataset to make a dataset with no batches

Currently I'm doing this -
column_names = ['Label','Sentence']
batchsize = 32
label = column_names[0]
train_dataset = tf.data.experimental.make_csv_dataset(
    'datasettrain.csv',
    batchsize,
    column_names = column_names,
    label_name = label,
    num_epochs=1
)

Due to this the batches being ordered dicts don't allow me to do certain things which a dataset directly loaded from tfds.load would allow. For instance in this tutorial - https://www.tensorflow.org/text/tutorials/text_classification_rnn
The author is able to run -
for example, label in train_dataset.take(1):
  print('text: ', example.numpy())
  print('label: ', label.numpy())

and I wish to do the same however in my case the type for example comes out to be <class 'collections.OrderedDict'> while in the tutorials it is <class 'tensorflow.python.framework.ops.EagerTensor'>.
Is there some way I can do the same with tf.data.experimental.make_csv_dataset?
Using -
ls = []
for example, label in train_dataset:
  ls.append([example['Sentence'][0],label[0]])

I was able to get the required form internally however the entire array is no longer a <class 'tensorflow.python.data.ops.dataset_ops.PrefetchDataset'> is there any way to do that from this ls?

Hi @aflah02 ! Can you please share the code which you were trying to execute as a Colab gist?

Hey
The code which is causing the error is the
one mentioned in the issue already
Which 2 methods are you referring to?

Incase some one faces a similar issue I eventually got a workaround by making 2 numpy arrays converting them into tensors and then feeding that into tf.data.Dataset.from_tensors but I guess there would be a better way than this the problem seems to be that Tensorflow does not recognize a np array of EagerTensors as a valid argument for making a dataset

Ok @aflah02 ! Glad it resolved. Could you please share the resolved code for community ?

Sure @mohantym Thanks for pointing it out
For perspective my dataset is of the following form -

A CSV
2 Columns
First Column Which I Need to Predict Named Label
Second Column Which I Use to Predict the Label which is a Sentence

So I first loaded it in using -
column_names = ['Label','Sentence']
batchsize = 1
label = column_names[0]
train_dataset = tf.data.experimental.make_csv_dataset(
    'datasettrain.csv',
    batchsize,
    column_names = column_names,
    label_name = label,
    num_epochs=1
)

This returns a tensorflow.python.data.ops.dataset_ops.PrefetchDataset
After that I used the following code to do the work mentioned in Comment above
ls_train = []
for example, label in train_dataset:
  ls_train.append([example['Sentence'][0],label[0]])

nparrtrainsentences = np.array([])
nparrtrainlabels = np.array([])
for i in range(len(ls_train)):
  nparrtrainsentences = np.append(nparrtrainsentences, [ls_train[i][0].numpy()])
  nparrtrainlabels = np.append(nparrtrainlabels, [ls_train[i][1].numpy()])

train_sentences = tf.convert_to_tensor(nparrtrainsentences)
train_labels = tf.convert_to_tensor(nparrtrainlabels)

train_dataset = tf.data.Dataset.from_tensors((train_sentences,train_labels))

In hindsight certain aspects do feel redundant now but since it works for now I won't be trying to fix those things

My friend, actually you can simply add an .unbatch() after the make_csv_dataset


Use example.org everywhere for consistency

This makes it a bit easier when doing a string search/replacement, by only having to replace example.org. Nginx's app.conf uses example.org.

That’s a very nice suggestion, thank you!


Update sbt-tpolecat to 0.1.21

Updates io.github.davidgregory084:sbt-tpolecat from 0.1.12 to 0.1.21.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.davidgregory084", artifactId = "sbt-tpolecat" } ]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #290.


Support release candidates cut

Breaking turnover into release candidates support

I renamed release candidate branches from candidate to rc


.publish folder is missing

I followed the instructions (working on Mac with VSCode) and couldn't find .publish folder so I couldn't update manifest.json with ngrok url
to fix this I "Download" the  package from App Studio, extract it, manipulate manifest.json manually and compress it manually before uploading to MS Teams
[Enter feedback here]

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

ID: 8f876156-cc1e-468f-4647-5143a5de03b1
Version Independent ID: 618f43a3-3371-ad99-13aa-087b9303298c
Content: Get started - Build and run your first app - Teams
Content Source: msteams-platform/build-your-first-app/build-and-run.md
Product: msteams
GitHub Login: @heath-hamilton
Microsoft Alias: lajanuar

Did you face any error during setup? I was able to set up with no issues.
Can you please try once again and let me know if it is still same.

there were no errors during setup
here is the project structure after I click finish:

I am checking it internally if there is any known issue on this.

@orlavy1976 really sorry our documentation update didn't make it out at the same time as the version update.  I am working on getting that approved ASAP.  It should be out today/tomorrow.  Sorry about the inconvenience.
In the meantime though the .publish folder is deprecated.  Once you create a new project in the toolkit all you need to do is hit F5/run and it will launch Chrome and install your package.  The package is stored in App Studio and we just use its id to install it.


missing comma?

Challenge Waypoint: Introducing JavaScript Object Notation JSON has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
var myMusic = [
  {
    "artist": "Billy Joel",
    "title": "Piano Man",
    "release_year": 1993,
    "formats": [
      "CS",
      "8T",
      "LP" ],
    "gold": true
  }, <---------------------perhaps is a missing a comma, here?
  // Add record here
  {
    "artist": "Tsuyoshi Hiroshi",
    "title": "Tonboyou",
    "release_year": 1987,
    "formats": [
      "CD",
      "DVD",
      "LP"]
  }
];

This just saved me! Couldn't figure out what I had done wrong, but adding the comma meant the test passed :)

I think there should be further instruction (maybe a "Note: ") that let the camper know about adding the comma between objects if there is more than one object in the array.  If it's already there when they start this waypoint, then they probably wouldn't even notice the comma there.

I agree. The notation to "// Add record here" is assumed to indicate, based on similar, earlier examples, that all modifications to the code should be below the notation. There should be something explaining how that comma works.

i'm so angry about it, b'cause there was no any word about com

Now there is but it still got me googeling
Note
You will need a comma in between objects in JSON objects with more than one object in the array.

Thank you so much!


Sticky headings are too close on window's edge when using multiple tab groups

Problem:
The sticky headings are too close on the window's edge when using multiple tab groups, making it slightly harder to read.
Image:

Settings:
I am using the Minimal Theme by kepano
Additional details:
The issue occurs when there are three tab groups or when the sidebar is used. I think that is because there are less horizontal space for each note when there are many tab groups or when the right sidebar is used.

There were indeed some minor compatibility issues with the Minimal theme. I have attempted to fix them. Could you please check if the issue has been resolved?

The issue has been resolved. Thank you!


Does vite provide a require.context-like API?

Is your feature request related to a problem? Please describe.
I would like to be able to dynamically read the file objects inside specific directory. Webpack provides API like require.context in order to achieve this.
Describe the solution you'd like
Is there any way within vite that one can perform something similar to require.context to read and parse file objects
Additional context
This would be useful to create dynamic route building with Vue router by reading the contents of the directory.

Perhaps an alternative would be to us fs.readdir?

like parcel: import imgmap from "./path/*.png"

Any update on this Feature?
I am currently trying to figure out a way to autoload Vuex Modules and Vue Components in Runtime without require.context and havent found any good Solution.

I am also looking for a substitute for require.context to be able to migrate to Vite.
I tried this rollup plugin:
https://github.com/elcarim5efil/rollup-plugin-require-context
But it doesn't seem to work out of the box.
Any advise is much appreciated!

rollup plugin maybe used in build mode, but in dev mode no rollup, how to support this feature?

I haven't thought of nice resolutions for this. But I have written a transform for this。https://github.com/luxueyan/vite-transform-globby-import. It support both dev and build mode.

Same here. Would be useful for autoloading Vuex modules for instance.

I wrote one that can support dynamic import and other functions https://github.com/vbenjs/vite-plugin-import-context. Welcome to try

I wrote one that can support dynamic import and other functions https://github.com/vbenjs/vite-plugin-import-context. Welcome to try

Renamed to glob import in 8d8e2cc (I realize the syntax and transform is very similar to @luxueyan 's plugin only after I have implemented it!)

Renamed to glob import in 8d8e2cc (I realize the syntax and transform is very similar to @luxueyan 's plugin only after I have implemented it!)

Can anyone give an example of how to use the newly implemented glob import?
I'd like to import all .svg files in a folder in my project.
Thanks!!

Can anyone give an example of how to use the newly implemented glob import?
I'd like to import all .svg files in a folder in my project.
Thanks!!

https://github.com/vitejs/vite/tree/main/packages/playground/import-glob

https://github.com/vitejs/vite/tree/main/packages/playground/import-glob

https://github.com/vitejs/vite/tree/main/packages/playground/import-glob

oh~I want to ask how to use it

https://github.com/vitejs/vite/tree/main/packages/playground/import-glob

oh~I want to ask how to use it

The syntax has been updated https://vitejs.dev/guide/features.html#glob-import

The syntax has been updated https://vitejs.dev/guide/features.html#glob-import

The syntax has been updated https://vitejs.dev/guide/features.html#glob-import

Oh, that's great! thanks

The syntax has been updated https://vitejs.dev/guide/features.html#glob-import

Oh, that's great! thanks

The syntax has been updated https://vitejs.dev/guide/features.html#glob-import

牛🍺  !

/**
 * 路由自动注册
 * 遍历views目录下所有 *.route.ts 文件，内部为路由信息
 * https://cn.vitejs.dev/guide/features.html#glob-import
 */
import { RouteRecordRaw } from "vue-router"

const modules = import.meta.globEager('/src/views/**/*.route.ts')
const autoRegisteredRoutes: RouteRecordRaw[] = []
for (const path in modules) {
  autoRegisteredRoutes.push(modules[path].default)
}

export { autoRegisteredRoutes }

import.meta.globEager 这个方法是同步的，等同于require.context
希望对大家能有帮助

webpack vs vite glob
/** webpack */
// const files = require.context('@/packages', true, /doc\.md$/);
// files.keys().forEach(component => {
//   const componentEntity = files(component).default;
//   const name = `${component.split('/')[1]}`;
//   pagesRouter.push({
//     path: '/' + name,
//     component: componentEntity,
//     name
//   });
// });

/** vite */
const modulesPage = import.meta.glob('/src/packages/**/doc.md');
for (const path in modulesPage) {
  let name = (/packages\/(.*)\/doc.md/.exec(path) as any[])[1];
  pagesRouter.push({
    path: '/' + name,
    component: modulesPage[path],
    name
  });
}

more...

vuex: webpack vs vite glob
/** webpack */
const files = require.context(".", false, /\.js$/);
const modules = {};

files.keys().forEach(key => {
    if (key === "./index.js") return;
    modules[key.replace(/(\.\/|\.js)/g, "")] = files(key).default;
});

export default modules;

/** vite */
const files = import.meta.globEager("./*.js");

const modules = {};
for (const key in files) {
    // if (key === "./index.js") continue;
    modules[key.replace(/(\.\/|\.js)/g, "")] = files[key].default;
}
console.log(files, modules);

export default modules;

Hi folks. Recently I implemented a vite plugin that can handle the require.context here. It's implemented by regexp search. Hope this can help a little bit.
BTW : contributions are welcome.

vite 2.0 docs Glob import

Vite supports importing multiple modules from the file system via the special import.meta.glob function:

const modules = import.meta.glob('./dir/*.js')
The above will be transformed into the following:
// code produced by vite
const modules = {
  './dir/foo.js': () => import('./dir/foo.js'),
  './dir/bar.js': () => import('./dir/bar.js')
}

You can then iterate over the keys of the modules object to access the corresponding modules:
for (const path in modules) {
  modules[path]().then((mod) => {
    console.log(path, mod)
  })
}


Added Candidate-Elimination Algorithm

For a given set of training data examples stored in a .CSV file, implement and demonstrate the Candidate-Elimination algorithm to output a description of the set of all hypotheses consistent with the training examples.

@NisargaJN thank you


🐛 [firebase_ui_auth] "Verify your email" route spinner is wide

Bug report
Describe the bug
The firebase_ui_auth "Verify your email" route (EmailVerificationScreen) now shows a spinner as wide as the paragraph above it instead of being the usual circle.
Steps to reproduce
Use my sample project. Run instead of debug to skip over exceptions since I don't implement Firebase Auth. Observe the only screen is the verify email route with a wide spinner as in image below.

Expected behavior
Round spinner
Sample project
https://github.com/EnduringBeta/flutter-bug/tree/verify-email-spinner

Additional context
None

Flutter doctor
Run flutter doctor and paste the output below:

$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.10.0, on Microsoft Windows [Version 10.0.22621.1702], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.2.6)
[√] Android Studio (version 2022.2)
[√] Android Studio
[√] VS Code (version 1.66.2)
[√] Connected device (4 available)
[√] Network resources

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
$ flutter pub deps -- --style=compact
Dart SDK 3.0.0
Flutter SDK 3.10.0
flutter_bug 1.0.0+1

dependencies:
- firebase_auth 4.6.0 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 2.12.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_ui_auth 1.4.0 [email_validator firebase_auth firebase_core firebase_dynamic_links firebase_ui_localizations firebase_ui_oauth firebase_ui_shared flutter flutter_localizations flutter_svg]
- flutter 0.0.0 [characters collection js material_color_utilities meta vector_math sky_engine]

dev dependencies:
- flutter_lints 2.0.1 [lints]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters collection js matcher material_color_utilities meta source_span stream_channel string_scanner term_glyph]
- integration_test 0.0.0 [flutter flutter_driver flutter_test path vm_service async boolean_selector characters clock collection fake_async file js matcher material_color_utilities meta source_span stack_trace stream_channel string_scanner sync_http term_glyph test_api vector_math webdriver]

transitive dependencies:
- _flutterfire_internals 1.3.0 [collection firebase_core firebase_core_platform_interface flutter meta]
- async 2.11.0 [collection meta]
- boolean_selector 2.1.1 [source_span string_scanner]
- characters 1.3.0
- clock 1.1.1
- collection 1.17.1
- crypto 3.0.3 [typed_data]
- desktop_webview_auth 0.0.12 [crypto flutter http flutter_web_plugins plugin_platform_interface]
- email_validator 2.1.17
- fake_async 1.3.1 [clock collection]
- file 6.1.4 [meta path]
- firebase_auth_platform_interface 6.15.0 [_flutterfire_internals collection firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 5.5.0 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser js meta]
- firebase_core_platform_interface 4.8.0 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 2.5.0 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_dynamic_links 5.3.0 [firebase_core firebase_core_platform_interface firebase_dynamic_links_platform_interface flutter meta plugin_platform_interface]
- firebase_dynamic_links_platform_interface 0.2.6 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_ui_localizations 1.5.0 [flutter flutter_localizations path]
- firebase_ui_oauth 1.4.0 [desktop_webview_auth firebase_auth firebase_ui_auth firebase_ui_shared flutter_svg flutter]
- firebase_ui_shared 1.3.0 [flutter]
- flutter_driver 0.0.0 [file flutter flutter_test fuchsia_remote_debug_protocol path meta vm_service webdriver async boolean_selector characters clock collection js matcher material_color_utilities platform process source_span stack_trace stream_channel string_scanner sync_http term_glyph test_api vector_math]
- flutter_localizations 0.0.0 [flutter intl characters clock collection js material_color_utilities meta path vector_math]
- flutter_svg 1.1.6 [flutter meta path_drawing vector_math xml]
- flutter_web_plugins 0.0.0 [flutter js characters collection material_color_utilities meta vector_math]
- fuchsia_remote_debug_protocol 0.0.0 [process vm_service file meta path platform]
- http 0.13.6 [async http_parser meta]
- http_parser 4.0.2 [collection source_span string_scanner typed_data]
- intl 0.18.0 [clock meta path]
- js 0.6.7 [meta]
- lints 2.1.0
- matcher 0.12.15 [async meta stack_trace term_glyph test_api]
- material_color_utilities 0.2.0
- meta 1.9.1
- path 1.8.3
- path_drawing 1.0.1 [vector_math meta path_parsing flutter]
- path_parsing 1.0.1 [vector_math meta]
- petitparser 5.4.0 [meta]
- platform 3.1.0
- plugin_platform_interface 2.1.4 [meta]
- process 4.2.4 [file path platform]
- sky_engine 0.0.99
- source_span 1.9.1 [collection path term_glyph]
- stack_trace 1.11.0 [path]
- stream_channel 2.1.1 [async]
- string_scanner 1.2.0 [source_span]
- sync_http 0.3.1
- term_glyph 1.2.1
- test_api 0.5.1 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.2 [collection]
- vector_math 2.1.4
- vm_service 11.3.0
- webdriver 3.0.2 [matcher path stack_trace sync_http]
- xml 6.3.0 [collection meta petitparser]

Thanks for the report. I am able to replicate the same using the repo shared.


Chinese language experiments

Hi shang,
have your team experimented this method on Chinese data? Could you share your progress and any plans on releasing Multi-lingual version of AutoNER?
THX

Thanks for your interests! We haven't tried this idea on Chinese data yet. And I don't have any plan for it so far.


front end cleanup

After opening this PR I see the work done in #976
This PR ideally can be added as a small iterative piece on the road to the goal of #976 seems like @mlunoe and I have about the same goals in mind :)
I just took a bit smaller bite out of the :apple:

Since we added the new building system I'm going to close this one.

no prob

I think we can prob cherry pick a few of these but I will open a new PR


Require EmbeddedAnsible playbook to create playbook service

Before we assume the playbook to create a playbook catalog item is of type ManageIQ::Providers::AnsibleTower::AutomationManager::Playbook. Now we need to switch to ManageIQ::Providers::EmbeddedAnsible::AutomationManager::Playbook
We can argue whether it is better to write playbook = ConfigurationScriptBase.find(info[:playbook_id]) which will work for both Embedded and external playbook. I think right now limiting to Embedded only can help to reject unqualified ConfigurationScript. We can further enhance when the time comes to support external tower.

@miq-bot add_label providers/ansible_tower
@miq-bot add_label enhancement
@miq-bot assign @gmcculloug

@syncrou please review

Looks good.  Was running with configuration locally as well.


Mention that int() can be used as an alternative to floor()

See discussion in #19262.

Thanks!

This can be a bit misleading though, because floor(-2.99) is -3 but int(-2.99) is -2.


authorization/action noExistingAuthIdSet

PROBLEM DESCRIPTION
It seems that I can't update authorizations using json.
REQUESTED INFORMATION
Make sure your have performed every step and checked the applicable boxes before submitting your issue. Thank you!

[x] Read the Contributing Guide and Policy and the Code of Conduct
[x] Searched the problem in issues
[x] Searched the problem in discussions
[x] Searched the problem in the docs
[x] Searched the problem in the chat
[x] Device used (e.g., ESP32-S3): ESP32-S3
[x] Nuki Hub binary firmware version number used: 9.02-master6<PHONE_NUMBER>5.117.1) (also on  9.01<PHONE_NUMBER>1.2.1)

[x] Pre-compiled
[ ] Self-compiled

[ ] Provide the output of http://nukihubIP/info: For some strange reason, /info  301-redirect me to /

TO REPRODUCE

Query nuki/authorization/json. get back the following:

[{"authId":123456,"idType":0,"enabled":1,"name":"Lee","remoteAllowed":1,"dateCreated":"2023-10-11 00:00:05","lockCount":91,"dateLastActive":"2024-10-28 17:32:53","timeLimited":0,"allowedFrom":"0000-00-00 00:00:00","allowedUntil":"0000-00-00 00:00:00","allowedWeekdays":[],"allowedFromTime":"00:00","allowedUntilTime":"00:00"},{"authId":654321,"idType":1,"enabled":1,"name":"NukiHub","remoteAllowed":1,"dateCreated":"2024-10-08 20:21:43","lockCount":15,"dateLastActive":"2024-10-27 13:48:35","timeLimited":0,"allowedFrom":"0000-00-00 00:00:00","allowedUntil":"0000-00-00 00:00:00","allowedWeekdays":[],"allowedFromTime":"00:00","allowedUntilTime":"00:00"}]

Trying to update the name using authorization/action with the following json: {"action": "update", "authId": "123456", "name": "test"} and getting back the following error authorization/commandResult noExistingAuthIdSet

EXPECTED BEHAVIOUR
The name of authId 123456 should be changed to "test"
I've tested it on the latest released version 9.01 and the latest development version.
Thank you!

Having similar issues on<PHONE_NUMBER>6.1501.1 - https://github.com/technyon/nuki_hub/actions/runs/11474751836
Other JSON actions (add / delete) seem to be working fine tho.

Hi,
Latest version<PHONE_NUMBER>6.129.1) is still having issues regarding authorization/action.
Now it no longer gives "noExistingAuthIdSet", it just does nothing. I've also noticed that authorization/json only return 4 results (instead of 4 users + nuki_hub in my case).
Thanks,
Lee.

Pretty sure this is fixed.
Note that a couple of MQTT topics have moved in the latest 9.02 build.
The authorization action topic is now under nukihub/lock/authorization/action (where it previously was under nukihub/authorization/action) or under nukihub/opener/authorization/action

Thank you for clearing the issue around the topics.
However, I've noticed that the user list for nuki_hub only updates upon resetting my MQTT server and waiting for the hub to reconnect.
I can confirm that sending an update works properly for the 'name' and 'enabled' fields.
Additionally, I'm experiencing issues with the /lock/authorization/json topic. I have 4 users, but while the JSON lists 4 users, one of them is named "NukiHub," which I assume is the hub itself. I am uncertain about the missing user. The 'Max authorization entries' under '/advanced' is set to 5. Let me know if I can provide any information to help resolve it.
Thank you!

However, I've noticed that the user list for nuki_hub only updates upon resetting my MQTT server and waiting for the hub to reconnect.

Not sure what you mean by this. The /lock/authorization/json and /lock/authorization/entries (if you enabled Publish a topic per authorization entry and create HA sensor) topics update together with the Nuki configuration based on the Query interval configuration or manually by setting /lock/query/config = 1 or by sending an action to /lock/authorization/action

Additionally, I'm experiencing issues with the /lock/authorization/json topic. I have 4 users, but while the JSON lists 4 users, one of them is named "NukiHub," which I assume is the hub itself. I am uncertain about the missing user. The 'Max authorization entries' under '/advanced' is set to 5. Let me know if I can provide any information to help resolve it.
Thank you!

Can't reproduce this. I have mine set to 10 and get 10 authorizations (this will also include NukiHub itself, keypads, fobs etc.)
Are you sure you get 4 and not 5? Note that the index goes from 0 to 4 in this case and not 1 to 5.

There was probably a configuration error on my side regarding the topic update.
I'm getting 4 users, index 0 to 3.
For testing I've added another user via the app, and now I am getting 5 users on the topic. I can't see the new user I've added, but the missing user from before is there. Very strange...
Could you try maximizing the authorizations and verify if you're receiving all of them or -1?

I have 10 "users" (1 app user, 4x NukiHub, 1x keypad, 4x fob) connected to my lock and have max auth users in Nuki Hub set to 10.
And I get 10 users as expected in my auth json and 10 separate entries when enabling this option in NukiHub.
You could just set Max authorization entries to 10 or 15. Do take note of the recommended stack sizes when upping this number and adjust accordingly.

Max authorization entries already set to 10.
For debugging I've created another user, and now I see 6 (should be 7 with NukiHub). It seems that I am always missing the last user on the list. strange.

I am going to close this as the main issue here is fixed. If you have another problem please open a new issue and provide (USB) serial logs using a program like hterm while running a debug build (OTA update using http://nukihubip/otadebug?) of 9.02-beta2 or up or 9.02-master14 or up


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.10 to before v5.16.8",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=b5de0c60cc30c2a3513c7188c73f3f29acc29234",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=3365f9daa2a7060309979d55105f0a1c12354683"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "b5de0c60cc30c2a3513c7188c73f3f29acc29234",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "3365f9daa2a7060309979d55105f0a1c12354683",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "RDMA/cma: Use correct address when leaving multicast group\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.16.8 by commit 3365f9daa2a7060309979d55105f0a1c12354683, it was introduced in version v5.10 by commit b5de0c60cc30c2a3513c7188c73f3f29acc29234. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000666


Travis: Use Focal-distribution, so we no longer have to downgrade Docker

Focal is Ubuntu version 20.04, which is the latest available in Travis: https://docs.travis-ci.com/user/reference/linux

I'm good with this change once we get Travis green

Coverage decreased (-0.5%) to 93.992% when pulling f53a8f723c260f4ecf0cbe7988dc0f2afbb1402d on bblommers:upgrade-travis-linux_distribution into 2f70373c2e9d91405082ddbd12eb61fb43914b20 on spulec:master.

Looks like it's working now @spulec - Travis just needed a kick


RuntimeError: Program 'ffmpeg' is not found; perhaps install ffmpeg using 'apt-get install ffmpeg'.

When i run the python file, "run_pretrained_interactive.py" i get the error RuntimeError: Program 'ffmpeg' is not found; perhaps install ffmpeg using 'apt-get install ffmpeg'. can someone tell me what this means and how to fix it?

You need to install ffmpeg

The error message you are getting means that FFmpeg is not installed on your system, or it is not recognized by the Python script. To fix this, you need to install FFmpeg on your Windows PC and add it to your system path. Here are the steps to do that:
Download the zip file of the latest version of FFmpeg from here.
Extract the zip file using a file archiver such as 7-Zip or WinRAR.
Rename the extracted folder to FFmpeg and move it to the root of your C: drive.
Open Command Prompt as an administrator and run the following command: setx /m PATH "C:\ffmpeg\bin;%PATH%"
Restart your computer and verify the installation by running: ffmpeg -version in Command Prompt.

Thank you i just had to restart my pc


can I use bool in struct, but use int in mysql table?

I have a struct like this
type User {     Deleted bool  }
when I tried to query like this:
db.Where("deleted = ?", false)
I found that the final SQL like this
where (deleted = 'false')
I tried to add tag
sql:"type:int"
or
gorm:"type:int"
but no lucky, what should I do?

it's my fault, the log didn't show real executed sql


helm list -n  doesn't give accurate output

Describe the bug a clear and concise description of what the bug is.
We use the spec targetNamespace for some of our helm releases to install them in a particular namespace.
but the release shows up when we query kube-infrastructure namespace.
ex- helm list -n kube-infrastructure should list only the releases installed under namespace kube-infrastructure. But we get some that are installed in different namespaces.
What's your helm version?
version.BuildInfo{Version:"v3.9.0", GitCommit:"7ceeda6c585217a19a1131663d8cd1f7d641b2a7", GitTreeState:"clean", GoVersion:"go1.18.2"}
What's your kubectl version?
Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.2", GitCommit:"f66044f4361b9f1f96f0053dd46cb7dce5e990a8", GitTreeState:"clean", BuildDate:"2022-06-15T14:14:10Z", GoVersion:"go1.18.3", Compiler:"gc", Platform:"darwin/amd64"}
What's the chart version?
flux2-0.20.0
What happened?
helm list -n kube-infrastructure list helmreleases that are not installed in kube-infrastructure namespace.
What you expected to happen?
helm list -n kube-infrastructure should list only helmreleases that are installed in kube-infrastructure namespace.
Enter the command that you execute and failing/misfunctioning.
helm list -n kube-infrastructure

Hello @stefanprodan, could you tag the other developers so that we could get some assistance here?

What is the storageNamespace and targetNamespace you have configured on the said release?

We do not have storageNamespace configured for our release.
example:
metadata:
name: rasa-x
namespace: kube-infrastructure
spec:
targetNamespace: mal-development

You need to set the storageNamespace to the same value as targetNamespace for Helm CLI to work, the Helm CLI is not as flexible as Flux and expects the storage and the release to be in the same namespace.


Added support for basic Matrices operation: addition

As well as provided structure for future Matrices operations to be added, this Closes #4

Structure looks good as of now


Fix 2292

the problem was that in the simple implied map
i = 1; // i goes into implied
let i; // this should error

var a = () => { i = 1; /* i goes into implied */ };
let i; // this should not error

there was no way to differentiate between the two situations.
So I took a leaf out of the blockscope manager and created one for implied that manages the different scopes.
But then as I've been doing this I've become unsure about the approach of the (blockscope) manager and implied manager - that logic is already present for var's in order to detect var's out of scope
function() {
    for(var i = 0; i < 8; i++) {
        var j = 1;
    }
    for(j = 0; j < 8; j++) {
    }
}

because even though j is function scope, jshint detects that j was not declared in the outer scope.
and it seems to me the logic should be combined. whats more I prefer the explicitness of the implied manager and blockscope manager (I think the logic outside of it is difficult to follow, with little overall explanation).
so @caitp @jugglinmike .. what do you think.. do you think the approach here is okay? which way should it be going?
Would you be happy if I continued refactoring and put implieds, blockscope and scope all into one "manager" with better names to test and poke on different scenarios ?

Coverage increased (+0.12%) to 96.49% when pulling 872dfbed6f171326225bcbbe871d565c644b15f0 on lukeapage:fix-2292 into 2b673d92107be93df80e48471dced0432a127f25 on jshint:master.

Coverage increased (+0.03%) to 96.5% when pulling 23bd529b585120ae044b1b895b3c3eacce98b5a4 on lukeapage:fix-2292 into c0edd9ff84a68946c615198a4a6f188f6fbbd54d on jshint:master.

@caitp @jugglinmike, now you've merged the other PR, would be good to get some discussion here. I think the best thing to do would be to discuss what should happen going forward so if I spend further time on this it will be accepted.

I think it's fine to fix this within some limits --- particularly, I don't want to go too crazy trying to enforce TDZ semantics that can't be determined statically. But the examples in the PR description don't seem unreasonable

I don't want to go too crazy trying to enforce TDZ semantics that can't be determined statically

I agree - determining whether or not a function is called before the declaration for instance would be going too far - right?
I'm most interested in your opinion on this:

Would you be happy if I continued refactoring and put implieds, blockscope and scope all into one "manager" with better names to test and poke on different scenarios ?

or do you think that whats ended up here is where things should be at?

Would you be happy if I continued refactoring and put implieds, blockscope and scope all into one "manager" with better names to test and poke on different scenarios ?

or do you think that whats ended up here is where things should be at?

The churn has probably died down for a while, so if we're going to do that, this is a good time to do it

Coverage increased (+0.03%) to 96.5% when pulling a8b55bb7f3dad22ffac5be8f59aa512ea0595f41 on lukeapage:fix-2292 into c0edd9ff84a68946c615198a4a6f188f6fbbd54d on jshint:master.

@caitp based on your last comment, is this ready to go?

Not sure if @lukepage wanted to work on it a bit more first or not.

Got it!

I am going to work on top of this, but I think its a big job and I don't know how much time I've got. I might have an update in a day or a week, or two. So, upto you.

Coverage increased (+0.03%) to 96.56% when pulling 1cd190239e7a867e9792e44785d3b0e66f8dcd9c on lukeapage:fix-2292 into e22b21a7bfeed9d1e38d0ed578c787aa9276690b on jshint:master.

Coverage increased (+0.03%) to 96.56% when pulling 1cd190239e7a867e9792e44785d3b0e66f8dcd9c on lukeapage:fix-2292 into e22b21a7bfeed9d1e38d0ed578c787aa9276690b on jshint:master.

Coverage increased (+0.03%) to 96.59% when pulling 76f376d1efcdf015db5f42d766bb8d7109e4ef36 on lukeapage:fix-2292 into 2ea9cb0f636cf005d48e53cc69b29175818f4ddf on jshint:master.

Coverage increased (+0.03%) to 96.59% when pulling 1c8294766dcb52c8a56012a1a2c75f2427378e14 on lukeapage:fix-2292 into 2444a0463e1a99d46e4afa50ed934c317265529d on jshint:master.

moved to #2435


[jdbc] define properties that indicate capabilities of DataSourceFactory

@timothyjward  properties like discussed in #418 ?

I also added some Spec-Text a and the implementation part.
test will be done in a separate PR


Echo During Call

Hey All,
I wanted to post and ask if anybody has experienced any echo during a call. If so, how did you resolve this? Is this plugin utilizing any of Twilios echo cancellation software? I have also submitted a ticket to Twilio asking about the echo cancellation software as well.
I'm utilizing this plugin for an Ionic 1 (specifically version 1.3.1) project to connect to Twilio to make phone calls. The plugin works great! Able to connect and everything is running smoothly on iOS and Android.
I have noticed during testing that this happens more on Android devices. Not sure if this is related.
If you need me to provide any more information please let me know. Thank you! :)

Hi,
I haven't heard any echos, but that doesn't mean that they wouldn't be there in your scenario. All of that would most likely be coming from the individual voice SDKs - the Phonegap plugin shouldn't be introducing any latency into the actual audio communications.
It could be something to be addressed in the Android Voice SDK, especially if you can make a scenario that reproduces it (for instance, over LTE or when you call a Skype number, or whatever else makes it happen).
Thanks for using the plugin!


balena-cli: 16.5.0 -> 16.6.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/balena-io/balena-cli/releases.
meta.description for balena-cli is: A command line interface for balenaCloud or openBalena
meta.homepage for balena-cli is: https://github.com/balena-io/balena-cli
meta.changelog for balena-cli is: https://github.com/balena-io/balena-cli/blob/v16.5.2/CHANGELOG.md
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 16.5.2 with grep in /nix/store/lc9vrnqmxm39ny3vpz1saxdvkfwxxyv8-balena-cli-16.5.2
found 16.5.2 in filename of file in /nix/store/lc9vrnqmxm39ny3vpz1saxdvkfwxxyv8-balena-cli-16.5.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
balena-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lc9vrnqmxm39ny3vpz1saxdvkfwxxyv8-balena-cli-16.5.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A balena-cli https://github.com/r-ryantm/nixpkgs/archive/e80dfd2549848e23717cfc56a3bd4e4119c23cf6.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lc9vrnqmxm39ny3vpz1saxdvkfwxxyv8-balena-cli-16.5.2
ls -la /nix/store/lc9vrnqmxm39ny3vpz1saxdvkfwxxyv8-balena-cli-16.5.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

balena-cli

Maintainer pings
cc @kalebpace for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/balena-io/balena-cli/releases.
meta.description for balena-cli is: A command line interface for balenaCloud or openBalena
meta.homepage for balena-cli is: https://github.com/balena-io/balena-cli
meta.changelog for balena-cli is: https://github.com/balena-io/balena-cli/blob/v16.6.2/CHANGELOG.md
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 16.6.2 with grep in /nix/store/wx84was03dn5ajfsh70rlsffsz8qm6bb-balena-cli-16.6.2
found 16.6.2 in filename of file in /nix/store/wx84was03dn5ajfsh70rlsffsz8qm6bb-balena-cli-16.6.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
balena-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wx84was03dn5ajfsh70rlsffsz8qm6bb-balena-cli-16.6.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A balena-cli https://github.com/r-ryantm/nixpkgs/archive/6bc31c600a1211450ef79f307070ba2006da616a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wx84was03dn5ajfsh70rlsffsz8qm6bb-balena-cli-16.6.2
ls -la /nix/store/wx84was03dn5ajfsh70rlsffsz8qm6bb-balena-cli-16.6.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

balena-cli

Maintainer pings
cc @kalebpace for testing.


Output .tar.bz2 artifacts

This configures the feedstock to output .tar.bz2 artifacts in addition to .conda artifacts.
We need this for cachine to work in CI pipeliens.
Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


⚠ Guya.moe Proxied has degraded performance

In d467f44, Guya.moe Proxied (https://ice.guya.moe/) experienced degraded performance:

HTTP code: 200
Response time: 11908 ms

Resolved: Guya.moe Proxied performance has improved in ac38ba6 after 1 hour, 19 minutes.


Adding ProviderName field so we can switch on clouds

Thinking it might be useful to have a configurable name for the cloud providers. This way we can easily switch on the cloud provider to serialize/deserialze the cloud provider to/from.
This would be really handy as we work on implementing this in kubicorn.

This change is

@justinsb @roberthbailey @pipejakob thoughts?

I think the provider name should be in the Spec but the constants should not be defined . There are too many of them to list in the api.

Having the names in the serialized provider config could be possible, but the entire point is too give the interpreting program an easy way to deserialize into a meaningful strict.

Sorry, I don't follow. I was trying to say that use ProviderName string .

@krousey

@kris-nova We've been using the approach of the ProviderConfig being an actual API type itself. Meaning that we decode based on the type meta in the blob like here: https://github.com/kubernetes/kube-deploy/blob/master/cluster-api/cloud/google/machineactuator.go#L316
Is that not good enough?

I have never used that before, but it seems like it should work. Does that support all the types? Amazon, Digital Ocean, etc?
Maybe I am just thinking about this wrong, but won't we have to have had the defined elsewhere? Also does anybody know of any docs or examples on using this with other clouds?
Thanks
Feel free to close if we really don't want the change.

@kris-nova @krousey @roberthbailey Is this still relevant? If not, could you please close it?

I'd propose that we close and instead rely on the provider string to include any such switch. If kubicorn wants to have a single config blob that supports multiple clouds, then that can be in the kubicorn versioned structure shoved into the provider config.

I'd suggest we add this to next week's meeting to discuss further.
A controller that supports multiple clouds - like we recently prototyped with Terraform - might not have a great experience to deserialize the provider string. My main concern is that the provider config becomes a super generic key/value string and no validation is possible until one deserializes, finds out the cloud, and then verifies every expected property for that particular property.
/remove-lifecycle stale

Why wouldn't the terraform provider config string look something like:
  "apiVersion": "terraformproviderconfig/v1alpha1",
  "kind": "TerraformProviderConfig",
  "cloud": "GCP",
  "terraform": ...

Where you just bump the raw terraform down one level instead of putting it at the root of the provider config? That allows you to add the wrapping metadata (api version, kind, etc) along with anything else that we find is needed in the future.

Given the direction we're moving in https://github.com/kubernetes/kube-deploy/issues/504, it will have the metadata necessary for the controller to deserialize and switch properly.
@rsdcastro validation is another concern and independent of specifying the cloud provider in the top level object or in the provider specific config. We can't bake provider specific validation logic into the API. That needs to be pluggable somehow through a webhook, admission controller, or some other mechanism. But it's a much larger discussion.

@roberthbailey I'd like to prototype how the validation process would look like to tell if any option might be cleaner or have clear advantages. Your suggestion is very fair, though, and that seems a reasonable way to go about this until we have more clarity on validation.
@krousey Validation discussion tracked at #623.

Please move the code or discuss to the ext-apiserver directory as we are targeting delete cluster-api directory soon.
Add a hold until it's finished.
/hold

One (maybe) related thing that I have found useful is to have two controllers running on one cluster (e.g. I have one for vmware and one for baremetal).  Something like this: https://github.com/kubernetes/ingress-nginx#running-multiple-ingress-controllers though I used a field for it in my hacking.

@justinsb - we've talked about the multiple controller use case from the beginning (I'd likened it to the multiple scheduler support, but the ingress precedence is good to see too). I don't think we've done anything to preclude that, have we?

@roberthbailey - right - we absolutely haven't done anything to preclude it.  We don't have a recommended pattern yet either (which is OK).  I don't know if people were thinking annotations or switching based on the type of the ProviderConfig (assuming it uses apimachinery).
I brought it up because I wasn't sure if the original ask would be satisfied if we had a clearer pattern for multiple controllers.

We had the same issue.
Currently our controller uses an annotation "machine.k8s.io/controller": "name"
The name can be specified via a -name flag.
https://github.com/kubermatic/machine-controller

Doesn't this change mean that adding a new provider would require core API changes and releases? That seems less than ideal. The API should have no knowledge of any provider at all, IMO.

I agree here with @karan .
The core API should be provider agnostic. Otherwise it gets overly complicated for others to write own implementations.
@kris-nova Our machine-controller does probably exactly what you want.
https://github.com/kubermatic/machine-controller
The providerConfig looks:
  providerConfig:
    cloudProvider: "digitalocean"
    cloudProviderSpec:
      token:
        secretKeyRef:
          namespace: kube-system
          name: machine-controller-digitalocean
          key: token
      region: fra1
      size: 2gb
      backups: false
      ipv6: false
      private_networking: true
      monitoring: true
      tags:
        - "machine-controller"

Or when using AWS:
  providerConfig:
    cloudProvider: "aws"
    cloudProviderSpec:
      accessKeyId:
        secretKeyRef:
          namespace: kube-system
          name: machine-controller-aws
          key: accessKeyId
      secretAccessKey:
        secretKeyRef:
          namespace: kube-system
          name: machine-controller-aws
          key: secretAccessKey
      region: "eu-central-1"
      availabilityZone: "eu-central-1a"
      vpcId: "vpc-819f62e9"
      subnetId: "subnet-2bff4f43"
      instanceType: "t2.micro"
      diskSize: 50
      diskType: "gp2"
      tags:
        "KubernetesCluster": "6qsm86c2d"

@mrIncompetent If the provider config works for you (and no concerns about validation, for example), it seems like we are converging that this param should not be at the top level.
In terms of usability, when we pick the provider/actuator, we are picking a "cloud". For an user it can be confusing to pick a top level "cloud" and then another "cloud" in the provider config. We might need to rethink and use a different term, so each implementation can target different cloud provider like Loodse does (otherwise). I filed #633 to track it. @karan
Unless there is any objection, the discussion so far indicates we can go ahead and close this PR. Waiting until end of day in case someone has any further thoughts.

/close


[1051] Fix: duplicate ids on radio inputs after validation error

Context
https://trello.com/c/Vh9IQurw/1051-duplicate-ids-on-errors-degree-grade-field-error-no-label-on-options-invalid-size-on-links-validate-html-with-validation-errors
Changes proposed in this pull request
Only apply link_errors: true to the first element. It's was only intended to be on the first.
Guidance to review

Load up the new UK degree page and submit empty form
Open the HTML inspector and confirm that only the first radio input element under "Degree grade" has the ID degree-grade-field-error. All the others are unique.

 Duplicate ids on errors degree-grade-field-error


Auto JSON decode

Hi there,
Firstly - great work on Flight - love the framework!
I have a problem with accessing json data from the body of a post request. As per the documentation and the code, if I set the content type of the request correctly, the body should be automatically parsed to json and the data property set.  However, unless I do this myself, it does not actually work like this.
To test/illustrate I added a simple router to my api with the following code:
Flight::route('/jsonissue', function(){
$flightData = Flight::request()->data;
$myOwnData = json_decode(Flight::request()->getBody(), true);
$testResponse['flight.data'] = $flightData;
$testResponse['own.data'] = $myOwnData;
Flight::json($testResponse);

});
I use runscope to test and here is the test request:

And then the response:

Hope this illustrates the problem?
is there some config setting I need to set/unset?
Regards
Heinz

It is because Flight::request()->data is not an array, but an object called a Collection, that lets you access data via array indexes or properties. If you want the raw data as an array you can do Flight::request()->data->getData().


ecwolf crashes if you try to load a pk3 that contains maps

Describe the bug
ecwolf crashes if you try to load a pk3 file that contains maps.
Steps To Reproduce
Steps to reproduce the behavior:

Make sure that you’re able to run the registered version of Wolfenstein 3D (WL6) in ECWolf.
Download the ECWolf Tech Demo (techdemo12.zip).
Extract techdemo12.pk3 from techdemo12.zip.
Run ecwolf <path-to-techdemo12.pk3>
If the “IWAD” selector appears, choose “Wolfenstein 3D (*.WL6)”.

Expected behavior
A new window with the “To be replaced with console…” screen appears. The game continues past that screen.
Additional context
Here’s what ECWolf outputs when it crashes:
$ ecwolf techdemo12.pk3
ReadConfig: Reading the Configuration.
IWad: Selecting base game data.
W_Init: Init WADfiles.
 adding /nix/store/7pqhqd9dcsilbxy44dk0v98lqpxiqddl-ecwolf-1.3.3/share/ecwolf/ecwolf.pk3, 72 lumps
 adding ./AUDIOT.WL6, 288 lumps
 adding ./GAMEMAPS.WL6, 120 lumps
 adding ./VGAGRAPH.WL6, 149 lumps
 adding ./VSWAP.WL6, 588 lumps
 adding techdemo12.pk3, 1685 lumps
InitGame: Setting up the game...
mapinfo.txt:4:1:Warning: Unknown gameinfo property 'signoncolors'.
mapinfo.txt:16:1:Warning: Unknown gameinfo property 'creditpage'.
mapinfo.txt:17:1:Warning: Unknown gameinfo property 'pagetime'.
VL_ReadPalette: Setting up the Palette...
ClassDef: Loading actor definitions.
S_Init: Reading SNDINFO defintions.
S_Init: Reading SNDSEQ defintions.
US_Startup: Starting the User Manager.
latchcfg.txt:0:0:Warning: Utilizing temporary status bar configuration script.
CreateMenus: Preparing the menu system...
double free or corruption (!prev)
Aborted (core dumped)
$

I wasn’t able to reproduce this bug in an Ubuntu VM using an ECWolf package from the ECWolf project itself.
Notify maintainers

@svanderburg
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.74, NixOS, 22.05 (Quokka), 22.05.3737.3933d8bb912`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(root): `"nixos-22.05"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

OK, so when I filed this issue, I misinterpreted ecwolf’s default.nix file, and, as a result, didn’t put the correct person in the notify maintainers section. I just updated the OP, but I don’t know if adding @svanderburg in an edit will actually ping him.


Quick fix for issue #71: remove '/usr/local/bin' from managed folders

Quick fix for issue #71
(duplicate declaration in PE v2017.2.1 puppet_enterprise/manifests/symlinks.pp:31)

@mcgege thanks for your PR!
@bitvijays that do you think?

@mcgege master is fixed now, could you please rebase our PR on the latest master?

@artem-sidorenko rebase done


fix: move test sources to the right places

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Weezing abilities

Hello, this references an issue over at Pokeapi opened by @SDM0. It seems that Weezing(110) has two abilities (Levitate and Neutralizing Gas) and one hidden (Stench). Currently, in Veekun data there is only Levitate.
I checked on Bulbapedia and Pokemon.com and the 3 abilities seem to be legit.
Do you think this data should be incorporated? Would you like a PR?

The Neutralizing Gas ability was added in Sword/Shield. Likewise, although Stench existed before then, Weezing and Koffing had no hidden ability before Sword/Shield.
So this is a duplicate of #284


Update: KDE.Dolphin version 22.12.1

Result: Installation Successful

Manifest
Add/Remove Programs

Name
Dolphin
Dolphin

Version
22.12.1
22.12.1

Publisher
KDE e.V.
KDE e.V.

ProductCode

Dolphin

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 5707
Microsoft Reviewers: Open in CodeFlow

@msftbot reset feedback

Close with reason: Cleanup of merge conflicts from fabricbot errors;


[WIP] SDF implementation for Python SDK + Python ULR

Please add a meaningful description for your change here

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

This has been superseded by other PRs.


Upgrades elm-test to the last version.

elm-test has been upgraded to the last version. Now it comes with cool colors!
Green tests

Red tests

@amitaibu maybe you want to have a review it.

This part in .travis.yml should probably change, as we rely on casperJs to check for us if the tests passed or not.
(The passing travis of this repo is of the yo generator, not the scaffolded one - which we have here

👍

Thanks. Published a new version in npm?
On Oct 31, 2016 5:31 PM, "David Hernández"<EMAIL_ADDRESS>wrote:

Merged #40 https://github.com/Gizra/generator-elmlang/pull/40.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Gizra/generator-elmlang/pull/40#event-842074015, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHrC4y1DmJ1oeFVNlee4OZPCqS1kORtks5q5gnYgaJpZM4KixrE
.

Yes, I published it as a major release (2.0.0) as the last Elm version will break the existing tests.

Thanks.
On Oct 31, 2016 6:31 PM, "David Hernández"<EMAIL_ADDRESS>wrote:

Yes, I published it as a major release (2.0.0) as the last Elm version
will break the existing tests.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Gizra/generator-elmlang/pull/40#issuecomment-257344140,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHrC4sMsDv5MeTxGxYHudmB3SM6XP9Wks5q5hfXgaJpZM4KixrE
.


CDI to 1.36.0 which and add support for DataSource

Signed-off-by: Michael Henriksen<EMAIL_ADDRESS>
What this PR does / why we need it:
Add support for CDI DataSOurce API.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

CDI to 1.36.0 with DataSource support

/hold

/retest

/hold cancel

/retest

/retest

/lgtm

/approve


Improve config-daemon logging

This PR aims to improve logs in config-daemon component, as:

utils.setSriovNumVfs(...): log  desired number of VF
utils.RunCommand(): avoid logging command twice, better print error.
Log daemon.setNodeStateStatus() in case of errors

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 14 (0.0%)  changed or added relevant lines in 2 files are covered.
14 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.2%) to 25.51%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/utils/utils.go
0
2
0.0%

pkg/daemon/writer.go
0
12
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/daemon/writer.go
1
0%

api/v1/helper.go
3
41.2%

controllers/sriovibnetwork_controller.go
4
65.35%

controllers/sriovnetwork_controller.go
6
65.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
1900

Relevant Lines:
7448

💛  - Coveralls

Thanks!
/lgtm

/lgtm

Can one of the admins verify this patch?

/test-all


Style/HashSyntax incorrectly output with UseHashRocketsWithSymbolValues option

Syle/HashSyntax cop incorrectly output with UseHashRocketsWithSymbolValues: true.
And despite the same cord, the results are different.

.rubocop.yml
Style/HashSyntax:
  Enabled: true
  EnforcedStyle: ruby19_no_mixed_keys
  UseHashRocketsWithSymbolValues: true
  PreferHashRocketsForNonAlnumEndingSymbols: false

Expected behavior
resources :photos, only: 'create'    # OK
resources :videos, :only => 'create' # NG
resources :books, only: :create      # NG
resources :movies, :only => :create  # OK

resources :comments, :shallow => true # NG
resources :messages, shallow: true    # OK

resources :users, only: 'create'     # OK
resources :groups, :only => 'create' # NG
resources :segments, only: :create   # NG
resources :units, :only => :create   # OK

Actual behavior
It is reported that some OK codes are incorrect and some NG codes are correct.
lib/style_hash_syntax.rb:4:20: C: Style/HashSyntax: Use the new Ruby 1.9 hash syntax.
resources :videos, :only => 'create' # NG
                   ^^^^^^^^
lib/style_hash_syntax.rb:5:19: C: Style/HashSyntax: Use hash rockets syntax.
resources :books, only: :create      # NG
                  ^^^^^
lib/style_hash_syntax.rb:9:22: C: Style/HashSyntax: Use hash rockets syntax.
resources :messages, shallow: true    # OK
                     ^^^^^^^^
lib/style_hash_syntax.rb:11:19: C: Style/HashSyntax: Use hash rockets syntax.
resources :users, only: 'create'     # OK
                  ^^^^^
lib/style_hash_syntax.rb:13:22: C: Style/HashSyntax: Use hash rockets syntax.
resources :segments, only: :create   # NG
                     ^^^^^

Steps to reproduce the problem
As above.
RuboCop version
$ rubocop -V
0.49.1 (using Parser <IP_ADDRESS>, running on ruby 2.4.1 x86_64-linux)

I believe this is due to the bug described by @jonas054 in #4229. Closing as a duplicate for now. Please follow that issue for updates. 🙂


⚠ Auth Portal has degraded performance

In 341c6ac, Auth Portal (https://auth.zyradyl.moe) experienced degraded performance:

HTTP code: 523
Response time: 17256 ms

Resolved: Auth Portal performance has improved in 2a2b1f0 after 5 minutes.


auto compile/interpret causes freeze

Challenge Name

https://www.freecodecamp.com/challenges/sum-all-odd-fibonacci-numbers
Issue Description
I missunderstood the task at first, so my function does some heavy calculation which freezes the browser due to the auto compile/interpret interface thats is provided  (chrome, ff, ie). (The code in a local file opened with any browser actually works, which shows a array containing many elements)
Since everything is frozen, I can't click on the reset button nor delete the code quickly enough to make to auto compile stop.
Everytime I open the challenge it just freeze instantly.
Is there a way to reset the code in this challenge so I can rewrite the code?
Screenshot

Here are two options:

Disable Code Auto-run
Clear localStorage.

Thanks and happy Coding!


[ISSUE]: Update failing when primary key is @PrimaryColumn instead of @PrimaryGeneratedColumn

Description:
When attempting to update an entity that has a PrimaryColumn decorator the update fails because it doesnt include the id of the entity in the mutation setting it to null. Tried using an interceptor to force the payload body to have he ID inferred from the path param but that resulted in a failed mutation too because it counted as a new insert that violates unique constraint.
Steps to Reproduce:
Define an entity with a PrimaryColumn primary key
Attempt to update
Expected Behavior:
Update successful and new entity returned.
Actual Behavior:
A 409 http error code exception is thrown that signals that the row's primary key is null
Screenshots:

Environment:
Database: Postgres SQL
Library Version: v0.12.0
Node version: v21.5.0

another side effect is updating an entity that has a nested entity or a nested entity array that has PrimaryColumn as primary key also doesnt update them

resolved

re opening, update still failing, you can try the example of OneToOne in typeorm docs https://orkhan.gitbook.io/typeorm/docs/one-to-one-relations , if you try to update the profile, it always fails because its inserting the userId as null, also tried using params option in the update route, still fails

hi, @chihabhajji
Can you please check if the issue you have is the following?

You are trying to change 2 entities in a relation through one api?

if it is true,
The currently implemented CRUD library is focused on an Entity.
You will be able to use it by separating endpoints per entity.
CRUD for relation entities is features to will be implemented in the future. (like https://github.com/woowabros/nestjs-library-crud/issues/419)
I don't have an immediate need for this feature, but if someone contributes it, I'll include it.

hello @jiho-kr , thank you for your swift responses as always.
Furthermore, i'm using seperate API's for the entities, here is my code
@Entity({
    name: 'users',
    schema: 'public',
})
export class User extends BaseEntity {
    @Index({ unique: true })
    @PrimaryColumn()
    @IsString({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.PARAMS, GROUP.SEARCH] })
    @IsOptional({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.SEARCH] })
    username!: string;

    @Index({ unique: true })
    @Column()
    @IsEmail(undefined, { groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.PARAMS, GROUP.SEARCH] })
    @IsOptional({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.SEARCH] })
    @ApiProperty({ type: String, description: 'Email of the user', format: 'email' })
    email!: string;

    @ApiHideProperty()
    @Exclude()
    @IsString({
        groups: [GROUP.CREATE],
    })
    @IsOptional({ always: true })
    @Column({
        nullable: false,
    })
    password: string;

    @IsEnum(ERole, { groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.PARAMS, GROUP.SEARCH] })
    @IsOptional({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.SEARCH] })
    @ApiProperty({ enum: ERole, description: 'Role of the user', type: String })
    @Column({
        type: 'enum',
        enum: ERole,
        default: ERole.USER,
        nullable: false,
    })
    role: ERole;

    @Type((_type) => Boolean)
    @ApiProperty({ type: Boolean, description: 'Is the user active' })
    @IsOptional({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.SEARCH] })
    @IsOptional({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.SEARCH] })
    @Column({
        type: 'boolean',
        default: true,
        nullable: false,
    })
    isActive!: boolean;

    @ValidateNested({ always: true })
    @Type((_type) => UserProfile)
    @Allow({
        always: true,
    })
    @OneToOne((_type) => UserProfile, (profile) => profile.user, {
        eager: true,
        nullable: false,
        cascade: true,
    })
    @JoinColumn()
    profile: Relation<UserProfile>;
}

@Entity({
    name: 'user_profile',
    schema: 'public',
})
export class UserProfile extends BaseEntity {
    @IsOptional({ groups: [GROUP.READ_MANY, GROUP.UPSERT, GROUP.PARAMS, GROUP.SEARCH] })
    @Allow({ always: true })
    @ApiHideProperty()
    @PrimaryColumn()
    @OneToOne(() => User, (user) => user.profile)
    user: string;

    @IsOptional({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.SEARCH] })
    @ApiProperty({ type: String, description: 'User avatar' })
    @IsString({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.PARAMS, GROUP.SEARCH] })
    @Column({ type: 'varchar', nullable: true, default: null })
    avatar?: string;

    @Column({ nullable: true, default: null })
    @IsString({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.PARAMS, GROUP.SEARCH] })
    @IsOptional({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.SEARCH] })
    lastName?: string;

    @Column({ nullable: true, default: null })
    @IsString({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.PARAMS, GROUP.SEARCH] })
    @IsOptional({ groups: [GROUP.CREATE, GROUP.UPDATE, GROUP.READ_MANY, GROUP.UPSERT, GROUP.SEARCH] })
    firstName?: string;

    constructor(partial: Partial<UserProfile>) {
        super();
        Object.assign(this, partial);
    }
}

@Crud({
    entity: UserProfile,
    routes: {
        readMany: {
            paginationType: 'offset',
            numberOfTake: 10,
            softDelete: false,
            sort: 'ASC',
            swagger: {
                response: UserProfile,
            },
            relations: false,
        },
        readOne: {
            params: ['user'] as (keyof UserProfile)[],
            swagger: {
                response: UserProfile,
            },
        },
        update: {},
        search: {
            paginationType: 'offset',
            numberOfTake: 10,
            swagger: {
                response: UserProfile,
            },
        },
    },
    only: [Method.READ_MANY, Method.READ_ONE, Method.UPDATE, Method.SEARCH],
})
@Controller('user-profile')
export class UserProfileController {
    // noinspection JSUnusedLocalSymbols
    constructor(private readonly crudService: UserProfileService) {}
}

and when i try to edit a user-profile entity

i get 409 error
{
  "query": "INSERT INTO \"public\".\"user_profile\"(\"user\", \"avatar\", \"last_name\", \"first_name\") VALUES (DEFAULT, $1, $2, $3) RETURNING \"avatar\", \"last_name\", \"first_name\"",
  "parameters": [
    "string",
    "string",
    "string"
  ],
  "driverError": {
    "length": 228,
    "name": "error",
    "severity": "ERROR",
    "code": "23502",
    "detail": "Failing row contains (null, string, string, string).",
    "schema": "public",
    "table": "user_profile",
    "column": "user",
    "file": "execMain.c",
    "line": "2003",
    "routine": "ExecConstraints"
  },
  "length": 228,
  "severity": "ERROR",
  "code": "23502",
  "detail": "Failing row contains (null, string, string, string).",
  "schema": "public",
  "table": "user_profile",
  "column": "user",
  "file": "execMain.c",
  "line": "2003",
  "routine": "ExecConstraints"
}

tried forcing the username by using this interceptor on the update
class A implements NestInterceptor {
    intercept(context: ExecutionContext, next: CallHandler<any>) {
        const r = context.switchToHttp().getRequest();
        console.log(r.params.user);
        if (r.params.user) {
            r.body.user = r.params.user;
        }
        return next.handle();
    }
}

but got the following error: 422
Error: Unprocessable Entity
{
  "message": "Cannot changed value of primary key",
  "error": "Unprocessable Entity",
  "statusCode": 422
}

Hello!
In UserProfile entity, you have applied both PrimaryColumn and OneToOne to the user property, with the type of user being a string instead of the related entity type.
I think this is not a normal use-case. - official docs
You should separate the relation field and the property field that manages the relationship to ensure TypeORM correctly handles the entity.
Like below...
export class UserProfile extends BaseEntity {
    @PrimaryColumn()
    userProfileName: string;

    @OneToOne(() => User, (user) => user.profile)
    user: User;
}

tried that i get the same error

tried that i get the same error

Is the custom interceptor disabled?
When I tested it the first issue, where insert query occurred instead of update, seemed to be resolved.

yes, i disabled the interceptor, i can do a minimal reproduction repo if you want

yes, i disabled the interceptor, i can do a minimal reproduction repo if you want

Yes, please share the modified code example again.


Package Upgrade

Describe the issue and expected behaviours. If this is a bug, describe the steps to reproduce the problem, what you observe, and what you expected to observe.
Acceptance Criteria

[ ]

Steps for QA:

No known build issues related to this upgrade, closing issue as fixed


Allow setting Influxdb TLS via config option

Currently the exporter cannot connect to Influx DB Database over https.

Thanks for the PR, looks good.


Speed up linting

Thanks to @ateliercw for the wonderful hack

Linting was taking about 8 minutes. With this workaround, it's now under 3 minutes. @ateliercw++ 🎉


Use modify!! from BangBang in GroupBy

Codecov Report

Merging #274 into master will decrease coverage by 3.42%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #274      +/-   ##
==========================================
- Coverage   89.29%   85.87%   -3.43%
==========================================
  Files          20       20
  Lines        1364     1260     -104
==========================================
- Hits         1218     1082     -136
- Misses        146      178      +32

Impacted Files
Coverage Δ

src/Transducers.jl
90.00% <ø> (-0.91%)
:arrow_down:

src/library.jl
93.35% <100.00%> (-0.89%)
:arrow_down:

src/interop/dataframes.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/basics.jl
35.00% <0.00%> (-31.67%)
:arrow_down:

src/simd.jl
78.12% <0.00%> (-21.88%)
:arrow_down:

src/core.jl
72.15% <0.00%> (-17.14%)
:arrow_down:

src/unordered.jl
79.59% <0.00%> (-16.71%)
:arrow_down:

src/interop/onlinestats.jl
70.00% <0.00%> (-10.96%)
:arrow_down:

src/lister.jl
80.48% <0.00%> (-7.32%)
:arrow_down:

... and 10 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 f5d8673...504be40. Read the comment docs.

It's somewhat below the noise on GitHub Actions https://github.com/tkf/Transducers.jl/pull/274#issuecomment-628388167.
In Travis:
              ["groupby", "sum", "sac"]              1.10 (5%) :x:    1.00 (1%)
     ["groupby", "sum", "xf-with-init"] 0.74 (5%) :whitecheckmark:    1.00 (1%)
  ["groupby", "sum", "xf-without-init"] 0.74 (5%) :whitecheckmark:    1.00 (1%)

https://travis-ci.com/github/tkf/Transducers.jl/jobs/333583419#L876-L878
On my laptop:
Before:
1-element BenchmarkTools.BenchmarkGroup:
  tags: []
  "sum" => 3-element BenchmarkTools.BenchmarkGroup:
          tags: []
          "xf-without-init" => Trial(309.752 μs)
          "sac" => Trial(150.147 μs)
          "xf-with-init" => Trial(296.652 μs)

After:
1-element BenchmarkTools.BenchmarkGroup:
  tags: []
  "sum" => 3-element BenchmarkTools.BenchmarkGroup:
          tags: []
          "xf-without-init" => Trial(201.071 μs)
          "sac" => Trial(149.924 μs)
          "xf-with-init" => Trial(193.529 μs)


g-l-s working with ycm+vim

Thanks for the useful software! So, I managed to get this working in vim using the ycm plugin. I added it to a repository which can set it up automatically. Just wanted to make you aware in case you wanted to link it somewhere in the project  :-) https://github.com/ycm-core/lsp-examples/pull/13

Nice! Thanks!


Drop-in UI Does Not Show Last Used Payment Method as Default

The drop-in interface should show the last used payment method when defaultFirst is false, but it doesn't.

To be more specific, I'm using the braintree drop-in in the following fashion:
`  function createBraintreeInterface() {
var payload = {
'_token': csrf_token,
};
return $.post('/get-braintree-client-token', payload, function(data) {
    clientToken = data.token;
    braintree.setup(clientToken, "dropin",
	{
	    container: "payment-form",
        defaultFirst: false,
	    onPaymentMethodReceived: function (paymentMethod) {
            // Ajax call to create the sale server-side in a unsettled state
	        createSale(paymentMethod.nonce, $('#amount').val()).done(function (result) {
                brainTree.teardown(function() {
                    createBraintreeInterface();
                });
	        });
	    },
        onReady: function(integration) {
            brainTree = integration;
        }
	});
});

}
`

Hi @directrix1, can you provide a description of the behavior you're seeing? When I tested this in sandbox, I wasn't able to duplicate.
The expected behavior is that a customer's payment methods are sorted with the most recently transacted upon payment methods at the top, with the rest of the payment methods sorted by when they were added.

It appears that it happens when selecting a different existing payment method (not entering in a new payment method). I've reproduced this in sandbox by:

Adding dummy mastercard and creating transaction with it
Adding dummy visa and creating transaction with it
Select dummy mastercard and create transaction with it
Upon going back, dummy visa is still selected in the interface by default.

@directrix1 Thanks for the info, I can duplicate this behavior. I'll look into this further.

Hi @directrix1,
After some investigation, I've found the root cause.
Each payment method has a timestamp that indicates when it was last used to create a transaction. A few months ago, we made a change so that these timestamps are only updated if more than 24 hours has passed since the previous timestamp. The purpose of this was to help optimize some database operations.
Braintree uses this timestamp to sort payment methods, so the side effect of this optimization is that there is a special case where if a customer creates a transaction with payment method A, creates another with payment method B, then creates yet another with A, and does this all within a 24-hour timespan, then their payment methods will not be sorted with the most recently used.
We will not be fixing this issue because this scenario is unlikely to occur in the real world. However, we will update our documentation to explain that the payment method sorting behavior may deviate in the aforementioned edge case scenario.
Please let us know if you have any concerns.

Yeah, that makes sense, and in the case of most apps will likely be irrelevant anyway. I was starting to wonder if this had something to do with the granularity of the last used timestamp, and I can see the value in that optimization.


Improve documentation on intervals

Fixes #1518, only merge once https://github.com/duckdb/duckdb/pull/9643 hits main.

@hawkfish is there anything else you'd like me to add to the documentation of intervals?

@hawkfish thanks!


Fix typos on oracle, redhat and centos

Thanks to @john-metcalfe-sbg for pointing this out

Released as: 11.1.11


Memory leak suspected

Watching a few test runs using top shows a growing demand on memory on my 8 GB SCRTP machine. I noted up to 70% of memory being used, slowly ramping up as the code proceeds.
I checked deletion of the trajectory container since that should be quite a big chunk of memory. That wasn't it. Explicit clean and delete of the container at the end of event changed nothing. Then I suspected the ROOT output since it doesn't appear to stream to disk but writes its substantial file size only at the end. I still suspect that to be a problem but mitigated by introducing that heavy trajectory filtering from the root script, creating much smaller file sizes. Something one should do anyway, I reckon. Still, memory use goes up and up to unacceptable levels. Something is not being cleared. Maybe a fresh pair of eyes can see things I miss.

I can take a look later this week. If you have time in the meantime, a quick run with valgrind might point to issues:

Configure the build using cmake -DCMAKE_BUILD_TYPE=Debug <otherargs> to ensure the correct flags are applied.
valgrind is installed in the LCG_97 release, so the memchecking tool can be run on the exe exactly as here: https://valgrind.org/docs/manual/quick-start.html

See also the full memcheck manual: https://valgrind.org/docs/manual/mc-manual.html
In case it needs debugging info from Geant4 itself, the debug variant of the LCG stack can be used. From a fresh shell, delete/recreate the build directory, and source:
$ . /cvmfs/sft.cern.ch/lcg/views/LCG_97python3/x86_64-centos7-gcc9-dbg/setup.sh

Note the dbg instead of opt at the end of the last directory name!
The just rebuild against that with cmake -DCMAKE_BUILD_TYPE=Debug <args> and make, then valgrind.

It has got a bit better, monitoring the memory behaviour. Nevertheless, here is the valgrind summary I get after 4 events (takes ages to run)
==15888== LEAK SUMMARY:
==15888==    definitely lost: 408,352 bytes in 3,350 blocks
==15888==    indirectly lost: 140,953,770 bytes in 3,066,071 blocks
==15888==      possibly lost: 62,780,307 bytes in 1,504,772 blocks
==15888==    still reachable: 100,809,320 bytes in 500,960 blocks
==15888==                       of which reachable via heuristic:
==15888==                         newarray           : 17,069,592 bytes in 5,679 blocks
==15888==         suppressed: 0 bytes in 0 blocks
==15888== Reachable blocks (those to which a pointer was found) are not shown.
==15888== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==15888==
==15888== Use --track-origins=yes to see where uninitialised values come from
==15888== For lists of detected and suppressed errors, rerun with: -s
==15888== ERROR SUMMARY: 732 errors from 731 contexts (suppressed: 0 from 0)
I am hoping this is fairly good news since none of the 732 errors seem to relate to the code. they all refer to building the physics table. Also the 'definitly lost' memory is quite small. Let me know if I didn't understand the output properly.

Thanks, that indeed looks like the fundamental leaks aren't in our code. I'll have a look tomorrow and see if I can track down any further. It sounds like you eliminated the possibility of the ROOT buffering/streaming being the problem.
Do you have any rough measure of the memory increase per event?

Just to update here, though #29 has the full details. Definitely something related to trajectory storage, as disabling this leads to a very stable memory use over long MT runs. With biasing there are of course a huge number of trajectories and hence trajectory points, and with the use of memory pools for speeding up allocation/deallocation, the memory used in these pools will always (I think) be enough to hold the largest number of objects seen to date.
Whilst that can account for a large fraction of the memory use, even clearing the highest capacity allocators between each event does not appear to drop the resident memory of the process. Some further investigation is still needed to check that the WLGD trajectory objects are free of leaks, and that they are being assigned/accessed to/from G4Tracks correctly. Also, the expected behaviour of the allocator memory pools should be confirmed (can ask in Geant4 about that).

It looks like it's also possible to only create trajectories for particles/regions of interest, see for example:
https://github.com/Geant4/geant4/blob/c02c370437c529eb0447aa14b6ee5a678fb0843d/examples/extended/runAndEvent/RE01/src/RE01TrackingAction.cc#L43-L56
Perhaps could adapt that only for neutrons/muons?

Sorry, forgot to comment on your last post. I tried that when implementing the RE01 example initially. It doesn't work for our use case.
Restricting trajectories to specific particles creates broken track histories (re-tracing the event does not reach the primary) since there can be quite surprising particles showing up intermediately (I've seen plenty of pions and Kaons and some other PDG codes). The muon energy simply is pretty high. On regions, we are interested in all regions/volumes where neutrons could be produced to see whether there is any pattern, any preferred volume/material which would have consequences on the experiment design. We can therefore not afford to restrict us to regions.


Show Licenses for Requirements.txt

Hi! I'm interested in being able to feed in a specific requirements.txt file and get the licenses for the packages listed in the file. There doesn't appear to be a CLI option supporting this. I'm curious -- does that seem like a reasonable feature? Thanks!
For reference, it could look something like:
pip-licenses --requirements-file=/path/to/requirements.txt

Most likely a duplicate of #108 as well.

Ah yup, I think this is a duplicate. Apologies as I couldn't find that, and didn't have a good understanding of how your library works (might be worth adding a brief How It Works section in the README?).
I am wondering if you've considered any alternate ways to get license info in the case where creating a venv is infeasible -- downloading the info from pypi or the package's repo possibly?

The repository already states:

Dump the license list of packages installed with pip.

and

pip-licenses is a CLI tool for checking the software license of installed Python packages with pip.

So it already states that it is about local packages.

I am wondering if you've considered any alternate ways to get license info in the case where creating a venv is infeasible -- downloading the info from pypi or the package's repo possibly?

I am neither the maintainer nor the author of this package, but just a user. Nevertheless, IMHO this is out of scope for pip-licenses as this basically requires a completely different approach (fetching and parsing JSON from PyPI etc.) I am not aware of a corresponding package which would already implement this.
Speaking of myself, I already use a combination of pip-licenses and custom scanning of packages without having to install them, although without resolving any dependency chains due to pip-licenses providing me all the necessary Python package names.

I can get the requirements.txt for a couple dozen repositories here but running all of them locally, installing all their packages and extracting the license information from there is a total non starter.
Weird that it's not possible to get this from requirements.txt or pyproject.toml.

OK. I've found that cdxgen works fine for my use case.


Upper bound chex for possible docs fix

[x] Closes #2131
[ ] Tests added and passed if fixing a bug or adding a new feature
[ ] All code checks passed.
[ ] Added type annotations to new arguments/methods/functions.
[ ] Added an entry in the latest docs/release_notes/index.md file if fixing a bug or adding a new feature.
[ ] If the changes are patches for a version, I have added the on-merge: backport to x.x.x label.

meeseeksdev backport to 1.0.x

meeseeksdev backport to 1.1.x


Complete C interface

Currently include/mammut/mammut.h is only a partial C interface to the mammut C++ code. In particular, it only exposes energy counters and energy measurements fonctions.
Topology, thread and processes management features are missing.
It would be helpful to build bindings to have the whole library exposed.
Thanks :)

Sure, actually it is quite straightforward but a bit time-consuming. At the moment I only added those for energy because I needed them for a python binding :) I'll probably keep adding them a few at a time.  If you would like to add some of the missing features I can provide you support.


Add More Test Cases

[x] for GithubContainsKeywordsPredicate
[x] for TelegramContainsKeywordsPredicate

Minor Tweaks

Codecov Report

Merging #272 (8cff508) into master (ad96684) will increase coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #272      +/-   ##
============================================
+ Coverage     69.77%   69.89%   +0.11%
- Complexity      860      866       +6
============================================
  Files           111      111
  Lines          3587     3587
  Branches        345      345
============================================
+ Hits           2503     2507       +4
  Misses          981      981
+ Partials        103       99       -4

Impacted Files
Coverage Δ

.../model/person/GithubContainsKeywordsPredicate.java
100.00% <0.00%> (+25.00%)
:arrow_up:

...erson/TelegramHandleContainsKeywordsPredicate.java
100.00% <0.00%> (+25.00%)
: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 ad96684...8cff508. Read the comment docs.