CI: add azure integration

Reference Issues or PRs

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Any other comments?

I think you're encountering the same issue I did when I last attempted to update Azure to use OIDC: https://github.com/nebari-dev/nebari/issues/1694

@viniciusdc can you take a look at this and see if you can pick this up?

Looks like it's working now and should be ready for review @viniciusdc @marcelovilla

LGTM!

Is this intended to run on every pull request?

Is this intended to run on every pull request?

@aktech No, the intent is just to run weekly with the others.  I can fix later today or feel free to fix if desired.

Fixed now - https://github.com/nebari-dev/nebari/pull/2498#event-13002430072

Awesome, thanks! 🎉


[kie-issues#1154] Make KieSession auto-closeable

Fixes: https://github.com/apache/incubator-kie-issues/issues/1154
Makes KieSession implement the AutoCloseable interface, so it can be used in try-with-resources code blocks. No new tests added, because KieSession does not have any listener for dispose events. Therefore, some tests were changed to use try-with-resources to dispose the KieSession.

Jenkins failure is not related to my changes, it is caused by environment.


ConnectionHandler

Hi,
Just a question here.
Rails 4(.2?) introduced a threadsafe runtime registry to store the connection handler, which you don't use.
Wouldn't that be an interested addition?

Sure!


Fix regex escape issue when using --root on Windows

The code was using directory separator in the regex itself,
which means on Windows it was leaving unescaped backslash.
Added explicit escape for such cases.

I do apologize for the delay in getting to this; previously, no one was currently watching this repository. Thank you for your contribution.


Error during compilling

I get an error during compiling event with blank code like:
 #include <analogmuxdemux.h>

void setup() {
}
void loop() {
}

/home/dmtrs/sketchbook/libraries/analogmuxdemux/analogmuxdemux.cpp: In constructor 'AnalogDeMux::AnalogDeMux(uint8_t)':
/home/dmtrs/sketchbook/libraries/analogmuxdemux/analogmuxdemux.cpp:107:68: error: cannot call constructor 'AnalogDeMux::AnalogDeMux' directly [-fpermissive]
     AnalogDeMux::AnalogDeMux(AMDM_MS0, AMDM_MS1, AMDM_MS2, writepin);
                                                                ^
/home/dmtrs/sketchbook/libraries/analogmuxdemux/analogmuxdemux.cpp:107:68: note: for a function-style cast, remove the redundant '::AnalogDeMux'

Hmmm that's weird.  Not sure why it should be doing that with blank code as
the constructor shouldn't be being called.
If you actually create an instance does it work properly? (Apologies am
away from somewhere I can test this at the moment).
On Sat, Mar 28, 2015 at 11:25 PM Dimitrios Meggidis <
<EMAIL_ADDRESS>wrote:

I get an error during compiling event with blank code like:
#include <analogmuxdemux.h>
void setup() {
}
void loop() {
}
/home/dmtrs/sketchbook/libraries/analogmuxdemux/analogmuxdemux.cpp: In constructor 'AnalogDeMux::AnalogDeMux(uint8_t)':
/home/dmtrs/sketchbook/libraries/analogmuxdemux/analogmuxdemux.cpp:107:68: error: cannot call constructor 'AnalogDeMux::AnalogDeMux' directly [-fpermissive]
AnalogDeMux::AnalogDeMux(AMDM_MS0, AMDM_MS1, AMDM_MS2, writepin);
^
/home/dmtrs/sketchbook/libraries/analogmuxdemux/analogmuxdemux.cpp:107:68: note: for a function-style cast, remove the redundant '::AnalogDeMux'
—
Reply to this email directly or view it on GitHub
https://github.com/ajfisher/arduino-analog-multiplexer/issues/4.

I have resolved the issue with the code change in the #5 pull request, although I am not using that part at all in my code and I am cpp noob

Looks good - that form was working fine and it must be something weird going on in compiler with how explicit it is. Oddly the original works under ino so I suspect it's something arduino specific. It's all good though.
Merged.


Real time data feeds

Hi the charts shown here is delayed by 1 day and there is no intrday data available for nse index and stocks.

This should be resolved now.  Please check and confirm


Clarissa: init at v0.6

Motivation for this change
I want to share my tool.
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)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I'm not sure if we want to include tools in nixpkgs that probably just have a single user, since there's a non-zero cost to including new packages, namely maintenance, hydra build time and for NixOS modules it even slows down everybody's nixos-rebuild. See also the discussion in https://discourse.nixos.org/t/we-need-more-defined-guidelines-for-package-inclusion/3592
Maybe NUR is a better place to put such packages, or eventually as Flakes.

reopened in honour of the 1st anniversary of the v1 release 4 years ago

converted to draft until the test can be sorted out
main issues there are:

actually networking the two machines
extracting the MAC, IPv4 and/or IPv6 addresses from the second machine, to check if they're detected on the first

i'm intentionally ignoring the failing nixpkgs-check-by-name check
this adds 3 very closely related packages all based on the same source
i'd like to keep them grouped in 1 directory, rather than add 3 by-name directories

@evils See https://github.com/NixOS/nixpkgs/tree/master/pkgs/by-name#recommendation-for-new-packages-with-multiple-versions then to resolve the CI failure.

@infinisil huh, thanks, i didn't think to look there because i didn't know by-name was now a requirement
(i don't see that in CONTRIBUTING.md)
(note that jumping to that anchor doesn't seem to work (because of the loading time of such a long page?))
the recommendation there does not provide enough context for me to be able to use it
(where should inherit be used?, what's meant with a local package set (local where?), foo_1 refers to ../tools, wouldn't this be ../to/tools?)
is there an example package that already does this?
(just grepping for inherit yields a bit too much noise to find such)


Remove version: Jackett.Jackett version 0.21.424

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.21.424/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


Progress Indicator does not work in web client

Please add a remark that the Progress Indicator does not work in the web client.

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

ID: 163dedae-669d-4f9e-da19-f9812a0a96ab
Version Independent ID: 22554382-887c-84da-7d21-51b81912c733
Content: Open Method - Business Central
Content Source: dev-itpro/developer/methods-auto/dialog/dialog-open-method.md
Service: dynamics365-business-central
GitHub Login: @solsen
Microsoft Alias: solsen

Hello, thanks for the feedback! We have updated article to state that the progress indicator is not supported in the web client. The changes will be published soon.


Enable cors

Changes Unknown when pulling 817d5c1f0023949bf82ec57204a7c99325596a7d on OpenAddressesUK:enable-cors into ** on theodi:master**.

@pikesley Can you merge master into this to allow merge?


storagereplica PowerShell reference update

1880837
2 new cmdlets and 1 new param.

Docs Build status updates of commit 3b5efab:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/storagereplica/Dismount-SRDestination.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/storagereplica/Mount-SRDestination.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/storagereplica/Remove-SRPartnership.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/storagereplica/StorageReplica.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 627123a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/storagereplica/Dismount-SRDestination.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/storagereplica/Mount-SRDestination.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/storagereplica/Remove-SRPartnership.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/storagereplica/StorageReplica.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@tiburd - It looks like this one is ready for merge.


NotFoundError when running example

Hello,
When trying to run your code:
pip install vit-keras
from vit_keras import vit, utils
image_size = 384
classes = utils.get_imagenet_classes()
model = vit.vit_b16(image_size=image_size,activation='sigmoid',pretrained=True,include_top=True,pretrained_top=True)
I get the following error: NotFoundError: /usr/local/lib/python3.6/dist-packages/tensorflow_addons/custom_ops/activations/_activation_ops.so: undefined symbol: _ZN10tensorflow14kernel_factory17OpKernelRegistrar12InitInternalEPKNS_9KernelDefEN4absl11string_viewESt10unique_ptrINS0_15OpKernelFactoryESt14default_deleteIS8_EE
This was ran on google colab with Tensorflow 2.4.0, Keras 2.4.3

According to https://github.com/tensorflow/addons/issues/676#issuecomment-560159918 and https://github.com/tensorflow/addons/issues/753 this is a known compatibility issue between TF and TF-addons.  Upgrading TF-addons with pip install tensorflow_addons --upgrade solved the issue


Adaptation for embedded use

In taking a prelim look at adapting CodeBuff for embedded use, the wide use of statics presents a question of what would be acceptable.  Proposal would be refactor the CodeBuff Tool to match the form of the Antlr4 Tool.  If there is an alternate preference, please describe and I will prepare a pull request accordingly.

Yeah, the statics are either debugging or related. Code is thread-safe to run multiple formats at same time. I hit that area of code hard.

Branched as CodeBuff2 in https://github.com/grosenberg/codebuff/tree/codebuff2
Removes the statics, refactors to a more conventional programming style, and configures the tool for embedded use. Works, but not well tested. More to do.

Thanks Gerald. I will take a look when I get back to codebuff. I want to integrate a version into my plugin so this could help quite a bit.

Closing in favor of Adept at https://github.com/grosenberg/adept


Delete old subjects

Please delete

Musik
-BWR (Betriebswirtschaftslehre mit Rechnungswesen)
Deutsch als Fremdsprache

What about possible existing content? Did you move it somewhere else? Or do we delete that?

Only in Musik there was content. I already moved it to the subjects in construction.


add dependency to tanglegram

solves portion of #21 referenced by @jzrapp. Pandas was imported by main, but not by make_tanglegram(), so the scoping was off. This PR imports pandas!

hmm...Travis CI failed with:
Traceback (most recent call last):
  File "/home/travis/build/bjtully/BioData/.tox/py36/bin/KEGG-decoder", line 10, in <module>
    sys.exit(main())
  File "/home/travis/build/bjtully/BioData/.tox/py36/lib/python3.6/site-packages/KEGGDecoder/KEGG_decoder.py", line 1386, in main
    raise ValueError("Interactive mode requires fifty or more genomes")
ValueError: Interactive mode requires fifty or more genomes

@bjtully does this make any sense to you?

@taylorreiter Yeah. I know exactly the cause for this -- if you try and run 'interactive' when there is too few genomes, the output doesn't work. Forced a minimum amount to avoid it. I'll need to update Travis CI tests to avoid this in the future. I'll override the failure for the PR.

Ok great, thank you @bjtully!


How use it?

Hello, how use it app? My Laptop use Windows 10.

Hi @Spec86,
This is the source code of a java project. It uses the maven build tool.
There are many ways to run a java project.
The most simple is to install maven and run:
mvn exec:java -Dexec.mainClass="com.github.csfulop.sicpjava.lisp.ReadEvalPrintLoop"
See TestReadEvalPrintLoop.java file for some example.


Not able to visualize graph. As well as showing error of optimizers.

HI,
I am running the cart example given in website. Agents are getting trained but I am not able to use tensorboard. First it is saying graph has been made but later it is not getting into logdir. Therefore I am not able to visualize the graph. I am using tensorflow 1.8.0 and tensorforce 0.4.2.
/usr/local/lib/python2.7/dist-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
WARN: gym.spaces.Box autodetected dtype as <type 'numpy.float32'>. Please provide explicit dtype.
INFO:tensorflow:Graph was finalized.
INFO:tensorflow:Graph was finalized.
2018-05-24 11:52:31.934912: I tensorflow/core/platform/cpu_feature_guard.cc:140] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
INFO:tensorflow:Running local_init_op.
INFO:tensorflow:Running local_init_op.
INFO:tensorflow:Done running local_init_op.
INFO:tensorflow:Done running local_init_op.
INFO:openai_gym.py:Starting VPGAgent for Environment 'OpenAIGym(CartPole-v0)'
2018-05-24 11:52:32.132634: W tensorflow/core/grappler/optimizers/arithmetic_optimizer.cc:1441] Failed to build SimpleGraphView.
2018-05-24 11:52:32.133442: E tensorflow/core/grappler/optimizers/dependency_optimizer.cc:586] Iteration = 0, topological sort failed with message: Non-existent input ^ConstantFoldingCtrl/vpg/cond/add/Switch_0 for node ConstantFolding/vpg/cond/strided_slice_1/stack_const_axis
2018-05-24 11:52:32.133820: E tensorflow/core/grappler/optimizers/dependency_optimizer.cc:586] Iteration = 1, topological sort failed with message: Non-existent input ^ConstantFoldingCtrl/vpg/cond/add/Switch_0 for node ConstantFolding/vpg/cond/strided_slice_1/stack_const_axis
INFO:openai_gym.py:Finished episode 100 after 35 timesteps. Steps Per Second 729.41
INFO:openai_gym.py:Episode reward: 35.0
INFO:openai_gym.py:Average of last 500 rewards: 23.78
INFO:openai_gym.py:Average of last 100 rewards: 23.78
INFO:openai_gym.py:Finished episode 200 after 65 timesteps. Steps Per Second 811.95
INFO:openai_gym.py:Episode reward: 65.0
INFO:openai_gym.py:Average of last 500 rewards: 36.20
INFO:openai_gym.py:Average of last 100 rewards: 48.61
{'shape': (4,), 'type': 'float'}
{'type': 'int', 'num_actions': 2}
(1, {'state': array([ 0.01600748, -0.04044823,  0.00649351, -0.04812018])}, {})
INFO:openai_gym.py:Learning finished. Total episodes: 200

