data:rezarex

Sorry @SurajPratap10 , I dismissed by mistake, but I have made the changes!

Sorry @SurajPratap10 , I dismissed by mistake, but I have made the changes!

no worries


[BUG] C2s missing pseudostatic wormhole type options

There is no option for pseudostatics when editing wormholes in class 2 systems.
To remedy, all the w-space C2 statics could be added to the Wandering Wormholes section of the type list:

Z647
D382
O477
Y683
N062
R474

:heavy_check_mark: Issue resolved
This issue was resolved by the commit(s) and/or the comments above.
It will be auto-closed with the upcoming release ( merge into master branch). Please check the attached Milestone information.
Many thanks to all participants/contributors for the helpful comments! :+1:
:information_source: Please reopen/comment this, if you have problems/questions about it.

Auto-generated response


Write a PHP program to check vowel

Description
Write a PHP program to check vowel
Vowel check is a simple program to check whether a character is a vowel or not.
Input  : a
Output : Vowel

How to contribute

Save the solution in check_vowel.php file
Add check_vowel.php file in check-vowel folder

!assign

Hey @deepu13,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)

!assign


question: why name the wasm file <inflected_crate_name>_wasm.wasm

the wasm part seems redundant. was just curious why it's named this way ;)

to differentiate the wasm from rustc and wasm from bindgen. It should be replaced with something like bg or bind 📦

Ah yeah it basically needed to be anything other than $crate.wasm because $crate.js needed to exist and would cause conflicts (I think?) with import { x } from './$crate';
I just randomly picked _wasm off the top of my head but something like _bg_wasm + _bg_shims would probably be a better idea!

Okay this seems like a really good-first-issue I will try to take a crack on this one.

Certainly! @sendilkumarn if you need any help just lemme know!

Sure!


Draw hierarchy for unhovered labels and node

When writing custom label renderers, the label is always drawn on top of any nodes, including the one that belongs to the label.  #1444 is related to this issue. When adjusting the draw hierarchy of unhovered nodes via CSS so that they are drawn over labels, nodes are also drawn on top of labels of other nodes, which is not my intention. Is there a way to get nodes drawn only over their own labels right now?
Many thanks.

Another thing that control over this would prevent is situations like this:

Here the label of the right node is drawn above the laben of the left node, while its node is drawn under the label of the left node, which is visually confusing

Hi @StockerGaming,
Normally, there is an additional layer, sigma-hoverNodes, that renders only the highlighted and hovered nodes, on top of the hover labels. Weirdly, in your capture, this layer seems to be missing. By the way, I just added a new documentation page, to better describe sigma's layers and what they do, and how to manipulate custom layers:
https://www.sigmajs.org/docs/advanced/layers
It looks like it is either an issue from your code, where you would wrongly manipulate the layers, or you reached some case where sigma breaks for some reason... Could you please try to produce some minimal code to reproduce this issue? Thanks!


[Bug]: How to shutdown/close/stop gracefully ledger desktop? Duplicate/2 instances?

Impacted Library name
latest
Impacted Library version
latest
Describe the bug
How do you exit gracefully after running pnpm run dev:lld, since closing the window doesn't close the application fully and CTRL+C causes issues when starting the app later (like it fails to get some stuff from the cache and to delete the DB).
It also seems that it launches two instances of ledger, which results in my visual-code editor to crash (this is an out of memory error):
ledger-live-desktop:start: cache bypass, force executing e3cd587677b2a7f1
ledger-live-desktop:start:
ledger-live-desktop:start: ><EMAIL_ADDRESS>start /home/arman/ledger-live/apps/ledger-live-desktop
ledger-live-desktop:start: > cross-env NODE_ENV=development node ./tools/main.js
ledger-live-desktop:start:
ledger-live-desktop:start: (node:45755) [DEP0025] DeprecationWarning: sys is deprecated. Use util instead.
ledger-live-desktop:start: (Use `node --trace-deprecation ...` to show where the warning was created)
ledger-live-desktop:start: (node:45755) ExperimentalWarning: WASI is an experimental feature and might change at any time
ledger-live-desktop:start:   ➜  Local:   http://localhost:8080/
ledger-live-desktop:start:   ➜  Network: use --host to expose
ledger-live-desktop:start: [watch] build finished, watching for changes...
ledger-live-desktop:start: [watch] build finished, watching for changes...
ledger-live-desktop:start: [watch] build finished, watching for changes...
ledger-live-desktop:start: Ledger Live 2.70.0
ledger-live-desktop:start: Ledger Live version: 2.70.0 { type: 'system-info' }
ledger-live-desktop:start: Ledger Live 2.70.0
ledger-live-desktop:start: Ledger Live version: 2.70.0 { type: 'system-info' }
ledger-live-desktop:start: Logs console display setup (main thread): {"everyLogs":false,"filters":[]}
ledger-live-desktop:start: Logs console display setup (main thread): {"everyLogs":false,"filters":[]}
ledger-live-desktop:start: [45855:1009/162049.904010:ERROR:process_singleton_posix.cc(338)] Failed to create /home/arman/.config/Electron/SingletonLock: File exists (17)
ledger-live-desktop:start: (node:45914) ExtensionLoadWarning: Warnings loading extension at /home/arman/ledger-live/tools/react-dev-tools:
ledger-live-desktop:start:   Manifest version 2 is deprecated, and support will be removed in 2023. See https://developer.chrome.com/blog/mv2-transition/ for more details.
ledger-live-desktop:start:
ledger-live-desktop:start: (Use `electron --trace-warnings ...` to show where the warning was created)
ledger-live-desktop:start: (node:45914) ExtensionLoadWarning: Warnings loading extension at /home/arman/.config/Electron/extensions/lmhkpmbekcpmknklioeibfkpmmfibljd:
ledger-live-desktop:start:   Manifest version 2 is deprecated, and support will be removed in 2023. See https://developer.chrome.com/blog/mv2-transition/ for more details.
ledger-live-desktop:start:   Permission 'notifications' is unknown or URL pattern is malformed.
ledger-live-desktop:start:   Permission 'contextMenus' is unknown or URL pattern is malformed.
ledger-live-desktop:start:
ledger-live-desktop:start: (node:45855) ExtensionLoadWarning: Warnings loading extension at /home/arman/ledger-live/tools/react-dev-tools:
ledger-live-desktop:start:   Manifest version 2 is deprecated, and support will be removed in 2023. See https://developer.chrome.com/blog/mv2-transition/ for more details.
ledger-live-desktop:start:
ledger-live-desktop:start: (Use `electron --trace-warnings ...` to show where the warning was created)
ledger-live-desktop:start: (node:45855) ExtensionLoadWarning: Warnings loading extension at /home/arman/.config/Electron/extensions/lmhkpmbekcpmknklioeibfkpmmfibljd:
ledger-live-desktop:start:   Manifest version 2 is deprecated, and support will be removed in 2023. See https://developer.chrome.com/blog/mv2-transition/ for more details.
ledger-live-desktop:start:   Permission 'notifications' is unknown or URL pattern is malformed.
ledger-live-desktop:start:   Permission 'contextMenus' is unknown or URL pattern is malformed.
ledger-live-desktop:start:

Expected behavior
Gracefully shutdown the app when all windows are closed, should be the default behavior.
Additional context
I am on Ubuntu/Linux.

Hi @Osiris-Team
have you solve this?

Yes it simply was because my VM had not enough memory. CTRL+C seems to close it somewhat gracefully.


Fixed where all mobs that are not explicitly registered have dropped …

Description
I have fixed the error where all mobs that were not registered were dropping a skeleton head. This was because the skeleton head was the default value, and if it wasn't overwritten, the skeleton head would be dropped. Since the methods shouldn't return null, I used optionals. This avoids a situation where an unregistered creature, such as a pig, drops a skeleton head.
Closes #426
Checklist

[x] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you).
[x] I have performed a self-review of my own code.
[x] I have commented my code in areas it may be challenging to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes are ready for review from a contributor.

Yep, like I said in the linked issue, I think this is an ideal solution. Nice work, and thanks for contributing!


Failed to execute 'getComputedStyle' on 'Window': parameter 1 is not of type 'Element'

Error on overlayFactory() implementation
<BootstrapTable keyField="id"                             data={this.state.data}                             columns={this.state.columns}                             striped                             bootstrap4={ true }                             loading={ true }                             overlay={ overlayFactory({ spinner: true, background: 'rgba(192,192,192,0.3)' })}                             />
Here's the error message:

app.js:103595 The above error occurred in the  component:
in TableLoadingOverlayWrapper (created by BootstrapTable)
in BootstrapTable
in SortProvider
in DataProvider (created by BootstrapTableContainer)
in BootstrapTableContainer (created by DailyTimeRecord)
in div (created by DailyTimeRecord)
in div (created by DailyTimeRecord)
in DailyTimeRecord (created by Route)
in Route (created by Main)
in Switch (created by Main)
in div (created by Main)
in div (created by Main)
in Switch (created by Main)
in Router (created by BrowserRouter)
in BrowserRouter (created by Main)
in div (created by Main)
in Main

@HumbleCarl i will check this, currently, I will not suggest people to use react-bootstrap-table2-overlay instead you can custom by yourself, you can check this example.
Currently, react-bootstrap-table2-overlay use third party library and it produced some bugs.

Anyway, I will look into it but not sure the fixing time.

Copy that. Thank you sir!

I faced the same error in the browser, and I found that the overlay functionalities works when the noDataIndication prop is provided.
So what I did was to provide a blank noDataIndication, then I can use the overlay with no error.


SoapClient and authentication headers

Hi,
I'm currently working on the integration of SoapClient in the API client.
This doesn't work, and I think this is due to the authentication headers.
The documentation ask to send theses headers
<soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:yous="http://www.yousign.com" xmlns:aut="http://authenticationejb.yousign.com/">
   <soapenv:Header>
      <username>YOUR_USERNAME</username>
      <password>YOUR_PASSWORD</password>
      <apikey>YOUR_APIKEY</apikey>
   </soapenv:Header>
   <soapenv:Body>
      <yous:connect/>
   </soapenv:Body>
</soapenv:Envelope>

But the soap heade specification and the SoapHeader class don't allow an empty namespace, it is mandatory.
Did you already success to connect to your Soap WebService with the PHP SoapClient ? Do you plan to add namespaces to the authentications headers ?
Thanks

Hi,
I'm really sorry for my late answer.
Do you have solved your problem.
If no, I made a test with SoapClient, and as you said there is no namespace for the header.
But you can add a parent and put inside a fake namespace like as below :
$wsdl = 'yousign_api_wsdl_url'; // @see http://developer.yousign.fr/
$soapClient = new \SoapClient($wsdl);

