Bad instructions in front-matter when applying template

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[ ] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[ ] Publish.  External Site Publish
[x] Markdown.  Markdown features, such as syntax support and features

Describe the bug
First bug : when a template is applied in a note, the order of the instructions in the front-matter is no longer the same as in the template.
Second bug: when a template is applied in a note, all comment are gone, as well as line breaks.
For example, my template is:
---
# --- base ---
id: qzmv9fiafd3lmvinuk619s8
title: Software template
desc: ''
updated:<PHONE_NUMBER>170
created:<PHONE_NUMBER>311

# --- template.article.software ---
template: 'article.software'
tags : [software]
name: '-'
developer: '-'
repository: '-'
programmingLanguage: '-'
platform: '-'
website: '-'
---

## Informations

- **Name :** {{ fm.name }}
- **Developer :** {{ fm.developer }}
- **Repo :** <{{ fm.repository }}>
- **Written in :** {{ fm.programmingLanguage }}
- **OS :** {{ fm.platform }}
- **Web site :** <{{ fm.website }}>

## Installation

after applying, the result is:
---
id: lgfub757uo2lr4lz414wx30
title: Test
desc: ''
updated:<PHONE_NUMBER>834
created:<PHONE_NUMBER>834
tags:
  - software
template: article.software
name: '-'
developer: '-'
repository: '-'
programmingLanguage: '-'
platform: '-'
website: '-'
---

## Informations

- **Name :** -
- **Developer :** -
- **Repo :** <->
- **Written in :** -
- **OS :** -
- **Web site :** <->

## Installation

As you can see, the tags property has been reversed with template,  all comment and line breaks are gone, and even the tags has changed.
To Reproduce
Steps to reproduce the behavior:

Create a template
Execute the command : Apply Template
See the result

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information)

OS: [e.g. iOS]
Browser (if with published websites; include version): [e.g. chrome, safari]
Versions (all that apply to bug):

Dendron Extension: 0.118.0 (the last version)
VS Code version: 1.73.1 (the last one)
Dendron CLI:

Additional context
Add any other context about the problem here.
Dendron Log file
Please attach the output of > Dendron:Dev: Open Logs here

Thank you for your answer. No problem for me to label this post as enhancement.


Contact us not working after going to team page

Description
Contact us in the navbar is not linked to the footbar of the homepage.

@Sapna2001 can you work on this?


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1092 ms

Resolved: Bofrost.lu performance has improved in 8153331 after 13 minutes.


chore: lint on ci and lint test files