FYI There is no directory named as optimizers in my tensorflow directory. I believe there is some versioning issue.
PS I think the first information error is just a warning of tensorflow.

@michaelschaarschmidt @krfricke Can you check above issue. Thanks in advance.

Closing as duplicate


[RNAdam-58] Upgrade to ADAM 0.15.1-SNAPSHOT

Resolves #58. Upgrades to ADAM 0.15.1-SNAPSHOT, adds RNAdamFunSuite for testing Spark based features, and moves Hadoop and Spark to provided scope.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/RNAdam-prb/66/
Test PASSed.


Change orientation of image after uploaded

Hello.
How I can change orientation of uploaded image?

Hi @aleksand88 , you can do so with the orientation option:
https://github.com/blueimp/JavaScript-Load-Image#orientation
Please see also: https://github.com/blueimp/JavaScript-Load-Image#usage


feat(core): support .set(val) on a signal

Resolves https://github.com/preactjs/signals/issues/299
This implements .set() to conform to the Svelte store contract as we have already implemented .subscribe() this felt logical.
Svelte Ref

Closing due to inactivity

Any reason the branch was deleted and the functionality not incorporated?

Having two ways of doing the same thing felt confusing and we preferred to have one way to set signal values by default. The Signal class is exposed so folks who really want this can just patch it back in on the prototype.

It's a shame, it would be very useful when using with Svelte following the store contract

FWIW the Svelte store contract also includes update(), so this wasn't really getting us to the point of being drop-in. It would make a lot more sense for someone to build and publish a Svelte integration that does the following:
import Signal from '@preact/signals-core';

Signal.prototype.set = function(value) {
  this.value = value;
};

Signal.prototype.update = function(callback) {
  this.value = callback(this.peek());
};

svelte's built-in store has an .update() method, but the store contract only asks for .subscribe() and .set(). It wouldn't really make sense to create a solution to add only the .set() method considering that the core library already has the .subscribe() method that was only inserted in the core library for compatibility with the svelte store contract, since for use outside of svelte there was already the effect() function

The .subscribe() method wasn't inserted only because of Svelte. It's useful when interfacing with imperative code too.


Always create subscriptions for anonymous groups

Previously, the binder would only create Pub/Sub subscriptions for
anonymous consumer groups when resource auto-creation was turned on.
This change ensures that we uncodnitionally create subscriptions for
anonymous consumer groups regardless of the auto-creation setting.
Fixes #1180.

Codecov Report

Merging #1220 into master will increase coverage by 0.08%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1220      +/-   ##
============================================
+ Coverage        67%   67.09%   +0.08%
- Complexity     1290     1294       +4
============================================
  Files           185      186       +1
  Lines          5316     5318       +2
  Branches        545      544       -1
============================================
+ Hits           3562     3568       +6
+ Misses         1520     1519       -1
+ Partials        234      231       -3

Flag
Coverage Δ
Complexity Δ

#integration
?
?

#unittests
67.09% <100%> (+0.08%)
1294 <17> (+4)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...er/pubsub/properties/PubSubProducerProperties.java
100% <100%> (ø)
1 <1> (ø)
:arrow_down:

...er/pubsub/properties/PubSubConsumerProperties.java
100% <100%> (ø)
1 <1> (-2)
:arrow_down:

...nder/pubsub/properties/PubSubCommonProperties.java
100% <100%> (ø)
3 <3> (?)

.../pubsub/provisioning/PubSubChannelProvisioner.java
93.93% <100%> (+12.68%)
13 <12> (+3)
: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 e1ed867...4dc238e. Read the comment docs.


Statement with bindParam equal does not work correctly

Node.js version: 14.6.0
idb-connector version<EMAIL_ADDRESS>idb-pconnector version<EMAIL_ADDRESS>IBM i version: 3 7

Describe the bug
the equal operator is supposed to work the same with both a direct query and a prepared query, however they are working differently
To Reproduce
const { Connection, Statement, IN, CHAR } = require('idb-pconnector');

const sql = 'SELECT * FROM QIWS.QCUSTCDT WHERE CITY = ?';
let bindParams = [ [ 'Dallas2', IN, CHAR  ] ]

let connection = new Connection({ url: '*LOCAL' });
let statement = new Statement(connection);

await statement.bindParam(bindParams);
await statement.execute(sql);
let resultSet = await statement.fetchAll();

Expected behavior
It is assumed that no record in the test table matches the city Dallas2, since Dallas is actually written without the number 2. but the system still finds Dallas being Dallas ! = Dallas2
Screenshots
with bindParam statement

normal query

Additional context
It can be solved by modifying the sentence but it is a waste of resources. ej
SELECT * FROM QIWS.QCUSTCDT WHERE CITY = %?%

CC @dmabupt

Since the CITY column is only 6 characters long, the parameterized condition is presumably getting truncated from "Dallas2" to "Dallas".

CC @forstie, any insight/thoughts on what the proper behavior is here? Should we instead be throwing truncation warnings/errors?

That makes sense. Looking at the SQLBindCol I see
The application must ensure enough storage is allocated for the data
to be retrieved.If the buffer is to contain variable length data, the
application must allocate as much storage
as the maximum length of the bound column requires;
otherwise, the data might be truncated.

makes sense but the truth is unexpected.
In my opinion, they should behave in the same way as the simple query, as it can lead to unexpected or unwanted results when looking for precision and validation. I think in this case I should not return anything or issue a warning.

makes sense but the truth is unexpected.
In my opinion, they should behave in the same way as the simple query, as it can lead to unexpected or unwanted results when looking for precision and validation. I think in this case I should not return anything or issue a warning.

I agree

This looks to be working as expected, although I do have to question only having 6 characters for a city name.
Retrieval assignment:
The length of a string assigned to a variable (but not a parameter or transition
variable) can be greater than the length attribute of the variable. When a string is
assigned to a variable and the string is longer than the length attribute of the
variable, the string is truncated on the right by the necessary number of characters.
When this occurs, an SQLSTATE of '01004' is assigned to the
RETURNED_SQLSTATE condition area item in the SQL Diagnostics Area (or the
value 'W' is assigned to the SQLWARN1 field of the SQLCA).
SQLBindParam has the concept of SQL_SUCCESS_WITH_INFO.
I'm not sure if that or the aforementioned warning would be surfacable here.

This looks to be working as expected, although I do have to question only having 6 characters for a city name.
Retrieval assignment:
The length of a string assigned to a variable (but not a parameter or transition
variable) can be greater than the length attribute of the variable. When a string is
assigned to a variable and the string is longer than the length attribute of the
variable, the string is truncated on the right by the necessary number of characters.
When this occurs, an SQLSTATE of '01004' is assigned to the
RETURNED_SQLSTATE condition area item in the SQL Diagnostics Area (or the
value 'W' is assigned to the SQLWARN1 field of the SQLCA).
SQLBindParam has the concept of SQL_SUCCESS_WITH_INFO.
I'm not sure if that or the aforementioned warning would be surfacable here.

This looks to be working as expected, although I do have to question only having 6 characters for a city name.
Retrieval assignment:
The length of a string assigned to a variable (but not a parameter or transition
variable) can be greater than the length attribute of the variable. When a string is
assigned to a variable and the string is longer than the length attribute of the
variable, the string is truncated on the right by the necessary number of characters.
When this occurs, an SQLSTATE of '01004' is assigned to the
RETURNED_SQLSTATE condition area item in the SQL Diagnostics Area (or the
value 'W' is assigned to the SQLWARN1 field of the SQLCA).
SQLBindParam has the concept of SQL_SUCCESS_WITH_INFO.
I'm not sure if that or the aforementioned warning would be surfacable here.

