Fix tile refinement when leaf is empty

We got a support issue about a tileset that doesn't refine property when skipLevelOfDetail is false, which is the default since CesiumJS 1.67. The tileset has empty leaf tiles which exposed an edge case in executeEmptyTraversal in Cesium3DTilesetTraversal in which a tile with empty leaf descendants would not refine. The fix is to refine even if the tile would refine to empty leaf tiles. Note that this only applies to leaf tiles - if there's a tile chain that consists of a content tile, an empty tile, and a content tile, it will wait for the final content tile to load before refining the top-level content tile, which is the main purpose of executeEmptyTraversal- to prevent holes from appearing while tiles stream in.
Technically this fix could lead to situations where the parent will refine and there will be holes wherever there's empty leaf tiles, but I think that's acceptable and in the spirit of how the tileset is constructed.
Local sandcastle with the fix
Hosted sancastle

Thanks for the pull request @lilleyse!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Forum user confirmed this branch fixes their 3D Tileset generated by Context Capture: https://community.cesium.com/t/3dtiles-are-only-partially-displayed-with-skiplevelofdetail-disabled/10066/5?u=omar

@loshjawrence Can you please take a look at this?

Looks ok, waiting for ci to pass.


Toggle to switch DataFrame API to use dask-expr (but not as default)

There's been a lot of momentum around dask-expr recently and we posted already a couple of blog posts (introduction and tpch benchmarks) that show how this new approach can be transformative to the performance and usability of the dataframe.
The project is not quite feature complete but the core API is implemented and functioning well enough for us to make the first step and add this to the main project via a feature toggle.
Implementing a feature toggle like this presents us with a certain challenge from an implementation POV since we have plenty of submodules in the dask.dataframe module that will be sensitive to such a switch. Many functions that can be imported from those modules will likely not work on a dask-expr DataFrame.
One proposal would be to drop all submodules in favor of allowing only imports from dask.dataframe.__init__.py and renaming all submodules s.t. they have an underscore signaling it to be private API.
The feature toggle could then easily be implemented at this central place and we could introduce proper deprecation warnings.
xref https://github.com/dask-contrib/dask-expr/issues/374

This is the default and only option now now


wip: Implements LUD22 from LNURL spec

The extension LNURLpay has capacity to set a webhook, this pr just formalises by changing the workflow to LUD22 https://github.com/fiatjaf/lnurl-rfc/pull/122/files
A webhook can also be passed and honoured by lnbits wallet if an lnurlpay is paid by the wallet

Actually, wont work for replacing lnurlp webhook on setup, as payer specifies the webhook

Closing due to no activity. Feel free to reopen!


[Feature Request] Navigation Buttons?

Given that this is supposed to be client for players, a minimal navigation UI would be very helpful to reload the page and go back to login portal and whatnot.
Thanks for the project, it's gonna be a lifesaver for players with "wild browsers".

I'll look into it!

This is meant to mirror the foundry client experience, the lack of buttons is intentional. To go back to login you can just logout from inside foundry, and you can just ctrl+r to refresh. I like the design as is without buttons this is not a feature I’m willing to add/approve

The Problem is that you can't go back to the server selection screen after connecting once

I'm aware of that, but I don't want a navigation bar taking up the screen when restarting the client takes 1 second

There's already some js being injected by the client, it should be possible to inject some more and add that button into the normal foundry UI i think

Yes, but I inject only in the login screen, I don't want to inject in the in game interface

In that case I will maintain a fork with that functionality: https://github.com/OmegaRogue/fvtt-player-client/tree/fork

https://github.com/OmegaRogue/fvtt-player-client/releases/tag/1.3.3
This should work

back and forward, and reload button, there must also maybe be a way to get the navigation bar to auto-hide. sorry for late comment.

A "back" and "forward", and "reload" button, there must also maybe be a way to get the navigation bar to auto-hide. Sorry for late comment. i'll have a look at current releaseé

What would the purpose of forward and back buttons be? I can somewhat see the use of a reload button, tho you can just press F5 for that


hack spawn and spawnSync

cc @shaoshuai0102

spawn(options.execPath, args, options)
-> options.execPath include? process.execPath then options.env << ENCLOSE_IO_USE_ORIGINAL_NODE
spawnSync(command[, args][, options])
-> command include? process.execPath then options.env << ENCLOSE_IO_USE_ORIGINAL_NODE

Current coverage is 93.10% (diff: 100%)

Merging #10 into master will increase coverage by 4.31%

@@             master        #10   diff @@
==========================================
  Files             2          2
  Lines           116        116
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            103        108     +5
+ Misses           13          8     -5
  Partials          0          0

Powered by Codecov. Last update f3b81e3...90c0185


Remove (pre-Shelley) Cardano support

Description
Current Cardano version has no staking, removing recently-added support.  Next version (Shelley) is expected soon, with staking, will be added again.
Testing instructions
Unit tests.
Types of changes

Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] Prefix PR title with [WIP] if necessary.
[x] Add tests to cover changes as needed.
[x] Update documentation as needed.

probably we can quickly add new address format support? so we can keep the coin definitions

probably we can quickly add new address format support? so we can keep the coin definitions

Shelley address support is available on branch https://github.com/trustwallet/wallet-core/tree/ar/shelley , but even coin definition would be different, even SLIP-44 ID is different (1852 vs. 1815).
Do you suggest to do the remove of pre-shelley and add Shelley address support in one PR?

This has been replaced by other PR where Cardano/Shelley address support is also added, #845


Add new benchmark: Spanish bench

SpanishBench is a benchmark for tasks in Spanish that cover several evaluation areas. The datasets consist of professional translations of relevant English datasets and newly created datasets in Spanish. The README.md contains detailed information on all the tasks included in the benchmark.

 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.

@zxcvuser need little help for running pre-commit run --all-files and should be good.

Getting a evaluate/blue related error (not due to this PR):
AttributeError: 'DownloadConfig' object has no attribute 'use_auth_token'

Will fix the underlying cause before merging!

These are the changes done:

Added the benchmark info in lm_eval/tasks/README.md
Replaced "-" by "_" in the create_files script in flores_es and added weight_by_size: false
Run linters
Remove grouping in paws_es, xnli, and mgsm tasks (they were pointing to pre-existing benchmarks) and renamed the tasks with the suffix "_spanish_bench", both in each of the task yamls and in the spanish_bench.yaml
Changed "group" to "tag" in phrases_es

With these, it should all be fine now. Thank you!


荣耀android10,录像失败

必现, 调用cameraview.startVideoRecord(File(file), this),摄像头界面会卡主
这边在cameraview.stopVideoRecord()调用onCameraOpenError回调只返回了错误
2021-10-18 18:57:52.674 31196-31322/xxx E/libc: Access denied finding property "media.stagefright.extractremote" 2021-10-18 18:57:52.736 31196-31409/xxx E/ion: ioctl c0044901 failed with code -1: Not a typewriter 2021-10-18 18:57:52.767 31196-31322/xxx E/libc: Access denied finding property "media.metrics.enabled" 2021-10-18 18:57:52.772 31196-31322/xxxr E/libc: Access denied finding property "media.stagefright.extractremote" 2021-10-18 18:57:52.823 31196-31322/xxx E/libc: Access denied finding property "media.metrics.enabled"


PAIs can have micro bomb implants

Description

PAIs can have microbomb implants implanted into them and they can trigger them.
Reproduction

Be PAI
Have your owner implant you with a microbomb implant
Accidentally gib your owner
Screenshots

N/A
Additional context

N/A

dupe

dupe

Where? I did a quick search of pai issues before making this. Nothing about pai getting micro bombs.

same cause as #16900, not checking whitelist before implanting (afaik only dna scrambler has a whitelist)

Ok then.


[eslint-plugin-lit-a11y] change default to not check lit-html imports by default

TL;DR:
We should make it the default to always lint any html tagged template literal. If people use multiple rendering libraries in their projects, they should opt in to only check lit-html imported tagged template literals, by doing either: "litHtmlSources": true or "litHtmlSources": ["ing-web"] in the eslint settings**
Details:
We currently are able to check if a html tagged template literal is actually a lit-html tagged template literal by checking the imports
We have several options regarding this:

By default, we allow lit-html to only be imported from lit-html and lit-element, but it could be the case that a package re-exports lit-html, like ing-web. In this case, ing-web can be added as a setting as “valid lit-html import”, like so: "litHtmlSources": ["ing-web"]
We can disable checking for a valid lit-html source, and just lint any html tagged template literal, like so: "litHtmlSources": false

The reason we have the check for a valid lit-html import, is because people may use multiple rendering libraries in a project, like both lit-html and htm in the same project. We only want to lint the lit-html tagged template literals in that case, and not htm templates because they use different syntax
The default we currently have is to only lint html tagged template literals imported from either lit-html or lit-element. If people dont import from lit-html or lit-element, they have to add an option:
  "eslintConfig": {
    "extends": [
      "eslint-config-ow",
      "plugin/lit-a11y:recommended"
    ],
    "settings": {
      "litHtmlSources": ["ing-web"]
    }
  },

I think we should make the default to completely disable checking for a lit-html import, then implementing projects only have to do:
  "eslintConfig": {
    "extends": [
      "eslint-config-ow",
      "plugin/lit-a11y:recommended"
    ],
}

People who use multiple rendering libraries in their projects should opt-in to the behavior of only checking lit-html tagged template literals
I think disabling the lit-html import makes it easier for more people to add the plugin to their project, than there are people really using multiple rendering libraries in their project
Todo:

[ ] Make linting any html tagged template literal the default
[ ] Update the docs in the packages main README.md

this was fixed in #1936


Proposal: Create our own toast library (@threefold/vue-toast)

