"Failed to download" error for ALOS FNF Microsoft PC dataset.

Hello!
Everything with the package is amazing so far. I am having some trouble with accessing the ALOS FNF dataset using the get_stac_data function. I've spent some time debugging this, but am currently feeling stuck.
To access ALOS FNF data from the planetary computer, I am running this code:
alos_coll <- "alos-fnf-mosaic"
alos_assets <- "C"
pc_stac_source <- "https://planetarycomputer.microsoft.com/api/stac/v1/"

rsi::get_stac_data(
  aoi = sf_poly,
  start_date = "2015-01-01",
  end_date = "2021-12-31",
  asset_names = alos_assets,
  pixel_x_size = 25,
  pixel_y_size = 25,
  stac_source = pc_stac_source,
  collection = alos_coll,
  sign_function = rsi::sign_planetary_computer,
  download_function = rsi::rsi_download_rasters,
  composite_function = NULL,
  output_filename = tempfile(pattern="fnf_df_", tmpdir=tempdir(), fileext=".tif")
)

After attempting to run, I am seeing these errors:
Error: [rast] file does not exist: /var/folders/2s/hq96vnhj3w37048l_1m9r2j00000gq/T//Rtmpc4xFEv/filede08ae0aa18.tif
In addition: Warning messages:
1: Failed to download N11W085_16_FNF from 2016-01-01T00:00:00Z
2: Failed to download N11W084_16_FNF from 2016-01-01T00:00:00Z
3: Failed to download N11W085_15_FNF from 2015-01-01T00:00:00Z
4: Failed to download N11W084_15_FNF from 2015-01-01T00:00:00Z
5: /var/folders/2s/hq96vnhj3w37048l_1m9r2j00000gq/T//Rtmpc4xFEv/filede08ae0aa18.tif: No such file or directory (GDAL error 4)
Warning messages:
1: In file.remove(unlist(download_results)) :
  cannot remove file '/var/folders/2s/hq96vnhj3w37048l_1m9r2j00000gq/T//Rtmpc4xFEv/filede08ae0aa18.tif', reason 'No such file or directory'
2: In file.remove(unlist(download_results)) :
  cannot remove file '/var/folders/2s/hq96vnhj3w37048l_1m9r2j00000gq/T//Rtmpc4xFEv/filede0823bf133c.tif', reason 'No such file or directory'
3: In file.remove(unlist(download_results)) :
  cannot remove file '/var/folders/2s/hq96vnhj3w37048l_1m9r2j00000gq/T//Rtmpc4xFEv/filede085587da79.tif', reason 'No such file or directory'
4: In file.remove(unlist(download_results)) :
  cannot remove file '/var/folders/2s/hq96vnhj3w37048l_1m9r2j00000gq/T//Rtmpc4xFEv/filede084a1869cd.tif', reason 'No such file or directory'

I know that rsi uses some rstac functionality to access the data, so I tried to download the ALOS FNF dataset using exclusively rstac to make sure there wasn't something funky happening there:
# Query for ALOS FNF data
stac_query <- rstac::stac(
  "https://planetarycomputer.microsoft.com/api/stac/v1"
)  |>
  rstac::stac_search(
    collections = "alos-fnf-mosaic",
    bbox = sf_bbox,
    datetime="2015-01-01/2021-12-31"
  ) |>
  rstac::get_request()

# Sign urls with PC authentication
signed_stac_query <- rstac::items_sign(
  stac_query,
  rstac::sign_planetary_computer()
)

# Get URLs
fnf_urls <- rstac::assets_url(signed_stac_query, asset_names = "C")
  # this returned 12 URLs. example:
  # "https://pceo.blob.core.windows.net/palsar/v200/alos_fnf_mosaic/2020/N15W085/N11W085_20_C.tif?st=2024-07-04T17%3A33%3A46Z&se=2024-07-05T18%3A18%3A46Z&sp=rl&sv=2024-05-04&sr=c&skoid=146be268-41f7-4e92-96ad-9cf622e35904&sktid=72f988bf-86f1-41af-91ab-2d7cd011db47&skt=2024-07-05T00%3A43%3A58Z&ske=2024-07-12T00%3A43%3A58Z&sks=b&skv=2024-05-04&sig=QpISmO8vWi0lmJWVy3E5uLOmzktVHh053LArjsq3kzE%3D"

# Use `stars` package to read and plot data
library(stars)
test_fnf <- stars::read_stars(fnf_urls[1])
plot(test_fnf)