This PR adds a linting job on CI to catch regressions.
A few lint issues have been fixed in the process (one lint rule has been removed as the restriction no longer applies, see https://github.com/vuejs/core/commit/7efb9dba3084816e54e2989d1812c13bba229ee9).
The test files are now also processed, and rules have been added to avoid letting focused or disabled tests slip through.
This has been discussed after https://github.com/vuejs/core/pull/5715

I introduced lint to CI in 1414f17 before seeing this PR - I grouped it with dts tests to reduce the parallel run since even with both combined they are faster than the main test suite.
The lockfile also has a bunch of conflicts now, could we rebase this?

@yyx990803 The PR is rebased and only kept the relevant changes: add linting for test files and ensure that we don't let focused tests slip through.


[FEATURE] make inline events have an end

Is your feature request related to a problem? Please describe.
I'd like to be able to pull out only part of a page for an %%aat-inline-event.
Describe the solution you'd like
I have various character pages which I'd like to be able to filter in different ways, so I can pull out the event for 'Bob and Alice have a conversation' and the event for 'Bob sends flowers to Chuck' separately.  One would be in timelines that are [bob, alice] and one would be in timelines that are [bob, chuck].  That would let me keep track of what alice knows separately.  So, some kind of ending like a third %% to tell the parser to stop would be fantastic.  In the screenshot below I've assumed that %% could be both the stop for one inline event and the start for another, but any symbol would work as long as it's clear.
Describe alternatives you've considered
There are no other timelines that are not buggy that do what I want.  I could create separate tiny sub-pages for all of these events, but I don't think I should fragment my data in that way.  An inline event could also have the description embedded in it, between %% tags, as part of the basic syntax.
Additional context

Hey @DianaProbst, thank you for bringing this up
I'm quite sure you are asking for something that has already been adressed
https://github.com/April-Gras/obsidian-auto-timelines/issues/97#issuecomment-1874716661

Thank you, you're right, that is what I was after.  That makes this into a brilliant thing I'm going to go 'squee' over.
Can I suggest that under the Inline Events heading in the readme, that end tag gets added?


fix: snyk

Summary
Updates eslint & jest to latest.
...
Changelog
- Update eslint
- Update jest
- Add no-unsafe-argument rule
- Fix no-unsafe-optional-chaining errors

Testing
Platforms

Please select any platforms where your changes have been tested.

Desktop

[ ] MacOS
[x] Linux
[ ] Windows

Mobile

[ ] iOS
[ ] Android

Instructions

Please describe the specific instructions, configurations, and/or test cases necessary to test and verify that your changes work as intended.

...
Checklist

Please tick the following boxes that are relevant to your changes.

[ ] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[ ] I have verified that new and existing unit tests pass locally with my changes
[ ] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

Getting this error (on Linux) when updating the dependencies:
yarn install v1.22.19
[1/4] Resolving packages...
warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>[2/4] Fetching packages...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "^14.15.0 || ^16.10.0 || >=18.0.0". Got "16.4.2"
error Found incompatible module.

We also need to fix the workflows to pass before we can merge this in.

I know that this PR is for the purpose of fixing Snyk's complaints, but it's usually better to do each of these in a separate PR b/c it's easier to review, easier to debug if something goes wrong, etc.
Thanks for making these fixes :pray:


V0.12.1.x govobj submission

Doing some general fixing up of the governance submission process to restore the syncing capability of the client.

Looks good
utACK 6e919af
PS. needs squashing
@schinzelh


Schema.js cleanup

A collection of semi-random changes that are progress towards database adapters.

Current coverage is 92.96%

Merging #1540 into master will increase coverage by +0.07% as of 2f0dc53

@@            master   #1540   diff @@
======================================
  Files           87      87
  Stmts         5506    5501     -5
  Branches      1022    1021     -1
  Methods          0       0
======================================
- Hit           5115    5114     -1
  Partial          9       9
+ Missed         382     378     -4

Review entire Coverage Diff as of 2f0dc53

Powered by Codecov. Updated on successful CI builds.

@drew-gross do you think it's time to move it to src/Controllers/SchemaController.js

Yah I think once these changes are merged I will do that.

LGTM, besides the baby comment, for later


Update the maintenance scripts

I said I'd look into the scripts that retrieve SQF commands. My initial plan was to scrape the intercept pointers deceleration file the same way I did for the sqf repository (this is still an option as far as that is concerned). However I've realised that this package also reads the description and other information of each command so I'm in the process of checking why the scripts aren't working.
The first thing I notice is that armadoc.py only processes (alphabetically) up to curatorEditingArea.
Here is the information initially queried by the armadoc.py script. It seems that the code to continue requesting more information (I assume that's what this is meant to do) isn't working:
https://github.com/acemod/language-arma-atom/blob/c4a6acfb510b7d16ee24a1604f1ba08ed160b3e6/rsc/armadoc.py#L18-L22

It's very strange how it suddenly doesn't work though, as I know this script originally ran with no issues. It could be some sort of flood detection on the wiki perhaps? Honestly not sure.

Does this need to be done in Python? I can throw something together in PHP to copy all commands / functions on the Wiki if needed,

It can honestly be in whatever language as long as it ends up doing the same thing.

The existing Python scripts have been made to automate the full process of retrieving the commands and then generating the required JSON contents to be placed into the relevant files for Atom to read. Any implementation of this same system would be fantastic, and would be even better if it supported doing the same for BIS functions as well as the current scripts don't do that. I personally don't have any experience in Python or PHP, so its much appreciated that you guys @SilentSpike @HazJ are helping and would be awesome if either the current scripts could be fixed or new ones made to do the same thing.

Hello. I have bene extremely busy the last week or so. Real issues and working on web app / other stuff. I assume this is still need of some attention? I'll get back on it this weekend. @ShadowRanger Regarding your point about JSON and Atom? What is the specific format that you need?


TLS Handshake failed: Cannot read handshake packet: EOF (Only TLS1.2 + PKCS disabled)

I am running my golang program to connect to SQL server 2016 SP1 on Windows Server 2012. I enabled only TLS 1.2 (TLS1.0 and 1.1 is disabled). Also, I disabled PKCS key exchange.
On this setup when my program is making a call to SQL server. I get below mentioned error.
**

TLS Handshake failed: Cannot read handshake packet: EOF.

**
This program works on Windows 2016 on TLS 1.2 with PKCS disabled. I am using IIS Crypto tool to disable protocols and key exchanges. Here is screenshot what I have in my configuration.

Also, DEBUG level to 63 did not yield any more information.
Code Sample: Please change username, password and host before running the code
package main

import (
	"database/sql"
	"fmt"
	"log"
	"net/url"
	"strconv"

	_ "github.com/denisenkom/go-mssqldb"
)

func main() {
	uid := "sa"
	password := "your password"
	hostName := `<IP_ADDRESS>`
	finalConnectionString := getConnectionString(uid, password, hostName)
	fmt.Println(finalConnectionString)
	conn, err := sql.Open("sqlserver", finalConnectionString)
	if err != nil {
		log.Fatal("Open connection failed:", err.Error())
	}
	defer conn.Close()
	result, err := conn.Exec(`SELECT name, database_id, create_date
	FROM sys.databases`)

	fmt.Println(result, err)
}

func getConnectionString(uid, password, hostName string) string {
	query := getCommonQueryString("master")
	u := &url.URL{
		Scheme:   "sqlserver",
		User:     url.UserPassword(uid, password),
		Host:     hostName,
		RawQuery: query.Encode(),
	}
	return u.String()
}

func getCommonQueryString(dbName string) url.Values {
	query := url.Values{}
	query.Add("Database", dbName)
	query.Add("connection timeout", strconv.Itoa(120))
	query.Add("Encrypt", "true")
	query.Add("TrustServerCertificate", "true")
	query.Add("log", "63")
	query.Add("app name", "defect")
	return query
}

Hi @shaileshpandey, please follow this wiki https://github.com/denisenkom/go-mssqldb/wiki#reporting-bugs to provide additional information about your issue

Hi @denisenkom ,
Added complete running code for reference. Log Level 63 generates nothing.

Hi @denisenkom ,
Is there any update on this issue?

@shaileshpandey Can you confirm that other SQL Server drivers that are running over TCP/IP also work in this configuration? To do that, I think you would need to disable named pipes and VIA on SQL SERVER itself. Or at least try to configure the .net client to use TCP/IP.

@kardianos I don't know why should we configure the .net client.  I have the same problam.

Any update on this? I am also facing this issue with SQL2012 connection.
TLS Handshake failed: cannot read handshake packet: EOF

@shaileshpandey encrypt=DISABLE worked for me. You can try this in your connection string.


error while using import

bebel version6.26.0 installed as per react docs :install babel-cli@6 babel-preset-react-app@3
chrome rendering html error:

{/*test.js*/}
import { useState } from 'react';

const root10 = ReactDOM.createRoot(document.getElementById('Xy'));
const element10 = <Pile />;
root10.render(element10);

 function Pile() {
  return (

  );
}

<!DOCTYPE html>

    <meta charset="UTF-8" />
    <title>Test Page-React</title>

<link rel="icon" type="./x-icon" href="./favicon.ico">

https://beta.reactjs.org/learn/describing-the-ui

https://transform.tools/html-to-jsx

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Modules

Like I mentioned in another issue you filed, you can't expect to use syntax like import and export in the browser directly like you're trying to do right now. We use it throughout the docs because in a practical way, most apps using React will put components in different files. But if you want to keep using this syntax, you need to use a bundler.
I agree we need to make this clearer in the docs, but this isn't specific to React — you'll run into this problem with every other library if you try to use import and export in your code.

What you can do instead is write React.useState.

(by the way i havent used bundler before) ..so please send me some links
regarding the bundler aspect you mentioned so that I can try..Thanks
On Fri 3 Feb, 2023 11:26 pm dan, @.***> wrote:

Like I mentioned in another issue you filed, you can't expect to use
syntax like import and export in the browser directly like you're trying to
do right now. We use it throughout the docs because in a practical way,
most apps using React will put components in different files. But if you
want to keep using this syntax, you need to use a bundler.
I agree we need to make this clearer in the docs, but this isn't specific
to React — you'll run into this problem with every other library if you try
to use import and export in your code.
—
Reply to this email directly, view it on GitHub
https://github.com/reactjs/reactjs.org/issues/5542#issuecomment-1416210698,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP3CCXROZPWE5JDV5GXZ6DDWVVBE7ANCNFSM6AAAAAAUQN37Q4
.
You are receiving this because you authored the thread.Message ID:
@.***>

React.useState worked ...thanks a lot ..example below
test.js

const root2 = ReactDOM.createRoot(document.getElementById('buton'));
const element2 = <MyApp />;
root2.render(element2);

function MyButton() {
	  const [count, setCount] = React.useState(2);

   function handleClick() {
    setCount(count + 3);
  }

  return (
    <button onClick={handleClick}>
      Clicked {count} times
    </button>
  );
}

export default function MyApp() {
  return (

Welcome to my app777

      <MyButton />

  );
}

index.html

Test Page-React

```


Add switch for multiple companies

Let's say a user is assigned as a manager to more than one company. We should have a way to switch the company while logged into the system.
Currently, we're considering that the user can have just one role admin || manager || customer, this is actually wrong though. Our backend is flexible enough to handle that, we should do the same in the frontend.

We have these functions to decide which role the user can get, let's exclude them and update our reducer. Also, we must make sure that the screens are being rendered properly according to the user's role.
To switch the company, we need a property in the redux state. something like current_company.
Refactor
Also, all screens should refer to only one company. The companies list screen (when logged as manager) for example will no longer exist.

I can try to solve this one

After the removal of those functions mentioned above, the current role of the user won't be set by the reducer,  correct?
In this case, I think the switcher could only refer to the list of companies in which the user has a role, as we switch among those options, we can automatically switch the role of the user in order to display the correct layout.
WDYT @belgamo ?


Add Message Format support for encryption

To support encryption at rest,

Introduce a new MessageFormat Blob Encryption Key Record.
Introduce a new version of the MessageFormat Header to support
the new record.
Introduce a new versioned MessageFormat MessageMetadata associated
with each message that is sent out via MessageFormatSend. The first
version of MessageMetadata will only contain the encryption key.
Broaden MessageInfoListSerDe to also include MessageMetadata as part
of it (and so rename it to MessageInfoListAndMetadataSerDe). Every
message in PartitionResponseInfo will therefore have a MessageInfo
and an optional MessageMetadata.
Introduce encryption flag in BlobProperties and add a new version of
BlobPropertiesSerDe to support the new field.
Add/update tests for coverage of the above changes.

This is a gist with additional information about this change.

Codecov Report

Merging #782 into master will decrease coverage by 0.11%.
The diff coverage is 66.35%.

@@             Coverage Diff              @@
##             master     #782      +/-   ##
============================================
- Coverage      66.1%   65.98%   -0.12%
- Complexity     3939     3970      +31
============================================
  Files           343      344       +1
  Lines         25926    26359     +433
  Branches       3241     3316      +75
============================================
+ Hits          17138    17394     +256
- Misses         7885     8032     +147
- Partials        903      933      +30

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com.github.ambry.protocol/GetResponse.java
82.85% <ø> (ø)
21 <0> (ø)
:arrow_down:

...ain/java/com.github.ambry.router/PutOperation.java
90.18% <100%> (ø)
70 <0> (ø)
:arrow_down:

...n/java/com.github.ambry.messageformat/BlobAll.java
100% <100%> (+12.5%)
5 <2> (+2)
:arrow_up:

...ain/java/com.github.ambry.protocol/PutRequest.java
82.66% <100%> (ø)
21 <0> (ø)
:arrow_down:

...ithub.ambry.messageformat/BlobPropertiesSerDe.java
93.02% <100%> (+1.59%)
13 <1> (+6)
:arrow_up:

...src/main/java/com.github.ambry/rest/RestUtils.java
96.21% <100%> (ø)
54 <0> (ø)
:arrow_down:

...com.github.ambry/messageformat/BlobProperties.java
95.91% <100%> (+0.26%)
17 <2> (+1)
:arrow_up:

...in/java/com.github.ambry.server/AmbryRequests.java
69.4% <100%> (+0.52%)
59 <0> (+3)
:arrow_up:

...bry.messageformat/PutMessageFormatInputStream.java
100% <100%> (ø)
9 <8> (+7)
:arrow_up:

....github.ambry.messageformat/BlobStoreRecovery.java
57.14% <3.03%> (-22.86%)
5 <0> (ø)

... and 32 more

Continue to review full report at Codecov.

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

Recording things discussed offline

Should we also increase the ReplicaMetadataResponse version?
Can we stop serializing isPrivate (and remove if from the "default" constructor) for the new blobs. The old ones (V1) still need the flag.

@vgkholla @nsivabalan Addressed comments and updated. Please take a look.

@vgkholla @nsivabalan Addressed comments.

@nsivabalan addressed your comments.

@vgkholla Addressed comments.


Logo missing at Edenred

ENVIROMENT: production
COMPANY: Edenred
URL: Edenred peviitor.ro
Company's Career website: Edenred official website
Browser: Google Chrome
Device: laptop
OS: MacOS
Precondition:
The site must be open in browser.
Steps:
Click on Firme from footer.
Enter Edenred in the search bar.
Click the button under the company's logo.
Expected result:
You can see the company's logo.
Actual result:
The company's logo is missing.

@AdelinaGuliman  logo este pe site. te rog retesteaza


9FUSY32 with firmware <IP_ADDRESS>A_201912271707, won't flash

Just got new cameras, they won't flash likley due to the new 8.3 firmware. Anyone working on fixing this. Any way I can help?
When I attempt to flash the orange light comes on and stays on. When I pull out the card, the cameras boot normal and no changes have been made to the card.

Probably it's a different model, not a different fw.
Try to make a backup using the procedure described in the wiki.

I just tried the backup (SD card method), same thing. Orange light stays on forever, no changes to the card.
Model is a 9FUSY32LYS (QR card on back) Model YYS.2518 (bottom of stand) YI Home Camera 3 Date 2020.04 (Back of Box)

I think you have to use tee serial cable method.

9FUSY32LYS.zip
Well that wasn't easy for a noob like me. Here is the image dump. I hope you can create/help me with this.

Do you want to try this hack?
y25ga_0.1.9.tar.gz

I certainlyappricieate trying to help with this. I tried this new on one two diffrent cards, same thing. continous oragnge light. I tried it a third time with a serial cable attached. Here is the output.
hachtry2.log

############## Starting Hack ##############

The hack starts but something is wrong in the syntax.
Check if the file config.sh is in linux format with the correct new line char.

Thats above my knowledge base. Did you write it?

I would like to understand if the file has changed during the copy.
Open the config.sh file in the microSD and check if the lines end with LF char or CR+LF chars.
For example with Notepad++ (View --> Show Symbol).
https://en.wikipedia.org/wiki/Newline
If the file changed the format and now is in Windows format, it can't run.

I think you are on to something. I'm not at home with the cameras to test, but looking at the files: I unzipped the files directly to the card using winzip. When I open the files with Notepad++ and show symbols. I see every line ends with CRLF. I then uninstalled Winzip, downloaded 7Zip, and unzipped again and reopened in Notepad++, the lines now end in just LF.
When I get home I will test again and report back.

Awsome, I'm so Happy, One donation coming your way roleroleo.
ANyone who comes accross here this same problem: Its the file on comment 6 of this thread that worked for me. I went back and tried the official release and it didn't work for me. Not sure why my zip program was converting the file to a diffrent format but it was, and 7zip did once as well. Each time now when I unzip to the SD card I check it in Notepad++ to be sure it only has LF.

Please check it all features work correctly and send me a feedback.
I will add your model to the supported cams.
Thank for your donation!

Everything works. When I goto the snapshot page, the camera takes a snap then seems to go through a reboot or reconnection,  but that is the only thing I have noticed.
I cant get home assistant to see ONVIF, but I think that is a network issue on my side. I have added the cameras to motioneye inside home assistant.

The reboot is a known issue.
Try to enable swap file.
To check onvif, use onvif device manager and try to connect on the same subnet.


add “ELN_Author_ID” as a field to user table

Add a new field to store the ELN Author ID.  When creating notebooks from StudyTracker in the ELN, we can use the logged in study tracker user to get the correct ID with which to create the notebook

Going to add a Map<String, String> field, attributes to the User model to support this. Will work just like program or assay type attributes.


Allow using local mapnik installation for building

Changes to allow building against a local installation of mapnik by using make release_base or make debug_base; it requires  mapnik-config to be in the PATH.
The default build is still using mason (downloads mason and its dependencies and sets the PATH to use it).  I'm not completely sure if the changes in postinstall are correct, though. I've tested this only on LInux, with these changes on top of v3.6.2 and mapnik 3.0.15 (with a cherry-pick for icu-59)
It should cover https://github.com/mapnik/node-mapnik/issues/784

Great work on this @Algunenano - this looks great. Could you also add docs to the main README.md?

Done. I've also modified the pre-build check (mapnik-config in PATH) to fail in the Makefile instead of failing randomly in the build process.
Please let me know if you need anything else.

Realized one more problem. The postinstall.sh currently pulls from the values of --proj-lib, --gdal-data, and --icu-data. However these are currently broken for mapnik core. Fix incoming in https://github.com/mapnik/mapnik/pull/3759. Until then the right approach here is to not set these values at all (they the defaults will be used for those libraries rather than overriding the environment variables at https://github.com/mapnik/node-mapnik/blob/dd3dc93b035742cb20061a52599a71a37b68e614/lib/mapnik.js#L17

Touched up your work and landed in #803. Thanks so much for the help on this! Been planning on fixing this myself for ages (#784) but just did not have the time. Thanks again.


How to call async component's internal function

I'm having a little problem.
Say I have an async component that is encapsulated with react-async-component, like this:
const AsyncComponent = asyncComponent({
  resolve: () => new Promise(resolve =>
    require.ensure([], require => {
      resolve(require('draft-js'))
    }
  )
})

then call its internal functions by refs
<AsyncComponent ref={c => this.editor = c} />

this.editor.focus();

but will get Uncaught TypeError: _this.editor.focus is not a function error, and I use console.log(this.editor) see it's a async component and has not focus function so that I can't access it.
I spent a lot of time but still have no idea how to fix it,
does anyone can help ? thanks 😄

Ok, finally I found the solution solved it,
just pass a prop to AsyncComponent and return it.
in your main component
<AsyncComponent
  onRef={c => this.editor = c}
/>

in Async Component
componentDidMount() {
  this.props.onRef(this);
}

then you can call its internal functions.
this.editor.clear();
this.editor.focus();


Documentation on settings

hello!
nice to see that you've opened a git repo for this.
I am trying to figure out which settings you've adopted within you zip.
Perhabs you could elaborate in your readme for this?
This would also enable others to do suggestions for improvements / updates.
thanks!

No need to document the settings IMO, simply import the package and then check the new settings via. the NetBeans preferences. - Not sure what the benefit to me documenting these would be given that they are plain text in the source code anyway :)


VPin-Studio.sh still not executable

#16 fixed the newlines but not the executable flag
Probably there is no way to keep this flag when cloning in windows?

looking at https://github.com/syd711/vpin-studio/blob/553379ca2271f879353140422974fe1a4192da9f/vpin-studio-ui/pom.xml#L491-L502
I now understand why unzip does not see this file as a zip, it's actually a 7-zip file with zip instead of 7z extension.
I wonder if there's a way to let 7z know it should apply the executable flag.

So 7z is the reason the flag gets lost? It wouldn't get lost when "regular" zip/unzip is used?

So 7z is the reason the flag gets lost? It wouldn't get lost when "regular" zip/unzip is used?

Don't think so, I guess windows has no way to keep this flag so it's already lost at clone. Maybe some more standard maven plugin can do the zipping and has an option to set executable flag?
On a side note i would make sure the file is a true zip.

Finally, I switched to "zip" so that this extraction error message is gone :)
Regarding the executable flag: For me its just a corner case I don't want to focus on too much. So I'll close this ticket for now.


Icon Font not captured under Windows7 & IE11

Bug reports:
Font Icon does not display under Windows7 & IE11.
it works well under Windows7 & IE9/10 and Windows10 IE11/Edge.
here is  an demo to reproduce the issuse.
I tried anthor test, found that canvas element render font icon well under Windows7 & IE11.
so, I guess it should be a html2canvas issuse or sth related or IE11 issuse.
Specifications:

html2canvas version tested with: latest release, 1.0-alpha
Browser & version: IE11
Operating system: Windows7

Please help to solve it. Also thanks your great work.

spend a few hours, the problem was located. so I note it here.
NOTE: this issuse only reproduce under windows7 + IE11.
CASE 1:
if your css code is:
font-family: "FontAwesome";
...

then you should modify the source code at src/renderer/CanvasRender.js line:216,
font.fontFamily.replace(g['"]/g, '')

CASE 2:
if your css code is:
font-family: FontAwesome;
...

then you should modify the source code at src/renderer/CanvasRender.js line:216,
JSON.stringify(font.fontFamily)

We are running into the same issue where the font-family in IE11 are not properly rendered.  This fix from @yessky seems to resolve the issue for us.


A heap-buffer-overflow vulnerability was discovered

While testing fuzzipp, I found a stack overflow vulnerability located in the ippWriteIO function in the /src/libcups/cups/ipp.c file.
In line 5263 of the function, an irregular use of the memcpy function causes a heap overflow vulnerability.

Ok, I've attached the corpus。
crash-d8616e43ff1db1b8e21f333287caa47d88d6a8fa.zip

[master a7a28e643] Error out if an attribute conversion is unsuccessful.


Fix typo in tokenize.word_tokenize() docstring

Unless I'm misunderstanding something, nltk.tokenize.word_tokenize() splits text into words, not into sentences (which nltk.tokenize.sent_tokenize() does).

Um, yes, indeed! Thanks @gmarmstrong


Generated TypeScript Models Lack File Extensions in Import Statements When Using Node 16 Module Resolution

Hello,
first, thanks for this library and the efforts to maintain it!
I've encountered an issue using it in a TypeScript project with Node 16 and moduleResolution set to node16 (or nodenext) in the tsconfig.json.
When the TypeScript models are generated, the import statements in the generated files lack the .js extension for relative file paths. This is problematic because Node 16 requires explicit file extensions in ES module imports: https://nodejs.org/api/esm.html#mandatory-file-extensions
Example Error:
models/contentful-schemes/TypeDiscoverLandingPage.ts:3:48 - error TS2835: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Did you mean './TypeBrandCarousel.js'?

Incorrect Import Statement:
export { isTypeArticleCarousel } from "./TypeArticleCarousel";

Expected Import Statement:

export { isTypeArticleCarousel } from "./TypeArticleCarousel.js";

tsconfig.json Configuration:
{
    "module": "Node16",
    "moduleResolution": "node16"
}

Questions and Offer to Contribute:

Might this be a configurational problem on my end? Unfortunately, I am not able to change module or moduleResolution in my project, but maybe I missed a flag in the library to configure the module compatibility?
Is this a known issue? If so, is there a planned fix?
I'd like to contribute a fix for this issue. For that, could you guide me to the part of the codebase where the import statements are generated?

Thank you!

Still waiting for an answer here

had the same issue, solved it using the moduleSpecifier, e.g.
  const moduleSpecifier = outputFile.getRelativePathAsModuleSpecifierTo(inputFile.getFilePath()) + '.js';

  outputFile.addImportDeclaration({
    moduleSpecifier,
    ...
  });


no workaround for initial Google signin

The issue tracker is for reporting product deficiencies. How do I questions should be posted to the discussion forum at https://groups.google.com/group/got-your-back. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GYB release from https://github.com/jay0lee/got-your-back/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/got-your-back/wiki

Full steps to reproduce the issue:

Follow the steps for an initial setup and sign in to your Google account using the Wiki and https://github.com/jay0lee/got-your-back/issues/134

Expected outcome (what are you trying to do?):  Initial setup by logging into Google and allowing it to link to GYB.  (I've done this successfully in the past with another account.)
Actual outcome (what errors or bad behavior do you see instead?):  "Sign in with Google temporarily disabled for this app.  This app has not been verified yet by Google in order to use Google Sign In."
 .  I also see other users having trouble as reported in the Google forum.

I have also heard reports about this problem. Specifically when trying to restore emails. Does gyb ask for any additional scopes that would trigger this?

I am here because I am experiencing this same issue.
OSX version 1.11 and 1.1
This error occurs with any attempt to authenticate, with an action or not.
./gyb --email<EMAIL_ADDRESS>

Same issue. It looks like the app will need to be verified as outlined here: https://support.google.com/cloud/answer/7454865?hl=en&ref_topic=3473162
This may have to be done by jay0lee.

Same here =(

This tool is awesome, but I can’t use it because of the issue with app verification :(.

Same

same issue.

Same

Interestingly I get that warning on my regular gmail account but not gsuite legacy account.

mute the thread
letechnology<EMAIL_ADDRESS>于 2019年3月20日周三 上午11:08写道：

Interestingly I get that warning on my regular gmail account but not
gsuite legacy account.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jay0lee/got-your-back/issues/194#issuecomment-474603946,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL6jiNNsb_dEoiw2ZsXf2gioKJJXizIzks5vYV_ygaJpZM4ba1jK
.

Got this email today—related to this issue?

Hello,
Although you don’t need to take any action, we wanted to let you know that the following third-party apps will no longer be able to access some data in your Google Account, including your Gmail content. This change will go into effect starting March 31, 2019.
Got Your Back: Gmail Backup
We are making this change as part of ongoing efforts to make sure your data is protected and private. These apps haven’t yet complied with our updated data privacy requirements (https://cloud.google.com/blog/products/g-suite/elevating-user-trust-in-our-api-ecosystems) announced on October 8, 2018.
You can always view, manage and remove apps you’ve given access to your account by visiting your Google Account.
Thanks,
The Google Accounts team

GYB 1.20 is now available. Basic instructions for installing and fixing the issue are in the release notes:
https://github.com/jay0lee/got-your-back/releases
can a few people try upgrading and fixing and give me feedback on the process?

Awesome thanks Jay!

Thanks, Jay. Went through the entire authorization process without issue, but here's what happened when I tried running command: gyb --email<EMAIL_ADDRESS>--search "label:Test". I tried to revoke my previous authorization, but same error
Select the actions you wish GYB to be able to perform for<EMAIL_ADDRESS>[ ]  0)  Gmail Backup And Restore - read/write mailbox access
[ ]  1)  Gmail Backup Only - read-only mailbox access
[ ]  2)  Gmail Restore Only - write-only mailbox access and label management
[*]  3)  Gmail Full Access - read/write mailbox access and message purge
[ ]  4)  No Gmail Access
[]  5)  Groups Restore - write to G Suite Groups Archive
[]  6)  Storage Quota - Drive app config scope used for --action quota
  7)  Continue

7
Traceback (most recent call last):
File "gyb.py", line 2026, in
File "gyb.py", line 1365, in main
File "gyb.py", line 321, in requestOAuthAccess
File "oauth2client_helpers.py", line 133, in positional_wrapper
File "oauth2client\client.py", line 2135, in flow_from_clientsecrets
File "oauth2client\clientsecrets.py", line 165, in loadfile
File "oauth2client\clientsecrets.py", line 122, in loadfile
File "json_init.py", line 296, in load
File "json_init_.py", line 348, in loads
File "json\decoder.py", line 337, in decode
File "json\decoder.py", line 353, in raw_decode
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 11 column 1 (char 47
0)
[8612] Failed to execute script gyb
C:\gyb>

Looks like a typo that leaves a bad character in the client_secrets.json
file. Edit your client_secrets.json file and you'll see an out of place `
character at the start of one line. Delete that character and see if it
solves.
I'll also see about creating an updated build.
Jay
On Fri, Mar 22, 2019 at 2:16 PM sachin427<EMAIL_ADDRESS>wrote:

Thanks, Jay. Went through the entire authorization process without issue,
but here's what happened when I tried running command: gyb --email
<EMAIL_ADDRESS>--search "label:Test". I tried to revoke my
previous authorization, but same error
Select the actions you wish GYB to be able to perform for
<EMAIL_ADDRESS>[ ] 0) Gmail Backup And Restore - read/write mailbox access
[ ] 1) Gmail Backup Only - read-only mailbox access
[ ] 2) Gmail Restore Only - write-only mailbox access and label management
[*] 3) Gmail Full Access - read/write mailbox access and message purge
[ ] 4) No Gmail Access
[
] 5) Groups Restore - write to G Suite Groups Archive [] 6) Storage
Quota - Drive app config scope used for --action quota