@threefold/vue-toast
Issue
Basically, We were looking for a good toast (Aka. Notifications) library out on npm/github and We hit some great options but sadly they hardly support any kind of global installed plugins/framework; for example vuetify.
Proposal
Create our own toast library with a solid under the hood functionality with a very clear and easy to use api (Aka. interface).
How
Well, Yesterday I came up with an idea (hope it's clever enough)

useToastPlugin
As the name suggest this method should be used while creating a new vue app

// main.ts
import { createApp } from "vue"
import {  useToastPlugin }  from "@threefold/vue-toast"

import App from "./App";

createApp(App)
  .use(useToastPlugin(/* Some options will be discussed below */))

  .mount("#app")

useToast
This one is all about notifying the users (Aka. showing toast/notification/popups or whatever you called it)
Basically, This is our api/interface for our toast library and it should only include a single method for now called notify
import { useToast }  from "@threefold/vue-toast"

<script lang="ts">
export default {
  name: "WhatEver",
  setup() {
    const toast = useToast()
    toast.notify(/* should be tons of options here will be discussed below */)
  }
}
</script>

// Global plugin options
// These options should be global and same options should be exported in `useToast` and use should
// be able to override it while opening any toast

import type { Component, Plugin } from "vue"

export interface ToastPluginOptions {
  timeout?: number // 3_000
  component?: Component  // Default component to be used while opening a new notify
                         // This component will get an options prop with some configuration
                         //  Will be discuss below
  plugins: Plugin[] // Array including all plugin that should be register in plugin to be able to use component
              // For example to use `VAlert` from vuetify you will need to pass `[vuetify]` same as your app

  // There will be more options later after discussing with our team...
  showProgressBar?: boolean
  backgroundColor?: string
  // ...etc
}

Note: For toastOptions should be the same as plugin and will override it when calling notify
Flow
Flow is how to library parts should connect to each other and this flow is my own thought neither standard nor the only way.

Note: We can have as many app as we need

Create another app while installing our plugin using useToastPlugin([options: ToastPluginOptions]): Plugin
Configure this app with plugins and values passed in ToastPluginOptions (If the user passed any)

Note: This app is just a regular vue component so we can have state/expose/inject/provide, ...etc

So, We can expose a global object for example { notify([options]): void }
Then,  We expose a global object with our own spec key @threefold/vue-toast in user passed app
This object should include a notify method { notify([options]): void } this method
So Whenever the user call notify which injected in app1 we can access app2 instance and call the exposed notify with same options after merging both options objects together
And we're done with it that should be it as should works as we expect

Conclusion
This approach might seem to be very hard and unclear at first but once it's ready to use it became a magic with tons of feature and options.

Implemented in https://github.com/mohamedElmdary/vue3-notifier


add a guide how to declare plugin typings

I'm seeing some users struggling how to declare typings for plugins (e.g. https://github.com/vuejs/vue-class-component/issues/128), especially the plugin that augments instance property/component options. I think it can be tricky because we have to use a feature that most of ts users may not know.
This PR adds an example code to do that. I think it would be helpful not only for plugin developers but also users that uses a plugin not include typings since they can know how to make it by themselves.

@ktsn Looks great! I just made some small tweaks to the wording.


Error when publishing/installing NAV Extension

I am trying to publish an app with VS Code to a local NAV Server and get the following error
"The request for path /KronanDataImport/dev/apps?SchemaUpdateMode=recreate failed with code InternalServerError. Reason: Requested value 'Blob' was not found."
However the Extension is visible in the client but when I try to install the following error is displayed:

I was previously able to publish and install the app on the same NAV Server, I have also restored another clean database but the error also occurs there.
Any ideas what might be causing this?

Please follow the issue templates


can not find gpu with linux system

I can find gpu in window system, after install linux system in same computer, I can not find any gpu, what happened?

Could you please install the gpu driver following this document? https://ipex-llm.readthedocs.io/en/latest/doc/LLM/Quickstart/install_linux_gpu.html#install-gpu-driver

@sgwhat

If the driver you post can support those package and oneapi?

By the way how can I know my Linux kernel?

By the way how can I know my Linux kernel?

uname -r


Allow double jump

Allow the player to jump once more by pressing space again before hitting the ground

/assign

Thanks for your interest. Looking forward to your PR!


bootstrap-rtl min is missing navbar-right

In minified file off bootstrap-rtl there is no navbar-right class
.navbar-right {
    float: left !important;
}

if you are using a module loader you need to import "bootstrap-flipped.css", then it will work
I faced the same


Hide avatar tooltips unless you're looking at your own profile

This hides the tooltips and links to configure your profile picture, unless it's your profile. This also links directly to the profile image, similar to how GitHub does, if you're looking at another user's profile.
Also included a commit to change the size of the resized uploaded avatars, from 200px to 234px, like was talked about in issue #1107. But not sure if this is how you wanted this fixed!

Looks good, thanks!

https://try.gogs.io/unknwon is updated as well. :grinning:

@Unknwon not sure why I get an error on that new statement on that link. Obligatory "works on my PC." :P

There is a template error, has pushed fix

Thanks, I thought I had to compare IDs not names.

Normally, yes, compare ID.
But when there is no logged in user, .SignedUser is nil and Go template cannot compare with it. So it panics


Using XYPieChart inside CollectionViewCells

Great project!
I am trying to add XYPieCharts inside each of my UICollectionViewCell, and so far implemented the datasource and delegate methods inside my UICollectionViewCell.m file (should I be doing it this way?). The slice of each pie is supplied in the UICollectionViewController method: - (UICollectionViewCell *)collectionView:(UICollectionView *)collectionView cellForItemAtIndexPath:(NSIndexPath *)indexPath.
However, I am running into the issue that none of the pies are showing. Am I correct in setting the datasource and delegate inside the cells as opposed to the view controller? What is the recommended way to doing this?
Thanks!

I'm also experiencing this issue, seems like a bug.


Does it Work with Tibia 11?

does it work with tibia 11 ?      or will it be developed for it ?

Mytherin:

The beta client has a different internal structure than the regular client, and Tibialyzer will have to be patched to work with it.
I have done some digging into the memory of the client and have found that strings are stored as follows.
struct {
short int length;    // 2 bytes indicate length of string
char string[length]; // the actual string
};
The strings aren't null-terminated, thus the length must be used to figure out when the strings end.
The main problem is that log messages are split up into several separate strings. Chat messages, for example, are split up into the separate strings Name and Message (possibly also Time and Level).
That means that to get a complete log message we need to understand how these different strings are combined, which means we need to reverse engineer the structure of the server log. I haven't yet found out how they are tied together. I will need to dig deeper later.

Can u somehow encapsulate that part of the problem if u figure it out.  so that its not visible anymore.. so that botters have a hard-time figuring out how to come back after that update?.

Botters need to figure out something entirely different from what I need to figure out. I don’t think bots even use the chat log at all. They need to figure out how to find the exact positions of creatures, their life, how many/which creatures there are, the position of the player in the world, etc. They don’t really need the chat log for botting, while that’s the only thing I need for Tibialyzer.

Stuff like heals bots.. can use the chat log. a fully functional bot is a different thing.

Don't mean to rush, but has there been any progress to this? Since Tibia 11 is now in open beta I'd imagine it's not too far away from release.

Yes, I have done some additional exploratory work on Tibia 11. I now know how to read the chat logs but haven't looked into other stats yet (life/mana/etc), and I still need to implement it in Tibialyzer as well. When Tibia 11 releases I will release a Tibialyzer that works with it unless CIP massively overhauls the internal structures, although I can't promise that all features will work with it (e.g. HUDs).

@Mytherin  Could you release an unsupported version to Tibia 11 with only looting stuff? I mean, that was originally the first step for tibialyzer and I guess it's the most relevant function until today. Some feedback could also help you when Tibia 11 finally arrives

@gabrielrdrguez Done.

@Mytherin Thanks man! You're awesome, I gonna try it and give you all the the feedback that I can provide.

Trying to use the version 1.4.2 with Tibia 11 but I just can't find a way to see the summary window while playing =/.


How to run a container as a specific user? (Equivalent of docker run --user)

I wish to run a container as a specific user on my system. In the command line I can do:
docker container run --user 1000:2000
Where 1000 is the user id and 2000 is the group id.
How do I accomplish this with the Portainer GUI?
PS. I have searched for this, but I'm afraid that my search terms are too broad and nothing relevant came up.

@janosrusiczki
I wanted to echo the response that was shared in our Community Slack:  https://join.slack.com/t/portainer/shared_invite/zt-190q9nm5v-uT0p0WFkn~K1yv2QlnHJNQ
You would enter the User in the User Field:

Thanks!


[0.8] Shady style processor doesn't drop operator for ::content

Shadow land
In shady land the ::content pseudo element selector is simply dropped, but it's valid to treat it as a real element in selectors and surround it with operators like >:
#main > ::content [core-drawer-toggle]

  <content></content>

  <button core-drawer-toggle>Toggle drawer</button>

That should be "select all [core-drawer-toggle] that is a descendant of a <content>, which is a child of #main". The distributed button is selected in the above case when ?dom=shadow is enabled.
Shady land
In shady land the ::content is just dropped:
Actual:
#main > [core-drawer-toggle]

Now the selector expects a different DOM structure: "select all [core-drawer-toggle] that are children of #main". The button in the above example is no longer selected.
Expected is to drop the operators surrounding ::content:
Expected:
#main [core-drawer-toggle]

Thx for the clear write up and suggestion. Will make it so.

Cool thanks. I think operators on both sides have to be dropped and replaced with a single descendant selector to match shadow DOM as closely as possible.. For example:
/* Yes this is _weird_, but the selector works with real shadow DOM */
#main > ::content + [core-drawer-toggle]

should become:
#main[style-scope] [core-drawer-toggle]

Well that replacement will screw up in odd cases, but that might be close enough.

My example might be invalid because the recommendation is to wrap a <content> element and make it the only child of that wrapper.

With the above change, these selectors should now be shimmed as follows:
a <left-separator> ::content <right-separator> b

a <right-separator> b

This should ensures that:
// selector
#main > ::content [core-drawer-toggle]
// shimmed selector
#main  [core-drawer-toggle]

// selector
#main > ::content > [core-drawer-toggle]
// shimmed selector
#main > [core-drawer-toggle]

// selector
#main ::content > [core-drawer-toggle]
// shimmed selector
#main > [core-drawer-toggle]

We also have an idea to improve shimming of ::content selectors that involves specifically tagging distributed elements.

:beers: Thanks, @sorvell.


After first time closing cannot reopen

Hello
First thank you for your module.
Issue
After clicking on a journal entry with an embedded url it works perfectly fine.
When closing and reopening then it won't show up.
As a GM
Reaplying the url and then it works again.
Selecting another journal entry then it works again.
As a Player
Only F5 works

could you maybe send me the url and the settings you have enabled when exporting to journal?
if it's a sensitive url feel free to dm it to me on discord! ardittristan#0001

This was a problem with Avast antivir.


Issue with testing after upgrade

Hey!
It seems like testing system changed and there's no anymore
import initializeTestHelpers from 'simple-auth-testing/test-helpers';
initializeTestHelpers();

....
And I have very strange error in logs:
ot ok 5 PhantomJS 2.1 - Acceptance | Agency Visiting: visiting agency overview
    ---
        actual: >
            false
        expected: >
            true
        stack: >
            http://localhost:7357/assets/test-support.js:2:6365
            exception@http://localhost:7357/assets/vendor.js:18:28516
            s@http://localhost:7357/assets/vendor.js:16:23816
            trigger@http://localhost:7357/assets/vendor.js:23:9998
            http://localhost:7357/assets/vendor.js:23:22775
            invoke@http://localhost:7357/assets/vendor.js:7:50013
            flush@http://localhost:7357/assets/vendor.js:7:50542
            flush@http://localhost:7357/assets/vendor.js:7:47594
            end@http://localhost:7357/assets/vendor.js:7:55345
            run@http://localhost:7357/assets/vendor.js:7:56482
            l@http://localhost:7357/assets/vendor.js:13:61764
            h@http://localhost:7357/assets/vendor.js:18:58186
            http://localhost:7357/assets/vendor.js:19:3438
            u@http://localhost:7357/assets/vendor.js:19:6861
            http://localhost:7357/assets/vendor.js:19:8871
            y@http://localhost:7357/assets/vendor.js:23:175
            b@http://localhost:7357/assets/vendor.js:23:270
            g@http://localhost:7357/assets/vendor.js:23:78
            http://localhost:7357/assets/vendor.js:16:47792
            invoke@http://localhost:7357/assets/vendor.js:7:50013
            flush@http://localhost:7357/assets/vendor.js:7:50542
            flush@http://localhost:7357/assets/vendor.js:7:47594
            end@http://localhost:7357/assets/vendor.js:7:55345
            run@http://localhost:7357/assets/vendor.js:7:56482
            l@http://localhost:7357/assets/vendor.js:13:61764
            http://localhost:7357/assets/vendor.js:18:59680
        message: >
            TypeError: undefined is not a constructor (evaluating 'e.transitionToRoute("login")')
        Log: |
    ...

Here's my code
import Ember from 'ember';
import { module, test } from 'qunit';
import startApp from 'app/tests/helpers/start-app';
import { currentSession, authenticateSession, invalidateSession } from 'app/tests/helpers/ember-simple-auth';

module('Acceptance | Agency Visiting', {
  beforeEach: function() {
    this.application = startApp();
  },

  afterEach: function() {
    Ember.run(this.application, 'destroy');
  }
});

test('visiting agency overview', function(assert) {
  authenticateSession(this.application);
  visit('/agency');

  andThen(function() {
    assert.equal(currentPath(), 'agency.index');
  });
});

// start-app
import Ember from 'ember';
import Application from '../../app';
import Router from '../../router';
import config from '../../config/environment';
import './click-on';
import './assert-page-includes';
import './assert-page-not-includes';

export default function startApp(attrs) {
  var application;

  window.confirm = function() { return true; };

  var attributes = Ember.merge({}, config.APP);
  attributes = Ember.merge(attributes, attrs); // use defaults, but you can override;

  Ember.run(function() {
    application = Application.create(attributes);
    application.setupForTesting();
    application.injectTestHelpers();
  });

  return application;
}

Do you have any idea what's wrong here?

I have a similar issue, I'm assuming the session is not authenticating despite calling authenticateSessoion(this.application) and thus the test is failing.

Agreed, attempt to transition to "login" is clear hear. However, I still think I miht have done something wrong
@marcoow can I have a few minutes of your precious time ?

@Blackening999: the code looks fine actually. Could you try to setup a small demo project that shows the same behavior?

Hey @marcoow
You've answered just in time I've found the issue :)
The issue were in the way I'm populating the user.
populateCurrentUser() {
//fix
    if (config.environment === 'test') {
      this.get('currentUser').set('content', this.store.findRecord('user', 'me'));
      return Promise.resolve();
    } else {
      return this.store.findRecord('user', 'me')
        .then(user => this.get('currentUser').set('content', user) && user, () => {
          this.get('session').invalidate().then(() => {
            const currentUser = this.get('currentUser');

            if (currentUser) { currentUser.set('content', null); }
            this.store.unloadAll('user');
            this.transitionToRoute('login');// <<< route of evil
          });
        });
    }

for some reason, when I'm running $ ember test, in TEST env ember don't recognise "this".
I've found another places proving that.
After I've fixed that - everything worked.
Thanks for feedback!


nixos/lxd-agent: add system path for exec

Description of changes
Allow commands run through incus exec to find the system path.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@ofborg test lxd.container incus.container

Backport here https://github.com/NixOS/nixpkgs/pull/277413


elastic/filebeat has not been pushed to the helm repo

helm repo add elastic https://helm.elastic.co
helm repo update
helm install --name filebeat elastic/filebeat --version 7.0.0-alpha1

Error: failed to download "elastic/filebeat"

This chart is still a WIP but will be pushed to the helm repo during the next release. https://github.com/elastic/helm-charts/pull/121 is the current PR that is putting the finishing touches on it. If you wanted to already play around with it you could install it directly from that branch.
In retrospect merging in the docs with those instructions is a bit confusing for anyone browsing the charts from GitHub. Sorry about that!

Meanwhile, got it to work by installing the chart from master. However, I had to comment out the livenessProbe that was killing the containers after 30s...
Monitoring on port 5066 is an X-Pack only feature, isn't it ?

Meanwhile, got it to work by installing the chart from master. However, I had to comment out the livenessProbe that was killing the containers after 30s...

Yup this was already fixed in the pull request/branch that I recommended you install from. This pull request has been merged so it is also fixed in master.

Monitoring on port 5066 is an X-Pack only feature, isn't it ?

Nope, it is also available in the oss release. There is an oss example which is also part of the automated testing for this chart to confirm that this setting and other future changes remains compatible with all versions and license types.  https://github.com/elastic/helm-charts/tree/master/filebeat/examples/oss

This was released in 7.1.0: https://github.com/elastic/helm-charts/releases/tag/7.1.0


Tokenizer returns empty string for the last line

Hi 👋  I'm using pegen for parsing a Python-like language, encountered something unexpected, and wondering if anyone can take a look.
Problem
It seems that pegen.tokenizer.Tokenizer stores the empty string for the last line of the source if the source does not end with a NEWLINE.
import io
import tokenize

from pegen.tokenizer import Tokenizer

source = "line 1\nline 2\nline 3"

print(source)

tok_stream = tokenize.generate_tokens(io.StringIO(source).readline)
tokenizer = Tokenizer(tok_stream)

# load all tokens
while True:
    tok = tokenizer.getnext()
    print(tok)
    if tok.type == tokenize.ENDMARKER:
        break

print(tokenizer._lines)

print("last line: ", tokenizer.get_lines([3]))

I expect the last line to print ["line 3"] but it prints [''].
After some investigation, I found that the empty line is coming from NEWLINE tokens generated by Python's tokenize module.
If the source does not end with a NEWLINE, tokenize injects one. For example, tokenizing the above input would produce
TokenInfo(type=1 (NAME), string='line', start=(1, 0), end=(1, 4), line='line 1\n')
TokenInfo(type=2 (NUMBER), string='1', start=(1, 5), end=(1, 6), line='line 1\n')
TokenInfo(type=4 (NEWLINE), string='\n', start=(1, 6), end=(1, 7), line='line 1\n')
TokenInfo(type=1 (NAME), string='line', start=(2, 0), end=(2, 4), line='line 2\n')
TokenInfo(type=2 (NUMBER), string='2', start=(2, 5), end=(2, 6), line='line 2\n')
TokenInfo(type=4 (NEWLINE), string='\n', start=(2, 6), end=(2, 7), line='line 2\n')
TokenInfo(type=1 (NAME), string='line', start=(3, 0), end=(3, 4), line='line 3')
TokenInfo(type=2 (NUMBER), string='3', start=(3, 5), end=(3, 6), line='line 3')
TokenInfo(type=4 (NEWLINE), string='', start=(3, 6), end=(3, 7), line='') 👈
TokenInfo(type=0 (ENDMARKER), string='', start=(4, 0), end=(4, 0), line='')

The second-to-last token does not exist in the original source but was added by the tokenizer. The problem is that this injected token does not have line (which on its own is reasonable).
https://github.com/we-like-parsers/pegen/blob/fab0c5b012836fcba07c1c1c828874745c8a4bfd/src/pegen/tokenizer.py#L59
Pegen's tokenizer uses this to cache the text on that line. So after execution, tokenizer._lines is set to {1: 'line 1\n', 2: 'line 2\n', 3: '', 4: ''}.
Possible Fixes
If this is an unintentional behavior, a possible fix I can think of is to update _lines only if the same line has not been set.
I've added a test case, implemented this change, and it seems to be working.
https://github.com/we-like-parsers/pegen/compare/main...shumbo:pegen:fix-last-line?expand=1
Is there anything else to consider, especially around the condition of the if statement? I can create a PR if this may benefit someone else.

That does sound like a bug. Could you open a PR with your fixes ?

@MatthieuDartiailh Thanks for your reply. I opened a PR with my fix. #77
I'd appreciate it if you can take a look.


Don't duplicate some mod actions in room modlogs

globalModlog writes to both the global modlog and the room one. Calling modlog in addition to this means stuff is logged twice.
I suppose modlog can have some slightly different behavior with the noip ad noalts options, but we have been logging the version with IP and alts in the roomlog for years as well now so this doesn't introduce any new behavior. This is also only for stuff that's globally relevant enough to get put in the global modlog, so incuding alts/IPs should not be an issue.

Looks good to me. @bumbadadabum, how do you feel about including alts/IPs?

I personal think that while leaking IPs and alts to non staff is a serious issue, this is probably not the place to address the issue. Better to solve this in part of the modlog parsing code.

So yeah seems fine

We already filter out all IPs for non-global staff in the modlog command. But we can't really filter out all alts there, because for some punishments, like roombans, roomstaff should be able to see them.


Implement proper field of view algorithm for vision

Depends on #27

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
Congratulations! You've won a share of the $500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

Don't miss out on your $1500 in BNB! Act now to secure your tokens.
Winners: @nmartineli, @FelipeBurg, @lscalado, @FelipeSD, @italo-mauricio, @jGledsonSilva, @clasenback, @henry235, @JoaoCamara13, @ericklpc, @pdecastilho, @gukiboy, @Lukanny, @leo-ribeiro, @vinnyalb, @gustavo-angeli, @DarioJunior, @hiago-balbino, @renatogcruz, @thephantomscar, @Guimatsu75, @mssantos, @tgn76ok, @luizfernandonb, @jsiloto, @fharaujo, @lukaz914, @EvertonFerreiraa, @lucasgstumpf, @Jef1-Dev, @Korothy, @anaelisaq, @KaDelanegri, @nelsondevsquad, @jfilipedias, @brianzzs, @math-reis, @liaverso, @maribeiromendes, @FragosoDev, @acgoularthub, @MarioViniciux, @dudubass15, @cleberjperi, @kalanedev, @LemonNTJ, @deriknyvo, @renanalencar, @RenanJochem98, @Trokke


Include section about replicating old asp.net signal QueryString functionality

Adding parameters at the end of the connectionBuilder .withUrl("/transportHub?param=foobar") like this, and accessing it in the Hub OnConnectedAsync override like this Context.GetHttpContext().Request.Query["foobar"].

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

ID: de8cdc99-48bb-3266-05bb-eda98f59f2e2
Version Independent ID: 66d252c3-6300-7e28-9aa0-ef2ff66e7a76
Content: ASP.NET Core SignalR configuration
Content Source: aspnetcore/signalr/configuration.md
Product: aspnet-core
GitHub Login: @rachelappel
Microsoft Alias: rachelap

@BrennanConroy  please comment

Maybe we should add a section in the hubs doc for the Context object, similar to the Clients one https://docs.microsoft.com/en-us/aspnet/core/signalr/hubs?view=aspnetcore-2.1#the-clients-object

@tomzorz Thanks for the suggestion. PR #8486 adds a Context property section to the doc, including a note on how to get query strings.

@tdykstra excellent, thank you! :)


