Different Route Servers to advertise routes to Virtual Network Gateways and to VNets

My feedback is a request for additional information in this theoretical architecture, and a sanity check on prescribed configuration.
Specifically in the titled section.
I was investigating this architecture because I have a hub-and-spoke azure network + AzureVmWare + onprem environments and need all traffic from Azure spoke to traverse the firewall in the hub.
I am trying to understand the value add of including another route server to manage the default route of the spoke vnets.  It would be great if this architecture completely removed the need for UDRs on the spoke vnets - but it seems like this relatively complicated architecture only removes a single route from a required route table... <IP_ADDRESS>/0.
My understanding is that the routeserver#2 is used to propagate <IP_ADDRESS>/0 to all of the spokes it is peered with.
In this configuration, the spoke networks will all have an injected default bgp route <IP_ADDRESS>/0 that will send all 'unknown' traffic to the Firewall.  This would include internet outbound traffic, as well as any routes known on the VPN gateway(s) in the hub.
The value add here is that I no longer need to add <IP_ADDRESS> -> fw on the spoke vnet route table(s), right?
However, I will still need a route table on the gateway subnet to force incoming (and return traffic) to flow through the firewall.  I believe this is true regardless if the incoming traffic needs to be filtered by the firewall, because otherwise, there will be an asymmetric route when traffic is routed from the hub gateway over the VNET peering.
And, referencing the RouteServer FAQ:

I will still need UDRs on every spoke subnet(s) to force Virtual Network and VNET peered traffic to go through the firewall, right?
This makes the previously mentioned value pretty low.  Especially considering the route server is ~$300/month.
And even if I didn't have the requirement to filter intra-azure vnet/spokes through the firewall, i could accomplish this by disabling BGP propagation on the spoke route tables and advertising <IP_ADDRESS> to the firewall, for free... right?
If there is a solution or architecture that removes the need for Route Tables and UDRs for every system route on the peered spokes, that would be great.  Especially if is dynamic and requires little future configuration.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 9a0ed7db-aadb-494f-15fa-e51f279b6b8d
Version Independent ID: 45b4e0e9-422c-87a3-c64e-bf70ec059eb5
Content: Default route injection in spoke virtual networks - Azure Route Server
Content Source: articles/route-server/route-injection-in-spokes.md
Service: route-server
GitHub Login: @halkazwini
Microsoft Alias: halkazwini

@Jared975
Thanks for your feedback! We will investigate and update as appropriate.
I've delegated this to @halkazwini, a content author, to review and share their valuable insights.

Hi @Jared975
Thank you for your feedback, and apologies for not get back to you earlier.
If you want to use spoke subnets in the same virtual network where you have the firewall, then, yes you need to use UDRs.
The topology in https://learn.microsoft.com/en-us/azure/route-server/route-injection-in-spokes#different-route-servers-to-advertise-routes-to-virtual-network-gateways-and-to-vnets doesn't use spoke subnets in the same virtual network, instead, it uses spoke VNets with Use the remote virtual network's gateway or Route Server disabled when peering with the Hub VNet where the firewall is.
I'll close this GitHub issue for now, but feel free to comment as necessary. #please-close


Fix lint

Describe the addition

Context

Proposed Solution

Done already, ongoing thing but most of the egregious linting rules have been taken care of.


Example has implicit dependency of 'hof-controllers'

https://github.com/UKHomeOfficeForms/hof-bootstrap/blob/master/example/apps/example-app/index.js#L3
hof-controllers should be in https://github.com/UKHomeOfficeForms/hof-bootstrap/blob/master/example/package.json

It does: https://github.com/UKHomeOfficeForms/hof-bootstrap/blob/master/package.json

but the example/package.json doesn't, it coincidently works in node 6 because of its flat dependencies

fair

Good spot @chrisns - will update.  Just as a note though - HOF only works with npm>=3 due to a bug down the line in frontend toolkit so there will always be a flat directory structure (al least until this is fixed)

@joefitter it'd be coincidently there but potentially not the version you're expecting :)

I know - it needs to be added, I was just pointing out as a side note we aren't able to use npm<3, more as a reminder to myself that there is an issue that should be fixed!


Getting the latest tweet of a user

