Allow converted outs

Part of #117

Turns out this doesn't work either
let mut pack_type = Default::default();

Since these are outparams we could get away with just mem::uninitialized().

IMHO better don't do it on current stage.
Also currently none of gtk/object_reform branches compiles, so I can't check function correctness.

Sorry about that. I've updated glib/object_reform and gtk/object_reform_gir and they seem to build with no other overrides now.
You need at least a beta compiler, because the glob import bug is only fixed in 1.4.

I've fixed the appveyor issue, now it fails to build for a legitimate reason. We'll definitely have to redo the used_types thing to keep track of versions...

Thanks, then I build current nightly and try build again.

GTK builds on current nightly, thanks.

Minor problem about mem::uninitialized that it need use statement.
We can use as std::mem::uninitialized and use std always,
or add std::mem to used_types in analysis::functions::analyze_function
I plan call function in this way
let mut padding_top = unsafe{ mem::uninitialized() };

Adding to used_types seems fine. Maybe rename them to imports.
Minimizing the unsafe blocks is not a good idea, I'd prefer the assignments and the function call to be in a single unsafe block.

Changed outs initializing.
Only one minor problem, that use looks like this:
use object::*;
use Bin;
use Buildable;
use Container;
use Widget;
use std::mem;

BTW there's another reason to make the unsafe block bigger (possibly just make them contain the whole fn body): from_glib_* functions are unsafe and when we start putting objects in the tuple it will need to be moved inside the unsafe block.
The imports... looks like we might throw all kinds of stuff in there and make them look nicer in a centralized manner.

Make it look like this?
    fn query_child_packing<T: Upcast<Widget>>(&self, child: &T) -> (bool, bool, u32, PackType) {
        unsafe {
            let mut expand = mem::uninitialized();
            let mut fill = mem::uninitialized();
            let mut padding = mem::uninitialized();
            let mut pack_type = mem::uninitialized();
            ffi::gtk_box_query_child_packing(self.upcast().to_glib_none().0, child.upcast().to_glib_none().0, &mut expand, &mut fill, &mut padding, &mut pack_type);
            (from_glib(expand), from_glib(fill), padding, pack_type)
        }
    }

Yes, this will be more reliable.

Changed

Nice, thanks!
@homu r+

:pushpin: Commit 4325539 has been approved by gkoz

:hourglass: Testing commit 4325539 with merge 25280e4...

:sunny: Test successful - status


support BatchEvaluateFeature action #11

Closes #11

Codecov Report

Merging #12 (97be5a9) into main (39439df) will decrease coverage by 18.02%.
The diff coverage is 25.53%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main      #12       +/-   ##
===========================================
- Coverage   92.15%   74.13%   -18.02%
===========================================
  Files           6        6
  Lines         255      348       +93
===========================================
+ Hits          235      258       +23
- Misses         14       83       +69
- Partials        6        7        +1

Impacted Files
Coverage Δ

handler/evaluation.go
46.79% <20.45%> (-34.09%)
:arrow_down:

handler/project.go
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

need to add batchEvaluateFeature test


mermaid diagram syntax doesn't work on vscode

Hey Developers
When I try to use the excalidraw vscode extension and use select the mermaid tool the entire workspace becomes blank and I'll have to close and reopen my whiteboard to get my content.
Help on this regard is greatly appreciated!
Cheers

Bumping this one. Same issue here. Any plans to support or workarounds?

Same for me


Don't call check_all_pending! for older Rails

Closes https://github.com/Shopify/ruby-lsp-rails/issues/512

Hello @vinistock could you publish a new version for VSCode with this fix please 🙏  ?


Create Secondary text button blue

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

Color to be used:- Normal : bg-blue-600 & on hover : bg-blue-700
To view other details, Please check figma design
After building the component please raise a PR with a screenshot of the component and add the component in Button/src/Secondary/text-button.html by replacing the existing code &  rename the file to text-button-blue.html.
So, In short you need to do:-

Make the Component
Rename text-button.html to text-button-blue.html
Replace the code of Button/src/Secondary/text-button-blue.html with your code

If you need to use any icon please use it from Hero Icons
Good luck.

I would like to work on this issue.

yes sure!


[WIP] Fix Android test loop failure on Mac

Fixes #901
JCenter shutdown: https://docs.microsoft.com/en-us/appcenter/build/android/jcentershut caused our build loops to fail.
Changes
Merit of all these changes is to ensure that we no longer attempt to pull anything from jcenter anymore. Thus, resolving the issue that this old artifacts repo was shutdown in May. There is no change to SDK nor NDK versions, only build tools have been upgraded.

switch to mavenCentral binary artifacts repo before jcenter (eventually need to remove jcenter)
update reactivecircus/android-emulator-runner to latest presently available version
use latest gradle tools com.android.tools.build:gradle:4.2.0
use latest gradle-6.7.1
use --refresh-dependencies flag to refetch latest versions, ignoring cache
add equivalent to Unix test script for a Windows runner (just a dev-focused script, not used in CI)
upgrade buildToolsVersion to 30.0.2

UPDATE: the build is getting stuck due to so many things to be downloaded. I'm adding GitHub cache Action here to try speed it up.

@saurabh-sagrawal @larvacea - would you be able to take a look at this issue? I tried fixing the problem by upgrading to more recent set of dependencies. But it appears to be handing while starting the emulator on GitHub actions runner.

@lalitb - I think it'd make sense to at least temporarily switch Android test loop on Mac. That'd unblock all other pending PRs. Since GitHub Action is really letting us down (latest AVD hangs), we could have re-enabled custom hosted runner on our own physical machine, preinstalling the necessary versions and AVD / emulator. I can give you the details of our Mac mini.

@lalitb - I think it'd make sense to at least temporarily switch Android test loop on Mac. That'd unblock all other pending PRs. Since GitHub Action is really letting us down (latest AVD hangs), we could have re-enabled custom hosted runner on our own physical machine, preinstalling the necessary versions and AVD / emulator. I can give you the details of our Mac mini.

@maxgolov  - I have disabled both the failing android builds now  ( Mac and Windows ) temporarily, but looks like it will be effective for only new PRs. I can enable custom-hosted runners on physical mac once you provide more details of it. Thanks.

Your change to disable the loops should be merged in master, then all existing PRs would have to be updated to intake your change, then they'd pass.

Your change to disable the loops should be merged in master, then all existing PRs would have to be updated to intake your change, then they'd pass.

Ok. I actually disabled them through the github admin interface. Will raise a PR instead.

@saurabh-sagrawal @larvacea - would you be able to take a look at this issue? I tried fixing the problem by upgrading to more recent set of dependencies. But it appears to be handing while starting the emulator on GitHub actions runner.

@saurabh-sagrawal @larvacea - Would you be able to help us here. Thanks.

@saurabh-sagrawal @larvacea - would you be able to take a look at this issue? I tried fixing the problem by upgrading to more recent set of dependencies. But it appears to be handing while starting the emulator on GitHub actions runner.

@saurabh-sagrawal @larvacea - Would you be able to help us here. Thanks.

@saurabh-sagrawal @larvacea - would you be able to take a look at this issue? I tried fixing the problem by upgrading to more recent set of dependencies. But it appears to be handing while starting the emulator on GitHub actions runner.

@saurabh-sagrawal @larvacea - Would you be able to help us here. Thanks.

@lalitb sorry I'm not aware of the build process here so I'm not of much help here.
Also sorry for the mis-click from my side which closed the PR by mistake.


hello I cannot download or export PDF my resume[BUG]

Describe the bug

Product Flavor

[ ] Managed (https://rxresu.me)
[ ] Self Hosted

To Reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

Screenshots

Desktop (please complete the following information):

OS:
Browser
Version

Additional context

Hello! Your issue provided no information at all about your problem. Please provide some details and someone might be able to help you.

Hey even I am unable to export the pdf.
Can you please help me out by exporting this ^1 as a pdf? I have to submit my resume very soon and I will be really grateful for that. @tryallthethings @AmruthPillai (Sorry for the tags its urgent...)
Thanking you

I, too, have faced this same issue, here is the error in the console

I, too, have faced this same issue, here is the error in the console

I get the exact error as the comment above. I suspect anybody who would like to have pdf exported and is using the web hosted https://rxresu.me/ can reproduce this issue. Additionally, it looks like there is some sort of a timeout error, it attempts to get the pdf for approximately 37 seconds then would return this error.

The issue should be fixed now, sorry for the delay and inconvenience.


Software Configuration Management

The team member looks sad at the Sprint 2. We have a total of ~13 sprints.
I think Risk Management should consider this.


Customisable slots per epoch

I found manipulating that variable pretty useful for private testnets, where finalisation speed is more important than network security. Is there a reason why you did not make it configurable from the get go?

Which clients have you tested this with? AFAIK not all clients support modifying the preset value of slots per epoch. Our approach for faster testing is to reduce the seconds per slot, achieves the same thing and is supported by atleast 4/5 CLs in our experience.

Yeah our go to is usually testing with 3s slots :) But ofc it could always be faster
There's a concept of a minimal config that is supported by all clients. Perhaps that's a better approach to go down, afaik the minimal config already supports faster forks and is useful for limited testing scenarios.

I'd personally be against including a variable in the config that is not in the spec:
https://github.com/ethereum/consensus-specs/blob/dev/configs/mainnet.yaml

#104 should fix your issue


Core Dump!

Got excited about this project and ordered two.
Tried the first couple of examples but got following error.
any ideas?   both are doing same thing on all two of my devices.
Thanks

E (157) esp_core_dump_flash: Core dump flash config is corrupted! CRC=0x7bd5c66f instead of 0x0
Rebooting...
ets Jul 29 2019 12:21:46
rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0030,len:1344
load:0x40078000,len:13964
load:0x40080400,len:3600
entry 0x400805f0
E (134) spi_flash: Detected size(4096k) smaller than the size in the binary image header(16384k). Probe failed.
assert failed: do_core_init startup.c:328 (flash_ret == ESP_OK)

