Invalid Arrow Buffer

I'm trying to load an arrow file in my typescript application. I'm using https://github.com/parcel-bundler/parcel to bundle my app.
import { Table } from "@apache-arrow/es2015-esm";

fetch(require("../data/flights-10k.arrow")).then(response => {
  response.blob().then(blob => {
    const table = Table.from([blob]);
    console.log(table.toString());
  });
});

I get the following error Uncaught (in promise) Error: Invalid Arrow buffer.

Maybe I need to convert the blob to a buffer
fetch(require("../data/flights-10k.arrow")).then(response => {
  response.blob().then(blob => {
    let arrayBuffer;
    const fileReader = new FileReader();
    fileReader.onload = function() {
      arrayBuffer = this.result;
      const table = Table.from(arrayBuffer);

      console.log(table.toString());
    };
    fileReader.readAsArrayBuffer(blob);
  });
});

Now I get Uncaught TypeError: Cannot read property 'fields' of undefined.

cc @trxcllnt @TheNeuralBit. The version of the project on npm is slightly stale, do we need to do a 0.4.0 JS release soon?
@domoritz in general we prefer to use either JIRA or the mailing list for these discussions, but we have GitHub issues available to capture information from users who aren't yet familiar with how the project works

I can post on the mailing list or create JIRA tickets instead. My questions are probably very naive and I wasn't sure where the right place is to ask them. I'll close this issue and move it to JIRA.

No worries, thanks!

https://issues.apache.org/jira/browse/ARROW-2686

@domoritz you should use response.arrayBuffer(), and then wrap the ArrayBuffer in a Uint8Array like this:
import { Table } from "@apache-arrow/es2015-esm";

fetch(require("../data/flights-10k.arrow")).then(response => {
  response.arrayBuffer().then(buffer => {
    const table = Table.from(new Uint8Array(buffer));
    console.log(table.toString());
  });
});

Oh, perfect. That works!


[hotfix][docs] fix readme 404 link

Proposed changes
Issue Number: close #22393

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

@hf200012 PTAL

@JNSimba PTAL

run buildall


Set up cookie key for production

Need to be able to deploy this piece consistently (so that all lambdas can access one another's cookies)

See #9

Done with lambda environment variables. Closing this in favor of rolling the permanent solution into #9


Gitignore global abuses CPU

Hey,
I am using IntelliJ 14.1.4 on Ubuntu 15.04 in VirtualBox 5, running with JRE8 64bit.
When I open a .gitignore file AND a .gitignore_global file is at place in my ~ folder, the Idea process starts to abuse my CPU. Though I have the more than enough horse power to handle it, this is really annoying.
How can I be of help tracking down the core of the problem?
Thanks in advance!

Hi, I am seeing the same behaviour on 1.2 with PHPStorm 10. CPU usage jumps to 100% when a .gitignore file is open in a project. Most noticeable after waking up from sleep.
MacBookPro, El Capitan
Thanks!


request help: can use crd config apisix global rule

Issue description
I want use apisix global rule to process some problem，like route not found and the gateway 500 error etc. ,but I read the ingress source code not found hao to use crd to config this ,is this support?

ApisixClusterConfig resource

ApisixClusterConfig resource

this only support monitoring skywalking ,I want use custom plugins

Global rule cannot be used in apisix ingrees controller right now.

Global rule cannot be used in apisix ingrees controller right now.

Whether there is plan support it

If all goes well, it could be in v1.6, otherwise it will be in v1.7.
Also, I will close this issue, we can use https://github.com/apache/apisix-ingress-controller/issues/1104  for track


Object / Array Destructuring not supported

Hello !
First, thank you all for this great job :)
It seems that object destructuring (or maybe even any assignation) made at the root of the  <script> section is not recognized by the parser and make it crashes.
When a SFC contains something like this :
<template>
    ...
</template>

<script>
    ...
    const { HIGH, NORMAL } = AppTypes.FlagLevel
    <<OR>>
    const LOGS_ENABLED = config.logs.router
    ...
    export default {
        name: 'FlagBanner',
        computed: {}
        methods: {}
        ...
    }
</script>

This kind of warning is raised when compiling :
Warning: Cannot parse src/components/app/FlagBanner/FlagBanner.vue: TypeError: Cannot read property 'HIGH' of undefined

It usually means that vue-docgen-api does not understand your source code or when using third-party libraries, try to file an issue here:
https://github.com/vue-styleguidist/vue-docgen-api/issues

I tried in vain to find out where this issue could come from. Do you have any idea about where it actually crashes and how we could fix this ?

Moved in vue-styleguidist : https://github.com/vue-styleguidist/vue-styleguidist/issues/182

It is related with https://github.com/vue-styleguidist/vue-docgen-api/issues/23#issuecomment-371202215


genji.IsNotFoundError should include errs.ErrDocumentNotFound

What version of Genji are you using?
main
What did you do?

	doc, err := a.database.QueryDocument("SELECT * FROM image WHERE hash = ?", hash)
	if genji.IsNotFoundError(err) { ... }

What did you expect to see?
genji.IsNotFoundError() should also include errs.ErrDocumentNotFound else it is not possible to check for a not found error (since errs is an internal package).

Invalid, since errors.ErrDocumentNotFound is actually included within the IsNotFoundError() check


How to get the http video stream instead of rtsp?

Hi,
I can get the rtsp stream from a IP camera but there is lots of lagging. (several seconds)
I can use opencv to capture the http video stream and the lagging is small
Is there anyway I can capture the http stream in your code? Thx
Note: I don't understand is that why my typical NVR (Network video recorder) does not have lagging, how it's protocol work? Just http? Can I do the same thing in nano?
Plan A: reduce the lagging in rtsp stream. I have added the --input-rtsp-latency = 0, it still has 4 seconds delay for 1920x1080 (h264)
Plan B:
use openCV to capture the http, then convert it using this example
https://github.com/dusty-nv/jetson-utils/blob/master/python/examples/cuda-from-cv.py
But mjpeg only has max 704x576 in the IP camera sub-stream...
Thanks.

Hi @AK51, can you try the suggestion from this post?
https://github.com/dusty-nv/jetson-inference/issues/1323#issuecomment-992544823

ok, I will ask question in #1323 , thx


bg-purple is misspelled

Changing the background color using bg-purple will not function is since it misspelled as bg-pruple

Fixed in v0.1.2


Contibutor Access

Add me as a contributor

@Lebura welcome! And thanks for your interest in being a contributor. You can be a contributor in various ways listed here https://contribute.freecodecamp.org/. Otherwise, feel free to join the conversation on ongoing developments in our Gitter chat room https://gitter.im/FreeCodeCamp/Contributors. Happy coding!


Upgrade to webpack 4 and unify webpack configs

Webpack

upgrade to webpack 4, comes with many default optimizations for dev and prod
Use webpack-config-utils to unify webpack configs
Use MiniCssExtractPlugin  instead of ExtractTextPlugin (used for CSS only and highly recommended by ExtractTextPlugin for webpack 4)
Use one rule for sass and css -> ExtractText/MiniCssExtract now also extracts .css files
Remove indentedSyntax, is now automatically set per file (needed for sass/css): https://github.com/webpack-contrib/sass-loader/pull/196
Rule for .yml: Remove depracated json-loader and use js-yaml-loader to directly convert to JS
Remove DefinePlugin process.node.ENV as this is set now automatically by mode
Use OptimizeCSSAssetsPlugin to minify CSS
Parallelize for UglifyJS. (Without optimize.minify, webpack would default to UglifyJS in production mode without parallelization)

Other

Use const instead of var in server.js

PS: Wasted a few days upgrading to webpack 4 and cleaning up the webpack config. I hope I never have to do that again.

Closing this in favor of #335


migrate components and pages to typescript

Description

install typescript in project

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Custom install folder

What about adding possibility to change default installation folder from 'node_modules' to something else in 'napa-config':
"napa-config": {
"install-folder": "web_modules"
}
Why it could be useful?
For example webpack is set by default to search for modules in 'web_modules' and 'node_modules' directories. By installing all libraries available in npm to 'node_modules' and libraries that aren't published in npm to 'web_modules' we could use 'npm shrinkwrap' without any issues.
Not sure if this idea have some downsides but I think it could work well. What do you think?

I would rather not as napa tries to stay inline with npm and they will never allow custom install folders. The benefit of only ever having a single install folder is tooling can make some big assumptions on dependencies. I fear if we allow custom install folders, existing tooling geared for npm won't be compatible.
This is one of the reasons I only use webpack at the app level and use Browserify on the module level.

Yeah, make sense. Thanks for explanation, closing.


Remove event tile on main screen

For the moment I will collect here also contributions from the community with link:

Current Implementation

Check-In related functionalities are not bundled in one single place:

As a host I have to use the main screen: the event tile
Also, for creators of QR codes, the only way to view an existing QR code is to click on the button "CREATE QR CODE". That doesn't feel right that I have to tap "CREATE" to in order to "VIEW". (https://github.com/corona-warn-app/cwa-wishlist/issues/469#issuecomment-824034396)
As a attendee I have to use the Check-In tab

Suggested Enhancement

Bundle all check-in related functionalities at a single point.
With the current design that would be: access all check-in related functions via the check-in tab.
Add a button to create a code in the "Check-In" page.
Or add an entry in the submenu of the "Check-In" page to generate a new event code. https://github.com/corona-warn-app/cwa-wishlist/issues/469#issuecomment-831435870
Maybe a more general label like "Manage QR codes" would be better than "Create QR code" https://github.com/corona-warn-app/cwa-wishlist/issues/469#issuecomment-824081870
The current design is indeed a bit hard to understand. A user correctly noted that it is confusing that the tab with the QR code icon does not allow one to create QR codes. If no better solution is found, there could be a link between worlds the attendee and the organizer view through the overflow drop-down menu. https://github.com/corona-warn-app/cwa-wishlist/issues/469#issuecomment-824328372