in a few hours I will have access to an iseries, I will confirm if the response to SQL_SUCCESS_WITH_INFO arrives. in case it reaches SQL_SUCCESS_WITH_INFO:
What do you think of creating a function or setting in the library that allows you to ignore that response and return as if nothing has been found? (although results were actually found) would be a waste of memory and perhaps a vulnerability. Is there a possibility to modify this behavior from the query?

I found some code that have potential problems related to this issue.
https://github.com/IBM/nodejs-idb-connector/blob/master/src/db2ia/dbstmt.cc#L2593
The code assumes that the input parameter is always shorter than the column defined length. But in this case, it is not true.
The idb-connector level code does not truncate the input parameter data in the bindParam() function. It may be related to the different behavior.
Maybe I should truncate the too long input parameters in idb-connector.

I found some code that have potential problems related to this issue.
https://github.com/IBM/nodejs-idb-connector/blob/master/src/db2ia/dbstmt.cc#L2593
The code assumes that the input parameter is always shorter than the column defined length. But in this case, it is not true.
The idb-connector level code does not truncate the input parameter data in the bindParam() function. It may be related to the different behavior.
Maybe I should truncate the too long input parameters in idb-connector.

Ah... The problem is that the bindParam truncates the string while exec not. So it is not the root cause.
And I also can not catch any SQL_SUCCESS_WITH_INFO

This looks to be working as expected, although I do have to question only having 6 characters for a city name.
Retrieval assignment:
The length of a string assigned to a variable (but not a parameter or transition
variable) can be greater than the length attribute of the variable. When a string is
assigned to a variable and the string is longer than the length attribute of the
variable, the string is truncated on the right by the necessary number of characters.
When this occurs, an SQLSTATE of '01004' is assigned to the
RETURNED_SQLSTATE condition area item in the SQL Diagnostics Area (or the
value 'W' is assigned to the SQLWARN1 field of the SQLCA).
SQLBindParam has the concept of SQL_SUCCESS_WITH_INFO.
I'm not sure if that or the aforementioned warning would be surfacable here.

in a few hours I will have access to an iseries, I will confirm if the response to SQL_SUCCESS_WITH_INFO arrives. in case it reaches SQL_SUCCESS_WITH_INFO:
What do you think of creating a function or setting in the library that allows you to ignore that response and return as if nothing has been found? (although results were actually found) would be a waste of memory and perhaps a vulnerability. Is there a possibility to modify this behavior from the query?

Hello @jhonnattan123, have you got the SQL_SUCCESS_WITH_INFO returned code? I can not get that when debugging the code.
If this is not an idb-connector issue. I will assume that it is working as expected and add corresponding test cases.

Does that mean that with the last commit now if they should work like the normal query?._

Does that mean that with the last commit now if they should work like the normal query?._
I had problems that day, to put the console log, and I was waiting this week (by issues of work permits) to put the console.log

No, the latest version does not change current behavior. As far as I can see, the truncation does not happen in the idb-connector level.


Test nonce too low

Wrote this test in the hunt of another bug. Doesn't actually change anything, just adds an additional test.

Thanks @aakilfernandes!


Tabs and sidebar titles appear italic [suffering to read]

Hi there,
Titles in sidebar and and tabs look like this. Is there some solution for this issue to make font looks normal?

I believe that's an intended affect for what's called "transient" tabs. If you want, you could disable the click to preview feature  with
"preview_on_click": false,


Upgrade @skpm/builder

closed in favor of https://github.com/BohemianCoding/print-export-sketchplugin/pull/9


allow specifying ingester port for registration

I have to run the ingesters in a port mapped setup, so this allows passing in the actual external port so they can register in consul correctly.

What does this change mean for anyone who used -server.grpc-listen-port to set the port to something other than 9095?

good catch. this would break their setup. I'll fix

@bboreham fixed


PLAT-10478  Connection pooling with BDK 2.0

Closes #429
Cannot reproduce the issue, however the message in the logs:
PoolingHttpClientConnectionManager Connection request [route: {tls}->[proxy server] -> ***.symphony.com:443][Total available: 0; route allocated: 2 of 2; total allocated: 2 of 20]
means that a connection cannot be allocated because the pool allow only 2 with the same route and they are both used.
The connections are closed when the response is closed so here https://github.com/SymphonyPlatformSolutions/symphony-api-client-java/blob/201859e8b3b8043895f18cbc6aea91c5b5f278e1/symphony-bdk-http/symphony-bdk-http-jersey2/src/main/java/com/symphony/bdk/http/jersey2/ApiClientJersey2.java#L187
As it is inside a finally block, it is unlikely that connection is not released; also, a connection timeout is already set https://github.com/SymphonyPlatformSolutions/symphony-api-client-java/blob/201859e8b3b8043895f18cbc6aea91c5b5f278e1/symphony-bdk-http/symphony-bdk-http-jersey2/src/main/java/com/symphony/bdk/http/jersey2/ApiClientBuilderJersey2.java#L51 (with a default value of 15000 ms) and it can be changed.
Most likely the issue (with a random occurrence) is due to more than two threads using the same route for a long time. With this modification, the PoolingHttpClientConnectionManager is created manually, by default with the same number of connection but not limited to two on the same route

Does it apply to the other HTTP client that we support (Spring webclient)?

We will also need to make configurable from BdkClientConfig.java + doc update here

Does it apply to the other HTTP client that we support (Spring webclient)?

It don't think: Spring Webclient use Netty, that has a default connection pool of 500
https://projectreactor.io/docs/netty/snapshot/reference/index.html#_connection_pool but, most important, the pool should be generic, not with a limited by route, that is the main problem of default PoolingHttpClientConnectionManager (I think that 20 connection in the pool are more than enough...if they are not limited to two for the same route!)
For this reason I don't added the configuration in the generic configuration interface. By the way I set the default max connection value has same value than...default one! So we should not have any difference in memory footprint/performance...the idea is only set max connections per route = max connections

Hi Guys,
Thanks for the quick response.
I did manage to hack around the old SDK version we used before to get the routes allocated to 20, but it quickly gobbled up all 20 connections. As mentioned by @symphony-thibault, having config to control the route allocated and total allocated would be useful. Once these changes are merged, I'll grab a copy and set the allocated routes and total allocated to a higher value than the default of 2/20 respectively and try that.
Incidentally, the response.close(); mentioned is within a try block that does nothing if the close fails.  Not sure I fully follow the comment as to why an exception here doesn't matter; I guess because it goes out of scope and would be gc'd?, but just wonder if a little debug message here might prove useful in the event of an exception?

Hi Guys,
Thanks for the quick response.
I did manage to hack around the old SDK version we used before to get the routes allocated to 20, but it quickly gobbled up all 20 connections. As mentioned by @symphony-thibault, having config to control the route allocated and total allocated would be useful. Once these changes are merged, I'll grab a copy and set the allocated routes and total allocated to a higher value than the default of 2/20 respectively and try that.
Incidentally, the response.close(); mentioned is within a try block that does nothing if the close fails. Not sure I fully follow the comment as to why an exception here doesn't matter; I guess because it goes out of scope and would be gc'd?, but just wonder if a little debug message here might prove useful in the event of an exception?

The close can fail when response is null (exception during calling) or when it is already closed. I will add a warning log, when the response is not null

Hi Guys,
Thanks for the quick response.
I did manage to hack around the old SDK version we used before to get the routes allocated to 20, but it quickly gobbled up all 20 connections. As mentioned by @symphony-thibault, having config to control the route allocated and total allocated would be useful. Once these changes are merged, I'll grab a copy and set the allocated routes and total allocated to a higher value than the default of 2/20 respectively and try that.
Incidentally, the response.close(); mentioned is within a try block that does nothing if the close fails. Not sure I fully follow the comment as to why an exception here doesn't matter; I guess because it goes out of scope and would be gc'd?, but just wonder if a little debug message here might prove useful in the event of an exception?

The close can fail when response is null (exception during calling) or when it is already closed. I will add a warning log, when the response is not null

Just another thought on this, could the finally block be avoided altogether if using try-with-resources where assigning the response which supports AutoCloseable and allow autoclose to look after closing the resource closure. Just means the system looks after tidying up resource closures rather than dealing with in a finally block. This is just pseudo code so may be a reason it wouldn't work or be desirable.
try (
  Response response = null;
  if ("GET".equals(method)) {
    response = invocationBuilder.get();
  } else if ("POST".equals(method)) {
    response = invocationBuilder.post(entity);
  } else if ("PUT".equals(method)) {
    response = invocationBuilder.put(entity);
  } else if ("DELETE".equals(method)) {
    response = invocationBuilder.method("DELETE", entity);
  } else if ("PATCH".equals(method)) {
    response = invocationBuilder.method("PATCH", entity);
  } else if ("HEAD".equals(method)) {
    response = invocationBuilder.head();
  } else if ("OPTIONS".equals(method)) {
    response = invocationBuilder.options();
  } else if ("TRACE".equals(method)) {
    response = invocationBuilder.trace();
  } else {
    throw new ApiException(500, "unknown method type " + method);

}
) {
...
}

Hi Guys,
Thanks for the quick response.
I did manage to hack around the old SDK version we used before to get the routes allocated to 20, but it quickly gobbled up all 20 connections. As mentioned by @symphony-thibault, having config to control the route allocated and total allocated would be useful. Once these changes are merged, I'll grab a copy and set the allocated routes and total allocated to a higher value than the default of 2/20 respectively and try that.
Incidentally, the response.close(); mentioned is within a try block that does nothing if the close fails. Not sure I fully follow the comment as to why an exception here doesn't matter; I guess because it goes out of scope and would be gc'd?, but just wonder if a little debug message here might prove useful in the event of an exception?

The close can fail when response is null (exception during calling) or when it is already closed. I will add a warning log, when the response is not null