Changed the Flash mode to DIO from QIO, and now I get different error.
ets Jul 29 2019 12:21:46
rst:0x8 (TG1WDT_SYS_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0030,len:1184
load:0x40078000,len:13260
load:0x40080400,len:3028
entry 0x400805e4
E (8) gpio: gpio_set_level(227): GPIO output gpio_num error

Not sure what is up with this. Have you gone through the Troubleshooting steps?

Thanks for the troubleshooting step.
You might want to add following.
You may need to put the device into download mode by press and hold "BOOT" and click "RESET" to change mode to download and then press and hold when you see "Connecting ........"  after the compile to properly upload the code.
not sure why but above method works, it won't just flash like other esp32 devices.


Hash file for cache filename

Addresses https://github.com/dylanaraps/pywal/issues/307. As mentioned there, there are two open questions here:

Whether md5 is appropriate for this application (over, say, sha256).
Whether this should be an optional feature or the default.

I like this addition a lot, but I'm guessing colors.py is not the right place for the hashf function, consider moving it to the util.py module, maybe that's a more appropiate place for that functionality?
you would then call util.hashf in colors.py

@deviantfero Good call :+1: I didn't pay enough attention to the existing setup while adding this.

In the example I gave in https://github.com/dylanaraps/pywal/issues/307, the filename looked like:
e57c9a6627a5c51244b65e88e50539ab_dark_wal__1.1.0.json
it might be worth hashing all of the info together (so, add in the dark, wal, and version number), rather than have a partially (but not completely) human readable filename.
Thoughts?

I think it's better to keep the other information readable.

Cool, I'm on board with that.
Is there anyone who can merge this aside from @dylanaraps? Looks like he hasn't been active on github for a few months.

I don't think there is, anyway you should try to fix those Travis CI errors

Fix the lint errors and I'll merge this :+1:

:+1: @dylanaraps fixed the ones that I introduced -- there were a couple of other small ones that I left because they were already there and not particularly bad, but I can get rid of those too if you'd prefer.

Cheers :+1:

Reverting this till I can test the performance impact.


Dirty HTML from Word leads to unwanted results

When content from Word is pasted, a lot of extra bonus-HTML is passed as well.
This can lead to words strung together: "like: thequick brownfoxjumps over the lazydog".
I fixed this (for a great deal) in a local project by simply stripping all unwanted things from the HTML.
  function sanitizeHtml(html) {
       // This destroys all (useless Word)  and <o:p> tags Word uses
       //  to set fonts on parts of the text and whatever
       html = html.replace(/]*>|<\/span>/g, '');
       html = html.replace(/<o[^>]*>|<\/o[^>]*>/g, '');

       // This cleans all wanted tags by replacing anything after "<tag" until and excluding ">"
       // By eradicating the second part, all extra settings per HTML tag are removed
       html = html.replace(/(<\w+)([^\>]*)/g, '$1');

       // Clean up bold / italics mess that can lead to converson issues
       // This covers only the basic cases
       html = html.replace(/<\/b>/g, '');
       html = html.replace(/<\/i>/g, '');
       html = html.replace(/\s+<\/b>/g, ' ');
       html = html.replace(/\s+<\/i>/g, ' ');
       html = html.replace(/\s+<\/p>/g, '
');
       return html;
   }

Using it:
pastebin.addEventListener('paste', function() {
        setTimeout(function() {
            // Used here
            var html = sanitizeHtml(pastebin.innerHTML);

            output.value =convert(html);

            output.focus();
            output.select();
        }, 200);
    });

De solution is not tested for all cases.

Feel free to make a pull request for this fix. It looks good to me 👍


Replace weave-local-pods ipset with physdev iptables rule

by-pass ingress network policy chains and accept the packet if
destination is not one of the local pods
return the packets so the egress network policy chains are skipped
in case packet's source is not one of the local pods
Fixes #3344

Addressed all comments. PTAL


Trim parquet_write_test to reduce integration test runtime [databricks]

Relates to #4538.  This significantly reduces the runtime of parquet_write_test.py by reducing the overhead of some types being tested and reducing redundant test configurations.  On the type overhead reduction front, one type being tested was an array-of-array-of-map-of-struct-of-map-of-string-to-string.  Each string was averaging 15 characters, and each map was 20 elements, and each array level was 10 entries, so there were quite a lot of string characters being processed on the CPU for data generation.  This was causing tests involving those types to execute on the order of minutes for that specific type.  Reducing the length of the maps and arrays solved this issue while still providing the same type coverage.
On the redundant test configuration front, there were many tests that were varying the timestamp write configs for all types, even those without timestamps anywhere in the type hierarchy.  This was solved by creating a separate test to test timestamp types and varying the timestamp config for just that test, and removing that dimension on all the other tests.
The tests were also varying the reader configuration and datasource version as dimensions across many tests, but this file is for testing Parquet writes rather than reads, and these settings don't change the behavior of the write code.  Parquet read tests are already covered by parquet_test.py, so these test dimensions were removed.

build

Looks like it timed out connecting to the Databricks cluster.  Kicking again.

build

build

Databricks clusters continue to encounter rsync failures preventing CI from proceeding.


Bluetooth: Mesh: Extended advertising support

Adds support for extended advertiser commands in the mesh. This doubles
throughput for common packet sending, and significantly improves timing
accuracy for the Friend and Low Power features.
The proxy module's advertisement control has been moved inside the adv
module to abstract away the different advertiser modes.
The extended advertiser mode does not need an advertising thread to
operate, and ends up with a net reduction in RAM usage.
Signed-off-by: Trond Einar Snekvik<EMAIL_ADDRESS>

@trond-snekvik  looks like you got several build failures after the latest push

Sorry about that, fixed.


Allow putting strings inside .

Fixes #1953

I moved the undefined check to separated if -statement.

@jsyrjala Looks fine, thanks


Daemon scripts

Related to #7, we should have some scripts that allow for running as a daemon

This is part of the packaging. Closing.


Invalid reference to "DEFAULT_CARDINALITY" for count aggregate

This query should work:
x = scan(public:vulcan:snapshot1818Hash);
a = [from x emit count(*)];
store(a, a);

But it throws this error:
Traceback (most recent call last):
  File "/base/data/home/runtimes/python27/python27_lib/versions/third_party/webapp2-2.5.2/webapp2.py", line 570, in dispatch
    return method(*args, **kwargs)
  File "/base/data/home/apps/s~myria-web/1.388195978131666431/myria_web_main.py", line 493, in post
    push_sql=push_sql))
  File "/base/data/home/apps/s~myria-web/1.388195978131666431/myria_web_main.py", line 119, in get_logical_plan
    return get_plan(query, language, 'logical', connection, push_sql=push_sql)
  File "/base/data/home/apps/s~myria-web/1.388195978131666431/myria_web_main.py", line 105, in get_plan
    processor.evaluate(parsed)
  File "/base/data/home/apps/s~myria-web/1.388195978131666431/raco/myrial/interpreter.py", line 339, in evaluate
    method(*statement[1:])
  File "/base/data/home/apps/s~myria-web/1.388195978131666431/raco/myrial/interpreter.py", line 380, in assign
    self.__do_assignment(_id, expr)
  File "/base/data/home/apps/s~myria-web/1.388195978131666431/raco/myrial/interpreter.py", line 366, in __do_assignment
    child_op = self.ep.evaluate(expr)
  File "/base/data/home/apps/s~myria-web/1.388195978131666431/raco/myrial/interpreter.py", line 77, in evaluate
    return method(*expr[1:])
  File "/base/data/home/apps/s~myria-web/1.388195978131666431/raco/myrial/interpreter.py", line 103, in scan
    self.catalog.num_tuples(rel_key),
  File "/base/data/home/apps/s~myria-web/1.388195978131666431/raco/backends/myria/catalog.py", line 53, in num_tuples
    return DEFAULT_CARDINALITY
NameError: global name 'DEFAULT_CARDINALITY' is not defined

Note that the relation was imported via a side channel, and I think it has numTuples=-1.  Not sure if relevant.

This looks like a bug caused by poor test coverage, looking at it

Does this branch work?
https://github.com/uwescience/raco/tree/fix-myria-catalog

I should have mentioned that this was against the catalog on the production service.  It's not a blocking issue for me, but I won't have a chance to try the fix (which I assume will work) this week.


No more broccoli

Are you sure we should delete the README section in the mainmodule as well?
And maybe a smaller ASCII Broccoli would be fitting rather than removing it completely.

If Sceleratis could merge it and do that after it's fine. I'm more highlighting it's a problem than not. It does decrease load time.

I didn't even know this was a thing


Dask and Tensorflow Compatibility

There was a package called dask_tensorflow which is currently archived but they stated there are many way to get dask to work with tensorflow.
So I thought I'd try it with the mnist model as follow:
%load_ext tensorboard
import tensorflow as tf
from tensorflow import keras
import datetime
import matplotlib.pyplot as plt x

EPOCHS = 50
BATCH_SIZE = 128
N_HIDDEN = 128
VALIDATION_SPLIT = 0.2
VERBOSE = 1
DROPOUT= 0.3
NB_CLASSES = 10
RESHAPED = 784

mnist = keras.datasets.mnist

(X_train, Y_train), (X_test, Y_test) = mnist.load_data()

Y_train = tf.keras.utils.to_categorical(Y_train, NB_CLASSES)
Y_test = tf.keras.utils.to_categorical(Y_test, NB_CLASSES)

import dask.array as da
import numpy as np
import joblib

X_train = X_train.reshape(60000, RESHAPED)
X_test = X_test.reshape(10000, RESHAPED)
X_train = X_train.astype('float32')
X_test = X_test.astype('float32')

X_train /= 255
X_test /= 255
print(X_train.shape[0],'train samples')
print(X_test.shape[0], 'test samples')

X_train = da.from_array(np.asarray(X_train))
Y_train = da.from_array(np.asarray(Y_train))
X_test = da.from_array(np.asarray(X_test))
Y_test = da.from_array(np.asarray(Y_test))