Expected Benefits

Improve UX
Clearer design of the app
Uncluttered main screen
improve feel and look of an app, main screen feels like a website

Internal Tracking ID: EXPOSUREAPP-6617

Also, for creators of QR codes, the only way to view an existing QR code is to click on the button "CREATE QR CODE". That doesn't feel right that I have to tap "CREATE" to in order to "VIEW".

@MikeMcC399 Thanks. I will add it to the OP, if you don't mind.

@dsarkar

I will add it to the OP, if you don't mind.

Certainly! Maybe a more general label like "Manage QR codes" would be better than "Create QR code". The text above would need changing as well.

Great request. I was really confused when I first used CWA today because I couldn't find the "create QR code" functionality. In the end I only found it after looking at screenshots because I would have never guessed to find it on the main screen.

The current design is indeed a bit hard to understand.
A user correctly noted that it is confusing that the tab with the QR code icon does not allow one to create QR codes.
If no better solution is found, there could be a link between worlds the attendee and the organizer view through the overflow drop-down menu.

Add a button to create a QR-Check-In code in the "Check-In" submenu.
Current Implementation

Currently the button to create a QR code only exists on the main page and the users are unable to create a QR code from the "Check-In" submenu.
This is against the expected behaviour of the application and leads to confusion.
Suggested Enhancement

Add a button to create a code in the "Check-In" page.
Or add an entry in the submenu of the "Check-In" page to generate a new event code.
Expected Benefits

better UX, closer to an expected behaviour of the app

@nilsriesenberg  Thanks. Added your suggestion to Internal Tracking ID: EXPOSUREAPP-6617

One thing that I want to add here: Inform the user that you have moved the feature to another place to avoid confusing similar to what happened when moving the certificates to it's own tab (:

The situation @MikeMcC399 mentioned in https://github.com/corona-warn-app/cwa-wishlist/issues/469#issuecomment-824034396 got worse with the introduction of the "Warn for others" feature.
You have to click "Create QR code" to use it.
I meanwhile rather consider this as an (easy to fix (text change only)) bug, what about you @MikeMcC399?


Handle executing PAL.ps1 from other directory

Updated PAL.ps1 to handle being invoked from a different working
directory
** Includes fixing inherited XML logic
Added parameter to NOT clear log or output, if part of a larger
sequence (defaults to true)
Updated file generation for log to include attempt count and timestamp
rather than just optionally attempt

-- Resolves https://pal.codeplex.com/workitem/65582 - I didn't see issues migrated over with change to GitHub

@VonOgre, thanks for your contribution!


Table creation in Hive or Impala fails with UnicodeDecodeError: 'asci…

Table creation in Hive or Impala fails with UnicodeDecodeError: 'ascii' codec can't decode byte
Internal jira: CDPD-22129
Due to change in the thrift code to look at python base version and based on that use utf-8 encoding the issue started happening.  Based on the information from one of the internal jira python 3 uses the else part which is similar to what was there in before sys.version_info was added to the thrift code.
So removed if condition and retained only the else part example
From: oprot.writeString(self.log.encode('utf-8') if sys.version_info[0] == 2 else self.log)
To: oprot.writeString(self.log)

We can't see the diff? Could we put the change here if small?

We can't see the diff? Could we put the change here if small?

This is the change, bring back the old info. with Python 3 it would use self.log and with CDH 6.x and earlier also uses oprot.writeString(self.log).  Only from CDP 7.x with python2 the code tries to encode and causes utf 8 issues and invalid ttype when run from impala.  This was tested with Ying on one of his cluster and it worked like a charm.
From: oprot.writeString(self.log.encode('utf-8') if sys.version_info[0] == 2 else self.log)
To: oprot.writeString(self.log)

@romainr ,  I keep getting this even after making the indentation correct to 2 spaces.
apps/beeswax/gen-py/TCLIService/ttypes.py:20: [W0311(bad-indentation), ] Bad indentation. Found 4 spaces, expected 2
apps/beeswax/gen-py/TCLIService/ttypes.py:21: [W0311(bad-indentation), ] Bad indentation. Found 4 spaces, expected 2
apps/beeswax/gen-py/TCLIService/ttypes.py:22: [W0311(bad-indentation), ] Bad indentation. Found 4 spaces, expected 2
apps/beeswax/gen-py/TCLIService/ttypes.py:23: [W0311(bad-indentation), ] Bad indentation. Found 4 spaces, expected 2
+++++++++++++++++++
apps/beeswax/gen-py/TCLIService/ttypes.py:7137: [C0326(bad-whitespace), ] No space allowed before bracket
(1, TType.I32, 'i32Value', None, None, ),  # 1
^
apps/beeswax/gen-py/TCLIService/ttypes.py:7138: [C0326(bad-whitespace), ] No space allowed before bracket
(2, TType.STRING, 'stringValue', 'UTF8', None, ),  # 2
^
apps/beeswax/gen-py/TCLIService/ttypes.py:7143: [C0326(bad-whitespace), ] No space allowed before bracket
(1, TType.MAP, 'qualifiers', (TType.STRING, 'UTF8', TType.STRUCT, [TTypeQualifierValue, None], False), None, ),  # 1
I have corrected both of these in the code and Ying also verified the same.  Can you please help on this one???

Could you add a
  grep - v apps/beeswax/gen-py/ | \
in
https://github.com/cloudera/hue/blob/master/tools/ci/check_for_python_lint.sh#L28
?

We can't see the diff? Could we put the change here if small?

This is the change, bring back the old info. with Python 3 it would use self.log and with CDH 6.x and earlier also uses oprot.writeString(self.log). Only from CDP 7.x with python2 the code tries to encode and causes utf 8 issues and invalid ttype when run from impala. This was tested with Ying on one of his cluster and it worked like a charm.
From: oprot.writeString(self.log.encode('utf-8') if sys.version_info[0] == 2 else self.log)
To: oprot.writeString(self.log)

So looks like we are going back to the autogenerated code so big :+1:

Could you add a
 grep - v apps/beeswax/gen-py/ | \
in
https://github.com/cloudera/hue/blob/master/tools/ci/check_for_python_lint.sh#L28
?
And originally issue are probably the , )

Done!!!


Apt-get update and apt-get upgrade fail

Your Windows build number:  18965.1000

Whever I use Ubuntu on WSL, apt-get update and apt-get upgrade work fine. And pinging works as well.   They have been ignored, or old ones used instead.
Whenever I try pinging google I receive:
ping: google.com: Temporary failure in name resolution
I've tried switching networks, turning off Firewall, and reinstalled Ubuntu.

This looks like a duplicate of https://github.com/microsoft/WSL/issues/4342 or https://github.com/microsoft/WSL/issues/3438 . Please try some of the workarounds [1] [2] posted in those threads, and if those fix your issue we can mark this as a duplicate and close it.

The powershell script available there was able to fix my issue!

I'm facing the same problem after I updated Windows Insider to 18965.1005.

i cannot ping baidu.com and google.com ,my version is 19008.1 insider and i am using v2ray ,"wsl2"

deleting  /etc/resolv.conf and reopen wsl terminal worked for me

/etc/resolv.conf was getting re-created for me every time. this fixed it for me:

Create a file: /etc/wsl.conf.
Put the following lines in the file
[network]
generateResolvConf = false
In a cmd window, run wsl --shutdown
Restart WSL2
Create a file: /etc/resolv.conf. If it exists, replace existing one with this new file.
Put the following lines in the file
nameserver <IP_ADDRESS>
Repeat step 3 and 4. You will see git working fine now.

found here

i faced to this issue in Windows10 ver2004, wsl2, and Ubuntu20.04LTS.
this issue should reopen.

@Sergej-Popov
hi, have you installed docker desktop for windows?
docker ps is dead in my env. can it run in your env?

I faced this issue too, on windows10 ver2004, wsl2, and Ubuntu18.04LTS

I faced this issue too, on windows10 ver2004, wsl2, and Ubuntu18.04LTS

I am also having this issue when i attempt to run Docker with MLflow.

This seems to be some access issue from within my WSL2 shell.  I've tried the suggestions mentioned here, but no luck.
Anyone have a resolution?

Doesn't work for me either with Windows 10 Enterprise, Version 10.0.18362, WSL2, and Ubuntu18.04 LTS. None of the workarounds have worked for me yet.

Guys, check your 3rd party firewall settings. BitDefender firewall broke my internet connection in WSL

Turning off my VPN got it working for me.

Started sounding like a VPN issue, glad I'm not the only one with the VPN issue. Oddly I have whitelisted apps for my VPN, but it must be the DNS resolver for the VPN screwing things up.

if you still not found this solution,
For me i was checking the /etc/resolv.conf
and add the nameserver <IP_ADDRESS> inside it
I think this problem because we not add the global DNS, when we set our local DNS

if you still not found this solution,
For me i was checking the /etc/resolv.conf
and add the nameserver <IP_ADDRESS> inside it
I think this problem because we not add the global DNS, when we set our local DNS

Adding nameserver <IP_ADDRESS> to /etc/resolv.conf worked for me, too.

I added the company dns ip but does not fix the issue

Resolved this issue on windows10 ver2004, wsl2, and Ubuntu18.04LTS with @craigloewen-msft help and his link to the fix:
sudo bash -c 'echo "nameserver <IP_ADDRESS>" > /etc/resolv.conf'
sudo bash -c 'echo "nameserver <IP_ADDRESS>" >> /etc/resolv.conf'

Adding nameserver <IP_ADDRESS> to /etc/resolv.conf worked for me, too.

Worked for me too. had trouble after i added fish ppa

Changing the server to <IP_ADDRESS> worked for me too.
sudo nano /etc/resolv.conf