The stars package was able to read in and plot the data. I then looked into the rsi source code, found the download_function which calls to the "Failed to download" error I was seeing:
tryCatch(
  {
     sf::gdal_utils(
       "warp",
       paste0("/vsicurl/", url),
       dl_location,
       options = current_options,
       quiet = TRUE,
       config_options = gdal_config_options
      )
     },
     error = function(e) {
       rlang::warn(
         glue::glue(
           "Failed to download {items$features[[which_item]]$id %||% 'UNKNOWN'} from {items$features[[which_item]]$properties$datetime %||% 'UNKNOWN'}" # nolint
          )
         )
       download_locations[which_item, ] <- NA

I manually replicated the sf::gdal_utils function outside of my script, using the fnf_urls variable from before to see if the utility failed outside of the code. However, I was able to successfully run the gdal warps utility manually.
I'm wondering if the url variable within the source code gets impacted in some way before the gdal_utils function is called. Maybe it's something to do with the PC signing that is unique to the ALOS FNF dataset? I haven't had any other issues using get_stac_data for other PC datasets like ESA Worldcover, for example, so it's just a bit strange that it's happening with this dataset.
Let me know if there is anything you would like for me to try, and thank you so much for all the work you've put into rsi !!!

The issue seems to be on PC's side of things; for instance, here's a valid STAC item:
https://planetarycomputer.microsoft.com/api/stac/v1/collections/alos-fnf-mosaic/items/N37W082_16_FNF
But if you scroll down to the "assets" and click the link for C, you get an error message:
https://pceo.blob.core.windows.net/palsar/alos_fnf_mosaic/N37W082_16_C_F02DAR.tif
I opened a discussion on the Planetary Computer repo to see what's up, but I have no idea if/when those assets will get restored:
https://github.com/microsoft/PlanetaryComputer/discussions/367
Now that said: rsi should give you the data that it could download anyway; that's why this is a warning and not an error in the code. I'll try and fix that issue.

Ah okay, that makes sense for why I was running into some trouble. Thank you for opening that discussion!

I'm marking this as closed because the rsi bit of the issue -- the entire download failing when only some assets couldn't be downloaded -- is now fixed in #75. Thank you so much for opening the issue -- your examples made this one pretty easy to debug!


unable to install better-sqlite3 on Linux(Centos Stream 9 ) Server

I am trying to deploy strapi application on cenos stream 9 serve.
installed node V22.0 +,   and trying to install the dependencies.   While my serve is in China so the network may has problem.
according to error below, it shows that (gyp http fetch GET https://nodejs.org/download/release/v20.16.0/node-v20.16.0-headers.tar.gz attempt 1 failed with ETIMEDOUT).  I can download it locally and upload to server. While I donot know which folder should I put into .?
yarn install     yarn install v1.22.22
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
[4/5] Linking dependencies...
warning "@strapi/plugin-cloud > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-cloud > @strapi/design-system > @uiw/react-codemirror > @uiw/codemirror-extensions-basic-setup ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@strapi/plugin-graphql ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "@strapi/plugin-graphql ><EMAIL_ADDRESS>has unmet peer dependency "koa@^2".
warning " ><EMAIL_ADDRESS>has unmet peer dependency "react-is@>= 16.8.0".
warning "styled-components > babel-plugin-styled-components ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning Workspaces can only be enabled in private projects.
[5/5] Building fresh packages...
[6/10] ⠐ sharp
[9/10] ⠐ @apollo/protobufjs
[-/10] ⠐ waiting...
[-/10] ⠐ waiting...
error /www/wwwroot/strapi_neon/strapi_blog/node_modules/better-sqlite3: Command failed.
Exit code: 1
Command: prebuild-install || node-gyp rebuild --release
Arguments:
Directory: /www/wwwroot/strapi_neon/strapi_blog/node_modules/better-sqlite3
Output:
prebuild-install warn install Request timed out
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| linux | x64
gyp info find Python using Python version 3.9.19 found at "/usr/bin/python3"
gyp http GET https://nodejs.org/download/release/v20.16.0/node-v20.16.0-headers.tar.gz
gyp http fetch GET https://nodejs.org/download/release/v20.16.0/node-v20.16.0-headers.tar.gz attempt 1 failed with ETIMEDOUT
gyp WARN install got an error, rolling back install
gyp ERR! configure error
gyp ERR! stack FetchError: request to https://nodejs.org/download/release/v20.16.0/node-v20.16.0-headers.tar.gz failed, reason:
gyp ERR! stack at ClientRequest. (/usr/lib/node_modules/npm/node_modules/minipass-fetch/lib/index.js:130:14)
gyp ERR! stack at ClientRequest.emit (node:events:519:28)
gyp ERR! stack at _destroy (node:_http_client:880:13)
gyp ERR! stack at onSocketNT (node:_http_client:900:5)
gyp ERR! stack at process.processTicksAndRejections (node:internal/process/task_queues:83:21)
gyp ERR! System Linux 5.14.0-480.el9.x86_64
gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild" "--release"
gyp ERR! cwd /www/wwwroot/strapi_neon/strapi_blog/node_modules/better-sqlite3
gyp ERR! node -v v20.16.0

On Linux node-gyp should download and extract files to /tmp. So unpacked it should look like
/tmp/prebuild/node/20.16.0/
├── include
└── installVersion

But regarding this, node-gyp would be a better place to ask.
I would rather try to deploy a prepacked bundle with the all required (native) modules instead of manually recreating an auto setup build environment.


Fix setrank & fire

I got a bug report that setrank and fire was broken so I can test later if this fixed it.

Didn't work


Report duplicates in configuration as Configuration issues

I noticed that knip reports unused items in ignoreDependencies for example, but doesn't report if ignoreDependencies contains duplicated entries, for example:
{
  "ignoreDependencies": [
    "package-a",
    "package-a"
  ]
}

I think it'd be nice if knip would report such issues

Any option could be checked for duplicates. Is there any tool that actually does this? I'm kinda worried to overdo this kind of checks. Maybe there's even a way to do this statically with JSON Schema and/or TypeScript?

with JSON Schema

No, it doesn't support such things

TypeScript

If we somehow enforce this at the type level, users will not be able to dynamically pass string[] values

I'm kinda worried to overdo this kind of checks

This makes sense. I totally understand you, thanks for the reply!
This feature is pretty nit-picky, so if you think it shouldn't be presented in the Knip, let's close the issue

Closing this as "wontfix" for now.


Message delete by using message ID in android

Hello,
I need to delete message from chat dialog screen? So how can I delete single message at a time when user click on this any message?
I noticed that the message ID is changing when UI refresh or re enter the chat dialog. So how can I  implement this message delete fucntion?
Please help

hello @firojandev, message ID never can change, maybe you mean message's position in adapter? You can realize deleting messages for example by long press on message bubble. For deleting message you can use QBRestChatService.deleteMessage(messageId, forceDelete). You can set forceDelete=true for own messages and forceDelete=false for opponent's messages. Additionally you should provide for notifying opponents about force deleting ща message, you can use System Notification or common messages with parameter save_to_history=false, after receiving this messages opponent should update UI and DB for deleting message.

Thanks for replying....
I have done below process
Step 1:
I have created a interface like below
public interface QBBubbleClickListener {
void onBubbleLongClicked(QBChatMessage qbChatMessage, int positionInAdapter);
}
Step 2:
At the QBChatMessagesAdapter, I use
holder.bubbleFrame.setOnLongClickListener(new View.OnLongClickListener() {
@Override
public boolean onLongClick(View view) {
qbBubbleClickListener.onBubbleLongClicked(chatMessage,position);
return false;
}
});
Step 3:
At the BaseDialogActivity class
protected class BubbleLongClickListener implements QBBubbleClickListener{
@Override
public void onBubbleLongClicked(QBChatMessage qbChatMessage, int positionInAdapter) {
ToastUtils.shortToast("MessageId:"+qbChatMessage.getId()+" dialogId:"+qbChatMessage.getDialogId()+" Message Body:"+qbChatMessage.getBody());
Glog.logMsg(TAG,"MessageId:"+qbChatMessage.getId()+" dialogId:"+qbChatMessage.getDialogId()+" Message Body:"+qbChatMessage.getBody());
deleteMsg(qbChatMessage.getId(),positionInAdapter);
}
}
public void deleteMsg(final String messageId,final int positionInAdapter){
    QBRestChatService.deleteMessage(messageId, false).performAsync(new QBEntityCallback<Void>() {
        @Override
        public void onSuccess(Void aVoid, Bundle bundle) {
            hideProgress();
            Glog.logMsg(TAG,"onSuccess delete message");
            if (combinationMessagesList.size()>0){
                combinationMessagesList.remove(positionInAdapter);
                messagesAdapter.notifyDataSetChanged();
            }
        }

        @Override
        public void onError(QBResponseException e) {
            Glog.logMsg(TAG,"onError delete message:"+e.toString());
            hideProgress();
            ToastUtils.shortToast("Failed to delete message!");
        }
    });
}

Result:
Exception happen. No resource found against message ID
So what I make mistake?????

Are you sure that message with id 5a7dac0d9d02de5d0ac059af was sent success before, and this message had parameter save_to_history=true? You can check it in logs. Check your adapter, how you get chatMessage from viewHolder, this is the same message?

Hi,
At my QBMessagesAdapter
@Override
public void onBindViewHolder(QBMessageViewHolder holder, int position) {
T chatMessage = getItem(position);
int valueType = getItemViewType(position);
switch (valueType) {
case TYPE_TEXT_RIGHT:
Log.e(TAG, "onBindViewHolder TYPE_TEXT_RIGHT");
onBindViewMsgRightHolder((TextMessageHolder) holder, chatMessage, position);
break;
case TYPE_TEXT_LEFT:
Log.e(TAG, "onBindViewHolder TYPE_TEXT_LEFT");
onBindViewMsgLeftHolder((TextMessageHolder) holder, chatMessage, position);
break;

protected void onBindViewAttachRightHolder(ImageAttachHolder holder, final T chatMessage, final int position) {
setDateSentAttach(holder, chatMessage);
displayAttachment(holder, position);
int valueType = getItemViewType(position);
String avatarUrl = obtainAvatarUrl(valueType, chatMessage);
if (avatarUrl != null) {
displayAvatarImage(avatarUrl, holder.avatar);
}
holder.bubbleFrame.setOnLongClickListener(new View.OnLongClickListener() {
@Override
public boolean onLongClick(View view) {
qbBubbleClickListener.onBubbleLongClicked(chatMessage,position);
return false;
}
});

Surprisingly ...
When I enter chat screen every time I get new messageId for same message
Example
---> MessageId:5a7dac0d9d02de5d0ac059b1 dialogId:null Message Body:hi this is for delete test msg
--->MessageId:5a7dac0d9d02de5d0ac059cf dialogId:null Message Body:hi this is for delete test msg
-->MessageId:5a7dac0d9d02de5d0ac059ed dialogId:null Message Body:hi this is for delete test msg
When I click on that message bubble after entering chat screen I got above  message ID.
I so frustrated.....
Any suggestions?

Hi @tatanka987
Just for your kind information....I am using Quickblox Adavnced plan.

looks like you call 'new QBChatMessage()' somewhere in your code and use it message for delete, please check it.

Thanks @tatanka987
Did you mean I am creating new instance of QBChatMessage() at the QBChatMessagesAdapter?  OK... I am checking...

yes, because constructor QBChatMessage() every time generates new message id, from your issue's description I did this conclusion.

I didn't call 'new QBChatMessage()' other place in code.... I checked....

Hi @tatanka987
We would be very benefited if this message delete feature by using message ID was implemented with q-municate-android project.


[dart2native] Embed resource files into Dart2Native executables

Currently resources files are excluded from the native binary created using dart2native tool and have to provide all the resource files relative to the binary location.
Eg: I have an https server where I wanted to embed the server TLS certs (self-signed) along with the binary. I am using the following code to read the resource data.
var cert = await Resource("lib/certs/cert.pem").readAsString(encoding: utf8);
var key = await Resource("lib/certs/cert.key").readAsString(encoding: utf8);

But the generated binary doesn't include the resource files and always looks for the data under lib/certs location.  One trick is to copy/paste that content as a dart string literal, but not very useful for binary or large data.

It seems package:resource is discontinued - https://pub.dev/packages/resource . For dart-native, I think it would still be useful to embed stats asset like in flutter.

Is this still not supported in plain Dart? I know that Go added support for this, even C apparently can do it now... so I thought it's a good time to ask if this feature has been or could be added in the immediate future?


Classes and styles not aplied after Gatsby Build

I have been experiencing some unexpected behavior when trying to use styled-components in a Gatsby build that only seems to happen in production builds. I have a hook that changes the value of props being passed to a styled component. When the page is loaded in the browser it appears as though the styles are correct in react dev tools but the DOM does not reflect these changes.
It works as intended with npm start, but not with npm build
Example CodeSandbox: https://codesandbox.io/s/styled-components-issue-2629-js89j
Example Repo: https://github.com/BraveLilToaster/styled-components-issue-2629
Environment
System:

OS: Linux 4.15 Ubuntu 18.04.1 LTS (Bionic Beaver)
CPU: (8) x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
Memory: 5.31 GB / 15.12 GB
Container: Yes
Shell: 5.4.2 - /usr/bin/zsh

Binaries:

Node: 10.16.0 - ~/.nvm/versions/node/v10.16.0/bin/node
Yarn: 1.12.1 - /usr/bin/yarn
npm: 6.9.0 - ~/.nvm/versions/node/v10.16.0/bin/npm

npmPackages:

babel-plugin-styled-components: ^1.10.1 => 1.10.1
styled-components: ^4.3.1 => 4.3.1

Reproduction

Create new Gatsby project from the default-starter
npm install --save gatsby-plugin-styled-components styled-components babel-plugin-styled-components
Add a styled component that takes a prop like color.
Add a hook to set the color based on screen size.

Steps to reproduce

Run npm run build
Run npm run serve

Expected Behavior
Screen size is detected to be over 900px wide. Value of props passed down to the styled components change. Classes/styles update in the DOM.

Actual Behavior
Screen size is detected to be over 900px wide. Value of props passed down to the styled components. Classes/styles update in react dev tools, but the DOM maintains the same classes/styles from the initial load.

You should consider reading this documentation about styled component with gatsby
You forgot to add gatsby-plugin-styled-components in your dependencies.

Thanks @Slashgear. I totally forgot to add gatsby-plugin-styled-componets to gatsby-config.js in the code sandbox. I do have it on my local environment and am still experiencing this issue in production builds. I have updated the CodeSandbox so you can see it working in the dev environment, and I am including a repo to clone since I was unable to find a way to run the production build in CodeSandbox.

@BraveLilToaster I just booted up your sandbox and it looks like it's working as expected?

@probablyup thanks for checking out my issue. I have no problem getting it to work in the sandbox because it is using the the Development build of Gatsby. When I try it with the production build I experience the issue that I reported. I included a repo so you may try it for yourself. Apologies for any confusion.

@probablyup, did you get a chance to try building for production yet. I am still experiencing this bug and haven't made any headway.

This is a weird one... I see the class being injected into the stylesheet but the dynamic style class on the HTML itself isn't changing...
Aside from that you're actually violating the rules of hooks with your useMedia hook. Rewritten to be compliant:
import { useCallback, useMemo, useState, useEffect } from "react"

function useMedia(queries, values, defaultValue) {
  console.log("check media query", queries, values)

  // Array containing a media query list for each query
  const mediaQueryLists = useMemo(
    () =>
      typeof window === "undefined"
        ? []
        : queries.map(q => window.matchMedia(q)),
    [queries]
  )

  // Function that gets value based on matching media query
  const getValue = useCallback(() => {
    // Get index of first media query that matches
    const index = mediaQueryLists.findIndex(mql => mql.matches)

    // Return related value or defaultValue if none
    return typeof values[index] !== "undefined" ? values[index] : defaultValue
  }, [defaultValue, mediaQueryLists, values])

  // State and setter for matched value
  const [value, setValue] = useState(getValue)

  useEffect(
    () => {
      // Event listener callback
      // Note: By defining getValue outside of useEffect we ensure that it has
      // current values of hook args (as this hook callback is created once on mount).
      const handler = () => setValue(getValue)

      // Set a listener for each media query with above handler as callback.
      mediaQueryLists.forEach(mql => mql.addListener(handler))

      // Remove listeners on cleanup
      return () => mediaQueryLists.forEach(mql => mql.removeListener(handler))
    },

    [] // Empty array ensures effect is only run on mount and unmount
  )

  return value
}

export default useMedia

I even debugged the s-c injection phase and see the correct class going through into render. Appears to some sort of strange React behavior

cc @gaearon there might be a react bug in production mode w/ hooks here. It definitely can see a render() pass happening with the correct props and the DOM partially updates (the explicit text of "color") in production mode but the className is not updated in the DOM.
If a render happens before rehydration completes would that explain devtools having correct values but the DOM being out of sync?

@probablyup, thank you for taking a look at this and for helping me make the useMedia. I put an issue in the Gatsby with as well. They suggested that the issue may be a hydration problem too.

Hey @probablyup and @gaearon,
Here is a link to the response I got from The issue I opened with Gatsby.
From my limited understanding it seems like it is an issue of a mismatch in the HTML and the virtual DOM. Is this right or am I misunderstanding something?
From https://reactjs.org/docs/react-dom.html

React expects that the rendered content is identical between the server and the client. It can patch up differences in text content, but you should treat mismatches as bugs and fix them. In development mode, React warns about mismatches during hydration. There are no guarantees that attribute differences will be patched up in case of mismatches. This is important for performance reasons because in most apps, mismatches are rare, and so validating all markup would be prohibitively expensive.

Hi, I've come across a similar issue. The way I came up to alleviate this was to render null at first and then display the component. Hardly a fix, but at least the layout is correct on initialization.
In your example the IndexPage would look something like this
const IndexPage = () => {
  const color = useMedia(["(min-width: 900px)"], ["green"], "red")
  const [render, setRender] = useState(false)

  useEffect(() => {
    // setRender(true)
    const timeout = setTimeout(() => setRender(true), 100)
    return () => clearTimeout(timeout)
  }, [])

  if (!render) return null

  return (
    <Layout>
      <SEO title="test" />

color: {color}

      <ColorBox color={color}>

box should be green on screens larger than 900px

      </ColorBox>
    </Layout>
  )
}

The pro is that layout is correct. The con is that the page will always blink when reloading and takes ~100ms to appear.

We're experiencing this same issue.
System:
OS: macOS 10.14.5
Node: 10.16.0
Yarn: 1.17.3
npm: 6.9.0
gatsby: ^2.15.14
babel-plugin-styled-components: ^1.10.6
styled-components: ^4.3.2
Everywhere on the site where we have a variable (enhanced comes from localstore) that affects the style, these styles are NOT be applied/rendered.
Example:

Everything above in that style definition DOES render... but everything in that !enhanced test DOES NOT. Even though it work in dev/local and works if you make the value of enhanced MATCH what was rendered during SSR.
Also, when we TEST that enhanced value inside of the style definition is consoles as the correct false value. So we know styled-components is getting the right value.... its just not applying THESE styles dependant on this variable to the component on hydrate.

Can people test this with v5 rc3?

@decepulis thanks for putting this minimal repro together 🙌🏻
Can confirm this same behavior, even with the current latest versions of all of the packags
  "dependencies": {
    "babel-plugin-styled-components": "^1.10.7",
    "gatsby": "^2.20.13",
    "gatsby-plugin-styled-components": "^3.2.1",
    "react": "^16.13.1",
    "react-dom": "^16.13.1",
    "styled-components": "^5.1.0"
  },

This is what we're running into, isn't it?
https://joshwcomeau.com/react/the-perils-of-rehydration/

@decepulis Thank you for the example repo! This was (I guess still is) driving me crazy.
I have nearly identical code to yours and exact same issues: My theme styles are loaded perfectly from localStorage in development mode. But when it comes to production, only some styles load and some do not (seems to be all the child components that do not load the theme styles). But if I were to change the theme through user interaction, voila! Everything works perfectly.
I currently am using your "useEffect" temporary solution. I suppose the "flash" is better than the mess of colors of multiple themes. I suppose we're still not at a resolution to this issue?

I haven’t tried it myself because my projects have taken me a different direction, but Josh Comeau has a great follow-up article to the one I posted earlier that presents a potential solution to this hydration-caused flash.
https://www.joshwcomeau.com/gatsby/dark-mode/
His solution isn’t specific to styled-components but maybe his work could still point you in the right direction!

@BraveLilToaster did you solve your initial issue?
I followed the response you got on your issue within Gatsby project. In my case I could see this error in dev mode:
Warning: validateDOMNesting(...):
 cannot appear as a descendant of
.

It was an element a bit earlier than the element I had problem with. When I fixed that, everything was back to normal. Hopefully this helps someone out there.

@decepulis Thank you for the example repo! This was (I guess still is) driving me crazy.
I have nearly identical code to yours and exact same issues: My theme styles are loaded perfectly from localStorage in development mode. But when it comes to production, only some styles load and some do not (seems to be all the child components that do not load the theme styles). But if I were to change the theme through user interaction, voila! Everything works perfectly.
I currently am using your "useEffect" temporary solution. I suppose the "flash" is better than the mess of colors of multiple themes. I suppose we're still not at a resolution to this issue?

I too am having almost an identical issue. Its a bit disheartening.

I haven’t tried it myself because my projects have taken me a different direction, but Josh Comeau has a great follow-up article to the one I posted earlier that presents a potential solution to this hydration-caused flash.
https://www.joshwcomeau.com/gatsby/dark-mode/
His solution isn’t specific to styled-components but maybe his work could still point you in the right direction!

Update -- In the above comment, i mentioned I was feeling pretty disheartened by the hydration issue. I just read Josh's blog post on rehydration and setting up dark mode in Gatsby and I'm feeling much better about his solution! I'm definitely going to give it a try!

So how did you guys solved this ? we are facing the same issue with a simple styled component..

So how did you guys solved this ? we are facing the same issue with a simple styled component..

I had to rework the way I was implementing dark mode entirely and follow the approach the Josh w Comedy took for dark mode here. It wasn't ideal but worked well for my case.
https://www.joshwcomeau.com/gatsby/dark-mode/

This seems like an external issue, but for now I'm closing this, since the topic is "SC with X" and not an issue with styled-components on its own or itself.


Mosaic and Zoom

Version

Development Environment

Current Behavior

hello, this is a great imageEditor, i use it in my project,  but my boss want Mosaic and Zoom feature, i try but fail. so i want to ask if you have plan to add these feature.
Expected Behavior

I'm not sure what the mosaic feature is. A little explanation would help you decide.

How about the zoom feature @jinwoo-kim-nhn ? Will the zoom feature on repo's feature list?

How about the zoom feature @jinwoo-kim-nhn ? Will the zoom feature on repo's feature list?

well,you can add this code in 'graphics.js'
but in other tools like crop,you may need to use canvas.setZoom(1) to reset the zoom ratio.
_onMouseWheel(fEvent) {
        const delta = fEvent.e.deltaY / 1000;
        let zoom = this._canvas.getZoom();

        zoom = zoom - delta;
        if (zoom >= 5) {
            zoom = 5;
        }
        if (zoom <= 1) {
            zoom = 1;
        }
        this._canvas.setZoom(zoom);

        this._canvas.calcViewportBoundaries();
        fEvent.e.preventDefault();
        fEvent.e.stopPropagation();
    }

@cocoroise thanks for the guide code and file.
here is a demo of fabric canvas that demonstrate zoom in zoom out on mousewheel
http://fabricjs.com/fabric-intro-part-5
by combining the hammer.js touch gesture library, you might implement zoom in zoom out on pinch on mobile device
http://hammerjs.github.io/
hope it helps

would be nice to have two buttons integrated for zoom in/out

I created a Fork to address Zoom and Pan issue for myself .

@cocoroise thanks for the guide code and file.
here is a demo of fabric canvas that demonstrate zoom in zoom out on mousewheel http://fabricjs.com/fabric-intro-part-5
by combining the hammer.js touch gesture library, you might implement zoom in zoom out on pinch on mobile device http://hammerjs.github.io/
hope it helps

thanks, it works for me!


Review

Description

Code review of the entire module. Refer to the Files changed for most of the review comments.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Appropriate labels applied
[x] Targeted PR against correct branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Apart from the comments and suggestions in the Files changed section, I'm also curious about something. This module essentially enables governance to take tokens from any address.

What about situations where this is misused such as to move tokens out of a whale's address into somewhere else? Of course this will require governance to agree to do this, but the large amount of tokens in the whale's address might be a big enough incentive.

@migueldingli1997 You're understanding how the module works correctly. It requires governance to agree to do it. We design the module excluding the possibility for the community to agree such proposal and it gets passed.

Of course, as you said, it can act as a malicious attack vector, but governance takes two weeks, and in the meantime, if the whale finds the possibility of its address being used as a budget source, it's enough time to move the asset to another wallet, Another usecase can also be used as a means to move the asset from a wallet that has lost its private key.
However, it is true that this can confuse other module ecosystems. However, in order to use inflation as a budget source, It is the biggest purpose and a primary use case of the module. this was the best to develop it without maximum dependence on the current sdk module structure, where assets minted in the FeeCollector are collected. If the mint and distribution module are improved in a more general direction in the future, the budget module will also be modified accordingly.

Close the issue because all suggestions have been discussed or resolved. Thank you again.


TestCheckReportDisable fails

https://circleci.com/gh/istio/istio/48740?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

Most likely related to the recent change https://github.com/istio/istio/pull/4073
When wait for TcpPort,  it will send Mixer a Check call,  it will mess up Check count.

I'll let you developers more knowledgeable in this to design a fix. For
now, disabling.
On Wed, Mar 14, 2018 at 4:00 PM Wayne Zhang<EMAIL_ADDRESS>wrote:

My suggestion is to revert that change, and come up some other ways to
make sure TcpPort is ready before Tcp Mixer tests.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/4265#issuecomment-373203321, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJGIxpVu_5DYx1H3uvSf-Ir9s38PnzZsks5teaEZgaJpZM4SrBrt
.


Treating unnamed nodes and repeat nodes as the same isn't valid

Alternative parsers emitting lezer trees will also sometimes use anonymous nodes, so the isRepeat flag on node types, which is used to check whether they need to be included in iteration, is too coarse.

This has been replaced by isAnonymous


support CEIL operator for ArmNN

I use ARM-NN v22.02 release package ArmNN-linux-x86_64.tar.gz load yamnet model,but get some error:

error message:
Can't load libOpenCL.so: libOpenCL.so: cannot open shared object file: No such file or directory
Can't load libGLES_mali.so: libGLES_mali.so: cannot open shared object file: No such file or directory
Can't load libmali.so: libmali.so: cannot open shared object file: No such file or directory
Couldn't find any OpenCL library.
Error: Failed to parse operator #6 within subgraph #0 error: Operator not supported. subgraph:0 operator:6 opcode_index:6 opcode:104 / CEIL  at function ParseUnsupportedOperator [/devenv/armnn/src/armnnTfLiteParser/TfLiteParser.cpp:960]
terminate called after throwing an instance of 'armnn::ParseException'
  what():  Failed to parse operator #6 within subgraph #0 error: Operator not supported. subgraph:0 operator:6 opcode_index:6 opcode:104 / CEIL  at function ParseUnsupportedOperator [/devenv/armnn/src/armnnTfLiteParser/TfLiteParser.cpp:960]

yamnet link address:
https://www.tensorflow.org/hub/tutorials/yamnet
So how fix it?

So CEIL operator not support by ArmNN?

Hi @choujayyl
CEIL is not currently supported in the ArmNN TFLiteParser you can see a list of the supported operators here:
https://arm-software.github.io/armnn/latest/parsers.xhtml#S6_tf_lite_parser
If you want to you could use the ArmNN TFLite Delegate instead of the TFLite Parser.
If the TFLite parser encounters an unsupported operator then it exits with the error you've seen but if the TFLite delegate encounters an unsupported operator then those will be handled by the default TFLite implementation.
How are you running the model? If you're using ExecuteNetwork you can add -T delegate to the command line to run your model using the ArmNN TFLite Delegate.
Something like
ExecuteNetwork -f tflite-binary -m //path/to/model.tflite  -i inputnames -o outputnames  -c CpuRef -T delegate

@MikeJKelly
I was try ExecuteNetwork with delegate,cmd

./ExecuteNetwork -f tflite-binary -m ./lite-model_yamnet_tflite_1.tflite -T delegate -i waveform  -o Identity_2 -c CpuRef -d ./1.PCM

but get some error:
Info: ArmNN v28.0.0 Info: Initialization time: 0.18 ms. INFO: TfLiteArmnnDelegate: Created TfLite ArmNN delegate. ERROR: Attempting to use a delegate that only supports static-sized tensors with a graph that has dynamic-sized tensors. Fatal: Could not register ArmNN TfLite Delegate to TfLiteInterpreter! Info: Shutdown time: 0.01 ms.

@MikeJKelly
Any suggestion?

Hi @choujayyl
you can add to your executeNetwork command: --infer-output-shape
it seems your model does not have explicitly mentioned all the shapes
for more options you can do ./ExecuteNetwork --help
Regards

How to download binaries for ArmHF platform. Official documents listed platform for arm64 but I want to implement on arm v7-l(armhf).


Exclude Maven built-in property from circular placeholder reference check

What's changed?
Exclude Maven built-in property from circular placeholder reference check
What's your motivation?
Error Circular placeholder reference 'XYZ' in property definitions is thrown when a built-in property, such as project.version, is redefined in the root POM and used in a child in multi-module projects.
I would not expect running recipes to fail given that mvn clean verify succeeds.
Attached is a minimum project to reproduce the issue.
Circular-placeholder-reference.zip
You can run any recipe to trigger it. I ran the one for SpringBoot
mvn -U org.openrewrite.maven:rewrite-maven-plugin:run -Drewrite.recipeArtifactCoordinates=org.openrewrite.recipe:rewrite-spring:RELEASE -Drewrite.activeRecipes=org.openrewrite.java.spring.boot3.UpgradeSpringBoot_3_2

Anything in particular you'd like reviewers to focus on?

Anyone you would like to review specifically?
@timtebeek
Have you considered any alternatives or workarounds?

Any additional context

Checklist

[x] I've added unit tests to cover both positive and negative cases
[x] I've read and applied the recipe conventions and best practices
[x] I've used the IntelliJ IDEA auto-formatter on affected files

That does not sound a like unit test for the changed method. If I wanted to add a unit test, which test class would I add it to?

I'd say MavenParserTest is the best place to guarantee that your issue is fixed, and remains fixed with a test similar to this one.
https://github.com/openrewrite/rewrite/blob/ad91222a2869e7baba013394c80ea349822397e7/rewrite-maven/src/test/java/org/openrewrite/maven/MavenParserTest.java#L3042-L3109

Thank for the suggestion! Working on adding a test there helped me find the correct strategy.


Add CalculiX

Calculix - Three-Dimensional Structural Finite Element Program
Homepage: http://www.calculix.de
Packages on other distros:
AUR: https://aur.archlinux.org/packages/calculix (PKGbuild)
FreeBSD: https://www.freshports.org/cad/calculix

We can’t reopen this issue until a PR is opened, unfortunately, consider making a pull request!


Use a slightly darker background

Describe your changes
Make the dark darker
Checklist before requesting a review

[X] I have performed a self-review of my code

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality
[ ] Major changes
[X] Minor changes
[ ] Bug fixes

[ ] Proposed changes are described in the CHANGELOG.md (no, not yet released)

[ ] CI Preview succeeds and looks good!

Before:

After:


Filtering GET results - update section 8.3 in API DG

Clarifications and example of comparison attributes in 8.3 Filtering
What type of PR is this?
Add one of the following kinds:

documentation

What this PR does / why we need it:
Clarification and example how to use filtering attributes using comparisons like 'greater or equal'
Which issue(s) this PR fixes:
Fixes #10
Special notes for reviewers:
Modified original examples as  not clear
Changelog input
Clarification for filtering attributes of GET request using comparisons like 'greater or equal'  added

Additional documentation
This section can be blank.
docs

Here if we read the API guidelines designed, we are applying operators gt, lt etc to enums too, that means that we are allowing to compare strings with lt, gt.... that not make sense.
In adition we are not allowing with the actual guidelines to compare a value that is exactly that value , we have not defined the equals operator. Then my proposal is basing on the TMF 630 use:

For numeric and dates value allow:
.gt > /  .gte >= /  .lt < /  .lte <= /  .eq = / .neq !=
And for string enums:
.eq = / .neq !=

"For operations on numeric, date or enumerated fields, the parameters with the suffixes .(gte|gt|lte|lt)$ need to be defined, which should be used as comparators for “greater - equal to, greater than, smaller - equal to, smaller than” respectively. Only the parameters needed for given field should be defined e.g. with .gte and .lte suffixes only."
As I exposed , one string cannot be gt o lt than other. My comments was not take into account.


add clarifying info on how MF joins works

this pr adds more detail about the joins supported by mf. currently it mentions left joins but there's no info on outer joins or semantic valiations. this pr adds more info on the types of joins available, when they're used, semantic validations available, and practical sql examples
raised by internal slack
Resolves #6650

🚀 Deployment available! Here are the direct links to the updated files:

https://docs-getdbt-com-git-update-mf-joins-dbt-labs.vercel.app/docs/build/join-logic

thanks @runleonarun and great flags! have made add'l tweaks and would love a re-review please


logos added

Issue #17
Logo added for Blogger

Thanks. Also won't we have tags for this merged PR ? @Swarnimashukla @Kajol-Kumari


Generate Bellatrix (Merge) fork test vectors

Issue
The fork test generator currently doesn't generate Bellatrix (Merge) fork test vectors.
How to fix it
Update fork test generator main.py

fixed by #2777


RFC: Remove item-level updates from IGListCollectionContext, require them done in batch updates

It is way too easy to execute an item-level update out-of-sync with coalesced batch updates and break things. We've observed this in Instagram. Although rare, at our scale its still a measurable problem.
By item-level updates, I'm specifically talking about:

Reload items
Insert items
Delete items
Reload the whole section controller

The Problem
The preferred way to use them is to batch update your state (e.g. w/e powers numberOfItems) and item changes:
[self.collectionContext performBatchAnimated:YES updates:^{
  NSMutableArray *models = [self.models mutableCopy];
  [models removeObjectAtIndex:2];
  self.models = models;
  [self.collectionContext deleteInSectionController:self
    atIndexes:[NSIndexSet indexSetWithIndex:2]];
} completion: nil]

However I've seen lots of examples not-using the batch update block:
NSMutableArray *models = [self.models mutableCopy];
[models removeObjectAtIndex:2];
self.models = models;
[self.collectionContext deleteInSectionController:self
  atIndexes:[NSIndexSet indexSetWithIndex:2]];

We even do this in one of the examples.
The problem w/ this is that if a batch update is in progress, you might update while the UICollectionViewData (private structure) is changing the state of the UICollectionView. While this is all main-thread affined, the whole operation happens over multiple runloop turns, so it should be treated as an async operation (e.g. racing).
Proposal
I propose we remove all of the item-update operations from IGListCollectionContext so they cannot be done w/out batch updates. Instead, we provide a new IGListBatchUpdateContext object as a parameter to the update block that has all of the above batch update methods.
So now your batch updates would look like:
[self.collectionContext performBatchAnimated:YES updates:^(IGListBatchUpdateContext *context) {
  NSMutableArray *models = [self.models mutableCopy];
  [models removeObjectAtIndex:2];
  self.models = models;
  [context deleteInSectionController:self atIndexes:[NSIndexSet indexSetWithIndex:2]];
} completion: nil]

Considerations
That way the design and compiler forces when we can update. The only thing we can't enforce is when the state changes happen. We can use documentation and examples to try get the message across, but there's nothing stopping someone from doing:
// BAD!
NSMutableArray *models = [self.models mutableCopy];
[models removeObjectAtIndex:2];
self.models = models;

[self.collectionContext performBatchAnimated:YES updates:^(IGListBatchUpdateContext *context) {
  [context deleteInSectionController:self atIndexes:[NSIndexSet indexSetWithIndex:2]];
} completion: nil]

Maybe we could add some fancy asserting or something just before performBatchUpdates: that checks the data source's numberOfSections and numberOfItemsInSection: and throws if the data source and UICV are out of sync?
That might be useful as its own thing. We could pinpoint which section controller is borked.

💯

Wow this is great! I just started learning IGListKit a couple days ago and was wondering how those two update APIs would play together.
What would it mean for two section controllers to do something like this in the same run loop? Would the updates be coalesced and performed all in one batch?
Tangential but related: if I wanted to build a fully data driven arch (~ #38) would something like this be appropriate?
// Pseudocode
- (void)didUpdateToObject:(id)object {
  self.model = object;
  NSArray *oldItems = self.items;
  self.items = [self.class itemsFromObject:object];
  IGListDiff *diff = …;
  [self.collectionContext applyDiff:diff inSectionController:self];
}

Another tangential point, would performUpdatesAnimated:additionalUpdates:completion: or something similar be more appropriate? It (1) shows that this is the method for updating whether it's batch or no, and (2) indicates that the updates you provide are in addition to the section-level updates from the list adapter.

Maybe we could add some fancy asserting or something just before performBatchUpdates: that checks the data source's numberOfSections and numberOfItemsInSection: and throws if the data source and UICV are out of sync?

Here the consumer of the API is alredy on the path down a slipperly slope of "unkowns errors so if you can help him "do the right thing" early on, this sounds like a good idea.

Diff up internally to make this change! 🎉

This change actually broke my project. I had a list of sections where each section could be expanded when selected, but if I selected another section, the previously selected section would collapse and then the new section would expand. By expanding a section I mean adding some cells to it.
In a section controller I maintained an expanded variable and when it changed its value I would call reloadSectionController. So basically there were two reloadSectionController calls at a time (one for collapsing the previously expanded section and the other for expanding the new one) and it worked fine. But now it doesn't.
Calling - performBatchAnimated:updates:completion twice at a time causes an exception.
Here is the code I used before:
var expanded = false {
    didSet {
        collectionContext?.reload(self)
    }
}

and after:
var expanded = false {
    didSet {

        let updates = { (context: IGListBatchContext) in
            context.reload(self)
        }

        collectionContext?
            .performBatch(animated: true, updates: updates, completion: nil)
    }
}


Organization Email Templates is now in a different location

In the following link: https://docs.microsoft.com/en-us/dynamics365/commerce/email-templates-transactions
Says the following:
"In Commerce headquarters, go to Retail and Commerce > Headquarters setup > Organization email templates."
However, when looking for that in the application, it does not shows up:

When I search for it, is on a different location (App version 10.0.10, PU43) :

The thing is to please change the location of the Organization email templates in the documentation.

@aleguerrero, thanks for the feedback.
@josaw1, please review.

@bicyclingfool, please review.

FYI: It appears to depend on which sku you have. I'll update the doc to call out that the other location

Just submitted a pull request to address this: https://github.com/MicrosoftDocs/Dynamics-365-Operations/pull/8390


restarting the server and refreshing admin pahe causes TypeError

When I restart the server and refresh the admin page the session seems to get corrupted.
Can't convert nil into String
File: cookie.rb
Location: hexdigest
Line: 152
Also I remember that admin used to work without cookies in an earlier padrino version, can that be?

Hi pke, this is a Rack bug (see https://github.com/rack/rack/issues/223). You can fix this by adding gem 'rack', '1.4.0', :git =>  "git://github.com/rack/rack.git" to your Gemfile.
Hope this helps!

Ah, thanks. I'll do that.
From: Michael Maltese
Sent: 17.01.2012 01:05
To: Philipp Kursawe
Subject: Re: [padrino-framework] restarting the server and refreshing
admin pahe causes TypeError (#752)
Hi pke, this is a Rack bug (see
https://github.com/rack/rack/issues/223). You can fix this by adding
gem 'rack', '1.4.0', :git =>  "git://github.com/rack/rack.git" to
your Gemfile.
Hope this helps!

Reply to this email directly or view it on GitHub:
https://github.com/padrino/padrino-framework/issues/752#issuecomment-3520840

According to my bundler I already have 1.4.0 installed.
On Tue, Jan 17, 2012 at 1:05 AM, Michael Maltese <
<EMAIL_ADDRESS>
wrote:

Hi pke, this is a Rack bug (see https://github.com/rack/rack/issues/223).
You can fix this by adding gem 'rack', '1.4.0', :git =>  "git:// github.com/rack/rack.git" to your Gemfile.
Hope this helps!

Reply to this email directly or view it on GitHub:
https://github.com/padrino/padrino-framework/issues/752#issuecomment-3520840

Same issue here, add  the rack gem to Gemfile and all works now.
Thanks mike

I have this in my Gemfil
gem 'rack', '1.4.0'
and its not working.
How do you guys boot up the server?
I am using
bundle exec padrino start
On Wed, Jan 18, 2012 at 4:27 PM, rex79 <
<EMAIL_ADDRESS>
wrote:

Same issue here, add  the rack gem to Gemfile and all works now.
Thanks mike

Reply to this email directly or view it on GitHub:
https://github.com/padrino/padrino-framework/issues/752#issuecomment-3547348

You need the github version of the gem, so in your gem file you have to write:
gem 'rack', '1.4.0', :git => "git://github.com/rack/rack.git"
after this, launch "bundle install"
and then restart the server, it should work

I'll try that
From: rex79
Sent: 18.01.2012 17:41
To: Philipp Kursawe
Subject: Re: [padrino-framework] restarting the server and refreshing
admin pahe causes TypeError (#752)
You need the github version of the gem, so in your gem file you have to write:
gem 'rack', '1.4.0', :git => "git://github.com/rack/rack.git"
after this, launch "bundle install"
and then restart the server, it should work

Reply to this email directly or view it on GitHub:
https://github.com/padrino/padrino-framework/issues/752#issuecomment-3548761

This helps indeed. Why does it have to be the git version of the gem?


feat: skip deps when installed is false

fixes: https://github.com/helmfile/helmfile/issues/1053

@mumoshu @xiaomudk test locally. all is ok.


Update rails and other gems

Along with fixing some deprecated things...

I'll wait until this is reviewed before fixing the merge conflicts...

I believe the rebase doesn't require reviewing again, so I'm going to go ahead and merge this.


empty col when exporting csv

When trying to export a csv with the following command:
api.exporter.csvExport(uiGridExporterConstants.ALL, uiGridExporterConstants.ALL);

The following column get always exported as an empty value (value is undefined) can you help me ?
{
    enableColumnMenu: false,
    enableCellEdit: false,
    enableSorting: false,
    name: 'packing',
    width: 150,
    displayName: $translate('APP.UI.catalog.grid.packing'),
    cellClass: 'ui-grid-cell-contents unselectable',
    cellTemplate: '<div>{{ (row.entity.quantity || 1) | number:0 }} {{ row.entity.unit }}</div>'
},

It doesn't have a field, it has a custom template that is rendering values from other columns.  Exporter cannot get at that value, it doesn't read the content off the screen, it gets it from the underlying data.
If you bound your column to a function instead, then that would likely work.  To do that you have to iterate through your data array adding a function to each row - the function would be something like:
  data[i].packing = function() {
    return packing = (data[i].quantity || 1) + ' ' + data[i].unit;
  ];

And then use packing as your column field.  See the binding tutorial for examples.

works great, thank you :)


Log files filling up disk

Looks like log files are rotated daily and a new log is created every day but the problem is, old log files are not automatically removed. As a result, the logs are filling up the disk quickly.
How can I disable the log rotation by traccar so as to use the Linux logrotate instead?

I found a way to do it for the wrapper.log :
wrapper.logfile=/var/log/traccar/wrapper.log
#wrapper.logfile=/var/log/traccar/wrapper.log.YYYYMMDD
#wrapper.logfile.rollmode=DATE
but I'm still looking for doing it to the tracker-server.log file.
any help will be appreciated :)


1_download.sh uses -C option on git command but git doesn't recognize it.

I changed it to -c (lower case) and that worked.

-c and -C are not equivalent, see https://www.kernel.org/pub/software/scm/git/docs/#_options
What git version do you have? -C should be available since 1.8.5

Ah, i see. My version is <IP_ADDRESS>. After updating to 2.11 it works fine. Thank you.

Thanks for getting back to me Lukas. I've managed to get to 3.0. The build
completes but the configuration test loses its mind.  Any thoughts? (the
odd characters are being displayed)
Thank you in advance.
my g++ version is g++ (Ubuntu/Linaro 4.7.3-2ubuntu1~12.04) 4.7.3
Running configuration tests...
Checking for gold linker...

cd /home/rjackson/raspi/src/qtbase/config.tests &&
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++
-fuse-ld=gold -o conftest-out conftest.cpp

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
1:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
ELF : not found

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
2:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
cannot open
: No such file

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
2:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
@ @@@@@0 0 p p @p @ @@4�
4�
: not
found

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
3:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
Syntax error: word unexpected (expecting ")")
test config.qtbase.tests.use_gold_linker FAILED
Checking for machine tuple...
+
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++
-dumpmachine

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
1:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
ELF : not found

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
2:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
cannot open
: No such file

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
2:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
@ @@@@@0 0 p p @p @ @@4�
4�
: not
found

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
3:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
Syntax error: word unexpected (expecting ")")
test config.qtbase.tests.machineTuple FAILED
Checking for valid makespec...

cd /home/rjackson/raspi/src/qtbase/config.tests/common/verifyspec &&
/home/rjackson/raspi/src/qtbase/bin/qmake "CONFIG -= qt debug_and_release
app_bundle lib_bundle" "CONFIG += shared console" "QMAKE_CFLAGS +=
--sysroot=/home/rjackson/raspi/sysroot" "QMAKE_CXXFLAGS +=
--sysroot=/home/rjackson/raspi/sysroot" "QMAKE_LFLAGS +=
--sysroot=/home/rjackson/raspi/sysroot" -early "CONFIG += cross_compile"
/home/rjackson/raspi/src/qtbase/config.tests/common/verifyspec

Info: creating stash file
/home/rjackson/raspi/src/qtbase/config.tests/.qmake.stash
/home/rjackson/raspi/src/qtbase/mkspecs/features/toolchain.prf:129:
Variable QMAKE_CXX.COMPILER_MACROS is not defined.

cd /home/rjackson/raspi/src/qtbase/config.tests/common/verifyspec &&
MAKEFLAGS= /usr/bin/make

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++
-c -march=armv8-a -mtune=cortex-a53 -mfpu=crypto-neon-fp-armv8
-mfloat-abi=hard --sysroot=/home/rjackson/raspi/sysroot -O2 -Wall -W -fPIC
-I. -I/home/rjackson/raspi/src/qtbase/mkspecs/devices/linux-rpi3-g++ -o
verifyspec.o verifyspec.cpp

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
1:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
ELF : not found

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
2:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
cannot open
: No such file

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
2:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
@ @@@@@0 0 p p @p @ @@4�
4�
: not
found

/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
3:
/home/rjackson/raspi/tools/arm-bcm2708/arm-rpi-4.9.3-linux-gnueabihf/bin/arm-linux-gnueabihf-g++.real:
Syntax error: word unexpected (expecting ")")

make: *** [verifyspec.o] Error 2
Note: Also available for Linux: linux-clang linux-kcc linux-icc linux-cxx

ERROR: Cannot compile a minimal program. The toolchain or QMakeSpec is
broken.
On Thu, Mar 16, 2017 at 12:00 PM, Lukáš Novotný<EMAIL_ADDRESS>wrote:

-c and -C are not equivalent, see https://www.kernel.org/pub/
software/scm/git/docs/#_options
What git version do you have? -C should be available since 1.8.5
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Kukkimonsuta/rpi-buildqt/issues/2#issuecomment-287104553,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIB5xNEA5sl2akhBQsTvvt9mPjV6jK0Dks5rmVy5gaJpZM4MfgZy
.

Version of your g++ should be irrelevant since we are using precompiled toolchain downloaded and patched in 1_download.sh.
Are you running the script 3.0_build_qtbase.sh or ./configure / make manually?

Running the scripts.
The only variables I modified in env.sh is the IP address of the Pi.
On Thu, Mar 16, 2017 at 1:25 PM, Lukáš Novotný<EMAIL_ADDRESS>wrote:

Version of your g++ should be irrelevant since we are using precompiled
toolchain downloaded and patched in 1_download.sh.
Are you running the script 3.0_build_qtbase.sh or ./configure / make
manually?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Kukkimonsuta/rpi-buildqt/issues/2#issuecomment-287131175,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIB5xBsryktgAIK6C5lly2DX8XiUT0h_ks5rmXCTgaJpZM4MfgZy
.

Maybe re-check that 2_sync.sh successfully copied all files from the pi /lib, /usr/include, /usr/lib, /opt/vc to sysroot, but other than that I'm sorry, I don't know.

Okay, well thanks for looking into it for me. Congrats on getting it to
work. Many people have spent many days. I'm already on day 3 and no closer.
Regards,
Richard
On Thu, Mar 16, 2017 at 1:44 PM, Lukáš Novotný<EMAIL_ADDRESS>wrote:

Maybe re-check that 2_sync.sh successfully copied all files from the pi
/lib, /usr/include, /usr/lib, /opt/vc to sysroot, but other than that I'm
sorry, I don't know.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Kukkimonsuta/rpi-buildqt/issues/2#issuecomment-287137084,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIB5xLrBSwYFkZ327ByQIhN8ehbHXUWaks5rmXUbgaJpZM4MfgZy
.

I understand the pain, it took me few days to get it running too. Would it help you if I published binaries built on and for ubuntu 16.04 x64 and rpi3?

It would help. I think my problem is my Linux distro is a VM on my Windows
laptop and it's way out of date - like 12.04. I suspect that's where I'm
falling down. I have a new machine waiting to be set up so I'll get 16.04
on it and try again.
Actually, my friend just succeeded and he's using 16.04  - I guess that
answers that.
So what are you intending on doing with your Qt based Pi?
On Thu, Mar 16, 2017 at 2:18 PM, Lukáš Novotný<EMAIL_ADDRESS>wrote:

I understand the pain, it took me few days to get it running too. Would it
help you if I published binaries built on and for ubuntu 16.04 x64 and rpi3?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Kukkimonsuta/rpi-buildqt/issues/2#issuecomment-287147026,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIB5xFOT6gBsg2P6M9Hetn4xToTAt-HWks5rmXzxgaJpZM4MfgZy
.

https://github.com/Kukkimonsuta/rpi-buildqt/releases/tag/20170317
I'm not sure if it's gonna be of any help since it's built for different ubuntu version, but it's worth a shot. Glad to hear your friend successfully built it.
We were doing experiments with video playback on some devices including rpi for one of our customers, but can't tell the specifics :)

Haha, understood. Thanks for the binary and good luck.
On Mar 17, 2017 11:08 AM, "Lukáš Novotný"<EMAIL_ADDRESS>wrote:

https://github.com/Kukkimonsuta/rpi-buildqt/releases/tag/20170317
I'm not sure if it's gonna be of any help since it's built for different
ubuntu version, but it's worth a shot. Glad to hear your friend
successfully built it.
We were doing experiments with video playback on some devices including
rpi for one of our customers, but can't tell the specifics :)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Kukkimonsuta/rpi-buildqt/issues/2#issuecomment-287380441,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIB5xGN_llXWEBh6CSCIfh1z4qPQ3ulOks5rmqICgaJpZM4MfgZy
.


Add validation to meta assignment

Some cases that should be disallowed:

Self assignment: puzzle A cannot have a metapuzzle that points to puzzle A
Cycles: if puzzle A is assigned to meta puzzle B, puzzle B cannot have meta puzzle A

@asdfryan , I believe this is resolved now?


Clinking 2nd or 3rd item in dropdown table doesn't call onSelect

I have a signup form and I am using this AutocompleteTextfield. But when I click on 2nd or 3rd Item, onSelect is not called, it is only called for 1st Item click.
I am guessing, I am seeing this issue because behind 2nd and 3rd item, I have my own fields.

Hi @pawannimje  , i wasn't able to replicate this issue. But I have updated the code for the latest swift version. Please try if this will work for you first. Thank you.

I also was not able to replicate this issue. Working fine for me.

@wvdk thanks for the feedback.
@pawannimje are you able to find out what cause this on your end?

Today, I encounter with similar kind of issue. I open an issue. Please check issue #57 .


Remove version: FreeCiv.FreeCiv version 3.0.1

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/freeciv/files/Freeciv 3.0/3.0.1/Freeciv-3.0.1-msys2-win64-gtk3.22-setup.exe/download
Status Code: 200
(Automated message - build 896)

Close with reason: Package still available.;


Add note about expiring queues to DLX documentation

The current DLX documentation states that a message will be dead-lettered when:

The message expires due to TTL;

This statement is ambiguous, as this only applies to messages that expire due to a message-ttl argument. Messages on a queue that expires are not dead-lettered.
A PR "Expiring queues should dead-letter messages when a dead-letter-exchange is defined" was closed, but the limitation it addresses is not currently mentioned in the documentation.

@leongersen Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@leongersen Thank you for signing the Contributor License Agreement!

Added a note directly in live on your behalf. Thank you.


"Cannot find command: julia"

Have Julia 0.4.3 installed. Have LightTable 0.8.1 installed. added a launcher (Ubuntu). Ran
Pkg.update()
Pkg.add("Atom")
from julia. All is well. When I launch LightTable from the launcher I get "Cannot find command: julia" When I launch LightTable from the command line. it finds Julia just fine. Restarting doesn't help. Blowing LightTable away and reinstalling doesn't help. Any hints?

Hi. LightTable doesn't interface with julia out the box. Since this is an issue with Julia, you should report this to the Julia plugin


Dropdown does not work properly with angular 1.4

The dropdown does not work properly when it is not appended to the body element.
This behavior can be seen here.

Looks like this was a red herring - something funky was going on before, looks like it works just fine in the Plunker now.


Add Edge instructions

Nvm, just found out it's on the Chrome Web Store now so manual installation is no longer necessary.
https://chrome.google.com/webstore/detail/eight-dollars/fjbponfbognnefnmbffcfllkibbbobki


Check for peers before broadcasting smart contract transactions

See 3893.

@rowandh The tests seem to be failing


Is this different enough from uralex?

See https://github.com/lexibank/uralex

Yes, it is, as it was used in List et al. (2017), as part of six by then not yet cldf datasets, which were by then modified (similar to sidwellbahnaric, starostintujia, etc.). I add these now to have them in CLDF, as they represent legacy data that was used. It does not mean the data needs to be featured anywhere specifically. Uralex also does not have phonetic transcriptions. This one does.

But uralex should be added to clics3 or clics4, I just saw ;)

Uralic language data might be a perfect example for some quality / inter-annotater-agreement checking.

Yes, e.g., we could check the 7 languages against Uralex (which is the same, of course), or against NorthEuralex.

And IDS has 13 Uralic languages, too.

I guess, that could be the next step after "lexibank-core": Go from just selecting gold-standard datasets to synthesizing "best" datasets per family.


is this project dead?

i was considering to start a new project using sails, however, seeing that the last update was in 2020, prevents me from using this project.
so the question is : is this project dead ?

Hi @creative-ae,
No, Sails is not dead and is still maintained.
We recently celebrated the project's 11th birthday, and our last release was Sails 1.5.3  in August 2022.
You can read @mikermcneil's statement on the current state of Sails here: https://gist.github.com/mikermcneil/4e03180398298bd3bf22394a7b72d012
We also recommend checking out @mikermcneil's Sailsconf 2021 talk here: https://youtu.be/_T-UR9mU4-o

I've reported two vulnerabilities which really only require an update to the package.json to patch and never received a response after many months (and it's still not patched). Also there are multiple serious vulnerabilities with the client-side lodash library which can be viewed on Pagespeed, this has also not been patched. Most of the updates that are done are fixes to typos in the docs. Most of the updates to Waterline were done 3-5 years ago. And Sails.js core has 489 issues and in 2022 received about 19 updates (guess how many of them are updates to the docs)
I would say the project has been mostly abandoned. The Sails.js team seems to be working on FleetDM now.

Hi folks! Thanks for your contributions.
Sails.js is alive

The Sails.js team seems to be working on FleetDM now.
Confirmed, at least for some of us.  Eric and I are definitely working on Fleet.

I would say the project has been mostly abandoned.
Thanks for sharing your perspective.  I would disagree and say the project is mostly stable.

For example, we use Sails.js at Fleet, in production.  Sails.js is also in use at Postman, Stripe, Amazon, and many others in production.  Sails Flagship has current customers as well.
My Sailsconf talks in 2020 and 2021 have more info, and I'll be doing another talk at Sailsconf 2022.  Hope that helps!
Why I'm locking this issue
I've been doing this open source thing for long enough now that I know how easy it can be for "foo is dead" issues to cause confusion and further questions, which can be a merry-go-round of misunderstandings and time investment for everyone involved, so I'm going to go ahead and close and lock this issue.  (Please feel free to link to this reply for similar issues that open in the future!)  If anything changes, I will let the community know.  But to be clear, Eric and I are committed as ever to making sure critical bugs and vulnerabilities are addressed quickly, and we have a strong incentive to do so.
Sails continues to be the best way I know how to build scalable, maintainable apps quickly.
Re: issue #7228 specifically

I've reported two vulnerabilities which really only require an update to the package.json to patch and never received a response after many months (and it's still not patched).

Hi @sudo-apt-get-updates, Eric and I took a look at your issue months ago.
Unfortunately, we missed the step of following through on the response we planned to share with you in your issue.  Sorry about that!  (Aside: Would you please use the approach outlined in the responsible disclosure policy for future vulnerability reports?)
Fortunately, we did prioritize and then take action in relation to your issue at that time.  We decided not to patch, and instead to update the docs.  Since Sails.js applications are not typically deployed in the web root of Java applications, and since the best practice for Sails apps is to use S3 or another more scalable solution for file uploads (rather than uploading to disk, which only works easily for single-instance deployments)
This response closes #7228.


Update api.py

Quick fix for "too many redirects" as mentioned here archive.is has been relocated to archive.vn. With this change, the library works again.

Interesting. Why did they change the TLD?

Coverage remained the same at 56.923% when pulling 1fb59ee53650e66c6467b6be47d3155bca745b28 on Zwemvest:patch-1 into 496ecb012ac043d76a62ade1c3dd89b57d4015d5 on pastpages:master.

Coverage remained the same at 56.923% when pulling 1fb59ee53650e66c6467b6be47d3155bca745b28 on Zwemvest:patch-1 into 496ecb012ac043d76a62ade1c3dd89b57d4015d5 on pastpages:master.

Coverage remained the same at 56.923% when pulling 1fb59ee53650e66c6467b6be47d3155bca745b28 on Zwemvest:patch-1 into 496ecb012ac043d76a62ade1c3dd89b57d4015d5 on pastpages:master.

Interesting. Why did they change the TLD?

No clue. Everything I'd say is pure speculation.


ScrollManager: Add ScrollIntoView

Description

This PR adds a ScrollIntoView method, which was previously missing from the ScrollManager. The closest existing equivalent is scrollToFragment, but this function accepts an elementId rather than a CSS selector. This implementation takes the first instance of the CSS selector and scrolls it into view, if it exsists.
If desired, this functionality could be used to implement a scroll to error feature on the MudForm.
How Has This Been Tested?

None. I do not see any tests for the other scroll manager methods either. Should these be excluded from coverage?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

I think its a good feature I would prefer ScrollToSelector with selector being either id or class.
In html terms Selector covers both id and class.
That would be good API IMO.  You can check to see the the selector parameter contaiins # and act accordingly
And forward the old ScrollToFragment ->ScrollToSelector.
Or just adapt ScrollToFragment if we want to keep the existing API.

What do you think? @henon

In JS it is calling Element.ScrollIntoView, so this is an appropriate name for the function. And yes, the ScrollToFragment function could be forwarded to this one and made obsolete.


Allow invalid length inputs in rust tests

This PR will check that each input value is ok, if not it will assert that the output is null.

I ran into issues with overflowing the stack, so Blob::from_bytes returns a boxed value now.

thread 'tests::test_verify_blob_kzg_proof_batch' has overflowed its stack
If there's a better way of handling this, please let me know.

The test.input.get_* functions return a result now, instead of always unwrapping.

Looks great! Merging!


Legend does not update after modifying CartoCSS

When manually editing cartoCSS for categories on a visualization, the legend does not update with the modified colors.  If we are applying custom colors in cartoCSS, must we also manually update the legend?

This would mean parsing the whole cartocss and a lot of guessing. I'd discard this until we work on the new editor.

Any news on this front? Just ran into this today when using a custom colorscheme.

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Java Style Guide: single line javadoc with at-clauses

https://google.github.io/styleguide/javaguide.html#s7.1.1-javadoc-multi-line

The single-line form may be substituted when there are no at-clauses present, and the entirety of the Javadoc block (including comment markers) can fit on a single line.

Here is ideas that guide is not correct:
https://github.com/google/guava/issues/1891#issuecomment-64281498
please confirm what is correct rules to allow single line javadocs.

@kevinb9n any objection to just removing "there are no at-clauses present, and " from §7.1.1?

@kevinn9n , please reply , there are more and more requests on this issue.
One more: checkstyle/checkstyle#3762

How about we just call them "block tags," as the Javadoc documentation does?

if you define what is "block tags" in guide it is completely fine.

Looks like we amended this a while back
https://google.github.io/styleguide/javaguide.html#s7.1.1-javadoc-multi-line


Logging Hooks

The C client's logging leaves a lot to be desired. Since log_callback_fn is simply void (*)(const char *message), we do not have a way to do basic things:

Quickly post-filter by level
Use a void* or other user data
Transform the output in any way that requires context

We probably just want to work around this...or fix the C API's logging.

Pushing this to v0.2 so we can think about configuration as a whole.

Pushing to 0.3 because ZK C API...


Page 404 resolving conflicts

Page 404 🚀
Resolving conflicts of pr #94
This pr is thanks to @juanpablo-is and @Zyruks 🙌
https://github.com/Afordin/hackafor-2/assets/110699874/bc915761-0dee-4ca9-b6a6-eba90bce8611
The hamster doesn't believe it, I take it from https://codepen.io/jkantner/pen/wvqeXrQ

Thank you very much @Zyruks 🙌


Fix README.md

System Specs

OS: Windows 10 (x86_64)
Python Version: 3.8
fugashi Version: 1.0.4

Description
Using tagger = fugashi.Tagger('-d{}'.format(unidic.DICDIR)) as per the readme causes fugashi to throw a RuntimeError due to the arguments being duplicated. However, using tagger = fugashi.Tagger(unidic.DICDIR) seems to resolve the issue.

Not quite sure why this happens, as other threads here suggest that the initialisation in the readme works just fine for others.

Hm, that's weird.

tagger = fugashi.Tagger(unidic.DICDIR)

This should not work, if you just pass a path without a flag in front it shouldn't do anything useful and should give a no such file or directory error.

tagger = fugashi.Tagger('-d{}'.format(unidic.DICDIR))

This might be failing because you're on Windows and the path quoting isn't working. Can you post the error details fugashi outputs and the value of unidic.DICDIR?
You can try this:

tagger = fugashi.Tagger('-d "{}"'.format(unidic.DICDIR))

Also note that if you're using fugashi you actually don't have to pass the dictionary dir, unidic will be used by default if installed.

... due to the arguments being duplicated.

If the -d argument is duplicated that doesn't cause an error, the last one is used. So whatever the cause is that's not it.

Using tagger = fugashi.Tagger('-d{}'.format(unidic.DICDIR))  gives:
Failed initializing MeCab. Please see the README for possible solutions:

    https://github.com/polm/fugashi

If you are still having trouble, please file an issue here, and include the
ERROR DETAILS below:

    https://github.com/polm/fugashi/issues

issueを英語で書く必要はありません。

------------------- ERROR DETAILS ------------------------
arguments: [b'fugashi', b'-C', b'-r', b'C:\\Users\\Redacted\\AppData\\Local\\Programs\\Python\\Python38\\lib\\site-packages\\unidic\\dicdir\\mecabrc', b'-d', b'C:\\Users\\Redacted\\AppData\\Local\\Programs\\Python\\Python38\\lib\\site-packages\\unidic\\dicdir', b'-dC:UsersRedactedAppDataLocalProgramsPythonPython38libsite-packagesunidicdicdir']
error message: param.cpp(69) [ifs] no such file or directory: C:UsersRedactedAppDataLocalProgramsPythonPython38libsite-packagesunidicdicdir\dicrc
Traceback (most recent call last):
  File "C:/Users/Redacted/PycharmProjects/RomajiDensity/test.py", line 8, in <module>
    tagger = fugashi.Tagger('-d{}'.format(unidic.DICDIR))
  File "fugashi\fugashi.pyx", line 313, in fugashi.fugashi.Tagger.__init__
  File "fugashi\fugashi.pyx", line 220, in fugashi.fugashi.GenericTagger.__init__
RuntimeError: Failed initializing MeCab
----------------------------------------------------------

My bad, the duplicate arguments are a non-issue. Quite obviously the Windows path wasn't formatted properly. But the default argument seems to be there.
However, using any of the options fugashi.Tagger(), fugashi.Tagger(unidic.DICDIR) and tagger = fugashi.Tagger('-d "{}"'.format(unidic.DICDIR)) gives the desired result for me.

Glad it works! Still not sure why fugashi.Tagger(unidic.DICDIR) works but I'll fix the README and then close this.


The 'workbench.action.terminal.sendSequence' command stops working after running a task.

Issue Type: Bug
The 'workbench.action.terminal.sendSequence' is very useful for sending commands to the integrated terminal with keyboard shortcuts. When tasks run, they open a new terminal on top of the existing one, along with the message "Terminal will be reused by tasks, press any key to close it". However, sending a key using the 'sendSequence' command no longer works while the "task terminal" is open. Neither does it close it, which it should according to the displayed message. This is very annoying as the whole point of these keyboard shortcuts is to avoid manually manipulating terminals.
VS Code version: Code 1.56.2 (054a9295330880ed74ceaedda236253b4f39a335, 2021-05-12T17:44:30.902Z)
OS version: Darwin x64 19.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
16.00GB (0.50GB free)

Process Argv
--crash-reporter-id 3e7d515b-b0fd-42e3-b8d6-159f0162d735

Screen Reader
no

VM
0%

@yohan-pg our May release, which will probably release this week, has a new task presentation property close:
https://github.com/microsoft/vscode/blob/b7009c5b546acbe7ee57d3e875dc490a84249182/.vscode/tasks.json#L198-L206
You can try it out in our Insiders build already.

My understanding of why workbench.action.terminal.sendSequence doesn't close task terminals is because the terminal is literally waiting for a key press, which is different from a sequence of characters that doesn't originate from a key press. @Tyriar in case I am getting that wrong.

sendSequence doesn't do this since it bypasses the terminal and passes the sequence directly through the pty. As designed seems right.

@alexr00 Thanks. While that makes perfect sense logically, from a usability point of view it is very disappointing!  I really feel like using a command like sendSequence, should never just fail silently and do nothing.


Redesigned the whole website

resolved issue #49
Screenshots:
already shared in issue #49

@AyushSarode

Good work!

@AyushSarode I am extremely sorry to inform you that your repo is presently not participating in hacktoberfest. Maintainers need to add the tag "hacktoberfest" in order to participate in it.

@AyushSarode I am extremely sorry to inform you that your repo is presently not participating in hacktoberfest. Maintainers need to add the tag "hacktoberfest" in order to participate in it.

I am sorry!
I forgot to add that label.
I did. After no time it will change

@AyushSarode I am extremely sorry to inform you that your repo is presently not participating in hacktoberfest. Maintainers need to add the tag "hacktoberfest" in order to participate in it.

I am sorry! I forgot to add that label. I did. After no time it will change

yeah it is, thanks :)

@AyushSarode I am extremely sorry to inform you that your repo is presently not participating in hacktoberfest. Maintainers need to add the tag "hacktoberfest" in order to participate in it.

I am sorry! I forgot to add that label. I did. After no time it will change

yeah it is, thanks :)

Can you send me ss?

ah Thank you!


[WIP] Fix checkbox corner cases

POC to fix Cmd-a and Cmd-shift-left behavior when using checkboxes. The behavior is fine but the code is not very clean...
This is probably not the best place but I have a few (interrelated) thoughts to share so I'll dump them here for now. Happy to move then to e.g. a GH Issue or whichever place is more adapted.
Thought 1 (update the List class)
The approach to handling checkboxes is starting to feel a bit spaghetti-like with these offsets I'm adding at the last minute. I feel like the List class needs to be amended, but I'm not 100% how yet. A checkbox is neither a bullet not a plain "content":

it's not "just a bullet" because it should not be blindly copied to the next line when pressing Enter (- [x] myline| should insert - [ ] , not - [x] ).
it's not "just a bullet" because when handling pressing Enter in - [ |] myline (to split the box in half, when settings.stickCursor is false), the implementation currently uses lines to determine whether the cursor is inside the box. This requires the checkbox to be part of lines (which the bullet isn't).
it's not "just contents" because obviously we need to handle checkboxes specially to get cursor out of it etc.

I'll try to wrap my head around that.
Thought 2 (avoid moving the cursor again after a move)
Currently, the implementation is such that a cursor that is in the bullet/checkbox area will be automatically moved out of it. I think this participates in the two bugs that these tests surface (looks like there's a kind of race condition due to Obsidian changing the checkbox from text-to-image and vice versa, and the selection's anchor ch gets set to 0). It is also what makes Cmd-left look glitchy (the cursor moves to the left, Obsidian changes the checkbox into text, then outliner moves the cursor after the checkbox, Obsidian changes the checkbox back into a square image).
Do you think it would be desirable to handle this differently, namely to override keystrokes (left, right, etc.) in such a way that the cursor never ends up within the bullet/checkbox (even for a fraction of a second)? This is for example what is done for leftArrow handling today.
Having to handle the different hotkeys individually is more work, but it would mean we avoid the non-trivial issues above.
If the cursor ends up in that bullet/checkbox area (e.g. when clicking there in source mode), then... we'd leave it in there? I'm thinking it might avoid potential incompatibilities in the future, e.g. with other plugins that won't expect the cursor to automatically move. It's also possible to keep the current "move away cursor" behavior as a last resort mechanism.
Thought 3 (custom checkboxes UX):
This is a tricky one: on the one hand it would be great to manage custom checkboxes such as [!], but on the other hand, if we don't let the cursor get in there, users won't be able to amend the value. Regular checkboxes have the Cmd-L hotkey for that so they're fine.
Moreover, I would say the current behavior is a regression from pre-4.1.0: users that use custom checkbox have a worse UX now than they used to do (see first item below).
Possible approaches:

have outliner ignore custom checkboxes (current behaviour). Main issues: it's hard for the user to change - [ ] to - [!]. Also, the cursor behavior is inconsistent between - [ ] (handled by outliner) and - [!]  (not handled).
have outliner consider custom checkboxes like regular ones. Main issues: it's hard for the user to change - [ ] from and to - [!]. But at least, it's custom- vs regular- checkbox behavior is consistent, and power users with custom-checkbox hotkeys have the best experience.
change the "stickCursor" setting to allow "anywhere", "outside bullet", "outside bullet and checkbox" (can be done in conjuction with either 1. or 2.). Main issues: custom-checkbox users won't reap the benefits from outliner's checkbox handling, but at least their experience is not a regression compared to pre-4.1.0 version.

CC @vslinko FYI

@vslinko PTAL - we should probably use it for a bit before releasing but I wanted you to have a look.
This changes the stickCursor setting to make it ternary (with backward compatibility with bool for people upgrading).
It also fixes the couple of corner cases you found earlier.
Note: the test suite needs to be upgraded to the latest version of Obsidian, which changed the behavior of the enter should fallback behavior while multiline selection test. I did not include the one-space fix in this PR to let the test suite pass but you'll see it will fail locally on the latest version of Obsidian.

The three settings names in the PR are descriptive but are slightly awkward in terms of UI because of the varying length. Maybe not a big deal but don't hesitate to suggest alternatives :-)

Another note: I did not amend getContentStartCh() directly as doing so has unintended consequences - not to say that it can't be done that way but I wanted to avoid this riskier approach.

@sitaktif Sorry for the long absence. Awesome job! Made some minor edits. Going to the release.

Released in https://github.com/vslinko/obsidian-outliner/releases/tag/4.3.0

Great, thanks!


Add support for stopping (not deleting) connectors

KIP-875 introduced a new target state for connectors: STOPPED. This state differs from PAUSED in that it completely shuts down all tasks for the connector, instead of leaving them active but on standby.
It's also a requirement for connectors to be in that state before their offsets can be reset or altered.
Since support for this new state was added in Kafka 3.5, we can and should add support for it in kcctl.

Definitely makes sense to support this. PR welcome :)

Self-assigned, plan on implementing tomorrow :)

Awesome, thanks a lot!

Message ID: @.***>


Document requirements for UX polish

Summary
Polishing the project is a huge task. We will need to make several issues to track to creation of assets for the game template. For now, I will use this issue to create a list of overall enhancements that need to be applied before this is passable and helpful to the open-source community.

We should include a sort of "Welcome" experience as part of UX polish. This should explain what the app is, and how you play.
Press [Space] to Jump

Type of stuff.


gRPC C# server reflection FileDescriptor dependencies doesn't include package name of proto file

I have implemented a gRPC service in .NET Core and Enabled gRPC server reflection on it as mentioned in https://github.com/grpc/grpc/blob/master/doc/csharp/server_reflection.md
It's working fine if all proto files of the services have the same package defined. But if proto files have different package name then it doesn't work.
Example:
I have two proto files.
test.proto
package test;
service testService
{
    rpc getTest(google.protobuf.Empty) returns(TestResponse)
}
message TestResponse
{
    string Name = 1;
}

and empty.proto  .I have included this file from protocolbuffers/protobuf repository wtih package name "google.protobuf"
https://github.com/protocolbuffers/protobuf/blob/master/src/google/protobuf/empty.proto
What version of gRPC and what language are you using?
Grpc.Reflection 2.25.0
Grpc.Core 2.25.0
What operating system (Linux, Windows,...) and version?
Tried on Windows 8.1 and Ubuntu
What did you do?
I am testing server-reflection using grpcui (https://github.com/fullstorydev/grpcui), it produces the error
Failed to compute set of methods to expose: Symbol not found: test.testService
caused by: File not found: empty.proto

I tried to debug the code for both Grpc.Reflection library and grpcui.
https://github.com/grpc/grpc/blob/3f108fc5f57a529b0d2912b8b7af952b8117933e/src/csharp/Grpc.Reflection/ReflectionServiceImpl.cs#L109-L124
I found that when calling method ServerReflectionInfo with ServerReflectionRequest.MessageRequestOneofCase.FileContainingSymbol,
property SerializedData(line 122) doesn't contain descriptor dependency full name. So on the client-side (grpcui) receives dependency name as "empty.proto" although it should
receive "google/protobuf/empty.proto".
https://github.com/grpc/grpc/blob/3f108fc5f57a529b0d2912b8b7af952b8117933e/src/csharp/Grpc.Reflection/SymbolRegistry.cs#L54-L59
Now if I call ServerReflectionRequest.MessageRequestOneofCase.FileByFilename with received dependency "empty.proto", it returns not found error.
Because filesByName Dictionary contains "google.protobuf.empty.proto" as Key, not "empty.proto".
What did you expect to see?
I expect that ServerReflectionRequest.MessageRequestOneofCase.FileContainingSymbol request should return dependencies with Full name (including package name)
OR the Dictionary keys should not have the package name in their keys.
OR I am missing something here and doing it the wrong way, please help me to correct it.

https://github.com/jhump/protoreflect/blob/10815c273d3f96493752e245336d4b29ce8715a8/grpcreflect/client.go#L296-L302
Here depName variable has value "empty.proto", it should be "google/protobuf/empty.proto"

Hi, from the description, there doesn't seem to be an obvious actionable bug to the gRPC repo. For questions in general, please redirect them to a wider community like stack-overflow. Thanks!
Please re-open if I'm mistaken.

I'll take a look.

The file empty.proto has to be inside a directory structure same as package name. For my case, the issue got resolved when I put the file inside ./google/protobuf/ directory as the package for the proto file is google.protobif

@ashu82492

I found that when calling method ServerReflectionInfo with ServerReflectionRequest.MessageRequestOneofCase.FileContainingSymbol,
property SerializedData(line 122) doesn't contain descriptor dependency full name. So on the client-side (grpcui) receives dependency name as "empty.proto" although it should
receive "google/protobuf/empty.proto".

Can you provide your .proto file that imports empty.proto? Have you imported "empty.proto" as import "google/protobuf/any.proto"; (the way you should) or as import "empty.proto"; (which is wrong).
Your description doesn't make it fully clear what you've done (ideally you would have provided a project that reproduces the issue), but it seems unlikely that fileDescriptor.SerializedData would include wrong data.

Closing because it seems that test.proto mentioned in the description imports empty.proto in and incorrect way (it should imported as import "google/protobuf/any.proto";, so that the filename is preserved correctly in the descriptor)

@jtattermusch Yes, you are right. I imported it in an incorrect way (same as you have mentioned).
Thanks.

Just wanted to leave a comment here because I faced a similar issue. I imported everything according to the correct path, however I didn't set ProtoRoot but just used
<Protobuf Include="../../PathToProtos/**/*.proto" />

This unfortunately sets the Name of the file to just the filename itself. You need to include the ProtoRoot directive and set it to the PathToProtos in this example.
Maybe this helps someone that's stuck on a similar issue.


Kernan flexibility

Use my superior version of kernan, it has to advantages:

it works with code that has annotations (by simply ignoring them at the moment), instead of causing moth to hang
the parser is more flexible, in particular the indentation rules are relaxed (so they appear to accept at least what andrews minigrace accepts), and binary operators don't need spaces around them.

Hi @IsaacOscar, is this PR for the right branch?
There seem to be a lot of changes there that do not seem to match the description of the PR.

Hi @IsaacOscar, is this PR for the right branch?
There seem to be a lot of changes there that do not seem to match the description of the PR.

Thanks @marr, yes I had selected the wrong branch!