How would i get the latest tweet of a specific user?
Im asking this because appereantly the endpoint:
https://api.twitter.com/1.1/statuses/user_timeline.json?user_id=12345678
doesnt always return the latest tweet.
(https://developer.twitter.com/en/docs/twitter-api/v1/tweets/timelines/guides/working-with-timelines) (The max_id parameter)

Hello there.
My understanding is that you have a problem with retrieving the latest tweets published by a specific user.
As mentioned in the article you pointed towards, you should not use the MaxId parameter or the SinceId parameter.
When both of these property are set to null you should receive the latest tweets. This logic is actually tested in Tweetinvi End to End tests.
If you are actually experiencing this issue at the moment, I would suggest communicating with Twitter about it on https://twittercommunity.com/ and sharing a link to your question here so that I can follow up on it.
Cheers,
Linvi

Ah okay, it works. THanks for your help


Badge text colour does not show well

The default text colour for badges seems to be some shade of grey, possibly the secondary colour. It does not show well with almost all of the background colours. Maybe a lighter shade would be better?

fixed in 0.5.1 / https://github.com/saltcorn/tabler/commit/45c930a2005a11d6e010b1caaad892529c352031


Fuel Codes | Prepopulating previous version data causes partially renewable box to always be checked |1|

Background:

IDIR user
Location: Administration > Fuel Codes > New Fuel Code
Action: Adding a new version of an existing fuel code and choosing to pre-populate the data from the previous version.
Note: This issue only affects fuels that can be partially renewable (e.g. biodiesel, ethanol, HDRD, etc.) but are not partially renewable in this case (i.e. the "Is partially renewable?" checkbox is not checked).

Issue:

When adding a new version of an existing fuel code, and choosing to pre-populate the data from the previous version, the **Is partially renewable?" checkbox is automatically checked regardless of if the previous version was indicated as being partially renewable (regardless if the checkbox was checked in the previous version)

Desired state:

When pre-populating data from a previous version of a fuel code, the "Is partially renewable?" checkbox should only be checked if the previous version was indicated as being partially renewable (i.e. the checkbox was checked for the previous version).  https://trello.com/c/9iS7d1DF/1571-fuel-codes-prepopulating-previous-version-data-causes-partially-renewable-box-to-always-be-checked-1

 Fuel Codes | Prepopulating previous version data causes partially renewable box to always be checked |1|

done in sprint quasar


EMBR-3940 integration test basic test app

Adds a basic test app for use in integration testing (https://github.com/embrace-io/embrace-react-native-sdk/pull/25)
Not too much to see here, these are mostly auto generated files from initializing a new RN app + the integration steps to add the Embrace SDK. For now just exposes a simple "End Session" button, will layer in more functionality as the need arises when adding tests:

I'll take a look at why the GHA are failing


Kernel, Where to start.

As promised on twitter.
Here is what I sent to a friend of mine that was interesetd in writting a kernel for R.

The easiest to start would be to read this blog post from the author of the haskell kernel.
https://www.fpcomplete.com/school/to-infinity-and-beyond/older-but-still-interesting/ihaskell
More formal doc is here:
http://ipython.org/ipython-doc/dev/development/messaging.html
And a few implementation examples.
actives:
https://github.com/ipython/ipython
https://github.com/gibiansky/IHaskell
https://github.com/JuliaLang/IJulia.jl
https://github.com/BayardRock/IfSharp
less active.
https://github.com/minad/iruby
https://github.com/mattpap/IScala
https://github.com/jpraher/ipython-ocaml-kernel
https://github.com/takluyver/igo

If you have any question your are welcome on IPython ML, github...etc
None of the core dev are really experienced javascript developers, so we would gladly appreciate feedback from js community

This is nice loading data via spark and do d3.js graphs out of it http://zeppelin-project.org/


pread returns EINVAL in linux

Run simple example in PR https://github.com/rust-lib-project/rocksdb-rs/pull/6: cargo run --example simple_example, and program will failed in Engine::open. The detail errno is EINVAL.
According to man 2 read:

EINVAL fd  is attached to an object which is unsuitable for reading; or the file was opened with the O_DIRECT flag, and either the address specified in buf, the value specified in count, or the file offset is not suitably aligned.

I noticed that get_current_manifest_path will invokes FileSystem::read_file_content, which eventually invoke AsyncRandomAccessFile::open, and O_DIRECT is added to open, but there doesn't seem to be any alignment in read_file_content.

I have disable the direct io until someone could refactor the async file system to give read size alignment.


AppIndicator support needed

Hi,
I'm trying to use networkmgr in Enlightenment DE (fresh beta). Evidently, the tray icon is absent as their systray module only supports AppIndicator protocol now.
I know that this protocol is not supported in GTK and there is no future plans to support it. Fortunately, Linux Mint offers a solution - XApp.StatusIcon (see https://blog.linuxmint.com/?p=3795). They claim this solution to be "cross-desktop and cross-distribution". It would be nice to add AppIndicator support to networkmgr using this approach, so it could be used on any modern DE.
Peter

I'm trying to get it working with libappindicator in my branch appindicator started by 8723aeedea87009ec8ebceacbe2770a28932109c

The tray was deprecated in the later GTK3 version and removed in GTK4. We will have to replace it with an AppIndicator.


Does the spec step of Gauge consider adding conditional judgments? example: If you are not logged in, then the Run login scenario If you are already logged in, skip the login scenario

Describe the bug
Does the spec step of Gauge consider adding conditional judgments
for example:
If you are not logged in, then the Run login scenario
If you are already logged in, skip the login scenario

Don't think there is any specific way in Gauge can do this. But this use case can be easily achieved by handling conditions in step implementations I believe. For this particular case on Login scenario have you tried using hooks ? Logging in beforeScenario or beforeSpec might solve this case.


Questions about running codes

I run 'bash compress_llama.sh' but I got an error
FileNotFoundError: [Errno 2] No such file or directory: 'whitening/jeffwan_llama_7b_hf_whitening_0.2.pt'
In the bash, the ratio is 0.2 and the evaluation command requires jeffwan_llama_7b_hf_whitening_0.2.pt, but when saving models the name of saved model is 'jeffwan_llama_7b_hf_whitening_only_0.8.pt'. Is the bash command wrong?
I revised the second and third command to my saved model name 'jeffwan_llama_7b_hf_whitening_only_0.8.pt'. Am I correct?
Thanks

Yes, the name should be modified to "jeffwan_llama_7b_hf_whitening_only_0.8.pt". Thanks for pointing out the mistake.


Bad bootstrap_config.sh

In 1.9.2, I ran ./bin/bootstrap_config.sh  and chose '512' and 'standalone'.  Tserver fails to start with
2019-02-27 12:45:49,681 [tserver.TabletServer] ERROR: Uncaught exception in TabletServer.main, exiting
java.lang.IllegalArgumentException: Maximum tablet server map memory 83,886,080 block cache sizes 24,117,248 and mutation queue size 52,428,800 is too large for this JVM configuration 129,761,280
	at org.apache.accumulo.tserver.TabletServerResourceManager.<init>(TabletServerResourceManager.java:186)
	at org.apache.accumulo.tserver.TabletServer.<init>(TabletServer.java:408)
	at org.apache.accumulo.tserver.TabletServer.main(TabletServer.java:3206)
	at org.apache.accumulo.tserver.TServerExecutable.execute(TServerExecutable.java:33)
	at org.apache.accumulo.start.Main$1.run(Main.java:130)
	at java.lang.Thread.run(Thread.java:748)

Setting tserver.memory.maps.max to 30M in the accumulo-site.xml got past the issue

This script was removed in 2.0 in favor of improved self-documented example configs. So, this only needs to be fixed in 1.9.3.

PR in #1055


I've been getting a connection refused after a few seconds after connection to firebase

So i've been working on a project doing IoT based IR transmitter using ESP32 C3 Super Mini with a couple of Firebase.getInt and Firebase.setInt in a 1000 ms interval.
though since my code doesn't work and keep getting the connection refused error, i've decided to try the example "basic" from the firebase ESP32 client library and try to recreate the error.

i've tried the original "basic" code and let it run for a while, and it turns out fine even up until 500+ repetitions. then after that I tried changing the millis delay to 1000ms instead of 15000ms the original code has. it runs up until like 15 repetition and then gave the "connection refused" error. so i wonder if there's some kind of specific minimal delay the library can handle that makes the rapid change doesn't work?

if you need me to provide more details or try something regarding the issue, i'd be happy to provide it. Thanks

This is the repo for ESP8266.
It is because the data bandwidth quota limit or network issue.
If you want to get the instant data update, use stream instead.
https://github.com/mobizt/Firebase-ESP8266/tree/master/examples/DataChangesListener


@conda-forge-admin,please add bot automerge

@conda-forge-admin,please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/spechomo-feedstock#6.


🐛 Customize only provider deployments

What this PR does / why we need it:
If there are several deployments in the manifest file, operator tries to apply provider changes to all of them, which leads to a failure.
In order to prevent this, we will customize only deployments with "cluster.x-k8s.io/provider" label.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #388

/cherry-pick release-0.8


fix(example) with-graphql-tools launch on wrong port number

I just notice that example use 3001 instead of 3000 that tell in console 😐

Ha! Thanks! 👍


Only shows date in header

Doesnt show On this day in....

Could you please provide some more informations? What language are you using? What is exactly shown and what do you expect? Thanks.

English and i thought for sure this application used to say "On this day on..." then the date but i could be mistaken lol

This module just takes the title from Wikipedia (https://en.wikipedia.org/wiki/Main_Page in your case) and prints it as the header. Feel free to open a PR to improve it with this feature. You could also try to use a custom function in your module config in the header property.

oh i see, thank you!


Which article does Fast-Meta-Optimizer come from?

Thanks for your code, i learn a lot from it. But i can't find  the source of Fast-Meta-Optimizer. It looks like meta-SGD, but not common entirely. Please tell me the source of this method, thanks!

Hi, I have just the same question here.


Fix nullable partition values

If checkpoint contains the partition with null value, the populate_hashmap_with_option_from_parquet_map will fail to read it because it expects the value to be present, despite the map being HashMap<String, Option<String>>

k, working on it

@houqp tests hash blocked by https://github.com/delta-io/delta-rs/pull/503 by now ready


[HANDS-ON BUG]  unit 3: cannot find dqn.yml file

Describe the bug
I am doing the hands-on of Unit 3. In section "Train our deep Q-learning agent to ply space Invaders", an error appeared  when I am running "!python -m rl_zoo3.train --algo dqn  --env SpaceInvadersNoFrameskip-v4 -f logs/ -c dqn.yml". It states that
"
Loading hyperparameters from: dqn.yml
Traceback (most recent call last):
File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/usr/local/lib/python3.10/dist-packages/rl_zoo3/train.py", line 274, in
train()
File "/usr/local/lib/python3.10/dist-packages/rl_zoo3/train.py", line 256, in train
results = exp_manager.setup_experiment()
File "/usr/local/lib/python3.10/dist-packages/rl_zoo3/exp_manager.py", line 187, in setup_experiment
hyperparams, saved_hyperparams = self.read_hyperparameters()
File "/usr/local/lib/python3.10/dist-packages/rl_zoo3/exp_manager.py", line 291, in read_hyperparameters
with open(self.config) as f:
FileNotFoundError: [Errno 2] No such file or directory: 'dqn.yml'
"
I found this dqn.yml file in the folder hyperparams/ and also I copied it from hyperparams/ to its upper folder rl_zoo3/, the error is still there.
I am wondering if anyone also met such problem? Can anyone help solve it? Thanks a lot.
I am using Google Colab

Hey there 👋 in this tutorial you need to create your own dqn.yml. "1. Create a hyperparameter config file that will contain our training hyperparameters called dqn.yml."

Train our Deep Q-Learning Agent to Play Space Invaders 👾
To train an agent with RL-Baselines3-Zoo, we just need to do two things:

Create a hyperparameter config file that will contain our training hyperparameters called dqn.yml.

This is a template example:
SpaceInvadersNoFrameskip-v4:
  env_wrapper:
    - stable_baselines3.common.atari_wrappers.AtariWrapper
  frame_stack: 4
  policy: 'CnnPolicy'
  n_timesteps: !!float 1e7
  buffer_size: 100000
  learning_rate: !!float 1e-4
  batch_size: 32
  learning_starts: 100000
  target_update_interval: 1000
  train_freq: 4
  gradient_steps: 1
  exploration_fraction: 0.1
  exploration_final_eps: 0.01
  # If True, you need to deactivate handle_timeout_termination
  # in the replay_buffer_kwargs
  optimize_memory_usage: False

Thanks, I followed the tutorial and created one dqn.yml file and also changed the n_timesteps to 1e6 according to the suggestion. The current problem is: I put this "dqn.yml" file to the folder "/usr/local/lib/python3.10/dist-packages/rl_zoo3/", then the error comes and states it cannot find dqn.yml file. Moreover, I read the code of "exp_manager.py" and find another "dqn.yml" in "/usr/local/lib/python3.10/dist-packages/rl_zoo3/hyperparams/". So I am very confusing even there are two "dqn.yml" files, why it still has the error: FileNotFoundError: [Errno 2] No such file or directory: 'dqn.yml'

You don't need to put it in the package. Put it in the root folder directly.

You define the path of your dqn in -c "where the hyperparameter config is after -c."

You don't need to put it in the package. Put it in the root folder directly.

Thank you, it works now.

Thanks again, the problem is solved and I will close this stack


SSL Certifications Verification Error

Hi!
While trying to launch the generated artifact executable on a linux running inside of a corporate network, it will fail due to SSL certificates being verified before accepting handshake:
thread 'main' panicked at 'called Result::unwrap() on an Err value: IoError(Custom { kind: InvalidData, error: InvalidCertificateData("invalid peer certificate: UnknownIssuer") })', src/profiles/mod.rs:117:49
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

Please introduce an option to disable certificates verification.
Regards,
Mariusz.

Hello!
Unfortunately the HTTP library I am using does not support using self-signed SSL certificates. I am in the process of migrating it over to a different HTTP library since it has caused issues with domain fronting. I can also include a build option to disable certificate validation once the migration is finished.


feat: updated to use latest version of private-api-sdjk-java

CC-1495
Short description outlining key changes/additions
JIRA Ticket Number
Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Infrastructure change

Pull request checklist

[ ] I have added unit tests for new code that I have added
[ ] I have added/updated functional tests where appropriate
[ ] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: No security warnings found

CI: No security warnings found


emit pending/failed job metrics even when queue is empty

Historically metrics would only be emitted when there were pending/failed jobs for a given queue in the database at the time the PeriodicUpdater triggered. This means that for quick moving queues, the metric may never get emitted.
For pull-based metrics aggregators like Prometheus this causes even bigger problems. Metrics would appear "stuck" at the value the queue had the last time the PeriodicUpdater ran when the queue was non-empty.
Here's some examples:
Before

Notice how Prometheus was not able to scrape the metric until it was non-zero (2) and that the metric never goes back down.
After

Notice how Prometheus is able to scrape zero-valued metrics and the queue returns back to zero.
Additional Context
Since each cf-api-server is scraped at a different time, I think the best way to actually track this metric is with the max promql function. E.g.
max(cc_job_queue_length_cc_cf_api_server_5b94765866_qql2x)

CAPI Story #174516310

MySQL failure was an unrelated services test flake.


Fix codecov.sh to use variable instead of hard-coded path.

/lgtm
/approve

@smawson: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
d99624c3d23dc342be8023d462e716c32ca8b425
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@smawson: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
d99624c3d23dc342be8023d462e716c32ca8b425
link
/test istio-unit-tests

prow/istio-presubmit.sh
d99624c3d23dc342be8023d462e716c32ca8b425
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests
/test istio-presubmit

@smawson: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
d99624c3d23dc342be8023d462e716c32ca8b425
link
/test istio-presubmit

prow/istio-unit-tests.sh
d99624c3d23dc342be8023d462e716c32ca8b425
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@smawson: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
d99624c3d23dc342be8023d462e716c32ca8b425
link
/test istio-unit-tests

prow/istio-presubmit.sh
d99624c3d23dc342be8023d462e716c32ca8b425
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


libraqm update to 0.9.0

Updates libraqm to v0.9.0, tested on x86-windows-static.
The second commit updates the portfile to use the new vcpkg functions, as explained in the maintainer guide.

Does your PR follow the maintainer guide?

Yes

Thank you!


Add Connectors

otherwise it will show Notice:  "Undefined property: Illuminate\Queue\QueueManager::$connectors" when execute "php artisan"

This is invalid syntax as travis is showing.

There's a + in line 26 that needs to be removed. Probably inserted accidentally.
@jianfengye Can you resubmit without that line?

I have resend a new pull


docs: correct highlighting

ok now it randomly works.

You filed an issue because GitHub doesn’t work? 🤔

yeah github syntax highlighting randomly doesn't work (just found out now. I thought it was just invalid value


fix Recurrence.after passing cache as dtend to rruleset

Thanks @enkore - I'll get the release notes updated and then I think we can cut a new release with this in.


[Powertoys Run] Re-register programs feature

Provide a description of the new feature / enhancement
Is there a way to re-register apps and games so they appear on powertoys run? Any way is welcome.
I have formated my C drive but have all my games (steam/gog etc) on another drive and it doesn't see the games although they run properly through their launchers.

When you re-added your Steam Library, did it (re)create startmenu/desktop shortcuts?

@olonas
It looks like your games are not in one of the default application locations. Where exactly your games are located(path)? If you create a shortcut on the desktop does the game still isn't shown in the PT Run?

sorry i didnt see your replies.
I use my second ssd drive for games. when i reinstalled windows, i ran on each title "validate data" option on steam and shortcut icons reappeared but wont show in pt run. same with gog games.

When you re-added your Steam Library, did it (re)create startmenu/desktop shortcuts?

yes after validating data

When you re-added your Steam Library, did it (re)create startmenu/desktop shortcuts?

yes after validating data

After validating the data, do they appear in PT Run now :) ?

When you re-added your Steam Library, did it (re)create startmenu/desktop shortcuts?

yes after validating data

After validating the data, do they appear in PT Run now :) ?

no :(

@olonas
Right-click on the PowerToys icon in the tray menu and select Report Bug

Drag and drop report into a GitHub comment.

PowerToysReport_2021-06-03-18-25-48.zip

Note: From logs it looks like Program plugin indexed everything successfully. From the other side, shortcuts are present on desktop/start menu so they should be indexed.

@olonas
Can you start PowerToys as not admin?

you want a non-admin bug report? i still doesn't load steam games.

@olonas
Just to clarify, you have steam games shortcuts on the desktop. Is it correct?

Actually I store all my shortcuts in a folder on my second drive so that if (when) I reformat I know what to download again.

@olonas
Can you create a shortcut to one of your games on the desktop and then try to find it in the PT Run?

Thanks! It works that way... can I add other folders in PT Run?

@olonas
It's not possible to add folders for the Program plugin. Programs have to be in default locations.
You can try to add your folder to the Windows Search indexing.

It is tracked by https://github.com/microsoft/PowerToys/issues/9293


llk_3c.h and dataflow_api.h cleanup and refactor

https://docs.google.com/document/d/1em0MZ9SbhpAPYq60DSdPYZnFZM8dkcaAJxGiO5Q_20I/edit?pli=1
A superset of https://github.com/tenstorrent-metal/tt-metal/issues/247

[x] Move compute kernel api to top level include
[ ] Move dataflow kernel api to top level include and split it into user facing and internal
[ ] Introduce compute namespace
[ ] Introduce dataflow and dataflow_internal namespace
[ ] Move various functions in dataflow_internal to dataflow which need to be exposed to user

@mywoodstock we completed some of this planned work.
We can close and open new cases if we plan for similar features?

we decided not to add these namespaces, and keep the dataflow api as it is now. Closing.


Crowdsale.sol contract has a bug in setting rate

The Wei received in transaction to fallback function are multiplied to rate and there is a check for rate in constructor  (see second screenshot) for it to be greater than 0. Currently, if we want to set rate for a single token to 10000 Wei, we would have to pass rate's value in the format of 0.000000000000001 and the transaction will fail and will only pass if we set the rate such that
we can set rate of 2 token for 1 Wei, but not 1 token for 2 Wei.

I made the code work by using division instead of multiply and providing the rate's value in the format of<PHONE_NUMBER>000..

@sheraz104 check the rate comment in Crowdsale.sol, it said How many token units a buyer gets per wei. 1 token for 2 Wei means the price.

@jacktang If token supply is more or equal 1e18 the current implementation works. What if there are only 100 tokens in the contract?

@jacktang exactly. Even when the tokens are 1e18, which is a huge amount, if I'm not wrong the crowdsale can be bought for 1 ether.

@sheraz104 agreed. I've been trying to figure out why it was written this way for the past week. Glad someone else has noticed this.

note that there's a difference between 1 TKN and 1 TKNbit where what people usually call "one token" is actually 1 * 1e18 TKNbits. This is the same relationship that ether and wei have. And what you're always doing calculations in is TKNbits.
So if you want to issue someone "one token for every 2 wei" and your decimals are 18, your rate is actually something like 0.5e17. Then, when I send you 2 wei, you issue me 2 * 0.5e17 TKNbits, which is exactly equal to 1 TKN.
And, to show the answer to the first question, "one token for 10000 Wei", your rate should be 1e14. Then, for each wei that is sent, I'm issued 1e14 TKNbits, which, once You send 10000 wei will be 10000 * 1e14 === 1e18. And then because your decimals are 18, that's displayed as 1 TKN.

Issuing less than 1 TKNbit/wei isn't really useful in practice, but there are a few ways to do it:
You're correct that because of the multiplication, you can't issue less TKNbits than the wei that was provided, but you can issue less TKN, total, by changing your decimals. If you change your decimals to 10, for example, you can now use a rate of 1 to say that "one TKNbit for one wei", meaning that it costs 10 Gwei to own 1 TKN.
If you don't want to change your decimals, you can also just inherit from crowdsale and (i) change the rate to a divisor and then (ii) change _getTokenAmount  to use div and (iii) tell your users to just ignore the rounding errors

Finally found the truth after hours of debug , THX ! 😂
But why not write things so important in document ?!?!

@linkdesu You should always, always operate in the smallest number of a currency when doing automatic operations on it. This is generally-well-known in programming and mentioned a variety of times in the ethereum docs.
We will add a specific note to this contract, thank you for your report


ARROW-8555: [FlightRPC][Java] implement DoExchange

This is a complete implementation of DoExchange for Java. It is not tested against the C++ implementation yet, however, it still passes integration tests, so the internal refactoring should not have broken compatibility with existing clients/servers.
In this PR, I've refactored DoGet/DoPut/DoExchange on the client and server to share their implementation as much as possible. DoGet/DoPut retain their behavior of "eagerly" reading/writing schemas, but DoExchange allows the client/server to delay writing the schema until ready. This is checked in the unit tests.
I also ran into some test flakes and tried to address them, by making sure we clean up things in the right order, and adding missing close() calls in some existing tests.

Thanks for the review! Replies inline above.

Thanks, I think the tests are clearer now, left a few more comments.  @jacques-n do you want to review?  Otherwise, I'll take one more review pass later in the week (but I expect this is mostly mergeable).

Thanks all for the reviews. I've rebased this again.

+1 thanks.


Infinispan as embedding storage support

Infinispan is implementing the Vector Search capabilities.
Tagging the issue here

@karesti I am sure what your timeline for this is, but I just wanted to let you know that this extension releases much quicker than the uptream langchain4j project.

@geoand we need 15.0.0.Final version in the infinispan-client upstream quarkus as well

@geoand somewhere in january will be releasing 15.0.0.Final, then with the latest quarkus release containing the version, we can integrate all here. for now i'm working locally with snapshot and dev versions to fill all the gaps for the integration in langchain upstream and here

Sounds good


MGCNet on Colab

I am currently trying to set up MGCNet on Colab, and when I am installing bazel, an error message pop up:
ERROR: bazel does not currently work properly from paths containing spaces (com.google.devtools.build.lib.runtime.BlazeWorkspace@38f2b815).
I do not think my path contains any spaces...
Did anyone encounter the same problem. If so, how did you solve it?
And did anyone successfully set up MGCNet on Colab?

I wrote a  colab demo.@lkh9908
https://colab.research.google.com/drive/1G8aQFFXp5a3-9x4RbraOuEo7TaA6WPRh?usp=sharing

@cuizhanyuan Hello, I hope this message gets to you well.
I tried to run your collab and as well follow the installation prerequisites on my local machine but it seems that there will always be version conflicts between the python version used, tensorflow-gpu, and opencv-python. Are there any updates to the collab?

I'm very sorry. Now my job is not in that direction, I haven't run code in a long time. It should be a conflict caused by Colab updating the python version. When I configured it successfully, its python version was 3.7.The environment configuration of this project is quite complex and requires a little more patience. It took me several days to successfully configure it.@BanjiBear

The current python version of colab is Python 3.10.12.

@cuizhanyuan Thank you so much for your response and please do not be sorry. Thank you for your time.
I tried to install all dependencies with newer versions and monitor their compatibility. Yet, there are some issues still with bazel and g++ in the collab. I have found other work regarding facial geometry modeling. Please don't worry.
Thanks again


allow tuples and other iterables instead of just lists

>>> t = trict.Trict({})
>>> t[('a', 'b', 'c')] = 'd'
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/tmp/trict/trict/trict.py", line 55, in __setitem__
    key = self.key_to_list(key)
  File "/tmp/trict/trict/trict.py", line 76, in key_to_list
    raise TypeError('Key is not list or str')
TypeError: Key is not list or str
>>>

I think Trict should check whether the key is a string (usually  isinstance is preferred over type), and if it's not a string, then assume that it behaves like a list. Most Python code is written that way, and works well with different kinds of iterables, not only lists.

Great point, agree completely. Will implement

d298e925b957a7c03287ed77a43f97afe3cee494


FailOnCVSS option does not respect suppression file

Describe the bug
Using the --failOnCVSS 9 flag and passing in suppression file with all the critical vulnerabilities suppressed still fails the build.
Version of dependency-check used
The problem occurs using version 7.1.0 of the cli (automatically downloaded by Jenkins).
Log file
Cannot provide since it is closed-source project.
To Reproduce
Steps to reproduce the behavior:

Have a project with critical vulnerabilities. Create a suppression file for all the critical vulnerabilities.
Run the CLI with --failOnCVSS 9 option
Jenkins build still fails

Expected behavior
Build doesn't fail (since all the critical vulnerabilities are suppressed).
Additional context
Critical vulnerabilities were suppressed successfully.

I am using Jenkins pipelines and passing in the CLI option like this:
dependencyCheck (
    additionalArguments: "--failOnCVSS 9",
    odcInstallation: 'latest' // 7.1.0
)

Workaround:
Managed to work around this by using the following code in Jenkins after the dependencyCheck. (Inspired by https://stackoverflow.com/a/66013797/6113915)
dependencyCheckPublisher (
    pattern: '',
    failedTotalCritical: 1
)

I tried with the exact same suppression file and the build now passes. When I remove one suppression it fails.

If you did the suppressions correctly the dependencyCheck CLI check should pass with all your criticals in the suppressed section of the XML... whether the Jenkins plugin reads the report wrongly and therefor still fails the build,  your suppression is not effective or there is some other error happening in the analysis that makes your build fail (non-zero return code due to 'errors occurred' rather than 'too high severity) is hard to tell as you can not provide a report.

Note that the Jenkins plugin for ODC is a project separate from this repository. This repository is only about the ODC tool, not the integration in Jenkins.

You are right in that it could be Jenkins failing the build and the CLI working perfectly. I should probably have tested the CLI directly. I have already spent a good amount of time and I am happy with the workaround so the issue can be closed. ✌
Thank you for looking at the issue and replying.


Double @ logos around atsign on PyPI

Describe the bug
Since switching to light/dark logos on README.md this results in PyPI showing both.
Steps to reproduce

First I browse to https://pypi.org/project/atsdk/
See double logo under Project description

Expected behavior
PyPI only shows a single logo
Screenshots

sln: https://stackoverflow.com/questions/75552648/github-darkmode-vs-lightmode-images-and-pypi


Reorganization of our samples.

Done. PTAL.

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


Wrong stat path in Dynamic includes (chapter 6)

# Include extra tasks file, only if it's present at runtime.
- name: Check if extra_tasks.yml is present.
  stat: path=extras/extra-tasks.yml
  register: extra_tasks_file
  connection: local

- include_tasks: tasks/extra-tasks.yml
  when: extra_tasks_file.stat.exists

The path parameter to stat should be tasks/extra-tasks.yml and not extras/extra-tasks.yml

Right you are! Thanks for posting this fix, I am incorporating it into the next minor book text update which I hope to publish in a few weeks.


Series/2.x docs and build prep for default

superseded by PR https://github.com/zio/zio-dynamodb/pull/142


Documentation updates

@dufoli Please do a quick review in case you have any feedback.  Otherwise once merged, we can submit a PR to https://github.com/quarkiverse/quarkiverse-docs/blob/master/antora-playbook.yml

Sounds good to me. We have a lot to add later
Code generation with maven plugin
Main method instead of container managed
A sentence about corba
Cient injection
Cxf Async
Wsdl
Literal, document, bare, raw mode
ServiceName and port name
WebServiceClient annotation
Link to cxf doc
Fastxml
Performance
...


Alleviate accepted media-types

Currently, the Accept header automatically strictly requires application/json responses from the server.
A conformant server should respond with 406 if it cannot deliver the resource using the requested content type.
Setting the Accept header to application/json is a too restrictive default, I hereby propose to change it to application/json, text/json, */*.

Thank you! 😀

Despite the examples, release notes and everything else 0.9.4 still sends application/json only for me:
POST /whatever/accepts/json/but/returns/text HTTP/1.1
Accept: application/json
Accept-Encoding: gzip, deflate
Connection: keep-alive
Content-Length: 109
Content-Type: application/json
Host: somehost
User-Agent: HTTPie/0.9.4

{
    "someJson": "here"
}

It's probably me, but please advice.

@jbaruch it hasn't been released yet


devSourcemap: true not generating sourcemaps for preview mode

Describe the bug
When running the command vite preview it should be treated as "dev" and hence should produce sourcemaps for easy debugging. even when NODE_ENV and mode is set to "development".
Reproduction
https://stackblitz.com/edit/vitejs-vite-2h4hxn?file=vite.config.ts&terminal=preview
Steps to reproduce
npm run preview

System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    vite: ^4.0.0 => 4.0.3

Used Package Manager
npm
Logs
vite v4.0.3 building for development...
✓ 6 modules transformed.
dist/index.html                      0.45 kB
dist/assets/javascript-8dac5379.svg  1.00 kB
dist/assets/index-1bb5262e.css       1.70 kB
dist/assets/index-de04364a.js        2.27 kB │ map: 1.61 kB
  ➜  Local:   http://localhost:4173/
  ➜  Network: use --host to expose

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

css sourcemap for build is not supported yet. Closing as a duplicate of #2830

@sapphi-red Okay that's cool, but not sure how can a basic and industry standard feature not be supported? There must be some technical limitations? I can try implement it.. is a PR welcomed?

PR is welcome! This is the related part.
https://github.com/vitejs/vite/blob/13ac37df8e135fe6b4990fac6f149a36a19e5409/packages/vite/src/node/plugins/css.ts#L354-L714


Update CLI/config for new API

We need to add two main features:

enable/disable individual services (NodeService, SmesherService, etc.)
pick a version of the API (see https://github.com/spacemeshos/api/issues/55#issuecomment-644946443)

For reference:

geth: https://github.com/ethereum/go-ethereum/wiki/Management-APIs, https://geth.ethereum.org/docs/rpc/server
openethereum: https://wiki.parity.io/JSONRPC

This was done in #2056


Hide top navigation bar in iframes

For now when embedding Hylo in an iFrame we hide the top navigation so people cant navigate to a different context. This is for Coffee Shop but may make sense for everyone

this is not needed right now because we can do layoutFlags=mobileSettings and it hides the nav!


[MB-15162] Fix various pre-commit issues

Jira ticket for this change
Summary
I noticed at some point that pre-commit started warning me about some fields being regex vs. glob:
[WARNING] The 'files' field in hook 'swagger' is a regex, not a glob -- matching '/*' probably isn't what you want here
[WARNING] The 'exclude' field in hook 'swagger' is a regex, not a glob -- matching '/*' probably isn't what you want here

This may have started when I picked up pre-commit 3.x locally when updating my brew dependencies.  This PR fixes that issue.  Note: I've rolled back to pre-commit 2.x locally for now because I was having runaway golangci-lint memory usage with pre-commit 3.x.
I also updated the markdownlint-cli hook to the latest version.  Note that I'm holding off on updating golangci-lint to a later version because there is an issue with the gci linter that is blocking us (it moves and then deletes a linter suppression comment which doesn't appear easy to work around due to other expectations we have on placement of these comments).
When I was playing with the newer golangci-lint, it did point out an issue with one set of tests where we were ignoring a return value from an Equal call as opposed to doing a suite.Equals.  I went ahead and fixed those so we don't have to deal with them later.
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps
Just make sure all tests pass and pre-commit still works locally as expected.
Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

Screenshots

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 0aaf5b5ef394fcd9aeaa5786ea84bb622bf2b978

Messages

:book:
:link: MB-15162

  Generated by :no_entry_sign: dangerJS against cf32c342f73f128435c91d22d65c29ac8c3463db


Easier upload to multiple hosts/destinations

Is this a similar or duplicate feature request?

[ ] I don't know. I will go check it.
[x] No.

Is your feature request related to a problem? Please describe.
The SFTP upload feature works quite well. However, the problem is that one needs to set the destination first ('sftp: set profile' and then choose the destination ) and then do an upload (sftp: SYNC local -> remote). When I need to upload to another Server, I need to change the profile first and then do the upload again. It quickly gets tiring during development when you want to upload the code to multiple Test servers.
Describe the solution you'd like
If the menu operation can be changed a little - STEP 1 - "sftp: SYNC local -> remote" and it provides a list of all destination hosts where the SYNC can be done. And maybe a last automatic option of "All hosts".
Describe alternatives you've considered
I've looked for alternatives but could not find any that worked. I've considered DEPLOY, but that didn't work either.
Does this project help you?

[x] Yes. SFTP IS AWESOME!
[ ] Yes. I want to support you.

I would add that it would be nice to be able to deploy simultaneously to multiple hosts.


Fix inconsistent locking behaviour in JoinFutures

JoinFutures could potentially deadlock, at least according to abseil's
deadlock detection.
This commit changes the locking behaviour to smaller locking scopes that
can't deadlock anymore.

fell free to merge is this cannot be unit-tested. which I suspect is the case.

It can. Test was added.


Webmail login display problem on smartphones

Hi,
The login page is really not well displayed - it seems that there is a scaling problem.
I'm not an html pro but it seems this line is missing in the code :
<meta name="viewport" content="width=device-width" />

You're right, the viewport is not properly configured.


test: [M3-7768] - Add VM Placement Group landing page empty state test

Description 📝
Adds a quick integration test to confirm the VM Placement Groups landing page empty state. The real purpose of this PR is to provide a bare minimum example showing how to mock one of our new object-based feature flags, but it's not much different from mocking our boolean flags.
Changes  🔄

Add integration test to confirm VMPG landing page empty state
Adds a mock util for Placement Group fetching

How to test 🧪
yarn cy:run -s "cypress/e2e/core/vmPlacement/vm-placement-landing-page.spec.ts"

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[x] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

Looks good and thanks for getting this started. My only reservation is whether the folder and spect should be renamed to placementGroups/ and placement-group-landing-page.spec.ts. This change would reflect the feature and component names in the rest of the codebase.

My only reservation is whether the folder and spect should be renamed to placementGroups/ and placement-group-landing-page.spec.ts. This change would reflect the feature and component names in the rest of the codebase.

Great callout, thanks @carrillo-erik! If that's more consistent and makes more sense then I absolutely agree. I'll try to take care of that before I sign out tonight!


Broken example introduced

Dear Airflow Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Version of Airflow (e.g. a release version, running your own fork, running off master -- provide a git log snippet): master branch, commit 4865ee66ba6d53e4a0b809ccbb60a65b5ede1915
Screen shots of your DAG's graph and tree views:
Stack trace if applicable:

[2016-04-06 19:19:09,361] {models.py:233} ERROR - Failed to import: /usr/local/lib/python2.7/dist-packages/airflow-1.7.0-py2.7.egg/airflow/example_dags/example_skip_dag.py
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/airflow-1.7.0-py2.7.egg/airflow/models.py", line 231, in process_file
    m = imp.load_source(mod_name, filepath)
  File "/usr/local/lib/python2.7/dist-packages/airflow-1.7.0-py2.7.egg/airflow/example_dags/example_skip_dag.py", line 60, in <module>
    create_test_pipeline('1', 'all_success', dag)
  File "/usr/local/lib/python2.7/dist-packages/airflow-1.7.0-py2.7.egg/airflow/example_dags/example_skip_dag.py", line 48, in create_test_pipeline
    op = MyEmrOperator(task_id='my_task_id', dag=dag,
NameError: global name 'MyEmrOperator' is not defined

Operating System: (Windows Version or $ uname -a) Ubuntu 14.04
Python Version: $ python --version 2.7.6

Description of Issue

What did you expect to happen? All examples are loaded properly
What happened instead? One example can't be loaded

Reproduction Steps

airflow webserver

@withnale, you may know what's going on?

Thx for reporting. @jlowin is working on a PR to address this.

Sorry about that @ty707 -- an extra operator related to a discussion we were having offline snuck into the PR -- @r39132 fixed it in #1316 and I discovered that not all example dags are being unit tested, #1317.

No sweat, glad some unit tests came out of it 😁

My bad. Sorry this got through. I fixed it an hour later in the PR branch when I noticed it, but it had been merged by them.


Separate versioning and maintenance of the vocabularies

The issue at hand is that all vocabularies are currently housed in a single repository. This arrangement reduces transparency, particularly when one vocabulary is updated without corresponding updates to the others. The use of GitHub version tags and release tags is also problematic, as they refer to the entire repository rather than individual vocabularies.
Could you please clarify the rationale behind consolidating all vocabularies into a single repository? Additionally, could you weigh in on whether, considering the balance between maintenance and clarity, it would be beneficial to separate individual vocabularies into their own repositories?

@burespe1 thank you for your comment! Each vocabulary is assigned its own URI using the w3id redirection mechanism for publication. This guarantees the unique identification of each vocabulary, the correct management of versioning and the decoupling of the URI from the hosting solution chosen. For this reason, we can at any time decide to split the content of this repository into different ones (e.g., one for each vocabulary).
We decided to use a single repository for the hosting to facilitate the "findability" of the repository from a maintenance perspective. Indeed, for an end user, the vocabularies are published as separate entities (each one with its URI) and this aspect affects only issue management.
However, considering your comment, I will add two improvements:

a table in the README to keep track of the current version of each vocabulary
a template for issues to force the user to declare the target controlled vocabulary


Undefined function or variable

Hello， I want to use your code to process the satellite imagery of the sentinel radar, but in the process of running this program , the following error occurred.
**Undefined function or variable'SPEED_OF_LIGHT'.
error： meta2sicd_s1product (line 120)
common_meta.Grid.Row.SS = (SPEED_OF_LIGHT/2) * delta_tau_s;**
I can't solve this problem through debugging. I hope to hear from you.
Thank you very much.

Should be in MATLAB_SAR/Utilities/constants/
I would advise just adding the entire toolbox to your MATLAB path, and everything should work.
Let us know if you have any other issues.


HSEARCH-3400 Use org.hibernate.search.jpa.Search in the documentation when we call getFullTextEntityManager

https://hibernate.atlassian.net/browse/HSEARCH-3400

Pull Request Test Coverage Report for Build 1

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.895%

Totals

Change from base Build 7:
0.0%

Covered Lines:
23743

Relevant Lines:
28301

💛  - Coveralls


Untitled

With the latest version of the driver (1.1.4) the persistent connection doesn't work every time you use the "connect" method.
The method create a brand new connection which lead to a problem under heavy traffic.
Now the connect method is only issues if the connection is not already connected.

stunti I have pulled your changes from be66896 into my branch at https://github.com/gwagner/Shanty-Mongo
These changes are too good to go un-merged so i am going to start managing pulls from my form and we can get the pushed upstream later once coen-hyde has time to get back to them.

@stunti, @gwagner Sorry I missed your commit, I didn't know it existed. It should be merged still I can't merge it until it has tests.


[Dropdown] Dropdown should not open when swiping to scroll on mobile devices.

The Semantic-UI dropdown opens when you start a swiping action over the dropdown control when trying to scroll the page on a mobile device. This is undesirable as pages with several dropdowns will make for a bad UX when trying to scroll the page.
The generic select control handles this properly; it distinguishes between swipes and taps.

Hi @merravid, thanks again, we’re tracking it here: #5180.


Allow higher than 500?

The default on most Linux and OS X installs is between 1000 and 1500, can we please have a higher upper limit on the cursor blink period?

This package doesn't control the blinking period. I don't know if you can change that. Atom sets the cursor blinking period at 500ms by default. In default Atom, the blinking period is the time between the cursor becoming visible and then invisible again. With this package it is the time between primaryColor and secondaryColor.
The setting you're talking about is the pulseDuration of the cursor, which isn't the same as the blinking period. The pulseDuration is the duration of the cursor animating from primaryColor to secondaryColor. If I would allow you to set this higher than 500, the cursor would never fully transition to the secondaryColor, so then setting the secondaryColor would be misleading.

Fair enough, I'll fork and do it myself since I know you can set it higher than 500ms as I did it already.

I thought about it, and I could add an option to also control the blinking period. If I add that, I could also lift the limit off the pulse duration. How do I change it?

I created a new issue for this: #4

I implemented customizing the blinking interval in version 0.9.0. Along with it I lifted the limit you're talking about. Please let me know if it works :)

:+1: on the feature but I can't see the option.

Hm... That's weird. There should be an option 'Blink Interval'. Does it work if you add this to your config.cson?
  'block-cursor':
    blinkInterval: 2000

It worked after your last update, I could see the settings.  That is legit awesome, thank you so much!

You're welcome. Happy to help :)


Library not available in Arduino Library Manager

Hello,
Nice library. Do you have any plans to release it in the Arduino Library Manager?
Installing in Arduino IDE as ZIP library fails unfortunately, I had to manually copy the folder and then move the opt3001.h from include folder into src folder to get it to work.

Hi Bernd, thank you for the feedback. I will publish an update in the next
couple days.

Message ID: @.***
.com>

Should work as expected since v0.1.1 👍.


Remove reviewpad action

Changes proposed
The reviewpad action no longer exists.
This PR removes the reviewpad action
Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

We can also delete the config file for Reviewpad, as it is used by the Action, and it will get removed with this PR. https://github.com/EddieHubCommunity/LinkFree/blob/main/.github/config/reviewpad.yml

Do we know if the reviewpad app uses config files? I was leaving it here in case we started using the app.

Thank you 👍


Indent inside strings after line-ending backslash

Fix #9.
Note there are a few different ways it tries to determine how to indent a line within a string.  Here are some examples from the tests:
pub fn aligned_with_str_begin() {
    format!("abc \
             def");
}

pub fn indented_to_next_level() {
    format!("\
        abc \
        def");
}

pub fn arbitrary_indents_with_and_without_backslashes() {

    println!("
Here is the beginning of the string
            and here is a line that is arbitrarily indented \
            *and a continuation of that indented line
     and another arbitrary indentation
  still another
        yet another \
        *with a line continuing it
And another line not indented
");

}

In the first function, the continuation aligns with the start of the string.  In the second, it pushes out to the next indentation level, since there is nothing in the string on the previous line.  (This is intended to be similar to how it indents lists inside parentheses--aligns to the open paren if there is anything following it on the same line, indent to next level otherwise.)
If the line ending with a backslash is not the first line in the string, the indent level of that line is used to continue it.  In the third function, the indentation will change the whitespace before the lines beginning in * in order to align them with their previous lines.  It will leave all the other lines of that string entirely untouched.

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)


Improve Travis speed: switch back to container

If we switch back to Travis's container infrastructure, our CI will start faster and possibly have more CPU resources.
The downside is that we lose sudo ability.
From what I can think of, the biggest holdback would be apache, since it's a system-installed application and we need to mess with its config file.
Note that we can install apache in the container using the apt stanza in .travis.yml, as apache is in their package whitelist.
This could be more of a "nice to have", as the effort to do this might not be worth the time.

We'd still need to configure apache, which may be harder to do without root.
Can we maybe split it off into it's own section of the build matrix, and use containers for most stuff, only having the stuff that needs root in the vm builds? Or does travis not let you mix & match?

From what I can tell, one only gets a single .travis.yml, which has a single global "sudo" statement.
Another option would be to add another CI service like CircleCI, though this would increase the points of failure.

Quoting Jason Hennessey (2017-07-28 18:07:16)

From [1]what I can tell, one only gets a single .travis.yml, which has
a single global "sudo" statement.

Darn.

Another option would be to add another CI service like CircleCI, though
this would increase the points of failure.

Not worth the trouble, I think.

It turns out I was wrong -- we can mix sudo and non-sudo travis jobs!
Here was the amazing response Travis support sent back to me:

It is possible to configure different jobs mixing
sudo: required and false in a single project! To achieve this, you can
use the Build Matrix feature and define different jobs for sqlite,
postgres and a third one in which sudo: required is explicitly
configured.You can do so by adding a configuration similar to the
following example in your .travis.yml file:language: python

matrix:
  include:
  - sudo: required
    dist: trusty
    python: "2.7"
    env: SUDO_ENABLED=yes DB=other
  - sudo: false
    dist: trusty
    python: "2.7"
    env: DB=sqlite
  - sudo: false
    dist: trusty
    python: "2.7"
    env: DB=postgres

install:
  - if [[ "$SUDO_ENABLED" == "yes" ]]; then echo "your special Apache config commands here!"; fi

Then, as shown in the install phase above, you can use conditional
statements to run any specific commands for the Apache configuration
only in the sudo-enabled job, and avoid running them through the other
jobs if they're not needed. Do you think that a setup similar to this
would be an option for you and your team?As a bit more information, in
this case, as the sudo: required job is defined as one more job in the
matrix, these three jobs will run in parallel. If you prefer to run
these jobs sequentially I'd recommend taking a look at our Build
Stages beta feature, that will allow you to define different stages in
your build that will run only if the previous stages were successful.
For example, you can do this to only run a stage (job) for Apache if
the previous jobs succeeded.

BTW - I still think we'll get more performance bang for our buck by doing #833 first.

@zenhack Do you think it's worth switching to a container in the future? If not, we can close this.

This might be worth it at some point, but it strikes me as relatively low priority. I'd say keep it open though.
The slowest run we have right now though is the apache run, which we can't do in a container, so I dunno.


alexa on raspberry pi does not play on bluetooth speaker

Hi guys, I've bought a new bluetooth speaker and paired it with my RPi. It works fine playing sounds from youtube and VLC, but when I start the Alexa script, it doesn't play on the bluetooth speaker. Does someone know something about this? I can share any info you guys need, just tell me about.

Hi @kiwisaurio,
While the SDK supports sending audio to a connected Bluetooth speaker, customers must utilize another component to perform the actual audio routing. The SDK will send the audio data to whatever sink you are using in the gstreamer pipeline, but the audio data needs to be routed to the Bluetooth stack. One straight forward option is to use PulseAudio. Another option if you are using ALSA directly is to explore BlueALSA. However there will still be some additional work required if you go this route.


Pms system activate Step 1

Hi,
For now, you can Send Messages to other People, but there are some things we need todo:

If i send a message, the message appears in my Inbox too
I cant read the Messages in my Outbox
I can only read the messages in my Inbox, i schould not recieve this messages
MD Support for Messages
Create Subject for Messages
Only Show Subjects in In-/Outbox
And many other things

My nowledge of OCM is very low, i need to read some stuff about this. This is why i dont made more =(
Maybe it is a good idea to use this:
https://github.com/cmgmyr/laravel-messenger
maybe i can try to implement this if you want.

Okay, i'm nearly finished with implementing the Laravel Massenger system the Nice Way. (Means: I'm integrate this with Thread-View-Style. Wait, i think i can push this today =))


Links

If a long note has a link, say https://example.com it will render plain, without linking it. Can it be made, so links are actually linked?

I noticed that --- aren't being rendered as "—" either (and I sure -- is in the same situation)[1]. It would be nice to have curly quotes too (which njump.me renders nicely).
[1] See:
https://ferengi.one/i/#3ac8b3dbbebc5203c696f1852eb2e64de21b8c87d6ce5fb95157e602682a8ec6
https://njump.me/nevent1qqsr4j9nmwltc5src6t0rpfwktnymcsm3jradnjlh9g40eszdq4ga3szypy2z0hr26xxnne0wtvwlhuss8clrqz9w4ssmmdgtzl485et8yr5cx0squ8

Fixed.

I noticed that --- aren't being rendered as "—" ...

Please open a different issue.

Were the links addressed on this issue? I just pulled and I still see https://... unlinked.

Can you share a nevent?

Sure, this one: https://njump.me/nevent1qqsvh4mztkxus2pnnfhkw07y23yt8rd8mtpcx5u0sl45wxa08xj3r2qzypy2z0hr26xxnne0wtvwlhuss8clrqz9w4ssmmdgtzl485et8yr5clj5c9g
It shows fine on njump.me, but not on Oracolo.

Thanks. The problem is related to the final parenthesis, and the link is broken on Njump, too. I will investigate.

This is still broken. See: https://ferengi.one/i/#cbd7625d8dc828339a6f673fc45448b38da7dac383538f87eb471baf39a511a8


[v1.5.2] Is it possible to use Modal.setAppElement with SSR ?

When i try to use Modal.setAppElement just before declare some ContainerComponent:
ReactModal.setAppElement('#app');
class ContainerComponent extends React.Component {
  render() {
    return (
      <Modal {...} />
    );
  }
}

i have next error:
react-modal/lib/helpers/ariaAppHider.js:5
 var el = document.querySelectorAll(element);
              ^

 ReferenceError: document is not defined

Hi @DimaRGB, have you tried with v1.6.5+? Not sure if we have a fix for this.

Hello @diasbruno, actually i can't, it's very old issue. I think it's more related to version 1.5. So, since released 1.6 and i can't test again, we can close issue, i believe.

Ok, @DimaRGB. Thank you.

For reference, since it was a pain for me, if you are doing SSR, use the following code to prevent errors server-side:
if (typeof(window) !== 'undefined') {
    ReactModal.setAppElement('body')
}

You could put this in componentDidMount() anywhere you use a modal or I put it in a custom modal component so it's nice and DRY.
#580


"Boghești" is listed in the drop-down menu

Description:
"Boghești" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Boghești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Boghești , VRANCEA".


TypeError: null is not an object (evaluating '$state.$current.locals[name]')

We collect client side errors via stacktrace.js and log them to our server. Recently I have seen this error in our logs. I can't reproduce it in Chrome 41, IE 11, or Firefox 35. I'm reporting it here as it seems to be an error internal to ui-router, not my own error. We are using ui-router v0.2.13.
Here is the stack trace in its entirety.
"errorUrl": "https://ourwebsite.com/?org=bluecube",
"errorMessage": "TypeError: null is not an object (evaluating '$state.$current.locals[name]')",
"stackTrace": [
"updateView@https://ourwebsite.com/js/angular-ui/angular-ui-router.js:485:159",
"https://ourwebsite.com/js/angular-ui/angular-ui-router.js:482:584",
"$broadcast@https://ourwebsite.com/js/angular/angular.min.js:114:332",
"load@https://ourwebsite.com/js/angular-ui/angular-ui-router.js:474:49",
"https://ourwebsite.com/js/angular-ui/angular-ui-router.js:469:1324",
"d@https://ourwebsite.com/js/angular/angular.min.js:35:41",
"proceed@https://ourwebsite.com/js/angular-ui/angular-ui-router.js:39:327",
"invoke@https://ourwebsite.com/js/angular-ui/angular-ui-router.js:39:228",
"https://ourwebsite.com/js/angular-ui/angular-ui-router.js:37:91",
"resolve@https://ourwebsite.com/js/angular-ui/angular-ui-router.js:41:101",
"https://ourwebsite.com/js/angular-ui/angular-ui-router.js:469:1436",
"r@https://ourwebsite.com/js/angular/angular.min.js:7:396",
"resolveState@https://ourwebsite.com/js/angular-ui/angular-ui-router.js:469:739",
"transitionTo@https://ourwebsite.com/js/angular-ui/angular-ui-router.js:455:138",
"go@https://ourwebsite.com/js/angular-ui/angular-ui-router.js:450:318",
"https://ourwebsite.com/js/app.js:1:12871",
"d@https://ourwebsite.com/js/angular/angular.min.js:35:41",
"https://ourwebsite.com/js/angular-ui/angular-ui-router.js:458:122",
"F@https://ourwebsite.com/js/angular/angular.min.js:100:187",
"https://ourwebsite.com/js/angular/angular.min.js:101:351",
"$eval@https://ourwebsite.com/js/angular/angular.min.js:112:68",
"$digest@https://ourwebsite.com/js/angular/angular.min.js:109:173",
"$apply@https://ourwebsite.com/js/angular/angular.min.js:112:405",
"h@https://ourwebsite.com/js/angular/angular.min.js:72:460",
"u@https://ourwebsite.com/js/angular/angular.min.js:77:796",
"onreadystatechange@https://ourwebsite.com/js/angular/angular.min.js:79:25",
"nrWrapper@https://ourwebsite.com/?org=bluecube:1:22757"
],
"cause": ""
}

:+1: We are getting the same issue, I also can't repro

https://gist.github.com/eddiemonge/f6a58169c2846731a1eb


AI cards : when user requests over13 or over18 or agerange get all 3 cards

To get the 3 credentials we call the 3 API with same photo and server will answer for each request but check with yoti only once.

Size: M


connect.SmartConnectNoSSL deprecated

I tried to use the module to deploy a vSphere cluster, I had the issue that connect.SmartConnectNoSSL is not present anymore in pyvmomi.
I managed to make it work by chaning this:
si = connect.SmartConnectNoSSL(
    host=esx_host, user=esx_user, pwd=esx_pass, port=443
)

to this:
si = connect.SmartConnect(
    host=esx_host, user=esx_user, pwd=esx_pass, port=443, disableSslCertValidation=True
)

I ran into the same issue and the above solution worked for me. Thanks!

Thanks for your previous interest in this project.
I'm closing all open issues to deprecate and archive this project.
See #65 for more details.


community: fix DirectoryLoader progress bar

Description: currently, the DirectoryLoader progress-bar maximum value is based on an incorrect number of files to process
In langchain_community/document_loaders/directory.py:127:
        paths = p.rglob(self.glob) if self.recursive else p.glob(self.glob)
        items = [
            path
            for path in paths
            if not (self.exclude and any(path.match(glob) for glob in self.exclude))
        ]

paths returns both files and directories. items is later used to determine the maximum value of the progress-bar which gives an incorrect progress indication.
I am not sure whether checking with path.is_file() is the right approach here (e.g. links etc) - if there are any thoughts about it let me know.
Also, I am not sure whether and how to add a test for this - let me know if you think it is necessary and have ideas how to do it (since everything is encapsulated within the function and there's no external interface tot he progress bar.

@efriis - I know this is a small change but can you or someone help with this?

:-(
Maybe someone else can take a look at this?
@baskaryan @eyurtsev, @hwchase17 - sorry if I am spamming, I know this is not a big change but still would be nice to follow this through (and a learning experience contributing to this project for me)


Object is not extensible error - using ngrx store-freeze

Error Message:

This is where the code is breaking:

I am using store-freeze from ngrx, if I remove store-freeze, I do not get this error. I am breaking reference from the object retrieved from the store using cloneDeep from Lodash.
const jsonObj = this.jsonConverter.deserializeObject(jObject, ctor);
So this jObject that is being used here has been obtained using cloneDeep.
I have also checked this object with Object.isFrozen() - which returns false.
This is the Object:
{
"id": "7012cf22-4874-454d-8d9a-12f9e45603dd",
"isDynamicWidget": true,
"pluginID": "0",
"parentID": "",
"pluginKey": "Plugin_Default",
"firstProperty": "this is initialized",
"secondProperty": "0",
"jsonWithType": "json with a type",
"jsonWithoutType": "json without a type",
"jsonWithAnotherName": "json with another name",
"requiredJson": "some required jason",
"styling": {
"spacingTop": 0,
"spacingBottom": 0,
"spacingLeft": 0,
"spacingRight": 0,
"float": 0,
"classes": ""
},
"someObjectItem": {
"propertyOne": "one",
"propertyTwo": 2
},
"objectArrayItems": [
{
"propertyOne": "one",
"propertyTwo": 2
},
{
"propertyOne": "one",
"propertyTwo": 2
}
],
"basicArray": [],
"basicNumberArray": [],
"basicBooleanArray": [],
"basicDateArray": []
}

Code:
@JsonObject('ObjectClass')
export class ObjectClass {
constructor(init?: Partial) {
if (init) {
Object.assign(this, init);
}
}
@TextInput({ label: 'First Property' })
@JsonProperty('propertyOne')
propertyOne = 'one';
@NumberInput({ label: 'Second Property' })
@JsonProperty('propertyTwo')
propertyTwo = 2;
}

Please let me know if you need any additional information. Am I missing something simple regarding this issue I am encountering?

Is this somehow related or could help? https://github.com/brandonroberts/ngrx-store-freeze/issues/39
I haven't worked with ngrx store yet so I am not sure if I can help there.
It looks like once you freeze an object (instance), it may not be manipulated in the way json2typescript does.

@Myster1o619 does this issue still persist in version 1.2.4? I am just asking because we have changed the way that property info is checked and if we are lucky, it fixes your problem.

@andreas-aeschlimann - thanks for getting back to me regarding this.
I will test on version 1.2.4 and let you know.
Thanks again.

@andreas-aeschlimann 1.2.4 seems to have sorted this issue out.
Thanks for all your help. Will close the issue and re-open if necessary.


Mark ios_app_with_watch_companion as flaky

Description
Mark as flaky while I investigate the runtime installed on all the devicelab machines.
Related Issues
https://github.com/flutter/flutter/pull/54887
https://github.com/flutter/flutter/pull/51126

Merging to green the build.

Tracking re-enabling with https://github.com/flutter/flutter/issues/54892.

LGTM. For future reference, I believe the process is usually to add devicelab tests as flaky, and then remove the flaky status when it's proven stable.

About a second after I merged it I remembered that detail.


new "vdiskstorage" type

Abstracts the collection of a primary block cluster, a secondary block cluster and an object cluster.
(name up for discussion)

duplicate of https://github.com/zero-os/0-orchestrator/issues/1055


chore: bump ol-util v10.3.1

:tada: This PR is included in version 6.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Simplify/doc moving of ndwidget json

https://github.com/ioam/nbsite/blob/master/nbsite/ipystartup.py#L7

Default is to copy json to same location as notebook html. Can be overridden (e.g. you want to put all json on a separate server, maybe?) via option, but then you need to copy the json files yourself.


update db model and webapp db connection with charset 'utf8mb4'

Hi Thomas,
This PR contains changes in database model and node server to support 4 byte chars.
I have update the testdb with this changes by using Database > Synchronize Model feature of MySQL Workbench, but it seems that it is not effective properly.
So that I have to run sql queries to update them manually. Please refer the attached sql script set_charset_tables.zip (replace testdb with your schema).
Then  verify the database, tables, columns charset changes with below queries (replace testdb with your schema):
select @@collation_database;
select @@character_set_database;
select table_name, table_collation from information_schema.tables WHERE table_schema='testdb';
select column_name, character_set_name, collation_name from information_schema.columns WHERE table_schema='testdb' and (data_type='varchar' or data_type like '%text');

The charset should be 'utf8mb4' and collation should be 'utf8mb4_general_ci'.
testdb database have been changed properly, you can verify the changes by using the updated *log2db tools in below pull requests:
https://github.com/test-fullautomation/python-pytestlog2db/pull/27
https://github.com/test-fullautomation/robotframework-robotlog2db/pull/42
Besides, apply change in node server in this PR to make the webapp display 4 byte string properly.
Please contact me in case you have any issue when applying changes.
Thank you,
Ngoan

Hi Thomas,
I have updated the DB instance for BIOS_ROBFWAIO_SELFTEST.
But I see that that web server instance is still problem when getting branch info.
Besides, the merge conflict in database model has been resolved.
Thank you,
Ngoan


Teamfight icon on timeline not being shown on safari (and probably other browsers too)

OS: Mac
Screen Resolution: 1080p
Browser/Version: Safari
Link to page with the issue, or screenshot: Every match

I can't reproduce this in safari. Can you link a screenshot and your safari/mac version?
OS: yosemite 10.10.5
Safari: 9.0.1

OSX: 10.11.1
Safari: 9.1.2
And the teamfights page actually kills my browser almost. I suspect the "Measure"

I did notice that performance is very terrible in safari. Not sure what's going on there, but it's an order of magnitude worse than chrome. I don't mind someone looking into it but I don't really think safari is a massive priority.

Also can you include a screenshot?

maybe we can do a quick test with disabled transitions and see if it improves safari perf?

I can test it out if you have a branch for me.

Here's the screenshot.

Cool, no console errors, right? And does this exist on the team fights page as well?

Yup, no errors. And yup. Same on teamfights page. The teamfight icons on map don't show as well.
And I have to close the tab immediately otherwise, my browser becomes unresponsive


#130341269 Convert project from grunt to gulp

What does this PR do?
Sets up gulp equivalents for the the existing grunt tasks
Description of Task to be completed?

Install gulp and required gulp modules
Create gulp tasks to replace the existing grunt tasks

How should this be manually tested?
same as before. run:
    npm test

What are the relevant pivotal tracker stories?
Find the relevant pivotal tracker story here

Please note that the build is supposed to fail at this point. Fixing the tests are not in the scope of this story.


Execute blocks to current

We had this separation into deploys and transfers at the beginning, months ago but we purposefully started handling them both together b/c that separation lead to bugs where on every call-site developers had to remember that there are two items they need to handle and that was easily missed. Why are we separating them again? I cannot see anything in the code here that would require this?


Update approleassignment list to v1.0 API. Closes #1604

Update approleassignment list to v1.0 API. Closes #1604

Awesome! Thank you for such a quick turnaround! 👏

Hi @aakashbhardwaj619 - thanks for your work on this.  Can you provide a brief description how to test the 'approleassignment list' command?

Hi @mpowney, you can assign some application permission roles to an AAD app registration in your tenant and then check that the command lists those app roles for that specific app.
For example, I have provided CallRecords.Read.All and Reports.Read.All application permission roles for Microsoft Graph to a custom AAD app.

When I execute the m365 aad approleassignment list --displayName "Custom PnP CLI for Microsoft 365" command, I get the below results with resourceDisplayName and roleName for the app roles.

I hope this helps.

Thanks @aakashbhardwaj619! Merged manually. 👏


Add inverse_transform in TimeSeriesImputerTransform

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Type of Change

[ ] Examples / docs / tutorials / contributors update
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Add inverse_transform in TimeSeriesImputerTransform along with tests
Little refactor of test_impute_transform.py
Fix bug with inplace transform with option running_mean

Related Issue
#132.
Closing issues
Closes #132.

Codecov Report

Merging #135 (b5fbc2f) into master (d11ab37) will decrease coverage by 1.07%.
The diff coverage is 100.00%.

:exclamation: Current head b5fbc2f differs from pull request most recent head 4a2802f. Consider uploading reports for the commit 4a2802f to get more accurate results

@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
- Coverage   88.06%   86.98%   -1.08%
==========================================
  Files          66       66
  Lines        2865     2867       +2
==========================================
- Hits         2523     2494      -29
- Misses        342      373      +31

Impacted Files
Coverage Δ

etna/transforms/imputation.py
100.00% <100.00%> (ø)

etna/models/nn/deepar.py
73.91% <0.00%> (-26.09%)
:arrow_down:

etna/models/nn/tft.py
75.00% <0.00%> (-25.00%)
:arrow_down:

etna/transforms/pytorch_forecasting.py
92.20% <0.00%> (-7.80%)
:arrow_down:

etna/loggers/base.py
92.15% <0.00%> (-2.08%)
:arrow_down:

etna/loggers/wandb_logger.py
33.33% <0.00%> (-1.50%)
:arrow_down:

etna/loggers/console_logger.py
92.30% <0.00%> (-0.29%)
:arrow_down:

etna/metrics/base.py
96.42% <0.00%> (-0.07%)
:arrow_down:

etna/models/base.py
97.43% <0.00%> (-0.04%)
:arrow_down:

etna/analysis/plotters.py
13.97% <0.00%> (ø)

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 d11ab37...4a2802f. Read the comment docs.


Creating a database for storing product images into s3 bucket

S3 Bucket has been created.

completion: 0.5hrs


[Feature Request]: Add Animaton to logo

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Sliding Logo Animation:
Functionality: The logo should slide in from the left side of the screen upon page load, capturing the user's attention immediately.
Animation Details:
The animation should start with the logo positioned off-screen to the left and move into view smoothly.
A keyframe animation called logoAnimation will control this movement, ensuring a fluid transition.
Letter Drop Animation for "Ambu":
Functionality: Each letter in the brand name "Ambu" should drop into place sequentially, adding an engaging effect that enhances user interaction.
Animation Details:
Each letter will be wrapped in a  element with a class that allows it to animate individually.
A new keyframe animation called letterDrop will manage the dropping effect, with each letter experiencing a slight delay based on its position in the word.
Use Case
Improved Brand Recognition:
Description: By styling the brand name "Ambu" in a vibrant red color, users will more easily associate the logo with the service provided. This visual differentiation helps in creating a lasting impression.
Impact: Stronger brand recognition can lead to increased trust in the service, encouraging users to rely on the platform in emergency situations.
Increased User Engagement:
Description: The sliding and letter drop animations create a visually dynamic entrance for the logo, making the user experience more engaging right from the start.
Impact: Users are likely to stay longer on the site, as engaging visuals can reduce bounce rates. This is particularly important for emergency services, where user trust and satisfaction are paramount.
Enhanced User Experience:
Description: Smooth animations contribute to an intuitive and modern interface that feels responsive and professional. The sequential letter drop effect adds an element of surprise and delight.
Impact: A pleasant user experience encourages users to explore the site further, increasing the likelihood of them using the features and services offered.
Benefits
Benefits of Enhanced Logo Animation and Styling Feature

Brand Recognition: The new logo animation enhances visibility and memorability, fostering trust in emergency situations.

User Engagement: Dynamic visuals attract users, encouraging exploration of the site and services.

Improved User Experience: Smooth animations make navigation intuitive, enhancing overall satisfaction.

Competitive Edge: The modern design sets the platform apart, signaling reliability and professionalism.

Community Trust: A polished appearance builds confidence among users, increasing the likelihood of return.

In summary, these enhancements strengthen branding and improve service effectiveness for users in need of assistance.
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Assign me for this issue

can you please assign this issue I will make my best


adding humidity sensor

I have an RM4 unit with the temperature and humidity sensor. I had add to the following areas code to implement the humidity sensor.
in def broadlink_rm_temperature_timer section added following lines below mqttc.publish(topic, temperature, qos=qos, retain=retain)
    humidity = str(device.check_humidity())
    topic = mqtt_prefix + "humidity"
    logging.debug("Sending RM humidity " + humidity + " to topic " + topic)
    mqttc.publish(topic, humidity, qos=qos, retain=retain)

and to keep response from indicating an warning added following in def on_message(client, device, msg): after the if command == 'temperature' or \ statement :
command == 'humidity' or \
you might consider adding to your code or a variation if you want to have independent timers for temperature and humidity.
Thanks for the excellent software, it is working great with broadlink and Nymea

Should be done now. Please check

Have checked. works as expected.
PS good catch on checking for the RM4.
Thanks


Object shadows (in ver 100)

Hi, this lib is very easy to use. Thanks a lot!
However, it's not able to recieve shadows in any materials.
Here is the screenshot.(the under layer which casting shadow correctly is MeshLambertMaterial I implemented.)

I implemented like this:

// Generate a terrain
const map_gras = texLoader.load('./img/snow.jpg');
const map_sand = texLoader.load('./img/sand.jpg');
const map_ston = texLoader.load('./img/ston.jpg');
const map_snow = texLoader.load('./img/snow.jpg');
tex_load_manager.onLoad = function () {
	const blend = THREE.Terrain.generateBlendedMaterial([
		{ texture: map_sand },
		{ texture: map_gras, levels: [-80, -35, 20, 50] },
		{ texture: map_ston, levels: [20, 50, 60, 85] },
		{ texture: map_snow, glsl: '1.0 - smoothstep(65.0 + smoothstep(-256.0, 256.0, vPosition.x) * 10.0, 80.0, vPosition.z)' },
		{ texture: map_ston, glsl: 'slope > 0.7853981633974483 ? 0.2 : 1.0 - smoothstep(0.47123889803846897, 0.7853981633974483, slope) + 0.2' }, // between 27 and 45 degrees
	]);

	let xS = 128, yS = 128;
	let terrainScene = THREE.Terrain({
		easing: THREE.Terrain.Linear,
		frequency: 2.5,
		heightmap: THREE.Terrain.DiamondSquare,
		material: blend,
		maxHeight: 100,
		minHeight: -100,
		steps: 1,
		useBufferGeometry: false,
		xSegments: xS,
		xSize: 1024,
		ySegments: yS,
		ySize: 1024,
	});
	terrainScene.receiveShadow = true;
	// Assuming you already have your global scene, add the terrain to it
	scene.add(terrainScene);
}.bind(this);

I'm using threejs version 100, but it seems not working since ver 91.
It seems related to #6 but I'm new at GLSL so it's bit confusing to investigate this.
would you henp me?

Thank you for the response.
I changed lights_pars_maps to  envmap_physical_pars_fragment, but it has no change.
I tested it on ver r91.
Is the shadowmap is enabled default in blended texture shader material? Or is there any options like recieveShadow?
Anyway, I will continue investigations!

Yes, I tested with below version combination.

r91 and lights_pars_maps
r101 and envmap_physical_pars_fragment

I know that demo receives shadows the terrain casts onto itself, but I want to recieve shadow from object.
(For example, in the THREE.Terrain demo, I want to cast the shadow from each trees)

I haven't specifically tried it, but I think this should be fixed in v1.6.1+ thanks to the improved way the textures are generated. Please feel free to reopen if that's not the case.


dbeaver/dbeaver#22305 Added Functions support for Snowflake SQL Editor

Added functions support for Snowflake SQL Editor Issue Link
The functions that are added are all referenced from https://docs.snowflake.com/en/sql-reference/functions-all

There are lot of functions present in the documentation, but I have only added support which where present in dbeaver repository at time of making this PR (Reason - Already present functions were all generic and mostly covers all scenarios for developers)
In future, we can surely add the functions in file, whenever needed.

UI Changes -
Before -
https://github.com/dbeaver/dbeaver/issues/22305#issue-2062742622
After -

Thanks for your contribution @sarveshhiwase
It looks good to me.
Now, we are waiting for a second reviewer and QA Team.

Thanks for your contribution @sarveshhiwase
It looks good to me.
Now, we are waiting for a second reviewer and QA Team.

Thanks @LonwoLonwo for the review 🧪.
Surely will wait ⏳ and will be contributing more 🚀.
Happy Coding👨💻!

Hello again @sarveshhiwase
Could you please change the formatting and set keywords in one column, not in two?
Like here https://github.com/dbeaver/dbeaver/blob/4584687584ef1e8379100afa028fec87852fd415/plugins/org.jkiss.dbeaver.ext.postgresql/src/org/jkiss/dbeaver/ext/postgresql/model/PostgreDialect.java

Hello again @sarveshhiwase
Could you please change the formatting and set keywords in one column, not in two?
Like here https://github.com/dbeaver/dbeaver/blob/4584687584ef1e8379100afa028fec87852fd415/plugins/org.jkiss.dbeaver.ext.postgresql/src/org/jkiss/dbeaver/ext/postgresql/model/PostgreDialect.java

@LonwoLonwo sure i can do that, but don't you think that will make the file like really long?

that will make the file like really long?

It is just our code style. So it's ok, yes.

that will make the file like really long?

It is just our code style. So it's ok, yes.

Sure, understood 📙, will do the changes.

@LonwoLonwo Please check ✅ now, hope you find it good! 🤞
Thanks for your help 💁.

Looks much better now, thanks!
And, I do not see the getdate in the list of functions (https://docs.snowflake.com/en/sql-reference/functions-all). Probably, we can't add it.

It is PRO compatible now, I've checked (no duplicates)

@LonwoLonwo I believe we'd like to remove enumerated in this PR keywords from SnowflakeSQLDialectExt class to prevent keywords definition duplication. It requires PR to the private repository from our side.

Thanks again for your contribution.
This implementation will be available in the next DBeaver version, 23.3.4, on 05-02-2024.

Thank you @LonwoLonwo for accepting the PR, and for all the help 💁🏻♀ you did which helped me understanding the whole workflow.
Would be contributing more.
Happy Coding! 👨🏻💻


New docker transport

This transport uses the docker api to upload files and run commands.
It is handy when you have restrictions to connect via SSH to your
containers.
I'm not pretending to replace the current default transport, but add this as an alternative for some particular use cases.

This needs, at a minimum, tests and docs. It also needs a full compat check with all the various modes that this plugin supports.

(please re-open when you have a more complete implementation)


load  pretrained de-en model raise RuntimeError in save_datastore.py

runtimeError: Error(s) in loading state_dict for TransformerModel:
	Missing key(s) in state_dict: "decoder.distance_func.0.weight", "decoder.distance_func.0.bias", "decoder.distance_fc1.0.weight", "decoder.distance_fc1.0.bias", "decoder.distance_fc1.2.weight", "decoder.distance_fc1.2.bias", "decoder.distance_fc2.0.weight", "decoder.distance_fc2.0.bias", "decoder.distance_fc2.2.weight", "decoder.distance_fc2.2.bias".

must set strict=False can fix it when loading model


Image sequence for texture not working

To play an image sequence for texture I have to input: //cache_ocean\foam_$f4.exr (I tried with F4 but it didn't work, it worked with f4 but only the first frame)
Here is the error it gives me. It renders the first frame no trouble with the foam, second frame appears with no foam and once the render is done I get this error:

Fixed.  Sorry we were converting the F4 in the input file when txmaking, not the output.  Incidently you can use either $f4 or $F4 its just one worked for you because it was cached.

Ok, I'll test that ;)

Seems to be working, I'll let the full 24 frames of animation render and I'll post if there is still issues, but so far, the 3 first frames rendered it ;)

Yup, definitely working ;) Awesome work Brian!

Awesome, worked flawlessly :) Added a video of this working on the forum
https://renderman.pixar.com/forum/showthread.php?s=&postid=182048#post182048


CLI versioning should always follow semantic versioning

As an example, the current release should be 0.5.0.
https://semver.org/

Fixed starting with v0.6.0.