# This file was automatically generated by WSL. To stop automatic generation of this file, add the following entry to /etc/resolv.conf
# [network]
# generateResolvConf = false
nameserver <IP_ADDRESS>

Adding nameserver <IP_ADDRESS> to /etc/resolv.conf worked for me, too.

This worked 100%, thanks!!!!


Example 2: -EnableDialInConferencing

Example 2 users EnableDialinConferencing other documentation states this can only be set on New-CsConferencingPolicy
Documentation also has this as a setable option.
Running the command as stated gives
"You are not permitted to invoke Set-CsConferencingPolicy with the following parameters:
EnableDialInConferencin"
on Skype for business online

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

ID: 2eb630b9-b94d-a647-ca21-821e1d520b21
Version Independent ID: ede8297e-aa49-fc5e-23bb-2d678bc23e6b
Content: Set-CsConferencingPolicy (skype)
Content Source: skype/skype-ps/skype/Set-CsConferencingPolicy.md
Service: skypeforbusiness-powershell
GitHub Login: @kenwith
Microsoft Alias: kenwith

@dariomws306 Thank you very much for the contribution and updating the documentation with PR. @chriswharton2 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Extension Managar hangs a lot

Extension Manager hangs, a lot.
System Information
Windows Build No: 10.0.22621.2715
Other Software: OS Build Version: 10.0.22621.2715.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.23
System Information:
Physical Memory: 11.89GB (1.28GB free)
Processor Architecture: x64
Extension Info: Python - Profiling and Python - C++ Project debugging support or something like that.
Additional Information
I tried without the extensions but it still hangs.
An image for reference

Hi @skanda890 ,
Your issue is related to Visual Studio, not Visual Studio Code. I think the best location to find info and/or report issues for Visual Studio would be at https://visualstudio.microsoft.com/vs/support/?tab=support
Hope this helps
/outOfScope


Update isMessageMatching method and fix the minimumConfidence

They (wit.ai) have replaced the intent with the search strategy. So, there is no intent. Also, the minimumConfidence inserted in the create function is not taked into consideration as it's not initialised in the constructor.

Can you update the PR so it has no conflicts and add the appropriate tests to it?


Add SfB and ASB

See bachelor thesis https://nbn-resolving.org/urn:nbn:de:bsz:14-qucosa2-211701 to follow on.

Both, SfB and ASB, use a Wiki for maintaining their classification:

https://www.sfb-online.de/wiki/index.php?title=Hauptseite
https://asb-kab-online.de/wiki/index.php?title=Hauptseite

So, converting them to JSKOS would mean a lot of copy and paste work. So did the author of the bachelor thesis (s. page 50, last paragraph).
Or is there a thinkable solution to scrape the content of the wiki with a script (e.g. written in Python)?

https://www.crummy.com/software/BeautifulSoup/bs4/doc/ seems to bee an option. Will try it out with ASB some time.

There is an API to MediaWiki, e.g. https://www.sfb-online.de/wiki/api.php. In particular method parse to get a page: https://www.sfb-online.de/wiki/api.php?action=help&modules=parse
example query:

https://www.sfb-online.de/wiki/api.php?action=parse&page=BID&prop=wikitext (source)
https://www.sfb-online.de/wiki/api.php?action=parse&page=BID&prop=text (HTML)

The latter could be analyzed with BeatifulSoup.


Pip install not working

Hi I tried to install mango-explorer via pip. We got an error
ERROR: Could not find a version that satisfies the requirement mango-explorer (from versions: none)

@mschneider any one can answer here


Implement SSH client in virtctl

This PR adds the virtctl ssh command leveraging the /portforward endpoint to create a basic SSH connection with a VM/VMI.
It also documents using the same mechanism for establishing other management connections like RDP.
Note: This PR is based on https://github.com/kubevirt/kubevirt/pull/4944 which should merge first.
Meanwhile this PR can be used for discussing the SSH client further while unblocking the backend work.
/hold
Special notes for your reviewer:
Follow ssh.md to try it out.
Release note:
- Add a `ssh` command to `virtctl` that can be used to open SSH sessions to VMs/VMIs.

Bringing over two comments that are not addressed yet from the old PR:
https://github.com/kubevirt/kubevirt/pull/4944#discussion_r662892443

Reading files with<EMAIL_ADDRESS>should be very easy.
Writing them by using<EMAIL_ADDRESS>may not be too hard either.
If we write them with the vmi/namespace.name syntax they an easily be identified and managed in more advanced ways later.
I am uncertain if that is something for this PR.
I'll look into it, but would probably prefer following-up on it as I'm not entirely sure yet if it should be in here just yet.
But I like the idea.

https://github.com/kubevirt/kubevirt/pull/4944#discussion_r662924969