model = tf.keras.models.Sequential()
model.add(keras.layers.Dense(NB_CLASSES,
                             input_shape = (RESHAPED,),
                             kernel_initializer = 'zeros',
                             name = 'dense_layer',
                             activation ='softmax'))

model.compile(optimizer ='SGD',
             loss= 'categorical_crossentropy',
             metrics = ['accuracy'])

with joblib.parallel_backend('dask'):
        model.fit(X_train,Y_train,
                  batch_size = BATCH_SIZE,
                  epochs = EPOCHS,
                  verbose = VERBOSE,
                  validation_split = VALIDATION_SPLIT)

But I get an error:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-28-2b3bc3bdb162> in <module>
      4                   epochs = EPOCHS,
      5                   verbose = VERBOSE,
----> 6                   validation_split = VALIDATION_SPLIT)

~/anaconda3/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py in _method_wrapper(self, *args, **kwargs)
    106   def _method_wrapper(self, *args, **kwargs):
    107     if not self._in_multi_worker_mode():  # pylint: disable=protected-access
--> 108       return method(self, *args, **kwargs)
    109
    110     # Running inside `run_distribute_coordinator` already.

~/anaconda3/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, validation_batch_size, validation_freq, max_queue_size, workers, use_multiprocessing)
   1034           data_adapter.train_validation_split((x, y, sample_weight),
   1035                                               validation_split=validation_split,
-> 1036                                               shuffle=False))
   1037
   1038     with self.distribute_strategy.scope(), \

~/anaconda3/lib/python3.7/site-packages/tensorflow/python/keras/engine/data_adapter.py in train_validation_split(arrays, validation_split, shuffle)
   1393     raise ValueError(
   1394         "`validation_split` is only supported for Tensors or NumPy "
-> 1395         "arrays, found following types in the input: {}".format(unsplitable))
   1396
   1397   if all(t is None for t in flat_arrays):

ValueError: `validation_split` is only supported for Tensors or NumPy arrays, found following types in the input: [<class 'dask.array.core.Array'>, <class 'dask.array.core.Array'>]

Does this mean tensorflow 2.0 no longer supports working with Dask Arrays?

@vinhdiesal
I have tried in colab with TF versions 2.2, 2.3-rc1 and i am seeing the error (ValueError: To use Joblib with Dask first create a Dask Client).Please, find the gist here.Thanks!