$authentication = new \stdClass();
$authentication->apikey 	= 'your_apikey';
$authentication->username 	= 'your_username';
$authentication->password 	= 'your_encrypted_password'; // (cf doc: sha1(sha1(clearPassword)+sha1(clearPassword)) )

// The name of namespace and parent are not fixed, you can put as you want
$header = new \SoapHeader('http://www.yousign.com', 'Auth', $authentication);
$soapClient->__setSoapHeaders($header);

$soapClient->__call('connect', array()); // should be return true

Bonjour Yousign/yousign-api-client-php,
Merci pour votre message, nous vous répondrons dans les meilleurs délais. Le numéro de ticket associé à votre demande est : 7087825.
L'équipe Yousign


Suite of suite-files has CloseChannelException

TestNG Version
6.9.13
Expected behavior
Successful read of a suite-file.
Actual behavior
Caused by: org.testng.TestNGException: java.nio.channels.ClosedChannelException
	at org.testng.xml.SuiteXmlParser.parse(SuiteXmlParser.java:20)
	at org.testng.xml.SuiteXmlParser.parse(SuiteXmlParser.java:9)
	at org.testng.xml.Parser.parse(Parser.java:170)
	at org.testng.TestNG.initializeSuitesAndJarFile(TestNG.java:279)
	at org.testng.TestNG.run(TestNG.java:1081)

Notes
Both files exist and readable. Running just tests.xml is successful. Running with individually enumerated suite xml eg. tests1.xml, tests2.xml executes successfully.
Encountered while using TestNG in a maven project and assigning the suite xml programmatically with testNg.setXmlSuites(List<XmlSuite>);
Also, built 6.9.13 (Ubuntu 16.04 x64). Successful build. Jar in testng/build/libs/testng-6.9.13.jar. However there is 'no main manifest attribute'; so couldn't just do java -jar testng-6.9.13.jar suite-files.xml.
Is the issue reproductible on runner?

[ ] Shell
[ ] Maven
[ ] Gradle
[ ] Ant
[x] Eclipse
[ ] IntelliJ
[ ] NetBeans

Test case sample
suite-files.xml
<?xml version="1.0" standalone="no"?>
<!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd">
<suite name="Suite" preserve-order="true">
  <suite-files>
    <suite-file path="/tmp/tests.xml"></suite-file>
  </suite-files>
</suite>

tests.xml
<?xml version="1.0" standalone="no"?>
<!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd">
<suite name="ParallelTests" preserve-order="true" parallel="tests">
  <test name="TestGoogle">
    <classes>
      <class name="com.sios.stc.coseng.tests.demo.Google">
        <methods>
          <include name="connect1" />
        </methods>
      </class>
    </classes>
  </test>
</suite>

However there is 'no main manifest attribute'; so couldn't just do java -jar testng-6.9.13.jar suite-files.xml.

Related to https://github.com/cbeust/testng/issues/1190

