Column filter doesn't work

Expected Behavior
If I search for a column by the exact name or a string with a wildcard, I would expect to see search results listing tables containing that column.
Current Behavior
Currently, no search results are returned when applying the column filter as shown in the screenshot below.

I also see this same error message when searching for a column by the exact name.
Possible Solution
I was able to resolve this bug by changing the value of the column field in this line to column_names.keyword as shown in the screenshot below.

Steps to Reproduce
I ran the search, metadata, and frontend service locally on my computer. I used this AwsSearchConfig module for the search, a DevConfig module that connects to a Neo4j proxy client for the metadata, and this LocalConfig module for the frontend.
Context
I couldn't deploy a version of the frontend with the column filter as it would incorrectly return no search results. As instructed in these docs, I ran the configured SearchMetadatatoElasticsearchTask and confirmed that the new table index has the new mappings. If I get the new index via the Kibana console, what I do notice is that there is a column_names property in addition to the columns property.
"mappings" : {
      "_meta" : {
        "version" : 2
      },
      "properties" : {
        ...,
        "column_names" : {
          "type" : "text",
          "fields" : {
            "keyword" : {
              "type" : "keyword",
              "ignore_above" : 256
            }
          }
        },
        "columns" : {
          "type" : "text",
          "fields" : {
            "general" : {
              "type" : "text",
              "term_vector" : "with_positions_offsets",
              "analyzer" : "general_analyzer"
            },
            "keyword" : {
              "type" : "keyword"
            },
            "ngram" : {
              "type" : "text",
              "term_vector" : "with_positions_offsets",
              "analyzer" : "ngram_analyzer_table_columns"
            }
          },
          "term_vector" : "with_positions_offsets",
          "analyzer" : "stemming_analyzer"
        },
        ...
     }

I also called the Elasticsearch REST APIs to confirm what I was seeing above based on the request body parameters logged out by es_proxy_v2_1. For example, when I submit this GET request, I get no hits:
GET table_search_index_v2_1/_search
{"query": {"bool": {"filter": [{"bool": {"should": [{"wildcard": {"columns.keyword": "*key"}}], "minimum_should_match": 1}}]}}, "from": 0, "size": 10, "highlight": {"fields": {"name": {"type": "fvh", "number_of_fragments": 0}, "description": {"type": "fvh", "number_of_fragments": 0}, "columns.general": {"type": "fvh", "number_of_fragments": 10, "order": "score"}, "column_descriptions": {"type": "fvh", "number_of_fragments": 5, "order": "score"}}}}

whereas this GET request (same as above except for the key after the wildcard key) returns hits
GET table_search_index_v2_1/_search
{"query": {"bool": {"filter": [{"bool": {"should": [{"wildcard": {"column_names.keyword": "*key"}}], "minimum_should_match": 1}}]}}, "from": 0, "size": 10, "highlight": {"fields": {"name": {"type": "fvh", "number_of_fragments": 0}, "description": {"type": "fvh", "number_of_fragments": 0}, "columns.general": {"type": "fvh", "number_of_fragments": 10, "order": "score"}, "column_descriptions": {"type": "fvh", "number_of_fragments": 5, "order": "score"}}}}

Your Environment
Amundsen version used:

amundsendatabuilder == 7.4.3
amundsensearch == 4.1.0
amundsenmetadata == 3.12.0
amundsenfrontend == 4.2.0

I also have an AWS OpenSearch Service (OSS) domain using Elasticsearch 7.10.

You should not have both a columns and column_names field in your mappings. For us we only have columns and the existing filtering functionality works for the column filter. Do you periodically drop and recreate your indices? If so does it create both fields every time?

I just made a small fix, this is a python inheritance issue that makes it so the mapping of the parent class is used rather than the newest search class. Thanks for raising this! https://github.com/amundsen-io/amundsen/pull/2121

You should not have both a columns and column_names field in your mappings. For us we only have columns and the existing filtering functionality works for the column filter. Do you periodically drop and recreate your indices? If so does it create both fields every time?

It creates both fields every time.

@beauttie are you using the packages?the release for that fix was just made

I am updated what was in the release for search-4.1.1 for the metadata, search, and frontend service, and I still see the same bug. Again, I think this is an issue with how the mappings are created in amundsendatabuilder. We currently use the latest version 7.4.3.

hi @beauttie, I'm going to close this issue as a triaged state, and people can upvote it or someone can choose to work on the issue from there


chore: fix link in system_contracts.md

What ❔
At the moment, old link is very outdated. In addition, it opens quite unstable (404 error).
cc: @perekopskiy

Why ❔

Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.
[x] Code has been formatted via zk fmt and zk lint.
[x] Spellcheck has been run via zk spellcheck.

@Deniallugo @perekopskiy @slowli @popzxc
Colleagues, please review this PR.

Thank you, though at this moment we don't merge small contributions (see contributing FAQ).


Add event for remote and local notifications (close #478)

This PR to create an out-of-the-box event usable on both trackers.
The fields in the event are common fields used in the notifications of both iOS and Android.
Some fields specific for the platform have a special comment indicating the platform.
The custom data associated to the notification can't be tracked in this event and should be tracked with custom entities built specifically for the case by the developer.

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


Unable to update state

Logger: homeassistant.helpers.entity
Source: helpers/entity.py:696
First occurred: 2 novembre 2023 à 10:13:36 (57 occurrences)
Last logged: 09:44:35
Update for climate.chambre fails
Update for climate.sejour fails
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 696, in async_update_ha_state
await self.async_device_update()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 959, in async_device_update
await hass.async_add_executor_job(self.update)
File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/air_cloud/climate.py", line 233, in update
devices = self._api.load_climate_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/air_cloud/api.py", line 57, in load_climate_data
response = ws.recv()
^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/websocket/_core.py", line 364, in recv
opcode, data = self.recv_data()
^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/websocket/_core.py", line 387, in recv_data
opcode, frame = self.recv_data_frame(control_frame)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/websocket/_core.py", line 408, in recv_data_frame
frame = self.recv_frame()
^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/websocket/_core.py", line 447, in recv_frame
return self.frame_buffer.recv_frame()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/websocket/_abnf.py", line 340, in recv_frame
self.recv_header()
File "/usr/local/lib/python3.11/site-packages/websocket/_abnf.py", line 296, in recv_header
header = self.recv_strict(2)
^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/websocket/abnf.py", line 375, in recv_strict
bytes = self.recv(min(16384, shortage))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/websocket/_core.py", line 531, in _recv
return recv(self.sock, bufsize)
^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/websocket/_socket.py", line 124, in recv
raise WebSocketConnectionClosedException(
websocket._exceptions.WebSocketConnectionClosedException: Connection to remote host was lost.

Hi!
The API consists of 2 modules, REST for sending commands and a web socket for receiving status. The web socket has been unstable for a long time, but unfortunately, sending a command requires an ID that can only be extracted from the status. But at the same time, if you did not reboot, HA integration may continue to work, since device identifiers are stored in the cache but without updating state.


Column resizer line not disappearing on column resize

My FixedDataTable's column resizer line is not hiding when I drag and release it, I've been trying to solve this for several hours and am not sure what is going wrong.
I've noticed that in online examples, whenever I release the resizer, an initially present class called "fixedDataTableColumnResizerLineLayout_hiddenElem" is re-added to the resizer via JavaScript, which hides the resizer.
That class isn't being re-added to my resizer. I've noticed that everything else about the resizer seems to match the online examples.
I've checked my own JavaScript, and don't think there is anything that might be interfering with FixedDataTable's own JavaScript for re-adding the class. For reference, I'm using an older React 0.14.6 (and the latest FixedDataTable 0.6.0). But looking at FixedDataTable's package.json, this seems like it shouldn't make a difference?
Thanks!

I can confirm. Having same issue.

Any solutions?

Found :) https://github.com/facebook/fixed-data-table/issues/167#issuecomment-118022854

Hey @burakcan - just read the issue you linked and it seems like the fix for one of the users was to "implement isColumnResizing"? Did anything in that issue fix it for you, and if so how did you do it? I was under the impression that isColumnResizing was implemented by FixedDataTable itself, not something that users of the library implement.

I figured it out. As it turns out, this isn't a bug - the answer is simply that if you are implementing a Table component with resizable columns, you must pass isColumnResizing={false} as props to that Table.
I realized this after reading the issue @burakcan linked above, as well as inspecting the source code for the FixedDataTable example that demonstrates resizable columns: https://github.com/facebook/fixed-data-table/blob/master/examples/ResizeExample.js#L61.
The documentation on isColumnResizing is really unclear: https://facebook.github.io/fixed-data-table/api-table.html. Both the name of the prop and the fact that it needs to be set to false don't make sense. Also, there is no in-depth documentation on it.


Update fungi-frog-maze.js

I fixed some assets on my game and the screen size of the two last levels so they could be 10x10 as well.
This is my last PR for my frog game :) 🐸

Our website is currently down due to us-east-1 AWS server issues

Yup! Thanks @LucasHT22 , I was going to mention that on the sprig channel


missing file

hello, your work is really excellent, would you mind share you fully code, I really want to learn how you implements this paper. many thanks dude.

Hi @daeing ,
Thanks for showing interest in this work.
Currently I am studying various attention papers. I will update this code base once I have reviewed the attention based papers.


Emulation - Setting percentHeight prevents layout from happening on resize

In this app
<?xml version="1.0" encoding="utf-8"?>
<s:Application
	xmlns:fx="http://ns.adobe.com/mxml/2009"
	xmlns:s="library://ns.apache.org/royale/spark"
	creationComplete="init()"
	xmlns:mx="library://ns.apache.org/royale/mx"
	>
	<fx:Script>
		<![CDATA[
			private function init():void
			{
				myGrandChildGroup.height = 100;
			}
		]]>
	</fx:Script>
	<s:VGroup id="myVGroup"
		style="background-color: red; alpha: 0.5"
		>
		<s:Group id="myChildGroup" style="background-color: blue; alpha: 0.5" height="100%" width="100%">
			<s:Group id="myGrandChildGroup" style="background-color: yellow; alpha: 0.5" width="100%"/>
		</s:Group>
		<s:Button width="100%" height="20"/>
	</s:VGroup>
</s:Application>

Button should be offset 100px from top, instead it floats at the top.

It looks like in SparkLayoutBead
	if ((!isNaN(target.percentWidth)) || (!isNaN(target.percentHeight)))
		if (!sawSizeChanged)
			return false;

Kicks in preventing proper layout from happening.

It looks like in SparkLayoutBead
	if ((!isNaN(target.percentWidth)) || (!isNaN(target.percentHeight)))
		if (!sawSizeChanged)
			return false;

Kicks in preventing proper layout from happening.

That's not the issue. The issue is that in org.apache.royale.core.LayoutBase.childResizeHandler() host.isHeightSizedToContent() returns true, which prevents performLayout() from happening.
Woraround is to use top=0, bottom=0 instead of height=100% (which seems inconsistent).

If heightIsSizedToContent, then the component should resize/layout in response to the change.  If not sized to content because it is percent sized, it would wait for the parent to layout first.
By setting height=100, then it is explicitly sized, not sized to content, but the framework may not be catching that and forcing a layout on the parent, but it should be.  The height change should cause "heightChange" event which should be picked up by myChildGroup.  That needs to trigger a re-measure and layout of the myVGroup.  myChildGroup's measuredHeight should be 100.

in org.apache.royale.core.LayoutBase.childResizeHandler() host.isHeightSizedToContent() returns true

I meant returns false, my mistake.
We have this code in UIComponent.isHeightSizedToContent
    if (!isNaN(percentHeight))
        return false;

Everything else you described works in the ChildResize branch.

I can confirm it now works in the ChildResize branch.

I can confirm it now works in the ChildResize branch.

Should this issue be closed?


Special Handling for Eq Conditions

If a rewrite as a condition of the form a = b, then it would be nice if we could check if it is satisfied, without explicitly requiring a fact of this form. For example, this is useful for automated satisfaction of rfl-conditions like 0 = 0, which can pop up in if-then-else expressions.

A relevant example is List.get?_modifyNth_eq in batteries.

Resolved by #47 and subsequent commits.


Use hatch backend

Hatch is newly added pypa backend with more flexibility, and helps avoid https://github.com/pypa/pip/issues/11110, which is breaking builds.

Codecov Report

Merging #29 (b16e1b3) into main (bd7f53e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   60.39%   60.39%
=======================================
  Files           7        7
  Lines         356      356
  Branches       75       75
=======================================
  Hits          215      215
  Misses        124      124
  Partials       17       17

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 bd7f53e...b16e1b3. Read the comment docs.


📃: adding an advanced AUTH page.

:red_circle: Title :  advanced auth page
:red_circle: Aim : this has jwt , bycrypt and proper tokenization approch to ensure proper safety.this comes with a mailing system aswell.
:red_circle: Tech Stack Used :

@Avdhesh-Varshney can you please assign me this task.

:white_check_mark: To be Mentioned while taking the issue :

Full name : T rahul prabhu
What is your participant role? GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

What will be the tech stack of this project?

mern , JWTs mailtrap and cookies

Already present here


Re-export color crate

Could you re-export color crate to allow users to use it without specifying it explicitly in Cargo.toml?
Sometimes users can be confused, see https://github.com/thscharler/spreadsheet-ods/issues/25.

I will see to this and your other issues in the next few days.

@thscharler no problem, take your time. I opened PRs to address them.

Will add this, but slightly different to your PR.


Add React/ReactDOM as peer dependencies for node package

Current behavior
In my App.jsx file, I call import GoldenLayout from 'golden-layout'; This compiles just fine, but I get the following runtime error:
Uncaught ReferenceError: ReactDOM is not defined
    at lm.utils.ReactComponentHandler._render (goldenlayout.js:4763)
    at lm.container.ItemContainer.emit (goldenlayout.js:257)
    at lm.items.Component._$init (goldenlayout.js:3408)
    at lm.items.Component.callDownwards (goldenlayout.js:2818)
    at lm.items.Stack.callDownwards (goldenlayout.js:2821)
    at lm.items.RowOrColumn.callDownwards (goldenlayout.js:2821)
    at lm.items.Root.callDownwards (goldenlayout.js:2821)
    at lm.LayoutManager._create (goldenlayout.js:1370)
    at lm.LayoutManager.init (goldenlayout.js:715)
    at mightThrow (jquery.js:3583)

For this reason I'm guessing they're being referenced as UMD globals instead of being required by the golden-layout package.
Expected behavior:
calling import for GoldenLayout would properly resolve React and ReactDOM.
Codepen example:
Codepen doesn't support npm afaik
App.jsx.txt

Hm maybe we should just add it as optional dependency? Since golden layout works well with or without React

similar to this issue, but for Webpack2 instead of Webpack1 #88

Does anyone have a temp workaround for this?

@robherley you can use the webpack ProviderPlugin

After I added ProviderPlugin as @andrewcapodieci suggested, I got another error:

Uncaught TypeError: Cannot read property '__SECRET_DOM_DO_NOT_USE_OR_YOU_WILL_BE_FIRED' of undefined
at unpkg.com/react-dom@0.14.7/dist/react-dom.js:41
at unpkg.com/react-dom@0.14.7/dist/react-dom.js:37
at unpkg.com/react-dom@0.14.7/dist/react-dom.js:40
(anonymous) @ unpkg.com/react-dom@0.14.7/dist/react-dom.js:41
(anonymous) @ unpkg.com/react-dom@0.14.7/dist/react-dom.js:37
(anonymous) @ unpkg.com/react-dom@0.14.7/dist/react-dom.js:40

Also, here is some warning:

common.bundle.js:6886 Warning: Accessing createClass via the main React package is deprecated, and will be removed in React v16.0. Use a plain JavaScript class instead. If you're not yet ready to migrate, create-react-class v15.* is available on npm as a temporary, drop-in replacement. For more info see https://fb.me/react-create-class

Can someone provide a working sample React project with webpack2/3 ?
Thanks.

@jerronl I upated the repo to use the ProvderPlugin; hope it helps.

@andrewcapodieci thanks a lot!

This means we can't use it with create-react-app without ejecting, which is a showstopper. Any chance to fix this without modifying the webpack configuration?

@cristiano-belloni how about importing React and ReactDOM then explicitly export it as global variable?
in any file:
import React from 'react';
import ReactDOM from 'react-dom';

window.React = React;
window.ReactDOM = ReactDOM;

@dvkndn That would probably work, but it's an antipattern. Wouldn't probably be better to just add React and ReactDOM as peer dependencies?

sure they should be peer dependencies :D it is just that React-related things seems not be done soon here :(

Cleaned up the suggestion @andrewcapodieci made.  In your webpack config, add:
{
  plugins: [
    new webpack.ProvidePlugin({
      React: 'react',
      ReactDOM: 'react-dom'
    })
  ]
}

@andrewcapodieci thanks for posting that example. I have a question though about
GoldenLayoutWrapper.contextTypes = {     store: React.PropTypes.object.isRequired };
in GoldenLayoutWrapper.jsx

I get this error and I'm not sure what to do with it.

Have you added prop-types package as a dependency? It looks like you are referencing PropTypes in your component.
https://www.npmjs.com/package/prop-types
M@

On 12 Feb 2018, at 07:33, Shawn Mitchell<EMAIL_ADDRESS>wrote:
@andrewcapodieci thanks for posting that example. I have a question though about
GoldenLayoutWrapper.contextTypes = { store: React.PropTypes.object.isRequired };
in GoldenLayoutWrapper.jsx
I get this error and I'm not sure what to do with it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks @mattspeller I did add that dependency.
`
import GoldenLayout from 'golden-layout'
import React from 'react'
import {Provider} from 'react-redux'
import PropTypes from 'prop-types';
import MarketTableContainer from '../containers/MarketTableContainer'
import Dummy from './Dummy'
class GoldenLayoutWrapper extends React.Component {
componentDidMount() {
// Build basic golden-layout config
const config = {
settings: {
showPopoutIcon: false,
showMaximiseIcon: false,
showCloseIcon: false
},
content: [{
type: 'row',
content:[{
type: 'react-component',
component: 'testComponent',
title: 'Orders',
props: { label: 'A' },
isClosable: false
},{
type: 'column',
content:[{
type:'react-component',
component: 'marketTableContainer',
title: 'Market Data',
props: { label: 'B' },
isClosable: false
},{
type:'react-component',
component: 'testComponent',
title: 'Blotter',
props: { label: 'C' },
isClosable: false
}]
}]
}]
};
    function wrapComponent(Component, store) {
        class Wrapped extends React.Component {
            render() {
                return (
                    <Provider store={store}>
                        <Component {...this.props}/>
                    </Provider>
                );
            }
        }
        return Wrapped;
    };

    var layout = new GoldenLayout(config, this.layout);
    layout.registerComponent('marketTableContainer',
                             wrapComponent(MarketTableContainer, this.context.store)
    );
    layout.registerComponent('testComponent', wrapComponent(Dummy, this.context.store));
    layout.init();

    window.addEventListener('resize', () => {
        layout.updateSize();
    });
}

render() {
    return (
        <div className='goldenLayout' ref={input => this.layout = input}/>
    );
}

}
GoldenLayoutWrapper.contextTypes = {
store: React.PropTypes.object.isRequired
};
export default GoldenLayoutWrapper;
`
In @andrewcapodieci repo the wrapped components import PropTypes but he doesn't import it in GoldenLayoutWrapper.jsx and there's also the issue of the React prefix. In TestComponent.jsx for instance his proptypes look like this:
TestComponent.PropTypes = {     label: PropTypes.number.isRequired }
No React.PropTypes...

It seems maybe React.PropTypes is deprecated in the latest?

Yeah you need PropTypes not React.PropTypes at bottom as React broke it out to a separate package.
import PropTypes from 'prop-types';
...
...
GoldenLayoutWrapper.contextTypes = {
store: PropTypes.object.isRequired
};
export default GoldenLayoutWrapper;
M@

On 12 Feb 2018, at 08:16, Shawn Mitchell<EMAIL_ADDRESS>wrote:
It seems maybe React.PropTypes is deprecated in the latest?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@mattspeller please make a PR on my repo if something needs to be updated; I'd appreciate it greatly

https://github.com/andrewcapodieci
@andrewcapodieci nothing to push, it was an error with the sample code posted above, not the repo.
M@

On 12 Feb 2018, at 16:48, andrewcapodieci<EMAIL_ADDRESS>wrote:
@mattspeller please make a PR on my repo if something needs to be updated; I'd appreciate it greatly
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@cristiano-belloni The maintainers of the project have been very busy. I step in for the odd issue if I have time and can see whats going on.
However a lot of the issues are not bugs and could have easily been asked on stack overflow or can be figured out with 20 mins of messing about and looking at the code. Therefore it can be time consuming to look through issues for something to work.

Thanks, @ButchMonkey. I didn't mean to criticize. I'm evaluating if I should use GL in a couple of projects and I was wondering if the project was active, dormant or abandoned.


Correct Misspelling of Tool Name

Related Issue
Closes #525

Description

Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[ ] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

You have updated the lockfileversion in pnpm-lock.yaml  so revert it


Make sure new file is tested and published

This will hopefully pass when the opposites are fixed.  Both files are tested and published for now.

Merging this so that the tests will be run on the good ontology, tests failing bc ontology bugs, which are addressed in #56


Refer to all front end toolkit assets in gov_modules

Instead of copying some files to other places. For example 'selection-buttons.js' but there may be others.

Hi @joelanman, has this issue been resolved?
The govuk folders from node_modules (govuk_frontend_toolkit, govuk_template and govuk-elements-sass) are copied to govuk_modules.


AMPI broadcasts should be [nokeep]

Original issue: https://charm.cs.illinois.edu/redmine/issues/1116

AMPI_Bcast currently calls ampi::generic with an AmpiMsg. For broadcasts, we should invoke a version of ampi::generic that is tagged [nokeep] so the runtime can avoid making multiple copies of the message on the receiver side. Then we need to distinguish between 'expected' bcasts and 'unexpected' bcasts in AMPI: if a bcast is expected (already blocked on), then we can serialize the message data directly into the user's recvbuf. If it is not expected, then we copy the message and put it in our unexpected message queue to be processed whenever the receiver gets to the bcast.
This is similar to what I've done for (i)(all){reduce,gather(v)} here: ~~https://charm.cs.illinois.edu/gerrit/#/c/1276/~~ https://github.com/UIUC-PPL/charm/commit/8b8b8aaab007023f31c571f035204ad57939d59a

Original date: 2016-08-23 15:06:56

~~https://charm.cs.illinois.edu/gerrit/#/c/1806/~~ https://github.com/UIUC-PPL/charm/commit/48836de6c768a7c53e1d63347693fb9620aff16a

Original date: 2018-11-09 16:20:36

~~https://charm.cs.illinois.edu/gerrit/#/c/charm/+/4672/~~ https://github.com/UIUC-PPL/charm/commit/7c60d20e24e37907aeef023539be273addd09c38


[Request]: gjson support wild mode like *

eg:
        jsonContent := `{"name":"john", "score":100, "childs":[{"name":"john", "score":100}]}`
	j := gjson.New(jsonContent)
	j.Remove("childs.*.score")
	fmt.Println(j.MustToJsonString())

@BeanWei Good suggestion, but it is a little bit complicated, let's make it scheduled.


Restricting to specific IP Adress not working

Description
Hi, @bjarneo appreciate your efforts.
I'm hitting this blocker when restricting the secret to a specific IP address, the secret won't open.
I tried my individual IP address and tried the CIDR/32 but this functionality isn't working for me.
I've tested this with a friend's IP address as well.
Screenshots

Additional information
I have hemmelig with docker compose behind Traefik.
hemmelig:
    image: hemmeligapp/hemmelig:latest # Check the tags for what version to use
    hostname: hemmelig
    container_name: hemmelig
    init: true
    volumes:
        - $DOCKERDIR/appdata/hemmelig/:/var/tmp/hemmelig/upload/files
        - $DOCKERDIR/appdata/hemmelig-database/:/home/node/hemmelig/database/
    environment:
        - SECRET_LOCAL_HOSTNAME=$SECRET_LOCAL_HOSTNAME # The local hostname for the fastify instance
        - SECRET_PORT=$SECRET_PORT # The port number for the fastify instance
        - SECRET_HOST=$SECRET_HOST # Used for i.e. set cors/cookies to your domain name
        - SECRET_ROOT_USER=$SECRET_ROOT_USER # User as the root admin user
        - SECRET_ROOT_PASSWORD=$SECRET_ROOT_PASSWORD # The admin user password (change this after signed in)
        - SECRET_ROOT_EMAIL=$SECRET_ROOT_EMAIL # The email for the admin user
        - SECRET_FILE_SIZE=$SECRET_FILE_SIZE # Set the total allowed upload file size in mb
        - SECRET_FORCED_LANGUAGE=$SECRET_FORCED_LANGUAGE # Set the default language for the application
        - SECRET_JWT_SECRET=$SECRET_JWT_SECRET # Override this for the secret signin JWT tokens for log in
        - SECRET_MAX_TEXT_SIZE=$SECRET_MAX_TEXT_SIZE # The max text size for the secret. Is set in kb. i.e. 256 for 256kb
    restart: always
    stop_grace_period: 1m
    healthcheck:
        test: 'wget -O /dev/null localhost:3000 || exit 1'
        timeout: 5s
        retries: 1
    networks:
      - proxy
    labels:
      - 'traefik.enable=true'
      - 'traefik.http.routers.hemmelig-rtr.rule=Host(`hemmelig.$DOMAINNAME_CLOUD_SERVER`)'
      - 'traefik.http.routers.hemmelig-rtr.entrypoints=https'
      - 'traefik.http.routers.hemmelig-rtr.tls=true'
      - "traefik.http.routers.hemmelig-svc.middlewares=chain-authelia@file"
      - "traefik.http.routers.hemmelig-rtr.service=hemmelig-svc"
      - "traefik.http.services.hemmelig-svc.loadbalancer.server.port=3000"

Hi, thank you for reporting.
While using traefik, do you know what IP header is being shipped to the Hemmelig docker instance?

This is the current approach, which is not the best. I think Hemmelig should av an option where we can set what header to look for. https://github.com/HemmeligOrg/Hemmelig.app/blob/main/src/server/helpers/client-ip.js

Hey mate,
whoami
Cf-Connecting-Ip: MyRealPublicIP
Cf-Ipcountry: AU
Cf-Ray: REDACTED
Cf-Visitor: {"scheme":"https"}
Cookie: REDACTED
Sec-Fetch-Dest: document
Sec-Fetch-Mode: navigate
Sec-Fetch-Site: none
Sec-Fetch-User: ?1
Upgrade-Insecure-Requests: 1
X-Forwarded-For: MyRealPublicIP, <IP_ADDRESS>
X-Forwarded-Host: whoami.domain
X-Forwarded-Port: 443
X-Forwarded-Proto: https
X-Forwarded-Server: REDACTED
X-Real-Ip: <IP_ADDRESS>

this is what's being passed inside the container, those IP addresses are Cloudflares.

@Tonyzaa-V, thank you!
I have added cf-connecting-ip to the list. Let us hope this fix is doing it for you: https://github.com/HemmeligOrg/Hemmelig.app/commit/33d9218a18e9fb6b7bfc6732a1baf38a9b403168
New release is v.5.9.2
🤞

It has fixed it and It's working now, really appreciate your prompt responses mate.
cheers

Awesome 👏

REDACTED

Hi, @Tonyzaa-V I faced the same issue. I think I miss configured the traefik headers. Would you please help by showing your header configuration file? TIA


[Bug] [Chat] LLMServer Generate Error, Please CheckErrorInfo.: No module named 'openai'

Search before asking

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

Operating system information
Linux
Python version information

=3.11

DB-GPT version
latest release
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[ ] Chat Knowledge
[ ] Model Management
[X] Dashboard
[ ] Plugins

Installation Information

[ ] Installation From Source

[ ] Docker Installation

[x] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
无关
Models information
LLM :  chatgpt_proxyllm
What happened

What you expected to happen
chatgpt_proxyllm  根本用不了
How to reproduce
用docker compose 起来的
需要的进到db-gpt-webserver-1 容器里面 执行 pip install "db-gpt[openai]"  才能 fix No module named 'openai'
直接   install  openai  好了
Additional context
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Search before asking

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

Operating system information
Linux
Python version information

=3.11

DB-GPT version
latest release
Related scenes

[ ]  Chat Data
[ ]  Chat Excel
[ ]  Chat DB
[ ]  Chat Knowledge
[ ]  Model Management
[x]  Dashboard
[ ]  Plugins

Installation Information

[ ]  Installation From Source
[ ]  Docker Installation
[x]  Docker Compose Installation
[ ]  Cluster Installation
[ ]  AutoDL Image
[ ]  Other

Device information
无关
Models information
LLM : chatgpt_proxyllm
What happened

### What you expected to happen
chatgpt_proxyllm 根本用不了
How to reproduce
用docker compose 起来的
需要的进到db-gpt-webserver-1 容器里面 执行 pip install "db-gpt[openai]" 才能 fix No module named 'openai'
直接 install openai 好了
Additional context
Are you willing to submit PR?

[ ]  Yes I am willing to submit a PR!
是的，是需要手动装下，后续会补充到文档里面。


Rename String.is_abs_path() to String.is_absolute_path()

This is more consistent with NodePath.is_absolute().
See https://github.com/godotengine/godot/issues/16863#issuecomment-845276240.

Thanks!

What about is_rel_path -> is_relative_path?


Add configuration option to carthage Action

By default carthage bootstrap uses the Release configuration, but in some scenarios it's helpful to switch that. See https://github.com/Carthage/Carthage/issues/281 and comment by @jspahrsummers "You could also pass --configuration Debug to Carthage, to select the Debug codesigning settings."

Sure, feel free to add the option here: https://github.com/fastlane/fastlane/blob/master/lib/fastlane/actions/carthage.rb

@adcooley that PR from @uny should take care of the issue! Please feel free to reopen this issue if necessary!


Import all roles from DPUB-ARIA and test them

ARIA role is meant to be extensible by other specifications. Digital Publishing WAI-ARIA Module 1.0 is doing just that. This change adds the new roles that DPUB ARIA defines.

Coverage remained the same at 95.993% when pulling a60e5bd58fde35088ad34ed7a4548f6fcf39126e on scienceai:dpub-aria into d8933d102741c08927edcb55ef95a4f81c633eb7 on evcohen:master.

This is awesome, thank you! I didn't know that 👍
v1.2.0 published.


Godot 4

This PR tracks the changes required to run inkgd in Godot 4. (Also see #55)
Global

[ ] Replace keywords by annotations (for instance @tool and @onready).
[ ] Replace all function names that clash with protected keywords, such as continue, assert or null.
[ ] Replace remaining spaces by tabs.
[ ] Add the super keyword in front of unqualified . accesses.
[ ] Replace Reference by RefCounted.
[ ] Update the getter / setter syntax.
[ ] Replace Array.remove() by Array.remove_at()

Runtime

[ ] Replace JSON.parse() logic by an instance of JSON.
[ ] Update all Variant.Type enum values in InkUtils.

Editor Plugin

[ ] Update the syntax connecting signals (now obj.funcA.connect(funcB)).
[ ] Rename get_icon to get_theme_icon.
[ ] Update the argument order of OS.execute.
[ ] Replace PoolStringArray by PackedStringArray.
[ ] Replace PackedScene.instance() by PackedScene.instantiate().
[ ] Review and update all LineEdit connections.
[ ] Remove Label.add_font_override() calls and use Label.font property instead.

This script might also be useful, although I wouldn't run it on the codebase as-is. https://gist.github.com/aaronfranke/79b424226475d277d0035b7835b09c5f

I converted the entire project to Godot 4 using the built-in tool, and here are some things you have to look out for:

FuncRefs and threads should use callables
Match statements need static values, so some enums have to be made static variables
Manually converting the setter/getters is a lot of work, maybe it should be automated in some way
Cyclic references are still a problem sadly

@Jummit perfect, thanks for investigating this. Did you push the conversion somewhere? I'm really curious to see what it looks like and if it maintains member orders.
I still need to land a couple of things in main before starting the conversion. I want prevent the Godot 3.x and the Godot 4.x versions of inkgd from diverging too much, but it's pretty close to being done.
We should be able to start the migration soon.

Seems that "minimum_size" was changed to "custom_minimum_size"

Cyclic references are still a problem sadly

I currently have cyclic dependencies in scripts using class_name, so it seems like it's working!

Gave this a shot again.
I removed the get_class functions and used the is operator in is_ink_class. This works, but a lot of code needs to be replaced to use class references instead of strings.
All the "imports" can be removed, as cyclic references are now possible.
Setters and getters are interesting, I think they work different than before. In 3.5 you could use the . syntax to use the getter, now it uses it by default. I think this is the main cause for issues at the moment.
Maybe I'll try again another time, but it's too tedious for me at the moment.

Closed in favour of #75.


Azure portal RBAC tool tip typo

The tooltip in Azure portal for adding a Contributer RBAC to a user on a subscription is having a typo.
It should say "..except granting access.." (https://docs.microsoft.com/en-us/azure/role-based-access-control/built-in-roles#all)
It says "..except access.."
Please below image

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

ID: 99581afb-7950-6daa-7227-055d04446b7e
Version Independent ID: f1a5bf89-9e54-aede-3a9a-70788a95927b
Content: Azure built-in roles - Azure RBAC
Content Source: articles/role-based-access-control/built-in-roles.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@santhoshb-msft Thanks for the feedback. We are looking into this and will get back to you shortly.
@rolyon FYI.

Hi @santhoshb-msft
Thanks for the feedback. I talked with the product team and they will work to update this descriptions.
thanks

Awesome, thank you @rolyon !

#please-close


Add dataset classification field in editor panel

Design: Fidesops-Management-UI (old)
When no category has been manually assigned, the selected category shows classifier's best category match.
Clicking on "search taxonomy" (left half) shows the original checkbox tree to manually set the categories.
Clicking on the selected categories area (right half) shows a dropdown with a list of the suggested categories and their confidence. Clicking on a suggested category selects it, just like selecting it from the checkbox tree.

❓Open questions:

Is there an API to assign set this field? Does it live on the dataset model? - This should ultimately be returned by this new endpoint I believe: https://github.com/ethyca/fidesctl-plus/issues/99
What even is this? - _This is the list of classification suggestions which were generated for the field. Classify will often return more than one suggestion and we want to give users the option to review and select from them if necessary. (Note: this question was asked when the design had the "classification" separate from the category dropdown. It should be clear now.)

Note: this ticket can cover updating the visual changes and selection state without worrying about how the data comes from the API. When the API is ready I'll transform the result into whatever is convenient for this interface.

@mfbrown can you confirm if the output of this glorified select multi-box is a single value, or an array of values? the left side (checkbox tree) gives an array at the moment, but the right side looks more like a single select? if someone chooses something from the right side, does that remove anything chosen by the left? or is it always adding?

We've decided that we will output arrays, and the box on the right will be a multiselect


Expose new args API hook / or neccasary internals

Is your feature request related to a problem? Please describe.
I would like to have the possibility to link a state of an interactive element in a story directly to the control/arg value in URL and use a hook API method as a change handler that modifies the URL param (which might populate the control state). So story component, control, and URL are all in sync.
Basically, the way how useArgs from Storybook works: https://storybook.js.org/docs/react/writing-stories/args#:~:text=useArgs,hook.
Here you can see useArgs in action: https://medium.com/urbandataanalytics/changing-args-values-state-in-storybook-without-usestate-hook-15cc096716f7
Describe the solution you'd like
A) API hook which is compatible with useArgs from Storybook. :)
B) Make all the necessary functions, types, and enums exported, so at least I can build the hook myself from those bits. (It's a bit difficult now).
Describe alternatives you've considered
Digging it / copying it from the ladle's source raw, starting by getQuery, etc, but that is too cumbersome and/or complicated.

You are indeed right, I had for some reason the impression that I will have to manually reparse the real values of the controls, but actually seems like it's not needed, so even smth naive like this works nicely:
type Args = Record<string, any>;

function useLadleArgs() {
  const { globalState, dispatch } = useLadleContext();

  const controlValues = Object.keys(globalState.control).reduce<Args>((acc, controlKey) => {
    acc[controlKey] =
      globalState.control[controlKey].value ?? globalState.control[controlKey].defaultValue;
    return acc;
  }, {});

  const updateHandler = (newArgs: Args) => {
    dispatch({
      type: ActionType.UpdateControl,
      value: {
        ...globalState.control,
        ...Object.keys(newArgs).reduce<(typeof globalState)['control']>((acc, controlKey) => {
          acc[controlKey] = {
            ...globalState.control[controlKey],
            value: newArgs[controlKey],
          };
          return acc;
        }, {}),
      },
    });
  };

  return [controlValues, updateHandler];
}

Thanks for the right directions. 👍


Add system test case to support custom query refactor code changes

We should add a BigQuery integration test for the custom query refactor.

This issue will be resolved with PR #679


Can't update past 0.21.2

Extension version: 0.21.2
VSCode Version: 1.51.1
OS: Mac

Steps to Reproduce:

Install version 0.21.2 on Mac
Extension doesn't get updated when clicking 'Check for Extension Updates' although 0.21.4 is the latest released version
Installing manually the 0.21.4 version (using the .vsix) is impossible (see screenshot):

Unable to install extension 'github.vscode-pull-request-github' as it is not compatible with VS Code '1.51.1'.

Extension page shows v0.21.2 as latest

This is actually working as expected. Versions 0.21.3 and higher require VSCode version 1.52 (which is the current VSCode insider's version) because of an API change. This should be released to as the stable version of VSCode this week.

Ok I see. It's not obvious when you look at the Git repository or the Marketplace what version of VSCode the extension requires. I'll wait for the next stable version then, thanks for letting me know.


[$500]  Composer height is not resized when resizing browser

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Type anything long on the first line of composer
Resize the browser window to be smaller

Expected Result:
The composer's height should increased to show all the text
Actual Result:
The composer's height (number of lines) stays the same
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: 1.3.76-1
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/ed1dd805-d1a2-47ed-8ff9-51b501b6e36c
https://github.com/Expensify/App/assets/43996225/daa245ef-ee1c-4261-a878-5bff6943c358
Expensify/Expensify Issue URL:
Issue reported by: @bernhardoj
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695059166660979
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b81aeded72e6d5a7
Upwork Job ID:<PHONE_NUMBER>042897408
Last Price Increase: 2023-10-02

Proposal by @bernhardoj
Please re-state the problem that we are trying to solve in this issue.
Resizing the browser window does not adjust the composer height/number of lines.
What is the root cause of that problem?
We don't recalculate the composer number of lines when the browser width changes. This issue was previously fixed in https://github.com/Expensify/App/pull/15020 by recalculating the composer number of lines when window width changes but a recent functional component migration accidentally removes this behavior.
What changes do you think we should make in order to solve the problem?

Recalculate the number of lines when the window width changes by adding windowWidth to updateNumberOfLines deps array.
https://github.com/Expensify/App/blob/6be8fcac69aa11ff84baaae94d3ef1d495fe5123/src/components/Composer/index.js#L369

Proposal
Please re-state the problem that we are trying to solve in this issue.
The composer's height doesn't adjust when the browser width is changed.
What is the root cause of that problem?
In Composer, updateNumberOfLines is enclosed within a useCallback, and it is not configured to update when the browser width changes.
What changes do you think we should make in order to solve the problem?
We can include windowWidth as a dependency for updateNumberOfLines.
const {windowWidth} = useWindowDimensions();
const updateNumberOfLines = useCallback(() => {
  .....
}, [value, maxLines, numberOfLinesProp, onNumberOfLinesChange, isFullComposerAvailable, setIsFullComposerAvailable, windowWidth]);

Proposal
Please re-state the problem that we are trying to solve in this issue
The composer textarea height doesn't update when the browser width is resized.
What is the root cause of the problem
updateNumberOfLines is wrapped in a callback that doesn't have windowWidth in the dependency array, thus It doesn't reflect the current width.
const updateNumberOfLines = useCallback(() => { ...  [value, maxLines, numberOfLinesProp, onNumberOfLinesChange, isFullComposerAvailable, setIsFullComposerAvailable]);
What changes do you think we should make in order to solve the problem?
adding windowWidth from useWindowDimensions() hook into updateNumberOfLines's useCallback dependency array should address the issue.
What alternative solutions did you explore? (Optional)
N/A

Hi all! Thank you all for your proposals. All of you propose the same solution for this issue so let's proceed with the first published one - @bernhardoj
🎀 👀 🎀 C+ reviewed

@kavimuru
Could you add me as the bug reporter?
I reported it earlier here
https://expensify.slack.com/archives/C049HHMV9SM/p1695059354067419?thread_ts=1695059166.660979&cid=C049HHMV9SM
https://expensify.slack.com/archives/C049HHMV9SM/p1693583529627299

@grgia waiting for your decision and assignment

@bernhardoj I need you to comment to assign.

assigning!

Hey @garrettmknight could you help ensure that the payments were processed for this one?

Summary of payments:

Reporter: @c3024 $50 paid via upwork
Contributor: @bernhardoj $500 paid via upwork
Reviewer: N/A

Upwork Job: https://www.upwork.com/en-gb/ab/applicants/1708968328042897408/job-details


document.activeElement is wrong

Version
2.6.0
Reproduction link
https://jsfiddle.net/ktsLp3n0/
Steps to reproduce
Go to the fiddle and open the console, and click between the form inputs and the page body.
What is expected?
Anytime the blur event happens the console will show which element is active by printing A or B for the two form fields or nothing when the page body is active. The fiddle shows correct behavior because it's using Vue 2.5.22
What is actually happening?
Change 2.5.22 to 2.6.0 in the script tag and you'll see the console always prints nothing which is incorrect.

The behavior of nextTick has been adjusted in 2.6. This is documented in the release note: https://gist.github.com/yyx990803/d1a0eaac052654f93a1ccaab072076dd

Even using 2.5.22 you really never see A or B logged? I've tried it in the latest firefox and chrome and it's definitely working that way in both.

The demo does what you say it does, for me at least.
Nonetheless, Evan's statement is correct. $nextTick changed for reasons outlined in the linked notes.
The solution to your problem would be to use a setTimeout instead.


cf install-plugin cflocal never finishes

Hello
cf install-plugin cflocal never finishes,
and there is no any errors or warning why it can't or what it is doing.
P.S. Environment Windows 10, Gish Bash and cmd.

the same with Windows cmd

The same for cfdev https://github.com/cloudfoundry-incubator/cfdev/issues/99
Please fill out the issue checklist below and provide ALL the requested information.

[ ] I tried updating to the latest version of the CF CLI to see if it fixed my problem.
The latest version have been just installed the same day
[ ] I attempted to run the command with CF_TRACE=1 to help debug the issue.
[ ] I am reporting a bug that others will be able to reproduce.

Describe the bug and the command you saw an issue with
Provide details on what you were trying to do (and why).
What happened
A clear and concise description of what happen.
Unknown reason for install-plugin not proceeding.
Expected behavior
A clear and concise description of what you expected to happen.
plugin installed with clear success message.
To Reproduce
Steps to reproduce the behavior; include the exact CLI commands and verbose output:

cf install-plugin cflocal

It is unknown why exactly this happens, as not enough logging from cf util.
Provide more context

platform and shell details :
Windows 10

version of the CLI you are running:
cf version 6.44.1+c3b20bfbe.2019-05-08

version of the CC API Release you are on:
How to check?

Note: As of January 2019, we no longer support API versions older than CF Release v284/CF Deployment v1.7.0 (CAPI Release: 1.46.0 (APIs 2.100.0 and 3.35.0).

Hi @paulvi thanks for raising this issue. Have you tried reaching out to the plugin authors to see if they have any insights? Their information is here: https://plugins.cloudfoundry.org/. We'll try to help when we can. Thanks for your patience.

Hi @paulvi,
We attempted to reproduce the issue locally on a Windows 10 Virtual Box image from Microsoft using cmd, and were unable to.  Also, our pipelines, which test the plugin commands, have remained green on Windows for quite a while.
Looking over the code in question, we don't really see where a "forever" hang could even happen.  The download should either fail with an error or time out.  Since we can't reproduce the problem, we're at a loss for how to help with your situation.
Our only guess would be to try the command on a different machine.
Please feel free to re-open this issue if you have ideas on how your environment may differ from ours so that we might be able to reproduce it.
Best regards,
Steve && @bwasmith


IDE: Move compiler invocation utils to break CMake cycle with Frontend

Move compiler invocation utilities up into the IDETool library which is designed to be dependent on the Frontend and FrontendTool libraries.

@swift-ci please smoke test and merge


Update to serde 1.0

Deserialize and Visitor now requires a lifetime... just a small update to add this.

Thanks!


Add support for mipsel-sony-psp

I wanted to use this to build a test runner for mipsel-sony-psp targets and figured I'd upstream. Requires #![feature(used_with_arg)] to be used for correct linking. 🙂

Published in 0.3.22.


What Is Code?

by @1wheel (1383 points, 352 comments)

by @bradezone at 2015-06-17T19:15:52.000Z
TLDR, good lord

by @thephyber at 2015-06-16T02:08:15.000Z
The Charlie Rose interview about this piece:
http://charlierose.com/watch/60575137

by @basseq at 2015-06-16T01:38:48.000Z
TMitTB (in his "mid-30s") seems in over his head. And he's not a "tech guy",  but a "Scrum Master". And he spends so much time (and money) at conferences that he is specifically brought to task for wastefulness ("he has apparently spent all of his time at conferences and no time actually working"). When he eventually delivers, months late, it's "a plain and homely thing" and he's still cagey about a go-live.Yet at the end of all this, "TMitTB will get his bonus."WHAT!?What's the message here? Big companies are hard and inefficient places? Programmers and techies are confusing and dress funny?Other than being technically illiterate, the VP seems to be the hero of this story. Not recognized as such, of course ("Money? Hours? Due date? Value? Bah!").The CTO ("who has several projects on roughly the same footing [e.g., horribly mismanged] scattered across the organization") and TMitTB should have been fired long before the 30,000 words came to a close.

by @alanrahn at 2015-06-14T22:42:04.000Z
Very interesting read... I enjoyed it.One thing I noticed though is that the author is definitely stuck in the old "Microsoft is the great Satan" mindset. If he ever finds out about all the open-source stuff MS is doing these days under Satya Nadella, I think his head would probably explode.He doesn't know what to say to a C# developer (nothing in common), but automatically trusts a Python developer?  Really?  sigh

by @mteruel at 2015-06-14T15:44:54.000Z
And this article just crossed :D

by @sullyj3 at 2015-06-14T13:14:36.000Z
I'm Aussie, I know the vice president of the US off the top of my head, but I couldn't tell you the Australian vice prime minister. Or even if that's a real thing.

by @visarga at 2015-06-14T07:26:32.000Z
>  It’s a comedy of ego, made possible by logic gates.

by @mangeletti at 2015-06-13T20:21:20.000Z
Oh that's where he got the number. I read it on mobile, so I didn't notice that.

by @Too at 2015-06-13T11:57:59.000Z
Huh, is this what caused the whole page to go pixelated and ask for permission to use my camera each time I changed orientation on my tablet? I had to reload the page each time this happened and scroll down to where I was before. Very annoying.

by @Too at 2015-06-13T11:51:54.000Z
Not likely.I've seen non developers try to write specs in whichever format they like: word, excel, drawings, hand written, in speech, mockup tools, anything. They decide exactly how they want to express their idea without any constraints. And yet, they always fail.There are always too many edge cases they do not think of. They only cover the "happy path" and quite often not even that. Just take the email conversation from the article as example, they didnt even touch the subject of implementation and it was already jibberish even for a developer. You need someone to actually sit down and squint their eyes over something, do research and run some test cases for a few hours before these emerge. Once you start doing this you are already by defition a software developer.

by @ErikRogneby at 2015-06-13T00:44:37.000Z
Not just distributed very quick but deemed worthy or unworthy very quick by the pseudo-meritocracy that is the "upvote".

by @igravious at 2015-06-12T22:52:29.000Z
ah crap, no -> any (how embarrassing (and how the hell?))

by @parasubvert at 2015-06-12T20:45:26.000Z
LOL, that's funny.  I'll admit I saw a few articles on this survey and the Daily Mail's was first on my results list.   Does cringing while posting it lend forgiveness? ;)

by @tempestn at 2015-06-12T20:20:41.000Z
Actually it says they gave the citizenship test to 1000 Americans to see how they would do.  29% didn't know who the VP was (so 71% did).  1000 isn't a massive sample, but I expect it's enough to say that the majority of American adults do know.  Anyway, it isn't a very interesting discussion regardless.

by @maratd at 2015-06-12T20:08:27.000Z
The data is from the US Citizenship test, so those taking it aren't "Americans" yet and have been specifically studying for the test.I stand by my assertion that the majority of Americans have no clue who Biden is and I swear to you, I wish I didn't either. The whole political arena is absolutely revolting and I'm not surprised the majority of people are apathetic to it.

by @DanBC at 2015-06-12T19:20:41.000Z
> Stumped: In the U.S. citizenship test, only 38 per cent of Americans passed> Although the majority passed, more than a third - 38 per cent - failed,Daily Mail is fucking useless.

by @tempestn at 2015-06-12T19:10:57.000Z
From your link: "...who is the Vice President of America? (29 per cent did not know)"So the majority of Americans do indeed know who the vice president is. (Although yes, a large minority don't.)

by @jweather at 2015-06-12T18:44:17.000Z
The certificate of participation at the end claims it's 38,000 words...

by @stigi at 2015-06-12T16:38:57.000Z
For reference: What is love by Haddaway https://www.youtube.com/watch?v=K5G1FmU-ldg

by @jameshart at 2015-06-12T16:09:40.000Z
This article is a sociology of code, not a technical manual. You absolutely could get a lot out of it.I get the GP's point about this being reminiscent of GEB, not in the sense that it covers the same topics or is at the same 'level', but in that it describes an intangible idea by approaching it from different angles and describing that same core concept from the shadow it casts in different directions. In GEB that core concept of self-reference was tackled from multiple perspectives so that an image of this common theme emerges as you read these different views onto it. Similarly, this article tries to conjure an image of 'code' as a cultural artifact, by portraying the shapes it casts in different directions - on the people who create it, the people who have to fund it, the tools and artifacts it generates. And it does so, like GEB, with wit and intelligence.

by @jameshart at 2015-06-12T16:02:21.000Z
This article doesn't attempt to answer the dull, dry, boring, well-understood question "What is code?" in the sense of "What is it that computers use to make them do what they do?". It tries to answer the much more interesting, intangible, existential question "what is code?", in the sense of what does it mean for the world to depend on a culturally isolated priesthood of technologists who control what computers do? How does that work?

by @recursive at 2015-06-12T15:58:45.000Z
I could be convinced either way.

by @apaprocki at 2015-06-12T15:58:05.000Z
The print edition has landed!https://twitter.com/ftrain/status/609388625596416001

by @jameshart at 2015-06-12T15:57:43.000Z
"This is supposed to be an introduction just to the abstract concept of code"Are you sure that's what it's supposed to be? I mean, the author didn't supply unit tests, all we have to go on is the specification, which was that the editor of BusinessWeek asked Paul Ford "Can you tell me what code is," and Paul Ford said "No", and instead wrote this.

by @rwallace at 2015-06-12T15:19:45.000Z
Everything I've heard about the lives of doctors and lawyers says that by and large (discounting the handful of percent of outliers) those professions make living on welfare or working at McDonald's for minimum wage look like brilliant career moves. I wouldn't look to them for models of how to do things.I do, mind you, think it is a good idea to encourage programmers to improve their negotiating skills and understand a little about business and politics.

by @api at 2015-06-12T14:45:51.000Z
I don't see that... or rather, if it's true here it's true to a significantly greater extent in most other industries.Money continues to be available, and often lots of it. It's available on better terms than most others in most other professions can even imagine receiving.To put it bluntly: in most industries you are meat and own nothing and never have any chance of owning anything. This has been the condition for nearly all human beings who have ever lived, today and in the past.There are also more alternatives to VC today: larger angel rounds, crowd funding, etc. It's also easier to bootstrap since everything (but people) has fallen in price. Those two things together have made the funding environment more competitive for VCs -- they have to offer more value or compete at the higher end.

by @parasubvert at 2015-06-12T14:38:42.000Z
My anecdote:  high school student in a smallish (100k) northern Canadian city, circa 1992-1996. computer science courses ran every year.  We were all taught DOS, Windows, programming in QBasic and Turbo Pascal, and (!) building web pages / using the Internet (in 1995!).The day Netscape 1.0 came out, the teacher had us all download it from a few of CDs passed around had burned after downloading it on the class modem.  The classroom was networked (coax) and figured out how to get Trumpet Winsock to work over the next few weeks to share the network with the computer with the modem.  By 1996 we had a frame relay connection.There was no curriculum other than what these two teachers could envision and sell to the achool board.  Pretty thankful for that.

by @jmccree at 2015-06-12T14:32:19.000Z
https://en.wikipedia.org/wiki/PHP#Release_history It was PHP 1.0, then PHP/FI 2.0, then became the PHP Hypertext Preprocessor at PHP 3.0 I believe.

by @eli at 2015-06-12T14:28:59.000Z
It's a joke.

by @parasubvert at 2015-06-12T14:27:02.000Z
Nope: http://www.dailymail.co.uk/news/article-1368482/How-ignorant...Lots of people hate politics and really avoid any study of it.

by @pitay at 2015-06-12T13:42:47.000Z
I have read elsewhere that programming is giving the computer instructions.  So clicking the close button on a window or typing into a word processor is technically programming (although it is not coding).  My search-fu was unable to find that though.

by @keithpeter at 2015-06-12T13:30:05.000Z
"It's 2015. The audience of this article shouldn't even exist."Well, the audience most certainly does exist whether it should or not. The thing to think about is what to do about that.

by @clebio at 2015-06-12T13:22:44.000Z
Just to clarify, my earlier comment is a direct quote from the article. I am not the author, just thought it was an apt anticipation of some of criticisms in this thread.

by @Willson50 at 2015-06-12T13:22:34.000Z
Aggregate sites like Hacker News and Reddit make distribution of news and ideas very quick and viral compared to more organic growth such as word of mouth and google.

by @rdrey at 2015-06-12T13:12:25.000Z
Well, you could read the source of course, to see whether the photo data is sent to the server or whether the cert is generated client-side. And then there is this: https://github.com/bloombergmedia/whatiscode/

by @empthought at 2015-06-12T13:04:24.000Z
You must not know many doctors.

by @dragonwriter at 2015-06-12T13:02:36.000Z
Lobbying for legislation is a tangential part of what a labor union does, and lots of organizations that aren't labor unions also do that. Labor unions first and foremost do collective bargaining on behalf of employees with the business owners who employ them. ABA and AMA don't do that.ABA and AMA are professional associations, like ACM, that happen to have effective lobbying arms, and their members are often independent business owners rather than employees.

by @zeckalpha at 2015-06-12T12:54:51.000Z
Let's add some whitespace to the README. Let's remove some whitespace from the README. Let's add some whitespace to the README.

by @meowface at 2015-06-12T12:48:06.000Z
There is a fine line between mess and art. :)

by @SSLy at 2015-06-12T12:33:33.000Z
Nope, it wasn't. recursive was later tackled on (probably still before 1.0)

by @jveld at 2015-06-12T12:10:11.000Z
To me, the defining feature is automation. A "programmable" system is one where smaller actions can be composed into larger ones and saved for invocation later (by name, in response to some event, etc). I don't know much about photoshop, but judging from what people have said, it seems to meet this definition.I will concede that "programming," in the sense of "I've been programming for the last couple hours" or "He isn't very good at programming" implies the use of a Turing complete language. Photoshop would probably fail here, along with more programmer-y things like writing html.

by @m52go at 2015-06-12T12:03:42.000Z
Haha, agreed. I'd argue there's a strong chance of that happening with any tool meant to give non-professionals a 'professional' result.WYSIWYG web editors, divorce 'kits', many DIY 'kits' for home improvement, etc. Something inevitably goes wrong and the lawyer / contractor wonders just why the hell the client didn't just call a professional in the first place.

by @meric at 2015-06-12T11:53:42.000Z
Hmm, shouldn't it be Sum(abs(deltaLoC per change))? Otherwise if you added 50 lines a day and removed 50, your abs(deltaLoC) will be 0.

by @Dewie3 at 2015-06-12T11:18:24.000Z
> It's circling the wagons around people who had the privilege and opportunity to learn these things before everybody else.The underprivileged Bloomberg readership.

by @lucozade at 2015-06-12T11:12:44.000Z
Many years ago I managed to convince my management that, if they had to use deltaLOC as a performance metric, that they at least use abs(deltaLoC).I then spent the next year cutting huge chunks of crap out of a C++ application that I had inherited.Was a most satisfying experience.

by @fit2rule at 2015-06-12T10:48:16.000Z
Keep writing.  The space is there for you.

by @sfk at 2015-06-12T10:31:44.000Z
And you are an idiot.

by @johnny22 at 2015-06-12T10:20:08.000Z
I've been involved for a long time, and it sure doesn't seem that way to me. Lots of that "code bloat" is simply making things portable across compilers, interpreters, operating systems and languages.

by @tormeh at 2015-06-12T10:07:16.000Z
I didn't really understand that part. Care to explain?

by @atemerev at 2015-06-12T09:39:30.000Z
Capitalists want people to be generally happy. Happy people buy more things. People who have useful well-paid jobs are generally happy (and can buy more things). Everybody knows that there will be huge drop in available jobs next few years, but software development skills will still be in high demand. Hence, capitalists want more people to learn how to code, so they could keep their jobs, so they can be generally happy and buy more things.Capitalism is profit-oriented, but happy people bring more profit.


Release v0.6.2

Release Checklist

[x] OWNERS must LGTM the release proposal.
At least two for minor or major releases. At least one for a patch release.
[x] Verify that the changelog in this issue and the CHANGELOG folder is up-to-date

[x] Use https://github.com/kubernetes/release/tree/master/cmd/release-notes to gather notes.
Example: release-notes --org kubernetes-sigs --repo kueue --branch release-0.3 --start-sha 4a0ebe7a3c5f2775cdf5fc7d60c23225660f8702 --end-sha a51cf138afe65677f5f5c97f8f8b1bc4887f73d2 --dependencies=false --required-author=""

[x] For major or minor releases (v$MAJ.$MIN.0), create a new release branch.

[x] An OWNER creates a vanilla release branch with
git branch release-$MAJ.$MIN main
[x] An OWNER pushes the new release branch with
git push release-$MAJ.$MIN

[x] Update the release branch:

[x] Update RELEASE_BRANCH and RELEASE_VERSION in Makefile and run make prepare-release-branch
[x] Update the CHANGELOG
[x] Submit a pull request with the changes: #1965

[x] An OWNER prepares a draft release

[x] Write the change log into the draft release.
[x] Run
make artifacts IMAGE_REGISTRY=registry.k8s.io/kueue GIT_TAG=$VERSION
to generate the artifacts and upload the files in the artifacts folder
to the draft release.

[x] An OWNER creates a signed tag running
git tag -s $VERSION
and inserts the changelog into the tag description.
To perform this step, you need a PGP key registered on github.
[x] An OWNER pushes the tag with
git push $VERSION

Triggers prow to build and publish a staging container image
gcr.io/k8s-staging-kueue/kueue:$VERSION

[x] Submit a PR against k8s.io,
updating registry.k8s.io/images/k8s-staging-kueue/images.yaml to
promote the container images
to production: https://github.com/kubernetes/k8s.io/pull/6681
[x] Wait for the PR to be merged and verify that the image registry.k8s.io/kueue/kueue:$VERSION is available.
[x] Publish the draft release prepared at the GitHub releases page.
Link: https://github.com/kubernetes-sigs/kueue/releases/tag/v0.6.2
[x] Run the openvex action to generate openvex data. The action will add the file to the release artifacts.
[x] Run the SBOM action to generate the SBOM and add it to the release.
[x] For major or minor releases, merge the main branch into the website branch to publish the updated documentation.
[x] Send an announcement email to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>with the subject [ANNOUNCE] kueue $VERSION is released.   https://groups.google.com/a/kubernetes.io/g/wg-batch/c/dmMkUxdcnog
[x] Update the below files with respective values in main branch : #1969

Latest version in README.md
Release notes in the CHANGELOG
version in site/config.toml
appVersion in charts/kueue/Chart.yaml
last-updated, last-reviewed, commit-hash, project-release, and distribution-points in SECURITY-INSIGHTS.yaml

[x] For a major or minor release, prepare the repo for the next version:

[x] create an unannotated devel tag in the
main branch, on the first commit that gets merged after the release
branch has been created (presumably the README update commit above), and, push the tag:
DEVEL=v0.$(($MAJ+1)).0-devel; git tag $DEVEL main && git push $DEVEL
This ensures that the devel builds on the main branch will have a meaningful version number.
[x] Create a milestone for the next minor release and update prow to set it automatically for new PRs:

Changelog
Changes since `v0.6.1`:

### Bug or Regression

- Avoid unnecessary preemptions when there are multiple candidates for preemption with the same admission timestamp (#1880, @alculquicondor)
- Fix Pods in Pod groups stuck with finalizers when deleted immediately after Succeeded (#1916, @alculquicondor)
- Fix preemption to reclaim quota that is blocked by an earlier pending Workload from another ClusterQueue in the same cohort. (#1868, @alculquicondor)
- Reduce number of Workload reconciliations due to wrong equality check. (#1917, @gabesaba)

### Other (Cleanup or Flake)

- Improve pod integration performance (#1953, @gabesaba)

+1

/close


KAFKA-3396 : Unauthorized topics are returned to the user

Modified KafkaApis to return Errors.UNKNOWN_TOPIC_OR_PARTITION if
principal has no Describe access to topic
Unit tests expanded
Some paths cause the client to block due to bug
https://issues.apache.org/jira/browse/KAFKA-3727?filter=-2
tests work around this by executing in separate thread

Hi this work has been done with @mimaison.
Please note that we had issues with the tests because of
https://issues.apache.org/jira/browse/KAFKA-3727
"Consumer.poll() stuck in loop on non-existent topic manually assigned"
and
https://issues.apache.org/jira/browse/KAFKA-3728

Can you please rebase this now that #1425 has been merged?

Thanks @ijuma, I did;
some tests still look ugly because of https://issues.apache.org/jira/browse/KAFKA-3727

@hachikuji Thoughts on how to handle this? It seems to me that we can't really fix this without fixing KAFKA-3727.

@ijuma I'd be happy to work on KAFKA-3727, but I think we need a bit of discussion on what the expected behavior should be. My comments are in that Jira issue https://issues.apache.org/jira/browse/KAFKA-3727

@ijuma Seems we need to decide how the client should handle non-existent topics both before and after KIP-4. I can see a case for both retrying and raising an exception. I hate to suggest it, but perhaps the behavior should be configurable?

@hachikuji I agree we can probably find use cases that justify both behaviours. What surprised us mostly was the difference between assign and subscribe.
Also for the looping behaviour, it would be nice to be able to control it with a max timeout or retries, as having to wakeup() from another thread is not very nice. In the end it sounds like a lot of new options so it might not be ideal.

@mimaison Can you clarify what surprised about the difference between assign and subscribe?

@hachikuji as we wrote in https://issues.apache.org/jira/browse/KAFKA-3727

The behavior of a consumer on poll() for a non-existing topic is surprisingly different/inconsistent
between a consumer that subscribed to the topic and one that had the topic-partition manually assigned.
The "subscribed" consumer will return an empty collection
The "assigned" consumer will loop forever - this feels a bug to me.

@edoardocomar Ah, I missed that. I agree the behavior is a little surprising, but it makes sense when I think of the implementation. We currently block setting the initial position of each partition. When using subscribe(), it's not possible to be assigned a non-existing partition so this is never a problem. But obviously a manually assigned partition for a non-existent topic is a non-existing partition, and we'll block setting the position. So to fix the inconsistency, we'll have to think through the implications of removing that blocking.

@hachikuji @ijuma why should a missing topic or partition be a retriable exception
I am thinking at least of the case where auto-create is turned off.

@edoardocomar Sorry for the slow response. The consumer has traditionally supported the ability to be started before any producers and that means retrying on unknown topics. Even if we had some way to inspect whether autocreate has been disabled, we probably still have to support this for compatibility. I'm also thinking about the impact of the CreateTopic API in KIP-4 (which is under discussion right now). The most likely eventual outcome is that autocreation will be taken out of the broker and pushed into the producer, so it seems that the picture doesn't really change for the consumer, but we should consider whether this outcome is ideal.
One option for the problem at hand is to keep the current behavior (i.e. do nothing). Users can use the partitionsFor() API to verify the existence of any needed topics and raise an exception themselves if they expect it to exist ahead of time. Alternatively, we would probably need a separate configuration that lets the consumer raise an exception instead of retrying. This would call for a new KIP. Topic deletion may be an interesting case to think about as well since the partitionsFor workaround may not help (I'm not too sure).
All that said, I don't think we need to block this issue on a resolution to that problem. It doesn't seem like a big difference to me whether we retry indefinitely because the user is not authorized to the topic or because we are waiting for a topic that may never be created. In either case, the user ultimately has to monitor the progress of the application and take steps when

My concern is that configuring security is challenging enough as it is. Configuring authorization incorrectly is very common and harder to diagnose than a topic that hasn't been created (even the latter sometimes causes people to be confused and waste time).
Replacing an exception with indefinite blocking is making the user experience during development even worse than it is at the moment (which is already suboptimal in a number of situations). Ideally, we would be moving in the other direction.
If we decide to do this, then at the very least, we need to think about how users will diagnose the problem and update the authorization documentation with the information.

@ijuma At the very least, we can change the consumer/producer log messages to warn the user that the topic might not exist or they might not have access to it. I agree this is less than ideal, but I don't see an alternative other than what I suggested above (changing how the client handles unknown topics). That is, unless we think this patch is not worth having at all. I was a little torn on this initially, but it does seem better to avoid leaking topic information.

@hachikuji The log warning you have suggested seems like our best bet at the moment.

Hi @ijuma @hachikuji
we have made a PR https://github.com/apache/kafka/pull/1535
to address the non existent topic or partition issue https://issues.apache.org/jira/browse/KAFKA-3727

I've been thinking about this and I think we should go ahead with this and then look at ways at improving it further once more of the KIP-4 work is in.
I left a few review comments. In addition to that, we should make the change to the log message as per Jason's suggestion.
Also, users can use listTopics() or partitionsFor() that don't hang as a workaround (Jason's suggestion).

Thanks for the review, I'll update the PR.

@mimaison, do you think you'll have a chance to look at this sometime soon?

Sorry for the delay, yes I'm working on it today

@mimaison no worries about the delay and there's no rush. :) I was just trying to get a rough idea about the timing. Thanks!

I've updated the PR and rebased on trunk

@mimaison, it would be good to port the tests from https://github.com/apache/kafka/commit/44ad3ec77364fab869cb5e01a182341bd0ae2698 to make sure the behaviour of pattern subscriptions is correct.

Thanks for all the feedback ! I've updated the PR

@ijuma @hachikuji I believe we've addressed all the comments, can you have another look ?

Sorry for the delay @mimaison and thanks for the reminder. Will try to look at this soon.

@mimaison, with the feature freeze in place, this is one of the main bugs we want to get in for <IP_ADDRESS>. Could you please merge trunk into this branch? KIP-79 made some changes to the list offsets request so it may need some updates. Sorry about that. :(

@ijuma @hachikuji - in hindsight, shouldn't we also handle an offsetRequest not to return TOPIC_AUTHORIZATION_FAILED where Describe auth is checked. e.g.
private def handleOffsetRequestV0(request : RequestChannel.Request) ... {
...
    val (authorizedRequestInfo, unauthorizedRequestInfo) = offsetRequest.offsetData.asScala.partition {
      case (topicPartition, _) => authorize(request.session, Describe, new Resource(auth.Topic, topicPartition.topic))
    }

    val unauthorizedResponseStatus = unauthorizedRequestInfo.mapValues(_ =>
      new PartitionData(Errors.TOPIC_AUTHORIZATION_FAILED.code, List[JLong]().asJava)
    )

it seems we had missed it

@ijuma @hachikuji the Jenkins build failure looks unrelated to our changes. Can you please review ? thanks

@edoardocomar, we should, yes. I think that was "missed" because that code was moved as part of KIP-79, but I haven't checked in detail yet. Will check the PR today or tomorrow. Thanks!

@ijuma Jenkins has failed with a timeout, the tests locally pass for me can you please kick Jenkins again ? thanks

@ijuma @hachikuji can you please take a look at the revised PR? thanks

Thanks for the feedback, we'll update the PR on Monday.

@ijuma We have handled the FETCH and PRODUCE commands

As I have squashed some commits, I am opening  a new PR


Model instance should have toJSON method

When getting a record via Model.find(), the returned object ignores the visible/hidden fields declared in the Model declaration. Also; the record itself is inside the attributes key & calling model.toJSON() doesn't work.
Getting the record via model.where().fetch() works fine. However; I can't access the first object of the returned array.
const record = yield Model.where('id', '=', 1).fetch()
console.log(record)
res.send(record[0])

When logging the record object in console, it looks like an array [{ ... }] but getting the first object of that array record[0] does not work. It does work after calling the model.toJSON() method.

Why do find() & where() behave so differently?
Why isn't it possible to get the first object of the array?

Couple of things here.
First .find method returns the model instance not a collection, so that is the reason .toJSON does not work. Ideally model instance also should have .toJSON method.
 So in order to get the first record, you have to say.
const record = yield Model.where('id', '=', 1).fetch()
console.log(record.first())

Also i am refactoring Lucid, so in coming future all these inconsistency issues will be resolved.

Any reason why find is a oh-so-special method that doesn't return a collection? Was it just forgotten?
What's the point in using Collection, if some methods use it, but others don't? Not to mention that it uses an old version of lodash...
It seems more like an inconvenience, than a convenience...

Coz Collection means collection of multiple rows not collection of a single row.
When you have multiple rows wrapped inside a collection, you can perform operations like map, filter etc. Why would you perform those operations one a single row?

where().first() returns just one element (not inside a list), but it's considered a collection (aka has a lodash wrapper, because that's really what Collection is...

If you are on adonis-lucid:^2.0 than yes. In 3.0 it will return the model instance.

Anytime when a single row is fetched intentionally, it will always be an instance otherwise a collection.

Also, I notice that whenever a Collection is used, the wrapped values are not model instances, thus creating another inconsistency.
Is this also fixed in 3.0?

Yes

@thetutlage @marines
const feed =  yield user.feed().fetch()
let index = 0
while( index < _.size(feed.value()) ) {
  // as feed.value() will fetch collection not instance of model
 // i need instance to update data in feed table
  index++
}

All good with Database query update, wanna do with db instance. Please guide.
Thanks!!


Build only for go 1.6

Because our builds starts to fail due to problems with third party tools on go 1.7 we want to run build only on stable releases.

👍


Do not render last_review_date in generated helm charts docs

Avoid changing chart content every day.

FYI @pipo02mix


[textsum]Attention heatmap error

Compared with the headmap showing in "A Neural Attention Model for Abstractive Sentence Summarization"(A.M.Rush), there are some problems here.
A heatmap should map the attention relationship between input and output, which means that it should have a higher attention if the output word is copied from input sentence. However, the heatmap on this project is just a mass, which is hard to get a point of which input word attentions which output word.

Hi, Thanks for your feedback.
The method is the project is a little bit tricky and just a short-term solution to get the Attn-Mask tensor out (Tensorflow 1.0). Since tensorflow 1.2 already deleted the old archived RNN functions and changed to  the common dynamic_rnn() function. I will find a more elegent way to do that in the new tf version. Any ideas are very welcome.

@rockingdingo
Thanks for replying.
And I have another question. Lots of methods of visualization has been focused recently, such as sensitivity analysis and relevance propogation. In which method are you going to use to generate the heat map?


The export type of namespace presents the wrong typing alert

TypeScript Version:  3.7.2
Search Terms:
namespace, typing
Code
type Nullable<T> = T | null

export namespace ProfileFields {
   type name = string;
   type WHYYY = string | null;
   type id = string | null;
  // export type WHYYY = Nullable<string>;
  // export type id = Nullable<string>;
}

export interface Profile {
  name: ProfileFields.name;
  WHYYY: ProfileFields.WHYYY;
  id: ProfileFields.id;
}

export type General = Pick<Profile, 'WHYYY' | 'name' | 'id'>

Expected behavior:
type General = {
WHYYY: ProfileFields.WHHYY
name: ProfileFields.name
id: ProfileFields.id
}

Actual behavior:
type General = {
WHYYY: ProfileFields.WHHYY
name: ProfileFields.name
id: ProfileFields.WHHYY
}

Playground Link: Provided

Two type aliases for the same type are considered interchangeable (because they are), and declaration emit can choose arbitrarily between two equivalent types when finding a name to emit.

But it make me confused.  I adjust the type of WHYYY & id to Nullable, and it make code more readable.
Has any better recommended method to use it ?

#37444  should improve this


Update image url in readme

I replaced the URL pointing to the image in the old document with the latest image on the website.
I erased the code related to the theme, but the dark mode still works. It is embedded inside the image.

I'll copy this PR into internal repo and include it for next release.

Moved and waiting approve now


Look into nested map regression

Likely a change, or possibly bug, with node-sass.
https://github.com/jgranstrom/sass-extract/blob/master/test/map-keys.js#L75-L79

Enable this test for dart-sass only.

https://github.com/sass/node-sass/issues/3383


create unit tests for cloud api

Setup running in-memory mongo for tests.
Write some tests for existing endpoints.

3/29 We are comfortable with the level of unit tests


Unable to use the new Symfony CLI

Describe the bug

When using tools: symfony-cli, we can't use the command symfony because of a symbolic link issue:
==> Setup Tools
✓ composer Added composer 2.2.6
ln: failed to create symbolic link '/usr/local/bin/symfony-cli' -> '': No such file or directory

Version

[x] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems

Ubuntu 20.04.3
PHP versions

PHP 8.0.15
To Reproduce

This is the step we use:
        - name: Install PHP
          uses: shivammathur/setup-php@v2
          with:
              php-version: ${{ env.PHP_VERSION }}
              coverage: ${{ inputs.php_coverage }}
              extensions: amqp, apcu, ctype, fileinfo, gd, iconv, imagick, intl, json, xml, mysql, redis, sockets, xml, zip, ${{ inputs.php_additional_extensions }}
              tools: symfony-cli, ${{ inputs.php_additional_tools }}

However I'm sure you can reproduce the issue with:
        - name: Install PHP
          uses: shivammathur/setup-php@v2
          with:
              php-version: 8.0
              tools: symfony-cli

Expected behavior

The symfony binary should be usable.
Screenshots/Logs

Additional context

Are you willing to submit a PR?

Also opened the issue on Symfony CLI repository: https://github.com/symfony-cli/symfony-cli/issues/105

@Kocal
I could not reproduce the issue.
https://github.com/shivammathur/test-setup-php/runs/5322510677?check_suite_focus=true
Try restarting your workflow, and if it fails again try it with shivammathur/setup@verbose and provide me the logs.

Hum that's working now, maybe that was just a temporary issue, or it has been fixed upstream.
Thanks anyway!

Closing now, open reopen if this happens again


Support 32-bit indices for reality meshes

Is your feature request related to a problem? Please describe.
RealityMeshParams.indices is a Uint16Array which means a reality mesh can contain no more than 64k/3 triangles. CesiumTerrainProvider.readMesh appears to try to handle 32-bit indices but ultimately stores them into that 16-bit array.
Describe the solution you'd like
Permit more than 64k indices.
Describe alternatives you've considered
None.
Additional context

@Ellord207 requested on behalf of users who are obtaining 256x256 elevation data from a government GIS service in their TerrainMeshProvider implementation. This already hits that maximum 64k vertices and precludes generating additional skirt vertices.
User may want the enhancement in a 3.5 patch release.


Franz in Microsoft Store (UWP App)

Feature Description
I want to install Franz directly from the Microsoft Store to use it as a much better replacement of the WhatsApp UWP App.
Motivation and Context

Easier installation
Easier updates
Easier "trust"

Thanks,
Alex

There are already requests to bring Franz to the Microsoft Store: #158 and #769
As noted in the other issues it shouldn't be too much effort to do it with the available converter. Store apps like WhatsApp and Slack are using this converter.
I also hope that we will see Franz very soon in the Microsoft Store.

Also 'works in S mode'

Plus the option to make it work with "my people" 😉


Trouble handling generated code

I'm trying to add coveralls CI with Travis to Heimdal (see also my fork).
cpp-coveralls ends up using the wrong path to generated code files.  Eventually I concluded that out-of-source builds are not well supported and modified the .travis.yml (in my fork) accordingly, but still, there's always some file cpp-coveralls cannot find, as you can see in the logs (scroll to the bottom):
Traceback (most recent call last):
  File "/home/travis/.local/bin/coveralls", line 11, in <module>
    load_entry_point('cpp-coveralls==0.3.12', 'console_scripts', 'coveralls')()
  File "/home/travis/.local/lib/python2.7/site-packages/cpp_coveralls/__init__.py", line 96, in run
    cov_report = coverage.collect(args)
  File "/home/travis/.local/lib/python2.7/site-packages/cpp_coveralls/coverage.py", line 447, in collect
    with io.open(source_file_path, mode='rb') as src_file:
IOError: [Errno 2] No such file or directory: '/home/travis/build/nicowilliams/heimdal/intr.c'

EDIT: intr.c is not generated, it is normally actually tests/bin/intr.c.  I don't know why or how the dirname of that got lost.
I'm running cpp-coveralls like this:
      coveralls --gcov-options '\-lp'       \
                -x c                        \
                -r `dirname "$PWD"`         \
                -b "$PWD"

I've tried a variety of... variations.  I've tried using -e with various forms of the troublesome filenames.

If I use in-tree builds and drop the -r and -b and -x options I still get this:
Traceback (most recent call last):
  File "/home/travis/.local/bin/coveralls", line 11, in <module>
    load_entry_point('cpp-coveralls==0.3.12', 'console_scripts', 'coveralls')()
  File "/home/travis/.local/lib/python2.7/site-packages/cpp_coveralls/__init__.py", line 96, in run
    cov_report = coverage.collect(args)
  File "/home/travis/.local/lib/python2.7/site-packages/cpp_coveralls/coverage.py", line 447, in collect
    with io.open(source_file_path, mode='rb') as src_file:
IOError: [Errno 2] No such file or directory: '/home/travis/build/nicowilliams/heimdal/libtommath/bn_mp_reduce_2k_l.c'


Master Child Form??

Then form for above two tables is like this:

@Muhammadinaam no. You need to do this manually by your own way. The Aim of this package is to create simple CRUD with some customizable options and to reduce & save a time of generating various files and simple redundant apis.


Boot to main Rom after SD2 card failure?

Is there a way to boot arkos from the main sdcard in the event that sd2 is corrupted? Because as it is now the main sd won't boot with a new sd2 card because it can't be initialized by the SD2 rom script in the options menu.  The only option I see in this situation is writing a new main sdcard image in order to get a bootable system to then be able to run the SD2 rom script.

No.  Just provide another sd card with the same format of the last sd card and it should boot at that point.


Add eslint-config-react-app dependency

Resolves error ERROR in Failed to load config "react-app" to extend from. when starting app on Mac

This is will be included in https://github.com/quay/quay-ui/pull/1


⚠ Bofrost.lu has degraded performance

In 58eadb3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1144 ms

Resolved: Bofrost.lu performance has improved in 1ee6a68 after 6 minutes.


Cross-platform T4 template generation

This PR addresses #381.
I don't think these should be invoked with every build so as not to slow down build.cmd and build.sh further. There should be a check during CI that the generated code is fresh, like was done in 8ffe07c060c7d8a51243a6a149c4ef8a43d366a3 for #235.

@fsateler You okay with my follow-up changes & explanation? Hopefully it's a quick one to approve for merge. 😉

can fix that up in a later PR

Done too in 082382c74ceffd88a942652dbab9c62552994ceb.


Output file Naming

The output file name is determined from the config project value. This is different to other builders an can cause problems in "surrounding" build systems triggering for dependencies and installations.
E.g. the html builders use the root_doc/master_doc value from the conf.py to write to <root_doc>.html.
The pdf/latex builders us the sphinx.util.osutils function make_filename_from_project to generate filename friendly output filenames.
Should this be "sanitized" like for the pdf builder or does this make no sense?
This could e.g be a problem using spaces, unicode or German Umlaute....

I think we should provide a simplepdf_output option for conf.py, which sets a name defined by the user.
If not given, the current behavior shall be used.
This would make the usage in CI more stable, as the final pdf filename does not change, even if the project name go changed slightly.

As i see the project name refers to a title (cover) and therefore you are right. :+1:  for the option to specify output name in config.
The name of the option may be similar to an existing configuration value, but here is no conformity in the builders.
Best approach seems to be latex_ oder texinfo_documents where these options are set via "nice" names.
But you are totaly free


initial changes

Currently our API allows only one perturbation acceleration. However, in case of J3/J4 perturbations, their effect is usually accounted together with, say, J2 perturbation. So it is reasonable to allow passing multiple perturbation forces at once.

And also, if we try to be very general and play too much with partial objects, I fear we might have a hard time in the future accelerating these with numba. Now we're blocked by https://github.com/numba/numba/issues/2952, but I suppose that this will be corrected in the future.

Let's better close this and follow a different approach.


Flutterbum: Archiving Projects

Issue 1: When user clicks the 'trash' icon on a project, in the modal dialogue that pops up, the 'Close' and 'X' buttons don't work.
Issue 2: Clicking 'OK' completely deletes the project. Shouldn't it Archive it, instead? This would be very bad for a company using this app to lose all records of finished projects, I think it should work more with the archiving system and you can see Completed Projects.

Fixed the Close and Cancel buttons.

I agree with your Archive statement, but whenever I exclude the delete functionality, I receive several emails asking how to delete. ;)

Haha fair enough! I’m still plugging away through the videos (which are great). After I build the app I plan to go through and custom design it, tweak some functionality, etc… so if I come up with any good solutions, I’ll let you know! Though of course I defer to your expertise :)
I admit though that as I watch you type, a lot of these functions seem like magic to me. Like “jeez, where does that come from?” Do you go through and read all the docs for every package, and for Meteor itself? Any recommended method for getting such familiarity with everything?

On Feb 2, 2015, at 3:39 PM, George Mcknight<EMAIL_ADDRESS>wrote:
I agree with your Archive statement, but whenever I exclude the delete functionality, I receive several emails asking how to delete. ;)
—
Reply to this email directly or view it on GitHub https://github.com/mcknightg/flutterbum/issues/16#issuecomment-72562711.

I definitely could do a better job slowing it down, and explaining more.  I familiarize myself by doing it over and over again.  I sit down with an idea, and say "how long will it take me to build a ..."  social app, project management, etc.  Then I build everything and trip several times before I figure out a good way.  Then... I have it down.

Thanks for all of the great comments.  I appreciate you helping me to make this better.


ARROW-6311: [Java] Make ApproxEqualsVisitor accept DiffFunction to make it more flexible

Related to ARROW-6311.
Currently ApproxEqualsVisitor will accept a epsilon for both float and double compare, and the difference calculation is always Math.abs(f1-f2)
For some cases like Validator it is not very suitable as:
i. it has different epsilon values for float/double
ii. it difference function is not Math.abs(f1-f2)
To resolve these, make this visitor accept both float/double epsilons and diff functions.

cc @pravindra

the change lgtm. some minor questions

is it useful to have different epsilons for floats and doubles (i guess a double epsilon can cover both)?
did you consider using a Java.Util.Comparator instead of the diff functions ?

Thanks for your comments,
for 1, most cases an epsilon can cover both, but in some cases they may be different, such as Validator.java(1.0E-6f and 1.0E-12f for each). And we have different constructor(with one epsilon for both or have two epsilon) for different cases.
for 2, the diff functions are used to calculate differences between values like Math.abs(f1-f2) by default or something as below in Validator.java, and maybe other implementations.

float average = Math.abs((f1 + f2) / 2);
float differenceScaled = Math.abs(f1 - f2) / (average == 0.0f ? 1f : average);

java.util.comparator seems only compare two values which is larger i think can’t handle this?
Thanks!