You need to connect to a client.
Try inserting the following at the beginning
from dask.distributed import Client
client = Client()
client`

@vinhdiesal
I tried adding above piece of code. I am seeing the error(TimeoutError: Worker failed to start).please, find the gist here.Thanks!


Analysing data on a self hosted atuin server

I'm running atuin with a local network Postgres instance to self host my data.
I was hoping to run queries on that data but realized from https://github.com/ellie/atuin/blob/main/atuin-client/src/encryption.rs that all the data was encrypted by default 😄 . Which is a great design default.
Was wondering if you'd have any recommendations on how could I run some more complex queries beyond what  stats currently offers.

All the data from the sync server should be mirrored with your client database. So you should be able to do the types of analysis on your local sqlite file (should be found in .local/share/atuin by default)

Thanks will look into that.
But not incase I use the server for multiple clients though?

Ah okay thanks! that works for me.

Apologies on reopening this, but I'm using a mac and I cannot find the db at ~/Library/Application Support/com.elliehuxtable.atuin/history.db .  or the path you pasted above.  Any suggestions on where this file maybe located?

nvm, found it /System/Volumes/Data/Users/<username>/.local/share/atuin

@conradludgate  can you help confirm if this is expected? If so should the default config file and readme be updated to reflect this.

In docs/config.md

We store data in ~/.local/share/atuin (unless overridden by XDG_*).


Editing a role with the default permission value resets it to 0

Steps to reproduce

Edit a role
Don't pass in any permission
See what happens

Expected Result
It should edit the role with the default permission value
Actual Result
It edits the role with permission value 0
System info
hikari 2.0.0.dev85
Checklist

[x] I have made sure to remove ANY sensitive information (bot token, passwords,
credentials, personal details, etc).
[x] I have searched the issue tracker and have made sure it's not a duplicate.
If it is a follow up of another issue, I have specified it.

I am unable to reproduce this. Could you please provide some sample code?


Terminating the login possible

We can easily escape from the login by pressing CTRL + C.

I is already terminating...


Update copyright and remove version api

What this PR does / why we need it:

Update copyright because some files got from OpenStack have been changed.
Version api is not required currently because we are developing the v1 api, there is no other api such as v2 or v3

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

@skdwriting Please check this.

Codecov Report

Merging #67 into master will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
- Coverage   29.98%   29.83%   -0.15%
==========================================
  Files          59       57       -2
  Lines        3739     3630     -109
  Branches      414      412       -2
==========================================
- Hits         1121     1083      -38
+ Misses       2601     2530      -71
  Partials       17       17

Impacted Files
Coverage Δ

dolphin/api/common/__init__.py
28.35% <ø> (-1.06%)
:arrow_down:

dolphin/api/common/wsgi.py
22.60% <ø> (-1.09%)
:arrow_down:

dolphin/api/extensions.py
22.69% <ø> (ø)

dolphin/api/v1/router.py
0.00% <ø> (ø)

dolphin/cmd/api.py
0.00% <ø> (ø)

dolphin/common/config.py
0.00% <ø> (ø)

dolphin/common/sqlalchemyutils.py
20.58% <ø> (ø)

dolphin/context.py
57.14% <ø> (-0.86%)
:arrow_down:

dolphin/cryptor.py
0.00% <ø> (ø)

dolphin/db/api.py
54.87% <ø> (ø)

... and 27 more


Plane Finder client does not install properly on aarch62.

The armhf Plane Finder client package fails to install reporting it is missing the libc6 package when being installed on systems based on the aarch64 architecture such as what the Raspberry Pi 4 is based on.

The planefinder.sh script now adds the armhf and installs the libc6:armhf package. Commit eb58ca62ff35211c09aa9a509aec83bbebc281e9 addresses this issue and will be a part of v2.8.1 when it is released. Until then it is recommended that anyone using a device using the aarch64 architecture and wishing to install the Plane finder client use the phoenix-fixes branch.


Update cainjector to use stable API versions

Updated all the cainjector "injectables" to the v1 APIs.

admissionregistration GA since K8S 1.16: https://github.com/kubernetes/kubernetes/commit/3dd8add5d25d29358fb239ac23461643f1d8effc
apiextensions GA since K8S 1.16: https://github.com/kubernetes/kubernetes/commit/ef397e5b368d530e712165a4bb78ec482d739c2b
apiregistration.k8s.io (aggregation) GA since K8S 1.10: https://github.com/kubernetes/kubernetes/commit/35ed5338b104f41c91efb2b842e92fded6d0b509

Re-enabled the apiregistration E2E tests since the original reason for their being disabled was that they interfered with Helm's Tiller, which we no longer rely on (AFAIK).
Fixes: #3289, #2354
Updated `cainjector` to use v1 API versions of admissionregistration, apiextensions and apiregistration.

/test pull-cert-manager-e2e-v1-16

/test pull-cert-manager-e2e-v1-16

/test pull-cert-manager-e2e-v1-16
/test pull-cert-manager-e2e-v1-20

/test pull-cert-manager-e2e-v1-16

/test pull-cert-manager-e2e-v1-16

/test pull-cert-manager-e2e-v1-16
/test pull-cert-manager-e2e-v1-17
/test pull-cert-manager-e2e-v1-18
/test pull-cert-manager-e2e-v1-19

/test pull-cert-manager-e2e-v1-16

/test pull-cert-manager-e2e-v1-16

/test pull-cert-manager-e2e-v1-16

/test pull-cert-manager-e2e-v1-16

/test pull-cert-manager-e2e-v1-16
/test pull-cert-manager-e2e-v1-17
/test pull-cert-manager-e2e-v1-18
/test pull-cert-manager-e2e-v1-19
/test pull-cert-manager-e2e-v1-20

/kind cleanup

Unhold when you're ready! :) seems like a good step forward given we don't support k8s 1.15 or below anymore
/lgtm
/hold

/unhold


About dual-template representation and dynamic template update.

I have been following the repository for two years. When will the code be updated?  @t-martyniuk @borsukvasyl

I have been following the repository for two years. When will the code be updated? @t-martyniuk @borsukvasyl

Nobody reply?


Deleted Relationship record prevents deleting Contact

Create two Contacts 'First' and 'Second'.
Create a Relationship record for the two Contacts.
Attempt to delete a Contact. Get error "Your attempt to delete First could not be completed because it is associated with the following relationships..." See screen shot.

Delete the Relationship record.
Attempt to delete the Contact again. Get error
There were custom validation error(s) encountered while saving the affected record(s). The first validation error encountered was "Apex trigger hed.TDTM_Contact caused an unexpected exception, contact your administrator: hed.TDTM_Contact: execution of AfterDelete caused by: System.ListException: Missing id at index: 0: (hed)".

Note that similar to a Relationship, an Affiliation on a Contact will prevent the deletion of the Contact, but after deleting the Affiliation record, the Contact in question may be deleted properly.

**lurch: add

Tracking W-019339

Included in beta release 1.39 (Beta 1)

Included in production release rel/1.39


add .py~ to the setup.py clean method.

Tasks/Steps/Elements
Versions checklist

Spyder version: >= 5
Python version: 3.7, 3.8
Qt version: all
PyQt version: all
Operating System name/version:

server : Ubuntu 20.04 Linux (aarch64 on RPI)
client : Windows/Linux/macOS

also add *.py~ to .gitignore

done


How to use arm_lstm_unidirectional_s16_s8? Are there any tutorials？

Besides TFLM, are there any tutorials, especially the example codes,  for us to learn how to use the arm_lstm_unidirectional_s16_s8 function? It is so complicated.

I think TFLM is a good starting point with a lot of supportive scripts. Other than that it might be a good idea so ask in the SIG Micro forum and/or join the monthly meetings: https://github.com/tensorflow/tflite-micro#getting-help

@mansnils OK, thanks!


XXL ♻ Update Platejs to latest version

#4619
Hey Team,
This PR aims to upgrade our existing editor to the new Plate version. This also adds / (SlashCommands) though some more work will be needed on this.
Includes,

New Example project next-2024 - adds a upgraded Next.js project that we can test against. I noticed a quicker start up time! (Though no sure why....)
Upgrades all remaining packages Typescript to version ^5
Adds Shadcn/ui styles/colours to our tinatailwind config<EMAIL_ADDRESS>Replaces some lodash deps with either the specific function i.e. lodash.set or implements them in a utility file
Updates and removes old version of plate (plate-headless) for latest version ^36
Starts removing and cleaning up some of the old Plate code.

Since this is already a massive PR - The remaining work will be carried out over separate PR/PBIs
Screenshots
Before:

Figure: Previous Editor UI
After:

Figure: New Plate UI

Figure: ToolbarOverride + New Plate Editor

question: Is the ...tinacms/src/toolkit/fields/plugins/mdx-field-plugin/plate/components/floating-toolbar.tsx file meant to be empty?

@JackDevAU
When I build the repository and attempt to run pnpm start:e2e:dev, I get the following errors in the console:
Error: Invalid <Link> with <a> child. Please remove <a> or use <Link legacyBehavior>.
Learn more: https://nextjs.org/docs/messages/invalid-new-link-with-extra-anchor
    at LinkComponent (webpack-internal:///../../node_modules/.pnpm/next@14.2.5_@playwright+test@1.46.0_react-dom@18.2.0_react@18.2.0__react@18.2.0/node_modules/next/dist/client/link.js:282:23)
    at renderWithHooks (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5658:16)
    at renderForwardRef (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5842:18)
    at renderElement (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6005:11)
    at renderNodeDestructiveImpl (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6104:11)
    at renderNodeDestructive (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6076:14)
    at renderNode (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6259:12)
    at renderChildrenArray (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6211:7)
    at renderNodeDestructiveImpl (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6141:7)
    at renderNodeDestructive (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6076:14)
    at renderNode (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6259:12)
    at renderHostElement (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5642:3)
    at renderElement (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5952:5)
    at renderNodeDestructiveImpl (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6104:11)
    at renderNodeDestructive (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6076:14)

I see the screenshot below in the browser (the link)

Figure: Error seen in browser

@JackDevAU
When I build the repository and attempt to run pnpm start:e2e:dev, I get the following errors in the console:
Error: Invalid <Link> with <a> child. Please remove <a> or use <Link legacyBehavior>.
Learn more: https://nextjs.org/docs/messages/invalid-new-link-with-extra-anchor
    at LinkComponent (webpack-internal:///../../node_modules/.pnpm/next@14.2.5_@playwright+test@1.46.0_react-dom@18.2.0_react@18.2.0__react@18.2.0/node_modules/next/dist/client/link.js:282:23)
    at renderWithHooks (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5658:16)
    at renderForwardRef (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5842:18)
    at renderElement (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6005:11)
    at renderNodeDestructiveImpl (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6104:11)
    at renderNodeDestructive (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6076:14)
    at renderNode (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6259:12)
    at renderChildrenArray (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6211:7)
    at renderNodeDestructiveImpl (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6141:7)
    at renderNodeDestructive (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6076:14)
    at renderNode (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6259:12)
    at renderHostElement (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5642:3)
    at renderElement (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:5952:5)
    at renderNodeDestructiveImpl (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6104:11)
    at renderNodeDestructive (/home/ncn/Source/Tina/tinacms/node_modules/.pnpm/react-dom@18.2.0_react@18.2.0/node_modules/react-dom/cjs/react-dom-server.browser.development.js:6076:14)

I see the screenshot below in the browser (the link)
 Figure: Error seen in browser

✅ Done - Fixed the example experimental-examples/kitchen-sink

As per my discussion with @JackDevAU, the examples will be refactored as documented in https://github.com/tinacms/tinacms/issues/4720


Python example. (#1)

Added python example.
Closes #1
My Discord handle is cilla#9719. Thanks!

Thanks @lucacillario, we will review this :)

Thanks @SuzeShardlow 😋

Thanks for your contribution, @lucacillario!  I have given you the special Hacktoberfest 2022 role on our Discord server :)


CXuesong.JsonRpc.AspNetCore nuget package needs update

Currently available CXuesong.JsonRpc.AspNetCore package version 0.4.3 still depends on CXuesong.JsonRpc.Standard while other packages have been bumped to 0.5.0+ which had it renamed to CXuesong.JsonRpc.Commons. This results in issues such as still having to use types in the old namespace (e.g. JsonRpc.Standard.Server.JsonRpcService), or JsonRpc.Standard.Server.JsonRpcServiceHost failing to catch exceptions of type JsonRpc.Server.JsonRpcException, unless one switches to a pre-0.5.0 version for all the other dependencies.

Oops. I must have missed to upload JsonRpc.AspNetCore package onto NuGet. I've picked the package from v0.5.0 and uploaded it just now. You may update your package dependency in a while.

Looks good, thanks ;)


Treat 404 responses as errors

At the moment, 404 responses are returned in google-api-nodejs-client as
regular responses, not errors. From the user perspective, this is an odd
pattern.
This commit forces 404s to be returned as errors, and not be treated as
a successful requests.

See also:
https://github.com/google/google-api-nodejs-client/issues/424

Coverage increased (+0.4%) to 93.833% when pulling a74aa5dd195cb32b2615edb4c2dcde12f192a9bd on saicheems:404 into 1956db0f24abf7b7eca36638579da1f343300fb7 on google:master.

Coverage increased (+0.4%) to 93.833% when pulling 96f619b11da19c773d401d92392eb534d090873d on saicheems:404 into 1956db0f24abf7b7eca36638579da1f343300fb7 on google:master.

Coverage increased (+0.4%) to 93.833% when pulling 993dd4d77277f538dedcc4cd15cc70cbd9920b10 on saicheems:404 into 1956db0f24abf7b7eca36638579da1f343300fb7 on google:master.

fyi @tcoffee-google @vchudnov-g

+jmdobry

@jmdobry
@JustinBeckwith

@ofrobots

Should we special-case 404, or possibly let any non-2xx trigger an Error?

Hah, I had similar feedback - at least any 4xx or 5xx.

Coverage increased (+0.4%) to 93.72% when pulling 380d892c4cf419f916be405e1234b05412f2291f on saicheems:404 into bd13376281a3af6e73fb729ea867950739c3c899 on google:master.

OK, looks like there was some conflict with a just merged PR - I resolved by casting all errors to RequestError, but I don't know if that's correct.
Otherwise I've changed it to consider all response status codes >= 400 to be errors.

Coverage increased (+0.4%) to 93.72% when pulling 53997ba0d239b4789806056d6968a352f78b70da on saicheems:404 into bd13376281a3af6e73fb729ea867950739c3c899 on google:master.

Note, also had to add the exception in oauth2client.ts for the 403 case, which also has a special error message.

Coverage increased (+0.4%) to 93.72% when pulling bb7cb65bd5b0d2db62620e7c5118a4ab8b448ea3 on saicheems:404 into bd13376281a3af6e73fb729ea867950739c3c899 on google:master.

Ping, don't have permission to merge this myself

I do not have permission either.

Landed. Also, I have added @lukesneeringer as an owner of this repo.


Added "Tamil" Language Stop words

 Issue by agnelvishal
Fri Oct 12 17:28:44 2018
Originally opened as https://github.com/codelucas/newspaper/pull/636

agnelvishal included the following code: https://github.com/codelucas/newspaper/pull/636/commits

added 0.9.3


Milestone for new statement templates

1.0 is now in the master branch and we are sorting out issues. I understand - if an issue requires a plugin update then it is 1.1 if not then 1.0.1
Are new statement templates to be worked the same way? For example VLE statements which will be generated by the plugins for 1.1 while library statements go in 1.0.1 or are all new statements released with 1.1?

Weekly call: All new ones go in 1.0.1 only edits of existing plugin output goes in 1.1


feat: Add selectionProvider to link an element

This PR adds a .selectionProvider optional attribute to the SelectionObserverMixin. The idea is that it can be used as an alternative to the selection-for attribute, when passing an ID isn't sufficient (e.g. children within a Shadow DOM that are in a different scope than the provider).
This will be used by the d2l-selection-header component that I'm creating as part of US144131. The header uses SelectionObserverMixin itself, and then passes its ._provider to its children via .selectionProvider.
Rally: https://rally1.rallydev.com/#/?detail=/userstory/656749634867&fdp=true

So not to add more complications to this, but a goal for the selection mixins was to eventually use the subscription controllers under the hood, but we haven't gotten around to that story. I created a POC almost a year ago when first making the controllers of what that would look like.  The actual merged controllers ended up here  and probably look a bit different, this just gives an idea.
In that _provider actually goes away altogether, and comes from the controller.  I suspect we'd need to add a new SubscriberController type to handle this case, or expand the id one.  Feel free to ignore this if it totally overcomplicates things and whoever tackles the story can worry about it then. But figured I should raise this in case there's a way to set things up now so they're in a good place for that work, or even do that work if it makes things easier.

(The POC I linked also takes a stab at moving the form and validation stuff which is totally separate from the selection work).

subscription controllers under the hood
I suspect we'd need to add a new SubscriberController type to handle this case

@svanherk I had a look and yep that's my impression too. If anything that would have us moving back in the direction of a more "public" property instead of _provider. Probably a decision for the future though.

subscription controllers under the hood
I suspect we'd need to add a new SubscriberController type to handle this case

@svanherk I had a look and yep that's my impression too. If anything that would have us moving back in the direction of a more "public" property instead of _provider. Probably a decision for the future though.

Yup makes sense - would be good to enhance the selection mixin observer tests to help catch regressions of your use case when that work does eventually happen

Closing in favour of #2979.


The markdown string with command link parsed incorrectly (returned by the language server)

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

VS Code Version: 1.95.3
OS Version: MacOS Sonoma 14.4.1

Steps to Reproduce:

The language server returns a markdown string with a command link.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.96.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

After updating to the latest version of VS Code (1.96.0), the issue still persists.

The language server returns a markdown string with a command link.

Can you please explain what you are trying to do and what the problem is in more detail

The language server returns a markdown string with a command link.

Can you please explain what you are trying to do and what the problem is in more detail

I want to insert the vscode command into the markdown text, as discussed in this issue: #140733
But if the markdown text is returned by the language server, the command does not be parsed correctly, I think maybe this markdown text is not trusted?

Can you share your code? I think you need to set isTrusted on the MarkdownString you create

The markdown string is returned by language server: CompletionItem.documentation, so it can not be set trusted property, it looks like can only return a normal JSON data: https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#completionItem

You can't set it per-string, but when you register your markdown language client, you should be able to configure this using markdown.isTrusted. That's how it works with the node lsp at least. Other lsp implementations may work differently

I set it up just now, and the command connection still hasn't been successfully resolved, so it doesn't seem to be an untrusted problem

Try putting together a self contained example. This most likely is an issue with your code or the lsp implementation, not VS Code itself

I directly return a fixed markdown text in LSP, this command is already registered.
When I display this markdown text in vscode extension, it can be successful, and there is no problem to excute the command directly, but command link can not be parsed when the markdown text is returned through lsp.

I directly return a fixed markdown text in LSP, this command is already registered.
When I display this markdown text in vscode extension, it can be successful, and there is no problem to excute the command directly, but command link can not be parsed when the markdown text is returned through lsp.

Please share a small, easy to understand example that shows this

but command link can not be parsed when the markdown text is returned through lsp.

Most likely the command link is sent along and parsed just fine, but it ends up being removed when we sanitize the rendered result. That's what happens if the markdown is not trusted

I am sorry that I forgot to restart watching compilation command after updating the vscode version, it turns out that your initial statement about setting the LSP client option to trust the markdown text works, and finally, thank you very much!


Combine 2 RTSP Streams

I am trying to combine the RTSP of two identical cameras (Reolink E1 Zoom) into one stream.
Under Windows it works fine with the following command:
ffmpeg
-rtsp_transport tcp -i "rtsp://<IP_ADDRESS>:8554/kind1?video=all"
-rtsp_transport tcp -i "rtsp://<IP_ADDRESS>:8554/kind2?video=all"
-filter_complex ""crop=1152:1296:0:0[v0];crop=1152:1296:0:0[v1];[v0][v1]hstack[v]""
-map [v]
-vcodec libx264
-pix_fmt yuv420p
-preset ultrafast "test.avi"

How do I reformat this so that go2rtc understands it correctly?
My attempt was:
go2rtc:
  streams:
    child1:
      - "rtsp://x:x@<IP_ADDRESS>:554/h264Preview_01_main#video=h264"
    child2:
      - "rtsp://x:x@<IP_ADDRESS>:554/h264Preview_01_main#video=h264"
    children:
      - ffmpeg:child1#raw=-rtsp_transport tcp -i "rtsp://<IP_ADDRESS>:8554/child2?video=all" -filter_complex "crop=1152:1296:0:0[v0];crop=1152:1296:0:0[v1];[v0][v1]hstack[v]" -map [v] -rtsp_transport tcp -f rtsp {[output}}

Unfortunately it does not work.
Does anyone have a tip?
Thanks in advance!

Unfortunately it does not work.

Hi
Can you please share valid setting?
If you found the solution.


CI: Pin syn to 2.0.32

I'm a little surprised that any syn 2.0.x works with 1.48.
The point of the existing serde pins is to keep syn at 1.x. I wonder if it would be possible to remove or relax those now?

My approach to this CI failure would be to look at whatever's depending on syn 2.x and downgrade that.

 We should drop serde_derive which should eliminate all the proc-macro nastiness and move on. Which is a bigger project and independent of this PR.

@danielabrozzoni thanks for fixing this!  I looked into fixing this but I didn't think to try syn 2.0.x.  How did you decide which version of syn to use?

@yancyribbens in this case I think I just looked at the compiler error and downgrade the syn dependency until it compiled. Generally, I think using cargo tree with the latest toolchain and using the output to figure out which of our dependencies is importing the broken dependency and pinning that is a better approach, as Andrew suggested above.


seehecht: 3.0.2 -> 3.0.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/seehecht/versions.
meta.description for seehecht is: "A tool to quickly open a markdown document with already filled out frontmatter"
meta.homepage for seehecht is: ""
meta.changelog for seehecht is: ""
Updates performed

Rust version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0.3 with grep in /nix/store/8cpdr9ik61q8qz6afdab54fgvzi96k2g-seehecht-3.0.3
found 3.0.3 in filename of file in /nix/store/8cpdr9ik61q8qz6afdab54fgvzi96k2g-seehecht-3.0.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
seehecht

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8cpdr9ik61q8qz6afdab54fgvzi96k2g-seehecht-3.0.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A seehecht https://github.com/r-ryantm/nixpkgs/archive/09f3acf17d9051a9339c123261e04caef540df62.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8cpdr9ik61q8qz6afdab54fgvzi96k2g-seehecht-3.0.3
ls -la /nix/store/8cpdr9ik61q8qz6afdab54fgvzi96k2g-seehecht-3.0.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

seehecht

Maintainer pings
cc @auroraanna for testing.

Program is running fine and a simple test worked
1 package built:
seehecht

Build is successfull. LGTM


Dependency Failed for /boot

Ever since the new update that introduces different file types. Arch will not boot and I get these errors. I tested with the first version and it works fine.
[TIME] Timed out waiting for device /dev/disk/by-uuid/4C45-1F05
[DEPEND] Dependency failed for /boot
[DEPEND] Dependency failed for Local File Systems
[DEPEND] Dependency failed for File System Check on /dev/disk/by-uuid/4C45-1F05
You are in emergency mode....

Same issue, no clue how to fix since beginner and just wanted to try out this "super easy" thing to replace those "stinking distributions".

Same problem here on a thinkpad x260


[Amazon AWS] Adds metadata for DDB tables and CRUD operations for items

Description

Adds UpdateItem, DeleteItem and Query functionalities for the tables
Adds metadata for the tables in the List view
Adds Deletion Protection Enablement capability for tables

Screencast
Updated Metadata

Update Item and Query

Delete Item and Query

Advanced Query use-cases (sort keys and indexes)

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

closes https://github.com/raycast/extensions/issues/12616

@victor-falcon / @JonathanWbn are you able to check this out
You can see this guide on how to check it out 🙂

And as always, thanks for checking @JonathanWbn 🔥


Crash when renaming

Happened after fresh start of the app.
debug.txt
debug_fulltrust.txt

Me too

Me too

Attach your log files, please.


Fix yetus errors and remove deprecated documentation

This PR fixes some yetus errors from codespell plugin and takes the opportunity to remove some deprecated information from documentation.

If you included the fixes to the typos which @mishrasamiksha found can you add credits to the PR for that?

I didn't pick her changes, I followed errors from yetus and fix one by one, but you are right, since she pointed out first, it's fair to pick her commit and give her the credits... I've update the PR with her commit, but I couldn't find her email address, so I let the fake one from GH....


support for rails 5, fixes #153

Looks like this is the same as PR #150, my mistake for not checking first.

Coverage remained the same at 100.0% when pulling bdd44b14a824659193c1a1ccaf37672faac0da67 on emars:master into dffdf5ed201b315c393233190cc5ed28d9737d31 on Ink:master.


Install Steam os

How do we install Steam deck ui on chimera os (I’m on aya neo air) and is it compatible with this console?

https://github.com/ChimeraOS/chimeraos/wiki/FAQ#how-can-i-access-the-new-steam-deck-ui-or-the-desktop

https://github.com/ChimeraOS/chimeraos/wiki/FAQ#how-can-i-access-the-new-steam-deck-ui-or-the-desktop

But how to use the terminal on big picture? (I’m not very good at that 😂)

In that same page: https://github.com/ChimeraOS/chimeraos/wiki/FAQ#how-can-i-access-the-terminal-from-chimeraos

It's hidden away in troubleshooting, but if you want access to a terminal without a separate machine, you can use: https://github.com/ChimeraOS/chimeraos/wiki/Troubleshooting#option-2-not-recommended

If you can’t use another computer to SSH into your ChimeraOS box, the other option is to plug a keyboard, and hit CTRL+ALT+F2 to switch from the GUI to a TTY terminal. When you’re finished, to switch back to the GUI, hit CTRL+ALT+F7.

The default username/password is gamer/gamer


CMath stdlib package depends on core Math module

CMath includes Math, so reflect this dependency in the cargo features of artichoke-backend.
https://github.com/artichoke/artichoke/blob/44b60ed9a8f3922cfac6c8e07ecc343e3091bc3e/artichoke-backend/src/extn/stdlib/cmath/vendor/cmath.rb#L23-L28

This dep was made clear to me in https://github.com/artichoke/artichoke/pull/1040.

This dep was made clear to me in https://github.com/artichoke/artichoke/pull/1040.


close recently accessed files

Can I close the recently accessed files on the home page？

You can right-click a recently accessed file and remove the file or clear the entire list.

You can right-click a recently accessed file and remove the file or clear the entire list.

I know, but I'd like to turn this off completely, hopefully in a later version, after all, I do n' t want anyone else to see what I've been watching lately when I' m using my compute

Merging with #366.


ROX-24045: cache Updated Secured Cluster Identity events

Description
The Updated Secured Cluster Identity event is triggered by cluster metrics update, which is not only frequent, but also produces duplicate telemetry data most of the time.
This PR introduces an expirable cache to store previous data reported per cluster, and stop reporting telemetry if there were no changes since last report.
As secured clusters may come and go, the cache expiration is set to 24 hours to remove disconnected clusters data from memory.
The cache will keep up to 100 clusters data in memory. If there are more than 100 clusters connected, every other cluster will push a random cluster out of the cache in a loop, and that will turn into reporting duplicates for N-100 clusters.
An alternative solution would be to cache on the sensor side, and send a flag whether or not there were any changes since last report. This would require altering the sensor-central pipeline, and reconstruct telemetry data on the sensor side somehow: central reads data from the DB.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

Can we store the hash of the map in the cache instead of the contents?


api_server_authorized_ip_ranges needs to be set to "<IP_ADDRESS>/32" with public access enabled. "<IP_ADDRESS>/32" is only "<IP_ADDRESS>"

Is there an existing issue for this?

[X] I have searched the existing issues

Greenfield/Brownfield provisioning
greenfield
Terraform Version
1.3.9
Module Version
7.0.0
AzureRM Provider Version
3.5.7
Affected Resource(s)/Data Source(s)
azurem_kubernetes_cluster
Terraform Configuration Files
module "aks" {

...

public_network_access_enabled         = true
api_server_authorized_ip_ranges       = ["<IP_ADDRESS>/32"]
}

tfvars variables values
.

Debug Output/Panic Output
Error: Resource precondition failed
│
│   on .terraform/modules/aks/main.tf line 446, in resource "azurerm_kubernetes_cluster" "main":
│  446:       condition     = !var.public_network_access_enabled || try(contains(var.api_server_authorized_ip_ranges, "<IP_ADDRESS>/32"), false)
│     ├────────────────
│     │ var.api_server_authorized_ip_ranges is set of string with 1 element
│     │ var.public_network_access_enabled is true
│
│ When `public_network_access_enabled` is set to true, `<IP_ADDRESS>/32` must be
│ added to `authorized_ip_ranges` in the `api_server_access_profile block`
│ (https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/kubernetes_cluster#public_network_access_enabled).

Expected Behaviour
when public api access is enabled, I'd expect netmask needing to be anything but <IP_ADDRESS>/32 (effectively blocking all addresses).
Or am I missing something here?
Thanks in advance!
Actual Behaviour
I'm forced to set <IP_ADDRESS>/32, thus blocking all api server access.
Steps to Reproduce
No response
Important Factoids
No response
References
No response

Hello,
this was introduced in https://github.com/Azure/terraform-azurerm-aks/pull/361
What is implemented comes from the provider documentation:
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/kubernetes_cluster#public_network_access_enabled

This originally comes from this commit:
https://github.com/hashicorp/terraform-provider-azurerm/commit/afd18e49d63d4d0b3a9e573fe93686fc083a66cd
@ms-henglu please have a look. I think this note is confusing.
If we look at the Azure CLI options --api-server-authorized-ip-ranges
https://learn.microsoft.com/en-us/cli/azure/aks?view=azure-cli-latest#az-aks-create

It says to use the range <IP_ADDRESS>/32 to restrict apiserver traffic to node pools, because it blocks basically everything.
Setting api_server_authorized_ip_ranges with ["<IP_ADDRESS>/32"] in the list prevents any other connection to the API server, even if other ranges are added to the list, so this cannot be a pre-condition.
Actually other ranges are not event allowed at that point, If you attempt to do this with the Azure Portal you even see this problem:

@lonegunmanb should we revert https://github.com/Azure/terraform-azurerm-aks/pull/361 ?

I'm studying the commit that introduced this restriction into the document, it seems like the document is wrong. I'll do some tests to verify it.

I've verified that api_server_authorized_ip_ranges won't be affected by public_access_enabled anymore. We should revert #361 to solve this issue.


Add node-video-lib

Pure JavaScript library for working with MP4 and FLV video files and creating MPEG-TS chunks for HLS streaming.
By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆

Nice work! 👌


Accept [x.y] and (x.y) PR title prefixes

Fixes #401

I started working on something that would add the prefix automatically if not present.
I didn’t finish it. I’ll see if I can revive it.
But in the meantime, we can probably merge and deploy this.

@jmartisk I pushed an additional commit that goes a little further and try to normalize titles to include the branch for version branches. This will avoid us missing the information and merging to back branches without noticing it (which happened a few times).

BTW, thanks for creating the PR, I wouldn't have made it a priority if you didn't and it was definitely annoying :).


FitBit OAuth2

Hi,
Could you update the code to use OAuth 2.0 in Fitbit?
Thanks!

Fork and Pull Request if you want... or close this issue
https://dev.fitbit.com/docs/oauth2/


httpclient is out of date and throws warnings in Ruby 2.3

We see these warnings:
Object#timeout is deprecated, use Timeout.timeout instead.

https://github.com/nahi/httpclient/issues/292

One or more sub-gems specify spec.add_dependency 'httpclient', '=2.4.0', ex:

https://github.com/cloudfoundry/bosh/blob/develop/agent_client/agent_client.gemspec#L22

any update on this?
because those deprecation warnings are really annoying..

@ramonskie I just updated the gemspec file in question locally to use 2.7.1 and it eliminates the error until the gem can be patched properly.

In the meantime you can also run these two on your httpclient-2.4.0 directory:
find -name '*.rb' -type f -exec sed -i 's/ timeout(\@/ ::Timeout.timeout\(/g' {} \;
find -name '*.rb' -type f -exec sed -i 's/ timeout\(([[@]]\)/ ::Timeout.timeout\1/g' {} \;

Tracker story for reference: https://www.pivotaltracker.com/story/show/111538726

Closing - httpclient was updated to 2.7.1 via f789444f4 and released as of 3189.


Version bump in package.js

@jaredreich I am bumping the version number in the package.json to match the version specified at in the notie.js file


Different meter subcategory

Hello in my azure portal, I don't see the meter subcategory that you mentioned. I see just ltr backup storage.

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

ID: 9492cedc-cea0-af9d-29a0-52cbca8c5e21
Version Independent ID: 969a8654-8f95-d5de-351d-73c8e02e5d3a
Content: Automatic, geo-redundant backups - Azure SQL Database
Content Source: articles/sql-database/sql-database-automated-backups.md
Service: sql-database
Sub-service: backup-restore
GitHub Login: @anosov1960
Microsoft Alias: sashan

@taromielnaz We are checking this and will get back to you.

@taromielnaz We are assigning this to our author and raised it with Product Group as well. We expect an answer soon for you.

#reassign: @MashaMSFT
Pls get input from @Sait-MSFT

Hi @taromielnaz , thanks for reaching out! Meter categories and subcategories aren't available for counters that you are not currently using. For example, if you do not have a managed instance deployed, you will not see any managed instance counters. Likewise, if you have not consumed backup storage, you may not see backup storage counters.
If you are missing counters for a category that you're certain you should see, please reach out to Microsoft Support to get assistance with this issue, as this is outside the scope of documentation support.
I hope that helps and please let me know if we can be of further assistance. I will close out this Git Issue but please comment should you need anything else.
Thanks and hope you have a wonderful rest of your day!
Masha from the SQL Docs team
#please-close


Is it possible to bypass schema validation?

I only want a simple store without schema validation. Can I pass null to the second parameter, so something like that? I'm trying to avoid having to use another framework (zod in this case) in my app.

Hey! Currently, unfortunately the answer is no(ish). You can theoretically pass z.any() as the schema, which will cause Zod to accept any schema, and set the writable's type to be any. However, you'd still need Zod to do this, but only to get the z.any function.
Since I personally think that naively parsing data from a client environment you can't control is pretty risky, I'm not sure if I'd rather add the ability to disable schema parsing entirely or only add support for parsers other than Zod for projects that already use another parsing library.
I'm currently leaning towards supporting different parsers and no parser at all in a new version, but it'd be a breaking v0.2 since I'd want the schema argument to be optional if it can be null. Unfortunately can't make any promises as to when this will be possible right now.
Either way, keeping this issue open to track the request 👍


Notifications sometimes displays wrong contact name

Sometimes the notification toast has the wrong sender name. I haven't detected a pattern yet, but I have seen it a few times both on the Android app and the Windows app.

I think this is due to not resetting the app after syncing with a new (different) server. Closing. We can re-open this if a resync doesn't fix the issue.


Error upgrade version

I/Process ( 5819): Sending signal. PID: 5819 SIG: 9

@giovannilima262 we need more info in order to know the context for your error report.  Such as:

What command did you issue?
Can you attach a flutter doctor --verbose output
What output do you see if you run with --verbose?
Is it 100% reproducible, or did  you just see it once?

Actually wanted to know how to update the flutter project without knowing how to create a new one

to create a project with update and pass the files to it,
this error appears when you include the apply plugin: 'com.google.gms.google-services' at the build.gradle

Sorry for the late response.

update the flutter project

What do you mean by that?

create a project with update and pass the files to it,

what files to pass, and for what purpose?

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Invalid Grant error processing request

There is nothing wrong with my API credentials.
However whenever I run Urs.py -r vizards N 100
the program says that my API credentials are not right and give me that error.
I know my API credentials are right and tested them on another piece of code. What could be creating this error?

It looks like you just joined GitHub so you might not be familiar with how bug report templates work. For future reference, do not delete the default issue template that is loaded when you create a new issue in a repository. Its purpose is to maintain a structured, detailed set of information so that it is possible for developers to replicate the bug that you are describing.
Please revise your bug report by filling out the default template.

I had the same thing happen to me.  I had to change
        username = os.getenv("USERNAME"),

to
        username = os.getenv("REDDIT_USERNAME"),

and to REDDIT_USERNAME in the config file as well.
Because in windows there is already a "username" environment variable which overrode the value in my config file.

@jamesrichter thank you for bringing this to my attention. I was not aware Windows already has a username environment variable. I will amend the corresponding scripts so that Windows users will not run into this issue again.


FeaturedPostsPlugin displays all posts instead of selected posts

The plugin FeaturedPostsPlugin requires to select some posts to display, and then the get_posts method returns all posts of the blog:
https://github.com/nephila/djangocms-blog/blob/d95802c1475fc7e06ba121a178bb0e755be9bba9/djangocms_blog/models.py#L614

@adrien-delhorme Thanks for reporting this: will try to fix and release a new version today

@adrien-delhorme We've just released 2.0.6 version: could you please check if now it's working as expected?

Perfect, thank you (and happy new year 🥳)


Durable Entity critical section still can receive requests in wrong order

We are trying to handle an issue in our durable workflow with order of updates comming on a particular Entity.
The issue is that although two orchestrations will not enter the Lock section at the same time, it is still possible that one orchestration gets to the Lock faster that the other.

We have a HttpTrigger that receives payloads.
Trigger starts orchestration which will try to Lock on an entity ID in the very first step.
We are scalling to multiple partitions and workers for performance.

If http call A arrives first, B second, it is still possible B will get to the Lock section first and be processed before A.
Are there any suggestions how to overcome such an issue?

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

ID: 87e12177-bb3a-d00c-99a5-422e00f9b03f
Version Independent ID: 2fa1cf0d-2a5f-2fb0-ef25-bccfac3279fd
Content: Durable entities - Azure Functions
Content Source: articles/azure-functions/durable/durable-functions-entities.md
Service: azure-functions
Sub-service: durable
GitHub Login: @cgillum
Microsoft Alias: azfuncdf

@dojo87
Thanks for your feedback! We will investigate and update as appropriate.


Remove deprecated force_text

force_text is deprecated since version 3.0 and will be removed in Django 4.0
https://docs.djangoproject.com/en/dev/releases/4.0/#features-removed-in-4-0

I have now created a fork and uploaded a new release to PyPi that can be used if needed. I do however strongly advise you to migrate to django-enumfields or another enum package, as I will not guarantee that I will fix future deprecations.
@jkevingutierrez @brammittendorff

I have released version 3.0.0 with these changes included. Thanks for the pull request, and sorry for the delay.


fix(config): log rotation max files

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
526

Relevant Lines:
526

💛  - Coveralls


update sb-admin theme

The sb-admin theme introduced by @mftruso added a pleasant interface to this project. However, the sb-admin theme doesn't always follow semantic html in a coherent manner. There have been updates to support Bootstrap 5, so we should try to update to a more recent sb-admin theme with Bootstrap 5 support.
Perhaps we could even look into an mdbootstrap admin theme (such as https://github.com/mdbootstrap/bootstrap-5-admin-template/ ), which probably has better support and follows html semantics in a more coherent manner...
Am currently trying to update to a newer sb-admin theme, however it's not very flexible... The sidebar seems to be pretty much hardcoded to a dark theme... I think that an mdbootstrap theme may be more flexible and adaptable...

it seems that bootstrap margin utilities are not working at all in the newer sb-admin theme? Does it perhaps remove a number of bootstrap utilities?
Also, I can't seem to get just a left border, even when using border-start? I have tried border-start border-primary, border-primary border-start, and border-start-primary to no avail...

it seems that bootstrap margin utilities are not working at all in the newer sb-admin theme? Does it perhaps remove a number of bootstrap utilities?

It seems that margin and padding utilities are actually working, you just have to use ms (margin-start) in place of ml (margin-left) and me (margin-end) in place of mr (margin-right), and idem for padding utilities, in bootstrap 5...

I did finally succeed in getting a border to work on one side only, but it's so much more complicated now. See the page on extending the calendar on a national level. The card widget has a blue border on the left side, this is how I got it to finally work: border-4 border-top-0 border-bottom-0 border-end-0 border-primary rounded-3. First set the border everywhere, then remove the border from the sides you don't want it on. Then you can define the color and radius. Seems a bit of a hack, but it does work...


Import/export xls files

Because some people still use xls files

@ddooley @cmrn-rhi Could one of you check out this branch and see if you can save xls files, and open them without issue in Excel? Also if you can edit an xls file in Excel, save it, and then open it in the validator? It works fine for me, but I'm on Ubuntu, so I'm using a third-party spreadsheet application.

Will do.

Worked great. Did the following checks:

Pasted new data directly into validator, validated, saved as .xls, opened in excel; experienced no data loss or unexpected formatting.
Pasted new data directly into validator, skipped validation step, saved as .xls, opened in excel; experienced no data loss or unexpected formatting.
Imported excel generated .xls file, imported it into the validator; experienced no data loss or unexpected formatting.

Excellent. Thanks @cmrn-rhi


AadIssuerValidator usage

The code in this file: https://github.com/AzureAD/microsoft-authentication-extensions-for-dotnet/blob/master/src/Microsoft.Identity.Client.Extensions.Web/Resource/AadIssuerValidator.cs
is replicated in many MSFT samples using AADv2, aka https://github.com/Azure-Samples/active-directory-aspnetcore-webapp-openidconnect-v2/blob/master/Microsoft.Identity.Web/Resource/AadIssuerValidator.cs
Also, this code is not part of a nuget package, even a preview, at this time.
should one just copy this code into their own project if they want to use the AadIssuerValidator?

@rhizohm we are working on productizing this code (coming from samples indeed), with the goal of releasing a NuGet package
cc: @MarkZuber @brentschmaltz @henrik-me

@jmprieur @MarkZuber : Perhaps we should add a roadmap to the wiki?

@henrik-me : yes we should

@jmprieur @henrik-me I did first pass cleanup of AadIssuerValidator.

@brentschmaltz
can I merge all the changes you proposed in the PR? (I would think so)
cc: @jmprieur @MarkZuber @kalyankrishna1 @TiagoBrenck

@jmprieur please pull them in. I expect to be able to continue in about a week. Is there any specific place you would like to prioritize?

Any update on this? It is very confusing if the sample code in documentation doesn't work.

@allenwux : did you mean that https://github.com/Azure-Samples/active-directory-aspnetcore-webapp-openidconnect-v2/blob/master/Microsoft.Identity.Web/Resource/AadIssuerValidator.cs still does not work?
We have unit tests etc .... are we missing a case? can you please elaborate ?

the code sample in the documentation doesn't work:
https://docs.microsoft.com/en-us/azure/active-directory/develop/scenario-web-app-sign-user-app-configuration
Can't find the AadIssuerValidator class after adding the nuget package
That documentation page needs some update anyway. I was not able to follow it.

Is this in a nuget package yet for AspNetCore v2.2?

@udlose : not yet. Meanwhile you can use the following project:
https://github.com/Azure-Samples/active-directory-aspnetcore-webapp-openidconnect-v2/blob/master/Microsoft.Identity.Web/README.md

@allenwux : I plan to update https://docs.microsoft.com/en-us/azure/active-directory/develop/scenario-web-app-sign-user-app-configuration soon

I made it clear that the code is no longer here. See PR https://github.com/AzureAD/microsoft-authentication-extensions-for-dotnet/pull/45
See https://github.com/AzureAD/microsoft-authentication-extensions-for-dotnet/tree/master/src/Microsoft.Identity.Client.Extensions.Web


Posssible Routing Peering Capabilities BGP protocols

The problem statement is as follows:

Can ohmyglb be built in such a way it would peer with a routable network using either bgp (Border Gateway Protocols)
Could you use the AS Number Private Ranges or an ospf Area config
Can you use authenticationon either BGP or Ospf to make sure it can be secured for only peer to peer session establishment
Can you use an ACL which can be used to present network ip's via the above two routing protocols
I am asking if the project feature request can be possibly looked at for this as this would mean that you could be location agnostic

As per discussions with @ZakAnderson this feature is best implemented in a separate project.


Add search button to mobile top navbar

Fixes: #839
When search box is not active:

and when search box is active:

Hey, it looks great!
I noticed that toggling between states takes a while, that's because it causes entire Wrapper to re-render:

Can you move this logic and state to Topnav itself, so it won't update entire Wrapper but only Topnav?

Nice catch! I will move state to Topnav :)

Hello @bonustrack, thanks for good feedback. I've just applied changes and PR is ready for next review.


gh760 Checks and tests for VU 00303-00308

Implement Valid Usages checks that verify that a vkImageMemoryBarrier's oldLayout/newLayout are compatible with the VkImage's usage flags.
Add both negative and positive tests.  Fix a couple existing tests that the new checks revealed were incorrect.

bump


Sync and View with external Viewer

Hello,
is there a way to sync and view with, lets say, sumatraPDF in windows?

Do you want to use the local VSCode instead of GitHub codespace? In that case sure.
Add this to .vscode/settings.json file:
"latex-workshop.view.pdf.viewer": "external",
"latex-workshop.view.pdf.external.synctex.command": "C:/Users/<UserName>/AppData/Local/SumatraPDF/SumatraPDF.exe",
  "latex-workshop.view.pdf.external.synctex.args": [
    "-forward-search",
    "%TEX%",
    "%LINE%",
    "-reuse-instance",
    "-inverse-search",
    "\"C:\\Users\\<Username>\\AppData\\Local\\Programs\\Microsoft VS Code\\Code.exe\" \"C:\\Users\\<U>\\AppData\\Local\\Programs\\Microsoft VS Code\\resources\\app\\out\\cli.js\" --ms-enable-electron-run-as-node -r -g \"%f:%l\"",
    "%PDF%"
  ],

In settings.txt of SumatraPDF, set
InverseSearchCmdLine = "C:\Users\<Username>\AppData\Local\Programs\Microsoft VS Code\Code.exe" "C:\Users\<Username>\AppData\Local\Programs\Microsoft VS Code\resources\app\out\cli.js" --ms-enable-electron-run-as-node -r -g "%f:%l"
EnableTeXEnhancements = true

Reference:
https://github.com/James-Yu/LaTeX-Workshop/wiki/View#sumatrapdf

I am using it in a docker on wsl2 environment.
Instead of path to c i have to assign path to mnt/c?

{
    "name": "latex-docker-workspace",
    "image": "your-docker-image:latest",
    "remoteUser": "root",
    "hostRequirements": {
        "storage": "32gb"
    },
    "remoteEnv": {
        "user": "your-username",
        "sumatraPath": "/mnt/c/Program Files/SumatraPDF/SumatraPDF.exe",
        "vscodePath": "/mnt/c/Users/${user}/AppData/Local/Programs/Microsoft VS Code/Code.exe",
        "vscodeCliPath": "/mnt/c/Users/${user}/AppData/Local/Programs/Microsoft VS Code/resources/app/out/cli.js"
    },
    "features": {},
    "customizations": {
        "vscode": {
            "settings": {
                "latex-workshop.view.pdf.viewer": "external",
                "latex-workshop.view.pdf.external.synctex.command": "${sumatraPath}",
                "latex-workshop.view.pdf.external.synctex.args": [
                    "-forward-search",
                    "%TEX%",
                    "%LINE%",
                    "-reuse-instance",
                    "-inverse-search",
                    "\"${vscodePath}\" \"${vscodeCliPath}\" --ms-enable-electron-run-as-node -r -g \"%f:%l\"",
                    "%PDF%"
                ]
            },
            "extensions": [
                "James-Yu.latex-workshop"
                // Add other extensions if needed
            ]
        }
    }
}

This works in one direction for now


pythonPackages.debugpy: 1.2.1 → 1.3.0

Motivation for this change
Update to the latest version: https://github.com/microsoft/debugpy/releases/tag/v1.3.0
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
No binary files, only a Python library.
Tested through dap-mode in Emacs.
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.debugpy
python39Packages.debugpy

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

2 packages built:

python38Packages.debugpy
python39Packages.debugpy


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_3d42dc4f-f3df-4b9a-919d-7ed405e4af68/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_3d42dc4f-f3df-4b9a-919d-7ed405e4af68.git

Docs Build status updates of commit 97ce28b:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-11-07-09-08-22 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-11-07-09-08-22

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


ImportError : DLL load failed: The specified module could not be found

I am trying to import the module signal but as soon as i try to import it
from scipy import signal
The below error is outputted
File "C:\Users\forza\anaconda3\lib\site-packages\scipy\signal\__init__.py", line 301, in <module>
    from .filter_design import *
  File "C:\Users\forza\anaconda3\lib\site-packages\scipy\signal\filter_design.py", line 18, in <module>
    from scipy import special, optimize, fft as sp_fft
  File "C:\Users\forza\anaconda3\lib\site-packages\scipy\optimize\__init__.py", line 390, in <module>
    from ._minimize import *
  File "C:\Users\forza\anaconda3\lib\site-packages\scipy\optimize\_minimize.py", line 30, in <module>
    from ._trustregion_constr import _minimize_trustregion_constr
  File "C:\Users\forza\anaconda3\lib\site-packages\scipy\optimize\_trustregion_constr\__init__.py", line 4, in <module>
    from .minimize_trustregion_constr import _minimize_trustregion_constr
  File "C:\Users\forza\anaconda3\lib\site-packages\scipy\optimize\_trustregion_constr\minimize_trustregion_constr.py", line 4, in <module>
    from scipy.sparse.linalg import LinearOperator
  File "C:\Users\forza\anaconda3\lib\site-packages\scipy\sparse\linalg\__init__.py", line 116, in <module>
    from .eigen import *
  File "C:\Users\forza\anaconda3\lib\site-packages\scipy\sparse\linalg\eigen\__init__.py", line 11, in <module>
    from .arpack import *
  File "C:\Users\forza\anaconda3\lib\site-packages\scipy\sparse\linalg\eigen\arpack\__init__.py", line 22, in <module>
    from .arpack import *
  File "C:\Users\forza\anaconda3\lib\site-packages\scipy\sparse\linalg\eigen\arpack\arpack.py", line 45, in <module>
    from . import _arpack
ImportError: DLL load failed: Impossibile trovare la procedura specificata.

Probably is not  a problem only related to the scipy libary. I get the same behavior when trying to import statsmodel, using :
from statsmodels.tsa.api import ExponentialSmoothing
I am using windows 10 home , and executing the program on the terminal of VS code.
I also tried to launch the program from the prompt of anaconda with the same result.
I did try to uninstall anaconda e install it again but nothing changed.
**python version information: **
Python 3.7.6 (default, Jan  8 2020, 20:23:39) [MSC v.1916 64 bit (AMD64)] :: Anaconda, Inc. on win32
**Scipy version: **
1.4.1

Hi, have you found a solution for this problem?

Hi @Barberto906 @eugeniozoni. Apologies for the very long delay somehow we missed this. Such problems typically happens when pip install is used while the rest is installed with the anaconda package manager. Hence pip and conda installs don't play nice together. We have many similar issues such as #10998 and often a clean and consistent reinstall solves the problem.

Thank you @ilayn I somehow got around with it by using this. I tried to uninstall completely anaconda and reinstall but that did not work. Anyway I now manage to run it and for me that is OK!

@eugeniozoni how did you manage to run it?

@isilefe just made the link given in the comment a bit more obvious

Looks like this can be closed. Thanks @Barberto906 , @eugeniozoni @ilayn


document intent versus entity for wit users

wit no longer has a native concept of an intent , so for new users of wit they won't know what that means. we'll also have to figure out how to parse exported wit apps which don't have any intents defined

this will be addressed when we merge the documentation branch #26


horizon('collection').removeAll() should remove all documents

Currently this does not work from what I can tell. One should be able to delete all documents in a collection without having to first find all the document ids and then passing the array into removeAll.

going to merge this into #576 which is a more general proposal


python311Packages.pkutils: replace nose with nose3

Description of changes
For Python 3.11 compatibility.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

2 packages failed to build:

python311Packages.pygogo
python311Packages.pygogo.dist

6 packages built:

python310Packages.pkutils
python310Packages.pkutils.dist
python310Packages.pygogo
python310Packages.pygogo.dist
python311Packages.pkutils
python311Packages.pkutils.dist

Maybe pygogo needs a similar fix.

Yes, thanks for noticing. I've been thinking about this and feel that:

It's a good idea to send PRs upstream to migrate from nose to nose3 and pull those patches or updates in.
For Python 3.11 update in a timely way, I want to explore patching our packaged version of nose to continue working.

It seems that there are quite a few packages that depend on nose.
If the patch works, I think it would be better for smooth progress.


General Optimizations

Remove the renderer factory - it didn't reduce the Sonar cycle count anyway
Build a customizable sub-query renderer which will reduce some extraneous garbage
Several other small changes that will reduce intermediate garbage

coverage: 100.0%. remained the same
when pulling 17ac0c06d3885967ebb039417566c6d2aa850c60 on jeffgbutler:optimizations
into 0e6342a5d4833a35cd98fcf7bef6e9720bb41317 on mybatis:master.


Avoid unnecessary split/join for in-clauses.

Current broker code joins the values in in-clause with '\t\t', that
requires server to get the individual values using String.split(). This
becomes extrememly inefficient for queries with large number of values
in the in clause (hundreds). This change elimiates this unwarranted
join/split, thus improving perfomance of affected queries. To avoid
breaking any broker/server compatibility:

Added config (off-by-default) in broker to not join the values, and
send them as list. Server checks if the size of list > 1, then
recognizes then skips split, or else falls back to original behavior.

Brokers and servers can be deployed with this feature off. Once all
are updated, the feature can be turned ON in broker, and ther servers
would already have been updated to handle the new protocol.

The config will be removed eventually to keep the code clean.

Added unit test for the new code.

Codecov Report

Merging #1819 into master will decrease coverage by 0.17%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master    #1819      +/-   ##
==========================================
- Coverage   65.51%   65.33%   -0.18%
==========================================
  Files         885      885
  Lines       42584    42594      +10
  Branches     5597     5599       +2
==========================================
- Hits        27898    27829      -69
- Misses      12814    12897      +83
+ Partials     1872     1868       -4

Impacted Files
Coverage Δ

...pinot/pql/parsers/pql2/ast/InPredicateAstNode.java
75.86% <100%> (+2.78%)
:arrow_up:

...ot/broker/requesthandler/BrokerRequestHandler.java
88.06% <100%> (+0.03%)
:arrow_up:

...a/com/linkedin/pinot/pql/parsers/Pql2Compiler.java
100% <100%> (ø)
:arrow_up:

...om/linkedin/pinot/pql/parsers/Pql2AstListener.java
85.71% <60%> (+0.09%)
:arrow_up:

...kedin/pinot/core/common/predicate/InPredicate.java
66.66% <66.66%> (-8.34%)
:arrow_down:

...com/linkedin/pinot/minion/metrics/MinionMeter.java
0% <0%> (-100%)
:arrow_down:

...in/pinot/minion/executor/TaskExecutorRegistry.java
0% <0%> (-100%)
:arrow_down:

...inkedin/pinot/minion/metrics/MinionQueryPhase.java
0% <0%> (-100%)
:arrow_down:

...nkedin/pinot/minion/executor/BaseTaskExecutor.java
0% <0%> (-100%)
:arrow_down:

.../java/com/linkedin/pinot/minion/MinionContext.java
0% <0%> (-100%)
:arrow_down:

... and 37 more

Continue to review full report at Codecov.

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


[WIP] Fix import for share page

@AndrewGable, please review
Details
Fixes the build error. I had missed this
Fixed Issues

Fixes GH_LINK
Tests
Ran npm run web and npm run ios to verify that the builds are working again.
Tested On

[x] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots
N/A
Web
N/A
Mobile Web
N/A
Desktop
N/A
iOS
N/A
Android
N/A

I have read the CLA Document and I hereby sign the CLA

Reverted so this is irrelevant


feat: automate preview releases [ROAD-624]

@bastiandoetsch please take a look again. I've chained CI pipeline into the release with the latest updates, so should be as desired now :)

Hyy

@michelkaporin, LGTM 👍


feat(hbase): install the hostname command

Description
part of https://github.com/stackabletech/hbase-operator/issues/508
Definition of Done Checklist

Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
Please make sure all these things are done and tick the boxes

- [ ] Changes are OpenShift compatible
- [ ] All added packages (via microdnf or otherwise) have a comment on why they are added
- [ ] Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there
- [ ] All packages should have (if available) signatures/hashes verified
- [ ] Add an entry to the CHANGELOG.md file
- [ ] Integration tests ran successfully

TIP: Running integration tests with a new product image
The image can be built and uploaded to the kind cluster with the following commands:
bake --product <product> --image-version <stackable-image-version>
kind load docker-image <image-tagged-with-the-major-version> --name=<name-of-your-test-cluster>

See the output of bake to retrieve the image tag for <image-tagged-with-the-major-version>.

obsolete. SDP doesn't use the graceful shutdown script.