Continue

7
Traceback (most recent call last):
File "gyb.py", line 2026, in
File "gyb.py", line 1365, in main
File "gyb.py", line 321, in requestOAuthAccess
File "oauth2client_helpers.py", line 133, in positional_wrapper
File "oauth2client\client.py", line 2135, in flow_from_clientsecrets
File "oauth2client\clientsecrets.py", line 165, in loadfile
File "oauth2client\clientsecrets.py", line 122, in
loadfile File "json_init.py", line 296, in load
File "json_init_.py", line 348, in loads
File "json\decoder.py", line 337, in decode
File "json\decoder.py", line 353, in raw_decode
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 11 column 1
(char 47
0)
[8612] Failed to execute script gyb
C:\gyb>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jay0lee/got-your-back/issues/194#issuecomment-475727272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEaMsEJqvTYIF93T18av_7dtbDIDzLyZks5vZR4WgaJpZM4ba1jK
.

That solved that issue, but another one came up when trying to restore:
C:\gyb>gyb --email<EMAIL_ADDRESS>--action restore
Using backup folder<EMAIL_ADDRESS>Traceback (most recent call last):
File "gyb.py", line 2026, in
File "gyb.py", line 1614, in main
sqlite3.OperationalError: database resume is locked
[4324] Failed to execute script gyb