@james-crocker I'm not able to reproduce the issue. Could you share a full runnable sample?
My test was and work as expected (minus #863):
  @Test
  public void test() throws IOException, SAXException, ParserConfigurationException {
    TestNG tng = new TestNG();
    tng.setXmlSuites(new Parser("suite-files.xml").parseToList());
    tng.run();
  }

Went back to create a simple example. In doing so, found that during some pre-processing I set the xmlSuite.setFileName() which in turn made it self-referential I think. At any rate, removed this step and all is good in reading a suite xml file with suite-files declarations. Thank you for time and consideration.

Ok. If you think something is wrong with setFileName, feel free to open another issue.


Is this repo still in use?

...or did we move everything to https://github.com/r9y-dev ?

everything should be moved.  are you suggesting i actually deprecate all this? :)
(do you see anything that is left here which wasn't ported over before i do so?)


gzdoom: g3.4.1 -> g3.5.0

Motivation for this change
Update package to latest
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build gzdoom

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: gzdoom
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Failure on x86_64-linux (full log)
Attempted: gzdoom
Partial log (click to expand)
building '/nix/store/67dpnhvr8wz6v45sg8k37a50la2kcmmw-source.drv'...

trying https://github.com/coelckers/gzdoom/archive/g3.5.0.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   124    0   124    0     0    300      0 --:--:-- --:--:-- --:--:--   300
100 9742k    0 9742k    0     0  4792k      0 --:--:--  0:00:02 --:--:-- 9861k
unpacking source archive /build/g3.5.0.tar.gz
cannot build derivation '/nix/store/kv3gsspximscv5dbkm56srxkjprbjdpg-gzdoom-3.5.0.drv': 2 dependencies couldn't be built
error: build of '/nix/store/kv3gsspximscv5dbkm56srxkjprbjdpg-gzdoom-3.5.0.drv' failed

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: gzdoom
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.


Overhaul

Overall cleanup of unused imports.
Added block and touchOther calls. With this, all privileged functions except destroy are available.
Updated README.md.
Added support for zombienet .
Renamed statemintand statemine to ...-asset-hub .
Cleaned up typos.
TODO:

[ ] Set up destroy .

[ ] Update and test dependencies.

[ ] Test zondax-ledger interaction.

If you intend on using yarn berry you need to add the following to the .gitignore:
# Yarn berry
.yarn/*
!.yarn/releases
!.yarn/plugins

This will ensure you dont push up the cache folder which is what is increasing the files for review.
In addition to that, you need to remove the package-lock.json if you want it to be leveraging yarn only.

Still waiting for (PR #18)[https://github.com/LedgerHQ/app-statemint/pull/18] and (PR #34)[https://github.com/LedgerHQ/app-statemine/pull/24] on LedgerHQ to go through to test Ledger integration


Cmake configuration fails with the latest version of gcc (5.1.1).

The latest version (5.1.1) of gnu compiler return just its major version ("5") when given the "-dumpversion" argument. GCC compiler developers don't consider this to be an error. A workaround for this case has been made.

https://bugzilla.novell.com/show_bug.cgi?id=941428#c4
@alalek

Just update PR source branch. No need to reopen PR.

I wanted to squash the last to comments, but github didn't let me. I thought closing the pull request would help. ... but all in vain.

Try to push  with "force" flag: git push -f ...

:+1:

thanks! yes, the force option does the trick.


Fix kube-push using gcloud compute instance-groups managed set-instance-template

GCE cluster update / upgrade is broken due to incompatible gcloud command. This pr should fix that.
Should this one be cherrypicked?

cc/ @brendandburns @roberthbailey

@wojtek-t
This broke in https://github.com/GoogleCloudPlatform/kubernetes/pull/11692, so it should only be cherry picked if that PR was cherry picked.

LGTM.

@roberthbailey @dchen1107
The PR that broke it was cherrypicked in #12192 (as requested by @roberthbailey in https://github.com/GoogleCloudPlatform/kubernetes/pull/11692#issuecomment-127329212 )
BTW - sorry for breaking

Thanks (I couldn't remember). Dawn, can you cherry pick this one as well?

@wojtek-t and @roberthbailey: Sure, thanks!

Thanks for fixing it @dchen1107 !


feat: https support

With this PR by using HTTPS env var will be possible to start the server with https instead of http.
Server cert and key will be auto generated and stored on store directory using node pem
cc @jmgiaever
ref #479

Pull Request Test Coverage Report for Build 554329908

0 of 88 (0.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 19.705%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

build/webpack.dev.conf.js
0
1
0.0%

config/index.js
0
6
0.0%

app.js
0
81
0.0%

Totals

Change from base Build 554121897:
-0.1%

Covered Lines:
1996

Relevant Lines:
10386

💛  - Coveralls


invidious: unstable-2022-11-02 -> unstable-2022-11-17

Description of changes
https://github.com/iv-org/invidious/compare/09942dee6621e7047a63dffcc61b3bbf78cef2c6...516efd2df3f7d242c2d1df416053b4991a554116

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

maintainers:
@sbruder @infinisil

Result of nixpkgs-review pr 201713 run on x86_64-linux 1

1 package built:

invidious


SIZRF new code for PEG_BVOL_2021

For the new version of the PEG BVOL, we need a respective code in SIZRF https://vocab.ices.dk/?ref=303

code PEG_BVOL2021 is now added


android-studio update is available but "brew cask update" doesn't work.

Current is 1.31 and 1.41 is available, in fact even the cask has it.
As mentioned in the title, running brew cask update android-studio returns:
Updated Homebrew from 47494569 to 53d3735b.
No changes to formulae.

How do I upgrade a brew cask?
I am not upgrading it from within the Android Studio app itself for fear of conflicts and breaking something.

How do I upgrade a brew cask?

You don’t. Best you can currently do is brew update && brew cask install --force {{your_app_here}}.
brew cask update is just an alias to brew update. It takes no arguments.


implement upgrade path from 4.6.0 to 4.6.1

The 4.6 branch needs an upgrade path to 4.6.1. I looked at how this was done before and think this will do. Can you give it a second pair of eyes please @DaanHoogland ?
When this is indeed OK, I'll do the same for master for the 4.6.0 -> 4.7.0 path.
I've tried a build and that works: mvn clean install
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 07:33 min
[INFO] Finished at: 2015-11-15T13:45:58+01:00
[INFO] Final Memory: 120M/1591M
[INFO] ------------------------------------------------------------------------

Once this is a PR, I'll try an actual upgrade from 4.6.0 to 4.6.1-SNAPSHOT.

looks good, let's run a test anyway. the jenkins should run it but it it's running on a INFRA-10703 host so it will most likely fail.

@DaanHoogland I rebased it, as it needs the checkstyle fix in 4.6.

merged with 4.6 branch and:
INFO  [c.c.s.ManagementServerImpl] (main:ctx-1c73e37e) Startup CloudStack management server...
INFO  [c.c.c.ClusterManagerImpl] (main:ctx-1c73e37e) register cluster listener class com.cloud.server.LockMasterListener
INFO  [c.c.c.ClusterManagerImpl] (Cluster-Heartbeat-1:ctx-62e492b0) We are good, no orphan management server msid in host table is found
INFO  [c.c.c.ClusterManagerImpl] (Cluster-Heartbeat-1:ctx-62e492b0) No inactive management server node found
WARN  [o.a.c.alerts] (Cluster-Notification-1:ctx-7db732cd)  alertType:: 14 // dataCenterId:: 0 // podId:: 0 // clusterId:: null // message:: Management server node <IP_ADDRESS> is up
WARN  [c.c.c.ClusterManagerImpl] (Cluster-Notification-1:ctx-7db732cd) Notifying management server join event took 9 ms

LGTM

This is an upgrade from 4.6.0 to 4.6.1:
INFO  [o.a.c.s.l.CloudStackExtendedLifeCycle] (main:null) Running system integrity checker com.cloud.upgrade.DatabaseUpgradeChecker@419c5949
INFO  [c.c.u.DatabaseUpgradeChecker] (main:null) Grabbing lock to check for database upgrade.
INFO  [c.c.u.DatabaseUpgradeChecker] (main:null) DB version = 4.6.0 Code Version = 4.6.1-SNAPSHOT
INFO  [c.c.u.DatabaseUpgradeChecker] (main:null) Database upgrade must be performed from 4.6.0 to 4.6.1-SNAPSHOT
INFO  [c.c.u.DatabaseUpgradeChecker] (main:null) Cleaning upgrades because all management server are now at the same version
INFO  [c.c.u.DatabaseUpgradeChecker] (main:null) Cleanup upgrade Upgrade460to461 to upgrade from 4.6.0-4.6.1 to 4.6.1
INFO  [o.a.c.s.l.CloudStackExtendedLifeCycle] (main:null) Configuring CloudStack Components

The database shows:

MariaDB [cloud]> select * from version;
+----+---------+---------------------+----------+
| id | version | updated             | step     |
+----+---------+---------------------+----------+
|  1 | 4.0.0   | 2015-11-06 13:33:14 | Complete |
|  2 | 4.1.0   | 2015-11-06 13:33:25 | Complete |
|  3 | 4.2.0   | 2015-11-06 13:33:25 | Complete |
|  4 | 4.2.1   | 2015-11-06 13:33:25 | Complete |
|  5 | 4.3.0   | 2015-11-06 13:33:25 | Complete |
|  6 | 4.4.0   | 2015-11-06 13:33:26 | Complete |
|  7 | 4.4.1   | 2015-11-06 13:33:26 | Complete |
|  8 | 4.4.2   | 2015-11-06 13:33:26 | Complete |
|  9 | 4.5.0   | 2015-11-06 13:33:26 | Complete |
| 10 | 4.5.1   | 2015-11-06 13:33:26 | Complete |
| 11 | 4.5.2   | 2015-11-06 13:33:26 | Complete |
| 12 | 4.6.0   | 2015-11-06 13:33:26 | Complete |
| 13 | 4.6.1   | 2015-11-15 15:38:58 | Complete |
+----+---------+---------------------+----------+
13 rows in set (0.00 sec)

I'd say this works as expected :-)
Pinging @DaanHoogland

so, you have my lgtm, need a second one?

Yes, let's stick to the rules :-)

LGTM
Works on my test deployment.
2015-11-15 19:52:41,711 DEBUG [c.c.u.DatabaseUpgradeChecker] (main:null) Cleanup script /usr/share/cloudstack-management/setup/db/schema-460to461-cleanup.sql is executed successfully
2015-11-15 19:52:41,736 DEBUG [c.c.u.DatabaseUpgradeChecker] (main:null) Upgrade completed for version 4.6.1

Hi,
@remibergsma  and @DaanHoogland tested this PR and I reviewed the code. it LGTM :+1:
Cheers,
Wilder


V8 needs to be tested for native locations

It is ambiguous how to access non-file location strings by the v8 stack api. I need to see an example of it.

Added a code which uses getFileName() by non-file locations. I am not sure about this, I have to test it.

I won't be able to use captureStackTrace with native locations. It is meant for custom errors. I have to use a string parser by v8 native errors.

Native locations probably cannot be part of a custom stack, but eval locations can.
function main (){
	var error = new Error("xxx");
	Error.prepareStackTrace = function (error, frames) {
		return frames;
	};
	Error.captureStackTrace(error);
	delete(Error.prepareStackTrace);
	return error;
}

var error = eval("main()");

console.log(error.stack)

Error: xxx
    at main (x.html:8)
    at eval (eval at <anonymous> (x.html:13), <anonymous>:1:1)
    at x.html:13

These need to be tested for being non-file location.

function main (){
	Error.prepareStackTrace = function (error, frames) {
		return frames;
	};
	var error = new Error("xxx");
	Error.captureStackTrace(error);
	var frames = error.stack;
	delete(Error.prepareStackTrace);
	return frames;
}

var frames = eval("main()");

console.log(frames)

note:
Only this order works by getting the stack, since the prepareStackTrace is called only by then.

function main (){
	Error.prepareStackTrace = function (error, frames) {
		return frames;
	};
	var error = new Error("xxx");
	Error.captureStackTrace(error);
	var frames = error.stack;
	delete(Error.prepareStackTrace);
	return frames;
}

var frames = eval("main()");

var frame = frames[1];

console.log({
	toString: frame.toString(),
	getThis: frame.getThis(),
	getTypeName: frame.getTypeName(),
	getFunction: frame.getFunction(),
	getFunctionName: frame.getFunctionName(),
	getMethodName: frame.getMethodName(),
	getFileName: frame.getFileName(),
	getLineNumber: frame.getLineNumber(),
	getColumnNumber: frame.getColumnNumber(),
	getEvalOrigin: frame.getEvalOrigin(),
	isToplevel: frame.isToplevel(),
	isEval: frame.isEval(),
	isNative: frame.isNative(),
	isConstructor: frame.isConstructor()
});

	toString: "eval (eval at <anonymous> (file:///C:/Users/inf3rno/Desktop/x.html:14:14), <anonymous>:1:1)",
	getThis: Window,
	getTypeName: "Window",
	getFunctionName: "eval",
	getMethodName: null…}
	getColumnNumber:1
	getEvalOrigin:"eval at <anonymous> (file:///C:/Users/inf3rno/Desktop/x.html:14:14)"	getFileName:undefined
	getFunction:main()
	getFunctionName:"eval"
	getLineNumber:1
	getMethodName:null
	getThis:Window
	getTypeName:"Window"
	isConstructor:false
	isEval:true
	isNative:false
	isToplevel:true

According to this by eval we have to use evalOrigin, by native we have to set native location. It is not clear what happens by unknown location, probably nothing.


Progress bar's dot isn't rendered at all

The title is self explanatory.

works for me

@SunsetTechuila it doesn't work sometimes!
@windowz414 even i experience this issue sometimes!
closing and reopening the spicetify desktop app fixes this issue. but sometimes it doesn't work either.
so i'll start working on this issue as soon as possible.

Never faced. Is it random?

@nimsandu, thanks for the tip on reopening it. Now I don't face it...for now.
@SunsetTechuila yes it's random. But I guess it's reproducible by applying a Spicetify config and so make it restart Spotify.

Edit 2: Going full screen causes this issue.
Now it makes more sense. Which extension do you use for full screen?

Edit 2: Going full screen causes this issue.

Now it makes more sense. Do you use spotify built-in fullscreen?

Yes I do.

using css also looks possible

Code
I have no time for it now, but maybe @nimsandu will

With updated bloom?

Yep sorry that's fixed

@windowz414 problem no longer exists?

@SunsetTechuila, I'll check when I get on my laptop.

@SunsetTechuila, it's perfect now. Thanks for contributing to Bloom and fixing this bug!

@windowz414 @SunsetTechuila i always appreciate you both 😊


[now-cli] Ensure that stdin exists when calling setRawMode

Ensure that stdin exists when calling setRawMode.
Error on Sentry
PRODUCT-637

Codecov Report

Merging #3321 into master will increase coverage by 1.91%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3321      +/-   ##
==========================================
+ Coverage   11.77%   13.69%   +1.91%
==========================================
  Files         267      267
  Lines       10162    10405     +243
  Branches     1331     1255      -76
==========================================
+ Hits         1197     1425     +228
- Misses       8869     8909      +40
+ Partials       96       71      -25

Impacted Files
Coverage Δ

src/util/input/text.ts
0% <0%> (ø)
:arrow_up:

src/util/input/prompt-bool.ts
0% <0%> (ø)
:arrow_up:

src/util/prompt-options.js
0% <0%> (ø)
:arrow_up:

src/util/dev/builder-cache.ts
84.66% <0%> (+2.64%)
:arrow_up:

src/util/get-files.ts
91.08% <0%> (+4.26%)
:arrow_up:

src/util/dev/builder.ts
70.49% <0%> (+5.1%)
:arrow_up:

src/util/dev/yarn-installer.ts
89.09% <0%> (+5.36%)
:arrow_up:

src/util/dev/server.ts
64.61% <0%> (+5.77%)
:arrow_up:

src/util/output/create-output.ts
65% <0%> (+7.42%)
:arrow_up:

Continue to review full report at Codecov.

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


Istio auto-inject with environment variable name invalid value error

Describe the bug
Some code from auto-inject
[[ range $k,$v := .ObjectMeta.Labels ]]

name: ISTIO_META_[[ $k ]]
value: "[[ $v ]]"
[[ end ]]

if Service,Deployment metadata labels have special char '/' , while auto-inject it will generate a env name which has '/'. Finally , error like below
Error creating: Pod "hello-5b9968fc7b-dn2jw" is invalid: [spec.containers[1].env[6].name: Invalid value: "ISTIO_META_app.alauda.io/name": a valid environment variable name must consist of alphabetic characters, digits, '_', '-', or '.', and must not start with a digit (e.g. 'my.env-name', or 'MY_ENV.NAME', or 'MyEnvName1', regex used for validation is '[-._a-zA-Z][-.a-zA-Z0-9]*'), spec.containers[1].env[9].name: Invalid value: "ISTIO_META_service.alauda.io/name": a valid environment variable name must consist of alphabetic characters, digits, '', '-', or '.', and must not start with a digit (e.g. 'my.env-name', or 'MY_ENV.NAME', or 'MyEnvName1', regex used for validation is '[-._a-zA-Z][-._a-zA-Z0-9]*')]
Expected behavior
if istio can replace the special char , or any better solution?
Steps to reproduce the bug
Deployment like below can reproduce the bug
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
name: hello-v1
spec:
replicas: 1
template:
metadata:
labels:
app: hello
hello/v1: hello-v1
version: v1
Version
istio1.0.3
[root@k8s-master lgz-cs]# kubectl version
Client Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.0+d4cacc0", GitCommit:"d4cacc0", GitTreeState:"clean", BuildDate:"2018-09-26T12:30:09Z", GoVersion:"go1.10.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.0+d4cacc0", GitCommit:"d4cacc0", GitTreeState:"clean", BuildDate:"2018-11-09T15:12:26Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Installation
{{ Please describe how Istio was installed }}
Environment
{{ Which environment, cloud vendor, OS, etc are you using? }}
Cluster state
{{ If you're running on Kubernetes, consider following the
instructions
to generate "istio-dump.tar.gz", then attach it here by dragging and dropping
the file onto this issue. }}

This looks like a duplicate of https://github.com/istio/istio/issues/9605
Can you try it on 1.0.5?


Advice on publishing a plant usages dataset

Hi folks, I have a dataset of plant usages (mostly medicinal + food) which I want to publish to GBIF. I think it's probably most appropriate as a checklist?
There are multiple usages per plant, and each usage is associated with a particular geographical location and with particular vernacular names. I was thinking if I publish a checklist with one row per usage I would have duplicate scientific names BUT I would be able to:

use the taxonid with the speciesdistribution extension http://rs.gbif.org/extension/gbif/1.0/distribution.xml in order to tie specific plant usages to places, and
use the taxonid with the vernacularnames extension http://rs.gbif.org/extension/gbif/1.0/vernacularname.xml to tie specific plant usages to common names.

If there's a better way to go please let me know. I guess an alternative would be to collapse the data and publish a normal non duplicate checklist, losing the common name <-> usage and place <-> usage relationships, but that's not great.

I am also very interested in this issue.
In my opinion, sometimes the information about vernacular names and plant usages looks more like occurrence data.
But for some reason, the IPT allows only checklist datasets to be extended with the vernacular names extension.
A practical example: an herbarium collection of labeled plant specimens, which are already published to gbif as an occurrence dataset. For now, only some core-occurrence concepts have been mapped, like institutionCode, collectionCode, basisOfRecord, catalogNumber, recordedBy, ScientificName, and several location related concepts.
But what if in the specimen labels there are also vernacular names and plant usages which we want to be published to gbif?
Let's say, one specimen collected in a certain country, contains 3 labels informing about the different vernacular names and popular usages of the species, in 3 different places around the location where the specimen was collected.
And the same for other specimens of the herbarium, collected in different places around the world.
As we cannot apply the vernacular names extension to the occurrence data, we have to cope with producing a checklist. But doing so, how can we link the vernacular names information to the original herbarium specimens where it comes from?
Looks like the only way is using the dcterms:source field, but I don't think that will permit people to navigate between the vernacular name and the herbarium specimens, when they are browsing either the occurrence or checklist datasets, Am I wrong?
Another problem I found is mapping the medicinal/food plant usages:
I don't see an appropiated field to store this information, either in core definitions (checklist / occurrence) or extensions (vernacular names). @rukayaj mentions the species distribution extension for that purpose, but I don't see any appropiated field there either.
Perhaps the measurements or facts extension? This can be applied to both occurrence and checklist data, but I am not sure if it is appropiate for this purpose.
Finally, I am a bit lost about how to link the extension to the checklist core. as @rukayaj says the checklist would have duplicated names, but I don't see why should that happen.
Isn't the taxonid supposed to act as a 1:N key which should avoid that problem, so it is used to link each taxon name with its corresponding vernacular names, once both text files are merged into the darwincore file?
Please somebody at gbif could enlight us on this? It would be great having a sample file to download and use as a model to follow.
Thanks a lot

@abubelinha I ended up using a checklist core with duplicate scientific names as I described, so that each use (stored in measurement or facts) is related to one record, e.g.:
https://www.gbif.org/species/164940157/verbatim
https://www.gbif.org/species/164940159/verbatim
This means that a specific plant use can be linked to a specific location and a specific vernacularName, which is important in this dataset (e.g. to know that in this part of Norway this plant was used for veterinary purposes and it was called X, whereas in this part of Sweden it was used as animal food and it was called Y).
Anyway even though I have published the dataset I will leave this issue open as I'm still interested in feedback from GBIF, and the case you describe where you want to link occurrence data with vernacular names seems relevant.

OK, I see now why you suggested to use the species distribution extension.
But the way you constructed your checklist core data looks a bit odd to me, in the sense that you are not using a single record for each taxon name. You are generating many different taxonIDs for the same taxon (for example, Achillea millefolium has near 25 different taxonids, as I see in the taxon.txt of your DwC zip file).
Of course you get your data published and the info is available. But it is not easy to browse: you can not click on a taxon and see all different vernacular names and usages associated to that species in the different locations and dates.
I would like to have some gbif feedback here (perhaps @mdoering or @timrobertson100 ?), but in my opinion this approach goes against the idea of how a taxon checklist should be constructed.
Or at least from this image in  https://github.com/gbif/ipt/wiki/BestPracticesChecklists#checklist-data , I get the idea that a taxonID should be like a primary key (only one taxonID for each species present in the core A file),

The point is: Should I follow your example when I publish our similarly structured data?
In our case: many plant specimen samples collected at schools; associated to each one there will be a taxonomic identification (so, for now, this is just an occurrence dataset).
But there will also be specimen-attached enquiries made to neighbours asking them for information about vernacular names and human usages of that plant in a particular location and date. Also, an enquiry could have associated media (images and sound or video recordings, mostly stored at youtube/google drive).
What way should I publish it, so all that info -names and plant usages from every location- is alltogether visible in the taxon page?

It seems the DwC scheme does not fit perfectly here. Checklists with a taxon core feel mostly right, but the core taxon record ideally should indeed be unique and not duplicated if you know these are the same taxa. There is no location or time information in the taxon core, it is purely the taxon concept. Both the distribution and vernacular names extension allow to assert a geographical context, but they do not allow to cross reference each other. But you can say that this particular vernacular names is used in this location of sweden.
As for human use of a taxon I am not aware of such an extension. But if you/we would craft one I assume it would also have a geographical context. I have not seen you examples, but how would you use the distribution in that case? Where would you put the kind of use/usage?

Thanks @mdoering
The example is not mine but from @rukayaj (who opened this issue, although their nice dataset of nordic plant uses is already published).
I am interested in the issue because I want to publish similar data from another country (but based in oral enquiries rather than in published references).
I understand they used the distribution extension as a mean of locating each plant usage in Norway.
But, as distribution extension is supposed to be referred to a taxon, I guess that was the reason they decided to repeat taxons so many times (with different taxonids): that way they could refer different locations and usages of the "same" taxon.
As for the human usages of species, they used the Measurement or Facts extension: in "measurementType" column they put "usage" for all rows, and in "measurementValue" column they described things like "animal food", "veterinary", "medicine" and so on. This extension does not provide geographical context, but as they are linking to different taxonids each time, the aforementioned distribution extension solved the problem.
A couple of their examples:
https://www.gbif.org/species/164940120/verbatim
https://www.gbif.org/species/164940135/verbatim
As you can see, both are referred to the same species (Achillea millefolium).
They created different "taxon instances" (taxonIDs), so each one could be linked to a bunch of usages (measurements or facts + their distributions), vernacular names, and also literature references.
So, that is not a "taxon checklist" anymore. It is more like a repetition of taxons, so to me it makes more sense to (1) publish it as occurrence data as far as it can be somehow extended with plant usages and vernacular names as well (vernacularName is an available term in the occurrence data, but for plant usages I would need something else).
If we still prefer to (2) produce a real taxon checklist, one possibility which comes to my mind is extending that checklist data with occurrence data.
This sounds odd since both of them are normally used for publishing the core of a dataset (either species or ocurrences).
But when playing with our IPT, I noticed that after selecting "taxon checklist" as the core dataset, then "occurrence" becomes also available as an extension for that dataset.
Can you tell me what is the expected use case for that combination?
Again an example dataset would be great: I can't predict how that would look like in gbif portal (and I don't think it is OK to publish a dataset just for testing purposes, and deleting it later on).
I don't think (3) an event dataset would be appropiate. In the sense that enquiries about vernacular names or plant usages could be considered a kind of "event". That would be a "human sampling event" rather than a "biological sampling event", so I don't think this is one of the intended purposes of event datasets.

Yes that's right, I have duplicated taxa because I wanted essentially to publish a checklist of plant uses, rather than a checklist of plant names. Rather than showing "this particular vernacular name is used in this location of sweden", I wanted to show that this particular plant usage (which I have published as a MeasurementOrFact) is tied to this vernacularName and this location. I could have included locality data in the VernacularName extension as well as (or instead of) the Distribution extension.
I'm not sure that I've done the optimal thing here, as you say it's not all that easy to browse but on the other hand at least it is possible to reconstruct the original data without losing valuable information.
As it's possible for taxon X to appear in several checklists, I feel like perhaps it's not a big deal to have a single checklist with a taxon multiple times?  I am not really sure though, it feels like there isn't a very good fit with any of the different datatype options as they are.
If I had used the one core to many extensions model (the one in the diagram you posted), how would I have linked particular VernacularNames to particular MeasurementOrFacts? I don't think it's possible. Publishing it as occurrence data as you suggest is a more sensible option. But on the other hand, this is not really a list of occurrence data.
I have never heard of a checklist dataset which has an 'occurrence' extension. You could always try publishing it on gbif-uat and a test IPT and seeing what happens? If you have a one to one relationship between them (i.e. each taxon record has only one occurrence record) that might work? But as I say, I feel like these datasets aren't really occurrence data.

Thanks for your advice @rukayaj
I never tried that gbif-uat testing thing. How does that work?
A test IPT needs to have a public IP or could it run in localhost for that testing purposes?

GBIF has a testing environment https://www.gbif-uat.org/, your test IPT (which must have a public IP address) will publish to there. I think it does need to be publicly accessible, as far as i'm aware it's an exact mirror of production. So everything should work exactly the same. If you want you can use Norway's test IPT, just send me an email to<EMAIL_ADDRESS>

Sorry @mdoering I didn't notice your suggestion of a new extension. It could be something like this.

id
taxonID
vernacularNames (associated with uses and distribution)
vernacularNamesLanguages
taxonParts
usage
distribution
additional
reference

1
86da7f38-2fcb-4719-8433-c77b9744323c
tilg|molfoor-tilg|talg|tælg|tælgblom
NO|NO|NO|SV|SV
seedlings, germinated seeds
alcoholic drinks
Tromsø, Norway|Helgeland, Norway|Beitstad, Norway
{"treats": ["gout", "joint pain"], "application": "oral ingestion", "preparation": "boiled"}
Høeg OA. (1974). Planter og tradisjon: floraen i levende tale og tradisjon i Norge 1925-1973. ISBN<PHONE_NUMBER>.

2
86da7f38-2fcb-4719-8433-c77b9744323c
orme-blom|ormeblom-ving|største ormeblom
NO|NO|SV
leaves
poultice
Siberia
{"treats": ["wounds"], "application": "placed on wound for 5 days", "preparation"" "pounded and mashed"}
Jørgensen PM, Weidemann E, Fremstad E. (2016). Flora Norvegica av JE Gunnerus. På norsk og med kommentarer. ISBN 978-82-8322-077-3.

3
86da7f38-2fcb-4719-8433-c77b9744323c
cheek
FI
fruits
food
Finland

Jørgensen PM, Weidemann E, Fremstad E. (2016). Flora Norvegica av JE Gunnerus. På norsk og med kommentarer. ISBN 978-82-8322-077-3.

That should also give you an idea of what the data might look like. Feels a bit weird to be adding that when we already have an extension for vernacular names though. I'm also not sure how to treat all the additional information my dataset has, so I stuck it in as json.

Thanks, @rukayaj, @abubelinha,
this is an interesting discussion. It indeed seems that no constellation fits perfectly, and it will also depend on whether you are mostly valuing your own, local resource describing the contents of your data as completely as possible, or more focusing on the integration with other datasets.
If the information is taxon-focussed (and now echoing @mdoering), then starting from a taxon core does sound right, though indeed ideally each taxon should only appear in the taxon core once in that case. What you are trying to model, though, is a matrix where each taxon has x vernacular names and y uses, and where a specific use is tied to a (or several) specific vernacular name(s), and vice versa. The vernacular name extension (https://tools.gbif.org/dwca-validator/extension.do?id=gbif:VernacularName) does allow to specify a locality, so depending on your goal, you may not need the distribution extension. The use (via Extended Measurements Or Facts), though, would still not be cross-referenced to the vernacular names; as you say, this would not be possible - it is one of the limitations of the "star schema", as in the sketch above. As a result, the information is available in the verbatim data, but appears a bit more confusing at the higher level of a search result (https://www.gbif.org/species/search?q=Achillea millefolium&dataset_key=e1fd4493-a11a-438d-a27f-ca3ca5152f6b&origin=SOURCE&advanced=1). This is a trade-off - the 1:1 mapping allows to document that relationship for users of the source file and the verbatim data, but makes it more obscure on the integrated view.
Concerning extending checklist data with occurrence data: that model is by no means uncommon. It is basically a shift of the view, from a more object-focussed (occurrence core, names already included: identified specimens from collections, or species spotted in the wild at a certain time and place), to a taxon oriented one. that could be a floristic dataset, with records of concrete occurrences attached as evidence; a checklist of newly described taxa, referencing the specimens examined or deposited; or a taxonomically focused catalog  capturing distribution information and evidence of occurrences.

GBIF has a testing environment https://www.gbif-uat.org/, your test IPT will publish to there. I think it does need to be publicly accessible, as far as i'm aware it's an exact mirror of production. So everything should work exactly the same. If you want you can use Norway's test IPT, just send me an email to<EMAIL_ADDRESS>
Thanks a lot for your offer @rukayaj
I preferred to set up a test IPT server, since I need to map against a mysql which is not accesible to external machines.
But I am having troubles to get a simple test published. @ahahn-gbif : how long does it take a 100 records checklist to get visible in gbif-uat when registry health is reported as "Operational"?

Thanks for your feedback @ahahn-gbif.
Another option I thought of is to publish a version of the data on GBIF which doesn't include the data linkages, but to dump the original spreadsheet on a data archival system like Zenodo. One could then note in the metadata that some of the information has been lost through the limitations of the star schema, but that the full dataset is available for download separately on Zenodo, and provide the link. As you say though, it's a trade off so I suppose each data publisher will just decide for themselves how best to do it. Looking forward to seeing your dataset @abubelinha :)

@abubelinha, it should not take that long; the first step would be for it to turn up in the uat registry though; after that, indexing can pick it up. At the moment, the last dataset registration there shows to be from July 13th, but that is an internal one.
Could you give us a few more details? Which organization did you register the dataset against in the IPT, or do you still need the token for one of the test organizations to do so? What dataset title and organization should we look out for? Please feel free to write to<EMAIL_ADDRESS>if any of the information you plan to send is security-relevant.

Sorry @ahahn-gbif , my fault.
Some ipt configuration steps are still missing: I forgot about organization token/password (thought it was not needed for test installations).

@ahahn-gbif one question about this warning when you click the "Register" resource button:

I do not understand why it says  "deletion from the GBIF Index is automatic, but cannot be undone".
Isn't that a mistake?
We are not deleting anything when we register a new resource. We are doing the opposit thing. Am I wrong?

Hi @abubelinha, I agree the phrasing is a bit confusing in the context.
The intention is to avoid a situation where a resource is registered somewhat prematurely, maybe just to get a first impression of what the public version would look like, but then deleted again to withdraw the now public version, in order to make some more changes before attempting to register it again for good. And that is not possible through the IPT - a dataset can be registered, and it can be deleted again if need be, but that deletion is then irreversible, at least from within the IPT.

The intention is to avoid a situation where a resource is registered somewhat prematurely - a dataset can be registered, and it can be deleted again if need be, but that deletion is then irreversible, at least from within the IPT.

Thanks @ahahn-gbif
So the warning is mostly for not loosing the url of the resource.
But if you don't mind changing that url, you can always publish a resource, delete it (if there is a problem), and republish it later again as a new resource (with a different url). Correct?

Technically correct, @abubelinha. This would mean starting the configuration over on the IPT, though - it will not allow you to register the same setup a second time. If it becomes necessary, the earlier registration can be manually 'resurrected', but that requires helpdesk intervention.
We would generally recommend to keep dataset IDs as stable as possible, not least because external data (usage reports, data citations) will likely be referencing them. That may not be so much of an issue early on, of course.

@ahahn-gbif, can you please summarise any changes are needed or should be considered in the IPT in light of this discussion?
Or can this be closed please?


Remove version: hrzlgnm.mdns-browser version 0.9.1

This version is older than what has been set in max-versions-to-keep by the publisher.
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/hrzlgnm/mdns-browser/releases/download/mdns-browser-v0.9.1/mdns-browser_0.9.1_x64-setup.exe
OK
(Automated message - build 923)


Feature request: register custom annotation that can mutate an annotated command being built

Sometimes you want to modify a large group of commands the same way, for example, adding a flag or generating certain descriptions. For this task it would be very useful to be able to modify the command built by the annotation parser, right before it is registered.
Pseudocode:
parser.registerCommandBuilder(MyAnnotation.class, (annotation, builder) -> {
    return builder.flag(CommandFlag.newBuilder("flag").build());
});

@MyAnnotation
@CommandMethod("command test")
private void commandTest(
   etc...
) {
}

If possible, there should also be a way to mutate the arguments with custom annotations.

If possible, there should also be a way to mutate the arguments with custom annotations.

What do you mean? Can you give an example? The arguments aren't mutable, and you can already use annotations to modify the parser to customise the argument depending on annotations.

If possible, there should also be a way to mutate the arguments with custom annotations.

What do you mean? Can you give an example? The arguments aren't mutable, and you can already use annotations to modify the parser to customise the argument depending on annotations.

I don't particularly have an example of what I suggested. Maybe I'm mixing stuff. Do you happen to have an example of using custom annotations to customize the argument parser?


Mask & Filters

I'm quite new to Arduino and even newer to CANBUS, I've uploaded the code and got everything to work perfectly after a few tweeks (setting clock speed, can speed, etc).
now to the next step, I don't understand the filtering. for example, how would I code it to only show me a specific CAN i.d?
Just a little snippet of i.d's below but how would I set a filter to only show me 0x34F?
0x35C
0x34F
0x349
0x336
0x35E

Did you resolve it? I have the same problem.

You done it? I also need a filter


Swernli/merge/feature/decomp

This PR merges the working branch from #421 back into the feature/decomp branch so we can collect some more changes there. See that PR for the change history, which includes updates from CR feedback and several merges from main.

/azp run

/azp run


chore: remove unused lemmas about bit0/1

Deletions:

bit0_mul
mul_bit0
bit1_mul
mul_bit1
Polynomial.natDegree_bit0
Polynomial.natDegree_bit1
Polynomial.bit0_comp
Polynomial.bit1_comp
Tropical.bit0

Looks reasonable, thanks!
maintainer merge

Actually: #14644 makes a different choice on some of these lemmas, and rewrites them instead. I haven't weighed that very carefully. Some of the later lemmas certainly do exist in non-bit0/1 form already.

Let's wait for #14745 to land first, and then merge master into this PR.

#14745 has landed: marking as awaiting-author


fix: change startIndex option type from string to number

startIndex should be an 0-indexed integer. But as it is was expecting a string which is incorrect and caused a lot of confusion, preventing compilation and requiring odd workarounds.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #1350 🙏

this file should not be manually modified, as it is generated from the Discovery doc (with the command npm run types).

@alvarowolfx Ah got it, I will close this PR then but the issue remains open. I am happy to contribute a solution, but not sure where to find/update the relevant Discovery Doc definition. If you want to point me to it I can attempt. Or someone who knows the procedure might be better suited to make the quick fix.

Another thing to consider is that the startIndex attribute expects an uint64 and that can't be represented with a integer in Javascript, which holds at most 2^53 - 1. So users would have to use a BigInt and convert to string anyway.


Make array broadcasts node aware

Very little change was required to get this working at a basic level, because array bcasts are based on group broadcasts. Main changes were adding array broadcast message types (one for the initial broadcast, and one for forwarding the broadcast to each group on a node), stopping CkArray from modifying the message, and adding a nokeep broadcast to CkArray.

The change is now fixed and ready for everything but ZC and BigSim. I will need to work with Nitin to figure out what is going wrong with ZC, and BigSim is now being deprecated.

The change is now fixed and ready for everything but ZC and BigSim. I will need to work with Nitin to figure out what is going wrong with ZC, and BigSim is now being deprecated.

I've pushed the fixes to the ZC implementation based on the changed semantics for chare array broadcast. These fixes include the following changes:

Not resetting the message type to ForBocMsg (which was required previously in order to reuse the same message)
For broadcast from non-zero root, by default recvBroadcast was called. This has been modified to called the expedited and noKeep versions of recvBroadcast based on entry method options.

Updated to master that removes BigSim. @epmikida is this ready for review?

Updated to master that removes BigSim. @epmikida is this ready for review?

Yes.

There is a crash in MPI-smp during automated testing in examples/charm++/zerocopy/entry_method_post_api/prereg, which is a p2p message test. It seems spurious, so I'll restart this build.
make[5]: Entering directory '/home/travis/build/UIUC-PPL/charm/mpi-linux-x86_64-smp/examples/charm++/zerocopy/entry_method_post_api/prereg'
make -C simpleZeroCopy test OPTS='-optimize -production  -g -Werror=vla' TESTOPTS='+setcpuaffinity'
make[6]: Entering directory '/home/travis/build/UIUC-PPL/charm/mpi-linux-x86_64-smp/examples/charm++/zerocopy/entry_method_post_api/prereg/simpleZeroCopy'
../../../../../../bin/testrun  +p4 ./simpleZeroCopy 32 +balancer RotateLB +setcpuaffinity
Running on 4 processors:  ./simpleZeroCopy 32 +balancer RotateLB +setcpuaffinity
charmrun>  /usr/bin/setarch x86_64 -R  mpirun -np 4  ./simpleZeroCopy 32 +balancer RotateLB +setcpuaffinity
Charm++> Running on MPI version: 3.1
Charm++> level of thread support used: MPI_THREAD_FUNNELED (desired: MPI_THREAD_FUNNELED)
Charm++> Running in SMP mode: 4 processes, 1 worker threads (PEs) + 1 comm threads per process, 4 PEs total
Charm++> The comm. thread both sends and receives messages
Converse/Charm++ Commit ID: 0fa61e4
CharmLB> Load balancer assumes all CPUs are same.
Charm++> cpu affinity enabled.
Charm++> Running on 1 hosts (1 sockets x 1 cores x 2 PUs = 2-way SMP)
Charm++> cpu topology info is gathered in 0.078 seconds.
Charm++> Warning: the number of SMP threads (8) is greater than the number of physical cores (2), so threads will sleep while idling. Use +CmiSpinOnIdle or +CmiSleepOnIdle to control this directly.
WARNING: Multiple PEs assigned to same core, recommend adjusting processor affinity or passing +CmiSleepOnIdle to reduce interference.
CharmLB> RotateLB created.
send: completed
Fatal error in MPI_Irecv: Invalid buffer pointer, error stack:
MPI_Irecv(153): MPI_Irecv(buf=(nil), count=840, MPI_BYTE, src=0, tag=1378, comm=0x84000002, request=0x7ffff003b910) failed
MPI_Irecv(118): Null buffer pointer

I see some potential race conditions with the ZC changes which may need to be addressed.

I've modified that code to not write into the envelope and just use the GroupID instead.

I see some potential race conditions with the ZC changes which may need to be addressed.

I've modified that code to not write into the envelope and just use the GroupID instead.

Ah I didn't realize those calls were only on one PE. But I agree, I think the consistency of not modifying the message is a good change to make anyways.


Send Appropriate Response on Error

Users can now place custom error pages in static/error/<error_code>.html
e.g. static/error/404.html
If no custom page is found, a default plaintext describing the error is served.

I think the file looked up should be templates/404.html but that means we need to ensure it's going to be generated, like the sitemaps etc

Yes, so nginx/netlify etc can display the 404 if there is one. I'd prefer templates/404.html, I don't think there are other error codes to be added afaik.
Also if you can change the branch to do the PR on the next branch that would be great!

Maybe something like 503.html ?

would 503 be used for static sites?

Note that I merged a PR to move from iron to actix-web as iron is unmaintained so this PR would need to be rewritten. I still want it in though

Peeking at the actix-web middleware intro, seems like the implementation would be virtually identical. Mostly the work would be to switch out the types for their actix counterparts.
As an aside, I host my gutenberg site on s3, and I only remember the option to provide a single generic "error" document. Wonder if we could emulate both behaviours by checking for a status code file, then falling back to check for a generic error file.

Anyone wants to pick this up?

I can probably take a look over the weekend if someone else doesn't get to it first.

I have something compiling now, but need to test and polish. Not sure how best to handle the hand-off for the PR. Close and open a new one?

Yes, i'll close this one as I don't think any part of the code can be re-used


Docker external uri for redirection

Setup:
IS4 + ASPNET Identity + MVC Client

https://mybestdomain.best.com - hosted docker container IS4 with IssuerUri = "https://mybestdomain.best.com" // hostname best.is4
https://mysecondbestdomain.best.com - hosted docker container web mvc client // hostname best.mvc
2.1 If I set Authority = "https://mybestdomain.best.com" - I get error cant open discover doc
2.2 If I set Authority = "best.is4" ( my docker host name ) - I get redirect to best.is4/connect/*

How to change redirect from https://DOCKERHOSTNAME/connect/authorize to https://BESTDOMAIN/connect/authorize
I found answer SO
// It is important this matches the actual URL of your identity server, not the Docker internal URL
options.Authority = "http://localhost:5000";

if (Environment.IsDevelopment())
{
    // This will allow the container to reach the discovery endpoint
    options.MetadataAddress = "http://identityserverwithaspnetidentity/.well-known/openid-configuration";
    options.RequireHttpsMetadata = false;

    options.Events.OnRedirectToIdentityProvider = context =>
    {
        // Intercept the redirection so the browser navigates to the right URL in your host
        context.ProtocolMessage.IssuerAddress = "http://localhost:5000/connect/authorize";
        return Task.CompletedTask;
    };
}

I changed MetadataAddress to docker-hostname and ProtocolMessage.IssuerAddress to public domain uri and it works!
But its tweak not primary solution
SO
https://stackoverflow.com/questions/43911536/how-can-i-use-identityserver4-from-inside-and-outside-a-docker-machine
https://stackoverflow.com/questions/59250716/identity-server-4-with-the-docker-compose-redirects-to-login-page-after-connect
Issues
is4 4592 I don't using reverse proxy or I have no idea how to do it.

Also I fixed some new problem with proxy
My MVC have redirect mybest.domain.com:433 to mybest.domain.com:6666 in Docker
`
                options.Events.OnRedirectToIdentityProvider = context =>
                {
                    // Intercept the redirection so the browser navigates to the right URL in your host
                    context.ProtocolMessage.IssuerAddress = "https://sso-mybest.domain.com/connect/authorize";
                    context.ProtocolMessage.RedirectUri = "https://mybest.domain.com/signin-oidc";
                    return Task.CompletedTask;
                };

`
Can I replace this code some primary solution?

I have similar issue when using a JS client ( using angular-oauth2-oidc package) and IS4 hosted on external docker. My issue is when returning from IS4 authentication I got Cors blocked.


Problem : Geolocator gives different coordinates at the same location

🔙 Regression

Old (and correct) behavior
Current behavior
Reproduction steps
Configuration
Version: 1.x
Platform:

[ ] :iphone: iOS
[x] :robot: Android

Geolocator gives different coordinates at the same location


IP2TOR+LestEncrypt: Check if working

It looks like that sometimes it takes some time until duckdns is updated on a new IP (I mean when the local DNS resolution with TTL catches on to a change). So there should be a warning on services that use LetsEncrypt - that its waiting on updating DNS.
In general a better "Test" if everything is running (HTTPS delivering 'a page', etc) would be great.

Now integrated to checks into blitz.subscriptions.letsencrypt.py:

domain resolves to the wanted IP
https gives a 200 response on a testport given

The menu elements of LNbits and BTCPayServer check now on this when called and show warning if one of the tests fail.

Tests looking good. Closing for v1.6.1 release.


File Upload not validating in version 2 docs

Description
After migrating to v2, the file upload stopped working. I then went through the docs and copied the exact same code as in the Single File Upload Doc example, and the validation keeps failing on the server side.  I don't know how to solve this if even the example from the official superform docs doesn't work..
here are the versions of the dependencies i am using in my stack:

schema & server side
// NOTE: Import fail from Superforms, not from @sveltejs/kit!
import { superValidate, fail, message } from 'sveltekit-superforms';
import { zod } from 'sveltekit-superforms/adapters';
import { z } from 'zod';

const schema = z.object({
  image: z
    .instanceof(File, { message: 'Please upload a file.'})
    .refine((f) => f.size < 100_000, 'Max 100 kB upload size.')
});

export const load = async () => {
  return {
    form: await superValidate(zod(schema))
  }
};

export const actions = {
  default: async ({ request }) => {
    const form = await superValidate(request, zod(schema));

    if (!form.valid) {
      return fail(400, { form });
    }

    // TODO: Do something with the image
    console.log(form.data.image);

    return message(form, 'You have uploaded a valid file!');
  }
};

frontend:
<script lang="ts">
  import { superForm, fileProxy } from 'sveltekit-superforms'
  import { zodClient } from 'sveltekit-superforms/adapters'
  import { schema } from './schema.js'

  let { data } = $props();

  const { form, enhance, errors } = superForm(data.form, {
    validators: zodClient(schema)
  })

  const file = fileProxy(form, 'image')
</script>

<form method="POST" enctype="multipart/form-data" use:enhance>
  <input
    type="file"
    name="image"
    accept="image/png, image/jpeg"
    bind:files={$file}
  />
  {#if $errors.image}<span>{$errors.image}</span>{/if}
  <button>Submit</button>
</form>

Using superdebug, this is when i chose the file:

and when i submit:

After 3 days of trial and error I found the issue - I had
+  define: {
+    SUPERFORMS_LEGACY: true
+  }


Explore moving to native fetch() API

The problem
We are using axios library for syntactic sugar for XHR-like things and ensuring the same code can be used for client-side and server side, but as of today fetch() API is available with all browsers we support.
The solution
Using node-fetch, which is no-op for browsers and has implementation for node.js.
Acceptance Criteria

[ ] A PoC code that demonstrates how we can replace axios with native fetch() API

@asudoh @RobertaJHahn moving this out a few sprints, web component work should be priority.

@RobertaJHahn moving this out out more.

Will stick with Axios.  Closing


emulator to remember the previously entered text

Can we have the emulator "remember" recently entered text? like maybe using the arrow up key to issue the previously enter text. just trying to same some keystroke here.. thanks..

Good suggestion. We've added this to our backlog, thanks!


Switch to TypeScript

It's best for libs, and means we don't have to manually manage the definition file

It's best for libs, and means we don't have to manually manage the definition file

@mattyork is that opinion for fact?


[Reminder Notification - Design Implementation] Watering Notification on Site

Quick summary:
We want to notify users in the summer months (hot months for southern hemisphere) that they should water their adopted new trees that may need water more frequently. The solution is to have a notification be present when they open the site (both mobile and desktop).
Issues:
This is a new design that has never been built on the app. The design is essentially the same with mobile being attached to the bottom of the screen and desktop floating.
Get Started:
Get started with our onboarding, https://github.com/waterthetrees/waterthetrees/wiki/Onboarding and read through the epic for relevant details of this ticket. https://github.com/waterthetrees/wtt_front/issues/433
Resources:

Design Figma: https://www.figma.com/file/5C3v1LUNwMPQy9JOgnKEtr/Water-the-Trees?node-id=1355%3A9085

Design of Banner:

the "learn more" will open a new tab to this page https://waterthetrees.com/treecare


submission status report

aggregated status of all the results
aggregated status of all the submissions count

polling agent:

when turned into a A+ project
open IntelliJ and the plugin is installed and the project is A+ project (startup activity)
when submission form is opened
a separate refresh button (poll manually)
if there are pending tasks incremental check interval increase (+15 +30 +60... )

https://apluslms.slack.com/archives/CSDV9CA31/p1591717757037500

https://github.com/JetBrains/intellij-community/blob/78a91459c4d8454b2d5df36ade108807ef772373/platform/util/src/com/intellij/util/ConcurrencyUtil.java#L21

https://plus.cs.aalto.fi/api/v2/courses/163/points/19457/

openAPI concurrency stuff?

objects are more into knowing how to update themselves?

Submission and SubmissionResult


demo 001 - with message toast new properties

hi @oblomov-dev
does this take so long to build downport ?

Yes something seems not to work, try to check it later. But maybe it's more a GitHub issue and has nothing to do with the code 🤷♂

thanks, deactivated the check temporary, now it works


[Suggestion] Link to Issues from CurseForge

A link to the Issues page from CurseForge where Files and Source are displayed would be great, thanks!

I took a look at the main page and this doesn't appear to be resolved.
A majority of mods provide a direct link to issues, it would be appreciated if this mod could do the same.

Sorry that, added.


sliding animation paus while hovering

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

i would like to work on this issue

what if we had this feature?

with sliding activated only by mouse drag

Great, Go for it! @Yh010


💚 Fix codecov upload

It's been broken for sometime as codecov.msbuild was deprecated and
archived ages ago. I've just been giving it the attention it deserves.

Merging for now, because the OIDC token situation is broken for more than just myself.


Android Q issue

Hi, there seems to be something wrong with android Q. it says no process when i click the connect button. Have you tested it with android 10?

@mj700 Is this issue? schwabe/ics-openvpn#1084 or can you post log files and errors or any images that describe the error?

Found this exception:
Cannot run program "/data/user/0/com.buzz.vpn/cache/c_pie_openvpn.arm64-v8a": error=13, Permission denied
from
VpnStatus.logException("Error reading from output of OpenVPN process", e);
at
de/blinkt/openvpn/core/OpenVPNThread.java:173

So, at least for me, it's the same problem.

@yurik94 @mj700  Please refer to this
https://github.com/schwabe/ics-openvpn/issues/1179#issuecomment-709324457
or
https://stackoverflow.com/a/64373156/7173141
I hope that this will resolve your issue.


Linker flags for sbcl on Darwin

@jwiegley

@joelteon This had no effect on my system, the bug is the same.  When I try to build sbcl I get:
cc -g -Wall -O2 -fdollars-in-identifiers -mmacosx-version-min=10.4 -arch x86_64 -fno-omit-frame-pointer -pagezero_size 0x100000 -I../src/runtime   determine-endianness.c  -lSystem -lc -ldl -o determine-endianness
ld: -rpath can only be used when targeting Mac OS X 10.5 or later

What sbcl version are you using?


Update version to 1.2.85-1 on production

Update version to 1.2.85-1

Hey! I see that you made changes to our Form component. Make sure to update the docs in FORMS.md accordingly. Cheers!


Downloadable data

Where can I download csv timeline data of new daily cases by county? Preferrably from 13.03.2020

Found already :D


[Method 1] Is it required to use non China mainland apple account to join the the AI waitlist?

Additional Information
I have run all steps in method 1 and I can enter the AI settings panel in preference, however when I click join there's no response from the UI.
I've already used English(US) and switched region to US but I'm still using China region apple account on the device.

yes

Such logic is controlled by Apple Server and may change over time.
I guess the answer is yes for now.
Close due to out of scope of this repo.

Additional Information
I have run all steps in method 1 and I can enter the AI settings panel in preference, however when I click join there's no response from the UI. I've already used English(US) and switched region to US but I'm still using China region apple account on the device.

Yes.
non-Chinese mainland Apple Account is required to join the Apple Intelligence waitlist.
Actually, I have a United States Apple Account, the system region is set as United States, but I'm using English (UK) as my system language (except Siri), and I couldn't join the waitlist until I changed my system language as English (US).


Error Parsing curl Command Without URL Quotes: Found "variable_assignment" AST Node

When using curlconverter to convert a curl command with a URL containing query parameters that is not wrapped in quotes, the following error is thrown:
Error: found "variable_assignment" AST node, only "command", "pipeline" or "redirected_statement" are supported
curl -X GET https://example-mission.com/missions?app=example&channel=3

However, when the same URL is wrapped in quotes, the error does not occur and the conversion works as expected:
curl -X GET "https://example-mission.com/missions?app=example&channel=3"

It seems like the parser is misinterpreting the query parameters in unquoted URLs as some form of variable assignment. This behavior may cause confusion for users, as curl generally does not require URLs to be wrapped in quotes unless special characters are involved.

+1, I'm experiencing the same issue

& is a special character in bash/shell. If you try running your command, it won't do what you think.
~$ curl -vvv -X GET http://example.com/missions?app=example&channel=3
[3] 34048
[2]   Done                    curl -vvv -X GET https://example.com/missions?app=example

as you can see it actually makes a request for https://example.com/missions?app=example not http://example.com/missions?app=example&channel=3 and the [3] 34048 stuff is Bash job control stuff.


Week 4

Collectively learn website design basics and languages (HTML, CSS, GH pages).
Being led by @karegapauline.
Prepare and share the whole lesson plan and slides with the group
October 17th deadline
Come up with our roadmap

Licences: what type of license suits our project CC-BY, CC0, MIT??
Make sure our README is very detailed, it should be self explanatory


Add alt_aliases to CanonicalAliasEventContent

https://github.com/matrix-org/matrix-doc/pull/2432/files#diff-a9ccbaea3cb53ab4c6147d0a7e7f10c2R61

So adding this wouldn't be complicated. But it's still not released yet and also right now adding any fields is a breaking change. I think we should probably transition to (generated) builders for every event content struct + making them #[non_exhaustive]. I guess we could start a branch with these not-yet-released spec things though.


Issue 80

This is a factor of cluster and data to remove the dependency on config. Unfortunately, the most recent code for data is sitting in #46, which hasn't been pulled in master.
@oxhead please merge #46 into #80 and assign the PR to me.

I got it covered.  Thanks.


Can I use this Kyma Runtime

Hello,
Can we use this SDK on Kyma Runtime?

Good morning @SanthoshCheekoti,
thanks for your question. We are currently working on this and we started already the investigation on the topic. We plan to have everything tested and documented by the end of Q3. The SDK will support the k8s based runtimes Kyma and Gardener in the near future. So currently it depends what you want to do and in which timeline.
We see that a lot is working but there are some blind spots:

The SDK works for cloud destinations as long as the client credentials (VCAP_SERVICES in cloud foundry) are mounted as secrets in the k8s cluster @tomfrenken - clould you comment on this and perhaps give one guide.
For OnPremise connection a connectivity proxy is needed which needs to be configured manually on k8s runtimes in contrast to cloud foundry where this is done automatically when subscribing to the connectivity service. In principle this should work, but we have not done an E2E test.
We have not tested the E2E flow of authentication and token handling.
The whole logic like the typed client and convenience in request creation is not affected by the runtime.

I hope this answer your question.
Best
Frank

Thank you. It answers my question.

Hey @SanthoshCheekoti maybe to add to Frank's answer, if you've created secrets containing the VCAP_SERVICES, you have to mount them in a very specific path, e.g. following this pattern:
volumeMounts:
          - name: <volume-name>
            mountPath: "/etc/secrets/sapcp/<service>/<service-name>"
            readOnly: true

for example, the destination service would look like this:
volumeMounts:
          - name: my-secret-volume
            mountPath: "/etc/secrets/sapcp/destination/my-destination-service"
            readOnly: true

as mentioned we will have a more in-depth guide on how to use the SDK in k8s environments in the upcoming week(s), where we will also discuss migration from CloudFoundry applications to k8s.


Unable to use software raid as "state" device

RancherOS Version: (ros os version): 0.8.1
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.): baremetal
My install of RancherOS is trying to use a mdadm software raid setup for a state drive, I'm aware of the rancher.state.dev=UUID=123abc option in the kernel flags, but it doesn't seem to even try to load the software raid when the kernel starts.
The devices in /dev/md* are missing unless I manually run mdadm --assemble --scan but I can't do that on boot since I don't have a writeable file system to perform it in cloud-config.
Is there anyway to enable this through either kernel args, grub.cfg or changing the kernel?
Thanks

Turns out this is possible, after some back and forth with @SvenDowideit on Twitter, I was able to piece together a working solution:
https://github.com/sthulb/posts/blob/master/rancher-raid.md


small refactoring in lb to start to unravel managed sg

Coverage decreased (-0.1%) to 23.293% when pulling 7bb2c8c33320e3a5715f9d253d939469a80cbfc2 on refactor-lb into a6565ea427473c41e004983e9b9ba12f5b0617d1 on master.


Provide better way to change drag axis

Right now XZ is assumed for dragging by default. Provide button to let users choose the axis/plane to drag along instead.  This is better for mobile devices too.

Doesn't one of the hotkeys do this, ala Trenchbroom?

Doesn't one of the hotkeys do this, ala Trenchbroom?

You can hold Alt, but it only allows moving on the Y axis: https://github.com/blackears/cyclopsLevelBuilder/issues/48#issuecomment-1546838407

Yeah, I'm having trouble moving a face along the Z axis without it also going off into the X axis, it would be useful if there was a way to just lock to a single axis (I dont think there is?)

I've just merged code that provides a gizmo that should make it easier to move you selection along a particular axis.


Automated tests

Implementing story:
https://github.com/griddynamics/nexus-replication-plugin/issues/16
This PR overrides:
https://github.com/griddynamics/nexus-replication-plugin/pull/24
Please review and merge if possible.

@gd-tmagrys the first 2 commits here are outdated, is that right?

Yes, because this work was created on top of #29 .
I'll rebase it on master.

@ctapobep, I rebased work regarding this PR on master. Feel free to review.

Hm. But I see classes like JerseyClientFactory added within this commit, so it's not just tests

@ctapobep, I hope that I fix things you've mentioned in my last commit.


i18n message example

Spoke to @stefanpenner awhile back about this, and sticking to my word to investigate it.
Will poke at it at some point this week to see how we'll be able to leverage the services of ember-intl || ember-i18n to resolve translation keys and handle formatting with params.

Yeah that was on my todo list as well. I don't think it should be too difficult.... all error messages are stored in app/validators/messages so porting that to ember-intl or ember-i18n should be pretty straight forward

Spoke to @stefanpenner awhile back about this, and

:+1:

@offirgolan I have the sense it will not be that straight forward because both ember intl and i18n require a container lookup and ember intl handles ICU message syntax where your examples use Ember.String.fmt.

Accessing the messages in the current context is also done via a container lookup. As for the ICU message syntax, that would be something that we would both need to explore.

A few ideas:

what do we think about implementing this as a validation-messages service? Either way, we'll need to convert the utils/messages.js file to be an Ember Object so we can try to inject the i18n/intl service and pull from them if present. this would be nice because the defaults would be cached in the singleton service.

we'll need to invalidate the messages on locale change

I'm thinking each flavor of library will need to handle converting to their desired format of string formatting. this library uses String.fmt, i18n uses curlies and intl uses ICU.

What do y'all think?

I'm thinking each flavor of library will need to handle converting to their desired format of string formatting. this library uses String.fmt, i18n uses curlies and intl uses ICU.

I think we may need to step away from String.fmt since it seems like it's being deprecated. We can always move over to using curlies or ICU formatted strings and then create two separate files and choose based on which library is being used.


Veer Prereg Updates

continuation of https://github.com/CenterForOpenScience/osf.io/pull/5289

Took a quick glance at the new form locally; looks good! :octopus:


Only add derived data parameter if it's set, and command is update, build…

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
This pull request fixes #15145
Description
Changed the parameter checking in the carthage action, to only add derived_data parameter if it is set, and the command isn't archive.

@googlebot I signed it!

I this we could test that even though derived_data parameter is set, it is not taken into account when a command parameter is set to archive.
carthage_spec.rb

context "when specify derived_data" do
    context "when command is archive" do
        let(:command) { 'archive' }
        # TODO: Check if `derived_data` is set, it is not taken into account for `archive` command.
        expect(result).to eq("carthage archive")
    end
end

Feel free to ping me if something's not clear 😉

Thanks for the comments @mollyIV
I'll be sure to give it a go, later today 👍

BTW I think it would be helpful to add the tests against supported parameters for each command. May be a good candidate for a separate issue 😉 Thoughts @janpio ?

I agree with both. Would be good to have, but also we don't want to ask @iOSGeekster to implement that just because he jumped in to fix something else - so new issue would be best. And if @iOSGeekster now feels motivated, feel free to have a go at it of course :)

Would be good to have, but also we don't want to ask @iOSGeekster to implement that just because he jumped in to fix something else - so new issue would be best.

Absolutely 👍
FYI @janpio @iOSGeekster I created a separate issue:  https://github.com/fastlane/fastlane/issues/15173 🎉

@mollyIV I've now added a test case which should cover the issue.

Looks good 🎉
I did some manual testing too - regression tests and archive command tests 👍
Thanks again for fixing it 🙇

Awesome, no problem. And thanks for the guidance 🙌


failing on homebrew

brew install deepsourcelabs/cli/deepsource

==> Fetching deepsourcelabs/cli/deepsource
==> Downloading https://github.com/DeepSourceCorp/cli/releases/download/v0.7.0/deepsource_0.7.0_darwin_arm64
curl: (22) The requested URL returned error: 404

Error: deepsource: Failed to download resource "deepsource"
Download failed: https://github.com/DeepSourceCorp/cli/releases/download/v0.7.0/deepsource_0.7.0_darwin_arm64.tar.gz

Is this tool actually maintained?

Hey @Goldziher. Apologies for the delay.
This is now fixed.


chore(gui-client): log uptime and run ID when a window is shown

I want to make sure there aren't bugs that prevent the GUI clients from reaching high uptime, so this makes it easy to check the uptime with e.g. tail -f. The run IDs would let us re-construct when the app restarts.
"chore" doesn't seem right but it's not a user-facing feature, test, doc, fix, or refactor, either.

Yeah it could go in connlib, although since the clients can stop and restart connlib, if it's instantiated in both places then we can also watch for client instability in the sign-out-sign-in cycle.
The use case is to quickly prove that the client was restarted or not restarted as intended during manual tests or debugging:

If I leave the client up all weekend, seeing 4 days of uptime shows that it can idle despite the computer being locked and time passing, and that I didn't restart it and then forget that I restarted it.
Gabi was helping me debug something with DNS and asked if I had restarted the client before or after a certain step. I found it from the logs but it took a couple minutes to be sure I hadn't missed anything. (In that case this PR should also print a human-readable calendar time, and not just the Duration in seconds)
When debugging logs, it would be easier to refer to a specific run by a UUID we can dump and then copy-paste quickly instead of looking up the start time and referring to it as "The run starting at T17:48:00Z".

This could all be derived from the logs, but I don't have a one-click command to do that yet, and it's lost if the logs are cleared.


Question: Defer login until user click button if no active session

Hi,
Is there a simple way to defer the authorize login flow until user clicks a button if there is no active session?
So if there is an active session then load the user profile but if not then show a login button and then auth.signOn() on button click. I have this working if i log out and then log back in without reloading the page but would like the option when user first navigates the page to give them some info and a login button rather than redirect to identity provider for credentials
Thanks, Ed

found it, apologies autoSignIn={false}

found it, apologies autoSignIn={false}


Make pcf_vms, pcf_templates and pcf_disk configurable for vsphere install

In the original config-opsdir task,   bosh_vm_folder, bosh_template_folder and bosh_disk_path were hard-coded to "pcf_vms", "pcf_templates" and "pcf_disk". This is not ideal when multiple teams work on separate PCF instances in the same vCenter. They would prefer to have their PCF VMs stored in different VM folders. This change makes pcf_vms, pcf_templates and pcf_disk configurable for PCF vsphere install pipelines.
I've committed my change to "master". Please let me know if this is the right branch to merge as I see the "install-pcf-vsphere" branch also needs this fix.

Thanks!


Add iframeHeight and iframeWidth properties to pageInfo

I had originally contributed to adding the pageInfo object to iframe-resizer. It turns out that in most case, you also need to know the height of the iframe to properly calculate absolute position of overlays in the iframe. Therefore I added iframeHeight and iframeWidth to the mix.

Sounds good, super busy atm. Will try and take a look in the next couple of
weeks.
Thanks,
D.
On Wed, Feb 3, 2016 at 8:11 PM, Pierre Olivier Martel <
<EMAIL_ADDRESS>wrote:

I had originally contributed to adding the pageInfo object to
iframe-resizer. It turns out that in most case, you also need to know the
height of the iframe to properly calculate absolute position of overlays in
the iframe. Therefore I added iframeHeight and iframeWidth to the mix.
You can view, comment on, or merge this pull request online at:
https://github.com/davidjbradshaw/iframe-resizer/pull/314
Commit Summary

Added iframeHeight and iframeWidth properties to pageInfo
Typo fix

File Changes

M README.md
https://github.com/davidjbradshaw/iframe-resizer/pull/314/files#diff-0
(2)
M js/iframeResizer.contentWindow.min.js
https://github.com/davidjbradshaw/iframe-resizer/pull/314/files#diff-1
(4)
M js/iframeResizer.js
https://github.com/davidjbradshaw/iframe-resizer/pull/314/files#diff-2
(2)
M js/iframeResizer.map
https://github.com/davidjbradshaw/iframe-resizer/pull/314/files#diff-3
(2)
M js/iframeResizer.min.js
https://github.com/davidjbradshaw/iframe-resizer/pull/314/files#diff-4
(6)
M spec/childSpec.js
https://github.com/davidjbradshaw/iframe-resizer/pull/314/files#diff-5
(4)
M src/iframeResizer.js
https://github.com/davidjbradshaw/iframe-resizer/pull/314/files#diff-6
(2)

Patch Links:

https://github.com/davidjbradshaw/iframe-resizer/pull/314.patch
https://github.com/davidjbradshaw/iframe-resizer/pull/314.diff

—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/iframe-resizer/pull/314.

--
David J. Bradshaw )'<EMAIL_ADDRESS>'The people we think are a little weird are the ones who change the world'

Simon Sinek

Thanks and sorry for the delay. Released in v3.5.2

Awesome! Thanks for the follow-up David.


"System.baseURL = ..." style configuration doesn't work

Much of the SystemJS configuration documentation suggests configuring system like
System.baseURL = '/js/lib/';
System.paths['jquery'] = '//code.jquery.com/jquery-1.10.2.min.js';

It isn't possible to use this same configuration in the file passed to builder.loadConfig(), because the System object isn't a loader, just a shim with a .config() method.
This means that after following the SystemJS documentation, you have to rewrite your configuration to work with the builder.

Great point - yes we should support these configuration styles too.

Ok this has been resolved. Thanks for posting.


boolean values update issues

With Postgresql database, update are not handled correctly in some cases.
Working :

old value : false => new value : true
old value : true => new value : NULL
old value : NULL => new value : true
old value : NULL => new value : false

Not working :

old value : true => new value : false
old value : false => new value : NULL

After some search I found that there is an issue in the update(array $row, array $current) function of src/Loaders/InsertUpdate.php file.
This section use loose comparison between old values array and new values array :
if ($row == array_intersect_key($current, $row)) {             return;         }
The $current array return boolean from database as a PHP Boolean while the $row array take string('true', 't', 'false', 'f') or NULL value to define booleans.
This prevent database update when it should be.

Thank you @tristanbsn we are going to look at it.

@tristanbsn could you clarify this point please, I am not sure to understand :bow:

... while the $row array take string('true', 't', 'false', 'f') or NULL value to define booleans.

@ecourtial In my case I use a transformer with a callback that take a date as source and transform it as string 'false' if date < current_date, 'true' otherwise :
return (date($row->get($column)) < date('Y-m-d')) ? 'false' : 'true';
This value get stored in a Boolean field in my PostgreSQL database.

To clarify the issue is mainly caused by the loose comparision in the update(array $row, array $current) function that cause unwanted behaviors because of thoses cases :

true == "false" return true
true == "f" return true
false == "false" return false
false == "f" return false
false == NULL return true

In my case if the value is true in my PostegreSQL database, it cannot be updated to false because this line prevent the update to be triggered :
 if ($row == array_intersect_key($current, $row)) {            return;        }

Thank you. I will look at it today.

Thank you, I made a quick patch if it can help you :
    /**
     * Execute the update statement.
     */
    protected function update(array $row, array $current): void
    {
        if (false === $this->doUpdates) {
            return;
        }

        if (null === $this->update) {
            $this->prepareUpdate($row);
        }

        // CURRENT VERSION
        /*
            if ($row == array_intersect_key($current, $row)) {
                return;
            }
        */

        // CORRECTED VERSION
        $correctedRow = $row;

        foreach ($correctedRow as $key => $value) {
            if (isset($current[$key]) && gettype($current[$key]) === "boolean" && gettype($value) === "string" && in_array($value, ["true", "t", "false", "f"])) {
                $correctedRow[$key] = in_array($value, ["true", "t"]) ? true : false;
            }
        }

        if ($correctedRow == array_intersect_key($current, $row)) {
            return;
        }
        // END CORRECTED VERSION

        if ($this->timestamps) {
            $row['updated_at'] = $this->time;
        }

        $this->update->execute($row);
    }

After a quick look, I am still not sure to understand 😅

This section use loose comparison between old values array and new values array :
if ($row == array_intersect_key($current, $row)) { return; }

So here the comparison is between keys of the array.
Hence despite the fact that we should have === instead of == at line 188, we are going to fix that on rigour sake, it won't change anything because as we can see in the documentation of PHP:

The two keys from the key => value pairs are considered equal only if (string) $key1 === (string) $key2 . In other words a strict type check is executed so the string representation must be the same.

So if we would like to really strictly compare keys, we could use a callback. But I am not sure why we should do that, because the keys of the array are supposed to represent columns names, not values.
Let me know if I am missing something, I would be very glad to help you.

if ($row == array_intersect_key($current, $row)) { return; } compare not only the keys but also the values of the array.
In my situation,
$row = ["date_naissance" => "1962-06-18", "actif" => "false"]
$current = ["date_naissance" => "1962-06-18", "actif" => true]

the comparison is true because "false" == true is true and this cause the update to not be triggered.

It works for me, thank you !