Just another thought on this, could the finally block be avoided altogether if using try-with-resources where assigning the response which supports AutoCloseable and allow autoclose to look after closing the resource closure. Just means the system looks after tidying up resource closures rather than dealing with in a finally block. This is just pseudo code so may be a reason it wouldn't work or be desirable.
try (
  Response response = null;
  if ("GET".equals(method)) {
    response = invocationBuilder.get();
  } else if ("POST".equals(method)) {
    response = invocationBuilder.post(entity);
  } else if ("PUT".equals(method)) {
    response = invocationBuilder.put(entity);
  } else if ("DELETE".equals(method)) {
    response = invocationBuilder.method("DELETE", entity);
  } else if ("PATCH".equals(method)) {
    response = invocationBuilder.method("PATCH", entity);
  } else if ("HEAD".equals(method)) {
    response = invocationBuilder.head();
  } else if ("OPTIONS".equals(method)) {
    response = invocationBuilder.options();
  } else if ("TRACE".equals(method)) {
    response = invocationBuilder.trace();
  } else {
    throw new ApiException(500, "unknown method type " + method);

}
) {
...
}

Yes, it is way better and I refactored it

We will also need to make configurable from BdkClientConfig.java + doc update here

Ok @symphony-thibault , @20-goto-10  I refactored to make it configurable. BTW they are other configuration parameters already partially implemented but the code was not complete, I implemented them also. Now we have (with their default values):
connectionTimeout: 15000
readTimeout: 60000
connectionPoolMax: 20
connectionPoolPerRoute: 20

Hi guys,
Sorry to keep on, but was just reading a little on PoolingHttpClientConnectionManager and it seems that in order to ensure resources are released FULLY so that the connection can be released back to the pool, its a good idea to call EntityUtils.consume(response.getEntity()); Would it be an idea to make this call at the end of the try block for the response? Just belts and braces? It seems to touch on the problem that I am seeing where connections in the pool drop to 0.
References:
http://hc.apache.org/httpcomponents-client-ga/quickstart.html
https://stackoverflow.com/questions/15969037/why-did-the-author-use-entityutils-consumehttpentity

orry to keep on, but was just reading a little on PoolingHttpClientConnectionManager and it seems that in order to ensure resources are released FULLY so that the connection can be released back to the pool, its a good idea to call EntityUtils.consume(response.getEntity()); Would it be an idea to make this call at the end of the try block for the response? Just belts and braces? It seems to touch on the problem that I am seeing where connections in the pool drop to 0.
References:

EntityUtils.consume() is for closing HttpEntity. However, when using Jersey Invocation builder, normally you need just close Response as far I understand


Setting a "none" infowindow in builder breaks editor

Reproduced testing #8309, which fixes the embed error for a related problem.

Create a map with the builder.
Set light infowindow.
Set none infowindow (2 + 3 enforces storing "none" as template).
Open the map in the editor. You get this JS error:

tracker.js:10 cartodb/table/views/tooltip/custom_templates/none not founda.(anonymous function) @ tracker.js:10
infowindow_html_pane.js:202Uncaught TypeError: cdb.templates.getTemplate(...) is not a function

cc @xavijam @matallo

on it!


First feature extractors

I created PR to incorrect base repository, and cannot remove the PR permanently, so I leave it here as closed. Sorry!


Fix cacheDir  is not in effect

Proxy mode cacheDir was unused.
set the downloadRoot  when init.

LGTM


thuang-fix-organism

Reviewers
Functional:
@Bento007
Readability:
@seve

Changes

modify

Quick update to the reduce function !

Thanks for the review, @seve !!

Thanks for the review, @seve !!


Update.kodiak config

Update kodiak config to add the correct watchers and post tickets to the correct Epic.
Question: Did we say we want to remove @JingleH as the default assignee?
Resolves: MWPW-145744
Test URLs:

Before: https://main--express--adobecom.hlx.page/express/?martech=off
After: https://configure-kodiak--express--adobecom.hlx.page/express/?martech=off

We can make David the assignee too

https://git.corp.adobe.com/kodiak-config-examples/default-notification-config/blob/main/Example configs/base-config.yaml confirms that assignee is an optional field.
I also added myself and AG in as watchers

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 36.62%. Comparing base (a3e2b10) to head (b51bc34).
 Consider uploading reports for the commit 677e7ed to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #785      +/-   ##
==========================================
- Coverage   37.33%   36.62%   -0.71%
==========================================
  Files         132      132
  Lines       25932    25932
==========================================
- Hits         9681     9497     -184
- Misses      16251    16435     +184

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Compilation time – drains battery and doesn't finish compiling

While editing code, you naturally break a function or two while editing. The problem is that ionide doesn't finish compiling, so code-completion is broken forever, until VSCode is restarted.

The number is the "Energy impact" column.
EDIT: If this is Suave's fault, I may have a real-world example to fix performance because of.

This should be fixed with request cancelling we've added recently


ASP.NET Core language pack support

Ditto to title

Hi @stevefan1999-personal thanks for opening this issue! Unfortunately, nobody that I am aware of on the team has .NET experience. We would, of course, be open to a pull request if you have any interest in working on this.

@stevefan1999-personal  feel free to check this https://github.com/knative/func/pull/1484

/remove-lifecycle stale


After filling out PIN & confirmation fields presented with a blank view

Description of problem
After filling out the PIN & confirmation inputs I am presented with a blank view. If I restart the application it once again prompts me to create a PIN.
See screenshots below.
Expected behavior
I would like to be able to use the software!
Steps to reproduce
Presumably you want to start with a clean slate, start the application, and attempt to navigate through the initial flow.
Screenshots and/or log output

Environment

[ ] Occurs on Android
[x] Occurs on iOS

Build #: (presumably latest version available on the App Store.) v1.0.8
Android Device Model: n/a
iOS Device Model: iPhone 13 Pro; more importantly iOS 17.0 (21A5268h)
Workaround
Not that I know of.
Severity

[ ] High
[ ] Medium
[ ] Low

Additional context: I've noticed that this occurs after I enter the sixth number in the PIN confirmation input but before I click "Create PIN".

I've tried this on an iPhone 13 Pro with iOS 15.7 and 16.4.1. This issue did not manifest. I wonder if it has to do with iOS 17? it is still in beta. My advice would be to go back to an official iOS release. Otherwise, when iOS 17 is officially released we will revisit this issue if it exists.

@jamesdphillips , we are no longer using the individual boxes, please try again with 1.0.9, and if the issues is till there please re-open this issue.


Cb 10637

Fixed link location which was previously not correct.

@dhams Thanks for the PR! This is no longer an issue in master because the android config.md file was removed as part of the Android Guide improvements in 21afd07d1e14ff4b6dc524738ffaa4fc7594de1f. Would you mind closing this PR?

@dhams please close this PR when you have a moment.

closing this PR as that changes has already been covered.

@dhams thanks!


Repo not working

Hi, I downloaded the latest Repo of 1.7 and installed it but it shows could not connect to the Repository error while trying. Please help to fix. Thanks.
https://github.com/Gujal00/GujalKodiWork/releases/download/1.7/repository.gujal-1.7.zip

Can't replicate the problem, working fine on the couple of machines that I tested with.


Cordova Sample

Are there plans or anything in the works to add an example for Cordova apps?
Would you take a PR for one?  If so, any constraints or things the example should do?  Should it have the same functionality as the other examples?

@orangemocha I'm working on a cordova angularjs example and will submit a PR.

@ksplache sounds good, thank you!

PR with the basics: https://github.com/janeasystems/nodejs-mobile-samples/pull/2
I may add more in the future, but I think we can start with this.


[v2.8] KDM Update for K8s Aug 2024 patch release

KDM Updates
TO BE RELEASED: as part of v2.8.7

New Kubernetes Version
Min Rancher Version
Min RKE Version

1.28.13-rancher1-1
2.8.3-patch0
1.5.0-rc0

Rancher System Images & Add-ons
Note: Please update system images and add-ons in the template as needed.
TBD
Developer Changelog

Add templates for new K8s versions. If a template that hasn't been released yet already exists, update it in place.
Update the default K8s version, and add or update the version information (MinRancherVersion and MinRKEVersion) in k8s_version_info.go for new K8s versions if needed (Use table from above).
Confirm versions for system images, add-on templates are correct using versioning reference table from below (expand details to view the table). Update the table if new versions are introduced.
Confirm add-on template version constraints in template.go are correct using versioning reference table from below (expand details to view the table). Update the table if needed.

Versioning Reference

Expand for system images & add-on templates versioning table

Note: Truncated repository names with a * indicates there are multiple images with that prefix. All repository names that match a truncated prefix should be checked for the same version stated in the table.
For example: rancher/weave-* matches weaveworks/weave-kube and weaveworks/weave-npc, and they should both have the same version.
v1.28.13-rancher1-1

Repository
Tag

rancher/mirrored-coreos-etcd
v3.5.10

rancher/hyperkube
v1.28.10-rancher1

rancher/rke-tools
v0.1.96

rancher/mirrored-k8s-dns-*
1.22.28

rancher/mirrored-cluster-proportional-autoscaler
1.8.9

rancher/flannel-cni
v0.3.0-rancher9

rancher/mirrored-flannel-flannel
v0.24.2

rancher/mirrored-calico-*
v3.27.0

weaveworks/weave-*
2.8.1