Sorry, that was a mistake on my part...everything works! Thank you, Sir

Works for me now, thanks!

GYB 1.20 is now available

Thanks, Jay!  I am curious, though:  https://cloud.google.com/blog/products/g-suite/elevating-user-trust-in-our-api-ecosystems says that a security assessment fee "is paid by the developer and may range from $15,000 to $75,000 (or more) depending on the size and complexity of the application."  Do you have a wealthy patron who is covering this, or are you paying it yourself out of the goodness of your heart, or are you setting up a paid version of GYB, or ...?  If you are paying this yourself for our benefit, thank you even more!  Wow!

Each GYB user will now create their own project. You don't need to be verified to authenticate against your own project. Problem solved (or worked around I guess).


Whether the volume can be mounted with non-root user

Describe the solution you'd like
Whether the volume can be mount with non-root user
Anything else you would like to add:
N/A
Environment:
AKS

Secrets Store CSI Driver version: (use the image tag):  mcr.microsoft.com/k8s/csi/secrets-store/driver:v0.0.13
Azure Key Vault provider version: (use the image tag): mcr.microsoft.com/k8s/csi/secrets-store/provider-azure:0.0.8
Kubernetes version: (use kubectl version): v1.16.13
Cluster type: (e.g. AKS, aks-engine, etc): AKS

@Shuanglu Could you elaborate more on this issue? What is the current behavior you're seeing and what is the use case?

Hi Team,
Use case is like this. Application container is running with non-root user and secrets are getting mounted with root privileges because of which container application is not able to access those secrets.

PR to make file permission configurable is merged: https://github.com/Azure/secrets-store-csi-driver-provider-azure/pull/751. This will be available in v1.1.0 release planned for next week.


Use tox for testing everywhere

This avoids going through all the dancing of setting up appropriate environments for each supported version locally
(That we run flake8 and then ignore everything seems… sad. We should either run it and care about it, or drop it, IMO.)

Also: instead of just asking people to run the tests before submitting a PR, running Travis on PRs would be nice!

Hmm, I've never used tox before. What does it bring us?

This avoids going through all the dancing of setting up appropriate environments for each supported version locally

What dancing are you referring to?

 We should either run it and care about it, or drop it, IMO.)

Sigh, yeah. @jfbastien set it up a while back, but I ended up reverting a number of the things anyway (especially line lengths, ugh) and haven't spent the time to research it to actually make it work for me. It needs fixing.

running Travis on PRs would be nice!

Indeed, I need to figure out how to turn that on.

Yeah, hm, https://github.com/tabatkins/bikeshed/pull/1726 shows a Travis failure on a recent PR, so I guess they are building automatically?

I forgot that I'd done that, but GitHub hasn't: https://github.com/tabatkins/bikeshed/commit/fb7ffbc9e53e56ecf7183d26eec01c6f452bddac#diff-354f30a63fb0907d4ad57269548329e3
I don't have a particular opinion. I've found flake8 useful because I don't Python often enough to have these things in my muscle memory. Any other tool that prevents silly mistakes is useful IMO, if it's not noisy (as it sounds flake8 was?). Maybe the noise is configurable?

Yeah, hm, #1726 shows a Travis failure on a recent PR, so I guess they are building automatically?

Only because it was pushed as a branch on this repo. Branches run on Travis, PRs don't.

Only because it was pushed as a branch on this repo. Branches run on Travis, PRs don't.

Ah, I see. And I finally spotted the settings option that turns on PR building, and have turned it on.

@gsnedders the flake8 issues aren't being ignored anymore, and the CI is flipped over to GitHub Actions. Does the Tox part still make sense?

@gsnedders the flake8 issues aren't being ignored anymore, and the CI is flipped over to GitHub Actions. Does the Tox part still make sense?

Yeah, looks like the new CI you've set up makes this issue moot.

Yeah, looks like the new CI you've set up makes this issue moot.

It would be nice to more fully document how to run the tests if you're not going to use tox to deal with the orchestration of the install (which when I created this at least it had to be a develop install) to run the tests.

It would be nice to more fully document how to run the tests if you're not going to use tox to deal with the orchestration of the install (which when I created this at least it had to be a develop install) to run the tests.

What dancing are you referring to?

To match what's run in GitHub Actions:
virtualenv -p python3.7 ../py37-env
../py37-env/bin/pip install -e bikeshed
../py37-env/bin/bikeshed test
virtualenv -p python3.8 ../py38-env
../py38-env/bin/pip install -e bikeshed
../py38-env/bin/bikeshed test
virtualenv -p python3.9 ../py39-env
../py39-env/bin/pip install -e bikeshed
../py39-env/bin/bikeshed test

v.
tox

What dancing are you referring to?

To match what's run in GitHub Actions:
virtualenv -p python3.7 ../py37-env
../py37-env/bin/pip install -e bikeshed
../py37-env/bin/bikeshed test
virtualenv -p python3.8 ../py38-env
../py38-env/bin/pip install -e bikeshed
../py38-env/bin/bikeshed test
virtualenv -p python3.9 ../py39-env
../py39-env/bin/pip install -e bikeshed
../py39-env/bin/bikeshed test

v.
tox

@tabatkins ping ^^


React UI: ReID algorithm

Motivation and context
The PR adds a feature from the old UI: Re-Identification algorithm

How has this been tested?

Checklist