The possibility to change the text of the items for "Available actions" into icons.

Checklist

[x] I checked for similar existing requests (both open and closed) before posting.
[x] My request is generic, other users may benefit from it too.

Proposal
The possibility to change the button text of the items for "Available actions" into icons.

Additional info
You can choose to use text or only an icon

Because i only want to use an icon instead of text.

I think this is for alarmo-card?

Oops  sorry. You're right. I'm gonna post the feature_request on alarmo-card.
You can delete this feature_request.


feat(user_profile_form): Add photo size validation

Types of changes
Thanks for sending a pull request! Please fill in the following content to let us know better about this change.
Please put an x in the box that applies

[ ] Bugfix
[x] New feature
[ ] Refactoring
[ ] Breaking change (any change that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Other (please describe)

Description
Describe what the change is

Add photo size validation in profile update form

Steps to Test This Pull Request
Steps to reproduce the behavior:

Go to user profile page
Upload 8 x 8 image that is larger than 10M.

Related Issue
If applicable, refernce to the issue related to this pull request.
https://trello.com/c/j0ezAODh/24-fix-all-the-bugs-found-during-the-proposal-system-testing
More Information
Screenshots

Additional context
Add any other context about the problem here. You may also want to refer
to how to wirte the perfect pull request

 Fix ALL the bugs found during the proposal system testing


__loc() in trait default fun definition confuses compiler

In the example at https://playground.ponylang.io/?gist=e14b0bb063257f2f75169d01f43c4779 / below:
use "debug"

trait Foo
  fun ref odd() =>
    @printf[I32]("I'm odd at line %d\n".cstring(), __loc.line())
    @printf[I32]("odd() I am %s, for resilient %s, checkpoint_id %s\n".cstring(), __loc().type_name().cstring(),
      "asdf".cstring(),
      "qwer".cstring())

class Foo1 is Foo

class Foo2 is Foo
  fun ref odd() =>
    @printf[I32]("I'm expecially odd at line %d\n".cstring(), __loc.line())

actor Main
  new create(env: Env) =>
    let a = Foo1
    a.odd()

    let b = Foo2
    b.odd()

... I see the error:
0.27.0 [release]
compiled with: llvm 3.9.1 -- cc (Ubuntu 5.4.0-6ubuntu1~16.04.11) 5.4.0 20160609
Defaults: pic=false ssl=openssl_0.9.0
Error:
main.pony:6:88: couldn't find 'apply' in '$1$1'
    @printf[I32]("odd() I am %s, for resilient %s, checkpoint_id %s\n".cstring(), __loc().type_name().cstring(),
                                                                                ^
Error:
main.pony:6:88: couldn't find 'apply' in '$1$3'
    @printf[I32]("odd() I am %s, for resilient %s, checkpoint_id %s\n".cstring(), __loc().type_name().cstring(),

I kept the __loc.line() in the example because I've managed to trigger a similar error with __loc.line() in Wallaroo.  I'm not sure why __loc.line() doesn't cause the same error in the playground version.
trait _EventLogPhase
  // ... stuff omitted ...
  fun ref checkpoint_state(resilient_id: RoutingId,
    checkpoint_id: CheckpointId, payload: Array[ByteSeq] val,
    is_last_entry: Bool)
  =>
    @printf[I32]("checkpoint_state() line %d\n"
      .cstring(), __loc().line())

    @printf[I32]("checkpoint_state() I am %s, for resilient %s, checkpoint_id %s\n"
      .cstring(), __loc().type_name().cstring(),
      resilient_id.string().cstring(),
      checkpoint_id.string().cstring())
  // ... stuff omitted ...

The errors there (using Pony 0.25.0) are:
Error:
/Users/scott/s/src/wallaroo/lib/wallaroo/core/recovery/event_log_phase.pony:45:24: couldn't find 'apply' in '$32$258'
      .cstring(), __loc().line())
                       ^
Error:
/Users/scott/s/src/wallaroo/lib/wallaroo/core/recovery/event_log_phase.pony:48:24: couldn't find 'apply' in '$32$259'
      .cstring(), __loc().type_name().cstring(),

Sorry, I don't understand this bug report - what is the bug here?
Note that __loc() (with parentheses after the __loc literal) will cause an error like that regardless of where it occurs, because the compiler assumes you are trying to call the apply() method of the __loc literal.
Removing the parentheses in your gist from __loc() to __loc solves the issue.

[...] I'm not sure why __loc.line() doesn't cause the same error in the playground version.

:facepalm: I've become partially parenthesis-blind.  My future as a Lisper is in doubt.


Identify the one GA report for tracking traffic

To enable us to successful deliver against the OKRs, we'd like to identify the one Google Analytics report that we will all reference as the source of metric of our KPI.

@ljcarot I think we said this issue can have the documentation added and then closed.  Thanks.

This is report from GA for tracking our pageviews in Carbon: https://analytics.google.com/analytics/web/?authuser=0#/dashboard/rWGJRg8ESuaNdNpdOdPXyg/a143580012w207210660p200215585/


New release with nodejs16

Hi @ishworkh
Thanks for the wonderful plugin.
I see that you have upgraded dependencies from nodejs12 to nodejs16 in https://github.com/ishworkh/docker-image-artifact-download/pull/6. Will you be making a v2 release for this plugin?

Hi, no v2 will be released for the action. The dependencies upgrade did not have any major changes to the action, so they were released in the same v1 tag.
Plz file another issue if you have further concerns :)


Adds missing namespace property to Helm configuration schema

When using Helm with azd users can specify the namespace of where a helm release will be deployed to.  The Helm integration already supports this but the namespace property was unintentionally omitted in the schema.

/check-enforcer override


merge V2.6.x develop to master

Codecov Report

Merging #12 (064a256) into master (2305ba0) will increase coverage by 0.09%.
The diff coverage is 29.78%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   51.22%   51.31%   +0.09%
==========================================
  Files         688      688
  Lines       36343    36359      +16
==========================================
+ Hits        18617    18658      +41
+ Misses      17726    17701      -25

Impacted Files
Coverage Δ

itsm/gateway/urls.py
100.00% <ø> (ø)

...pipeline_plugins/components/collections/bk_sops.py
16.08% <0.00%> (-0.23%)
:arrow_down:

itsm/gateway/views.py
17.55% <8.69%> (-0.38%)
:arrow_down:

itsm/workflow/views.py
31.58% <22.58%> (ø)

itsm/service/views.py
52.87% <45.07%> (+0.27%)
:arrow_up:

blueking/component/open/apis/sops.py
100.00% <100.00%> (ø)

itsm/workflow/backend.py
66.31% <0.00%> (+0.70%)
:arrow_up:

itsm/workflow/validators.py
35.18% <0.00%> (+6.68%)
:arrow_up:

itsm/component/utils/bk_bunch.py
80.95% <0.00%> (+23.80%)
: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 2305ba0...064a256. Read the comment docs.


feat: memory storage

BREAKING CHANGE: storage instances now must be passed into the Umzug constructor. strings ('json', 'sequelize', 'mongodb' etc.), as well as storageOptions, are no longer supported.
This also changes folder structure slightly:
before:
src/storages/JSONStorage.ts
src/storages/MongoDBStorage.ts
...

after:
src/storage/json.ts
src/storage/mongodb.ts
...

And all built-in storages are exported from the package's main module, so they can be imported with (e.g.) import { JSONStorage } from 'umzug'

Nice catch on changing the `UmzugStorage` interface to it. Much better.


[Bug report] Transportation segments with linear referencing shouldn't have a @name

Outline bug
If there are LR names, the primary name does not reflect the label of the whole segment, so exclude the name:
https://github.com/OvertureMaps/io-site/issues/118
Issue dependency with other TF Groups
No response

create or replace view segments as select * from read_parquet('/srv/data/overture/2024-07-22.0/theme=transportation/type=segment/*');
select count(*) from segments;
> 310298422

queries:
select count(*) from segments where names is not null;
> 89282746

29% of segments have a name
select count(*) from segments where names is not null and (names->'rules') is null;
> 83084965

93% of segments with a name do not have any rules
select count(*) from segments where json_array_length(names->'rules') = 1;
> 3916744

4% of segments with a name have only one rule
select names from segments where json_array_length(names->'rules') = 1 and (names->'rules’->>0->’between') = [0.0, 1.0];
> 0

there are no segments with geometric scopes where the scope is 0-1 (good)
select count(*) from segments where json_array_length(names->'rules') = 1 and (names->'rules'->>0->'between') is null;
> 2863694

Of the segments with 1 rule, 73% of them are a non-geometric scoping rule
select names from segments where json_array_length(names->'rules') = 2 and (names->>0->'value') = (names->>1->'value');
> 0

Of the segments with 2 rules, there are none where the name value is consistent between the first and second rule


Bug: fix underscores literal operators c++ compatible error again.

redo #7136 , because #7132 revert the patch.
Check #7132 libs/math/include/math/quat.h. Or git show e0a37de1ba2325d21f191253dd4c2c22bb7f8a9e.
@pixelflinger could you review this PR?

#7192 fixes this again


Error in C function svg: Expected  just one argument on line 59 of scss/apnscp/_ajax.scss

Hi developer(s),
I am getting an issue running grunt command, please check the screenshot attached.
Thanks

Issue need more info:

what's the environment?
anything edited in the theme already? if yes, did it work before?
why is he running grunt alone? docs say grunt watch or grunt dist
https://github.com/apisnetworks/apiscp-bootstrap-sdk#getting-started

Hey, thanks for the response

Getting same error of  grunt dist

Environment is Ubuntu 20.04, have not made any changes.
Why the grunt dist is throwing the same error.
Am I missing something?
Any help will be appreciated

What's your Node version? Recommended one is 6.10, you can try installing that via nvm and see how that works out. Make sure to reinstall modules with that version.

Hello, I am using 15, latest one.

v12 is required per package.json. It's a change in the node-sass bindings installed on v14+

v12 is required per package.json. It's a change in the node-sass bindings installed on v14+

So, I should downgrade to v12?

Yes. nvm is what I recommend to manage multiple Nodes.


Fix: Single use of quiz room

Added a condition to check if the room is already used, if it is used it should deny entering the room

Display rooms that are not used
Deny access to used rooms

@kinloveko pls approve

@jaymar921 @ha-rold1999  @HaroldCuico Approved!


Add media queries to downsize headers. Fixes #474.

Coverage remained the same at 100.0% when pulling 923667147adcf57ff92123725cf8be50a7f77001 on 474_resize_headings into 3f136cab0e3a9dd7b8905df575781f4654f0ac75 on dev.

Coverage remained the same at 100.0% when pulling 6a78055c1e6d4f4b8a9502a88a321dfe59ea47b1 on 474_resize_headings into 1953d038a7fd3bedb4bdafe557c67eb876380d0e on dev.

Just nitpicking: You mean headings not headers right? @tom95


slow on iterative solves

How can we pass the solvelink=5 option? This is useful a solution I am obtaining iteratively.

Actually, GAMS.jl currently forces the use of solvelink=5 at the moment. Do you need other solvelink values in other cases?

Really nice you thought of putting that by default. Unfortunately, in my case it does not seem to be working. I have this model that I solve iteratively. Runs about about 60 times. In Ipopt.jl this takes 1.7 seconds to go through all 60. Using KNITRO/Conopt-GAMS.jl it is taking 17 seconds. Of course the solutions are eventually the same. So my guess was trying solvelink. But then it must be something else.

Hm, that doesn't sound good. :-/ Are you able to share your model so I can have a close look? Otherwise I will try to find the bottleneck running some benchmark models.

The code below illustrates the issue. Just change the solver from Ipopt to GAMS. With Ipopt all iterations are done in 2 seconds. Conversly GAMS.jl takes 16 seconds.

using JuMP
using Ipopt
using GAMS

function NLP()
    nlp = Model(optimizer_with_attributes(Ipopt.Optimizer))
    #nlp = Model(optimizer_with_attributes(GAMS.Optimizer, "Solver" => "CONOPT","ModelType" => "NLP", "solve_link" => 5))
    set_silent(nlp)
    @variable(nlp, x>=0, start = 1)
    @NLobjective(nlp, Max, log(x))
    optimize!(nlp)
    display(value(x))
    display(termination_status(nlp))
end

function solve()
    cnt = 100
    while cnt > 3
        NLP()
        cnt -= 1
    end
end

@time solve()
```

I'm afraid I can't offer a good solution here. On my machine a single solve with one iteration only takes approx 0.002s for Ipopt.jl while GAMS.jl (with CONOPT) takes 0.014s. This is of course a significant overhead that multiplies with each solve in this loop. Of this one GAMS solve, 14.5% are for initializing the GAMS workspace (e.g. finding GAMS on your system), 1.0% for translating the model to GAMS and 74.2% for running the GAMS program. It is quite expensive to start a process in comparison to calling an API directly. I also compared the time needed to run the process from Julia and from command line and that's pretty much the same, so I don't see a way to improve timing here. I would love to fill GAMS data structures more directly in GAMS.jl using our APIs, but unfortunately, this isn't currently possible due to legal restrictions.
If you have a model that takes long solving, such an overhead is of course negligible, but this is not the case in your setup. The one thing I can do: I'll create an option where you can pass the GAMSWorkspace such that you can init it once instead of in every solve. But this won't solve the issue.
@odow Do you know how I could have an option of type GAMSWorkspace in MathOptInterface (via MOI.AbstractOptimizerAttribute)? I currently get the following error:
ERROR: MethodError: no method matching map_indices(::MathOptInterface.Utilities.var"#7#8"{MathOptInterface.Utilities.IndexMap}, ::GAMSWorkspace)

It seems like that options are allowed to only take certain types. Or shouldn't a GAMSWorkspace be passed as MOI.AbstractOptimizerAttribute.

For environments like this, the easiest way is probably like Gurobi's Env.
Modify Optimizer to take a workspace argument:
workspace = GAMS.Workspace()
model = Model(() -> GAMS.Optimizer(workspace))

The map_indices thing is here: https://github.com/jump-dev/MathOptInterface.jl/blob/master/src/Utilities/functions.jl#L55-L163. You probably just want
MOI.Utilities.map_indices(::MOI.Utilities.IndexMap, x::GAMSWorkspace) = x

Thanks @odow, I followed the way Gurobi.jl does it.
Since GAMS.jl version 0.2.3, a GAMS workspace can be initialized with either:
ws = GAMS.GAMSWorkspace()
ws = GAMS.GAMSWorkspace("<gams_system_dir>")
ws = GAMS.GAMSWorkspace("<gams_system_dir>", "<gams_working_dir>")

and the model then initialized with
model = Model(() -> GAMS.Optimizer(ws))

With this you are able to create a GAMSWorkspace once and use it for all models in your loop, @Jovansam. Unfortunately, this only reduces the execution time slightly. Because I cannot improve the GAMS command call at the moment, I'll close the issue. If you think that the bottleneck is somewhere else, please don't hesitate and reopen the issue.


Django 1.8 compatibility (continued)

#224 fixed an AttributeError caused by changes to ForeignObjectRel objects, but that same error exists with ManyToOneRel objects.
  File "django-filter/django_filters/filterset.py", line 111, in filters_for_model
    field = get_model_field(model, f)
  File "django-filter/django_filters/filterset.py", line 90, in get_model_field
    model = rel.rel.to
AttributeError: 'ManyToOneRel' object has no attribute 'rel'

Model structure that causes this:
Model A
Model B
Model C
 - fk to A
 - fk to B

FilterSet is for A; filter field is c__b. If that makes sense?

@melinath Fancy submitting a PR?

Normally I would, but my time's pretty limited right now, and this isn't a blocking issue for us (yet).

@carltongibson I see you merged into the develop branch the "fix get_model_field when using django 1.8. Is it possible to build and push to pypi?
thanks

@aRkadeFR — Yeah sorry for the delay here: I've had zero capacity here.
I will roll a new release just as soon as I can gather a few hours together to do it. In the meantime the best bet is to get pip to install the branch directly. You can revert to the PyPI release just as soon as it's made.

Exactly the same problem here with a beautiful 'ManyToOneRel' object has no attribute 'opts'
Sad :(

Hello all — sorry to leave this hanging so long — what's (cough) "sad" is the general lack of time and/or pull requests here :-)
Anyhow — I'm looking to do a new release now. I'm going to close this since I can't reproduce.
If you look in tests/test_filterset.py:56 you can see the test added in #224. I've tried all sorts trying to trigger the AttributeError seen here.
An example:
    def test_get_model_field_m2m(self):
        result = get_model_field(Book, 'lovers__username')
        self.assertEqual(result, User._meta.get_field('username'))

This passes. Again:
        result = get_model_field(Article, 'author__username')

... and so on...
What I can't see is what call to get_model_field triggers the exception. If someone can show me that I'm happy to re-open (and fix).

Looking at the code I can't quite see how the AttributeError comes up anyway. The offending line is on the else branch of this:
if isinstance(rel, ForeignObjectRel):
  # Fix from #224 applied here
else:
    # `AttributeError` raised here

But ManyToOneRel is a subclass of ForeignObjectRel — as are all the related fields — so how do we get here, ever?
As I say, totally happy to re-open but I need a reproduce.

Just checked on this again, still getting the same error on 0.11.0

I'll see if I can come up with a reproducible scenario for you in the next couple of days.

Oooh, seems to be fixed on develop! Guess we'll just have to wait for the next release!

FTR did a git bisect and the first working commit was 3d4a4e6ae282794f43846cb2c7a401cf25eac7a2.

Any word on when the next release might be? I see that django-filter now works with 1.9, but I still don't have a release that works for me on 1.8 :-p

Hi @melinath
@carltongibson said he was planning on doing a release at the beginning of January.

Great, thanks!


Adds support for custom docker options

Adds support for custom docker options.
Chose to nest docker under an options container so that all custom service options can be consumed via a generic map and cast to the correct type when needed.
name: todo-nodejs-mongo-aca
metadata:
  template<EMAIL_ADDRESS>services:
  api:
    project: src/api
    language: js
    host: containerapp
    options:
      docker:
        path: ./Dockerfile
        context: ../
  web:
    project: src/web
    language: js
    host: containerapp

Resolves Azure/azure-dev-pr#1298

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100 --version '' --verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-100

As part of this also update schemas/v1.0/azure.yaml.json with this new optional property.

+1 on Docker options and the ability to setup the Context in azure.yaml. With .NET projects, I was hitting this issue when multiple .csproj files are referenced that are not within the same folder. We are trying to setup an end-to-end sample with azd and hit this issue. As a best practice, we copy multiple .csproj files and restore the packages so subsequent builds are faster. With the options to setup the context similar to the docker-compose files, we could easily achieve this. See this simple app where we do it in compose https://github.com/erjain/azdSampleMicroservice/tree/shared-poc and works but fails with azd deploy.

Tested my sample with the exe generated in this PR and it worked ❤

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100 --version '' --verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-100

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100 --version '' --verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-100

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100 --version '' --verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/100/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-100


Add Web service reference whicth required a ssl certificate

i have to add web serveice reference "https://cst.olis.ssha.ca/ssha.olis.webservices.ER7/OLIS.asmx?WSDL"  to .net core mvc project, but when i copy url and click on go , vs did not ask for ceritificate and give error
"An error occurred while attempting to find services at 'https://cst.olis.ssha.ca/ssha.olis.webservices.ER7/OLIS.asmx?WSDL'. The request was aborted: Could not create SSL/TLS secure channel.  "
how can i add it in .net core asp.net core mvc project?

when i add it in .net framework project, it ask for certificate and add reference properly.

@ChAhmadBasha -- thank you for your feedback.  This repo is solely for documentation issues, and this is implementation-related.  Please consider these resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Deprecation of EntityManager copy method

To follow @alcaeus' procedure mentioned in #6794, I create this PR to deprecate the method EntityManagerInterface::copy() on the master branch.

Deprecation of copy() is now added to the upgrade information file.

@SenseException thanks, let's hold this a bit though. The idea is to start adding deprecation notes in v2.7, so we need to get v2.6 release ASAP.

Since we're about to start using @trigger_error(..., E_USER_DEPRECATED) pattern as per doctrine/dbal#2846, should we probably use it here as well?

I can't imagine how this could be a real BC break since the only implementation ever was an exception. 🤣 But well, it's in the interface anyway...

@SenseException a rebase is still needed, sorry


Successfully Solved Leetcode 421 Solution.

Successfully Solved Leetcode 421 Solution for the HackOctoberfest2022.

Great work @trivedi2u please give this repo a star if it really helped you to complete your PR.
Also follow me for follow back if you want to get connected with me for any such open source contribution program. Thank you 😊


[microsite] Setup Google Analytics 4

I'm opening this as part of a CNCF-wide effort to upgrade project websites to GA4. For context, see:

https://github.com/cncf/techdocs/issues/108

Tasks:

[x] Create a GA4 tag (measurement ID) under the CNCF account

The new measurement ID is G-KSEVGGNCJW. Note that the UA and GA4 ids are now connected.

[x] Upgrade the microsite to use gtag.js for configuring the new GA4 site tag

[x] #12659
[x] Manually connect the UA ID to the GA4 tag

[x] Confirm that the GA4 site tag is receiving events and posting data

To learn more, including links to other resources, see:

https://github.com/cncf/techdocs/issues/108

Related:

https://github.com/backstage/backstage/issues/12019#issuecomment-1185574359

/cc @nate-double-u

/assign @chalin

We're done! The GA4 tag is processing website events:


update for macOS 13 (Ventura)

As the interface changed as well as the acessibility policies, a re-build using automator app was needed.

todo: update docs (screenshots etc)

solves #12

Thanks!


Update taskparameters.ts

Task name: AzureWebAppContainerV1
Description:  support \n caracter for multiline
As some other Task  (example : AzureFunctionAppV1) use appsettings parameter :  Add multiline support.
I suggest to use \n as space (mandatory). It's better for readibility. A space is not visible and easy to forget it.
It's also more consistent with other tasks.
Documentation changes required: (N) => not required but maybe interesting to add information about this possibility
Added unit tests: (N)
Attached related issue: (N)
Checklist:

[ OK] Task version was bumped - please check instruction how to do it
[ OK] Checked that applied changes work as expected

Result
steps:
- task: AzureWebAppContainer@1
  displayName: 'Azure Web App on Container Deploy: test'
  inputs:
    azureSubscription: resourcegroup
    appName: test
    appSettings:
      -input1 val1 \n
      -input2  val2 \n

@v-mohithgc new one with updated version, and merged master.
Could you adapt this one or validated ?

@v-mohithgc new one with updated version, and merged master. Could you adapt this one or validated ?

Thanks, I will test it and revert back.

/azp run

/azp run

/azp run

/azp run

@michelcedric , looks the changes weren't build properly so checks are failing here, I will fix that now.
meanwhile, if possible, can you brief about the proposed fix here? like how to reproduce this issue (required for testing) and how these changes will fix that.
Thanks

I have rebuild this task, will use this PR https://github.com/microsoft/azure-pipelines-tasks/pull/19094

Manually integrated in https://github.com/microsoft/azure-pipelines-tasks/pull/19128
Thank you ;)


Fix bug in upload config script

Signed-off-by: Rex<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Linter tests not passing because an issue with isort. Disabling linter check and merging manually.


Is there a complete list of nodes and a node's information?

Hi, your work is impressive :) I have some doubts, what is the complete format of parsetree's json? I will need it because I need to traverse the parsetree to extract specific information.
For example, the parse tree of PgQuery.parse ("select count (*) from page") has node FunCall, while PgQuery.parse ("select * from page") does not. If I don't know the existence of FunCall node, it will make my traversal difficult.
Also, I cannot know the meaning of the "CommonTableExpr" node in "WithClause"? What information does this node contain? I am also confused about some other nodes
Is there a complete list of nodes and what does each node mean?

@ZhirongYuan0628 The new 2.0 release now has full class types for each node, making it much easier to to what you're describing.
You can see this in the example in the README here: https://github.com/pganalyze/pg_query#parsing-a-query
They are based on the Protobuf format of the parse tree, you can see it for reference here: https://github.com/pganalyze/libpg_query/blob/13-latest/protobuf/pg_query.proto


YUNIKORN-38. YuniKorn UI behaves incorrectly when max resource is not set

@yangwwei @sunilgovind @wilfred-s  please review the PR

Hi @akhilpb001
Thanks for working on this!! I tested the changes locally and attached the screenshot here, there are 2 things I want to discuss

for the logo, can you please use the logos we have here: https://github.com/apache/incubator-yunikorn-core/tree/master/docs/images/logo. Since the background is dark blue, I think we can try the gray one.
for the usage bar, now it is nicer, but are they static values? Can we calculate the relative percentage to the biggest queue-usage and use that for the usage-bar?

@yangwwei

Will use the new logo
I took the value from response. I thought they are calculated by scheduler itself.  How do you want me to calculate this? I did not understand what you said above.

@yangwwei

I have updated the PR with the new logo for side nav bar
I think it would be better the scheduler to send the absUsedCapacity in the api response (which is used to show usage bar), because sometimes configured capacity could be empty and UI can not do that calculation in that case.

Great, looks good to me. +1


Dynamic ref NMS

Has NMS op output a dynamic shape (ONNX spec behavior)
Allows for dynamic input shape to NMS op
Depends on #1224
Need to decide how to preserve previous zero output padding behavior (done with member variable)
Unnecessary compute_shape() called at eval time for NMS as output shape will stay dynamic until after the calculation is complete (refactored)

I found a bug that I erroneously carried over from Shucai's code. The maximum output shape needs to change with the batch size. The max_num_boxes should actually be number of batches * total number of boxes. Made the fix in a recent commit. I'm somewhat surprised this operator has been working thus far?

I found a bug that I erroneously carried over from Shucai's code. The maximum output shape needs to change with the batch size. The max_num_boxes should actually be number of batches * total number of boxes. Made the fix in a recent commit. I'm somewhat surprised this operator has been working thus far?

Good fine. It probably worked because number_of_batch was only 1.

Noticed an issue: there's not going to be a use_dynamic attribute for NMS nodes in ONNX models. We need to be able to set use_dynamic through some other method. I think we would want to have another value in struct onnx_options of basically use_dynamic_output and to make a new parse_nms.cpp file.

I made a rather roundabout way to set use_dyn_output using the onnx_options. It sets or creates the attribute for any node with the same op_type. The name field for a node can be absent in the ONNX spec, so I could not use that.
In the future, I expect that we will change the code such that dynamic output operators always use the new dynamic output behavior. That would make the use_dyn_output stuff unnecessary.

We can merge this and then remove the header in a later PR if you want.


[ffigen] Switch to package:dart_flutter_team_lints

FYI @liamappelbe @Mohammad3id

I'm not sure why the lints didn't show up locally, but this PR requires some more migration.


After upgrade to <IP_ADDRESS>107041908 the dash icon disappears and it's nowhere to be found except directly calling dbeaver-ce from the cli

System information:

Ubuntu 20.04.2 LTS 64-bit, GNOME 3.36.8
<IP_ADDRESS>107041908
no extensions

Connection specification:

there is no problem with any DB connection

Describe the problem you're observing:
After an overnight update in snap tonight the icon in the dash disappeared and I can't seem to find it in the applications list neither. The only way I can run it is via cli calling "dbeaver-ce" but I cannot even pin it to the dash
Steps to reproduce, if exist:
every time I run the software ...
Include any warning/errors/backtraces from the logs

I can see a few suspicious things in the stdout when running it via cli
SWT SessionManagerDBus: Failed to RegisterClient: GDBus.Error:org.freedesktop.DBus.Error.AccessDenied: An AppArmor policy prevents this sender from sending this message to this recipient; type="method_call", sender=":1.121" (uid=1000 pid=21015 comm="/snap/dbeaver-ce/136/share/dbeaver/jre/bin/java -X" label="snap.dbeaver-ce.dbeaver-ce (enforce)") interface="org.gnome.SessionManager" member="RegisterClient" error name="(unset)" requested_reply="0" destination=":1.23" (uid=1000 pid=1887 comm="/usr/libexec/gnome-session-binary --systemd-servic" label="unconfined")
(DBeaver:21015): Gtk-CRITICAL **: 06:27:27.431: gtk_box_gadget_distribute: assertion 'size >= 0' failed in GtkScrollbar
there was nothing else in the logs (/home/*****/snap/dbeaver-ce/136/.local/share/DBeaverData/workspace6/.metadata)

thank you for the report

Fixed. Try to update dbeaver-ce snap package.

nice, it works great now again :) awesome support guys !!! thank you


Fix typo in playground: speect → speech

Action button title/aria-label says 'speect' instead of 'speech'.

Thank you!


Support flow parser in prettier

Running this plugin on project with flow types causes an error as follow:

The reason is explicit indication to prettier to use babel-ts parser in utils/are-nodes-same-in-text.js

Thanks 🙏
It looks like a bug for any non-standard JS-like languages.

So, I'd tried to set up Flow with ESlint in test env, but I failed. Could you make a PR with test for your use-case?
I'm not a big specialist in Flow, and I cannot set up it properly.

Ok, I'll try to do it as soon as possible


[DISCUSS] Set DataFusion settings for maximum "out of the box" performance

Is your feature request related to a problem or challenge?
I want people's first impression of DataFusion to be "that is very fast" without having to tune parameters
DataFusion has many configuration options that control various performance optimization.
There is a tradeoff between some of these options between faster query execution (for more than linear resource consumption) and pure efficiency.
We have benchmarks such as a tpch runner that typically run with a single core. These are great as performance unit tests in well controlled environments (and avoid task overhead, and other non determinism introduced with multi-core execution), however they don't mimic what users typically run with.
Up to now we have taken a conservative approach and only enabled optimizations by default if they make everything faster.  I would like to change our philosophy and optimize for "out of the box" performance
You can see examples of other systems tuning knobs up for performance:
IOx: https://github.com/influxdata/influxdb_iox/blob/ad28ebb7650d1cd21b995ee5b514d8da3580f22b/datafusion_util/src/config.rs#L20-L23
Here is a recent example from https://hussainsultan.com/posts/unbundled-datafusion/
runtime = RuntimeConfig().with_disk_manager_os().with_fair_spill_pool(100000000)
config = (
    SessionConfig()
    .with_create_default_catalog_and_schema(True)
    .with_target_partitions(8)
    .with_information_schema(True)
    .with_repartition_joins(True)
    .with_repartition_aggregations(True)
    .with_repartition_windows(True)
    .with_parquet_pruning(True)
    .set("datafusion.execution.parquet.pushdown_filters", "true")
)

ctx = SessionContext(config, runtime)
ctx.register_parquet("orders", "../../../fanniemae-benchmark/sf10/raw/orders.parquet")

Describe the solution you'd like
I would like to change the ConfigOption defaults to optimize performance in the common case rather than avoid
performance regressions in all cases
Specifically that means:

Repartition always when possible (to increase parallelism by default)
Push down all parquet filters  (e.g. https://github.com/apache/arrow-datafusion/issues/4085 and https://github.com/apache/arrow-datafusion/issues/3463)

Describe alternatives you've considered
We can leave the defaults alone and make users change the defaults
Additional context
No response

Here is another example of where we had to finagle settings to make DataFusion fast: https://github.com/apache/arrow-datafusion/issues/5942

Another example what I have found is increasing batch size solves some performance troubles related to excessive collection allocation or loops.
I suppose there is also a trouble when the code gets triggered for empty batches, so the code spins CPU but in fact does nothing, I think noticed this behavior couple of times but need to have a good reproduce test to prove it.

Thank you @alamb for putting this together. I definitely agree that we should choose the default configuration settings to squeeze maximum performance. OOTB performance is a common blogging subject, and search results that yield to such posts is a big entry to point to the user funnel. We should give users the right impression in terms of where Datafusion stands in terms of the performance.

#6325  is another project to improve the performance out of the box


Add shell completion for --version flag

This proposal is to add shell completion for the --version flag.  As an example
helm show all stable/fluentd --version <TAB>
helm pull stable/fluentd --version <TAB>
helm install myfluentd stable/fluentd --version <TAB>
helm upgrade myfluentd stable/fluentd --version <TAB>

would suggest
1.0.0   1.10.0  1.2.0   1.4.0   1.5.1   1.7.0   1.9.1   2.0.1   2.0.3   2.2.0   2.3.0   2.3.2   2.4.0
1.1.0   1.10.1  1.3.0   1.5.0   1.6.0   1.9.0   2.0.0   2.0.2   2.1.3   2.2.1   2.3.1   2.3.3   2.4.1

This improvement is not as much about saving users some typing as it is about helping users figure out which versions are available.

@marckhouzam how would this perform if someone had hundreds of releases? I'm thinking about CI scenarios.

@marckhouzam how would this perform if someone had hundreds of releases? I'm thinking about CI scenarios.

It's not as much a question of the number of releases but the size of the index file. A large index file takes time to parse to extract the versions needed for the completion. Performance is already poor for this completion (in my preliminary tests) when the chart is on the 'stable' repo because the yaml parsing of that giant index file is slow.
We've optimized this for completing chart names by adding a charts.txt cache file for each repo which contains a flat list of chart names.  See https://github.com/helm/helm/pull/6809#issuecomment-560977848
I didn't feel this--version completion was a common scenario, so I didn't focus on optimizing.
Let me take this opportunity to ask: do we consider the charts.txt cache file format an API? If it's not, it could be enhanced to also cache the list of versions.
But all this performance is during user interaction doing completion.  What do you mean by CI scenarios?


_replicationInfo no longer exists and breaks the browser example code.

Changed to _replicationStatus to match tests and latest code and the example works again.

Nice catch!

Indeed great catch! Thank you @strotter for the PR ❤


AWS: Authentication and TLS Support

https://www.kubeflow.org/docs/aws/authentication/
Aws does not provide with ARN for the custom domain has the auth.us-east-1.amazoncognito.com belongs to AWS and the document does not say how to configure the Custom Domain ARN.
And the cognitoUserPoolDomain : field parameter is confusing. domain/your-pool-name
for eaxmple : kubeflow.example.com/kubeflow (our pool name)

cognitoUserPoolDomain is the cognito pool name. Custom Domain ARN will be addressed in #1435

@charlesparasa Can you check if #1661 is clear enough for you?

This has been address in #1661 and it's also a duplicate of #1435. I think we can close this one and track in #1435
/close


Changed jsonp to loadJsonp

Closes #2579
Created a new file loadJsonp and deprecated jsonp. Renamed occurrences of jsonp to loadJsonp. Updated CHANGED.md.
There was no Spec for jsonp, but I can add one for loadJsonp if needed.

There was no Spec for jsonp, but I can add one for loadJsonp if needed.

Yes, please.

JsHint is failing: https://travis-ci.org/AnalyticalGraphicsInc/cesium/builds/85558063

You may be able to base your tests off of this: https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Specs/Core/loadJsonSpec.js

Looks good otherwise!

Added Test. I based it off the tests for loadloadJsonSpec, and removed what was not applicable, for example, when an error occurs, the error does not specify the error code or status.

I would also add a test that flexes each of the options objects and makes sure they work as expected.  Other than that and my couple of comments, this looks good.

Thanks @ggetz

@mramato is there a way to simulate loading a url and getting a response with data? The methods used in loadJsonSpec doesn't work here, I believe.

Updated the unit tests.

Just that one last comment.

@mramato Thanks, made the change.

Thanks again @ggetz!


Create a labcreation page

Create some kind of page where an instructor can enter in questions / answers to create new labs
Make it API ready so that once the API is setup it can switch from a dummy data structure to the API's call.

Finsished


znc: 1.6.5 -> 1.6.6

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
ran /nix/store/z51kkm0wj1wv4bxkib8lv7nqizdly9jw-znc-1.6.6/bin/znc -h got 0 exit code
ran /nix/store/z51kkm0wj1wv4bxkib8lv7nqizdly9jw-znc-1.6.6/bin/znc --help got 0 exit code
ran /nix/store/z51kkm0wj1wv4bxkib8lv7nqizdly9jw-znc-1.6.6/bin/znc -v and found version 1.6.6
ran /nix/store/z51kkm0wj1wv4bxkib8lv7nqizdly9jw-znc-1.6.6/bin/znc --version and found version 1.6.6
ran /nix/store/z51kkm0wj1wv4bxkib8lv7nqizdly9jw-znc-1.6.6/bin/znc -h and found version 1.6.6
ran /nix/store/z51kkm0wj1wv4bxkib8lv7nqizdly9jw-znc-1.6.6/bin/znc --help and found version 1.6.6
found 1.6.6 with grep in /nix/store/z51kkm0wj1wv4bxkib8lv7nqizdly9jw-znc-1.6.6

cc @viric @schneefux @lnl7 for review

@GrahamcOfBorg eval

Success on x86_64-darwin (full log)
Attempted: znc
Partial log (click to expand)

 If you need help with using ZNC, please visit our wiki at:
   http://znc.in
post-installation fixup
gzipping man pages under /nix/store/mygwdmndj9k6ikdwsbszk95821qffzhs-znc-1.6.6/share/man/
strip is /nix/store/vb6mgjqgd9h11nbv60fvrp1ls9nxck2y-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/mygwdmndj9k6ikdwsbszk95821qffzhs-znc-1.6.6/lib  /nix/store/mygwdmndj9k6ikdwsbszk95821qffzhs-znc-1.6.6/bin
patching script interpreter paths in /nix/store/mygwdmndj9k6ikdwsbszk95821qffzhs-znc-1.6.6
/nix/store/mygwdmndj9k6ikdwsbszk95821qffzhs-znc-1.6.6/bin/znc-buildmod: interpreter directive changed from "/bin/sh" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/sh"
/nix/store/mygwdmndj9k6ikdwsbszk95821qffzhs-znc-1.6.6

Success on x86_64-linux (full log)
Attempted: znc
Partial log (click to expand)
shrinking /nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6/lib/znc/stickychan.so
shrinking /nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6/lib/znc/chansaver.so
shrinking /nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6/lib/znc/webadmin.so
gzipping man pages under /nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6/share/man/
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6/lib  /nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6/bin
patching script interpreter paths in /nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6
/nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6/bin/znc-buildmod: interpreter directive changed from "/bin/sh" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6...
/nix/store/kgwglg17nwr6sj4z1si7yq9lrxfagnkc-znc-1.6.6

Success on aarch64-linux (full log)
Attempted: znc
Partial log (click to expand)
shrinking /nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6/lib/znc/webadmin.so
shrinking /nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6/lib/znc/autocycle.so
shrinking /nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6/bin/znc
gzipping man pages under /nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6/share/man/
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6/lib  /nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6/bin
patching script interpreter paths in /nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6
/nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6/bin/znc-buildmod: interpreter directive changed from "/bin/sh" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6...
/nix/store/d2gbhfb4qpcgjh0b3kb3r5jm66mja4ln-znc-1.6.6


chore(main): release 0.2.2

:robot: I have created a release beep boop
0.2.2 (2024-05-15)
Bug Fixes

update dependencies (7077e21)

This PR was generated with Release Please. See documentation.

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

:exclamation: No coverage uploaded for pull request base (main@d4e9d3c). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage        ?   92.54%
=======================================
  Files           ?       11
  Lines           ?      778
  Branches        ?        0
=======================================
  Hits            ?      720
  Misses          ?       58
  Partials        ?        0

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

:robot: Release is at https://github.com/IFCA-Advanced-Computing/anjana/releases/tag/v0.2.2 :sunflower:


node 6 - nuxt@next - build fails with SyntaxError: Unexpected token function

Since the last nuxt@next (1.0.0-gh-3ec8f14), I'm not able to build nuxt anymore with Node Version 6. I saw a commit regarding node 8, so is node 6 officialy not longer supported? Not even for the build process?
# nuxt build --universal && backpack build
output /source/node_modules/nuxt/lib/common/utils.js:25
output exports.waitFor = async function waitFor(ms) {
output ^^^^^^^^
output SyntaxError: Unexpected token function
output - module.js:20 require
output internal/module.js:20:19
output - index.js:1 Object.<anonymous>
output [source]/[nuxt]/lib/common/index.js:1:77

Thanks
Simon

Hi. And thanks again for your early testings. Yes. Is there any reason blocking you to upgrade node 8.0.0? We forced for this decision for many internal perf improvements. the most important is native async/awaits.

Well ;) the problem is actually just my deployment pipeline is not supporting node 6 right now. At least not out of the box. Servers are running already on node 8.
Is there a way to include a babel transpile to support node 6 just for the build process? Or is this even more effort than upgrading my deployment pipeline? ;)
thanks for your quick replies!
Simon

If you can upgrade everything to 8.x that would be much better.  Can i ask which pipeline do you use for build ?

I'm using https://deploybot.com/ .. not very satisfied anyways, should take this as a sign an move on ;)
If you have any recommendations, just let me know.
Thanks though, closing this for now.

Google cloud functions also not compatible with node 8.x yet:
https://cloud.google.com/functions/docs/release-notes
This makes firebase cloud functions fail bad also...

The latest build of nuxt-edge supports Node 6.x again by automatically falling back to the legacy build.

PS: Don't forget to add babel-polyfill dependency to your project if you need legacy build.


feat: add argo workflow templates to argo cd application automation

Adding our argo-workflows and dependency manifests as an Argo CD Application.

@skrobul I think you also mentioned some ideas here. Should we package up the workflows as a helm chart and have ArgoCD deploy that chart? Then each deployment can have its own values.yaml to override things.

I am looking for thoughts on how I am handling the overlay repo here. Do we want to use the same UC_DEPOLOY overlay repo?

If you see the use case for something like that, we can add an additional environment variable that is looked up first, if it's not present, we default to $UC_DEPLOY. Same thing for the references.

Should we package up the workflows as a helm chart and have ArgoCD deploy that chart?

I think long term it makes sense, but for now let's just have something that deploys those workflows.


Feature request: Leveraging good forecasting methods implemented in R packages

It would be useful to incorporate in sktime some of the forecasting methods implemented in R packages, and in particular the packages done by Prof. Hyndman and his team: fable, forecast, feasts, seer, hts, thief (see https://robjhyndman.com/software/).
This way one can have access in Python to those methods, through the excellent API that was designed for sktime.
I know that a few of the methods  in those R packages were already implemented in sktime. For the remaining ones, a possible approach would be not to reimplement them, but rather to call these packages within sktime, given that R packages can be utilized within Python code (and viceversa).

Yes, that sounds like good suggestion!


./Rcplex2.h:4:10: fatal error: 'ilcplex/cplex.h' file not found

Describe the bug
Installed CPLEX and set the path in bashrc, installed sequenza and copynumber. However couldn't install JaBba, shows error meg as follows:
In file included from Rcplex2.c:2:
./Rcplex2.h:4:10: fatal error: 'ilcplex/cplex.h' file not found
#include <ilcplex/cplex.h>
^~~~~~~~~~~~~~~~~
1 error generated.
make: *** [Rcplex2.o] Error 1
ERROR: compilation failed for package ‘JaBbA’

removing ‘/Library/Frameworks/R.framework/Versions/4.3-arm64/Resources/library/JaBbA’
Warning message:
In i.p(...) :
installation of package ‘/var/folders/c2/q4h5h5s940s76rf4dyhchh9dh_wk0p/T//Rtmpmq7z2L/file955f54a45d69/JaBbA_1.1.tar.gz’ had non-zero exit status

any help is appreciated.

What is the output of echo $CPLEX_DIR?

its /Applications/CPLEX_Studio_Community2211/cplex; I have set the path for cplex.h also in bashrc.
export CPLEX_DIR=/Applications/CPLEX_Studio_Community2211/cplex
export CPLEX_INCLUDE_PATH=$CPLEX_DIR/cplex/include
export CPLEX_LIBRARY_PATH=$CPLEX_DIR/cplex/lib

I believe JaBbA only uses the CPLEX_DIR to derive the include and library paths, exporting them directly may not work. Can you please verify:

NOTE: if CPLEX_DIR is set correctly then $CPLEX_DIR/cplex/include and $CPLEX_DIR/cplex/lib should both exist.

If I remember right, the new versions of CPLEX have an extra cplex directory in the path.

我相信 JaBbA 仅使用CPLEX_DIR来派生包含和库路径，直接导出它们可能不起作用。您能否验证：

注意：如果CPLEX_DIR设置正确，则 $CPLEX_DIR/cplex/include 和 $CPLEX_DIR/cplex/lib 都应该存在。

如果我没记错的话，新版本的 CPLEX 在路径中有一个额外的目录。cplex

在ARM64的机器上如何使用？我无法装上cplex


Maven

Note that the gradle build is broken if you've not run maven.
This is because the gradle build doesn't yet sign the testjar.
I'm working on this later https://github.com/jmock-developers/jmock-library/issues/67

Oli, we have to coordinate on this, we can't continue with commit-tennis. If you want to take it, that's fine but we need to agree a destination.

That's why I'm not pushing to jmock2 which is the master branch for this project.
maven is an integration branch, that we can use to communicate and coordinate.

I would advice not pulling this into jmock2 until gradle is working correctly.
Currently it's not building the signed and unsigned jars #67.
Which breaks the travis jdk1.7 build.


[python3] build failure

Host Environment

OS: OS X (Big Sur beta8)
Compiler: Clang

To Reproduce
Steps to reproduce the behavior:
./vcpkg install python3
Failure logs
-Cut and past the appropriate build messages from the console output.
-Please attach any additional failure logs mentioned in the console output.
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:141 (message):
Command failed: /Users/jast/vcpkg/buildtrees/python3/src/v3.8.3-43b03f2bf0.clean-x64-osx-release/configure --prefix=/Users/jast/vcpkg/buildtrees/python3/src/v3.8.3-43b03f2bf0.clean-x64-osx-release/../../make-build-x64-osx-release --with-openssl=/Users/jast/vcpkg/installed/x64-osx "CPPFLAGS=-I/Users/jast/vcpkg/installed/x64-osx/include -framework CoreFoundation" LDFLAGS=-L/Users/jast/vcpkg/installed/x64-osx/lib LIBS=-liconv
Working Directory: /Users/jast/vcpkg/buildtrees/python3/src/v3.8.3-43b03f2bf0.clean-x64-osx-release
See logs for more information:
/Users/jast/vcpkg/buildtrees/python3/config-x64-osx-rel-out.log
/Users/jast/vcpkg/buildtrees/python3/config-x64-osx-rel-err.log
Call Stack (most recent call first):
ports/python3/portfile.cmake:125 (vcpkg_execute_build_process)
scripts/ports.cmake:79 (include)
Error: Building package python3:x64-osx failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with ./vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: python3:x64-osx
Vcpkg version: 2020.06.15-unknownhash
Additionally, attach any relevant sections from the log files above.
The following packages will be built and installed:
GNU nano 2.0.6                             File: /Users/jast/vcpkg/buildtrees/python3/config-x64-osx-rel-err.log
Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
configure: error: in /Users/jast/vcpkg/buildtrees/python3/src/v3.8.3-43b03f2bf0.clean-x64-osx-release': configure: error: cannot compute sizeof (long double) See config.log' for more details
Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

Could you provide the full log?
/Users/jast/vcpkg/buildtrees/python3/config-x64-osx-rel-out.log
/Users/jast/vcpkg/buildtrees/python3/config-x64-osx-rel-err.log
Thanks.

config-x64-osx-rel-err.log
config-x64-osx-rel-out.log

@JackBoosY Sure, already uploaded.

configure: error: cannot compute sizeof (long double)

Is there any workaround to fix this error? @JackBoosY

@chenming-wu My osx machine has problem, waiting for fix it.

@chenming-wu Can you provide the log /Users/jast/vcpkg/buildtrees/python3/src/v3.8.3-43b03f2bf0.clean-x64-osx-release/config.log?

config.log

clang: error: invalid version number in 'MACOSX_DEPLOYMENT_TARGET=11.0'
Should not be compatible with the beta version of macOS.

Will upstream cpython to the latest version solve this problem? See https://github.com/python/cpython/pull/21113

@chenming-wu  Maybe, you can have a try.


Binaries missing/incorrect release notes for v20201117 release

From the release notes:
Binaries used to build these AMIs are published :
s3://amazon-eks/1.18.9/2020-11-07/
s3://amazon-eks/1.17.12/2020-11-07/
s3://amazon-eks/1.16.15/2020-11-07/
s3://amazon-eks/1.15.12/2020-11-07/

Neither one of the K8s version directories contains a 2020-11-07 subdirectory, only a 2020-11-02 subdirectory is present.
Are the binaries missing or are the references in the release notes incorrect?

@vishalkg, I see you updated the changelog for v20201117, perhaps you can identify what the problem is here?

@janbeerden We have fixed the bucket name.

@vishalkg, awesome, thanks!


Fixed copy-paste errors

Description:
Fixing copy-paste errors in tests
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

Looks good to me