noiro/* (All ACI images)
<IP_ADDRESS>.81c2369

rancher/mirrored-pause
3.7

rancher/nginx-ingress-controller
nginx-1.9.6-rancher1

rancher/mirrored-nginx-ingress-controller-defaultbackend
1.5-rancher1

rancher/mirrored-jettech-kube-webhook-certgen
v20231226-1a7112e06

rancher/mirrored-metrics-server
v0.7.0

rancher/mirrored-coredns-coredns
1.10.1

rancher/mirrored-cluster-proportional-autoscaler
1.8.9

rancher/mirrored-k8s-dns-node-cache
1.22.28

Add-on Name
Add-on Template Version Constraint

Calico
calicov3261Rancher2

Canal
canalv3261Rancher2

Flannel
flannelv0214

CoreDNS
coreDnsv183Rancher2

KubeDNS
kubeDnsv116

MetricsServer
metricsServerv061

Weave
weavev122

Aci
aciv6032

NginxIngress
nginxIngressv194Rancher1

Nodelocal
nodelocalv121

Fresh install checks - PASS

On rancher version v2.8-head KDM pointing to dev-v2.8 branch, cluster creation succeeds for 1.28.13-rancher1-1  on all network providers - canal/flannel/weave/calico
Network checks look good
Canal network checks look good

Upgrade checks - PASS

On rancher v2.8.7, KDM pointing to release-v2.8, cluster creation succeeds for k8s 1.28.12-rancher1-1 version with all network providers.
Pre-upgrade checks look good.
Upgraded rancher to v2.8-head
Post upgrade checks look good.
KDM points to dev-v2.8
Upgraded cluster to k8s versions - 1.28.13-rancher1-1.
Post upgrade checks look good.

Minimum Rancher checks - PASS

On rancher version v2.8.3, KDM pointing to dev-v2.8 branch, cluster creation succeeds for v1.28.13-rancher1-1 on all network providers - canal/flannel/weave/calico
Network checks look good
Canal network checks look good

RKE default k8s version - PASS
dev-v2.8 should have the default as v1.28.13-rancher1-1
 "RKEDefaultK8sVersions": {
  "0.3": "v1.16.3-rancher1-1",
  "default": "v1.28.13-rancher1-1"
 }

RKE Checks passed with rke v1.5.13-rc.2. Also, checked if images are getting pulled successfully.

Ran rke config --system-images
Pulled all the docker images manually.
Images are getting pulled successfully.


PLANNER-618 Workbench: Use org.jboss.errai.ioc.client.api.ManagedInstance instead of BeanManager where possible

Removes unused methods from DomainHandler class hierarchy.

+1 (I'd change the commit message though to Remove unused methods from DomainHandler or add that to the existing one).

@mcimbora @csadilek
Hello guys,
I saw that in previous PRs you made some changes that affects things like JPADomain, DroolsDomain, etc. All this XXXDomain things are related to the content of the data modeler tool windows, etc.
So the question that came up to my mind is, are you @mcimbora testing/verifing that all the tool windows continues working properly with the changes, i.e., checking that drools parameters configuration works properly, persitable data objects, etc. (with for example two editors opened at the same time), or are you only testing the planner related stuff?
Regards,
Walter.

@wmedvede, I have to admit the testing is mainly planner-related. I am aware this possibly affects multiple modules, however the change was based on an assumption that the methods have no usage. Are you aware of any code which utilizes the removed methods?

Hello @mcimbora ,
I haven't checked exactly if only a couple of un-used methods were removed or the changes also include some other modification. But I believe this is the typical case that requires a verification to be sure all tool windows continues working. It shouldn't take too much time. So I'll appreciate if you please can take a look and be sure it is so.
Thanks!!

@wmedvede I'll go through it, thanks for the insight :-)

@wmedvede I went through the affected screens and everything seems to work fine. Additionally I was able to set drools-wb in multi module IDEA setup (drools-wb + kie-wb-common + guvnor) - created https://github.com/droolsjbpm/drools-wb/pull/244

@csadilek Commit message fixed.

Thanks for the heads-up @wmedvede and for verifying @mcimbora. My +1 is based on the fact that the methods aren't used / have no call sites.


useHttp field does not default to false when not set

What happened in your environment?
We applied a new oras store CR with useHttp not set and the behavior indicates that the requests are still made with http.
What did you expect to happen?
https should have been used since useHttp was not defined
What version of Kubernetes are you running?
1.25.4
What version of Ratify are you running?
rc1
Anything else you would like to add?
No response
Are you willing to submit PRs to contribute to this bug fix?

[X] Yes, I am willing to implement it.

Conducted a deeper investigation of the issue:
Ratify is correctly serializing the config json data and the k8 spec data to the appropriate value for useHttp. It defaults correctly to false if not specified. No action is needed on Ratify to update this.
But then why are we seeing useHttp equal to true when we kubectl apply an update to the oras store CR with the useHttp flag missing?
Looks like the apply command does not properly update the store CR. The implicit patch/merge functionality of apply is not picking up the change of the missing field. This means that the old CR with useHttp is still used.
Here's a similar issue with an explanation
Switching to the replace command works since that is an imperative operation. Read more about it here.


starting point for first announcement

Logo in teh middle of the blog entry is a bit large, but hey..


When using "setTitleBarStyle('hidden')" and "maximize()" on windows the app maximize to full screen hiding the taskbar

When using windowManager.setTitleBarStyle('hidden') or  windowManager.setAsFrameless() with windowManager.maximize() to maximize the window on windows the app maximize to full screen hiding the taskbar.
Is this a bug, or is this the expected behavior?
if it is the expected behavior, how do I get it to not do that and maximize above the task bar like normal apps.
thanks for the great work on this library

Fixed, will be released in next version


Fix: "remember me" not translated.

The "remember me" string is not translated in the new session form.

Or we could just do it the way devise does:
https://github.com/plataformatec/devise/blob/88724e10adaf9ffd1d8dbfbaadda2b9d40de756a/lib/generators/templates/simple_form_for/sessions/new.html.erb#L7


Created new plugin 29 AD Authentication.ps1

Created a new plugin to display Active Directory settings and domain membership status for each host. A screenshot of the output is attached.

That's a great idea for a plugin!
I guess the only thing to improve would be to filter out the "Ok" results by default (as vCheck is really designed as a report to show potential issues), but maybe have an option to include the full information.

Thanks!
I'll look into filtering OK results via a setting and I've considered adding an expected Domain Name and Admin Group setting.


llama3 in run_knowedit_llama2.py

Loading checkpoint shards:   0%|          | 0/4 [00:00<?, ?it/s]
Loading checkpoint shards:  25%|██▌       | 1/4 [00:00<00:02,  1.18it/s]
Loading checkpoint shards:  50%|█████     | 2/4 [00:01<00:01,  1.23it/s]
Loading checkpoint shards:  75%|███████▌  | 3/4 [00:02<00:00,  1.20it/s]
Loading checkpoint shards: 100%|██████████| 4/4 [00:02<00:00,  1.52it/s]
Loading checkpoint shards: 100%|██████████| 4/4 [00:02<00:00,  1.39it/s]
The tokenizer class you load from this checkpoint is not the same type as the class this function is called from. It may result in unexpected tokenization.
The tokenizer class you load from this checkpoint is 'PreTrainedTokenizerFast'.
The class this function is called from is 'LlamaTokenizer'.
You are using the default legacy behaviour of the <class 'transformers.models.llama.tokenization_llama.LlamaTokenizer'>. This is expected, and simply means that the legacy (previous) behavior will be used so nothing changes for you. If you want to use the new behaviour, set legacy=False. This should only be set if you understand what it means, and thoroughly read the reason why this was added as explained in https://github.com/huggingface/transformers/pull/24565
Traceback (most recent call last):
File "/scratch2/mas/jiangkailin/EasyEdit-main/examples/run_knowedit_llama2.py", line 207, in
editor = BaseEditor.from_hparams(hparams)
File "/scratch2/mas/jiangkailin/EasyEdit-main/../EasyEdit-main/easyeditor/editors/editor.py", line 59, in from_hparams
return cls(hparams)
File "/scratch2/mas/jiangkailin/EasyEdit-main/../EasyEdit-main/easyeditor/editors/editor.py", line 89, in init
self.tok = LlamaTokenizer.from_pretrained(self.model_name)
File "/home/jiangkailin/miniconda3/envs/EasyEdit/lib/python3.9/site-packages/transformers/tokenization_utils_base.py", line 2089, in from_pretrained
return cls._from_pretrained(
File "/home/jiangkailin/miniconda3/envs/EasyEdit/lib/python3.9/site-packages/transformers/tokenization_utils_base.py", line 2311, in _from_pretrained
tokenizer = cls(*init_inputs, **init_kwargs)
File "/home/jiangkailin/miniconda3/envs/EasyEdit/lib/python3.9/site-packages/transformers/models/llama/tokenization_llama.py", line 169, in init
self.sp_model = self.get_spm_processor(kwargs.pop("from_slow", False))
File "/home/jiangkailin/miniconda3/envs/EasyEdit/lib/python3.9/site-packages/transformers/models/llama/tokenization_llama.py", line 196, in get_spm_processor
tokenizer.Load(self.vocab_file)
File "/home/jiangkailin/miniconda3/envs/EasyEdit/lib/python3.9/site-packages/sentencepiece/init.py", line 961, in Load
return self.LoadFromFile(model_file)
File "/home/jiangkailin/miniconda3/envs/EasyEdit/lib/python3.9/site-packages/sentencepiece/init.py", line 316, in LoadFromFile
return _sentencepiece.SentencePieceProcessor_LoadFromFile(self, arg)
TypeError: not a string

Thank you very much for your interest in EasyEdit. We will investigate and fix this issue shortly.


output file can't be played everywhere. something missing

Hi,
I managed to get the muxing working but the output file seems to be corrupted in some way. it can be played with VLC and chrome on my computer. but it fails playing on my android phone or with windows media player, preview is not shown in windows folder and it can't be opened by https://gpac.github.io/mp4box.js/test/filereader.html
if I just remux with ffmpeg with copy, it works fine. ffmpeg adds something when I compare the ffprobes :
[mov,mp4,m4a,3gp,3g2,mj2 @<PHONE_NUMBER>84af40] Processing st: 0, edit list 0 - media time: 0, duration: 127492
[mov,mp4,m4a,3gp,3g2,mj2 @<PHONE_NUMBER>84af40] Unknown dref type 0x206c7275 size 12
[mov,mp4,m4a,3gp,3g2,mj2 @<PHONE_NUMBER>84af40] Processing st: 1, edit list 0 - media time: 0, duration: 483336
also mediainfo shows some extra lines for the remuxed ffmpeg version :

for global:
Overall bit rate mode                    : Variable
encoded-date and tagged-date
video :
stream-size , encoded-date and tagged-date
audio :
Bit rate mode                            : Variable
stream-size , encoded-date and tagged-date

I use avc1.420033 and mp4a.40.2
my 2 test files :
https://keny.toys/tmp/test_mp4muxer.mp4
https://keny.toys/tmp/test_remuxedffmpeg.mp4

did some more tests. fails on iOS (iphone, ipad and mac) with safari, quicktime and native read. works only with a desktop chrome and VLC

Alright, very interesting, I was able to reproduce the error. So, when I run the demo of this library, the resulting file actually runs perfectly in QuickTime and everything - but seems like I'm still doing something wrong, but only sometimes. Could you perhaps share how you're creating this file exactly? I remember when coding this library initially, files with highly varying frame rate refused to play in some players. But clearly, FFmpeg seems to be fixing something here, so I can try to inspect what it's doing differently.
So yeah, could you share a bit of the code that's generating this video, and also confirm that the demo is indeed working for you?

hi,
made a bunch of tests with the demo, changing all the parameters, and I think I've found the issue. it's linked to the avc: { format: 'annexb' } in the videoEncoder config. which was mandatory in chrome 113. I don't know exactly what is it for. it works without this option and it seems chrome 114 don't require it.
I will make some more extensive tests monday. I use your library in a complex thing that produces a great variety of resolutions, framerate, length, codecs. Will let you know about the results
thanks a lot

Was annexb required? I never set it and it worked fine for me - that's also what's done in the demo. But yes, keep me updated and we'll figure this out!

I've realized that without annex-b, I can open the file everywhere but I can't hear the sound if I'm not opening with chrome or VLC.
everything seems to be there and remuxing with ffmpeg fixes the problem. the demo with the microphone input is working fine so I'm trying to find out what's different

Thanks for doing some more research! So; is there anything this library should do or change? Or is this not a bug?

not a bug. I hope google will keep webcodec as it is now for h264/aac

Awesome. If you think this is an issue with Chromium though, make sure to report it!


Document changes in System.Random in .NET Core

Issue description
There is a substantial change in the behaviour of System.Random in .NET Core. This change is however not reflected in the documentation, so the claims provided by the documentation are incorrect.
Target framework

[x] .NET Core 2.x, 3.x, perhaps 1.x too
[ ] .NET Framework
[ ] .NET Standard

Details
The behaviour of the System.Random class in .NET Framework is that the parameterless constuctor  takes the seed value from the current system time (Environment.TickCount). This has been documented here: https://docs.microsoft.com/en-us/dotnet/api/system.random?view=netframework-4.8#instantiating-the-random-number-generator. This behaviour leads to the well-known issue: several random generators created in quick succession produce the same value sequences.
The behaviour has changed in .NET Core: now the initial seed value is randomized as well, so several random generator's instances produce different sequences even if created in quick succession. However the documentation doesn't reflect these changes: https://docs.microsoft.com/en-us/dotnet/api/system.random?view=netcore-3.0#instantiating-the-random-number-generator.
I think, System.Random is a very often-used tool, so refuting the traditional wisdom about its behaviour is important.

@vladd The section you pointed does seem to point out that the problem only arises on .NET Framework:
"However, on the .NET Framework only, because the clock has finite resolution, using the parameterless constructor to create different Random objects in close succession creates random number generators that produce identical sequences of random numbers."
"To avoid this problem, create a single Random object instead of multiple objects. Note that the Random class in .NET Core does not have this limitation."
However, given that you created this issue, it seems that this information about .NET Core vs. .NET Framework is not prominent enough.

@mairaw Should I transfer this issue to the API docs repo? Or since the information is already there, should we just close it?

I'll transfer. The information seems a bit hidden after the code block.


[1.0.0-alpha2] Swipeable tabs are not responsive to content

Expected Behavior
Tabs should be responsive to content
Current Behavior
Tabs are always 400px height
Steps to Reproduce (for bugs)
CodePen: https://codepen.io/anon/pen/BmvMVE
Your Environment
Version used: 1.0.0 alpha 2
Browser Name and version: Chrome 62.0.3202.94
Operating System and version (desktop or mobile): Windows 10

A possible solution or rather workaround is to set the tabs-content height to auto using js kind of like this (css will not work as the _setCarouselHeight-function will dynamically reset the height):
$('.tabs-content.carousel.carousel-slider').css("height","auto");

and then set the current active tab to relative position while leaving the others absolute:
.tabs-content.carousel.carousel-slider .carousel-item.active{
    position: relative;
}

Seems this is a continuing issue - https://github.com/Dogfalo/materialize/issues/4148, https://github.com/Dogfalo/materialize/issues/4159, among others but these seem to be the earlier issues

.tabs-content.carousel.carousel-slider .carousel-item.active{
position: relative;
}
.tabs-content.carousel.carousel-slider{ height: auto !important; }

A possible solution or rather workaround is to set the tabs-content height to auto using js kind of like this (css will not work as the _setCarouselHeight-function will dynamically reset the height):
$('.tabs-content.carousel.carousel-slider').css("height","auto");

and then set the current active tab to relative position while leaving the others absolute:
.tabs-content.carousel.carousel-slider .carousel-item.active{
    position: relative;
}

WORKS PERFECTLY!
It's nice and clean... Thanks @Luke5 +1
Just for reference... If you have your CSS loaded after materialize's like me, another way of doing it without jQuery is to directly put into the code:
.tabs-content.carousel.carousel-slider{ height: auto !important; }
Thank you so much

.tabs-content.carousel.carousel-slider .carousel-item.active{
position: relative;
}
.tabs-content.carousel.carousel-slider{ height: auto !important; }
the addition of the two worked for me

Thanks, this worked for me
.tabs-content.carousel.carousel-slider .carousel-item.active{
position:relative;
}
.tabs-content.carousel.carousel-slider{ height: auto !important; }
I added it in the materialize.min.css
PS.
there are  in the tabs


store: add assertion for error cases

Some failure errors are printed in debug format to the client. We should make assertions to ensure our output remains stable even in future changes.
@zhangjinpeng1987 PTAL

What error does this pr fix?

LGTM

@hicqu @wentaoxu PTAL

/run-unit-test

/run-all-tests

/run-all-tests

PTAL @AndreMouche

PTAL @overvenus

error[E0061]: this function takes 3 parameters but 2 parameters were supplied
--> src/storage/mod.rs:1212:9
|
1212 | /         ReadPool::new(&readpool::Config::default_for_test(), || {
1213 | |             || ReadPoolContext::new(None)
1214 | |         })
| |__________^ expected 3 parameters
|
::: src/server/readpool/mod.rs
|
57   | /     pub fn new<F, CF>(name_prefix: &str, config: &Config, context_factory_builder: F) -> Self
58   | |     where
59   | |         F: futurepool::Factory,
60   | |         CF: futurepool::Factory,
...    |
89   | |         }
90   | |     }
| |_____- defined here

@zhangjinpeng1987 Oh it's a merge mistake. Fixed. PTAL again.


"Go to Disassembly" support on Linux.

Add "memory transactions" in CoreApp.
Add transaction id to response handler.
Simplified SymbolsManager using new transaction id.
Add "ReadProcessMemory" helper for Linux and Windows in Utils.h.
"Go to disassembly" now works on Linux (right click on a function).

can you rebase this branch on top of master?


Dev/doc encrypt

Continuation of PR #917  & #243
Hope to get some advanced review on the PR as this is quite a significant addition,
What?
This PR add the feature for user to be able to Encrypt an unencrypted file.
Why?
Enables the encryption feature in this library
How?
  const pdfDoc = await PDFDocument.create();
  const existingPdfBytes = fs.readFileSync('test.pdf');
  const pdfDoc = await PDFDocument.load(existingPdfBytes);
  const page = pdfDoc.addPage();
  page.drawText('Creating PDFs in JavaScript is awesome!', {
    x: 50,
    y: 700,
  });

  pdfDoc.encrypt({
    userPassword: 'abcd',
    ownerPassword: '1234',
    permissions: { modifying: true },
  });

 // useObjectStreams true and false works
 // note on the following section on bugs on adobe acrobat reader
  const pdfBytes = await pdfDoc.save({ useObjectStreams: false });
  fs.writeFileSync('sample.pdf', pdfBytes);

Testing?
Manual testing by verifying the permission in PDF Reader application.
Added Integration test test19.ts and test20.ts in node
File should only be able to open and get decrypted in the PDF Reader if everything is implemented correctly.
Suggestions on testing is welcomed.
New Dependencies?
Crypto.js
Saslprep
Both are famous public libraries.
Screenshots
N/A
Suggested Reading?
Yes
Anything Else?
There are 2 areas that I would need help with on this PR.

How to verify that the build is working, as raised in #917

Currently, there is a bug when using pdfDoc.save({ useObjectStreams: true }), the file is not able to be opened in Adobe Acrobat Reader, but works fine on browser and other PDF readers.

Checklist

[x] I read CONTRIBUTING.md.
[x] I read MAINTAINERSHIP.md#pull-requests.
[ ] I added/updated unit tests for my changes.
[x] I added/updated integration tests for my changes.
[x] I ran the integration tests.
[x] I tested my changes in Node, Deno, and the browser.
[x] I viewed documents produced with my changes in Adobe Acrobat, Foxit Reader, Firefox, and Chrome.
[ ] I added/updated doc comments for any new/modified public APIs.
[x] My changes work for both new and existing PDF files.
[x] I ran the linter on my changes.

@PhakornKiong thanks for reopening this PR! It's pretty hefty, so it'll be awhile before I can look through all this.
Sidenote: If you'd like for folks to be able to use your work without waiting, you might consider making a separate library that people can pass a PDFDocument object to for encryption.

@PhakornKiong thanks for reopening this PR! It's pretty hefty, so it'll be awhile before I can look through all this.
Sidenote: If you'd like for folks to be able to use your work without waiting, you might consider making a separate library that people can pass a PDFDocument object to for encryption.

No worries, take your time.
It would be difficult to spin off as a separate library as the change rely heavily on the pdfWriter to write the encrypted content into the file.

This looks awesome; it would be great to see this pulled in and also the ability to decrypt pdfs =] (dealing with that now, trying to decide how to handle it)

Hi, how can I use it in a browser environment?
It seems that npm run build creates a dist compatible only with node
In my case the inclusion of the saslprep library produced a few lines of code that point directly to the local filesystem to access code-point.mem
Can you add some instruction on how to use this mod in standard web pages?
Thanks and congratulations again for the excellent work done

Hello,
I'm very interested by this one. Any chance to see this feature added in the near future ?
Thanks.

Hey @Hopding @PhakornKiong, this feature is a great and i see how it could re written outside of pdf-lib so i can keep using the latest release without doing any refactoring in the future, but before i do is it going to be added in the near future? Not a problem at all just curious.

Hey @Hopding @PhakornKiong, this feature is a great and i see how it could re written outside of pdf-lib so i can keep using the latest release without doing any refactoring in the future, but before i do is it going to be added in the near future? Not a problem at all just curious.

At this rate, i doubt it will be merge anytime soon. Might be a great solution to those that need this functionality.
How would you write it outside of pdf-lib ? (Just curious, since i relied heavily on the parser in pdf-lib)

This idea is really more a guess. But a while back I found a solution on here to update spot colours in a pdf and has been expanded for other features. So using the research from your blog plus the pdf specification I'm thinking that might work.

This would be incredibly useful for us. Any idea when it might get merged?

Hi～ @PhakornKiong  the PR code I use cannot be opened in the adobe reader after encrypting the pdf in my workplace, but it can be opened in other pdf readers (the browser can be opened)

This feature would be immensely useful. I'm voting to move forward with merging.  Any ETA?

@PhakornKiong What an incredible job!
Maybe, while this processes until merge, can you release your fork on npm?

+1 for adding it in. Anybody from the repo bossmen available for a review? @MatheusHCP I see it was approved, could we merge it in?
@PhakornKiong Some merge conflicts here, a small fix for man, a giant PDF encryption leap for all developer kind. Plz resolve.

+1 for adding it in. Anybody from the repo bossmen available for a review? @MatheusHCP I see it was approved, could we merge it in? @PhakornKiong Some merge conflicts here, a small fix for man, a giant PDF encryption leap for all developer kind. Plz resolve.

Ok, let me do it this weekend.

Thanks so much @PhakornKiong!

This looks awesome; it would be great to see this pulled in and also the ability to decrypt pdfs =] (dealing with that now, trying to decide how to handle it)

decrypt is super helpful as well, any updates on this? thanks!

@PhakornKiong I fork your repo and build, but error about fs founded, would u kindly tell me how to build correctly?

Would love to get this implemented. As of right now, there are not a lot of ways to encrypt an already existing pdf using node.

@PhakornKiong
Still some merge conflicts 🥲. Or are there some new ones now? Let's get this party started please.
@gonyulian415 are you using the correct version of nodejs?

Hey @peterpoliwoda, I faced 2 issues while building Phakorn's fork for browsers.

The fork uses crypto-js and saslprep. Rollup config bundles them too when building the package for browsers using (yarn build:esm or yarn build:umd)
Even if you do not bundle crypto-js(as most browsers support it), saslprep has statements like const fs = require('fs') which throws errors in browser.

Can you guide me how to bundle the fork to make it work on browser?

Awesome! Well done sir!

Hell

Awesome! Well done sir!

Hello, I want to ask this feature will be merge or still is review?

Closing this? So is there no hope for this to go in? What's happening?

I forced pushed and it automatically closed it, probably have to do with history

reopen

Hi @PhakornKiong, with encryption enabled (ownerPassword and permissions only), document's metadata seems to be encrypted as well. Is this expected result or a bug?
Without Encryption	With Encryption

I'm not 100% sure, would need to check the standard to verify.
What tools do you use to display this? Maybe i can take a peek
Left is unencrypted, middle is encrypted by ilovepdf, right is from the PR

What tools do you use to display this? Maybe i can take a peek

Simply viewing the pdf in MS Edge (chromium to be specific).

Left is unencrypted, middle is encrypted by ilovepdf, right is from the PR

Both the images of the encrypted pdf are not showing the metadata.
Also from Adobe Acrobat Reader

I'm of the opinion that the default is to encrypt metadata. So probably i would not change this implementation here.

https://opensource.adobe.com/dc-acrobat-sdk-docs/pdfstandards/PDF32000_2008.pdf

The encryption is grate, however I’m facing the same issue as PROxZIMA ,metadata are also encrypted, and I can’t reset team by pdfDoc.setTitle(“”) (same as setAuthor, setSubject)

In most PDF encryption apps and libraries there is an option to enable/disable the encryption of document metadata.  I would recommend adding this option so that the developer can choose to encrypt the metadata or leave as-is.

The encryption is grate, however I’m facing the same issue as PROxZIMA ,metadata are also encrypted, and I can’t reset team by pdfDoc.setTitle(“”) (same as setAuthor, setSubject)

I try to clean the meta info and save as a temporary pdf then reload the clean meta pdf  for encrypted. You know what, the meta info recovered, I just don’t know how it happened, is crazy 😂

The encryption is grate, however I’m facing the same issue as PROxZIMA ,metadata are also encrypted, and I can’t reset team by pdfDoc.setTitle(“”) (same as setAuthor, setSubject)

I try to clean the meta info and save as a temporary pdf then reload the clean meta pdf for encrypted. You know what, the meta info recovered, I just don’t know how it happened, is crazy 😂

It works If I remove the meta by adobe reader instead of pdf-lib

The encryption is grate, however I’m facing the same issue as PROxZIMA ,metadata are also encrypted, and I can’t reset team by pdfDoc.setTitle(“”) (same as setAuthor, setSubject)

I try to clean the meta info and save as a temporary pdf then reload the clean meta pdf  for encrypted. You know what, the meta info recovered, I just don’t know how it happened, is crazy 😂

Can you provide steps to reproduce this? If it works then this might be more straightforward to implement.
Else the solution would require substantial change in how the PDFwriter keep tracks of all the object, which I haven't really got into

`const pdfName = "./somePDFWithMeta.pdf";
//step 1 remove meta and save to disk
const pdfContent = fs.readFileSync(pdfName);
const pdfDoc = await PDFDocument.load(pdfContent);
//wipeout meta
pdfDoc.setAuthor("");
pdfDoc.setTitle("");
pdfDoc.setSubject("");
pdfDoc.setKeywords("");
const pdfBytes = await pdfDoc.save();
fs.writeFileSync(cleanMeta.${pdfName}, pdfBytes);
//step 2 encrypt PDF
const pdfContent2 = fs.readFileSync(cleanMeta.${pdfName});
const pdfDoc2 = await PDFDocument.load(pdfContent2);
pdfDoc.encrypt(encryptConfig);//password and permissions
const pdfBytes2 = await pdfDoc.save({ useObjectStreams: false });
fs.writeFileSync(encrypted.${pdfName}, pdfBytes); `
A simple step for your reference.
The meta info is clean in cleanMeta.pdf however it recovered and encrypted in encrypted.pdf

Can anyone know when this can be merged?. Im desperately looking for this PR to get it merged.
Thank you.

If I get a working PR on @cantoo/pdf-lib we can merge it. But it has to be working out of the box.

stack: 'TypeError: pdfDoc.encrypt is not a function facing this issue while adding encrption to the pdf document. How to resolve tis issue

stack: 'TypeError: pdfDoc.encrypt is not a function facing this issue while adding encrption to the pdf document. How to resolve tis issue

Test on @cantoo/pdf-lib

can confirm encrypt() works on @cantoo/pdf-lib

While @cantoo/pdf-lib has the encrypt method, it seems to use 40-bit RC4 and not AES.


mssql: schema builder - attempt to drop default constraints when changing default value on columns

Depends on #4319
When changing a default value on a column in mssql, the previous default constraint needs to be dropped before a new one can be assigned. This PR attempts to drop the constraint and then adds the constraint via a separate query:
DROP CONSTRAINT [constraint_name]; //does some look up to find the constraint name
ALTER TABLE [table] ALTER COLUMN ...; // other altering of column
ALTER TABLE [table] ADD CONSTRAINT [constraint_name] DEFAULT [defaultVal] FOR [column_name]; // applies default constraint

Help required:
I've tried to get a grasp of how the internals of the builder work for this and  implemented this in a way that I think makes the most sense.
Under some circumstances this change can result in a needless alter query being run when the only change to a column is its default value; in this case we should only be running the drop and add constraint queries, but I'm not clear on how best to skip the other query that is produced under these circumstances.
To do:

[x] Create integration tests to cover this feature specifically

      -ALTER TABLE [users] ADD CONSTRAINT [users_foo_default] DEFAULT test FOR [foo]
      +ALTER TABLE [users] ADD CONSTRAINT [users_foo_default] DEFAULT 'test' FOR [foo]

Looks like escaping changed.

Thanks!

Released in 0.95.0


Bump T1 dependencies

Updated: nix/t1/dependencies/_sources/generated.json
nix/t1/dependencies/_sources/generated.nix

@sequencer Seems like the arithmetic module required a rewrite with latest Chisel.

Arithmetic module is now not planned for Chisel migration.


3 new notebooks on making reference files

One is specifically for flat fields. The other tries to give a more general recipe for known reference files, and the third is basically a summation of what the schema expects.

I haven't fully reviewed this yet, but I wanted to quickly mention that all the dq arrays need to go to uint32 (I am making that change in RAD/RDM/RomanCAL). This came up as I am working on DQ code presently.


07/09/2018 21:54 : Create Pull Request for branch prTesting_2018070921 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Search times out for long queries

searching for small queries like nerijs  or nerijs pixel works, but not for nerijs pixel art xl in https://huggingface.co/datasets/multimodalart/lora-fusing-preferences
It seems to be due to time out because long queries take longer to run

I think increasing the delay before time out should be fine @AndreaFrancis no ?
Maybe cc @severo if you know the place in moon landing to modify it

see https://github.com/huggingface/moon-landing/pull/7579 (internal)

https://huggingface.co/datasets/multimodalart/lora-fusing-preferences now shows
Error code:   JobManagerExceededMaximumDurationError

Refreshing it

Note that this dataset is also reported here: https://github.com/huggingface/datasets-server/issues/1859 for having too big row groups

same error:  JobManagerExceededMaximumDurationError

It seems to be working now for https://huggingface.co/datasets/multimodalart/lora-fusing-preferences. Maybe the worker's memory increase helped.

Could we close this issue?

Yes OP is fixed, thanks !


master: pkg: rpy2

let
  mach-nix = import (builtins.fetchGit {
    url = "https://github.com/DavHau/mach-nix.git";
    ref = "refs/heads/master" ;
    rev = "b2aede9aa2f89a7bb150d5927812b3c3e505dcf6"; # new
  } ) { } ;

  nixPkgs = (import mach-nix.nixpkgs.path {config= { allowUnfree = true;  };} ); # permittedInsecurePackages = [ "openssl-1.0.2u" ];
  py = nixPkgs.python37Packages ;
  R = nixPkgs.rPackages ;

  pkgsNix = import /home/ae/ae_share_VM/z_DS_setup/home-manager/nix/nixPkgs.nix nixPkgs ;

  pkgsR = with R; [
      dplyr
      data_table
      DataExplorer
  ];

  pyEnv = mach-nix.mkPython rec {
    pkgs = nixPkgs;
    python = nixPkgs.python37;
    requirements = ''
    ipython
    jupyterlab
    pandas
    seaborn

    rpy2
    '';
    _.rpy2.buildInputs.add = pkgsR ;

    providers = {
      rpy2                = "nixpkgs" ;
    };
  };
 envShell = nixPkgs.mkShell rec {
  buildInputs =
    [
    (pyEnv )
     ]
  ;

};
in envShell

error: while evaluating the attribute 'buildInputs' of the derivation 'nix-shell' at /nix/store/n2n1sxx75mfwwnvrm0akifrafympfblh-nixpkgs/pkgs/build-support/mkshell/default.nix:28:3:
'_mkPython' at /nix/store/4i8400z2nj3hffja31r3wg4j0p8gawa0-source/default.nix:253:5 called with unexpected argument 'pkgs', at /nix/store/4i8400z2nj3hffja31r3wg4j0p8gawa0-source/default.nix:250:87

I deprecated the pkgs argument recently. pkgs now have to be passed during import. I also now added a proper deprecation error message. I will release a new version soon and adapt all docs.
This here builds:
(To bring your own nixpkgs, un-comment line 7)
let
  mach-nix = import (builtins.fetchGit {
    url = "https://github.com/DavHau/mach-nix.git";
    ref = "refs/heads/master" ;
    rev = "aa0d7cb6b3dc4468c3229083eedea01a6f4a7ecd"; # new
  }) {
    # pkgs = import <nixpkgs> {};
  } ;

  pkgs = mach-nix.nixpkgs;

  pkgsR = with pkgs.rPackages; [
      dplyr
      data_table
      DataExplorer
  ];

  pyEnv = mach-nix.mkPython rec {
    python = "python37";
    requirements = ''
    ipython
    jupyterlab
    pandas
    seaborn

    rpy2
    '';

    _.rpy2.buildInputs.add = pkgsR ;

    providers.rpy2 = "nixpkgs" ;
  };
  envShell = pkgs.mkShell rec {
    buildInputs = [
      pyEnv
    ];
  };
in envShell

PS: I'm curious if the R packages actually work. In that case, we should add it to the examples.

mport rpy2.robjects as robjects
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/nix/store/n068mqbr9kq10sw5cmaj77lpg6kyl7fb-python3-3.7.9-env/lib/python3.7/site-packages/rpy2/robjects/__init__.py", line 19, in <module>
    from rpy2.robjects.robject import RObjectMixin, RObject
  File "/nix/store/n068mqbr9kq10sw5cmaj77lpg6kyl7fb-python3-3.7.9-env/lib/python3.7/site-packages/rpy2/robjects/robject.py", line 10, in <module>
    rpy2.rinterface.initr_simple()
  File "/nix/store/n068mqbr9kq10sw5cmaj77lpg6kyl7fb-python3-3.7.9-env/lib/python3.7/site-packages/rpy2/rinterface.py", line 799, in initr_simple
    status = embedded._initr()
  File "/nix/store/n068mqbr9kq10sw5cmaj77lpg6kyl7fb-python3-3.7.9-env/lib/python3.7/site-packages/rpy2/rinterface_lib/embedded.py", line 119, in _initr
    raise ValueError('openrlib.R_HOME cannot be None.')
ValueError: openrlib.R_HOME cannot be None.

I tried with a few different expressions but cannot reproduce the error. Maybe it's due to some recent improvements.
Please also see the example I added here: https://github.com/DavHau/mach-nix/blob/master/examples.md#r-and-python

Closing due to inactivity and me not being able to reproduce. Feel free to add more info in case this problem still exists.


Commit conda-lock.yml to repository

It would be useful to commit the unified lockfile to this repository in addition to the platform-specific
https://github.com/CryoInTheCloud/hub-image/blob/main/conda-linux-64.lock
This would facilitate installing the cryocloud environment (at least the conda-forge part) on different systems
https://github.com/ICESAT-2HackWeek/website-2024/pull/17#discussion_r1706100624

Maybe time for me to pick up on https://github.com/CryoInTheCloud/hub-image/pull/14 again :laughing: Do we want both the conda-lock.yml file and conda-linux-64.lock file?

Oh wait actually, #14 might have been chewing on too much as it was trying to install from the conda-lock.yml via a postBuild script. I can open a separate smaller PR for just adding a conda-lock.yml file.

Yeah for starters just linux-64 conda-lock.yml in addition to conda-linux-64.lock would be useful.
conda-lock lock -p linux-64
conda-lock render -k explicit -p linux-64

Looks like other platforms aren't happy with a few of the packages:
Could not lock the environment for platform osx-64
Could not solve for environment specs
The following package could not be installed
└─ ocl-icd-system does not exist (perhaps a typo or a missing channel).


add cluster monitoring label to applications namespace

Add cluster monitoring label to redhat-ods-applications

https://issues.redhat.com/browse/RHODS-7978
How Has This Been Tested?
verified namespace label with live-builder image
Testing Instructions:

install live builder image
verify redhat-ods-applications has the label openshift.io/cluster-monitoring=true

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] For commits that came from upstream, [UPSTREAM] has been prepended to the commit message.
[x] JIRA link(s): https://issues.redhat.com/browse/RHODS-7978
[x] The Jira story is acked.
[x] Live build image: quay.io/vedantm/rhods-operator-live-catalog:1.28.0-7978
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work.

/approve

I think it still needs qe-approve label to auto-merge


Loading pre-rendered peaks display issue?

Help please, what causes this to happen with pre-rendered peaks?

data is not normalized, use the wavesurfer option or make sure the peaks are between -1 and 1

@aburai Setting normalize: true results in no waveform shown at all when loaded with peaks. Confirmed, peaks are between -1 and 1.
I'm using the following command to create the peaks:
var pcmData = '[' + wavesurfer.backend.getPeaks(500, 0, 500) + ']';

ok, was my first guess
i don't know why you convert the peaks to string, but without more code i have no idea

@aburai Thank you - The issue was because it was being stored as a string and not an object, I hadn't realised this would have affected it. I was putting the peaks in to pcmData to then ajax them into a mySQL database to be stored with the specific track. When retrieving the peaks I just had to do JSON.parse(pcm) and it displayed correctly.
Would it be more efficient to store the peaks data in a .json file?

more efficient, maybe not, but it is my favorite way to store the peaks
i use audiowaveform on the server side to create the peaks and other audio metadata


fix: update to latest ddev (v1.22.7)

I am still getting an error when running ddev drush si demo_umami -y

Summary by CodeRabbit

Chores

Updated PHP version to 8.3 and database version to 10.6 to enhance performance and security.
Improved post-database import processes for better data handling and application updates.
Updated the Gitpod base image and tools for a more current development environment.

This is also resolved by #146

And of course it's DDEV v1.23.0 now.

fixed by #146


Use SVProgressHUD wisely! ...

From the readme:

Use SVProgressHUD wisely! Only use it if you absolutely need to perform a task before taking the user forward. Bad use case examples: pull to refresh, infinite scrolling, sending message.

Can you explain why?
It is a progress HUD so loading something over the network is a viable case for using such library.

I agree that one could think thats a viable use case. However, in my opinion, loading from something over the network is one of the worst cases where to use such a library. This is due to the following reasons: A user might be one a bad network connection. Then loading of data could take a long time (sometimes multiple seconds). Blocking the UI meanwhile using the HUD is not good. Here a user could do some other tasks in your app while waiting for the data to arrive, e.g. in a Twitter client a user should be able to read the timeline (with old tweets) while waiting for new data. Here i would use [UIApplication setNetworkActivityIndicatorVisible:] to indicate loading in the background. I would only show the HUD when for example your updating the UI (e.g. after loading the tweets) and the user is explicitly not allowed to interact.

@honkmaster Interesting.. A while back I was using MBProgressHUD, and attached a UITapGestureRecogniser to it. Whenever the user tapped on the HUD, I cancelled the network connection and got out of the user's way.

You could do the same using the SVProgressHUDDidTouchDownInsideNotification. However, think about your iOS-experience. Have you seen any popular, network-connected app doing something like this (i don't think so)?

Yeah you're right..
I'd be interested in finding the right solution for these issues though, it has never crossed my mind that its bad UX.

As mentioned before, my usage:

Use SVProgressHUD wisely and only if you want to show progress and prohibit user interaction (by using a background mask)
Perform loading and all hardcore tasks on a background thread
When loading network data show the networkActivityIndicator

:+1: Alright. Thanks for responding quickly.


[Support] How to resolve foreing keys Restangular

Hi, it's me again, i have this technical concern of how to solve the foreing keys with Restangular. I have for example this json that i get from my RESTful API
{
    id: 43,
    users_scheduler_id: 1,
    created_at: "2016-04-24 19:34:12",
    updated_at: "2016-04-24 19:34:12"
}

there are a lot  of them that i get from a getList(), but how can i show in angular the name of the users_scheduler_id instead of it's id

Hi,
Can you please show a sample code.. I'm not sure I get it

sure, i'm using:
this.appointments = this.API.all('apointments').getList().$object;
to get an array of objects containing:
{
    id: 43,
    start_datetime: "2016-04-18 00:00:00",
    end_datetime: "2016-04-18 00:15:00",
    notes: "",
    users_scheduler_id: 1,
    is_urgent: 0,
    is_unavailable: 0,
    users_provider_id: 1,
    client_id: 2,
    id_services: 1,
    created_at: "2016-04-24 19:34:12",
    updated_at: "2016-04-24 19:34:12"
}

and the values that have an id in the name like: users_providers_id and id_services are foreing keys from other tables.
users_scheduler_id and users_providers_id are from the Users table
client_id is from the Clients table
and id_services is from the Services table
and in my angular app when i get the array of objects, i want to show the name of the 3 above instead of the foreing key, cause when i click a card for example i need those details, like for the name of the client, and what service is

If I understand correctly,  this is something that needs to be fixed from the API
Official docs tutorial

Thanks!


Emmet's inline completions works unintentionally in JSX strings literal

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

VS Code Version: Version: 1.69.0-insider (Universal) Commit: f361c5b71d6676cfc6de97cdb1cc40b08bf7d994
OS Version: macOS 12.4

Steps to Reproduce:

Create JSX (TSX) File
type import Hoge from '../../a'
type ../ between / and a

https://user-images.githubusercontent.com/1955233/175196045-837742fd-fb4b-4ab1-a5a1-b66dbcdd10e4.mp4
Emmet's inline-completions works even in areas enclosed by quotes and seems a little noisy.
This happens not only in import from statement, but also when writing simple strings and spread operators.
ref: https://github.com/microsoft/vscode/pull/151559

The completions are the same as those suggested by the completion item provider aka the non-inline completion provider. But, with inline completions on, it's even more likely for users to see noisy suggestions if they only see one completion rather than a list of completions.

Labelling this as a duplicate of #51537