[x] I have raised an issue to propose this change (required)
[x] My issue has received approval from the maintainers
[x] I've read the CONTRIBUTION guide
[x] I have added description of my changes into CHANGELOG file
- [ ] I have updated the documentation accordingly
- [ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

Related #1341

@ActiveChooN
Mainly the feature is done (it works in base scenarios). I am going to test it deeper, that is the reason why it has WIP tag. You can already look at the PR also.

what's the underlying algorithm? do you have a link to a paper?

does it associate labelled data or does it label new data based on an already labelled track?

It is one of person-re-identification models from https://github.com/opencv/open_model_zoo/


Getting Epic Games Store and Battle.net working in Lutris

To get the Epic Games Store Working I had to follow the Nvidia driver instructions from Lutris (link)
And run the command:
sudo pacman -S --needed nvidia-dkms nvidia-utils lib32-nvidia-utils nvidia-settings vulkan-icd-loader lib32-vulkan-icd-loader
Run to get the Battle.net connection working I had to run:
sudo sysctl -w net.ipv4.ip_default_ttl=80
Some games will require winetricks to have .Net or Visual C++ runtime installed.
Use winetricks to install dotnet472 vcrun2015 (command line works)

Added to Wiki Page


Environment variables for a specific command

Should it be possible to set environment variables that apply for a specific command? The current way to handle this would be to set the environment variables inline with the command but this can be cumbersome and reduce readability.
Sample of current devfile snippet to show example of usage/format:
- exec:
     commandLine: "MAVEN_CONFIG=~/mvn/config JAVA_OPTS="-XX:MaxRAMPercentage=50.0 -XX:+UseParallelGC -XX:MinHeapFreeRatio=10
               -XX:MaxHeapFreeRatio=20 -XX:GCTimeRatio=4 -XX:AdaptiveSizePolicyWeight=90
               -Dsun.zip.disableMemoryMapping=true -Xms20m -Djava.security.egd=file:/dev/./urandom
               -Duser.home=/home/user" ./buildSchema.sh"
     component: build-tools
     alias: buildSchema

Example with proposed changes:
commands:
 - exec:
     commandLine: "./buildSchema.sh"
     env:
     - name: MAVEN_CONFIG
       value: "~/mvn/config"
     - name: JAVA_OPTS
       value: "-XX:MaxRAMPercentage=50.0 -XX:+UseParallelGC -XX:MinHeapFreeRatio=10
               -XX:MaxHeapFreeRatio=20 -XX:GCTimeRatio=4 -XX:AdaptiveSizePolicyWeight=90
               -Dsun.zip.disableMemoryMapping=true -Xms20m -Djava.security.egd=file:/dev/./urandom
               -Duser.home=/home/user"
     component: build-tools
     alias: buildSchema

cc @johnmcollier @elsony

That would be very easy to do, model-wise, and quite consistent with the way environment is mentioned in other parts of the devfile (in container components for example).

Reviewed and approved

I am wondering what is the benefit of doing this way as opposed to having the ENV defined for the component container?
Also what happens if multiple commands are referencing the same component and require the same ENV? Will I need to mention it twice in the respective command? That seems redundant IMO..

@maysunfaisal This allows you to easily set variables for just one command, without needing to change the default value for the entire container, and to allow separate commands to have separate values for the same variable.
The environment variable value will only be set in the context of the command, and shouldn't conflict with other commands setting the same variable.
The example given in issue description is a good one, setting different values of JAVA_OPTS for each separate command as need.

@johnmcollier will 2.0 spec support defining  ENV on the component level or just on the command level?

@maysunfaisal Both

Implemented


[13] Program Task Module

We were able to create our Task Module using the Codeigniter Framework and it was already running in the browser.


Map setting does not save when the zoom value has a decimal

Current behavior
When updating the zoom for a map and the value has a decimal an error is displayed when you click on the update button
To Reproduce
Steps to reproduce the behavior:

To the Maps tab in the Organization settings page
Select a country code, admin boundary, latitude and longitude
Enter a zoom value that has a decimal e.g 6.4
Click on the update button
"There was a problem updating data" error is displayed

Expected behavior
When updating the zoom for a map and the value has a decimal the page should be saved successfully
Screenshots

Hey @sannleen I think the Zoom level shouldn't accept decimals as we're trying to follow a standard zoom level scale: https://wiki.openstreetmap.org/wiki/Zoom_levels. But good point, user may not know this so we can look at adding help text. If you think that would be enough, maybe we can revise this issue to add help text instead?

Can the field be made a integer only field? If not we can add a help text.

@sannleen Good catch. Indeed the zoom is defined as an integer in the Workflow app: https://github.com/hikaya-io/activity/blob/dev/workflow/models.py#L191
I don't know exactly what this zoom level entails, but is rounding it to the nearest integer in the form (before insertion) not a viable option? Or does it make a noticeable difference?
If it is the same, then we would rather have the form adapt the input, than changing the model and adding an additional migration

UPDATE: Let's keep the zoom level as an integer and not support decimals. We can use this card to add a help text that says
Select a zoom level between 1-18 for the map.
Note:

It looks like 0 doesn't work nicely and the zooming stops at 18.

I just implemented this.


chore: use unzipped config fixtures

For ease of debugging I created a script that zips every folder in tests/fixtures/config, so the idea is to commit uncompressed folders instead of zips.
I'm wondering if we should have some exceptions for this. e.g. tooBigOfAZip -> we can keep the zipped file since its smth we're not going to touch further. It makes the script less straight away thought, but wondering how github will deal with a folder with so many entries (I don't want to make the repo slow to navigate...)
Still haven't added those folders (want to double check before).
Also I used yazl instead of the zip cli since I don't want to worry about platform compatibility (despite using a bash script would be muuuch simpler)

I probably won't be able to review this until I get back on August 12. Just FYI!


Fix support for empty responses in the schema

@turkerdev could you please publish new version?

maybe I could get npm permission to publish myself? also kibertoad there

maybe I could get npm permission to publish myself? also kibertoad there

I sent the invite.


[<IP_ADDRESS>]  version accepts any block version > 2

Version <IP_ADDRESS> accepts any block version > 2.
This opens a door for malicious users to create a block with a randon block version number, whichnthen will lead to a fork of the block chain, as <IP_ADDRESS> clients will accept the block, while 0.8.7.x and <IP_ADDRESS> will reject the block.

0.8.7.x and <IP_ADDRESS> will be eliminated.  we can send alert message.

They are eliminated when all users have upgraded. We can't force users to upgrade, so we have to secure the client against  such a situation.
It should be a small change only anyway: Replace 'if Version >=2' with 'if ((Version== 2) or (Version ==4))
just a one liner ;)

Part of documenting  "the total release procedure" is to bring people on board and warn them of changes, we have a number of updates to integrate it makes no sense to release that dev version, when we a proper <IP_ADDRESS>. is virtually ready to go and could be published properly.
I am suffering extra support issues (possibly because of <IP_ADDRESS>) when we should be spending effort setting up a 0.11 testnet, and working on getting the updates and testing of that  ....

The testnet must reflect the prodnet, so we need 0.8.7, 0.9.3 and 0.11 in the testnet

I think <IP_ADDRESS> accept block 2 and block 4 , but mine block 2 only . 0.11 mine block 4 only. so <IP_ADDRESS> is a bridge.  When the main pool is installed, we will broadcast the V4

@Lizhi this seems  to be causing a lot of extra work for no apparent reason and adding a LOT of potential problems. It is causing a lot of stress we don't need. Let's concentrate on updating 0.11 to <IP_ADDRESS> standards or above and give people a "reason to upgrade".
Unfortunately, we have tried to make it go faster but the upgrade to 0.11 can only go at a certain speed and by taking this unnecessary step (of having a bridge) we are actually slowing it down.....

Fixed in 0.9.6  in main.cpp line 2941:
if ((block.nVersion !=2)&&(block.nVersion !=4))
{
LogPrintf("AcceptBlockHeader,incorrect block version=%d \n",block.nVersion);
return(state.DoS(100, error("AcceptBlock() : incorrect block version")));
}


Add enable-api-fields: alpha to pipeline's CI runs

Feature request
Once https://github.com/tektoncd/pipeline/pull/3881 is merged we'll be in a good position to start testing alpha fields in our CI. In order to exercise new alpha features we'll need to add an additional round of integration tests and example runs in a cluster with enable-api-fields: alpha set in the feature-flags configmap.
Use case
Contributors to Tekton Pipelines should be able to observe the effects of their changes in environments with both "alpha" and "stable" feature gates during CI runs.

thanks @sbwsg yup, we need this to validate any new alpha features being added. I have bumped up the priority for the same reason.


Support for the docker-compose entrypoint option

Add support for the entrypoint option as described in the documentation:
https://docs.docker.com/compose/compose-file/#entrypoint
https://stackoverflow.com/questions/44316361/difference-between-docker-entrypoint-and-kubernetes-container-spec-command

Already implemeted


Fix linting errors

Seems there's a really aggressive linter that comes with the prototype kit, so I've fixed the warnings we were getting on the cookies JS. Might be related to the error Karen is seeing?

ah bums looks liek this conflicts now - probably the change I made the other day?

No worries, thought this might be to do with the bug we were seeing but as it's not I'll close this!


Better example of multi series pie chart and functioning legend using generateLabels & onClick callback

Let's say we want to create a mutli series pie chart were we show Yes vs No votes across three different groups.
Example dataset:

Group A: 33/67
Group B: 20/80
Group C: 10/90
Overall: 21/79

The sample multi series pie chart functionality leaves something to be desired for this because we cannot differentiate between Groups and Votes with colors.
If you look at this example, you will see that you are able to change the colors per dataset, but the legend and onclick filtering of the data series leaves much to be desired:
https://codepen.io/anon/pen/bygNqa
After a bunch of digging around stackexchange and trial and error, I figured out how to use the  generateLabels and onClick options for the legend to drastically improve the functionality. The trick here was figuring out how to override the legend item's color and its displayed state, while also linking it with a different data series. The docs don't really explain how to do this.
Solution:
https://codepen.io/anon/pen/JqEomG
Hopefully this example I made can be included in the docs or at least in the samples page.

@PaulWieland Could I do this issue ?

@WizOzHeir I don't follow...


Generic type "Object" preferred over hint

Hi
I had a problem with some JAXB2-generated lists of type List.
Despite using a hint (and/or deep hint), Dozer was deriving the items to be of type Object, so the mapping to a field in the real class type was failing.
I wonder if ReflectionUtils, line 169 (if genericType != null) should be checking if the generic type is anything other than Object if a hint is present?
Thanks,
Lee

Maybe related to: https://github.com/DozerMapper/dozer/pull/241


Does it make sense to push dist/*.map files to repository?

Expected behavior and actual behavior.
Temporary development files shouldn't be pushed
Steps to reproduce the problem.
Look at the repository
Specifications like the version of the project, operating system, or hardware.
__

We've solved this by ignoring map files in the repository, but building and including them in the package distribution upon release.

In the npm package up until 0.22.17, canvas-datagrid.map was included, but from there it does not seem to be... canvas-datagrid.debug.map and (from 2.5.1) canvas-datagrid.module.js.map are there. Was there a reason for this change?

Do you mean that canvas-datagrid.js does not have a corresponding map file, whereas the .debug.js and .module.js files do? I think the reasoning was along the lines that only debug needs a map, file, as you wouldn't want one in production. The ESM version is produced by Rollup, and I think I missed that it produces a sourcemap when I merged the PR. It's inconsistent, for sure.
Do you have a preference, and if so, why?

While in the general sense, having a map file only for the debug version should be all that's necessary, I don't really see much reason not to include it, either. If you don't want it in production, then don't put it in production. I could also imagine some (albeit extremely rare) cases where the debug code might work differently than the non-debug code. Just looking at the first part, === is changed to ==, for example. I do find it more convenient, though, as well, to use the same source as much as possible in development and release, so rather than adding some logic at some step to use the .js or the .debug.js, I personally would like to use the .js and be able to debug with it when necessary. Not to mention that it eases debugging in production, when you have to do that.
The only minor hitch is that you might still get unwanted requests for the map file if someone has developer tools open in the browser, or if there's a browser that gets map files even when developer tools isn't open, but that's the only argument against it that I can think of at the moment...
I'm also not a heavy JavaScript coder, though, so there may be other good reasons that I don't know about, and I also don't know what the general consensus is in the JavaScript world, so my opinions may not reflect those of the average consumer ^_~


try to use web clipboard api when possible

With this changes we won't need to implement the bridge method in iOS, as Web Clipboard Api works fine in Safari webviews

¡¡¡Thanks!!!


Can't run on Win7 32-bit

Windows error:
The version of this file is not compatible with the version of Windows you're running. Check your computer's system information to see whether you need an x86 (32-bit) or x64 (64-bit) version of the program, and then contact the software publisher.
This is where I need the Fusion log viewer: on a VM to test that I have all dependencies included in my build and to help find the missing ones.
Am I missing something or can I really not run the app in this environment?

New to me. I did a rewrite in version 1.2 in .NET Core. Can you try version 1.1 from the releases tab? That one was built using the full framework.
Would be nice if you could provide feedback on this.

Just tried installing "aspnetcore-runtime-3.0.3-win-x86.exe", but same error. Will try 1.1.

Tried installing "dotnet-runtime-3.0.3-win-x86.exe" but same error. Will try 1.1.

Please try 1.1. As said, this version was built with the good old Full-Framework (v4.7.2). You might need to install the v4.7.2 runtime if your Windows image is plain vanilla.

1.1 runs fine. I tried to log 1.2's startup with 1.1 but it is halted before logging starts.

Thanks for the report. I will have a look at this.
Yes, you can‘t log .NET Core apps with Fusion++ or FUSLOGVW, Microsoft changed pretty much everything about assembly loading in .NET Core. It might be applicable to .NET 5 again, see issue10.

Thanks - it's looking like a great tool - am going to have fun exploring my apps with it.

News on this?


Find and catch Pokémon to any location on the road

My observation, the pokemon born around pokestop, gym and near the road. Some time I want to my bot find and catch Pokémon to any location on the road. Can you add new feature.
thank

It is already implemented, with, every move the Bot search for pokemon with the catchable_pokemon event.

Sorry, in detail I want to bot walk on the road from pokestop to anther pokestop. somtime, my bot swim to pokestop. it's impossible.

#3854


Flaky test failure in ProgressBar?

https://ci.preferred.jp/pytorch-pfn-extras.torch110-win/96230/196033#L744

              assert 8 <= iters_per_sec <= 12 |
              assert 8 <= 7.9981

yeah, maybe we should do a round up


Updating the job description

@critzo for some reason the emphasis is weird on the Call to Apply near the end of the post. Also, not sure what's up with my pre-commit hooks, so we may need to close and re-open a new PR.

This change is

@georgiamoon It looks like on your system adding the hooks removed the _hooks/ folder and files inside. I'm wondering if this is something I need to test on a mac.  In any case, this PR LGTM. I can add the hooks back in an upcoming PR.

Noting that we are going to close this PR and open a new one.


Ensure MaxConcurrentRequests is a positive number

@DylanDmitri

If MaxConcurrentRequests is negative (or zero), there's currently no feedback and the server never responds to any requests. This is BAD behavior and we want to prevent this.
However, some tests rely on using 0 for MaxConcurrentRequests to make sure that nothing sneaks out of the queue. Adding the validation check breaks those tests, so it's all trickier than expected.

The check allows MaxConcurrentRequests to be 0 so the tests will pass. Also if MaxConcurrentRequests is negative we have a exception anyway https://github.com/aspnet/AspNetCore/blob/master/src/Middleware/RequestThrottling/src/Internal/RequestQueue.cs#L23

@Kahbazi good catch on the issue!


Remove deprecated changelog task

I would like to release https://github.com/voxpupuli/voxpupuli-release/pull/71 before we merge this one


Added gobject proposals hashes to trigger/superblock events

This is needed to be sure which proposal was paid if proposals have same address (which they can have and do have in Testnet)

Thanks for the PR! It's a little more complicated that just adding this -- we'll need to add it to the DB as well. I've already started a branch for this work -- have a look at the sb-proposals-hashes branch for more info.
https://github.com/nmarley/sentinel/tree/sb-proposal-hashes


*: added a script mode

This commit adds a script mode to acbuild, whereby build scripts can be
written in an acbuild DSL and then run. When acbuild is run in this way,
acbuild will exec itself with the debug flag, a work-path flag, and the
contents of each line. This lives under the "script" subcommand, but
this subcommand will be implied if the run acbuild has the name
"acbuild-script" (this was necessary to support /usr/bin/env shebangs).
The apache example has been rewritten to be in the acbuild DSL.
With a DSL mode various enhancements are now possible (like per-command
build caches), but the immediate benefit is the fact that acbuild can
enforce cleanup of builds that use the script mode.

Curious to get a review from @lucab on this

I'm typically not a fan of DSL, but I have to say that I'd really like having this scripting mode in acbuild! I've left some comments inline related to multicall usage, input validation and reducing runtime complexity.

I think this implementation of a DSL is a fairly sane way to add one, since the semantics are identical to whatever shell script you'd otherwise write, and if you want more power or control over a build you can still write a shell script just fine.

Ready for another review pass

Addressed nits, added the ability to have multi-line commands with trailing \ characters, and added a suite of test cases to try to hit the edge cases in the tokenizer and related functions.

@squeed another one it might be nice to have you review

No further concerns from my side.

This testcase fails for TestTrimComments, should it?
testcase{
    "this is a \"comment in a #string\".",
    "this is a \"comment in a #string\".",
},

It might be worth the extra effort to throw together a yacc grammar for this...

No, that shouldn't fail. I had forgotten about comments in strings. Since my comment parsing logic was run before the logic that's checking for strings, I removed trimComments, added its functionality to tokenizeLine, and added the cases TestTrimComments was checking for to TestTokenizeLine.

AWESOME work @dgonyeo ! I've been waiting for a functionality like this to be added. Just when I get back into looking at acbuild again in order to implement a DSL, I see this! Thanks for the awesome work!
I'd love to starting throwing out some examples for for acbuild scripts, one suggestion I have is the ability to use capitalization for each command.
Ex:
#!/usr/bin/env acbuild-script
BEGIN
SET-NAME example.com/apache

DEP add quay.io/coreos/alpine-sh

RUN -- apk update
RUN -- apk add apache2
RUN -- /bin/sh -c "echo 'ServerName localhost' >> /etc/apache2/httpd.conf"

PORT add http tcp 80

MOUNT add html /var/www/localhost/htdocs

SET-EXEC -- /bin/sh -c "chmod 755 / && /usr/sbin/httpd -D FOREGROUND"

WRITE --overwrite apache-latest-linux-amd64.aci
END

Also, is there the ability to remove the -- for each command as it'll be implied in the scripting language?

Thanks :)
I have no issue with capitalizing the subcommand, acbuild can just make the first token of each line lowercase before running the command.
As far as removing the --, that exists due to the need to differentiate between flags to acbuild and flags to the command you're running. Right now, you could have a script with this line:
run --working-dir=/opt -- mycoolapp --some-option .

Without the --, cobra will try to parse --some-option as a flag to acbuild.
Maybe a way to remove it would be to have acbuild insert a -- before the first argument that doesn't start with a -? This would mean that if you want to run a binary that starts with the - character it wouldn't work how you expect, but in that case you can just provide an absolute path to the binary.

@dgonyeo Thanks for replying!
Yeah, removing -- would be great as it'll simplify the experience when building an .aci.
Although you are right. It'll create some large differentiation between the command line as well as the scripting language.
Would the best way be to create documentation explaining it? Similar to other composable formats (I'm thinking kubernetes) as long as we have a decent amount of documentation flowing it'll be easier for users to switch to building via a DSL.

It is documented here, if you think the description there isn't sufficient I could always write a little more on the reasoning behind it.
And I'm comfortable with UX related diverges from the cli syntax. It should still be pretty clear the relation between the two syntaxes, and the -- has been a common complaint.

Just pushed a commit that lowercases the first token of a line, and inserts a -- before the first argument on a run command that's lacking a --.

This seems reasonable to me.
A question: what are the security implications of running untrusted acbuild scripts?

Were I to hand someone a malicious acbuild script, a couple things come to mind:

I could copy any files from your host and ship them off somewhere
I could attempt to break out of systemd-nspawn or the chroot engine (the latter is probably easier), and if I did I'd have root on the box
I could write an ACI to any path, potentially overwriting things you'd find important

So I'd highly recommend reviewing an acbuild script before running it. There's an issue from forever ago (here: https://github.com/appc/acbuild/issues/86) for running acbuild inside of rkt, and with the recent chroot engine this should now be much easier to implement, if someone wanted a sandboxed way to run an acbuild script.

Ahh, all good points! Thanks!

Review ping?

@squeed @lucab @alban

👍 LGTM!

LGTM too.


SwagSong has no field noteStyle

i got this error
source/Note.hx:63: characters 27-36 : SwagSong has no field noteStyle help me

are u trying to a death arrows?


members of the working tool group

My name is still not on the list as a member and as I am the co-chair I think it should be corrected.

On it!
Le mer. 12 sept. 2018 17 h 13, JaniqueGodin<EMAIL_ADDRESS>a
écrit :

My name is still not on the list as a member and as I am the co-chair I
think it should be corrected.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/canada-ca/OS-Advisory_Conseil-SO/issues/74, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABnJ5cjaVK-7BaKn-AEsJHDyDqV-uYZhks5uaXj2gaJpZM4WmLXw
.

On it!
Le mer. 12 sept. 2018 17 h 31, Guillaume Charest <
<EMAIL_ADDRESS>a écrit :

On it!
Le mer. 12 sept. 2018 17 h 13, JaniqueGodin<EMAIL_ADDRESS>a
écrit :

My name is still not on the list as a member and as I am the co-chair I
think it should be corrected.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/canada-ca/OS-Advisory_Conseil-SO/issues/74, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABnJ5cjaVK-7BaKn-AEsJHDyDqV-uYZhks5uaXj2gaJpZM4WmLXw
.

Hi Janique, I'm so sorry about that. I just added your name under the tools page as well.

Closing as it has been fixed


Keybindings for ctrl+tab, ctrl+shift+tab do not comply with operating system norms

Issue Type: Bug
The default keybindings for ctrl+tab and ctrl+shift+tab do not comply with Windows / OSX operating system norms. Standard tabbed application behavior on those platforms is workbench.action.previousEditor / workbench.action.nextEditor, which VS Code should respect.
VS Code version: Code 1.35.0 (553cfb2c2205db5f15f3ee8395bbd5cf066d357d, 2019-06-04T01:17:12.481Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.93GB (3.93GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

gitlens
eam
9.8.1

vsc-material-theme
Equ
2.9.0

Keybindings are configurable.


Release 24.10.1

Includes:

Remove unused i18n keys (PR #2038)
Update postcode regex for PII stripping (PR #2043)

Actually, if you can wait five minutes, can I add #2044 in?


Window buttons on custom titlebar are blurry at 1x DPI

VSCode Version: 1.39.0
OS Version: Windows x64 10.0.18362

Steps to Reproduce:

Ensure that you're using VS Code on Windows and on a monitor set to 1x DPI scale, and that you have the custom titlebar enabled
Observe the window buttons and see that the glyphs appear smeared in the bottom-right direction

Compare to a window using a native titlebar:

I assume this could possibly be related to the icon refresh (see https://github.com/microsoft/vscode/issues/78830), but I can't imagine this one is actually intentional. The standard width for the strokes on these icons is most certainly 1px and I can only assume that was the intended target here.

Does this issue occur when all extensions are disabled?: Yes

Can confirm that these icons are pixelated at 1x. Working on updating the icons, see below for before (top) and after (bottom):

@sbatten these are now added to Codicons (via https://github.com/microsoft/vscode/commit/4448d83f00c0c1419db2ba6254e1174cb72d61f6), you can refer to them as:

codicon-chrome-minimize
codicon-chrome-maximize
codicon-chrome-restore
codicon-chrome-close

All they need to do is follow the similar syntax as the Octicons classes: codicon codicon-ICON-NAME

These still seem blurry to me @misolori but @sbatten says that there is still an open issue.

That's covered here: https://github.com/microsoft/vscode/issues/83316


Please clearify license

According to your package.json, this project is licensed under BSD-2-Clause.
The LICENSE.md contains the text of BSD-3-Clause.
Under which License is it licensed?

@leipert Thanks for bringing this to my attention, seems like the license is BSD-3-Clause according to https://github.com/facebook/fbjs/blob/89ffda59d923183dd34efb2622da0176ccb04d41/package.json. May have been a typo on my part.
Care to submit a PR updating the package.json?


Kubectl install fails with 404 because of incorrect url

The github.com/openfaas/certifier CI is failing because arkade fails to install kubectl
Expected Behaviour

This script should work, as it did previously
KUBE_VERSION=v1.18.8

echo ">>> Installing kubectl $KUBE_VERSION"
arkade get kubectl --version $KUBE_VERSION
sudo mv $HOME/.arkade/bin/kubectl /usr/local/bin/

Current Behaviour

>>> Installing kubectl v1.18.8
Downloading kubectl
https://storage.googleapis.com/kubernetes-release/release/vv1.18.8/bin/linux/amd64/kubectl
Error: Process completed with exit code 1.

The url is incorrect, it contains two v in the path: vv1.18.8
Possible Solution

Ideally the installation should be able to handle specifying the version with or without a v and still build the correct URL
Steps to Reproduce (for bugs)

Run arkade get kubectl --version v1.18.8

Context

Your Environment

What Kubernetes distribution are you using?
Not applicable

What arkade version is this?

arkade version

Version: 0.7.19
Git Commit: a6a37d06ae25663b03f6279c3aaf7b01d3505eca

Are you a GitHub Sponsor (Yes/No?)
Check at: https://github.com/sponsors/alexellis

[x] Yes
[ ] No

From my perspective, this is still working as designed.
alex@am1 ~ % arkade get kubectl
Downloading kubectl
https://storage.googleapis.com/kubernetes-release/release/v1.20.0/bin/darwin/amd64/kubectl
44.09 MiB / 44.09 MiB [--------------------------------------------------------------------------------------------------------] 100.00%
Tool written to: /Users/alex/.arkade/bin/kubectl

# Add (kubectl) to your PATH variable
export PATH=$PATH:$HOME/.arkade/bin/

# Test the binary:
/Users/alex/.arkade/bin/kubectl

# Or install with:
sudo mv /Users/alex/.arkade/bin/kubectl /usr/local/bin/

alex@am1 ~ % arkade get kubectl --version 1.20.0
Downloading kubectl
https://storage.googleapis.com/kubernetes-release/release/v1.20.0/bin/darwin/amd64/kubectl
44.09 MiB / 44.09 MiB [--------------------------------------------------------------------------------------------------------] 100.00%
Tool written to: /Users/alex/.arkade/bin/kubectl

# Add (kubectl) to your PATH variable
export PATH=$PATH:$HOME/.arkade/bin/

# Test the binary:
/Users/alex/.arkade/bin/kubectl

# Or install with:
sudo mv /Users/alex/.arkade/bin/kubectl /usr/local/bin/

alex@am1 ~ %

The v cannot be hard-coded for apps because quite often, what you'll see is URLs that have this:
SEGMENT/version_without_v/BINARY_NAME-version-withv
or
SEGMENT/version_with_v/BINARY_NAME-version-without-v
So as a rule, the v prefix isn't passed in to apps. (It's fair to say that this isn't documented in the README)
So to fix this issue, I would write: KUBE_VERSION=1.18.8
Or KUBECTL_VERSION=1.18.1
Option 2 is to do what @yankeexe is suggesting, however it may lead to issues where people then pass --version v1.0.0 into all apps, and get surprised by the cases where it fails.
I wish download URLs were consistent.

While the opening post is about hardcoded v-prefix, the issue described in the title has become relevant since Google has moved hosting kubectl in different domain:
https://dl.k8s.io/release/v${version}/bin/${os}/${arch}/kubectl

Hi @okuuva perhaps you could raise a separate issue and describe what problem you're running into?
I've had no issues with kubectl or with specifying a version.
alex@bq:~$ arkade get kubectl
Downloading: kubectl
Downloading: https://storage.googleapis.com/kubernetes-release/release/v1.24.2/bin/linux/amd64/kubectl
43.59 MiB / 43.59 MiB [------------------------------------------------] 100.00%
/tmp/kubectl written.
2023/02/27 10:00:35 Copying /tmp/kubectl to /home/alex/.arkade/bin/kubectl

Wrote: /home/alex/.arkade/bin/kubectl (45.71MB)

# Add arkade binary directory to your PATH variable
export PATH=$PATH:$HOME/.arkade/bin/

# Test the binary:
/home/alex/.arkade/bin/kubectl

# Or install with:
sudo mv /home/alex/.arkade/bin/kubectl /usr/local/bin/

🐳 arkade needs your support: https://github.com/sponsors/alexellis
alex@bq:~$ arkade get<EMAIL_ADDRESS>Downloading: kubectl
Downloading: https://storage.googleapis.com/kubernetes-release/release/v1.26.0/bin/linux/amd64/kubectl
45.80 MiB / 45.80 MiB [--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------] 100.00%
/tmp/kubectl written.
2023/02/27 10:00:44 Copying /tmp/kubectl to /home/alex/.arkade/bin/kubectl

Wrote: /home/alex/.arkade/bin/kubectl (48.02MB)

# Add arkade binary directory to your PATH variable
export PATH=$PATH:$HOME/.arkade/bin/

# Test the binary:
/home/alex/.arkade/bin/kubectl

# Or install with:
sudo mv /home/alex/.arkade/bin/kubectl /usr/local/bin/

🐳 arkade needs your support: https://github.com/sponsors/alexellis
alex@bq:~$

Alex

/lock: off topic, original issue resolved


File cleaning

Specifically cleaning detritus like thumbs.db, DS_Store, etc.
This is something I handle hardcoded in the CBR/CBZ Applescript droplets and BetterZip provides a good user editable preference for.

Added in version 2.3.0

This feature has been temporarily removed in version 2.5.0, due to the zip library not able to exclude certain files when zipping an entire directory. (The problem can be solved by adding the files one by one, but that's a last resort solution). It may require me to rewrite the ZipUtilities library a bit, adding it as a feature.

Back again in 2.5.1.

Works well for non-wild card names. I tried adding ".*" to remove all hidden metadata files that Mac OS or other systems might create (one non-Mac thumbnail cache is .xvpics).
If user added wildcards are too difficult, I suggest you remove all hidden dot-files as a default. Either include a pref to turn on/off or annotate it in the readme and (potentially) in the pref heading area for user additions.
Given that this is a comic archives, would it also make sense to remove any 0 byte files regardless of filename? Empty folders as well?

Wildcards can't be added, as it must currently be in a regex pattern. But this should work:  ^(.\w+)
But, perhaps it makes more sense to include only image files (.jpg/.jpeg, .png .gif), as the app should only process those files, and remove the user defined filter all together?

I would recommend NOT removing all non-image files as other viewers/platforms use several of these. SFV (validation), PAR/PAR2 (validation/repair), NFO/TXT files are common. Comic Rack (Windows, Linux) uses a html file for metadata, etc. Simple Comic (Mac) supports PDF files, etc.
I would remove hidden dot-files by default as the typical user has no way to know these are there and I can think of no good reason for them in a comic archive (all the examples above are not hidden). I have seen Windows comic viewers choke on Mac metadata files. While I think this a bug as they shouldn't try to process them, it would be best not to include them.
I would also remove 0-length files.

Rereading your last comment, I recommend the following defaults if you remove user filters:
thumbs.db
all dot-files (includes .DS_Store)
__MACOS (and all zero length files)
empty folders
Personally, I'd prefer to keep user added filters as an advanced feature since covering all user needs is problematic. The file exclusion filter/function needs to communicated to the user in any event to prevent confusion.

Added dot files, thumbs.db and __MACOS as default filters (though you need to remove your .plist file first, it gets added to that the first time the plist file is created). Also, empty subfolders are not compressed any more, and empty folders cannot be added to the list.


Cancel button not showing

Am on development mode, So i disable my network to simulate network error, Paystack inline pop up opens with network error as expected. When i reload the paystack inline pop, the user lose the ability to close the pop up as there are no cancel button and clicking on the backdrop also does nohing

Please, the cancel button is not showing on my own end, am using react-paystack. It only shows when you hover over it .

Hi @Mezeobidi this repo is for Angular paystack not react

Alright, thanks


[BUG] Missing Select Security Groups in Advaced Zone with SG Enabled

Describe the bug
While Deploying a VM in a Zone with SG Enabled, the option to select the security groups is missing.
Ref Legacy UI :

@utchoang Could you look into this ? Will create a zone with SG Enabled on QA to test

Created SGAdvanced Zone

@davidjumani I tried with SGAdvanced on
http://primate-qa.cloudstack.cloud:8080/client/
But I couldn't find the step security group when I switched from affinity group to network. Thanks.

@utchoang Could you test now ?

@davidjumani OK. I've seen it now. Thanks.

@utchoang Could you look into this too

Missing Guest VLAN ID in create zone wizard for Advanced Zone with SG Enabled
'Fix Errors' button still present after fixing errors and zone successfully created

@davidjumani is this still an issue or fixed? cc @utchoang

PR https://github.com/apache/cloudstack-primate/pull/567 merged


Can't change hostname in WSL

Windows Terminal version
1.11.2921.0
Windows build number
10.0.19043.0
Other Software
No response
Steps to reproduce
I'm using Ubuntu 20.04 in WSL:
cat /etc/os-release
NAME="Ubuntu"
VERSION="20.04.1 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.1 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal
Try to change the hostname using the below command:
hostnamectl set-hostname <test-host-name>
Expected Behavior
Change the hostname to the new one test-host-name
Actual Behavior
It gives this error message instead:
System has not been booted with systemd as init system (PID 1). Can't operate. Failed to create bus connection: Host is down

@baseely that's a feature, not a bug. WSL is not using system daemon (systemd) and that's by design. To change the hostname on a WSL instance you have to add /etc/wsl.conf if not present already. If you cat /etc/hosts you'll find this at the top:
# This file was automatically generated by WSL. To stop automatic generation of this file, add the following entry to /etc/wsl.conf:
# [network]
# generateHosts = false

Here's sample wsl.conf I use:
[network]
hostname = ubuntu
generateHosts = false

There are more options to add to /etc/wsl.conf if you'd like to customize your instances.

If anything, this discussion belongs over at https://github.com/microsoft/WSL, I'm not sure we on the Terminal team have anything to add here. Thanks!

It's quite tricky since you need to change at several places.
First, set this in /etc/wsl.conf
[network]
hostname = wsl2
generateHosts = false

Then, make sure you have replace the old entries with your new hostname entry in /etc/hosts
<IP_ADDRESS>      your_new_hostname.localdomain        your_new_hostname
``

no need to change in `/etc/hostname`

Open powershell, shutdown wsl and you are done:

wsl --shutdown

I did a tutorial here: https://datmt.com/linux/change-ubuntu-wsl2-hostname-permanently/


Not able to load the local resources

Hi,
When I use the plugin, My application was not able to load the local resources that are created dynamically on a Cache folder.
It may be because of the content policy.  I see that this plugin serve the decrypted sources on a http localhost with the configured port.
I wonder what if the configured port has been already used by different application ?
Also What happens If the port can listened by browsers, will the application render on the browsers ?

Hi,
I am also facing the same issue on Android. I am trying to load the downloaded resources from cache folder and I am getting error as (http://localhost/app_file/data/user/0/com.*./cache/contactImages/WSi7wIMG-20190602-WA0002.jpg)  net::ERR_CONNECTION_REFUSED. I am using ionic web view 4.0.1, Android SDK target to 28.
Knidly, find below code I am using to read the local file.
$scope.getImagePath = function(item) {
var str = cordova.file.cacheDirectory;
var imgPath = str + "contactImages/"+ item.FileName + item.Extension;
return window.Ionic.WebView.convertFileSrc(imgPath);
};
Without cordova-plugin-crypto-file this source code is working perfectly fine.
Kindly, look into this issue and please suggest solution for the same.

@bhushanpd add your code in a repository and the steps that you are doing so I can try it and test it

The problem is the change in shouldInterceptRequest.
Using super.shouldInterceptRequest we are not using localServer with local file implements in WebViewLocalServer.java :/

@fdmartins can I contact you by email?

@PeterHdd of course yes

I found a solution for this issue.
In angular we need to convert uri to http using this command:
image = window.Ionic.WebView.convertFileSrc(imageUri);

In IonicWebViewEngine.java change to:
@Override
public WebResourceResponse shouldInterceptRequest(WebView view, WebResourceRequest request) {
WebResourceResponse r = localServer.shouldInterceptRequest(request.getUrl(), request);

if (r==null)
{
     return super.shouldInterceptRequest(view, request);
}

return r;
}

and in WebViewLocalServer.java change line 231 returning null.
if (isLocalFile(uri) || uri.getAuthority().equals(this.authority)) {
Log.d("SERVER", "Handling local request: " + uri.toString());
return handleLocalRequest(uri, handler, request);
} else {
**//return handleProxyRequest(uri, handler);
return null;   // <<<<<<<--- HERE**
}