But at the end we will see in the follow-up PR what to do with it. We can also consider adding a virtcl config-ssh command which can for instance add a few nice entries to .ssh/config where people can then simply write ssh vmi/myvm.mynamespace afterwards and we are not involved anymore at all.
The idea would be like at https://cloud.google.com/sdk/gcloud/reference/compute/config-ssh, but not necessarily limited to currently running VMs since we can add generic entries:
Host vmi/*
   User root
   ProxyCommand virtctl ssh --runAsProxyCommand=true %h --port %p

I like the idea, would suggest this as a separate PR to keep this one focused on just the basic SSH command.
Until then we have the docs mentioning it.

/assign @rmohr

/retest

Yes I think either using a local openssl binary or maybe there is something official for golang to generate keys like that (I'd expect so).

And we can support to just provide a keypair and the command will use that instead, but also talk to the credentials API to register they public key first.

/retest

/lgtm
/approve
The docs can be moved later on, and I will add e2e tests in about a week. Thanks! :+1:

/unhold


Too many open files error during dependency detection

I suspect this only happens during dependency detection (and not regular compilation) because a greater percentage of time is spent loading headers.  This is pretty easily fixed with ulimit but it might be good to limit the number of detection processes (it's currently limited to the task limit, which I would have thought would be sufficient).
error: Too many open files (os error 24)
   ┌─ ergo-cpp-0.12.1/src/operations.ergo:68:21
   │
68 │                 child = exec ~env=toolchain:tool-exec-env $compiler ^[
   │ ╭───────────────────────^
   │ │ ╭─────────────────────'
69 │ │ │                 ^tc-flags
70 │ │ │                 ^flags
71 │ │ │                 ^public-include-flags
   · │ │
75 │ │ │                 $tracked-file
76 │ │ │             ]
   │ ╰─│─────────────^ while spawning this process
   │   ╰─────────────' in this call

one or more errors occurred

This will occur if something is persisting the child process values rather than using them immediately. This used to occur before a few script changes I made. I was able to check the number of open files using procfs to debug/verify fixes.


feat: Adding AWS S3 as notification method options for Alerts/Reports

SUMMARY

We are adding the implementation and usage of the AWS S3 upload feature in Apache Superset. This feature allows users to send reports directly to an AWS S3 bucket. There are three methods available for authentication: taking credentials from the user in the frontend, taking credentials from the superset_config.py file, or utilizing an AWS IAM role associated with the ECS task role.

TESTING INSTRUCTIONS

Installation and Setup

Ensure Apache Superset is installed and running.

Ensure S3 bucket policy is configured for interacting with AWS S3.

Configure Apache Superset to enable the AWS S3 upload feature.

Follow the method-specific setup instructions outlined below.

Common Configuration for Enabling AWS S3 feature for all below method.

User should update the superset_config.py (docker/pythonpath_dev/superset_config.py) file with following:

Uncomment the following code in superset_config.py.

# AWS S3 reporting - uncomment to use
# FEATURE_FLAGS = {"ALERT_REPORTS": True, "ENABLE_AWS": True}
#ALERT_REPORTS_NOTIFICATION_DRY_RUN = False
# AWS Credentials
#AWS_ACCESS_KEY = '####'
#AWS_SECRET_KEY = '####'

Method 1: Frontend User Credentials

When accessing the feature in the frontend, users will be prompted to enter their AWS credentials.

Users should provide their AWS Access Key and the AWS Secret Access Key in the user interface itself.

Ensure proper IAM policies are applied to the user's credentials to grant necessary permissions for uploading to the specified S3 bucket.

Method 2: config.py Credentials

Update the config.py file in your Apache Superset installation directory.

Add the AWS credentials as configuration variables in superset_config.py.

Ensure proper IAM policies are applied to the user's credentials to grant necessary permissions for uploading to the specified S3 bucket.

Method 3: AWS IAM Role

Ensure that the ECS task running Apache Superset is associated with an IAM role.

Configure the IAM role's permissions to allow uploading to the desired S3 bucket.

Apache Superset will automatically use the IAM role associated with the ECS task for authentication when uploading to S3.

Usage
Once setup has been done, the AWS S3 can be used for sending reports/alert directly to the S3 bucket using following0 AWS S3 method options:

AWS_S3_credentials

AWS_S3_pyconfig

AWS_S3_IAM

· Navigate to settings, then on Alerts & Reports.
· Create alert /report like we use to do with the Email or Slack.
· In the Notification Method select S3.
· Then select the available S3 method options: AWS_S3_credentials, AWS_S3_pyconfig, AWS_S3_IAM.
· Fill the required aws credentials in case of AWS_S3_credentials else just give the bucket name to use.
Like Email, we can also create the reports directly to the Aws S3 bucket from the dashboard and charts page. All three S3 method options would be available from the dashboard and charts page.
· Navigate to any of the dashboard or Chart.
· Create the report using Set up a S3 report and use any of the S3 method options available.
· Generated report can be seen in the Alerts & Reports page.
Note: If the User doesn’t enable the feature flag of ENABLE_AWS, but has ALERT_REPORTS set to True then Aws s3 option wouldn’t be available but he can still use the default Notification method from the Apache Superset i.e. of Email.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ x ] Required feature flags:
[ x ] Changes UI
[ x ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ x ] Introduces new feature or API
[ ] Removes existing feature or API

Documentation
This is an external link to Document for Frontend
This is an external link to Document for Backend

HI @NeHaSinghTeqfocus, thanks for your contribution. It looks like you're targeting the 2.1 release branch; can you retarget your PR to the latest master branch instead?

Hi @giftig As per client we developed the code on top of version 2.1.0, so we need to raise the PR against the same. However you have mentioned to raise the PR on the master branch but it shows major code conflicts. Can you please suggest us to raise the PR for the version 2.1.0 only.

Hi @giftig As per client we developed the code on top of version 2.1.0, so we need to raise the PR against the same. However you have mentioned to raise the PR on the master branch but it shows major code conflicts. Can you please suggest us to raise the PR for the version 2.1.0 only.

Unfortunately, we don't actively support 2.x releases anymore. In fact, it won't be long until 4.0 is out and 3.1 becomes the LTS support for patches (not features). We typically don't review/merge features onto older release branches, unless there's an opportunity to cherry-pick a fix from master onto a patch release (which would not contain features. There are no plans for a 2.2 release that would include features.
My advice would be to upgrade to 3.1, and rebase your code so it can be PR'd into master and we can pick it up in releases from now on. If it were merged onto 2.x, it's unlikely then that it would ever be in a future release, since those are cut from master.
The other option is to fork Superset and build on the 2.1.x release branch freely. You're just subject to fork drift at that point though, and may miss out on various features or security fixes in future versions... trust me that you don't want to cherry-pick everything from master onto an old fork... it can get hairy.

@rusackas @giftig Thanks for your response, Can you please just give me a confirmation on, what is the current Apache superset version on master branch so that we can further give this information to our client ?


Add download pseudo translations flag

This uses the pseudo attribute in API to download
files with pseudo translations.
The change is on the pull command, in which, if
pseudo flag is true we use the source file
download instead of the translations file download
because for compatibility issues. The pseudo
attribute will be removed from the API for translations
in the future and will be used only in source.

tx pull  --pseudo --force  is downloading the pseudo content in target language files, while it suppose to create or update en_pseudo.json file.


Analog Keypad wrong ranges

I have this keypad, it's 3x4 with analog output and it has 3k resistors for rows and 1k resistors for columns but the pulldown resistor is 10k instead of 1k as in the schematic you provide, the range of analog readings are totally diferent due this issue.
My analog values are
button 1 = 1023
button 2 = 930
button 3 = 850
button 4 = 790
button 5 = 730
button 6 = 680
button 7 = 640
button 8 = 600
button 9 = 570
button * = 540
button 0 = 510
button # = 490
How to fix this
because the range given by your default and changing resistor values is this
OnewireKeypad <Print, 12 > KP(Serial, KEYS, 4, 3, A0, 3000, 1000, MediumPrec );
V:5.00, AR: 1023.00 | V:2.50, AR: 511.50 | V:1.67, AR: 341.00 |
V:1.25, AR: 255.75 | V:1.00, AR: 204.60 | V:0.83, AR: 170.50 |
V:0.71, AR: 146.14 | V:0.63, AR: 127.87 | V:0.56, AR: 113.67 |
V:0.50, AR: 102.30 | V:0.45, AR: 93.00 | V:0.42, AR: 85.25 |

Regards

Solved, I look into the h file and find the constructor can add a third resistor value to do that, so I note that I tried the example and thorws a few wrong keys by bad debouncing mostly # after some keys
How to fix this

The code assumes the pull down resistor is the same as the other column resistors (1k).
I would need to add a way to change this. Give me a few days and I should have a fix for you.
Sent from my iPhone

On Aug 30, 2017, at 2:32 PM, dragoblaztr<EMAIL_ADDRESS>wrote:
I have this keypad, it's 3x4 with analog output and it has 3k resistors for rows and 1k resistors for columns but the pulldown resistor is 10k instead of 1k as in the schematic you provide, the range of analog readings are totally diferent due this issue.
My analog values are
button 1 = 1023
button 2 = 930
button 3 = 850
button 4 = 790
button 5 = 730
button 6 = 680
button 7 = 640
button 8 = 600
button 9 = 570
button * = 540
button 0 = 510
button # = 490
How to fix this
because the range given by your default and changing resistor values is this
OnewireKeypad <Print, 12 > KP(Serial, KEYS, 4, 3, A0, 3000, 1000, MediumPrec );
V:5.00, AR: 1023.00 | V:2.50, AR: 511.50 | V:1.67, AR: 341.00 |
V:1.25, AR: 255.75 | V:1.00, AR: 204.60 | V:0.83, AR: 170.50 |
V:0.71, AR: 146.14 | V:0.63, AR: 127.87 | V:0.56, AR: 113.67 |
V:0.50, AR: 102.30 | V:0.45, AR: 93.00 | V:0.42, AR: 85.25 |
Regards
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@AndrewMascolo
I already fix this as next
OnewireKeypad <Print, 12 > mk1(Serial, KEYS, 4, 3, A0, 3000, 1000, 10000, 30 );
But sometimes it throws inacurrate values (I think poor debouncing)
Thank you in advance

In the function just replaced
float V = (voltage * float( R2 )) / (float(R2) + (float(R1) * float(R)) + (float(R2) * float(C)));
with this
float V = 0;
      if (R3 != 0) {
        V = (voltage * float( R3 )) / (float(R3) + (float(R1) * float(R)) + (float(R2) * float(C)));
      } else {

        V = (voltage * float( R2 )) / (float(R2) + (float(R1) * float(R)) + (float(R2) * float(C)));
      }

The denounce is based on the precision number, so instead of 30, try 40 or 50
Sent from my iPhone

On Aug 30, 2017, at 4:54 PM, dragoblaztr<EMAIL_ADDRESS>wrote:
In the function just replaced
float V = (voltage * float( R2 )) / (float(R2) + (float(R1) * float(R)) + (float(R2) * float(C)));
with this
float V = 0;
if (R3 != 0) {
V = (voltage * float( R3 )) / (float(R3) + (float(R1) * float(R)) + (float(R2) * float(C)));
} else {
    V = (voltage * float( R2 )) / (float(R2) + (float(R1) * float(R)) + (float(R2) * float(C)));
  }

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Imperial Raider translation

-Translated all the cards from the Imperial Raider expansion pack.
-Added several of them that were blank on the Card Browser (most of the Huge Ships Only crew).
-Added translations for the Wave 7 Expansion Packs and ships, as we already know what will be the official translation. Last time this broke something on another file causing a couple problems (no pilots on a certain ship, another without certain slots if I remember correctly), so you may want to check it. In any case, I will notify you of any issue after the merge.
And that's it, I think :P

Checking the Travis CI report, seems to be the same problem as last time ( https://github.com/geordanr/xwing/pull/187 ), with all the new ship translated names entering in conflict with the test files.
Hope this time there isn't any unseen typo on them :P


added missing dependencies in package.json

The app build was failing, so I added these dependencies, now it's working!

@SunitRoy2703  May i know the issue when you running the code? As we are running the code successfully without the dependencies


Error making gguf: KeyError: '<|user|>'

System Info / 系統信息
transformers: 4.44.0
llama.cpp: latest
Hi, when I try to make a gguf I get this error:

Traceback (most recent call last):
File "/home/david/llm/llama.cpp/convert_hf_to_gguf.py", line 4074, in
main()
File "/home/david/llm/llama.cpp/convert_hf_to_gguf.py", line 4068, in main
model_instance.write()
File "/home/david/llm/llama.cpp/convert_hf_to_gguf.py", line 388, in write
self.prepare_metadata(vocab_only=False)
File "/home/david/llm/llama.cpp/convert_hf_to_gguf.py", line 381, in prepare_metadata
self.set_vocab()
File "/home/david/llm/llama.cpp/convert_hf_to_gguf.py", line 3713, in set_vocab
special_vocab._set_special_token("eot", tokenizer.get_added_vocab()["<|user|>"])
~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^
KeyError: '<|user|>'

Do you know how to fix this?
On huggingface someone else has the same problem:
https://huggingface.co/THUDM/LongWriter-glm4-9b/discussions/1#66bc33eccd16fda66e7caa1f
But I don't know how to apply this solution:

Hi! You can get the token id by tokenizer.get_command("<|user|>").

Is the EOT even needed?
Thank you!
Who can help? / 谁可以帮助到您？
No response
Information / 问题信息

[X] The official example scripts / 官方的示例脚本
[X] My own modified scripts / 我自己修改的脚本和任务

Reproduction / 复现过程
With llama.cpp:
python convert_hf_to_gguf.py /home/david/llm/LongWriter-glm4-9b --outtype f32
Here is the code:
    special_vocab = gguf.SpecialVocab(dir_model, load_merges=False)
    special_vocab.merges = merges
    # only add special tokens when they were not already loaded from config.json
    special_vocab._set_special_token("eos", tokenizer.get_added_vocab()["<|endoftext|>"])
    special_vocab._set_special_token("eot", tokenizer.get_added_vocab()["<|user|>"])
    # this one is usually not in config.json anyway
    special_vocab._set_special_token("unk", tokenizer.get_added_vocab()["<|endoftext|>"])
    special_vocab.add_to_gguf(self.gguf_writer)

Expected behavior / 期待表现
For it to make a quantization.

Hi! You can get the token id by tokenizer.get_command("<|user|>").

Have you updated to our most recent model files? Also, please use transformers>=4.43.0.

@bys0318 thank you, it appears that the token id is:

151336

Is this correct?
in llama.cpp:
llm_load_print_meta: general.name     = LongWriter Glm4 9b
llm_load_print_meta: EOS token        = 151329 '<|endoftext|>'
llm_load_print_meta: UNK token        = 151329 '<|endoftext|>'
llm_load_print_meta: PAD token        = 151329 '<|endoftext|>'
llm_load_print_meta: LF token         = 128 'Ä'
llm_load_print_meta: EOT token        = 151336 '[PAD151336]'
llm_load_print_meta: max token length = 1024

@echnio
I did this (find the lines starting at 3711) and replace:
    # only add special tokens when they were not already loaded from config.json
    special_vocab._set_special_token("eos", tokenizer.get_added_vocab()["<|endoftext|>"])
    token_id = tokenizer.get_command("<|user|>")
    print(token_id)
    special_vocab._set_special_token("eot", token_id)
    # this one is usually not in config.json anyway

This is correct. Thanks for sharing!

@bys0318谢谢，看来令牌 ID 是：

151336

这是正确的吗？
在 llama.cpp 中：
llm_load_print_meta: general.name     = LongWriter Glm4 9b
llm_load_print_meta: EOS token        = 151329 '<|endoftext|>'
llm_load_print_meta: UNK token        = 151329 '<|endoftext|>'
llm_load_print_meta: PAD token        = 151329 '<|endoftext|>'
llm_load_print_meta: LF token         = 128 'Ä'
llm_load_print_meta: EOT token        = 151336 '[PAD151336]'
llm_load_print_meta: max token length = 1024

@echnio
我这样做了（找到从 3711 开始的行）并替换：
    # only add special tokens when they were not already loaded from config.json
    special_vocab._set_special_token("eos", tokenizer.get_added_vocab()["<|endoftext|>"])
    token_id = tokenizer.get_command("<|user|>")
    print(token_id)
    special_vocab._set_special_token("eot", token_id)
    # this one is usually not in config.json anyway

Thank you very much, the format conversion was successful.

@bys0318 thank you, it appears that the token id is:

151336

Is this correct?
in llama.cpp:
llm_load_print_meta: general.name     = LongWriter Glm4 9b
llm_load_print_meta: EOS token        = 151329 '<|endoftext|>'
llm_load_print_meta: UNK token        = 151329 '<|endoftext|>'
llm_load_print_meta: PAD token        = 151329 '<|endoftext|>'
llm_load_print_meta: LF token         = 128 'Ä'
llm_load_print_meta: EOT token        = 151336 '[PAD151336]'
llm_load_print_meta: max token length = 1024

@echnio
I did this (find the lines starting at 3711) and replace:
    # only add special tokens when they were not already loaded from config.json
    special_vocab._set_special_token("eos", tokenizer.get_added_vocab()["<|endoftext|>"])
    token_id = tokenizer.get_command("<|user|>")
    print(token_id)
    special_vocab._set_special_token("eot", token_id)
    # this one is usually not in config.json anyway

Thanks for the detailed steps!
I was able to convert the model.
PLease find quants at QuantFactory/LongWriter-glm4-9b-GGUF


Soft delete

Description
Outline the issue here:

Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here:
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here:
OS Version: What OS and version are you using?
Answer here:
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Answer here:

Moves to the next sprint.

Moved out of the sprint 26 to backlog as the service is not ready.

Done and released as of version 2.0.27


Finished unity install not auto-detected

After finishing installation in UnityHub of the suggested version, RW4D does not automatically continue, going one step back and then forwards again picks up the installation.

I think I forgot the refresh button (or some polling in the background). Could also be related to #4


CPCO-417: Upstream Datadog Monitors

what

Upstreaming datadog monitors

why

Upstream datadog monitors to our public catalog

references

CPCO-417

@nitrocode @aknysh — Looks like these would be pretty useful… Any plan to come back to upstream them?


BGP listen port is not changed. (calico v3.16.3)

This issue is what I asked on slack and is related to #4098
https://calicousers.slack.com/archives/CPTH1KS00/p1602833003052900

Calico has been updated to version 3.16.3

[root@master ~]# docker images|grep calico
calico/node                                        v3.16.3             f0d3b0d0e32c        6 days ago          164MB
calico/pod2daemon-flexvol                          v3.16.3             a0b97353aa18        6 days ago          22.9MB
calico/cni                                         v3.16.3             fe49caa20c30        6 days ago          133MB
[root@master ~]# calicoctl version
Client Version:    v3.16.3
Git commit:        7d066703
Cluster Version:   v3.15.2
Cluster Type:      kubespray,bgp,kubeadm,k8s

And I wanted to apply the BGP port change spec, but it didn't apply even after restarting the callico-node pod.

[root@master ~]# calicoctl get BGPConfiguration default -o yaml --export
apiVersion: projectcalico.org/v3
kind: BGPConfiguration
metadata:
  creationTimestamp: "2020-09-09T10:08:42Z"
  name: default
  resourceVersion: "11573515"
  uid: 0ef5e0ef-db19-4afa-a906-1a7c95d7aaab
spec:
  asNumber: 64512
  listenPort: 11178
  logSeverityScreen: Info
  nodeToNodeMeshEnabled: true
[root@master ~]# ps -ef|grep bird
root      53074  53022  0 17:36 ?        00:00:00 runsv bird6
root      53075  53022  0 17:36 ?        00:00:00 runsv bird
root      53214  53074  0 17:36 ?        00:00:00 bird6 -R -s /var/run/calico/bird6.ctl -d -c /etc/calico/confd/config/bird6.cfg
root      53215  53075  0 17:36 ?        00:00:00 bird -R -s /var/run/calico/bird.ctl -d -c /etc/calico/confd/config/bird.cfg
root      60956  55828  0 17:43 pts/0    00:00:00 grep --color=auto bird
[root@master ~]# netstat -nap |grep LISTEN|grep bird
tcp        0      0 <IP_ADDRESS>:179             <IP_ADDRESS>:*               LISTEN      53215/bird
unix  2      [ ACC ]     STREAM     LISTENING     228609567 53215/bird           /var/run/calico/bird.ctl
unix  2      [ ACC ]     STREAM     LISTENING     228610782 53214/bird6          /var/run/calico/bird6.ctl

So I tried to apply it manually, but it didn't work either.
[root@worker /]# echo 'listen bgp port 11179' >> /etc/calico/confd/config/bird.cfg
[root@worker /]# grep 11179 /etc/calico/confd/config/bird.cfg
listen bgp port 11179
[root@worker /]# sv hup bird
[root@worker /]# exit
exit
command terminated with exit code 127
[root@master ~]# netstat -nap|grep LISTEN|grep bird
tcp        0      0 <IP_ADDRESS>:179             <IP_ADDRESS>:*               LISTEN      121692/bird

Expected Behavior
Change BGP listen port. (179->11179)
Current Behavior
BGP listen port is not changed.

Anyone can answer? :(

@neiljerram Could you look into this?

@song-jiang @suiny Sure, I've started to take a look.  Please be patient though, as I don't see a really obvious cause here!

@suiny Sorry for such a long hiatus.  Is this issue still live for you?

So I tried to apply it manually, but it didn't work either.

I think you need a semicolon at the end of listen bgp port 11179.
Recommend birdcl configure, instead of sv hup bird, as birdcl will tell you if the new configuration is valid.

calicoctl get BGPConfiguration default

When you use BGPConfiguration to set the listen port, can you use cat /etc/calico/confd/config/bird.cfg to show us the BIRD config that you have?

I checked that changes bgpport (179->11179) as the yaml file on the v3.18.4 version.

@suiny Thanks.  Reviewing this whole issue, I have a guess at the problem.

[root@master ~]# calicoctl version
Client Version:    v3.16.3
Git commit:        7d066703
Cluster Version:   v3.15.2
Cluster Type:      kubespray,bgp,kubeadm,k8s

Note v3.15.2 - It looks like you haven't actually upgraded the running Calico components on each node.  That would explain why the BGP port configuration was not reflected in your bird.cfg (until you added it manually).

That's strange.
I upgraded calico to 3.16.3 but it was not reflected.  (follow this link)
However, the image was used as 3.16.3.
[root@master ~]# calicoctl version
Client Version:    v3.16.3
Git commit:        7d066703
Cluster Version:   v3.15.2
Cluster Type:      kubespray,bgp,kubeadm,k8s
[root@master ~]# docker images |grep calico
calico/node                                        v3.16.3             f0d3b0d0e32c        8 months ago        164MB
calico/pod2daemon-flexvol                          v3.16.3             a0b97353aa18        8 months ago        22.9MB
calico/cni                                         v3.16.3             fe49caa20c30        8 months ago        133MB

The docker image might not be same as what the Kubernetes runtime has available, or is actually using.  What do you get for kubectl get ds calico-node -n calico-system -o yaml | grep image: ?  (Or you might need kube-system instead of calico-system.)

@neiljerram
I used same image (v3.16.3)
[root@dev-suinyk8s001-ncl ~]# kubectl get pods -n kube-system calico-node-89pm4 -oyaml | grep image:
            f:image: {}
            f:image: {}
            f:image: {}
            f:image: {}
    image: calico/node:v3.16.3
    image: calico/cni:v3.16.3
    image: calico/cni:v3.16.3
  - image: calico/pod2daemon-flexvol:v3.16.3
    image: calico/node:v3.16.3
    image: calico/cni:v3.16.3
    image: calico/cni:v3.16.3
    image: calico/pod2daemon-flexvol:v3.16.3

Looks good, thanks.  Did you change that just now, or has it been like that for a long time?

It's been like that for a long time.
The bgport did't change when using v3.16.3 version's image, and I think that it was a bug.
I recently created a new cluster with v3.18.4 , the bgp port changed.
It's goot that the bug is resolved.


How do I download from all pages I follow?

I am pretty new here and to this program. I cannot figure out how to download all of the pages that my profile is following. How do I do so?

By 'pages' do you mean PROFILEs? Plz be more clear.
Also does 'download all of the pages' mean all the posts on that page/profile?

By 'pages' do you mean PROFILEs? Plz be more clear.
Also does 'download all of the pages' mean all the posts on that page/profile?

I apologize. Let me be more descriptive.
I follow around 500 profiles, currently. I would like to download all the profiles that I follow on a currently updated basis. I have been using 4KStogram for a while but the program doesn't have a way to check for Instagram's rate limits. It's only downfall really. So the steps I would to take would be:

Download all profiles I follow.
Update said profiles when I run the program.
Be able to add/check for new profiles I follow and download.

It sounds like you simply want to download all followees. This can be achieved by using the target @YOUR_PROFILENAME like described in the documentation.
To update the profiles you can use --fast-update.

OK. I almost got it... I think. What am I missing?
".\instaloader  --login=username @profile"
Do I enter my profile username for @profile like "@username"? I tried that and it doesn't work.


[CIT-293] Add lock to allow only one mapper running

Proposed changes
This introduces crate flockfile which allows for only single mapper running on the system.
Types of changes
What types of changes does your code introduce to thin-edge.io?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[x] I used cargo clippy as mentioned in CODING_GUIDELINES
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Codecov Report

Merging #231 (f306f95) into main (6e7da14) will increase coverage by 0.17%.
The diff coverage is 76.08%.

@@            Coverage Diff             @@
##             main     #231      +/-   ##
==========================================
+ Coverage   64.85%   65.03%   +0.17%
==========================================
  Files          56       57       +1
  Lines        3224     3269      +45
==========================================
+ Hits         2091     2126      +35
- Misses       1133     1143      +10

Impacted Files
Coverage Δ

mapper/tedge_mapper/src/main.rs
0.00% <0.00%> (ø)

common/flockfile/src/lib.rs
87.50% <87.50%> (ø)

Continue to review full report at Codecov.

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


Remove boilerplate license headers

This is just an itty bitty nit I have, dunno if you want to go through with it. The Apache license doesn't require a header and we haven't been using one in new files for ages now, so I just wanted to make our code consistent and get rid of our current headers (except for the twitter code which has the special modifications license header). What do you think?

@SUPERCILEX we actually can't do this :-( technically we are running afoul of Google policy by not adding the header to new files.  So if we do anything about this we should probably use the Android Studio license adder thing to fix it across the board.
But let's save all that code churn for a more quiet time.

Haha, ok thanks for the info! 😄


Msfr updates

Job VTB Documentation on 996cdea wanted to post the following:
View the site here
This comment will be updated on new commits.


Track expense lhn

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/41183
PROPOSAL: https://github.com/Expensify/App/issues/41183#issuecomment-2090617822
Tests

Track an expense
Select any of the actionable whisper options
Confirm that the LHN displays the last visible message instead of 'What would you like to do with this expense?

[x] Verify that no errors appear in the JS console

Offline tests

Track an expense
Go offline
Select any of the actionable whisper options
Confirm that the LHN displays the last visible message instead of 'What would you like to do with this expense?

QA Steps

Track an expense
Go offline
Select any of the actionable whisper options
Confirm that the LHN displays the last visible message instead of 'What would you like to do with this expense?

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/93134676/867564b7-da21-4e18-8196-4f1a7757c46c

Android: mWeb Chrome

https://github.com/Expensify/App/assets/93134676/584af70b-d958-4424-b259-7df9c8ead663

iOS: Native

https://github.com/Expensify/App/assets/93134676/e623f9b9-d983-4551-824f-3da09cac4d5a

iOS: mWeb Safari

https://github.com/Expensify/App/assets/93134676/eea6d9fb-3121-4e04-be74-2a5703139abd

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/93134676/5863d966-7afa-44ce-b97a-18219736b3aa

MacOS: Desktop

https://github.com/Expensify/App/assets/93134676/b8bcdd15-d850-4ed3-b131-018edb67fa45

@c3024 We need to address this on the backend too. It works fine in offline mode, but when the open report API is called, the last message text reverts to 'What would you like to do with this expense?
We still have one unhandled case regarding expense deletion. Let's first test this solution, and if everything works fine, we'll decide whether we need to fix that here too.

I see it working fine for all but Do nothing because the expense is moved to another report in those cases.
I found the last message showing as What do you want to do with this expense? when I choose Do nothing for an expense. But it should show the last tracked expense.
I see we only use the lastMessageText from report as fallback here
https://github.com/Expensify/App/blob/23297deeab8d16c98ef2af028171a0c7ec70a039/src/libs/OptionsListUtils.ts#L625
and this here
https://github.com/Expensify/App/blob/2d621b786fc4afbf94a51ed3a48b2da1e927c1c6/src/libs/ReportActionsUtils.ts#L304
is filtering track actions as well and removing the track action in the sorted report actions to display here
https://github.com/Expensify/App/blob/23297deeab8d16c98ef2af028171a0c7ec70a039/src/libs/OptionsListUtils.ts#L274
and only CREATED action is left in the actions and the fallback is taken as last message. Since the backend still sends the whispers also as report actions I think we need not change the lastMessage sent from backend. We should fix the above to include the track action in the actions to display.

@Nodebrute
Please check this. I don't think it needs a backend fix.

@c3024 I'm on it. I'll complete it today.

@c3024 The issue with the 'do nothing' behavior has been resolved after making this change. However, we're still encountering the same message from the backend when the 'open report' API is called
https://github.com/Expensify/App/assets/93134676/a0c72c60-3522-46d0-8452-9afbb12785ad

@Nodebrute
please fix the lint and ts failures.

@c3024 Hey, could you help me solve this TypeScript error? I've tried but haven't had any luck.

@Nodebrute there are conflicts as well now.

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

All works well except with a small issue.
Do nothing fails again because this filter
https://github.com/Expensify/App/blob/525ad6ffede2f0127c0759855f020e54fc704403/src/libs/OptionsListUtils.ts#L299-L302
now does not use shouldReportActionBeVisibleAsLastAction where isResolvedActionTrackExpense change is included.
So, we should change this
https://github.com/Expensify/App/blob/525ad6ffede2f0127c0759855f020e54fc704403/src/libs/OptionsListUtils.ts#L299-L302
to
ReportActionUtils.shouldReportActionBeVisible(reportAction, actionKey) &&
                    !ReportActionUtils.isWhisperAction(reportAction) &&
                    reportAction.actionName !== CONST.REPORT.ACTIONS.TYPE.CREATED &&
                    reportAction.pendingAction !== CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE &&
                    !ReportActionUtils.isResolvedActionTrackExpense(reportAction),

@c3024 Hey, I have resolved the conflicts in my code. I have tested it and it still shows the message when the openReport API is called. Can you help me understand what the next steps are to finish this up? Let's get this issue closed.

Yes, this needs a backend fix separately.

Please fix this and it will be done.

@thienlnam there is a backend fix required as well because when all track expenses are submitted/categorized (when all track expense whispers are resolved) there are no valid visible report actions and backend sends last message for the report as "What would you like to do with this expense?" even though all the whispers are resolved and expenses are moved to other reports.
I think backend should not send this as last message text when all the track expense whispers are resolved.

Thanks, I'll create an issue to track that internally

Is this still a problem? https://github.com/Expensify/App/pull/41668/files#r1600275772

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/thienlnam in version: 1.4.77-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.4.77-11 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.4.77-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Whether high-speed motion of the camera during sampling will affect the accuracy of 3D data ？

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
L515

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 18

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC

SDK Version
2.53.1

Language
C++

Segment
Robot

Issue Description
<Describe your issue / question / feature request / etc..>
I used the L515 to assist in positioning on a mobile robot. At that time, there was a phenomenon that when the robot moved slowly, the positioning effect was significantly better than when the robot moved faster (of course, it could also be that the recognition effect was poor when the robot moved faster). Later, when I communicated with our colleagues working on two-dimensional radar (who also perform auxiliary positioning on high-speed mobile robots with a data update period of 25ms), I discovered that they have implemented a dynamic compensation for the data, which can play a significant role in their auxiliary positioning.
Then I thought that high-speed mobile robots have such a significant impact on the radar for 25ms of data updates, and that would also have an impact on the camera. So I wanted to follow their approach to dynamic compensation, but due to different data update principles, I haven't figured out how to specifically implement it.
I just want to ask if the camera is moving at high speed during camera sampling, which may have some adverse effects on the depth or 3D data of the camera? If there is an impact, can we use dynamic compensation methods to offset this impact, as well as reference dynamic compensation methods?

The L515 is designed to be able to scan fast moving objects with no motion blur, as described in its official YouTube launch video at the link below.
https://www.youtube.com/watch?v=Rx-75ZEAMeg
If you are using the RGB color stream then it may be worth having depth set at 30 FPS (the only depth speed available on L515) and RGB at 60 FPS.  The RGB sensor on the L515 has a rolling type shutter, which can be susceptible to lag / blurring when capturing fast motion.  Increasing RGB FPS to 60 can help to negate this.

OK, I will have a try.

No, thanks. You can turn it off at will.

Thanks very much for the update!


BUG: Incompatible with recent Kimai Version 2.0.33

I've upgraded Kimai to 2.0.33 some minutes ago.
Now I don't get a Project, NOR actions for a new task.
Not for new Tasks, and even not for any recent tasks:

Whole Cache-Object is empty :

Yes, refresh-button, disable (also re-enabeling) caching in dev-tools, etc.
I'd closed it till now and used the integrated (web-base) tracker of kimai.
But now I've started it again and ... strange enough, currently it work again, and I don't know why.

Hi,
I mean this button, no any cache in dev-tools.

I think this was happends:
When you start CodeTimer, cache was old, so CodeTimer tries to load actual data. Kimai server was not ready yet, so there was no response from server, cache was empty. Then you tried to load timesheets, server was ready, but cache was still empty - that's why you have not any customer/activity/project. When you click to this button, cache should be refreshed and everything should work normalny. Cache is valid for one hour, so after you reopen CodeTimer after one hour, cache is automatic calling Kimai server, then loads actual data and app will work.
I also installed Kimai v 2.0.33 and have not any issue with CodeTimer.


if using service worker, b64 decode of token in extractTokenPayload of oidcserviceworker.js fails for some tokens

Issue and Steps to Reproduce
if using token like following one :
eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCIsImtpZCI6ImtleXN0b3JlLUNIQU5HRS1NRSJ9.eyJzdWIiOiJ0ZXN0RXh0ZXJuYWwiLCJmYW1pbHlfbmFtZSI6IuOCouOCr-OCtSIsImdpdmVuX25hbWUiOiLlpKrpg44iLCJlbWFpbCI6InRlc3RAdGVzdC5jb20iLCJ1aWQiOiJ0ZXN0RXh0ZXJuYWwiLCJ1c2VyX3R5cGUiOiJleHRlcm5hbCIsImF1dGhfdGltZSI6MTY2MzIyMTczOCwibm9uY2UiOiJtbTRsRzNUNU5jbzAiLCJhdF9oYXNoIjoieGlfemNwYlMwZFY5MG45OUhPa21ldyIsImF1ZCI6ImZvbyIsImV4cCI6MTY2MzIyNTQwMSwiaWF0IjoxNjYzMjIxODAxLCJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjMwMDIifQ.xQa9pfcfqO14BaEIF6UZQxe3i7mzqo0D4HxXnXgTMBAJEe2Q5424_nW5TIiGzcrD_yWG6yusv6hDauFkcRDBNjmHiyyqR3yESfCN3SXwTQvi7wIPiVvn92rynbU09EpPQcff9_5QCIFx9h7K-W91J-1WPnI3sxL6AwLfb-KIjoS_5UV1szW4IL6MeMXypbgUpAZksvlKkQM78qPgmC64KqdxKhYDxt8kFZjyg4jyFf_oIGp7FLDQyGA8u9B21gndrJEtRkKXB8Z3A2s5Ujv3cDpyV_gQa3OL1i42Ev7rU86OXCFFJsO71kL3PDKF9vg3n9m9DvClQhLTUIAwRGJ7hg

the decode of token in extractTokenPayload function of oidcserviceworker.js file fails in atob function call saying it is not encoded properly.
Versions
latest
Screenshots

Expected
decode of token should work
Actual
Uncaught DOMException: Failed to execute 'atob' on 'Window': The string to be decoded is not correctly encoded.
Additional Details
A Solution would be:
It is a unicode problem cf here https://stackoverflow.com/a/30106551/5277071
the 8th solution from this post: https://www.anycodings.com/1questions/145912/how-to-decode-jwt-token-in-javascript-without-using-a-library is good to solve the issue.
Following code solve the issue:
const b64DecodeUnicode = (str) =>
  decodeURIComponent(Array.prototype.map.call(atob(str), (c) => '%' + ('00' + c.charCodeAt(0).toString(16)).slice(-2)).join(''));
const parseJwt = (token) => JSON.parse(b64DecodeUnicode(token.split('.')[1].replace('-', '+').replace('_', '/')));
const extractTokenPayload = (accessToken) => {
  try {
    return parseJwt(accessToken);
  } catch (e) {
    console.warn(e);
  }
  return null;
};

while this fix the parsing of token correctly, i now see following in console logs:

it seems str is not a string in function countLetter ?

en fait le prob est dans packages/react/src/oidc/vanilla/parseTokens.ts (je testais sans le service worker)

Do you have an imput sample of that function just before the crash? I would be able to set up à unit test on it.
Thank you again @fffan64 ^^ I hope your going well in Japan 😜

@guillaume-chervet , problem is linked with my other problem https://github.com/AxaGuilDEv/react-oidc/issues/867, i can't debug properly :D

it is called 2 times, one of which str is an object and not a string :D

Error is here : https://github.com/AxaGuilDEv/react-oidc/blob/7250657ef33326115d265cbd60c28063c8a3bf81/packages/react/src/oidc/vanilla/parseTokens.ts#L44
passing the whole object instead of token (.accessToken i believe) only

Thank you very much @fffan64 , the version 6.6.3 fixed it !

Does the bug fixed for yoy @fffan64 ?
Thank you very much.
May we close the issues?

sure, works fine ! thanks again


arch: xtensa: check for xt-clang instead of xcc-clang

Check against new variant name. xcc-clang usage is deprecated.
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

Can we also remove main/cmake/modules/FindDeprecated.cmake#L40 ?

will need to be done in a different PR


package manager tool?

So far the zopen tool has been great for developers to work on porting OSS tools.  We will soon need a command/tool for installing and managing (upgrading?) these OSS tools and their version numbers, similar to other platforms' package management tools.  Any ideas here?
One idea is to have commands that allows for searching, listing local installations, and installing packages.  Example:
zopen list # List the locally installed packages
zopen search git # Search the list of available packages
zopen install git # Install the specified package
zopen purge git # Uninstall the specified package

Ideally, zopen would be both open sourced but also shipped with z/OS Unix so user's don't have to first install zopen on a fresh system, if that is possible.

Would this work? - https://github.com/basherpm/basher

Maybe. Have you tried it on another system?

Nope, but it seems straightforward enough to evaluate for those who are comfortable with the current zopen workflows.

not sure who opened this - it's against ZOSOpenTools. @IgorTodorovskiIBM ?

Ah, these were migrated from utils, this was the original issue: https://github.com/ZOSOpenTools/utils/issues/49. @jshimoda9323


[Feature Request] DATE_ADD() with optional timezone

At the moment it takes some conversions to securly add a day to a date that is not UTC. This query shows what I mean:
RETURN {
    a: DATE_LOCALTOUTC(DATE_ADD(DATE_TIMESTAMP(DATE_UTCTOLOCAL(1561932000000, 'Europe/Berlin')),200,'d'), 'Europe/Berlin'),
    b: DATE_ADD(1561932000000,200,'d')
}

This returns 2 different values due to day light saving is changing the timezone offset:
[
  {
    "a": "2020-01-16T23:00:00.000Z",
    "b": "2020-01-16T22:00:00.000Z"
  }
]

It would be great to just do this:
RETURN {
    a: DATE_ADD(1561932000000,200,'d','Europe/Berlin'),
    b: DATE_ADD(1561932000000,200,'d')
}

and get
[
  {
    "a": "2020-01-16T23:00:00.000Z", // or "2020-01-17T00:00:00.000+01:00"
    "b": "2020-01-16T22:00:00.000Z"
  }
]

I want to write a PR to this issue. There are 2 options to make this work.

Only add a timezone to each method like DATE_ADD(date, 1, 'd', 'Europe/Berlin')
Add the timezone to the query options like this{
   "query" : "RETURN DATE_ADD(1511132400000, 1, 'd')",
   "options" : {
     "timezone" : "Europe/Berlin"
   }
}

both, where the in-query declaration is dominant{
   "query" : "RETURN DATE_ADD(1511132400000, 1, 'd', 'Asia/Bangkok')",
   "options" : {
     "timezone" : "Europe/Berlin"
   }
}

That may result in the following method overload examples:
DATE_ADD(date, 'P1D') // Add one year
DATE_ADD(date, 'P1D', 'Europe/Berlin')  // Add one year assuming date is in timezone 'Europe/Berlin'
DATE_ADD(date, 1, 'd') // Add one year
DATE_ADD(date, 1, 'd', 'Europe/Berlin') // Add one year assuming date is in timezone 'Europe/Berlin'

There is one issue:
Since DATE_ADD also accepts string dates it might be confusing, that a UTC time string is another timezone. Also the result will be a UTC time string, even though it would be calculated in a given timezone.

Specifying a timezone in the query options looks a bit strange to me and would prohibit the valid use case of calculating multiple date/times with different timezones in the same query.
I think the timezone should to be an additional optional parameter to all date/time functions that should get timezone support. Alternatively, we could also add dedicated timezone-aware date/time variations of the existing date/time functions with modified call signatures (timezone parameter needs to go somewhere).

@jsteemann I wanted to actually contribute and write the code. I just wanted to know what you think is the best solution.

Specifying a timezone in the query options looks a bit strange to me and would prohibit the valid use case of calculating multiple date/times with different timezones in the same query.

My thoughts on that were, that the query option only holds the default timezone and if a DATE_-function with a timezone parameter is called, the functions parameter will be used instead of the default one. But this is just an idea. If you don't like the idea, I'd just go with something like this:
DATE_ADD(date, 'P1D', 'Europe/Berlin')

I think the timezone should to be an additional optional parameter to all date/time functions that should get timezone support. Alternatively, we could also add dedicated timezone-aware date/time variations of the existing date/time functions with modified call signatures (timezone parameter needs to go somewhere).

The following functions may have a timezone:
DATE_DAYOFWEEK()
DATE_YEAR()
DATE_MONTH()
DATE_DAY()
DATE_HOUR()
DATE_MINUTE()
DATE_SECOND()
DATE_MILLISECOND()
DATE_DAYOFYEAR()
DATE_ISOWEEK()
DATE_LEAPYEAR()
DATE_QUARTER()
DATE_DAYS_IN_MONTH()
DATE_TRUNC()
DATE_ROUND()
DATE_FORMAT()
DATE_ADD()
DATE_SUBTRACT()
DATE_DIFF()
DATE_COMPARE()
Most of them could take an optional parameter. The only problem would be:
DATE_DIFF()
DATE_COMPARE()
They already have an optional parameter, so for compatibility reasons the timezone would have to be added as last parameter. IMO that would be ok. What do you think?
Wouldn't having extra timezone aware functions kind of overload the date function section?
DATE_DAYOFWEEK_TIMEZONED()
DATE_YEAR_TIMEZONED()
DATE_MONTH_TIMEZONED()
DATE_DAY_TIMEZONED()
DATE_HOUR_TIMEZONED()
DATE_MINUTE_TIMEZONED()
DATE_SECOND_TIMEZONED()
DATE_MILLISECOND_TIMEZONED()
DATE_DAYOFYEAR_TIMEZONED()
DATE_ISOWEEK_TIMEZONED()
DATE_LEAPYEAR_TIMEZONED()
DATE_QUARTER_TIMEZONED()
DATE_DAYS_IN_MONTH_TIMEZONED()
DATE_TRUNC_TIMEZONED()
DATE_ROUND_TIMEZONED()
DATE_FORMAT_TIMEZONED()
DATE_ADD_TIMEZONED()
DATE_SUBTRACT_TIMEZONED()
DATE_DIFF_TIMEZONED()
DATE_COMPARE_TIMEZONED()

Hi @jsteemann my PR is more than 6 months old. Will it have a chance to get into a release any time?

Many thanks for your contribution and apologies for it not receiving the attention that it deserved sooner. We have just merged it to devel.


Hide specific or empty drives

What feature or improvement do you think would benefit Files?

Have an option to hide all empty drives (like an empty optical drive)
Have an option to hide specific drives (e.g. with very important data or drives that you do not have permission to open.)

Requirements

It would look cleaner
It would save space
It makes the use safer

Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22621.819
Windows Version
Windows 11 Pro 22H2 22621.819
Comments
No response

Merging with #9170

But that is not the same feature request

But that is not the same feature request

Can you elaborate?

In the feature request you mentioned, the drives (only empty ones) should be hidden in the sidebar.
I want to hide them everythere in the app, because i have no permissions even to see if the drives are empty.

The "hover over the drive to see Information"- feature wouldn't even work for the drives I mentioned.

You can right click on the drives section and click the option to hide it

But I only want to hide the hard drives that I can't use, that would be 3 out of 6, not all of them.


Add a logo

Right now the app is just using the default react logo. We should add a real logo.

Addressed as part of https://github.com/codeforsanjose/disaster-response-sj/pull/84


Project website not working, incl. download

The Project website https://pdnsmanager.org/ is not working correctly subsites couldn't be accessed.

Yes this was a error caused by a change I did a few days ago it should be fixed now.


Successive Strongs

This allows for successive strong tags to be created without breaking any existing tests.
Fixes #111

Merged and adjusted it for underscore also. Thank you!


[BUG] Formatting changed in version 0.6.3

Describe the bug
In the machine field there are no longer spaces
To Reproduce

run macchina

Expected behavior
Formatting like 0.6.2
Screenshots

System Information
You don't have to provide this information if you're not comfortable doing so, but it'll help us solve the issue a lot faster.

Distribution: Arch
Desktop Environment bspwm
Operating System Linux
Terminal Alacritty
Macchina's version 0.6.3

I'll check this out when I get home, should be an easy fix.
But to provide some context as to why this is happening, one of the recent versions included a little trick to remove duplicate words from product information, and like you mentioned, it isn't being formatted properly.

Newest libmacchina version fixes this issue, I would've mentioned the issue in the commit message if it were in libmacchina's repository.

I ended up fixing it in the car haha.

Haha thanks.

Please tell me if the issue is gone :)

Yes the issue is fixed.


Portable mode: old profile folder is recreated

Testing #172015
Version: 1.75.0-insider
Commit: 0a6c277e4faa44a6b6eb82ddca8bfeccf7fecef6
Date: 2023-01-24T05:22:55.489Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22623
Sandboxed: Yes

download and extract the VS Code insiders zip build to %HOMEPATH\VSCode-win32-x64-1.75.0-insider
create a data folder VSCode-win32-x64-1.75.0-insider\data
start code-insiders and create some profiles
open a folder and set one of the new profiles
close VS Code, rename VSCode-win32-x64-1.75.0-insider to VSCode-win32-x64-1.75.0-insider2
start VS Code again. It opens on the previous folder with the profile
check the disk. A folder VSCode-win32-x64-1.75.0-insider has been recreated: VSCode-win32-x64-1.75.0-insider\data\user-data\User\profiles\7898c455\snippets` (no files, just folders)

Great catch. Thanks for reporting.


Node card slots are inventories

LaserIO version
1.3.3
Minecraft Version
1.18.2
Forge Version
40.1.67
Modpack & Version
Direwolf20 1.18.2
Do you have optifine installed?
No
Describe the issue
Not calling as a bug - Nodes have 6 inventories, by design or not
Steps to reproduce
1 Connect any side of a node to a pipe or conduit and a chest, bottom also interacts with hopper
2 Put card into that side of the node
3 Card is pulled out.
Expected behaviour
Don't know... but was not expecting that
Screenshots
No response
Log files
No response
Additional information
It isn't a huge problem, likely edge case type thing. I was modifying a build, moving 2 blocks up and inserting a node was the simple solution, and I tried to put a node with a counter over a hopper only to discover the node went missing, figured out where and why but curios to find out if this was by design, accidental or the only way Dire knows to get the mod to work, I have no idea if there is another solution but I suspect there is, I mean I have never, that I can recall, connected pipes etc to an IF or thermal machine and extracted the upgrades.
Just curious.

It’s intended :). I was hoping it would lead to some cool things (like automatically changing out cards) but usually it leads to problems with hoppers instead. Might remove it in 1.19


Check on buggy Ansible version for ntp role

Only upgrade if ansible_version.full == 2.9.6

Doing version compare on pre_tasks


Models for supporting VMware Storage Profile

Models for inventory of storage profiles to support VMware Storage Profile.

Storage Profile vs Datastore
A Storage Profile is related many datastores that satisfy its requirements. A datastore can have satisfy multiple Storage Profile.

Storage Profile vs VM/VMDK
A VM/VMDK can be associated with one Storage Profile while one Storage Profile can be applied to multiple VM/VMDK.

At this stage of implementation (and this PR), we only support this.

@miq-bot add_labels enhancement, providers/vmware/infra

@miq-bot remove_label wip

LGTM :+1:

@Fryguy GB is out, can you check this out and see if it can be merged?
Thanks

LGTM, my only question is what is the Rails norm for model ordering.  Regardless of how ugly it might be we should use whatever Rails uses.

Would it be storage_profiles_storages or storage_profile_storages?
Rails suggests ordering alphabetically [1] so I think it'd be
storage_profile_storages over storages_storage_profiles, but storage_storage_profiles over storage_profiles_storages
P.S. I'm not renaming host_storages again :smile:
[1] http://guides.rubyonrails.org/association_basics.html#creating-join-tables-for-has_and_belongs_to_many-associations section 3.3.2

storage_profile_storage seems to what generated migration prefers.

@Fryguy Please see if this can be merged.  Thanks

P.S. I'm not renaming host_storages again 😄

😆

Since this has SQL schema changes, I am marking as darga/no


Crash with Advanced Generators

Forge: <IP_ADDRESS>6
bdlib: <IP_ADDRESS>
Advanced Generators: <IP_ADDRESS>
Crash Log: http://pastebin.com/kEb7Qb6G

Update bdlib, it's way out of date and will not work with newer builds of my mods.


User-provided version

Hi!
Is it possible to specify custom version id for deployment?
Thanks!

At the best of my knowledge and in the case I understand you correctly, a deployment results in a new version timestamp, if the deployment is triggered, so this should not be possible. Can you give me some more information, maybe what you want to achieve? Thank you.

@aldipower I used Elastic Beanstalk a lot before Marathon and what I want from Marathon is:

API for creating version of application (without deployment). Very useful for CI: build server is creating version and notifying ops about new available version. Then ops guy can review this version and if everything is fine (instance amount, memory, env vars) - it will press "deploy" (see 2) ).
API for deploying existing version from "catalog" (already implemented as "configuration history")

I do understand that 1) could be implemented by storing manifests somewhere, but it will cause duplication (somewhere + config. history)
This is why "custom version" is important - CI can create this version with id like "foo-service-v1.3.4-ab83a7ef1", and release request will looks like "please release foo-service v1.3.4"

Hey @bsideup !
Thank you for your explanation, now I understand and see the use case for this.
For now this isn't possible and planed, but we will have this in mind.
You could store app definitions in JSON-format of course by yourself and then apply them to Marathon. And maintain a custom catalog. This would be a solution for the meantime.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3048. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3048. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


fix: add script to execute sf

What does this PR do?

Adds a script that will execute sf. This is preferable to specifying the node_modules sf path for the bin since we can't guarantee that node_modules will exist when npm creates the sf executable.
Better handle errors during postupdate hook
Add more information to failed sf installs from postupdate hook

Testing
Local Install

mkdir test-dir && cd test-dir
npm init
npm install<EMAIL_ADDRESS>
Global Install

npm install -g<EMAIL_ADDRESS>
What issues does this PR fix or reference?
@W-9875023@
@W-9875088@
@W-9831748@
https://github.com/forcedotcom/cli/issues/1174
https://github.com/forcedotcom/cli/issues/1175

checking understanding:
everything in node_modules/.bin is a symlink to wherever the module's bin actually is?
and then this'll fix the dependencies bug because sf will get installed at the top-level?
or because .bin symlinks to wherever the module is, even if it's nested somewhere else?

@mshanemc Not quite. The issue is that the entire node_modules directory does not exist when npm is creating the executables (based on the bin property in the package.json). Because of this, the specified file path cannot be anything inside of node_modules
To solve that, I've added a new script to the bin folder which resolves the absolute path to the sf executable and runs it

Fixed on 7.118.1-6fd25cc with no error:
PS C:\Users*> sfdx update stable-rc
sfdx-cli: Updating CLI from 7.117.0-b5a8ac9 to 7.118.1-6fd25cc... done
sfdx-cli: Updating CLI... done
PS C:\Users*>