@fdmartins Thanks a lot.. It worked perfectly...

@bhushanpd great!


General Issue: Adding 'weeks' to valid_time_units()

Background Information
If assert_that(is_timeunit(in_unit)) in compute_duration() is being replaced with assert_character_scalar(in_unit, values = valid_time_units()), then  should 'weeks' be added to the valid_time_units to avoid the last row below?

Definition of Done

Include 'weeks' in valid_time_units

@sadchla-codes , I think "weeks" should not be added to valid_time_units() because it is permitted for out_unit only but not for in_unit. You can replace the line highlighted above with lines 162 and 163 of derive_vars_duration.R:
  assert_character_scalar(in_unit, values = valid_time_units())
  assert_character_scalar(out_unit, values = c(valid_time_units(), "weeks"))

okay, that make sense, thank you @bundfussr !

Hey @sadchla-codes we don't close issues until releases.  This is so people don't remake the issue as 3 months is a long time :)

@bms63 , I think in this case closing it is ok as no code changes were made.

oh lol sorry i thought this was with a PR


Could you please provide the ".d.ts""? I really need his thanks :)

I developed it with typeScript, but I didn't find that you provided a definition file. Can you help me?Thank you so much!~ : )

+1

@AegeanRagdolls @vuliad I might not be able to do it until this weekend. I'll accept a PR :)

I’ll try publishing it this weekend.
Sent from my iPhone
On Nov 18, 2020, at 3:50 AM, Sepehr Hosseini<EMAIL_ADDRESS>wrote:

I can see there are some typings in core, but they're present on npm, any chance of having them published?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/pixijs/pixi-ui/issues/35#issuecomment-729531420, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFLJDB2S7VCKLLEA6DPZ3MLSQODFPANCNFSM4MHEP3WA.

I republished @puxi/core,tween 1.0.2 and they should have the declaration files

Thank you :)


dyndns was removed on latest Network update

On latest official Network release 7.2.92, dyndns is gone from the list of services. Is there an alternative service we can use? They are now running inadyn 2.8.1 into console, I think we can pick any service from the list.

@willswire I believe dyndns is not showing into list because I already use it. I wanted to create an additional option for a different worker.

I use dyndns for another Service already. Is it possible to choose one of the others?

Currently this isn't a supported option. IIRC, the way the dyndns configuration passes the ip and hostname URL query parameters is unique from other services. This would require significant rewrite/refactor, but I'm more than happy to look over some code if someone makes an MR!


Transactions period filter: improve behavior for "End Date"

In the transactions period filter (but this might be true for expenses/order filter too), if you pick the same day for Start Date / End Date, you will usually end up with no results. This is because the filter translates this query to:

startDate: 2021-06-01 00:00:00
endDate: 2021-06-01 00:00:00

While endDate should really be 2021-06-01 23:59:59, to make sure we include the transactions happening during the day.

I will take it.


"supply a valid key" error when registering with mac client

Using 1.1 I was unable to register using the Bitwarden mac app. iOS worked though.
Error was "supply a valid key"

just fixed yesterday in newest version 8a2b16eadccf9896f8d63a4d83300b3a5823610b

Thanks @vvondra and thanks for making this. It makes so much sense!


fix #620 Modules only apply after closing and re-opening the glasses inventory

PLEASE READ THE GUIDELINES BEFORE MAKING A CONTRIBUTION

Please check if the PR fulfills these requirements

[x] The commit message are well described
[x] All changes have fully been tested

What kind of change does this PR introduce? (Bug fix, feature, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their scripts due to this PR?)

Other information:

Thank you for your help! I really struggled with this one. I love the improvements you made, especially the one I didn't see


OpenCV will soon be NuGet-ized

Soon you will not need to include the OpenCV file and just need to add a nuget ref thanks to https://github.com/opencv/opencv/issues/7903 .
I will update you on this soon.

Wow. Thanks man


Upgrade/next 11.1

Upgrade to next 11.1 with ES Module Support
The update is more complicated than I would have like as it involves upgrading all unified js (remark / rehype) libraries to use the latest publish esm versions.
Main changes

Upgrade next.js related packages to 11.1.0
Add sharp image processing library as dependency, as recommended by nextjs docs
Set  experimental: { esmExternals: true } in next.config.js
Clean up unused devDependencies
Upgrade remark / rehype related packages to esm published libraries
Refactor lib/mdx.js  to use import instead of require
Standardize naming of mdx extensions - remark plugins are called remarkxxx and rehype plugins are called rehypexxx
Refactor remark / rehype plugin scripts to use es6 syntax

Will wait till I sync the updates on the typescript branch before merging and publishing a new release but feedback is always welcome.

Might I help you with something ?

All the necessary changes should have been made, but let me know if you encounter issues updating your i18n blog.


New Request: How can I add support for multiple videos or images ?

Hii,
I am very much impressed by your work.
I just want to add multiple images and videos support similar to Instagram in your existing project.
Can you guys help me out with this

I think you need write a shader to render multiple texture

I am looking for something like this where I can select multiple image or video and add use your editor on that.


Add a snippet for Hapi.js plugin

Hi @gordysc
Thanks for contribution 👍 . May you add the command to README as well for helping users notice the command? 😄

@deerawan sorry on the long response time.  I've updated the README as requested.


Club ID Update Customer.io Event

What's this PR do?
This pull request adds a Customer.io event dispatched when a user's club_id attribute is updated.
How should this be reviewed?
Does it make sense not to dispatch the event in case of an invalid club? Is this executed properly?
How are the tests for this?
Is the event name sound?
Any background context you want to provide?
We're interested in two emails being triggered through this event (two 'Campaigns' in customer.io). One, using the provided ID will be to the user or customer welcoming them to the club. (No specific attributes needed for that beyond the club name). Two, to the club's leader informing them that the user joined their club (hence those leader attributes).
I first took a page out of @aaronschachter's work in Rogue (https://github.com/DoSomething/rogue/pull/1018) to add a general CreateCustomerIoEvent job so we can easily track individual events. I also imitated the flow from those referral events in that work.
A pain point here was some of the test mocking. For one, I had trouble balancing the general mock in CreatesApplication for all the other tests involving club_id updates to work, while specifically testing against the arguments/return values in our individual tests. We seem to need to re-mock the GraphQL class / Customer.io class in order for this to work.
Relevant tickets
References Pivotal #174106264.
Checklist

[x] This PR has been added to the relevant Pivotal card.
[ ] Documentation added for new features/changed endpoints. - do we document these events anywhere?
[x] Added appropriate feature/unit tests.
[ ] If new attributes were added to users, there is a corresponding PR to surface these in Aurora.
[ ] If new attributes were added to users, then the data team already knows about these changes.

I'm a bit confused about these test failures. I don't see how this new work would cause our Blink service's userCreate to be invoked an extra time 🤔


Next config env should be string, not json value

When type checking the config with /** @type {import('next').NextConfig} */, the env variables like nextImageExportOptimizer_quality should be strings.

Hi @zifeo,
Thanks, I changed this in this PR #141

@Niels-IO That was fast, thanks! 🙌

@Niels-IO just tried with the latest version and nextImageExportOptimizer_quality still require number and the readme may need to be updated as well


chore: README update and CI test

Having transferred the code from a personal repo, this PR makes a few README changes and will also test the CI workflow.

All checks have passed


UI5 doesn't preserve core.HTML Component state. It re-renders the component from scratch.

OpenUI5 version: 1.84.17
Browser/version (+device/version): 98.0.4758.82
Any other tested browsers/devices(OK/FAIL): Microsoft Edge
Steps to reproduce the problem:

Render a core.HTML component with afterRendering prop that render children nodes to the core.HTML component with Deku JS. (3rd Party js library)
Stop the afterRendering function from running (the second time). This would show that UI5 didn't preserve the previously existing DOM.

What is the expected result?
The core.HTML component is re-rendered instead of being preserved like it used to in version 1.60.39.
The expected result would be the children nodes would disappear that was created by the intial afterRendering because the state of the core.HTML was not preserved.

@rafidashab this works exactly as documented https://ui5.sap.com/#/api/sap.ui.core.HTML%23methods/getPreferDOM
Even though I agree that https://github.com/SAP/openui5/commit/6c56823ad1bd5663fd512d1436d640c1c7b798a2 seems responsible, for your use case, rendering 3rd party component, should have been set the preferDom to true.

Hello Aborjinik. It seems like the preferDom had nothing to do with the render. I have tried setting preferDom to true manually.  UI5 doesn't preserve the DOM element at all. The commit you shared mentions : https://github.com/SAP/openui5/commit/6c56823ad1bd5663fd512d1436d640c1c7b798a2
With this change, moving "to-be-preserved" control DOM nodes to the
preserved area is left to the control responsibility. Controls should do
this onBeforeRendering hook via RenderManager.preserveContent API.
However the node I am using already has data-sap-ui-preserve="__html3" tag. Do I still need do something else to make sure the DOM is preserved along with it's children.